non-standard concept identifiers used for types

In the following tables a non-standard type concept is being used for the specified columns.  Please use 32817
condition_occurrence - condition_type_concept_id
drug_exposure - drug_type_concept_id
procedure_occurrence - procedure_type_concept_id
device_exposure - device_type_concept_id
measurement - measurement_type_concept_id
using non-standard concept id.  please use 32817
https://athena.ohdsi.org/search-terms/terms/32817

Fixed


initial values and custom directives on inputs

Hello,
I ran into another edge case while trying to use imask directives on a standard DOM input.
Element value was changed outside of mask. Syncronize mask using `mask.updateValue()` to work properly.

Without the initialValues, it would mask the value nicely on page load.
Would it be possible and make sense to have felte look for other directives on inputs/components and trigger them possibly when the initialValues are set?
If that's not the case, I suppose one can filter the initialValues object when handing it off to felte.

Felte is largely vanilla JS with just light wrappers to make it work for a specific framework (which is what allowed it to also be used in Solid). So triggering actions or other directives is not that trivial to do. I'm guessing the reason why imask is not working here with initialValues is because programmatically changing an input's value does not trigger any event (this is how Felte sets initialValues when set on the config).
I don't have experience with imask but judging from that error message I'm guessing it should be possible to force it to run after initialisation of the form within an onMount call?

some components wouldn't expose re-update mechanisms, that's why I asked. I ended up filtering the inititalValues to exclude the special cases, and then bind separately. Thank you for the explanation.


Add Score, Increase Game Speed

Adding Score to the game
Increase Gamespeed after n-Lines are deleted

Feature added.


When video is invalid, VideoEvent comes with 'VS_PAUSE' state instead of 'VS_ERROR'

This fix is needed so that the content team can fallback to a different video or settings when it's detected that the original video had problems playing for the user.

will reach production with E@ v0.40


[CWS] custom events cleanup

What does this PR do?
Cleanup PR which:

merge all custom event types into a single one (no need to have both event types AND rule ID for each of them)
adapt/simplify test helpers to filter custom events with wanted rule ID
remove dead custom events code (CustomForkBombEventType / CustomTruncatedParentsEventType)
remove both CustomLostReadEventType (wich was not used at all) and CustomLostWriteEventType (disabled by default and not used, we use metrics instead)

Motivation
Do some cleanup
Describe how to test/QA your changes
Launch tests
Possible Drawbacks / Trade-offs
Additional Notes

/merge

/gitlab trigger-ci

/merge


Data Structures: Implement Heap Sort with a Min Heap (Passing)

Hey there,
Here is what the challenge description says regarding the sort method.
"The sort method should return an array of all the elements in the min heap sorted from smallest to largest."
I am literally returning an empty array and the challenge keeps on passing.
Data Structures: Implement Heap Sort with a Min Heap

Browser Name: Chrome
Browser Version: Version 86.0.4240.111 (Official Build) (64-bit)
Operating System: Windows 10 Home

Has this worked everytime, even after you cleared your browser cache?

@farhankk360 Can you please post your full code instead of a screen shot?
Thank you.

@RandellDawson Sure,
function isSorted(a){
  for(let i = 0; i < a.length - 1; i++)
    if(a[i] > a[i + 1])
      return false;
  return true;
}

// Generate a randomly filled array
var array = new Array();
(function createArray(size = 5) {
  array.push(+(Math.random() * 100).toFixed(0));
  return size > 1 ? createArray(size - 1) : undefined;
})(25);

var MinHeap = function() {
  // Only change code below this line
  this.insert = () => {}

  this.remove = () => {}

  this.sort = () => []
  // Only change code above this line
};

@JoshuaPelealu Yes couple of time but still the issues is there. I believe it's the isSorted method if you give it an empty array it returns true.


User-defined block functions should be allowed to get explicit types

Since CodeWorld enables scoped type variables, it's possible to implement this in the pattern.  For example, https://code.world/#PF-6FWEOWxwYUV5oUgJ3Fkg

To be clear, this should NOT be the default.  But it could be an option, in addition to inferred parameters.


refactor: get_normalized_directions

The get_normalized_directions and shift_directions_for_tcnn performed the same function but were placed in two separate files. This PR places them in base_field and refactors other usages.

LGTM. Just need to fix the test then it can be merged

Fixed in 6ff033d48d084a33ef6d6ed946e3a15a4360d24c


Fix Dockerfile for swarm under go1.7.1.

Fix #2531.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

Test failure on engine-master is expected. Pending fix in #2492.
cc @allencloud @nishanttotla.

LGTM


[FEA] Support reading Avro

This is the epic issue to trace the tasks for reading Avro.
cuDF has some basic support on reading Avro.
High priority

[x] Avro: cuDF JNI support
Primitve types

[x] https://github.com/NVIDIA/spark-rapids/issues/4935 (Numeric, Boolean, String)
[x] Check how Spark/cuDF handle null values for primitive types.

Logical types

[ ] https://github.com/NVIDIA/spark-rapids/issues/4982 (cudf_dependency)
[ ] Support decimal type. NOTE: the decimal in avro can be backed by fixed or bytes (cudf_dependency)

Complex types

[ ] Support Array type (cudf_dependency)
[ ] Support Record type (Spark maps to StructType) (cudf_dependency)
[ ] Support Map type (cudf_dependency)

[ ] Support avro read options
[ ] Support avro read configurations
[ ] https://github.com/NVIDIA/spark-rapids/issues/5018  (cudf_dependency)
[ ] Add integration tests for reading avro files with corrupt files

Low priority

[x] https://github.com/NVIDIA/spark-rapids/issues/4983
[x] #5148
[x] #5149
[x] https://github.com/NVIDIA/spark-rapids/issues/5312
[x] https://github.com/NVIDIA/spark-rapids/issues/5347
[ ] Support pushed down filters for Avro
[ ] More tests to cover all the compression types.
[ ] Support the from_avro  SQL function running on GPU

Needs Triage

Primitve types

[x] Check how Spark/cuDF handle columns of null type.
[ ] Support null type (Can support on plugin side in post-handling)
[ ] Support the Bytes type (cuDF native supports it by mapping to string, differing from the Spark mapping to BinaryType) (cudf_dependency)

Logical types

[ ] Check the support of Local timestamp (millisecond and microsecond precision and ) in Spark (If yes, add a new task for adding the support)
[ ] Check the support of Time (millisecond and microsecond precision and ) in Spark (If yes, add a new task for adding the support)
[ ] Check the support of UUID and Duration in Spark (If yes, add a new task for adding the support)

Complex types

[ ] Support Enum type, (cuDF native supports it by mapping to String or INT32, but Spark maps only to StringType.) (cudf_dependency)
[ ] Support Fixed type (cudf_dependency)
[ ] Support Union type (cudf_dependency)

Hi @firestarman @GaryShen2008, shall we deem #5691 as a sub-issue of this epic?

Yes please.


Add prefix case for incrementFileName

Fixes https://github.com/Microsoft/vscode/issues/47014
CC: @isidorn

While this is naive way to add new case there are couple of problems with incrementFileName in general:

It's always using . as separator of version prefix/suffix, it would be good to support also other separators (like -, app-1.js should be incremented to app-2.js)
Since it's just parsing version string to int we lose leading zeros (right now app.0001.js is incremented to app.2.js which is probably wrong behavior.)

@Krzysztof-Cieslak thanks a lot for your PR.
I would be happy merging this in if we add some unit tests - they would increase my confidence and would use as a documentation how this beahves. Could you export function incrementFileNameand add a fileActions.test.ts and test out that method.

@Krzysztof-Cieslak yeah that would be great! Adding tests for those cases would also be awesome! Thanks

This is ready for review.

@Krzysztof-Cieslak thanks a lot! I love the tests, they give me confidence looking at all these regexes.
Merging in, thanks again!


{{.Name}} not working

Hello
It seems to me that {{.Name}} is always empty, vboxmanage commands do not work, here is the error message:
==> vagrant: Executing custom VBoxManage commands...
    vagrant: Executing: modifyvm <no value> --memory 2048
==> vagrant: Error executing command: VBoxManage error: VBoxManage: error: Could not find a registered machine named '<no value>'

Here is the template:
{
  "builders": [
    {
      "type": "vagrant",
      "box_name": "ubuntu/trusty64",
      "box_provider": "virtualbox",
      "box_file": ".ovf",
      "builder": {
        "type": "virtualbox-ovf",
        "headless": true,
        "guest_additions_mode": "disable",
        "vboxmanage": [
          [ "modifyvm", "{{.Name}}", "--memory", "2048" ],
          [ "modifyvm", "{{.Name}}", "--cpus", "2" ]
        ],
        "ssh_username": "vagrant",
        "ssh_password": "vagrant",
        "ssh_private_key_file":"vendor/vagrant/keys/vagrant",
        "shutdown_command": "echo 'vagrant' | sudo -S shutdown -P now"
      }
    }
  ]
}

Thanks
Nabil

@kadimi does {{.Name}} work without packer-builder-vagrant plugin?

@themalkolm Yes, see https://github.com/chef/bento/blob/master/ubuntu-16.04-amd64.json usage examples.

I have the same issue with Packer 0.12.2.
Here is the relevant snippet from my packer.json file:
  "builders": [
    {
      "type": "vagrant",
      "box_name": "bento/centos-7.3",
      "box_provider": "virtualbox",
      "box_file": ".ovf",
      "builder": {
        "type": "virtualbox-ovf",
        "guest_additions_mode": "disable",
        "ssh_username": "vagrant",
        "ssh_password": "vagrant",
        "ssh_pty": true,
        "headless": false,
        "vboxmanage": [
          [ "modifyvm", "{{.Name}}", "--cpus", "{{user `vm_cpus`}}" ],
          [ "modifyvm", "{{.Name}}", "--memory", "{{user `vm_mem_mb`}}" ],
          [ "modifyvm", "{{.Name}}", "--cableconnected1", "on" ]
        ],
        "shutdown_command": "echo '/sbin/halt -h -p' > /tmp/shutdown.sh; echo 'vagrant' | sudo -S sh '/tmp/shutdown.sh'"
      }
    }
  ],

