Separate leader port and local port

For running multiple derecho instances on a single node as different processes, the nodes need to use different ports for TCP connections. However, ports for all the nodes are assumed to be the same in the code currently. Instead we can provide the leader port as an input and have the port for the local node in the config file. Then the leader can be contacted by any starting member and the leader can then share the port of the existing members so that other nodes can be contacted. This can be conveniently achieved by sharing the ports using the SST, just as IP addresses currently are shared.

Whoever tackles this change might consider just having us store a Linux ip_addr struct, rather than treating IP addresses as an int_32 and adding an additional int_32 for the port number.  The standard Linux struct has a type field (IPv4 or IPv6), a value field, and a field for the port number.  So the code change is minimized (one int_32 becomes one instance of the struct), existing code that needs the IP address does need to change but in a fairly simple way, and we benefit because of some network out there is pure IPv6, we could run on it rather than being limited to IPv4 networks.

Related to this issue, we verified that Derecho works on a local machine. We tested:

Separate derecho instances on the same set of machines
Derecho test with two VMs on the same machine
Derecho test with a VM and a different laptop
Derecho test with a VM and a Fractus machine

It would be worth the effort to think about situations where some Derecho processes are TCP only (and yes, not locked to specific port numbers unless capable of playing the leader role), while some have RDMA interfaces and “switch” to those if all members of a given subgroup or shard are equally capable.
Then we could have a big group, a rack of RDMA-capable servers, and a larger set of TCP-only “clients” using the LibFabrics to TCP mapping, but still benefitting from the high efficiency of that version of TCP, the zero-copy marshalling logic, etc.  I think this might be an ideal mix, in the long term, for many developers

While that is definitely something to do in the long run because we will have clients outside the system unable to use RDMA, we currently only support all TCP or all RDMA nodes. Weijia might know how much time implementing this will take.
The feature will be that every node can join the group regardless of whether they use TCP or RDMA (but only one). All nodes can talk to all nodes. So if a subgroup has some members using RDMA and some TCP, atomic multicast should still work just fine.

I agree.  Basically, it is probably either easy to do this, or impossible to do this!  And it will depend entirely on whether LibFabrics already supports the needed functionality.

Commit 3ee59730da83505fd9d867059b0a40ee4c8e0c66 says it closes this issue, but Git didn't notice and auto-close it. (The exact text has to be "closes #60", not "closes issue #60").


[Q] Do we have a api that can create a project from python or command line?

For the time being
run = wandb.init(project="new-project")
with run:
    run.log({"metric": metric})

this should create a project named "new-project" where "metric" is logged.

Hey @HaoLiuHust
Could you describe the problem a little better?
For the time being
run = wandb.init(project="new-project")
with run:
    run.log({"metric": metric})

this should create a project named "new-project" where "metric" is logged.

thanks for your reply, I thought init must ha


How to use with other drop-in replacements for request?

I'd like to use co-request with other drop-in replacements for request, in particular https://www.npmjs.com/package/requestretry (https://github.com/FGRibreau/node-request-retry), is this possible or do you know how I could do this?

@rightaway you need to provide list of extended methods, co-request does not support this. I will take a look, maybe we can create method to push new method names to handle.


Allow overriding menu, widgets, and links from outside theme directory

I'd like to control my maupassant configuration outside of the theme directory (see https://hexo.io/docs/configuration#Alternate-Theme-Config). For most of the settings this works fine. For the menu, widgets, and links settings, however, the resulting merged config makes it impossible to remove items.
For example. In my _config.maupassant.yaml file, I have the following:
widgets:
- search
- tag
- links

In the themes/maupassant/_config.yaml file, there is the following default:
widgets: ## Six widgets in sidebar provided: search, category, tag, recent_posts, rencent_comments and links.
  - search
  - category
  - tag
  - recent_posts
  - recent_comments
  - links

When I run hexo serve, I see the following list of widgets:

search ✔
tag ✔
links ✔
recent_posts ❌
recent_comments ❌
links ❌

If I try appending - NULL to my lists, I get errors. The only workaround I've found so far is to comment out the contents in the themes/maupassant/_config.yaml file, but this obviously defeats the purpose.
I'm using:

maupassant from https://github.com/tufu9441/maupassant-hexo/commit/dc2e0e44fd6e033e1b7c600eaf565ce9f3f55a43
hexo: 5.2.0
node: 15.0.1

I am sorry that I haven't used the alternate-theme-config of Hexo yet, so I cannot clarify the reason for your problems at present, maybe there are some syntax conflicts in those .pug files of the widgets.


feat: s3 upload and cluster scripts

This PR adds 2 major features:

A module for uploading data to s3
A system for submitting arbitrary jobs to a cluster, heavily inspired by (read: copied from) https://github.com/JaneliaSciComp/spark-janelia, but in this case we use https://jobqueue.dask.org/ instead of Spark.

Sorry for the monolith, this should have been 2 separate PRs but I got carried away.
For an example cluster job, see https://github.com/AllenNeuralDynamics/nd-data-transfer/blob/feat-s3-upload/scripts/README.md
Benchmarks
I uploaded a directory of 2,758 tiff slices from the Isilon using 32 HPC cores (split across 4 nodes, 8 tasks per node, 1 cpu-per-task)
369,103,409,262 bytes uploaded in 123.6s, or 2.986 GB/s
Command used:
python cluster/submit.py generate-and-launch-run \
--job_cmd="python s3_upload.py --input /allen/scratch/aindtemp/data/anatomy/exm-hemi-brain/fused_tiffs --bucket fake-bucket --s3_path=upload-test/fused_tiffs --part_size=256000000 --batch_num=10 --cluster --recursive" \
--conda_activate=/allen/programs/aind/workgroups/msma/cameron.arshadi/miniconda3/bin/activate \
--conda_env=nd-data-transfer \
--queue=aind \
<EMAIL_ADDRESS>\
--ntasks_per_node=8 \
--nodes=4 \
--cpus_per_task=1 \
--mem_per_cpu=2000 \
--walltime=01:00:00

I still need to test with the Vast
Things I wasn't sure about
Tests - what is the best way to test uploads to s3? Presumably we would mock a request and not actually hit a real endpoint. The way AWS tests the CRTTransferManager is shown here
Upload retries - I keep a list of filepaths that failed to upload, so those could be retried either within each worker or at the end in a serial pass. We could also run a aws s3 sync command at the end.
Code organization - is /scripts the best place to put the cluster submission code? It is general enough that it could potentially go in a separate repo

I think it might be better to merge this branch into main if it's ready. It might be better to open another pull request for the tests and separating out the scripts if wanted.

got the go-ahead from @dyf , merging


Pipette Gen2: Remove Gen2 feature flags from app

overview
Prior to the customer launch of GEN2 pipettes, we need to promote to following feature flags to default behavior:

The app's enablePipettePlus flag

Acceptance Criteria

[ ] GEN2 pipettes appear in app's change pipette wizard by default

PD FF moved to #4295


consumer heartbeats

When a push consumer disconnects from the network due to bad network that consumer will just stop working and the client will have no way to determine this happened, there's no way to confirm it's valid and no way to recover from that scenario. The same scenario that led to https://github.com/nats-io/stan.go#connection-status
We need a concept of a heartbeat so the consumer can get messages telling them things went bad, something like a nil body with a Header indicating heartbeat count or similar.
What I don't know is what kind of callback we'll fire or what we'll do in the absence of that heartbeat?
sub, err := nc.Subscribe(nats.NewInbox(), handler)

how does that communicate this failure and what happens when that failure happens? It has to be subscription specific, we don't want every user to (badly) implement the handling of this, I imagine we'll need some helper that:

Remembers creation parameters
Tracks the message sequences including whats been ack'd
Transparently recreates the consumer if it heartbeat fails using the tracked sequence as starting position

But the recreation is very problematic since we can't know about out of order deliveries due to redeliveries and so the simple scheme or subscribing to last acked+1 will cause duplicates. I can't think of a good design with what we have.
But remembering how really painful this was in STAN to retrofit to existing code bases we need an answer for this during tech preview timeline

This was a bit about my desire to put in how far behind you are in the reply subject / metadata for delivered messages. This does not help with the very next one after the last, but can help quite a bit in some designs.
Question these sorts of things bring up is what will the application do with the information? Is it helpful or actually just more confusing. Not totally clear.
I do think adding in the last sequence known for the stream to delivered messages is important.

We will have to create a helper that observes these heartbeats and not expose them to the user, that recreates/resubscribes when there is issues.
Easy with durables they will do the right thing but ephemerals are a real pain you just dont know where to start

Not sure HBs are the right thing tbh..  What will the app do with that information?

How else will you tell the consumer he's getting no data because there's no new data vs no data because he is dead if not a heartbeat?
Just spitballing the first thing that come to mind, but we should provide something very roughly like this:
sub, err := stream.NewEphemeralConsumer(jsm.FilterStreamBySubject("foo"), jsm.StartWithLastReceived(), handler)

handler gets called for every message just like nats.Subscribe() - handler not called for heartbeats - but the NewEphemeralConsumer will create something to monitor the absence of heartbeats and recreate up to last acked against the same inbox - this way the subscription continues subject to standard underlying nats connection retries.

How we do it is not as important, my question is what does the app do? Exit? Wait for NATS to self heal? In other words giving that information sometimes is not a good thing. And many times its wrong in one form or another and causes the app to do the wrong thing. Especially with NATS where it will try to self heal as best it can.

As I showed above, that's what is done with the heartbeats or absense thereof.

Especially with NATS where it will try to self heal as best it can.

In this case the self heal leads to a broken consumer - the ephemeral timed out, no data will ever be received again.

Well for ephemerals we do wait a bit for them to come back and do not react too quickly, but how else would the server know to remove the state for it?
For a durable, does it do the right thing after a self heal? I agree it should if it does not.

durables are fine.
An ephemeral just stop working if the interruption is longer than our grace period and the application has no way to detect that or recover.

The app of our layering could ping to JS and ask about the ephemeral consumer and if it detects the system is there but the consumer is gone resume where it left off.

