Cannot Access CVAT from Outside Localhost (i.e., using my own domain)

My actions before raising this issue

[X] Read/searched the docs
[X] Searched past issues

Expected Behaviour

Access CVAT from Chrome by navigating to http://cvat.ai-lab.edu:8080. I should also mention that I everything works fine when accessing CVAT from localhost. But, multiple people will be using this server and so it needs to work on our domain network.
Current Behaviour

In Chrome I get, "This site can’t be reached. The connection was reset."
Steps to Reproduce (for bugs)

Followed CVAT Installation Instructions for v1.4.0.
Set the CVAT_HOST parameter to cvat.ai-lab.edu in the docker-compose.override.yml file.

version: '3.3'

services:
  cvat_proxy:
    environment:
      CVAT_HOST: cvat.ai-lab.edu

  cvat:
    environment:
      CVAT_SHARE_URL: 'Mounted from ai-lab /data NFS'
    volumes:
      - cvat_share:/home/django/share

volumes:
  cvat_share:
    driver_opts:
      type: none
      device: /data
      o: bind

cvat/docker-compose.yml

#
# Copyright (C) 2018-2021 Intel Corporation
#
# SPDX-License-Identifier: MIT
#
version: '3.3'

services:
  cvat_db:
    container_name: cvat_db
    image: postgres:10-alpine
    networks:
      default:
        aliases:
          - db
    restart: always
    environment:
      POSTGRES_USER: root
      POSTGRES_DB: cvat
      POSTGRES_HOST_AUTH_METHOD: trust
    volumes:
      - cvat_db:/var/lib/postgresql/data

  cvat_redis:
    container_name: cvat_redis
    image: redis:4.0-alpine
    networks:
      default:
        aliases:
          - redis
    restart: always

  cvat:
    container_name: cvat
    image: openvino/cvat_server
    restart: always
    depends_on:
      - cvat_redis
      - cvat_db
    environment:
      DJANGO_MODWSGI_EXTRA_ARGS: ''
      ALLOWED_HOSTS: '*'
      CVAT_REDIS_HOST: 'cvat_redis'
      CVAT_POSTGRES_HOST: 'cvat_db'
      ADAPTIVE_AUTO_ANNOTATION: 'false'
    volumes:
      - cvat_data:/home/django/data
      - cvat_keys:/home/django/keys
      - cvat_logs:/home/django/logs

  cvat_ui:
    container_name: cvat_ui
    image: openvino/cvat_ui
    restart: always
    networks:
      default:
        aliases:
          - ui
    depends_on:
      - cvat

  cvat_proxy:
    container_name: cvat_proxy
    image: nginx:stable-alpine
    restart: always
    depends_on:
      - cvat
      - cvat_ui
    environment:
      CVAT_HOST: localhost
    ports:
      - '8080:80'
    volumes:
      - ./cvat_proxy/nginx.conf:/etc/nginx/nginx.conf:ro
      - ./cvat_proxy/conf.d/cvat.conf.template:/etc/nginx/conf.d/cvat.conf.template:ro
    command: /bin/sh -c "envsubst '$$CVAT_HOST' < /etc/nginx/conf.d/cvat.conf.template > /etc/nginx/conf.d/default.conf && nginx -g 'daemon off;'"

networks:
  default:
    ipam:
      driver: default
      config:
        - subnet: <IP_ADDRESS>/24

volumes:
  cvat_db:
  cvat_data:
  cvat_keys:
  cvat_logs:

cvat/cvat_proxy/conf.d/cvat.conf.template

server {
    listen       80;
    server_name  _ default;
    return       404;
}

server {
    listen       80;
    server_name  ${CVAT_HOST};

    proxy_pass_header       X-CSRFToken;
    proxy_set_header        Host $http_host;
    proxy_pass_header       Set-Cookie;

    location ~* /api/.*|git/.*|opencv/.*|analytics/.*|static/.*|admin(?:/(.*))?.*|documentation/.*|django-rq(?:/(.*))? {
        proxy_pass              http://cvat:8080;
    }

    location / {
        proxy_pass              http://cvat_ui;
    }
}

Ran containers with docker-compose up -d
Navigated to http://cvat.ai-lab.edu:8080 in Chrome and got a "This site can’t be reached. The connection was reset." page.

INFO: However, if I navigate to http://cvat.ai-lab.edu I get the default Nginx CentOS landing page.

 No
Operating System and version (e.g. Linux, Windows, MacOS): CentOS v7
Code example or link to GitHub repo or gist to reproduce problem: I tried my best to follow the  CVAT Installation Instructions.

Additional Logs

Output from docker exec -it cvat_proxy nginx -T
docker_exec_it_cvat_proxy_nginx_T.txt

Output from docker exec -it cvat_ui nginx -T
docker_exec_it_cvat_ui_nginx_T.txt

Output from docker logs cvat_ui
docker_logs_cvat_ui.txt

Output from docker exec -it cvat_ui cat /usr/share/nginx/html/index.html

<!DOCTYPE html>

        <meta charset="UTF-8" />
        <meta http-equiv="X-UA-Compatible" content="ie=edge" />
        <meta name="viewport" content="width=device-width, initial-scale=1.0" />
        <meta
            name="description"
            content="Computer Vision Annotation Tool (CVAT) is a free, open source, web-based image and video annotation tool which is used for labeling data for computer vision algorithms. CVAT supports the primary tasks of supervised machine learning: object detection, image classification, and image segmentation. CVAT allows users to annotate data for each of these cases"
        />
        <meta name="”robots”" content="index, follow" />
        <title>Computer Vision Annotation Tool</title>

Output from docker ps

Output from docker logs cvat_proxy returned nothing.

I am stumped... Please help if you can.

Compare output of these commands (from the machine with docker containers):
curl -v http://<IP_ADDRESS>
curl -v --connect-to cvat.ai-lab.edu:80:<IP_ADDRESS>:80 http://cvat.ai-lab.edu
curl -v http://cvat.ai-lab.edu

@azhavoro Could you please look here?

Compare output of these commands (from the machine with docker containers):
curl -v http://<IP_ADDRESS>
curl -v --connect-to cvat.ai-lab.edu:80:<IP_ADDRESS>:80 http://cvat.ai-lab.edu
curl -v http://cvat.ai-lab.edu

Here is output from curl -v http://<IP_ADDRESS>:
curl_v_<IP_ADDRESS>.txt
Here is the output from curl -v --connect-to cvat.ai-lab.edu:80:<IP_ADDRESS>:80 http://cvat.ai-lab.edu
curl_v_connect-to.txt
Here is output from curl -v http://cvat.ai-lab.edu:
curl_v_connect-to.txt
In all cases I seem to get the default CentOS landing page.

Hi @digiwang
I am not a network specialist, just have an idea. Do not you think it can be related with 2nd domain level cvat . ai-lab . edu?
I mean that it was not our use case before and not sure CVAT setups support it.
As far as I know "ai-lab.edu" should have its DNS table to resolve subdomain "cvat". Have you setup it?
Did you try to setup CVAT on host "ai-lab.edu"? (maybe with a different port)

Hi @digiwang
I am not a network specialist, just have an idea. Do not you think it can be related with 2nd domain level cvat . ai-lab . edu?
I mean that it was not our use case before and not sure CVAT setups support it.
Did you try to setup CVAT on host "ai-lab.edu"? (maybe with a different port)

@digiwang
What is hostname of the server where you have installed CVAT?
What is output for host -v user-Z170M-D3H.inn.intel.com. Does it match IP address of the server in LAN?

@
Author
digiwang Thank you for update!
Try to fetch content from cvat-proxy directly:

find IP address: docker inspect --format '{{ .NetworkSettings.IPAddress }}' cvat_proxy
run curl -v http://<IP>:80 (and other experiments with replaced <IP_ADDRESS> => IP)

get the default CentOS landing page

This is strange, because:
  cvat_proxy:
    container_name: cvat_proxy
    image: nginx:stable-alpine

"alpine" should not emit CentOS messages.
Please shutdown CVAT containers and check availability of port 80. You can reuse curl -v http://<IP_ADDRESS> from above.
Expected output is "Connection refused" or similar. If you still see "CentOS landing page" then you need to turnoff that process/service (not related to CVAT).
Use ss -lnpt to find process which occupies port 80 (root user is required to see process names).

@digiwang , I hope you solved the issue. Don't hesitate to ask help on our https://gitter.im/opencv-cvat/public


Making test pass in angular 1.5

Fixes https://github.com/ohjames/angular-promise-extras/issues/9

cool thanks, didn't realise circleci had Xvfb setup and chrome installed.

Welcome! :tada:

@ohjames Could you please create a new release with this changes ?


Plural types

Hey guys. Was wondering the proper use of pluralization for this library. It appears that the pluralization types are a bit off. I am wondering if this is something that could be merged in with a PR or if I should fork and use my own.
eg. French differentiates between 0, 1, and 2+. Czech differentiates between 1, 2, 3, 4, 5+.
These types don't quite line up with the plural types object that you have built in this library.
Thanks,
Elliott

Can you provide some examples?
http://www.unicode.org/cldr/charts/latest/supplemental/language_plural_rules.html#cs suggests that Czech does not in fact distinguish between 2, 3, and 4, for example, and similarly http://www.unicode.org/cldr/charts/latest/supplemental/language_plural_rules.html#fr does not imply that 0 and 1 are distinguished in French.

I need to add SL locale.

@xpepermint thanks, PRs (with docs, tests, and citations) welcome :-)

