Desktop emulator feature (text editor, ide and/or compiler for non resource intensive languages, browser, terminal, systemd, virtual keyboard that is navigated with controller L1 R1 an and b buttons, calculator, music player, email, [pentester tools]) just some ideas of what could be done, ( I use an anbernic rg351p)

Before submitting your issue or feature request.  DO NOT REQUEST ASSISTANce FOR PRELOADED IMAGES.  I do not support or condone the use of preloaded ArkOS images.  Preloaded meaning images that come with copyrighted roms preincluded.  Look for support from the creator or supplier of those preloaded images, not from here!  If a ticket is submitted for assistance for such images or if there's an indiciation that a preloaded image is what the support request is for, the issue ticket will be closed with no further response.

This is not the Stock OS that comes with your device, AmberElec, JelOS, or UnOfficialOS.  ArkOS is a different distribution from those systems and will have differing features.  The only somewhat similarity is that they both use EmulationStation as a frontend.  Even with that, the versions of EmulationStation that's used in between ArkOS and these other distributions is different with different capabilities.  If there are features you prefer with the other available distributions, just use those distributions instead.  The choice is yours.

Have you reviewed the Frequently Asked Questions section available for your supported device in the wiki?

If your issue is related to being able to run certain emulators or ports, have you reviewed the Emulators and Ports section available at https://github.com/christianhaitian/arkos/wiki/ArkOS-Emulators-and-Ports-information for needed bios files and supported rom extensions?

Have you reviewed the known issues section available at https://github.com/christianhaitian/arkos/wiki/Known-issues ?

Describe the Issue (If applicable)
A clear and concise description of what the bug is.
How can the issue be reproduced? (If applicable)
What device are you using? (Only the Gameforce Chi, RK2020, OGA 1.0, OGA 1.1, RG351MP, RG351V, RGB10, R35s, RG353V/VS, RG353M, RG503, RGB30, and RK2023 units are officially supported)
What version of ArkOS are you on?

You can find this during the initial boot process right before the Emulationstation loading screen

Anything else you'd like to include that may help to review this issue or feature request?

Not interested in implement most of these things.  It is possible to use ArkOS as a development environment.  There's instructions how to do so on the wiki.  I haven't supported the rg351p in years though.


Editor (current one) breaks when creating map from dataset

A client has just reported this, and it is reproducing both in team and free accounts.
When clicking on VISUALIZE from a dataset (either data or map view) the Editor shows an error and the map is not created
STR

Go to Datasets dashboard and open a dataset
Click on VISUALIZE and confirm
This happens

In the console we can see this:
cartodb.uncompressed.js:28 POST https://ernesmb.cartodb.com/api/v1/viz 403 (Forbidden)

We are aware of this and a fix is underway:
https://github.com/CartoDB/cartodb/pull/8118

Just started deploying, should be fixed in ~20 minutes.
Thanks for reporting!


Light mode

When searching through the code base, it looks like this was implemented and never turned on. Would be really nice to have as I have other documentation which is always in light mode. It is quite jarring to switch from light to dark.

Are you referring to the docs site? It should have dark and light mode

Sorry no, the app itself.

@dotansimha @kamilkisiela I can take this task after laboratory enhacements


multiple page routing doesn't work

I ran into the impossibility of using the multiple pages mode because I got only one error in this case::
Package subpath './server.js' is not defined by "exports" in D:\projects\js\vite_ssg\vite-react-ssg\examples\multiple-pages\node_modules\react-router-dom\package.json imported from D:\projects\js\vite_ssg\vite-react-ssg\examples\multiple-pages\node_modules\vite-react-ssg\dist\shared\vite-react-ssg.CCvTYWeJ.mjs.
This is the same with the example from the appropriate folder in the repository.
I use the following versions:
react: 18.3.1,
react-dom: 18.3.1,
react-router-dom: 7.0.2,
vite-react-ssg: 0.8.1
If I made a mistake here, I would appreciate it if someone could tell me how to fix it. Thanks for your support!

Have you used react-router-dom v6? I haven't tested v7 yet, as it was just released.
Note: This does not mean that I won't support v7. I'm just trying to determine if the issue is caused by the react-router version. If not, I will prioritize resolving this issue.


Add monitor filters

Looks good, but you have the rebase on master before it can be merged

Yes that is correct, you can no combine it with OkanshiMonitor. You have to create a monitor manually before passing it in, or you extend OkanshiMonitor with the wrapping methods

So how do we proceed?
We could extend okanshimonitor with a WithFilter eg
OkanshiMonitor.WithZeroFilter().Counter("c")

become
    let config = MonitorConfig.Build(name).WithTags(OkanshiMonitor.DefaultTags).WithTags(tags)
    monitorRegistry.GetOrAdd(config, fun x -> xxxFilter(new MaxGauge(x)))

Im not sure we can prevent the following (which I think is ok)
OkanshiMonitor.Counter("c")
OkanshiMonitor.WithZeroFilter().Counter("c")

which creates two monitors with the same name but different filtering
Finally, the filters do not take a MonitorConfig in the ctor - all the monitors do that - so should the filters do the same?

It seems like we need to change the poller MetricMonitorRegistryPoller as well as it sends data to the observer irregardless of any data in the monitor

So you want me to revert so observers are always called ?

So you want me to revert so observers are always called ? The influx observer doesn't check if values is empty either..

So you want me to revert so observers are always called ? The influx observer doesn't check if values is empty either..


arch/sim: add CONFIG_SIM_STACKSIZE_ADJUSTMENT to reduce variability

Summary
Add CONFIG_SIM_STACKSIZE_ADJUSTMENT to reduce variability between sim and other different platform stack size setting
Signed-off-by: Jiuzhu Dong<EMAIL_ADDRESS>Impact
N/A
Testing
daily test

@Donny9 can you please give an example when or how this is used?

We all know that a task may be somewhat larger on the sim than the stack on the board because some debug like asan etc。
Before this patch, the default stack size is 65536 for ARCH_SIM, but these task may overflow on the board because their stack size are 2048 because of !ARCH_SIM. So for the different arch, there will be two different configurations.
This patch can automatically extend the size of the extension stack without changing config so keeping a configuration.

i'm not sure if this is a good idea because

this breaks existing configurations
it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_attr_setstacksize)
stacksize is inherently arch-dependent

i'm not sure if this is a good idea because

this breaks existing configurations

Could you explain more?

it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_attr_setstacksize)

SIM_STACKSIZE_ADJUSTMENT is used only for sim, the stack on sim already has huge difference to other arch. This patch is try to fix the program hard code the stack size in code or config with a fixed default value.

stacksize is inherently arch-dependent

DEFAULT_TASK_STACKSIZE could cover the most arch difference.

i'm not sure if this is a good idea because

this breaks existing configurations

Could you explain more?

if you have a config with stack sizes adjusted carefully, it won't work as expected anymore.

it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_attr_setstacksize)

SIM_STACKSIZE_ADJUSTMENT is used only for sim, the stack on sim already has huge difference to other arch. This patch is try to fix the program hard code the stack size in code or config with a fixed default value.

the difference from other arch is not a problem as it's arch-dependent in the first place.
my concern is inconsistency within sim.
with this change, an app using pthread_attr_setstack and an app using pthread_attr_setstacksize
need to have different ideas of stack size.

stacksize is inherently arch-dependent

DEFAULT_TASK_STACKSIZE could cover the most arch difference.

the stack usage is actually different.
it's more natural to use different values to reflect the reality than trying to maintain the illusion of "one value fit all".

i'm not sure if this is a good idea because

this breaks existing configurations

Could you explain more?

if you have a config with stack sizes adjusted carefully, it won't work as expected anymore.

it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_attr_setstacksize)

SIM_STACKSIZE_ADJUSTMENT is used only for sim, the stack on sim already has huge difference to other arch. This patch is try to fix the program hard code the stack size in code or config with a fixed default value.

the difference from other arch is not a problem as it's arch-dependent in the first place.
my concern is inconsistency within sim. with this change, an app using pthread_attr_setstack and an app using pthread_attr_setstacksize need to have different ideas of stack size.

If the user tune the stack size by pthread_attr_setstacksize for real device, it normally stop work on sim, so do you prefer the user define the different size between sim and other arch. But, is it good to let user take care about the stack size on sim?

stacksize is inherently arch-dependent

DEFAULT_TASK_STACKSIZE could cover the most arch difference.

the stack usage is actually different. it's more natural to use different values to reflect the reality than trying to maintain the illusion of "one value fit all".

I just want to illusion of the sim(that's why we name it CONFIG_SIM_STACKSIZE_ADJUSTMENT) not other real arch. sim is just a test platform for functional development, nobody expect to tune the stack size on it, so it's boring to enforce the user to define the different stack size for sim either by defconfig or source code.
We have found many people just turn on some application under apps/, and crash immediately on sim just because that application tune the stack size to a particular value.

i'm not sure if this is a good idea because

this breaks existing configurations

Could you explain more?

if you have a config with stack sizes adjusted carefully, it won't work as expected anymore.

it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_attr_setstacksize)

SIM_STACKSIZE_ADJUSTMENT is used only for sim, the stack on sim already has huge difference to other arch. This patch is try to fix the program hard code the stack size in code or config with a fixed default value.

the difference from other arch is not a problem as it's arch-dependent in the first place.
my concern is inconsistency within sim. with this change, an app using pthread_attr_setstack and an app using pthread_attr_setstacksize need to have different ideas of stack size.

If the user tune the stack size by pthread_attr_setstacksize for real device, it normally stop work on sim, so do you prefer the user define the different size between sim and other arch. But, is it good to let user take care about the stack size on sim?

yes. it's something only the app can deal with some #ifdef.

stacksize is inherently arch-dependent

DEFAULT_TASK_STACKSIZE could cover the most arch difference.

the stack usage is actually different. it's more natural to use different values to reflect the reality than trying to maintain the illusion of "one value fit all".

I just want to illusion of the sim(that's why we name it CONFIG_SIM_STACKSIZE_ADJUSTMENT) not other real arch. sim is just a test platform for functional development, nobody expect to tune the stack size on it, so it's boring to enforce the user to define the different stack size for sim either by defconfig or source code.
We have found many people just turn on some application under apps/, and crash immediately on sim just because that application tune the stack size to a particular value.

how those applications tune the stack size?
if they hardcode values for a specific arch, it's their problem, not sim.

https://github.com/apache/incubator-nuttx/pull/5365

i'm not sure if this is a good idea because

this breaks existing configurations

Could you explain more?

if you have a config with stack sizes adjusted carefully, it won't work as expected anymore.

it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_attr_setstacksize)

