auth error

When creating the link for authentication, google authentication returns this message:
"Unable to login because project-367116221053 sent an invalid request. Try again or contact the developer about the issue. Learn more about the error
If you are a developer of project-367116221053, please see the error details.
Error 400: invalid_request"

just use this version for now: #665
https://github.com/BugCode1/gdrive/releases/tag/2.1.2

it didn't work, I'm using an instance on AWS.
I can't log in there, I need to log in on my machine.

I had the same problem on ubuntu 22.
This other approach worked for me:
https://medium.com/@acpanjan/download-google-drive-files-using-wget-3c2c025a8b99

just use this version for now: #665
https://github.com/BugCode1/gdrive/releases/tag/2.1.2

please make a gif tutorial pleaase >>

I just wanted to chime in and say that I installed gdrive via brew yesterday, and wasn't able to use it because of this issue.  Other pull requests say that this is because Google is deprecating (has deprecated?) oob.
I was not able to get gdrive to accept my copy+pasted code from the redirect url

Please post a screen recording of the process, blurring out specific data >>>

Hi.  I just wanted to let you know that I ran into the same problem, downloaded and tried v2.1.2, and it seems to work now.

Thanks 2.1.2 seems to work.
Please note that when you click allow it will redirect to a non-working link; that's the workaround, and you need to use the code in the url (see the placeholder below):
https://localhost:1/?state=state&code=<THIS_CODE_HERE>&scope=https://www.googleapis.com/auth/drive

where to get the code?
i got same error like this.... why localhost?
The webpage at http://localhost:1/?state=state&code=4/0AWgavde6eXXXXdZwefCsaJtmJcE8nXXXXX0pvVGjS2rL-SBOXXXXXXBPA&scope=https://www.googleapis.com/auth/drive might be temporarily down or it may have moved permanently to a new web address

ok sorry.. i understand now.....
4/0AWgavde6eXXXXdZwefCsaJtmJcE8nXXXXX0pvVGjS2rL-SBOXXXXXXBPA
this is the activation code.....
thank you so much

it doesnot wirk inside gogle colab

hope this can help.
https://www.youtube.com/watch?v=a-zI2tN7kaE

I manage to solve this using gdrive --service-account, you have to create G drive API key from Google API Console
download the key as .json file and store it on  ~/.gdrive folder , and download the shared file like this
gdrive --service-account myKeyName.json download '1234gfsdvsvsdfsdf'
You should use File ID not the shared G drive url

For those looking to use a service-account, I found this guide helpful on all the steps.

regarding @Phredward 's statement:

[this error is happening] because Google is deprecating (has deprecated?) oob.

That is correct. The existing gdrive app has not been updated to comply with Google's changed support for the OAuth protocol. Google provided instructions, but the main source has not been updated.
The update available in the BugCode1 clone allows the gdrive OAuth dance to succeed. To use it:

download the .tar.gz file for version 2.1.2 from BugCode1's clone
extract gdrive.exe from that tgz file
In a cmd or powershell window, run gdrive about.  This will then prompt you with a URL
copy that URL from your cmd or powershell window and paste it into a browser address bar.
Google will ask you to authenticate. Do so. Then approve "project-367116221053". This is the consent UI for OAuth.
The Google consent UI will then try to redirect to localhost:1 and the browser will show an error. This is expected.
In the error message you will see a URL of the form http://localhost:1/?state=state&code=******&scope=...  Copy the code , what is in place of the **** in that URL.  This will be a long odd-looking string like 4/0AWtgzh78TWfF8K66P6JCEBVNsg-Rqctd.
go back to your cmd or powershell window and paste in that code.
done.  Now gdrive.exe will work.

I manage to solve this using gdrive --service-account, you have to create G drive API key from Google API Console
download the key as .json file and store it on ~/.gdrive folder , and download the shared file like this
gdrive --service-account myKeyName.json download '1234gfsdvsvsdfsdf'
You should use File ID not the shared G drive url

This solution working like a charm thank @SalimF ❤❤


3.10.0 - Examining an item in ScavengerInventoryScreen doesnt show actual item

ScavengerInventoryScreen doesnt let you examine items and see what the item is

Originally written by Kaeno

No longer an issue, closing

Originally written by CWX


3 Phase only when MODE_NORMAL

if(Mode == MODE_NORMAL && enable3f)
CONTACTOR2_ON;
Can you explain why this MODE_NORMAL is necessary?

As far as I can see, the original code does not support 3 fase solar/smart modes.
E.g. In solar mode and EVSE sees -6A is going to the grid, it'll say the car can load at 6A, but if it charges with 3fases it will load 6A x 3f which is 18
So to avoid any issues this is currently limited for manual mode only as the logic will most likely start flipping on/off otherwise

I thought that using two contactors gives the option to use only 1 phase; not enabling 3 phase.
In the current original code, where you refer to, C1 and C2, for now, are switched at the same time. This is the same as having just one 4 pole contactor, in which situation 3-phase will always be enabled (with proper wiring of course).

C2 is for enabling 3f, you have to install them in serie, not parallel

@deqw  that is exactly what it is doing. When following the original EVSE wiring schema it is wired for 3phases via a 4p contactor. The goal of the second contactor is to be able to toggle that for two reasons: a) the original smart/solar code does not seem to take into account 3f when calculating energy b) in most cases when charging solar the minimal energy on 3p will exceed the solar energy available and start to draw power from batt/grid, to lower that you'll have to switch to 1phase charging.
So, if you put a NO 2p (C2) in series with the NO 4p (C1) (as @k-janssens indicates) then in smart/solar mode the EVSE will operate from now on in 1phase mode (as C2 is not closed). If you switch to normal mode it will activate C2 and you'll get 3phase charging.

For safety reasons we decided to go for the serial setup. If for some reason the C1 is OFF and C2 is still ON, there are 2 phase cable with power still connected with the car without an active neutral wire.
Maybe it is overly cautious but better safe than sorry was our opinion.

Do I understand correctly, I need to wire as follows?
Can you update the documentation to clarify this?

@k-janssens can you elaborate why you think the code does not take into account all three phases? If we look at the code we can see at https://github.com/serkri/SmartEVSE-3/blob/master/SmartEVSE-3/src/evse.cpp#L1333 that it will take the highest loaded 'channel' (or phase) to do its calculations. This would in theory mean it will never charge more than the highest loaded phase, which is basically what you want as there is no way to tell the car to load the other phases differently.
Or am I completely off-base here? I do understand Solar charging mode is more difficult for three-phase operation, especially if you have a single phase inverter on a three phase mains supply like me. But at least 'smart mode' with balancing should be safe with enable3f true.

I mainly talk about solar functionality as the goal there is to use 0 power from the grid.There it is an issue as the EVSE does not take into account the amount of phases the car will use.
Meaning if the EVSE tells it to charge with 6A if there is 6A solar available, a 3 phase car will draw 18A resulting in the EVSE to continuously start/stopping the charging process.
Not 100% sure there is an issue for smart mode (never used it).

Thanks for your reply. It makes perfect sense now. In that case I guess simply excluding MODE_SOLAR in that line where it enables contactor 2 as well should suffice to make three phase smart charging possible.


Correct term (Continuation)

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/8834
this edit was missing from https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/pull/8892

@CarolynRowe PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Removing test data page so we can use it as a student project.

I removed the other references to the default data.


RigidBody2D bounces when passing on top of two perfectly aligned StaticBody2Ds

Godot version:
Happens on 3.1 stable and 3.1.1rc1
OS/device including version:
ubuntu 18.04 / nvidia GTX1070 8GB driver 418.56
Issue description:
When creating two StaticBody2Ds, no matter how well aligned they are, a RigidBody2D passing over them will jump upwards sometimes. I expected the RigidBody2D to just roll over the connection since there is no gap at all, but it jumps instead.
GIF video of the issue
https://media.giphy.com/media/d7SgPewvXmuwsfw8w9/giphy.gif
Steps to reproduce:
Attach the GDScript code below to a Node2D in a new project, and run:
extends Node2D

const STATIC_SIZE = 2000
onready var img = preload('res://icon.png')

func _ready():
	var rb := RigidBody2D.new()

	var s = Sprite.new()
	s.texture = img

	rb.add_child(s)

	# Add a circle shaped collision
	var rb_col := CollisionShape2D.new()
	rb_col.shape = CircleShape2D.new()
	rb_col.shape.radius = 50
	rb.add_child(rb_col)

	# Configure to have no friction and no damp
	rb.linear_damp = 0
	rb.physics_material_override = PhysicsMaterial.new()
	rb.physics_material_override.friction = 0

	# apply impulse
	rb.apply_impulse(Vector2.ZERO, Vector2(1050, 0))

	# follow with a camera
	var camera := Camera2D.new()
	camera.current = true
	rb.add_child(camera)

	add_child(rb)

	# create 10 consecutive StaticBody2D perfectly aligned
	var pos := Vector2(STATIC_SIZE / 2, 200)
	for x in range(0, 10):
		var ground := StaticBody2D.new()
		var ground_col := CollisionShape2D.new()
		ground_col.shape = RectangleShape2D.new()
		ground_col.shape.extents = Vector2(STATIC_SIZE / 2, 100)
		ground.add_child(ground_col)

		# Add a sprite so we can see where the division is
		s = Sprite.new()
		s.position = Vector2(-STATIC_SIZE/2, -50)
		s.texture = img
		ground.add_child(s)

		ground.position = pos
		add_child(ground)

		pos.x += STATIC_SIZE

Happens for 3d as well

Fixed by #56801.


[Request]: Add blur animation to the top bar and other elements.

Is your request related to a problem?
No
How is the feature request related to the problem?
Skipped
Describe your proposed solution

Notifiction button was placed near the Close and Maximize. Close and Maximize are linear and non-background icons, but Notifiction icon with a white background. Can their display styles be unified?

The black icons in the sidebar are not clear against a dark background(like the green mark on image).

Describe alternatives you've considered
This is a UI design issue and doesn't require an alternative solution to instead.
Additional context
None

The dark icon might have been forgotten during our removal of left bar (addition of auto transparent left bar) so the UI button in light mode is a bit occluded/hard to see on certain background.
@shatyuka can you help me on this one? I'm still confused myself reading the main app theme xamls

@neon-nyan has mitigated this in Preview branch release.
Reopen this ticket if you think there still needs improvements to this issue~
Dark Mode
Light Background