For it to serve purpose the ping would have to be done regularly, say every n seconds after y seconds of no updates.
-> is consumer there
<- yes
vs
every n seconds after y seconds of no updates:
<- yes consumer is there
heartbeats is 50% of the traffic.

The intelligence would be the same if server side or app side, only ask or HB in absence. But semantics are better on app side and spreads out the processing vs making server do everything.

ok, doesnt really matter for this issue which approach we take just now, as long as we agree its a thing thats needed

Agree that if possible ephemeral consumers should self heal, but I believe this is on the app side, not the server side.

With both above models the server need to support this? Or do you suggest the consumer just repeatedly call the existing is known consumer api for this?

My problem with the consumer side only approach is same as my problem with the nats streaming server behaviour, everyone has to get it right, every language binding has to get it right, 10s of library implementations and if you dont get it right from the get go the structure of your program changes so significantly later that it's effectively a rewrite.
https://github.com/nats-io/stan.go#connection-status is listed under "Advanced Usage" when in fact it's should be required for every single consumer ever to implement this else the system fails silently in a unrecoverable and undetectable manner.  It's a terminal failure in library design imo
So in JetStream I think doing the right thing should be obvious and visible from day one.  If the server emits heartbeats its 1 implementation to get right and everyone who ever make ephemeral consumers discovers this need.

I agree client code needs to be correct in reference impl regardless if its receiving HBs or doing inactive polls to see if consumer still exists.

There are a few significant differences:
If the server sends HBs:

Clients will discover this feature and be actively prompted to use the correct approach
Clients, while having to behave correctly, have a much lower amount of code to manage
Server can handle this in a single place

If the client handles this entirely:

Clients have to find out from documentation, experience, past failures that this is required. Worst case scenario imo
Clients will then have to do quite large code reorgs to handle this scenario, just like it was with STAN
Clients have to do a whole lot of work just right - calling the API, possibly in go routines if those exist etc, would be hard to do on low spec machines
Servers have to pass all this through the API entrance
Huge load on future RBAC systems that will do access control to the API
Huge amount of API audit advisories, making this not 50% more traffic but a very large amount more. Every ephemeral consumer will be hitting this API regularly
Every client needs access to the API calls that can potentially leak information about other consumers that is none of their business

This has been done


texture_array_reflections is broken on mobile (GLES3)

Godot version
3.3.3.stable, 3.3.2.stable
System information
Linux Manjaro 21.1.0, Android 10(Xiaomi Redmi 8, arm64-v8a, OpenGL 3.0.2, confirmed to occur on other phones as well), using GLES3
Issue description
For backstory see #52064
texture_array_reflections seems broken on mobile.
On PC:

texture_array_reflections set to true on PC.

On mobile:

texture_array_reflections.mobile set to false, looks good.

texture_array_reflections.mobile set to true, broken.

Steps to reproduce

Create new GLES3 project
Import 3D model
Create new spatial material with albedo texture.
Apply material to the model and save it as a scene.
Instantiate child node in main scene using result of previous step(model with the material).
Create 3D camera in main scene.
Install android build template
Setup export to android.
Export to android and compare result from phone with the result when running F5 or using editor.

Minimal reproduction project
Note: screens in issue description were made with normal/metallic/roughness maps and HDRI, but they are not included in project as they are not necessary to reproduce the bug.
reproduce.zip

Isn't this issue a duplicate of https://github.com/godotengine/godot/issues/52064? I don't understand how this issue and https://github.com/godotengine/godot/issues/52064 differ.

@Calinou #52064 is a visual bug on desktop platforms when  texture_array_reflections is false.
This is an android bug where the reflections don't work when texture_array_reflections is ``true`


Allow object destructuring assignment

$ cat 1.js
'use strict'

let a = {x: 1, y: 2}

let b = {...a, c: 3}

console.log(b)
$ cat .jshintrc
{
  "asi": true,
  "esversion": 6,
  "strict": true,
  "node": true,
  "undef": true,
  "unused": true
}

$ jshint 1.js
1.js: line 5, col 10, Expected '}' to match '{' from line 5 and instead saw '...'.
1.js: line 5, col 13, Missing semicolon.
1.js: line 5, col 16, Expected an assignment or function call and instead saw an expression.
1.js: line 5, col 17, Missing semicolon.
1.js: line 5, col 16, Unrecoverable syntax error. (62% scanned).

5 errors
$ jshint --version
jshint v2.9.5

expected： no error

Thanks for the detailed report! The technical name of the language feature you are describing is "object spread properties" and "object rest properties". Since we already have a tracking issue for the feature request (see gh-2991), I'm going to close this report as a duplicate.
Good news, though: support has been implemented on a feature branch and is expected for the next minor release of JSHint!


feat: install googlecompute packer plugin

What problem does this PR solve?:

bakes googlecompute packer plugin in Devkit and KIB wrapper docker images

Which issue(s) does this PR fix?:

https://jira.d2iq.com/browse/D2IQ-90153

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Manually tested image creation. Teamcity need to setup with GCP credentials to run it with automation.
	PROVIDER=gcp \
	ADDITIONAL_OVERRIDES= \
	ADDITIONAL_ARGS="" \
	IMAGE=images/gcp/centos-79.yaml \
	VERBOSITY=0 \
	BUILD_DRY_RUN=true
make[2]: Entering directory '/kib'
make/images.mk:8: Warning: BUILD_DRY_RUN is true
Executing target: bin/konvoy-image
go build \
	-ldflags="-X github.com/mesosphere/konvoy-image-builder/pkg/version.version=v1.16.0-4-gc2b0a1cd5ac6" \
	-o ./bin/konvoy-image ./cmd/konvoy-image/main.go
./bin/konvoy-image build gcp images/gcp/centos-79.yaml \
--dry-run=true \
-v 0 \
 \
 \

2022/06/01 20:18:32 writing new packer configuration to work/centos-7-1654114712-dfdOy
2022/06/01 20:18:32 starting packer build
googlecompute: output will be in this color.

==> googlecompute: Checking image does not exist...
==> googlecompute: Creating temporary RSA SSH key for instance...
==> googlecompute: Using image: centos-7-v20220519
==> googlecompute: Creating instance...
    googlecompute: Loading zone: us-west1-a
    googlecompute: Loading machine type: n1-standard-1
    googlecompute: Requesting instance creation...
    googlecompute: Waiting for creation operation to complete...
    googlecompute: Instance has been created!
    googlecompute: Waiting 20s before adding SSH keys...
==> googlecompute: Waiting for the instance to become running...
    googlecompute: IP: <IP_ADDRESS>
==> googlecompute: Using SSH communicator to connect: <IP_ADDRESS>
==> googlecompute: Waiting for SSH to become available...
==> googlecompute: Connected to SSH!
==> googlecompute: Provisioning with shell script: /tmp/packer-shell2217156501
...
...

So i understand the dev-kit pieces. Can we also have either a make entry to download the plugin for us or have it installed ourselves locally for dev?

I have not been able to use KIB without devkit container when testing locally. I think consistent approach would be to test it through devkit container only. for ex: we install and use goss and some ansible plugins in devkit only.

I dont agree necessarily but I understand. Im ok with merging this, my thoughts on these situations is that if we ask customers to use a wrapper, maybe we should be testing this way too. Anyway lgtm

I dont agree necessarily but I understand. Im ok with merging this, my thoughts on these situations is that if we ask customers to use a wrapper, maybe we should be testing this way too. Anyway lgtm

Totally agree. We are shipping this as a wrapper by default, so the testing and shipping are the same.

I dont agree necessarily but I understand. Im ok with merging this, my thoughts on these situations is that if we ask customers to use a wrapper, maybe we should be testing this way too. Anyway lgtm.

The wrapper is covered by https://github.com/mesosphere/konvoy-image-builder/pull/373/files#diff-dd2c0eb6ea5cfc6c4bd4eac30934e2d5746747af48fef6da689e85b752f39557R29 .
+1. we should be using wrapper in our e2e test.


Link to strategy description

As a user, when adding strategies to a tournament it would be useful to have a link to something that explains how the strategy behaves.
Something similar to the 'Who are they?' button on http://thefightclub.herokuapp.com/ might be an idea or, alternatively, some kind of hover popup.
The content could be derived from docstrings in the strategy source code.

I like the hover popup idea but with an indication on the site somewhere to the documentation for axelrod that I'm working on here: https://github.com/Axelrod-Python/Axelrod/issues/199


Add data.tar.gz file containing all data #17

Make the data available for the user when they install the package. (tar.gz file in data)
Solving issue #17

Hi @TomMB45  Thanks for updating this file! The size might be too large (~6MB). May you please push a smaller version of it i.e. max 1MB? Thank you,


Sign Bank - examples from both ASLLVD (citation-form signs) and our continuous signing datasets

Just recording this for the record:

The isolated sign data should appear at the top of the list, for each sign, labeled as such: "Citation-form examples", followed by the examples from the continuous signing datasets: "Examples from continuous signing."

For the continuous signing data, the button "Play composite video" should instead read "Play utterance."

The counts are off for the continuous signing examples.  It seems that when we remove a file from the DAI, its contributions to the DAI fail to be removed.  So that when they are reuploaded, there are duplications in the counts and the examples.

Fixed


Error thrown when creating event with :on-event-create and :dragToCreateEvent="false"

    <vue-cal
      :events="events"

      class="week-planning-calendar vuecal--full-height-delete"

      :editable-events="{ title: false, drag: true, resize: true, delete: true, create: true }"
      :dragToCreateEvent="false"

      :on-event-create="onEventCreated"
      />

      with method
          methods: {
  onEventCreated(event)
  {
    console.log("created an event : "+JSON.stringify(event));

    this.events.push(event)
  }
}

      produces the error message :
      Cannot destructure property '_eid' of 'this.utils.event.createAnEvent(...)' as it is undefined.

