Dingo API is not working with dynamic relationships of model

I have a relationship like
 public function relationXYZ() {
        if(class_exists('Prefix'.ucfirst($this->type), true)) {
            return $this->hasMany('Prefix'.ucfirst($this->type), 'refId');
        }
    }

Here I have multiple type, A, B, C, D for example.
and in Transformer when I use code like
public function includeRelationXYZ($model) {
        $data = $model->relationXYZ;

        if(!empty($data)) {
            $className = 'TransformersPrefix'.ucfirst($panel->type);
            return $this->collection($data, new $className, 'relationXYZ');
        }
    }

But $model->relationXYZ is not giving proper results. To make it working i had to call
$model->load('relationXYZ');
After that it works. Is it a bug or I am missing anything? How can I make it working without loading relationship again.

Dynamic includes are never a pretty thing, especially when it comes to using Eloquent collections.
I'm going to go out on a limb and say you need to rethink this. What I believe is happening is that because Dingo automatically eager loads your includes, and because this is a collection, it's eager loading for all the models in the collection but it's using the attributes of the first model for the dynamic section.
You can disable eager loading on the adapter if you'd like, either globally, or in the controller method.
app('api.transformer')->getAdapter()->disableEagerLoading();

@gauravjain028 You can try this weiwenhao/tree-ql


Not sure you can mount volumes from host for MySQL data on Windows Docker

I had been using the Percona image for awhile with the following run command:
docker run -d -e APP=db -e \
MYSQL_ROOT_PASSWORD=pass -e MYSQL_DATABASE=db \
-e MYSQL_USERNAME=user -e MYSQL_PASSWORD=pass --name=db \
-p 3306:3306 percona:latest

Specifically note I am not using -v option.
It works, but no data is retained in the container. I figured this is due to mapping /var/lib/mysql to an endpoint expected to come from host volume.
After some experimenting with -v option to docker run command, I find the Percona container only stays running for maybe 7 seconds. Here is the relevant portion of the logs.
Initializing database
2016-12-01T20:18:09.168809Z 0 [Warning] 'NO_ZERO_DATE', 'NO_ZERO_IN_DATE' and 'ERROR_FOR_DIVISION_BY_ZERO' sql modes should be used with strict mode. They will be merged with strict mod
e in a future release.
2016-12-01T20:18:09.168846Z 0 [Warning] 'NO_AUTO_CREATE_USER' sql mode was not set.
2016-12-01T20:18:17.961137Z 0 [Warning] InnoDB: Retry attempts for writing partial data failed.
2016-12-01T20:18:17.961206Z 0 [ERROR] InnoDB: Write to file /var/lib/mysql/xb_doublewritefailed at offset 0, 1048576 bytes should have been written, only 0 were written. Operating syste
m error number 22. Check that your OS and file system support files of this size. Check also that the disk is not full or a disk quota exceeded.
2016-12-01T20:18:17.961236Z 0 [ERROR] InnoDB: Error number 22 means 'Invalid argument'
2016-12-01T20:18:17.963717Z 0 [ERROR] InnoDB: InnoDB Database creation was aborted with error Generic error. You may need to delete the ibdata1 file before trying to start up again.
2016-12-01T20:18:18.564793Z 0 [ERROR] Plugin 'InnoDB' init function returned error.
2016-12-01T20:18:18.564947Z 0 [ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
2016-12-01T20:18:18.564978Z 0 [ERROR] Failed to initialize plugins.
2016-12-01T20:18:18.564998Z 0 [ERROR] Aborting

This leads me to wonder if there is any way to use the container as intended on Windows. I may need to build one in which the data can persist inside the container using docker commit.

That looks like permission issues on the bind mount and is similar to https://github.com/docker-library/mariadb/issues/30, https://github.com/docker-library/mariadb/pull/50, https://github.com/docker-library/mariadb/issues/58, and https://github.com/docker-library/mysql/issues/153; which is fixed by https://github.com/docker-library/percona/pull/21 (assuming you are using docker installed by Docker Toolbox and not "Docker for Windows").

Closing since it seems like this was solved. Let us know if this was closed in error.

i have same problem. Percona(or mariadb or mysql) can not start.
Docker stats:
docker -v
Docker version 17.03.0-ce, build 60ccb22
docker-compose -v
docker-compose version 1.11.2, build f963d76f
docker-machine -v
docker-machine.exe version 0.10.0, build 76ed2a6
Windows 10.
docker-compose.yml
version: '2'
services:
db:
image: percona:latest
volumes:
- ./data/db:/var/lib/mysql
restart: always
container_name: mysql
environment:
MYSQL_ROOT_PASSWORD: 1234
MYSQL_DATABASE: db
MYSQL_USER: db_user
MYSQL_PASSWORD: 1234
Docker log:
Initializing database
2017-03-22T21:00:02.880303Z 0 [Warning] 'NO_ZERO_DATE', 'NO_ZERO_IN_DATE' and 'ERROR_FOR_DIVISION_BY_ZERO' sql modes should be used with strict mode. They will be merged with strict mode in a future release.
2017-03-22T21:00:02.880349Z 0 [Warning] 'NO_AUTO_CREATE_USER' sql mode was not set.
2017-03-22T21:00:05.468269Z 0 [Warning] InnoDB: Retry attempts for writing partial data failed.
2017-03-22T21:00:05.468309Z 0 [ERROR] InnoDB: Write to file /var/lib/mysql/xb_doublewritefailed at offset 0, 1048576 bytes should have been written, only 0 were written. Operating system error number 22. Check that your OS and file system support files of this size. Check also that the disk is not full or a disk quota exceeded.
2017-03-22T21:00:05.468321Z 0 [ERROR] InnoDB: Error number 22 means 'Invalid argument'
2017-03-22T21:00:05.469852Z 0 [ERROR] InnoDB: InnoDB Database creation was aborted with error Generic error. You may need to delete the ibdata1 file before trying to start up again.
2017-03-22T21:00:06.070990Z 0 [ERROR] Plugin 'InnoDB' init function returned error.
2017-03-22T21:00:06.071021Z 0 [ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
2017-03-22T21:00:06.071025Z 0 [ERROR] Failed to initialize plugins.
2017-03-22T21:00:06.071028Z 0 [ERROR] Aborting

It's not permissions problems because some files was created. look this screen: http://joxi.ru/n2Y05v5CxGLLm6
I think that some problems with write db files to host machine.
There are ideas for solving the problem.

Still reviving this error on Docker for windows.


Add Support for Resource Manager

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
The same works without OCI Resource Manager.
I am not able to use the module with Resource Manager due to 2 reasons:

when using the module as a reference to my yaml file. There is this error:

The local name "oci.home" in the root module represents provider hashicorp/oci", but ""oci.home"" in module.oke represents ""oracle/oci".

when downloading the full repository and applying it, there is this error:

-> Invalid Terraform version specified in the Terraform configuration file: >= 1.3.0. Update the configuration file to specify a supported Terraform version.
The max version of Resource Manager seems to be 1.2.x...
New or Affected Resource(s)

Potential Terraform Configuration
variable "tenancy_id" {}
variable "region" {}
variable "compartment_id" {}

terraform {
  required_version = ">=1.2.0"
  required_providers {
    oci = {
      source  = "oracle/oci"
    }
  }
}

provider "oci" {
  alias        = "home2"
  auth         = "SecurityToken"
  config_file_profile = "DEFAULT"
  region       = var.region
}

module "oke" {
  source  = "oracle-terraform-modules/oke/oci"
  version = "5.1.0"

  home_region = var.region
  region      = var.region
  ssh_private_key_path = "/home/opc/data/lz/tfok2/ssh_key_starter"
  ssh_public_key_path = "/home/opc/data/lz/tfok2/ssh_key_starter.pub"

  tenancy_id = var.tenancy_id

  # general oci parameters
  compartment_id = var.compartment_id

  worker_disable_default_cloud_init = true
  bastion_allowed_cidrs       = [ "<IP_ADDRESS>/0" ]

  providers = {
    oci.home = oci.home2
  }
}

I can upload also a zip file that fails if it helps.

tfok2.zip
Upload this file in OCI Resource Manager and it will fail...

This module is using features available in terraform starting with version 1.3.0. If you are looking for a deployment through ORM, you can use this source:
The current latest Terraform version available for ORM is 1.2.0.

Thanks, forgot to say that there is a WA (thanks Alberto) it is to use a branch for 1.2 like this:
module "oke" {
source  = "github.com/oracle-terraform-modules/terraform-oci-oke.git?ref=5.x-tf12&depth=1"
providers      = { oci.home = oci.home }
tenancy_id = var.tenancy_ocid
compartment_id = var.compartment_ocid
Identity
create_iam_resources = false
create_iam_tag_namespace = false
create_iam_defined_tags = false
use_defined_tags = false
...
Then it works perfectly.

Resolved in #907


When I lost many time on non working multi file upload, I was surprised why it's not supported. Moreover, fix is trivial, something like

When I lost many time on non working multi file upload, I was surprised why it's not supported. Moreover, fix is trivial, something like
[ContentType.FormData]: (input: any) =>
  Object.keys(input || {}).reduce((formData, key) => {
    const property = input[key];
    if (Array.isArray(property) && property.every(p => p instanceof Blob)) {
      property.forEach(p => formData.append(key, p));
    } else {
      formData.append(
        key,
        property instanceof Blob
          ? property
          : typeof property === 'object' && property !== null
            ? JSON.stringify(property)
            : `${property}`,
      );
    }
    return formData;
  }, new FormData()),

Originally posted by @darky in https://github.com/acacode/swagger-typescript-api/discussions/815#discussioncomment-10907960

I have a similar issue in React Native but I use uri with local system files so I added one more check:
[ContentType.FormData]: (input: any) =>
  Object.keys(input || {}).reduce((formData, key) => {
    const property = input[key]
    if (Array.isArray(property) && property.every((p) => 'uri' in p)) {
      property.forEach((p) => formData.append(key, p))
    } else if (Array.isArray(property) && property.every((p) => p instanceof Blob)) {
      property.forEach((p) => formData.append(key, p))
    } else {
      formData.append(
        key,
        property instanceof Blob ? property : typeof property === 'object' && property !== null ? JSON.stringify(property) : `${property}`,
      )
    }
    return formData
  }, new FormData()),


File_names

File names causing issues with paths.  Need to refactor code to exclude characters not allowed in path names.

fixed


Put DF_UDF plugin code into the main uber jar.

This puts the data frame UDF plugin code into the uber jar. It should now work just like our regular SQL plugin to get DF_UDF functionality.

build

@jlowe I put it under the SQL plugin because I heard some push back on creating another API jar in addition to a shimed implementation jar. I am fine with doing whatever people want me to do, but we just need to decide what that is. This patch was by far the simplest way to get something working so I went with this.

This patch was by far the simplest way to get something working so I went with this.

What I'm proposing is even simpler.  Given it's essentially a standalone component, we could leave everything in the existing df_udf project.  We add the df_udf project to the aggregator project so it ends up being part of the dist jar.  The df_udf project would depend on the sql-plugin-api so it can get access to ShimLoader, and we unshim the "front door" classes for df_udf.  That would let us keep the code in a separate project which would make it easier to separate if we ever decide to do that.
I'm fine with it being part of sql-plugin if that's the way you want to go.  I was wondering if you considered the aggregator approach and eliminated it for some reason.

@jlowe I thought about it, but I wasn't sure how the shimming would work. I am happy to give it a try and get back to you on it.

build

@jlowe and @abellina please take another look. I tried to do what Jason Suggested, but I ran into some issue with the public APIs not working as I expected. I think I could make it work, but this is already working and I don't want to spend too much more time on it right now.


Serialize models with root

Question

Python version import sys; print(sys.version): 3.6
Pydantic version import pydantic; print(pydantic.VERSION): 0.30

How to serrialize models with root
from pydantic import BaseModel
from typing import List

class Pets(BaseModel):
    __root__: List[str]

pets = Pets(__root__=['dog', 'cat'])
print(pets.json())

result: {"__root__": ["dog", "cat"]}
How can I get ["dog","cat"]?

pets.__root__

pets.__root__ is list. But I need to serialize it to json string. Class Pets is just simple example, __root__ can be list of another model.

Just for future reference "["dog","cat"]" is ambiguous, it could be a python object or a JSON string without the quotes, best to be clear what you're taking about.
I guess json() should be changed as per v1 to serialize m.__root__ if it's defined that would fundamentally change it's behaviour from dict() but I think makes sense.
Anyone else, thoughts?

what is the pydantic 2 version of this question? because root is gone

@tommyjcarpenter , the v2 property is just .root.


PowerVM - Informing user of # of in-use VMs

There are a ton of PowerToys.  This the first for VMs :)
@benpye has a great gist for a systemtray that tells a user for how many VMs are in use.
https://gist.github.com/benpye/4d011e65abc4d9c585b8a59f877a2d29
Things to figure out

How we display this information to the user as PT will only have 1 system tray icon
As more things come in, how do we have sanity for settings and end user experience
Looks like the gist polls versus an event push for seeing if something is running
Can we make this expandable past Hyper-V

screenshots from Ben's prototype

This idea looks like it was spawned off of SwiftOnSecurity, FYI.  https://twitter.com/SwiftOnSecurity/status/1199539189177552896

adding in @Swiftonsecurity too :)

So - bit of research here:
Hyper-V WMI API used in the PoC: https://docs.microsoft.com/en-us/windows/win32/hyperv_v2/windows-virtualization-portal
VirtualBox looks like it has a COM API, we should be able to get a list of machines there too: https://www.virtualbox.org/sdkref/interface_i_virtual_box.html
I'm much less familiar with VMWare but it looks like they have a REST API: https://code.vmware.com/apis/412/vmware-workstation-pro-api

we should validate this won't turn on for when sandbox, WSL2 and others are on


Improve layout for 3000x2000 screen with 250% scaling

https://discord.com/channels/372137812037730304/532702442446520321/772901526619553793

#7 is a duplicate. Just consolidating to that one.


Return 401 response for failed authentication attempts.

As per the docucumentation, the authentication backend has to define an authenticate_header method for it to return 401 response for failed authentication attempts. Otherwise a 403 response is returned.

Hi @umgelurgel,
A little bit later then expected, but we just released 0.0.2.
Mike

Awesome! Thanks for letting me know @mikedebock and for your work on this project!


Image padding

Hey, just discovered the lib and great job.
I'm currently using PhotoView which allows padding on images. As far as I can tell padding is ignored when determining the initial image size when using subsampling-scale-image-view.
If it helps this was the previous issue raised for PhotoView:
https://github.com/chrisbanes/PhotoView/issues/29
Cheers

I might be able to add this soon, but would a margin not do the same job?

Setting a margin would stop the image from expanding into the region. I've attached an image to show this:

I'm currently looking to integrate this into "Swipe for reddit" and then "reddit sync".

Okay, that makes sense but I'm not sure this is how I'd interpret padding. That said, there's no more logical behaviour for padding on this view (since it doesn't support child views or display text) so I might be able to implement this. I'm snowed under at the moment so I have a couple of other alternatives, which I haven't tried but I think will achieve this.
I assume the view is the full size of the screen whether the chrome is displayed or not, and the chrome is an overlay.
The first option is to fit the view inside the chrome when it's visible, and expand it to full screen on tap. The view's onSizeChanged method preserves the scale and center, so in theory, as long as the header and footer are the same height, the image should not move or scale when you hide the chrome.
The second option is more complicated, and also assumes the header and footer are the same height.