I believe the next release will include Slovenian (#113); Czech and French are correct already. Closing.


Store only one page object per URL

Gerben provided a nice deduplication framework, where we can flexibly define the rules on when and if to store duplicates.
I may be wrong about that, but It seems to me that we currently won't need to store a new object for every visit, as long as there is no versioning.
It also would make it simpler if we just had one version per page and could index all of them, instead of worrying which versions to index. We may want to store and search multiple versions later though.

 We accept bounties via ?utm_campaign=plugin&utm_content=tracker%2F59103681&utm_medium=issues&utm_source=github Bountysource.

New thoughts:

For now, we should only store one page object per url - the most recent. And add all other visits as visit objects.

Expected behaviour:
Whenever a page is visited/imported, that is already existent: replace old version of text/content AND freeze-dry with the most recent.
The freeze dry versions can easily get very big. one page can have anything between 100kb-2MB
So we should reduce the amount of freeze dry as much as possible.
Later we might also think of adding some more metrics to the decision to keep a freeze dry version in the first place. Maybe something related to how long a person stays on an article, or only when a user choses to store the freeze dry version manually.

fixed with #69


Trimble

🗣 Description

💭 Motivation and context

🧪 Testing

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[ ] Tests have been added and/or modified to cover the changes in this PR.
[ ] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Add a tag or create a release.

@TVACyberRisk Not seeing any changes with this PR.  Looking at commit history for your branch, it looks like with commit ecfe55d5816d52926b750d6459d0687b5d965d60 you erased the Trimble entry you are wanting to add?


Add console-based hex editors for binary formats

@mhils  I have tested it and it is now working.
But, on Windows, I faced a problem related to that shutil.which() sometimes cannot locate a hex-editor.
I have made some research related to this problem and found this workaround. But, I think it is dependent on the user machine using the tool and if he has changed the PATHEXT as this workaround suggests or not.
So, what do you think will be the best solution for this to continue my work :) ?

@mhils I have tested it and it is now working. But, on Windows, I faced a problem related to that shutil.which() sometimes cannot locate a hex-editor. I have made some research related to this problem and found this workaround. But, I think it is dependent on the user machine using the tool and if he has changed the PATHEXT as this workaround suggests or not. So, what do you think will be the best solution for this to continue my work :) ?

references:
https://stackoverflow.com/questions/56399696/shutil-which-not-finding-programs-without-appending-file-extension
https://stackoverflow.com/questions/61097406/jupyter-kernelspec-not-found-while-installing-iqsharp-however-it-exists-on-pat/


fix: expand/collapse through isOpen prop

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
It fixes #735
Fixes # (735)
Type of change
Please delete options that are not relevant.

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

Breaking changes
Please document the breaking changes if suitable.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

@rluders , I worked on this one but now I'm thinking: seems that Flowbite-React never wanted to expose isOpen and setOpen properties directly, based on what I saw in the code. Maybe this one can be treated as a feature instead of a bug? Up to discussion.

Marked as  bug and feature. @tulup-conner , what you think about this one?

@tulup-conner  also sorry for bad commiting type. I will be more precise and I'll re-read again the contribution guide to give the best following the contribuition rules.

@IuriPires could you please, instead merge the main use rebase instead? It is better for history tracking on git.

@IuriPires could you please, instead merge the main use rebase instead? It is better for history tracking on git.

Done :)

@IuriPires I think that it isn't actually working as expected, or there is something wrong, check it here: https://flowbite-react-3hzdjhhhs-themesberg.vercel.app/docs/components/accordion

@IuriPires I think that it isn't actually working as expected, or there is something wrong, check it here: https://flowbite-react-3hzdjhhhs-themesberg.vercel.app/docs/components/accordion (ps: also the docs sidebar isn't working after the change)

Hi there @rluders , could you confirm a thing? Main isn't work on my local env. I'm currently using node v18.15.0 (not building/serving Storybook) could you confirm this please? I've checkout older branches and it worked fine.
This is main:

This is a 3 week old branch

Sorry, @IuriPires just had time to reply to you now. So... Not sure what is happening. I'm using node v19.2.0 and I don't have any issues with the storybook.

@IuriPires I think that this one just needs a rebase, right?

@IuriPires I think that it isn't actually working as expected, or there is something wrong, check it here: https://flowbite-react-3hzdjhhhs-themesberg.vercel.app/docs/components/accordion (ps: also the docs sidebar isn't working after the change)

Or maybe not... 🤔

any progress on this?


Add support for custom method

I fixed the lint errors and moved the tests to a separate file.
But when I added a test for $that->trans it unfortunately appears as the setting { name: '$this->trans' } also matches $that->trans()

Add a check to validate that the method exists and fixed issue with functionsCall so it don't rember all functions and methods when you create a new instance of wpPot.


feature(client): disconnecting from the server without closing the client app

Is your feature request related to a problem? Please describe.
The problem is that I use more than one vpn client and I need to switch between them. I close the FireZone client each time to connect to another one. They have their own DNS services.
Describe the solution you'd like
When starting the application there is no auto connection to the server (can be configured)
We can click on the connect button and connect, we can also click on the disconnect button and disconnect.
Describe alternatives you've considered
-
Additional context
I'm using macOS, perhaps on Windows or Linux this can be done more easily.
P.S. These are 2 different requests, but they are possibly related. I decided to split them into two different ones

This would make dogfooding easier too. It could be done for Linux and Windows but I'll wait to see how this gets triaged


chore: refactor settings class to functional component

Description
This giant class was causing me all kinds of pains when i tried to implement the revoke phone number task in the sprint. Turning this into a functional component so that we can use hooks and reduce boilerplate.
No actual changes aside from adding an analytics event to track disabling analytics 😅
Test plan
n/a
Related issues

Fixes RET-717

Backwards compatibility
Y

@kathaypacific it looks like the confirm account removal modal isn't displaying after completing the backup quiz.

@kathaypacific it looks like the confirm account removal modal isn't displaying after completing the backup quiz.

@MuckT is there anything in this PR that could have caused that? i'm confused about what the flow should look like, and whether the behaviour already exists on main


Improve terranix module for GitHub

The module works fairly well, but there are some lacking features.
For one, there is no support for pages blocks, so any repository that has pages enabled is not able to be imported into the configuration as it will attempt to delete the resource (which fails in my experience, luckily)

#24 Added support for repos with github pages. I don't think this issue should be closed with only that. There are many more features that could be supported.


Enable association and disassociation of extra_credentials with job_templates

This PR is against tag/v3.1.8, since releases don't appear to be fully merged with master.
Borrowing the basic logic of associating labels to job templates, this PR adds the ability to associate and disassociate extra credentials beyond the initial machine credential for a job template.

The versioning idea is that master supports v2 API and branch v1 supports the v1 API.
extra_credentials are only really supported through the v2 API, so we will want this in the master branch (as you've done here). The only potential problem is that this was branched off of the v1 branch.
I'm going to merge the v1 branch into master and then check back on what git history this PR shows.

That didn't quite work as I hoped it would.
Could you try just re-opening this pull request? Other branches forked from the same point have no trouble when I try it, so I suspect something specific to the PR's history is at work here.
if that doesn't work, you could checkout the master branch and then git cherry-pick 4076b53d93dda39f49f5cdae55e2465f5a27f40b, and make a new PR from that.

I originally coded off master but then copied to v1. This would be easier if I just go back to my original branch. I'll close this PR and resubmit.


WIP IR-418: use installer bootstrapped storage config for Azure

I'm going to close this for now to not cause confusion - I'm not sure if we will pursue this. For now we'll ask customers to create the registry config manifest manually.
/close


When targets are ignored, only build delve if debug compose file is used

This PR fixes a hard-to-reproduce bug encountered when building the CDN in a Box images from within a Jenkins pipeline.
The bug is that Docker build targets are ignored, so even though dlv should not be installed in non-debug images, it ends up being installed in the trafficops, trafficmonitor, trafficstats, enroller, edge, mid-01, and mid-02 images. As a result, if there is any issue building dlv in this environment, building the entire image will fail.
This PR adds, in the optional/docker-compose.debugging.yml file, a DEBUG_ENABLE build arg to each of the compose services listed above. That way, even if build targets are ignored, dlv will not be built unless optional/docker-compose.debugging.yml is included in the docker-compose command.

Which Traffic Control components are affected by this PR?

CDN in a Box

What is the best way to verify this PR?

docker-compose -f docker-compose.yml -f optional/docker-compose.debugging.yml build --build-arg DEBUG_ENABLE=false trafficops

then
docker-compose -f docker-compose.yml -f optional/docker-compose.debugging.yml build --build-arg DEBUG_ENABLE=false trafficops

Finally,
docker run --rm -it trafficops sh -c dlv

expected output:
sh: dlv: command not found

If this is a bugfix, which Traffic Control versions contained the bug?

PR submission checklist

[ ] This PR has tests
[ ] This PR has documentation
[ ] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

The target-ignoring bug described above is no longer encountered if you set DOCKER_BUILDKIT and COMPOSE_DOCKER_CLI_BUILD=1 in the environment. Closing the PR as no-longer-necessary


Two recipes?

Any idea why am I getting two identical recipes registered?

They both have different IDs.
Using 1.12.2-<IP_ADDRESS>.

They are not identical, one has the Restone and Glowstone rotated. This is not a bug.

Overlooked it, whoops. Thanks!


TypeError: unsupported operand type(s) for -: 'numpy.ndarray' and 'Tensor'  when runnerf on real images

hi, hashNerf works well on synthetic images, however, it doesn't work on nerf_liff_data, such as fern.
I run
python3 run_nerf.py --config configs/fern.txt --finest_res 512 --log2_hashmap_size 19 --lrate 0.01 --lrate_decay 10
it do can train. but when it supposed to render, i guess, there's an error.
TypeError: unsupported operand type(s) for -: 'numpy.ndarray' and 'Tensor'

and line 169 is

did anybody meet the same error, help me
thanks!!! orz

hi, hashNerf works well on synthetic images, however, it doesn't work on nerf_liff_data, such as fern.
I run
python3 run_nerf.py --config configs/fern.txt --finest_res 512 --log2_hashmap_size 19 --lrate 0.01 --lrate_decay 10
it do can train. but when it supposed to render, i guess, there's an error.
TypeError: unsupported operand type(s) for -: 'numpy.ndarray' and 'Tensor'

and line 169 is

did anybody meet the same error, help me
thanks!!! orz

已收到邮件


Performance bump in PSTH

Single-threaded speedup through JIT histograms and parallelization over neurons. This introduces numba and joblib dependencies, but there is an option to turn off parallelization. If it's turned off, joblib is not imported. -1 option means use all available threads.
On Steinmetz Neuropix dataset, this gets a 3x single-threaded speed increase and theoretically linear speed increase with the number of threads
Tests are passing, except for strf:  TypeError: object of type <class 'float'> cannot be safely interpreted as an integer. but I think that's unrelated since this PR does not deal with ML part. Edit: Fixed that test, indeed it wasn't related to this PR.

Currently the failing test seems to do with tensorflow version. It's probably enough to pin TF to 2.2


Blazor

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you give details about the issue you're having?


Countdown being reseted due long time in background

Hello there! How being doing?
Thanks for this awesome pod! :)
I'm currently having issues while running a countdown at background. If somehow the counter is more then 20 minutes or so, it resets when i get back to the app... How can i avoid this?
Best!
Rafa