Here is the output from packer build packer.json:
vagrant output will be in this color.

    vagrant: (vagrant) Builder source_path ...
    vagrant: (vagrant) Builder source_path: /Users/ernst/.vagrant.d/boxes/bento-VAGRANTSLASH-centos-7.3/2.3.5/virtualbox/box.ovf
    vagrant: (vagrant) Builder type ...
    vagrant: (vagrant) Builder type: virtualbox-ovf
    vagrant: (vagrant) Builder ...
    vagrant: (vagrant) Builder: OK
    vagrant: (vagrant) Builder prepare ...
    vagrant: (vagrant) Builder prepare: OK
    vagrant: (vagrant) Builder run ...
==> vagrant: Downloading or copying OVF/OVA
    vagrant: Downloading or copying: file:///Users/ernst/.vagrant.d/boxes/bento-VAGRANTSLASH-centos-7.3/2.3.5/virtualbox/box.ovf
==> vagrant: Importing VM: /Users/ernst/.vagrant.d/boxes/bento-VAGRANTSLASH-centos-7.3/2.3.5/virtualbox/box.ovf
==> vagrant: Creating forwarded port mapping for communicator (SSH, WinRM, etc) (host port 2791)
==> vagrant: Executing custom VBoxManage commands...
    vagrant: Executing: modifyvm <no value> --cpus 2
==> vagrant: Error executing command: VBoxManage error: VBoxManage: error: Could not find a registered machine named '<no value>'
==> vagrant: VBoxManage: error: Details: code VBOX_E_OBJECT_NOT_FOUND (0x80bb0001), component VirtualBoxWrap, interface IVirtualBox, callee nsISupports
==> vagrant: VBoxManage: error: Context: "FindMachine(Bstr(a->argv[0]).raw(), machine.asOutParam())" at line 500 of file VBoxManageModifyVM.cpp
==> vagrant: Unregistering and deleting imported VM...

Before, when I was using the virtualbox-ovf builder directly, I had no such issue.

OK, got what is the problem. We render builder config without taking in account how builder itself wants to render it. In our case we try to interpolate {{.Name}} while it must be skipped and left as is i.e.
$ cat vendor/github.com/mitchellh/packer/builder/virtualbox/ovf/config.go
...
func NewConfig(raws ...interface{}) (*Config, []string, error) {
	c := new(Config)
	err := config.Decode(c, &config.DecodeOpts{
		Interpolate:        true,
		InterpolateContext: &c.ctx,
		InterpolateFilter: &interpolate.RenderFilter{
			Exclude: []string{
				"boot_command",
				"guest_additions_path",
				"guest_additions_url",
				"vboxmanage",
				"vboxmanage_post",
			},
		},
	}, raws...)
	if err != nil {
		return nil, nil, err
	}
...

This means we need to either defer rendering for the wrapped builder or render according to its rules.

OK, this is fixed now.

I updated test to contain both cases: https://github.com/themalkolm/packer-builder-vagrant/blob/v2017.05.10-1/test/virtualbox/template.json#L24
New release is made https://github.com/themalkolm/packer-builder-vagrant/releases/tag/v2017.05.10-1

@znerd ping me if you need help with wmware support


Add support for download counts

Closes https://github.com/leoafarias/pub_api_client/issues/57

Thank you @Rexios80


Changing Sidebar Style Changes

Why:
Closes https://github.com/github/docs-internal/issues/19283

What's being changed:
This PR is updating the sidebar so that the current active page sidebar is viewable via bold and the marker.

Check off the following:

[x] I have reviewed my changes in staging (look for the deploy-to-heroku link in your pull request, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (eg. a "before and after video")

✨ Looks great! Thanks, @gracepark 🙂

You're welcome @lucascosti! <3


References between projects in paket.references

Is it possible to add project references by paket.references ?
As posible feature is swap package reference by inculuded sources while debug.
This feature supoprted in new nuget3.

This is not supported yet, but @k_cieslak and @cloudRoutine and others
requested that before. Currently I'm not completely sure if this is a good
idea (we would work against basic VS features).
On Mar 21, 2016 11:09 AM, "Red Beard"<EMAIL_ADDRESS>wrote:

Is it possible to add project references by paket.references ?
As posible feature is swap package reference by inculuded sources while
debug.
This feature supoprted in new nuget3.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1532

And AFAIK it's more a change of project system (and everything above) than
a new feature of nuget.
On Mar 21, 2016 11:11 AM, "Steffen Forkmann"<EMAIL_ADDRESS>wrote:

This is not supported yet, but @k_cieslak and @cloudRoutine and others
requested that before. Currently I'm not completely sure if this is a good
idea (we would work against basic VS features).
On Mar 21, 2016 11:09 AM, "Red Beard"<EMAIL_ADDRESS>wrote:

Is it possible to add project references by paket.references ?
As posible feature is swap package reference by inculuded sources while
debug.
This feature supoprted in new nuget3.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1532

Yes it is not about nuget.
But for paket it must easy feature.
It will good to manage this by paket.
It is simpler then Pr -> AddReference -> PrReferences->(check some lines)
I hate csproj file, and prefer to do not touch this=)

Maybe we can do that in v3, but I would need volunteers...
On Mar 21, 2016 11:22 AM, "Red Beard"<EMAIL_ADDRESS>wrote:

Yes it is not about nuget.
But for paket it must easy feature.
It will good to manage this by paket.
It is simpler then Pr -> AddReference -> PrReferences->(check some lines)
I hate csproj file, and prefer to do not touch this=)
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1532#issuecomment-199213214

Sounds like paket.dependencies starts to become like project.json ;-) hides

In this case it's paket.references.
On Mar 21, 2016 11:23 AM, "Isaac Abraham"<EMAIL_ADDRESS>wrote:

Sounds like paket.dependencies starts to become like project.json ;-)
hides
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1532#issuecomment-199213620

Yes, true. I was thinking of another issue that I saw which was about putting paket.references embedded within the paket.dependencies file. Which I think has some legs in it.

@isaacabraham, no, no, no, no =)
dependencies is perfect, do not make project.json with him =)

Yes, true. I was thinking of another issue that I saw which was about putting paket.references embedded within the paket.dependencies file. Which I think has some legs in it.

Can u give link?

Can't find it - maybe I imagined it. But link to #1497 - this seems similar.

@isaacabraham, 10x.

project myproject path/to/myproject.csproj

it is interesting - manage project list by paket.dependencies?
but i about manage project-to-project references and swap nuget/[other] references by included project.

I was more thinking within dependencies file having a section e.g.
project abc
   mydep
   myotherdep

rather than a paket.references for project abc with those elements in. I'll make a new issue if I can't find the original one.

@isaacabraham, NO!
references.paket for project refecences
dependencies.paket only store deps list with sources and versions
Is it original https://github.com/fsprojects/Paket/issues/590?

@ggrnd0 no, that's not the original.
paket.references is not for project references. It's for project dependencies (and not for transitive dependencies at that). All I am proposing is allowing us to specify that in the paket.dependencies if we want as opposed to having several files with them in.
Primarily this would make creating new projects in a solution and adding them to paket slightly easier.

paket.dependencies is global. How would that help to specify inter-project dependencies?

Duplicate with #1497


syntax-highlighting not found when installing as gem

I'm trying to install this theme as a gem, and i'm getting the following error message when I run bundle exec jekyll serve:
Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/main.scss':
File to import not found or unreadable: syntax-highlighting. on line 9
I noticed that you had some recent commits related to syntax highlighting, so I'm not sure what I'm doing wrong.
I tried running the repository both locally and here: https://dmcwo.github.io/temp-j4/ https://github.com/dmcwo/temp-j4
Let me know your thoughts and advice!