Extend the class
Override onImageReady
When onImageReady is called, calculate a scale that will reduce the image to the correct size using the image dimensions, view dimensions, and header and footer height.
Set the minimum scale and current scale to this value.

    float scale = Math.min(getWidth() / (float) sWidth(), (getHeight() - (2 * [headerHeight])) / (float) sHeight());
    setMinimumScaleType(SCALE_TYPE_CUSTOM);
    setMinimumScale(scale);
    setScale(scale);
    invalidate();

If that doesn't work, take a look at the other public and protected methods, because I'm reasonably confident there's a solution that doesn't require changes to the class itself.

The margin method does work but this stops content being shown behind the transparent bottom bars when a user zooms in.
I couldn't quite get your second scale based method to work. Seemed to not display.

I deleted the first method from my comment because I realised it wouldn't work, although not for the reason you've described.
Could you send your layout file and the subclass for the second method?

This is my simple implementation. I'm posting a runnable as "onImageReady" seemed to call back on a non-ui thread...
https://gist.github.com/laurencedawson/21028a35f55ba14205d3
Thanks for the help!

Thanks for that, there's just one thing I need to know, and that's how you hide the chrome and what happens to the image view when you do. When the chrome is visible is the view sized to fit within it then resized to full screen on tap, or is the chrome an overlay so the view is the full size of the screen whether the chrome is visible or not?

The SubsamplingScaleImageView is the first child in a RelativeLayout with a Toolbar and LinearLayout overlaid.
Hiding the chrome does not change the size of the SubsamplingScaleImageView.

I've spotted a bug in the setMinimumScaleType method - it results in the view redrawing using the initial value of minScale which is zero, so key state variables get set to NaN. I'll fix this, but there's an easy workaround for you - just move the setMinScale line above setMinimumScaleType.
onImageReady is called off the UI thread to make sure that if subclasses do any heavy lifting in that method, the rendering is not blocked.
A bit of additional hacking is required to stop the image from appearing briefly at full size before the padding takes effect. This appears to work well enough: https://gist.github.com/davemorrissey/9133860908fb4977644c

That seems to have done the trick! Thanks.
On 1 February 2015 at 12:34, davemorrissey<EMAIL_ADDRESS>wrote:

I've spotted a bug in the setMinimumScaleType method - it results in the
view redrawing using the initial value of minScale which is zero, so key
state variables get set to NaN. I'll fix this, but there's an easy
workaround for you - just move the setMinScale line above
setMinimumScaleType.
onImageReady is called off the UI thread to make sure that if subclasses
do any heavy lifting in that method, the rendering is not blocked.
A bit of additional hacking is required to stop the image from appearing
briefly at full size before the padding takes effect. This appears to work
well enough: https://gist.github.com/davemorrissey/9133860908fb4977644c
—
Reply to this email directly or view it on GitHub
https://github.com/davemorrissey/subsampling-scale-image-view/issues/39#issuecomment-72363017
.

No worries. I'll think about how to support padding, the thing I need to work out is how it should behave if top padding is different to bottom padding so the image starts off-centre. I may not support that.
Would you like your apps listed in the readme?

I think PhotoView only supports identical padding too.
On 1 Feb 2015 14:13, "davemorrissey"<EMAIL_ADDRESS>wrote:

No worries. I'll think about how to support padding, the thing I need to
work out is how it should behave if top padding is different to bottom
padding so the image starts off-centre. I may not support that.
Would you like your apps listed in the readme?
—
Reply to this email directly or view it on GitHub
https://github.com/davemorrissey/subsampling-scale-image-view/issues/39#issuecomment-72366285
.

I have added padding support (including asymmetric padding) to the view. I'll release a new version in a few days.

Awesome thanks!
On 1 Feb 2015 22:54, "davemorrissey"<EMAIL_ADDRESS>wrote:

