Extension ringing but does not open call popup

I have come across the following problem:
When making a call, I can see in the Asterisk log that the extension is ringing, however no call popup opens in the home assistant android app.
This also happens when I use browsers, whether chrome or edge on my smartphone.
When I use a desktop too.
It happens many times.
My intercom rings, but I can't answer it on my smartphone as the call popup doesn't open.

This will be fixed on the next release.
You can track this here #71

This will be fixed on the next release.
You can track this here [#71]

@damiaorocha the new update is released. Can you check if it works?
If it doesn't, check the Asterisk logs, and the browser console (F12).

This problem keeps happening. When I use the browser, whether on desktop or smartphone, the number of times it happens is less than when I use the app. It happens to hear the ringtone of the call but not to open the popup, it happens that neither opens the popup nor hears the ringtone, as well as it happens to open the popup call and it is not possible to hear the ringtone. The asterisk log says that the extension I am calling is ringing. I couldn't see the logs in the app or in the browser when I use my smartphone. In the browser logs on the desktop nothing happens, as if no call had been received.

Hmm.. Could be some weird caching issue.
Do you use the addon?
Could also be a mix of different issues. Maybe a codec issue on your phone? Do you use video?

Yes, I use the add-on. I don't use video, I use a camera for the intercom extension.

Hi @TECH7Fox, thanks for the explanation.
the in-card view is wokring very well, sometimes the ringtone is played twice at the same time, it's a bit strange :-) but if i reload the page the ringtone is played correctly. I think some sort of caching problem if two calls are coming in very close.
I will rework my dashboard to have a better use of the in-card view. I'm using it on a Wallpanel with Fully Kiosk to interact with my Dahua doorbell.

Sorry for the late reaction. Was very busy these last days.
I also noticed the issue where it sounds as the ringtone is being played multiple times. Guessing indeed also a cache issue when you reload the card by going away from and back to the dashboard.
Not sure how to fix something like this. Will keep this open for now.


Using internal search function (-s) invalidates placement results

if I use the internal search function to limit results to a particular person, the "place" value fails.  It looks like the place value is computed AFTER the search results instead of BEFORE the search results.  For example, if I search the southeast district and use 'grep' to limit the output, I see the following (which shows the district standings):
$ atastandings -d southeast -c BCOB | grep -i atkins
 1 ISAAC ATKINS 55 SANDY SPRINGS, GA
 1 ISAAC ATKINS 40 SANDY SPRINGS, GA
 32 ISAAC ATKINS 7 SANDY SPRINGS, GA
 8 ISAAC ATKINS 25 SANDY SPRINGS, GA
 1 ISAAC ATKINS 67 SANDY SPRINGS, GA
 1 ISAAC ATKINS 42 SANDY SPRINGS, GA
 1 ISAAC ATKINS 54 SANDY SPRINGS, GA
 2 ISAAC ATKINS 39 SANDY SPRINGS, GA

whereas if I use the internal search, the standings return '1' in all cases (NB: the extra grep at the end is just to limit the output):
$ atastandings -d southeast -c BCOB -s atkins | grep -i atkins
 1 ISAAC ATKINS 55 SANDY SPRINGS, GA
 1 ISAAC ATKINS 40 SANDY SPRINGS, GA
 1 ISAAC ATKINS 7 SANDY SPRINGS, GA
 1 ISAAC ATKINS 25 SANDY SPRINGS, GA
 1 ISAAC ATKINS 67 SANDY SPRINGS, GA
 1 ISAAC ATKINS 42 SANDY SPRINGS, GA
 1 ISAAC ATKINS 54 SANDY SPRINGS, GA
 1 ISAAC ATKINS 39 SANDY SPRINGS, GA

This will take some investigation to see what could be done


Wrong position in SubViewport (Object Picking) for scaled viewport

Godot version
v4.0.alpha.custom_build [cfcdd576d]
System information
Manjaro
Issue description
If the subviewport is scaled, the input picking does not reflect the position on the screen for the object
Steps to reproduce
var factor = 1.0;
func _on_button_pressed():
	if factor < 1.0:
		#get_tree().root.set_content_scale_factor(1.0);
		factor = 1.0
	else:
		#get_tree().root.set_content_scale_factor(2);
		factor = .75;

	$SubViewportContainer/SubViewport.size = get_viewport().size * factor;

	$Button.text = "Change scaling - cur: " + str(factor)

to scale the subviewport for a 3D-scene
on a StaticBody listen for the _on_static_body_3d_input_event
if the subviewport has a sacling-factor of 1 everything is fine
if the subviewport has a scaling-factor of 0.75 the mouse-cursor triggers the input_event NOT on the object but with an offset to the top left

Minimal reproduction project
testProject.zip

Duplicate of https://github.com/godotengine/godot/issues/34805 (same cause).


Add SDR++ French bandplan

Hello,
I added support for SDR++ for the French bandplan by converting bestel74's SDR# file

@Noderyos Thank you for the addition!


Add description for step

Describe the feature you'd love to have in Orchest
Would love to add a description input box , support markdown  in step properties

I love the suggestion. Would you be okay with the description only showing in the properties pane? Meaning that you won't be able to see a step's description unless you open its properties.

Yes, for description mainly for explain the title, so just properties pane is good enough.

It would be great if the description supports Markdown


New Device: Door Locks

Describe your device
Tuya Door Locks. Sample Product
Example: Generic door locks with open and close commands
Partial Support?
Currently there is no partial support
We can support via a paypal donation

I currently don't have plans to add locks due to how different protocols may handle locks. Can you send your schema, I'll try to experiment.


[feat] Add environment variables to deployment/services def'n

This PR makes it possible to set environment variables for services through a deployment config file.
name: QuickStart

control-plane:
  port: 8000

default-service: dummy_workflow

services:
  dummy_workflow:
    name: Dummy Workflow
    source:
      type: local
      name: src
    path: workflow:echo_workflow
    env:
      VAR_1: x
      VAR_2: y
    env_files:
      - .file1.env
      - .file2.env

There are two ways to set env variables:

via the env which requires the user to pass via mappings (i.e., with keys and values for the variables)
via the env_files which requires the user to pass the path of the .env file in relation to the source name.

In order to avoid name clashes with the environment variables from multiple services in a single deployment, the env variables are prefixed with the service id i.e., {service_id.upper()}_{env_varname}.
NOTE:

env-files currently always read from local even if the source type is git and not local

coverage: 73.576% (-0.02%) from 73.597%
when pulling 953608a659b4c6e9af82b11b458938525acec8c3 on nerdai/add-environment-variables-to-deployment-defn
into 47efff882cffa9b662f5a78e3032f5a13ea7bc17 on main.

LGTM, nicely tested!

Thanks for the assist, as always!


Add ability to search by market

Listing all blocks is nice, and so is searching by library and name, but also the ability to narrow down to a specific market would be helpful for 'list' command.

New format for searching list of blocks.
[<market>].[<library>].[<name>]
Any block title component can be skipped. For instance, searching through all available blocks is equivalent as searching for ...
Searching all blocks in market open-ip:
legohdl list open-ip.
Searching all blocks belonging to library util:
legohdl list .util
Searching all blocks with same name as design1:
legohdl list ..design1
Searching for specific block with full title:
legohdl list open-ip.util.design1

Commit f4de1be615f10dbb reflects these new enhancements.


Import type assist doesn't work near syntax errors

struct S;

impl Debug for S // need to add {} to be able to trigger the assist on Debug

Might be related to #5144.

Looks like we fall through here: https://github.com/rust-analyzer/rust-analyzer/blob/aa69757a01c26cfad12498053c55cbc3d66a4bdb/crates/ra_assists/src/handlers/auto_import.rs#L97.

Probably fixed in #5280.


Update build.gradle

Targetsdk have to be 30
Please ensure that your pull request meets the following requirements - thanks!

Does not contain merge commits. Rebase instead.
Contains commits with descriptive titles.
New code is written in Kotlin whenever possible.
Follows our existing codestyle (gradlew ktlintCheck; will be checked by CI).
Does not break any unit tests (gradlew testDebugUnitTest; will be checked by CI).
Uses a descriptive title; don't put issue numbers in there.
Contains a reference to the issue that it fixes (e.g. Closes #XXX or Fixes #XXX) in the body text.
For cosmetic changes add one or multiple images, if possible.

Finally, please replace this template text with a description of the change and additional context if necessary.

Finally, please replace this template text with a description of the change and additional context if necessary.

This makes me doubt you have read the documentation on behavior changes for API 30.

See PR #5573.


🐛 : Image layout for early-adoptors badge when app starting

Screenshots

Smartphone (please complete the following information):

iPhone13 mini Simulator

If you want speed up process of this issue, please send 🚀 as a reaction.

It's fixed with RN version.


Improve menus due to improvements in electron

Just updated to the newest version of electron and used some of the new features, to streamline our menus.
See each commit for details.
Tested on windows 10 so far.

Thanks @Razzeee! Can I have your help to also post the latest artifacts on circleci.com, so we can have others in the community help test the pull request?

What do you mean? Upstream automatically builds, when you send a pr https://circleci.com/gh/mattermost/desktop/463#artifacts

Sorry for the confusion, a link to the corresponding artifact on circleci.com is what I meant
Thank you for adding it!

Just curious, "Toggle Fullscreen" is only localized on OS X. Do you know anything about this?

No clue, sorry. Maybe ask in https://github.com/electron/electron/pull/6166

Rebased, not sure if/how to add it to the changelog

Thanks again @Razzeee! For the changelog entry, will this have any effect on the user interface, or on performance?
Would we be able to have two community members testing this pull request, one each in OS X and Linux/Ubuntu? We would like to have at least two reviewers before it's merged.
The artifacts to test can be found here: https://circleci.com/gh/mattermost/desktop/505#artifacts

Well it should be a little bit more platform specifical correct. For example the quit button on Windows is now named Exit as it's for pretty much every other app on Windows. Little things like that.

Ah cool, thanks @Razzeee  - I'll make a list of changes I've noticed while testing it on Windows, and we can maybe prepare a similar one for OS X and Linux?
Will know better than how to update our docs

Thanks again @Razzeee - tested on Windows and didn't find issues with functionality.
However, I compared to other messaging platforms such as Slack and Franz, which use "Quit" for quitting the app. I'm not sure if it's obvious whether "Exit" closes or quits the app.
We're also losing a shortcut, which is CTRL+Q to quit the app.
Any thoughts?

Well that's standard for windows, as I can't think of an app that has a close shortcut on Windows. On osx it's still named quit and has a shortcut. As that's the platform way of doing it.
Compare the menu to notepad++ or sublime on Windows, not Franz/Slack as that's also a electron app and not native.

I think it's not important whether the app is made of Electron.
Generally, Windows apps can be closed by system shortcuts like Alt+F4, so the shortcut is not necessary. But this app keeps the window as hiding and not be closed in order to communicate with the server. So I feel it's not bad idea to keep the shortcut.

Are you sure that alt + f4 doesn't close the app? Would seem like a bug to me.

Not bug, but "not be closed" was not enough to describe the behavior. When closing the main window, the app doesn't close and just hide it in close-event. As the result, the window still exists internally when using Alt+F4. So I wrote "not be closed".

Got you. So when we want that, we should overwrite the shortcut for "Exit" on windows with Alt + F4

I expect Alt+F4 to close a "window" like Ctrl+W. For example, Chrome closes a window on Alt+F4, but the app is still running when there are other windows or background tasks, and Ctrl+Shift+Q is provided to exit. So I feel we should use other shortcuts to finish the app (So I had added Ctrl+Q).
How do you feel?

I really don't have an opinion about this. In my mind the best parts about letting it be handled by electron is the consistent wording, familiar shortcuts and implementations. And I don't think we make it worse if we add one shortcut to one of the entries.

Thanks for the thoughtful discussion here!

I did a little research and found Microsoft's Design apps for the Windows desktop guide. Here they spell out that File > Exit in the menu bar is standard and File > Quit doesn't really exist, except in the command line.

To make the Desktop App consistent with other apps on Windows and therefore more obvious to end users, agree @Razzeee to change it to File > Exit.

However, propose CRL+Q would still be used as the shortcut to Exit (or CMD+Q to quit on Mac), and ALT+F4 to Close the app to the system tray. This is current behaviour and is similar to other apps when there is a program running in the background (along the lines of what @yuya-oc mentioned above).

@Razzeee If you're fine with 2), can we have your help updating the menu bar so that CTRL/CMD+Q is used to quit the app?

Quit as in shutdown with nothing running in the background? Just to be clear?

Yes, that would be correct @Razzeee. Thanks for confirming

@jasonblais
changed
@yuya-oc
you might want to redo the review, as I had to add some special handling for the exiting via STRG-Q on windows

Great, will use this the CircleCI during my work tomorrow, thanks @Razzeee for the updates!
Any OS X or Linux testers who would like to help test it out? https://circleci.com/gh/mattermost/desktop/537#artifacts

Update: I tested on Windows 10 and at one point the app completely froze for me and I had to quit it.
Might have been a one-off thing and not related to this PR, but I'll try again tomorrow to see if it reproduces. Otherwise no issues spotted

Haven't reproduced it, so pass from me. No issues found during testing.
@Razzeee Might need one more rebase before we merge it.
@yuya-oc can we have your help reviewing this PR before we merge it?

Rebased

It looks okay except electron version.

Fixed your comments, thank you.

Tested on OS X 10.11.5. It looks okay. Thanks!

Thanks @yuya-oc! I think this is ready to merge, has been tested in OS X and Windows.
@Razzeee excellent work here, thank you!!

There are now new roles for the zoom functions:
https://github.com/electron/electron/pull/6777
Should probably done in a follow up PR.

Thanks!


Eroor in Cartesian to Geodetic Transformation

Hi,
we are trying to transform the position data coming up from the ins in ENU coordinate system to the geodetic coordinate system so that we can match up the location where the gps is missing. However, we are getting very weird value.For example the gps coordinate of our area is
41.7458322 , -111.8035665, in lat,long, and altitude. And the coordinate(position x,y,z) of the corresponding place coming from the ins is -11328.84277,<PHONE_NUMBER>,  -106.8024216.  It should match up with the coordinate of the gps after the transformation but it does not. we are using matlab enu2geodetic transformation function.
Does it have anything to do with the ins calibration ?
Any help would be very appreciated.
Thanks

Have you set your reference lla?
See the 'set_ref_lla' service.

Hi,
Thank you very much for your kind reply. Ok, I got your point. But can you tell me something line the data that i have collected before setting up the ref_lla, will that be garbage  or I can somehow modify the data ?

@superjax , we just changed the reference but it is still giving us error. Does it has anything to do with the calibration ? If so, how to do this calibration ?

Hi @ethan20120 ,
Can you give a more in depth example of your input parameters? The matlab enu2geodetic function requires
xEast, yNorth, zUp, lat0, lon0, h0, and spheroid as input parameters.
This may be obvious but I'm going to describe everything just to make sure you don't miss anything. lat0, lon0, and h0 should match the reference Lat, Lon, and Alt used in the uINS. This represents the point from which the ENU coordinates will be measured.
You can enter this manually in ROS (set_refLLA_value) or you can set the current location of the uINS as the reference LLA(set_refLLA_current). These values can also be set or obtained using the EvalTool.
xEasy, yNorth, zUp are the coordinates in meters of the point of which you want to calculate the LLA position. The uINS outputs an NED location in x,y,z coordinates so you need to transform NED to ENU.
spheroid should be wgs84Ellipsoid which is the spheroid that is used by GPS systems.
lat, lon, h (the output) is the location that is calculated. To reiterate, this is the point measured in meters from the reference LLA and converted to lat lon, alt.
Hopefully that helps. If that doesn't clear things up please send more information or data so we can dig deeper.
-Andrew Priddis
Inertial Sense Support

Hi Andrew ,
Please correct us if we are wrong we need to calculate the offset between the gps and ins by ourselves right ?
The following picture represents the vehicle trajectory (white dot from gps and black from the ins)

Now after doing the offset calculation we get the

Still the gps and the ins is not aligning properly. I guess it's because of the drifting error of the ins right ?

Hi @siddirp ,
Our INS and GPS are not aligning. Can you please give us some hints on how to resolve this ?
Thanks
Ethan

Hi @siddirp
we have updated the firmware to the latest version 1.7.6 but still we are facing the same error that the INS and the GPS points are not overlapping with each other. Any help will be very appreciated. Also can you please tell us how we can increase the update rate of the INS. Currently it's updating in every 10 milliseconds. Can we update it faster than this?
Thanks
Weng

@Chenweng1 - Since we discussed this thoroughly over the support email I am closing this issue.


On displaying, wide error messages are not correctly wrapped

Environment

Elixir & Erlang/OTP versions (elixir --version):Erlang/OTP 24 [erts-12.0] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [jit]

Elixir 1.12.0 (compiled with Erlang/OTP 22)

Operating system: Linux
Livebook version:  HEAD: 26226a4
How have you started Livebook (mix phx.server, livebook CLI, Docker, etc): mix phx.server and CLI

To reproduce the issue, you can run this livemd:
https://gist.github.com/qhwa/d35371322f7522f74a912c18fc98841d
Current behavior
When the error message contains a line of wide string, it is not wrapped on the webpage. So it's hard to read, as the following screenshot:

Expected behavior
The error message is correctly wrapped.

A quick workaround can be to replace the CSS class name from whitespace-pre to whitespace-pre-wrap. However this might not be ideal because now the error message goes wrapped and might have a greater chance to overflow vertically. While this is not a big deal on laptop screens, but maybe a frequent issue on small screens. So maybe we need a better interactive way of displaying error messages?

Hey @qhwa, your solution with pre-wrap sounds good to me, especially that these kind of errors are relatively rare (most errors can be caught and shown directly in the output). Looking at the diff you referenced, we want to keep pre for the outputs, and only use pre-wrap for the flash messages. Do you want to submit a PR? :)

Sure, my pleasure! @jonatanklosko

Closed in #304, thanks @qhwa :)


Proposed changes to repo based on Julia 1.11

As mentioned in #482 now julia-1.10 is LTS and some extra useful features have been add to julia-1.11.
I think these are two small changes we should make:

[x] #484
[ ] #486

Yes agree. We may also want to use [sources] in benchmark.

Closing as now just a dup oof #486


17.7 service configuration buttons not inline

@fabianfrz I don't think there is a very nice solution, but this should work https://github.com/opnsense/core/commit/1e87fd4a6a9eef34e835c97c50acb3e2f377a588

in the smallest possible screen size this still wraps
this stuff can be fixed in general, but we really need to get rid of tables in favour of divs

It will always wrap at some level, even with divs... hiding columns on xs size is also still an option. I don't expect the widgets will be converted any time soon (to less to gain, too much effort to restructure all).

that's true, wrap or overflow. the problem with tables at the moment is that every other row will overflow, while the other doesn't. I tried to fix this, but it's impossible. Maybe that is a better context. :)

true, the question for this issue is..... is the fix good enough (within the boundaries we can fix), or do we wait for something better.
I vote for close :)

