Projections of the earth in the examples on the front page

Looking at the examples on https://github.com/JuliaPlots/Makie.jl ,
First earth example from
https://gist.github.com/SimonDanisch/8f5489cffaf6b89c9a3712ba3eb12a84
simplified and with lighting adjusted (current documentation does not say too much about the light attribute :) )
using Makie, FileIO, Colors
earth = load(download("https://svs.gsfc.nasa.gov/vis/a000000/a002900/a002915/bluemarble-2048.png"));
scene = Scene(resolution = (500, 500), color = :black)
m = GLNormalUVMesh(Sphere(Point3f0(0), 1f0), 60)
light = Vec{3,Float32}[[1.0, 1.0, 1.0], [0.1, 0.1, 0.1], [0.9, 0.9, 0.9], [0.0, -20.0, 0.0]]
Makie.mesh(m, color = earth, light = light)

Antarctica looks big

Antarctica should look like this (takes from Wikipedia)

https://upload.wikimedia.org/wikipedia/commons/thumb/f/f2/Antarctica_(orthographic_projection).svg/250px-Antarctica_(orthographic_projection).svg.png
This example is a nice one
https://gist.github.com/SimonDanisch/58cb1050581658859f7a5df51fa1e5a1#file-earth-jl
but it has two problems
1)
Projection of the tempeture map has the same projection error as above.
2)
The precipitation is not matching the temperature map, but is 90 deg rotated and scaling is distorted

These are low priority, but since these are on the front page, I thought to file an issue.
Also, gists as far as I know do not accept PRs to update them to Julia v1.0.

I think this is because of https://github.com/JuliaPlots/Makie.jl/issues/205

trying another equirectangular projection test:
using Makie, FileIO, Colors, GeometryTypes
scene = Scene(resolution = (500, 500), color = :black)
m = GLNormalUVMesh(Sphere(Point3f0(0), 1f0), 60)
moon = load(download("https://svs.gsfc.nasa.gov/vis/a000000/a004600/a004675/phases.0001_print.jpg"))
mesh(m, color = moon)

there is quite a lot of polar distortion in the image. Craters are elliptical:

ok, seems to have something to do with the coordinate spacing. I if I adapt the image on a sphere example, everything works:
using Makie, FileIO, Colors, GeometryTypes

linspace(a,b,n) = LinRange(a,b,n) |> Vector
n = 512
θ = linspace(0,pi,n)
φ = linspace(0,2*pi,2*n)
x = [cos(φ)*sin(θ) for θ in θ, φ in φ]
y = [sin(φ)*sin(θ) for θ in θ, φ in φ]
z = [cos(θ) for θ in θ, φ in φ]
pts = vec(Point3f0.(x, y, z))
moon = load(download("https://svs.gsfc.nasa.gov/vis/a000000/a004600/a004675/phases.0001_print.jpg"));
surface(x, y, z, color = moon)

@SimonDanisch the distortion seems to be traceable to the way the sphere points are sampled by HyperSphere. Does this function generate an equirectangular coordinate grid for the 3-sphere? Should it be doing something like the image on a surface example adapted above?

So the error is in the code wrapping the color on a spherical mesh. Where is that color defined in Makie?

It's basically here:
https://github.com/JuliaGeometry/GeometryTypes.jl/blob/master/src/decompose.jl#L367
And here:
https://github.com/JuliaGeometry/GeometryTypes.jl/blob/master/src/decompose.jl#L381
(uv coordinates are in 0..1 mapping to the coordinates of the image)

But decompose works, right? Look at my image above - the points are correctly placed but the underlying heatmap isn't (or is it the other way around)? The "North pole" convergence of the decompose points are at the points that correspond to longitude-latitude (0, 0), the South Pole at (180, 0) on the heatmap.
But the correspondence should be at (_, 90) and (_, -90) (where _ corresponds to any value as the longitudes collapse in the poles).

The UV map seems pretty wrong (decompose(UV, Sphere(...)))

This works:
uv = [UV{Float32}(φ, θ) for θ in ux, φ in ux]
f = decompose(Face{3, GLIndex}, SimpleRectangle(0, 0, 1, 1), (n, n))
m = GLNormalUVMesh(vertices = pts, faces = f, texturecoordinates = vec(uv))
scene = mesh(m, color = moon, shading = false)

Maybe that's how I should define the decompose for sphere

Although that's still inconsistent with surface:
![image](https://user-images.githubusercontent.com/1010467/47991599-eabca200-e0ea-11e8-8586-db4cd8909e9a.png)

Thanks everyone, I think this is fixed now with https://github.com/JuliaGeometry/GeometryTypes.jl/pull/158
using Makie, FileIO, Colors
earth = load(download("https://svs.gsfc.nasa.gov/vis/a000000/a002900/a002915/bluemarble-2048.png"));
scene = Scene(resolution = (500, 500), color = :black)
m = GLNormalUVMesh(Sphere(Point3f0(0), 1f0), 60)
light = Vec{3,Float32}[[1.0, 1.0, 1.0], [0.1, 0.1, 0.1], [0.9, 0.9, 0.9], [0.0, -20.0, 0.0]]
Makie.mesh(m, color = earth, shading = false)

Awesome, you're a wizard!

Note that this does not completely resolve the issue with the points decomposition:

Note the polar position of the points wrongly at (0,0) and the clearly misplaced rainy belt over Sahara. The bar heights could be something in the meshscatter function but at least the point decomposition is not consistent with that of a globe.

For info, the world-clim precipitation layer is only defined over land, so the correct projection would put all the flat bars over the ocean.

Yeah the uv look up code needs to be adapted for sampling the precipitation:

Do you mean in the example or in GeometryTypes?

The example needs an update ;)

OK, so you essentially just fixed the to_msize function? I wouldn't mind adding this unless you've essentially already done it for your plot above.

Yeah it's fixed ;) https://github.com/JuliaPlots/Makie.jl/blob/master/examples/bigdata.jl#L2 needed some julia 1.0 updates as well!

Oh great. Yeah I'd fixed the other things locally because I wanted to update this example, that was how I fell over this to begin with.

So, this can be closed I guess?


How and where to aggregate all issues from all janssen repositories

how and where to aggregate all issues across janssen so that community members can easily see where the contribution is needed

After moving to monorepo, jans repo has consolidated all issues.


zfs: 2.1.9 → 2.1.11

Skipping over 2.1.10 now that it was found to have possible data corruptions
Release notes @ OpenZFS’s Git forge: https://github.com/openzfs/zfs/releases/tag/zfs-2.1.11
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Hopefully the reviews from 2.1.10 make this easy to review. I‘m on vacation so I haven’t physically tested it out, but I can make any suggested fixes.

Should this be backported?

Should this be backported?

I'm not sure it fits our rules for backports


Data Preprocessing Code of DialogueGCN

Thank you for releasing the DialogueGCN code. Can you post the data preprocessing code of DialogueGCN or tell me where I can find it. I don't understand the data in file IEMOCAP_features.pkl.
Thanks.

If I use features that DialogueRNN used, the model is easy to be overfitted. So can you tell us how to extract the features better? Thanks.

I am also confused about the data in IEMOCAP_features.pkl. The sentence in dialog in 'videoText' is a vector with 100 dimensions. Is this the output of CNN embedding?

The model will overfit if you set lr to 0.001.
Yes, this is the output of CNN embedding.

The model will overfit if you set lr to 0.001.
Yes, this is the output of CNN embedding.

We set lr to 0.001, it also be overfitted.  The model is sensitive to the features. Can you share the  code for feature extraction ?   I want to know when to stop the training in the feature extraction stage. The feature file for DialogueRNN is different from the file for DialogGCN. Thanks.

The DialogueGCN and DialogueRNN features should be the same. The way we format the data in the pkl is a bit different.

The DialogueGCN and DialogueRNN features should be the same. The way we format the data in the pkl is a bit different.

Yes, maybe something is wrong in my experiment. Thank you very much!

Closing the issue.


feat: stdlib docs generator

Description
Problem
Part of #5797
Summary
This is just an experiment to see if we can already generate stdlib docs from our code.
Pending:

[ ] Before being able to merge this, we need to document all the standard library (copy doc comments from .md files into .nr files)
[ ] Let the generator handle globals
[ ] Let the generator handle traits
[ ] Let the generator handle trait impls
[ ] Let the generator handle type alises

Additional Context
The current doc generator currently has just one goal in mind: be able to generate stdlib docs from the .nr files, without us having to manually write .md files with doc comments. This means that the doc generator can't be used for other crates (or maybe it can, but the output is not ideal). The reason is that to generate docs we traverse the AST so we don't have type information. Ideally a doc generator would put links between types, etc., but doing that is much more complex and more time consuming. It can be done later, but the current goal is just to simplify the docs writing process.
Documentation
Check one:

[ ] No documentation needed.
[x] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Closing for now. We can reopen or revive once we have more stdlib docs in the .nr files.


Create proposal for new evaluation library repo

Can be based off of https://github.com/instructlab/dev-docs/pull/63

Issues such as https://github.com/instructlab/evaluation/issues/9 should be captured as part of this


Test and use sync.Map

In d7829b96f9db7b130f21d9795c45cdc6d72a5bc1 (https://github.com/CosmWasm/wasmvm/pull/315) we see the use of a sync.Map to handle iterators better. When I did some heavy testing on my Intel Mac this produced issues around pthreads. I also observed dead lock behaviour. Since it's unclear where this is coming from, let's work on this change separately.

Not impressed by the numbers from #502. Closing here to avoid spending more time on a solution searches problem case.


php-simputils-62 Overall tests and coverage

Closes #62

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@f2fd403). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev      #73   +/-   ##
======================================
  Coverage       ?   64.30%
  Complexity     ?     1233
======================================
  Files          ?       69
  Lines          ?     2953
  Branches       ?        0