I'm seeing strange things too when the timer goes in the background for 20+ mins. To get around this, I had to do this:
    override func viewDidLoad() {
        countdownTimer = MZTimerLabel(label: myLabel, andTimerType: MZTimerLabelTypeTimer)

        // When app becomes active, re-run to refresh timer
        NSNotificationCenter.defaultCenter().addObserver(self,
            selector: "updateCountDownTimer",
            name: UIApplicationWillEnterForegroundNotification,
            object: nil)
    }

    override func viewWillAppear(animated: Bool) {
        timerDate = //some date
        countdownTimer.setCountDownToDate(timerDate)
        countdownTimer.start()
    }

    func updateCountDownTimer() {
        // Countdown looses its place after app leaves foreground for 20+ mins
        if timerDate > NSDate() {
            countdownTimer.reset()
            countdownTimer.setCountDownToDate(timerDate)
            countdownTimer.start()
        }
    }

hey @basememara thanks first. But is that anyway i can bundle your workaround into MZTimerLabel that not requires any further custom implementation from user? otherwise i will try to put this into readme ;)


Adding the enlilviz package.

I'm adding in the enlilviz package to the projects list. I went through and added a quick self-evaluation as well.
Community Standard
I am currently the only contributor, so I'm calling this partially met because of that. It is fully open source under an MIT license so anyone can contribute.
Testing Standard
I have automated tests, but have not been rigorous on testing every individual unit of the code. It could be improved, so I'm calling this partially met.

Looks good, thanks Greg!


Does not return the text written certain font

Hello. I can't to parse some pdf files. These have some text written fonts such as
https://drive.google.com/folderview?id=0BzDb0f_tVYoAVFFual9KS0VEbjg&usp=sharing
After parse i got text = "", and font = nil.
I checked 200 different fonts and only those have caused problems
pdf-reader version  1.3.3
PDF::Reader.open("GulimChe.pdf") do |reader|
    reader.pages.each do |page|
        font = page.fonts[:F1][:BaseFont].to_s
        text = page.text
    end
end

I think I'm seeing this problem (or something like it?), for Raleway SemiBold and Raleway Regular. When using either of those, reader.pages.map(&:fonts) returns an array of all {} values, and pages.map(&:text) has all empty strings. It seems as if those typefaces lead to the text acting as if it's completely missing. Switching to an unspecified typeface value of "sans-serif" (through font-family: sans-serif; with a HTML -> PDF generator) makes the text available.

Apologies for taking so long to review this - it seems the sample file is no longer available.
I'm going to close the issue for now, but feel free to re-open it with additional info if required.

Cambria Math.pdf
Dotum.pdf
DotumChe.pdf
GulimChe.pdf
It it files from issue.

Thanks for the sample files @flaminestone.
I've checked Cambria Math.pdf and GulimChe.pdf. Neither has a font in the page resources, and I suspect the tool that generated the files has converted the fonts to vector lines. If that's the case, then no tool will be able to extract the text.
Have you tried copying the text in adobe acrobat? I suspect that will have the same issue.


Port Alerts missing basic information

There alert for ports on the flash array is are missing names/descriptions, providing no context to the engineer in scom.

I believe this is addressed in <IP_ADDRESS>, but I'll check. If not, it will be in an upcoming release.

Resolved in <IP_ADDRESS>.


Cloud function to import scenario test scores

What's included?
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
Risk - how likely is this to impact other areas?
🟢 Low risk
Additional context

Discussed as part of the QT II epic today. It was agreed that this ticket was a priority to be included in April's work.

After feedback in the Standup this issue is now on hold

@HalcyonJAC  Based on discussions at the 02/07/21 Digital Stan-up, we will not need to continue with this work due to Dries work on the Merit list and UI to enter scored directly onto the platform.


Fatality on iPhone 7

Hello,
I am using v0.7.0
applying the following lines below:
//spacing:2
V:|~[box]~|
H:|-(20)-[box(170)]~|
V:[box:-[r0(40)][r1(r0)][r2(r0)][r3(r0)][r4(r0)]-[r5(r0)][r6(70)]-[r7(r0)][r8(r6)]-]
H:[r1:-[x0]-[y0(x0)]-]
H:[r2:-[x1]-[y1(x1)]-]
H:[r3:-[x2]-[y2(x2)]-]
H:[r4:-[x3]-[y3(x3)]-]
H:[r6:[A][B(A)]]
H:[r8:[C][D(C)]]

on this editor :
https://rawgit.com/IjzerenHein/visualformat-editor/master/dist/index.html?vfl=example
result is :

so that is pretty fine, until I try it on iphone7 and my heart gets broken into little tiny pieces 💔
I tried many variances but it always threw these errors on iPhone7

Undefined is not an object (evaluating 'a.isResricted')
Undefined is not an object (evaluating 'a.hashCode' )
https://github.com/IjzerenHein/autolayout.js/issues/21 (same issue same workaround so no problem)

So it could be either me writing it the wrong way or there is a bug in the library. or bitter truth both of us which is most likely to be happening.
Also, my vfl could be messed up, it will be much appreciated if you could tell me a better way of writing it as am not so vfl savvy
Thank you in advance,
keep the good work up 👍

Hi, thanks for letting me know of the problem. I'll get back to you later when I have more time on my hands. As for the hashcode problem, please take a look at my last message in this thread https://github.com/IjzerenHein/autolayout.js/issues/22
cheers


Incorrect type warning on object parameters of type { [key: string]: any; }

This issue occurs when coding with a definition file that declares a function which takes a parameter object with properties like:
// in d.ts file:
propertyName?: { [key: string]: any; };

When passing this parameter as a plain object, VScode is happy:
// from javascript - no warning
functionCall({
    propertyName: { foo: "bar" }
})

But I get a warning when doing the same thing thusly:
// from javascript - VScode shows warning
var obj = { foo: "bar" };
functionCall({
    propertyName: obj
})

The warning is:

Types of property 'propertyName' are incompatible. Type '{ foo: string; }' is not assignable to type '{ [key: string]: any; }'.

To reproduce, try putting ractive.d.ts into a project, and then enter code like this:
// ok:
var a = new Ractive({
	partials: {foo: 'bar'}
});

// ng:
var obj = {foo: 'bar'};
var b = new Ractive({
	partials: obj
});

Closed with the adoption of Salsa (https://code.visualstudio.com/Updates#_languages-javascript).


Solana Pay message signing

Edit: The information below is no longer up-to-date. A current version of the proposed changes can be viewed at #169.
All feedback is appreciated. I can be reached on telegram @samhogan.
Original Issue
Solana Pay should support message signing via HTTP request. The following is an attempt to summarize how a dapp can request that a wallet sign a message via Solana Pay.
The flow is as follows:

The user scans a QR code or taps NFC.
The wallet parses the link from the QR code and makes a GET and  POST request in accordance with the Solana Pay transaction request specification.
The server responds with the following fields in the request body:
The <data> field is the data that will be signed. It must be a base64-encoded value adhering to the Proposal for off-chain message signing solana#26915 specification once it is finalized.
The <state> field is a MAC that the wallet will pass back to the server in order to verify that the contents of the <data> field were not modified before signing.
The <message> field is an optional UTF-8 string value to display to the user.

{
  "data": "aGVyZS1pcy1hLW1lc3NhZ2UtdG8tc2lnbg==",
  "state": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzaWduYXjF9.43WZK82a_sGU-ImVvCUnMJmNprs6Fe30pm0",
  "message":  "Login with Solana Pay"
}

The wallet displays the message and asks the user if they would like to sign the data.
If the user signs the data, the wallet makes a second POST request to the same url with the following fields in the request body:
The <account> field is the base-58 encoded value of the user's public key
The <state> field is the same as the <state> field from the first POST request response body. It should be returned to the server unmodified.
The <signature> field is the base-58 encoded signature from signing the <data> field with the users private key

{
  "account": "C9uYZinjZmmqxaF7FENdmzMVMuNVqRkKHXpbmd933W98",
  "state": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzaWduYXjF9.43WZK82a_sGU-ImVvCUnMJmNprs6Fe30pm0",
  "signature": "CUghXMN18pTQPoxm9zmrQY1PQXctp3xrGHpXxSJd1UYkRcBQBaUetZwKuc57VgjwjH7cC1Zbm6t1Zz1WJkVBMnW"
}

The dapp server will decode the <state> field to ensure that it has not changed and then attempt to verify the <signature> against the content from the <state> field and <account> field. If the signature is verified, the server knows that the wallet controls the private key for the <account>.

Some thoughts --

adhering to this specification.

Bad link, I assume referring to https://github.com/solana-labs/solana/pull/26915?

Since message is used in transaction responses too, data makes sense to differentiate. But signedMessage when the data is what gets signed is a bit confusing. signedData could work though. I'll use it to refer to this concept.

On that subject, however, what are the use cases envisioned for message to be provided alongside data? The argument for including it with transaction is more obvious.

The  field is the same as the  field from the first POST request response body

signedData may not be the same if it was prefixed (which is why we include it here) but arguably we could do away with this entirely if we specify in the spec that the data must be prefixed according to https://github.com/solana-labs/solana/pull/26915. This requires that spec to become finalized, though this is a major blocker for Mobile Wallet Adapter 1.0, so I would expect this to happen soon.

signature should be specified as the base64 encoded signature bytes (many wallets use base58 for message signatures as well as tx signatures).

We probably need to think about how practical a stateless design for this is. If a malicious client modifies the data, then posts the modified signedData along with its signature, the server has know way of knowing without persisting something. It could persist it statefully by the account, or persist it statelessly by adding some message authentication or signature of the data to the data itself, or perhaps in another field that must be returned.

/* req 1 */ { "account": "base58" }
/* res 1 */ { "account": "base58", "data": "base64", "mac": "base64" }

/* req 2 */ { "account": "base58", "data": "base64", "mac": "base64", "signature": "base64" }
/* res 2 */ 200 OK

An overall downside of all this is increasing complexity of the protocol, mostly from the API server's end. Servers need to be pretty smart about what they put in the message (because it can't be modified/corrected), how they authenticate it (because they need to generate MACs and verify MACs as well as the signatures), how they store private keys (to generate MACs), and how to prevent replay attacks.