@AdSchellevis me too. If the display is small enough this should be possible but not on a screen wide enough.


Ek 428 gatsby caching issue

Installed @netlify/plugin-gatsby as an alternative to using gatsby's caching system for the time being due to an error caused by gatsby-source-filesystem.  It seems that the gatsby community has only got around this error by running gatsby clean at the start of each build which would be the opposite of the purpose of this branch.
See here and here
Test:

Gatsby build time has been notably reduced.
Faculty Nodes have consistent ID's between builds
Gatsby uses cache instead of fetching image when a photo exists - Note, for this test to pass, we will need to initiate a rebuild without clearing the cache 2-3 times and ensure that when there exists cached images, the number of requests hitting the api should be 0 (unless there was a change for a corresponding node in the api coincidentally happening while we are testing)

Pre deploy: Merge https://github.com/Stevens-EWS/gatsby-source-custom-api/pull/1/files and make a commit changing package.json to reference master instead of the commit hash

There is no need for this PR anymore, the issue with caching is being solved in https://github.com/Stevens-EWS/faculty-profiles/pull/49

Closing before merge because this site has been decommissioned.


[feat][ci] replace stale bot with ping-pong workflow

Motivation
Replace stale bot with ping pong action:

waiting-for-user
waiting-for-reviewer

Modifications