======================================
  Hits           ?     1899
  Misses         ?     1054
  Partials       ?        0

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 f2fd403...a45c1a6. Read the comment docs.


The point cloud scanning software used by Intel RealSense SR300 scanner

Hi, thanks for your impressive work!
Your results, especially under real scans scanned by SR300, are very robust for different point cloud artifacts.
So I want to experiment self-sample with more real scans scanned by my own SR300, but I only scanned depth maps before based on SDK thus I am not very familiar with point cloud scanning.
My question is which software are used by Intel RealSense SR300 scanner to scan the point cloud? The software in the official SDK is not very convenient for object scanning. Besides, I have found itSeez3D, but it seems can only produce mesh but not point cloud.
Looking forward to your reply.

Hi, so the point clouds from the SR300 sensor were obtained thanks to Guy, a good friend of mine.
He published the code for his project that used the SR300, which can be found here, together with a thorough README explaining how to use the program.
He also kindly pointed me to this script that should be a good reference for getting point clouds using python.
I hope that answers your question :)

Awesome！ I will follow its instructions and try to scan shapes by myself. Thank you very much!


update to cordova 5.2

https://cordova.apache.org/news/2015/08/13/tools-release.html

also check bower, npm (generator and project)

cannot update karma #235


error[E0063]: missing fields

Trying to build aur/simp.
error[E0063]: missing fields `preload_range_hint`, `release_error_msg`, `release_file_range` and 1 other field in initializer of `heif_reader`
   --> /home/builder/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libheif-rs-1.0.2/src/reader.rs:104:50
    |
104 | pub(crate) static HEIF_READER: lh::heif_reader = lh::heif_reader {
    |                                                  ^^^^^^^^^^^^^^^ missing `preload_range_hint`, `release_error_msg`, `release_file_range` and 1 other field

Looks like libheif changed a header file. I libheif-sys uses bindgen to generate the rust bindings and it looks like the arch package is too new. 134d46b disables bindgen so it should build. It is not a great solution though as it might just crash now instead.


Create docker network fail!

Hi guys:
Working with This link.
Can't create network databases success.
I have run calico/node-libnetwork:latest and calico/node:latest and etcd
root@10-9-40-246:~# docker  ps
CONTAINER ID        IMAGE                                 COMMAND                  CREATED             STATUS              PORTS                                                                NAMES
ea5d1aa0df84        calico/node-libnetwork:latest         "./start.sh"             18 minutes ago      Up 18 minutes                                                                            calico-libnetwork
7f4944de704c        calico/node:latest                    "/sbin/start_runit"      18 minutes ago      Up 18 minutes                                                                            calico-node
49166b8e6c02        index.alauda.cn/xdzhang/etcd:latest   "/etcd -name etcd0 -a"   30 minutes ago      Up 30 minutes       <IP_ADDRESS>:2379-2380->2379-2380/tcp, <IP_ADDRESS>:4001->4001/tcp, 7001/tcp   etcd

after I run docker network create --driver=calico --ipam-driver=calico databases
get nothing
root@10-9-40-246:~# docker network create --driver=calico --ipam-driver=calico databases
7a54bcc7f88d8184d7599f5c240d17c1f73764a42dc2d715a42b0a4926554837
root@10-9-40-246:~# docker network ls
NETWORK ID          NAME                DRIVER
bb414f8646df        none                null
f7b847c49748        host                host
2ac657422781        bridge              bridge

Did I miss sth?
Thank you!~

Hi @nashasha1,
It looks like you may have missed the step to set Docker's --cluster-store parameter (this is a docker requirement to use Docker libnetwork).  This is covered in our Calico as a Docker Networking Plugin guide. We link to this setup from within the Calico with the Docker Containerizer Manual Install guide, which is where you likely followed instructions for getting calicoctl onto your machine.
Let me know if this helps!

@TrimBiggs  Thank you for your reply.
After I set Docker's --cluster-store parameter, network can create as expect.
But another issue occurred.
When I run docker  run -d --name=c1 --net=databases tutum/ubuntu:latest
response:
root@10-9-59-120:~# docker  run -d --name=c1 --net=databases tutum/ubuntu:latest
4ce9ffbdc3432a233c0f9f159594bc379c7bb7b58001511a499b9ef1c7c7af4f
Error response from daemon: Cannot start container 4ce9ffbdc3432a233c0f9f159594bc379c7bb7b58001511a499b9ef1c7c7af4f: failed to create endpoint c1 on network databases: Plugin Error: NetworkDriver.CreateEndpoint, {
  "Err": "'LibnetworkDatastoreClient' object has no attribute 'get_default_next_hops'"
}
root@10-9-59-120:~#

My calicoctl version
root@10-9-59-120:~# ./calicoctl version
0.19.0-dev

My Docker info:
root@10-9-59-120:~# docker  info
Containers: 3
Images: 30
Server Version: 1.9.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 36
 Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.13.0-63-generic
Operating System: Ubuntu 14.04.3 LTS
CPUs: 1
Total Memory: 1.955 GiB
Name: 10-9-59-120
ID: T3IG:4TLW:YAZA:UYHV:MLE2:NC3R:7LBI:PN2W:WPOI:QQBZ:QCFD:5ESW
WARNING: No swap limit support
Cluster store: etcd://<IP_ADDRESS>:2379

Any idea about this.
 In projectcalico, master images are not guaranteed to be bug free. Releases are!
Try downloading calicoctl from the 19 release: https://github.com/projectcalico/calico-containers/releases/tag/v0.19.0
If you rerun calicoctl node --libnetwork, it will pull stable images.

@djosborne  Thank you very much.Release version work well.


Preserve document title and append notebook name

Hey @scottdraves, this change is more for embeddable beaker, but wasn't sure if you would be interested in having it in desktop beaker as well.  Right now if you switch from beaker app to a notebook, the document title will change from Beaker to New Notebook.  This change makes it so it changes from Beaker to Beaker - New Notebook.
I can look into another approach for embeddable if this isn't a change you'd be interested in, but thought I would present it as an option.  :dancers:

Testing this in another use case didn't work with this implementation, so closing for now, but let me know if you'd be interested in the functional change.

the user can often only see the beginning of a window title (when they have many tabs open).
so i don't think this makes sense for any version.
please skip it completely.


isort processes left behind after VS code closes

The following isort processes don't terminate after VS code is closed.
41002   ??  S      0:00.63<EMAIL_ADDRESS>/Users/gnought/.vscode/extensions/ms-python.isort-2022.8.0/bundled/tool/server.py
41038   ??  S      0:00.17<EMAIL_ADDRESS>/Users/gnought/.vscode/extensions/ms-python.isort-2022.8.0/bundled/tool/runner.py

Environment: OSX, VS code 1.74.0

similar issue for me. I killed about 30 of these processes recently. I just found another 13 running from ~/.vscode/extensions/ms-python.isort-2022.8.0/bundled/tool/server.py

Fixed in https://github.com/microsoft/vscode-isort/pull/207
The fix is available in pre-release. There is also a server less mode, set "isort.serverEnabled": false in user settings to disable server mode.


Custom Nginx Error Messages?

It seems like most people use CDNs for this type of functionality these days, but is there some way to have "site will be back momentarily" sign until all of the nginx-proxy-fed docker services finish starting? This seems like a pretty common use case, so I figured someone here would have a good suggestion or extensions they've written to do this. So if visiting the domain while the services are still coming up - instead of seeing an nginx error (meaningless to most people) - it shows a "please check back in a few moments" message.
I had previous posted this in the wrong project:
https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion/issues/574

in vhost.d nginx config for your container have something like
        error_page 503 = @custom_error;
        location @custom_error {
                internal;
                return 503 'please check back in a few moments';
        }

not 100% if its 503 or some other error you have to cover.

Great, very useful... But how do I get it so the error appears immediately in the browser. If the site isn't up and this gets triggered, then it seems the browser will always display this first:
Attackers might be trying to steal your information from [domain] (for example, passwords, messages, or credit cards). Learn more
NET::ERR_CERT_AUTHORITY_INVALID
Then you have to click twice to proceed.

@dm17 I dont think you can do anything with that. Browser remembers that site is being loaded over HTTPS and if your site is down/certificate is invalid, user will get this error from browser.

Note that you might want to use the proxy_intercept_errors on; directive to intercept and redirect proxied error responses to nginx for processing with the error_page directive in order to prevent information exposure through error messages.


Move the CLI from hug to click

Disclaimer - this is just a suggestion. Feel free to close it. click is IMHO may be a better suited framework for creating CLIs than hug. I think where hug really shines is for creating web APIs but for clis I think click has a competitive edge since it was designed with just CLI applications in mind. Click supports better documentation, input types and has click.testing for testing support. It also handles type annotations better than hug. I know hug is maintained by @timothycrosley and since cruft is your project you may want to use it.
Other benefits are that cookiecutter itself use click. This way we can avoid adding another dependency for our downstream users and possibly reuse more of cookiecutter internals, making the cruft wrapping logic even thinner. This will allow for better compatibility with cookiecutter.

If this is okay to do, I am willing to do the migration.

How would you feel about using typer instead? https://github.com/tiangolo/typer In theory, since it is backed by click it should maintain good interoperability with cookiecutter, while maintaining the conciseness of a hug CLI

Typer is great as well! Absolutely a fan of all of @tiangolo's projects :) I will make the move to typer in the next PR


Fixing mkl dependency issue, and further improvements.

Issue, if available:
Because we moved correctly the numpy dependency to the conda env directly, we had an additional mkl version installed within anaconda, additionally to the mkl version that came from the mxnet_mkl version. Anaconda once again was not able to resolve this. This lead to one of the jenkins builds timing out.
Description of changes:

Changed mxnet_mkl to mxnet.
Some minor dependency improvements

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #661 into master will not change coverage.
The diff coverage is n/a.


CORS Error: Response to preflight request doesn't pass access control check

Hello, after following the instructions in the README to run the python server and the react client, I am seeing several CORS errors in the browser javascript console. How can I resolve the CORS error?
Following are the errors that I see repeated in the console. The errors appear periodically without any interaction, and also when I attempt to upload a PDF using the FAISS dialog UI.
Access to fetch at 'http://localhost:5000/api/check-vector' from origin 'http://localhost:3000'
has been blocked by CORS policy: Response to preflight request doesn't pass access control check:
No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response
serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Hi @slee-aws -
I've found that CORS errors can be difficult to track down as it may indicate an issue other than CORS, but I'll do my best to help.
Can you tell me what browser you're using?
Are you running this on your local machine or Cloud9?

Closing this issue as we haven't head back.


#37 #38: Reordered buttons and added Social Media buttons

Link the issues that this PR is addressing/fixing


Feature: Introduce ObjectId as primary key for mongo

Motivation
Auto-increment primary keys are introduced in cases where no fields can be used as primary naturally.
However, there are non-negligible overhead when using autoInc in mongo driver.
Therefore we propose to use the native ObjectId as auto-generated primary key for mongo and provide universal interface to support it while still maintaining consistency with other drivers.
Design
Introduce new tagged type Primary (name is not decided yet) and new field type primary
The Primary type contains auto-generated value, and can only use as primary keys or foreign keys. Also it is unordered.
When Primary occurs as primary key, autoInc and optimizeIndex must be set.
In mongo driver, Primary is treated as ObjectId (therefore omitted in many situations), while in SQL-based drivers, Primary is treated as proper auto-increment intergers.
先占坑，看看有什么别的思路x

f19854410ead2702726aa20ae2a24d5cf89c4913
waiting for other PRs

f198544

我感觉可以直接默认 (忽略) autoInc 属性。即有 primary 的时候，无论是否 autoInc，在 SQL 上均视为 autoInc，在 mongo 均视为 ObjectId。


Update our dependency to the latest pulumi/terrform-provider-aws

Merging this in.  Let me know if you have any concerns.


[Bug]: While running a create Query in the appsmith using postgres database , it shows "Execution failed with status 5005 ". Can someone pour some light about the issue here .

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Normally when I have connected other databases like mongodb atlas and even nocodb , I have observed that it has ran successfully . Now with the postgres database only it is showing error 5005

Please look into this .
Steps To Reproduce

Create a postgres database using docker and input some values and table into the database named of your choice .
Run queries in the psql prompt just  to check if its working and the tables are created or not .
Now connect the database to appsmith using the plus icon in database section on the left sidebar
Once the database is connected , test it .If its successful then got the next step .
Go to Add queries section and then write your create query .
You will find that it shows an Execution error with status 5005 .

Public Sample App
No response
Version
Ran inside docker .

@etherral17 it's likely that the amount of data in your database is too large to load in a single query. Could you try by adding a limit to your query and paginating your data?
https://docs.appsmith.com/core-concepts/data-access-and-binding/displaying-data-read/display-data-tables#pagination

@Nikhil-Nandagopal  Hey nikhil , I have freshly prepared that postgres database and has only 3 tables with few data points. I would need some points where you require more info . Can it be that I am using free docker version of appsmith , thats why the problem exists ?

You can see in this image that there is only two data points in this table "author" . Please help me as I am a fresher and this is my second issue post . So please feel free to guide me .

@etherral17 it's then likely that you connection is not configured correctly/. Can you go to the datasource and try to hit the test button? Maybe a value in the connection is incorrect

it got solved by recreating the database , a fairly simple solution delete the database and its metadata and recreate it .


Fix for Mousetrap to prevent event propagation.

I noticed that when you add combos like Ctrl + a (which is used by some browsers to select everything in the page), the hotkey callback is fired successfully but the event is not cancelled and hence the default action of the combo is executed (Select All in this case)

I'd argue this is best left for the user of the library to decide. I always call event.preventDefault() as the first line of all my hotkey callbacks, but it is conceivable that someone might want to augment the default action of a hotkey combo rather than replace it.


Unable to start activity ComponentInfo{org.nativescript.mapboxdemo/com.tns.NativeScriptActivity}

NativeScript: 1.6.2
Today I have checkout your current mapbox demo but I'cant get it running on my system. I get this error (only a part of fully error text, couldn't copy it all).
java.lang.RuntimeException: Unable to start activity ComponentInfo{org.nativescript.mapboxdemo/com.tns.NativeScriptActivity}: com.tns.NativeScriptException:
Calling js method onCreate failed