SIM_STACKSIZE_ADJUSTMENT is used only for sim, the stack on sim already has huge difference to other arch. This patch is try to fix the program hard code the stack size in code or config with a fixed default value.

the difference from other arch is not a problem as it's arch-dependent in the first place.
my concern is inconsistency within sim. with this change, an app using pthread_attr_setstack and an app using pthread_attr_setstacksize need to have different ideas of stack size.

If the user tune the stack size by pthread_attr_setstacksize for real device, it normally stop work on sim, so do you prefer the user define the different size between sim and other arch. But, is it good to let user take care about the stack size on sim?

yes. it's something only the app can deal with some #ifdef.

Why do you want to see a general application add ifdef CONFIG_ARCH_SIM? As I said before, sim is just for developing, nobody want to fight the stack size issue on the simulator. We have 50+ developers use simulator daily who doesn't have the deep knowledge how sim work internally. The top 1 issue(complaint) is that the application run correctly on the real device, but crash on sim. After investigating by NuttX expert, 90+% is just because they copy the stack size config to sim.

stacksize is inherently arch-dependent

DEFAULT_TASK_STACKSIZE could cover the most arch difference.

the stack usage is actually different. it's more natural to use different values to reflect the reality than trying to maintain the illusion of "one value fit all".

I just want to illusion of the sim(that's why we name it CONFIG_SIM_STACKSIZE_ADJUSTMENT) not other real arch. sim is just a test platform for functional development, nobody expect to tune the stack size on it, so it's boring to enforce the user to define the different stack size for sim either by defconfig or source code.
We have found many people just turn on some application under apps/, and crash immediately on sim just because that application tune the stack size to a particular value.

how those applications tune the stack size? if they hardcode values for a specific arch, it's their problem, not sim.

If you don't like this patch, please revert DEFAULT_STACK_SIZE too. From my review, DEFAULT_STACK_SIZE provided by you also try to fix the same stack size. And let's modify all the hardcode stack size in application or add CONFIG_xxx_STACK_SIZE=65536 in all defconfig under boards/sim.

i'm not sure if this is a good idea because

this breaks existing configurations

Could you explain more?

if you have a config with stack sizes adjusted carefully, it won't work as expected anymore.

it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_attr_setstacksize)

SIM_STACKSIZE_ADJUSTMENT is used only for sim, the stack on sim already has huge difference to other arch. This patch is try to fix the program hard code the stack size in code or config with a fixed default value.

the difference from other arch is not a problem as it's arch-dependent in the first place.
my concern is inconsistency within sim. with this change, an app using pthread_attr_setstack and an app using pthread_attr_setstacksize need to have different ideas of stack size.

If the user tune the stack size by pthread_attr_setstacksize for real device, it normally stop work on sim, so do you prefer the user define the different size between sim and other arch. But, is it good to let user take care about the stack size on sim?

yes. it's something only the app can deal with some #ifdef.

Why do you want to see a general application add ifdef CONFIG_ARCH_SIM?

i don't.
but sometimes it's necessary.

As I said before, sim is just for developing, nobody want to fight the stack size issue on the simulator. We have 50+ developers use simulator daily who doesn't have the deep knowledge how sim work internally. The top 1 issue(complaint) is that the application run correctly on the real device, but crash on sim. After investigating by NuttX expert, 90+% is just because they copy the stack size config to sim.

interesting. thank you for the info.
what's your suggestion to them when their app was using pthread_attr_setstack-like api?

stacksize is inherently arch-dependent

DEFAULT_TASK_STACKSIZE could cover the most arch difference.

the stack usage is actually different. it's more natural to use different values to reflect the reality than trying to maintain the illusion of "one value fit all".