Verifying this change
See https://github.com/tisonkun/pulsar repo for use case
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Closed. I suggest we first meet a consensus on the strategy and actually I can write a script to run periodically on behalf of me instead of wasting online resource especially to trigger it per comments.


Can not set breakpoint in .c files

Testing #194071
I have the DWARF extension installed but I can not set breakpoint in .c files.
Should Dwarf contribute the ability to set breakpoints in c files?
Or am I missing something?

Which c file are you trying to set breakpoints in, and which HTML page are you on when you try to set breakpoints?

This was for fibonacci.c

Oh, right. We need to register what languages our extension lets us set breakpoints in... but there's an arbitrary number of languages that can compile to WebAssembly. And if we add some kind of "*" language, then that will effectively disable the per-language breakpoint ability in VS Code since js-debug is built-in.
@roblourens thoughts? For the moment I could just register common languages people are compiling to wasm (C/++, Rust, maybe Zig?)

Yeah, it bugs me too, on one hand it's weird to just let breakpoints be set in every .md and .json or whatever but also you can't really enumerate all the possible bp languages in scenarios like this. I'd stick with the common known set for now.


Jobber Dependency Broken

All my latest builds with jobber tell me that the following dependency has moved:
package code.google.com/p/go.net/context: unable to detect version control system for code.google.com/ path
make: *** [build] Error 1
Full build log here:
https://circleci.com/gh/blacklabelops/jobber-cron/14

Balls.
I'm traveling today so I'll fix it tomorrow.

On Jan 26, 2016, at 7:58 AM, Steffen Bleul<EMAIL_ADDRESS>wrote:
All my latest builds with jobber tell me that the following dependency has moved:
package codegooglecom/p/gonet/context: unable to detect version control system for codegooglecom/ path
make: *** [build] Error 1
Full build log here:
https://circlecicom/gh/blacklabelops/jobber-cron/14
―
Reply to this email directly or view it on GitHub.

Fixed in v1.0.1.


Check for redundant Collection.contains calls.

Could Error prone check for a Collection.contains call followed by an immediate Collection.remove call?
I see a fair amount of
if (myList.contains(value)) {
  myList.remove(value);
  // Do something.
}

which could be
if (myList.remove(value)) {
  // Do something.
}

This is covered in the tools alluded to in #649 which we are working on getting open sourced.


Major refactor

Abstract the redundant bits.
Map SharpHook keys to Godot keys in the dictionary. Idk why it was the other way around anyway.
Clean up the redundant use of the private keyword.
Disable "Declare types in namespaces" warning.
Use _pascalCase for naming private fields.
Less indirection for accessing event data.

Wow. Great work in reducing the clutter lol.


test: add SinglelinePage tests

Description
Adds tests for single line page

Hi @DanLeshinsky ,
Thank you for the feedback! I've addressed the concerns you raised and implemented the requested changes. Please take another look at the code, and let me know if everything looks good to you.
If you have any further suggestions or if there's anything else you'd like me to modify, please don't hesitate to let me know.
Thanks again for your time and feedback!


IoT device working offline: explore Edge and solution rearquitecture

Some notes are captured in Edge.md

Moving out to a future milestone, as we are going to instead focus on the mobile app as the device infrastructure to manage and since moving to edge adds a degree of complexity for scale we don't yet have.


Added Hacktoberfest Contributions - talk 2/10 and blog post 31/10

Added in contributions:-

MaccTech Talk 2/10/2024 - Delivering UMBRAAD talk in person on how we can learn from Umbraco to improve accessibility across the web
Blog post 31/10/2024 - Looking at running Umbraco 12 vs Umbraco 13 on a variety of Mini PCs, balancing performance, cost and sustainability issues.

Thanks @CodeBunTes for those great contributions👍
May I suggest that you create a separate PR for each of those? Then it will count as two contributions and you will be eligible for some Hacktoberfest Umbraco swag 😁

@mikecp Good shout there actually, I'll resubmit it separately later today when back at the computer. Thanks!


Add targetUtilization for percentile prediction

Add targetUtilization for percentile prediction
Refactor percentile prediction

What type of PR is this?
feat
What this PR does / why we need it:
Add targetUtilization for percentile prediction
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

shouldAnalytics := c.ShouldAnalytics(analytics) 	if !shouldAnalytics { 		klog.V(4).Infof("Nothing happens for Analytics %s", req.NamespacedName) 		return ctrl.Result{}, nil 	}
Do you want to include the fix for this pr?

shouldAnalytics := c.ShouldAnalytics(analytics) 	if !shouldAnalytics { 		klog.V(4).Infof("Nothing happens for Analytics %s", req.NamespacedName) 		return ctrl.Result{}, nil 	}
Do you want to include the fix for this pr?


Missing WiFi.h when including BootWifi.h

I am trying to use BootWifi module. I develop in eclipse with esp-idf.
However when including bootwifi.h I get following error:
fatal error: WiFi.h: No such file or directory
I can bot find "WiFi.h" in any esp-idf moduel. I see though esp_wifi.h
How can I fix it?

You have to add cpp_utils folder to components folder in your project

Thanks for quick response.
And for all valuable info you are sharing.

Closing for just now, re-open as needed.


HDDS-8657. Remove Clone and make use of copyObject in OmDirectoryInfo and OmKeyInfo.

What changes were proposed in this pull request?
Remove Clone and make use of copyObject in OmDirectoryInfo and OmKeyInfo to have better maintainability.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8657
How was this patch tested?
Built locally and tested as same steps in HDDS-8292

@szetszwo, Please help to review, Thanks.


Add authentication routes and controllers and include refresh and access token

Description
Add authentication routes and controllers and include refresh and access token including revokedtokens table and many to many changes in image and labels table. Also changed the directory structure.
Fixes #424
Type of change

[ ] 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)
[ ] This change requires a documentation update

Also, include screenshots of app for the verification and reviewing purpose.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] 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
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@Tushar19varshney @ivantha @UdeshUK review this please.


Remove Duplicates

CherryPy appeared twice in the file, and CloudFront seemed to have two description elements.
There's probably a way to validate this vs the schema but I haven't had time to look into it.
Signed-off-by: kingthorin<EMAIL_ADDRESS>

It seems this can't be done in the Schema. However, if you'd like me to setup a GitHub Action workflow to run jsonlint (which does/would catch such mistakes) against PRs for the technologies.json file please let me know.

Thanks!

I just noticed your comment, I'm actually wanting to move from Travis CI to GitHub Actions. I'll probably need to do some work there first.

Well feel free to ping me if you need any help or advice.


fixed-responses-for-api

Data to store
deposit collections

account
deposit market
deposit commitment
deposit id
balance - This will be updated in another PR, "addToDeposit" seems to be flawed!
savings interest

loan collections

account
loan market
loan commitment
loan amount
loan id
collateral market
collateral amount
isSwapped(bool true/false)
current market
current amount

This will be updated in another PR, "addToDeposit" seems to be flawed!
What is it..? are we not fixing this ?

Balance field will be added in another PR
On Sat, Feb 19, 2022, 20:50 Markish @.***> wrote:

This will be updated in another PR, "addToDeposit" seems to be flawed!
What is it..? are we not fixing this ?
—
Reply to this email directly, view it on GitHub
https://github.com/0xHashstack/offchain/pull/59#issuecomment-1046041696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYETTMW2YPX76O2Y7DLEM3U36YNFANCNFSM5O2RBXAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Will discuss with you on Monday once.
This PR can be merged.
On Sat, Feb 19, 2022, 20:51 Saket Shrivastava @.***>
wrote:

Balance field will be added in another PR
On Sat, Feb 19, 2022, 20:50 Markish @.***> wrote:

This will be updated in another PR, "addToDeposit" seems to be flawed!
What is it..? are we not fixing this ?
—
Reply to this email directly, view it on GitHub
https://github.com/0xHashstack/offchain/pull/59#issuecomment-1046041696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYETTMW2YPX76O2Y7DLEM3U36YNFANCNFSM5O2RBXAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Google OpenID Connect authorization token problems

The backend for Google OpenID Connect is broken. Here are some notes toward where the problem lies, in case someone else is tempted to go on a goose chase.
In 61dfbc2ea7a897981a15db8904c5a68d471b0515 the authorization flow for the GoogleOpenIDConnect backend fails at BaseOAuth2.auth_complete(): raising an AuthCanceled exception due to an underlying 400 response by Google -- PSA is passing nonce, which is not supported by the Google API.
If we remove the nonce from the request (e.g. by removing lines 298-300 in backends.open_id.OpenIDConnectAuth.auth_complete_params()), we get an ok response from Google, but PSA fails to verify Google's JWT -- we get an AuthTokenError here with an underlying complaint from jwt.decode that "The specified alg value is not allowed").
From what I vaguely understand of OpenID Connect, we should expect an alg header in the JWT, and so it makes sense that the PSA OpenID backend is asking jwt.decode to verify (default) the token. The exception above is raised because PSA's OpenID backend passes algorithms=['HS256'] to  jwt.decode, but since Google does not pass the alg claim in the JWT verification fails.
The missing alg claim is by design on Google's part: Google claims that

Normally, it is critical that you validate an ID token before you use it, but since you are communicating directly with Google over an intermediary-free HTTPS channel and using your client secret to authenticate yourself to Google, you can be confident that the token you receive really comes from Google and is valid.

Here's how Google wants you to validate their tokens: https://developers.google.com/identity/protocols/OpenIDConnect#validatinganidtoken
So, not sure what the path forward is for PSA, other than that a different validation strategy needs to be implemented.

Hi All I have the same issue!

The fix for this problems got ported to the social-core component.


Latest version of package is not listed on pypi.org or installable through pip

Description
I have built a new version of my package and I have uploaded it to pypi.org through twine.  The new package does not list as the latest version. https://pypi.org/project/pytest-rpc/
Expected behavior
I expect to see my new release listed as the latest on the page for pytest-rpc
I expect to be able to see my release in the Release History section of pytest-rpc's page on PyPI
I expect to be able to pip install pytest-rpc==0.10.0 successfully
To Reproduce
Attempt pip install pytest-rpc==0.10.0
Notice that you can navigate to this package on pypi.org https://pypi.org/project/pytest-rpc/0.10.0/
My Platform
Package built on Mac (10.13.5)
Python (2.7.10)
Uploaded with twine (1.11.0)
Attempted install with pip (9.0.3) & (18.0)