I have added padding support (including asymmetric padding) to the view.
I'll release a new version in a few days.
—
Reply to this email directly or view it on GitHub
https://github.com/davemorrissey/subsampling-scale-image-view/issues/39#issuecomment-72389562
.

Any chance I could get a look at the code pre committing? Thanks
On 2 Feb 2015 08:53, "Laurence Dawson"<EMAIL_ADDRESS>wrote:

Awesome thanks!
On 1 Feb 2015 22:54, "davemorrissey"<EMAIL_ADDRESS>wrote:

I have added padding support (including asymmetric padding) to the view.
I'll release a new version in a few days.
—
Reply to this email directly or view it on GitHub
https://github.com/davemorrissey/subsampling-scale-image-view/issues/39#issuecomment-72389562
.

I've already pushed the code changes, but not built a new release yet. See https://github.com/davemorrissey/subsampling-scale-image-view/commit/e88575898e49725160685298db718b1bae6b36db


Google Feed API will shutdown soon

Since Google Feed API will get shut down on 15th of December, is there a plan to update the RSS part of this plugin with another API like YQL or is it just going to bit the dust?

The RSS part is currently using Yahoo YQL: https://github.com/pavelk2/social-feed/issues/147


Propose overPath to apply a function to a nested property.

Ramda has both path and assocPath for using the [key] strategy to get/set data respectively.
Curious if you guys would find overPath handy, where the signature could look something like:
overPath :: [String] -> (a -> b) -> {k:v} -> {k:v}

and the implementation would be analogous to:
const overPath = (pl, fn, obj) => {
    const was = path(pl, obj);
    return assocPath(pl, fn(was), obj);
}

The motivation being that, where lenses have view, set, and over. Path could be a lighter abstraction with path, assocPath, and overPath respectively.
If that sounds like something you guys dig, I wouldn't mind making a PR for it sometime this week.

While not adamantly opposed, I'm more inclined in the opposite direction.  @davidchambers has made the case for removing some of the functions that overlap with lenses in favor of promoting a consistent lens-based interface.  While I might not be so happy with losing ones like prop or path, ISTM that assoc and its cronies could well go.
Part of the point is that while there is a slight advantage to the conceptual weight of overPath, the usage is not much simpler:
overPath(['a', 'b', 'c'], myFunc, myObj);

is only slightly simpler than
over(lensPath(['a', 'b', 'c']), myFunc, myObj);

And the latter comes along with all sorts of useful abstractions not available to the former.

I happen to totally agree with stripping out the redundancy.
I would add though, IME "spreading the rhetoric" about how cool FP and Ramda are, I've noticed that the lens abstraction tends to shake new players. I find I've had to work up to it in most cases, especially in listeners who don't yet have that "FP itch".
Kind of what I planned to do with this change was to bait and switch listeners. Show them the various path functions and then put a lens in front of them at the last minute in an attempt to make the introduction less ground shattering.

Though, seeing how similar your example functions are, I'm starting to wonder if there would be benefit in merging the two styles?
over(['a', 'b', 'c'], myFunc, myObj);
// could be equivalent to
over(lensPath(['a', 'b', 'c']), myFunc, myObj);

If you're familiar with partial.lenses, this is a similar approach they take. Using a String as a valid lensProp. Which is nice, I enjoy how minimal their lens compositions are.
I've got a PR ready to go if you guys see overPath as being useful. Or If you dig over, view, and set taking arrays as an implicit lensPath, I'd be willing to make an attempt. :)

Using a String as a valid lensProp.

Good luck expressing this in a type signature. ;)

No kidding. Not one to pass up a challenge though!
classtype LensLike a where
    view :: a -> s -> s
    set  :: a -> a -> s -> s
    over :: a -> (v -> v) -> s -> s

instance LensLike (Lens a s) where
    view = R.view
    set  = R.set
    over = R.over

type Path = [String]

instance LensLike Path where
    view = R.path
    set  = R.assocPath
    over = R.overPath  -- Error: overPath is not found :P

Yeah, I'm pretty sure that's all bogus, best I can do.

Though, seeing how similar your example functions are, I'm starting to wonder if there would be benefit in merging the two styles?

over(['a', 'b', 'c'], myFunc, myObj);
// could be equivalent to
over(lensPath(['a', 'b', 'c']), myFunc, myObj);

This is something we're quite loathe to do in Ramda... unless we find a common abstraction shared by the different types we'd accept.  Your suggestion of LensLike doesn't seem to fit the bill, as our lenses are not defined by the relationship to over, view, and set; it's more generic than that.

Oh Sure, it was a stretch and types are fun. :D
Anyway, overPath. Yay, nay? Got it if you want it. Used this library a lot. Always though it would be useful. Team don't want lenses (no matter how awesome they are). Make it easier to switch out all the overs I peppered everywhere.
That's all I got. Go ahead and close this sucker if Nay.

That's all I got. Go ahead and close this sucker if Nay.

I'd like to hear a few more voices before making a call on this.  Again, I don't have a major objection, just an aversion to bloat, especially when there are simple enough techniques to accomplish the same goals with what's already there... and where it doesn't add a great deal in the way of readability.
Anyone else?  @scott-christopher? @raine?  @asaf-romano?

:+1:, this is very common IMHO, I always add it to my ramda wrapper.
How about assocWith and assocPathWith?

Sounds good, Looking at types for similar functions, the closest I've seen is: pathSatisfies, which both uses path and takes a function. I assume we'd want the same layout?
assocWith :: Function → String → Object → Object
assocPathWith :: Function → Path → Object → Object

I totally just opened a PR with the arguments in reverse from what I just suggested... Don't know why I did that. (yeah i do, because I just copy-pasted from assoc/assocPath)
Which would be preferred?
assocWith :: Function → String → Object → Object
assocWith :: String → Function → Object → Object

All of our *With functions take the function first.  So that definitely seems preferable.

PR should be ready to go, I've aligned the functions to the *With format


etcd-operator no longer works

etcd-operator was deprecated two years ago, and does not seem to still work with newer versions of k8s. Installing gives me this error:
 helm install stable/etcd-operator --generate-name
WARNING: This chart is deprecated
Error: INSTALLATION FAILED: failed to install CRD crds/etcd-cluster-crd.yaml: resource mapping not found for name: "etcdclusters.etcd.database.coreos.com" namespace: "" from "": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first

This is the coredns helm repo.

Yes, but the instructions for this chart specify that it requires another chart that is not just broken, but deprecated. I can’t file a bug with the etcd-operator chart, it’s deprecated and archived
Given that etcd-operator is not going to come back, I think this chart needs to be updated with new instructions/dependencies.
—
Ashton Kemerling
Sent from my iPhone.

On Jun 21, 2022, at 10:41 PM, chrisohaver @.***> wrote:

This is the coredns helm repo.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Ah.  I wasn't aware of the third bullet use case...

CoreDNS as an external dns provider for kubernetes federation. This is a sub case of 'external dns service' which uses etcd plugin for CoreDNS backend. This deployment mode as a dependency on etcd-operator chart, which needs to be pre-installed.

Is this a hard dependency on etcd-operator as implied?  Or does this just mean that the mode requires etcd?
IOW, should this instead say:
"This deployment mode requires an etcd service to be present at install time."

I think the instruction intend to convey the following:

This 3rd mode requires an etcd cluster.
This chart does not install an ectd cluster.
An etcd cluster must therefor be installed by some other means.

@chrisohaver according to https://groups.google.com/g/kubernetes-sig-multicluster/c/aZv-YuafSko it seems that kubernetes federation is deprecated and no other project is slated to replace it.
Given that, should we just remove the reference to kubernetes federation from the chart use cases?


Can't Copy Nerd Font Icons/Glyphs

I'm not 100% sure on the right verbiage to refer to these little icons that are available in nerd patched fonts and/or font awesome. But, whenever I have wl-clip-persist running, I cannot copy those items. They just won't copy at all. Closing wl-clip-persist and they are immediately copyable. However, then, they obviously don't persist closing the window I copied from.
Here are some icons that I mean:

󰤨
󰝟
I don't even known how to make them show up here. As I type them I just get tiny boxes.

My mistake. The whole issue is the selection of text is jacked up with wl-clip-persist, not the copying of the glyph.


Analyzing the bottleneck

The first bottleneck occurs here:
https://github.com/liusida/gpuVoxels/blob/dev-CUDA-0.1/Voxelyze3/src/VX3/VX3_Link.cu#L18
while initializing links, every link needs to search the entire voxel list. so it's at least O(t*v).
I am thinking of using a dictionary, but in CUDA there's no such thing...

I am a dummy. This line runs on the host. (After I made a CUDA version of dictionary...) But I am quite satisfied with my dictionary.
Quickly use std::map to make this cache... lol


Fit to screen option

We use the plug in on a large monitor in the office to show the build status.  Since we moved to the most recent version we only get some of our jobs displayed and a scrollbar on the right.
Whilst we have changed font sizes and added columns to manually fit to the screen we can still get issues if the number of jobs displayed on the board changes (we have some jobs that only appear if they have failed).
It would be good if there was a 'fit to screen' option that ensured that all displayed jobs fitted without the need for a vertical scrollbar.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32990343-fit-to-screen-option?utm_campaign=plugin&utm_content=tracker%2F580790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F580790&utm_medium=issues&utm_source=github).

Hey @leemarmara and thanks for your feedback.
The 'fit to screen' functionality is a bit experimental at the moment and I'm continuously working on improving it.
Could you please let me know:

how many jobs do you display on one build monitor
the resolution of your screen
also a screenshot would be really helpful :-)

All the best,
Jan