This should work now, if you run bundle update and try jekyll serve` again. Let me know if you need any help! Thanks again for raising this! 😄

Works! Thanks!


Add BSD support

Would it be appropriate to add support for detecting the different families of BSD?

FreeBSD
NetBSD
OpenBSD
DragonFlyBSD
TrueOS

I would love to see this. Do you have access to a VirtualBox or other virtualization software @woodsb02 ?

Do you mean vbox on BSD as host?

I can spin up a FreeBSD guest within my Vultr account, and give you access, if that would help with testing?
I also use the FreeBSD bhyve type 2 hypervisor at home for my own testing, but can't give external access to this.

Well I could do the virtualization myself. :) I was just wondering if you were up to making the contribution yourself.

It would be awesome to support these. I would need a contribute for that though as I don't currently have the time to invest.
Anyone? :)

I can work on this, I'll fire up a Virtualbox instance and see what I can do!

I'll try to make a dent in this!

So I've installed and configured an OpenBSD installation and I've found that there is no /etc/*-release files and no lsb-release installed by default. It seems that the only way to get version info is via uname -rs which outputs OpenBSD 6.2. So perhaps we should try that if lsb-release isn't installed?

lsb stands for LINUX standard base.
On BSD use uname. If you really need to parse userland version, there is an option to grep for version in headers.
$ awk '/^#define/ && /__NetBSD_Version__/ {print $6}' /usr/include/sys/param.h
8.99.7

Thanks for the tip, I was doubtful LSB would be available but was surprised that there wasn't any release info in /etc. I think I'll go down the uname -rs route.

Note that on FreeBSD you can use the command "freebsd-version -k" to give you the same details of the kernel version. The helpful part is that if you want the userland details, you use "freebsd-version -u" (it can be different to the kernel if patches have come out for a release that only affect either kernel or userland).

Note that distro 1.2.0 has now been added to the FreeBSD ports collection, with Seth's patch in #207. If the BSD implementation changes, we can easily update the patch in the ports tree, but at least this has it working on FreeBSD :)
https://www.freshports.org/sysutils/py-distro

Why would be there /etc/*release, it's a Linux specific file. BSD are one - from the kernel to userland, and in typical scenarios uname(1) retrieves the version for everything at once.

I was reporting information back. I've never investigated or booted into *BSD before, so I'm not sure how I would know that before investigating and learning myself.


Revert "Remove SQL until it bumps log4j 2.17.1. (#1419)"

This reverts commit a7a7e96f364aad433dbc6c9e3662d2562468fc62.
Description
Revert the removal of SQL after https://github.com/opensearch-project/sql/pull/354.
Check List

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

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

Codecov Report

Merging #1420 (0682a6f) into main (a7a7e96) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1420   +/-   ##
=========================================
  Coverage     94.13%   94.13%
  Complexity       11       11
=========================================
  Files           139      139
  Lines          3086     3086
  Branches          8        8
=========================================
  Hits           2905     2905
  Misses          173      173
  Partials          8        8

Continue to review full report at Codecov.

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


Unable to pull containerPort from yaml

Type of question
Are you asking about community best practices, how to implement a specific feature, or about general context and help around the operator-sdk?
Help with an Operator feature
Question
What did you do?
With the controller, I am creating a StatefulSet and pulling the configuration from the cr.yaml file I have. In the .yaml, I have a section
ports:
   - name: mysql
      containerPort: 3306

What did you expect to see?
I expected to see the port listed when I run kubectl describe sts mysql
What did you see instead? Under which circumstances?
Port section was empty
Environment

operator-sdk version:
v0.10.0-31-ga827815d
insert release or Git SHA here

Kubernetes version information:

1.14.6

Kubernetes cluster kind:
Docker-for-Desktop

Additional context
Add any other context about the question here.

Hi @danyollee,
I understand that you would like to see the MySQL container in a deployment using the info added in the CR.
Note that the CRD is the definition of a new /customized object which will represent your solution in the cluster, for example, the MySQL. The CR is where you are able to set the values for the CRD specs/attributes.
So, for your MySQL container in the cluster use info from the CR you need define it programmatically in the code of your operator when you are developing the deployment/container on it. See here an example where the deployment is using the value defined in the CR as a port of the DB container.
I hope that it is the info which you are looking for.
Please, let us know if it answers your question or not.

Thanks for your reply!
So I've added all of the values from the types.go into the CRD and specified the types. Unfortunately, it still looks like the controller is unable to pull the ports from the CR.
What strikes me as odd is that the controller is able to pull other objects/arrays from the CR just fine. I just don't know why the container port is having issues. In my CR, I have something along the lines of
  volumemounts:
  - name: storage
    mountPath: /var/lib/mysql
    subPath: mysql
  - name: mysql-conf
    mountPath: /etc/mysql/conf.d
  volumes:
  - name: mysql-conf
    configMap:
      name: mysql

  # Special for mysql
  containerPorts:
  - containerPort: 3306
    name: mysql
    protocol: TCP

My controller is able to pull the volumes and the volumemounts just fine. In fact, I set an if statement where the controller will pull the volumes, volumemounts, and ports. If they exist, then it'll log that the specifications exists. I get existing logs for the volumes and volumemounts, just not the container ports :(
Any ideas?

Another piece to note is that in the same CR yaml file, I have another spec source which looks like
  # Service specs
  serviceSpec:
    tier: backend
    ports:
    - name: mysql
      port: 3306
    clusterIP: None
    selector: mysql

These values are used to create a Service. The controller is able to pull the ports from the serviceSpec and successfully use the ports in the Service creation.

Did you run the command "operator-sdk generate k8s" to update the deep copy files? Is it finishing with success? If yes, how are they defined in the *_type.go?

The command operator-sdk generate k8s as well as operator-sdk build ... and docker push ... are all completing successfully. The *_types.go is defined as
type SustainSpec struct {
	// INSERT ADDITIONAL SPEC FIELDS - desired state of cluster
	// Important: Run "operator-sdk generate k8s" to regenerate code after modifying this file
	// Add custom validation using kubebuilder tags: https://book-v1.book.kubebuilder.io/beyond_basics/generating_crd.html
	Namespace       string                      `json:"namespace,omitempty"`
	Replicas        *int32                      `json:"replicas,omitempty"`
	Name            string                      `json:"name,omitempty"`
	Image           string                      `json:"image,omitempty"`
	Secret          string                      `json:"secret,omitempty"`
	VolumeMounts    []corev1.VolumeMount        `json:"volumemounts,omitempty"`
	Volumes         []corev1.Volume             `json:"volumes,omitempty"`
	EnvFrom         []corev1.EnvFromSource      `json:"envFrom,omitempty"`
	ImagePullPolicy corev1.PullPolicy           `json:"imagePullPolicy,omitempty"`
	**Ports           []corev1.ContainerPort      `json:"ports,omitempty`**
	Resources       corev1.ResourceRequirements `json:"resources,omitempty"`
	LivenessProbe   *corev1.Probe               `json:"livenessProbe,omitempty"`
	ReadinessProbe  *corev1.Probe               `json:"readinessProbe,omitempty"`
	Env             []corev1.EnvVar             `json:"env,omitempty"`

	ServiceSpec ServiceSpec `json:"serviceSpec,omitempty"`
}