This might be okay for the protocol to give it more flexibility, but it makes me wonder if we will want to just implement a full authentication solution OOB, because it's the most obvious use case, every server is going to have to create their own, and many will end up half-baked and unsecure.

Another possibility is to take a very different direction, something like Glow's signIn() functionality. Instead of having the server generate a message to be signed, we have the wallet generate a stock message with a URL, account, timestamp, and nonce, sign it, and provide that to the server. This has numerous pros and cons. One pro is that it can be done in one hop, because the server doesn't need to know the account first. One con is that this likely requires a new URL spec (can't just blindly sign in to tx request URLs).

Tagging @jnwng for feedback, he's been interested in this as well.

I changed <signedMessage> to <signedData>. This makes much more sense.

On that subject, however, what are the use cases envisioned for message to be provided alongside data? The argument for including it with transaction is more obvious.

Including <message> allows dapps to add context to the message signing request. For example, depending on the action a user is taking, a dapp may send {"message":  "Login in with Solana Pay"} or {"message":  "Confirm address ownership to continue"}, or whatever makes sense in that context. It also keeps the spec more inline with transaction requests, which Solana Pays devs are already familiar with.

signedData may not be the same if it was prefixed (which is why we include it here) but arguably we could do away with this entirely if we specify in the spec that the data must be prefixed according to https://github.com/solana-labs/solana/pull/26915. This requires that spec to become finalized, though this is a major blocker for Mobile Wallet Adapter 1.0, so I would expect this to happen soon.

I like the idea of adhering to this spec as it offers consistency within the ecosystem. This means that <signedData> would contain only the <message> (bytes 0x14..n) from the <data> payload, I think.

signature should be specified as the base64 encoded signature bytes (many wallets use base58 for message signatures as well as tx signatures).

Updated the initial issue to reflect this.

An overall downside of all this is increasing complexity of the protocol, mostly from the API server's end. Servers need to be pretty smart about what they put in the message (because it can't be modified/corrected), how they authenticate it (because they need to generate MACs and verify MACs as well as the signatures), how they store private keys (to generate MACs), and how to prevent replay attacks.

This might be okay for the protocol to give it more flexibility, but it makes me wonder if we will want to just implement a full authentication solution OOB, because it's the most obvious use case, every server is going to have to create their own, and many will end up half-baked and unsecure.

It may make sense to keep the Solana Pay spec relatively simple and create a separate package, or another package within Solana Pay,  to encapsulate the server-side complexity involved in creating data values that properly hold a signature of themselves, a timestamp to prevent replay attacks, etc. This package could also contain an OOB authentication solution. While most dapps would likely opt to use this package, it would be possible for a developer to write their own message signing solution that tracks data values in another manner.

Another possibility is to take a very different direction, something like Glow's signIn() functionality. Instead of having the server generate a message to be signed, we have the wallet generate a stock message with a URL, account, timestamp, and nonce, sign it, and provide that to the server. This has numerous pros and cons. One pro is that it can be done in one hop, because the server doesn't need to know the account first. One con is that this likely requires a new URL spec (can't just blindly sign in to tx request URLs).

This is an interesting idea that deserves more exploration. My initial thought is that it's a fairly significant departure from how transaction requests work. I'm not sure saving a single hop is worth changing the URL spec. I'll noodle on this item a bit more over the next couple days. A few questions:

Is there any downside to disallowing servers from creating their own data values?
Assuming we want to include a message field for context, how would that get encoded in the URL?

3. This means that <signedData> would contain only the <message> (bytes 0x14..n) from the <data> payload, I think.

Ah, what I meant is that signedData will always be equal to data if we do this, so we would just have (ignoring MAC for this example):
/* req 1 */ { "account": "base58" }
/* res 1 */ { "data": "base64" }

/* req 2 */ { "account": "base58", "data": "base64", "signature": "base64" }
/* res 2 */ 200 OK

It may make sense to keep the Solana Pay spec relatively simple and create a separate package, or another package within Solana Pay, to encapsulate the server-side complexity involved in creating data values that properly hold a signature of themselves, a timestamp to prevent replay attacks, etc.

The server needs to be able to know that data hasn't changed between res 1 and req 2. data may also need to be human-readable, and length is an important constraint. We don't need the user to sign the MAC, so we shouldn't put it in the data, but we do need the MAC, because we can't trust the data without it.
This suggests to me another field that's just passed back and forth as-is. It's intentionally opaque to the user and wallet. They don't need to see this value because they aren't signing it, and we don't care how long it is (within reason).
For example, mac = encrypt({ account, data, timestamp, nonce }, secret), which is passed to the wallet, then returned as-is and decrypted. We shouldn't specify what goes in the value at the level of the spec, but we should provide a field in the spec (maybe not called mac) that lets you pass arbitrary state back and forth.

My initial thought is that it's a fairly significant departure from how transaction requests work. I'm not sure saving a single hop is worth changing the URL spec.

I'm not sure neither, but it's an interesting constraint. We should try to understand what use cases there are for message signing besides "sign in".

before i saw this issue, i was working on a proof-of-concept of a server-side verification flow that accommodates various efforts to standardize the semantics of "Sign-in with Solana". i was primarily focused on creating a reference example of a verifying the encoding of a signed transaction without submitting it to the network (workaround for Ledger), but wanted to bake in some of the in-flight thoughts here just to try them on for size.

The server needs to be able to know that data hasn't changed between res 1 and req 2.

definitely agree. the strategy outlined to handle the machine encryption seems suitable, although in my reference i skip it. i can give it a try and see how it feels!
one pattern that i thought felt rather clean... instead of having a third hop to the server-side API, i baked in the message to sign in the GET request as a third parameter. since my reference client-side isn't using Solana Pay, i'm not sure how this affects the user workflow (haven't thought about it yet), but subsequently the POST to the URL could have the signature and go through the verification flow.
this means that:

the server generates the message to verify too (which follows the CAIP-122 standard)
the server response doesn't need to hold a transaction; we're already done at this point (and some server-side behavior like linking to a user account has completed), allowing us to omit the transaction return field define some other behavior

open to critique here! again, haven't really thought through the GET flow, although it made this client-side implementation fairly straightforward

OTOH, this is also a lot of work to subvert the gas fee. in the short-term submitting a transaction with Memo using a nonce and validating with Solana Pay is a reasonably inexpensive process to execute, provided it is backed up on the server with a cookie / JWT that the client can use afterwards

The server needs to be able to know that data hasn't changed between res 1 and req 2. data may also need to be human-readable, and length is an important constraint. We don't need the user to sign the MAC, so we shouldn't put it in the data, but we do need the MAC, because we can't trust the data without it.

Got it. OAuth supports passing a plaintext state parameter that allows you to validate to responses. Instead of mac, we may want to call this field state or something similar. Devs can technically put whatever they want in this value, though JWTs seem like a good fit here.

I'm not sure neither, but it's an interesting constraint. We should try to understand what use cases there are for message signing besides "sign in".

I did a 10 minutes of digging and authentication seems like the only active use-case so far. I'll put out some feelers and see if anyone is doing something more creative.
From @jnwng's proof-of-concept:

Primarily, this strategy can be used along with OAuth-related account management to "associate" multiple Solana wallets to an OAuth account.

This seems like a really good end goal for an OOB Solana auth solution, whether a user is using Solana Pay or a traditional wallet connection.

OTOH, this is also a lot of work to subvert the gas fee. in the short-term submitting a transaction with Memo using a nonce and validating with Solana Pay is a reasonably inexpensive process to execute, provided it is backed up on the server with a cookie / JWT that the client can use afterwards.

The Solana Pay dapps I've seen will usually pay these gas fees as requiring your users to pay to sign-in is not great UX. This opens the dapp up to attacks where a malicious actor repeatedly requests memos and drains the dapps fee paying account. Maybe not a huge concern in the short term. Offering real message signing based on this proposal also provides a more consistent dev experience.
Here's a reference implementation of how we do Solana Pay authentication in Bedrock using memo transactions. This has only been used for toy apps so far but currently we don't pay fees for sign-in to avoid this exact issue. Live version here.

OTOH, this is also a lot of work to subvert the gas fee.

While I agree, I view it as less about saving on fees and more about the UX benefits of enabling a lower trust threshold. Users must be more careful of signing a transaction than signing a message, so wallets have to handle the presentation differently.
It's also private, being off-chain. You may not want to put something on chain to symbolize that your users were physically somewhere at some time doing something.

OAuth supports passing a plaintext state parameter that allows you to validate to responses.

Yeah, this sounds about right. We should definitely learn as much from OAuth as we can here.

authentication seems like the only active use-case so far

It may be worth getting feedback from some POAP and event/ticketing protocols (Cupcake, SOAP, and Cardinal come to mind).

It's also private, being off-chain. You may not want to put something on chain to symbolize that your users were physically somewhere at some time doing something.

This is huge for live events. High profile consumers don't want to broadcast their location when scanning into an event.

It may be worth getting feedback from some POAP and event/ticketing protocols (Cupcake, SOAP, and Cardinal come to mind).

I reached out to Cardinal, SOAP, Disco, Decaf, and Fantastix. Feedback incoming.

Glad to see this being solved! There are a few ideas that this would make easier.

Event ticketing can be extended to "hold any of these nfts for free entry", where the user can enter the event/building by having a certain pre-existing NFT (not explicitly ticket). With signing, the host can authenticate if the person holds a certain SOAP or membership NFT. I could imagine an iPad with a dynamic QR code that the user scans (through regular camera or within wallet), and signs a request. The host then can check on chain if the person indeed owns a certain NFT or not.

Without signatures, the only possible way rn is memo tx (which is very unoptimal) or the host scanning a qr on the guest's phone (qr as image of the ticket NFT) which brings almost no added benefit to NFT ticketing.

IRL raffles can be made possible by users scanning a QR, authenticating and the host storing their entry. (This would need to have some form of "allowlist" to not let a single person enter with any number of wallets.)

How this gets implemented on a technical level is out of my scope of expertise, but this allows for a lot of yet undiscovered use cases.

glad to see this being discussed! Im from Monstrè @longeye_monstre, we built Solana Pay powered NFC Gift Cards for the hackathon. This is definitely not my area of expertise but keen to share some thoughts.
Another use-case outside of events (but still falls under authentication) is wallet to wallet messaging. Dialect protocol would benefit greatly from this, imo. Will be dming you @samheutmaker on tg!

Really cool to see this being worked on! One concern I have is do we want to specify whether/how apps can 'reject' a sign request?
For example if I have an NFT-gated part of my app and somebody sends the POST request in this spec, I get their public key at that point. I could look up whether they have any of my NFTs, see they don't and send back a 403 instead of letting them authenticate with that key and then having my own in-app logic to check their access afterward.
Is that a valid way for an app to behave? Or should a compliant app accept all signature requests with this scheme and only check privileges later? I think it's more obvious with browser wallets because AFAIK an app can't get in the middle of the connect flow and throw in their own checks, but this specification explicitly gives them an identity and ability to fail at that first (or second) hop.
This probably applies to transaction requests too ATM, but there are probably more use cases for "you don't have access" than "you can't perform a transaction".

For example if I have an NFT-gated part of my app and somebody sends the POST request in this spec, I get their public key at that point. I could look up whether they have any of my NFTs, see they don't and send back a 403 instead of letting them authenticate with that key and then having my own in-app logic to check their access afterward.
Is that a valid way for an app to behave?

Yes, this is valid and expected. The spec doesn't specify the structure of error responses (yet) but it does specify that the wallet must handle 200 and 3xx-5xx responses: https://github.com/solana-labs/solana-pay/blob/master/SPEC.md#post-response

For example if I have an NFT-gated part of my app and somebody sends the POST request in this spec, I get their public key at that point. I could look up whether they have any of my NFTs, see they don't and send back a 403 instead of letting them authenticate with that key and then having my own in-app logic to check their access afterward.

Keep in mind that a user is not authenticated until they sign the message regardless of whether or not the address you received passes a token gate you may have set up. The user must sign the message to prove they hold the private key to that address.
Given this, it might be helpful to think of authentication and authorization as two separate steps. It may be better to let them sign the message (authentication) and then block access if they fail to pass the token gate (authorization).
The actual specification will leave it up to you to decide what is right for your dapp.

This has been specified here: https://github.com/solana-labs/solana-pay/blob/master/message-signing-spec.md
This is still alpha and unimplemented. If you're following this issue or have a use case for this, please review this spec!


Fix potential null pointer dereference

The original code first conditionally dereferences the pointer, then asserts they are not null and then later unconditionally dereferences them:
*out_quads = (CvCBQuad*)cvAlloc((root->total+root->total / 2) * sizeof((*out_quads)[0]));
*out_corners = (CvCBCorner*)cvAlloc((root->total+root->total / 2) * 4 * sizeof((*out_corners)[0]));

which yields undefined behavior. The intent of function is to unconditionally dereference them (with a pre-assert).
This was found with Cppcheck.

I don't see any problems with the original code. You simply swap two pieces of code.

@ilya-lavrenov The compiler is allowed to remove the checks from the original code because of unconditional pointer dereference in later code which means it can remove the checks and the asserts.

@ilya-lavrenov Because cv::error() is not attributed with "noreturn" attribute the compiler cannot understand that if either of the pointers is null control does not proceed to the lines with unconditional dereference. This means the compiler can assume the dereference always happens and so pointers are never null and remove the asserts.

Actually the same problem is with the changed code.


Small tooling improvements

Issue
NA
Description
Small tooling improvements after a few days of use of both renovate and sematic release:

add a badge in readme with a link to the download page
update Renovate config to limit CI runs

Gravitee.io Automatic Deployment

🚀 A prerelease version of this package has been published: <IP_ADDRESS>53

:tada: This PR is included in version 2.1.0 :tada:
The release is available on:

2.1.0
GitHub release

Your semantic-release bot :package::rocket:


Select2 url mutable

With the built in select2 functionality, the data-autocomplete-light-url can be read and modified dynamically.
For Example:
var companySelect = $("select[name=company]");
var deliveryAddressSelect = $("select[name=delivery_address]");
companySelect.change(function (e) {
    var url = "/addresses/auto-complete?company_id="
        + e.target.options[e.target.selectedIndex].value;
    deliveryAddressSelect.attr("data-autocomplete-light-url", url);
});

If the company changes in the form, I update the query set of the addresses with the help of the query string. Like this I only get the delivery addresses possible for the given company.

Could we perhaps add the code example that's in your PR description into your commit ?

Sure thing.

K, please ping me when you update the commit ;)
git commit --amend
git push -f

A little example in documentation would be really kewl to share some of your insight :)