TypeError: Cannot read property 'widgets' of undefined
File: "/data/data/org.nativescript.mapboxdemo/files/app/tns_modules/ui/frame/frame.js, line: 327, column: 39

StackTrace:
	Frame: function:'NativeActivity.onCreate', file:'/data/data/org.nativescript.mapboxdemo/files/app/tns_modules/ui/frame/frame.js', line: 327, column: 40

	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2416)
	at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2476)
	at android.app.ActivityThread.-wrap11(ActivityThread.java)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1344)
	at android.os.Handler.dispatchMessage(Handler.java:102)
	at android.os.Looper.loop(Looper.java:148)
	at android.app.ActivityThread.main(ActivityThread.java:5417)
	at java.lang.reflect.Method.invoke(Native M[...]

Generally I have amount of experience in software development, but I'm new to nativescript. Maybe is a really simple fault and I don't recognize it ...

Hi, I just updated my NativeScript to 1.7.0 and checked this plugin. Updated the iOS podpec while I was at it (newer native Mapbox SDK) and fixed an error with Android setTilt. All seems fine, so please update your NativeScript and try again.

Ok, new checkout ... some new errors. What could be wrong?
Executing before-prepare hook from /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/hooks/before-prepare/nativescript-dev-typescript.js
Found peer TypeScript 1.8.7
app/examples/example-base-page.ts(13,31): error TS2306: File '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/sidedrawer/sidedrawer.d.ts' is not a module.
app/examples/example-base-page.ts(13,31): error TS2656: Exported external package typings file '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/sidedrawer/sidedrawer.d.ts' is not a module. Please contact the package author to update the package definition.
app/examples/listview/layouts/listview-layouts.ts(2,33): error TS2306: File '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module.
app/examples/listview/layouts/listview-layouts.ts(2,33): error TS2656: Exported external package typings file '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module. Please contact the package author to update the package definition.
app/examples/listview/layouts/listview-layouts.ts(17,46): error TS2339: Property 'os' does not exist on type 'typeof android'.
app/examples/listview/layouts/listview-layouts.ts(19,38): error TS2339: Property 'WindowManager' does not exist on type 'typeof view'.
app/examples/listview/layouts/listview-layouts.ts(24,46): error TS2339: Property 'os' does not exist on type 'typeof android'.
app/examples/listview/layouts/listview-layouts.ts(26,40): error TS2339: Property 'WindowManager' does not exist on type 'typeof view'.
app/examples/listview/reorder/listview-reorder-model.ts(42,9): error TS1095: A 'set' accessor cannot have a return type annotation.
app/examples/listview/reorder/listview-reorder.ts(2,33): error TS2306: File '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module.
app/examples/listview/reorder/listview-reorder.ts(2,33): error TS2656: Exported external package typings file '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module. Please contact the package author to update the package definition.
app/examples/listview/reorder/listview-reorder.ts(25,21): error TS2339: Property 'os' does not exist on type 'typeof android'.
app/examples/listview/reorder/listview-reorder.ts(26,44): error TS2339: Property 'WindowManager' does not exist on type 'typeof view'.
app/examples/listview/reorder/listview-reorder.ts(30,21): error TS2339: Property 'os' does not exist on type 'typeof android'.
app/examples/listview/reorder/listview-reorder.ts(31,42): error TS2339: Property 'WindowManager' does not exist on type 'typeof view'.
app/examples/listview/reorder/listview-reorder.ts(32,55): error TS2339: Property 'Color' does not exist on type 'typeof graphics'.
app/examples/listview/reorder/listview-reorder.ts(45,21): error TS2339: Property 'os' does not exist on type 'typeof android'.
app/examples/listview/reorder/listview-reorder.ts(46,42): error TS2339: Property 'WindowManager' does not exist on type 'typeof view'.
app/examples/listview/reorder/listview-reorder.ts(49,21): error TS2339: Property 'os' does not exist on type 'typeof android'.
app/examples/listview/reorder/listview-reorder.ts(50,44): error TS2339: Property 'WindowManager' does not exist on type 'typeof view'.
app/examples/listview/selection/detail-page.ts(7,24): error TS2307: Cannot find module 'Page'.
app/examples/listview/selection/main-page-model.ts(3,27): error TS2306: File '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module.
app/examples/listview/selection/main-page-model.ts(3,27): error TS2656: Exported external package typings file '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module. Please contact the package author to update the package definition.
app/examples/listview/selection/main-page-model.ts(211,37): error TS2339: Property 'items' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(246,22): error TS2339: Property 'notifySwipeToExecuteFinished' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(279,48): error TS2339: Property 'getSelectedItems' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(286,44): error TS2339: Property 'getSelectedItems' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(302,38): error TS2339: Property 'getSelectedItems' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(315,50): error TS2339: Property 'getSelectedItems' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(324,30): error TS2339: Property 'deselectItemAt' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(324,54): error TS2339: Property 'items' does not exist on type 'View'.
app/examples/listview/selection/main-page-model.ts(339,38): error TS2339: Property 'getSelectedItems' does not exist on type 'View'.
app/examples/listview/selection/main-page.ts(3,27): error TS2306: File '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module.
app/examples/listview/selection/main-page.ts(3,27): error TS2656: Exported external package typings file '/home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/node_modules/nativescript-telerik-ui/listview/listview.d.ts' is not a module. Please contact the package author to update the package definition.
Project successfully prepared

:config phase:  createDefaultIncludeFiles
        +found plugins: fresco
        +found plugins: nativescript-telerik-analytics

:config phase:  createPluginsConfigFile
        +creating product flavors include.gradle file in configurations folder...

:config phase:  pluginExtend
        +applying configuration from: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/configurations/fresco/include.gradle
        +applying configuration from: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/configurations/nativescript-telerik-analytics/include.gradle
        +applying configuration from: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/configurations/include.gradle

:config phase:  copyAarDependencies

:config phase:  addAarDependencies
        +adding dependency: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/libs/aar/widgets-release.aar
        +adding dependency: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/libs/aar/Chart-2015.3.1307-trial-release.aar
        +adding dependency: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/libs/aar/Common-2015.3.1307-trial-release.aar
        +adding dependency: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/libs/aar/Data-2015.3.1307-trial-release.aar
        +adding dependency: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/libs/aar/Input-2015.3.1307-trial-release.aar
        +adding dependency: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/libs/aar/List-2015.3.1307-trial-release.aar
        +adding dependency: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/libs/aar/Primitives-2015.3.1307-trial-release.aar
:preBuild UP-TO-DATE
:preNativescript-frescoNativescript-telerik-analyticsDebugBuild UP-TO-DATE
:checkNativescript-frescoNativescript-telerik-analyticsDebugManifest
:preNativescript-frescoNativescript-telerik-analyticsReleaseBuild UP-TO-DATE
:prepareChart201531307TrialReleaseLibrary UP-TO-DATE
:prepareComAndroidSupportAnimatedVectorDrawable2321Library
:prepareComAndroidSupportAppcompatV72321Library
:prepareComAndroidSupportSupportV42321Library
:prepareComAndroidSupportSupportVectorDrawable2321Library
:prepareComFacebookFrescoDrawee070Library UP-TO-DATE
:prepareComFacebookFrescoFbcore070Library UP-TO-DATE
:prepareComFacebookFrescoFresco070Library UP-TO-DATE
:prepareComFacebookFrescoImagepipeline070Library UP-TO-DATE
:prepareCommon201531307TrialReleaseLibrary UP-TO-DATE
:prepareData201531307TrialReleaseLibrary UP-TO-DATE
:prepareInput201531307TrialReleaseLibrary UP-TO-DATE
:prepareList201531307TrialReleaseLibrary UP-TO-DATE
:preparePrimitives201531307TrialReleaseLibrary UP-TO-DATE
:prepareWidgetsReleaseLibrary UP-TO-DATE
:prepareNativescript-frescoNativescript-telerik-analyticsDebugDependencies
:compileNativescript-frescoNativescript-telerik-analyticsDebugAidl
:compileNativescript-frescoNativescript-telerik-analyticsDebugRenderscript
:generateNativescript-frescoNativescript-telerik-analyticsDebugBuildConfig UP-TO-DATE
:cleanLocalAarFiles
:deleteJavaDir UP-TO-DATE
:ensureMetadataOutDir UP-TO-DATE
:collectAllJars
:isMetadataGenerationNecessary
:buildMetadata
Exception in thread "main" java.lang.IllegalArgumentException: Class android.support.v7.widget.AppCompatCheckedTextView conflict: /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/build/intermediates/exploded-aar/com.android.support/appcompat-v7/23.2.1/jars/classes.jar and /home/mbernhardt/Development/Projects/NativeScript/nativescript-marketplace-demo/platforms/android/build/intermediates/exploded-aar/com.android.support/appcompat-v7/22.2.1/jars/classes.jar
        at com.telerik.metadata.ClassRepo.cacheJarFile(ClassRepo.java:21)
        at com.telerik.metadata.Builder.build(Builder.java:38)
        at com.telerik.metadata.Generator.main(Generator.java:44)
:buildMetadata FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':buildMetadata'.
> Process 'command '/usr/lib64/jvm/jdk1.8.0_73/bin/java'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Okay, tried it again and now it works! Thank you for your support!


Don 102 pph data migration

Description
Added DB models for the PPH Salesforce integration, seed scripts for those tables and updated seeding routines. Re-added the original Wondertix models to ensure API stability.
Per Johnny:
"invite me, Eben, Ben Williams, Justin, Henry, Jaeger and Adam."
Risks
Some of the seeding routines will compare the data given to it, and if no relation can be made, it will skip over that instance of data. Johnny approved this step, as one-sided relations broke the seeding process- changing the fields to optional or not requiring a relation didn't fix this.
Because of this, dataset validation and integrity is key to this working appropriately. At present, a lot of the DEV seed code isn't linked appropriately to the other tables' relations, which leads to a lot of seeded data being skipped over. Production tables have a better success rate.
Validation
I've had to beef up the seeding scripts to sanitize data as it came in. Salesforce had a lot of mixed data types in any given field (I suspect all of their data is sanitized to strings), or dangling nodes of data that were probably assigned to a catch-all account.
Over several days I added changes to the seeding scripts to make them more robust. I checked it with the deployment tables, the dev tables, and instances of custom data to ensure they worked.
Operating System
Mac, M2-series CPU

I think Main may need to be merged into the branch so that we don't overwrite changes that have been made since the branch was created.

Closing per Johnny's direction.


Add FSharp support

Nice to see you took advantage of the multicolor feature for the ascii logo.
Unfortunately, it looks a little noisey and unpolished.
Looking forward to merge your PR 💯

I change the ascii. How about now?

Looking much better @Emanon42
Few remarks:

The first color has to be {0} and not {1}
To better match the real logo, you should use two colors, Color::BrightBlue for the first half and Color:Cyan for the second

Looking forward to merge your PR

As your suggests!


Taxon overview: add taxonomy description from wiki

As discussed earlier, get taxonomy description using wikipedia API and display it on the taxon overview panel.
-Maulik

Per discussion, we will add wiki description for selective ranks: genus and family


gave an example for f.cda_query

Thanks for the PR, but the example should not be 10000, because 1000 is the maximum that the API supports. Happy to merge if you could change that, please :)