Almost certainly due to #4183, will confirm.

Confirmed. Executing the purge successfully exposed the new release and no failures/exceptions were found in log regarding the purge that should have occurred. This is being considered a duplicate of #4183.

Thanks! @ewdurbin


Add Xoshiro256StarStar

Hello
Could you please add xoshiro256ss pseudorandom generator?
https://xoshiro.di.unimi.it/xoshiro256starstar.c
I understand it is mostly reusing xoshiro256pp but with a slight change.
Thank you in advance!

Hello!
Please, explain, what is the fundamental difference for you between star-star and plus-plus? For what cases would someone use the star-star?

It is used for UR encoding:
https://github.com/BlockchainCommons/Research/blob/master/papers/bcr-2020-005-ur.md#ur-encoding
"Hence seqNum and checksum are concatenated and then passed through a SHA256 hash to generate a 256-bit seed for a particular pseudorandom generator algorithm: Xoshiro256**, henceforth "Xoshiro256"."

The differences between ++ and ** were really quite small.
I added the Xoshiro256** generator to the new version of the package (0.7.0) under the name Xoshiro256ss. The new version is already on pubdev.

Thank you, much appreciated.


Drop multiple ways for referencing modules in Kyma CR

Description
Currently, it is possible to reference the Kyma Module in multiple ways. There should be only one way - using the label value of operator.kyma-project.io/module-name.
Reasons
There is no need to support multiple ways of referencing Module Templates, we plan to stop synchronizing ModuleTemplates to SKRs.
Acceptance Criteria

[x] FQDN is no longer supported as a method of referencing Kyma Modules
[x] Passing the namespace/name of a ModuleTemplate CR  is no longer supported as a method of referencing Kyma Modules
[x] Documentation adjusted
[x] Tests adjusted

Feature Testing
No response
Testing approach
No response
Attachments
No response

While working on this issue, several problems with the local test setup guide of the Kyma Lifecycle Manager were discovered. The fix / updated documentation has been proposed here.

Additional AC:

[ ] Please create Follow-Up Issue to check whether any SKR cluster uses the "old" way of referencing the modules.

check whether any SKR cluster uses the "old" way of referencing the modules.

IMHO, this must be verified before merging the related PR. We need to know if we break any existing clusters and if so, first migrate them on using the approach supported that is left.

As discussed we want to do a thorough verification on Prod landscape with namespaced name and FQDN references as a last step to close this issue.

As discussed we want to do a thorough verification on Prod landscape with namespaced name and FQDN references as a last step to close this issue. And please clean-up/archive related issue: #1682

Comment: during refinement of the other ticket, it seemed like the alternative approaches for referencing a module were not working because of an other change introduced earlier... If it can indeed be verified that other approaches are not working right now already, and given that nobody complained until now, we can move forward with this change and close the other issue.


Some questions about the NL module

Hello
I would like to ask why it needs to redefine the backward for NL modules, instead of writing the module and using the backward in pytorch, because of the "processing" operation?
I am not familiar with custom functions and I hope you can answer this for me.

Becasue our normalized self-attention block re-modify the naive self-attention module. In other words, our block re-define the forward process, and thus, the backward process also needs to be defined.
You could refer to the customed operation presented in the official pytorch tutorials.
Thank you again,
Ge-Peng.

Becasue our normalized self-attention block re-modify the naive self-attention module. In other words, our block re-define the forward process, and thus, the backward process also needs to be defined.
You could refer to the customed operation presented in the official pytorch tutorials.
Thank you again, Ge-Peng.

Thank you for your reply
What I mean is that Constrained Self-Attention can also be implemented by calling the API that already exists in pytorch, now by writing your own CUDA arithmetic, is that to speed up the processing?

Sorry for the late reply cause I missed your question.
I haven't validated the difference between the two different modules. But I guess the answer is 'NOT'. You could write a comparison code for that. I think it is not very complicated.
Best,
Ge-Peng.


Fix the incorrect sequence path when using pre-hierarchy for render creator

Changelog Description
When we create render instance with pre-hierarchy options enabled, it will raise the error and the system can't manage to find the correct path. This PR is to fix it.
Additional info
We should allow user settings for the sequences path
Testing notes:

Create some level sequences inside /Game/Ayon/Sequences/
Create Render Instance with pre-hierarchy
Should not be error out

This indeed solved the issue where you could not create a publish producht when "use hierarchy" was enabled/ I still get the error:
"We have been unable to find a sequence in the files. Please ensure the files are named appropriately. Files: []"
When trying to publish.
Your log printed: "/Game/Ayon/AyonPublishInstances/renderICVFXMain_INS" for me

The "use hierarchy" issue is resolved, but I still get the "We have been unable to find a sequence in the files. Please ensure the files are named appropriately. Files: []" error from the "validate_sequence_frames.py". Maybe it can't find the rendered frames or something?

TODO list:

[ ] Fix the bug of not getting ClipIn & ClipOut in folder entity and task entity

Alright, were getting closer. When I first ran this code, it said that the frame rages weren't correct. (First time I got that error) But after setting the frame ranges, the error persisted. Finally after setting the frame ranges to all tasks in the project, it worked. Could be a bug, but it could also be a mistake om my side. After I set the frame ranges, I got another error:
  File "C:\Users\thijs.smeele\AppData\Local\Ynput\AYON\addons\core_0.4.1\ayon_core\plugins\publish\integrate.py", line 369, in prepare_product
    folder_entity = instance.data["folderEntity"]
                    ~~~~~~~~~~~~~^^^^^^^^^^^^^^^^
KeyError: 'folderEntity'

It did render a .mp4 from all the exr frames, so that's a win

Alright, were getting closer. When I first ran this code, it said that the frame rages weren't correct. (First time I got that error) But after setting the frame ranges, the error persisted. Finally after setting the frame ranges to all tasks in the project, it worked. Could be a bug, but it could also be a mistake om my side. After I set the frame ranges, I got another error:
  File "C:\Users\thijs.smeele\AppData\Local\Ynput\AYON\addons\core_0.4.1\ayon_core\plugins\publish\integrate.py", line 369, in prepare_product
    folder_entity = instance.data["folderEntity"]
                    ~~~~~~~~~~~~~^^^^^^^^^^^^^^^^
KeyError: 'folderEntity'

It did render a .mp4 from all the exr frames, so that's a win

Yup, folder entity is a requirement for instances. If it's not set for a publish instance, that's a bug in the Unreal integration for that particular product type.
Please share a publish JSON report.

Already shared it in the other PR, but to keep the discussion complete... Here it is again.
publish-report-240729-15-18.json

There was also an issue with rendering. If the directory where the rendered images doesn't exist, it returns an error. Maybe it would be nice if it just created the folder instead?

would be great if you can show me the log for the error.

When I run the code, it still can't find the "ClipIn" value even though it is set in the AYON settings
unreal\client\ayon_unreal\plugins\publish\validate_sequence_frames.py", line 81, in process
    folder_attributes["clipIn"],
    ~~~~~~~~~~~~~~~~~^^^^^^^^^^
KeyError: 'clipIn'

Could that be becase "ClipIn" is zero?

Fixed in a657d8a it will use 1 for clipIn and 1 for clipout as ayon server has set for default if there is no value for these two.

at this point, I think the PR shouldn't be set as draft.


React-intl #391

fix extra line
and add npm command after generate new language

@chaintan17 @mxstbr this can be closed 👍


[WIP][Conv2d] Conv2d Padding Fix

This patch resolves conv2d calculation is crooked in some cases
Changes proposed in this PR:

Conv2d Layer Golden Test
Conv2d Fix
Padding::Compute Fix

Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1570. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1570-202109151110340.42215800285339-3dcc51f1fc49925fe116e483092c3cf78a28792c/.


Build Failed

I have this issue every time i want to verify or build my arduino code, i don´t now whats wrong, aferter 2 seconds atom says "Build Failed", i have already check the Path and seems to be correct. I've also installed "build-arduino" and "language-arduino" packages. Any suggestion or help will be well appreciated.

Atom 1.31.0-beta1 x64
arduino-upload 1.3.3
Windows 10 Home Single Language 1803

@Wermmann We've made some changes recently, can you tell me if the issue is still present?

Closing due to stainless. Please feel free to reopen the issue if it is still a problem.


Bump AWS SDK to resolve reported vulnerability

This PR remediates the AWS SDK vulnerability that appears in the osv-scanner run shown in PR #136.  I don't expect a lot of churn in the v1 SDK, but if that's an issue then I'll move the hard-coded version to a property file.

This PR is a proposed fix for issue #146


remove empty folders

What do you think about removing the folders "examples" and "encoding", as not all projects have examples or an encoding.
If we keep the folders, at least "encoding" would need to be added to the setup, so that non-python files get installed too when installing and shipping the project.

Yes, there is no need for them.

Done in last PR.


Non-Fatpack sd card build failed

When trying to build a sd card with no fatpack -f 0 (without API/WebUI) using the following command on a fresh RaspiOS 64-bit:
wget --no-cache https://raw.githubusercontent.com/rootzoll/raspiblitz/dev/build_sdcard.sh && sudo bash build_sdcard.sh -f 0 -b dev
It failes on this error:
*** Adding deb.torproject.org keyring ***
gpg: no valid OpenPGP data found.
! FAIL: Was not able to import deb.torproject.org key

@openoms this is not v1.8.0 critical - but should be fixed soon after v1.8.0 release in v1.8 branch.

Likely times out as Tor is so bad nowadays. Will add a clearnet fallback to pull the keys.

@openoms let me know when this is in - its the last issue to close post-v1.8.0 release.

fixed in 35607331 both in dev and v1.8


Switch to unordered set for C++ version

Need to implement hash and == for the relevant type.

https://stackoverflow.com/questions/15103975/my-stdhash-for-stdtuples-any-improvements