WIP: Fix testsuite

Changes Unknown when pulling 2f4b084ad53dd6091aaa37c70e17b3048c675567 on master_fixtests into ** on master**.

Changes Unknown when pulling 01680ad40cd2987d14f4980b961b7dfc82255abd on master_fixtests into ** on master**.

Changes Unknown when pulling 81110874e2c750256292f490c5b43a859764e8d4 on master_fixtests into ** on master**.


LSPS2/README.md: Clarify requirements for MPP support.

As requested in the telegram group.

Updated: fail all parts once first part times out.

Guess this is good

OK, merge in from me.

Do we merge this in or...?


Update graphql server example to use ninjs 3.0

https://github.com/iptc/newsinjson/blob/events-planning/graphql/server.js needs to be updated by hand to match the 3.0 schema.

https://github.com/iptc/newsinjson/pull/209


No trailing spaces in comments

The comment formatter allows you to specify a prefix for comments. Setting this to something like //  results in the following when formatting "foo\n\nbar\n":
// foo
//
// bar

The middle line has a trailing space from the specified prefix.
How can I not have this? For example:
// foo
//
// bar

Is there a way to have a prefix for lines that contain characters and another for empty lines?

I see. Currently, there is no way to do that, but I believe we should extend the semantics of the trim configuration parameter to also trim empty comment lines on the right.


Renamed kpm restore to dnu restore

Should be dnu restore since dnx is used below in stead of kvm too. kpm is confusing.

Hi @henkmollema, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft Open Technologies, Inc. and real humans are currently evaluating your PR.
TTYL, MSOTBOT;


OSError: read error

When running the example.py file, I first have to change line 8 from
found_sub = KeyboardHelper.get_subdevice(0x951, 0x16e5, 1)
to
found_sub = KeyboardHelper.get_subdevice(0x951, 0x16dd, 1)
to get the correct device.
An error is then thrown at kb.start_thread(), giving me this error:
Exception in thread Thread-7 (__thread_loop):
Traceback (most recent call last):
  File "C:\Users\Family\AppData\Local\Programs\Python\Python310\lib\threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "C:\Users\Family\AppData\Local\Programs\Python\Python310\lib\threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\Family\AppData\Local\Programs\Python\Python310\lib\site-packages\hyperx_keyboard_rgb\keyboard.py", line 78, in __thread_loop
    self.__send_feature_report(
  File "C:\Users\Family\AppData\Local\Programs\Python\Python310\lib\site-packages\hyperx_keyboard_rgb\keyboard.py", line 90, in __send_feature_report
    read = self.hid.get_feature_report(0, 0x64)
  File "hid.pyx", line 346, in hid.device.get_feature_report
OSError: read error

I'm unsure if this is a problem with me changing the subdevice pid. I am not able to fix this.

What happens when you try other interfaces?

What happens when you try other interfaces?

The two available interfaces for the keyboard's subdevices contain one 0, and four 1s. All throw the same error.

What happens when you try other interfaces?

Could it be a problem with how my keyboard is set up? Does it have to be in a certain state?


Raise an error when the interaction is invalid

Check the interaction is defined correctly here. If the interaction is
not defined correctly, that can cause errors in pact verification in
Provider test. For example, "undefined method capitalize' for nil:NilClass" at #interaction_description_for_rerun_command`.
I sent follow up PR to pact-support repository: https://github.com/bethesque/pact-support/pull/24

Thanks!

I saw af0fde8ea1124920316a6e1a4219648602dae078, but it seems pact v1.10.0 was not released on rubygems.org. Did you intend to?

Sorry, I thought I had, but must have gotten distracted. Done now.

Thank you! 💎


Allow "custom" window.titleBarStyle with native context menus

It would be nice to have a setting to make context menus/right click menus native independently of title bar style. It allows users to take advantage of new command center.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

I submitted a PR to implement the feature. https://github.com/microsoft/vscode/pull/154544

The reason for this feature request is that with titlebar set to native, I can't use the new command center with native context menus.

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

The other way (native title bar with custom context menu) should also workaround #113175, is that supported?

@yume-chan that would be a fantastic reason to implement this option


open special app on notify click

Hello! Is it possible (or possible to add the feature) that i can open a special app on clicking a special notification?
For example -> someone rings at my door -> Loxone sends notification to igotify -> i click the notification -> Loxone App should open.
I think hopefully its possible, but it is apple.... so i think its not. :-/
Maybe there is an alternative way (button on notification, ..)

It might be possible, but the feature misses the intended purpose of the app, which is to securely notify rather than redirect to other apps. I’ll think about the feature request and will mark the issue as LTFR for now.

Yes! I understand that its not the intended purpose - i just thought about redirecting all notifications to gotify for my smarthome (washing machine finished, doorbell, ...) - so i think gotify is also usefull for this (because... it is for notifications :) ) - the thing is, for example loxone is not really good in sending notifications, some people get the notifications, other not.
So i want to find a better solution. But when i now tell my wife that she have to open Loxone manually, the WAF (Wife Acceptence Factor) is not high! ;-)
So i think its a really cool (optional) feature to open a special app when there is a notification from tool X - so why not implement it when there is time :-) ? Users who don´t need it don´t have to use it 😇 i would really like!
But for the moment, i will set iGotify up for the notifications what don´t need a opened app after seeing it. :-) Thank you for this awesome work (i really like it!!)

