Synchronously notify SceneTree before node is added

This PR is a proof-of-concept implementation for godotengine/godot-proposals#8850. Its purpose is to allow for dependency injecting tree nodes.
Testing steps

Set up a standard Godot project and set up an autoload for a inject.gd file with the following contents:

class_name Injector extends Node

func _enter_tree() -> void:
	get_tree().node_adding.connect(_on_node_adding)

func _on_node_adding(node: Node) -> void:
	print("Adding node")

Create a main scene with the following attached script:

class_name Main extends Node2D

func _enter_tree() -> void:
	print("Enter tree")

func _ready() -> void:
	print("Ready")

Expected output:
Adding node
Enter tree
Ready

What is insufficient with the node_added signal? It's not mentioned here or in the proposal


Sorting Algorithm

As discussed, we want to revise the sorting algorithm calc_sorted_vertices so I'm collecting the issues with the current one:

Sometimes it forgets to return the region for last generator. I could reproduce this error with the following points:

   points = np.array([[-0.78928481, -0.16341094, 0.59188373],
       [-0.66839141, 0.73309634, 0.12578818],
       [0.32535778, -0.92476944, -0.19734181],
       [-0.90177102, -0.03785291, -0.43055335],
       [0.71781344, 0.68428936, 0.12842096],
       [-0.96064876, 0.23492353, -0.14820556],
       [0.73181537, -0.22025898, -0.6449281],
       [0.79979205, 0.54555747, 0.25039913]])

Sometimes the sorting algorithm crashes. I could reproduce this error with the following points:

    points = np.array([[0, 0, 1],  [0, 0, -1], [1, 0, 0], [0, 1, 0], [0, -1, 0], [-1, 0, 0],])

The datastructure returned is a dictionary. Since this is uncommon in scipy we want to be more compatible (as close as possible to scipy.spatial.Voronoi).

At the moment the code is a bit difficult to read and not fully compatible with PEP8.

Sounds good--I see you recently made some adjustments and added a test.


Add Apache v2.0 License To Repo

Fixes #3
This PR adds the Apache v2.0 License to the repo, as well as provides a link to it from the README.md file.

@crweiner Thank you so much for your contribution - sorry this took so long to merge!


How to integrate with angular material 2?

This is my problem, i have a proyect with angular material 2, and i want use some component of covelant, there any way these two can coexist in same proyect?
because it tries to integrate them and do not get it

If you are using angular cli, take a look here https://github.com/Teradata/covalent/blob/develop/docs/GETTING_STARTED.md

How to use covalent with the latest version of CLI?
My Project
@angular/cli: 1.2.3
node: 8.1.2
os: win32 x64
@angular/animations: 4.3.2
@angular/common: 4.3.2
@angular/compiler: 4.3.2
@angular/core: 4.3.2
@angular/forms: 4.3.2
@angular/http: 4.3.2
@angular/platform-browser: 4.3.2
@angular/platform-browser-dynamic: 4.3.2
@angular/router: 4.3.2
@angular/cli: 1.2.3
@angular/compiler-cli: 4.3.2
@angular/language-service: 4.3.2

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

<EMAIL_ADDRESS>added 1 package and removed 882 packages in 65.165s

How to proceed ?

There might be some problems. I had the same thought for my project. Everything was build by using @angular/material and on some point I needed a stepper and discovered covalent.
Integrating all needed module and styles was no problem, but it seems to break the style of some components created before integrating covalent.
There are many small changes like setting another background-color to the sidenav content container, changing icon sizes, small displaces on some inputs. Haven't tracked down yet whether it's because some styles of @angular/flex-layout are overwritten or if it is something other.

The styles clashing are now optional, so you can use covalent without the platform.scss or covalent mixins


How to Display All Cases about Business Error?

our team define the response like this:
{
    code:123,
    message: "success",
    data:[
        {
            id:xxx,
            xxx:xxx,
            ...
        },
        ...
    ]
}

We use 'code' field to represent  various business error. So I want to display all cases in swagger-ui. Specially when the code is the correct one, I want it to show the structure of data. What should I do for this?

So I want to display all cases in swagger-ui. Specially when the code is the correct one, I want it to show the structure of data.

Your question isn't entirely clear to me. Could you explain with an example perhaps?


Adding new development environment

To facilitate local development this PR makes restarting bokeh server and qlf pipeline more straight forward.
For example if you want to develop on bokeh:

Change or remove BOKEH_TEST env variable on backend/docker-compose.yml
Start docker backend with ./start.sh backend
Start bokeh server after the backend container finishes starting with ./start.sh bokeh

You can now stop bokeh server with CTRL+C and start with ./start.sh bokeh
For qlf daemon (responsive to run ql pipeline):

Change or remove DAEMON_TEST env variable on backend/docker-compose.yml
Start docker backend with ./start.sh backend
Start qlf daemon server after the backend container finishes starting with ./start.sh daemon

You can now stop the daemon with CTRL+C and start with ./start.sh daemon

[x] Add tutorial on wiki

https://github.com/desihub/qlf/wiki/Developing-with-QLF-Daemon-(Docker)
https://github.com/desihub/qlf/wiki/Developing-with-Bokeh-(Docker)

Can you comment what is hard right now?

@ricardogando To see a change in our daemon you have to either restart the docker container that can about 10~30 seconds each restart or enter the container, stop the daemon process and start it from there. This can be tricky sometimes if the process is grouped inside the main process (run.sh).


Where is the "Write the ID of the current Active Device to config" button?

I'm connecting Spotify to Bitfocus Companion using the following instructions:

Help for spotify-remote v1.0.11
×
A module for controlling spotify via the Web API.
Setup
Go to here and create an application.
Copy the Client ID and Client Secret and add them to the config boxes.
Set https://spotauth.github.io/ as your Redirect URL both in the Config and on the spotify application you created.
When you save the modules config a URL will be placed on in the auth URL config box, make sure to refresh to load it, go to this URL and then copy the Approval Code into the config.
This time when you save the config an Access Token and Refresh Token will be wrote to the config.
Now go assign the "Write the ID of the current Active Device to config" button, start playing music on the device you wish to control and then press the button you just assigned.
The module is now fully configured and should work without issue.
If you have an issues with the module please open an issue on the module's GitHub repo or message Peter Stather on the official Slack.