Dark Background (Background credit: Shuno on Pixiv https://www.pixiv.net/en/artworks/100895729)

Light Mode
Light Background

Dark Background (Background credit: Shuno on Pixiv https://www.pixiv.net/en/artworks/100895729)

Thank you for your support <3


Release azure-core-http-jdk-httpclient Library

Tracking issue to prepare and begin releasing the azure-core-jdk-httpclient.

Is this going to be released soon?

Is a test artifact already available somewhere?

Is a test artifact already available somewhere?

It is not released yet but it is the next HTTP client we are trying to release.  Currently, it can build locally and test.

The first beta version of Azure core JDK HTTP client is release.
https://repo1.maven.org/maven2/com/azure/azure-core-http-jdk-httpclient/1.0.0-beta.1/


⚠ Bofrost.de has degraded performance

In 9f6aef5, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1492 ms

Resolved: Bofrost.de performance has improved in 0c381b1 after 1 hour, 1 minute.


Keep getting nginx error 500

Hi,
No matter what I try, I keep getting HTTP error code 500 when I visit my docker-machine ip.
I am running laradock through Docker tool box (OS: Windows 10) and have a default VM running. I have setup laradock as per the instructions in the video.
I have successfully mounted the folder that I need into the VM and have changed docker-compose.yml according to the location of my code in the VM:
Here is my docker-compose.yml file:

version: '2'
services:
Nginx Server Container
nginx:
    build: ./nginx
    volumes_from:
        - volumes_source
    volumes:
        - ./logs/nginx/:/var/log/nginx
    ports:
        - "80:80"
        - "443:443"
    links:
        - php-fpm

PHP-FPM Container
php-fpm:
    build:
        context: ./php-fpm
        dockerfile: Dockerfile-70
    volumes_from:
        - volumes_source
    expose:
        - "9000"
    links:
        - workspace

MySQL Container
mysql:
    build: ./mysql
    volumes_from:
        - volumes_data
    ports:
        - "3306:3306"
    environment:
        MYSQL_DATABASE: homestead
        MYSQL_USER: homestead
        MYSQL_PASSWORD: secret
        MYSQL_ROOT_PASSWORD: root

PostgreSQL Container
postgres:
    build: ./postgres
    volumes_from:
        - volumes_data
    ports:
        - "5432:5432"
    environment:
        POSTGRES_DB: homestead
        POSTGRES_USER: homestead
        POSTGRES_PASSWORD: secret

MariaDB Container
mariadb:
    build: ./mariadb
    volumes_from:
        - volumes_data
    ports:
        - "3306:3306"
    environment:
        MYSQL_DATABASE: homestead
        MYSQL_USER: homestead
        MYSQL_PASSWORD: secret
        MYSQL_ROOT_PASSWORD: root

Neo4j Container
neo4j:
    build: ./neo4j
    ports:
        - "7474:7474"
        - "1337:1337"
    environment:
        - NEO4J_AUTH=homestead:secret
    volumes_from:
        - volumes_data

MongoDB Container
mongo:
    build: ./mongo
    ports:
        - "27017:27017"
    volumes_from:
        - volumes_data

Redis Container
redis:
    build: ./redis
    volumes_from:
        - volumes_data
    ports:
        - "6379:6379"

Memcached Container
memcached:
    build: ./memcached
    volumes_from:
        - volumes_data
    ports:
        - "11211:11211"
    links:
        - php-fpm

Beanstalkd Container
beanstalkd:
    build: ./beanstalkd
    ports:
        - "11300:11300"
    privileged: true
    links:
        - php-fpm

Beanstalkd Console Container
beanstalkd-console:
    build: ./beanstalkd-console
    ports:
        - "2080:2080"
    links:
        - beanstalkd

Workspace Utilities Container
workspace:
    build:
        context: ./workspace
        args:
            - INSTALL_PRESTISSIMO=false
    volumes_from:
        - volumes_source
    tty: true

Laravel Application Code Container
volumes_source:
    build: ./volumes/application
    volumes:
        - /c/Users/pomodoro.xyz/code:/var/www/laravel

Databases Data Container
volumes_data:
    build: ./volumes/data
    volumes:
        - /var/lib/mysql:/var/lib/mysql
        - /var/lib/postgres:/var/lib/postgres
        - /var/lib/mariadb:/var/lib/mariadb
        - /var/lib/memcached:/var/lib/memcached
        - /var/lib/redis:/data
        - /var/lib/neo4j:/var/lib/neo4j/data
        - /var/lib/mongo:/data/db

Add more Containers below

When I log into the workspace and change files, they are reflected into the host machine, so I am good on that front.
Here is my folder structure in my work container (just as per the video):

ls -l
total 30
drwxrwxrwx 1 1000 staff 4096 Jul 20 19:58 app
-rwxrwxrwx 1 1000 staff 1646 Jul 20 19:58 artisan
drwxrwxrwx 1 1000 staff    0 Jul 20 19:58 bootstrap
-rwxrwxrwx 1 1000 staff 1272 Jul 20 19:58 composer.json
drwxrwxrwx 1 1000 staff 4096 Jul 20 19:58 config
drwxrwxrwx 1 1000 staff 4096 Jul 20 19:58 database
-rwxrwxrwx 1 1000 staff  503 Jul 20 19:58 gulpfile.js
-rwxrwxrwx 1 1000 staff   12 Jul 21 15:13 index.html
drwxrwxrwx 1 1000 staff 4096 Jul 23 09:24 laradock
-rwxrwxrwx 1 1000 staff  212 Jul 20 19:58 package.json
-rwxrwxrwx 1 1000 staff 1026 Jul 20 19:58 phpunit.xml
drwxrwxrwx 1 1000 staff 4096 Jul 23 10:39 public
-rwxrwxrwx 1 1000 staff 1918 Jul 20 19:58 readme.md
drwxrwxrwx 1 1000 staff    0 Jul 20 19:58 resources
-rwxrwxrwx 1 1000 staff  567 Jul 20 19:58 server.php
drwxrwxrwx 1 1000 staff    0 Jul 20 19:58 storage
drwxrwxrwx 1 1000 staff    0 Jul 20 19:58 tests
drwxrwxrwx 1 1000 staff    0 Jul 20 19:59 vendor

All nginx.conf files etc are being loaded into the container as I can see from docker exec -it into my nginx container.
For reference, here is my laravel.conf

server {
listen 80 default_server;
listen [::]:80 default_server ipv6only=on;
root /var/www/laravel/public;
index index.php index.html index.htm;

server_name laravel.dev www.laravel.dev;
location / {
     try_files $uri $uri/ /index.php$is_args$args;
}

location ~ \.php$ {
    try_files $uri /index.php =404;
    fastcgi_pass php-upstream;
    fastcgi_index index.php;
    fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
    include fastcgi_params;
}

location ~ /\.ht {
    deny all;
}

}

and nginx.conf:

user www-data;
worker_processes 4;
pid /run/nginx.pid;
events {
worker_connections  2048;
multi_accept on;
use epoll;
}
http {
server_tokens off;
sendfile on;
tcp_nopush on;
tcp_nodelay on;
keepalive_timeout 15;
types_hash_max_size 2048;
client_max_body_size 20M;
include /etc/nginx/mime.types;
default_type application/octet-stream;
access_log on;
error_log on;
gzip on;
gzip_disable "msie6";
include /etc/nginx/conf.d/.conf;
include /etc/nginx/sites-available/;
open_file_cache max=100;
}
daemon off;

And here is my laravel's root directory's .env file:

APP_ENV=local
APP_DEBUG=true
APP_KEY=SomeRandomString
APP_URL=http://localhost
DB_CONNECTION=mysql
DB_HOST=<IP_ADDRESS>
DB_PORT=3306
DB_DATABASE=homestead
DB_USERNAME=homestead
DB_PASSWORD=secret
CACHE_DRIVER=file
SESSION_DRIVER=file
QUEUE_DRIVER=sync
REDIS_HOST=<IP_ADDRESS>
REDIS_PASSWORD=null
REDIS_PORT=6379
MAIL_DRIVER=smtp
MAIL_HOST=mailtrap.io
MAIL_PORT=2525
MAIL_USERNAME=null
MAIL_PASSWORD=null
MAIL_ENCRYPTION=null

I have also setup the permissions of the folder by logging into the workspace container and typing:

chmod -R 777 storage bootstrap/cache

But still I keep getting the http 500 error on docker-machine ip. What am I missing?

It's a long shot but APP_KEY=SomeRandomString. Try generating an APP_KEY

It sounds like nginx is running ok, most likely, the error is on the php-fpm container. Try running docker logs -f CID.
Or run this script docker logs -f $(docker ps | grep php-fpm | awk -F $'[ ]{2,}' '{ print $7 }').
This should follow the log of the php-fpm container. You can then make a request to the server and see the out put in the terminal if there are any errors.


cleaner and faster operations wtih datasketches

I believe these changes should result in cleaner code and more efficient operations with datasketches. I would love to compare performance before and after, but I am not sure how to approach this at the moment since I have no experience with Impala yet.
I also added messages from original exceptions to re-thrown exceptions.

Thank you for your interest in contributing to Impala! We do code review via gerrit rather than GitHub PRs; you can see the process described at https://cwiki.apache.org/confluence/display/IMPALA/Contributing+to+Impala

resubmitted to Gerrit
https://gerrit.cloudera.org/#/c/17818/


Requesting access to the trained model

Could you share just the inference model?
Add complete details about the development, as the paper has a very minimal explanation of implementation.
Regards

any update?


Class drag-border doesn't disappear on fast drag and drop

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] other

What is the current behavior?
In the situation when I have multiple draggable and droppable elements, the class drag-border doesn't disappear when I'm dragging and dropping the elements very quickly.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar.
The bug is hard to reproduce, but I discovered it's easier to notice it if in the file draggable.directive.js in setTimeout() in the function Draggable.prototype.dragStart(), I change 10ms to 1000ms (line 98). The bug even stays if I lower it to 1ms, but it's much harder to reproduce. I think I fixed the bug by removing setTimeout() and keeping only the content of the function (lines 96-97), but maybe I just made it harder to reproduce or I am unknowingly causing some other issues.

What is the expected behavior?
The class drag-border always disappears when drag ends.

Please tell us about your environment:

Angular version: 5.2.0
Angular CLI Version: 1.6.4
Browser: [ tested only on Chrome 63.0.3239.132 ]

Any news about this bug ? I got the same problem with nested drag / drop container .

I have found a solution, however I don't know if it is the best aproach... however It works
I hope this help to any person in the future.
simply at the end of OnDrop function put:
`var items = document.querySelectorAll(".contenido div");
for (let j = 0; j < items.length; j++) {
  var element = items[j];
  element.classList.remove("drag-over-border");
}`

.contenido div is my particular case because i have a div with class Contenido and inside nested divs dragables.

Any updates on this bug. I have noticed that intermittently onDragEnd is not triggered and it that time when the dragClass and dragHintClass remain applied to concerned items.

Alright, this might be helpful for anybody who is facing this issue too.
For me, the actual problem lies in my project itself and not the ng-drag-drop module.
What caused my issue was:
I had a background task running, that in every few seconds updated the dom. Now this process momentarily block page (few milliseconds). Problem is that during this dom update, drag events are ignored. So for me if during that update process, dragend was triggered, it is never captured and hence ng-drag-drop is not even notified that the drag has ended, resulting in the drag and drop hint classes still being applied.
I fixed that part in my code and it works smooth.
So, make sure any such process is not scheduled in your application, which is causing the events to be lost. This doesn't seem to me as an issue with this library.


[xhamster] url regex fix for videos with empty title.

apparently xhamster allows for videos with an empty title, so the url turns out like url/.html
this made the regex (and therefore the download) to fail, so i just made a quick fix for that.

Cherry-picked in a4690b3244a42a833146c406e622c96045b23df5, thanks.
Do you want to be credited as one of the authors of youtube-dl? If so under what name?


Use relative file paths

Expected behavior
Use relative file paths instead of absolute file paths with PHP makers.
Steps to reproduce
Lint a PHP file using one of Neomake's makers.

Do you mean the output in the location list?
That might just depend on :cd, :tcd, :lcd etc.
Example output would be good.

I'm using PHP inside a Docker container. The location list displays || Could not open input file: /home/niek/test/index.php for a sample PHP file with syntax errors.

It appears to come from the maker already, i.e. it cannot open the file for some reason.
Please provide the output from :verbose NeomakeInfo (see the issue template).
How do you mount the local file?
What is the path in Vim itself?

I would assume it's an issue with shell files then, too.
Can you try :Neomake sh on some t.sh file with the following contents: s="?

I'm mounting the file as followed:
docker run \
    --init \
    --interactive \
    --rm \
    --volume "$(pwd)":/app \
    --workdir /app \
    auditemarlow/php:latest "$@"

The path of Vim itself is /home/niek/test in the case of my test example.
Output of :verbose NeomakeInfo:
makeprg=make
NVIM v0.3.0
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/cc -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -Wconversion -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=auto -Wno-array-bounds -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -I/build/neovim/src/build/config -I/build/neovim/src/neovim-0.3.0/src -I/usr/include -I/build/neovim/src/build/src/nvim/auto -I/build/neovim/src/build/include
Compiled by builduser
Features: +acl +iconv +jemalloc +tui
See ":help feature-compile"
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/usr/share/nvim"
Run :checkhealth for more info
"index.php" 3L, 28C#### Neomake debug information
Async support: 1
Current filetype: php
Windows: 0
[shell, shellcmdflag, shellslash]:['/bin/zsh -l', '-c', 0]
Enabled makers
For the current filetype ("php", used with :Neomake):

php

args: ['-l', '-d', 'display_errors=1', '-d', 'log_errors=0', '-d', 'xdebug.cli_color=0']
auto_enabled: 1
errorformat: '%-GNo syntax errors detected in%.%#,%EParse error: %#syntax error, %m in %f on line %l,%EParse error: %m in %f on line %l,%EFatal error: %m in %f on line %l,%-G\s%#,%-GErrors parsing %.%#'
exe: 'php'

phpmd

args: ['%:p', 'text', './phpmd-ruleset.xml']
auto_enabled: 1
errorformat: '%W%f:%l%\s%\s%#%m'
exe: 'phpmd'
WARNINGS:

maker's exe (phpmd) is not executable.

phpcs

args: ['--report=csv']
auto_enabled: 1
errorformat: '%-GFile,Line,Column,Type,Message,Source,Severity%.%#,"%f",%l,%c,%t%[a-zA-Z],"%m",%[a-zA-Z0-9_.-],%*[0-9]%.%#'
exe: 'phpcs'
WARNINGS:

maker's exe (phpcs) is not executable.

phpstan

args: ['analyse', '--errorFormat', 'raw', '--no-progress', '--level', 0]
auto_enabled: 1
errorformat: '%E%f:%l:%m'
exe: 'phpstan'
WARNINGS:

maker's exe (phpstan) is not executable.
NOTE: you can define g:neomake_php_enabled_makers to configure it (or b:neomake_php_enabled_makers).

Default maker settings:

output_stream: 'both'
remove_invalid_entries: 0
buffer_output: 1

For the project (used with :Neomake!):
None.
NOTE: you can define g:neomake_enabled_makers to configure it.
Settings
New-style (dict, overrides old-style)
g:neomake: {
  'automake': {
    'ignore_filetypes': ['startify'],
    'events': {
      'FileType': {},
      'FileChangedShellPost': {},
      'BufWritePost': {
        'delay': 0,
      },
      'BufWinEnter': {},
    },
  },
}
b:neomake: unset

Old-style
g:neomake_error_sign = {'text': ''}
g:neomake_info_sign = {'text': ''}
g:neomake_message_sign = {'text': ''}
g:neomake_open_list = 2
g:neomake_php_phpmd_args = ['%:p', 'text', './phpmd-ruleset.xml']
g:neomake_place_signs = 1
g:neomake_warning_sign = {'text': ''}

makeprg=make
:version
NVIM v0.3.0
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/cc -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -Wconversion -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=auto -Wno-array-bounds -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -I/build/neovim/src/build/config -I/build/neovim/src/neovim-0.3.0/src -I/usr/include -I/build/neovim/src/build/src/nvim/auto -I/build/neovim/src/build/include
Compiled by builduser

Features: +acl +iconv +jemalloc +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

:messages
"index.php" 3L, 28C

My sh maker is working fine, as I am not using Docker to containerize it: sh: unexpected EOF while looking for matching "' (E)`

My sh maker is working fine

I've meant to try it in the Docker container.
The debug output does not contain a failed run - the really interesting part.
I suggest using let g:neomake_logfile = '/tmp/neomake.log' and then triggering it.

I'm mounting the file as followed:

How do you use this container then for running the maker(s)?
Wouldn't you have to override _exe?  (the logs should show it)

Have you seen https://github.com/neomake/neomake/issues/1956?

Using this for g:neomake_php_php_exe works:
#!/bin/sh
docker run \
    --init \
    --interactive \
    --rm \
    --volume "$(pwd)":/app \
    --workdir /app \
    auditemarlow/php:latest "$@"

You could try the following patch, but it breaks other makers that use e.g. cwd (that might be fixable though, by changing before already):
diff --git i/autoload/neomake.vim w/autoload/neomake.vim
index ee698735..8f033940 100644
--- i/autoload/neomake.vim
+++ w/autoload/neomake.vim
@@ -592,7 +592,7 @@ function! s:command_maker_base._get_tempfilename(jobinfo) abort dict
         if empty(bufname)
             let temp_file = tempname() . slash . 'neomaketmp.'.a:jobinfo.ft
         else
-            let orig_file = neomake#utils#fnamemodify(a:jobinfo.bufnr, ':p')
+            let orig_file = neomake#utils#fnamemodify(a:jobinfo.bufnr, ':.')
             if empty(orig_file)
                 let dir = tempname()
                 let filename = fnamemodify(bufname, ':t')
@@ -648,7 +648,7 @@ function! s:command_maker_base._get_fname_for_buffer(jobinfo) abort
         endif
         let used_for = 'unreadable'
     else
-        let bufname = fnamemodify(bufname, ':p')
+        let bufname = fnamemodify(bufname, ':.')
         let used_for = ''
     endif

https://github.com/neomake/neomake/pull/1980 should fix it.

I can verify your changes to be working. Thanks a lot!

Great!
Thanks for bringing it up in the first place, and reporting back.


Update auth HTTP samples to use new makeHttpClient

Do this once Brent's change is added: https://github.com/google/google-auth-library-php/pull/162

Won't Fix


[bug] Firewall rules are missing (either in the code and in the doc)

Describe the bug
When I use portal on a firewall-enabled computer, I have absolutely no clues about which ports I have to open in order to make portal working
To Reproduce
Steps to reproduce the behavior:

Install ufw or firewalld and enable it
Install portal
Without any firewall modification, try to use portal
In my case, I face those errors: lookup localhost on <IP_ADDRESS>:53: no such host on the sender side, and failed to get reader: failed to read frame header: EOF on the receiver side

Expected behavior
Portal should work out-of-the-box, by either adding custom rules during installation, or by showing a warning during the first use
Platform / Portal version (please complete the following information):

OS and OS version: Arch Linux 6.4.12-arch1-1
Portal Version v1.2.3

Additional context
Strangely, this problem only occurred for me between two PCs with firewalls.
When one of them hasn't this protection or has correct firewall rules applied, transfers works perfectly

I already tried to open port 53. Same error

Nevermind, it's purely my fault! 😅
This error is thrown when the system is unable to resolve localhost domains.
It was my case: /etc/hosts was empty.
I fixed this problem and now everything works flawlessly! 👍🏼


Update references to support channels throughout docs

We're not providing the right recommendations, e.g., https://www.cockroachlabs.com/docs/stable/support-resources.html.
cc @tim-o

This has been resolved in the meantime.


[BUGFIX] Prevent Duplicates in toctree and menu

A page that has already been explicitely added to a toctree does not get added by the glob directive once more. As having multiple occurences of the same page within one toctree or menu does not make sense in general, I applied array_unique to the menu_items of one level of the toctree or menu.
@wouterj, @greg0ire is this behaviour also wanted in the doctrine and symfony docs?

AFAIK that's not something Doctrine would care about: it does not look like we use globs at all.


Not seeing Shell Integration Enabled message on Windows

Testing #153393
It turns out "Run Recent Command" works both with shell integration disabled and enabled, but I wasn't seeing the shell integration prompt on Windows.
I'm using pwsh 7. When I disable shell integration, I see the header that tells me I'm using version 7. When I enable shell integration, I see just the prompt.
My profile references the following script: https://github.com/microsoft/vscode/blob/32d13d5a8d5f1bfc3e3dd88ca39e1e61ed7208e2/src/vs/workbench/contrib/terminal/browser/media/shellIntegration.ps1

what do you mean that your profile references that script? could you provide the profile you're using?

also what's the value of your PS1?

we changed this so no longer show shell integration activated in the buffer

It doesn't seem to be in the tab list icon hover:

I think this is working as designed since we hid the message?

It turns out "Run Recent Command" works both with shell integration disabled and enabled, but I wasn't seeing the shell integration prompt on Windows.

This is also by design, it's just populating that list that only happens when shell integration is enabled.

not seeing it on mac either

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Issue with @NonNull field in unwrapped class

@graemerocher Found one issue when field is marked @NonNull deserialization throws an error telling that field is required and not set, although it is set.
It seems like some nested objects are being processed more than once during deserializaiton.

is there supposed to be a failing test here?

is there supposed to be a failing test here?

Yes, test test @JsonUnwrapped - levels is failing now. Not in github because this is PR against your branch. Basically, this is not needed, you can reproduce in your branch by adding @NonNull on any of the fields in nested classes. For example
@Serdeable
class InnerFooId {

    @NonNull
    private Long theLong;

    private String theString;

    public Long getTheLong() {
        return theLong;
    }

    public void setTheLong(Long theLong) {
        this.theLong = theLong;
    }

    public String getTheString() {
        return theString;
    }

    public void setTheString(String theString) {
        this.theString = theString;
    }
}

and the exception is

Unable to deserialize type [unwrapped.InnerFooId]. Required property [Long theLong] is not present in supplied data
io.micronaut.serde.exceptions.SerdeException: Unable to deserialize type [unwrapped.InnerFooId]. Required property [Long theLong] is not present in supplied data

because it seems nested objects are processed more than once and in second pass, properties are consumed from the first pass and then it throws error for not finding value for one of the fields.


feat: add missing icons and enforce icon sizes

No Jira ticket.
Description
According to BASE design (https://www.figma.com/file/TqaGgbpjGSUDf7qq153Isq/Product-Iconography?type=design&node-id=1-2969&mode=design&t=Sz5fcUd1rnrqmlWx-0), icons should always have 16px and 24px versions (https://toptal-core.slack.com/archives/C05CSR5RFBN/p1688574324432869?thread_ts=1687249523.324339&cid=C05CSR5RFBN).
This pull request adds missing icons (taken from BASE) and adds a pre-commit check that makes sure that icon is added.
⚠ Please note that Ach and CreditCard will be added after they are added to BASE
How to test

Check the icon list at 🔴 TBD
Try to push incomplete icon locally

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[N/A] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[N/A] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[N/A] Covered with tests

Breaking change

[N/A] codemod is created and showcased in the changeset
[N/A] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers


Crash with multi RootView by multi touch

I have many RCTRootView in my app, and got crash like this:
(my app based on 0.30.0 iOS, this repro below uses 0.25.1)
2016-08-21 18:19:36.476 [info][tid:main][RCTPerformanceLogger.m:75] Unbalanced calls start/end for tag 15
2016-08-21 18:19:36.477 [info][tid:main][RCTPerformanceLogger.m:75] Unbalanced calls start/end for tag 15
2016-08-21 18:19:36.482 [info][tid:main][RCTPerformanceLogger.m:75] Unbalanced calls start/end for tag 15
2016-08-21 18:19:39.205 UIExplorer[6490:3038384] sendEvent topTouchStart, 1
2016-08-21 18:19:39.206 UIExplorer[6490:3038384] sendEvent topTouchStart, 1
2016-08-21 18:19:39.206 UIExplorer[6490:3038384] *** Assertion failure in -[RCTEventDispatcher sendEvent:](), /Users/lituo/Documents/rn/react-native/React/Base/RCTEventDispatcher.m:145
2016-08-21 18:19:39.212 UIExplorer[6490:3038384] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Got event <RCTTouchEvent: 0x12f8e5340; name = topTouchStart; coalescing key = 1> which cannot be coalesced, but has the same eventID<PHONE_NUMBER>70560 as the previous event <RCTTouchEvent: 0x12f8e4c30; name = topTouchStart; coalescing key = 1>'
*** First throw call stack:
(0x180a42db0 0x1800a7f80 0x180a42c80 0x1813c81c0 0x1001642ac 0x10012c0fc 0x10012c250 0x186131f30 0x185d46728 0x185bd37a0 0x186133530 0x185b924d8 0x185b8f254 0x185bd1610 0x185bd0c0c 0x185ba104c 0x185b9f628 0x1809f909c 0x1809f8b30 0x1809f6830 0x180920c50 0x182208088 0x185c0a088 0x1000a8bc8 0x1804be8b8)
libc++abi.dylib: terminating with uncaught exception of type NSException`

UIExplorer's RCTRootView example can reproduces this bug (100%), try to touch the 'Embedded React Native view' and 'React Native Button' at the same time.
I noticed that in RCTEventDispatcher.m RCTGetEventID returns same value
static NSNumber *RCTGetEventID(id<RCTEvent> event)
{
  return @(
    event.viewTag.intValue |
    (((uint64_t)event.eventName.hash & 0xFFFF) << 32) |
    (((uint64_t)event.coalescingKey) << 48)
  );
}

for RCTTouchEvent
viewTag: is always 0
coalescingKey: each RCTRootView's RCTTouchHandler has this value starting from 0
So set a viewTag (maybe just the target view's reactTag) for RCTTouchEvent solves this,
any other better solutions?

And one more thing, I got warning 'Cannot find single active touch.' on JS side, stack:
reactConsoleError	@	ExceptionsManager.js:76
console.error	@	YellowBox.js:59
warning	@	warning.js:44
recordTouchTrack	@	ResponderTouchHistoryStore.js:174
extractEvents	@	ResponderEventPlugin.js:441
extractEvents	@	EventPluginHub.js:186
handleTopLevel	@	ReactEventEmitterMixin.js:28
(anonymous function)	@	ReactNativeEventEmitter.js:125
perform	@	Transaction.js:138
batchedUpdates	@	ReactDefaultBatchingStrategy.js:63
batchedUpdates	@	ReactUpdates.js:98
_receiveRootNodeIDEvent	@	ReactNativeEventEmitter.js:124
receiveTouches	@	ReactNativeEventEmitter.js:186
__callFunction	@	MessageQueue.js:193
(anonymous function)	@	MessageQueue.js:94
guard	@	MessageQueue.js:41
callFunctionReturnFlushedQueue	@	MessageQueue.js:93
onmessage	@	debuggerWorker.js:39

UIExplorer's RCTRootView example can reproduces this bug (not 100% easier on real device), try to hold the 'Embedded React Native view' and then tap the 'React Native Button'.
There's already an issue for this #79, reopen it?

For the first issue, after look into android's impl, I'm sure it's a bug.
Android TouchEvent
iOS RCTTouchEvent viewTag is missing

For the second issue: 'Cannot find single active touch', turnning off the RCTTouchHandler for the inner view solves this warning.

@littlesome  It happened to me, too. Did you send a PR?


Add maybe my block list?

The pages come mainly from emails I receive myself, or from scammers.info, which nobody seems to be collecting and putting them into a list.
Most of the addresses in my list are apparently not blocked in any other list. The phising pages are not blocked by Google Safe Browsing list either, making them even more dangerous.
The list is available as a domain list and also as a hosts file.
All the domains I've collected are sorted by addition date along with the reason and source. I am keeping domains for a few months that have been affected even if the page returns 404 currently or domain not found, just in case the domain is still registered and up for being used in the short future.

Hi Marcos @socram8888 thank you for this.
Does your list have a versionable history, or a track record that shows a history of curation?  That's important.
Using ghosts I see that there is very little overlap between the two lists, just six domains in common.  So including yours would give us 297 new domains.
$ ghosts -c https://orca.pet/notonmyshift/hosts.txt
----------------------------------------
Base hosts file summary:
----------------------------------------
Location: https://raw.githubusercontent.com/StevenBlack/hosts/master/hosts
Domains: 55,942
Bytes: 1.7 MB
----------------------------------------
----------------------------------------
Compared hosts file summary:
----------------------------------------
Location: https://orca.pet/notonmyshift/hosts.txt
Domains: 303
Bytes: 10 kB
----------------------------------------
Intersection: 6 domains

The domain list is a file available at my website's repository, along with the commit history: https://github.com/socram8888/orca.pet/commits/master/notonmyshift/domains.txt
I will be moving either today or tomorrow the file to its own repository, so changes to my pages are independent to that of the block list.

The list has been alive for about a month.
I'll keep an eye on it.  I'm happy to include it once we know it's active over the long haul.

Done, I've moved it into another repo so changes are easier to follow: https://github.com/socram8888/not-on-my-shift

This is now part of #1439 so can be easily closed.
Ping Steven @StevenBlack


branch-2-0 add show_table_data http api

In some cases, users need to get the data size of single replica of a table, and evaluate certain actions based on this, such as estimating the precise backup size.
Proposed changes
Issue Number: close #xxx
cherry pick from #28380
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run p0

run clickbench

run buildall

run buildall

run buildall


Escape ampersand in cinderblock.xml

It appears as though TinderBox doesn't have this issue, but other tools will die during parse when an unescaped ampersand is encountered.

Thanks! I didn't realize that, but it makes sense.


Add Method to Go to Next Event

Adds next_event(discreetly:bool) to Dialogic class.
Sometimes, you'll want to go to the next event without pressing the 'next' button, such as when you want to give an effect of sudden interruption.
The discreetly parameter is for whether you want the Passing audio to be played. Usually, it is played when you go to the next event.
Test project: dialogic_sandbox.zip
In addition to the default buttons, pressing R goes to the next event (it even skips the event if the text is not finished).

I have updated the test project to keep it up-to-date with other recently accepted merged, just in case I resolved conflict in the wrong areas:
dialogic_sandbox.zip
So far, nothing went wrong in the process.

Thank you! Looks good :)


Make a set of JWT decoding keys public

Hello!
Seems like the provided example does not work.
I am not able to access the keys.
The issues are:

field "keys" of Jwks is private.
field "decoding_key" of Jwk is private.

is it possible to make the keys property public?
/// A container for a set of JWT decoding keys.
///
/// The container can be used to validate any JWT that identifies a known key
/// through the `kid` attribute in the token's header.
#[derive(Clone)]
#[allow(dead_code)]
pub struct Jwks {
    pub keys: HashMap<String, Jwk>,
}

The same for
pub struct Jwk {
    pub decoding_key: DecodingKey,
}

I was trying to make a PR but unfortunately I am not able to push a new branch :(

Thanks for reporting, let's make it work :)

@slavik-pastushenko should be fixed now in 0.1.3, feel free to fork and send a pull request if you find anything :)

thanks a lot!


[AutoPR authorization/resource-manager] Python Authorization to multi api

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3151

(message created by the CI based on PR content)

Codecov Report

Merging #2730 into restapi_auto_authorization/resource-manager will not change coverage.
The diff coverage is n/a.

@@                             Coverage Diff                              @@
##           restapi_auto_authorization/resource-manager    #2730   +/-   ##
============================================================================
  Coverage                                        55.89%   55.89%
============================================================================
  Files                                             7264     7264
  Lines                                           152632   152632
============================================================================
  Hits                                             85312    85312
  Misses                                           67320    67320

Impacted Files
Coverage Δ

...15_07_01/operations/role_definitions_operations.py
14.91% <ø> (ø)
:arrow_up:

...15_07_01/operations/role_assignments_operations.py
11.94% <ø> (ø)
:arrow_up:

...n/v2015_07_01/operations/permissions_operations.py
19.64% <ø> (ø)
:arrow_up:

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 5444f71...0a99086. Read the comment docs.

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2732


Add revision and trust_remote_code to from_pretrained calls

I was trying out Snowflake/snowflake-arctic-embed-m-long with onnx engine
infinity_emb --model-name-or-path Snowflake/snowflake-arctic-embed-m-long --revision 08e7a4449e3f07709fb9387bc3172d393a6cc5e2 --engine optimum --device cpu --dtype float32 --batch-size 4 --port 8000 --trust-remote-code

But trust_remote_code is not being passed along
  File "/app/infinity_emb/transformer/embedder/optimum.py", line 51, in __init__
    self.model = optimize_model(
  File "/app/infinity_emb/transformer/utils_optimum.py", line 107, in optimize_model
    unoptimized_model_path = model_class.from_pretrained(
  File "/app/.venv/lib/python3.10/site-packages/optimum/onnxruntime/modeling_ort.py", line 669, in from_pretrained
    return super().from_pretrained(
  File "/app/.venv/lib/python3.10/site-packages/optimum/modeling_base.py", line 373, in from_pretrained
    config = cls._load_config(
  File "/app/.venv/lib/python3.10/site-packages/optimum/modeling_base.py", line 231, in _load_config
    config = AutoConfig.from_pretrained(
  File "/app/.venv/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 1141, in from_pretrained
    trust_remote_code = resolve_trust_remote_code(
  File "/app/.venv/lib/python3.10/site-packages/transformers/dynamic_module_utils.py", line 622, in resolve_trust_remote_code
    raise ValueError(
ValueError: Loading Snowflake/snowflake-arctic-embed-m-long requires you to execute the configuration file in that repo on your local machine. Make sure you have read the code there to avoid malicious use, then set the option `trust_remote_code=True` to remove this error.

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 74.43%. Comparing base (8d78747) to head (3061154).

Files
Patch %
Lines

...ty_emb/infinity_emb/transformer/embedder/neuron.py
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #224      +/-   ##
==========================================
+ Coverage   74.37%   74.43%   +0.05%
==========================================
  Files          32       32
  Lines        1889     1889
==========================================
+ Hits         1405     1406       +1
+ Misses        484      483       -1

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

Thanks for your contribution!


Classic Start menu not working on Windows 10 build 17692 and later

This is a very serious issue that hits me with the slow ring build 17692:
The first click on the start button opens Classic Start Menu normally; The second click only opens half of it; Then all subsequent clicks display nothing. Shift+click displays the native Windows start menu normally.
The same problem was also reported on the Classic Shell forum for the fast ring build 17704:
http://www.classicshell.net/forum/viewtopic.php?f=12&t=8457
(Side issue: It's not possible to register on that forum anymore, as the captcha system it uses is shut down.)
Update: I just noticed that the problem was already known and reported there:
https://github.com/passionate-coder/Classic-Start/issues/11

Closing as duplicate of https://github.com/passionate-coder/Classic-Start/issues/11


FC003 no longer needed?

As of  chef-client 11.8.0 zero/local mode are supported out of the box, and thus server functions such as search are available.

It'd be useful to reverse FC003, so it catches old chef-solo code to be removed rather than eliminating it entirely.

I'd vote we make set this rule to only alarm for older chef versions and make a new rule to remove solo checks and the use of solo search

I wouldn't bother with alarming on older chef versions.  That just becomes cruft, and people writing cookbooks on 12.5.0 and uploading them to supermarket should still not be encouraged to be added chef_solo specific hacks.  In that case we should just remove FC003 and turn up the new rule.

6 months later me has seen the light. We should in fact just remove this rule. It doesn't make sense in modern Chef design.

🎂

FC003 has been removed from master


Error occurs on .net Core 2.0 based Services

Hi,
Library fails while running on .net Core 2.0 based service applications.
Error that occurs while application running:
Unhandled Exception: System.ComponentModel.Win32Exception: The service process could not connect to the service controller
at DasMulli.Win32.ServiceUtils.Win32ServiceHost.RunAsync()
at DasMulli.Win32.ServiceUtils.Win32ServiceHost.Run()
at QRAuth.UsageService.Program.Main(String[] args) in C:\Users...\Program.cs:line ...
Event Viewer logs:

hi!
What version of windows are you running on?
Could you try adding https://ci.appveyor.com/nuget/dotnet-win32-service as a NuGet Source and use the version 1.1.0-master-000080 to see if it helps?
You wrote "while application runs" - does it fail immediately or run for some time and then crash? (assuming it was started as a windows service)

My OS is Windows 10 Pro.
I tried with version 1.1.0-master-000080 and 1.1.0 but getting same error and dotnet.exe crashs.
I' m trying "dotnet run" and "dotnet run --register-service" commands separately but getting same result. In CMD screen cursor waits for a while but suddenly returns error after waiting between 20-30 seconds.

just to make sure: Are you running the cmd as Administrator?

Yes, running it as Administrator.

Good/bad news: compiling the sample (sample/Testservice) on 2.0.0 using the released 1.0.1 version by replacing the project reference with a package reference (needs reverting to CreateService from CreateOrUpdateService) works fine on my win10 1703 enterprise.
Could you try the same thing on your machine?
Did this work with .net core 1.0 or 1.1 before on the same machine?

I'm getting the same error - also on Windows 10.
It does not work on dotnet 1.1 either.

@pkjaer does the sample service work? could you clone the latest master and try dotnet run --register-service from an admin command prompt in the sample app's directory and see if it registers a service and could start it? (should be visible in the service management)
The problem is that the code path in the stack trace looks fine, the win32 exception is a standard windows exception that can occur in the following scenario (according to some google research):

Program started directly (cmd, explorer, task scheduler, …), not through service manager.
Restrictions imposed by system security configuration. (managed environments)
Weird file permissions setups. (that no one could explain)

Only other reason I can think of at the moment is a mismatch between registered service names and names offered via the API call (=> the name property that the service class has to implement).

Thanks! Would be interested to hear the outcome.

Oh, actually -- the sample project also doesn't run if I do --run-as-service instead of --register-service.

yes because --run-as-service is supposed to be passed in by the service controller and not invoked manually.

you should get the same exception when you use ServiceBase.Run(…) on .net framework without any checks to Environment.UserInteractive (which works differently on core)

I understand. Is there any way to debug the service, though?

Yes that was always 💩 to debug even with ServiceBase. See related https://github.com/dasMulli/dotnet-win32-service/issues/18#issuecomment-267798630

Closing this because the discussion about running in and out of service environment has been resolved and haven't heard back from the OP.
(Reopen / post with additional issues if it hasn't been resolved)


Mouse Highlighter leaving primary button circles behind

Microsoft PowerToys version
0.74.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Mouse Utilities
Steps to reproduce
I can't reliably reproduce
The issue is intermittent
✔ Expected Behavior
No artefacts left behind on screen - the mouse highlight circle to go away
❌ Actual Behavior
Intermittently the mouse highlight circle
only for primary button (no secondary)
gets left behind on the screen
If I switch off the "Mouse highlighter" feature then the screen is cleared of any persistent highlights
Other Software
No response

/dup #21099


Can't search for applications

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.22000.1936
Other software
OS Build Version: 10.0.22000.1936.amd64fre.co_release.210604-1628
.NET Version: .NET 6.0.16
Steps to reproduce the bug
Following the walkthrough for machine configuration. Selected a repository. That worked fine. Next step was to add an extension.
Expected result
I was expecting to see (or be able to search for ) extensions.
Actual result
The placeholder text said something like Try GitHub, but I couldn't type anything.  There was no text box visible and clicking on where I thought the textbox should be had no effect.  No
Included System Information
CPU: AMD Ryzen 7 2700X Eight-Core Processor
Physical Memory: 31.93GB (11.81GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_<IP_ADDRESS>_x64__8wekyb3d8bbwe

If the search box is disabled it likely means that the catalog connection to enable search has not been established yet, or an error occurred when attempting to connect. To troubleshoot this issue, Is it possible to check in the log file (path details below) for this string: Connecting to all catalogs completed. If the string was not found, then either the connection has not yet been established or an error occurred (latter should be logged in the same file).
Log file directory: %LOCALAPPDATA%\Packages\Microsoft.Windows.DevHome_8wekyb3d8bbwe\TempState\SetupFlow

I just installed version <IP_ADDRESS> and the application now works as expected.  Thanks.


I encountered an error while following the Habitat Sim Image Extractor Tutorial for image extraction.

Habitat-Lab and Habitat-Sim versions
Habitat-Lab: None
Habitat-Sim: v0.3.0
Habitat is under active development, and we advise users to restrict themselves to stable releases. Are you using the latest release versions of Habitat-Lab and Habitat-Sim? Your question may already be addressed in the latest versions. We may also not be able to help with problems in earlier versions because they sometimes lack the more verbose logging needed for debugging.
Master branch contains 'bleeding edge' code and should be used at your own risk.
Docs and Tutorials
Did you read the docs? https://aihabitat.org/docs/habitat-lab/
Yes
Did you check out the tutorials? https://aihabitat.org/tutorial/2020/
Yes
Perhaps your question is answered there. If not, carry on!
❓ Questions and Help
I encountered an error while building the ImageExtractor as I was following the image extraction tutorial provided on the https://aihabitat.org/docs/habitat-sim/image-extractor.html website
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 2 dimensions. The detected shape was (45, 3) + inhomogeneous part.

I know why I made a mistake. The reason for my mistake is habitat sim 0.3.0 requires numpy<1.24.0,>=1.20.0, but I have numpy 1.24.4

@nilinykh yes, you are right, thanks for your advice


Adding zib ValueSets to CodeableConcepts with required bindings

Sometimes FHIR defines elements with a required binding, whereas the zib prescribes a different ValueSet for the concept. For elements of type code, the CodeSpecification-extension is used (combined with a ConceptMap). However, for elements of type CodeableConcept, the expected way to handle this is to add a second CodeableConcept.coding.
Guidance needs to be developed on these topics:

How to prescribe in the profile that the second ValueSet should be used?
When is the addition of the second coding required/expected? Only when the mapping is not equal or in all circumstances?

I performed a quick analysis of our code-specification usage in STU3 for zib2017 profiles. We have only added it to datatypes such as code and coding and one time a string.
Shall we wait for making the exact guidance until we encounter this for real? Or do you know a case?
Preliminary thoughts for when we encounter a CodeadbleConcept element that have a required binding, the usage of slice on CodeableConcept.coding seems to be the way to go. We should only require an additional coding if the mapping is not equal or equivalent. If equal, it may be added too.

Yes, we do have a case: the concepts ProblemStatus and VerificationStatus of zib Problem, which are mapped to Condition.clinicalStatus and Condition.verificationStatus.
On the topic of equal mappings; if the zib defines a required binding, and not all elements map to the FHIR ValueSet, shouldn't the FHIR ValueSet be constrained?

This is also in the case Encounter.class

For the completeness here the required binding strength rules as applicable on a CodeableConcept:

The following rules apply when required bindings are used with the CodeableConcept data type:
•	at least one Coding element SHALL be present
•	one of the Coding values SHALL be from the specified value set
•	text can be provided as well, and is always recommended, but is not an acceptable substitute for the required code

Proposal (for addition in paragraph 7.3 in profiling guidelines):

Profiling a CodeableConcept.coding is required when the mapping of codes contain a code that cannot be mapped  equal.
Slice CodeableConcept.coding into one or two open slices discriminated by value/$this based on a required binding on the CodeableConcept.coding level.
-Mapping to the concepts of the functional description is applied both to the FHIR element and the CodeableConcept.coding slice.

Possibly a slice that recreates the FHIR element, see note below.

Slicename =  copy of the FHIR element name for the parent CodeableConcept
Binding = copy FHIR binding of parent CodeableConcept – (with copy of the ConceptMap reference extension too?)
Cardinality = 1..1

A slice that holds the zib terminology.

Slice name = “The slice is named after the full English name of the value set according to the functional description, in camelCased notation.”
Binding = zib valueset and required to make a valid slice definition
Cardinality = zib concept cardinality (most likely 0..1)
Comment = guidance on when to use this coding.
Note: The slice that copies the FHIR valueset binding may not be necessary because FHIR dictates the required rules from the CodeadbleConcept level. Copying this information may however be clearer to the implementer.

I made an example of both ways for Condition.clinicalStatus and Condition.verificationStatus as shown here in the image

Made two examples: one valid and another invalid to test this out. Seems to work as expected.

I think we should go for only creating a slice for our zib terminology. Don't copy FHIR.

On the topic of equal mappings; if the zib defines a required binding, and not all elements map to the FHIR ValueSet, shouldn't the FHIR ValueSet be constrained

That’s a good question… For Condition.verificationStatus that would mean we would constrain the FHIR valueset by removing the Entered-in-error code.
I think we should constrain the FHIR ValueSet to only the codes that match with the zib. However, I would like to not do this as this might be a lot of additional work and touches the open-world modelling principle. What’s your thought on this?
@Ashwin-nictiz

This is also in the case Encounter.class
Encounter.class is of type Coding.

Nice work @ArdonToonstra . I can see benefits and drawbacks of both approaches. Maybe we should put a textual hint in the slice documentation that the base FHIR binding still applies, and link to the relevant ConceptMap. In addition to that: shouldn't the ConceptMap be referenced using the extension as well?
I'm undecided on the requirement to include the zib code only when the code is not equal/equivalent. Wouldn't it be easier to always require the zib code when the ValueSet as a whole is incompatible, and not when it is compatible?

Agreed with the textual hint in the slice documentation (I proposed on the comment of the coding) including the mention of an available ConceptMap. I am not sure if you want the ConceptMap extension here too? Because you will not map codes to this particular binding.  But I am not completely opposed to the idea.

I'm undecided on the requirement to include the zib code only when the code is not equal/equivalent. Wouldn't it be easier to always require the zib code when the ValueSet as a whole is incompatible, and not when it is compatible?

This might potentially give a lot of work to systems that do not natively support zib terminology. Some research data platforms are modelling EPD data into a data model based on FHIR and our zib profiles. If they want to exchange data based on our zib profiles, they will need to (manually) map their internal codes to both FHIR and zibs terminology while this might not be necessary for all codes. @LilianMinne can you comment on this to see if I am making a correct point here?

Agreed with the textual hint in the slice documentation (I proposed on the comment of the coding)

Oops, missed that one, sorry

ConceptMaps are there to define the relationship between concepts, not specifically between a non-bound and a bound ValueSet. I think it makes a lot of sense to formally include it here, it tells the implementer how to go from the zib terminology to the FHIR terminology.

This might potentially give a lot of work to systems that do not natively support zib terminology. Some research data platforms are modelling EPD data into a data model based on FHIR and our zib profiles. If they want to exchange data based on our zib profiles, they will need to (manually) map their internal codes to both FHIR and zibs terminology while this might not be necessary for all codes.

Yeah, but you will always have to do this when the ValueSets are not fully equivalent, unless you can somehow guarantee that you will never send nor receive the codes that are not equivalent. So you need to create some complex logic for a fairly rare use case.
I also wonder why you need to map to both the zib and FHIR ValueSets. When you're doing zibs, you deal with zib ValueSets and use the given ConceptMap to go from there to the FHIR ValueSet.

@ArdonToonstra @pieter-edelman-nictiz it's indeed true that some hospitals map their source data in the research data platform directly to FHIR and it's additional work for them to map to ZIB's as well. It's true that the mapping can be done to ZIB's first and then use ConceptMaps to get to FHIR, but this is not the way it's currently implemented. But as they need to change this to be able to qualify for MedMij, it doesn't really matter if they have to do this for all cases or only for cases where the mapping is not equivalent.

Just spoke with Lilian and other colleagues working as data modellers and indeed concluded that it seems more logical to expect zib terminology for all concepts if a complete ValueSet cannot be mapped perfectly.

I seems like we're reaching a consensus. To improve on @ArdonToonstra's earlier proposal

A ConceptMap will be created to translate from the ValueSet of the functional model to the FHIR core ValueSet.
Profiling CodeableConcept.coding is required when not all the codes in the ValueSet from the functional description can be mapped to the base FHIR ValueSet with an equivalence of equivalent or equal.
CodeableConcept.coding will be sliced with a discriminator of value/$this, based on a required binding on the CodeableConcept.coding level.
A slice will be created that binds the ValueSet of the functional description:

The slice is named after the full English name of the value set according to the functional description, in camelCased notation.
Binding strength = required to make a valid slice definition.
The ConceptMap is documented using the permitted-value-conceptmap extension.
Cardinality = 1..1 (when the concept itself is included, bot the FHIR and function model codes are required).
Comment = A default string to describe the relation to the base binding. Proposal: "In addition to a coding from this ValueSet, the corresponding coding from the FHIR base ValueSet SHOULD be communicated. The ConceptMap <[canonical of ConceptMap]> can be used to relate these two ValueSets."

Mapping to the concepts of the functional description is applied only to the FHIR element.

Yes, I think we have a consensus. Nicely worded Pieter. I do have the next comments

"In addition to a coding from this ValueSet, the corresponding coding from the FHIR base ValueSet SHOULD be communicated. The ConceptMap <[canonical of ConceptMap]> can be used to relate these two ValueSets."
If it concerns a required binding from the FHIR base value set it concerns a SHALL from the FHIR spec itself. We provide a SHOULD because we forse this construct for extensible bindings too?

Mapping to the concepts of the functional description is applied only to the FHIR element.
This includes the ConceptMap reference with the permitted-value-conceptmap extension too, right?

If it concerns a required binding from the FHIR base value set it concerns a SHALL from the FHIR spec itself. We provide a SHOULD because we forse this construct for extensible bindings too?

Hmm ... maybe I didn't think this through. I think it should be a "SHALL". Extensible doesn't mean you can do just whatever if there's also a alternative code from another code system.

This includes the ConceptMap reference with the permitted-value-conceptmap extension too, right?

I meant the "root" of the concept, thus including the zib slice and including the permitted-value-conceptmap extension.

Ok! We will go for SHALL. I think we can wrap this up in the Profiling Guidelines.

Added this guidance to the profiling guidelines

When profiling AllergyIntolerance following the guidelines (https://informatiestandaarden.nictiz.nl/wiki/FHIR:V1.0_FHIR_Profiling_Guidelines_R4#Metadata_for_elements) I noticed 2 inconsistencies:

When slicing coding, the permitted-value-conceptmap extension is added to the slice ValueSet .binding, with binding.description set to: "In addition to a coding from this ValueSet, the corresponding coding from the FHIR base ValueSet SHALL be communicated. The ConceptMap <[canonical of ConceptMap]> can be used to relate these two ValueSets."
When adding the CodeSpecification extension, the permitted-value-conceptmap extension is added to the *FHIR* ValueSet .binding, with binding.description` set to: "Use ConceptMap [ConceptMap.name] to translate terminology from the functional model to profile terminology in ValueSet [ValueSet.name]" (https://informatiestandaarden.nictiz.nl/wiki/FHIR:V1.0_FHIR_Profiling_Guidelines_R4#Metadata_for_elements)

I would argue that it is more consistent to:

Always add the permitted-value-conceptmap extension to the FHIR element
Always add "Use ConceptMap [ConceptMap.name] to translate terminology from the functional model to profile terminology in ValueSet [ValueSet.name]" to the FHIR element binding.description.
Add "In addition to a coding from this ValueSet, the corresponding coding from the FHIR base ValueSet SHALL be communicated. The ConceptMap <[canonical of ConceptMap]> can be used to relate these two ValueSets." to comment of the .coding slice.

The only situation wel .coding is sliced at the moment in the integration branch seems zib-Problem, and here the three bullets above are followed instead of the guidelines.

I was not completely right when I said that zib-Problem already follows the proposal above. It seems Condition.verificationStatus does not refer to the ConcetMap using the permitted-value-conceptmap extension, only by comment on the slice.

I think your proposal is an improvement, and I would think we should always add the permitted-value-conceptmap extension to the FHIR element.
We had a discussion in this issue whether to include the extension on both elements. I believe the consensus was to add them to both?

ArdonToonstra commented on May 18
Agreed with the textual hint in the slice documentation (I proposed on the comment of the coding) including the mention of an available ConceptMap. I am not sure if you want the ConceptMap extension here too? Because you will not map codes to this particular binding. But I am not completely opposed to the idea.

pieter-edelman-nictiz commented on May 18
ConceptMaps are there to define the relationship between concepts, not specifically between a non-bound and a bound ValueSet. I think it makes a lot of sense to formally include it here, it tells the implementer how to go from the zib terminology to the FHIR terminology.

When reading the permitted-value-conceptmap definition (https://www.hl7.org/fhir/extension-11179-permitted-value-conceptmap.html) I notice a few things:

It should be used together with https://www.hl7.org/fhir/extension-11179-permitted-value-valueset.html, but if we agree that permitted-value-valueset in our case always is a zib valueset, we can get away with that.

The source for the concept map is the value set the extension appears on. The target is the permitted-value-valueset.

The extension should appear on the FHIR ValueSet, BUT this ValueSet should be the source, while the target is the zib ValueSet. At the moment we do this the other way around.


Session is not refreshed and always valid

Ask your question
I use nuxt-auth to authenticate my users agains my ID provider.
However, after a user signed in, he is always able to access the system. If I terminate his session via my ID provider or if I revoke a role which is required for the authentication, this data is not getting updated in my Nuxt application, the user is still authenticated there ...
How to work with this issue correctly?
Additional information
No response

Hey @Prutheus! 👋
Thanks for the question - what NuxtAuthHandler configuration do you use? credentials and then a $fetch to your own auth-system in the authorize hook or something different?

Hey @BracketJohn , I use the Keycloak Provider against my Keycloak instance

I think the cookie expiration timeout needs to be set to a low value instead of 1 month (default) to reload the session, right?

So, nuxt-auth uses NextAuth.js under the hood rotates the expiry to keep the session from ending, see https://next-auth.js.org/getting-started/client.
If you want to have a quicker client-session expiry, you can set maxAge of session to some very low value -> this way unless the user is (quite) active and refreshes the session often, the session will expire: https://next-auth.js.org/configuration/options#session
If you want to expire the session as soon as keycloak expires the session, you'll need to:

add a session expiry check into the session or jwt callback
if the session expiry check returns true (i.e.: keycloak says the session was invalidated), you set a custom attribute in the session data / jwt that you return from these hooks, e.g.: error: 'session-expired'
in the frontend add a middleware that checks for error === 'session-expiored' and calls the signOut function

Does this help? (:

Closing this now as no further response from questioner. Feel free to comment again here / open a new issue in the future!


Load fonts from disk, not Google Fonts

Does this have to be done for all the fonts referenced in the code?

#22

We should store all the fonts locally as those doesn't change too often (at all?).
Thanks for you PR by the way. 😄

Closed #22


Plugin lexon: link to gitlab 404

https://gitlab.com/lexonfoundation/lexon-remix

opened issue
https://gitlab.com/lexon-foundation/lexon-remix/-/issues/17


Fancy search

Ideally, the search bar should be able to process basic sentence structure, such as: "math class taught by Professor Rainsford at 10 AM on Monday and Thursday", which would then break down into the search components (department = math && professor = Rainsford && start-time = 10AM && (day = Monday || day = Thursday).

I doubt anyone is actually going to make searches like the above example, so we probably aren't going to make a grammar-parsing search. However, we still want a "fancy" intelligent search.

This might be able to be closed as "good enough" with the new full-text search.

@aosdict agreed, this can be closed. potential for search enhancements can be re-evaluated in the future, but this idea may conflict with the current implementation and the current implementation is "good enough" even without enhancement


RateLimiter stress test is failing occasionally

The RateLimiter stress test is failing sometimes:
https://ci.appveyor.com/project/GoogleCloudPlatform/google-cloud-dotnet/build/1.0.463-appveyor
The failure seems to only be one extra trace.
I can't seem to repo the issue with the current test on linux or windows.

(I've seen it two or three times on AppVeyor, but only AppVeyor.)

I also saw this locally when I was testing the initial stress test problems (before the task yields were added). I had increased the timeout to 30.1 seconds and forced the process to use a single core and I was almost always getting 31 traces instead of 30. So maybe even with the yields the original thread is not getting control again for another second. Maybe check for elapsed time before and after the delay and use the floor of the elapsed seconds as an upper bound on the allowed trace count.

@evildour I think that may be the solution.  When I increase the tasks and time limit the total run time goes well over the delay but the floor of the elapsed time seems to always be the correct count.

Ok cool. You may want to allow for it to be off by one though (one fewer than you expect) because if the elapsed time is close to 3000 milliseconds, you'll expect 3 traces, but the 3rd might not have gone through.

It's now failing more than occasionally (on AppVeyor), possibly related to the introduction of coverage. I'll skip it for now, but add some more diagnostics and investigate further.

Sample output with some more diagnostics:
Starting at 11:57:48.750
Incrementing at 11:57:50.772
Incrementing at 11:57:52.771
Finished at 11:58:02.763

Notice how there's a 10 second gap. I wonder whether the VM gets suspended periodically, which would definitely affect things. Adding more diagnostics...

When we start the threads, they're completely tight-looping, which means it takes quite a long time even to start all the threads.
Options (not mutually exclusive):

Create a semaphore so that we can wait for all the threads to start, then work out the end time, then release the sempahore
Just pause for a second inside the thread, so all the threads can be started
Create fewer threads
Sleep(1) inside the loop, effectively like Thread.Yield
Change the timing so that we get the threads going, reset the counter and time it from that point

@iantalarico Any thoughts?

I think the semaphore would be the next best thing to try if the sleep doesn't work.  Thanks for looking at this Jon!


the method get_onnx_model() should not need the path to original model

Hi,
After having obtained our ONNX model thanks to the export_and_get_onnx_model() method, we want to load it in order to use it in production.
There is a method called get_onnx_model() which requires the following arguments: model_name_or_path, onnx_models_path=saved_models_path and quantized=True
This is conceptually strange: why do I need to access the original model (via model_name_or_path) when I want to use the ONNX one?
I explored the code and figured out that model_name_or_path is needed for 2 things:

get the model name that was used to create the ONNX files names
get model name configuration

In order to avoid passing the model_name_or_path argument in get_onnx_model(), the export_and_get_onnx_model() method could save the model name configuration in the ONNX folder and could arrange a standard model name for ONNX files names as we can now (in the latest version of fastt5) customize ONNX folder path.
What do you think?

yes, it is bit confusing, it should be model_name, not model_name_or_path, I'll make this change in the next update.
the purpose of the model_name (the current model_name_or_path) is to select a particular model from a custom folder if you've stored more than one type of model in that folder.

the export_and_get_onnx_model() method could save the model name configuration in the ONNX folder and could arrange a standard model name for ONNX files names as we can now (in the latest version of fastt5) customize ONNX folder path.

can you elaborate more on this.

it should be model_name, not model_name_or_path

I agree. This is exactly my point: the name of the model is important to get back our ONNX model with get_onnx_model(), not the path to the model.
My second point is that get_onnx_model() calls the class OnnxT5 in step 4, and this class is initialized with the following code:
config = AutoConfig.from_pretrained(
            model_or_model_path, use_auth_token=get_auth_token()
        )

I think we could save the config file through export_and_get_onnx_model() . If not, we need model_or_model_path as argument of get_onnx_model() but we do not want that.


Change component init script name x-Init.lua to package.lua or in some other formats in the whitelist

Solved in a6a8407.


Support dotnet flavored NuGet projects

@osafi Just a reminder that this PR has addressable comments :smiley:


:recycle: Chore: Migrate classes to factories to make @_jxom on twitter happy

Description
Factory functions are more consistent with the rest of EVMts and viem
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #752

PR #753   👈

This stack of pull requests is managed by Graphite.


Other Level of Description

We are using your plugin to import a large amount (16,000 records) of data from an old records management system, and use the  "Other Level" for Description for most items as they were described at the box level. The plug in allows us to input "Other Level" in the Description Level column, without issue the "Other Level" field is blank. But without a column to input the text for the name of that level, when ever we need to edit a record we need to add it.  And as the column is blank I can not use batch find replace to add the information.
I noticed that you have this as a question on the spreadsheet:
"Should there be a column for Other Level of Description in Basic Information? And/or can it default to "unspecified"?"
I wanted to know if there was a plan to add this to the plug in.

Hi; sorry I've been slammed at work.  I hope to take a look at this today, and see if I can get you the answer you want.
--Bobbi

OK,  I've taken a look; it wasn't clear from your email exactly what you were doing to get the results you were getting.  Let me make sure we are on the same page:

Using the template spreadsheet, you've added "Other Level" to the Data Validation List for the "Description Level" column
You then are successful in specifying archival objects with a "Description Level" of "Other Level"
Once you get those objects loaded, they are all missing the free text "Other Level" field that, were you to create the archival object manually, is a required field.

Assuming this is correct (and I tried it myself):
Because the initial version of this plugin was to satisfy the needs of Harvard's archivists, and they don't use "other level" very much, we just didn't include it, or its testing.
Because the free-text "Other Level" field is required in ArchiveSpace when the level "Other Level" is selected, I would have expected that, when my plugin tried to save the archival object, the ArchivesSpace validation would catch it, and reject that line.
I've proven to my satisfaction that this doesn't happen (I'll probably file an issue, or otherwise let them know that this is a problem for Aspace).
I am going to have to modify the template to allow for "Other Level" to be selected, and add an "other level" column (defaulting to "unspecified").  When I've made the changes, tested them, and modified the User Documentation, I'll announce it to the aspace users list, as well as let you know.
Hopefully, this won't take more than a day or two (she wrote optimistically).

Thanks for looking into this. Sorry about being unclear but you seem to have understood what I was asking. I will be looking for an update whenever you have time to work on it.
Thanks,
Kate Herbert
From: Bobbi Fox at Harvard LTS<EMAIL_ADDRESS>Sent: Thursday, December 14, 2017 10:18 AM
To: harvard-library/aspace-import-excel<EMAIL_ADDRESS>Cc: Herbert, Kate F<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [harvard-library/aspace-import-excel] Other Level of Description (#16)
OK, I've taken a look; it wasn't clear from your email exactly what you were doing to get the results you were getting. Let me make sure we are on the same page:

Using the template spreadsheet, you've added "Other Level" to the Data Validation List for the "Description Level" column
You then are successful in specifying archival objects with a "Description Level" of "Other Level"
Once you get those objects loaded, they are all missing the free text "Other Level" field that, were you to create the archival object manually, is a required field.

Assuming this is correct (and I tried it myself):
Because the initial version of this plugin was to satisfy the needs of Harvard's archivists, and they don't use "other level" very much, we just didn't include it, or its testing.
Because the free-text "Other Level" field is required in ArchiveSpace when the level "Other Level" is selected, I would have expected that, when my plugin tried to save the archival object, the ArchivesSpace validation would catch it, and reject that line.
I've proven to my satisfaction that this doesn't happen (I'll probably file an issue, or otherwise let them know that this is a problem for Aspace).
I am going to have to modify the template to allow for "Other Level" to be selected, and add an "other level" column (defaulting to "unspecified"). When I've made the changes, tested them, and modified the User Documentation, I'll announce it to the aspace users list, as well as let you know.
Hopefully, this won't take more than a day or two (she wrote optimistically).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/harvard-library/aspace-import-excel/issues/16#issuecomment-351740305, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdeP0EmTUeNF-7hBM9tFRGha8e-9n0ntks5tATwwgaJpZM4Q9950.

I have made the changes, and released a new version v1.7.1


A little more sane path resolution for TRS

working on https://github.com/dockstore/dockstore/issues/3335
There are two routes for this PR:

Demonstrates backwards-compatible behaviour, behaviour which seems odd. Basically one test relies upon this trying a passed path as both a relative and an absolute path.
Look at https://github.com/dockstore/dockstore/pull/3414/commits/4bc22a2978d5c0b333b8530c3a3b5154233c1ba0 to see what that would look like
For example: even if the main descriptor is /cwls/cgpmap-bamOut.cwl, we allow cgpmap-bamOut.cwl (relative to the main), /cwls/cgpmap-bamOut.cwl (absolute) and /cgpmap-bamOut.cwl (no idea why)
This last case seems to stem from https://github.com/dockstore/dockstore/pull/1441
I'm not entirely sure what might rely upon that extra slash although https://github.com/dockstore/dockstore/issues/1422#issuecomment-392944165 seems to indicate that we had doubled slashes being passed around at one point

The final state, breaks backwards compatible behaviour. We rely upon the user properly specifying a leading slash to indicate an absolute path rather than a relative path (rather than trying both). This seems more sane, but might break something.

Thoughts?

General thoughts, I think we should see how many entries we'd break by changing this. I've seen workflows use all 3 path schemes (it confused me initially too) so a bit worried about that.

General thoughts, I think we should see how many entries we'd break by changing this. I've seen workflows use all 3 path schemes (it confused me initially too) so a bit worried about that.

Having had a tea, I think this isn't possible on our end. The problem is that, potentially, users of the API could send an absolute path and have us interpret it as a relative path by us ignoring the leading slash (e.g. interpret /cgpmap-bamOut.cwl as cgpmap-bamOut.cwl in the working directory).
I'm not actually sure that anyone actually did and I don't think we can tell.

I'm not 100% grokking what's going on, but the input for #3335 is determined by what the files endpoint returns. I think so, but to confirm, that use case will work, right?

Yes, all file paths generated by us in the files endpoint should be relative to the main descriptor and should work in this endpoint. I don't believe that this should change the situation, any calls that are using a leading slash incorrectly "should" be outside our code.


PR: Do not clean WebEngine cache when using WebKit

Fixes #114

+1 from me. Please merge and release 0.2.3 afterwards.


Minimum Insertion Steps to Make a String Palindrome

Given a string s. In one step you can insert any character at any index of the string.
Return the minimum number of steps to make s palindrome.
A Palindrome String is one that reads the same backward as well as forward.
Example 1:
Input: s = "zzazz"
Output: 0
Explanation: The string "zzazz" is already palindrome we do not need any insertions.
Example 2:
Input: s = "mbadm"
Output: 2
Explanation: String can be "mbdadbm" or "mdbabdm".
Example 3:
Input: s = "leetcode"
Output: 5
Explanation: Inserting 5 characters the string becomes "leetcodocteel".

/assign

Hi there!
I am the Mentor of this project.
Although this repo is not part of Hacktoberfest 2024, we have some great alternatives for you. Welcome to SkillShow, a repository focused on skill enhancement!
Guidelines:

Please adhere to the existing guidelines.
No additional rules, thanks!
Check it out here: SkillShow.

Note: Start by creating an issue. Once it’s assigned to you, feel free to open a pull request (PR). Directly opening a PR will lead to an invalid label.
Looking forward to your contributions!


Add Dmitrii Rozhkov to the owners.json

As a Tech Lead of the Frontend Team I'd like to be involved and help out especially with release trains.

@mesosphere-mergebot label ready for review

@mesosphere-mergebot bump-ee

@orlandohohmeier I think we gonna need overriding those Arm tests again 🙇

@nLight No, we don't – they're not required.


Create plastic.desktop

Adding a default Desktop file which downstream can use. The Debian Package uses this already. And the Arch Mantainer wants a default in the repo first before using it. (The Desktop file for Arch will need no modifications since i made the deb exactly like the Arch one)

Replaced with #40 for more managed debian release


How to run an swift app in background?

I want some information stored in the database server  if  disconnected the Internet stored database sqlite when  Internet connection to send data to the server (even when the app is closed) What is your suggestion for the job

I'd suggest you take your question to Stack Overflow. We use our GitHub project for bug reports and feature requests.
Also, we are not in the business of writing people's apps for them. Nor suggesting how they should build them. Please do some research on your own before asking questions like this.

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Visualising the colormap - Mona Lisa

Just seen the Scipy talk, and loved the graphic of Mona Lisa.  For me that gives a much better way to view the colourmap than the sample images given as we have a well known image that we can use for comparison.  At present we just see abstract shapes which (I think) our brains find more difficult to interpret and thus form a meaningful opinion on the colourmap.
So I would suggest adding the picture of the Mona Lisa (or similar) to the visualising tools output, perhaps also for the colour-blindness test as well than I think would really help us.  The brain afaik has a very good detection algorithm for faces, which makes it very easy for us to tell a good colormap from a bad one, and thus give a better opinion.

I don't have a strong opinion either way really, because ideally to assess
a colormap you want to look at a few dozen images. We kept it down to three
to make it manageable for sticking up on the webpage and fitting in a
single window, but it's always going to be a pretty extreme compromise.
.
I will say that the reason we didn't put a face or a photo in in the first
place is exactly that people are so good at reading images :-). It's a
pretty different situation where you're trying to interpret some abstract
data surface versus trying to interpret a full 3d image with shading and
shadows etc. -- our extreme skill at the latter makes it a great way to
cheat when trying to get a general idea across quickly during a talk, but
I'm not particularly convinced that good colormaps for photos (which is
basically, any colormap that has a good lightness ramp) are necessarily the
same as good colormaps for visualizing arbitrary scalar fields.
On Aug 22, 2015 4:54 AM, "OceanWolf"<EMAIL_ADDRESS>wrote:

Just seen the Scipy talk, and loved the graphic of Mona Lisa. For me that
gives a much better way to view the colourmap than the sample images given
as we have a well known image that we can use for comparison. At present we
just see abstract shapes which (I think) our brains find more difficult to
interpret and thus form a meaningful opinion on the colourmap.
So I would suggest adding the picture of the Mona Lisa (or similar) to the
visualising tools output, perhaps also for the colour-blindness test as
well than I think would really help us. The brain afaik has a very good
detection algorithm for faces, which makes it very easy for us to tell a
good colormap from a bad one, and thus give a better opinion.
—
Reply to this email directly or view it on GitHub
https://github.com/matplotlib/viscm/issues/5.

Ahh, good point about the lightness, I guess we look at that more when looking at real images... I have no idea about the different factors in making a good colormap for analysing scalar fields, though obviously lightness helps when it comes to printing in black and white...
For me the biggest problem for me with abstract shapes comes from their abstractness, that we have no base to compare it to, for example when looking at the galaxy formation video we saw more detail in some than in others despite all having "perceptual uniformity", and without a base to compare to, we cannot make a judgement on which features actually exist, and which ones result due to the colormap (like jet does frequently).

To try to see what the "real" data looks like, IBM researchers in the past
compared the image in the colormap of interest with the image in grayscale:
http://www.research.ibm.com/people/l/lloydt/color/color.HTM
On Wed, Sep 2, 2015 at 6:16 PM, OceanWolf<EMAIL_ADDRESS>wrote:

Ahh, good point about the lightness, I guess we look at that more when
looking at real images... I have no idea about the different factors in
making a good colormap for analysing scalar fields, though obviously
lightness helps when it comes to printing in black and white...
For me the biggest problem for me with abstract shapes comes from their
abstractness, that we have no base to compare it to, for example when
looking at the galaxy formation video we saw more detail in some than in
others despite all having "perceptual uniformity", and without a base to
compare to, we cannot make a judgement on which features actually exist,
and which ones result due to the colormap (like jet does frequently).
—
Reply to this email directly or view it on GitHub
https://github.com/matplotlib/viscm/issues/5#issuecomment-137271247.

--
Kristen M. Thyng
Assistant Research Scientist
Department of Oceanography
Texas A&M University
Eller O&M 618
http://kristenthyng.com

We now have Kovesi's colormap test image, which is IMO better than any of the alternatives discussed here, so I think we can close this.


refactor(jobs): migrate components to TypeScript

And why this is labeled with de?

why this is labeled with de?

https://github.com/quisquous/cactbot/blob/main/.github/scripts/auto-label.cjs#L156-L170

I'll fix that

@quisquous  I'm not familiar with other jobs, maybe I'll just migrate these 2 jobs first?

@quisquous I'm not familiar with other jobs, maybe I'll just migrate these 2 jobs first?

Two is more than enough to start!


Replace deprecated huggingface_hub.Repository by HfApi

Description

Copy of #462
Motivation and Context

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

[ ] 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 change)
[ ] Documentation (update in the documentation)

Checklist:

[ ] I've read the CONTRIBUTION guide (required)
[ ] I have updated the changelog accordingly (required).
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.
[ ] I have reformatted the code using make format (required)
[ ] I have checked the codestyle using make check-codestyle and make lint (required)
[ ] I have ensured make pytest and make type both pass. (required)

Note: we are using a maximum length of 127 characters per line

For the failure on CI: https://github.com/Stable-Baselines-Team/stable-baselines3-contrib/pull/243/commits/03db09ef40acaeb08a7a453023e872c2b8f4906e
and for ValueError: loaded state dict contains a parameter group that doesn't match the size of optimizer's group
I hope it is not https://github.com/DLR-RM/stable-baselines3/pull/1975