Feature will be in the update 1.4.2


Add new call spots

Add new call spots with updated page variable names, new methods and new diagnostics.

This involves a large reorganisation of some variables. Given that we are trying to make this a stable version so we don't have to rerun data, let's try get as many notebook name changes out of the way as soon as possible.
So @paulshuker this is probably a good time to remove the background and use_preseq options from notebook pages.
I will also remove the absolute intensity percentile from call spots, as it makes more sense to compute this in OMP given that we load the entire tile in at that point. This also raises the point that we can compute a different intensity threshold for each tile which might be worthwhile!


Methods "only" and "without" do not work on nested docs?

I've got the document (book with nested chapters with nested sections):
book: {:title: "Book Title", "chapters":[{title: "Chapter I", "sections":[{},{}] }, {...} ]}
If I want to get book without chapters I say
books[0].without("chapters").chapters ==> [] #i've got empty array
books[0].chapters[0].without("sections").sections ==> [.....] # I've got full array

What is your books, and how'd you initialize it? I ask because I'm not sure how you're able to do books[0].without("chapters"). For me, that calls #without on a book instance, which raises a NoMethodError.

Thanks. It was my mistake. "Only" can be used only on collection docs not on embedded as I tried, i needed to write:
books[0].without("chapters.sections").chapters -> []
Closed (or even would be better if deleted)


Manual version add for SETI-CommunityTechTree

Netkan-bot fails because of AVC version

Ty @dbent

Passes local, passes jenkins. Merging.


lint for ARG001, ARG002

Ruff was complaining about ARG001, ARG002, unused function and method (respectively)
arguments. All of these cases were arguments inside of mocks, so this is fine behavior.
Only related to ruff linting, not argument of whether or not mocks are good LOLZ!

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#190  👈
#189
#188
#187
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @chrisaddy and the rest of your teammates on  Graphite


[VS Code] Joining collaboration session: Unable to connect to Live Share cloud relay. Check your proxy, firewall and network settings and try again.

Error:
Joining collaboration session: Unable to connect to Live Share cloud relay. Check your proxy, firewall and network settings and try again.
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.379

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.35.1

platformName
Windows

platformVersion
10.0.17763

Hi @bkhushang  , thank you for reporting the issue. Would you give us some more information to investigate ? Does this consistently happen for you or happened once ? Also , your logs may provide us with helpful information. You can access them via the "Live Share: Export Logs" command from the command palette (ctrl + shift + p) and attach them to the issue.

@Priya91 , possible dupe of #2267


Look into the possibility of not converting everything / anything

(If I'm not mistaken) PyAudio can play anything PyDub can convert
mp3-Tags are not really needed if the meta information for offline playlists is stored in a database anyway, which is arguably the more sensible approach anyway
songs added from the hard drive are not converted anyway
conversion more than doubles the time for loading a song, which impairs user experience

Downsides:
keeping files in the format it is downloaded in possibly increases needed disk space

In my opinion, the upsides definitely outweigh the downsides here

@FelixGail what's your opinion on this?

I agree to your point of converting the files.
Some thoughts on your arguments:

keeping files in the format it is downloaded in possibly increases needed disk space

most of the songs are taken from play music and are therefore in .mp3-format anyway.

mp3-Tags are not really needed if the meta information for offline playlists is stored in a database anyway, which is arguably the more sensible approach anyway

i agree that using a database to keep the song information is the best approach for a running bot, but i would store mp3-tags for all songs in mp3 format anyway for multiple reasons.

if the database is lost or corrupted the song information is not lost
Importing offline songs will require mp3-tag handling anyway since it is the best way to garther song information and saving them allows to (re)import them later or on another bot
It is possible to use the songs in any media player (except groove maybe)

most of the songs are taken from play music and are therefore in .mp3-format anyway

yup, I would also argue that disk space isn't a big concern nowadays anyway

Importing offline songs will require mp3-tag handling anyway since it is the best way to garther song information and saving them allows to (re)import them later or on another bot

Of course mp3-tags need to be read for that. But I would argue that only the playlist_manager.py needs to handle those tags and stores the information in the database.
Songs imported from the hard disk are not and should not be modified, moved or copied, so they aren't relevant for the question of saving meta-information of songs to their mp3-tags anyway.

It is possible to use the songs in any media player (except groove maybe)

I'm not sure how much we should endorse that. It circumvents Google Play Music's download limit for uploaded songs, and even enables downloading songs normally only streamable.
Legally the Play Music usage is questionable anyway, but that's kinda crossing a line for me...

if the database is lost or corrupted the song information is not lost

While that is technically correct, it's actually not an issue:

For songs from web-services (in online mode)

the information is not stored in the database (only in-memory)
we get the song information when the song is returned with search results or suggestions, so we never need to look up the information out of thin air

For songs in offline playlists

if the database is lost, we won't have a reason to look up song info for songs from the database, do we? (we 'don't know' about the songs existence without the database)

When not saving id3 tags it is not possible to import the songs from
another bot or after corruption, which would render all downloaded data
useless.
Which is not that big of a deal after thinking about it again while the
legality issue seems way bigger.
Not saving the tags is probably the better solution.

Importing a database with local (from hard disk) songs from another bot would also be very problematic, because the songs are not stored in the same directory as the downloaded songs.


[SPARK-9530] [MLlib] ScalaDoc should not indicate LDAModel.describeTopics and DistributedLDAModel.topDocumentsPerTopic as approximate

Remove ScalaDoc that suggests describeTopics and topDocumentsPerTopic are approximate.
cc @jkbradley

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

LGTM pending tests

Test build #1275 has started for   PR 7858 at commit b574923.

Test build #1276 has started for   PR 7858 at commit b574923.

Test build #1275 has finished for   PR 7858 at commit b574923.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #1276 has finished for   PR 7858 at commit b574923.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merging with master.  Thank you!


Fixed Typo (REQ -> REG)

Fixed REQ-QUERY-REQ, REQ-QUERY-REP, and REQ-ADD-REQ to use REG instead of REQ

I don't want to add another pull request, but EXEC-POST-REP should be EXEC-POSH_REP


Add missing commits after rebase and change to main-branch

Add commits from master -> main
Original:
https://github.com/tiiuae/px4-firmware/pull/278

Could these be upstreamed?

Sure, I can do that


Update dependencies & config for upload-file-via-browser demo.

Packages used in the upload-file-via-browser demo is too old.
The PR update all of them and fixed all the warnings.

Awesome! Thanks you @edsgerlin


test: 💍 add example on how to mock $queryRaw

✅ Closes: #731

:tada: This PR is included in version 1.26.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


class attributes are overridden by className attribute if the static part follows the dynamic part

Because we use a custom name for some attributes like className and htmlFor.
The isAttrOverridden logic fails if the dynamic part also have the same attributes as of the static part.
<div {{className: 'something'}} class="something-else"></div>

In this the className of div should be something-else but because of incorrect check the className will be something.
This bug will be there for htmFor attribute as well.

Fixed


Don't send scopes on token request

The docs at:
https://developers.google.com/accounts/docs/OAuth2InstalledApp#handlingtheresponse
don't mention that scopes should be sent for the token request. Doing so seems to work fine when only requesting a few scopes but when many scopes are being requested, a invalid_scope error is returned by the Google servers.

Have you yet signed Google's Contributor License Agreement? We can't accept your contribution without your having done so.


Option to sort by key order

This would really improve usability. I read your response on this, but wasn't convinced since users shouldn't care how the V8 engine works. The whole point of a JSON browser extension is to make JSON readable.
Example below showing a use case where consistent key order is really helpful (using a different JSON extension):

I arrived here because I was wanting this, too.
Motivating use case: you have a REST API that returns JSON, you have a pretty JSON view showing, you make a change to the backend, and hit reload in the browser to see what effect your change has, but everything changes because the object keys are in random order, so you have to completely reorient yourself in the new layout to figure out what actually changed.
This is a fun property in video games that generate random maps every time you visit a level. It is not a fun property in a software development tool.  :)
The nikrolls fork claims to have this feature, but I've just tested 0.10.0 from CWS (the last version, released 3 years ago) and reloading the page shuffles the keys each time.
As for the FAQ argument that fixing this is difficult and slow:

Slow: So make it an enabled-by-default option.
All algorithms are fast for small N, an observation usually made to warn against using inefficient implementations on small test data sets. However, it also works in inverse: if you know you're usually going to have small data sets, it's shortsighted economy to avoid an improvement just because it would slow the implementation down. If it goes from 1ms to 5ms, that's a 5x slowdown, but it's still an imperceptible delay to humans.
Those rare few using this plugin to pretty-print huge JSON data structures can disable the new feature.

Difficult: What are tools for, if not to save the user work? Investments up front pay off manifold on the back end. I realize we're proposing work for you to do here, but it seems clear from the issues and the fact that you call this issue a "FAQ" that many people want the tool to do this.


TTY shell breaks when another admin opens shell session

Describe the bug: (a clear and concise description of what the bug is)
It seems that when more than one admin users try to access the TTY Console, it will break for the one that created it first. This is probably related to the way the kubeconfig is generated.
Expected behaviour would be to have two separate instances that do not interfere with each other.
Admin account shell should not cause conflicts with other admin shells.
To Reproduce
Steps to reproduce the behavior:

Log in to otomi with an admin account
Click the shell button and wait for it to load
Check that you can see all namespaces k get ns
On another browser(or incognito mode) log in as another admin (create account in keycloak)
Click the shell button and wait for it to load
Now if you run again k get ns on the old shelll it will show an authorisation error.

Expected behavior: (a clear and concise description of what you expected to happen)
Expected behaviour would be to have two separate instances that do not interfere with each other.
Admin account shell should not cause conflicts with other admin shells.
Screenshots: (if applicable, add screenshots to help explain your problem)
Cluster(s):

Cloud: [e.g. aws]
Cluster: [e.g. demo]

Versions: (seen in console's /settings)

Core Version [e.g. 0.12.34]
Console Version: [e.g. aws]
API version: [e.g. 0.4.56]

Desktop: (if relevant)

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone: (if relevant)

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context: (add any other context about the problem here)

Hey team! Please add your planning poker estimate with Zenhub @Ani1357 @dennisvankekem @ElderMatt @ferruhcihan @merll


d3js blocked by Content-Security-Policy

When the server sends a restrictive Content-Security-Policy header,
Content-Security-Policy: default-src 'self'; script-src 'self'; img-src 'self'

the following error comes up in Firefox:
Content Security Policy: The page's settings blocked the loading of a resource: An attempt to call JavaScript from a string (by calling a function like eval) has been blocked

and a similar to that of Chrome:
Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".

The culprit is the following code, where a Function object is being created:
    dsv.parse = function(text, f) {
      var o;
      return dsv.parseRows(text, function(row, i) {
        if (o) return o(row, i - 1);
        var a = new Function("d", "return {" + row.map(function(name, i) {
          return JSON.stringify(name) + ": d[" + i + "]";
        }).join(",") + "}");
        o = f ? function(row, i) {
          return f(a(row), i);
        } : a;
      });
    };

The following replacement looks sensible to me, am I missing something?
    dsv.parse = function(text, f) {
      var o;
      return dsv.parseRows(text, function(row, i) {
        if (o) return o(row, i - 1);
        var a = function(d) {
          var r = {};
          row.forEach(function(name, i) {
            r[name] = d[i];
          });
          return r;
        };
        o = f ? function(row, i) {
          return f(a(row), i);
        } : a;
      });
    };

Best regards,
uu1101.
-- edit: fixed code

I have been trying to use D3.js at work, where there is very strict IT security policy.  I have not been able to use  at all.  Nothing at all shows up.  I tried referencing the D3 library in my local computer but still not working.
However, I found a very limited way to get version 3 (  to work in my work environment where every data has to be hard coded.  If I link data using d3.csv then it doesn't work again.   Having to hard code all the data makes it difficult to use this great library.
Can you please help?  Thanks!

@magicmathmandarin Have you tried UNPKG? For example:

v3: <script<EMAIL_ADDRESS>v4: <script<EMAIL_ADDRESS>v5: <script<EMAIL_ADDRESS>

Hi Curran,
Thanks for the suggestion.  These alternative references work in my
personal computer.  But they still don't work in my work computer.  My work
is a very large organization.  And yet I don't know anyone who has
successfully  used D3 :(
On Wed, Mar 28, 2018 at 3:18 AM, Curran Kelleher<EMAIL_ADDRESS>wrote:

@magicmathmandarin https://github.com/magicmathmandarin Have you tried
UNPKG? For example:

v3:
v4:

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/d3/d3/issues/1904#issuecomment-376786315, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVz5e2DvBRIciUYrAPWgrSEHYlmLc4SIks5tizlIgaJpZM4CCQ2V
.

Unfortunately this is still an issue for most companies with a CSP policy, I got this one today:
function objectConverter(columns) {
--
8057 | return new Function("d", "return {" + columns.map(function(name, i) {
8058 | return JSON.stringify(name) + ": d[" + i + "] \|\| \"\"";
8059 | }).join(",") + "}");
8060 | }

Related PR https://github.com/d3/d3-dsv/pull/87

I found that the latest version of d3.js contains innerHTML, which causes csp problems. Is there any solution?


Add context and links to What is a DRep FAQ

https://docs.sanchogov.tools/faqs/what-is-a-drep

Fixed


Timezones in UI are inconsistent

Flyte specifies timezones in UTC. In some places in the UI (inputs/outputs and schedules), we show the values in the local timezone. These should be updated to show the value in UTC to maintain consistency and avoid confusion.

Fixed by lyft/flyteconsole#98
Available in flyteconsole v0.12.1


Grid Search File Support

Current Grid Search is to run all combinations. For a flexible grid search, user can specify each parameters in a file, each one as line:
Like this file:
FeatureSubsetStrategy:ONETHIRD;MaxDepth:7
FeatureSubsetStrategy:0.5;MaxDepth:6
FeatureSubsetStrategy:0.5;MaxDepth:6:LearningRate:0.04
So 3 jobs will be run for grid search.

Done by junshi.


fix: Return no resources when array filter param is zero-length

Closing, will open PR with regression tests.


Minimum value for OS support should be 8.0

After we update both xCode to 11.5 and deeplinkkit to 1.5.1 we're not able to upload our builds to iTunes connect.
I believe target version should be bumped to 8.0

I think this key should be removed from the Info.plist entirely. From the docs:

Don’t specify MinimumOSVersion in the Info.plist file for apps built in Xcode. It uses the value of the Deployment Target in the General settings pane.


its not issue i just want to help you in this amazing app

Hi ,
I'd like to offer my help to improve your app's performance, remove boilerplate, and fix any functions that may need attention.
I have some experience with flutter and dart , and I believe I could make some valuable contributions to your project. If you're interested, please let me know how I can help.
Thank you, and I look forward to hearing back from you.
Best regards,

Hey there!
Thanks for reaching out and offering your help. I really appreciate it! Yes, you can contribute to this project.


[#4] added conditional intercept option

now you can also define a method intercept for jwtInterceptorProvider (by default it returns true for all) to skip the interceptor from working for falsy values returned
added tests for skipping auth
added example for intercept usage

irrelevant now? should we abandon this?

Sorry for letting this sit for quite some time, we're now catching up on older PRs and issues.
We've made a few changes since your PR came in. Are your proposed changes still required now that there have been some changes? If you're interested, please feel free to rebase :)
Thanks for your PR 🙌

oh, no idea tbh, it's been over two years now ;-)


Encrypting twice DELETES all contacts

Steps to recreate bug:

Encrypt address book with some contacts already in entry.
Reopen address book.
Add a new contact.
Encrypt address book again.
Reopen address book.
Decrypt address book.
Reopen address book.
All previous contacts except for the newly added one, is all DELETED.

Critical bug since it involves all data lost without any prompts to user.

Resolved withe error handling and throwing exceptions


feat(gas): do not store self-code in dictionary of child contract codes

This should reduce forward fees for Jettons, for instance.
Another possible optimization would be to remove the child contract dictionary for the common case when there are just one or very few child contracts and use cell references to store the child contract codes. But let's do it as a separate PR so we can check its impact too.

Issue
Closes #1012.
Checklist

[x] I have updated CHANGELOG.md
[x] I have added tests to demonstrate the contribution is correctly implemented: it reduces gas consumption for the common cases, makes storage improvements, etc. and does not have too much negative impact for the rest of the use-cases
[x] I have run all the tests locally and no test failure was reported
[x] I have run the linter, formatter and spellchecker
[x] I did not do unrelated and/or undiscussed refactorings

for some reason it increases fees for some transactions in initof tests

do you still have the exact numbers?
because it definitely decreased gas in the benchmarks:

Nice results! Let's merge


Need help in Crash log reporting on react native

Environment: Android
Steps to Reproduce: Integrate crashlytic and react-native-fabric and then inject one crash in the JS code
Expected Behavior: Application should be able to get the exact file name and line number where crash is occurring.
Actual Behavior
We are trying to integrate Crashlytic in our application (on React Native platform). It's providing complete log traces for the native side but we are facing issue in capturing JS originated exceptions. It's capturing JS exception but not providing complete information to find the crash location, file name etc.
We are getting following logs:
Fatal Exception: com.facebook.react.common.JavascriptException: undefined is not an object (evaluating '(void 0).message'), stack:
value@306:637
touchableHandlePress@165:2877
_performSideEffectsForTransition@167:8505
_receiveSignal@167:7292
touchableHandleResponderRelease@167:4757
@165:3277
g@41:296
invokeGuardedCallback@41:519
invokeGuardedCallbackAndCatchFirstError@41:634
A@41:2433
D@41:3195
U@41:2992
@41:15520
batchedUpdates@41:135761
Ie@41:14871
ze@41:15366
receiveTouches@41:16203
value@18:3582
@18:1067
value@18:3009
value@18:1039
   at com.facebook.react.modules.core.ExceptionsManagerModule.showOrThrowError(ExceptionsManagerModule.java:54)
   at com.facebook.react.modules.core.ExceptionsManagerModule.reportFatalException(ExceptionsManagerModule.java:38)
   at java.lang.reflect.Method.invoke(Method.java)
   at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372)
   at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:160)
   at com.facebook.react.bridge.queue.NativeRunnable.run(NativeRunnable.java)
   at android.os.Handler.handleCallback(Handler.java:790)
   at android.os.Handler.dispatchMessage(Handler.java:99)
   at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:29)
   at android.os.Looper.loop(Looper.java:164)
   at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:192)
   at java.lang.Thread.run(Thread.java:764)

So it's not providing information in which class, line number and method this exception has occurred.
I have tried to integrate react-native-fabric and react-native-fabric-crashlytic but still no luck.
There is one more issue in crashlytic, all JS crashes are collating as single type of issues although issues had occurred on different lines.

ExceptionsManagerModule.java line 54#2com.facebook.react.modules.core.ExceptionsManagerModule.showOrThrowError

1.0 (1)
9CRASHES
1USER

Yes all relevant information has been added


Could u explain the parameters in detail?

Hi, thanks for your sharing!
Follow your instruction, I've run your demo code(exp_run.py) successfully.
But I still get confused of some parameters(list in the below image) in it. Could you explain them more specifically?
Thanks!

Here is the result of your demo.