I have successfully completed all steps up to the "Write the ID of the current Active Device to config"-- I can't find the button!
The connection works for a while, but when i come back the next day, it no longer works (I assume because my 'Device ID' field is empty. I've searched everywhere, can someone please help me find the 'write the ID' button?

Its not a button, but an action that you can place and use

Ah, thank you! I never would have thought to look there.
Is it necessary to include the button action permanently in my button macros, or can i activate it once and then remove it once the device ID has been recorded?

@Daruuk If you are planning to use the same device for a long period of time then you can remove the button. I have only had to use it the first time I setup the integration, over 18 months ago.


Deprecate ERAP app/skid

Summary

Application Name: ERAP
Application Urls: https://experience.arcgis.com/experience/285c8572fddd41a7b2221e731bf23ace

Staging: ut-dts-agrc-erap-dev
Production: ut-dts-agrc-erap-prod

The ERAP/Rent Relief program has ended and DWS does not need this map online anymore.
Action items

Assign a person who should complete the task by replacing name with their github @name.
Check [x] the box when the task is completed and add the date of completion.
~Strike~ out all items that do not apply.

~[ ] Update relevant gis.utah.gov application pages (name, completed: 2023/00/00)~
~[ ] Remove application-specific Databases (name, completed: 2023/00/00)~
~[ ] AGRC Projects team drive folder (name, completed: 2023/00/00)~
[x] Remove from Application Portfolio Management ("Business Applications" in ServiceNow) (@nathankota , completed: 2023/07/10)

Is there a website?

~[ ] Remove DNS entry (name, completed: 2023/00/00)~
[x] Archive source code repository (@jacobdadams , completed: 2023/07/12)
~[ ] Remove app-specific quad word from Discover system (name, completed: 2023/00/00)~
[x] Remove cloud project (@steveoh, completed: 2023/07/12)
[x] Remove terraform infrastructure (@jacobdadams , completed: 2023/07/12)

Choose one.

~[ ] Remove from the web server (name, completed: 2023/00/00)~
~[ ] Replace app with a static page with information (name, completed: 2023/00/00)~
~[ ] Redirect somewhere else (name, completed: 2023/00/00)~
[x] Deprecate and unshare web app and map (@jacobdadams , completed: 2023/07/12)

Is there a map service?

~[ ] Stop (name, completed: 2023/00/00)~
~[ ] Delete (name, completed: 2023/00/00)~
~[ ] Unregister Data Store in ArcGIS Server (name, completed: 2023/00/00)~
[x] Unshare and deprecate feature layer in AGOL (@jacobdadams, completed: 2023/07/12)

SFTP

[x] Contact DWS to stop automated csv dump to our SFTP (emailed Jennifer Edwards and Mike Murdock from DWS about who on their team can help, since Rick Kowalczik's email is no longer active, and he helped set it up, @jacobdadams or @nathankota , completed: 2023/07/10)
[x] Contact DTS to remove dws-erap share (share and user removed via INC1856498, @nathankota, completed: 2023/07/11)

Group Task Assignments

Check [x] the box when you have assigned all the tasks relevant to your group.

~[ ] Data Team (@gregbunce)~
[x] Dev Team (@steveoh)
~[ ] Cadastre Team (@rkelson)~

conductor results for tasks - 253

check
status

@nathankota has completed 0 out of 3 tasks
:no_entry:

@jacobdadams has completed 0 out of 5 tasks
:no_entry:

steveoh has completed 1 out of 1 tasks
:+1:

Looks like everything's done now, closing this issue.


server_names_hash issue

Running the ecs-nginx-proxy without any special config, I'm getting the following error as soon as I introduce a service:
2017/03/06 03:44:09 found cluster name to be: test-reverse-proxy
2017/03/06 03:44:09 updating config
2017/03/06 03:44:10 container is own container. skipping
2017/03/06 03:44:10 running signal command
2017/03/06 03:44:10 ===== output start =====
2017/03/06 03:44:10 2017/03/06 03:44:10 [emerg] 14#14: could not build server_names_hash, you should increase server_names_hash_bucket_size: 64
nginx: [emerg] could not build server_names_hash, you should increase server_names_hash_bucket_size: 64

2017/03/06 03:44:10 ===== output end =====
2017/03/06 03:44:10 failed to run signal command
2017/03/06 03:44:10 error:  exit status 1
2017/03/06 03:44:10 []

Any idea what could be wrong?
The task definition for the container is:
{
  "requiresAttributes": [
    {
      "value": null,
      "name": "com.amazonaws.ecs.capability.logging-driver.json-file",
      "targetId": null,
      "targetType": null
    },
    {
      "value": null,
      "name": "com.amazonaws.ecs.capability.docker-remote-api.1.19",
      "targetId": null,
      "targetType": null
    }
  ],
  "taskDefinitionArn": "arn:aws:ecs:ap-southeast-2:1234567890:task-definition/hello-world:1",
  "networkMode": "bridge",
  "status": "ACTIVE",
  "revision": 1,
  "taskRoleArn": null,
  "containerDefinitions": [
    {
      "volumesFrom": [],
      "memory": 128,
      "extraHosts": null,
      "dnsServers": null,
      "disableNetworking": null,
      "dnsSearchDomains": null,
      "portMappings": [
        {
          "hostPort": 81,
          "containerPort": 80,
          "protocol": "tcp"
        }
      ],
      "hostname": null,
      "essential": true,
      "entryPoint": null,
      "mountPoints": [],
      "name": "hello-world",
      "ulimits": null,
      "dockerSecurityOptions": null,
      "environment": [
        {
          "name": "VIRTUAL_HOST",
          "value": "helloworld.test-reverse-proxy.a.little.bit.longer"
        }
      ],
      "links": null,
      "workingDirectory": null,
      "readonlyRootFilesystem": null,
      "image": "tutum/hello-world",
      "command": null,
      "user": null,
      "dockerLabels": null,
      "logConfiguration": {
        "logDriver": "json-file",
        "options": {
          "max-size": "200m"
        }
      },
      "cpu": 10,
      "privileged": null,
      "memoryReservation": null
    }
  ],
  "placementConstraints": [],
  "volumes": [],
  "family": "hello-world"
}

http://charles.lescampeurs.org/2008/11/14/fix-nginx-increase-server_names_hash_bucket_size
Try adding that to the template.

@codesuki yes it seems the domain name is too long. Is there an easy way to do so without having to build my own ecs-nginx-proxy and ecs-gen image? Otherwise no worries I'll take care of that then do a PR

You could run with docker run -v src:dst to map some new template file and replace the CMD to use that.

hmmm it doesn't seem to work? Any idea?
[ec2-user@ip-10-13-81-40 ~]$ docker run 123456789.dkr.ecr.ap-southeast-2.amazonaws.com/ecs-nginx-proxy:latest cat nginx.tmpl
server_names_hash_bucket_size 64;
# If we receive X-Forwarded-Proto, pass it through; otherwise, pass along the
# scheme used to connect to this server
map $http_x_forwarded_proto $proxy_x_forwarded_proto {
  default $http_x_forwarded_proto;
  ''      $scheme;
}
# If we receive Upgrade, set Connection to "upgrade"; otherwise, delete any
# Connection header that may have been passed to this server
map $http_upgrade $proxy_connection {
  default upgrade;
  '' close;
}
gzip_types text/plain text/css application/javascript application/json application/x-javascript text/xml application/xml application/xml+rss text/javascript;
log_format vhost '$host $remote_addr - $remote_user [$time_local] '
                 '"$request" $status $body_bytes_sent '
                 '"$http_referer" "$http_user_agent"';
access_log off;
# HTTP 1.1 support
proxy_http_version 1.1;
proxy_buffering off;
proxy_set_header Host $http_host;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection $proxy_connection;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header X-Forwarded-Proto $proxy_x_forwarded_proto;
# Mitigate httpoxy attack (see https://github.com/jwilder/nginx-proxy for details)
proxy_set_header Proxy "";
server {
        server_name _; # This is just an invalid value which will never trigger on a real hostname.
        listen 80;
        access_log /var/log/nginx/access.log vhost;
        location = / {
                return 200 'nginx is alive';
                add_header Content-Type text/plain;
        }
        location / {
                return 503;
        }
}

{{ range $domain, $container := . }}
upstream {{ $domain }} {
    {{ range $_, $value := $container }}
         server {{ $value.Address }}:{{ $value.Port }};
    {{ end }}
}
server {
        server_name {{ $domain }};
        listen 80;
        access_log /var/log/nginx/access.log vhost;
        location / {
                proxy_pass http://{{ $domain }};
        }
}
{{ end }}

nevermind I had to increase it to 128. Should we put this as a default? I'm happy to do a PR
It'd be also good to do a deeper PR and not make the proxy crash is someone pushes a VIRTUAL_HOST that's too long for the proxy. That's a big risk in my opinion?

FYI nginx-proxy has the fix already https://github.com/jwilder/nginx-proxy/blob/6554a1cf287f79a23bff84672601422caca8cb10/Dockerfile#L14

Thanks for the report and the PR!

You're welcome. Still unsure about how your proxy would break if someone tried to enter a very very long name, but I guess that's unlikely at the 128 limit

Well if somebody needs support for that they could fix it by replacing the template which is something that we could leave to the user. You set a good default I think.


ML Projects : Heart Disease Prediction

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #7864

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Machine Learning

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@hrugved06 do the needful changes please.

I did the required changes. Please have a look once :-D


Update CMIP6 cmor tables from PCMDI at DReq 01.00.30

Update CMIP6 cmor tables to Data Request v01.00.30
using JSON files from PCMDI (https://github.com/PCMDI/cmip6-cmor-tables)

Why are the tests failing?

Why are the tests failing?

There was an error in the header of CMIP6_Efx.json.

Now tests fails apparently due to some issue with conda packages ...

@mattiarighi Any idea about the error resulting from the tests?

I'm not an expert, but it could be that some changes are needed here.
@bouweandela or @jvegasbsc can give you a better hint.

I am checking...

Test are working locally.

There was an error in the header of CMIP6_Efx.json.

Have you reported it to the tables project?

Test are working locally.

There was an error in the header of CMIP6_Efx.json.

Have you reported it to the tables project?

Changes are in commit 98d589c and local test were fine.
If it may help, also PR #942 (that was ok beside codacy) now fails the test python3_install

Changes are in commit 98d589c and local test were fine.
Yes, I know. I just asked if you have reported that as an issue also in
https://github.com/PCMDI/cmip6-cmor-tables/issues because I can not find it. If you don't want to add it, I will do it because it is a more general error that should be addressed there

@jvegasbsc I was writing down the issue !  Here is the reference https://github.com/PCMDI/cmip6-cmor-tables/issues/240

Now tests are fine, this was probably fixed after #965.

Good to go (assuming that issues with CircleCI test have nothing to do with this)

Now tests are all fine, so I would also say it is good to go !


fix storybook and package-lock brittleness by removing shoelace dependency

This PR does the following:

remove shoelace from dependencies and vendor the files needed
update ModalContainer to import these without doing so onMount
rebuild package-lock from scratch

After this PR, both storybook and npm run dev work for me locally.
Additionally this fixes #2673, since it rebuilds package-lock.json from scratch with a fresh npm install, which should allow us to check off that subtask in #1916. (cc @begelundmuller and @kaspersjo to make that call)

Note: latest commit also adds a @ts-ignore to an unrelated file to suppress an a svelte-check error. My guess is that perhaps the version of svelte-check changed and made the checks more stringent or something like that. I hate adding an ignore, but I think it's worth it in this case -- this PR brings the repo to an overall more reproducible and less brittle state, and the code in question was working fine before  this ts check started failing.

Wow, I'm now getting a really unbelievable set of @typescript-eslint errors: https://github.com/rilldata/rill/actions/runs/5360080684/jobs/9724501187?pr=2686
It looks like it's checking minified and/or build files and somehow losing it's mind all the error are in /home/runner/work/rill/rill/web-local/build/_app/immutable/chunks/*, which I'm pretty sure should not be targeted with prettier/eslint/typescript checks.
I think the next step would be to exclude that folder from the failing CI checks.

The reason you are getting eslint errors now is because something you changed is causing the build to produce .js files instead of .mjs files like they were before. eslint is picking up those .js files.
Regardless of final solution, we probably need a .eslintignore file that explicitly ignores build directories.
But also, would be good to understand what you changed that produced .js files instead of .mjs files for the build, and whether that breaks anything downstream for us

@skokenes I think we're good to go on this now, should be ready for final review and merge


Added support for MCU

I noticed that there is only one bare minimum example for STM32. It sends out scan command and then prints out response as raw data without parsing.
https://wiki.slamtec.com/pages/viewpage.action?pageId=13959187
In order to use RPLidar on MCUs, I ported the SDK to RT-Thread which works on STM32, RISC-V, i.mx RT, virtually all mainstream MCUs.
https://github.com/wuhanstudio/rplidar
I hope this will make RTLidar working with MCUs more feasible.

Han Wu, Master Student
School of Energy and Power Engineering
Huazhong University of Science and Technology
Email<EMAIL_ADDRESS>Website: http://wuhanstudio.cc

This works for both RP LIDAR A1 and A2? while interfacing with the MCU, the PWM input to LIDAR motor is to be carried out separately?

Hi, this should work for both A1 and A2 as they share the same protocol:
http://bucket.download.slamtec.com/f010c72be308cdc618e91746d643278185ed02b2/LR001_SLAMTEC_rplidar_protocol_v2.2_en.pdf
But this package is only tested on A1, because I do not currently have a A2 for testing.
The motor can be controlled using the serial protocol. thus only UART is required for the connection.

I had gone through the code, i believe you have written it for RT-thread RTOS ? I am aiming to use the lidar driver program for a project in freeRTOS on a stm32 hardware. I think i need to port  the rplidar.c file to freeRTOS, right?

The serial driver that requires porting is here:
https://github.com/wuhanstudio/rplidar/blob/master/drv/drv_rplidar.c
And the serial protocol is implemented here:
https://github.com/wuhanstudio/rplidar/blob/master/src/rplidar.c


Fix missing response schema bug

Fixes #1315

Do we need to make the same fix for empty Consumes array?


ENTITY_IS_DELETED error on Informatica integration

From https://powerofus.force.com/_ui/core/userprofile/UserProfilePage?u=0058000000BMwyH&tab=sfdc.ProfilePlatformFeed&fId=0D580000032FgmD.
After version 1.19 was pushed to production on 27/10/16, user reported getting the following error from an existing Informatica integration that was previously working correctly:
Error Type: System.DmlException
Error Date: 2016-10-27 11:50:48
Message: "Delete failed. First exception on row 0 with id a2R4E0000001L5UUAU; first error: ENTITY_IS_DELETED, entity is deleted: []"
Stack Trace:
Stacktrace available at the time of saving the record - Class.hed.TDTM_Config.setTdtmConfig: line 82, column 1
Class.hed.TDTM_Config.getClassesToCallForObject: line 124, column 1
Class.hed.TDTM_TriggerHandler.run: line 75, column 1
Class.hed.TDTM_Global_API.run: line 60, column 1
Trigger.hed.TDTM_Account: line 33, column 1

**lurch: add

Tracking W-011737

I was able to reproduce the issue with a script like this:
require 'restforce'

def api_client
  @api_client ||= Restforce.new api_version: '32.0',
                                refresh_token: 'xxxx',
                                client_id: 'xxxx',
                                client_secret: 'xxxx'
end

def create_account_via_api
  @account_id = @api_client.create('Account', Name: 'foo')
end

api_client

t1 = Thread.new do
	create_account_via_api
end
create_account_via_api
t1.join

Included in beta release 1.20 (Beta 1)

Included in production release rel/1.20

@njjc ~ See error message from client with full stack trace.


Question: Where the hell is Godot 4.3? It doesn't even come up in Google.

This addon can only be used with a Godot version from the future.
Ok but seriously: This addon requires a pull request which has not been merged yet, but will be merged for 4.3. I posted a link to this pull request in the README file.

The pull request I mentioned has been merged, so this repo can be used with the latest master branch of Godot.


allow adding directory

Codecov Report

Merging #44 (51cfc21) into master (b5621b2) will decrease coverage by 7.33%.
The diff coverage is 31.72%.

@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
- Coverage   52.96%   45.62%   -7.34%
==========================================
  Files           8       10       +2
  Lines        1316     1977     +661
==========================================
+ Hits          697      902     +205
- Misses        502      929     +427
- Partials      117      146      +29

Flag
Coverage Δ

unittests
45.62% <31.72%> (-7.34%)
:arrow_down:

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

Impacted Files
Coverage Δ

internal/ipfs/adder.go
0.00% <0.00%> (ø)

internal/ipfs/util.go
59.61% <ø> (ø)

mobile/discovery.go
30.14% <16.07%> (-0.89%)
:arrow_down:

internal/replication/replication.go
33.14% <22.89%> (-17.27%)
:arrow_down:

mobile/datahop.go
45.43% <42.20%> (-17.38%)
:arrow_down:

internal/ipfs/ipfs.go
56.60% <48.35%> (ø)

internal/security/security.go
64.28% <64.28%> (ø)

internal/config/config.go
75.00% <100.00%> (ø)

internal/matrix/matrix.go
72.90% <100.00%> (+47.06%)
:arrow_up:

internal/repo/repo.go
55.88% <100.00%> (-0.77%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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


calibration data seq_length

Hi, I have a question about calibration data (128, 2048 tokens, respectively)
Is there a particular reason to use 2048 tokens for each data?
I tracked SparseGPT, GPTQ, but I couldn't find it.
I hope I can get some insight.
Thank you.

2048 is the maximum context size of the OPT and LLaMA models. In other words, these LLMs can process a sequence with up to 2048 tokens.


[Inspector] Need to rewrite how Component Drawing works

In Unity, the inspector uses a SerializedObject version of the object it's inspecting
Changes are then saved back into the object
Our serializer, the TagSerializer outputs a tag that isn't entirely far off from what the SerializeObject and Properties are in unity and could be reused for a similar purpose.
Let's try and recreate the Unity PropertyDrawer API and custom editor API, we won't be able to get the EditorGUI class to be 1:1 due to ImGUI, but we could at least try and get a simple Unity Editor code to be able to run in prowl.
Ideally, Tags stay simple as they are for Serializing everything not just editor stuff
so some hurdles I can think of:
1. Detecting a change in Tags for Undo/Redo
2. Field Attributes need to be stored somewhere, ideally not inside the tags

Im gonna postpone this one for the foreseeable future.
I was working on this for a little while and while it does have some pros, I'm not sure the pro's are worth the effort, at least not now.
The Serializer needs a way to Populate existing objects with a serializedProperty, at first I thought Serialized.DeserializeInto() would do the job, but after actually doing it I quickly realized DeserializeInto is only half of the deserialization process, and lacks some needed features, which to implement would require re-writing deserialization.
So for the time being at least, I'm gonna close this, What we have now works just as well, its just not 1:1 with how unity does it.


tflite runtime conflicts with tensorflow version

Description
Hello all, I am using USB Coral TPU. I want to update my code from libedgetpu to Pycoral. I am using tensorflow 2.6, pycoral version: 2.0 and tflite_runtime : 2.5.0.post. I have successfully installed the packages but when I run my object detection code and I also want to use tensorflow in parallel at that time I am getting following runtime error. Any help is appreciated!
import tensorflow as tf
from ._api.v2 import compat
from . import v1
from tensorflow._api.v2.compat.v1 import lite
from . import experimental
from tensorflow.lite.python.lite import OpResolverType
from tensorflow.lite.python.interpreter import Interpreter  # pylint: disable=unused-import
from tensorflow.lite.python.interpreter_wrapper import _pywrap_tensorflow_interpreter_wrapper as _interpreter_wrapper
ImportError: generic_type: type "InterpreterWrapper" is already registered!

Click to expand!
Issue Type
Build/Install
Operating System
Ubuntu
Coral Device
USB Accelerator
Other Devices
No response
Programming Language
Python 3.6
Relevant Log Output
No response

@corvitacvision can you please check this https://github.com/google-coral/pycoral/issues/57#issuecomment-949997068 issue to fix it.

@hjonnala Thank you for your reply. I have solved the issue by downgrading tensorflow version from 2.6 to 2.4 and now it runs without any runtime error.


Only fix npm link of AVA as necessary.

PR #815 introduced a fix for npm link, but it broke NYC coverage.
Instead of always applying the "fix", this wraps it in a conditional, such that the old behavior is returned unless we detect a problematic npm link setup (npm link ava on node 6.0.0-6.?.?).

I don't think you meant to commit ava-0.14.0.tgz.

tarball removed.

👍


:ghost: Add questionnare required field to assessment (#588)

This is needed to aid UI in computing status for an assessment given a questionnaire is marked as not required. If an assessment exists for an archived questionnaire on an app or archetype, the assessment status is no longer valid & the assessment will need to be filtered from status consideration.
https://issues.redhat.com/browse/MTA-1956

release-0.3 has been rebased on main so this PR can be closed.


[BDD] Make the UI pretty

If we have time in this sprint, we can enhance the UI with collapsible accordion in place of the results table (there is a commented out example in the code - looks like this). This will also require altering BDD tests to fit the new markup of the page. The products results table can stay the same.

Hey team! Please add your planning poker estimate with ZenHub @Fivetoes @jiuliwei @XinyuC97 @yx2512 @ZX-WANG


How to generate demo site ?

Hi,
I didn't succeed to generate demo from src. Could you help me here ?
Cheers,
Xavier

Hi,
I think it is like... you clone the repo on you machine then run (do you saw package.json):
npm install
npm run build
npm run start

Hi,
thanks, I modified it locally to be able to build with webpack (starting a new project using @vue/cli)


fix cypress > 8.2 error where cypress tries reassing error name

Motivation
Starting from Cypress 8.3 when a test fails Cypress outputs the TypeError: Cannot set property name of [object Object] which has only a getter it happens because Cypress is trying to reassign the error name https://github.com/cypress-io/cypress/blob/853a6f5a332433971e451290b3aa961139d40793/packages/driver/src/cypress/command_queue.ts#L348 and we use custom errors with the read-only name property for interactors.
More details here
Approach
I made the PR for Cypress https://github.com/cypress-io/cypress/pull/18259 and use the plain property for our custom error instead of getters

Can't merge it because of canceled CI checks.

Oh, Exactly, I forgot the changeset 🤦


build: bump go to v1.22.10

What this PR does
There are a number of bug fixes included since v1.22.8[1]
[1] https://go.dev/doc/devel/release#go1.22.0

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
/cc @dhiller @xpivarc

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

Build KubeVirt with go v1.22.10

/approve
@brianmcarey PTAL


fix bugs of wrong event filters and network back online

fix wrong event filters when register.
resubscribe when network is back online.

Coverage decreased (-0.06%) to 40.183% when pulling 28ebc66c0c95b4e296e82db38c7472db7c0e1c02 on embbnux:bugfix-subscription into 6b57182e71498b89f4495cf43d0085f7cb7b8467 on ringcentral:master.


Do not give me default inteli-sense suggestions when there are better matches

Trigger inteli-sense inside an object creation literal.
I would expect to get the fields of the expected argument as only suggestions, not the global suggestions.
Picture attached

Which editor is this? The TypeScript language service isn't reporting all those other things -- those look like snippets being provided by the editor's plugin.

This looks like VS Code which adds template suggestions now and then. Maybe we should re-label this work item to either: (1) suggestions should include templates or (2) suggestion should hint what template scope is appropriate.

(1) suggestions should include templates or

what locations are ok to include templates and what are not? looks like you are in a function body in the example above, class, if are still valid, get is not though..

(2) suggestion should hint what template scope is appropriate.

I think this is a better approach. can you elaborate of some possible values? may be:

global
class body
object literal
function body
switch body
expression

Sublime would also greatly benefit from this. @zhengbli @billti

Should this be closed and marked as external?

@DanielRosenwasser we will have the same issue in VS with #312, all editors need some syntactic context if they want to filter templates.

AFAIK all editors do (or at least can) handle this in the preferred way now


Can't login with GitHub to signup for Doclets

I'm trying to login and try Doclets. I get to the GitHub authorization screen but when I click authorize I'm redirected to https://doclets.io/auth/github/callback?code=%hexstring%
And get the following error:
TypeError: req.session.save is not a function
   at Routes.authGithubCallback (/usr/src/app/lib/routes.js:15:15)
   at Layer.handle [as handle_request] (/usr/src/app/node_modules/express/lib/router/layer.js:95:5)
   at next (/usr/src/app/node_modules/express/lib/router/route.js:131:13)
   at complete (/usr/src/app/node_modules/passport/lib/middleware/authenticate.js:250:13)
   at /usr/src/app/node_modules/passport/lib/middleware/authenticate.js:257:15
   at pass (/usr/src/app/node_modules/passport/lib/authenticator.js:421:14)
   at Authenticator.transformAuthInfo (/usr/src/app/node_modules/passport/lib/authenticator.js:443:5)
   at /usr/src/app/node_modules/passport/lib/middleware/authenticate.js:254:22
   at /usr/src/app/node_modules/passport/lib/http/request.js:60:7
   at pass (/usr/src/app/node_modules/passport/lib/authenticator.js:267:43)
   at serialized (/usr/src/app/node_modules/passport/lib/authenticator.js:276:7)
   at /usr/src/app/lib/routes.js:394:9
   at model.<anonymous> (/usr/src/app/node_modules/mongoose/lib/document.js:1842:20)
   at next_ (/usr/src/app/node_modules/mongoose/node_modules/hooks-fixed/hooks.js:89:34)
   at fnWrapper (/usr/src/app/node_modules/mongoose/node_modules/hooks-fixed/hooks.js:186:18)
   at /usr/src/app/node_modules/mongoose/lib/model.js:292:13

This also affects returning users who are trying to sign back in with github.

thanks for reporting! digging into this...

@jniles @kensnyder could you please re-try logging in/out to see if this still happens? thanks

... seems like some ressource/memory problem with session store....

@lipp, I am able to log in again.  Thanks!

i'll keep this issue open as I don't understand what was going wrong ...

I have the same problem.
I've tried log out & in , reauthorize from github, it's still not work, please help!

@uedsky signup/login is working again. i have to resize my server vm, ... it seems to be a memory issue. sorry for the inconvenience & trouble.

Another problem, I've pushed a repo named docletstest, but doclets cannot detect it, even after clicked the re-sync respositories btn

that's interesting.... many projects have been added during the last day...
i will look into the logs...

It appear now, maybe i'm just didn't see it. coz you guys use "reponame/username" form in repo list.

this was a misconfigured redis / docker host issue

Thank you, it is working for me as well.


page 13, email editor will not close when email message is html formated

Lock in page 13 Email Editor when using codeunit 8904 Email Message,
EmailMessage.Create('', '', '', true) where it is html formatted.
Email.OpenInEditor(EmailMessage, Enum::"Email Scenario"::Default);
Page 13:
If you type in receipient it disappers.
Two body sections appears
Page can't be closed.
Works if html formatted is false

Let's get the issue to a team who will fix and backport it, as this repository is only for issues related to the AL compiler in latest developer preview environment for Dynamics 365 Business Central. We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
To open a support case, you can:
Open a support request to CSS through the PartnerSource portal
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company
If you file the bug in Collaborate, remember to include steps to reproduce the issue, and the Business Central build number and country version you're using.


Uncaught Error: ENOENT: no such file or directory, open 'd:\Documents\2015\folder\

[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.0
System: Unknown Windows Version
Thrown From: atom-minify package, v0.1.6
Stack Trace
Uncaught Error: ENOENT: no such file or directory, open 'd:\Documents\2015\Renault_Kadjar2\dev\Desert\js\slider.min.js'
At fs.js:519

Error: ENOENT: no such file or directory, open 'd:\Documents\2015\Renault_Kadjar2\dev\Desert\js\slider.min.js'
  at Error (native)
  at Object.fs.openSync (fs.js:544:18)
  at Object.module.(anonymous function) [as openSync] (ATOM_SHELL_ASAR.js:118:20)
  at Object.fs.readFileSync (fs.js:396:15)
  at Object.fs.readFileSync (ATOM_SHELL_ASAR.js:369:29)
  at C:\Users\Ciro\.atom\packages\atom-minify\node_modules\node-minify\lib\node-minify.js:127:37
  at ChildProcess.exithandler (child_process.js:720:7)
  at emitTwo (events.js:87:13)
  at ChildProcess.emit (events.js:169:7)
  at maybeClose (child_process.js:997:16)
  at Socket.<anonymous> (child_process.js:1166:11)
  at emitOne (events.js:77:13)
  at Socket.emit (events.js:166:7)
  at Pipe.close (net.js:464:12)

Commands
     -2:02.9.0 editor:newline (atom-text-editor.editor.is-focused)
     -1:55.1.0 core:backspace (atom-text-editor.editor.is-focused)
     -1:51.3.0 core:move-right (atom-text-editor.editor.is-focused.autocomplete-active)
     -1:30.6.0 atom-terminal-panel:toggle-autocompletion (atom-text-editor.editor.is-focused)
     -1:30.5.0 core:save (atom-text-editor.editor.is-focused)
     -1:29.9.0 atom-terminal-panel:toggle-autocompletion (atom-text-editor.editor.is-focused)
     -1:29.4.0 command-palette:toggle (atom-text-editor.editor.is-focused)
  6x -1:26.7.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -1:21.1.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -1:17.8.0 core:move-up (atom-text-editor.editor.mini.is-focused)
 10x -1:15.5.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -1:14 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -1:14 core:cancel (atom-text-editor.editor.mini.is-focused)
     -1:00.0 atom-minify:minify-to-min-file (atom-text-editor.editor.is-focused)
     -0:39.6.0 atom-minify:js-minifier-uglifyjs2 (atom-workspace.workspace.scrollbars-visible-always.theme-seti-syntax.theme-seti-ui)
  2x -0:33 atom-minify:minify-to-min-file (atom-workspace.workspace.scrollbars-visible-always.theme-seti-syntax.theme-seti-ui)

Config
{
  "core": {
    "themes": [
      "seti-ui",
      "seti-syntax"
    ]
  },
  "atom-minify": {
    "jsMinifier": "UglifyJS2"
  }
}

Installed Packages
# User
atom-beautify, v0.28.5
atom-minify, v0.1.6
atom-terminal-panel, v4.4.4
autoclose-html, v0.18.0
haxe, v0.6.1
language-haxe, v0.3.0
linter, v1.0.2
minimap, v4.10.0
seti-syntax, v0.4.0
seti-ui, v0.7.1

# Dev
No dev packages

Will be fixed with next version, coming soon!

Also happens for CSS with clean-css

Please try new version 0.2. I fixed the bug there.

Working great now! Thank you very much.


operator: allow setting security context on the deployment

The operator's chart does not currently allow setting either the Pod nor container security contexts. While this can be worked around with a Kustomize post-render step, it would be ideal to support this directly in the chart itself.
JIRA Link: K8S-316

It should be possible to set security context in operator helm chart version 0.4.29
Example of usage:
https://github.com/redpanda-data/helm-charts/blob/182bd51aeaaf2c53a86d9b5fd39a92be48ea5689/charts/operator/testdata/template-cases.txtar#L11-L31


Search Error - Got uncovertable type '[]interface ()

Describe the bug A clear and concise description of what the bug is.
Search query returns error shown in screenshot. It appears this search result has more than one source with errors but I cannot find any other details related to error
To Reproduce Steps to reproduce the behavior:
search query "(status:Incomplete AND sources:Epic)" --indices accountactivities -f "C:\Users\justin.haines\OneDrive - Optiv Security Inc\WakeMed\HealthCheck"
Expected behavior Return search results.
Actual behavior A clear and concise description of what actually happens.
Error occurs
Screenshots If applicable, add screenshots to help explain your problem.

OS: Windows 10

Hey @justinhaines,
This is fixed by https://github.com/sailpoint-oss/sailpoint-cli/commit/b6d847fecbf40404b1531dfc7a44281767a2cc84 and will be included with the next release!


How to embeded flutter to desktop app?

hi,all:
We know currently flutter already supports embed to existing mobile application(android & ios), but I am not sure how to do it on desktop(macos &windows), also no any documents found.

Hi @imiskolee
There is no support for this yet, closing as duplicate of https://github.com/flutter/flutter/issues/80339


Docker usage documentation Inconsistency

The instructions for using docker in the readme (line 227) are different than the instructions in the official docs pages here: https://neo4j.com/docs/apoc/current/installation/#docker
These should be updated for consistency and accuracy. I believe the ones in the readme are accurate, whereas using the env variables as specified in the official docs result in the error:

Failed to read config /var/lib/neo4j/conf/neo4j.conf: Setting 'apoc.export.file.enabled' for APOC was found in the configuration file. In Neo4j v5, APOC settings must be in their own configuration file called apoc.conf. Cleanup the config or disable 'server.config.strict_validation.enabled' to continue.

Hello :) Thanks for writing it. The official docs work on later versions of APOC, there was a bug in previous versions, which is why the README is different as it seems to have been missed in the update when the Docker commands were fixed, which version of APOC are you trying to load?
I can fix this up for consistency for sure :)


Patterns.json: add hierarchy level

Restructure the patterns.json to implement the following structure

version
patterns

doall
pipeline
...

Todo:

[x] update PatternBase
[ ] update vscx parser
[x] update patch generator
[x] ...

fixed in:

10db6bb7a730b46adecdbc68f10b2ce9f728db7a
63cd24cc08a9ee32020c13f05193c224f130a375

@goerlibe
Please close this issue whenever you are ready and have tested the changes in the VSCode extension.


Getting Money by farming sweet berries

Description of new idea or improvement on existing one
is it possible to add a new feature (like the break or place function) for earning money if you are farming the sweet berry bush by right clicking on the bush.
Detailed information what should do what and when

first of all: thanks a lot for your fast and perfect job.
could you pls add in the language file for the new type "collect" the message like this:
collect:
info: '&eCollect'
none: '%jobname% does not get money for collecting blocks.'
and also pls implement the sweet_berry_bush-3 into the translatable words file. at the moment its not recognizing the growing state, so i cant use another language.
thanks a lot

first of all: thanks a lot for your fast and perfect job.

Thanks!

could you pls add in the language file for the new type "collect" the message like this:
collect:
info: '&eCollect'
none: '%jobname% does not get money for collecting blocks.'

Oh, I forgot to add this. Thank you!

and also pls implement the sweet_berry_bush-3 into the translatable words file. at the moment its not recognizing the growing state, so i cant use another language.

No, I don't implement this material, because the plugin will automatically implement the materials to the file if has official ID. If a material has no ID, then not generates it, for saving memory.


closes #432 Add host option to serve command

What kind of change does this PR introduce?
feature: Adds a --host options to serve command

Did you add tests for your changes?
No
Summary
This fixes #432

Does this PR introduce a breaking change?
No

Other information
When testing with simplehttp2server I am running into SSL warning on localhost and it seems normal

Not relevant for next release (serve is being deprecated).


Add amber login support

fixes #3

* I was planning to use https://github.com/chebizarro/flutter-signer-plugin (vs Amberflutter) as it seems better and supports other NIP-55 signers. Also, what is the status of NIP-55 in NDK? Maybe it should be done through NDK?

We'll do it in NDK for sure, it's just not very high priority atm.
I just though we could already have amber support ASAP, since there are currently (AFAIK) no other signers besides Amber anyways.

* I want to keep the ability to sign in with pubkey or NIP-05

Hmm, should still work.

Cool, no rush with NDK, and also no rush with this – we have nothing to sign at the moment (except zaps). As far as I understand NWC would require NDK, correct?
https://github.com/chebizarro/flutter-signer-plugin is more recent and appears higher quality than amberflutter, so I'd go with that.

Hmm, should still work.

I haven't run the branch, but from what I see in the code logging in with external signer is the only option.

I haven't run the branch, but from what I see in the code logging in with external signer is the only option.

I didn't touch the existing login by npub/nip-05, neither the UI for that, so it should be possible to login in either by external signer or by old method.

I haven't run the branch, but from what I see in the code logging in with external signer is the only option.

I didn't touch the existing login by npub/nip-05, neither the UI for that, so it should be possible to log in either by external signer or by old method.

You're right, my mistake

Implemented based on this. Thanks!


Improve tooting of quoted tweets

Hey,
I'm loving Moa so far - it's actually persuading me to use Mastodon more! - and especially love that I have the option of self-hosting if, for example, I wanted to do things slightly differently to how the hosted version runs, so thanks for that!
My main minor issue with it at this point (besides not having an easy way of posting a specific post to only one or the other platform [I only use Twitter & Mastodon currently] - #43 is a nice solution to that, alternatively maybe we could use hashtags to opt-in or opt-out of cross-posting to other platforms, as I've just suggested in #77?) is that quoted tweets end up looking super confusing on Mastodon (and Instagram, maybe?) because the toot includes the content of the quoted tweet but doesn't make it obvious that part of it is quoted.

Would it be possible to add something to make the quote more obvious? I'm partial to the idea of surrounding the quoted tweet in the Unicode left & right double-quotes - “ and ” - but anything that has that effect would work :)

OK, I'll see what I can do.


Generate repository classes and automatically register them with BoundedContext

So far we do not observe a need to add custom logic to repository classes. Now the repository classes merely define type parameters.
In order to reduce the number of steps required in supporting a new entity type, let's generate the repository classes by corresponding entity class (which is in turn scaffolded from a properly annotated entity state message).
All the repositories belonging to a BoundedContext should be registered via a generated class, which is invoked by the application code after an instance of the BoundedContext is built.
If a repository needs a custom storage, it should be constructed manually, passing custom StorageFactory, and registered with the BoundedContext` before the call to automatic repository registration. In that case, the automatic registration would bypass creation of the duplicating repository instance.

@armiol, FYI.

@alexander-yevsyukov To me. DefaultRepository.of(RoverBot.class) qualifies as the desired shortcut. Should we close the issue?

Closing this issue, since DefaultRepository.of() addresses the need.


Test the program under bash v4.4

I found in bash v5.0 release notes the following:

New Features in Bash
...
m. If a non-interactive shell with job control enabled detects that a foreground
job died due to SIGINT, it acts as if it received the SIGINT.

It might mean that my recent fix for killing background jobs work only in bash v5.0, which I tested it on.

I am testing it with bash-4.4.18.

No problems detected.


Sessions should be enabled/disabled by User

Right now, there is not an easy way to disable the session container & session middleware. This seems like it's a little too much, Mandarine is supposed to provide things out of the box but it shouldn't cross the line of forcing the user to. So we want to provide a way to handle this at least for the session container

possibly dependent on https://github.com/mandarineorg/mandarinets/issues/209

Solved in #209


Mesos-master sees framework registration requests as coming from localhost

I have a 3 node Mesos cluster, on which I run masters, slave and Marathon. I run for the moment Chronos only on node1.mesos1.dev. If the active mesos master is not on the same node as Chronos, I get the following error:
Mar  8 01:39:26 node3.mesos1.dev mesos-master[17463]: I0308 01:39:26.158910 17481 master.cpp:818] Received registration request from scheduler(1)@<IP_ADDRESS>:36198
Mar  8 01:39:26 node3.mesos1.dev mesos-master[17463]: I0308 01:39:26.159025 17481 master.cpp:823] Framework 2014-03-08-01:01:57-3053457580-5050-17463-0000 (scheduler(1)@<IP_ADDRESS>:36198) already registered, resending acknowledgement

This is how I run Chronos:
java -cp /opt/chronos/target/chronos-2.1.0_mesos-0.14.0-rc4.jar com.airbnb.scheduler.Main --master zk://<IP_ADDRESS>:2181,<IP_ADDRESS>:2181,<IP_ADDRESS>:2181/mesos --zk_hosts <IP_ADDRESS>:2181,<IP_ADDRESS>:2181,<IP_ADDRESS>:2181 --hostname <IP_ADDRESS> --http_port 4400

Marathon does not seem to suffer of this issue, so my guess is that this is an issue with how Chronos advertises itself.

This is pretty old but it popped up when I was searching for a very similar issue and figured it might help anyone having similar issues.
My environment is Ubuntu Server (14 04 Tacky I think is what its called) running 4 small VM clusters: 1 ZK, 1 Master, and 3 Slaves.
As @vinodkone explained, setting up the LIBPROCESS_IP helped me solve the first problem of <IP_ADDRESS> appearing in the Master logs, but it didn't help with the re-registering logs in the Master.
What I found was that Ubuntu has a pre-installed firewall pretty much blocking all access to all ports. The master tried to connect to an open port on my slave process (in my case it was port 47250). The slave was listening on that port, but because of the firewall the Master couldn't get through. By poking a hole on port 47250 the whole thing magically worked for me and the process successfully completed.
I'm rather new to this whole Mesos thing and that really got me stuck. Once you start distributing on more than on machine these little gotchas seem to pop up all over the place.

Closing this, as different people reported that setting LIBPROCESS_IP fixes the issue.


Mark all read

Hey guys I've been thinking about a Mark all read option as we have in email tools to mark the emails read incase we don't want to read them, This feature currently works in the popup we get when we have I guess 500+ messages to read.
Suggestions are welcomed

Hey @anirudhjain75!  Do you think https://github.com/zulip/zulip/issues/3347 would cover this well?
I think one idea for where to place this feature would be to add a simple caret menu to the right of "home" call "Mark all as read".

Yeah I think that would be better

OK, I'm on board for someone implementing that plan.

Fixed via #8399.


NameError: name 'configuration' is not defined

I'm getting a "name 'configuration' is not defined" error. I followed the instructions carefully, and unsure of why this is causing an issue.
Running on:
Python3 Virtual Environment (installed all the packages here)
Device: 2011 Macbook Pro (8,1)
OS: 64bit Mac OS X 10.13.6 17G14042
Kernel: x86_64 Darwin 17.7.0
Intel Core i5-2435M @ 2.40GHz
Bash
(apienv) bash-5.1$ pip --version
pip 21.2.4 from /Users/noel/telstra/apienv/lib/python3.9/site-packages/pip (python 3.9)

(apienv) bash-5.1$ python3 app.py
Traceback (most recent call last):
  File "/Users/noel/telstra/MessagingAPI-SDK-python/app.py", line 9, in <module>
    configuration.host = "https://tapi.telstra.com/v2"
NameError: name 'configuration' is not defined

What the app.py looks like:
(apienv) bash-5.1$ cat app.py
from __future__ import print_function
import time
import Telstra_Messaging
from Telstra_Messaging.rest import ApiException
from pprint import pprint

# Defining host is optional and default to https://tapi.telstra.com/v2
configuration.host = "https://tapi.telstra.com/v2"
# Create an instance of the API class
api_instance = Telstra_Messaging.AuthenticationApi(Telstra_Messaging.ApiClient(configuration))
client_id = 'DOF8UIcPjmM8ciq1eqGIBzAYvBCG67Pt' # str |
client_secret = 'GDTGeVnP37FoaO14' # str |
grant_type = 'client_credentials' # str |  (default to 'client_credentials')
scope = 'NSMS' # str | NSMS (optional)

try:
    # Generate OAuth2 token
    api_response = api_instance.auth_token(client_id, client_secret, grant_type, scope=scope)
    pprint(api_response)
except ApiException as e:
    print("Exception when calling AuthenticationApi->auth_token: %s\n" % e)
(apienv) bash-5.1$

DId a grep search for the word "configuration" but my suspicion is that there is something wrong with the API URL.

the configuration object does not exist, you are trying to use an object which has yet to be defined.
This is how I use the API:
api_instance = Telstra_Messaging.AuthenticationApi()
api_response = api_instance.auth_token(
   settings.TELSTRA_CLIENT, settings.TELSTRA_SECRET, "client_credentials"
)

# Configure OAuth2 access token for authorization: auth
configuration = Telstra_Messaging.Configuration()
configuration.access_token = api_response.access_token

api_instance = Telstra_Messaging.MessagingApi(Telstra_Messaging.ApiClient(configuration))
payload = Telstra_Messaging.SendSMSRequest(
    to=to,
    body=sms_text,
    _from=settings.SMS_NAME,
    validity=settings.SMS_VALIDITY,
    notify_url=self.callback_url,
)
api_response = api_instance.send_sms(payload)

I've obviously got some stuff there which you'll need to change, but hopefully should help you out a bit


Create an RNPM plugin for create-library

An alternative to publishing upstream, which may be more amenable to generating Windows libraries, would be to make this an RNPM plugin.  A recent commit to RNPM allows any packages that start with react-native-* (which this does) to be recognized as a plugin.
Here's how we allow RNPM to pickup plugins from react-native-windows:
https://github.com/ReactWindows/react-native-windows/blob/master/package.json#L38-L40

This looks very interesting and I'm intrigued. This is definitely something I'll take a closer look during the next few days if I have a few hours to myself.

Anyone actively working on that? I am up to adding support since that's very useful and idiomatic way of doing things.

All done! 🎉 🍾
This has just been published as 3.0.2.


[Issue]: Outdated https://docs.ag2.ai/docs/contributor-guide/documentation

Describe the issue
We migrated doc website to mintlify. Need to update contributor-guide/documentation page with https://github.com/ag2ai/ag2/blob/17ab5d69e0bb4ccdd4e021fda5e2fca0fec5314b/website/README.md
Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
No response

@harishmohanraj Could you update it? thank you!


[Snyk] Upgrade @apollo/client from 3.5.8 to 3.7.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @apollo/client from 3.5.8 to 3.7.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 48 versions ahead of your current version.
The recommended version was released 2 months ago, on 2022-10-20.

Release notes

Package name: @apollo/client

3.7.1 - 2022-10-20Bug fixes (from CHANGELOG.md)

Fix issue where loading remains true after observer.refetch is called repeatedly with different variables when the same data are returned.
@ alessbell in #10143

Fix race condition where useFragment_experimental could receive cache updates before initially calling cache.watch in useEffect.
@ benjamn in #10212

What's Changed (auto-generated by GitHub)

changes install message by @ jpvajda in #10149
oct 2022 roadmap update by @ jpvajda in #10161
Small updates for AS4 by @ rkoron007 in #10175
Various edits to caching docs by @ StephenBarlow in #9984
Use nav nesting and tweak defer article slightly by @ StephenBarlow in #10187
Call iterateObserversSafely if vars change between calls to observer.next by @ alessbell in #10143
Update docs algolia filters by @ trevorblades in #10200
Fix typo in NetworkError by @ OliverWales in #10204
Update apollo-link-rest.md - Add graphql-anywhere to install list by @ pfcodes in #10138
Use custom config/bundlesize.ts script to avoid iltorb dependency, unbreaking npm install when using Node.js v19 by @ benjamn in #10206
Experiment with not caching the ~/.npm directory during Filesize/Tests CircleCI jobs by @ benjamn in #10209
docs: add TypedDocumentNode example by @ henryqdineen in #10031
Update Local Resolvers docs by @ bignimbus in #10211
Remove (deprecated) from Local resolvers doc title by @ bignimbus in #10213
Remaining Local Resolvers wordsmithing by @ bignimbus in #10214
Call forceUpdate immediately if diff changes between first useFragment call and first cache.watch call by @ benjamn in #10212

New Contributors (auto-generated by GitHub)

@ OliverWales made their first contribution in #10204
@ pfcodes made their first contribution in #10138

Full Changelog: v3.7.0...v3.7.1

3.7.0 - 2022-09-30 Read more

3.7.0-rc.0 - 2022-09-21This is the first "release candidate" for the upcoming 3.7.0 minor
release, signalling we will only be fixing bugs and adding missing
documentation before the final release. No new features, in other words.
Please run npm i @ apollo/client@next to install this version and kick
the tires in your real applications. We appreciate your feedback! 🐣

3.7.0-beta.8 - 2022-09-21Bump @ apollo/client npm version to 3.7.0-beta.8.

3.7.0-beta.7 - 2022-09-08Bump @ apollo/client npm version to 3.7.0-beta.7.

3.7.0-beta.6 - 2022-06-27Bump @ apollo/client npm version to 3.7.0-beta.6.

3.7.0-beta.5 - 2022-06-10Bump @ apollo/client npm version to 3.7.0-beta.5.

3.7.0-beta.4 - 2022-06-10

3.7.0-beta.3 - 2022-06-07

3.7.0-beta.2 - 2022-06-07

3.7.0-beta.1 - 2022-05-26

3.7.0-beta.0 - 2022-05-25

3.7.0-alpha.6 - 2022-05-19

3.7.0-alpha.5 - 2022-05-16

3.7.0-alpha.4 - 2022-05-13

3.7.0-alpha.3 - 2022-05-09

3.7.0-alpha.2 - 2022-05-03

3.7.0-alpha.1 - 2022-05-03

3.7.0-alpha.0 - 2022-04-27

3.6.10 - 2022-09-29 Read more

3.6.9 - 2022-06-21Bug Fixes (from CHANGELOG.md)

Leave fetchPolicy unchanged when skip: true (or in standby) and nextFetchPolicy is available, even if variables change.
@ benjamn in #9823

What's Changed

removes deprecation language by @ jpvajda in #9825
Leave fetchPolicy unchanged when skip: true (or in standby) and nextFetchPolicy is available by @ benjamn in #9823

Full Changelog: v3.6.8...v3.6.9

3.6.8 - 2022-06-10

3.6.7 - 2022-06-10

3.6.6 - 2022-05-26

3.6.5 - 2022-05-23

3.6.4 - 2022-05-16

3.6.3 - 2022-05-05

3.6.2 - 2022-05-03

3.6.1 - 2022-04-28

3.6.0 - 2022-04-26

3.6.0-rc.1 - 2022-04-19

3.6.0-rc.0 - 2022-04-18

3.6.0-beta.13 - 2022-04-14

3.6.0-beta.12 - 2022-04-11

3.6.0-beta.11 - 2022-04-05

3.6.0-beta.10 - 2022-03-29

3.6.0-beta.9 - 2022-03-10

3.6.0-beta.8 - 2022-03-10

3.6.0-beta.7 - 2022-03-10

3.6.0-beta.6 - 2022-02-15

3.6.0-beta.5 - 2022-02-04

3.6.0-beta.4 - 2022-02-03

3.6.0-beta.3 - 2021-11-23

3.6.0-beta.2 - 2021-11-22

3.6.0-beta.1 - 2021-11-16

3.6.0-beta.0 - 2021-11-16

3.5.10 - 2022-02-24

3.5.9 - 2022-02-15

3.5.8 - 2022-01-24

    from @apollo/client GitHub release notes

Commit messages

Package name: @apollo/client

e6ad3c9 Bump @ apollo/client npm version to 3.7.1.
81bc6b4 Detect if `diff` changes between `useFragment` and `cache.watch` (#10212)
b03dff2 Remaining Local Resolvers wordsmithing (#10214)
3c7736d Remove (deprecated) from Local resolvers doc title (#10213)
8bfdec1 Update Local Resolvers docs (#10211)
896e5d8 docs: add TypedDocumentNode example (#10031)
b4dd365 Stop caching `~/.npm` directory during Filesize/Tests CircleCI jobs (#10209)
643c45c Use custom `config/bundlesize.ts` script to avoid `iltorb` dependency (#10206)
b0cef62 Update apollo-link-rest.md (#10138)
b23234e Fix typo (#10204)
948eaeb updates roadmaps for oct
e7d1347 Update docs algolia filters (#10200)
ebcec85 Call `iterateObserversSafely` if vars change between calls to `observer.next` (#10143)
03c8fa4 chore(deps): update dependency mocha to v10.1.0
6a5bbac chore(deps): update dependency @ types/node to v16.11.66
935e773 chore(deps): update dependency jest-environment-jsdom to v29.2.0
8577690 chore(deps): update dependency jest to v29.2.0
b16776d chore(deps): update dependency @ types/node to v16.11.65
773e365 chore(deps): update dependency @ babel/parser to v7.19.4
3a98bbf Merge pull request #10187 from apollographql/sb/docs-nav-tweak
9cf4ae5 Use nav nesting and tweak defer article slightly
c915214 Various edits to caching docs (#9984)
ff6a490 updates rest link docs page
c03f92d Small updates for AS4 (#10175)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Main branch dependencies

Neodora Analysis ⚠
 ⚠ Incorrect runner in Github Workflow

Message: Github workflow must run on a self-hosted runner

Workflow file
Job name
Actual runs-on value
Expected runs-on value

perf-test.yml
docker
ubuntu-latest
Any of [eks-runner, eks-runner-php, eks-runner-nano, eks-runner-debug, eks-runner-compute]

🔍 How to fix it

 ⚠ Incorrect runner in Github Workflow

Message: Github workflow must run on a self-hosted runner

Workflow file
Job name
Actual runs-on value
Expected runs-on value

unit-test.yml
test
ubuntu-latest
Any of [eks-runner, eks-runner-php, eks-runner-nano, eks-runner-debug, eks-runner-compute]

🔍 How to fix it

 ⚠ Missing service namespace configuration

Message: The property namespace in repository.json must exist in SPOT and service must be deployed under correct namespace
Repository file: repository.json

Namespace status

Error: namespace is not defined

🔍 How to fix it

 ⚠ Incorrect runner in Github Workflow

Message: Github workflow must run on a self-hosted runner

Workflow file
Job name
Actual runs-on value
Expected runs-on value

functional-test.yml
docker
ubuntu-latest
Any of [eks-runner, eks-runner-php, eks-runner-nano, eks-runner-debug, eks-runner-compute]

🔍 How to fix it

 ⚠ Incorrect runner in Github Workflow

Message: Github workflow must run on a self-hosted runner

Workflow file
Job name
Actual runs-on value
Expected runs-on value

integration-test.yml
test
ubuntu-latest
Any of [eks-runner, eks-runner-php, eks-runner-nano, eks-runner-debug, eks-runner-compute]

🔍 How to fix it

 ⚠ Incorrect runner in Github Workflow

Message: Github workflow must run on a self-hosted runner

Workflow file
Job name
Actual runs-on value
Expected runs-on value

lint.yml
test
ubuntu-latest
Any of [eks-runner, eks-runner-php, eks-runner-nano, eks-runner-debug, eks-runner-compute]

🔍 How to fix it

Commit a355b6b  Job 126669


Bugfix/pod configuration

I was only able to use this library with the shown modifications. Since Pods will become more important to manage native iOS dependencies, I adapted the readme and created this PR. Happy to discuss any changes!

fixes #19

Thanks @tobiaseisenschenk! I’ll try it soon @guilhermedecampo

Nice!

Can I help further with resolving the conflicts?

Hi @tobiaseisenschenk, thanks for your contribution
I tested in a real project and it's still showing the error

I installed the new version, added the pod and installed the new pods, are there any steps I'm missing?

Hi @tobiaseisenschenk, thanks for your contribution
I tested in a real project and it's still showing the error

I installed the new version, added the pod and installed the new pods, are there any steps I'm missing?

Are you already managing your RN dependencies in your podfile? This article explains the general setup. Do you have libPods added to the target's "linked frameworks and libraries" in XCode?

Hi @tobiaseisenschenk, thanks for your contribution
I tested in a real project and it's still showing the error

I installed the new version, added the pod and installed the new pods, are there any steps I'm missing?

Are you already managing your RN dependencies in your podfile? This article explains the general setup. Do you have libPods added to the target's "linked frameworks and libraries" in XCode?

Yes, I have other dependencies using pods, they are working fine

Yes, I have other dependencies using pods, they are working fine
For me RCTBaseTextInputView.h points to this path in xcode:

Do you also have this binary linked under Libraries?

Yes, I have other dependencies using pods, they are working fine

For me RCTBaseTextInputView.h points to this path in xcode:

Do you also have this binary linked under Libraries?

yes, if I remove the selectable text from the linked libraries and build, then add again and build, it works

yes, if I remove the selectable text from the linked libraries and build, then add again and build, it works

Ah ok i think the issue is that you have selectableText as linked library at all! I have libPods as first item in "link binary with libraries" since any library that you install through pods comes through the libPods item and therefore doesn't have to be linked additionally anymore.

Would be nice to have something in the README about this possible error!

yes, if I remove the selectable text from the linked libraries and build, then add again and build, it works

Ah ok i think the issue is that you have selectableText as linked library at all! I have libPods as first item in "link binary with libraries" since any library that you install through pods comes through the libPods item and therefore doesn't have to be linked additionally anymore.

Indeed removed the link and is working now, sorry my bad, approved and after fix the conflicts can merge

nice good catch!

Thanks @tobiaseisenschenk !


[QUESTION] Parametrizing a class dependency on a sub-dependency

First check

[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.

Description
When dealing with sub-dependencies, the documentation introduces e.g.
oauth2_scheme = OAuth2PasswordBearer(...)

async def get_current_user(token: str = Depends(oauth2_scheme)):
    ...

Chained dependencies / sub-dependencies can be achieved with free functions.
The documentation also introduces parametrized dependencies in the form of callable instances, e.g.
class FixedContentQueryChecker:
    def __init__(self, fixed_content: str):
        self.fixed_content = fixed_content

    def __call__(self, q: str = ""):
        if q:
            return self.fixed_content in q
        return False

checker = FixedContentQueryChecker("bar")

@app.get("/query-checker/")
async def read_query_check(fixed_content_included: bool = Depends(checker)):
    return {"fixed_content_in_query": fixed_content_included}

Now is there a way I can write a CurrentUserFetcher class, parametrized on an OAuth2 instance?
This would take the following form:
oauth2_scheme = OAuth2PasswordBearer(...)

get_current_user = CurrentUserFetcher(oauth2_scheme)

@app.get("/foo/")
async def get_foo(current_user: User = Depends(get_current_user)):
  ...

Thank you

Do you want to authenticate an user in your system by using a dependency called get_current_user or creates a class dependency?

I want get_current_user to be a callable instance of a class dependency. So that I can parametrize the get_current_user on sub-dependency.

Maybe something like this?
import datetime
import jwt
import pydantic
from fastapi import HTTPException
from fastapi import Depends
from fastapi.security import OAuth2PasswordBearer
from fastapi.security import OAuth2PasswordRequestForm
from starlette import status

TOKEN_AUDIENCE = 'urn:users'
TOKEN_URL = '/auth/login/'
TOKEN_LIFETIME_SECONDS = 1000
TOKEN_ALGORITHM = 'HS256'
TOKEN_SECRET = 'yoursupersecret'

authorization = OAuth2PasswordBearer(TOKEN_URL, auto_error=False)

class User():
  """User data class."""

  id: int
  is_verified: bool
  create_date: datetime.datetime
  password: str
  name: str
  email: pydantic.EmailStr
  birth_date: datetime.date

async def get_current_user(
    token: str = Depends(authorization)
) -> User:
  """Get current user from the request metadata.

  Args:
    token (str): authorization token.

  Raises:
    HTTPException: whether there is no authenticated user.

  Returns:
    models.user.schemas.User: the user who performs the request.
  """
  if not token:
    _unauthorized()
  try:
    data = await decode_token(token, TOKEN_AUDIENCE)
    user_id = data.get('user_id')
    if not user_id:
      _unauthorized()
    # Get your user here from database
    return await User.get(user_id)
  except jwt.PyJWTError:
    _unauthorized()

async def generate_token(
    user: User,
    audience: str = TOKEN_AUDIENCE,
    lifetime: int = TOKEN_LIFETIME_SECONDS
) -> str:
  """Generates authentication token for the given user.

  Args:
    user (models.user.schemas.User): the user the token will be generated for.
    audience (str, optional): token audience. Defaults to st.TOKEN_AUDIENCE.
    lifetime (int, optional): token seconds lifetime. Defaults to
      st.TOKEN_LIFETIME_SECONDS

  Returns:
    str: the user authentication token.
  """
  expire = (
      datetime.datetime.utcnow() +
      datetime.timedelta(seconds=int(lifetime)))
  data = {
      'user_id': user.id,
      'aud': audience,
      'exp': expire
  }
  return jwt.encode(
      data, TOKEN_SECRET, algorithm=TOKEN_ALGORITHM).decode('utf-8')

async def decode_token(token: str, audience: str = TOKEN_AUDIENCE):
  """Decode token.

  Args:
    token (str): the token which will be decoded.
    audience (str, optional): token audience. Defaults to st.TOKEN_AUDIENCE.

  Raises:
    jwt.PyJWTError: whether the given token is expired or malformed.

  Returns:
    dict: token decoded data.
  """
  return jwt.decode(
      token, TOKEN_SECRET,
      audience=audience, algorithms=[TOKEN_ALGORITHM])

def verify_and_update_password(
    plain_password: str, hashed_password: str
) -> Tuple[bool, str]:
  """Verify the given password by matching it with the hashed one.

  Args:
    plain_password (str): plain password.
    hashed_password (str): hashed password.

  Returns:
    Tuple[bool, str]: a tuple which contains if the password has been verified
    and the new password hash.
  """
  return pwd_context.verify_and_update(plain_password, hashed_password)

async def authenticate(
    credentials: OAuth2PasswordRequestForm
) -> User:
  """Authenticates a user with the given credentials.

  Args:
    credentials (OAuth2PasswordRequestForm): user credentials.

  Returns:
    schemas.User: the user who match with the given credentials.
  """
  user = await User.get_by_email(credentials.username)

  verified, updated_password_hash = verify_and_update_password(
      credentials.password, user.password)

  if not verified:
    raise Exception('User not verified')

  # Update password hash to a more robust one if needed
  if updated_password_hash is not None:
    user.hashed_password = updated_password_hash
    await User.update(user)

  return user

def _unauthorized(headers=None):
  """Raise HTTPException with 401 status code.

  Args:
    headers: (dict, optional): additional headers to the response. Defaults to
      None.

  Raises:
    HTTPException: elevate handler unauthorized exception.
  """
  detail = 'Not authenticated'
  raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail=detail)

Take care with User get get_by_email update, the are operations examples and are not implemented in the example code.

@alvarogf97 I'm asking about sub-dependencies when writing dependencies as classes. If you read my question carefully, you'll realize the excerpt you pasted uses sub-dependencies when writing dependencies as free functions. All in all you're just adding noise my question 😐

@tiangolo Any idea please?

Sure, you can do that, just pass the subdependencies in the __call__().
E.g. copying your fragments, something like:

oauth2_scheme = OAuth2PasswordBearer(...)

class ParameterizedGetUser:
    def __init__(self, some_config: str):
        self.some_config = some_config

    def __call__(self, token: str = Depends(oauth2_scheme)):
        if self.some_config == "something":
            return token
        return False

@app.get("/items/")
async def read_items(user: User = Depends(ParameterizedGetUser(some_config="Foo"))):
    return {"message": "Hello World"}

:warning: I'm just copying your snippets and combining them, you'll have to adapt it and check it with your code.

But can the oauth2_scheme be passed to ParameterizedGetUser's constructor?

I have the same issue for endpoints where I have a sub-dependency that depend on an authorization dependency swagger doesnt seem to get that it need authorization (the lock icon is not shown) and swagger doesn send the token (Bearer in my case)

But can the oauth2_scheme instance be passed to ParameterizedGetUser's constructor?

@gpakosz I'm not sure I understand what do you mean with that :thinking:
@transfluxus please create a minimal self-contained example showing the problem, that way it's easier to help.

@tiangolo damn in my minimal self-contained example its working... :smiley:

@transfluxus Hehe that happens...
Comparing your self-contained example with your complete app will probably help you to find the bug/problem. It could be somewhere in your app's code, or even maybe in FastAPI itself. :shrug:

@tiangolo I would like write
oauth2_scheme_variant1 = ...

oauth2_scheme_variant2 = ...

@app.get("/foo/")
async def get_foo(user: User = Depends(ParameterizedGetUser(oauth2_scheme_variant1)):
    return {"message": "foo"}

@app.get("/bar/")
async def get_bar(user: User = Depends(ParameterizedGetUser(oauth2_scheme_variant2))):
    return {"message": "bar"}

So as to write the ParameterizedGetUser once, and inject a sub-dependency via the constructor.

@tiangolo in your suggestion:
def __call__(self, token: str = Depends(oauth2_scheme)):

You're hardcoding the dependency in the def __call__ definition.
In other words, I'm asking whether there's a way to make the dependency a member variable of the class that defines the __call__ function

Hmm, you can probably do something like:
from typing import Callable
from fastapi import FastAPI, Depends
from fastapi.security import OAuth2PasswordBearer

oauth2_scheme = OAuth2PasswordBearer(...)

app = FastAPI()

def get_dependency(auth: Callable, some_config: str):
    def dependency(value: str = Depends(auth)):
        print(some_config)
        return value

    return dependency

@app.get("/foo/")
async def get_foo(
    current_user: dict = Depends(get_dependency(oauth2_scheme, some_config="bar"))
):
    return "Hello World"


HTTPCLIENT-1830: await termination of the IdleConnectionEvictor when it is shut down (4.5.x)

Same as #67 but for 4.5.x branch.

As discussed in HTTPCLIENT-1830, updated this pull request to keep await termination separate from the shutdown and keeping the change set to a minimum.

This has been merged.


Add diachronic information to QuiVer

Describe the feature you'd like
Currently we display only the latest information about a workflow in QuiVer. We run a workflow A, the important metrics are saved and overwritten when we run workflow A again.
In order to measure how the changes in the OCR-D software impact the OCR quality as well as the hardware statistics we should introduce diachronic information to QuiVer, e.g. via a time stamp.
User story
As a developer I need an overview of how the changes in the software effect the OCR quality and hardware metrics in order to be certain that the newest contribution to OCR-D really improve the software's outcome.
Ideas we have discussed so far
How to display the information
For each GT corpus available there should be a line chart that depicts how a metric has changed over time. Each step in time (x axis) represents an ocrd_all or a ocrd_core release (clarify!)
Users can choose between the different metrics and can see a tendency whether the metric improves or not.
Underlying data structure
When selecting a GT corpus the front end uses an ID map file that points it to the right collection of JSON objects. Each OCR-D workflow that is executed on a GT corpus has a separate file in which all the runs per release are present.
Given GT workspace 16_ant_simple. We then have a file 16_ant_simple_minimal.json with all its benchmarking workflows, 16_ant_simple_selected_pages.json with all its benchmarking workflows etc. Each executed workflow has a timestamp by which the front end can then sort the single executions and retrieve the relevant data.
TODOs

[ ] clarify what our steps / increments in time are. A release of ocrd_all? A release of ocrd_core?
[x] add time stamps to workflow objects
[ ] add single files for each GT workspace + workflow. ideally, the data should be sorted chronologically right from the start (although the front end should not depend on that)
[ ] create id map file

clarify what our steps / increments in time are. A release of ocrd_all? A release of ocrd_core?

According to @kba this doesn't matter much so I will opt for ocrd_all.

+1 for basing this of ocrd_all.


revert back to remaining_gas for syscalls

Gas consumed is calculated on the SiR part, not here, got confused about that.
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #330 (57dd37e) into main (120f972) will increase coverage by 0.23%.
Report is 2 commits behind head on main.
The diff coverage is 19.23%.

:exclamation: Current head 57dd37e differs from pull request most recent head 7ac3b13. Consider uploading reports for the commit 7ac3b13 to get more accurate results

@@            Coverage Diff             @@
##             main     #330      +/-   ##
==========================================
+ Coverage   71.86%   72.09%   +0.23%
==========================================
  Files         117      117
  Lines       20773    20696      -77
==========================================
- Hits        14928    14921       -7
+ Misses       5845     5775      -70

Files
Coverage Δ

src/execution_result.rs
80.17% <75.00%> (ø)

src/libfuncs/stark_net.rs
14.52% <9.09%> (+0.09%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Revised & Expanded dice roll commands

I'd like to request an expansion to Gravebot's current dice function. These features are partially inspired by the popular Roll20 site.

Change syntax to XdY+MOD. X being the number of dice, Y being the die size,** MOD** being the modifier that gets added to a given roll. This allows the bot to be used quickly by any tabletop gamer as the aforementioned syntax has almost become standard.

Automated re-rolls aka "Exploding dice". This means that when - for example - a number of six-sided dice are rolled, any die that comes up a 6 gets re-rolled. Let us say I roll 4d6 (4 six-sided dice).If one of these comes up as a 6 I roll it again, which ends up putting out five die results instead of just four.

Success-based rolls. Some systems such as Vampire: The Masquarade or Shadowrun simply roll a certain number of dice and evaluate only how many of them come up above a certain number. For example, Vampire considers any roll above 8 a success. If I roll 8 10-sided dice and 5 come up 8, 9 or 10 this means 5 successes have been achieved.

I thank you for the work put in so far and look forward to your responses.
-Sincerely, Winter

I heavily support the first point. I often get complaints from users on my server about the non-standard rolling format. I don't know about the rest of the stuff, but I hope that the change of syntax for rolling isn't something too difficult to go about. :)

Sounds great, thanks for the awesome input! We've added it to the list. As you've detailed everything quite well and can code, feel free to put up a PR. Otherwise we'll get to it when we can. :)

Thanks a ton!


Limits on how many VNet service endpoints for storage accounts

https://docs.microsoft.com/en-us/azure/virtual-network/virtual-networks-faq#does-azure-active-directory-azure-ad-support-vnet-service-endpoints
         Azure service | Limits on VNet rules
         Azure Storage | 100

As per below link:
https://docs.microsoft.com/en-us/azure/storage/common/storage-network-security?tabs=azure-portal#grant-access-from-a-virtual-network --> Limit for Storage Account should be increased from 100 to 200
#################################################################
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 07cec477-f584-400f-889c-9a5403f34bad
Version Independent ID: f777fdfd-2f5a-56b9-574f-ae6d2b19f130
Content: Azure Virtual Network FAQ
Content Source: articles/virtual-network/virtual-networks-faq.md
Service: virtual-network
GitHub Login: @KumudD
Microsoft Alias: kumud

@diegodct , thank you for your feedback. We'll review and update as appropriate!

@diegodct , we've updated the doc with the Limits on VNet service endpoints for storage accounts as 200 and it will reflect shortly!


fix: typo in api-reference.mdx

Description (required)

typo in api-reference.mdx

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌑 This pull request will not trigger status changes.

Learn more
Lunaria automatically ignores changes on specific PRs by adding a ignored keyword in its title. Found: typo.
You can change this by either removing the keyword above from the PR's title, or modifying the ignoreKeywords property in your Lunaria configuration file.

Tracked Files

[!NOTE]
The notes below indicate what would happen if the pull request is merged when triggering status changes. Since a ignored keyword was found in the PR's title, the status changes indicated below won't be applied.

File
Note

en/reference/api-reference.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Change AdminFlags on commands to separate debug and moderation commands

About the PR

Change AdminFlags for some commands.

BanExemptionUpdateCommand changed from Admin to Ban.

Why / Balance

This is to separate debug commands from moderation commands.
Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

re-open if you come back to this


Remove a couple of unused dependencies

Removed in https://github.com/badges/shields/pull/2906/files

@babel/plugin-proposal-class-properties
@babel/plugin-proposal-object-rest-spread
@babel/preset-env

Built in:

url

No usage detected:

mkdirp
react-error-overlay
walkdir

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @znarf!

  Generated by :no_entry_sign: dangerJS against 5e66313adcdf7c9f78451d722456b4313a7d4957

@PyvesB I checked and react-error-overlay is effectively required. Looks like this file from gatsby is copied in the .cache directory. It's smelling a bit wrong but I don't have enough knowledge in gatsby internals to dig more into it.
I'm adding it back, however I think it's best to add it in version "^3.0.0" so that it matches what current version of gatsby (2.19.22) is expecting.


Clippy on ci-flow with -D warnings

Hello,
It's more of a question for discussion whether the clippy task should fail on any warnings. Maybe this can also be controlled with an env. variable?
E.g.
args = ["clippy", "--all-features", "--", "-D", "warnings"]

I think that's really good idea.
with the help of the [split function][https://github.com/sagiegurari/cargo-make#split] I could have a default empty env var and users could modify that env var to whatever args they want to add.
you can today just override that task, and set the args only.

Yes, that's what I did, but for the CI flow I thought it's a good idea to actually fail the build on such occasions. And I wanted to hear your thoughts on that.

I'm thinking of doing something like the following:
[env]
CARGO_MAKE_CLIPPY_ALL_FEATURES_WARN = "--all-features -- -D warnings"
CARGO_MAKE_CLIPPY_NO_ARGS = "--all-features -- -D warnings"
CARGO_MAKE_CLIPPY_ARGS = "${CARGO_MAKE_CLIPPY_NO_ARGS}"

[tasks.clippy]
description = "Runs clippy code linter."
category = "Test"
dependencies = [
    "install-clippy"
]
command = "cargo"
args = ["clippy", "@@split(CARGO_MAKE_CLIPPY_ARGS, )"]

this will basically mean that by default, clippy is invoked without any args like today.
but you could change to all features+warn by doing the following:
[env]
CARGO_MAKE_CLIPPY_ARGS = "${CARGO_MAKE_CLIPPY_ALL_FEATURES_WARN }"

in your makefile.
and if you wish to enable it for a specific profile only, you can do it as follows:
[env.ci]
CARGO_MAKE_CLIPPY_ARGS = "${CARGO_MAKE_CLIPPY_ALL_FEATURES_WARN }"

and run it
cargo make --profile ci

Sounds good to me. I will try to sit down today and fire a PR :blush:

actually this one i already developed (i took the example from what i did and tested).
but you could do a pr to the other issue you opened, but i would prefer there a bit better shell script solution if possible

@sagiegurari when can I expect this to be committed & released, so that I can check it and close this issue?

i actually pushed it to 0.19.5 branch 2 minutes ago.
so you can check it now if you install it from github directly

Tested it today. Works perfectly! Thanks!

thanks for confirming!!!


Feature: ARR Reviewer Assignment Quotas

This PR adds pre-process checks to the Invite_Assignment and Assignment invitations to ensure that at most 3 reviewers are assigned/invited at any time
Resolves:

#2220

I've added a test to ensure that the pre-process validation is skipped if it is not provided in the venue request form (in the mock June cycle), but I'm not sure if this is necessary


Reflect derive macro uses wildcard imports

Describe the bug
When using #[derive(Reflect)] inside a workspace where the clippy::wildcard_imports lint is disallowed, it fails the linter:
error: usage of wildcard import
  --> test.rs:75:35
   |
75 |   #[derive(Component, Clone, Debug, Reflect)]
   |                                     ^------
   |                                     |
   |  ___________________________________in this derive macro expansion
   | |
76 | | pub struct Foo {
   | |___^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wildcard_imports
   = note: `-D clippy::wildcard-imports` implied by `-D warnings`
   = help: to override `-D warnings` add `#[allow(clippy::wildcard_imports)]`
   = note: this error originates in the derive macro `Reflect` (in Nightly builds, run with -Z macro-backtrace for more info)

Is there any particular reason why these wildcard imports are used, and if so, can they be annotated
with a #[allow] internally?

Is there any particular reason why these wildcard imports are used

Makes our lives so much easier. Wildcard imports are good :)
Adding #[allow] is fine. Do you wanna submit a PR perhaps?


Project is tagged as Godot 4, but seems to be made in a version of Godot 3

Project is not labeled correctly in the asset library.

The project is made with Godot 4, but it's not updated to work with 4.0.beta4.


Fix code coverage

The CircleCI code coverage badge isn't working. This needs to be remedied.

Can I grab this one?

@nicolaspascual go for it!

It's done but in order for this to work you need to configure test coverage of codeclimate here.


Pooling of cells (bam files)

In the publication it's stated that kNN pooling of cells was used for getting a better estimate of gamma.
Is this done in the python implementation or in any implementation if more than one bam file is supplied. Specifically I'm working with SMART-seq2 data.
I'm expecting that the results may vary due to what counts are used to cluster cells, normalized vs non normalized etc.
Basically I'm wondering if it equivalent to submit a single bam file or all cell bam files at once.

Starting to look in to the output of velocyto it looks like the calculations of velocity is done after the original counts are obtained in the loom output.
Am I correct in this? Meaning if I concatenate the loomfiles the pooling will be done in my secondary analysis.


Suggestion to support postman collection schema for documentation generation

I want to suggest supporting the Postman Collection Schema https://schema.getpostman.com/ for document generation

Unfortunately, this is out-of-scope of this project. Postman btw has it's own documentation.
Maybe there is a convertor from postman collection schema to swagger


❓Survey: Your experience in this repo

Whether you're a frequent visitor or a relatively new visitor of the repo, please tell us about your experience with our issue support and/or PR contribution process. The survey should only take a few minutes, and you can answer for up to 3 repos:
https://www.surveymonkey.com/r/8QTQTGJ?Source=azure-sdk-for-python

Thank you for your interest in the experience survey. The survey has now been closed.
If you didn't get a chance to provide feedback this time, please look out for future surveys.


skyportal-fink-client : Initial Commit

Welcome ! This first pull request aims to add the first of many iterations to a new SkyPortal extension: skyportal-fink-client
What is skyportal-fink-client ?
skyportal-fink-client is an extension to be used along side with an instance of SkyPortal. But, as it only uses API calls to communicate with SkyPortal, it runs "outside" of it, it does not depend on any files/code from SkyPortal, and it does not modify the behavior of SkyPortal either.
What does it do ?
Thanks to skyportal-fink-client, you will be able to add the data from alerts coming from Fink Broker directly into SkyPortal, in real time. The only thing you need to do is to edit the config.yaml file that contains the necessary information to access to both Fink and SkyPortal, and allows you to choose which topics to listen to, and to what classifications they will correspond to in SkyPortal.
How does it work ?
On the Fink side, we use fink-client to poll alerts in real time from Fink broker. Your credentials can be configured in the config.yaml file (server, username, group, password, topics and associated probability and classification).
Then, we process the data and proceed to post it to SkyPortal's DB using only API calls. Same as before, your credentials can be configured in the config.yaml file (skyportal url and api token).
How do i use it ?
After filling in your credentials and other parameters in the config.yaml file (don't forget to set the testing parameter to False if you intend to poll real alerts from a non-local Kafka server, like Fink Broker), you need to create a python virtual environment using the option of your choice (conda, virtualenv, pyenv ...). After activating the virtual environment and installing the required dependencies found in requirements.txt, you can now use the command make poll_alerts to start polling alerts and posting them to SkyPortal.
To see the newly added data, just head to SkyPortal's dahsboard, or sources, or candidates page to see the results.
What's next ? To-dos ?
For us, the next step will be writing clear installation instructions in a README file, as well as adding additional make commands to run testing locally in a user friendly way.

when you say in the description above:

Thanks to skyportal-fink-client, you will be able to add the date from alerts coming from Fink Broker directly into SkyPortal, in real time.

you really mean the date from alerts or the alerts?

when you say in the description above:

Thanks to skyportal-fink-client, you will be able to add the date from alerts coming from Fink Broker directly into SkyPortal, in real time.

you really mean the date from alerts or the alerts?

Fixed ! Spelling mistake, i meant "data"

First batch of comments -- looks great so far :-)
I will push more later today.

Thanks for your review ! I'll address those comments today :)