Hi
Thanks for getting back to me so quickly.
We currently display between 6 and 10 builds (some builds only appear on
the display if they fail).
The resolution is 1920x1080
I will try and get you some shots with the minimum and maximum builds on
them.  I'll have to take a picture rather than get an actual screenshot.
On Sun, Apr 17, 2016 at 8:39 PM, Jan Molak<EMAIL_ADDRESS>wrote:

Hey @leemarmara https://github.com/leemarmara and thanks for your
feedback.
The 'fit to screen' functionality is a bit experimental at the moment and
I'm continuously working on improving it.
Could you please let me know:

how many jobs do you display on one build monitor
the resolution of your screen
also a screenshot would be really helpful :-)

All the best,
Jan
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jan-molak/jenkins-build-monitor-plugin/issues/214#issuecomment-211094389

No worries :) Looking forward to seeing the screenshots!
Best,
Jan

Hi Jan
You won't believe this.  I went to get the screenshots and suddenly the fit
to screen appears to be all working.  Sorry for the waste of time.  I'll
grab a picture if it fails again.
Keep up the good work
Lee
On Mon, Apr 18, 2016 at 4:53 PM, Jan Molak<EMAIL_ADDRESS>wrote:

No worries :) Looking forward to seeing the screenshots!
Best,
Jan
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jan-molak/jenkins-build-monitor-plugin/issues/214#issuecomment-211442473

No worries, thanks for letting me know! Automating the font scaling is a tricky thing to do, so please do let me know if you spot any issues :-)
By the way, I've improved the font scaling algorithm a bit in the latest build. If you'd like to play with it before it hits the Jenkins Update Centre and let me know what you think you can find it here.
All the best,
Jan


Update repo URL and add Gradle Script Kotlin note to Minecraft Dev example

Since moving the repo to the a Github organization, the repo URL has changed.
Seeing the request to add the comment for Gradle Script Kotlin in #163, I added a comment here as well. In addition to using Gradle Script Kotlin, the Minecraft Dev build script also shows examples for:

Running Grammar Kit (modified from the IntelliJ Rust plugin)
Mixing Java, Kotlin, and Groovy code
Using a Kotlin version not bundled with IntelliJ

the Minecraft Dev build script also shows examples for:

This is really cool! Could you add note about this right in README?

I'm not sure what you mean by that. Where do you want this note?

Just nearby uses Gradle Script Kotlin, like uses Gradle Script Kotlin, mixing java and kotlin sources code, etc..
Or under the example item

I've pushed that change. I placed them in a sub-list below the example item, stringing them out in one line looked a bit ugly and unreadable.

Also, should I update the readme with these changes as well in the same commit?

Also, should I update the readme with these changes as well in the same commit?

It would be nice. documentr tasks is responsible for updating README

Alright, done.

thanks!


table: responsive/touch support

Next step: design
Ideas: scrollbar on small screen

Here's a Figma design and prototype for responsive tables and sticky headers. Responsive isn't dependent on sticky headers, but I included it in case it informs how we do the responsive part.

Here are some reference examples of a responsive table and a responsive table with sticky header from Material UI.

Latest PR with small screen scroll is good-as-done. Closing.


[SPARK-26945][PYTHON][SS][TESTS] Fix flaky PySpark tests in StreamingTests

What changes were proposed in this pull request?
This PR proposes to make sure processing all available data before stopping and delete the temp directory.
See https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/102518/console
ERROR: test_query_manager_await_termination (pyspark.sql.tests.test_streaming.StreamingTests)
----------------------------------------------------------------------
Traceback (most recent call last):
 File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/sql/tests/test_streaming.py", line 259, in test_query_manager_await_termination
 shutil.rmtree(tmpPath)
 File "/home/anaconda/lib/python2.7/shutil.py", line 256, in rmtree
 onerror(os.rmdir, path, sys.exc_info())
 File "/home/anaconda/lib/python2.7/shutil.py", line 254, in rmtree
 os.rmdir(path)
OSError: [Errno 39] Directory not empty: '/home/jenkins/workspace/SparkPullRequestBuilder/python/target/072153bd-f981-47be-bda2-e2b657a16f65/tmp4WGp7n'