Bolded is where the ports should be pulling from. The controller to pull the ports looks like
				},
				Spec: corev1.PodSpec{
					Containers: []corev1.Container{
						corev1.Container{
							Name:            cr.Spec.Name,
							Image:           cr.Spec.Image,
							ImagePullPolicy: cr.Spec.ImagePullPolicy,
							EnvFrom:         cr.Spec.EnvFrom,
							Env:             cr.Spec.Env,
							**Ports:           cr.Spec.Ports,**

I also have a debugging line in the controller just before the creation of the resource where it checks to see the controller has pulled pieces from the CR file.
	val := cr.Spec.EnvFrom
	if val != nil {
		log.Info("EnvFrom exists")
		log.Info(strconv.Itoa(len(val)))
	}
	val2 := cr.Spec.Ports
	if val2 == nil {
		log.Info("Ports do not exist")
	} else {
		log.Info("ports exist")
		log.Info(strconv.Itoa(len(val2)))
	}

In the logs for the operator, I see the following logs associated from these lines.
{"level":"info","ts":1568150276.05756,"logger":"controller_sustain","msg":"EnvFrom exists"}
{"level":"info","ts":1568150276.0575795,"logger":"controller_sustain","msg":"1"}
{"level":"info","ts":1568150276.0575871,"logger":"controller_sustain","msg":"Ports do not exist"}

@danyollee In your _types.go your JSON tag is ports, but in your YAML, you're calling it containerPorts. I think you could fix it by changing your CR from containerPorts to ports.

@joelanford Ah my apologies. I had copied a previous version. After changing the CR yaml to match the *_types.go, I am still not able to pull the ports :-(
  # Special for mysql
  ports:
  - containerPort: 3306
    name: mysql
    protocol: TCP

Are you missing a quotation mark at the end of the JSON tag?
Looks like you have:
`json:"ports,omitempty`
And you may need:
`json:"ports,omitempty"`

Looks like the syntax was it 😫 I'm so embarrassed lol

I am glad that it is solved.

Thanks for the help!


Typo in README.md - Ng2Module

Any Ng2Module mentioned in README.md should be Ng2LoggerModule.

Thx for the contribution. Sorry for the delay, notifications for this repo was disabled 😢


Can't play the video when i click on the video ?!!

So, basically I've done everything to set up the videoplayerview. Loaded the view, with the URL, and when I click on the video. It's doesn't play. However, it start playing when i "access" the video itself. What I am saying here is that when I click on the name of the video, It is taking me to the "real" youtube player if we can say that. And as soon as I press play, It's start to play in full-screen, like I want. So the problem here is I have to click on the name of video to have to play the video ?
Regards,
Hary

Never mind turns out to be a URL mistake ! VeVo video are not allowed I guess or forbidden this way.


gnomeExtensions: auto-update

Motivation for this change
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Thank you for doing these. Please have a look at the collision and add it to extensionRenames.nix. Note that we already have a manually packaged version of<EMAIL_ADDRESS>and that we are using a custom name due to backwards compatibility.

Thank you for doing these. Please have a look at the collision and add it to extensionRenames.nix. Note that we already have a manually packaged version of<EMAIL_ADDRESS>and that we are using a custom name due to backwards compatibility. (Also, feel free to ping me for review on anything extensions-related)

I've seen the new entry but left it because of https://github.com/NixOS/nixpkgs/blob/933dab8b5f6c44ff8135ab875ca10b9de704876d/pkgs/desktops/gnome/extensions/extensionRenames.nix#L74 since this causes no collision. Should I rename it to disable-unredirect-fullscreen-windows-2?

You are right, technically there is no collision. However I would find it weird to know one as disable-unredirect and the other as disable-unredirect-fullscreen-windows. I'd suggest to stick with our current custom name and call it disable-unredirect-2. What do you think? Alternatively, we can also add a mechanism so that our extension is known under both names.

disable-unredirect-fullscreen-windows

IMO it makes more sense to call it the same as it is found on https://extensions.gnome.org. Since that is the name a user is used to and will probably search for. What would make sense is to create an alias for the other extension and that way we can call the new one disable-unredirect-fullscreen-windows-2.

Alias sounds good. I have a small concern about potential collisions on the aliased name: they won't be caught by our collisions.json. On the other hand, this is kind of true for a lot of attributes in the gnomeExtensions set so it wouldn't make things worse.

@piegamesde let me make an other update since some time has passed.


Previously 0-values were not plotted into chart, now they do.

For ChartJS, data with 0 values were not being plotted into the chart, so there were gaps in places where it shouldn't. That leads to confusion, as the 0 values were confused with null values (a.k.a no data). An use case is for example a chart that measures the heartbeats of an application, 0 values NEED to be plotted as that data exists and is valuable.

Hey, thanks @cizambra 👍  Looks like that was introduced in the latest release with this commit. https://github.com/ankane/chartkick.js/commit/80719366847a5fc6fc3192ec1805f7ce710c2114
I believe the code should read (typeof rows[i][j] === "undefined") ? null : rows[i][j] and don't think we want the comment change since it'll be about a fixed bug.

Since this is a pretty big issue, going to push out a fix now.

Awesome, thank you for consider the change. I added the comment as it was my first pull request and I didn't know that the code can be reviewed later haha.

This was a really big issue, so big thanks for not only being the first to report it but finding exactly what was going wrong and submitting a fix (this is more than 99% of people do). Not bad for a first PR :)

Thanks, if I can assist you with something else, just let me know ;).


mapStateToProps does not map to props using mount

I have this container
import { connect } from 'react-redux'
import { createType, getTypes } from '../modules/type'

import Type from '../components/Type'

const mapDispatchToProps = {
  createType,
  getTypes
}

const mapStateToProps = state => ({
  types: state.type.types
})

export default connect(mapStateToProps, mapDispatchToProps)(Type)

and I would like to test it using enzyme. To do it, I'm using this test
import React from 'react'
import { Provider } from 'react-redux'
import { mount } from 'enzyme'

import TypeContainer from 'routes/Type/containers/TypeContainer'

import createStore from 'store/createStore'
import { injectReducer } from 'store/reducers'

import reducer from 'routes/Type/modules/type'

const mockStoreInitialized = createStore();

describe.only('(Container) TypeContainer', () => {
  it('should get container', () => {
    injectReducer(mockStoreInitialized, { key: 'type', reducer })

    const wrapper = mount(
      <Provider store={mockStoreInitialized}>
        <TypeContainer />
      </Provider>
    )

    expect(wrapper.find(TypeContainer).props()).to.equal([])
  })
})

(I'm using react-redux-starter-kit as a boilerplate project)
The test is failing (at the assertion level) because wrapper.find(TypeContainer).props() is undefined. I can not find find why.
If I check the coverage report, the mapStateToProps function is called, an if I add a console.log inside, I see the right value (eg []).
I check the issue #396, but using wrapper.find('TypeContainer').props() causes this error

Method “props” is only meant to be run on a single node. 0 found instead.

Did I missed something or is it a bug ?

I was doing the assertion on the wrong component. If I switch to
import Type from 'routes/Type/components/Type'
// ...
expect(wrapper.find(Type).prop('types')).to.deep.equal([])

it is working


Draggable directive never emit onDragStart event

Describe the bug
onDragStart event is never emitted in Draggable directive.
To Reproduce
Steps to reproduce the behavior:

Try to hook an event onDragStart
Run the application
onDragStart is never triggered

Expected behavior
onDragStart should be triggered when configured
Screenshots

Example Code
Please provide some example code to help us understand the issue.
Desktop (please complete the following information):

OS: all
Browser all
Version 6.1.0

Smartphone (please complete the following information):

Device: al
OS: all
Browser all
Version [e.g. 22]

Additional context
Please emit onDragStart event.

Hi @xtianfelix,
thanks for reporting this issue! It seems like a line of code has gone missing somehow. I'm fixing it right now.

Okay, so I've just published a new version :)


Set default value “null” for “key” property in RESET action

This pr makes "key" property in RESET action optional as it is now in docs, by using null as its default value.
For more info please check discussion here https://github.com/react-navigation/react-navigation-core/pull/43

thx

i'll do a release later today

i'll do a release later today

Thanks!


Update benchmarks for Chalk (and other packages)

Chalk has got a new version, which is now also dependency-less, lightweight, and fast. cli-color and nanocolors also had updates. In other words, the benchmarks are now very inaccurate :)
A problem one might encounter is that Chalk is now ESM-only, which makes it impossible to use in the current benchmark setup. I've tried a basic bundle using ESBuild, and it works. Here's how:

Clone chalk/chalk
In source/index.js, replace

#ansi-styles with ./vendor/ansi-styles/index.js
#supports-color with ./vendor/supports-color/index.js

Run npx esbuild source/index.js --outfile=dist/chalk.js --bundle --platform=node
Copy dist/chalk.js to picocolors' repo under benchmarks/chalk.js
Inside ./benchmarks, replace across all files

let chalk = require("chalk") with let chalk = require("./chalk").default

I could run the benchmarks on my machine, but it's slower than the one used for README, so I'm not sure if it makes sense for me to send a PR with this data...

P. S. Since many packages still use Chalk v4, I propose adding another line to the benchmarks:
$ node ./benchmarks/size.js
Data from packagephobia.com
  chalk@4     101 kB
  chalk@5      41 kB
  cli-color   984 kB
  ansi-colors  25 kB
  kleur        21 kB
  colorette    17 kB
  nanocolors   15 kB
+ picocolors    7 kB

I'd definitely separate chalk v4 and v5 in the benchmark stats. As for ESM/CJS, it may possibly be easier to make a separate environment for running ESM-first benchmarks

I implemented @kytta's workaround for ESM-only packages for chalk 5 and yoctocolors in #61. picocolors was dethroned by yoctocolors in the execution speed, but not in module-loading speed yet :-)


Fixed broken link in README.md

The second point in "Want to Contribute", the link to snippet template was broken. as it was linking to /snippet_template.md. I changed it to /snippet-template.md and it works now.

Can you add the hacktoberfest-accepted label for this PR?


Key not present in word2vec model

key fruit not present in word2vec model
key vegetable not present in word2vec model
key meat not present in word2vec model
This is the error I am getting , when I tried to run the example  ?

@VSK02101996
This would suggest that the words are not present in the vectors dictionary you are currently using. I would suggest training a custom model or using one of the default, more elaborate, models provided by gensim. See the model section in the Readme.

Could you send me some reproducible code please?

`import spacy
from spacy import displacy
import concise_concepts
data = {
"fruit": ["apple", "pear", "orange"],
"vegetable": ["broccoli", "spinach", "tomato", "garlic", "onion", "beans"],
"meat": ["beef", "pork", "fish", "lamb", "bacon", "ham", "meatball"],
"dairy": ["milk", "butter", "eggs", "cheese", "cheddar", "yoghurt", "egg"],
"herbs": ["rosemary", "salt", "sage", "basil", "cilantro"],
"carbs": ["bread", "rice", "toast", "tortilla", "noodles", "bagel", "croissant"],
}
text = """
Heat the oil in a large pan and add the Onion, celery and carrots.
Then, cook over a medium–low heat for 10 minutes, or until softened.
Add the courgette, garlic, red peppers and oregano and cook for 2–3 minutes.
Later, add some oranges and chickens. """
model_path = "/content/word2vec.wordvectors"
nlp = spacy.load("en_core_web_md", disable=["ner"])
nlp.add_pipe("concise_concepts", config={"data": data,"model_path":model_path,"ent_score": True,})
doc = nlp(text)
options = {
"colors": {
"fruit": "darkorange",
"vegetable": "limegreen",
"meat": "salmon",
"dairy": "lightblue",
"herbs": "darkgreen",
"carbs": "lightbrown",
},
"ents": ["fruit", "vegetable", "meat", "dairy", "herbs", "carbs"],
}
ents = doc.ents
for ent in ents:
new_label = f"{ent.label_} ({float(ent..ent_score):.0%})"
options["colors"][new_label] = options["colors"].get(ent.label.lower(), None)
options["ents"].append(new_label)
ent.label_ = new_label
doc.ents = ents
displacy.render(doc, style="ent", options=options)`
This is the code I am using

from gensim.models import Word2Vec
dataset = api.load("text8")
model = Word2Vec(dataset)
Now the error is , cannot compute mean with no input .

For word2vec.model  file I got error argument of type 'Word2Vec' is not iterable"

I am trying this in colab , it is giving me error argument of type 'Word2Vec' is not iterable again

https://colab.research.google.com/drive/1G6VurQf4piogj7zylWB2VuHlrtOU0mtu?usp=sharing
Link for colab

Please if possible can you resolve this issue

@VSK02101996 can you send me the entire error stacktrace and a link to the "word2vec.wordvectors" file that you are using.

Also, please check whether you versioning aligns with the most recent version of concise-concepts and its requirements.

Thanks it works now.


Lazy behavior changed

It seems after recent change, lazy will cause on_source hook to be called for the bundle, which doesn't seem to be the case in the past.
I am wondering what is the difference between autoload and lazy for a bundle? Doesn't autoload imply lazy?

What's the problem?
I cannot understand your description.
Please describe it by the example.

@paulhybryant, autoload will affect only scripts located inside autoload/ dir, loading all code from other dirs. So lazy loads much less, if only plugin's author wasn't trying to greatly reduce it's own startup time. Therefore for vim-altr it will not have much difference, when for most others would have.
As for on_source -- see loading order. If I'm not mistaken, currently it's like:

vimrc + stuff inside neobundle#tap(...)
bundle.on_source -- after adding to runtime but before actually sourcing plugin stuff (so functions from autoload/ could be called here.
bundle.on_post_source

I think @Shougo will correct me if I'm mistaken.

bundle.hooks.on_source and bundle.on_post_source are correct names.
Yes, the description is right.

Hm, so help file has mistake? There specified usage of bundle.hooks.on_post_source. See:
NeoBundleLazy 'matchit.zip', { 'autoload' : { 'mappings' : ['%', 'g%'] }}
let bundle = neobundle#get('matchit.zip')
function! bundle.hooks.on_post_source(bundle)
    silent! execute 'doautocmd Filetype' &filetype
endfunction

@Shougo Sorry for the late reply as I was in a trip. The problem is this:
In the past, I have the following in my config
NeoBundle 'foo/bar', { 'autoload' : { 'filetypes' : ['vim'] }, 'lazy' : 1 }
let s:foobar = neobundle#get('bar')
function s:foobar.hooks.on_source(bundle)
let g:foobar_x = 1
endfunction
In the past, if the filetype I am editing is not of type vim, g:foobar_x will not be set.
However, after recent update, this behavior has changed, g:foobar_x will be set after opening vim, regardless of the filetype.

@amerlyq Thanks for the reply but I am not quite following the explanation. I think autoload in neobundle means the plugin will be loaded when some conditions are satisfied (e.g. filetype = x)
From the example in the help:
NeoBundleLazy 'Rip-Rip/clang_complete'
autocmd FileType c,cpp NeoBundleSource clang_complete
This seems to be equivalent to
NeoBundle 'Rip-Rip/clang_complete', { autoload : { 'filetypes' : ['c', 'cpp'] }, 'lazy' : 1 }

In the past, if the filetype I am editing is not of type vim, g:foobar_x will not be set.
However, after recent update, this behavior has changed, g:foobar_x will be set after opening vim, regardless of the filetype.

OK.  If so, it is neobundle problem.
But I cannot reproduce the problem.
Please create minimal .vimrc within 20 lines and upload reproduce ways.

Also, everywhere in user configs used let neobundle#hooks.on_source = 'file'
Never saw bundle.hooks.on_source used by anyone.

It is just syntax sugar of bundle.hooks.  It is not mistaken.


Cookie Settings are not being saved

I noticed on the live server that the settings were not being stored in the cookies after clicking the toggle.
I've added a fix to the issue also in my pr for the material-ui update #44

I think this is fixed in your PR! Thanks!
Once I finish up a few other things, we'll get it deployed to the live site.


ImagePicker.launchCamera not calling callback function on second call from a modal

Description
If ImagePicker.launchCamera is called from a modal dialog on iOS, it works fine the first time and returns to the original page. Then the second time the modal dialog is created and the camera called upon, the camera opens briefly before closing itself without any user interaction. The callback function does not get called at all. No response is set.
Android works totally fine. iOS also works fine if the camera is called from somewhere other than the modal.
The modal is a radio button selection when is closed before the camera is called normally, but is somehow reopened when the camera crashes. After a crash, the app is unresponsive. The scrollview containing the radio selection options allows scrolling up and down, but the selection chosen cannot be changed, and the modal cannot be closed.
Expected behaviour is the library would always call the callback function provided to it and return an error if something goes wrong. Or that the camera would not close itself prematurely.
How to repeat issue and example
calling-file.tsx:
beginUpload() : boolean {
  this.alertInput.showAlert();
  return false;
}

private completeUpload(documentDescription : string) {
  return new Promise((resolve, reject) => {
    ImageService.capture().then((response : <stuff>) => {
      resolve(response.uri);
      ... some logic
      this.setState(...);
    }).catch(not being called);
  }
}

render() {
  return(
    <View><Button-which-calls-beginUpload />
    <AlertInput
      ref={(alertInput : AlertInput) => this.alertInput = alertInput}
      callback={this.completeUpload}/>
    </View>
  );

alert-input.tsx:
public showAlert() {
  if (Platform.OS === 'ios') {
    AlertIOS.prompt(
      'title', 'message', text => this.props.callback(text)
    );
}

imageservice.ts:
export default class ImageService {
    static capture() {
        return new Promise<stuff>((resolve, reject) => {
            const options: any = {
                storageOptions: {
                    skipBackup: true,
                    waitUntilSaved: true,
                    cameraRoll: true,
                    path: '/'
                },
                mediaType: 'photo',
                cameraType: 'back',
                noData: true
            };
            ImagePicker.launchCamera(options, (response: any) => {
                console.log('this line not called second time');
                if (response.didCancel || response.error) { // (this part actually two separate if conditions in our code)
                    console.log('not called');
                    reject({cancelled: true}) || reject(response.error);
                }
                // not being called on second attempt.
            });
            console.log('this is called');
        }
    }
}

Solution
If it fails, image picker should always contain an error and the callback function should be called regardless, instead of just not being called in some cases.
Additional Information

React Native version: 0.57.1
Platform: iOS
Development Operating System: macOS
Dev tools: Xcode 10.1, IPHONEOS_DEPLOYMENT_TARGET = 10.3

same to me

I have the same problem

Umm, this has not seen any resolution. Why would you have a bot that closes issues that haven't been addressed.

Still have not heard a solution...

This is still an issue

Found the solution.....just make sure your modal is closed before calling launchCamera().

@dembeEdward fixed my problem.. thanks, you're my savior

ImagePicker.launchCamera(response => {
console.log('ResponseImage = ', response);
  if (response.didCancel) {
    console.log('User cancelled image picker');
  } else if (response.error) {
    console.log('ImagePicker Error: ', response.error);
  } else if (response.customButton) {
    console.log('User tapped custom button: ', response.customButton);
  } else {
    const source = {uri: response.uri};

    // You can also display the image using data:
    // const source = { uri: 'data:image/jpeg;base64,' + response.data };

    this.setState({
      avatarSource: source,
    });
  }
}

//camera get open but after clicking a pic  no response is coming

Same


Publication and dissemination

Publication and dissemination - Authorship - Affiliation - change the 1. to a black bullet point

thanks! fixed in 2cc06ff


Input pointer drag on empty container causes invalidation/refresh

Easiest way to see and replicate this issue is to run the theme demo.
Go to Tab 3 - everything on this page fits on screen, so there is no vertical scrollbar.
If you drag in any direction in the empty area to the right of the floating windows (in the area where there are no objects to interact with) the result is essentially a refresh of the entire screen.
You can see this on Tab 2 as well, if you drag in the area between the graph/text/keyboard.
The expected behavior would be no areas redrawn, since no objects were interacted with/nothing actually changed on screen.

Hi,
I fixed this bug dev-5.1. Can you test it?

Tried it out with dev-51 and it looks fixed! Closing this issue.

Thank you very much!


Create codeql-analysis.yml

STRY0063010 GHAS: Enable on Public Repo in our GitHub Organization
As part of our evaluation of GitHub Advanced Security (GHAS) let's make sure the feature is enabled on all our public repos so we can see what it does for us in these cases.

See https://github.com/byu-oit/hw-fargate-api/pull/517#issuecomment-1134878605


High CPU usage

I'm running Mac OS Big Sur on a macbook pro 16" and Ferdi is always using 20-24% cpu. Every time I run it the laptop kicks up the fan speed which in macbook is rare (as they love to cook the internals) and if I start doing anything somewhat heavy or multitasking the system starts lagging hard!
edit: For reference here's my I've included what cpu I have.
CPU: 2.3 GHz 8-Core Intel Core i9

same here.. i have to uninstalled unit it is fixed

This is was seen previously with darkmode... looks like it's still a problem, when I enable darkmode my Ferdi CPU usage goes from 12.7% to over 300%.

This is was seen previously with darkmode... looks like it's still a problem, when I enable darkmode my Ferdi CPU usage goes from 12.7% to over 300%.

how many things do you have open that it's using 12.7% of cpu? Asking so I can make a reference to my performance.

Same here on Macbook Air/Linux. Fans are nearly constantly running full speed whenever Ferdi is open (same issue with Franz in the past).
I simply run Ferdi with cpulimit now to keep it somewhat under control.
I actually don't think Ferdi is to blame here. I believe it has to do with either electron or the web services themselves. I'm running eight services on Ferdi (Mattermost, Telegram, Messenger, Whatsapp and 4 x Slack). What about you?

I have very high CPU usage from Ferdi when in dark mode. It is the GPU Task in Ferdi. Windows 10, i7-5600U.

Also having the same issue

Let me share an example how much Ferdi consume CPU:

Still an issue on Version 5.6.0-beta.5 (5.6.0-beta.5.2741) with/without Dark Mode enabled:
This has made Ferdi unusable

Still an issue on 5.6.0 with Dark mode on Ubuntu 20.04

@diegobersanetti - can you please also post the numbers without the darkmode?

Hi,
I was encountering  the same issue and it seems that disabling dark mode in Ferdi and restarting Ferdi does reduce the cpu. I am using Mac BigSur 11.6  and Ferdi 5.6.2

Thanks for getting back @TalZiv . Yes, the darkmode plugin is not in our control. Since it tries to render contrasting colors based on the user's OS choice, the service-recipe's default colors and then the overrides from the darkmode library, it can be very cumbersome for the overall process. Unfortunately, this is not something we can do a lot about.

Ferdi uses 115% of cpu when idle for me, with 8 services

@polson - which version of Ferdi are you on? Could you please try the latest nightly? We have upgraded electron to the latest version, and other dependencies have been upgraded as well.

I also have this issue with 5.6.2 also on MBP i9 8 Core. Installed with homebrew


When account is switched, the decrypted texts are still visible until the web is refreshed

Account switching should refresh all the local Storage and render again the page

i think this has been resolved - @manumonti could you please confirm using PR #1

i think this has been resolved - @manumonti could you please confirm using PR #1

Yes!


No relevent interactions file

Hi! I noticed that my output doesn’t include a relevant interactions file. Does this indicate an issue with the analysis, or could it simply mean that there are no significant interactions in my dataset?
Thank you for your help!

Hi Bstu! Yes I’m using the dge method :)

Hi again,
Therefore this should indicate that no relevant interactions could be found according to the relevancy criteria. I would also suggest to double-check the input formats, you can find the example files here.
Hope this helps!
Best,
Batu


Duplicate test case

There are two "There are no references to Ember.$, $ or jQuery" unit test cases

taking this.


Test cleanup: Remove defunct fail_mutators tag.

This tag is not used anymore, and there are no test configurations which exclude this tag. Remove it.


Update dependencies

GIVEN
The next CORGI release
WHEN
CORGI is deployed
THEN
CORGI builds and the interface works as usual
(This should not impact content generation at all)

Update dependencies on the frontend and backend.
I have been watching dependabot PRs pile up and, at this point, I thought it would be better to try to update all dependencies instead.
I also figured out why rollup was always complaining about broken sourcemaps: we were producing empty sourcemaps when we built the bundle for production. That should not happen anymore: now sourcemaps should only be generated when running in watch mode.

verified as part of m.m deploy to staging and prod - OK


Do not disable Google HTTP Client logging for custom logger in Gradle

Fixes #2356.
One of the ways to have a custom logger is through the system property java.util.logging.config.file. If it's defined, don't disable Google HTTP Client logging.

Travis seems to have completely stalled. Will try closing and reopening.


Update data-box-disk-faq.yml

Azure File Sync usage with Data Box disk is now supported (See: https://dev.azure.com/msft-skilling/Content/_workitems/edit/133048)

@alkohli

Can you review this PR? (after @huypub takes it out of draft mode)
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@alkohli
Could you please help to review this change? You may go to the following request to get more context for this change:
https://dev.azure.com/msft-skilling/Content/_workitems/edit/133048

@alkohli
Could you please help to review this change?

Reviewed the changes. As per the IcM 407942072, this behavior was changed the current fix addresses the changed behavior. Please merge.
#sign-off


[5.4] Make it possible to publish package routes

Adding a way to publish package routes.
This can be convenient where a package creates routes for your application, but you want to modify them a bit.
This is a small change which does not break anything, only adds possibilities.
The routes that the package will publish will be stored in routes/vendor/{packagename}.php
See https://github.com/laravel/docs/pull/3200 for docs changes

Breaking change since it changes method signature.

Will it be accepted for 5.5 then, @taylorotwell ?

Just an FYI, instead of flat out closing the PR for being made on the wrong base, you can click Edit next to the title and switch the base to use...


one hot encoder

scikit-learn one-hot-encoder impl
Scikit learn had API change that removed categorical_features, complicating things a bit

Codecov Report

Merging #193 into master will decrease coverage by 0.16%.
The diff coverage is 89.09%.

@@            Coverage Diff             @@
##           master     #193      +/-   ##
==========================================
- Coverage   93.72%   93.55%   -0.17%
==========================================
  Files          28       29       +1
  Lines        1609     1661      +52
  Branches      284      294      +10
==========================================
+ Hits         1508     1554      +46
- Misses         58       61       +3
- Partials       43       46       +3

Flag
Coverage Δ

#unittests
93.55% <89.09%> (-0.17%)
:arrow_down:

Impacted Files
Coverage Δ

...mingbird/ml/operator_converters/one_hot_encoder.py
88.67% <88.67%> (ø)

hummingbird/ml/operator_converters/__init__.py
100.00% <100.00%> (ø)

hummingbird/ml/supported.py
88.70% <100.00%> (ø)

...rd/ml/operator_converters/_tree_implementations.py
98.01% <0.00%> (-0.02%)
:arrow_down:

Continue to review full report at Codecov.

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

Ok I added few comments. Will merge once all checks pass

Closes #136.


[FLINK-18883][python] Support reduce() operation for Python KeyedStream.

What is the purpose of the change
This pull request adds reduce operation and reduce functions for Python DataStream API.
Brief change log

Adds reduce operation on Python KeyedStream.
Adds Python ReduceFunction.
Add tests in test_data_stream.py

Verifying this change
This change added tests and can be verified as follows:

Added integration tests in test_data_stream.py

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (PythonDocs)

Hi @hequn8128, thanks for your PR, the code looks pretty good to me. I think the flink-python/dev/glibc_version_fix.h should not be included in the PR since it has no change.

@shuiqiangchen I have removed the file and updated the PR. Thanks.


Native sort

Native key-only sort for libnd4j:

Parallel quicksort for CPU backend (both simple sort, and sort along dimension)
Odd-even sort for CUDA for sort along dimension
Radix sort for CUDA large float & double arrays (not a views)
Bitonic 2^N + arbitrary sort for CUDA fp16, float & double arrays. And for views.

The back40computing project as long since been superseded by Thrust and CUB:
https://code.google.com/archive/p/back40computing/
Unless there is a really good reason to use old, slow, and buggy code, we should probably use Thrust and/or CUB...

It's just one of fallback options for very large arrays. If you have better portable impls in mind - please let me know.

p.s. buggy? It's just sort. It's either works or not. As i see - tests pass :/

Well, IMO might as well use libraries that are maintained, optimized to new
hardware, new discoveries in algorithms, and yes it has already been
enhanced a lot since 2012. They just refactored everything and changed the
name, that's all.

Well, main problem for external radix implementation, is simple fact: it's pita to maintain. So tbh i'm thinking to get rid of it, OR write own implementation for our needs. Still thinking about the way here.
I'll have another take on sort soon, going to add symmetric sort of multiple arrays, as in 1 key array - N value arrays. Will see what we're going to have there after that.


Multiple x-forwarded-for IPs

I'm trying to limit access to my express server and I have multiple values for the x-forwarded-for header. I want to allow access to the server if any of those values match the IPs i've designated. Is there a way to do this?

@bialesdaniel you should be able to add any headers you want to use in the allowedHeaders option.

@ryanbillingsley Sorry I don't think I explained my problem correctly. I'm setting the allowedHeaders to x-forwarded-for but my x-forwarded-for has multiple values and it only checks the first one.
ex.: x-forwarded-for = "ipaddress1","ipaddress2","ipadress3"

@bialesdaniel can you post you configuration for ipfilter as well as a request that you think should work, I am still not understanding what you are trying to accomplish, so maybe that will make it clearer.

Hi guys, @bialesdaniel correct me if I am wrong but you mean something like this use case?
https://lostechies.com/derickbailey/2013/12/04/getting-the-real-client-ip-address-on-a-heroku-hosted-nodejs-app/
As stated in the post, Heroku uses some internal routing system to forward the original request. In this case, Heroku puts the client real IP as the last value of the array of Ips in the header 'x-forwarded-for'
In the lib you have the following code:
if (headerIp) {
      var splitHeaderIp = headerIp.split(',');
      ipAddress = splitHeaderIp[0];
    }

In this case you are only taking into account the first IP of the array, what if in the case of Heroku, the header contains multiple Ips?

Ah ok, now I understand the problem. Thanks for the clarification @javier-tarazaga. Currently, there is no way to do that, so if someone wants to take a stab it that would be great, otherwise it might be a while before I can get to it.

@bialesdaniel I added a way to write your own IP detection function in the latest version.  You should be able to solve this problem using it.


Update faq.md

@anikaz : Thanks for your contribution! The author, @Juliako, has been notified to review your proposed change.

@anikaz - Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you review staging and fix all validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
We'll close this PR. Thank you.


Update subscription-vending-content.md - Increase readability

Increase readability

@lukemurraynz
Thanks for raising these articles to our attention. Unfortunately, copy edits need to be done internally to ensure that articles follow Microsoft style and formatting guidelines. We'll close this PR and submit the file for an internal copy edit.
#please-close


Azure Repo support in APIM

Is it possible to configure Azure Repos? This provides more functionalities including branching policy, pull request and code review.

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

ID: 7d0120b7-a5fe-fead-e88b-cf1bab80ba73
Version Independent ID: 7371e474-727d-5bd1-600d-a502e9b17581
Content: Configure your API Management service using Git - Azure
Content Source: articles/api-management/api-management-configuration-repository-git.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

Thank you for your feedback! We will review and update as appropriate.

@PingPongSet As long as the underlying version control system is based on Git, I believe that this feature will work regardless of whether it's Github.com or Azure Repos.

Since we haven't heard back, we will now proceed to close this thread for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

@mike-urnun-msft

@PingPongSet As long as the underlying version control system is based on Git, I believe that this feature will work regardless of whether it's Github.com or Azure Repos.

But how to change it? On the Azure Portal, I cannot change it. Please see the link below, which is from the doc quoted at OP.
https://docs.microsoft.com/en-us/azure/api-management/media/api-management-configuration-repository-git/api-management-enable-git.png


1.0.0 beta7

Hi @sonuomi, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

Not sure what the intent is here. Beta7 content is old and we are currently maintaining docs for RC1. Regardless, the DNX content will be going away soon as we are in the process of moving onto the new .NET CLI


28139 - Bukharan Uzbeks - Reassign two songs; Change society lat-lon

Songs 4102 and 4108 are not from Bukhara, they are from Fergana. Suggest reassigning to society 62302 (Uzbeks).
Also, society lat-lon is currently in Afghanistan. Should be relocated to Bukhara (39.76, 64.45).

Suggestions implemented


Standardize human-readable output in bot-interfaces:export command

bot-interfaces:export should output "Bot interface exported to file ".
Unit tests may need to be adjusted.

This was resolved thorugh issue #126


NECO-192 removed unnecessary empty lines and added return to finish the method's procedure.

Ticket
NECO-192
Changes

Removed unnecessary empty lines.
Added returns to finish the method's procedure.

Tests

Tested all functions are run normally.
Tested procedures after added return doesn't execute when fixes condition.

Details and screen shots are in ticket.

Thank you for your coding !
I'm going to merge this pull request.


Adding R language to CircleCI testing

@richardbeare
This is based on the release branch. I hope it will help with your CRAN effort.


RazorEngine.Templating.TemplateCompilationException

UPDATE: We are using RazorEngine v3.8.2
I want to start off by saying I only encounter this issue on our test environment (deployed in azure app service) but not locally. Possibly due to debug configuration?
I am trying to compile e-mail templates. Originally, this was done in a worker role (cloud service) without issues. Now, I have enabled a preview functionality within our web application, and RazorEngine now runs in our web app to compile and display the e-mail template preview.
However, I run into issues once deployed to azure app service (not locally).
This is the error message details:
Type: RazorEngine.Templating.TemplateCompilationException
Message: Errors while compiling a Template.
Please try the following to solve the situation:
* If the problem is about missing/invalid references or multiple defines either try to load
the missing references manually (in the compiling appdomain!) or
Specify your references manually by providing your own IReferenceResolver implementation.
See https://antaris.github.io/RazorEngine/ReferenceResolver.html for details.
Currently all references have to be available as files!
* If you get 'class' does not contain a definition for 'member':
try another modelType (for example 'null' to make the model dynamic).
NOTE: You CANNOT use typeof(dynamic) to make the model dynamic!
Or try to use static instead of anonymous/dynamic types.
More details about the error:
- error: (0, 0) An expression is too long or complex to compile
Temporary files of the compilation can be found in (please delete the folder): D:\local\Temp\RazorEngine_ckxscmhx.1ih

This is the template (redacted some parts) it tried to compile:
@{ Layout = Model.BrandingType + "_MasterLayout"; ViewBag.IncludeFooter = false;}@section Overview{ @Include("HeaderGreeting")

Welcome to the online portal for managing your needs. }@if (Model.Order != null){

You have purchased
@Include("PurchaseDetails")
}

What happens next?

To activate your account, please log in to online portal using the details below.

Email: @Model.Email

Password: @Model.Password

Log in now

You will be prompted to change your password when you log in for the first time.

For information about the portal, please visit our website.

The interesting part is that the list of loaded assemblies is crazy long. These are the first 10 lines:
List of loaded Assemblies:
D:\Windows\Microsoft.NET\Framework\v4.0.30319\mscorlib.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_32\System.Web\v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a\System.Web.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\System\v4.0_<IP_ADDRESS>__b77a5c561934e089\System.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\System.Core\v4.0_<IP_ADDRESS>__b77a5c561934e089\System.Core.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\System.Configuration\v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a\System.Configuration.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\System.Xml\v4.0_<IP_ADDRESS>__b77a5c561934e089\System.Xml.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\System.Web.ApplicationServices\v4.0_<IP_ADDRESS>__31bf3856ad364e35\System.Web.ApplicationServices.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\System.Runtime.Caching\v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a\System.Runtime.Caching.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\Microsoft.Build.Utilities.v4.0\v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a\Microsoft.Build.Utilities.v4.0.dll
Loaded Assembly: D:\Windows\Microsoft.Net\assembly\GAC_MSIL\System.Web.RegularExpressions\v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a\System.Web.RegularExpressions.dll

However, it goes on for another 10,000+ more lines, 10,153 to be exact.
The stack trace points to the Compile method:
Source: RazorEngine
Target Site: System.Tuple`2[System.Type,RazorEngine.Compilation.CompilationData] CompileTypeImpl(RazorEngine.Compilation.TypeContext)
Stack Trace:
at RazorEngine.Compilation.DirectCompilerServiceBase.CompileTypeImpl(TypeContext context)
at RazorEngine.Compilation.DirectCompilerServiceBase.CompileType_Windows(TypeContext context)
at RazorEngine.Compilation.DirectCompilerServiceBase.CompileType(TypeContext context)
at RazorEngine.Templating.RazorEngineCore.CreateTemplateType(ITemplateSource razorTemplate, Type modelType)
at RazorEngine.Templating.RazorEngineCore.Compile(ITemplateKey key, Type modelType)
at RazorEngine.Templating.RazorEngineService.CompileAndCacheInternal(ITemplateKey key, Type modelType)
at RazorEngine.Templating.RazorEngineService.Compile(ITemplateKey key, Type modelType)
at RazorEngine.Templating.DynamicWrapperService.Compile(ITemplateKey key, Type modelType)
at RazorEngine.Templating.RazorEngineServiceExtensions.Compile(IRazorEngineService service, String name, Type modelType)

Any ideas? Thanks!

Good luck.... We are moving to azure function too... Honestly I think I will change engine... It's about a week I am trying to make it work and there is no support. Here people answer rarely, you have been lucky because I am here to doing a new question...
So, to solve your problem you must write a custom ReferenceResolver... This WAS mine:
    internal class ExternalAssemblyReferenceResolver : IReferenceResolver
    {
        private string[] _assembliesToLoad;
        public ExternalAssemblyReferenceResolver(params string[] assembliesToLoad)
        {
            _assembliesToLoad = assembliesToLoad;
        }

        public IEnumerable<CompilerReference> GetReferences(TypeContext context, IEnumerable<CompilerReference> includeAssemblies = null)
        {
            IEnumerable<string> loadedAssemblies = CompilerServicesUtility
                .GetLoadedAssemblies()
                .Where(a => !a.IsDynamic && !a.FullName.Contains("Version=<IP_ADDRESS>") && File.Exists(a.Location) && !a.Location.Contains("CompiledRazorTemplates.Dynamic"))
                .GroupBy(a => a.GetName().Name).Select(grp => grp.First(y => y.GetName().Version == grp.Max(x => x.GetName().Version))) // only select distinct assemblies based on FullName to avoid loading duplicate assemblies
                .Select(a => CompilerReference.From(a))
                .Concat(includeAssemblies ?? Enumerable.Empty<CompilerReference>())
                .Select(r => r.GetFile())
                .ToArray();

            yield return CompilerReference.From(FindLoaded(loadedAssemblies, "mscorlib.dll"));
            yield return CompilerReference.From(FindLoaded(loadedAssemblies, "netstandard.dll"));
            yield return CompilerReference.From(FindLoaded(loadedAssemblies, "System.dll"));
            yield return CompilerReference.From(FindLoaded(loadedAssemblies, "System.Core.dll"));
            yield return CompilerReference.From(FindLoaded(loadedAssemblies, "RazorEngine.dll"));

            yield return CompilerReference.From(typeof(ExternalAssemblyReferenceResolver).Assembly); // Assembly

            foreach (var assembly in this._assembliesToLoad)
            {
                yield return CompilerReference.From(FindLoaded(loadedAssemblies, assembly));
            }
        }

        private string FindLoaded(IEnumerable<string> refs, string find)
        {
            return refs.First(r => r.EndsWith(System.IO.Path.DirectorySeparatorChar + find));
        }
    }

What I understood is that you must load all references that are necessary to compile the cshtml... so you must add system.dll, mscorlib.dll ... and so on (perhaps you do not need netstandard.dll, it depends on the type of project where your templates are saved)... than you must add also your dll where your models are saved... you cannot see it in my code because I passed the name of my dll inside the variable _assembliesToLoad...
I think you can use my code and pass in input your dll name...
Finally, in your configuration you have to use your new ReferenceResolver:
TemplateServiceConfiguration configuration = new TemplateServiceConfiguration();
            configuration.ReferenceResolver = new ExternalAssemblyReferenceResolver(assembliesReferencePaths);

It should start to work but you should receive an advice that it is better that you run razorEngine in a new app domain, otherwise temp file are not deleted... infact if you check temp file will not delete.... It is about 3 days I am trying to resolve that problem.... but nothing...
If it still does not work let me know... I do not remember anymore all the passes I did to let it works (I am not sure you have to change also the template manager)... I have done a lot of reverse engineering to solve all my problems.. and I have not finished yet...
Hope this help


PrivateEndpoint & PrivateLinkService APIs update

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-java

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?

Given that two new APIs are related to PrivateLinkServices, maybe it would be better to add them to privateLinkServices.json instead of creating two new files?

Could you please review my comment above (response schema related). Adding previously recorded traffic for more context (there is response body received from server):
DEBUG: ============================ HTTP REQUEST ============================

HTTP Method:
PUT

Absolute Uri:
https://management.azure.com/subscriptions/<ID>/resourceGroups/test-evg/providers/Microsoft.Network/privateLinkServices/test/privateEndpointConnections/test.
19b38612-9576-4036-88d2-dc34e0cb5b18?api-version=2019-04-01

Body:
{
  "properties": {
    "privateEndpoint": {
      "properties": {
        "privateLinkServiceConnections": [],
        "manualPrivateLinkServiceConnections": []
      },
      "id": "/subscriptions/<ID>/resourceGroups/test-evg/providers/Microsoft.Network/privateEndpoints/test",
      "tags": {}
    },
    "privateLinkServiceConnectionState": {
      "status": "Rejected",
      "description": "change"
    }
  },
  "name": "test.19b38612-9576-4036-88d2-dc34e0cb5b18",
  "id":
"/subscriptions/<ID>/resourceGroups/test-evg/providers/Microsoft.Network/privateLinkServices/test/privateEndpointConnections/test.19b38612-9576-4036-88d2-dc34e0cb5b18"
}

DEBUG: ============================ HTTP RESPONSE ============================

Status Code:
OK

Body:
{
  "name": "test.19b38612-9576-4036-88d2-dc34e0cb5b18",
  "id":
"/subscriptions/<ID>/resourceGroups/test-evg/providers/Microsoft.Network/privateLinkServices/test/privateEndpointConnections/test.19b38612-9576-4036-88d2-dc34e0cb5b18",
  "etag": "W/\"0cd819b1-aa1c-4ccc-ae61-de4928d37de0\"",
  "properties": {
    "provisioningState": "Updating",
    "privateEndpoint": {
      "id": "/subscriptions/<ID>/resourceGroups/test-evg/providers/Microsoft.Network/privateEndpoints/test"
    },
    "privateLinkServiceConnectionState": {
      "status": "Rejected",
      "description": "change",
      "actionsRequired": ""
    }
  },
  "type": "Microsoft.Network/privateLinkServices/privateEndpointConnections"
}

@KrisBash could you please give feedback from ARM's side?

Signing off from ARM side with one minor comment. Not blocking but good to fix

What's up with the JS SDK failing?

@tjprescott @number213 who can help look at the SDK issue?

I think this is the step that was removed by @sergey-shandar (https://github.com/Azure/azure-rest-api-specs/pull/6391). My guess is that it's deprecated and Network's release branch is out of sync with master.

@bowen0106 can you try rebase your branch?

@shahabhijeet can you please with the go sdk generation? or point to someone who can help?

@jhendrixMSFT will the right person who can help you with Go SDK related questions.

The build errors are a known issue for network (not just Go is affected).  I'm following up with the owners of the swagger-to-sdk job but for now the failures can be ignored.

@AutorestCI regenerate azure-sdk-for-go

@AutorestCI regenerate azure-sdk-for-go

Encountered an unknown error
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 33, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/bot_framework.py", line 105, in manage_comment
    response = getattr(self.handler, orderstr)(webhook_data.issue, *split_text)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/restbot.py", line 54, in regenerate
    return self.rebuild(issue, repotag)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/restbot.py", line 36, in rebuild
    self.sdk_default_base
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github_handler.py", line 151, in rest_pr_management
    sdk_tag=sdk_tag
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/SwaggerToSdkNewCLI.py", line 310, in generate_sdk_from_git_object
    sdk_repo.git.push('origin', branch_name, set_upstream=True)
  File "/usr/local/lib/python3.6/dist-packages/git/cmd.py", line 548, in <lambda>
    return lambda *args, **kwargs: self._call_process(name, *args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/git/cmd.py", line 1014, in _call_process
    return self.execute(call, **exec_kwargs)
  File "/usr/local/lib/python3.6/dist-packages/git/cmd.py", line 825, in execute
    raise GitCommandError(command, status, stderr_value, stdout_value)
git.exc.GitCommandError: Cmd('git') failed due to: exit code(1)
  cmdline: git push --set-upstream origin restapi_auto_6332
  stderr: 'To https://github.com/Azure/azure-sdk-for-go.git
 ! [rejected]            restapi_auto_6332 -> restapi_auto_6332 (fetch first)
error: failed to push some refs to<EMAIL_ADDRESS>hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.'

@jhendrixMSFT who's the owner of the swagger to sdk job? we really need this PR to merge ASAP so we can start our work on AKS private clusters which is scheduled to go in preview in 6 weeks.

This was resolved yesterday by @lmazuel.  Regardless you can still merge the PR.


kodi platform: following jsonrpc-request version bump (0.3), let kodi…

Description:
Hand over the collection item, the url or the file to play directly to kodi's file abstraction layer
Related issue (if applicable): fixes #
fixes kodi platform's implementation of media_player.play_media()
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If code communicates with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Looks good! :dolphin:


Supabase Vector Store Problems

I have problems using Supabase as a Vectorstore. I have a textfile containing a speech of the German Chancellor Scholz as a testdocument. This speech is about the war in the Ukraine and of course contains the word "Ukraine" multiple times. I setup Supabase as described in the documentation (https://js.langchain.com/docs/modules/indexes/vector_stores/integrations/supabase)
I first generated the vectorstore using the fromDocuments()-method.
I then created a chain and queried "Was sagt Scholz zur Ukraine" (in english: what does Scholz tell about the Ukraine), I get the answer "I don't know". When I asked him "Was sagt Scholz zur Zeitenwende" (the speech is called Zeitenwende-speech and the word is used in the text too) the query returned a completelly wrong text.
I have then enhanced the testscripts to use chroma and hnswlib. When using these stores, I get a different, but correct results for both questions described above.
I know, that the results may differ between stores, but Supabase failed to answer many questions where the other stores worked just fine.
I attach the testdocument and my testscripts so that you can reproduce the problem
Regierungserklärung von Bundeskanzler Olaf Scholz am 27. Februar 2022.md
scripts.zip
package.json.zip

@henriette-einstein I wonder if it's an issue with the index, try dropping and recreating
drop index documents_embedding_idx;

create index on documents
  using ivfflat (embedding vector_cosine_ops)
  with (lists = 100);

and then try the query again afterwards

That worked! But as far as I can see, this statement is equal to the last statement you have in the documentation. Does that mean, that I always have to recreate the index after adding a new Document to the index?

@henriette-einstein some more info form supabase on this

Re-indexing should only be needed if the distribution of data changes significantly. pgvector uses IVFFlat (Inverted File) indexes which come with a few nuances:

The index enables an approximate nearest neighbour search (since exact search on high dimensional data can't be indexed efficiently). This means that query results on the same data will actually change after you add an index (trading recall for speed).
pgvector recommends adding indexes only after the table has sufficient data, so that the internal IVFFlat cell clusters are based on your data's distribution. Anytime the distribution changes significantly, indexes should probably be recreated.
When you create the index, you choose the number of inverted lists (cell clusters). Increase this number to speed up queries, but at the expense of recall.
Every query you can set the number of probes (1 by default), which is referring to the number of nearby cells to probe for a match. Increase this for better recall at the expense of speed.
Some good references I've found are:
pgvector has some documentation on their indexing: https://github.com/pgvector/pgvector#indexing
Facebook Research's Faiss library discusses IVF: https://github.com/facebookresearch/faiss/wiki/Faiss-indexes#cell-probe-methods-indexivf-indexes
James Briggs has a good video on indexes and has a section on IVF: https://www.youtube.com/watch?v=B7wmo_NImgM&t=1360s


Update the GitHub connector docs

The GitHub connector docs were very hard to read before, and did not include any reference to the very important preferredEmailDomain feature added by https://github.com/dexidp/dex/pull/2740

@nabokihms @abhisek this is a small but important update to the GitHub docs which should help new users.
I am grateful for your review.

Also, we might want to fix the "bug" that preferredEmailDomain is not used if the user has a public GitHub email, its caused by this code:

https://github.com/dexidp/dex/blob/v2.40.0/connector/github/github.go#L534-L541

@thesuperzapper thanks! If you think the current preferredEmailDomain, feel free to open a PR.

@nabokihms I have raised a PR https://github.com/dexidp/dex/pull/3584 to fix the preferredEmailDomain issue.


Subarray with given sum gfg

pls assign this problem to me i assure will provide easy and efficient code

Hello there!! I would also like to contribute to this issue. I am fairly confident in my DSA concepts, however I am fairly new to open source!


App would not open

Windows:
Scout-App Version:
Information from Dev Tools:
(Attach screenshot below)
I extracted the scout app several times and maybe that may be the issue but I cannot open the app itself. Please help me to solve this issue. Thank you.

did you follow the steps in this video:
https://www.youtube.com/watch?v=6zA78zMsH9w
If so, you may need to right-click > run as administrator. Or check your task manager and see if scout-app.exe is running and kill those processes. If it didn't open properly the first time (missing files or something), then the stray scout-app.exe files may prevent it from opening again later.


Add way of checking if existing ToCs need updating

I'm not currently a doctoc user, but if I were, one feature I imagine I'd like would be the ability to check whether doctoc needs running. This would be handy as part of a CI test suite to ensure that some careless soul hadn't updated the readme and forgotten to run doctoc.
All the check would need to do is locate all markdown files with a  in them and check that their current content exactly matches what their content would be after running doctoc.

That's actually a really great idea. I never thought about it, but that would actually make life a whole hell of a lot easier. I don't have access to a computer this week, but if nobody wants to work on this, I'd love to try and get it working.

Why not just run doctoc as an automated step to make sure it's always up to date?

Maybe a user may just want to see what needs updating without actually updating? A dry run, of sorts.

Well if people need that I'm open to a well tested PR.

Why not just run doctoc as an automated step to make sure it's always up to date?

Analogoously you could arguethat there's no need for a linting step that detects trailing whitespace when you could have an automated step that strips trailing whitespace. This is likewise a defensible idea, but the trouble is that if you're relying upon this step being run via a hook on people's dev machines, you have no guarantee it'll get run (since a new contributor might not add the hook), and if you have it done as part of your CI, you need your CI server to be making and pushing commits for you, which isn't something I've ever seen done in the wild, and is more complicated - both to implement and to understand - than simply checking whether something is broken and nagging a human to fix it.

@ExplodingCabbage not sure why you're still trying to convince me since I already said I'm open to a PR that adds this ;)


Have Pachctl repect PACHD_SERVICE_PORT evar when available, instead of defaulting to 30650

What is the goal / desired outcome?
Pachctl should "just know" how to connect to the most contextual pachd.
Currently, pachctl fails to do this when deployed in a kubernetes container.
If there is a way to accomplish this today via workaround, what does that require?
Yes, respect the kubernetes environment variables injected into the container to discover pachd.
(Optional) What is your proposal for a feature to solve this?
Be aware of an available k8s service, named pachd by default, by respecting the following env variables:

PACHD_SERVICE_HOST (optional - literal 'pachd' should work on most k8s configurations)
PACHD_SERVICE_PORT  (must be respected - at the very least port must change from 30650 to 1650 when on a k8s container)

If possible, also allow config variables to:

Change the service name. If the deployment names the service anything other than pachd, then this feature would fail to work.
Ignore the environment variables - disable this feature altogether.

Environment?:

Kubernetes version (use kubectl version):
Pachyderm CLI version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS (e.g. from /etc/os-release):
Others:

@nitinjainsj @ysimonson This is related to the context work that Yusuf did last year.  As stated, it's about using pachctl from within a container in the same k8s cluster.  Don't think I'd make it a super-high priority.  Might be good to just add docs showing how to easily do it in the Connecting to Pachyderm section (cc @svekars )


OLS - handle classifications

Following https://github.com/odpi/egeria/pull/2718 implementation on Asset Lineage OMAS for classifications, adds functionality in OLS for handling classifications:

Store classification in the buffer graph for ClassifiedEntityEvent/ReclassifiedEntityEvent event
Remove classifications from the buffer graph on DeletedEntityEvent
Store classifications to main graph - add all the classifications vertices and edges that are connected to the processes/columns that are added to the main graph
Handle update/ delete classifications for main graph

Following PRs address the changes needed for OLS classifications:
https://github.com/odpi/egeria/pull/3188
https://github.com/odpi/egeria/pull/3191/files

More changes are needed to add the classifications for the subProcess nodes displayed in the UI - new PR will follow in a few days


Starting the helper and inspiration material list

This is a general-purpose "things that help others reuse their mainboards" list, both creations intended to facilitate reuse, and projects one can draw inspiration from.


Local self-signed certs

Some of my internal servers use self-signed certificates, but I must monitor their interfaces. However, because they are self signed I get an error:
Cannot connect to host xxxx.local ssl:True [SSLCertVerificationError: (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self-signed certificate (_ssl.c:992)')]
Could this be handled differently (e.g. with a flag to ignore self-signed?)

I have a solution, which I have implemented in my local version. Once I have tested it all out I will create a pull request.

Since I dont have any self-signed certs, yet, I can't reproduce the issue. Looking forward for your PR! Thanks @GeekyTim

Pull request requested.


⚡ Chunk wal event inserts

#475  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RTLS and the rest of your teammates on  Graphite


Make website/wordpress database optional

Describe the task
The setup of website/WordPress database has become mandatory otherwise we can't create any job opportunity.
https://user-images.githubusercontent.com/79009032/131099978-523138a1-872a-45d8-ab8d-880ef07452b4.png
Expected behavior

[ ] Instead of enforcing another DB setup, make syncing with website an optional field.
[ ] Create a boolean .env variable WORDPRESS_ENABLED that must be checked wherever corcel is used.
[ ] The variable must be added to .env.example and default value should be false
[ ] Update documentation about this variable and what it does

@am-Anmol @GauravGusain98 this is not working on UAT.

This is also working on UAT. Closing.


Content pane's background changes to black when click on Edge

Repro steps:
• Navigate to https://makecode.microbit.org/ on Edge
• Drag blocks as below picture
• Click few times on content pane until the background become white
• Click any block
Expected result:
The content pane's background shouldn't change
Actual result:
The content pane's background change to black

What is the version of Edge. I can't repro this.


Error loading import source

I have downloaded and trying to run this from the index.html in the browser (Firefox).
Initially the page looks like this:

When I click on any button I get the error

What am I doing wrong?

The resource files are served by an additional server in development.
As well as
npm start
In a different terminal
npm run serveextras


[Bug] Replace "TE1 Spring 2024" from TH UI with "beta1.1 Spring 2024"

Describe the bug
Replace the "TE1 Spring 2024" UI with "beta 1.1 Spring 2024"
Steps to reproduce the behavior
No response
Expected behavior
No response
Log files
No response
PICS file
No response
Screenshots
No response
Environment
No response
Additional Information
No response

Done in release https://github.com/project-chip/certification-tool/releases/tag/v2.10-beta1.1%2Bspring2024


_.isNaN returns true for whatever created with new Number() instead of just NaN and new Number(NaN)

// All => true
_.isNaN(new Number(3))
_.isNaN(new Number(Infinity))
_.isNaN(new Number('donut'))

If _.isNaN is intended to return true for only NaN and new Number(NaN), I think in line 1304, obj !== +obj should be replaced with obj != +obj.
Correct me if I'm wrong : )

I don't understand what you're trying to say.

I don't know if his proposed solution will work.  But, if _.isNaN(new Number(3)) returns true, that's probably a bug.

Feel free to send a PR with the change to obj != +obj

If we change the code to obj != +obj, _.isNaN still returns true for things like new Number('dice'), but I guess that's effectively the same as NaN in operations : )

Since != won't make it through the build, I suggest we can use the good old isNaN with _.isNumber, thus
_.isNumber(obj) && isNaN(obj)

_.isNumber makes sure obj is a number, primitive or wrapped, and then isNaN makes sure only NaN and new Number(NaN) pass the test.


THRIFT-5134: Fix memory leak when the handler method return FALSE

Client: c_glib
Patch: wangyunjian
Signed-off-by: wangyunjian<EMAIL_ADDRESS>

It will be closed. Ping for review, thanks.

.

.


Adding roaring.dll to NuGet package with output copy

This will include roaring.dll in the NuGet package with the new .csproj format.

Took me a little while to figure out the correct way to do this with newer .csproj format -- in any case I've tested the NuGet package against a .NET Core 2.0 host and this works correctly.


feat: generate++

Closes #371.
This version covers steps 1 and 2, but not 3 and 4 yet.

Updated to add steps 3 and 4 - allow-dirty field added to config, added automatic checking during dist and manifest commands, and exercised allow-dirty in tests.