@neonichu thanks, that's updated

Thanks!


Scss source maps mismatch in angular 16

Which @angular/* package(s) are the source of the bug?
core
Is this a regression?
Yes
Description
After upgrading my project to Angular v16 from Angular v14, I noticed that the source maps of my scss files were mismatched
The source maps changed (not necessarily correctly) when I removed imports of external packages
I created a test project using angular cli and installed two of the packages we use - bootstrap and @ng-select/ng-select and I saw that the problem is restored
before importing bootstrap and @ng-select/ng-select scss
app.component.scss
.test {
  color: red;
}

after importing bootstrap and @ng-select/ng-select scss
@import "../../node_modules/bootstrap/scss/bootstrap";
@import<EMAIL_ADDRESS>
.test {
  color: red;
}

Importing bootstrap or ng-select separately works fine. I also tried other packages combinations and got mismatched source maps
Please provide a link to a minimal reproduction of the bug
https://stackblitz.com/edit/node-ej8xiu?file=index.js
Please provide the exception or error you saw
Scss source maps mismatch

Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 16.1.1
Node: 18.12.1
Package Manager: npm 8.19.2
OS: darwin x64

Angular: 16.1.2
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1601.1
@angular-devkit/build-angular   16.1.1
@angular-devkit/core            16.1.1
@angular-devkit/schematics      16.1.1
@angular/cli                    16.1.1
@schematics/angular             16.1.1
rxjs                            7.8.1
typescript                      5.1.3

Anything else?
No response

I had a look at this and it appears to be caused the by the Angular compiler style shimming logic in https://github.com/angular/angular/blob/811fe001c9ae4e65c73280a8cfa542f11e275820/packages/compiler/src/shadow_css.ts#L659


Adding cmip6 recipe

A test recipe to demo bulk uploading CMIP6 data from ESGF to cloud storage (as zarrs).

/run recipe-test recipe_run_id=49

/run recipe-test recipe_run_id=50

/run recipe-test recipe_run_id=51

/run recipe-test recipe_run_id=52

/run recipe-test recipe_run_id=52

/run recipe-test recipe_run_id=51

/run recipe-test recipe_run_id=52

/run recipe-test recipe_run_id=53

/run recipe-test recipe_run_id=54

/run recipe-test recipe_run_id=55

/run recipe-test recipe_run_id=56


Fix how xlsx-renderer handle vertical merged cells inside loops

all files are available in 9a87732
Actual

Expected

Hi @skypesky. I'm glad that you're asking.
Yes, the reproduction could be found in this commit: https://github.com/Siemienik/xlsx-renderer/commit/9a87732

@Siemienik
I ran the branch test and have seen the error. f/37-vertical-merged-cells.
The implementation of this part seems to be the most complex(ForeachCell). I plan to spend some time studying it.

I'll keep my fingers crossed. if any help is needed you can find me as always on the Gitter

Hi @Siemienik any updates on this topic? I'm facing ugly/unstable render for vertical merged cell. I would like to achieve something like in the image.

My template:


[#118479861] Use new, password generating RDS broker

What
Generating secure master password for Postgres Instances
Use latest broker from https://github.com/alphagov/paas-rds-broker/pull/11. Add new properties into spec and config file templates to configure the new features.
Testing
Deploy this release against existing environment. E.g. by using https://github.com/alphagov/paas-cf/pull/320. Observe new broker release being deployed. In the broker logs you can see that it tries to connect first, fails ping and updates password afterwards. After deployment, all tests should pass and everything should continue working as usual.
Merging
This PR depends on https://github.com/alphagov/paas-rds-broker/pull/11. Once that has been merged, remove https://github.com/alphagov/paas-aws-broker-boshrelease/commit/a3a6bfd117fb9e40ff5853f94df70c0dd91cdf3e and add a new commit pointing the rds-broker submodule at the merge commit of  https://github.com/alphagov/paas-rds-broker/pull/11.
Who
not @mtekel or @keymon or @saliceti

Submodule reference updated. Merging...


Possible enhancements

https://github.com/denolibs/event_emitter/blob/a030286e116f2be56cd85f402c8d25a324454f91/lib/mod.ts#L194
  if (arr.indexOf(listener) !== -1) {
    arr.splice(arr.indexOf(listener), 1);
    this.emit("removeListener", eventName, listener);
    if (arr.length === 0) {
      this.events.delete(eventName);
    }
  }

indexOf(listener) is called twice
by checking (arr.length === 1) before we splice we can avoid arr.splice entirely if delete the thing anyway
Also not sure if it is possible with your implementation to remove a listener added with .once()

All this can be replaced by checking if arr.length > 1
It's either the listener gets removed at the specific index or not.

you should keep this.events.delete(eventName);
otherwise you will be exposed to memory leaks after continuously subscribing and unsubscribing
see https://github.com/GrosSacASac/event-e3/blob/master/event-e3.js#L77
and tests

@GrosSacASac Are you fine with the changes? @ozjd If so, can you review and merge?

make sure to add tests for what I mentioned and then it is ok

Sorry about the delay, I seemed to have missed a previous notification - @Balou9 could you please verify that the removeListener event would be called everytime a listener is removed?


3 - Use contenteditable html5 attribute to edit file names

This is a suggestion. It will be nice to use Html5 at its whole power, at least in the browsers that supports it. Maybe it's a nicer approach to use contenteditable html5 attribute when the edit file names feature is on.

After some more thought, I don't think contenteditable is appropriate for this situation. A simple text input is best for renaming a file. As such, I'm closing this.


question about retain_perturbed.json in datasets locuslab/TOFU

Thank you for your work on machine unlearning!
I noticed that retain_perturbed.json in datasets only contains 400 pieces of data, about 20 authors. Then when calculating the Truth Ratio for Retain Set, can we only calculate these 400 pieces of data? In my opinion, it requires calculating the Truth Ratio for each of the 20 authors in the Retain Set.
Looking forward to your reply！

Did you solve this problem? I encountered the same issue.


chore: add Dockerfile

Heroku is discontinuing its free tier, so we're migrating to a Docker-based deployment.

LGTM, I'll add a Github workflow to push on the Github repository later this week.


Achievements

Describe your problem here.
How tf do you add achievments in the 0.5.2 update
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Only 2 Custom Events

wait for the documentation

wdym

wdym

Wait for the wiki to be updated

ok


custom_profile_fields: Make "Add and Edit custom profile field" form into modals.

The implementation is approximately the same as before I just migrated
"Add/Edit custom profile field" forms into modals, status update
notifications about these forms will be displayed inside their modal.
Issue #21634 describes pretty much everything about these new changes.

[x] At present, the UI for editing a custom profile field opens directly inside the table. This is confusing, especially if you have more than one open, which is possible. Instead, clicking the pencil icon should open a modal to edit the field. The modal heading can be "Edit custom profile field".
[x] Similarly, the "Add custom profile field" form should be migrated to a modal as well. The field labels in the modal should be above (on a separate line) from the text boxes. The button to open the modal should be on the same line as the "Custom profile fields" table heading; we can try right-aligning it.

Fixes: #21634
CZO

Screenshots and screen captures:

Error inside modal

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Nice, this is such an improvement!
The main thing I noticed is that when the inputs are invalid, we should disable the "Save changes" button. The error message text should be shown in a tooltip when hovering over the "Save changes" button, without the "Failed: " part. Some examples (not exhaustive) in case it's helpful:

I also noticed that the error banner doesn't get cleared when you switch field types (it should), but this may no longer be relevant after the change above.

Also, I don't know if this issue comes up elsewhere, but I noticed that the "g" in "Long text" is a bit cut off:

It is not happening in my case, in the normal size of my screen

But when I zoom chrome to 150%, then yes I am able to reproduce it, and it is also happening with other modals too(e.g edit user modal)

Also, I don't know if this issue comes up elsewhere, but I noticed that the "g" in "Long text" is a bit cut off:

It is not happening in my case, in the normal size of my screen

But when I zoom chrome to 150%, then yes I am able to reproduce it, and it is also happening with other modals too(e.g edit user modal)

Nice, this is such an improvement!
The main thing I noticed is that when the inputs are invalid, we should disable the "Save changes" button. The error message text should be shown in a tooltip when hovering over the "Save changes" button, without the "Failed: " part. Some examples (not exhaustive) in case it's helpful:

As you can see here #21864 we are trying to show this type of error inside the modal for all modals in settings, so In case of consistency, we may need to change other models too.

I also noticed that the error banner doesn't get cleared when you switch field types (it should), but this may no longer be relevant after the change above.

For now, I fixed this.

Replacing the giant red error banners with a different error display approach should be out of scope for this PR. I certainly want to get rid of those red banners, but it's a pattern across 20+ modals, and we'll want to have an independent discussion of changes there.
I'm also not sure that a change to disable the button after an error is correct -- we certainly wouldn't want to do that after a server 502 (i.e. a server capacity error), since retrying with no changes is the correct response to that sort of error.

Got it, I hadn't realized how complex this is -- makes sense to me!

In that case, we should fix the "Malformed URL pattern" error to be "Failed: Invalid URL." The current error text doesn't make sense in this context.

In that case, we should fix the "Malformed URL pattern" error to be "Failed: Invalid URL." The current error text doesn't make sense in this context.

I think this error message is actually correct. What we're asking you to do is provide a format that, if a username is substituted in, will form a URL. Here's the code:
    if s.count("%(username)s") != 1:
        raise ValidationError(_("Malformed URL pattern."))

I think an improvement would be to instead say "URL pattern must contain '%(username)s'.", but the thing you're inputting is a "URL pattern".

@timabbott are we changing the error message?

PR updated.

I merged 08e8a21da06541f1eedf5db526fe171c0498291b for the little error message change.

@yogesh-sirsat for the main commit; I'd appreciate your splitting it so that it's easier to review. When I look at git show, I see a lot of things like this:

It's far easier to read that change as a separate preparatory commit that just renames the field_elem to $field_elem than having that squashed with rewriting several modals. Check out the Zulip commit guidelines for more details: https://zulip.readthedocs.io/en/latest/contributing/version-control.html.
If you can do these variable name refactors in preparatory commits, it'd really help with reading your actual changes.

I am not just renaming these variables, I am changing them and because of this if we create a separate commit, it can produce errors.
For making it easy to review I can split this commit in this two different modals

@timabbott If I am missing something here please inform me.

LGTM.
One thing we can do is probably make the last commit cleaner by having a prep commit to remove get_profile_field_info function and instead simply define two different variables for the form and row and change the variables. And then have the original commit to add the modal code without requiring those variable-only changes.

@sahil839 I have made changes as you said have a look.

You can use the form_id parameter so that pressing Enter on an input field submits the form. See this for an example.

I tried but it is not working.
When I tried to update bots to see how it is working by pressing the enter button, it is giving this error  Error: Expected one element in jQuery set, 2 found after this error when I clicked on edit bot button again this error occurred twice at a single time, I was using owners log in account in dev and the branch was clean main.

Focusing the Label input when the modal opens would be nice.

You can use the form_id parameter so that pressing Enter on an input field submits the form. See this for an example.

@ganpa3 I tried but it is not working.
When I tried to update bots to see how it is working by pressing the enter button, it is giving this error  Error: Expected one element in jQuery set, 2 found after this error when I clicked on edit bot button again this error occurred twice at a single time, I was using owners log in account in dev and the branch was clean main.

I tried and it's working. Can you specify the exact place the error is occurring?

You can use the form_id parameter so that pressing Enter on an input field submits the form. See this for an example.

@ganpa3 I tried but it is not working.
When I tried to update bots to see how it is working by pressing the enter button, it is giving this error  Error: Expected one element in jQuery set, 2 found after this error when I clicked on edit bot button again this error occurred twice at a single time, I was using owners log in account in dev and the branch was clean main.

I tried and it's working. Can you specify the exact place the error is occurring?

In settings > bots, edit bot

Actually, I gettings this error on every modal -- open a modal and press enter.

LGTM.

Re-reviewing this PR, let's change the "confirm" button label to "Add" rather than "Save changes" for the adding modal. The editing modal is fine as-is. (see CZO discussion)

Otherwise, looks good to me!

Re-reviewing this PR, let's change the "confirm" button label to "Add" rather than "Save changes" for the adding modal. The editing modal is fine as-is. (see CZO discussion)

@alya PR updated with this change.

@timabbott reviewed by @ganpa3 , @sahil839 and myself, and ready for you to take a look!

Merged, after fixing the two issues noted above, and rewriting several of the commit messages to either motivate the changes or explain why they're correct.


example: add a continuous deployment (CD) workflow using terraform

add a new examples/cd/ folder, and then create  examples/cd/terraform.yml that exemplifies how to run terraform in a workflow, as described here. The image used in the workflow can be the official one

Cd tensorflow

In addition to #929, I will begin work on this as well.


the link to stable version of this codelab is broken

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/MyMethod
Description of issue (what needs changing):
Clear description
For example, why should someone use this method? How is it useful?
Correct links
Is the link to the source code correct?
Parameters defined
Are all parameters defined and formatted correctly?
Returns defined
Are return values defined?
Raises listed and defined
Are the errors defined? For example,
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/feature_column/categorical_column_with_vocabulary_file#raises
Usage example
Is there a usage example?
See the API guide: https://www.tensorflow.org/community/contribute/docs_ref
on how to write testable usage examples.
Request visuals, if applicable
Are there currently visuals? If not, will it clarify the content?
Submit a pull request?
Are you planning to also submit a pull request to fix the issue? See the docs
contributor guide: https://www.tensorflow.org/community/contribute/docs,
docs API guide: https://www.tensorflow.org/community/contribute/docs_ref and the
docs style guide: https://www.tensorflow.org/community/contribute/docs_style

@navczydev
Please share the link mentioned in the issue.

Here is the link : https://developer.android.com/codelabs/recognize-flowers-with-tensorflow-on-android/index.html?index=..%2F..index#0
and it's. in this codelab  👇
https://developer.android.com/codelabs/recognize-flowers-with-tensorflow-on-android-beta#0

Can you please point the page where this is hosted?
We have Recognize flowers with TensorFlow codelab tutorial on TF website on this web page

If you go to this link
https://developer.android.com/codelabs/recognize-flowers-with-tensorflow-on-android-beta#0
and there is a link to stable version of this codelab which is broken

Do you mean Colab from this web page?

Clicking on Colab takes you to https://colab.sandbox.google.com/github/tensorflow/examples/blob/master/lite/codelabs/flower_classification/ml/Flower_Classification_with_TFLite_Model_Maker.ipynb correctly.


Automated cherry pick of #2452: add update vm name method

Cherry pick of #2452 on release/2.11.
#2452: add update vm name method

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@b70c8a6). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.11   #2454   +/-   ##
==============================================
  Coverage                ?   7.07%
==============================================
  Files                   ?     454
  Lines                   ?   71720
  Branches                ?       0
==============================================
  Hits                    ?    5075
  Misses                  ?   66227
  Partials                ?     418

Flag
Coverage Δ

#aFlag
7.07% <ø> (?)

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 b70c8a6...5c26f73. Read the comment docs.

/lgtm

/approve


Fix: Attachments not shown for sender, stuck at 'Uploading attachment...'

@roryabraham
Details
Corrected the passed parameter/value for prevVisibleMessageText translateLocal.
Fixed Issues

$ https://github.com/Expensify/App/issues/22750
PROPOSAL: https://github.com/Expensify/App/issues/22750#issuecomment-1632638487
Tests

Send an attachment to any user
Verify the text Uploading attachment... replaced with the attachment once it is sent successfully

[X] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Send an attachment to any user
Verify the text Uploading attachment... replaced with the attachment once it is sent successfully

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/14358475/49a1f693-759c-43fe-a78f-5c6e5675aa55

Mobile Web - Chrome
https://github.com/Expensify/App/assets/14358475/32ed99eb-11f6-401e-ae7b-465246d57b15

Mobile Web - Safari
https://github.com/Expensify/App/assets/14358475/27693750-2e1b-4ef7-a2ea-31f4ff9fd963

Desktop
https://github.com/Expensify/App/assets/14358475/32797b44-1294-4df4-894f-c07d811b58ec

iOS
https://github.com/Expensify/App/assets/14358475/5f80cffc-51e7-478d-84b7-7a38b9445f90

Android
https://github.com/Expensify/App/assets/14358475/4af243e3-9799-4974-a57a-b32dd5f83941

@rushatgabhane its working for all platforms. Updated the screencast for all.

@Pujan92 can you try uploading a Document?

@Pujan92 can you try uploading a Document on iOS

Seems it gets uploaded.

@Julesssss can you please help us out here
I'm not able to upload documents on native. Is it just my device or the PR?

Use the links below to test this build in android and iOS. Happy testing!

:test_tube::test_tube: Use the links below to test this build in android and iOS. Happy testing! :test_tube::test_tube:

android :robot:
iOS :apple:

https://ad-hoc-expensify-cash.s3.amazonaws.com/android/22836/index.html
https://ad-hoc-expensify-cash.s3.amazonaws.com/ios/22836/index.html

desktop :computer:
web :spider_web:

https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/22836/NewExpensify.dmg
https://22836.pr-testing.expensify.com

I'm not able to upload documents on native. I get this error - https://github.com/Expensify/App/pull/22836#issuecomment-1635259003
Is it just my device or the PR?

It works for me on Android using the link above. I get a different error on web/dev.
@rushatgabhane Can you try the above links?

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.3.42-0 🚀

platform
result

🤖 android 🤖
cancelled 🔪

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
cancelled 🔪

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.42-26 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


windows: fix advanced settings loading

Based / Blocked on #2795
Fixes #2807 where I accidentally bail out of the controller task if the settings file is missing or isn't valid in any way.

Tested the exe produced by CI on my unprivileged account and it's fixed. Just waiting on #2795 to land and we can un-draft this.

I'm gonna redo this PR, I don't think it's supposed to be based off of main. Something looks weird.


Cleanup like and nlike utf8 kernels

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
There is quite some repitition in the like / nlike utf8 kernels.
Describe the solution you'd like

Create a generic method that can handle like/nlike kernels.
Describe alternatives you've considered

Additional context

label_issue.py automatically added labels {'arrow'} from #2743


Lock engine to prevent read/write conflict on engine containers map

Thank @vieux for the discussion.
Fix #1805. It's not very sure but we don't have reproduce steps available.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

LGTM
If you don't mind, can you describe a little more about the exact conflict that was being caused?

@nishanttotla There are a few places Engine.containers can be updated on the fly. It might cause access violation when it's recreated like below.
    // Wipes the internal container state.
    func (e *Engine) cleanupContainers() {
       e.Lock()
       e.containers = make(map[string]*Container)
       e.Unlock()
    }

LGTM

@dongluochen thanks! that makes sense.

Job: Swarm-PRs (engine 1.9) [FAILED](https://jenkins.dockerproject.org/job/Swarm-PRs (engine 1.9)/64/console):

---
- docker network rm
ok 160 mesos - docker network disconnect connect
ok 161 mesos - docker ps
ok 162 mesos - docker run with wrong user
ok 163 mesos - docker run
ok 164 mesos - docker run short lived
ok 165 mesos - docker run no resources
ok 166 mesos - docker run with long pull
ok 167 mesos - docker stats
ok 168 mesos - docker-compose ps
ok 169 mesos - docker-compose scale
ok 170 mesos - docker-compose up
ok 171 mesos - zookeeper
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Swarm-PRs (engine 1.9)] $ /bin/sh -xe /tmp/hudson5571676988716988481.sh
+ set +x

---


module 'bitsandbytes' has no attribute 'matmul_mixed'

hi, when I tried to test SwitchBackLinearBnb in order to verify the performance differences with the corresponding API in triton, I got this error message:
AttributeError: module 'bitsandbytes' has no attribute 'matmul_mixed'

Is there any specific version I need to run SwitchBackLinearBnb?
Thanks ahead.

Hi @FrostML! Have you solved this problem?


cursor marker disappear while hovering over item

Describe the bug
The cursor marker disappears while hovering over an item
To Reproduce
hover over item
https://codesandbox.io/s/n920olmyxj
Expected behavior
the cursor should be shown by default even though the cursor is over an item
Library Version
0.18
Desktop (please complete the following information):

OS: macOS
Browser Chrome

Additional context
This problem occurs because the MarkerCanvas has no z-index value whereas the item z-index is 80. I think the MarkerCanvas should get the highest z-index of all layers. This will make items though undraggable (Not sure how to go with this to be honest)

What do you think about maybe track the mouse over event on the root canvas div?

@Ilaiwi  I'll try this, or have the MarkerCanvas accept children, or something 😄


kf_lidar_track advertises /detected_objects but never publishes

Bug
Expected Behavior
kf_lidar_track advertises /detected_objects:
pub_detected_objects_ = node_handle_.advertise<autoware_msgs::DetectedObjectArray>( "/detected_objects", 10);
It would be expected to be able to use kf_lidar_track with nodes that subscribe to /detected_objects such as potential_field.
Actual Behavior
kf_lidar_track never publishes /detected_objects.
the Publish method is never called on pub_detected_objects defined above.
Steps to Reproduce the Problem

launch euclidean_cluster
launch svm_lidar_detect
launch kf_lidar_track
Monitor topics

Specifications

Ubuntu version: 16.04
ROS Version: Kinetic
Autoware branch: master

I think there’s a problem on the way you’re connecting the Euclidean cluster to the kf tracker
Please check

Euclidean cluster is publishing CloudClusterArray
You selected the correct output frame in Euclidean Cluster
If you’re using wayarea to remove clusters from outside the road, check that wayarea2gridmap is publishing the grid map and that the correct topic is being subscribed

You can check all of these in the app button from Euclidean Cluster node

I've noticed similar behaviour, though when selecting kf_contour_tracker instead of kf_lidar_track, euclidean_cluster publishes to /cloud_clusters.
I didn't need to change any of the settings within 'app'.

@amc-nu thanks for the response. I checked what you asked for:

Euclidean cluster is publishing a CloudClusterArray.
The output frame is selected correctly.
I am not using wayarea.

I think you may not have understood the issue I've observed correctly. kf_lidar_track simply does not publish /detected_objects. It only advertises it. publish is  never called for this topic in the source code.

yeah bro in the source code they never called the publish for the source code. i have added by myself.


How can I change the certificate name?

When downloading the certificate for decrying HTTPS decrypt on a device, the name of the certificate shows up as "mitmproxy" Is there any way to change its name?

There is no builtin way. If you want to do it, you have to read and modify the source code.

Thank you for the reply. That will not be a problem, but where is the file with the certificate's information so that I can change it? I attempted to go through all of the files related to certificates but it's too much


feat(VIST-764): Change value type of visitDate from Int to LocalDateTime

JIRA: link to jira ticket
https://champlainsaintlambert.atlassian.net/browse/CPC-764
Context:
This pull request is to switch the date value types in the visits-service to type LocalDateTime. LocalDateTime is a more precise and suitable data type for date and time information and is a step towards maintaining better consistency throughout the code base.
Changes
-Value type for Visit Dates has been changed to LocalDateTime from type Int
-All logic pertaining to findVisitByPractionerIdAndMonth has been disabled as it stands
Before and After UI (Required for UI-impacting PRs)
UI Before change

UI After change

Could you address why the UI has not been updated with the new DateTime. The different data type should be affecting the format of the date in the Visits-List front end, yet no UI change images were provided for this. The front-end related date functionality for visits-list needs to be updated before this pull request can be merged.

Hey will, this is the reply I meant to address. Thank you for checking my code thoroughly, this should have been implemented


Swift 3.0 support

Migrated the project to Swift 3.0. Do not merge this one before Xcode 8 is finally released. To use it in the meanwhile use the following pod:
pod 'ObjectMapper', :git=> 'git@github.com:e-Sixt/ObjectMapper.git', :branch => 'master'

I think it would be good to have an official Swift 3 branch here in the ObjectMapper repo.
Opened an issue about this.

Hi there, I have created a swift-3 branch. @batschz can you please create a new PR with your work onto that branch?


Using six for xrange

I went through a similar thought process - I considered adding a helper instead of six but then figured I'd be reinventing the wheel with code that wasn't particularly clear.


feat: Implemented reset method

Hello! It will be very useful to have reset method in Lzxd. I need it in rust-cab crate.

@Lonami it will be very cool to release these changes


[feature request] explaination to do a basic search replace on key/value/name traversing all the registry

I'm a basic python user searching for a way to find and replace (everywhere in the registry) "old" by "new" (in name, key, value, even in the protected keys).
It would be nice to add to your documentation basic explaination (for basic user)
(Python 3.6, windows 7).

hey @WiliTest
thanks for submitting this feedback! unfortunately, it sounds like python-registry probably will not work for your needs. this library only supports reading data, and cannot write to registry files. you might find the _winreg module (in the python standard library) more suitable.


Move css rules for #165169

Ref https://github.com/microsoft/vscode/issues/165169
Use css vars for the following files:

[x]  src/vs/editor/browser/viewParts/scrollDecoration/scrollDecoration.ts
[x]  src/vs/editor/browser/widget/diffReview.ts
[x]  src/vs/editor/standalone/browser/accessibilityHelp/accessibilityHelp.ts
[x] src/vs/editor/contrib/bracketMatching/browser/bracketMatching.ts

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

Thanks @weartist !


Not installing on coreos 1235.6.0

When I try to install a cluster on 4 Coreos nodes I get :

Running task: Install python 2.7
coreos-01                                                                     [ERROR]
---- STDOUT ----
which: no python in (/bin:/sbin:/usr/bin:/usr/sbin:/usr/local/bin:/usr/local/sbin:/opt/bin)
/home/core/.ansible/tmp/ansible-tmp-1484128659.62-36746745770822/python27.sh: line 2: apt-get: command not found
ln: failed to create symbolic link '/usr/bin/python': Read-only file system

---- STDERR ----
OpenSSH_6.7p1 Debian-5+deb8u3, OpenSSL 1.0.1t  3 May 2016
debug1: Reading configuration data /etc/ssh/ssh_config
debug1: /etc/ssh/ssh_config line 19: Applying options for *
debug1: /etc/ssh/ssh_config line 57: Applying options for *
debug1: auto-mux: Trying existing master
debug2: fd 3 setting O_NONBLOCK
debug2: mux_client_hello_exchange: master version 4
debug3: mux_client_forwards: request forwardings: 0 local, 0 remote
debug3: mux_client_request_session: entering
debug3: mux_client_request_alive: entering
debug3: mux_client_request_alive: done pid = 27278
debug3: mux_client_request_session: session request sent
debug1: mux_client_request_session: master session id: 2
debug3: mux_client_read_packet: read header failed: Broken pipe
debug2: Received exit status from master 1
Shared connection to <IP_ADDRESS> closed.

error running preflight: error running playbook: error running ansible: exit status 2
Why does it try to use apt-get on Coreos?
Kind regards, Paul

We don't support CoreOS at the moment. Take a look at https://github.com/apprenda/kismatic/blob/master/docs/PLAN.md#hardware--operating-system for a list of operating systems we currently support.


@zeit/next-workers: ValidationError: Invalid options object.

worker-loader released a new major update v3.0.0 on 2020-08-01. This update doesn't work with<EMAIL_ADDRESS>anymore.
The error it produces is:
error -
ValidationError: Invalid options object. Worker Loader has been initialized using an options object that does not match the API schema.
 - options has an unknown property 'name'. These properties are valid:
   object { worker?, publicPath?, filename?, chunkFilename?, inline?, esModule? }

In<EMAIL_ADDRESS>name is renamed to fileName like mentioned in the changelog.
There are 2 solutions:

Only use<EMAIL_ADDRESS>instead of v3.0.01.

Just change the name to fileName at line#14  in the source of @zeit/next-workers.

There are so many pending pull requests in this repo. So, I'm skeptical about the team accepting my pull request. So, this might help future developers who are having this issue.
If the maintainers come across this, please update the readme with this or even better update the plugin to use the latest worker-loader v3.0.1.

@stuartmemo I haven't explored other options as I've just reverted back to<EMAIL_ADDRESS>as it was enough for my use case. But it's not that hard to use webpack loaders in Next.js by yourself. I'm positive there would be plenty of loaders for webpack and you can follow this guide Custom Webpack Config to use that webpack loader in your Next.js.

Amazing, thanks for the pointer. That was remarkably easy. For those following along at home, just remove the plugin and add this to your next.config.js...
    webpack: (config) => {
        config.module.rules.push({
            test: /\.worker\.js$/,
            use: { loader: 'worker-loader' },
        });

        return config;
    }


Image tiling helper functions

This is for internal use only; if you'd like to open an issue or request a new feature, please open a bug or enhancement issue
Section 1: Design details
Relevant background
Rather than stitching FOV images together according to acquisition order, we want a stitch_tiled_image function that puts the image back together based on the original shape it was acquired from (assuming it was generated using the 2_create_tiled_mibi_run jupyter notebook in toffy, and will result in an RNCM image). After renaming the FOVs with the reorg notebook in toffy, the folders will be in RNCM format. We want to reassemble the images back into that same original shape, while also allowing for missing FOVs.
Design overview
This will mirror the stitch_images() flow in toffy, but will use new helper functions more appropriate to the desired resulting image. There will be a single stitched_tiled_images() function for the user to run.

Have some kind of validation in stitched_tiled_images to check if the image dir provided actually contains tiled image data. Folder name check (RNCM)??

get_stitched_dimensions() is a helper function to determine what the image dimensions should be based on the folder names, getting max row and col values
fill_blank_fov_data() is a helper function to fill in blank image data for any missing fovs
(this may actually be simple enough to just include the code in the load function and not be its own helper function)
load_tiled_img_data() will load in the image array from a base directory (similar to load_imgs_from_tree and other functions) that loads one fov at a time and uses fill_blank_fov_data() for any that are missing image data
stitched_tiled_images() will be the user function calling the above functions, and doing any validation checks and such

Code mockup
def stitch_tiled_images(data_dir, img_sub_folder=None, channels=None):

	# check for previous stitching
	tiled_dir = os.path.join(data_dir, 'stitched_images')
	if os.path.exists(tiled_dir):
    		raise ValueError(f"fThe tiled_images subdirectory already exists in {data_dir}”)

	folders = ns.natsorted(io_utils.list_folders(tiff_out_dir, substrs='fov-'))
         _, num_cols = get_stitched_dimenstions(folders)

	# no img_sub_folder, change to empty string to read directly from base folder
	if img_sub_folder is None:
    		img_sub_folder = ""

	# retrieve all extracted channel names, or verify the list provided
	if channels is None:
    		channels = remove_file_extensions(io_utils.list_files(
        	dir_name=os.path.join(data_dir, folders[0], img_sub_folder), substrs='.tiff'))
	else:
    		misc_utils.verify_in_list(channel_inputs=channels, valid_channels=remove_file_extensions(
        		io_utils.list_files(dir_name=os.path.join(data_dir, folders[0], img_sub_folder), substrs='.tiff')))

	# make tiled subdir
	os.makedirs(tiled_dir)

# main code logic
----------------

	# save the tiled images to the tiled_images subdir, one channel at a time
	for chan in channels:
    		image_data = load_tiled_img_data(data_dir, img_sub_folder, channels=[chan])
    		tiled = data_utils.stitch_images(image_data, num_cols)
    		current_img = tiled.loc[‘tiled_image', :, :, chan].values
    		io.imsave(os.path.join(tiled_dir, chan + ‘_tiled.tiff'), current_img.astype('float32'), check_contrast=False)

----------------

def load_tiled_img_data(data_dir, img_sub_folder=None, channels=None)

	validate_paths(data_dir, data_prefix=False)
	if img_sub_folder is None:
		img_sub_folder = “”

	max_img_size = get_max_img_size(data_dir)
	fov_list = ns.natsorted(io_utils.list_folders(data_dir))
	rows, cols = get_stitched_dimensions(fovs)
	expected_fovs = []
	for n in range(rows):
    		for m in range(cols):
        		expected_fovs.append(f"R{n+1}C{m+1}")

	if channels is None:
		channels = iou.list_files(dir_name=os.path.join(data_dir, fov_list[0], img_sub_folder), substrs=['.tiff’])

 	if len(fov_list) == len(expected_fovs):
                # load data normally
		img_xr = load_imgs_from_tree(data_dir, img_sub_folder, channels, max_img_size)
		return img_xr
	else:
                test_img = io.imread(os.path.join(data_dir, fov_list[0], img_sub_folder, channels[0]))
                # the dtype is always the type of the image being loaded in.
	        dtype = test_img.dtype

# main code logic
----------------

		img_data = np.zeros((len(expected_fovs), max_img_size*rows, max_img_size*cols, len(channels)), dtype=dtype)

		for fov, fov_name in enumerate(expected_fovs):
	            if fov_name in fov_list:
		       fov_digits = re.findall(r'\d+', fov)
                       # -1 for indexing
                       start_row = (fov_digits[0]-1)*max_img_size
                       start_col = (fov_digits[1]-1)*max_img_size

		       for img in range(len(channels)):
			  temp_img = io.imread(os.path.join(data_dir, expected_fovs[fov], img_sub_folder, channels[img]))
			  # fill in specific spot in array
			  img_data[fov, start_row:start_row+temp_img.shape[0], start_col:start_col+temp_img.shape[0], img] =temp_img

-----------------
	# check to make sure that dtype wasn't too small for range of data
	if np.min(img_data) < 0:
    		warnings.warn("You have images with negative values loaded in.")

	row_coords, col_coords = range(img_data.shape[1]), range(img_data.shape[2])

	# remove .tiff from image name
	img_names = [os.path.splitext(img)[0] for img in channels]
	img_xr = xr.DataArray(img_data, coords=[fovs, row_coords, col_coords, img_names],
                      		dims=["fovs", "rows", "cols", "channels"])

	return img_xr

Required inputs
The inputs are an data_dir and img_sub_folder which hold the images we want to stitch and an optional channels list.
Output files
There will be a new folder in the data_dir named tiled_images that contains a single tiled image of all fovs for each channel comprised.'
Timeline
Give a rough estimate for how long you think the project will take. In general, it's better to be too conservative rather than too optimistic.

[ ] A couple days
[ ] A week
[x] Multiple weeks. For large projects, make sure to agree on a plan that isn't just a single monster PR at the end.

Estimated date when a fully implemented version will be ready for review:
Thursday 9/15
Estimated date when the finalized project will be merged in:
Friday 9/23 (latest)

@ngreenwald pretty sure I have the image array logic set and added the timeline

When you're loading the image data here:
for img in range(len(channels)):
	temp_img = io.imread(os.path.join(data_dir, expected_fovs[fov], img_sub_folder, channels[img]))
	# fill in specific spot in array
	img_data[fov, start_row:start_row+temp_img.shape[0], start_col:start_col+temp_img.shape[0], img] =temp_img

you're creating a stitched image. But then you also call the stitch image function here:
# save the tiled images to the tiled_images subdir, one channel at a time
	for chan in channels:
    		image_data = load_tiled_img_data(data_dir, img_sub_folder, channels=[chan])
    		tiled = data_utils.stitch_images(image_data, num_cols)
    		current_img = tiled.loc[‘tiled_image', :, :, chan].values
    		io.imsave(os.path.join(tiled_dir, chan + ‘_tiled.tiff'), current_img.astype('float32'), check_contrast=False)

Which is expecting just a regular load_img_from_tree df. The point of using the existing stitched image code would be to avoid stitching it yourself. You can also just stitch it yourself, but then you wouldn't need this function. Up to you.
Given how far along the code is, I suspect it will take you less time than indicated, but always better to start off conservative. Looks good to go.


Android Can't Build: appcompat-v7 AAPT: error: resource android:attr/fontVariationSettings not found.

Issue
I know there is an recent issue from the new google api update. But I can seem to figure this out after many hours of debugging. Any help or insight would be greatly appreciated!
:react-native-push-notification:verifyReleaseResources/home/circleci/.gradle/caches/transforms-1/files-1.1/appcompat-v7-27.1.1.aar/537c9158e84b60ea7e5743fd405e631a/res/values/values.xml:251:5-69: AAPT: error: resource android:attr/fontVariationSettings not found.

/home/circleci/.gradle/caches/transforms-1/files-1.1/appcompat-v7-27.1.1.aar/537c9158e84b60ea7e5743fd405e631a/res/values/values.xml:251:5-69: AAPT: error: resource android:attr/ttcIndex not found.

android build.gradle
ext {
  compileSdkVersion = 26
  buildToolsVersion = "27.0.3"
  supportLibVersion = "27.1.1"
  googlePlayServicesVersion = "16.+"
}

buildscript {
    ext {
        buildToolsVersion = "27.0.3"
        minSdkVersion = 19
        compileSdkVersion = 26
        targetSdkVersion = 26
        supportLibVersion = "27.1.1"
        googlePlayServicesVersion = "16.+"
    }
    repositories {
        google()
        maven{
            url 'https://dl.bintray.com/android/android-tools'
        }
        maven {
            url 'https://maven.google.com/'
            name 'Google'
        }
        jcenter()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:3.1.4'
        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

app build.gradle
configurations.all {
    resolutionStrategy {
        force 'com.google.android.gms:play-services-gcm:16.1.0'
    }
}

dependencies {
    implementation(project(':react-native-push-notification')) {
        exclude group: 'com.google.android.gms'
    }
    implementation 'com.google.android.gms:play-services-base:16.1.0'
    implementation 'com.google.android.gms:play-services-maps:16.1.0'
    implementation 'com.google.android.gms:play-services-gcm:16.1.0'
    implementation 'com.google.android.gms:play-services-basement:16.1.0'
}

React Native version:
    Binaries:
      Node: 8.11.4 - /var/folders/p3/kvdgygxj5tv_wbvzk308kgz00000gn/T/yarn--1561057276714-0.4231988255188943/node
      Yarn: 1.16.0 - /var/folders/p3/kvdgygxj5tv_wbvzk308kgz00000gn/T/yarn--1561057276714-0.4231988255188943/yarn
      npm: 5.6.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
    IDEs:
      Android Studio: 3.4 AI-183.6<IP_ADDRESS>22156
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.0-alpha.8af6728 => 16.6.0-alpha.8af6728
      react-native: ^0.57.4 => 0.57.4
    npmGlobalPackages:
      react-native-cli: 2.0.1

Steps To Reproduce

./gradlew assembleSignedRelease

Same issue with the latest release, v0.59...
$ react-native info
info
  React Native Environment Info:
    System:
      OS: Linux 4.18 Ubuntu 18.04.2 LTS (Bionic Beaver)
      CPU: (4) x64 Intel(R) Core(TM) i5-4460S CPU @ 2.90GHz
      Memory: 4.91 GB / 15.58 GB
      Shell: 4.4.19 - /bin/bash
    Binaries:
      Node: 11.14.0 - ~/.nvm/versions/node/v11.14.0/bin/node
      Yarn: 1.16.0 - /usr/bin/yarn
      npm: 6.7.0 - ~/.nvm/versions/node/v11.14.0/bin/npm
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.9 => 0.59.9
    npmGlobalPackages:
      react-native-cli: 2.0.1

Any clue?
We are stuck desperate!

same issue. I did tryed<EMAIL_ADDRESS>but got others issues..

In this post I found the solution:
https://forum.ionicframework.com/t/firebase-app-unable-to-compile-on-android-with-aapt-error/166564/6

I think I finally got it. I called ./gradlew app:dependencies and made sure that every single library from https://developers.google.com/android/guides/releases#june_17_2019 was that being used was using the version that was not updated recently

How to resolve this error ?? Anyone help me?
android/build.gradle File
buildscript {     ext {         buildToolsVersion = "27.0.3"         minSdkVersion = 16         compileSdkVersion = 27         targetSdkVersion = 26         supportLibVersion = "27.1.1"         // googlePlayServicesVersion = "16.+"     }


Delete CODE_OF_CONDUCT.md

.github repo org wide coc present. this file present means that it opts-out of that which we don't want.

@swift-ci test


Hi

As the author of git-bug, it's interesting to see that we made the same tech choice (go, gocui, cobra). Very neat and clean code, good job !
I'm just slightly annoyed by how good your demo.gif is. Could you share which tool/config you used to make it ?
Thanks a lot :-)

LiceCap does the same thing I think.

I use Giphy Capture

We indeed made similar choices, @MichaelMure --and thanks for noticing the clean[ish] code! As for the demo gif I ended up using https://github.com/phw/peek , they did a pretty solid job with the interface and minimal options. And since we're on useful tools, goreleaser (https://github.com/goreleaser/goreleaser) has also been pretty helpful too throughout this process.
Git-bug is an awesome idea, I'm curious to see where it goes!

Thank you very much :-)


Cannot find txt files (AU.txt, EXPR.txt, and VA.txt)

I want to test on my own video, so I tried emotion_demo.py, and modifid the content of video_file. But I cannot find the three text files: AU.txt, EXPR.txt, and VA.txt. Where can I find or download them?

Hey @richard-bmc I am facing the same problem, can you tell me how you fixed it?


How to write extension  - to compile serenadejs views

Hi,
I am wondering how hard it would be to do a middleman extension to compile serenadejs views.
There is a serenade_rails  extension that does something similar for Rails 3.1 - https://github.com/elabs/serenade_rails
The core bit of the extension would need to load the view files and "parse" them with serenade, like so:
def parse
  context = ExecJS.compile(File.read(SERENADE_PATH))
  code = "Serenade.view(#{content.to_json}).parse()"
  context.eval(code)
end

Then it would need to "render" some javascript, like so:
def render
  "Serenade.view(#{name.to_json}, #{parse.to_json});"
end

Thinking to base it on the sprockets extension - any tips on where to start on an extension.
The middleman-jammit seems like it must be doing something similar-ish...
~chris

I agree, Sprockets is probably the right place. See also the rasputin and ember-rails gems which precompile templates (Handlerbars) in sprockets.

Closing this, but happy to provide direction in implementing this extension.

No problem and thanks - due to the nature of the current app (embedded on Smart TV), I am going for the option of getting the views remotely during startup, so no immediate need for this now.  Thanks for all the help/tips so far :)