See https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/102311/console
ERROR: test_stream_await_termination (pyspark.sql.tests.test_streaming.StreamingTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 202, in test_stream_await_termination
    shutil.rmtree(tmpPath)
  File "/usr/lib64/pypy-2.5.1/lib-python/2.7/shutil.py", line 256, in rmtree
    onerror(os.rmdir, path, sys.exc_info())
  File "/usr/lib64/pypy-2.5.1/lib-python/2.7/shutil.py", line 254, in rmtree
    os.rmdir(path)
OSError: [Errno 39] Directory not empty: '/home/jenkins/workspace/SparkPullRequestBuilder@2/python/target/7244f4ff-6b60-4f6c-b787-de4f15922bf5/tmpQbMZSo'

How was this patch tested?
Jenkins tests

retest this please

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8181/
Test PASSed.

Test build #102633 has started for PR 23870 at commit 0499aba.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8182/
Test PASSed.

Test build #102634 has started for PR 23870 at commit 0499aba.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8183/
Test PASSed.

Test build #102636 has started for PR 23870 at commit 0499aba.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8184/
Test PASSed.

Test build #102637 has started for PR 23870 at commit 0499aba.

Test build #102633 has finished for PR 23870 at commit 0499aba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102633/
Test PASSed.

Test build #102634 has finished for PR 23870 at commit 0499aba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102634/
Test PASSed.

Test build #102637 has finished for PR 23870 at commit 0499aba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102637/
Test PASSed.

Test build #102636 has finished for PR 23870 at commit 0499aba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102636/
Test PASSed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8186/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8187/
Test PASSed.

Test build #102639 has started for PR 23870 at commit 66c3282.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8188/
Test PASSed.

Test build #102640 has started for PR 23870 at commit 66c3282.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8189/
Test PASSed.

Test build #102641 has started for PR 23870 at commit 66c3282.

Test build #102639 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102639/
Test PASSed.

retest this please

Test build #102643 has started for PR 23870 at commit 66c3282.

Test build #102640 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102640/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8191/
Test PASSed.

Test build #102641 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102641/
Test PASSed.

retest this please

Test build #102645 has started for PR 23870 at commit 66c3282.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8192/
Test PASSed.

Test build #102646 has started for PR 23870 at commit 66c3282.

Test build #102643 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102643/
Test PASSed.

Merged build finished. Test PASSed.

Test build #102645 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #102646 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102645/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102646/
Test PASSed.

Merged build finished. Test PASSed.

retest this please

Test build #102650 has started for PR 23870 at commit 66c3282.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8197/
Test PASSed.

Merged build finished. Test PASSed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8199/
Test PASSed.

Test build #102654 has started for PR 23870 at commit 66c3282.

Test build #102650 has finished for PR 23870 at commit 66c3282.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102650/
Test FAILed.

Test build #102654 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102654/
Test PASSed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8201/
Test PASSed.

Test build #102659 has started for PR 23870 at commit 66c3282.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8204/
Test PASSed.

Test build #102661 has started for PR 23870 at commit 66c3282.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102659/
Test PASSed.

Test build #102661 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102661/
Test PASSed.

Yup .. let me run this few more times to be 100% sure. Looks it wasn't that flaky much so it's kind of difficult to be sure.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8208/
Test PASSed.

Test build #102668 has started for PR 23870 at commit 66c3282.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8210/
Test PASSed.

Test build #102670 has started for PR 23870 at commit 66c3282.

Test build #102668 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102668/
Test PASSed.

Test build #102670 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102670/
Test PASSed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8212/
Test PASSed.

Test build #102673 has started for PR 23870 at commit 66c3282.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8214/
Test PASSed.

Test build #102675 has started for PR 23870 at commit 66c3282.

Test build #102673 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102673/
Test PASSed.

Merged build finished. Test PASSed.

retest this please

Test build #102678 has started for PR 23870 at commit 66c3282.

Test build #102675 has finished for PR 23870 at commit 66c3282.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102675/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8217/
Test PASSed.

Test build #102678 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102678/
Test PASSed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8222/
Test PASSed.

Test build #102684 has started for PR 23870 at commit 66c3282.

Test build #102684 has finished for PR 23870 at commit 66c3282.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102684/
Test PASSed.

right, looks fixed. I'm going to merge this one and keep fixing flaky tests.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8224/
Test PASSed.

Test build #102689 has started for PR 23870 at commit 4061fe5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8226/
Test PASSed.

Test build #102691 has started for PR 23870 at commit 38601d0.

Test build #102689 has finished for PR 23870 at commit 4061fe5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102689/
Test PASSed.

Merged build finished. Test PASSed.

Test build #102691 has finished for PR 23870 at commit 38601d0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102691/
Test PASSed.

Merged to master.
Thanks, @srowen and @felixcheung

Thanks @HyukjinKwon


[Addition]: Hummingbird selfwitnessing cluster

Hotspot b58 Addresses
1126M5XLhFXDr254HJZzEHMwxPZHTg9iJhGUHrQkXwirk1Niveyz
112ETME825oDvNDxwzvPMHbcqEL6PHpthpUFiSKXRdZgFsVmpiBZ
112gXEu1obEuX5pHjFmMHWiR73fiD91NkF7Ld8EY1EstqjdcFFMZ
112jfmEymq6TPXYPTNS3FeoPBc4Vb9tjmPZt7SPf626nRuadmbTh
112LhPYcoMh9wyLi8taNZzSWCgh86n4gqChHsuRyks8th7RGSWxj
112SctJKzxcu3JGArQwfRc58qwMFGpvAikGC26YN6i8UnRQeC5LQ
112sQQUNGqLbt6Rt9TRvpPagtMAkwLJ68HaRGfxsyHSAL1Usy9WM
112szS3md7ezyqGBeZQPLNfNqsct7wVQxYKMy1jH9H4TnoHzXdFu
11fHWY46CWy2Uy7svz44fDQrQ8mUgVJc8jCr55fFJ8nLt3zUzXB
11JUUdEdVBgXQcCBdUMBKUTe56R4AU3oqQUZjwnP5hWre5HLFgB
11K3FDkZoBQXP7Gxbj2bYVBz3QurCTZfyaKTsetbKi6GKWabtVw
11MnJ4oVuHJ4dwkXrhkdaMZt2nKaX2MPA7kVTEdkWpuMGvsHrp4
11oGozCEWiA7UqqGChLGRfy8EFnA73NFKtLVu9gkUmHr1J6KNmC
11pxoVyagEurAptLKsffcRLd5uFrh1wD415EHPfLsJJf5xBUkT4
11rP5sv3mkfzew22ysyDpzUkyVFAsLVe8RAZ4WmAsXChTcLkrhq
11YqZUHCJTupcpMVhM8AukvFrRmLFz8z6eTujaZKVoqZvoesxAb
Discord Handle
No response
Reason(s)

All but 1 account for all Hummingbird gateway's didn't show any data transfers.
So data transfer is possible! only 109 out of 3853 showed packet transfer data, all these 109 are in same account.
the gateway's in that account are part of a selfwitnessing cluster so they probably all heared the same
signal.

Most of all Hummingbird gateway's witness only Hummingbird gateway's.

We iterated the witnesses of the witnesses to find the clusters.
We used the node-red server we setup for this to get the results out of api.

All HNT stolen with the total amount of 21067 HNT as by todays date/time.

We conclude that out of the 3852 gateway's we inspected only 109 gateway's in 1 account showed accidental
data transfer. so data transfer for gateway's Hummingbird is possible.

We conclude that out of the 3852 gateway's we inspected only a couple of accounts showed communication with other makers. so communication with other makers for Hummingbird gateway's is possible.

Rssi/ distance / SNR calculations are to perfect, we have found some invalids -59 dbm but we suspect they are from cables touching eachother.

We haven't targetted any maker, it was the result of a count WHERE actors1.actor_role = 'witness'
AND actors2.actor_role = 'challengee' are in same account.

I'd wait on this one a bit / needs a bit more time.
Showing some indicators of genuine activity.


TSS: sprint 71: dependabots

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.80%. Comparing base (88b1e54) to head (f2df229).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #232   +/-   ##
=======================================
  Coverage   90.80%   90.80%
=======================================
  Files          70       70
  Lines        2078     2078
=======================================
  Hits         1887     1887
  Misses        191      191

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


Don't depend of enabled pdf/epub to show downloads

Fixes #5497
We just need to rely if the epub/pdf exists, we already do this with the localmedia download.

Apart from the documentation_type #4638 I don't think so.


Is ungoogled-chromium-bin safe at the AUR?

ungoogled-chromium-bin

I've been using it for the last several months with no issues. It seems there is an official build for Arch Linux, but the the AUR package doesn't appear to be using it yet. Right now it's downloading the binary from https://cloud.woelkli.com/. According to the user hosting it there, he says you just have to trust him. The best move at this point would be for the packager to change the source URL, since fishy looking links like that may deter people from using it. It's up to you if you want to use it.

If the Arch Linux build is reproducible, then we could see if others can build and verify. Otherwise, there's no good way to determine safety.


Executing JavaScript?

Just a question - I can't seem to find any recent information on this so I'm just going to ask here. Please let me know if there is a better place for my question.
Is it possible to execute JavaScript on a page? For my use case, I am parsing data from urls like this: https://www.mountainproject.com/search?q=y-crack . That search page provides a "snippet" of the results and you need to click "See all 67 routes" in order to view all of them (and sometimes also click the "Load More" button on the bottom of the next page).
Both of these buttons trigger JavaScript to load in the other results. Is it possible to trigger these with AngleSharp so that all the results can be scrapped?

Not sure I can follow - what have you tried so far?
There are multiple questions on this on StackOverflow and a recent one in this repo, see #60. Does that help?


Fixed #31507 -- Added .exists() optimizations to .union().exists

Ticket: https://code.djangoproject.com/ticket/31507
Hi,
I added the optimizations that we have on the .exists() call (removing selected columns for a "SELECT 1", removing ordering and limiting the number of results to 1) to a call with the .union() combinator. So we also strip the selected columns and remove ordering, but we cannot add a limit clause in the subqueries.
We don't apply the optimization for other combination predicates (difference, intersection) - should I add test cases for those, asserting that don't have this behaviour for these cases?
Cheers,
David

Hi @charettes, thanks for the review, once again 😄
Splitted in 3 commits, so they can be amended/dropped more easily:

I moved the test into test_qs_combinators.py and added test cases for .intersection() and .difference() that fail if we change we include them in the optimization
I tried implementing a DB feature called supports_limiting_in_compound
adding the .exists() in the documentation examples of available functions for the combined QuerySets

Awesome @charettes 👍 that's exactly what I thought. I tried to pass the DB alias, because I didn't want to use the default one (because it's not necessarily the correct one), but I stumbled upon the call in Exists.__init__ which didn't have this information when being called. Thanks a lot for the suggested improvement 🚀
And also good call for merging the two test methods :)
Thanks for pointing this out @felixxm 👌  The patch became much more redable by using the existing DB feature.
I amended the second commit with the improvements

buildbot, test on oracle.


Fix adaptive height when multiple slides are visible

Issue
When multiple slides are visible and adaptiveHeight is turned on, then slider's height is calculated from first visible slide only.
Desired behavior
Slider height should be calculated as max. height of all slides visible.
How to reproduce problem:
var settings = {
  infinite: true,
  // Multiple slides visible
  slidesToShow: 3,
  slidesToScroll: 1,
  // Slider height is adaptive (set dynamically based on content height)
  adaptiveHeight: true
};

return (
  <Slider {...settings}>

1

2

Hello

3

See ....

Height is adaptive

  </Slider>;
);

@akiran May I ask you for a review so we can fix this bug?

This works for me. Thanks @Poky85
Added to npm
Anyone who wants this yarn add react-slick-adaptive-height-fixed then find/replace in project: react-slick with react-slick-adaptive-height-fixed

Works for me as well. When could we expect a review on this issue ?

Any updates on this?

Also have this annoying issue, can we merge please????

afaik react-slick-ad-height-fixed is a patched version.

afaik react-slick-ad-height-fixed is a patched version.

Correct, I am reporting a bug in the patched version. Without the patch the problem is much worse :(

afaik react-slick-ad-height-fixed is a
Correct, I am reporting a bug in the patched version. Without the patch the problem is much worse 😢

there is two patched versions, react-slick-adaptive-height-fixed and react-slick-ad-height-fixed, you talked only about the first one, did you try the later?

Could this please be merged?

Waiting for the merge!

waiting for merge in 2023 😂


projects: Add missing fields to struct

The original motivation was to add Project's HtmlURL field as we need it in https://github.com/terraform-providers/terraform-provider-github/pull/111
but I took some time to look at all missing fields in both Project & ProjectColumn, just for completeness.
The PR covers only one, small part of https://github.com/google/go-github/issues/884 - there is still more work to do before #884 can actually be closed - specifically around Project Collaborators and Team Projects.

@gmlewis Thanks for the review. I believe I addressed all your comments. PTAL.

@gmlewis done, PTAL


Secure secrets in develop cluster

The raster API requires a couple of secrets for access to restricted buckets.
Currently, these are included as clear text in the IAC, which is of course not sustainable.
https://github.com/EOEPCA/eoepca-plus/pull/34#issuecomment-2372293121
What options do we have? Sealed Secrets?
Perhaps look at what the minio setup does? https://github.com/EOEPCA/eoepca-plus/blob/28cd6d3e548a673047a832fc9ea602721ab30a5a/argocd/infra/minio/README.md?plain=1#L7
Acceptance criteria

[x] No more clear text secrets in git
[ ] Solution works

I think this still applies:
https://deployment-guide.docs.eoepca.org/current/cluster/cluster-prerequisites/#sealed-secrets
@rconway ?

@j08lue @kalxas
Yes - we are still using SealedSecrets.
Typically we use a helper script to create them - for example - https://github.com/EOEPCA/eoepca-plus/blob/deploy-develop/argocd/infra/minio/ss-minio-auth.sh
This relies on having kubeseal installed locally.

Two PRs went in. We need to check whether they work, which is not as straight forward.

I redeployed. This link should show data, but I have the challenge that the environment variables are  filled with map[valueFrom:map[secretKeyRef:map[key:AWS_ACCESS_KEY_ID name:data-access namespace:data-access]]] instead of the decrypted value. Going to do debug it.

It turns out:

envVars is specific to eoAPI's Helm charts and is used to populate a ConfigMap with variables that are then made available as environment variables to the application.
It is not possible to read values from a secret and include them during the creation of the ConfigMap.
To address this, I modified eoAPI to support a new argument, envSecrets, which allows entire secrets to be added as environment variables.
I deployed it an the test link (one post up) is now working correctly again.


AssertionError message when running reconstruct_contig.py (cogent 3.5)

Hi @Magdoll,
I am using Cogent 3.5 to process the high-quality isoforms generated from the isoseq3 pipeline. Now I still have one failed job in the reconstruction step. Below is the error message I got:

2.10-r770-dirty
/data/jiaochen/miniconda3/envs/anaCogent/lib/python2.7/site-packages/Bio/Seq.py:349: BiopythonDeprecationWarning: This method is obsolete; please use str(my_seq) instead of my_seq.tostring().
BiopythonDeprecationWarning)
[M::mm_idx_gen::0.0015.07] collected minimizers
[M::mm_idx_gen::0.0013.33] sorted minimizers
[M::main::0.0013.25] loaded/built the index for 1 target sequence(s)
[M::mm_mapopt_update::0.0013.09] mid_occ = 2
[M::mm_idx_stat] kmer size: 15; skip: 5; is_hpc: 0; #seq: 1
[M::mm_idx_stat::0.0012.92] distinct minimizers: 40 (100.00% are singletons); average occurrences: 1.000; average spacing: 3.325
[M::worker_pipeline::0.0050.97] mapped 6 sequences
[M::main] Version: 2.10-r770-dirty
[M::main] CMD: minimap2 -ax splice -t 1 cogent.fa in.trimmed.fa
[M::main] Real time: 0.005 sec; CPU: 0.005 sec
Traceback (most recent call last):
File "/home/jiaochen/miniconda3/envs/anaCogent/bin/reconstruct_contig.py", line 4, in
import('pkg_resources').run_script('Cogent==3.5', 'reconstruct_contig.py')
File "/home/jiaochen/miniconda3/envs/anaCogent/lib/python2.7/site-packages/pkg_resources/init.py", line 658, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/home/jiaochen/miniconda3/envs/anaCogent/lib/python2.7/site-packages/pkg_resources/init.py", line 1438, in run_script
exec(code, namespace, namespace)
File "/data/jiaochen/miniconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.5-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 337, in
main()
File "/data/jiaochen/miniconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.5-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 259, in main
run_Cogent_on_input()
File "/data/jiaochen/miniconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.5-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 239, in run_Cogent_on_input
post_minimap2_processing('cogent.fa', sam_file, 'cogent2', seqrecs=seqrecs)
File "/data/jiaochen/miniconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.5-py2.7.egg/Cogent/Utils.py", line 48, in post_minimap2_processing
assert r.sID.startswith('path')  # chr should be path0, path1, etc
AssertionError

Do you know why I got this message and how to deal with this? I can send you the in.fa and in.weights files. Thanks!
Chen

Hi,
Indeed, from the error it's not obvious to me what went wrong.
Please give me an email to request a file upload to.
--Liz

Hey,
I got exactly the same error for 4 of my reconstructions (Cogent 3.5). In the hello.log file of the respective files you can find the following:
2018-09-05 09:23:26,592 - Cogent - INFO - Setting k-mer size to: 60
2018-09-05 09:23:26,592 - Cogent - INFO - Setting expected error rate to: 1%
2018-09-05 09:23:26,625 - Cogent.splice_cycle - INFO - K-mer in-seq cycle detection: None found at k=60
2018-09-05 09:23:26,718 - Cogent - INFO - Doing nx.cycle_detection....
2018-09-05 09:23:26,720 - Cogent - INFO - Initial Graph Size: 4 nodes, 2 edges
2018-09-05 09:23:26,721 - Cogent - INFO - Post-Reduction Graph Size: 2 nodes, 0 edges
2018-09-05 09:23:26,722 - Cogent.process_path - INFO - number of paths now: 1
2018-09-05 09:23:26,722 - Cogent.process_path - INFO - number of paths now: 1
2018-09-05 09:23:26,722 - Cogent.process_path - INFO - number of paths now: 1
2018-09-05 09:23:26,722 - Cogent.process_path - INFO - number of paths now: 2
if I increase k to 200 the hello.log file shows the following:
2018-09-05 09:24:46,582 - Cogent - INFO - Setting k-mer size to: 200
2018-09-05 09:24:46,582 - Cogent - INFO - Setting expected error rate to: 1%
2018-09-05 09:24:46,630 - Cogent.splice_cycle - INFO - K-mer in-seq cycle detection: None found at k=200
2018-09-05 09:24:46,761 - Cogent - INFO - Doing nx.cycle_detection....
2018-09-05 09:24:46,764 - Cogent - INFO - Initial Graph Size: 2 nodes, 1 edges
2018-09-05 09:24:46,766 - Cogent - INFO - Post-Reduction Graph Size: 1 nodes, 0 edges
2018-09-05 09:24:46,766 - Cogent.process_path - INFO - number of paths now: 1
2018-09-05 09:24:46,766 - Cogent.process_path - WARNING - [BUG] Missing good_for for transcript/10742. Maybe known issue (graph cycles). Use own path.
2018-09-05 09:24:46,766 - Cogent.process_path - WARNING - [BUG] Missing good_for for transcript/10879. Maybe known issue (graph cycles). Use own path.
Maybe it will help you to find the bug?
Thanks,
Stephie

Hi Liz,
my email address is<EMAIL_ADDRESS>Thanks!
Chen

@jayceejiao , a request email has been sent to you to upload the file.
@steffi778 , let me see what happens with @jayceejiao's error. Maybe I can figure out why this happens in general.

Hi @Magdoll ,
Did you find the reason causing this problem? I have another project also reports this error for three families. Thanks for your help!
Chen

Hey @Magdoll
Thank you very much for your help! But where can I find version 3.6? When I try to install it via git and conda, the newest version I can get is still 3.5


Add Python Testing

Include Tox to make running tests easier
Fix a set of rules based on failures in the code
Write tests for a set of linting rules

Lots of linting fixes for python still required.  When I have some more time I will fix that as well.

I suggest to open a conversation first on how to integrate testing in this repo. It is a step in a good direction.
For fixes in the rules, please re-submit them independently.


Validate operation name in Optimize command tests - Fix #1756

Description
Validate the operation name for optimize command - Fix #1756
Also changed DeltaLog.snapshot to DeltaLog.unsafeVolatileSnapshot, as snapshot field is deprecated.
How was this patch tested?
Run tests locally
Does this PR introduce any user-facing changes?
No

@vkorukanti could you review it again?

@vkorukanti done. Thanks for reviewing it

Also removed unneeded take(1) and imports


[vrt] support label vrt skip

ref: https://blog.studysapuri.jp/entry/2024/3/26/introduce-roborazzi-2

JaCoCo Code Coverage 76.92% :white_check_mark:

Class
Covered
Meta
Status

  Generated by :no_entry_sign: Danger


Add functionality for downloading/parsing Mac1 data from CDD

Add functionality for downloading/parsing Mac1 data from CDD.

Codecov Report

Merging #420 (c07dbca) into main (8fbbe8e) will increase coverage by 3.71%.
The diff coverage is 100.00%.

Might be good to test the MAC1 download but as this fixes CI I will merge for now.


Missing to_h cast in repo discovery script

if write make dataset/repositories.csv with Total number of repositories to fetch from GitHub. > 30 and % 100 != 0 we get an exception:
cam/steps/discover-repos.rb:125:in `<main>': undefined method `values' for an instance of Array (NoMethodError)
lines = [found.first[1].keys.join(',')] + found.values.map { |m| m.values.join(',') }
Did you mean?  values_at

This happens because if we exit the loop before we have collected enough repositories, the script tries to slice hash - line with bug
i suppose we miss to_h cast

@yegor256 fyi
noticed it when noticed it while I was working on https://github.com/yegor256/cam/issues/186

fixed


Show GitHub profile link of every contributor in the release notes

Current release notes list the authors which is great because it motivates to contribute! However, the author is listed by name, without the link to GitHub profile. The profile would be useful for more exposure / fame for the contributors.
Last time I worked on it, there was no way to link user to GitHub profile.
TODO:

try to find GitHub user name given that we have Git author/author email. Engage community, ask on GitHub tracker, ask on twitter, etc.
as suggested by @TimvdLippe we can use GitHub search API<EMAIL_ADDRESS>
🚀 Looking for help!!!
If you like to party, try to implement this interface and submit a pull request! :) To get started, run "GitHubTicketFetcherTest" test, it actually hits GitHub to get data.
/**
 * Maps Git authors to GitHub users
 */