Specialize Std::hash for simple two size tuples where is type is hashable using the Hash requirement https://stackoverflow.com/questions/47714655/c-concepts-and-template-specialization-how-to-get-a-user-friendly-compiler-er

Probably overkill to specify as a concept since it is directly in (the whole) body, but who cares.

https://en.cppreference.com/w/cpp/utility/hash

https://github.com/JanEricNitschke/TicTacToe/commit/4a6b61386ddcdcf086de90db9abb7e88d8fb2bef


Fix plugin link tooltip flickering

Summary
The PR #10004 for "Eliminate unnecessary tooltip divs added for links" introduced a few issues when hovering over a  plugin LinkTooltip link:

In certain mouse and screen positions, the tooltip be placed right at the pointer of the mouse, causing sporadic mouseon and mouseleave events to fire. This makes the tooltip flicker continuously.
The protocol for this plugin feature has changed. Before the PR, the LinkTooltip component was mounted on initial render of the post containing the link. The component receives a show prop when it needs to react to being hovered. After the PR, the component is mounted on hover, making it so the component cannot do any local state caching.
The popover does not fade into appearance anymore. It just instantly shows instead.

The first two bullets together is causing a spam of requests to fetch data for the plugin's component.
The changes in the linked PR are a little complicated, and I'm not sure which pieces are causing which issues, so I've decided to revert the PR.
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-43906
Related Pull Requests
#10004
Release Note
Fix plugin link tooltip flickering issue

/e2e

/e2e-test

Cherry pick is scheduled.


TSServer crash

Type: Bug
TSServer keeps crashing.
VS Code version: Code 1.78.1 (6a995c4f4cc2ced6e3237749973982e751cb0bf9, 2023-05-04T09:48:08.683Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: unavailable_offvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
31.89GB (7.68GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

prettier-vscode
esb
9.12.0

copy-filename
jac
2.3.2

remove-unused-imports
kus
1.2.3

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.78.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I am getting the same thing...
Version: 1.78.2 (Universal)
OS: Darwin arm64 22.1.0

Updated to the latest version. Still the same  [error] TSServer exited. Code: 134. Signal: null

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!


Getting data with no content-type from a ResourceStore

How the SimpleResourceStore currently works is to look at the type value of the input RepresentationPreferences to know what kind of representation to return (this corresponds to the incoming Accept header). This is not possible if for some reason we would need a javascript object as representation (e.g., a stream of Quads), since those have no corresponding mediatype. Currently the SimpleResourceStore returns a stream of Quads if the type preference is missing, but this does make some assumptions and might become a problem for non-quad data, or if we have multiple javascript object types.

Currently the SimpleResourceStore returns a stream of Quads if the type preference is missing

On the one hand, this is acceptable behavior; but ResponseWriters should reject streams of non-strings if they do not support them. There could be a quad-based response writer.
But on the other hand, even with a type preference, I don't think it's this particular SimpleResourceStore's job to convert representations. Given that you are implementing SimpleResourceStore with an internal Quad buffer, the knowledge of how to convert quads to text should not be re-encoded across such similar stores (let's say, HdtStore). We rather want the decorator pattern for this, with RepresentationConvertingStore.
So note that both are possible; the store (possibly RepresentationConvertingStore) could convert to an RDF format, or ResponseWriter could. However, given that some stores will directly output in RDF formats (SparqlEndpointStore etc.). we might want to keep that functionality in a store.

a problem for non-quad data

Non-quad data likely has an inherent media type, such as image, or worst case, octets.

Currently the SimpleResourceStore returns a stream of Quads if the type preference is missing

So in conclusion, I think it should always do that, otherwise we re-encoded the serialization logic multiple times. A store should only cater to preferences if it is really easy for them. Let's say for instance a SPARQL endpoint, where it has to ask the backend for some serialization anyway, so it might as well ask for the right one. Or if reading from disk, and the disk happens to have the right format, don't re-serialize. But if conversion is needed; don't bother, someone else should do that.

So in conclusion, I think it should always do that, otherwise we re-encoded the serialization logic multiple times. A store should only cater to preferences if it is really easy for them. Let's say for instance a SPARQL endpoint, where it has to ask the backend for some serialization anyway, so it might as well ask for the right one. Or if reading from disk, and the disk happens to have the right format, don't re-serialize. But if conversion is needed; don't bother, someone else should do that.

So for example, an AclAuthorizer constructor would take as parameter a QuadConvertingStore since it needs Quads to query the acl files (and we just assume for a second that there is not a separate query class). And this QuadConvertingStore would have been made with for example the SimpleResourceStore as constructor parameter (or whatever resource store is being used internally), while other components would use that resource store directly as parameter instead of the converting store?

So for example, an AclAuthorizer constructor would take as parameter a QuadConvertingStore

Yes; exactly (to everything you wrote).
Although very specifically, I think that we could just have one RepresentationConvertingStore with multiple Convertors inside, i.e., QuadConvertingStore = RepresentationConvertingStore with a TextToQuadParser (or an NQuadParser, JsonLdParser, etc.).

How would the QuadConvertingStore know that the SimpleResourceStore can directly return Quads, or any type for that matter? Should ResourceStores perhaps have a function returning which types they actually support?

How would the QuadConvertingStore know that the SimpleResourceStore can directly return Quads, or any type for that matter?

At the moment, I think that QuadConvertingStore would just pass the RepresentationPreferences to the SimpleResourceStore, which would try (only simple efforts!) to satisfy them. If not, it just gives its best-effort representation back (which would be quads).

Should ResourceStores perhaps have a function returning which types they actually support?

That could be different per resource though.

Related since this is probably also going to be the responsibility of the RepresentationConvertingStore, how to know which content types a ResourceStore supports when writing data to it? This can be done entirely static by choosing the correct classes using dependency injection, since we know what the bodyparser is going to return (quad or binary data streams), and we know what the Resource store is going to accept (text/turtle in the case of a file store, probably quads for a sparql store). On the other hand, it might be nice to have a more flexible solution, so the converting store can determine to what type it should convert before calling the the set/add functions. Having something like that might also help when looking into implementing the Accept-Post header as mentioned in #45 .

Related since this is probably also going to be the responsibility of the RepresentationConvertingStore, how to know which content types a ResourceStore supports when writing data to it?

I hadn't designed anything for that yet.
It might be possible for a ResourceStore to implement an additional interface with a getter that indicates accepted formats.
Would not make that part of the main ResourceStore interface though.

since we know what the bodyparser is going to return (quad or binary data streams)

So normally the body parser wouldn't be invoked for a PUT (unless validation happens), but would be for a PATCH.


Update sbt-pgp to 1.1.2

Updates com.jsuereth:sbt-pgp from 1.1.1 to 1.1.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "com.jsuereth", artifactId = "sbt-pgp" }]

Codecov Report

Merging #99 into series/0.6.x will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           series/0.6.x      #99   +/-   ##
=============================================
  Coverage         61.18%   61.18%
=============================================
  Files                11       11
  Lines               304      304
  Branches             21       21
=============================================
  Hits                186      186
  Misses              118      118

Continue to review full report at Codecov.

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


confirm if leaving a modified form without saving

There may be a nicer way to track form modification; not sure.

Relates to #12202.

/botio-linux preview

According to https://developer.mozilla.org/en-US/docs/Web/API/Window/beforeunload_event, beforeunload has undesirable effects in general:

[...] Attaching an event handler/listener to window or document's beforeunload event prevents browsers from using in-memory page navigation caches, like Firefox's Back-Forward cache or WebKit's Page Cache.

Furthermore, years ago, we used to have a beforeunload event listener in another part of the code-base, but removed it because  it caused problems. (Unfortunately I don't remember all the details right now, since it was well over five years ago now.)
All-in-all, it seems really unfortunate as far as I'm concerned to use a beforeunload event unconditionally like this. Can't we at least limit this to documents with forms, to lessen the impact of these changes (unless a solution without beforeunload can be had)?

According to https://developer.mozilla.org/en-US/docs/Web/API/Window/beforeunload_event, beforeunload has really undesirable effects in general:

[...] Attaching an event handler/listener to window or document's beforeunload event prevents browsers from using in-memory page navigation caches, like Firefox's Back-Forward cache or WebKit's Page Cache.

Furthermore, years ago, we used to have a beforeunload event listener in another part of the code-base, but removed it because it caused problems. (Unfortunately I don't remember all the details right now, since it was well over five years ago now.)
All-in-all, it seems really unfortunate as far as I'm concerned to use a beforeunload event unconditionally like this. Can't we at least limit this to documents with forms, to lessen the impact of these changes (unless a solution without beforeunload can be had)?

I took a look at this. It looks like this.bindWindowEvents() happens before _initializeMetadata, and possibly even load, so I don't have a this.documentInfo.IsAcroFormPresent or a this.pdfDocument to refer to. Another solution could be to bind beforeunload elsewhere, outside of bindWindowEvents. Do you have any recommendations here? (Or am I mistaken about any of my assumptions?)

bdahl
aki: what about binding/unbinding on modified?
aki
that makes sense. is there a destroy() or similar?
i'll try that and test
bdahl
may need to add an event that is dispatched from annotation storage on modification and reset?

bdahl
aki: what about binding/unbinding on modified?
aki
that makes sense. is there a destroy() or similar?
i'll try that and test
bdahl
may need to add an event that is dispatched from annotation storage on modification and reset?

This patch doesn't appear to work. I git stashed that, then tried looking for how to send an event from annotation storage. dispatchEvent seems to only be in web/, and message handlers appear to only go between core and display?

bdahl
aki: could add two properties to class AnnotationStorage e.g. onReset and onModified that the viewer could set as callbacks. then when those are called in the viewer you'd add/remove the unload listener

This worked! https://github.com/mozilla/pdf.js/pull/12241/commits/488f6748cd43369d26ee03726042a7309a081da9

/botio-linux preview

/botio-linux unittest

Sorry, but I'm finding the decision to add a global event listener in web/base_viewer.js slightly strange and inconsistent (since there's no others anywhere in the file). Note that when the viewer was originally split into components, we purposely tried to move e.g. any direct DOM accesses out of them.
Also, I understand that the following is more difficult to implement, but nonetheless: Note that this will not warn when switching documents, which means that if you call e.g. PDFViewerApplication.close (or directly call BaseViewer.setDocument when using the viewer components standalone) there won't be any warning about losing form data.

Sorry, but I'm finding the decision to add a global event listener in web/base_viewer.js slightly strange and inconsistent (since there's no others anywhere in the file). Note that when the viewer was originally split into components, we purposely tried to move e.g. any direct DOM accesses out of them.
Hence it does seem, at least to me, that putting that code in PDFViewerApplication.load (in web/app.js) would have seemed slightly more expected overall.