I notice that the rmse turns smaller along with the larger value of para:total_budget([1, 2, 4, 6, 8, 10]). Does this para represent the num of parties in FL system? If so, in my view, the rmse should turn larger since the more parties the lower accurancy. Could u help me to figure out it? Thanks.

Hi @hacql2004 ,
The explanation of some parameters is available in the Parameter section here. The other parameters are used for our previous testing and may not be useful for you. We may delete them in the future.
The total_budget is the total privacy budget (\epsilon in differential privacy). If the privacy budget is bigger, the privacy level is lower and the accuracy usually is higher.

Thanks for your quickly reply. I found here from your AAAI2020 paper "Practical Federated Gradient Boosting Decision Trees". It seems I need to read your "Privacy-Preserving Gradient Boosting Decision Trees" first.


Provide protection against attempting to create an md5 hash on null.

Provide protection against attempting to create an md5 hash on null.

Can you tag / release too please?

yup, give me 30 minutes

Sure. No problems.

thanks, it's published v0.1.10


sap.m.DatePicker calendar does not check minDate and maxDate values properly (month/year only selection)

OpenUI5 version:
Latest SAPUI5 version (https://sapui5.hana.ondemand.com/./resources/sap-ui-core.js)
Browser/version (+device/version):
Chrome 89.0.4389.114 (64 bit) on Windows 10
Any other tested browsers/devices(OK/FAIL):
Firefox 87 on Windows 10 (FAIL)
Steps to reproduce the problem:

Create a page with a DatePicker with displayFormat = "MMMM y" and minDate and maxDate set to a JSON Model
min: new Date("2021-02-01T00:00:00")
max: new Date("2021-04-30T23:59:59")
Open the embedded calendar date selector

What is the expected result?
A calendar is shown displaying the year 2021 with limited selection to the 2021-02, 2021-04 range
What happens instead?
A calendar is shown displaying the year 2021 with all months selectable
If you press the < button, the year 2020 is displayed with all months selectable
If you press the > button afterwards, the year 2021 is displayed with only the correct range selectable (see screenshots)
Any other information? (attach screenshot if possible)
Starting status

After pressing '<'

After pressing '>'

I'm a developer on behalf a SAP customer, if this is not the correct way to report this issue (we are using SAPUI5, not OpenUI5), please let me know. Thank you.

Hello @pietromodi
As you are using SAPUI5 for a Customer I would recommend you to raise a Customer Incident in order to process that internally. Anyway DatePicker is an OpenUI5 control and if you would like to process that here we would an example your coding.
Could you please provide a jsbin example? It would speed up the process of resolving the issue.
Best Regards,
Martin
(GitHub dispatcher)

Hi Martin,
here is the jsbin example
Thanks,
Pietro

Hello @pietromodi
The issue has been already fixed and included in 1.87.(1 - 4)
Could you please try to upgrade and confirm?
jsbin
Regards,
Martin
Github Dispatcher

Hi Martin @MapTo0,
I've tested your jsbin and it works but I've seen that version 1.87 is a "short term" supported version (https://sapui5.hana.ondemand.com/versionoverview.html).
Since we're about to decide the version of SAPUI5 to use (we would like to settle on a specific version for Quality Assurance purposes and stability), do you have plans to backport the fix to a "long term" maintenance version? (e.g. 1.84)?
Thank you,
Pietro

Hello,
I will forward that request to the responsible team.
Regards,
Martin

Hello @pietromodi ,
The issue was fixed in 1.84 and will be available with the next patch.
Best Regards,
Tsanislav


Networking issue in only one copy of Ubuntu 20.04 WSL 2 Distro, Win 10 Firewall Public profile off is workaround

Only for the past couple of weeks (not prior), when I run
xdg-open https://microsoft.com
it takes several minutes for the website to appear in Chrome in Windows. Typically and in all other WSL distros on my machine, the website pops up in Windows in less than a second.
The delay only occurs in one apt updated/upgraded copy of Ubuntu 20.04 (1 of 3 Ubuntu WSL 2 distros) and does not occur in a another apt updated/upgraded Ubuntu 20.04 WSL distro that is freshly initialized with no other software on it on the same machine; or a semi fresh 18.04 for that matter.
Everything else seems to be running fine on the Ubuntu 20.04 WSL distro with the issue. This distro is my daily driver used for php development via ddev and docker.
The symptoms of the issue do not present when the Windows Defender Firewall Public profile is turned off. There is never an error and no packets are ever blocked based on win def firewall logs. The command always completes successfully but with a long delay on this one WSL distro when the win def public network firewall profile is turned on.
I had some success with a custom rule allowing inbound port 6000 in the Win Def Firewall public profile but that success was short lived.
I suspect that the ARP tables are corrupted (or something in the neighborhood) and an Ubuntu network reset would correct the issue but I am unsure of the WSL recommended method to accomplish this without breaking Docker integration and I am also unsure if this was caused by a bug or just a normal possibility that can occur when using so much new tech together (WSL, Docker, Ubuntu, DDEV-Local, Windows Defender Firewall, xdg-utils, xdg-open)
I am curious if the WSL team would recommend troubleshooting this one from...
#1 Win Def Firewall Public profile rules
or
#2 from within the WSL Ubuntu network stack.
I am sure wiping out WSL distro and starting over would fix it but would like to find out if there is a less drastic method of troubleshooting that takes less time than rebuilding the dev environment in the wsl distro.
Environment
Windows build number: Microsoft Windows [Version 10.0.19041.572]
Your Distribution version: Ubuntu 20.04
The issue is on WSL 2
Steps to reproduce
sudo apt xdg-utils
xdg-open https://microsoft.com                        - or any website causes a long several minute delay on only one Ubuntu 20.04 WSL distro
The long delay can be relieved by turning off the Win Def Firewall Public Profile
---Wireshark Packet capture---
1	0.000000	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	54750 → 6000 [SYN] Seq=0 Win=64240 Len=0 MSS=1460 SACK_PERM=1 TSval=524255763 TSecr=0 WS=128
2	1.019061	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	[TCP Retransmission] 54750 → 6000 [SYN] Seq=0 Win=64240 Len=0 MSS=1460 SACK_PERM=1 TSval=524256782 TSecr=0 WS=128
3	3.098889	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	[TCP Retransmission] 54750 → 6000 [SYN] Seq=0 Win=64240 Len=0 MSS=1460 SACK_PERM=1 TSval=524258862 TSecr=0 WS=128
4	5.018988	Microsof_6b:6c:bf	Microsof_38:28:31	ARP	42	Who has <IP_ADDRESS>? Tell <IP_ADDRESS>
5	5.019006	Microsof_38:28:31	Microsof_6b:6c:bf	ARP	42	<IP_ADDRESS> is at 00:15:5d:38:28:31
6	7.178969	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	[TCP Retransmission] 54750 → 6000 [SYN] Seq=0 Win=64240 Len=0 MSS=1460 SACK_PERM=1 TSval=524262942 TSecr=0 WS=128
7	9.629218	fe80::c481:8553:a2ab:fcbd	ff02::1:ffc2:eeab	ICMPv6	86	Neighbor Solicitation for fe80::473:2653:52c2:eeab from 00:15:5d:38:28:31
WSL logs:
https://gist.github.com/bc1121/2eae6c73e0321b7d7b45fafacea88d0c
Expected behavior
Website opens in Chrome on Windows in about 1 second
Actual behavior
Long several minute delay ending in success with no errors or blocked packets. The delay is the issue.

After running xdg-open https://microsoft.com I see a few dbus processes start up and then nothing happens at all for me.
The fact it's influenced by Windows Defender being enabled is strange.
For now I would recommend using wslview https://microsoft.com from wslutilities.

and then nothing happens at all for me.

Actually it eventually opens the link in Lynx, which I have installed and must be handling my web links.
If you don't have another browser installed then wslview will be registered as your web link handler, which is why it does work.
There seems to be some sort of hang up in dbus. It happens even when I have dbus launched and running.

I have discovered the same issue on another computer also running Windows 10 WSL 2 Ubuntu 20.04. Also my main WSL 2 machine on that computer. It must have quit working around the same time as the laptop because it was running fine right before the laptop started acting up and it wasn't working right today (same long delay but no error) which is the first time I've tried it since the laptop started acting up.
It caused my to wonder if it was a networking issue that had to do with Windows or Ubuntu updates. The curious thing is that I can spin up a new Ubuntu WSL 2 machine and it works fine. I just wonder if maybe something got updated while the machines were running and it caused some strange networking cache errors because the new machines and the machines that were not running were not affected.
wslview looks interesting. I'll check that out!


Drawing line doesn't appear in toolbox

Hi there,
Is drawing line feature (mark) deleted from e-charts v4 ?
Is there any plugin that can bring the function back or I have to switch to v2 ?

Hi, you may draw lines with graphic.line or series.markLine.


CMS951 Updated bundle extraction.

This bundle was previously extracted from an 11-17-2023 upload.
This commit is from extracting an updated version of that same bundle, uploaded on 12-1-2023.

What was the intent of this update? I see test files updated but I do not see the CQL or Measure, were they OK and only test data needed updating?

Yeah, that was that updated bundle.  After extracting, only the test files looked to be updated.  At that, it looked like they mostly just changed some bookkeeping type dates  (not periods or effectiveTimes)
From: Michael Holck @.>
Sent: Monday, January 8, 2024 12:35 PM
To: cqframework/ecqm-content-qicore-2024 @.>
Cc: Donahue, Dan @.>; Author @.>
Subject: Re: [cqframework/ecqm-content-qicore-2024] CMS951 Updated bundle extraction. (PR #43)
What was the intent of this update? I see test files updated but I do not see the CQL or Measure, were they OK and only test data needed updating?
—
Reply to this email directly, view it on GitHubhttps://github.com/cqframework/ecqm-content-qicore-2024/pull/43#issuecomment-1881704871, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEC2ZFCUBKSLTOFQYMWA2BLYNRC7JAVCNFSM6AAAAABBO7XZFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBRG4YDIOBXGE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Test for vote label

ds

/vote

/cancel-vote

/vote

/cancel-vote

/vote

needs-review'

needs-review

/vote