TypeError: Cannot destructure property '_eid' of 'this.utils.event.createAnEvent(...)' as it is undefined.
at eval (webpack-internal:///./node_modules/vue-cal/dist/vue-cal.es.js:947:21)
when you try and create an event... The event is still created and the error is thrown outside my function, so i cannnot catch it.

i hadn't understood how things worked.
onEventCreated(event)
{
console.log("created an event : "+JSON.stringify(event));
this.events.push(event)

return event

}
is how things should be done


Added set(value). Fixes #249.

Closes #249. Uses code from @confiks in https://github.com/facebook/immutable-js/pull/250 comment..

@geirsagberg This can be refactored to use standard API:
IndexedCursorPrototype.set =
KeyedCursorPrototype.set = function(key, value) {
    return arguments.length === 1 this.setIn([], key) ? this.setIn([key], value);
}

@Dashed tried your version, test fails with:
Cursor › it can set value of a cursor directly
  - TypeError: Object 1 has no method 'updateIn'
        at /Users/geirsagberg/Projects/immutable-js/contrib/cursor/index.js:148:14
        at updateInDeepMap (/Users/geirsagberg/Projects/immutable-js/dist/immutable.js:2500:22)
        at updateInDeepMap (/Users/geirsagberg/Projects/immutable-js/dist/immutable.js:2509:23)
        at src_Map__Map.updateIn (/Users/geirsagberg/Projects/immutable-js/dist/immutable.js:1816:26)
        at updateCursor (/Users/geirsagberg/Projects/immutable-js/contrib/cursor/index.js:307:38)
        at KeyedCursor.IndexedCursorPrototype.updateIn.KeyedCursorPrototype.updateIn (/Users/geirsagberg/Projects/immutable-js/contrib/cursor/index.js:147:10)
        at KeyedCursor.src_Map__Map.setIn (/Users/geirsagberg/Projects/immutable-js/dist/immutable.js:1794:19)
        at KeyedCursor.IndexedCursorPrototype.set.KeyedCursorPrototype.set (/Users/geirsagberg/Projects/immutable-js/contrib/cursor/index.js:89:40)
        at Spec.<anonymous> (/Users/geirsagberg/Projects/immutable-js/contrib/cursor/__tests__/Cursor.ts:247:20)
        at jasmine.Block.execute (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:1065:17)
        at jasmine.Queue.next_ (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2098:31)
        at jasmine.Queue.start (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2051:8)
        at Spec.jasmine.Spec.execute (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2378:14)
        at jasmine.Queue.next_ (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2098:31)
        at onComplete (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2094:18)
        at Spec.jasmine.Spec.finish (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2352:5)
        at null.onComplete (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2379:10)
        at jasmine.Queue.next_ (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2108:14)
        at null._onTimeout (/Users/geirsagberg/Projects/immutable-js/node_modules/jest-cli/vendor/jasmine/jasmine-1.3.0.js:2088:18)
        at Timer.listOnTimeout [as ontimeout] (timers.js:121:15)```

Ah I see. Nevermind then; I didn't actually test my suggestion.
Your PR works much better. :+1:

I've used cursors a little bit more extensively ever since I've suggested https://github.com/facebook/immutable-js/issues/249.
I think Cursor.set(value: any): Cursor doesn't seem to be consistent with the the rest of the API of the immutable library; especially since Cursor is to be a transparent type.
I think Cursor.update(updater) is a much better approach.

All I know is that it feels really unnecessary to do cursor.update(function() { return newValue; }) if cursor.set(newValue) would do the same thing. cursor.setIn([], newValue) is slightly better, but is not more intuitive nor easier to read, IMHO.
Can't we add .set(value) just for the sugar?

Or at least point out a scenario where .set(value) would cause undesirable effects, because I can't think of any off the top of my head.

@geirsagberg I totally understand the need; and there isn't anything wrong with the implementation of Cursor.set(value) as it is done through this PR.
I'm merely respecting the author's intent of Cursors as he has stated here: https://github.com/facebook/immutable-js/issues/242#issuecomment-67198651
Which to me, means the Cursor API should be consistent with the other Immutable collections.

This is reasonable, I understand that it's something people want to use Cursors to do.


Very bad performance when run inside pod

Hello!
I'm using the AWSCodeBuild plugin in a Kubernetes Cluster with Kubernetes plugin (https://github.com/jenkinsci/kubernetes-plugin), when i allocate a pod to run my pipeline/awscodebuild command the perfomance in zip action is very bad, normally my zip takes ~10seconds to complete but in this case the same action takes ~6minutes.
If i connect to a container and run the zip command manually the performance is normally, the "problem" only happen inside plugin.
My Jenkinsfile: https://pastebin.com/yCGMeA2R
There are could help me please?
Regards

Someone ?

Sorry to hear you're seeing poor performance in this use case. I will research the root cause, but a possible workaround could be the solution to this issue, which is to allow users to specify the path of a zip file to upload for the build. This way you could zip the source yourself in a step before the build. This feature is still pending, but I will update this issue when further progress is made.

I've seen this also, but didn't realize it was the zip operation that was taking so long.  I've seen this on ECS (EC2 launch type) slaves, and Fargate slaves now.  Also, the operation maxes out the CPU on the Jenkins master at the same time, making all jobs, even ones running on master, take twice as long.  And the job on the slave takes over 30 minutes to start (usually takes 10-20 seconds on master).
@xdexter were you able to workaround the issue by zipping the source yourself?

Hi @Petezah
I got it, my temporary workaround was ran the awscodebuild command on master node (and no in slave). Yes, i did the same test (zip manually on slave container and all works normally).
Regards

Until the feature I mentioned is implemented, a workaround could be to zip and upload your source to S3 manually, then use that source to run the build on CodeBuild. Not ideal, but we plan to make this use case easier in the future.

I believe the issue is that when a slave is running this step, the Master is trying to zip files that exist on the Slave--ie. the zip file is being built on the Master while files are being read off of the slave.  This is potentially really slow over IPC.  The solution would be to run the whole process on the Slave, and send the whole zip file to the master if necessary.
I'm seeing if I can plumb up a solution, hopefully.

No, sorry, I misspoke; the files being zipped are on the Slave--they are streamed over IPC to the Master and zipped, and streamed back over IPC into the zip file, which also exists on the Slave.  So the zipping action occurs on the Master process, but the files all exist on the Slave.  The reason the Zip utility step, mentioned above, is faster is because the whole thing happens inside a MasterToSlave callable.

I have tested this PR on a local instance + one of my remote slaves, and it fixes the issue for me.

@Petezah It's looks good! @leobaran-aws It's possible merge this PR to master? To be possible i install on my Jenkins?

Fixed by 69244e36be3cc363b41c2ba0fb2c45cd840893fc, which will be released in version 0.28 this week.

Thank you!


Wrong error message when login to OCI registry interactively.

When using a default registry with
export FUNC_REGISTRY=docker.io/invaliduser

and then do a func deploy without being logged in you get:
10:10 ➜ kn func deploy
function up-to-date. Force rebuild with --build
Please provide credentials for image registry (index.docker.io).
? Username: rhuss
? Password: ************************************
Incorrect credentials, please try again.
? Username:

although the password is correct (verified with podman login). The reason is probably that for the authentication still invaliduser is used.
I suggest to not asked for the username here, but just reuse the configured one (either via env or via cli option), and just print this out to the user when asking for the password.

@rhuss

The reason is probably that for the authentication still invaliduser is used

I do not think so. I believe the correct username is used, but the user is not authorized on the specific namespace; i.e authentication succeeds but authorization does not.

I suggest to not asked for the username here

I am not sure what you mean.
IMO we should ask for username. Imagine you want to push image into some orgnatization.
export FUNC_REGISTRY=docker.io/our-org

You still want to user your personal login (e.g. my-user-name) which belongs to the org, right?
However I agree the error messages could be better.

You could at least repeat the username or org for which authentication is done. And I don't think it's always about authorization, as the invalid user does not exist at Docker Hub, including much more context around the error. Of course, the user or orga could exist, too, so it's then about authorization. But still, it's then not an invalid credentials error, that clearly indicates an authentication error (it would be no access kind of message for authorization errors).
At least that error took me some hours to figure out that I had a typo in the env's repo name. Yes, there was a warning, but it fades out quickly.

And I don't think it's always about authorization, as the invalid user does not exist at Docker Hub

Yes, but I suspect that API would still return authorization error not 404, so I am not sure if we would be able to differentiate. Some APIs return authorization error even for non-existent entries for sake of security or something.
Again I agree the error should be better.

@gauron99 would you look into this?

Basically CheckAuth() have to return two distinct errors; authentication error and authorization error.
Some time ago the function was checking authentication, but then in #1130 we switched and the function directly checked authorization on given image.
The function should check authentication first, then authorization and return appropriate error if one of these fails.
The user should be then informed about what failed.

If you would work on this, @gauron99 , make sure it works against ghcr.io,docker.io,quay.io and gcr.io. Also please do not forget adding tests.

thanks Matej, will take a look

This also my required signature change of CredentialsCallback -- it might need additional parameter -- previous error.

I think the most important part is to add more context to the error message, like the image name for which you want to authenticate. Whether its an authorization, authentication or user-not-known error might be nice to have.


{FAQ} Automatic handling of scaled / stretched resolution.

It said in readme.md automatic handling scaled resolution, what does it mean? Does it mean resolution like 480p 720p is changing dynamically or what? Can you tell me detail about it.

For example, if I play CS:GO on a 1080 p monitor using scaled full-screen, 4:3, 1280x960 stretched resolution, the output of DxCam capture will be 1280x960, in the original output quality without the scaled effect.


[Feature] Support bypassing network transport data when two partitions in the same worker

Feature Description (功能描述)
Support bypassing network transport data when two partitions in the same worker

Hi, @coderzc , do you mean if the target partition is maintained by sender worker itself and we should avoid such memory copies?

Yes, if two partitions in the same process, we should avoid unnecessary network transport


Add empty state edge case handling to all existing widgets

Description
Adds an empty state display to all widgets. This is for cases when the data request is successful, but the system simply does not have any data for the selected widget. e.g. The selected scope is too narrow, all images are very up to date, etc.
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
TODO(replace-me)

Current dependencies on/for this PR:

master

PR #2138

PR #2166
PR #2190
PR #2204

PR #2238

PR #2255   👈

This comment was auto-generated by Graphite.


[BUG] AttributeError: 'PatchEmbed' object has no attribute 'dynamic_img_pad'

I am using "beitv2_base_patch16_224", "beitv2_large_patch16_224".
I used it as usual, but I get an error. AttributeError: 'PatchEmbed' object has no attribute 'dynamic_img_pad'
Please fix the bug.

@byunsunyoung you'll have to specify what 'using it as usual' means as it was tested and works fine for me, are you serializing the full model instead of the state dict in saved checkpoints? you cannot do that across versions reliably.

I've been using the beit model since 2 weeks ago, and I've been getting errors since I restarted the server today.
I guess there was a side effect of the August 28 update.
I am using timm==0.9.5. Can you check the code that uses the "dynamic_img_pad" parameter in the beit model?

For your information, I succeeded "create_model" method, and this error occurs during fine-tuning process using pytorch.

Cell In[45], line 3
1 lr = 0.001
2 epochs = 30
----> 3 train_model(model, dataloaders, criterion, optimizer, num_epochs=epochs)
Cell In[44], line 37, in train_model(model, dataloaders, criterion, optimizer, num_epochs)
33 optimizer.zero_grad()
35 with torch.set_grad_enabled(phase == 'train'):
---> 37     outputs = model(inputs)
38     loss = criterion(outputs, labels)
40     _, preds = torch.max(outputs, 1)
File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1501, in Module._call_impl(self, *args, **kwargs)
1496 # If we don't have any hooks, we want to skip the rest of the logic in
1497 # this function, and just call forward.
1498 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1499         or _global_backward_pre_hooks or _global_backward_hooks
1500         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501     return forward_call(*args, **kwargs)
1502 # Do not call functions when jit is used
1503 full_backward_hooks, non_full_backward_hooks = [], []
File /opt/conda/lib/python3.8/site-packages/timm/models/beit.py:427, in Beit.forward(self, x)
426 def forward(self, x):
--> 427     x = self.forward_features(x)
428     x = self.forward_head(x)
429     return x
File /opt/conda/lib/python3.8/site-packages/timm/models/beit.py:404, in Beit.forward_features(self, x)
403 def forward_features(self, x):
--> 404     x = self.patch_embed(x)
405     x = torch.cat((self.cls_token.expand(x.shape[0], -1, -1), x), dim=1)
406     if self.pos_embed is not None:
File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1501, in Module._call_impl(self, *args, **kwargs)
1496 # If we don't have any hooks, we want to skip the rest of the logic in
1497 # this function, and just call forward.
1498 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1499         or _global_backward_pre_hooks or _global_backward_hooks
1500         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501     return forward_call(*args, **kwargs)
1502 # Do not call functions when jit is used
1503 full_backward_hooks, non_full_backward_hooks = [], []
File /opt/conda/lib/python3.8/site-packages/timm/layers/patch_embed.py:83, in PatchEmbed.forward(self, x)
75         _assert(
76             H % self.patch_size[0] == 0,
77             f"Input height ({H}) should be divisible by patch size ({self.patch_size[0]})."
78         )
79         _assert(
80             W % self.patch_size[1] == 0,
81             f"Input width ({W}) should be divisible by patch size ({self.patch_size[1]})."
82         )
---> 83 if self.dynamic_img_pad:
84     pad_h = (self.patch_size[0] - H % self.patch_size[0]) % self.patch_size[0]
85     pad_w = (self.patch_size[1] - W % self.patch_size[1]) % self.patch_size[1]
File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1614, in Module.getattr(self, name)
1612     if name in modules:
1613         return modules[name]
-> 1614 raise AttributeError("'{}' object has no attribute '{}'".format(
1615     type(self).name, name))
AttributeError: 'PatchEmbed' object has no attribute 'dynamic_img_pad'

That attribute was added in 0.9.6, but it's disabled by default and wouldn't intefere with training of beit (it's not currently used by beit). I can fine-tune that model w/o issues, see example below...
Are you sure your environment is in a good state? You don't have conflicting timm versions, this error should only happen if the model has the attributes of a previous version of timm, but is using the code for the new version somehow. Usually happens when people save their model checkpoints by doing torch.save(model) instead of torch.save(model.state_dict()) and then restore across changes in the underlying code...
python train.py --amp --amp-dtype bfloat16 --data-dir /data/standford-dogs/Images/ --num-classes 120 --model beitv2_base_patch16_224 --pretrained
Training with a single process on 1 device (cuda:0).
Loading pretrained weights from Hugging Face hub (timm/beitv2_base_patch16_224.in1k_ft_in22k_in1k)
[timm/beitv2_base_patch16_224.in1k_ft_in22k_in1k] Safe alternative available for 'pytorch_model.bin' (as 'model.safetensors'). Loading weights using safetensors.
Model beitv2_base_patch16_224 created, param count:85854264
Data processing configuration for current model + dataset:
	input_size: (3, 224, 224)
	interpolation: bicubic
	mean: (0.485, 0.456, 0.406)
	std: (0.229, 0.224, 0.225)
	crop_pct: 0.9
	crop_mode: center
Learning rate (0.05) calculated from base learning rate (0.1) and effective global batch size (128) with linear scaling.
Using native Torch AMP. Training in mixed precision.
Scheduled epochs: 300. LR stepped per epoch.
Train: 0 [   0/160 (  0%)]  Loss: 4.79 (4.79)  Time: 1.359s,   94.18/s  (1.359s,   94.18/s)  LR: 1.000e-05  Data: 0.482 (0.482)

I was using it well before without serialization. I tried serializing as you advised and it worked well without errors.
Thank you!

@byunsunyoung I'd say that torch.save(model) IS serializing, as in it is pickling the full model class and all attributes, so when it restores to a different version of code the attributes may no longer match. torch.save(model.state_dict()) is just pickling the state dict, so only the parameters and buffers, any other python class attributes are not saved, so as long as parameter/buffer names and shapes don't change when the code does compatibility is maintained.


PytaskIO incorrect import path

Currently to install PyTaskIO class we need to import like
from pytask_io.pytask_io import PyTaskIO

But we should have:
from pytask_io import PyTaskIO

This issue will also remove the requirements.txt


Improve detection of transport closing

After https://bugs.python.org/issue31491 / https://bugs.python.org/issue32391, we must be able to simplify websockets, however not before everyone is on Python 3.7 (or 3.8), so no hurry :-)

Actually the code that called is_closing() no longer exists because it wasn't necessary on Python ≥ 3.6.


[Feature request] 啥时候增加支持大文件，1MB限制太低辣

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

https://github.com/JasonStein/Notepads/issues/378


[NEW] Handle setting to hide other users' email addresses from other users

Currently we're not showing any user e-mail.


Influx 0.9rc31 eat all server performance

Hi
We have a server that stores 9GB of data and eat memory with a 5.0 of load

Is it normal ?
How can i help you about fixing this issue ?

@dtouzeau can you tell us a bit more about what's going on with the server while this is happening? what sort of writes and queries are being performed?
both the read and write paths are being modified heavily for RC32. i'd like to see what this looks like once that version is released next week, but answers to the questions above would be very helpful.

Yeah! I'm monitoring latest versions each 10mn on your website!
Writes are million proxy requests with strings like IPADDRESSES (string), usernames long string , hits (integer) , size (integer), domain (string) ,subdomain
all are fields

Try to repro with RC33, especially with the line protocol. The JSON parsing costs are huge for that volume of writes. Reducing that load should help significantly.

RC31 had only JSON protocol and different datastore structure. Changes in 0.9.0 final and forward should alleviate this load issue.


Error on import Gist

Hi, when I try to import my Gist returns a error "SyntaxError: Unexpected Token".
I double check the token and it´s correct.
Follow the image of error.
Thank you amazing work.

Does it still import Gists - I'm having trouble? Set up access token, import Gists > enter URL appears to die silently

hi @WebSpectrum, this project is not maintained anymore


Adding -Wshadow=local as a default warning for warnings=moderate+ ?

 My question is whether the project will be interested in fixing these.
 That said, I'm not planning to work on that. Maybe we can close it?

Well even if we don't add -Wshadow=local to the warnings, I think it would be good to fix some of those warnings, so the issue can be kept.

Ok, if you see it profitable, then I can prepare patch for it.


"Contact Gallery" button appearing for works that are marked as uninquireable

See https://api.artsy.net/api/v1/artwork/amadeo-luciano-lorenzato-untitled-1 for example
https://www.artsy.net/artwork/amadeo-luciano-lorenzato-untitled-1

It appears from a chat message by @broskoski that this line is what’s causing the issue https://github.com/artsy/eigen/blob/24cce11c042f018ac2e7f0618c20b67b6a2780fb/Artsy/Views/Artwork/ARArtworkActionsView.m#L365.

Should this show anything else in lieu of the button?

Nvm I see the notForSaleLabel


Ansible INI module

A simple custom promise module wrapping the Ansible INI custom module, and simply passing on attributes from CFEngine into the Ansible module.
The one required attribute for the module is module_path which is the path to the Ansible INI module installed on the system.

For now I've added a cfbs.json file in the directory. However, I see all other modules are incorportated into the top-level cfbs.json. I'm guessing you want me to follow that example?

For now I've added a cfbs.json file in the directory. However, I see all other modules are incorportated into the top-level cfbs.json. I'm guessing you want me to follow that example?

@oleorhagen Yea. Either that, or you can also develop the module in your own repo, if you prefer.

@olehermanse feel free to have a last lookover


[Grid] Add "space-evenly" value for justify prop

The Grid component's justify prop doesn't accept the value "space-evenly" which is a valid value for justify-content.
(This is my first PR here and I have read the contributing docs, let me know if I've forgotten anything)

@iain-b I haven't updated the interactive demo either. I'm not sure we need a demo for this value. Thank you!


Bundling Error: code=-1, error:spawn ENOENT

Hi Aronuda,
I'm getting this error on deploying. I've run mup init and mup setup with no errors. It seems to have a problem bundling, but I can call meteor bundle bundle.tgz and it seems to work fine.
I fired up node-inspector with export NODE_OPTIONS='--debug-brk'; node-inspector & and message that states that it's listen pops up, but when I run mup deploy the message following outputs without node-inspector breaking.
Is there a good way to debug this?
Any thoughts on the error?
Thanks Marc
Lava:app cram$ mup deploy

Meteor Up: Production Quality Meteor Deployments
------------------------------------------------

Bundling Started: /Users/marc/Projects/METEOR/Sqwrl/app
Bundling Error: code=-1, error:spawn ENOENT
-------------------STDOUT-------------------

-------------------STDERR-------------------

I just had this issue as well. I had moved my local development folder to a new location and failed to update the mup.json settings.

can we make so that mup look up for current directory?


Watermarking Images with OpenCV in Python

Proposed title of article
Watermarking Images using OpenCV in Python.
Introduction paragraph (2-3 paragraphs):
Watermarks play an important part of businesses and online content credit making. They provide ownership or credits to the creator of objects in the digital world, preventing the content from being stolen or replicated without owner's consent. They may be present as a logo, signature, or stamp that is distinctive and unique to the creator. Making these watermarks is now easier than ever using OpenCV.
We will cover as much as possible in this article to make the reader understand OpenCV, a python library that allows us to do several operations like, tracking camera movements, extracting 3D models of any objects, tracking moving objects as well as creating text or image watermarks. The reader will learn how to use this python library to make image watermarks.
Key takeaways:
The reader will be able to:

Understand what OpenCV python library is.
How to use OpenCV to create image watermarks either textual or image.
How to define functions to add an image.
How to adjust the position of the text to suit the content.

References:
N/A
Templates to use as guides
N/A

Good afternoon and thank you for submitting your topic suggestion. @dentonya
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

How will this article differ from what is published our other blog sites

The article will focus more on how to add a single watermark to multiple images automatically at once  instead of watermarking each image manually.
@hectorkambow

Okay, automation is a great idea. But please make sure you build a unique project. @dentonya
Let's please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it).
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting it for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double-check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved

Thanks for the feedback I will for sure @ahmadmardeni1


fix: duplicated styles

Thanks for creating this library!
I found some duplicated styles when browsing through the repo

Thank you!


Hello_Services sample not working with Atom

Steps to Reproduce

Added the hello_services sample project to Atom.
When I try to run, it shows the error "No application found for TargetPlatform.android_arm.
Is your project missing an android/AndroidManifest.xml?".

Flutter Doctor
[✓] Flutter (on Linux, channel master)
• Flutter at /usr/local/google/home/quddusc/flutter
• Framework revision 5af67e1592 (52 minutes ago), engine revision 034ae44419
[✓] Android toolchain - develop for Android devices (Android SDK 24.0.0-preview)
• Android SDK at /usr/local/google/home/quddusc/Android/Sdk
• Platform android-N, build-tools 24.0.0-preview
• OpenJDK Runtime Environment (build 1.8.0_45-internal-b14)
[✓] Atom - a lightweight development environment for Flutter
• flutter plugin version 0.2.2
• dartlang plugin version 0.6.24
Logs and Crash Reports
No application found for TargetPlatform.android_arm.
Is your project missing an android/AndroidManifest.xml?
exited with code 1

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

It looks like flutter_tools requires the file android/AndroidManifest.xml to exist, and the hello_services project has it at android/app/src/main/AndroidManifest.xml.

Yeah, hello_services is a demo of a separate workflow from the Atom / flutter-tool workflow.  I'm working to integrate the two, but they aren't integrated yet.
The README.md file in hello_serves explains how to build and run the example.

For the codelab, we'd like the devs to start with a hello_services style project so that they can add sign-in and Firebase. This bug will prevent them from working in Atom in the earlier steps.

I ran through the readme.md to get a sense of what the hello_services workflow looks like, and what it would take to drive that style of project from flutter run. Here are some notes and issues I ran into:

we need to be able to locate gradle (via an existing PATH or user configuration)
the gradle script that I had installed (in my copy of Android Studio) did not have its executable bit set
when I ran gradle wrapper, the generated gradlew script did not have its executable bit set
the project's gradle config specifies version 2.10 - I have 2.2.1 installed
the project's android/app/build.gradle file specifies android sdk v22/22.0.1, I have v23/23.0.2 installed
we need to inject or verify if the flutter.sdk value exists in local.properties
the apk gets built into android/app/build/outputs/apk/app-debug.apk
installing the app via adb worked
you need to pass the activity id into flutter refresh - we could instead recognize the project type and look in a different place for the android manifest file

This works on iOS now.  @devoncarew is going to look at the Android aspects of this issue.


Fixed the issue of nav bar links ...

Related Issue

Information about the related issue

Closes: #
Describe the changes you've made

Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

Original
Updated

original screenshot
updated screenshot

Hey @Vansh-marda can you explain which issue this PR solve?  I think you are a beginner in Open Source. I'll encourage you to join our community and ask your doubts you have in open source, our community will help you there 😄
You can join our GitHub org by raising an issue here - https://github.com/web3community/start-here/issues/new?assignees=&labels=github-invitation&template=invitation.yml&title=Please+invite+me+to+the+community ✨
You can join our discord server, you'll get to interact with some awesome folks there - https://discord.gg/uVzRRUDZ 🤓 ✨
Can't wait to see you there 💛 ✨


Is it possible to have a frozen footer

Expected behavior and actual behavior.
looking for a way to have a total row total row that allows the body to scroll? if not how hard would it be to wire into the scroll event to fetch the viewable columns and render then to a secondary div under the grid
Steps to reproduce the problem.
Specifications like the version of the project, operating system, or hardware.

I imagine a single row footer wouldn't be a terrible feature.  It won't get done anytime soon, there's mountains of work before it, but I'll add it to the to-do list.  The second idea you had is very easy to do:
window.addEventListener('load', function () {
    var parentNode = document.body;
    var footer = document.createElement('div');
    var grid = canvasDatagrid({
        parentNode: parentNode
    });
    function updateFooter() {
        footer.innerHTML =
        	'Col 2 Max: ' +
        	Math.max.apply(null, grid.data.map(function (row) { return row.col2; }))
          + ' Col 2 Avg: ' +
          (grid.data.map(function (row) { return row.col2; }).reduce(function (a, v) { return a + v; }, 0) / grid.data.length) + '';
    }
    grid.addEventListener('datachanged', updateFooter);
    setInterval(function () {
      grid.data = [
        {col1: 'foo', col2: Math.random(), col3: 'a'},
        {col1: 'bar', col2: Math.random(), col3: 'b'},
        {col1: 'baz', col2: Math.random(), col3: 'c'}
      ];
    }, 100);
    parentNode.appendChild(footer);
});```

Note that the event `datachanged` will not fire when the data object is directly manipulated.  You'll have to run the `updateFooter` function yourself if you manipulate the data that way.  You'll also need to call `grid.draw` as well.  The reason is the grid cannot listen for changes to the elements of the array, just the setter `grid.data`.

make that
http://jsfiddle.net/qr1cythe/18/
and your example as well
http://jsfiddle.net/25ngbszd/12/
On Mon, Jul 16, 2018 at 8:28 PM, Richard Davison<EMAIL_ADDRESS>wrote:

thanks for the input
check out what i have figured out already
http://jsfiddle.net/qr1cythe/16/
On Mon, Jul 16, 2018 at 7:52 PM, Tony Germaneri<EMAIL_ADDRESS>wrote:

I imagine a single row footer wouldn't be a terrible feature. It won't
get done anytime soon, there's mountains of work before it, but I'll add it
to the to-do list. The second idea you had is very easy to do:
window.addEventListener('load', function () {
var parentNode = document.body;
var footer = document.createElement('div');
var grid = canvasDatagrid({
parentNode: parentNode
});
function updateFooter() {
footer.innerHTML =
'Col 2 Max: ' +
Math.max.apply(null, grid.data.map(function (row) { return row.col2; }))
+ ' Col 2 Avg: ' +
(grid.data.map(function (row) { return row.col2; }).reduce(function (a, v) { return a + v; }, 0) / grid.data.length) + '';
}
grid.addEventListener('datachanged', updateFooter);
setInterval(function () {
grid.data = [
{col1: 'foo', col2: Math.random(), col3: 'a'},
{col1: 'bar', col2: Math.random(), col3: 'b'},
{col1: 'baz', col2: Math.random(), col3: 'c'}
];
}, 100);
parentNode.appendChild(footer);
});```
Note that the event datachanged will not fire when the data object is directly manipulated.  You'll have to run the updateFooter function yourself if you manipulate the data that way.  You'll also need to call grid.draw as well.  The reason is the grid cannot listen for changes to the elements of the array, just the setter grid.data.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TonyGermaneri/canvas-datagrid/issues/137#issuecomment-405424752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFzyluFNskf_89obtOlBLneFFj4xwzeMks5uHTU-gaJpZM4VQeTi
.

Nice work!  I never thought I'd see a virtual scroll bar controlling a real scroll bar, but... there it is.  If you want it to fire more frequently remove the debounce code, that's there to prevent over-use of XHR resources.  Since you're just updating a DIV you can fire it off as frequently as you like.

Thanks.
Insomnia is great epiphany time -- use  a grid with the headers turned off
as the footer and syncronize those scrolls.. problem is you need to be able
to hide one grids scroll so it can control the other.... reading docs.
Thinking in the 33rd dimension.
RD
On Tue, Jul 17, 2018 at 2:55 AM, Tony Germaneri<EMAIL_ADDRESS>wrote:

Nice work! I never thought I'd see a virtual scroll bar controlling a real
scroll bar, but... there it is. If you want it to fire more frequently
remove the debounce code, that's there to prevent over-use of XHR
resources. Since you're just updating a DIV you can fire it off as
frequently as you like.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TonyGermaneri/canvas-datagrid/issues/137#issuecomment-405493850,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFzylkAXyUgnRA07e0YyGXl8SOoJvnxSks5uHZiDgaJpZM4VQeTi
.

There is no way to turn it off, but you can alter the dimensions so the height or width is 0 which should effectively turn it off.  I'm like 72% sure it will work like that.

appears to turn them both off I only want to turn off the horizontal
scrollbar.
Ideas?
On Tue, Jul 17, 2018 at 11:50 AM, Tony Germaneri<EMAIL_ADDRESS>wrote:

There is no way to turn it off, but you can alter the dimensions so the
height or width is 0 which should effectively turn it off. I'm like 72%
sure it will work like that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TonyGermaneri/canvas-datagrid/issues/137#issuecomment-405651138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFzylgHV0Q1luYX_o5UNnr8Pa1nGUXCxks5uHhXwgaJpZM4VQeTi
.

This works !! and is exactly what I was pondering last night.
Except that  the vertical scroll is turned off as well?
http://jsfiddle.net/qr1cythe/43/
RD
On Tue, Jul 17, 2018 at 1:07 PM, Richard Davison<EMAIL_ADDRESS>wrote:

appears to turn them both off I only want to turn off the horizontal
scrollbar.
Ideas?
On Tue, Jul 17, 2018 at 11:50 AM, Tony Germaneri<EMAIL_ADDRESS>
wrote:

There is no way to turn it off, but you can alter the dimensions so the
height or width is 0 which should effectively turn it off. I'm like 72%
sure it will work like that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TonyGermaneri/canvas-datagrid/issues/137#issuecomment-405651138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFzylgHV0Q1luYX_o5UNnr8Pa1nGUXCxks5uHhXwgaJpZM4VQeTi
.

I know it's been a while, but is your issue resolved?   If not please open the issue again.  FYI I've added overflowX and overflowY since we had this conversation.  Seems related.


Enable nullsafety

I am trying to use your package as part of an app which I am trying to migrate to nullsafety. Right now, the dependency on freezed-annotations 0.12 is preventing this from being successful. Is there a chance you could go to freezed-annotations 0.13?

I try to keep with the most stable versions of all packages when possible. And currently freezed 13 is still in prerelease. Once it's released in a stable version, I'll try and upgrade.

I try to keep with the most stable versions of all packages when possible. And currently freezed 13 is still in prerelease. Once it's released in a stable version, I'll try and upgrade.


Routing via interface

This PR allows to define routing for a super-interface of messages (see #1037).
It also addresses deprecations recently introduced by Truth.

@armiol, PTAL.

@armiol, PTAL.


M3-2033 Styling for "Total Traffic" stats

M3-2033 Update Styling for "Total Traffic" stats
Description
Update to Total Traffic section for Stats & Graphs
Type of Change

Non breaking change ('update', 'change')

Coverage remained the same at 31.876% when pulling 5e4cf37ae2485323eb6a4dfdc6c6f61d799a2e88 on WilkinsKa1:M3-2033 into 9143aaa435224ea0179bbca4cbb15cea941dc353 on linode:release-0.42.0.


[TC-LC-2.2]User label cluster- LabelList attribute fails to list the entries

Problem
As per Test-Plan, LabelList attribute should  list the entries, currently chip-tool fails to list entries

expected behavior:- LabelList should list minimum 4 list entries

actual behavior:-0 entries

steps to reproduce:-  ./chip-tool userlabel read label-list 1 0

PFA log below:-
[TC-LC-2.2].txt

Issue is resolved in latest commit ID.


spec: a possible inconsistency for promoted methods

What version of Go are you using (go version)?

go version go1.13 linux/amd64

Does this issue reproduce with the latest release?
true
What did you do?
package main

type S struct{}

func (*S) F() {}

type T struct {
	*T
	*S
}

func (*T) M() {}

func main(){
	T{&T{}, &S{}}.F()   // no problem
	T{&T{}, &S{}}.S.F() // no problem
	T{&T{}, &S{}}.M()   // cannot take the address of T literal
	T{&T{}, &S{}}.T.M() // no problem
}

What did you expect to see?
All lines should compile.
What did you see instead?
One line fails to compile.

En, it looks the implementation is correct, it is consistent with reflection:
package main

import (
	"fmt"
	"reflect"
)

type S struct{}

func (*S) F() {}

type T struct {
	*T
	*S
}

func (*T) M() {}

func main(){
	T{}.F()   // no problem
	T{}.S.F() // no problem
	//T{}.M()   // cannot take the address of T literal
	T{}.T.M() // no problem

	_ = T{}.F   // no problem
	//_ = T{}.M // cannot take the address of T literal

	// <invalid reflect.Value>
	fmt.Println(reflect.ValueOf(T{}).MethodByName("M"))
	// {  <nil> <invalid Value> 0} false
	fmt.Println(reflect.TypeOf(T{}).MethodByName("M"))
	// 1
	fmt.Println(reflect.TypeOf(T{}).NumMethod())
	// {F  func(main.T) <func(main.T) Value> 0}
	fmt.Println(reflect.TypeOf(T{}).Method(0))

	_, ok := (interface{}(T{})).(interface{M()})
	fmt.Println(ok) // false
}

Maybe, a small adjustment is needed in spec?

If S contains an embedded field T, the method sets of S and *S both include promoted methods with receiver T. The method set of *S also includes promoted methods with receiver *T.

In the final example I think you are suggesting that S should inherit the method M from the embedded field T, even though there is a method M on *S.  I think it's clear that the language is not intended to permit that.  It's possible that the spec does not clearly state that.  I think the place to state it would be https://golang.org/ref/spec#Selectors.
CC @griesemer

@ianlancetaylor Yes, I agree.
It looks the current design views S{}.M as a legal but invalid selector. However I haven't found how the spec distinguishes "legal" from "valid" for selectors.

I think this is a dup of #15708. In particular, I think this is my first example here.

I agree this is a dup of https://github.com/golang/go/issues/15708, so close it.
Maybe it is good to add a complement in the https://golang.org/ref/spec#Selectors section
For a primary expression x (always assume it is addressable) that is not a package name, the selector expression ....


add unique key for t_ds_process_definition

add unique key for t_ds_process_definition

Codecov Report

Merging #2669 into dev-1.3.0 will increase coverage by 0.54%.
The diff coverage is 66.12%.

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


Error with ChunkUnload Flag

Hi,
I receive the following console error when enabling the chunk unload option.
Console Error: https://pastebin.com/xRurB2Kn
Server Version: Paper 1.14.4
Plugin Version: 4.1.2
Regards,
Sam.

This was fixed in dc8cb84


Change max chars in workflow step to 1k

Change from 200 to 255(?)
Check validator rules
Update: 255 is added. Maybe change to 1k?

not too major issue, we can live with it. closing issue


Exercise 2.7 - Failing a test from pybryt but successful with the assert

Hi!
As seen in the screenshot, I was able to make a function that finds primes up to n (and pybryt agrees: "- SUCCESS: Your function returns the right primes."), but I can't get it to give me success on: "- ERROR: Hmm... It looks like you are not finding primes correctly."
How should I proceed? Is it something I should be spending time on?
Thanks :)

The assert also yields no errors!

Hi @edsml-77e2dd98, thank you for raising this issue. This is fine, and here is the explanation from README:
Although PyBryt provides much more detailed feedback, it is possible it complains about your code even though assert statement confirms your code is correct. This is because PyBryt testing is based on comparing your solution to reference solutions. Since there is virtually an infinite number of alternative solutions, it is possible that your solution is not in our references. On the other hand, assert statements check the final result of your code and if they do not raise an error, your code is correct and you do not have to address PyBryt's error messages (if any).


fix(perception_launch): fix apollo pipeline in detection module

Signed-off-by: yukke42<EMAIL_ADDRESS>PR Type

Bug Fix

Related Links
https://github.com/autowarefoundation/autoware.universe/pull/1336

Description
Same as https://github.com/autowarefoundation/autoware.universe/pull/1336.

Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

This PR is moved to https://github.com/tier4/autoware_launch/pull/412.

This PR is moved to https://github.com/tier4/autoware_launch/pull/412.


Ticking entities when making nested scale boxes

According to your testing, does the issue occur with ONLY MiniScaled mod and Immersive Portals mod and Farbic API?
Yes
Describe the issue
When I try to make a scale box inside another scale box inside another scale box ticking entities randomly appear
latest.log
crash-2024-01-04_13.20.55-client.txt
crash-2024-01-04_13.20.53-server.txt
Post the log here
No response

Probably fixed in 5.0.3


[fix][ci] Use correct annotation to prevent resource leaks in Elastic integration tests

Motivation
pulsar-io/elastic-search tests leak containers since the method initBeforeClass has the @BeforeMethod annotation when it should be @BeforeClass.
Modifications

rename @BeforeMethod -> @BeforeClass for the initBeforeClass methods
remove the testReuseFork=false workaround
add testcontainers-ryuk to ignored threads in ThreadLeakDetectorListener

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Codecov Report

Merging #21471 (185bd7a) into master (29db8f8) will increase coverage by 43.06%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #21471       +/-   ##
=============================================
+ Coverage     30.21%   73.27%   +43.06%
- Complexity      325    32613    +32288
=============================================
  Files          1711     1890      +179
  Lines        130786   140408     +9622
  Branches      14255    15430     +1175
=============================================
+ Hits          39516   102884    +63368
+ Misses        85324    29433    -55891
- Partials       5946     8091     +2145

Flag
Coverage Δ

inttests
24.29% <ø> (+0.10%)
:arrow_up:

systests
24.71% <ø> (-0.07%)
:arrow_down:

unittests
72.55% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1512 files with indirect coverage changes


Question about install

Hi, i appreciate your project
for install the rootkit
i need only the install.exe file or i need the dll's r77-x64 & r77-x86 on the same folder?

You only need Install.exe, it bundles all other required files, including both DLL's.


Header Block not accepting any custom logo

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #1219
Test URLs:

Before: https://main--moleculardevices--hlxsites.hlx.page/cellesce
After: https://brandlogo-fix--moleculardevices--hlxsites.hlx.page/cellesce

and

Before: https://main--moleculardevices--hlxsites.hlx.page
After: https://brandlogo-fix--moleculardevices--hlxsites.hlx.page

Hi @lydiapuric
Can we remove search and RFQ from the header and merge these changes?
Regards,
Rajneesh


Fix input variable to trigger macos test with testwasher

What does this PR do?
Triggering a pipeline on Github with True does not work, Github expect true as a string instead for boolean variables

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Output the resource URN next to the "Dismissed an error as retryable. marked as timeout"

I keep seeing these errors (only in verbose), but have very little context on which resource is causing them:
Dismissed an error as retryable. marked as timeout - Post "https://oauth2.googleapis.com/token": dial tcp: i/o timeout

Dismissed an error as retryable. marked as timeout - dial tcp: i/o timeout

These timeouts have plagued me and slowed down my CI btw.

I wonder if my underlying issue is related to this new error I'm seeing (while I destroy my stack):
error retrieving userinfo for your provider credentials. have you enabled the 'https://www.googleapis.com/auth/userinfo.email' scope? error: error retrieving userinfo for your provider credentials. have you enabled the 'https://www.googleapis.com/auth/userinfo.email' scope? error: Get "https://openidconnect.googleapis.com/v1/userinfo?alt=json": Post "https://oauth2.googleapis.com/token": dial tcp: i/o timeout

I believe this is a short coming of the retry semantics in the terraform GCP provider - https://github.com/hashicorp/terraform-provider-google/issues/7620. Perhaps you can try setting a higher requestTimeout e.g. 60s etc. here: https://www.pulumi.com/registry/packages/gcp/api-docs/provider/#requesttimeout_nodejs

Thank you @viveklak for that context.


Come up with better selling points in "Key features"

For example "Homoiconicity" could be perhaps replaced by something more attractive to developers. Initiated by discussion here: https://github.com/exercism/red/pull/17#discussion_r599312829

For homoiconicity, instead it could be "data oriented", or "reflective". Maybe "expressive", but that could be its own topic.
Could also mention:

both compiled (standalone executable) and interpreted
lots of built-in functionality

Temporarily I "homoiconicity" to "code is data" in #19 .

* both compiled (standalone executable) and interpreted
* lots of built-in functionality

Both are fine. I wonder what we could change it for, because there have to be exactly 6 features in the config. Maybe homoiconicity is the one to get rid of...

more info in updated exercism docs:
https://github.com/exercism/docs/commit/1be74f48bbdc0273138ec38b70f4a2cf69c2df75


Install script stops at "./launch-linux.sh: line 46: dotnet: command not found" when manually defining dotnet location

Issue: The install script fails to find the dotnet command and halts when using the modifications in the script file to specify a different install location for dotnet. Excerpt from the script file with the modifications in place:
# Note: manual installers that want to avoid home dir, add to both of the below lines: --install-dir $SCRIPT_DIR/.dotnet
./dotnet-install.sh --channel 8.0 --runtime aspnetcore --install-dir $SCRIPT_DIR/.dotnet
./dotnet-install.sh --channel 8.0 --install-dir $SCRIPT_DIR/.dotnet

The installer works fine when the location is not manually defined. I fully understand that this may be intended, but in that case the instructions could be made clearer to specify the requirements for using a manually specified location.
Expected behaviour: To find dotnet and continue installation.
OS details: Nobara 40 (based on Fedora 40), x86-64. SwarmUI master on latest commit https://github.com/mcmonkeyprojects/SwarmUI/commit/487bb198033018c8010e8f98667a12deb37c7651
Logs:
Cloning into 'SwarmUI'...
remote: Enumerating objects: 16754, done.
remote: Counting objects: 100% (16754/16754), done.
remote: Compressing objects: 100% (3647/3647), done.
remote: Total 16754 (delta 13155), reused 16621 (delta 13022), pack-reused 0
Receiving objects: 100% (16754/16754), 11.47 MiB | 1.40 MiB/s, done.
Resolving deltas: 100% (13155/13155), done.
rm: cannot remove 'dotnet-install.sh': No such file or directory
dotnet-install.sh    100% [=============================>]   61.71K  261.21KB/s
                          [Files: 1  Bytes: 61.71K [28.36]
dotnet-install: Attempting to download using aka.ms link https://dotnetcli.azureedge.net/dotnet/aspnetcore/Runtime/8.0.6/aspnetcore-runtime-8.0.6-linux-x64.tar.gz
dotnet-install: Remote file https://dotnetcli.azureedge.net/dotnet/aspnetcore/Runtime/8.0.6/aspnetcore-runtime-8.0.6-linux-x64.tar.gz size is 41925297 bytes.
dotnet-install: Extracting archive from https://dotnetcli.azureedge.net/dotnet/aspnetcore/Runtime/8.0.6/aspnetcore-runtime-8.0.6-linux-x64.tar.gz
dotnet-install: Downloaded file size is 41925297 bytes.
dotnet-install: The remote and local file sizes are equal.
dotnet-install: Installed version is 8.0.6
dotnet-install: Adding to current process PATH: `/home/niroku/Applications/SwarmUI/.dotnet`. Note: This change will be visible only when sourcing script.
dotnet-install: Note that the script does not resolve dependencies during installation.
dotnet-install: To check the list of dependencies, go to https://learn.microsoft.com/dotnet/core/install, select your operating system and check the "Dependencies" section.
dotnet-install: Installation finished successfully.
dotnet-install: Attempting to download using aka.ms link https://dotnetcli.azureedge.net/dotnet/Sdk/8.0.302/dotnet-sdk-8.0.302-linux-x64.tar.gz
dotnet-install: Remote file https://dotnetcli.azureedge.net/dotnet/Sdk/8.0.302/dotnet-sdk-8.0.302-linux-x64.tar.gz size is 222084318 bytes.
dotnet-install: Extracting archive from https://dotnetcli.azureedge.net/dotnet/Sdk/8.0.302/dotnet-sdk-8.0.302-linux-x64.tar.gz
dotnet-install: Downloaded file size is 222084318 bytes.
dotnet-install: The remote and local file sizes are equal.
dotnet-install: Installed version is 8.0.302
dotnet-install: Adding to current process PATH: `/home/niroku/Applications/SwarmUI/.dotnet`. Note: This change will be visible only when sourcing script.
dotnet-install: Note that the script does not resolve dependencies during installation.
dotnet-install: To check the list of dependencies, go to https://learn.microsoft.com/dotnet/core/install, select your operating system and check the "Dependencies" section.
dotnet-install: Installation finished successfully.
cat: src/bin/last_build: No such file or directory

WARNING: You did a git pull without building. Will now build for you...

mv: cannot stat './src/bin/live_release': No such file or directory
./launch-linux.sh: line 37: dotnet: command not found
./launch-linux.sh: line 39: src/bin/last_build: No such file or directory
./launch-linux.sh: line 46: dotnet: command not found

Oh - I see what happened, that was accidentally targeting the wrong path.
I pushed a commit that places it to the correct path, I think you'd need to re-do the install to have it work as intended.


Update libreoffice-fresh.json

Fix "https" error

This is not good way how to fix. As there is automatic load balancing and mirror finding, which will lead into http => https => http mirror and you have same problem
Download Manually or with aria2 it should work.

I agree with @Ash258. If the error is an expired cert, that’s an upstream error. A better solution would be to add a —no-check-certificate option. What, exactly, is the error.


Clear incorrect implementation of clone warning

Clippy emits:
error: incorrect implementation of clone on a Copy type
As suggested use *self instead of each individual field.

Every PR will need this to get green CI.
cc @apoelstra and @sanket1729 because they are the only two reviewers we have right now - no pressure :)

Gosh, look at all those other ones. Closing this and re-opening it on another branch name done properly.

BTW this lint is (usually) wrong. See discussion on https://github.com/rust-lang/rust-clippy/issues/11178 which has the same problem.

It's harmless to fix so we might as well do it, but it's a little annoying that they'd enable something by default which has such a high false positive rate.


since minimongo is in-memory data store.It has limitations on mobile device.solution?

is there any way to make minimongo or any alternate solution to make it persistent.
Something like websql?
Actually my datasets are very huge possibly it may more than 10GB..how to handle?

Some thinks to consider:
-Even webSQL doesn't allow very large databases (and it's deprecated).
-Other local storage solutions are also limited to a couple of tens of megs.
-Do you really need all of that data on the client at all time?
-Minimongo doesn't support indexes, so queries would be unacceptable slow anyway.
-Even on desktop, sending 10GB to the client will crash the browser.
So there's really no option without having a cordova build an querying a database through a native db plugin. And even then you'll likely run into problems with a 10GB database.

I think it's a good question but it should be posted on the Meteor forums as it's not Angular-Meteor specific - forums.meteor.com


[Suggestion] Add support for TwitchEmotes config

Describe the problem
Extending the current TwitchEmotes configuration to include our own emotes pack, along with detailed usage statistics, would be awesome!

Since we will make the addon independent in the future, support for this feature is no longer needed.


RDKTV-17862 Fixed order of ipv6 rules

Description
Previously the ICMPv6 rule was too low, making it not activated. Now the order is proper which means that communication should work fine.
Before the fix only the unicast address inside container was REACHABLE, as the router advertisement went from it, while all other ICMPv6 rules were dropped by the anti-spoofing rule due to them having "improper" address, By changing the the order of operations now we save all neighbor discovery frames.
Test Procedure

Start container with ipv6 networking on
Get bash inside container crun --root /run/rdk/crun exec --tty <container_name> /bin/bash
Check IP address inside container ifconfig
From outside container try to access local type IPv6 address (starts with fe80::) i.e. ping -c 1 fe80::dccf:8aff:fee1:923c
From outside container check neighbor table ip -6 neigh container should be in REACHABLE state
From inside container check neighbor table ip -6 neigh host should be in REACHABLE state

Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other (doesn't fit into the above categories - e.g. documentation updates)

Requires Bitbake Recipe changes?

[ ] The base Bitbake recipe (meta-rdk-ext/recipes-containers/dobby/dobby.bb) must be modified to support the changes in this PR (beyond updating SRC_REV)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Koczek9:x: Marcin KoczwaraMarcin Koczwara seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Some improper merges clattering the view, will re-upload this commit.


DOCS: Edits to README.md

Just mostly commas and wording and other edit-y stuff.

We really don't need to wait on tests for this.

Thanks, Daenny!
On Mon, Jun 15, 2015 at 12:23 PM Daniele Sluijters<EMAIL_ADDRESS>wrote:

Merged #533 https://github.com/puppetlabs/puppetlabs-apt/pull/533.
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppetlabs-apt/pull/533#event-331386640.


How can I migrate an old ES256 key file to the database?

I have a current key stored in the file format with the old alg ES256. Now the default is RSA. I'm trying to migrate from File store to the Database using EF but I wouldn't like my users to lose their current jwt.  Is there a way to migrate the old key file to the database? The new keys would be generated using the alg RSA.

Yes, but definetely will not a simple task. You need to inject FileSystemStore and IJsonWebKeyStore, then use FileSystemStore.GetCurrent() to get current Key, after that call IJsonWebKeyStore.Store().
Sure it helps


fix: travis.sh should exit with an error if any of the build steps fail.

Closes #542.
This was accomplished by adding the set -e flag documented
here.

Coverage remained the same at 95.46% when pulling 4266b20afd9f58f185fabc0363e8e4553a67ff15 on jamestalmage:fix-542 into bab99d9fc211379d59809185f03ef4d42d4b7d4f on firebase:master.

Thank you!! Will get to your other PRs soon.


add override css variable for --icon-size

should this need a changeset?

should this need a changeset?

I wouldn't add one. The CSS API added isn't part of our public API.


Fixed tenant operator target label behavior

Description
This PR fixes a little bug in the tenant operator related to the targetLabel behavior. The bug is that the filter performed at the event-level in the SetupWithManager function does not apply when entering the reconcile after returning from the previous reconcile with a non-nil RequeeAfter. This causes the resource to be reconciled by contrasting operators.
How it has been discovered
The issue appeared after changing the label on a tenant from production to pre-production. Doing this, the tenant will be later reconciled by the pre-production operator. But on the last reconcile performed by the production operator, a RequeAfter set the next reconcile in 1-2 hours, which will not be filtered by the event-filter mentioned before and will change the label on all resources related to the tenant (namespaces, roleBindings,..). This behavior leaves the label on the tenant with the value pre-production but effectively brings back the tenant from pre-production to production through the change on every related resource.
Note
The fix is not the best solution. The best solution would be probably trying to update the controller-runtime dependencies to the latest version and see if this has been fixed., if not, we could write to the kubeBuilder guys.
How Has This Been Tested?
This has been tested manually using kind locally

/rebase

/merge


Error in example code

Missing using statement in the example for Visual Studio (not Code) in the code block when registering the DbContext. https://docs.microsoft.com/en-us/aspnet/core/tutorials/first-web-api?view=aspnetcore-3.1&tabs=visual-studio#register-the-database-context.
Code example needs to include: Microsoft.EntityFrameworkCore.InMemory, otherwise the user will not see the 'UseInMemoryDatabase(string)' method on the options class. Its vaguely mentioned in the section on downloading nuget packages, but the code example as-is, is broken without adding this using statement, and installing the associated package.

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

ID: 583ae3e7-5915-f22b-e54c-6c100cf2a7da
Version Independent ID: 95f2549e-29f9-e31a-b4d0-432b971643ff
Content: Tutorial: Create a web API with ASP.NET Core
Content Source: aspnetcore/tutorials/first-web-api.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

It's in there.

With all due respect, (I understand you are the author of this, and I don't mean to waste your time), but if you could please show me where the 'Microsoft.EntityFrameworkCore.InMemory' using statement is included your example code in the tutorial, I will absolutely apologize and shut up.
Here is a screenshot of the relevant section as it currently exists on the tutorial for Visual Studio.

So given:
services.AddDbContext(opt =>
opt.UseInMemoryDatabase("TodoList"));
opt.UseInMemoryDatabase(string) would be unavailable unless the using statement was included:
using Microsoft.EntityFrameworkCore.InMemory;
This snippet is broken, unless that using is included, regardless of if you instructed the user to install the package, you did not include the using statement in the example.

Do a search, it's there.

Ok. Since you did not specify where to search, or even for what, we will attempt to do some documentation debugging here. The only 2 places I could search will be the tutorial, and the repo, so lets start there.
Searching the document as a whole:
Results 1 of 1:
This is where you state we need to install the nuget package. At no point does it state to go to Startup.cs and add the using statement. In fact, you tell the user to install the '.SqlServer' package here, which is never used, so I am not sure why its included?

In the section of the tutorial related to adding the DbContext to the ServiceCollection you explicitly state several using statements to include, including one for 'Microsoft.EntityFrameworkCore.SqlServer', which isn't used in your example. I suspect this is where your confusion is. I think you meant to include, the one that is actually needed, 'Microsoft.EntityFrameworkCore.InMemory' to use an InMemory database, vs a SQL server instance. See screenshot:

If I am correct, then in the section where you tell the users to install the nuget packages, that is also incorrect and the user can omit installing the 'Microsoft.EntityFrameworkCore.SqlServer' package, as its not consumed while using an In-Memory database.
Searching the example code in the github repo: Results 0 of 0. No mention of the using, even though the code requiring that statement is included in the snippet.

I don't mean to be rude, but I did spend a bit of time today, and have now searched through all of the documentation, and the github repo for related code/instructions to ensure I was not wasting your time. I can even google search this, and find others with the same issue, so clearly, its not that clear.
Seeing as this tutorial is geared towards beginners in .net core, this should be clear and obvious, and this really isn't. Yes, with a bit of deductive reasoning, you should be able to assume that if you installed that package, you need to consume it somewhere, but at no point do you actually tell the user to do so, and that is what I am trying to point out here.
To a new user, this would stump them, without external research into what methods are available to which packages in the various entity framework packages, or some stack overflow digging.

Download the sample. It works and is the code presented in the tutorial.

Ok, as I said I would, I apologize. This does work in the sample code without the using statement. I will admit to not understanding why it works without the using statement, but that's for me to figure out on my own.
Again, sorry for wasting your time.


Stop Images from being saved to Default Camera Roll. Let users (developers that use the package) decide to enable or disable that.

Description
I tried to invoke show camera method. After that took a picture from camera interface.
I expected to be returned a camera blob information without it being saved into default camera roll or DCIM folder.
I Donot want any information that is recorded from my app to move outside it.
The image is being saved to default camera roll. I donot want to do extra work of moving the image to application's internal document folder.
How to repeat issue and example
ImagePicker.showImagePicker(options, async response => {
      // console.log("Response = ", JSON.stringify(response));

      if (response.didCancel) {
        console.log("User cancelled image picker");
        // DO NOTHING
      } else if (response.error) {
        console.log("ImagePicker Error: ", response.error);
        // TODO: error handling
      } else if (response.customButton) {
        console.log("User tapped custom button: ", response.customButton);
        // TODO: Custom Button handling
      } else {
        switch (Platform.OS) {
          case "android":
               if (responsePath.indexOf(RNFS.ExternalDirectoryPath) >= 0) {
                 //  if the image is from temporary directory i.e. image is taken from launchCamera within the applicationmove it to document directory
                 await RNFS.moveFile(responsePath, filePath);
               } else {
                 //  if the image is from external location or performed action os choose from Library move it to document directory
                 await RNFS.copyFile(responsePath, filePath);
               }
               break;
          . . .
         }
      }
})

when options has storageoptions property set to undefined or doesnot have the property assigned at all, you will find that when image is captured within the application a copy resides in DCIM folder which is not needed.
I am unable to delete the image from within the application.
Solution
** Unknown
Additional Information

React Native version: 0.55.4
Platform: Android
Development Operating System: Windows and Mac
Dev tools: Visual Studio Code and Android Studio

Anyone can fixed that ?

https://github.com/react-community/react-native-image-picker/issues/928#issuecomment-438526153

Facing the same problem, It get worse if you request smaller sizes than the max resolution.
I get 3 copies!! (on Android):

/Pictures -> original size
/Pictures -> resized
/DCIM/Camera -> original size

It seems so strange this issue is around since 2017 having even a pull request to address it.
What is blocking the merge or solution to this undesired storage wasteful behaviour?

Hi, how u guys solved this??  I alway got the path with DCIM, and I'm using another folder.

Please reopen the issue as it is not solved.

Someone resolve this? I have the same problem,
I only need the base64 data but the picture is saved in the cameraroll with the original size

@marcoavazquez unfortunately I've solved it by using react-native-camera instead.

@marcoavazquez unfortunately I've solved it by using react-native-camera instead.

Ok, tank you very much. I was using react-native-camera, but I needed take the picture from the galery too, so I will use this library for that and the camera with the another one.


Add github action to automatically push to pypi on Release x.y.z commit

you need to create a token there https://pypi.org/manage/account/token/ and put it in there https://github.com/crowsonkb/k-diffusion/settings/secrets/actions/new name it PYPI_PASSWORD
The release will be triggered when you name your commit Release x.y.z I advise to change the version in setup.cfg in that commit

also used clip_anytorch instead of git clip


[stable/chartmuseum] Fixes compatibility with 1.16

Signed-off-by: Rafael Grigorian<EMAIL_ADDRESS>
What this PR does / why we need it:
Makes stable/chartmuseum compatible with 1.16. A quick overview of deprecated APIs can be found on their blog post: https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @jdolitsky

👍 Need that !

/ok-to-test

cc @jdolitsky Hello sir any feedback would be good.

/lgtm


Add generic implementation of failover cache

Codecov Report

Merging #12 (ba1b998) into master (3607797) will decrease coverage by 3.64%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
- Coverage   75.54%   71.89%   -3.65%
==========================================
  Files          11       11
  Lines         740      740
==========================================
- Hits          559      532      -27
- Misses        153      181      +28
+ Partials       28       27       -1

Flag
Coverage Δ

unittests
71.89% <0.00%> (-3.65%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

evict.go
0.00% <0.00%> (-84.38%)
:arrow_down:

sharded_map.go
78.62% <0.00%> (ø)

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 3607797...ba1b998. Read the comment docs.


chore(Analysis): fix mathlib3 names; automated fixes

Looking over the diff:

I understand where the exp -> NormedSpace.exp change comes from; I'm not sure if it's actually better. Happy to revert those manually.
False negative (my script so far missed one): continuous_linear_map.adjoint should also be pointed out

Thanks! I think NormedSpace.exp is good since it creates a link in the documentation.
bors r+

Thank you for the fast review!