I just want to illusion of the sim(that's why we name it CONFIG_SIM_STACKSIZE_ADJUSTMENT) not other real arch. sim is just a test platform for functional development, nobody expect to tune the stack size on it, so it's boring to enforce the user to define the different stack size for sim either by defconfig or source code.
We have found many people just turn on some application under apps/, and crash immediately on sim just because that application tune the stack size to a particular value.

how those applications tune the stack size? if they hardcode values for a specific arch, it's their problem, not sim.

If you don't like this patch, please revert DEFAULT_STACK_SIZE too. From my review, DEFAULT_STACK_SIZE provided by you also try to fix the same stack size. And let's modify all the hardcode stack size in application or add CONFIG_xxx_STACK_SIZE=65536 in all defconfig under boards/sim.

replied in the other pr.

Why do you want to see a general application add ifdef CONFIG_ARCH_SIM?

i don't. but sometimes it's necessary.

With this patch, developer don't need handle the stack size specially for sim in most case, except the code call pthread_attr_setstack with non-NULL stack pointer(Actually, no code in apps/nuttx call this api, which mean this approach work for all mainline code).

As I said before, sim is just for developing, nobody want to fight the stack size issue on the simulator. We have 50+ developers use simulator daily who doesn't have the deep knowledge how sim work internally. The top 1 issue(complaint) is that the application run correctly on the real device, but crash on sim. After investigating by NuttX expert, 90+% is just because they copy the stack size config to sim.

interesting. thank you for the info. what's your suggestion to them when their app was using pthread_attr_setstack-like api?

I never see the case which call pthread_attr_setstack with no NULL pointer. All developer call pthread_attr_setstack just want to adjust the stack size, which can handle correctly with this approach.

Why do you want to see a general application add ifdef CONFIG_ARCH_SIM?

i don't. but sometimes it's necessary.

With this patch, developer don't need handle the stack size specially for sim in most case, except the code call pthread_attr_setstack with non-NULL stack pointer(Actually, no code in apps/nuttx call this api, which mean this approach work for all mainline code).

As I said before, sim is just for developing, nobody want to fight the stack size issue on the simulator. We have 50+ developers use simulator daily who doesn't have the deep knowledge how sim work internally. The top 1 issue(complaint) is that the application run correctly on the real device, but crash on sim. After investigating by NuttX expert, 90+% is just because they copy the stack size config to sim.

interesting. thank you for the info. what's your suggestion to them when their app was using pthread_attr_setstack-like api?

I never see the case which call with no NULL pointer. All most developers call pthread_attr_setstack just want to adjust the stack size, which can handle correctly with this approach.

is it even a vaild usage of pthread_attr_setstack?
i guess you should use pthread_attr_setstacksize instead.

Yes, this approach isn't perfect, but it can fix 99.99% problem.

maybe. but the api consistency outweigh IMO.


libuv portability test: a few C core test cases failing on master consistently

E.g. here:
https://source.cloud.google.com/results/invocations/46a25296-9b0a-4e9a-824f-7cf25e360372/targets/github%2Fgrpc%2Fc_linux_dbg_uv/tests
Failing tests:
bins/dbg/memory_profile_test
bins/dbg/server_chttp2_test
bins/dbg/server_test

@jtattermusch
Probably me or @murgatroid99
Feel free to re-assign to me.

@kpayson64   bins/dbg/memory_profile_test  is fixed now,   the remaining ones are bins/dbg/server_test  and bins/dbg/server_chttp2_test (both seems to have the same rootcause).
https://source.cloud.google.com/results/invocations/1a21d5a5-6d24-4ea1-99df-af2aef83fb08/targets/github%2Fgrpc%2Fc_linux_dbg_uv/tests


fixed Connection type "fo"

Prometheus exporter's HELP typo fix

@gstiller Thank you. This fix has been successfully implemented in version 1.2.31.


Help: Basic example with NAT

I am trying to make the basic example (nginx-proxy / nging-proxy-companion / nginx combo) working but no cigar.
I've setup the 3 containers on Docker on a Debian 9 server.  The containers are set up exactly as described.
http://subdomain.yourdomain.tld is reachable but https://subdomain.yourdomain.tld is not. The former get the expected "Welcome to nginx!" message, but the latter gets "This site can’t be reached - subdomain.yourdomain.tld refused to connect.".
The containers are all running, and no other errors are given.  The 3 folders described in step 1 are all present and chmod 777 - but no content has ever been found in any of the folders, either before/during/after trying.
Netstat shows that docker-proxy is listening on both ports 80 and 443. Different PID's though, but docker ps only shows one nginx-proxy container running.
I would have expected nginx-proxy to accept traffic on any published port and proxy it to whatever is bound on the socket. What am I doing wrong? Help....
(redid tests without NAT, updated post to reflect this)

Do you have the exact commands or compose file you used and the containers logs ?

Actually somehow I got over this problem, even though I'm not sure how.   Now I have most of it working to the point I get the page up, but when I enter /media/usb/share/nextcloud/data as my data folder, it comes back with the error "Error Can't create or write into the data directory /media/usb/share/nextcloud/data"
Ownership and permissions should be correct (www-data.www-data and 755). I have a feeling docker might try to create it under /var/lib/docker in spite of me adding NEXTCLOUD_DATA_DIR=/media/usb/share/netcloud/data.  I want my data store out of docker, how do I do that?
`version: '2'
volumes:
nextcloud:
data:
db:
services:
nginx-proxy:
image: 'jwilder/nginx-proxy'
container_name: nginx-proxy
ports:
- '443:443'
volumes:
- '/etc/nginx/vhost.d'
- '/usr/share/nginx/html'
- '/etc/nginx/certs:/etc/nginx/certs:ro'
- '/var/run/docker.sock:/tmp/docker.sock:ro'
letsencrypt-nginx-proxy:
image: 'jrcs/letsencrypt-nginx-proxy-companion'
container_name: letsencrypt-nginx-proxy
volumes:
- '/etc/nginx/certs:/etc/nginx/certs'
- '/var/run/docker.sock:/var/run/docker.sock:ro'
volumes_from:
- nginx-proxy
mariadb:
image: mariadb
container_name: mariadb
command: --transaction-isolation=READ-COMMITTED --binlog-format=ROW
restart: always
volumes:
- db:/var/lib/mysql
environment:
- MYSQL_ROOT_PASSWORD=$rootpassword
- MYSQL_PASSWORD=$mysqlpassword
- MYSQL_DATABASE=nextcloud
- MYSQL_USER=nextcloud
nextcloud:
image: nextcloud
container_name: nextcloud
ports:
- 8080:80
links:
- mariadb
volumes:
- nextcloud:/media/usb/share/nextcloud
- /data:/media/usb/share/nextcloud/data
environment:
- VIRTUAL_HOST=$myhost
- LETSENCRYPT_HOST=$myhost
- LETSENCRYPT_EMAIL=$myemail
- NEXTCLOUD_ADMIN_USER=admin
- NEXTCLOUD_ADMIN_PASSWORD=$adminpass
- NEXTCLOUD_DATA_DIR=/media/usb/share/netcloud/data
restart: always
networks:
default:
external:
name: nginx-net
`

If you attempt to close port 80 of the proxy, domain authorizations will fail. Please check the requirements at the beginning of the doc.
Nextcloud specific questions should be asked on Nextcloud's repository.

I didnt' close port 80. I really have no idea why it didn't work, suddenly it did work. It seems there is a delay from the container runs until the certificate is ready, maybe that was the reason. Anyway, this part is resolved so I will take the nextcloud question to the nextcloud repo as suggested.

services:
  nginx-proxy:
    image: 'jwilder/nginx-proxy'
    container_name: nginx-proxy
    ports:
      - '443:443'
    volumes:
    [...]

There is no mapping for nginx-proxy port 80, meaning it's unreachable from outside the container's docker network.
It has to be reachable on your host's own port 80.

Ah, right, thanks!


Disable displayName stuff in production

We should check for process.env.NODE_ENV !== 'development' and disable displayName if that's the case.

The issue was created when we used IIFE and had no configuration flags.
These days, I'd recommend using https://babeljs.io/docs/usage/babelrc/#env-option to configure it explicitly.
@mxstbr  WDYT?

Sounds very reasonable!

In case it helps anyone else, the updated babel URL would be: https://babeljs.io/docs/en/config-files#apienv
Here's an example of disabling displayName for production:
// .babelrc.js

module.exports = function (api) {
  return {
    plugins: [
      ["styled-components", { ssr: true, displayName: !api.env("production") }]
    ]
  };
};


Fabric mods don't load nested mods more than one layer deep

I have a Fabric mod which JiJ's a mod, which in turn JiJ's other mods
On Fabric loader, this works fine and loads the nested mods
On Quilt loader, this causes a crash, because the nested mods aren't being loaded.
Log: https://mclo.gs/av13o8K
The log may appear weird, but that's an artifact of MoreMCMeta's xplat setup

Nevermind, this is an issue with moremcmeta


Remove char escape from string template

Purpose

To remove char escape from string template.

Fixes #23988
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@Chirantha Can you sync this with master?


Multiple process parameters interface

Updates the interface to allow multiple processing parameters. The interface is based in the parameter set name and shows the values to the user.
@antgonza this should be the last needed bit to support ITS.

Errors!! Could you fix?

Coverage increased (+0.07%) when pulling 3c97376a254e87f27e7da71243696f5af7352942 on josenavas:mult-process-parameters into 687b01f9d5c368088fbbf8a3c28a0168e25113db on biocore:master.

Works great but some comments on the code.

Coverage increased (+0.08%) when pulling 520414e3749b53287daeef100b333e9accf8e1e8 on josenavas:mult-process-parameters into 687b01f9d5c368088fbbf8a3c28a0168e25113db on biocore:master.


Benchmark for different types of serializations

Description
Adding Marshalling and UnMarshalling functions for a few different types of serliazations. included benchmarking code
Local benchmarking results for speed:

Local benchmarking results for size after encoding:

Closing since cbor was chosen as the preferred serialization.


Use the new Apple / google contact tracing API

see Contact tracing API

Cool, see the Stack-Overflow blog and podcast chapter about it:
https://stackoverflow.blog/2020/04/15/reading-the-spec-covid-19-contact-tracing-from-apple-and-google/

https://9to5mac.com/2020/04/29/ios-13-5-beta-covid-19-contact-tracing/

Thanks for that, this is being checked. Google/Apple  also imposes some limitations to use this API.
https://techcrunch.com/2020/05/04/apple-and-google-release-sample-code-and-detailed-policies-for-covid-19-exposure-notification-apps/

Just to clarify for others, the main problematic clause in the link above - to my understanding - is:

They can’t access or even seek permission to access a device’s Location Services, which provides specific geolocation data. Google and Apple note that apps already available from public health authorities that make use of location data will continue to be offered, but that no apps that make use of that info will also have access to the new Exposure Notification API

So there is no possibility to integrate into Hamagen unless it completely gives up it's current approach (the pro's and con's of which could be debated).  Note also that Google/Apple are only allowing one app per country.  Only other option I think is for a separate, 2nd app from the competent Hamagen dev team.

From what I understand, in outdoor places there  are less chances getting covid-19 than indoor places, So bluetooth solutions like the Google/Apple project are more efficient than location GPS solutions.

Let me clear a mix between a few things. We have bluetooth (BLE to be exact) technology and we have Google/Apple contact tracing API, which uses BLE, but also has many restrictions (e.g. no GPS).
As you can see at #213, BLE will be part of Hamagen 2.0, so we plan to enjoy both GPS and BLE. You can also look at https://github.com/MohGovIL/rn-contact-tracing for these efforts.

Thanks for the information

From what I read its not efficient to use other BLE solution since google+apply are the only ones that will keep bluetooth running in the background


#160 Paths endpoint

Closes #160
Context
Paths provide information about potential path payments. A path can be used to populate the necessary fields for a path payment operation.
https://developers.stellar.org/api/aggregations/paths/
Objective

To get access to all the aggregations exposed by Horizon.

Checklist

[x] List Strict Receive Payment Paths
[x] List Strict Send Payment Paths

@juanhurtado10 Done, thanks!!!


Basic module structure for Camel K projects

Going forward version 1.0.0 we should think about how people will organize complex integrations that they will run in production. We strongly want to keep a single file model for the integrations, but often you need to provide resources, like:

Properties
API definitions
Transformation and mapping files
Test files (yet to come)

That means that even in a simple project composed of few integrations that interact together, it will be difficult to understand what resources belong to what integration.
E.g. When you have multiple integrations, which one does an application.properties file refer to?
We need to define some module structure for integrations. It may be something encoded in the kamel CLI (e.g. kamel scaffold name) or simply a best practice we propose also in the examples.
The simplest model I can imagine is creating as less structure as possible, keeping a minimum of visual order in a project.
e.g.
module1
  module1integration.groovy
  application.properties
  trasnform.xslt

module2
  tests
    moduletest.feature
    moduletest2.feature
    moduletest3.feature
  module2integration1.groovy
  module2integration2.groovy
  application.properties
  README.md

rootmoduleintegration.groovy

Questions:

Is that the best structure we can have for integrations?
Should we provide some tools to scaffold it (e.g. create the basic quickstart)?
Should we provide some tools in the CLI to run a module?
Should we define a module configuration (as in the comments #1133)
Other ideas?

@lburgazzoli , @davsclaus, @astefanutti , @jamesnetherton, @valdar, @oscerd

@nicolaferraro I was about to ask this. So is it true that I currently cant create a route that uses several java classes defined in several files? Is this a major limitation, or can I simply place all those class definitions into a single ".java" file?
I would think that many meaningful routes may have a handful of classes, and modularity suggests that these should go in separate files.

One thing I absolutely could do is to put the building block classes into a "jar" file that is then loaded as a dependency. I know how to do this, and that may be the proper way to go, and to live with the single file restriction in kamel

You can run multiple files in a Camel K integration, it works also if they are written in different languages. Just run:
kamel run --name thename Routes1.java Routes2.java others.xml

It runs as a single integration named thename.
The "single file model" is the default model we're pushing for, but if it makes sense, I think there are no problems in collapsing multiple integration files into a single integration (often it makes sense for a performance point of view, to save some CPU/memory since it uses a single JVM).
I feel that generic supporting methods and classes are better placed in an external "jar", so that one separates the glue (routes) from the business logic.

@nicolaferraro Wow.. thats cool. Sounds like we have a couple of good options.

Outdated

Wait what happened, still curious to know how is this going to be handled? Outdated why?

Wait what happened, still curious to know how is this going to be handled? Outdated why?


Improve daemonset security

Make fs read-only. Mounted volume not affected.
Drop all linux caps
Remove needless privileged permission
Remove needless hostnetwork permission

Signed-off-by: Martin Kennelly<EMAIL_ADDRESS>

I confused that why we set
securityContext:
     allowPrivilegeEscalation: false
     privileged: true
     readOnlyRootFilesystem: true

when we need to copy binary file to host.


Touchscreen Gesture Customization

I feel with how many touchscreen devices out there we would benefit from customizable touch gestures. There are many different touch gestures that could be implemented.

swipe up from the bottom to go to desktop (similar to iPhone/Android)
swipe from the sides to switch desktops/applications
swipe to display multi-view of currently open apps
swipe and hold to quick split screen apps

I just want to be able to turn off swipe left to open action centre and swipe right to open task view.
Those are so annoying. Along with the system media transport controls that appear when you change the volume. They also happen to be inconsistent with the controls on the lockscreen. And god, the new task view (rewritten with W10 1803) drives me bonkers. It crashes pretty often too, bringing File Explorer down with it and as we all know, File Explorer handles a lot. Including the low battery notification! Yes, two years later and none of these things have been fixed. W10 2004 only made task view a little bit better but it's still nothing like it used to be.

Good idea

Swiping up should show in Tablet Mode the Start menu and not the Desktop. But this Idea to implement gestures into power tools  is 🔥

It would be great if a 'Custom Gestures' feature can be added, where you can pinch in and out with 3/4 fingers to open the multitasking view/show the desktop, pinch in from 2 sides to put the 2 recent windows side-by-side and more.

It would be great if a 'Custom Gestures' feature can be added, where you can pinch in and out with 3/4 fingers to open the multitasking view/show the desktop, pinch in from 2 sides to put the 2 recent windows side-by-side and more.

Note: If this isn't the right place for me to record this idea, then please direct me there.
I'd love to be able to use 3/4 fingers to perform a drag operation. This functionality is available in macOS. It used to be in the advanced touchpad settings, and then it was added to the accessibility options. When 3/4 fingers are placed on the touchpad, the computer behaves as if the left mouse button is being held down. Moving these 3/4 fingers around on the touchpad behaves the same is the computer would behave if the mouse was being moved when the left mouse was being held down. When the fingers are lifted off of the touchpad, then the computer behaves as if the left mouse button has been released.
This allows the user to hold down 3/4 fingers to move a window around, highlight text, or moving files around via drag-drop.
I see that there are ways to customize 3/4 finger swipes, but I don't see any way to customize the movement behavior when 3/4 fingers are held on the device.
Note: On macOS, this functionality does not interfere with any 3/4 finger swipe gestures. Those continue to work.
One way that I've found to get this behavior or Windows is to use an Apple Magic Trackpad with the Magic Utilities drivers installed.
I don't know enough about the low-level operations of the Widows touchpads to know if this is even possible to detect what I'm describing. I'm really curious about helping to implement a solution. I just need some help to get me pointed in the correct direction. My suspicion is that this would require a new device driver, but I'm open to other ideas, and I've never written a Windows device driver before, so that might be a bit of a learning curve.

yes please


[Feature] Expose loadBalancerIP for head service to users

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Users may want Ray head to be accessible from outside Kubernetes. One option is to use Kubernetes Service with type LoadBalancer and specify loadBalancerIP. However, KubeRay currently does not expose loadBalancerIP to users. See https://discuss.ray.io/t/specify-ip-adress-for-the-head-as-a-loadbalancer-service-type/9008 for more details.
Use case
No response
Related issues
https://discuss.ray.io/t/specify-ip-adress-for-the-head-as-a-loadbalancer-service-type/9008
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

We should expose as many fields as people ask for.
Probably we should have just exposed the entire service (maybe we did in the initial version)


URGENT! Tabbed chat is down right now.

URGENT! Tabbed chat is down right now. Completely, for everyone.

it's down, again.


Remove Option from any cursor position API

What problem does this solve or what need does it fill?
I'm trying to write a simple UI for some example. Nothing fancy. I keep hitting a wall against all the cursor position APIs returning an Option<Vec2>.

Why is None ever possible? This is documented nowhere. What does it ever mean to get an Interaction::Clicked with a None mouse cursor position?
It's extremely annoying to have to deal with None. This leaks into all code dealing with UI: button click, drag and drop, etc. and there's no good behavior to assign to it.

What solution would you like?
Remove the Option from all those APIs, just use Vec2 directly. If there's really a case where the mouse cursor position is not available, document why and possibly add an escape hatch for handling that, our try our best guess to find a position anyway.
What alternative(s) have you considered?
User has to wrap all APIs into their own helper function(s) to safely unwrap that Option or find a default based on the (primary) window and other factors, which systematically adds several queries to all systems and is far from trivial (see Additional context).
Additional context

Interaction has no cursor information on click
RelativeCursorPosition stores an Option<Vec2> but it's unused for cases where the cursor is outside the widget, which would be the only case where I find this would make sense.
It's unclear how you get the current window where the interaction occurred (vs. the primary one) to be able to retrieve its position and cursor position.

Why is None ever possible?
I think it's supposed to represent the cursor being outside of the window.

This is documented nowhere
I added a line in https://github.com/bevyengine/bevy/pull/8858 for Window.physical_cursor_position and Window.cursor_position. If it should appear somewhere else in the doc, or if it doesn't have the behaviour I think it has, please comment there!

What does it ever mean to get an Interaction::Clicked with a None mouse cursor position?
I think it's not supposed to happen. If it happened to you it might be linked to this bug: https://github.com/bevyengine/bevy/issues/8840

Thanks @Selene-Amanita for the docs. I approved your PR. I still think that None is a bad idea, the info it provides is already available from the Window, and the info it loses instead is valuable. Whether the cursor is inside the window should be a separate piece of information from the relative position of the cursor.

We don't get cursor position updates outside of the window from winit, so to implement this we'd have to manually track cursor position using `DeviceEvent::Mouse, which still will not give us a path where we would always know the position of the cursor.

It's unclear how you get the current window where the interaction occurred (vs. the primary one) to be able to retrieve its position and cursor position.

This is a separate issue that is being solved in https://github.com/bevyengine/bevy/pull/8852.

For example it might be a good idea to add a Vec2 position to Interaction::Clicked or similar.

Yes that was my initial reaction. But then I remembered Interaction is a component attached to many UI elements, and increasing the size of ALL of them just for the position of a single unique cursor feels like a waste. I don't have a good solution on top of my head.

About what I'm trying to do:

On right click, open a small menu UI that I wrote at the position of the mouse cursor. This requires knowing which Window the cursor is on, in case there's multiple (not really a blocker as I have a single one now, but the general issue remains).

I have a widget similar to a window or dialog, and when hovering over the "title bar" I want to be able to left click and drag the widget around. This requires knowing the exact position of the cursor on click relative to that widget, so that cursor moves can drag the widget around. Using delta cursor moves doesn't work from experience; it's prone to drift. So this requires knowing at all times the cursor position, subtracting the delta from when the left button was clicked, and moving the widget there.


Warn user if they're trying to bundle large unused folders into Cog image

It is not uncommon to have a giant (several GB) conda environment folder in the Cog working dir. We should detect that and warn the user, and also point to the .dockerignore docs.

Hi @andreasjansson and @afiaka87 I would love to take this issue as I am new to open-source and recently made few PRs and feeling confident to take new challenges.

Files matched by .dockerignore aren't included in images built by Cog. #1136 adds .dockerignore to the cog init template. https://github.com/replicate/cog/issues/209 tracks documenting this functionality. I think we can close this as a dupe of #209.
Tangentially, #1095 and #1151 adds a --separate-weights option to cog build, which puts these large files into a separate layer. So files that should be ignored but aren't shouldn't impact rebuild times as much.


nodePackages.typescript-language-server: add typescript dependency

Motivation for this change
typescript-language-server is a wrapper around tsserver and is useless without it.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @malob

This looks good to me! (Works on my macOS machine.)

Please rework your PR. It now has a merge conflict after PR https://github.com/NixOS/nixpkgs/pull/89184 has been merged

@prusnak I just force pushed a change to bring it up-to-date.

Just wondering would a local tsserver installation still override the default one with these changes?

PR got combined with a few others other node updates. Closing so that the update goes trough smoothly.

PR got combined with a few others other node updates. Closing so that the update goes trough smoothly.


Angastrom

Have you read the Contributing Guidelines on Pull Requests?
yes
Description
this program will check whether a number is ARMSTRONG NUMBER or not
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#2036

@HarshCasper  requested changes are done please check

@HarshCasper  requested changes are done please check


Add latest ruleset page

There is something that bothers me, now the "/" from the URL are encoded because of the separator in the path.
For the given ruleset path: boost/hidden/v2, the vuejs's router think that it's an endpoint instead of a path in the router declaration thus the /rulesets/:path/latest won't be reached.
Can we do something to avoid that?
This PR fixes #32.

Also, the rules are still editable. Even though it doesn't change data on the server, it's poor UX. We should only display the rules without being able to change them.

Are you sure? Because in my machine the rules aren't editable thanks to the editMode props.
Thank you for the plugin suggestion, I installed them, but the errors aren't fixed on saving.

@asdine 🆙
I fixed the eslint errors.
Can you make sure of this:

Also, the rules are still editable. Even though it doesn't change data on the server, it's poor UX. We should only display the rules without being able to change them.


Pagination does not work with loop is set to true.

This is a (multiple allowed):

[x] bug

Swiper Version: 4.0.0

Platform/Target and Browser Versions: macOs

Webpack: 3.6.0

What you did
When you turn on loop setting, pagination will stop working.
let swiper = new Swiper('.swiper-container', {
	pagination: {
		el: '.swiper-pagination',
		clickable: true,
	},
	loop: true,
	speed: 1500,
	autoplay: {
		enabled: true,
		delay: 5000,
		disableOnInteraction: false,
	}
});

I reproduced this issue on v4.0.1, too.

From what i see here it is working http://idangero.us/swiper/demos/200-infinite-loop.html

have the same issue

I got the same issue on 4.3.0.
The swiper-container is display:none; when new Swiper();.
When container displayed, ".swiper-pagination.swiper-pagination-bullets" is empty node.
And the console did not report any errors.
Then I switched to version 3.4.2,
just changed like
pagination: {     el: '.swiper-pagination',   },
to
pagination '.swiper-pagination',
then working fine.

Still broken, on the demo: http://idangero.us/swiper/demos/200-infinite-loop.html

yes, still can't slide loop


maybe: add S.encase

encase
verb
enclose or cover in a case or close-fitting surround

As I imagine it we're encasing a function which may explode in an indestructible wrapper. Alternative name suggestions welcome.

...and the puns you can make now... "Just encase"

Very punny, @buzzdecafe.


MIC-318 record-decision/additional-information screen

MIC-318

Codecov Report
Attention: Patch coverage is 79.16667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 77.41%. Comparing base (5e9ef6f) to head (eeb8ec9).

Files
Patch %
Lines

...cord-decision/additional-information/controller.ts
42.85%
4 Missing :warning:

.../record-decision/additional-information/schemas.ts
80.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
+ Coverage   77.39%   77.41%   +0.01%
==========================================
  Files         156      159       +3
  Lines        2141     2165      +24
  Branches      347      349       +2
==========================================
+ Hits         1657     1676      +19
- Misses        468      473       +5
  Partials       16       16

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


Remove avatarHTML from template helpers

The HTML code is more readable and more correct (it needs "ui avatar" class)

Ideally should use normal avatar helper, but I see the problem here is that it is indexed by email address instead of username.

Ideally should use normal avatar helper, but I see the problem here is that it is indexed by email address instead of username.

Actually, $push.AvatarLink is not for the "user". $push is a special model (*repository.PushCommits)

Can it be easily refactored to user regular avatar helper?

Can it be easily refactored to use regular avatar helper?

IMO, no. Or, how?

Don't wanna dig into this now :)


Format

Modify kpod diff --json to --format json
We want all kpod subcommands to use the formats code to output
formats like json.  Altering kpod diff --json to kpod diff --format json
like the kpod images command.

LGTM
@14rcole PTAL

LGTM

@baude lint failures

@rhatdan this includes all commands now ...

LGTM
@14rcole @umohnani8 PTAL

Couple nits, otherwise LGTM.

@rhatdan @nalind is this the same failure everyone else is seeing?

@baude Looks different to me - most of the kpod tests are passing, except for the rename test.

ok tests pass now

@mheon @nalind PTAL.

LGTM


RT-3.2: add missing instances of InterfaceEnabled deviation

This PR adds missing instances of InterfaceEnabled deviation:

ipv6 subif config in ATE version of the test
interface, ipv4 subif, ipv6 subif in the OTG version of the test

Reference - https://github.com/openconfig/featureprofiles/pull/1136
"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls

Changes look good, pending presubmit test.

/gcbrun

@xw-g  can we merge this one?


stop trusting cinder for devicename; use /dev/disk/by-id instead

wrong branch

Changes Unknown when pulling 11f272ee9db14c01b71d8ebebee027c38766de2d on justinclayton:openstack_volume_attach_fix into ** on emccode:master**.


Many "isRecyclable decremented below 0" error logs

04-19 18:39:25.047 967-967/us.mitene.development E/View: isRecyclable decremented below 0: unmatched pair of setIsRecyable() calls for ViewHolder{7325e91 position=213 id=199807, oldPos=-1, pLpos:-1}

Calls to setIsRecyable(false) and setIsRecyable(true) should always be paired according to document.

+1
I get a lot of these log messages too.
Wasn't always there, but seemed to start happening more sometime after I added some more efficient use of notifyItemX() methods; not sure if that has something to do with it or not.

See http://stackoverflow.com/a/29102232/3142018 for a solution

@csbenz what's the solution there? Seems unrelated

+1

+1

according to @csbenz the solution given is to put holder.setIsRecyclable(false); inside onClick method.
Thank you!

Does anyone have any insight as to how to fix this for animateChangeImpl? I have a viewholder using FastAdapter and toggle its selection. I have nothing extra myself, but disabling isRecyclable, doing the toggle, then reenabling it doesn't remove the error.

Nevermind. I'll post here in case someone else needs it, but animateChangeImpl is a replica of the one in DefaultItemAnimator
What was missing was
if (oldHolder == newHolder) {
    // Don't know how to run change animations when the same view holder is re-used.
    // run a move animation to handle position changes.
    return animateMove(oldHolder, fromX, fromY, toX, toY);
}

at the very beginning

I use the FlipInTopXAnimator with an OvershootInterpolator on a screen that user's can filter.  I am also using multiple viewTypes.  Without the animateMove fix from DefaultItemAnimator, the screen regularly crashes.


#75 broke Volar for me

Problem description
After updating to e492f12 (#75), Volar broke for me in every project. When opening a buffer on anything handled by Volar (in my case with takover mode Vue, ts, json, ...) I get this error:
Error executing vim.schedule lua callback: ...l/Cellar/neovim/0.8.0/share/nvim/runtime/lua/vim/lsp.lua:1285: RPC[Error] code_name = InternalError,
 message = "Request initialize failed with message: Cannot read properties of undefined (reading 'createLanguageService')"
stack traceback:
        [C]: in function 'assert'
        ...l/Cellar/neovim/0.8.0/share/nvim/runtime/lua/vim/lsp.lua:1285: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

Why do you think this is an issue with mason-lspconfig.nvim?
This doesn't happen with the previous commit 0eb7cfe.
Neovim version (>= 0.7)
NVIM v0.8.0
Build type: Release
LuaJIT 2.1.0-beta3
Übersetzt von brew@Monterey
Features: +acl +iconv +tui
See ":help feature-compile"
      System-vimrc-Datei: "$VIM/sysinit.vim"
 Voreinstellung für $VIM: "/usr/local/Cellar/neovim/0.8.0/share/nvim"

Run :checkhealth for more info
Operating system/version
Darwin khitomer 21.6.0 Darwin Kernel Version 21.6.0: Mon Aug 22 20:17:10 PDT 2022; root:xnu-8020.140.49~2/RELEASE_X86_64 x86_64
I've manually reviewed the Nvim LPS client log (:LspLog) to find potential errors

[X] Yes

I've recently downloaded the latest plugin version of mason.nvim, mason-lspconfig.nvim, and nvim-lspconfig

[X] Yes

Affected language servers
volar
Steps to reproduce

update to latest mason-lspconfig
install volar
open a buffer handled by volar

Actual behavior
Language server fails to start with above mentioned error message
Expected behavior
Language server should start successfully
LspInfo
Use [q] or [Esc] to quit the window

 Language client log: /Users/strayer/.local/state/nvim/lsp.log
 Detected filetype:   json

 1 client(s) attached to this buffer:

 Client: null-ls (id: 2, pid: 5001, bufnr: [1])
 	filetypes:       dockerfile, Dockerfile, sh, python, markdown.mdx, markdown, lua, handlebars, css, yaml, scss, less, graphql, jsonc, typescript, javascript, javascriptreact, typescriptreact, vue, json, html
 	autostart:       false
 	root directory:  /Users/strayer/dev/my-vue-app
 	cmd:             <function>

 Other clients that match the filetype: json

 Config: volar
 	filetypes:         typescript, javascript, javascriptreact, typescriptreact, vue, json
 	root directory:    /Users/strayer/dev/my-vue-app
 	cmd:               vue-language-server --stdio
 	cmd is executable: true
 	autostart:         true
 	custom handlers:

 Configured servers list: ansiblels, bashls, dockerls, eslint, emmet_ls, pyright, terraformls, yamlls, solargraph, volar, sumneko_lua

LspLog
No response
Healthcheck
mason: require("mason.health").check()
========================================================================
## mason.nvim report
  - OK: neovim version >= 0.7.0
  - OK: **Go**: `go version go1.19.2 darwin/amd64`
  - WARNING: **cargo**: unsupported version `cargo 1.50.0 (f04e7fab7 2021-02-04)`. Some cargo installations require Rust >= 1.60.0.
  - OK: **luarocks**: `/usr/local/bin/luarocks 3.9.1`
  - OK: **Ruby**: `ruby 2.6.8p205 (2021-07-07 revision 67951) [universal.x86_64-darwin21]`
  - OK: **RubyGem**: `<IP_ADDRESS>`
  - WARNING: **Composer**: not available
  - WARNING: **PHP**: not available
  - OK: **npm**: `8.5.5`
  - OK: **node**: `v16.15.0`
  - OK: **python3**: `Python 3.10.7`
  - OK: **pip3**: `pip 22.2.2 from /usr/local/lib/python3.10/site-packages/pip (python 3.10)`
  - OK: **javac**: `javac 19`
  - OK: **java**: `openjdk version "19" 2022-09-20`
  - WARNING: **julia**: not available
  - OK: **wget**: `GNU Wget 1.21.3 übersetzt unter darwin21.3.0.`
  - OK: **curl**: `curl 7.79.1 (x86_64-apple-darwin21.0) libcurl/7.79.1 (SecureTransport) LibreSSL/3.3.6 zlib/1.2.11 nghttp2/1.45.1`
  - OK: **gzip**: `Apple gzip 353.100.22`
  - OK: **tar**: `bsdtar 3.5.1 - libarchive 3.5.1 zlib/1.2.11 liblzma/5.0.5 bz2lib/1.0.8 `
  - OK: **bash**: `GNU bash, Version 5.2.2(1)-release (x86_64-apple-darwin21.6.0)`
  - OK: **sh**: `Ok`
  - OK: **python3_host_prog**: `Python 3.10.4`
  - OK: GitHub API rate limit. Used: 0. Remaining: 5000. Limit: 5000. Reset: Di 11 Okt 22:13:29 2022.

Screenshots or recordings
No response

I think I found the issue… previously new_config.init_options.typescript.tsdk was set to the folder of tsdk, but the new code points it to the same value as new_config.init_options.typescript.serverPath, which is the actual typescript js file.
Hacking a simple dirname equivalent in there fixes the language server for me:
local typescript = require "mason-lspconfig.typescript"

---@param install_dir string
return function(install_dir)
    return {
        on_new_config = function(new_config, workspace_dir)
            local tsdk = typescript.resolve_server_path(install_dir, workspace_dir)
            new_config.init_options.typescript.serverPath = tsdk
            new_config.init_options.typescript.tsdk = string.gsub(tsdk, "(.*/)(.*)", "%1")
        end,
    }
end

Ah, I must've missed some detail. Could you see if #77 (branch fix/typescript-tsdk-vs-tsserver) fixes the issue?

Works fine for me. Thanks for looking into this so quickly!


Java-Solution Number Of 1 Bits

Java-Solution Number Of 1 Bits
Easy 3 Line Java solution for "Number Of 1 Bits" using bit manipulation.
Fixes #1945
Replace #1945 with the issue number which is fixed in this PR
Type of change
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[ ] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@Amit366 @ravikr126 @Ayush7614
please review my this PR and merge it, I will thankful for you.


Add a way to manage workspace metadata via the admin API

Description
For example, modify the workspace API/secret pair.
Priority
Medium
Resolution
I'm willing to add this feature myself and raise a PR
More information
No response

This feature would be very important for some automations.

Hey @simonbrowndotje  if it's okay to you I would like to implement this feature.


[Improvement] Autolocalization of the "Search" field

It could be used getResources().getString(android.R.string.search_go) so we don't have to localize that string.
Actually I'm learning Android and I don't know how to localize that without modifying your library (which is a bad practice, as I would lose any changes if I update the library from your repo), so I think using the SO strings would be nice in this case.

that's very good suggestion :+1: Do you want to send a pull request or work on some existing PR #14? Thanks


Curl error

When I execute :  composer require laracasts/integrated --dev
It gives me the following error:
Problem 1
- instaclick/php-webdriver 1.4.3 requires ext-curl * -> the requested PHP extension curl is missing from your system.
- instaclick/php-webdriver 1.4.2 requires ext-curl * -> the requested PHP extension curl is missing from your system.
- instaclick/php-webdriver 1.4.1 requires ext-curl * -> the requested PHP extension curl is missing from your system.
- instaclick/php-webdriver 1.4.0 requires ext-curl * -> the requested PHP extension curl is missing from your system.
- laracasts/integrated 0.15.6 requires instaclick/php-webdriver ~1.4 -> satisfiable by instaclick/php-webdriver[1.4.0, 1.4.1, 1.4.2, 1.4.3].
- Installation request for laracasts/integrated ^0.15.6 -> satisfiable by laracasts/integrated[0.15.6].
But on same virtual machine when i execute : curl --version
It gives me the following information:
curl 7.22.0 (x86_64-pc-linux-gnu) libcurl/7.22.0 OpenSSL/1.0.1 zlib/<IP_ADDRESS> libidn/1.23 librtmp/2.3
Protocols: dict file ftp ftps gopher http https imap imaps ldap pop3 pop3s rtmp rtsp smtp smtps telnet tftp
Features: GSS-Negotiate IDN IPv6 Largefile NTLM NTLM_WB SSL libz TLS-SRP
Please Help!

You need to install curl for php 5
sudo apt-get install php5-curl

In my case it was to enable curl in php.ini


Compare traffic on-the-wire with focas2.log data.

focas2.log is a Linux only log from the Fanuc library. We have no control over it.
public static extern short cnc_startupprocess(long level, string filename);

I see. But could you not catch the output prior to writing it to file? I know, this is not that important, but it’s nice to have. wink

It looks like it would be helpful for reversing the protocol on the wire.

Indeed it would be useful for that. :wink:
Originally posted by @tukusejssirs in https://github.com/Ladder99/fanuc-driver/issues/4#issuecomment-864390147

@MRIIOT, I don’t mind marking this issue as wontfix, but what is the reason? Especially, what’s the reason for the security label?

Time and priority.  Unless we wanted to reverse the Focas protocol.


SW-4571 Update site selection buttons with active/hover/selected styles

with new info from Chudo
also refactored the component into 2, a selector and a button

Hey @karthikbtf, can you have the "selected" state override the the hover and active state? It feels a bit odd that once you click on a selection, it doesn't look like it until you hover out of the UI element. TIA!

Hey @karthikbtf, can you have the "selected" state override the the hover and active state? It feels a bit odd that once you click on a selection, it doesn't look like it until you hover out of the UI element. TIA!

Hey Chudo yes I noticed that, will fix it.


Fix goPath trimming in NewPackage func

Hi!
I have a list of paths as GOPATH, and I was getting absolute paths for all the goPackagePath values in my generated deps.nix.
So I refactored the gopath trimming used by currentPackage() into a separate function, and used that in NewPackage() instead of the strings.TrimPrefixthat was there.

Hi, thx for PR.
Could you provide an example of GOPATH that you want to support?

Sure. In my case it was a 2 path list : /home/hectorj/myProjectRepo/go:/home/hectorj/go.
And my issue was that I ended up with values like /home/hectorj/myProjectRepo/go/src/github.com/someUser/someRepo as deps' goPackagePath instead of just github.com/someUser/someRepo

Thx, I'll look at it shortly.


Cleanup

SHould we also set peer dependency resolution to strict in the root .npmrc? See viamrobotics/app#2740

Makes sense to me, will add it.

SHould we also set peer dependency resolution to strict in the root .npmrc? See viamrobotics/app#2740

Makes sense to me, will add it.

So I actually took it out because one issue. I want to use our svelte eslint/prettier configs but they rely on eslint-plugin-tailwindcss, which itself has a peer dep on tailwindcss. We can just install them, but we don't and probably won't use them in this project. I tried to extend the configs to just not that plugin, but it was not working. Might be worth looking into for when we want svelte libraries that don't include presentational components.


new TelemetryClient(); now deprecated, initialize with configuration

Using var telemetryClient = new TelemetryClient(); is now deprecated, one can replace new TelemetryClient() with new TelemetryClient(TelemetryConfiguration.CreateDefault()); instead to avoid this issue.

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

ID: b189efce-ce4e-f8f8-ecf8-8803a18f9b65
Version Independent ID: aec54012-dae3-ce18-0f8c-30749c46a79c
Content: Azure Application Insights for Console Applications
Content Source: articles/azure-monitor/app/console.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@ERGeorgiev, Thank you for sharing the update. We are working on this internally to incorporate the change to this doc.

@ERGeorgiev The changes should go live soon. Thanks again for reporting this.


CSS affects the slider itself

Users can create custom CSS on the slider and depending on if it's a flexbox or something, it'll break. We want to make sure that CSS can't leak and affect the behavior of the slider.

Decided that we did have a restriction… that is the wrapper element must have certain styles no matter what. They are:
#id {
  white-space: nowrap;
  overflow: hidden;

  position: relative;
}

There was also one bug we missed where the HTML nested children would be "flattened" to become all siblings.


Add some documents

This PR adds some documents with reference to RuboCop RSpec.
https://github.com/rubocop-hq/rubocop-rspec

First of all, I prepared a base documents. I will open some documents (Contribution Guidelines, Change History, and GitHub template) later as another PR.


Accept application/json for image manifest v2, schema 1

According to image manifest v2, schema 1, "application/json" should be accepted as the content-type, but I get the error:

encountered unknown type application/json

If I provide "application/vnd.docker.distribution.manifest.v1+prettyjws" then there is no such error.
Also "application/vnd.docker.distribution.manifest.v1+json" gives an error (it is unknown as well).

Scheme 1 is deprecated, please use Scheme 2 or OCI Image Spec.


Update CONTRIBUTING.rst

Switch pytest coverage config to pyproject.toml
Due to the removal of setup.cfg in the qcodes repository, the coverage configuration has been moved to pyproject.toml.

Thanks


nix-weather: init at 0.0.2

Description of changes

A fast rust tool to check availability of your entire system in caches. It so to speak "checks the weather" before going to update. Heavily inspired by guix weather.

https://git.fem.gg/cafkafk/nix-weather/
This PR relies upon #339854 to build. I could consider patching the version in Cargo.toml, as Rust 1.80.1 can otherwise build this just fine.
I have decided to use the GitHub mirror because @cafkafk expressed bandwidth and speed concerns with the upstream. I have added her as a maintainer per her request.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

maybe superseeded by https://github.com/NixOS/nixpkgs/pull/340518


adding in useragent info for Sxm and Pandora

Hi,
The Pandora RSS-useragent, which this file is intended for, is PandoraRSSCrawler/1.0 and is already included.

Hi,
The Pandora RSS-useragent, which this file is intended for, is PandoraRSSCrawler/1.0 and is already included.

wow, my apologies for making that mistake. Thank you for taking the time to review it and letting me know.

wow, my apologies for making that mistake. Thank you for taking the time to review it and letting me know.


Change selected tab label color swift

I'm trying to change the selected tab label color for the ButtonBarPagerStrip but the color simply doesn't change. I'm trying to set this on the viewDidLoad method of my class.
self.changeCurrentIndexBlock = { (oldCell : XLButtonBarViewCell!, newCell : XLButtonBarViewCell!, animated : Bool) -> Void in

            oldCell.label.textColor = UIColor(red: 125/255.0, green: 126/255.0, blue: 128/255.0, alpha: 1.0)
            newCell.label.textColor = UIColor.appRedColor()

        }

Any help on this?

@tiagopigatto Is changeCurrentIndexBlock being invoked?

library version?

how can I change the color of button bar view from child??


Add a label property to snippets to override the generated tab label

This allows for custom labels like below
<CodeSnippetWidget
  har={exampleHAR}
  snippets={[
    {
      prismLanguage: "javascript",
      target: "javascript",
      client: "jquery",
      label: "JavaScript (jQuery)"
    },
    {
      prismLanguage: "bash",
      target: "shell",
      client: "curl",
      label: "cURL"
    },
  ]}
/>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


HHH-16737 Widen types for numeric literals without explicit type suffix if necessary

https://hibernate.atlassian.net/browse/HHH-16737

@beikov does this change the semantics of HQL?
It looks like it changes the type of a floating point literal like 1.23.

It seems the Hibernate 6 parser/AST changed the behavior. In 5 we always produced exact literals, so even though this changes the behavior slightly, I think it is a good change. We'll discuss this on Wednesday in our team meeting though.

Let's discuss it here, since I can't be at the team meeting. WDYM by "exact literal"?

OK, so indeed there was a change of behavior here.

In Hibernate 5, select 1.3 from Book gives me a Double, which is consistent with the interpretation of floating-point literals in Java, but
in H6, it gives me a Float, which is not.

So it seems like reverting to the behavior of Hibernate 5 would be reasonable.
Now, I'm not sure what this patch does, and I'm probably misunderstanding, but it almost looks like the syntax select <literal> would be assigned a different type depending on how many digits of precision the <literal> value has. Is that correct?

Another thing: this whole issue would never have arisen if the JPA spec wasn't so atrociously ambiguous on the semantics of literal values. So I will make some time to write up a proposal to clarify all this in the spec.

As far as I understand the SQL specification, there are two types of literals, exact and approximate numeric literals. Denoting approximate literals requires the use of the e/E character to signify the exponent part. See https://www.contrib.andrew.cmu.edu/~shadow/sql/sql1992.txt section 5.3 <literal> or page 90.
Here comes a quote from the JPA spec section 4.6.1

Exact numeric literals support the use of Java integer literal syntax as well as SQL exact numeric literal syntax.
Approximate literals support the use Java floating point literal syntax as well as SQL approximate numeric literal syntax.
Appropriate suffixes can be used to indicate the specific type of a numeric literal in accordance with the Java Language Specification. Support for the use of hexadecimal and octal numeric literals is not required by this specification.

AFAIU this, JPA follow the SQL spec, but additionally allows to denote the type via the Java syntax type suffixes.
Playing a bit around with PostgreSQL which allows to introspect the runtime type of an expression, it seems that what I have implemented here is in accordance to what the SQL spec says and how most DBs behave. I'd even go one step further and only parse decimal literals as float/double if explicitly requested through the type suffix and not bother about checking if a value exactly representable.
Consider the following SQL query for playing aroung in PostgreSQL:
select pg_typeof(1), pg_typeof(1000000000000), pg_typeof(10000000000000000000), pg_typeof(1.0), pg_typeof(1e)

The result is integer, bigint, numeric, numeric, numeric, which is explained in the PostgreSQL docs. Oracle behaves the same way https://docs.oracle.com/cd/B14117_01/server.101/b10759/sql_elements003.htm#sthref385
My other PR which implements type inference for HQL parsing would take care of treating a numeric literal as float/double when comparing them to a float/double property.

Superseded by https://github.com/hibernate/hibernate-orm/pull/6885


Add function that counts chars in file

As a corollary to this, I wonder if it is possible to emulate something like du(1), to some extent? I don't know enough about how file sizes work or whether it's meaningful to count the number of characters in something that's not plain text.


BUG? Several start events in a flow trigger only one WorkflowInstance object.

When a workflow has several start events, they will all be triggered when the flow is started, according to bpmn standard.
So when a flow has two startevents/activities, in my opinion two workflowInstances should be created when the workflow is triggered (For example via the WorkflowEngineImpl.start(TriggerInstance) method.)
However, this is not the case, and I think it 's a bug. What now actually happens is that one workflowInstance is created, and both start events are executed for this instance (ie logged in the activityInstances array). The workflowInstance object is finally advanced to the activity where the last startevent was pointing to.
See the example image below

In this example after starting the workflow the result is that the flow contains one workflowInstance that resides in the "User 1" activity, while I would expect two workflowInstances: One in "Manual 1" and one in "User 1".

Learned a lot :-)
Turns out that my "Manual 1" activity type was not recognised during deploy. So for the engine the operation did not exist (I think a warning/error should be issued in that case, I will open another issue for that).
In the end both Start events are fired, but with one workflowInstanceId, while I would expect that two workflowInstances were created: one for each Start Event. On this one workflowInstanceId all activityInstances are logged, making it not very clear what flow path was followed, because these paths were followed in parallel.
But: In the end everything works, although a little bit different than I expected.

When there are multiple start events, starting all of them at the same time seems not desirable.  Users typically want to start only 1 start event.  The current logic (of starting all activities without incoming transitions) is based on the case where there are no start events and only have tasks / activities in your diagram.
I think that we should extend the TriggerInstance so that 1 event can be fired in 1 workflow instance.
The TriggerInstance should get a field called 'startActivityIds'.  In that field, the client can specify which activities should be started when the workflow instance starts.  A fluent convenience method startActivityId(String) should be added to the TriggerInstance that adds a startActivityId to the list.

I have already implemented this, and then I ran into the above issues. So if we agree that this is the way to go, I will write a test for it this afternoon and create a pull request.


Preserve WeID pubkey and auth ID and values

Summary

Concisely describe the issue and your solution:
Link to existing issue (type "#" to check):
Specify the tests done so far:

Solutions (Optional)
Please describe the details of your solution if it might be too complicated for reviewers.

Codecov Report

Merging #231 into develop will decrease coverage by 1.36%.
The diff coverage is 78.84%.

@@              Coverage Diff              @@
##             develop     #231      +/-   ##
=============================================
- Coverage      41.51%   40.15%   -1.37%
+ Complexity      1612     1536      -76
=============================================
  Files            232      232
  Lines          13928    12971     -957
  Branches        1955     1623     -332
=============================================
- Hits            5782     5208     -574
+ Misses          7448     7126     -322
+ Partials         698      637      -61

Impacted Files
Coverage Δ
Complexity Δ

...k/weid/service/impl/CredentialPojoServiceImpl.java
44.34% <33.33%> (-3.13%)
133.00 <0.00> (-35.00)

.../webank/weid/service/impl/EvidenceServiceImpl.java
39.70% <60.52%> (-0.75%)
31.00 <3.00> (-13.00)

...rvice/impl/engine/fiscov2/WeIdServiceEngineV2.java
76.25% <75.00%> (+2.41%)
47.00 <19.00> (+3.00)

...com/webank/weid/protocol/base/AuthorityIssuer.java
84.61% <85.71%> (-2.06%)
9.00 <5.00> (-4.00)

.../weid/service/impl/AuthorityIssuerServiceImpl.java
73.43% <95.23%> (-4.72%)
65.00 <1.00> (-7.00)

.../main/java/com/webank/weid/constant/ErrorCode.java
96.27% <100.00%> (+0.79%)
6.00 <0.00> (-3.00)
:arrow_up:

...in/java/com/webank/weid/constant/WeIdConstant.java
96.42% <100.00%> (-1.14%)
1.00 <0.00> (ø)

...ank/weid/protocol/base/AuthenticationProperty.java
75.00% <100.00%> (+8.33%)
4.00 <1.00> (+2.00)

...m/webank/weid/protocol/base/PublicKeyProperty.java
83.33% <100.00%> (+3.33%)
6.00 <1.00> (+2.00)

.../com/webank/weid/service/impl/WeIdServiceImpl.java
62.53% <100.00%> (-1.78%)
62.00 <0.00> (-8.00)

... and 17 more

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 205c75d...695c8e9. Read the comment docs.


[SDK BUG] Pinned Widgets with 'Enabled Clickthrough' aren't actually click through on PC Startup

Describe the bug
Pinned Widgets with 'Enabled Clickthrough' aren't actually click through on PC Startup until the Game Bar is opened and closed (Win + G pressed twice consecutively)
To Reproduce
Steps to reproduce the behavior:

Pin any widget.
Press 'Enable Click-through' in the Game bar menu.
Restart your PC and wait for the pinned widget to boot on startup, DONT press Win + G prior.
Try clicking inside the boundaries of the widget.

Expected behavior
Pinned widget with 'Enabled Clickthrough' should always be click through.
Screenshots
I took a video, I hope this helps :)
https://www.youtube.com/watch?v=y2a292AyMxM
Desktop (please complete the following information):

OS version: Windows 10 Pro, Version 10.0.19042 Build 19042
Game Bar version: 5.521.3093.0
SDK version: 5.3.200605002

I noticed with the most recent version of Game Bar v5.621.4142.0, the widgets no longer start on startup?

Hi jurkog,
Yes we are aware of this and it's a difficult problem to solve.  Due to how underlying windows handles the multiple HWNDs, we needed to come to the foreground (WinGForeground) before we could twiddle the settings to get click-through back on.  We've learned some more since we last looked at this, so I'll revisit and see if we can do better.
As far as game bar loading after reboot:  This doesn't happen automatically, something has to trigger game bar to start.  Some things that do this are: 1) If the user has startup tips enabled on game bar, it will launch when a known game launches. 2) Known Game List updates are checked for (infrequent periodic check), 3) Normal game bar hotkey presses, 4) Xbox app (Game Pass) is launched, 5) URI launch.  You likely just didn't hit one of these cases when you were testing v5.621.4142.0?  You can verify manually by running ms-gamebar://launch after reboot.  If you see your pinned widgets appear, then everything is working as expected.
Due to the above, we've discussed internally what to do with pinned widgets on game bar startup.  We're waiting for the experience team to review the scenario, but by default we enter PinnedOnly mode on startup, thus why the pinned Widgets show.  Since we aren't auto launch, it does create a unpredictable outcome on reboot.  One possibility is we don't show pinned widgets until the user brings Game Bar to the foreground for the first time in a given session, but we're still discussing.  If we went with this solution, the click-through problem described above would be a non-issue.
Hopefully that was transparent into what's going on.
Thank you for the feedback!
Ben

Hi Ben,
Is there any update on this?
This is probably our most pressing issue at the moment that we have no control over.
I don't know what happened but this is what I've noticed:

Game Bar widgets are still launching at boot without any activation.
When they are launched, users cant click through the pinned widgets. We've been advising them to press Win + G twice to bypass this but it's not good that this is a problem in the first place. A lot of people either refund the app, leave a bad review, etc
ms-gamebar://launch doesn't work for me anymore. It seemed like it did at one point and now it doesn't. I read the documentation about the URI launch and none of it works for me.

Best,
Jurko

Hey Ben,
The possibility you listed would be a great solution to what our customers have been complaining about. I have attached some support emails we have gotten to give you a better representation of the customer's mindset when they encounter the bug.

Since there is just a lot of confusion around where the bug is originating from, it would be a good call to not show the pinned widgets until the game bar is activated. We always instruct the customers on how to resolve the issue but this bug doesn't look good on us or the game bar platform.
Hope this info helps,
Best,
Sandra

Hi Jurkog and Sandra,
Thank you for the feedback here again.  That clarification helps.  Given that game bar shouldn't shutdown with widgets pinned, this should only occur if game bar crashes or the system reboots. However, it sounds like you have real scenarios where this is a problem for our joint customers. So I will run this by our Program Manager team to see if we can suppress showing pinned widgets until after the user explicitly brings up game bar at least once.
Sorry for the trouble here.
Thanks,
Ben

Update.
I just realized I had already fixed this issue in our 2108 branch.  Pinned widgets won't appear until the first time the user brings game bar to the foreground.  We just reviewed this behavior as a team and agreed to ship it for 2108.
You should already see this behavior in 5.721.7151.0 version of game bar (July 15th build) which flighted to External Fast and SDK flight rings (if you're in either of those).
Regarding ms-gamebar://launch.  This URI command only launches game bar if it's not already running.  If it wasn't already running, it starts and enters PinnedOnly mode. It does not toggle Foreground/PinnedOnly states.  So ms-gamebar://launch effectively does nothing if game bar is already running.
Here's the summary difference between 2106 (and prior) and 2108+ when doing ms-gamebar://launch (or any other background activation) when game bar wasn't already running:
2106 (and prior) - Game bar enters PinnedOnly mode and shows pinned widgets. Click through is not enabled, even if the user has it turned on.  Click through won't be enabled until after the first time game bar comes to foreground (Win+G or other method).
2108+ - Game bar enters PinnedOnly mode and does not show pinned widgets.  Click through isn't relevant as no widgets should be shown.  Pinned widgets are not shown until after the first time game bar comes to foreground (Win+G or other method).
Let us know if you have other questions!
Thanks,
Ben

Closing this as it should be resolved per the above.  If you see any further issues, please let us know!  Thanks!


Offset third person aim view

Improved Third Person View
Our camera when aiming is currently centered (see image below)

It would be ideal if it was offset to get a better view of where the player is aiming at (example: image below)

Scope of the task:

Camera view must be centered (same as first image) when not aiming.
When aiming (hold right mouse button) offset the camera.
In and out smoothing/interpolation. (the transition between centered and offset should be smooth both ways)

Possible issues

Changing the camera position could affect crosshair accuracy when shooting (But this should be a separate issue)

https://user-images.githubusercontent.com/36644824/164085253-de331ff2-c49d-4107-85fe-27e61a5ba415.mp4
added small demo video of functionality

Is anyone already working on this? I want to try to do it.

It's a test task for GitWillG


Implement rustdoc-scrape-examples

This implements rustdoc-scrape-examples for the docs that we host ourselves (not the docs.rs ones - I figure it's best to see how this goes first). Example.
This also:

Implements a full feature that incorporates all additional features
Resolves #2053
Fixes a link I messed up on the __call__ example in the guide
Fixes some links to anyhow and eyre's documentation
Adds cargo aliases for rendering docs
Renders and hosts internal documentation: Example

Note that this creates a dead link to "decorator" in the sidebar. This should be fixed by https://github.com/rust-lang/rust/pull/92146
I also haven't touched the makefile yet - should I?

Looks good to me! Feel free to merge this as-is. I'll let you decide whether to change the docs build CI step here or in a follow up PR.

I'm not sure I even want to change it now, seeing as it seems to work fine as it is (I'm guessing the rustdocflags don't get passed for some reason?).
Moving it to guide.yml and rendering docs for each possible combination of cfgs would require rebuilding pyo3 for each of those (?).

Mmm ok, let's keep as-is for now!


Dart VM on Android: Directory::CreateTemp("") should use Android-compatible temp directory.

This issue was originally filed by<EMAIL_ADDRESS>
Right now it uses /data/local/tmp, which is good enough for bring-up.
But when running in an Android application this code should probably return the same directory that android.content.Context.getCacheDir() does.

This CL adds support for setting the system temp path by the embedder.
https://codereview.chromium.org/2156243002


Inconsistencies when using "import from URL"

When on new event page (https://amy.software-carpentry.org/workshops/events/add/), clicking "Import from URL" tells you "Use the URL of the event's website, e.g. http://swcarpentry.github.io/workshop-template/ or http://www.datacarpentry.org/workshop/, and not the URL of its GitHub repository."
However when accepting a workshop request (https://amy.software-carpentry.org/workshops/request/XXX/accept/), it says "Either URL to the event's website or to it's repository index.html will work."
It seems like a link to the repo, and not necessarily the index.html file will work for the "accept workshop" page to populate the form.
However, the "URL" field of the form gets populated with the link to the repo, and then the form fails validation.  So if a user enters the github repo rather than the website URL, it should not populate the URL fied.

So there are actually two issues:

Inconsistent and misleading help texts below "url" field in "import from URL" modal. Here is PR: #1170.

Validation error in event form because of autofilling-in URL field. In that case, we can convert a link to GitHub repo to a link to website. This can be done manually (in that case we probably need to add a hint to error message) or automatically by AMY. @maneesha any opinions?

@maneesha any opinions on the second issue?

I'm reopening this, because #1170 solves only the first issue.


syntax highlighting in action logger

Overall, this looks cool.
Once we are okay with suggestions I made, we could take this in.

@arunoda added tests and comment

Okay. Great.

Published as v1.14.0


Make local maven settings applied to surefire plugin so that Quarkus …

…could build the application model accordingly

I was not able to run tests locally because of a build issue related to Quarkus not being able to download some dependencies.
After investigation, it turns out to be an issue where maven settings were not applied, similar to https://github.com/quarkusio/quarkus/issues/11433.
I'm able to have the build working locally with this PR. If it's breaking builds in other environments, please let me know.

Let's merge. Don't hesitate to revert in case it causes issues on other build platform/kind.


[0.25.0 update] createEagerFactory production only error

after update to 0.25.0 start getting
Error walking your react tree
TypeError: Class constructor OnMount cannot be invoked without 'new'
at createEagerElementUtil (/app/node_modules/recompose/utils/createEagerElementUtil.js:18:12)
at /app/node_modules/recompose/createEagerFactory.js:18:49
at ShouldUpdate.render (/app/node_modules/recompose/shouldUpdate.js:45:16)
at /app/node_modules/react-tree-walker/commonjs/index.js:148:35
at doTraverse (/app/node_modules/react-tree-walker/commonjs/index.js:78:21)

was all fine on 0.24.0
OnMount component is pretty plain
export default (timeout = 0) => BaseComponent =>
  class OnMount extends PureComponent {
    state = { isMounted: false };

    componentDidMount() {
      this.mounted = true;
      setTimeout(() => {
        if (this.mounted) {
          this.setState({ isMounted: true });
        }
      }, timeout);
    }

    componentWillUnmount() {
      this.mounted = false;
    }

    mouted = false;

    render = () =>
      <BaseComponent isMounted={this.state.isMounted} {...this.props} />;
  };

Can you provide a full example on jsbin?
As I'm not see createEagerElementUtil in your code above.
Is it reproduced in DEV only mode or also in production?
cc @deepsweet

@istarkov
only on production. Dev runs fine.  https://jsfiddle.net/p3vsmrvo/3/

That change haven't change anything in production mode, seems like this https://github.com/acdlite/recompose/pull/463/files

a bit more debugging.
I'm getting
type is  class OnMount extends _react.PureComponent {
// ..

from
var createEagerElementUtil = function createEagerElementUtil(hasKey, isReferentiallyTransparent, type, props, children) {
  if (!hasKey && isReferentiallyTransparent) {
    if (children) {
      return type(_extends({}, props, { children: children }));
    }
    console.log('type is ', type, props);
    return type(props);
  }

and if I try something like  stuff this
const hoc = compose(
  pure,
  setPropTypes(propTypes),
  createEagerElement(onMount(), {}),

getting same error
TypeError: Class constructor OnMount cannot be invoked without 'new'
 at ../recompose/compose.js:22:18

I just run that example in production mode and in development and see no error
Here is full example
import { compose } from 'recompose'
import React from 'react'
import ReactDOM from 'react-dom'

const onMount = (timeout = 0) => BaseComponent =>
  class OnMount extends React.PureComponent {
    state = { isMounted: false }

    componentDidMount() {
      this.mounted = true
      setTimeout(() => {
        if (this.mounted) {
          this.setState({ isMounted: true })
        }
      }, timeout)
    }

    componentWillUnmount() {
      this.mounted = false
    }

    mouted = false

    render = () =>
      <BaseComponent isMounted={this.state.isMounted} {...this.props} />
  }

const Counter = ({ count, increment, decrement }) =>

    Count: {count}

      <button onClick={increment}>+</button>
      <button onClick={decrement}>-</button>

const App = compose(onMount())(Counter)

ReactDOM.render(<App />, document.getElementById('root'))

I used create-react-app to build and run

Your React version?

Also in example you provided, again no eager factory is used

But I'm able to reproduce it adding pure HOC in compose

Minimal example to reproduce
import { compose, pure } from 'recompose'
import React from 'react'
import ReactDOM from 'react-dom'

const onMount = (timeout = 0) => BaseComponent =>
  class OnMount extends React.Component {
    render = () => <BaseComponent {...this.props} />
  }

const Counter = ({ count, increment, decrement }) =>
HELLO

const App = compose(pure, onMount())(Counter)

class Hello extends React.PureComponent {
  render() {
    return

  }
}

ReactDOM.render(<App />, document.getElementById('root'))

To fix don't bind render method
const onMount = (timeout = 0) => BaseComponent =>
  class OnMount extends React.Component {
    render() {
      return <BaseComponent {...this.props} />
    }
  }

I'll close this, feel free to reopen
If you are interesting about what the problem with bind and why it occurred now:
It is combination of 2 PRs #473 and #463
To check that component is a React Class Component we started to check the existence of render method in Component prototype in #463 and then in #473 we disabled that check used for optimisation purposes in DEV mode. So at production we get stuck.
I'm not sure do we need to provide an additional check that if type is not a react class it must be function. Imo yes it will be better to provide such check in devmode.
cc @wuct @deepsweet  You thoughts?

I think that the concept mentioned in #473 is correct and we shouldn't rely on any "post-production" optimizations in any way. Something should be done on isClassComponent side, like "an additional check that if type is not a react class it must be function" you mentioned.

@istarkov thanks for explaining this.
My main frustration is that it runs ok in development while crushes on production. This is not desired behavior and can lead to 'wtf' issues  on production. I'd personally prefer not to have this optimization then have unexpected behavior.
@deepsweet idea might be good option to look into

@istarkov I can't reproduce the problem using your minimal code. Here is the jsbin. Do I miss something?

@wuct  production only bug

Could we reopen this bug? We got burned by this as well. We use render = () => in lots of places it's breaking with mapProps

Just stop bind render method. There is no sense in this at all.

Just stop bind render method. There is no sense in this at all.
@istarkov it does really make much sense to prevent doing that. For purpose of recompose not to crush on production suddenly.
Even if we fix that there is not guaranty that someone won't do it again for whatever reason just to break only production build.
Imo it is better without this change then having only production bugs.

IMO its better without this optimizations too. Let us think a little what to do. May be we can provide additional checks at dev mode or something else cc @deepsweet

Any help will be very appreciated.

I'd like to somehow extract this optimization from Recompose at all.
On Tue, Sep 26, 2017 at 16:59, Ivan Starkov<EMAIL_ADDRESS>wrote:

Reopened #489.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/acdlite/recompose","title":"acdlite/recompose","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/acdlite/recompose"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Reopened #489."}],"action":{"name":"View Issue","url":"https://github.com/acdlite/recompose/issues/489#event-1265932861"}}}

I agree it's better to leave this optimization to React.

so... let's remove it then? :)

Ok

Waiting for PR ;-)

#538 optimisations removed please test 0.26

please test 0.26
Issue seem to be resolved. Thank you for the fix.

Still getting this error when using lifecycle.


Correction for raw affiliation Department of Mechanics, Mechatronics and Commands, Moulay Ismail University, 15290 ENSAM،, Meknès 50500, Meknes, Fes-Meknes, 15290, MOROCCO

Correction needed for raw affiliation Department of Mechanics, Mechatronics and Commands, Moulay Ismail University, 15290 ENSAM،, Meknès 50500, Meknes, Fes-Meknes, 15290, MOROCCO
raw_affiliation_name: Department of Mechanics, Mechatronics and Commands, Moulay Ismail University, 15290 ENSAM،, Meknès 50500, Meknes, Fes-Meknes, 15290, MOROCCO
new_rors:
previous_rors: 018pp1107
works_examples: W4387823882
contact: bf21dce330ad7dc992071dedb49642bb:ffe840efbb8e2e4772c576092d @ ensam.eu

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


tests: add Node.js 7, 8 and 9 for testing

Whilst we're here, what versions should we support?
Should we drop versions 4, 5 and 7?

Node.js 4 is in "Maintenance LTS" but soon reaches EOL on 2018-04-30.
Node.js 5 is EOL since 2016-06-30.
Node.js 7 is EOL since 2017-06-30.

https://github.com/nodejs/Release

In foundations-sites we use
https://github.com/zurb/foundation-sites/blob/6b2b85ba05489effceb791f084fb97571f2775dd/.travis.yml#L2-L5

Maybe we could do the same here?

Should be ok now.

If we're following semver, does this warrant a major version bump for the next release?

Not sure, this is just CI / testing related in general.