public interface GitHubUserMapper {

    /**
     * Provides mapping from Git author email to GitHub user name.
     *
     * @param authorEmails email addresses of git authors
     */
    Map<String, String> mapToGitHubUser(Collection<String> authorEmails);
}

I think it's better to search by login, NOT by e-mail. Search by e-mail doesn't work if you change Public e-mail to "Don't show my e-mail address" in Github settings page. I think this is better way: https://api.github.com/search/users?q=TimvdLippe&in=login

I looked into the code and GitHub API a little bit. Generating notes has 2 sources of true:

git log for read commits and
GitHub API /issues for read info about solved issues.

In first case we can read user git name and e-mail and from second source we can read GitHub login (user.login) and GitHub user URL (user. html_url). We can use this data to link to user profile in release notes, but sometimes we are unable to match this two informations (GitHub login can be differ from git user name).
Also we are unable to read link to GitHub profile from commits: https://developer.github.com/v3/git/commits/ We see only the same information like in git log i.e. name and e-mail.
I didn't ask on GitHub tracker / twitter.

Hey! Long time no hear :)
When the release automation is getting git log, we don't have access to the GitHub login (we have Git user name and email). Some users might have the same GitHub login as Git user name but it's unlikely. Hence, search by email is probably the best given circumstances.
Do you want to help out with the implementation? When you clone and open the project in IDEA, run "GitHubTicketFetcherTest" test. The test actually hits GitHub.
Thanks for suggestions!!!

If you like to party, try to implement this interface :). To get started, run "GitHubTicketFetcherTest" test.
/**
 * Maps Git authors to GitHub users
 */
public interface GitHubUserMapper {

    /**
     * Provides mapping from Git author email to GitHub user name.
     *
     * @param authorEmails email addresses of git authors
     */
    Map<String, String> mapToGitHubUser(Collection<String> authorEmails);
}