Also, I understand that the following is more difficult to implement, but nonetheless: Note that this will not warn when switching documents, which means that if you call e.g. PDFViewerApplication.close (or directly call BaseViewer.setDocument when using the viewer components standalone) there won't be any warning about losing form data.

Sorry, but I'm finding the decision to add a global event listener in web/base_viewer.js slightly strange and inconsistent (since there's no others anywhere in the file). Note that when the viewer was originally split into components, we purposely tried to move e.g. any direct DOM accesses out of them.
Hence it does seem, at least to me, that placing that code in PDFViewerApplication.load (in web/app.js) instead would have been slightly more expected overall.

Also, I understand that the following is more difficult to implement, but nonetheless: Note that this will not warn when switching documents, which means that if you call e.g. PDFViewerApplication.close (or directly call BaseViewer.setDocument when using the viewer components standalone) there won't be any warning about losing form data.


example cw data source

Example data source for CW on samdisandbox

Closing. Superseeded by two other PRs to this repo.


Please add the following Request.ServerVariable values to reported Exceptions

HTTP_X_REWRITE_URL
SCRIPT_NAME
PATH_TRANSLATED
APP_PHYSICAL_PATH

Would you be happy with these in the RequestInfo extended data or do you want to see them as first class on the request info object..
Would you be able to provide a short description of each of these items (might be needed for xml docs).

Would you be able to provide a short description of each of these items (might be needed for xml docs).

@badgerdigital Would you mind providing a description of these four paths.

I'm closing this unless others ask for it or the community feels it adds value overall. I think those fields only pertain to url rewriting?? Here is how you can do this today:
[Priority(95)]
public class AddCustomServerVariables : IEventPlugin {
    public void Run(EventPluginContext context) {
        var requestInfo = context.Event.GetRequestInfo();
        if (requestInfo == null)
            return;

        HttpContextBase httpContext = context.ContextData.GetHttpContext();

        // if the context is not passed in, try and grab it
        if (httpContext == null && HttpContext.Current != null)
            httpContext = HttpContext.Current.ToWrapped();

        try {
            if (httpContext?.Request?.Headers == null)
                return;

            requestInfo.Data["HTTP_X_REWRITE_URL"] = httpContext.Request.Headers["HTTP_X_REWRITE_URL"];
        } catch (Exception) {}
    }
}

Then register it on startup like this:
ExceptionlessClient.Default.Configuration.AddPlugin<AddCustomServerVariables>();


Update v1 to point to the latest v1.2

Don't both v1 and v1.2 tags, as well as the v1 release, all point to the same commit (d04208cef6e985db0830428975fc2ab3f28d89a1) already?

Yes, it is now, thanks. I think it wasn't before.


[BUG] Systray not working correctly with all:unset

Checklist before submitting an issue

[X] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[X] I have specifically verified that this bug is not a common user error
[X] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug
I use a property Global all: unset necessary for my bars to look good, but systray does not work as it should, it does not allow you to set the background color.  Attached images.

Reproducing the issue
No response
Expected behaviour
No response
Additional context
No response

My systray widget as the css class tray.
You can use my config as a reference to help you theme the systray menus
.tray menu {
	border-radius: 10px;
	padding: 5px 0px;

	background-color: rgba(0, 0, 0, 0.8);

	>menuitem {
		padding: 0px 5px;

		&:disabled label {
			color: gray;
		}

		&:hover {
			background-color: #0e4679;
		}
	}

	separator {
		background-color: #222222;
		padding-top: 1px;

		&:last-child {
			padding: unset;
		}
	}
}

Great, now it works, but with one small detail, the corners look black and remain round even though I indicate in the scss with border-radius: 0. I think I will ignore it for the moment.

Could you send your config ? For me it works with:
.tray menu {
	border-radius: 0;
}

My config. For more details, here you will find eww.yuck and ewe.scss: https://github.com/JuanH88/gh0stzk_1080p/tree/master/config/bspwm/rices/andrea/andy
.tray menu {
border-radius: 0px;
border: 1px solid $borderbg;
padding: 5px 0px;
background-color: $bg;

>menuitem {
	padding: 0px 5px;

	&:disabled label {
		color: gray;
	}

	&:hover {
		background-color: $magenta;
	}
}

separator {
	background-color: $borderbg;
	padding-top: 1px;

	&:last-child {
		padding: unset;
	}
}

}

Well, I copy-pasted your config on my computer and here is the result