You're welcome!
As I said in my first comment, searching by e-mail will usually not work, try this<EMAIL_ADDRESS>(I found your e-mail using GH API, you can delete this comment if you don't like to publish it ;) ).
Unfortunately this is a gap in GH API that we can't easy link commit author with GH page author. I see a few solutions.
Solution 1. GitHub team will add link to user profile in /commits endpoint (generation notes will be slowly because of possible big number of needed requests).
Solution 2. We can read GH profile link from /issuses (see: https://api.github.com/repos/mockito/mockito/issues?state=closed&filter=all&labels=&direction=desc generated by GitHubTicketFetcher.GitHubIssues.GitHubIssues(...)) and link only to creators of PullRequest using GitHub login and link to profile. See result of this API call: https://api.github.com/users/szczepiq then we will see in release notes something like this: szczepiq
Solution 3. GH team will add to /issuses git name, or in https://api.github.com/users/szczepiq or somewhere else, then we can generate notes like this: Szczepan Faber
Solution 4. We can use Github UI instead of API and get this information from HTML. But then change in UI can destroy everything.
I can implement something, but maybe it's better to first ask GitHub team for improvement in the API?

Thank you for researching!

is impractical, 2) will not match accurately (user assigned / changing issue not necessarily is the implementor)

How about using search API, e.g. searching for commits: https://developer.github.com/v3/search/#search-commits

The search API is in developers preview and it's hard to reduce results to reasonable size.
A little bit better looks commit API: https://developer.github.com/v3/repos/commits/#list-commits-on-a-repository
e.g.:
https://api.github.com/repos/mockito/mockito/commits
It's also hard to limit results, but reading sha of commits from git log can help us to match authors with GitHub profile pages. I can implement this.
I asked also GitHub API support for few improvements from this thread.

Great work @mstachniuk, we're about to use it in Mockito: https://github.com/mockito/mockito/pull/987
THANKS!!!


zh_Trans:  Fix preview issue

/lgtm
/approve

/lgtm
/approve

We should never do such huge PRs simply for fixing the previews. It is too dangerous to introduce many hidden defects ...

We added the pre-translated content to the document in case the link failed.

@Mr-Linus If that should be done, it should be done very very carefully. As is pointed out by @tengqm in https://github.com/kubernetes/website/pull/11311#discussion_r236575474, that mistake is not easy to find out when reviewing. Obviously, it was introduced by some local git operations such as "git rebase".

Sure.The catalog has been specially translated and the task project will ensure that our tasks are properly distributed.

If you look closely, you will find that only the pre-translated content of the glossary directory is uploaded to prevent the link from being invalidated. The rest is the static yaml configuration file in the example. The purpose of this is to prevent future links from failing.

The problem of this PR is that it had incorrect content (as is pointed out by @tengqm in https://github.com/kubernetes/website/pull/11311#discussion_r236575474) but was merged.
Please take a look at the file content/zh/docs/reference/glossary/quantity.md. It has some contents repeated and noisy git info.
<<<<<<< HEAD
||||||| merged common ancestors
=======
>>>>>>> master

This is not the result of git. The English language content still has the same content.   https://github.com/kubernetes/website/blob/release-1.12/content/en/docs/reference/glossary/quantity.md

OK.No Problem. 👌


intellisense in comments

VSCode Version: 1.5.2
OS Version: windows 7 professional sp1

Steps to Reproduce:

create a .js or .php file
type in a comment

Intellisense should be disabled while typing in a comment.

Duplicate of #11704
@xucaen Please go through the discussions in #11704 and other issues mentioned there to get to know the latest on this issue


refactor bootstrap

Requrements

separate inernal services start, bootstrap and public endpoints starts
try to start VVM with an app with different number of partitons or app workspaces -> panic
prepare storages for blobber and router apps as *IAppStorage before bootstrap
blobber and router apps are not builtin, not used as apps at all. Storages only are used

Internals

clusters/bootstrap

Depends on

#2022

start everything but:

API endpoints (router, mertics service)
async actualizers

bootstrap

requests directly to bus?
implement system API endpoint on http that accepts requests from <IP_ADDRESS> only?

start the rest of services

where to initialize IAppStorage for router and blobber apps? The doc says about it in 2 places: within Bootstrap() and within sync operator that calls Bootstrap()


How to insert foreign key using auto generated identity columns in Azure Functions using SQL OutputBinding in isolated model?

I am trying to use the SQL OutputBinding attribute in Azure Functions (isolated process model) to insert data into two related tables in Azure SQL database. The tables are Invoice and InvoiceLineItem, and they have a one-to-many relationship. The Invoice table has an identity column (InvoiceId) that is the primary key, and the InvoiceLineItem table has a foreign key column (InvoiceId) that references the Invoice table. I want to insert data into both tables using the SQL OutputBinding attribute, but I don't know how to get the auto-generated identity value from the Invoice table and use it as the foreign key value for the InvoiceLineItem table. Here is the C# code for the classes that represent the tables:
`public class Invoice
{
public long InvoiceId { get; set; }
public int? InvoiceClientId { get; set; }
public string CurrencyCode { get; set; }
public ICollection LineItem { get; set; }
}
public class InvoiceLineItem
{
public long InvoiceLineItemId { get; set; }
public long? InvoiceId { get; set; }
public int? Hours { get; set; }
public double? RatePerHours { get; set; }
public string WorkDescriptions { get; set; }
}
`
How can I use the SQL OutputBinding attribute to insert data into both tables and use the auto-generated identity value from the Invoice table as the foreign key value for the InvoiceLineItem table? Can anyone provide a demo or an example of how to do this? Thank you.

Hi @nyghtrocker, unfortunately there is no way to return auto-generated values from a SQL Output Binding. You can try using two functions:

one function with a SQL Output Binding to insert values into Invoice
and a second function with a SQL Input Binding to get the InvoiceId from Invoice and a SQL Output Binding to insert values into InvoiceLineItem

Closing this issue since there is no way to get values return from output bindings. @nyghtrocker Feel free to open an issue in the Azure Functions Host repo if this is functionality you'd like to see.


Feature: provide arguments when configuring test-cmd

Issue
I am heavily relying on test-cmd to safely request new changes at it will retry for a few times to successfully add changes to code.
The problem is it can very well accept a single-word. It can't accept "dotnet test" or "pwsh -File test_file.ps1 arg1 arg2"
It would be helpful to have a way either to specify arguments separately or within the existing cfg property "test-cmd".
When specifying "dotnet test" or anything similar the error is: The filename, directory name, or volume label syntax is incorrect.
Version and model info
Aider latest,
GPT-4o

Thanks for filing this issue. You can quote the multi-word test command like this:
aider --test-cmd "dotnet test"

For example:
$ aider --test-cmd "echo 1 2 3"
Aider v0.37.1-dev
Models: gpt-4o with diff edit format, weak model gpt-3.5-turbo
Git repo: .git with 38 files
Repo-map: using 1024 tokens
Use /help to see in-chat commands, run with --help to see cmd line args
────────────────────────────────────────────────────────────────────────────────────────────────
> /test

1 2 3

I've configured yml like this:
## Specify command to run tests
test-cmd: "dotnet test"

## Enable/disable automatic testing after changes (default: False)
auto-test: true

## Run tests and fix problems found
test: true

And startup doesn't work properly and looks like this:
(aider) C:\workdir>aider
Aider v0.37.0
Model: openai/gpt-4o-2024-05-13 with diff edit format
Git repo: .git with 14 files
Repo-map: using 1024 tokens
'"dotnet test"' is not recognized as an internal or external command,
operable program or batch file.

Thanks for filing this issue. You can quote the multi-word test command like this:
aider --test-cmd "dotnet test"

For example:
$ aider --test-cmd "echo 1 2 3"
Aider v0.37.1-dev
Models: gpt-4o with diff edit format, weak model gpt-3.5-turbo
Git repo: .git with 38 files
Repo-map: using 1024 tokens
Use /help to see in-chat commands, run with --help to see cmd line args
─────────────────────────────────────────────────────────
> /test

1 2 3

(aider) C:\Users\ELI\workdir>aider --test-cmd "echo 1 2 3"
Aider v0.37.0
Model: openai/gpt-4o-2024-05-13 with diff edit format
Git repo: .git with 14 files
Repo-map: using 1024 tokens
Use /help to see in-chat commands, run with --help to see cmd line args
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
> /test

'"echo 1 2 3"' is not recognized as an internal or external command,
operable program or batch file.

I am unable to replicate what you have showcased.

Something about your shell environment is preventing --test-cmd "echo 1 2 3" from being cleanly parsed as 2 command line arguments: --test-cmd and echo 1 2 3. For some reason your shell is parsing them as --test-cmd and "echo 1 2 3".
Both cmd.exe and PowerShell should parse them properly. I am probably not able to help debug your local shell environment.


add support for PowerScale topology

Description
A few sentences describing the overall goals of the pull request's commits.
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/334

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[x] I have maintained at least 90% code coverage
[x] I have inspected the Grafana dashboards to verify the data is displayed properly
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have maintained backward compatibility

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[x] Test A: UT

[x] Test B: manual testing

Manual inspection of the GUI
I have verified that the dashboards show the data properly while generating I/O and storage resources

[x] Yes
[ ] No

Can one of the admins verify this patch?

Hi Yian, please hold off on merging any PRs to the main branch until after CSM 1.3 release (06/21)

Sure, @hoppea2

Hi @hoppea2 , can we resume PR review and merge for Q3 features now?

run e2e test

Hi code owners, please kindly review and approve this PR. Thank you!


Added mulAdd and cosMinusOne operations for Real.

Defines two new operations for types conforming to Real:
/// cos(x) - 1, computed in such a way as to maintain accuracy for small x.
static func cosMinusOne(_ x: Self) -> Self

/// a*b + c, computed _either_ with an FMA or with separate multiply and add.
static func _mulAdd(_ a: Self, _ b: Self, _ c: Self) -> Self

These are mostly implementation hooks for Complex, but are generally useful, so I've made them public. Experimental, subject to renaming, especially mulAdd. The first is "trivial" to define using the half-angle formula, but it's a sufficiently useful implementation hook that it's worth having.

CC @NevinBR who has asked about functionality like _mulAdd in the past. It's not exactly what you've asked for, but it's sufficiently useful that you may want to borrow and adapt it.


Merge branch 'main' into sparsehistogram

Attempts to merge 'main' branch into sparsehistogram
Main contains a few big changes including the out-of-order support that conflicts with many parts of the code modified to bring the new sparse histograms.

Closing due to an accidental rebase.


Connect to ADLS Gen2 using client_id/client_secret

I want to connect to ADLSGen2 using a service principal, i.e. tenant_id, client_id, and client_secret.
According to #75 this should be possible.
But the current README says the opposite. adl for Gen1 and abfs for Gen2.
This works:
import dask.dataframe as dd
dd.read_parquet(
    path='abfs://container/folder/file.parquet',
    engine="pyarrow-dataset",
    storage_options={"account_name": STORAGE_ACCOUNT, "account_key": STORAGE_KEY}
    )

This fails:
dd.read_parquet(
    path='adl://storage_account/container/folder/file.parquet',
    engine="pyarrow-dataset",
    storage_options={"tenant_id": TENANT, "client_id": CLIENT_ID, "client_secret": CLIENT_SECRET}
    )

This solved it for me, as discussed in dask/dask#8677:
dd.read_parquet(
    path='abfs://CONTAINER/FOLDER/FILE.parquet',
    engine="pyarrow-dataset",
    storage_options={"tenant_id": TENANT, "account_name": ACCOUNT_NAME, "client_id": CLIENT_ID, "client_secret": CLIENT_SECRET}
    )

I think the README is a bit misleading, since it guides the user into using shared key in the quick guide, which is not recommended security wise. Maybe a note about using security principal in production should be added.


add queries error

@mwielgoszewski ,
When I add query , there is error happened(maybe input form has some problems:)
File "/data/doorman-master/doorman/utils.py", line 261, in validate_osquery_query
def validate_osquery_query(query):
# Check if this thread has an instance of the SQLite database
db = getattr(osquery_mock_db, 'db', None)
if db is None:
db = create_mock_db()
osquery_mock_db.db = db
try:
    db.execute(query)
except sqlite3.Error:

File "/data/doorman-master/doorman/utils.py", line 253, in create_mock_db
def create_mock_db():
mock_db = sqlite3.connect(':memory:')
for ddl in schema:
print ddl
mock_db.execute(ddl)
return mock_db
def validate_osquery_query(query):
# Check if this thread has an instance of the SQLite database
OperationalError: near "��𐁅񐁔����𠁁񐁌������������������������������������������������������������񐁔񐁇������������������������": syntax error
The debugger caught an exception in your WSGI application. You can now look at the traceback which led to the error.
And input form is  ::
name:processtest
query:select * from processes;
interval:3600
platform:linux
version 1.4.5
description:"test"
value:"test"
query options:log removed(default)
packs:no input
tags:no input

I am really not sure how you encountered this - I am unable to reproduce it. @andrew-d @dguido have you guys experienced this?

@mwielgoszewski
We found the root cause of this problem, because we use python 2.7, the input date is unicode, and   when we modify code from mock_db.execute(ddl) to mock_db.execute(ddl.encode()) and the problem is fixed.
Thank you very much.

@mwielgoszewski
One more question, does these items (Name	Make	Model	Serial	Cpu	Cpu Cores	Memory) of webstie https://xxx:5000/manage/nodes from osqueryd real time collecting by doorman ? or there is another program collecting by osquery log and push them to database, doorman is only a webui ?
Because after I set up doorman and register node by "ENROLL_SECRET=secret osqueryd --flagfile osquery.flags" ,but https://xxx:5000/manage/nodes webui do not have any data of Name	Make,Model,Serial,Cpu,Cores,Memory.

If there is any configure about this(eg. ) ,  can you tell more details?
Thanks.

@foreversunyao See DOORMAN_CAPTURE_NODE_INFO key in the configuration file for details. You'll need to schedule some queries to actually query that information.
If you create a distributed query, and the state always remains NEW, then the nodes the query is intended for are not getting them. What values do you have in your osquery configuration for the following flags?

--disable_distributed
--distributed_plugin
--distributed_interval
--distributed_tls_max_attempts
--distributed_tls_read_endpoint
--distributed_tls_write_endpoint

@mwielgoszewski
I saw DOORMAN_CAPTURE_NODE_INFO this configuration , but I don't know where can I configure this query(select * from system_info;), I tried to configure it at doorman query tab and /etc/osquery/osquery.conf at the node(local server) as below, but they did not work anyway, although osquery log has this message. ? Did I miss some details ? Thanks.
osquery.conf :
{
"options": {
"host_identifier": "hostname",
"schedule_splay_percent": 10
},
"schedule": {
"system_info": {
"query": "select * from system_info;",
"interval": 10
}
}
}
osquery flags configuration is as below:
--pidfile=/tmp/osquery.pid
--host_identifier=uuid
--database_path=/opt/osquery.db
--config_plugin=tls
--config_tls_endpoint=/config
--config_tls_refresh=10
--config_tls_max_attempts=3
--enroll_tls_endpoint=/enroll
--enroll_secret_env=ENROLL_SECRET
--disable_distributed=false
--distributed_plugin=tls
--distributed_interval=10
--distributed_tls_max_attempts=3
--distributed_tls_read_endpoint=/distributed/read
--distributed_tls_write_endpoint=/distributed/write
--logger_plugin=tls
--logger_tls_endpoint=/log
--logger_tls_period=5
--tls_hostname=serverip:5000
--tls_server_certs=./certificate.crt
--log_result_events=false
--pack_delimiter=/
--utc
--verbose
--tls_dump=true

@foreversunyao Configure the query on the queries page, and then tag the query with a tag that is shared by the nodes you wish to get this information for.
For example, I have a set of queries and packs that I've tagged with baseline. All my nodes are also tagged with baseline, which results in them receiving in their osquery configuration all the queries, packs, etc associated with the baseline tag.
See screenshots below:
On the queries tab:

On the packs tab:

On the individual node's page:

@mwielgoszewski
It works , thanks very much for your patient. It's a very nice framework.
if convenient we can make a friend, my name is Samuel, my facebook is foreversunyao,I am a SRE and work in Singapore now , we plan to use your platform for our system security monitor.

closed.


comparison doesn't make any sense

.Net SYNC is not concurrent at all. This would be fast in all languages.
.Net ASYNC does NOT have a million actors at any time. Other implementations probably as well because the task is so easy that they complete faster than they can be spawned.
Sorry, but you should state clearly what you want to compare and then create a fitting benchmark.
Currently I fear the only thing you've shown that underlying implementations are very different. But not if they are better or worse at anything bit this particular program...

While I agree with the overall assessment, the main reason is that comparing futures with coroutines and actors concerning their instantiation performance does not make much sense because a Future is ephemeral and performs only a single-shot computation while an Actor is remotely addressable, has proper lifecycle management and supervision etc. (a coroutine is somewhat in the middle between these two extremes). This means that one is optimized for creation performance while the other is optimized for longevity.
Nevertheless I have submitted #13 to make the Akka sample more idiomatic (i.e. everything is a message). This reduces the runtime down by a factor of 5 (roughly) on my machine.

Indeed, we're mixing Actors, Coroutines and Futures/Promises here, unfortunately :(

and CSP style. (goroutines) which somewhere sits between coroutines and actors.

@schmitch does it really sit inbetween? I would say it's "just" syntax sugar from the compiler and is exactly what Future/Promises are (which are just a abstract syntax over threads)
actors are something different


support for programmatic ingest from url

Coverage decreased (-46.09%) to 0.0% when pulling def7dde6a0076b3fc72d831d7b07a77e047fb4c8 on rfecher:GEOWAVE-1442 into 175b5e113d8d60f2404bc355ba20beb76d0da627 on locationtech:master.


Make the AnimTrack class and its dependencies public

Instances of the pc.AnimTrack class are made available available to users as part of the container asset resource. It should be made public to give users proper documentation of it and it's dependencies.
Fixes #3684
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

This PR has been around for a while so some conflicts have arisen in the meantime. Essentially you need to do two things:

Adopt the new, succinct JSDoc block format for getters/setters.
Remove the newly adopted @ignore tag from the classes you would like to expose to the API reference manual (which has replaced the use of the @private tag in #3918).


build: force rebuild of libdispatch always

This pays a small penalty in build times by invoking an extra call to
ninja.  However, unless there is a change in libdispatch, no actions
will be taken other than ensuring that it is up-to-date.  Should ensure
that the buildbots switching between builds DTRT.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test


Classifications of disease progression

what does disease prognosis look like, especially what downward spiral looks like, potentially we can triage (using a predictive AI system) and send "slow decline" to other treatment. Old people seem to decline fast – 24-72 hours. Also, identifying stages may help inflection points where it pays to apply more resources

Is ACE2 expression a predictor for disease progression?


Unable to add event to all channels

Hello,
I am unable to add events to all existing channels in my script. I have written the following:
**ch = 'all';
bmech_addevent(fld,ch,'max','max')
bmech_addevent(fld,ch,'min','min')**
Two errors come up:
_Cell contents reference from a non-cell array object.
Error in addevent_data (line 51)
ch = ch{i};
Error in bmech_addevent (line 62)
data = addevent_data(data,ch,ename,type);_
Could you please look into this issue?
Thanks,
Aiden

bmech_addevent and addevent_data were updated to address this bug report