However I noticed you were using :pack-direction on the systray widget, which has been removed by the second pull request for the systray (#1059).
Either you are using only the first pull request for the systray (commit 1b819fb), which may explain why it works for me and not for you (even though the second pull request didn't change a lot of things related to menus). Or, you're using a commit that have both pull requests and I have no idea why it doesn't work for you.

Oh, so could it be the composer? In picom I have set corner-radius = 6, now I set it to 60 and the error persists, the menu was not affected, it is exactly the same, maybe it is something else.

No, I don't think it's the compositor, I was saying that you may not have the very latest version of eww, which would explain why it works for me (I'm on the latest commit)

Oh, so could it be the composer? In picom I have set corner-radius = 6, now I set it to 60 and the error persists, the menu was not affected, it is exactly the same, maybe it is something else.

try setting border-radius to zero in your gtk.css. Eww windows somehow respect the system gtk configuration. Had the same issue, but fixed by setting border radius to zero in gtk.css. here is my ~/.config/gtk-3.0/gtk.css
/* Remove rounded corners */
button,
.titlebar,
.titlebar .background,
decoration,
window,
window.background
{
    border-radius: 0px;
}

Thank you all for your contributions that helped me customize my config.
However, I found out a problem with the current approach. That is there are no arrows to indicate sub-menus, check boxes or radio buttons.
After searching for long and some trial and error, I figured out how to get these working ...
(please read the CSS comments I left)
.systray menu {
	border-radius: $borderRadius;
	padding: 0.33em;
	color: $white;
	background-color: $altBg;

    /*
        >menuitem matches menuitem elements nested directly inside menu
    */

	>menuitem {
		padding: 0.3em;

        arrow {

            /*
                Symbolic icons from your icons theme

                EXAMPLE: (using Papirus-Dark icons)
                1. run `find /usr/share/icons/Papirus-Dark/symbolic | grep <YOUR-ICON-NAME>`
                2. use the name without the .svg part like what i did below

             */

            -gtk-icon-source: -gtk-icontheme("pan-end-symbolic");
            color: $white;

            // min-width and min-height are required for the icon to be rendered
            min-width: 1em;
            min-height: 1em;
        }

        /*
            & before pseudo element means nesting selector modifiers
            so instead of

            menuitem:disabled{

            }

            you type:

            menuitem {
                &:disabled{
                }
            }

         */

        radio{
            // min-width and min-height are required for the icon to be rendered
            min-width: 1em;
            min-height: 1em;
            padding-right: 0.5em;

            /*
                How to get the url for icon?

                1. Use your terminal to go to gtk-3.0 sub-directory inside your gtk theme directory,
                   `/usr/share/themes/<YOUR-GTK-THEME>/gtk-3.0/` NOTE: also works for `~/.themes`

                   EXAMPLE: I use arc-dark theme, so for me it's `/usr/share/themes/Arc-Dark/gtk-3.0/`

                2. View contents of either `gtk.css` or `gtk-dark.css`, for me it looks like this

                   `@import url("resource:///org/gnome/arc-theme/gtk-main-dark.css");`

                   the important part is `url("resource:///org/gnome/arc-theme/")`

                3. run `gresource list <NAME-OF-GRESOURCE-FILE>.gresource` in the same directory to
                   list all files inside of it or even easier, pipe it to grep to pick the icon you want.

                   EXAMPLE: `gresource list gtk.gresource | grep radio-checked-dark`

                   RESULT: `
                            /org/gnome/arc-theme/assets/radio-checked-dark.png
<EMAIL_ADDRESS>                           `
                   The @2 variant is a an additional scale
                   SEE: https://docs.gtk.org/gtk3/css-overview.html#using-themed-icons-in-css

                   I'm not concerned so I'll just pick the intended one

                4. Merging the two would be:

                   `url("resource:///org/gnome/arc-theme/assets/radio-checked-dark.png")`
             */

            /*
                Scale 2 can be added through:

                `-gtk-icon-source: -gtk-scaled(url(...), url(..<SCALE-2>..));`

             */

            -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/radio-unchecked-dark.png"));

            &:checked {
                -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/radio-checked-dark.png"));
                &:hover {
                    -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/radio-checked-selected.png"));
                }
            }

            &:hover {
                -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/radio-unchecked-selected.png"));
            }
        }

        check{
            // same as before, min-width and min-height are required for the icon to be rendered
            min-width: 1em;
            min-height: 1em;
            padding-right: 0.5em;
            -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/checkbox-unchecked-dark.png"));

            &:checked {
                -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/checkbox-checked-dark.png"));
                &:hover {
                    -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/checkbox-checked-selected.png"));
                }
            }

            &:hover {
                -gtk-icon-source: -gtk-scaled(url("resource:///org/gnome/arc-theme/assets/checkbox-unchecked-selected.png"));
            }
        }

        &:disabled label { color: $trayDisabled; }
		&:hover {
            arrow { color: $bg; }
			background-color: $accentColor;
			color: $bg;
		}
	}

	separator {
		background-color: $bg;
		padding-top: 0.06em;

		&:last-child {
			padding: unset;
		}
	}
}

So it just becomes a matter of getting the icon theme and GTK theme
As for the problem with rounded-corners, I run river (wayland compositor) and it doesn't give control over compositing as it will require significant change to code and the developer decided to delay eye-candy features, but I found out that changing the color from hex #282828 to rgba rgba(40,40,40,0.99) gives me some kind of  rounded corners (although not truly rounded).
Also, you'll find the tooltip of systray icons transparent ... to control that you can use tooltip CSS selector
tooltip {
    background-color: $altBgRadius;
    border-radius: $borderRadius;
    color: $white;
}

And if someone is interested here are the variables I use in my SCSS file
$bg: #282828;
$white: #d4be98;
$grey: #5a524c;
$red: #ea6962;
$yellow: #d8a657;
$green: #a9b665;
$blue: #7daea3;
$cyan: #89b482;
$purple: #d3869b;

$accentColor: $red;
$altBg: #45403d;
$altBgRadius: rgba(69, 64, 61, 0.99);
$trayDisabled: #7c6f64;

$borderRadius: 0.6em;

Appearance of systray in my system.

* {
  margin: 0;
  margin-top: 0;
  margin-bottom: 0;
  margin-left: 0;
  margin-right: 0;
  padding: 0;
  border: none;
  border-radius: 0;
  min-height: 0;
  min-width: 0;
  font-family: "Dina NF";
  font-size: 12pt;
  label {
    color: #ebdbb2;
  }
  button {
    background: #1c1c1c;
  }
  button:hover label {
    color: #8ec07c;
  }
  tooltip {
    background-color: #1c1c1c;
    color: #ebdbb2;
  }
  trough {
    background: #4c4c4c;
    min-height: 4px;
    min-width: 50px;
    border-radius: 15px;
    highlight {
      background: #8ec07c;
      border-radius: 15px;
    }
  }
}

.systray {
  background-color: #1c1c1c;
  color: #ebdbb2;
  border-radius: 0;
  * {
    border-radius: 0;
    padding: 2px;
    box-shadow: none;
    margin: 0;
  }
}

I stumbled upon this issue because I was seeing weird inconsistent behaviour when using all: unset with eww, using two monitors like so:
eww daemon --debug --config "${SCRIPTPATH}"/eww &

eww open --config "${SCRIPTPATH}"/eww --screen 0 p --arg target-monitor=1
eww open --config "${SCRIPTPATH}"/eww --screen 1 s --arg target-monitor=0

Despite using identical defwindow and css for the two windows, one of them would have weird padding issues, and reloading would sometimes fix the issue, and sometimes break one of the windows.
I ended up solving it by using open-many to achieve the same result:
eww open-many --config "${SCRIPTPATH}"/eww tab:p tab:s \
    --arg p:screen=0            \
    --arg p:target-monitor=1    \
    --arg s:screen=1            \
    --arg s:target-monitor=0

I'm thinking maybe the * { all:unset } clause fucks with either the way GTK or Eww handles styling when opening multiple windows like that, introducing a sort of race condition or conflicting style application, causing the inconsistency.


Add column headers to Hathitrust report

Anticipate header names from Lisa.

file:///C:/Users/william/Downloads/Oct%202022%20Specifications%20for%20submitting%20print%20holdings%20updates%20to%20HathiTrust%20.pdf


Issue 2406/add email template s3 logic

🗣 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.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

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

✅ Post-merge checklist

[ ] Create a release.

Confirmed that vulnerabilities will be addressed in a separate PR.


improving folder structure

Description
improved folder structure by segregating javascript files into components, pages reducers, context directory.
Fixes:  #104

Type of change

[] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project.
[x] My changes generate no new warnings.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK:

Hey @aklite, can you please attach a screenshot which can show me where the Dockerfile is there in the folder?


RuntimeError: CUDA error: an illegal memory access was encountered

The error was occured after changing batch_size from 8 to 4 in cityscapes.yaml.
Found 2975 train images.
Found 500 val images.
Epoch: [1/500] Iter: [1/185] LR: 0.00010049 Loss: 7.71337414:   1%|▌                                                                                | 1/185 [00:03<11:41,  3.81s/it]
Traceback (most recent call last):
File "tools/train.py", line 153, in
main(cfg, gpu, save_dir)
File "tools/train.py", line 97, in main
scaler.scale(loss).backward()
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/_tensor.py", line 255, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/autograd/init.py", line 147, in backward
Variable._execution_engine.run_backward(
RuntimeError: CUDA error: an illegal memory access was encountered
(semseg) liuwang@liuwang-OMEN-30L:~/Documents/projects/semantic-segmentation$ CUDA_LAUNCH_BLOCKING=1 python tools/train.py --cfg configs/DDRNet/cityscapes.yaml
Found 2975 train images.
Found 500 val images.
Epoch: [1/500] Iter: [1/743] LR: 0.00010012 Loss: 7.53493118:   0%|▏                                                                                | 1/743 [00:03<37:21,  3.02s/it]
Traceback (most recent call last):
File "tools/train.py", line 153, in
main(cfg, gpu, save_dir)
File "tools/train.py", line 95, in main
loss = loss_fn(logits, lbl)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/liuwang/Documents/projects/semantic-segmentation/semseg/losses.py", line 43, in forward
return sum([w * self._forward(pred, labels) for (pred, w) in zip(preds, self.aux_weights)])
File "/home/liuwang/Documents/projects/semantic-segmentation/semseg/losses.py", line 43, in
return sum([w * self._forward(pred, labels) for (pred, w) in zip(preds, self.aux_weights)])
File "/home/liuwang/Documents/projects/semantic-segmentation/semseg/losses.py", line 33, in _forward
loss = self.criterion(preds, labels).view(-1)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/nn/modules/loss.py", line 1120, in forward
return F.cross_entropy(input, target, weight=self.weight,
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/nn/functional.py", line 2824, in cross_entropy
return torch._C._nn.cross_entropy_loss(input, target, weight, _Reduction.get_enum(reduction), ignore_index)
RuntimeError: CUDA error: an illegal memory access was encountered
Exception in thread Thread-2:
Traceback (most recent call last):
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/utils/data/_utils/pin_memory.py", line 28, in _pin_memory_loop
r = in_queue.get(timeout=MP_STATUS_CHECK_INTERVAL)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/queues.py", line 116, in get
return _ForkingPickler.loads(res)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/site-packages/torch/multiprocessing/reductions.py", line 289, in rebuild_storage_fd
fd = df.detach()
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/resource_sharer.py", line 57, in detach
with _resource_sharer.get_connection(self._id) as conn:
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/resource_sharer.py", line 87, in get_connection
c = Client(address, authkey=process.current_process().authkey)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/connection.py", line 508, in Client
answer_challenge(c, authkey)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/connection.py", line 752, in answer_challenge
message = connection.recv_bytes(256)         # reject large message
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/connection.py", line 216, in recv_bytes
buf = self._recv_bytes(maxlength)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/connection.py", line 414, in _recv_bytes
buf = self._recv(4)
File "/home/liuwang/Softwares/anaconda3/envs/semseg/lib/python3.8/multiprocessing/connection.py", line 379, in _recv
chunk = read(handle, remaining)
ConnectionResetError: [Errno 104] Connection reset by peer

How large is the RAM on your GPU?

How large is the RAM on your GPU?

3090-24GB

I had an issue with model size and used [get_model_size()](https://github.com/sithu31296/semantic-segmentation/blob/35cc1bd6dba63b7d8bc63195aa7cf1e1637f03fb/semseg/utils/utils.py#:~:text=def-,get_model_size,-(model%3A Union[nn) to get a sense of it.

@StuLiu So many people also faced this illegal memory access error. But I still haven't. What is your pytorch version? If you happen to fix that, leave a solution here.

@StuLiu So many people also faced this illegal memory access error. But I still haven't. What is your pytorch version? If you happen to fix that, leave a solution here.

torch==1.9.0+cu111.
This error didn't occured when I trained SFNet in the cityscapes datasets. However, it occured when training in DDRNet.

The error disappered when I changed ddrnet config file as following.
`
DEVICE          : cuda              # device used for training and evaluation (cpu, cuda, cuda0, cuda1, ...)
SAVE_DIR        : 'output'         # output folder name used for saving the model, logs and inference results
MODEL:
NAME          : DDRNet                                           # name of the model you are using
BACKBONE      : DDRNet-23slim                                                 # model variant
PRETRAINED    : 'checkpoints/backbones/ddrnet/DDRNet23s_imagenet.pth'              # backbone model's weight
DATASET:
NAME          : CityScapes                                          # dataset name to be trained with (camvid, cityscapes, ade20k)
ROOT          : 'data/cityscapes'                                   # dataset root path
IGNORE_LABEL  : 255
TRAIN:
IMAGE_SIZE    : [1024, 1024]    # training image size in (h, w)
BATCH_SIZE    : 16              # batch size used to train
EPOCHS        : 500             # number of epochs to train
EVAL_INTERVAL : 10              # evaluation interval during training
AMP           : false           # use AMP in training
DDP           : false           # use DDP training
LOSS:
NAME          : OhemCrossEntropy          # loss function name (ohemce, ce, dice)
CLS_WEIGHTS   : true            # use class weights in loss calculation
OPTIMIZER:
NAME          : adamw           # optimizer name
LR            : 0.001           # initial learning rate used in optimizer
WEIGHT_DECAY  : 0.01            # decay rate used in optimizer
SCHEDULER:
NAME          : warmuppolylr    # scheduler name
POWER         : 0.9             # scheduler power
WARMUP        : 10              # warmup epochs used in scheduler
WARMUP_RATIO  : 0.1             # warmup ratio
EVAL:
MODEL_PATH    : 'output/DDRNet23_slim_citycapes.pth'     # trained model file path
IMAGE_SIZE    : [1024, 1024]                            # evaluation image size in (h, w)
MSF:
ENABLE      : false                                   # multi-scale and flip evaluation
FLIP        : true                                    # use flip in evaluation
SCALES      : [0.5, 0.75, 1.0, 1.25, 1.5, 1.75]       # scales used in MSF evaluation
TEST:
MODEL_PATH    : 'checkpoints/pretrained/ddrnet/ddrnet_23_city.pth'    # trained model file path
FILE          : 'assests/cityscapes'                    # filename or foldername
IMAGE_SIZE    : [1024, 1024]                            # inference image size in (h, w)
OVERLAY       : true                                    # save the overlay result (image_alpha+label_alpha)
`

The error disappered when I changed ddrnet config file as following. ` DEVICE : cuda # device used for training and evaluation (cpu, cuda, cuda0, cuda1, ...) SAVE_DIR : 'output' # output folder name used for saving the model, logs and inference results
MODEL: NAME : DDRNet # name of the model you are using BACKBONE : DDRNet-23slim # model variant PRETRAINED : 'checkpoints/backbones/ddrnet/DDRNet23s_imagenet.pth' # backbone model's weight
DATASET: NAME : CityScapes # dataset name to be trained with (camvid, cityscapes, ade20k) ROOT : 'data/cityscapes' # dataset root path IGNORE_LABEL : 255
TRAIN: IMAGE_SIZE : [1024, 1024] # training image size in (h, w) BATCH_SIZE : 16 # batch size used to train EPOCHS : 500 # number of epochs to train EVAL_INTERVAL : 10 # evaluation interval during training AMP : false # use AMP in training DDP : false # use DDP training
LOSS: NAME : OhemCrossEntropy # loss function name (ohemce, ce, dice) CLS_WEIGHTS : true # use class weights in loss calculation
OPTIMIZER: NAME : adamw # optimizer name LR : 0.001 # initial learning rate used in optimizer WEIGHT_DECAY : 0.01 # decay rate used in optimizer
SCHEDULER: NAME : warmuppolylr # scheduler name POWER : 0.9 # scheduler power WARMUP : 10 # warmup epochs used in scheduler WARMUP_RATIO : 0.1 # warmup ratio
EVAL: MODEL_PATH : 'output/DDRNet23_slim_citycapes.pth' # trained model file path IMAGE_SIZE : [1024, 1024] # evaluation image size in (h, w) MSF: ENABLE : false # multi-scale and flip evaluation FLIP : true # use flip in evaluation SCALES : [0.5, 0.75, 1.0, 1.25, 1.5, 1.75] # scales used in MSF evaluation
TEST: MODEL_PATH : 'checkpoints/pretrained/ddrnet/ddrnet_23_city.pth' # trained model file path FILE : 'assests/cityscapes' # filename or foldername IMAGE_SIZE : [1024, 1024] # inference image size in (h, w) OVERLAY : true # save the overlay result (image_alpha+label_alpha) `

your training is single gpu? I have used your same config, single gpu is 8000,48G, but i face the same problem. But when i change the IMAGE_SIZE to [512, 512], the training is ok.

I met the problem also. I think the error is caused by the label. I use torch 1.8.1+cu111, but it throws this error by the loss calculation. After checking the loss, I find the label id is wrong here. The label id -1 is invalid when calculating the CE loss. Am I right?

I met the problem also. I think the error is caused by the label. I use torch 1.8.1+cu111, but it throws this error by the loss calculation. After checking the loss, I find the label id is wrong here. The label id -1 is invalid when calculating the CE loss. Am I right?

I remove
'''-1 : -1'''
in ID2TRAINID and it works

I met the problem also. I think the error is caused by the label. I use torch 1.8.1+cu111, but it throws this error by the loss calculation. After checking the loss, I find the label id is wrong here. The label id -1 is invalid when calculating the CE loss. Am I right?

I remove '''-1 : -1''' in ID2TRAINID and it works

Thank you it's work for me too.

the same error.
for custom datasets, the ignore_label should be set carefully.


nodejs: going to drop instrumentation v1 support

Description

node.js tracer is working on dropping v1 support
https://github.com/DataDog/dd-trace-js/pull/3827

Motivation

v2 > v1

Workflow

⚠⚠ Create your PR as draft
Follow the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Once your PR is reviewed, you can merge it! :heart:
Reviewer checklist

[x] Check what scenarios are modified. If needed, add the relevant label (run-parametric-scenario, run-profiling-scenario...). If this PR modifies any system-tests internal, then add the run-all-scenarios label (more info).
[x] CI is green

[ ] If not, failing jobs are not related to this change (and you are 100% sure about this statement)

[x] if any of build-some-image label is present

is the image labl have been updated ?
just before merging, locally build and push the image to hub.docker.com

[x] if a scenario is added (or removed), add (or remove) it in system-test-dasboard nightly

Please check tests/appsec/waf/test_telemetry.py::Test_TelemetryMetrics::test_headers_are_correct test.
I think it is failing too


(noHDD) - Connect external HDD/SSD

Hi Everyone
Ive just got a RaspiBlitz from Fulmo but its not recognising the SSD.
Raspberry PI4, 8GB RAM
Running 1.6.3
1000 GB SSD (San Disk SSD plus)
Ive run the SSH query and get the following:
If HDD is connected - please report the result of the following command:
sudo /home/admin/config.scripts/blitz.datadrive.sh status

admin@raspberrypi:~ $ sudo /home/admin/config.scripts/blitz.datadrive.sh status
RASPIBLITZ DATA DRIVE Status
BASICS
isMounted=0
isBTRFS=0
SETUP INFO
isSSD=0
hddCandidate=''
hddBytes=0
hddGigaBytes=0
hddPartitionCandidate=''
RAID
isRaid=0
raidCandidates=0
SWAP
isSwapExternal=0
Could this be a power issue? im using the supplied Raspberry PI official USB-C power supply.

Thanks Ben10Brasileiro
I've checked the SSD is correctly connected and it is so have contacted Fulmo. Unfortunately I don't have another device to test the drive on so dont know if its the drive or the connection on the board that's causing the issue.

@walker377 Was this solved?

@nyxnor I got the node up and running by buying an external SSD case and connecting via USB. I couldn't get the installed SSD board to work. Waiting for an update from Fulmo on the issue.

@walker377 Pls close this. Hope your problem was solved with the shop by now.

@nyxnor Will close now as its a hardware issue not a Rasipblitz issue. Still waiting for Fulmo to resolve.


Unusual behavior with Firefox when recording

Prerequisite
Firefox 47.0, Mac OS X 10.11.5 El Capitan.
What I Have Do
Goto https://www.webrtc-experiment.com/msr/video-recorder.html and begin recording.
What Happened
Record is a success. However only the first segment. The following segments are only fragments of the first segment that there are so many of them, causing OutOfMemoryError and therefore crashed the browser.
What Is Expected
There should have been a steady output of video blob for each 5 seconds. Chrome 51 is known to work expected.
Workaround
None. I have to temporarily disable recording from Firefox. I have to make checks for user, telling them to leave or use Chrome if the user is using Firefox.

Recording and watching the individual videos worked perfectly for me. I can't watch the full videos, but I don't think that is your issue.
Firefox 47.0.1


Fix FancyFileWidget

Various fixes for the fancy file upload widget.
To fix #123 and some more problems I found.

A test is failing, I have no idea what is going on there, maybe you have an idea?

I can't look into this right now but quickly looking at the error from travis:
1) Bootstrap\Test\TestCase\View\Helper\FormHelperTest::testFormSecuredFileControl
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    0 => 'Contact.picture-text'
+    0 => 'Contact.picture]-text'
     1 => 'Contact.picture.name'
     2 => 'Contact.picture.type'
     3 => 'Contact.picture.tmp_name'
     4 => 'Contact.picture.error'
     5 => 'Contact.picture.size'
 )

Maybe you added a typo somewhere?

Yes I got this failure also with phpunit, I don't know where the extra ']' is coming from. I am not as experienced with Cake's templating so I thought I'd ask you first.
I will try to debug the test. In my own use case everything works as expected. That's at least something ;).

I think I got something similar and the beginning, and it's why I switch to the [text] field, so the issue might be that you reverted this change... I cannot investigate more right now unfortunately.

@ypnos-web Could you provide me a working example (e.g. from your own use case)? I will see if I can do something about that false phpunit warning.

This would be a working example:
    public function testFormSecuredFileControl() {
        $this->form->setConfig('useCustomFileInput', true);
        // Test with filename, see issue #56
        $this->assertEquals([], $this->form->fields);
        $this->form->file('picture');
        $expected = [
            'picture-text',
            'picture.name', 'picture.type',
            'picture.tmp_name', 'picture.error',
            'picture.size'
        ];
        $this->assertEquals($expected, $this->form->fields);
    }

@ypnos-web I was talking about a working application (because I did not manage to create one that reproduces everything... ), but if this test fixes the issue, feel free to update the TestCase with it.

I rewrote the code to work in both cases. I feared it would bite back at us as soon as someboday actually has the file as a member of another entity.

Codecov Report

Merging #127 into master will increase coverage by 0.39%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #127      +/-   ##
============================================
+ Coverage     82.55%   82.94%   +0.39%
- Complexity      331      334       +3
============================================
  Files            17       17
  Lines          1043     1061      +18
============================================
+ Hits            861      880      +19
+ Misses          182      181       -1

Impacted Files
Coverage Δ
Complexity Δ

src/View/Widget/FancyFileWidget.php
98% <100%> (+2.65%)
12 <3> (ø)
:arrow_down:

src/View/Helper/EasyIconTrait.php
100% <0%> (ø)
7% <0%> (+2%)
:arrow_up:

src/View/Helper/HtmlHelper.php
84.35% <0%> (+0.21%)
39% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

@ypnos-web Thanks for the update.
Could you merge testFormSecuredFileControl and testFormSecuredFileControl2 (place the 3 asserts in a single testFormSecuredFileControl)? If you don't have time, I'll merge and update it myself this week-end.

Sorry I didn't spend more time on how I could merge them the first time.

I was actually thinking about simply putting the two assertEquals in the same method, but this work, so I'm merging, thanks ;)

Well in that case I reckon the form would've needed to be resetted in between, or the second assert include the fields from the first.


Slight improvements to error handling in TestPurifiedMixedState.

This PR slightly improves the TestPurifiedMixedState unit test to help diagnose failures such as that seen in https://dev.azure.com/ms-quantum-public/Microsoft Quantum (public)/_build/results?buildId=21163&view=logs&j=b78a1ffd-6a4f-5c9a-fd79-c70d65470011&t=a3b90293-a206-53fd-a1c1-8b7b4b27ab86.

/azp run

/azp run


Added placeholder folder for Medical Accuracy effort

Just created an empty folder for Medical Accuracy working group to allow to collaborate

@googlebot I signed it!