Avoidance Interface CI

This PR is an attempt to get an automated test on the avoidance interface going to check if any future PR would brake the interface.
Test: the vehicle will fly a mission where flying a direct line between the two waypoints leads to a collision with a wall. The avoidance maneuver is simple enough to assure that the local_planner can always find a path around the wall. If the vehicle cannot finish the mission before a timeout, the test fails.
Current Work in Progress: I need to setup the catkin environment for the avoidance to run. All the steps are done in the script rostest_avoidance_run.sh. I see that in the other ROS tests the equivalent script is called here. How can I change it to run rostest_avoidance_run.sh for the avoidance test?
Future Work: we can use the Gazebo collision topic to detect if the vehicle has crashed

@mrivi I did option 2. Jenkins is fine with the test now, but there seems to be other issues. Take a look at http://ci.px4.io:8080/blue/organizations/jenkins/PX4_misc%2FFirmware-SITL_tests/detail/pr-avoidance_ci_rebased/13/pipeline/82 and https://logs.px4.io/plot_app?log=69c95f98-26a4-4d87-a739-73c6baa97fc2

@lamping7 thank you!!!
I think the problem is that the avoidance is not enabled (param MPC_OBS_AVOID is not set to 1). I'll quickly try to set that in the iris config. I think we need another iris vehicle config (like those *_iris_vision, *_irsi_opt_flow) to set the parameter.

I'm seeing `Laser Plugin (ns = )  <tf_prefix_>, set to ""
Aborted (core dumped)`

@lamping7 @dagar avoidance test is now working! thank you for the help!
I'll put the param set for MPC_OBS_AVOID on a different target than the normal iris probably tomorrow.

Awesome. The Jenkins failure is another data link lost issue.

@dagar good to merge?

@lamping7 fair points.

I have modified the avoidance.plan to include lading. This is what it does locally on my machine. Lets check if it matches the output on the ci

the obstacle is defined in this world file that is in the avoidance repository

Can you use 5 meters for take off alt to match other MC tests?
The takeoff altitude isn't the problem.

[ERROR] [1541060312.830477210, 44.581000000]: FCU: Failsafe enabled: no global position
gzserver: /usr/include/boost/smart_ptr/shared_ptr.hpp:648: typename boost::detail::sp_member_access<T>::type boost::shared_ptr<T>::operator->() const [with T = gazebo::rendering::DepthCamera; typename boost::detail::sp_member_access<T>::type = gazebo::rendering::DepthCamera*]: 'Assertion px != 0' failed.
I'd probably set the gazebo verbose flags in the launch file for testing to see what is going on.

@mrivi before there was a mission check failure on the takeoff altitude. I can increase it but I also need to modify the obstacle.
Let me first get the output from avoidance on the console.

@lamping7 before there was a mission check failure on the takeoff altitude. I can increase it but I also need to modify the obstacle.

Yes, I saw that, but I'm thinking it was a result of not having global position. You don't need to increase it. I was just suggesting it for consistency. I haven't pulled up the world to visualize it, so I didn't know if you could or not easily.

@mrivi going to try on my side and let you know

@TSC21 @lamping7 I run the contains on my laptop and retrieve the spwan log
[xmlrpc][INFO] 2018-11-23 11:50:54,698: XML-RPC server binding to <IP_ADDRESS>:0
[xmlrpc][INFO] 2018-11-23 11:50:54,698: Started XML-RPC server [http://9a1b26ca3e60:44181/]
[rospy.impl.masterslave][INFO] 2018-11-23 11:50:54,698: _ready: http://9a1b26ca3e60:44181/
[xmlrpc][INFO] 2018-11-23 11:50:54,698: xml rpc node: starting XML-RPC server
[rospy.registration][INFO] 2018-11-23 11:50:54,699: Registering with master node http://localhost:11311
[rospy.init][INFO] 2018-11-23 11:50:54,702: ROS Slave URI: [http://9a1b26ca3e60:44181/]
[rospy.init][INFO] 2018-11-23 11:50:54,703: registered with master
[rospy.rosout][INFO] 2018-11-23 11:50:54,703: initializing /rosout core topic
[rospy.rosout][INFO] 2018-11-23 11:50:54,705: connected to core topic /rosout
[rospy.simtime][INFO] 2018-11-23 11:50:54,712: initializing /clock core topic
[rospy.simtime][INFO] 2018-11-23 11:50:54,716: connected to core topic /clock
[rosout][INFO] 2018-11-23 11:50:54,721: Loading model XML from Gazebo Model Database
[rosout][INFO] 2018-11-23 11:50:54,721: Waiting for service /gazebo/spawn_sdf_model
[rosout][INFO] 2018-11-23 11:50:54,724: Calling service /gazebo/spawn_sdf_model
[rospy.internal][INFO] 2018-11-23 11:50:54,734: topic[/clock] adding connection to [http://9a1b26ca3e60:35775/], count 0
[rospy.internal][INFO] 2018-11-23 11:50:54,882: topic[/rosout] adding connection to [/rosout], count 0
[rosout][INFO] 2018-11-23 11:50:55,604: Spawn status: SpawnModel: Successfully spawned entity
[rospy.core][INFO] 2018-11-23 11:50:55,605: signal_shutdown [atexit]
[rospy.internal][INFO] 2018-11-23 11:50:55,609: topic[/rosout] removing connection to /rosout
[rospy.internal][INFO] 2018-11-23 11:50:55,609: topic[/clock] removing connection to http://9a1b26ca3e60:35775/
[rospy.impl.masterslave][INFO] 2018-11-23 11:50:55,609: atexit  ```

It seems that the model was successfully spawned

@mrivi you are talking about the below error?
gzserver: /usr/include/boost/smart_ptr/shared_ptr.hpp:648: typename boost::detail::sp_member_access<T>::type boost::shared_ptr<T>::operator->() const [with T = gazebo::rendering::DepthCamera; typename boost::detail::sp_member_access<T>::type = gazebo::rendering::DepthCamera*]: Assertion `px != 0' failed.

About
gzserver: /usr/include/boost/smart_ptr/shared_ptr.hpp:648: typename boost::detail::sp_member_access<T>::type boost::shared_ptr<T>::operator->() const [with T = gazebo::rendering::DepthCamera; typename boost::detail::sp_member_access<T>::type = gazebo::rendering::DepthCamera*]: Assertion `px != 0' failed.

Issue a make clean. Then, try putting source /usr/share/gazebo/setup.sh in https://github.com/PX4/Firmware/blob/master/Tools/setup_gazebo.bash#L17 and then run the pipeline again to see if it works.

@TSC21 both errors. Point cloud timeout ... is a consequence of the other error.

@mrivi have you tried to run Gazebo in gdb and check the backtrace? Try passing the debugger argument on https://github.com/PX4/Firmware/blob/master/Tools/sitl_run.sh.

We need avoidance to be versioned, which likely means building it in a container that's tagged.
This could be done with a Dockerfile that lives in https://github.com/PX4/avoidance.

Can we get this current branch squashed and rebased on master?

@dagar we will have a stable version of avoidance sometimes this week

https://github.com/PX4/Firmware/issues/11523

Getting there... rebase after https://github.com/PX4/Firmware/pull/11606 if everything checks out. Seems promising on the SITL side.

The simulation is now working. The remaining problem is that the avoidance doesn't let the vehicle land. There are two possible solutions:

disable the avoidance setpoint when the current triplet is type SETPOINT_TYPE_LAND or when the nav state is AUTO_LOITER on the Firmware side
fix the issue on the avoidance side

@dagar the avoidance test is now working against the stable avoidance release.
Anything else needed before merging?

@dagar can we merge this?

Also, do we want to cut down the messages spamming the console multiple times a second?
[ INFO] [1552481031.530698617, 33.248000000]: [SP]Tree (72 nodes, 2 path nodes, 10 expanded) calculated in 6.55ms.
[ INFO] [1552481031.553791537, 33.272000000]: [WG] Speed adapted WP: [10.778049 1.757204 4.033766].
[ INFO] [1552481031.589228580, 33.304000000]: [WG] Speed adapted WP: [10.777305 1.752048 4.026687].
[ INFO] [1552481031.621368949, 33.340000000]: [WG] Speed adapted WP: [10.779245 1.758199 4.021173].
[ INFO] [1552481031.653143228, 33.372000000]: [WG] Speed adapted WP: [10.167651 2.287094 4.085506].
[ INFO] [1552481031.653242088, 33.372000000]: [OA] Planning started, using 3 cameras

@lamping7 I addressed your comments.
I killed all the avoidance outputs from the console.
Yes, the warning on the third waypoint is due to the relative height being set to -1. This is necessary to make the vehicle land and disarm. Once the avoidance can handle landing correctly we can change the relative altiude back to 0.

dev call discussion:Get a clear failure message FYI: @mrivi

I rebased because I realized that the OBS_AVOID parameter wasn't up to date. It switched from MPC_OBS_AVOID to COM_OBS_AVOID in #11550

If we want to merge our efforts, you could cherry-pick 70c9c78dc9fb27a509d11ca718890ff72b187153

It has been long six months to get this PR working so I am not going to change anything at the moment. This is a good first step and I would like to keep the improvements in a separate PR.
@dagar could you please review it? I really would like to see this merged rather sooner than later.

Ok, I'll work on top of this then. Let's get it in

Thanks everyone!


Fixed broken RunSamples.cmd link

Hi @cosminBoaca, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Current coverage is 82.55% (diff: 100%)

Merging #512 into master will decrease coverage by 0.05%

@@             master       #512   diff @@
==========================================
  Files            61         61
  Lines          6869       6869
  Methods        1122       1122
  Messages          0          0
  Branches        882        882
==========================================
- Hits           5675       5671     -4
- Misses          871        874     +3
- Partials        323        324     +1

Powered by Codecov. Last update d20d281...7f61857

LGTM

LGTM


Weather data not correct (location)

Did you read the instructions?

[X] I have read the README, including the known-limitations section.

The problem
I have provided the configuration with the LAT / LON data from my location. The data in Home Assistant is not the same as the data I find on "weerlive" for the same location (city). I see some other value's at max temperature / air pressure.
Data in HA

Data on Weerlive

Hourly forecast

Integration version
2.1.0
Home Assistant version
2024.5.2 / 12.3
Python version
3.11.8
Home Assistant log
No response
Diagnostics
No response
Additional information
No response

Can you please provide your diagnostics file as well, that will make it easier to investigate

Sure!
config_entry-knmi-29ad1f977f6c674ee403f306d3954a75.json

When I look at the json data (parsed and raw) from the diagnostics file you provided the max temp for today is 24
...
      "wk_verw": [
        {
          "dag": "13-05-2024",
          "image": "regen",
          "max_temp": 24,
          "min_temp": 16,
        }
...

While indeed looking into the hourly forecast, the "current hour" is 24 while a few upcoming hours have 25:
...

      "uur_verw": [
        {
          "uur": "13-05-2024 12:00",
          "timestamp":<PHONE_NUMBER>,
          "image": "zonnig",
          "temp": 24,
          ...
        },
        {
          "uur": "13-05-2024 13:00",
          "timestamp":<PHONE_NUMBER>,
          "image": "zonnig",
          "temp": 25,
          ...
        },
        {
          "uur": "13-05-2024 14:00",
          "timestamp":<PHONE_NUMBER>,
          "image": "halfbewolkt",
          "temp": 25,
          ...
        },
        {
          "uur": "13-05-2024 15:00",
          "timestamp":<PHONE_NUMBER>,
          "image": "zonnig",
          "temp": 25,
          ...
        },
        {
          "uur": "13-05-2024 16:00",
          "timestamp":<PHONE_NUMBER>,
          "image": "bewolkt",
          "temp": 25,
          ...
        },
        {
          "uur": "13-05-2024 17:00",
          "timestamp":<PHONE_NUMBER>,
          "image": "regen",
          "temp": 22,
          ...
        },
...

Since the value is taken from the API, I'll need to report this to the API supplier

Dankje Sander, we wachten af!

I contacted the supplier today to address this issue. Apologies for the delay; I was dealing with some distractions. The problem has been addressed, and we should monitor the situation over the coming days to ensure it has been resolved.

This is still an issue:

Also interesting to see the max temp sensor is changing from 17/19 often


Create CommandInfo message type from application that is used to give more context in Command History

+string SourceApplication
+string SourceUser


CarouselController is imported from both

Steps to Reproduce
Run project with Flutter 3.24.0
Expected results
Build Done!
Actual results
Could not build the precompiled application for the device.
Error (Xcode): ../../../.pub-cache/hosted/pub.dev/camerawesome-2.0.1/lib/src/widgets/filters/awesome_filter_selector.dart:3:1: Error: 'CarouselController' is imported from both 'package:carousel_slider/carousel_controller.dart' and 'package:flutter/src/material/carousel.dart'.
About your device

Brand
Model
OS

Apple
iPhone 14 Pro
18

Your flutter version
3.24.0

android version of the issue:
`Running Gradle task 'assembleDebug'...
../../.pub-cache/hosted/pub.dev/camerawesome-2.0.1/lib/src/widgets/filters/awesome_filter_selector.dart:3:1: Error: 'CarouselController' is imported from both 'package:carousel_slider/carousel_controller.dart' and 'package:flutter/src/material/carousel.dart'.
import 'package:flutter/material.dart';
^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/carousel_slider-4.2.1/lib/carousel_slider.dart:9:1: Error: 'CarouselController' is imported from both 'package:carousel_slider/carousel_controller.dart' and 'package:flutter/src/material/carousel.dart'.
import 'carousel_controller.dart';
^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/camerawesome-2.0.1/lib/src/widgets/filters/awesome_filter_selector.dart:27:42: Error: 'CarouselController' is imported from both 'package:carousel_slider/carousel_controller.dart' and 'package:flutter/src/material/carousel.dart'.
final CarouselController _controller = CarouselController();
^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/carousel_slider-4.2.1/lib/carousel_slider.dart:48:15: Error: 'CarouselController' is imported from both 'package:carousel_slider/carousel_controller.dart' and 'package:flutter/src/material/carousel.dart'.
: CarouselController() as CarouselControllerImpl,
^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/carousel_slider-4.2.1/lib/carousel_slider.dart:62:15: Error: 'CarouselController' is imported from both 'package:carousel_slider/carousel_controller.dart' and 'package:flutter/src/material/carousel.dart'.
: CarouselController() as CarouselControllerImpl,
^^^^^^^^^^^^^^^^^^
Target kernel_snapshot_program failed: Exception
FAILURE: Build failed with an exception.`

https://github.com/Apparence-io/CamerAwesome/pull/502 addressed to fix it

Also #499 and this issue are duplicates if I'm not mistaken

And we have three open pull requests to fix this issue:
#502
#504
#505
#505 being the one with the least number of changes

Any way, in master branch you already have similar fix so it just can be released...

Any way, in master branch you already have similar fix so it just can be released...

You must have looked in the wrong repo...
For the record:

#502 puts the code up to date (removing lint issues), and is a version using carousel_slider_plus
#504 is a fork of the previous reusing carousel_slider
#505 "only" fixes the issues, without fixing the lint issues

@millerf fair enough, when can we expect an update on a release that fixes this issue?
@kuhnroyal

Hi everyone,
Sorry for not being so active here.
I am reviewing the code and trying it on iOS and Android.
Will merge it after that

Awesome, thank you for the update!

Version 2.1.0 published
Thank you all for your help 🙏🏻

You're Welcome!

@g-apparence #505 is a duplicate of #502 / #504


Create colorado.txt

Add colorado.edu

https://www.colorado.edu/cs/

@dustin-hooks I am sorry, the domain colorado.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Pack at https://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Doesn't help at all !!

I know I need a PAT to link LCS and VSTS (DevOps). I've now got one, but the ridiculous situation is that when I try to set-up the link between LCS and my DevOps project, the poorly written UI tells me that I need a Personal Access Token and then directs me to this URL !! Absolute nonsense and typical of the circles we all run around to get any Microsoft Integration working as intended!!

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

ID: 53f99d2d-363b-d01d-039d-4fdc636fa2df
Version Independent ID: f47330d5-e2de-36a4-e5b3-5bc77505b161
Content: Authenticate with personal access tokens - Azure DevOps
Content Source: docs/organizations/accounts/use-personal-access-tokens-to-authenticate.md
Product: devops
Technology: devops-security
GitHub Login: @chcomley
Microsoft Alias: chcomley

@advent77 Thank you for the feedback and we apologize for the issues you're having.
Please see if this post addresses the issue and consider reporting this issue to the product team here.
If there is a step in particular that is causing issue, please reference it below and we can assign an author to further investigate.

Thanks for response.
I got the issue fixed!
The problem is the form which is displaying the message that I need a token!
The form is not long enough! The field for entry of the token is “hidden” from view... you need to scroll down to see the input field!
Can I suggest that this form gets FIXED!!
Regards
Joe
(iPhone mail)

On 8 May 2020, at 07:23, Matthew Sebolt<EMAIL_ADDRESS>wrote:
@advent77 Thank you for the feedback and we apologize for the issues you're having.
Please see if this post addresses the issue and consider reporting this issue to the product team here.
If there is a step in particular that is causing issue, please reference it below and we can assign an author to further investigate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Closing this issue now.


Cannot add Tuya curtains / loonas_curtain

Describe the bug
Cannot add Tuya curtains (device is loonas_curtain), after entering all details Im getting "Sorry, there is no support for this device."
To Reproduce
Install tuya-local later version from repository.
Add integration, and in configuration page enter correct details:

device_id: [my device ID from iot.tuya]
device IP: [my local IP of curtain motor that is pingable]
local key: [device's key from iot.tuya]

Expected behavior
Device recognized and added into HA
Additional context
A) device details from iot.tuya. Note that product_id matches loonas_curtain
{
"result": {
"active_time":<PHONE_NUMBER>,
"category": "cl",
"category_name": "Curtain",
"create_time":<PHONE_NUMBER>,
"gateway_id": "",
"icon": "smart/icon/ay1513046654908xshtz/c7188920b20c257c673caa7a8d059808.png",
"id": [HIDDEN],
"ip": "[HIDDEN]",
"lat": "54.6100",
"local_key": "[HIDDEN]",
"lon": "24.9200",
"model": "",
"name": "curtains",
"online": true,
"owner_id": "152641179",
"product_id": "wraaecjqfiirj8go",
"product_name": "Curtain Motor",
"sub": false,
"time_zone": "+03:00",
"update_time":<PHONE_NUMBER>,
"uuid": "[HIDDEN]"
},
B) DPS IDs in iot.tuya also mathes ones in yaml file
Control = 1
Curtain Position Setting = 2
Current Curtain Position = 3
Motion Direction = 5
Fault = 12
Assistant log (Settings / System / Logs):
Device matches None with quality of 0%. DPS: {"updated_at":<PHONE_NUMBER>.0017009, "2": 0, "3": 100, "5": "forward"}
If applicable, add screenshots to help explain your problem.

It appears from the log that the "control" dp needs to be optional.

Worked:


fix(firestore-send-email) empty template attachments fallback to message attachments

fixes: https://github.com/firebase/extensions/issues/716

Codecov Report

Merging #719 (a6dfd42) into next (2509537) will decrease coverage by 0.20%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             next     #719      +/-   ##
==========================================
- Coverage   50.65%   50.46%   -0.19%
==========================================
  Files          30       30
  Lines        1313     1318       +5
  Branches      188      196       +8
==========================================
  Hits          665      665
- Misses        647      652       +5
  Partials        1        1

Impacted Files
Coverage Δ

firestore-send-email/functions/src/index.ts
5.21% <0.00%> (-0.15%)
:arrow_down:

https://github.com/firebase/extensions/pull/713 fixes the test issue in this PR

Can we stop committing .js files for this extension?


setprojectdir virtualenvwrapper style

I love pyenv and pyenv-virtualenv, but the only thing I wish for is the optional auto-switching to a project directory when the env is activated, ala virtualenvwrapper.
Keep up the great work yyuu!

Use direnv or some equivalent. Don't do everything with swiss army knife.
http://direnv.net/

Thanks for the point, direnv looks perfect

How does direnv work in that case?
If I understand it correctly you want to cd somewhere after pyenv activate foo, no?

I could switch it around and pyenv activate foo after cd ing into the project directory

Ah sorry I was misunderstanding. Anyway pyenv-virtualenv will never support the behaviour (to avoid complicated problems in users's shell).

Its fine direnv worked a treat, its actually better as cd-ing into the dir transparently changes the env whih is arguably more intuitive anyway.
Thanks


support for WebStorm IDE

Hello. Please could you add support for WebStorm? In your manual I read about VSCode extension and via CLI.
Could I use your plugin with another IDE like WebStorm?

Thank you for posting this issue!
As far as I've looked at the docs, it may not be possible to use this plugin with WebStorm.
https://www.jetbrains.com/help/webstorm/using-stylelint-code-quality-tool.html
To use this plugin, you need to specify the syntax provided by this plugin in the custom syntax option.
Probably WebStorm does not support custom syntax option.
To use this plugin with WebStorm, you need to request support for custom syntax option in WebStorm or request support for custom syntax option in stylelint configuration.

I don't understand. But WebStorm support stylus syntax. Could you help with this?

But WebStorm support stylus syntax.

It is not WebStorm's stylus syntax support. Needs support for stylelint's custom syntax option.
WebStorm allows you to apply stylelint configuration, but stylelint configuration does not support custom syntax option.
Therefore, you need to request custom syntax option support for WebStorm or stylelint configuration.
VS Code extensions support the custom syntax option.
https://github.com/stylelint/vscode-stylelint#stylelintcustomsyntax
I think WeStorm can use stylelint-plugin-stylus if it adds support for similar options.

"request support for custom syntax option in stylelint configuration" - how? I could write something in stylelint config?

Currently, custom syntax option is only available via CLI flags or the Node.js API.
I think stylelint-plugin-stylus can be used with WebStorm by allowing custom syntax option to be set from the stylelint configuration.
This is similar to the following issues.
https://github.com/stylelint/stylelint/issues/3859
https://github.com/stylelint/stylelint/issues/3858

I create issue https://github.com/stylelint/stylelint/issues/4843
Please check that I correctly write and complement it please)


feat: add shared lockbox spec

Description
Introducing the SharedLockbox spec and its corresponding upgrade process. This enhancement improves the Superchain’s interoperable ETH withdrawal user experience and prevents withdrawal failures caused by insufficient ETH in the OptimismPortal. Additionally, the dependency manager role is being added to SuperchainConfig to manage the network’s dependency set.
Additional context
Design doc: https://github.com/ethereum-optimism/design-docs/pull/84

Dropping this comment here as per request from @tynes. I think the most valuable think to add to this PR would be to clearly define the invariants that need to be maintained when we introduce this component.
Some examples here would be:

Only authorized Portal contracts should be able to use the lockbox
Portal contracts can only be authorized by
Only safe Portal contracts can be authorized (now we need to define "safe")

I think point (3) is the most important and it would be valuable to explore all of the ways in which this condition could be violated. What is "safe"? We need to be exhaustive.


Native Chrome Extension

Progress:
The extension currently sends the documentInfo object:
const documentInfo = {
    title: document.title,
    url: window.location.href,
    html: document.documentElement.outerHTML
};

On every change in the DOM.
Example of how does the console receive messages:

so on

Good work @KrishPatel13 !
Next up is connecting the extension to python and having the python module the updates.

@abrichr, Just to be clear, Currently I have a python module, browser.py that gets the updates in DOM and then it sends the reply_num (update count) and the whole documentInfo object via native messaging to the background.js script which logs in the console.
See here:
https://github.com/MLDSAI/OpenAdapt/pull/230/files#diff-c51a7447dc4c4bc4b75e74075f2d00cdf140a026fe7dc02e6de6d64fcec06be5R44
My screenshot here is the exact description of the browser.py in action with background and content script :)
https://github.com/MLDSAI/OpenAdapt/pull/230#issue-1746148482

Refer

From this, I interpret as, making a new python module that sends and then logs only the updated part of the DOM.
Is that right ? :)

https://learn.microsoft.com/en-us/windows/win32/ipc/pipes

Super early but looks very relevant

@abrichr
On researching and reading about pipes, I found that:
Creating a platform-independent solution that uses named pipes can be challenging, because named pipes work differently on Unix/Linux versus Windows systems. So essentially we would need different scripts for Windows and Unix/Linux and/or macOs.
A more platform-independent approach might be to use Python's built-in support for inter-process communication (IPC) via sockets. This approach works similarly to pipes but should function the same way across different operating systems.
Or
Another way I can think of is to make this work by writing into a temp txt file. And then we want to run openadapt, we can get the data from it. If this works, we can measure the performance difference. If is huge, we can work on improving the performance. But, at least we would have something working the end :)
Please let me know your thoughts.
THank you.

Please remove the .idea directory 🙏

A Demo: https://mldsai.slack.com/archives/D056HTS5KEG/p1687015265698859

Next Steps:

[ ] Integrate the named-piped scripts with browser extension to print the actual dom changes in extension
[ ] Integrate the whole environment with openadapt.

Closing in favour of: https://github.com/OpenAdaptAI/OpenAdapt/pull/337


Suboptimal "location searched" feedback on "no matching package"

If I put in a manual path for a dependency, and get something subtly wrong like the name of the package itself, the error report is not as helpful as it could be.
Example:
% cargo new spelling
% cargo new my-proj
% cd my-proj/
% cat >> Cargo.toml <<EOF
> [dependencies.zpelling]
> path = "/tmp/demo/spelling"
> EOF
% cargo build
no matching package named `zpelling` found (required by `my-proj`)
location searched: file:///private/tmp/demo/my-proj
version required: *
%

In this case, while it is easy to see that zpelling is different from spelling, it is worth noting that the path "/tmp/demo/spelling" is not being printed anywhere in the error output.

Triage: the output I get today is:
 $ cargo build
error: no matching package named `zpelling` found (required by `my-proj`)
location searched: file:///tmp/demo/spelling
version required: *

Since the path /tmp/demo/spelling is being printed, this seems fixed and I'm closing.


Create a getting started page in the docs repository

See http://help.exercism.io/getting-started-with-go.html as an example.
If appropriate, suggest a linter (see exercism/docs#54).

Superseded by docs/ locally. Fixed in #14


Support for in-memory SQLite

What should be done?
Currently SQLite backend only accepts directory path as URI. Is it possible to add support for in-memory? Sample URIs:
:memory:
file::memory:

Thank you.

Yes, that's planned.

@tbnguyen1407 it should work already with URI file:dir/?mode=memory. dir/ should be an existing directory, but it will not be actually used.
I will leave the issue open for documenting this.

@AlekSi thanks I just tested this to just use current directory and it seems fine.
file:./?mode=memory


Rollout UI / server-side paging

Some additional major changes

More flexibel swu parsing. Covers three different cases.
Rollout creation/deletion and pause/unpause addressing #27
JavaScript formatting, modernisation and lint issue cleanup
Server-side filtering/search/paging

Let me know when ready for review.  Biome doesnt seem to do HTML formatting (which is mildly annoying), but up to you which you prefer.  If we want to use prettier, see this comment - https://github.com/UpstreamDataInc/goosebit/pull/37#discussion_r1697274155

Let me know when ready for review. Biome doesnt seem to do HTML formatting (which is mildly annoying), but up to you which you prefer. If we want to use prettier, see this comment - #37 (comment)

@b-rowan Ready.


Annotation not working with OSD Reference Strip

Greetings and thanks for this nice plugin!
I have run into issues trying to annotate using the OSD with a reference image strip
 var viewer = OpenSeadragon({
        id: "openseadragon1",
        prefixUrl: "/userContent/lib/openseadragon-2.2.1/images/",
        sequenceMode: true,
        showReferenceStrip: true,
        referenceStripScroll: 'horizontal',
        viewportMargins:{bottom:120},
	tileSources:  [tilesArray]
    });
     viewer.initializeAnnotations();

I can always annotate the first image fine, but once I move on to the next image it stops working. The mouse just pans the image instead of annotating. Is a viewer with a reference strip supported?
Attaching an example below. Thanks again for your open source work!

Hey @imoutsatsos I have the same issue, but I am just experiencing it.  Have you tried to work around it in any way?  There is another post in the OpenSeadragon issues with a guy who is putting a reference strip outside of the viewer.  I wonder if that could help at all.  Just seeing if you have made any progress or knows what doesn't work.
Thanks

@imoutsatsos, thanks to you for your interest and for taking the time of creating this detailed issue. I have to confess that I just never thought about reference strips, I don't think it's supported to be honest.
If you have an example repo that you can share in github I can take a look and work on implementing this feature in the next version of the plugin, or in this one.

@Emigre Thank you for the latest updates. Even without the reference strip I'm unable to get annotations to work. Now, instead of using a reference strip I load images in an specific positions in the viewer.world. In this case the plugin adds the buttons, but I can't annotate at all. The draw button just moves the images

and using this code to activate annotations
var viewer = OpenSeadragon({
     id: "openseadragon1",
     prefixUrl: "/userContent/js/openseadragon-2.2.1/images/",
     preserveViewport: true,
     crossOriginPolicy:"Anonymous",
     viewportMargins: {
       bottom: 120
     },
     tileSources: []
   });
   viewer.initializeAnnotations();

@imoutsatsos, if you could provide me a github repo with an example of what you are doing, it would help me to identify the issue.

@Emigre Thank you for reviewing this. I will create a use case on github next week for you to be able to test the environment I work with.

@imoutsatsos hi, I hope that all is fine. I'm not sure if I got back to you regarding this? Apologies if not. I'm really sorry but I lack the time to invest in this plugin, I'm planning on terminating it. Sorry for not getting back to you earlier. Please, if you need this plugin, keep a fork and feel free to continue its development. I hope that all is well, best regards.

Thanks for the update @Emigre ! Hope all is well with you as well. I think your plugin was a great start! I will fork it in case I need to go back to use it in the future. But I understand how we all refocus on what is important for our daily work. Best regards

@Emigre I'm sorry to hear that, but I certainly understand!
I believe there are people who have found this plug-in useful… It would be great to see if we could find someone to adopt it.
@imoutsatsos any interest in being that person?

Thanks @iangilman ! Yes, sure... If there is somebody interested in it, we can do that.

@Emigre and @iangilman thanks for the invitation! I'm currently using OSD but have not revisited annotations. I will review the plugin source code to understand whether this is something I can tackle. Best regards

@imoutsatsos Great! We mostly need someone willing to "mind the store"; host the plug-in on their GitHub, respond to issues, review pull requests.
My strategy for open source management without burnout is to help people help themselves… I manage the OpenSeadragon community, but I don't do coding on it myself. When people come with questions or bugs, I give them information for how they can help fix the problems. Otherwise all I would ever do is fix things for people! It's working so far…

@iangilman, that makes sense, sounds like a good approach.
If not @imoutsatsos, maybe you are interested?...

Well, it doesn't appear anyone has raised their hand, so we can shift this over to the OpenSeadragon organization if you want. I'm actually not entirely sure how one does that… Do you know?


Feature: remember Draw # between captures.

remember Draw # between captures.
In OpenGL comes true this behaviour:
capture, analyze
capture, analyze
and user always must open the same "draw #" to compare.
Can RenderDoc preserve "draw #" between captures?
it will save time for searchin same "draw #" in next capture .

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


Cannot show logs in new relic dashboard when running logs-in-context-log4j2

Description
After configuring my newrelic configurations, call the ping endpoint in my local
while I can see the transaction section  has this /ping endpoint
I cannot see any logs in the trace
The pic shows the empty logs:

Steps to Reproduce

configure newrelic configurations
start logs-in-context-log4j2 in my local
Checking the newrelic dashboard

Expected Behavior
Should see some log messages in the dashboard
Your Environment
JDK: 17
OS: MAC M1

Hi @TDtianzhenjiu! I actually did some work recently to adjust this example to run purely with docker (i.e. no requirement to have java installed locally to run). In the process of doing this, I realized that some log records being generated by the app were being rejected because the log bodies were longer than the 4095 character limit New Relic imposes, so the logs were being dropped and generating NrIntegrationErrors. In the process of dockerizing the example, I added a collector processor to truncate log record bodies which are over the limit, which should fix issue you're seeing.
Marking this as closed as I believe its fixed, but please re-open if you give it a try and still don't see the logs.


doc: lift supported ceph version for snapshot/clone

snapshot and clone should be supported from octopus 15.2.4 instead of 15.2.3.
fixes #2731
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@Madhu-1 the fixes come back here :)

@Madhu-1 the fixes come back here :)

Thanks, Fixed now PTAL

@humblec PTAL

In short, it looks like the snapshot info code path got into escape path in initial testing, Considering 15.2.15 is running now and we can still revisit the requirement of supporting it from the user who are still on 15.2.3 , I think we can update the documentation about this change. Thanks.

@mergifyio rebase


master tag of docker image not update

What did you do?
Use the master branch for testing new features
https://quay.io/repository/coreos/prometheus-operator?tab=tags
What did you expect to see?
master tag of docker image is up to day.
What did you see instead? Under which circumstances?
master tag of docker image is outdated.

This is a fair request. I will make sure this is automated for the future. Thanks for the notice.

The master tag is now automatically updated on successful CI runs from master. Closing here.


Add missing methods for FormData type

Fixes https://github.com/Microsoft/TypeScript/issues/14813 by updating the existing append method to better match the Web APIs and adding missing methods. Please let me know if there is anything I should add or change, thanks!

@NColey,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

My apologies @mhegazy , I had been referencing the definitions here: https://developer.mozilla.org/en-US/docs/Web/API/FormData. I just updated the PR to match the appropriate definitions from the spec, thank you for sharing the link with me!

@Kovensky Is there another area beyond just as a typedef where it might
need to be added? I just updated my PR with the typedef for
FormDataEntryValue but the test is failing with the following error Error: The baseline file verifyDefaultLIb_dom.errors.txt has changed. which is
both a file that I don't seem to have and also one I suspect I shouldn't
touch. I can't seem to figure out what in my changes would have prompted
that but I'm guessing there is another area of the code where I would need
to add FormDataEntryValue?
On Tue, Mar 28, 2017 at 9:51 PM, Diogo Franco<EMAIL_ADDRESS>wrote:

@Kovensky commented on this pull request.
In baselines/dom.generated.d.ts
https://github.com/Microsoft/TSJS-lib-generator/pull/219#discussion_r108577174
:

@@ -3796,7 +3796,12 @@ declare var FocusNavigationEvent: {
}

interface FormData {

append(name: any, value: any, blobName?: string): void;

append(name: string, value: any, fileName?: string): void;

Ah, looking in the full diff I understand better what you meant.
It seems you need to add FormDataEntryValue to the json file as a typedef
(search "typedef" in it to find examples)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TSJS-lib-generator/pull/219#discussion_r108577174,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIWynBs5mf9v29zZFB4IkDat2cAFBeWTks5rqbk1gaJpZM4MppcA
.

thanks!

Thank you!


hotfix/vending -> dev

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Make the whole editorcontent clickable

Only the text part is clickable
Make the whole box clickable

Fixed in https://github.com/CS3219-AY2425S1/cs3219-ay2425s1-project-g14/tree/post-ms6


Problem with using htmlunit together with spring

I use the following code:
        java.util.logging.Logger.getLogger("com.gargoylesoftware").setLevel(Level.OFF);
        java.util.logging.Logger.getLogger("org.apache.commons.httpclient").setLevel(Level.OFF);

        WebClient webClient = new WebClient(BrowserVersion.CHROME);
        webClient.getOptions().setThrowExceptionOnScriptError(false);
        webClient.getOptions().setJavaScriptEnabled(true);

        HtmlPage page = webClient.getPage("https://wp.pl");

When I use it in plain java application it works perfectly, but when I put it in springBootApplication it produce a lot of errors:
htmlunitErrorWithSpring.txt

I think the difference between the two is that Spring Boot properly configures the logging and the vanilla project most likely doesn't.
So I'd assume that the same errors are logged in the vanilla project but you don't see them.
Is the side by chance yours? It would be helpful to provide the error messages with unminified versions of the javascript.

@pikonradNCU can we close this?

I stop to use htmlunit, i just call post which is called by the page I wanted to analyze, and received direct data.


⚠ Bofrost.be has degraded performance

In 0f8ff79, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1037 ms

Resolved: Bofrost.be performance has improved in 804737c after 6 minutes.


Adding the transition effect on the buttons

The UI doesn't seems to be modern looking as  it lacks some smooth transition on buttons and some other elements
Also there is some issue with the "How it works for tailors" part

I can contribute to the resolve the issue.
This is regarding the gssoc'23

Go for it @raghav1030

heyy there! I'm not being able to run it on my local machine. after forking, cloning when i do npm install , the dependencies are not getting installed. I read the readme and it says i have to enter google api key but i don't have one. Please help me with it?

I'm finding  it difficult to run the repo on my local machine.
When i do npm run dev it shows :
Error [ERR_MODULE_NOT_FOUND]: Cannot find module<EMAIL_ADDRESS>imported from C:\Users\91767\Desktop\StichHub\client\StichHub\node_modules\magic-string\dist\magic-string.es.mjs
Did you mean to import @jridgewell/sourcemap-codec/dist/sourcemap-codec.umd.js?
at new NodeError (node:internal/errors:387:5)
at finalizeResolution (node:internal/modules/esm/resolve:429:11)
at moduleResolve (node:internal/modules/esm/resolve:1006:10)
at defaultResolve (node:internal/modules/esm/resolve:1214:11)
at nextResolve (node:internal/modules/esm/loader:165:28)
at ESMLoader.resolve (node:internal/modules/esm/loader:844:30)
at ESMLoader.getModuleJob (node:internal/modules/esm/loader:431:18)
at ModuleWrap. (node:internal/modules/esm/module_job:76:40)
at link (node:internal/modules/esm/module_job:75:36)
I think the following error can be due to :

api keys that are needed to be inserted in .env file. I don't knnow how to find them.
when i do npm install , there are some dependency warning that come as  -

npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving<EMAIL_ADDRESS>npm WARN Found<EMAIL_ADDRESS>npm WARN node_modules/react
npm WARN<EMAIL_ADDRESS>from the root project
npm WARN 64 more (@emotion/react, @emotion/styled, ...)
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/ink/node_modules/react-reconciler
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/ink
npm WARN
npm WARN Conflicting peer dependency<EMAIL_ADDRESS>npm WARN node_modules/react
npm WARN peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/ink/node_modules/react-reconciler
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/ink
Please help me with it...

Try to manually install the package which is not found
With npm install packagename

can i work on this under gssoc'23

Hey  @raghav1030, Can you please assign this to me. I love to changes in the buttons and cards to look better and attractive hover effects and transition effect which is eye catcher for the user.


The role with name undefined cannot be found when using existing Cognito and existing data source.

Environment information
System:
  OS: macOS 14.5
  CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
  Memory: 44.37 MB / 16.00 GB
  Shell: /bin/zsh
Binaries:
  Node: 20.13.1 - /usr/local/bin/node
  Yarn: 1.22.19 - /usr/local/bin/yarn
  npm: 10.5.2 - /usr/local/bin/npm
  pnpm: undefined - undefined
NPM Packages:
  @aws-amplify/backend: 1.0.3
  @aws-amplify/backend-cli: 1.0.4
  aws-amplify: Not Found
  aws-cdk: Not Found
  aws-cdk-lib: Not Found
  typescript: 5.4.5
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Description
How to reproduce

Manually install AWS Amplify following this instructions https://docs.amplify.aws/react/start/manual-installation/
Connect to existing AWS cognito following this instructions https://docs.amplify.aws/react/build-a-backend/auth/use-existing-cognito-resources/#use-auth-resources-with-an-amplify-backend
Connect to an existing Mysql Database inside RDS following this instructions https://docs.amplify.aws/react/build-a-backend/data/connect-to-existing-data-sources/connect-postgres-mysql-database/
run npx ampx sandbox

The Sandbox failed to build with this error. See Your CloudFormation for details.
data/amplifyData/AuthRolePolicy01 (amplifyDataAuthRolePolicy01567A5654) Resource handler returned message: "The role with name undefined cannot be found. (Service: Iam, Status Code: 404, Request ID: xxxx" (RequestToken: xxxxxxx, HandlerErrorCode: NotFound)
and
data/amplifyData/UnauthRolePolicy01 (amplifyDataUnauthRolePolicy01355B9DCF) Resource handler returned message: "The role with name undefined cannot be found. (Service: Iam, Status Code: 404, Request ID: xxx)" (RequestToken:xxxx, HandlerErrorCode: NotFound)
and more...
Expected result
Sandbox builds successfully and uses existing datasource and existing Cognito.
Note
If a define a new Cognito resource with defineAuth( inside amplify/auth/resources.ts everything works as expected.

Update
If i use a custom authorization rule with a Lambda function. see https://docs.amplify.aws/react/build-a-backend/data/customize-authz/custom-data-access-patterns/
Everything works as expected.

Hey @marcomilon, thank you for reaching out. Connecting an existing auth resource to the backend is currently not yet supported and is being tracked on https://github.com/aws-amplify/amplify-backend/issues/1548 RFC(request for comment)
But i do think we could add a callout on the documentation on this behavior. Marking this as documentation.

Closing this as a callout had been adding to the documentation referencing the RFC: https://docs.amplify.aws/react/build-a-backend/auth/use-existing-cognito-resources/#use-auth-resources-with-an-amplify-backend


Audio/Video received doesn't play

Keeps loading but nothing happens, but if i enter to the web it plays both of them.

It's a new problem from my updates or you had that problem before?

I've started using this messenger for desktop yesterday, i've installed your version because of the fixes like start minimized on startup and still active for updates. Also the video not playing was reported to the original developer, can you check both Audio/Video at least for me none of them are working in the application but plays normally in the browser. Thanks

Just checked and I have the same problem. I will check what I can do. My guess is maybe its an old version of chromium running and it doesnt support the HTML5. If thats the case then its easy to fix, otherwise it will need further investigation...

Thanks, keep it up :)

Ok I got time to check the problem. My opinion is that its a framework based problem. Now, they released yesterday a beta of a new version of NWJS. Many changes occurred in the framework, but I will try to migrate the app tomorrow. It should resolve the issue.

Ok I got time to check the problem. My opinion is that its a framework based problem. Now, they released yesterday a beta of a new version of NWJS. Many changes occurred in the framework, but I will try to migrate the app tomorrow. It should resolve the issue.

NWJS doesnt support it for the moment, so cant do much about it.


Upgrade reliable to latest

Fix a memory leak when the sequence buffer wraps
Fix receiving packets close to max_packet_size

Do all tests pass in Yojimbo?

@gafferongames I'm about to find out as I'm also setting up the CI stuff for yojimbo so we can get pull request test feedback. ;)

Perfect

But yes, the tests pass on this pull request :) (I just realized that even if the GitHub Workflow is merged, it might not run on a pull request that was created before it was merged.)


Refactor: CLI binaries integration

:scroll: Description

to enable automatic updates in CI, as seen in https://github.com/getsentry/sentry-unity/pull/855
as a side-effect, closes #10
:bulb: Motivation and Context

:green_heart: How did you test it?
:pencil: Checklist

[x] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

CI for CLI update PRs
packaging in CI so that the person who does publishing to pub.dev doesn't forget to download the CLI?

@bruno-garcia please give permission to @vaind so we don't need to approve CI every time.


Fix getMaxZoom to use props.maxZoom

getMaxZoom looks like it was copied from getRadius, but was not changed to return maxZoom.
This corrects that issue, maxZoom is broken without this fix.

@jeremiahrhall Hey, would you consider merging this and updating the release on npm? Thank you.

Yes these changes make sense. Can you make the corresponding changes to the files in '/src' and run 'npm run compile' and push those changes? There are more complete instructions in the Submitting a Pull Request section of the 'CONTRIBUTING.md' file in the root of the project.

@jeremiahrhall Thank you, yes, I forgot to modify the src file for that last change. I'll follow the contributing.md file instructions.
Also it seems like this still may not fix maxZoom, I will check against my local map.

@jeremiahrhall Ok, I've made those changes according to contributing.md.
I seem to have misinterpreted "maxZoom", which Leaflet/Leaflet.heat seems to define as
maxZoom - zoom level where the points reach maximum intensity (as intensity scales with zoom), equals maxZoom of the map by default
I assumed it was a threshold for showing/hiding.
Anyway, these changes are still relevant to fixing the feature, thanks for the responsiveness.

I'll give these changes a run through later today.

Hi @hughlomas, Apologies for the delay. This looks good. I will merge and publish the new version!


Support Azure CosmosDB

For a production high-volume bundle environment, we would like to support a proper database on Azure. Let's add support for CosmosDB since it will work well with storing documents, and has the query support we are planning to add to Porter later in v1.

Storage plugin is no longer special. To support CosmosDB the MongoDB API should be enabled on CosmosDB


WIP: Fix bug Exit manager bug with beta19rc5

After removing selected Exit and mesh ip from the config and storing necessary data in a lazy static, only this ip
was being called when requesting exit information, which led to all exits getting only test exit information

Since we remove mesh_ip and use subnet.ip() instead, this relies on the fact that the subnets in the config are in the form of
<mesh_ip>/, so for example: "fd00"::1337/120" instead of "fd00::1300/120"


support null value include in @Scopes

I got error of below log and fixed it.
This error occurred by null value in @Scopes.
/home/user/app/node_modules/sequelize-typescript/lib/utils/object.js:10
            .getOwnPropertyNames(source)
             ^
TypeError: Cannot convert undefined or null to object
    at Function.getOwnPropertyNames (<anonymous>)
    at /home/user/app/node_modules/sequelize-typescript/lib/utils/object.js:10:14
    at Array.forEach (native)
    at deepAssign (/home/user/app/node_modules/sequelize-typescript/lib/utils/object.js:8:13)
    at assign (/home/user/app/node_modules/sequelize-typescript/lib/utils/object.js:34:21)
    at /home/user/app/node_modules/sequelize-typescript/lib/utils/object.js:11:46
    at Array.forEach (native)

Thank you - looks good :)


Add a new usage for codegen::clang::If

As Issue #917 has posted,  a different usage of If may be needed. This PR provides a new method in codegen::lang::If which will make it easier for users to call If and reduces confusion. The new usage will be like this.
llvm::Value *condition = ...;
llvm::Value *a = codegen::lang::If::If2(cg, condition, [&] {
  cg.CallPrintf("then branch", {});
  return cg.Const32(1);
}, [&] {
  cg.CallPrintf("else branch", {});
  return cg.Const32(2);
});

Support for codegen::lang::If will be added later.

Coverage decreased (-0.03%) to 75.198% when pulling db25d6b3a74753800eabc470722382b2aad7d931 on MalcolmUnWang:fornewif into dd29079012644e47392b9d015187d8ae1dadd569 on cmu-db:master.

Before reviewing this, let's discuss back in the original thread.

I don't think we'll be going with this approach, so I'm closing.


fix: typo on architecture page

noticed this small typo while scrolling earlier

@znd4 thanks for finding this!


possible timeout on mariadb start

Describe the bug
While the upgrade itself went smooth i got an error that mariadb.service could not get started in the final phase of the upgrade. I belive this to be an timing problem
2023-09-05 23:17:09,464 - INFO - Running: ['/usr/bin/systemctl', 'enable', 'mariadb.service', 'plesk-task-manager.service', 'plesk-web-socket.service', 'sw-cp-server.service', 'sw-engine.service']. Output:
2023-09-05 23:17:09,478 - INFO - Synchronizing state of mariadb.service with SysV service script with /usr/lib/systemd/systemd-sysv-install.
2023-09-05 23:17:09,479 - INFO - Executing: /usr/lib/systemd/systemd-sysv-install enable mariadb
2023-09-05 23:17:09,678 - INFO - Created symlink /etc/systemd/system/mysql.service → /usr/lib/systemd/system/mariadb.service.
2023-09-05 23:17:09,678 - INFO - Created symlink /etc/systemd/system/mysqld.service → /usr/lib/systemd/system/mariadb.service.
2023-09-05 23:17:09,678 - INFO - Created symlink /etc/systemd/system/multi-user.target.wants/mariadb.service → /usr/lib/systemd/system/mariadb.service.
2023-09-05 23:17:09,678 - INFO - Created symlink /etc/systemd/system/multi-user.target.wants/plesk-task-manager.service → /usr/lib/systemd/system/plesk-task-manager.service.
2023-09-05 23:17:09,678 - INFO - Created symlink /etc/systemd/system/multi-user.target.wants/plesk-web-socket.service → /usr/lib/systemd/system/plesk-web-socket.service.
2023-09-05 23:17:09,678 - INFO - Created symlink /etc/systemd/system/multi-user.target.wants/sw-cp-server.service → /usr/lib/systemd/system/sw-cp-server.service.
2023-09-05 23:17:09,679 - INFO - Created symlink /etc/systemd/system/multi-user.target.wants/sw-engine.service → /usr/lib/systemd/system/sw-engine.service.
2023-09-05 23:17:09,866 - INFO - Command '['/usr/bin/systemctl', 'enable', 'mariadb.service', 'plesk-task-manager.service', 'plesk-web-socket.service', 'sw-cp-server.service', 'sw-engine.service']' finished successfully
2023-09-05 23:17:09,866 - INFO - Running: ['/usr/bin/systemctl', 'start', 'mariadb.service', 'plesk-task-manager.service', 'plesk-web-socket.service', 'sw-cp-server.service', 'sw-engine.service']. Output:
2023-09-05 23:17:45,179 - INFO - Job for mariadb.service failed because the control process exited with error code.
2023-09-05 23:17:45,179 - INFO - See "systemctl status mariadb.service" and "journalctl -xe" for details.
2023-09-05 23:17:45,181 - ERROR - Command '['/usr/bin/systemctl', 'start', 'mariadb.service', 'plesk-task-manager.service', 'plesk-web-socket.service', 'sw-cp-server.service', 'sw-engine.service']' failed with return code 1
2023-09-05 23:17:45,182 - ERROR - Failed: starting plesk services!. The reason: Command '['/usr/bin/systemctl', 'start', 'mariadb.service', 'plesk-task-manager.service', 'plesk-web-socket.service', 'sw-cp-server.service', 'sw-engine.service']' returned non-zero exit status 1.
2023-09-05 23:17:46,327 - ERROR - centos2alma process has been failed. Error: Failed: starting plesk services!. The reason: Command '['/usr/bin/systemctl', 'start', 'mariadb.service', 'plesk-task-manager.service', 'plesk-web-socket.service', 'sw-cp-server.service', 'sw-engine.service']' returned non-zero exit status 1.

Right after the upgrade failed i looked into systemctl status mariadb and found the database to be running about 5 seconds later
● mariadb.service - MariaDB 10.2.41 database server
   Loaded: loaded (/usr/lib/systemd/system/mariadb.service; enabled; vendor preset: disabled)
  Drop-In: /etc/systemd/system/mariadb.service.d
           └─limits.conf, migrated-from-my.cnf-settings.conf
   Active: active (running) since Tue 2023-09-05 23:17:51 CEST; 2min 32s ago
     Docs: man:mysqld(8)
           https://mariadb.com/kb/en/library/systemd/
  Process: 43458 ExecStartPost=/bin/sh -c systemctl unset-environment _WSREP_START_POSITION (code=exited, status=0/SUCCESS)
  Process: 42637 ExecStartPre=/bin/sh -c [ ! -e /usr/bin/galera_recovery ] && VAR= ||   VAR=`cd /usr/bin/..; /usr/bin/galera_recovery`; [ $? -eq 0 ]   && systemctl set-environment _WSREP_START_POSITION=$VAR || exit 1 (code=exited, status=0/SUCCESS)
  Process: 42610 ExecStartPre=/bin/sh -c systemctl unset-environment _WSREP_START_POSITION (code=exited, status=0/SUCCESS)
 Main PID: 42949 (mysqld)
   Status: "Taking your SQL requests now..."
    Tasks: 35 (limit: 4308)
   Memory: 675.8M
   CGroup: /system.slice/mariadb.service
           └─42949 /usr/sbin/mysqld --basedir=/usr

Sep 05 23:17:50 abc123.mydomain.xy systemd[1]: Starting MariaDB 10.2.41 database server...
Sep 05 23:17:51 abc123.mydomain.xy mysqld[42949]: 2023-09-05 23:17:51<PHONE_NUMBER>16128 [Warning] option 'ft_min_word_len': unsigned value 0 adjusted to 1
Sep 05 23:17:51 abc123.mydomain.xy mysqld[42949]: 2023-09-05 23:17:51<PHONE_NUMBER>16128 [Note] /usr/sbin/mysqld (mysqld 10.2.41-MariaDB) starting as process 42949 ...
Sep 05 23:17:51 abc123.mydomain.xy mysqld[42949]: 2023-09-05 23:17:51<PHONE_NUMBER>16128 [Warning] Could not increase number of max_open_files to more than 20000 (request: 96186)
Sep 05 23:17:51 abc123.mydomain.xy systemd[1]: Started MariaDB 10.2.41 database server.

In case im right, would it be possible to increase the timeout to 120 seconds for example ?
Also : I tried to upgrade the system again one day later without any changes and it went through without errors.
Feedback archive
centos2alma_feedback.zip

As I can see, startup timeout for MariaDB  systemd module is 900 seconds for 10.2 installed from official repository:
# Set an explicit Start and Stop timeout of 900 seconds (15 minutes!)
# this is the same value as used in SysV init scripts in the past
# Galera might need a longer timeout, check the KB if you want to change this:
# https://mariadb.com/kb/en/library/systemd/#configuring-the-systemd-service-timeout
TimeoutStartSec=900
TimeoutStopSec=900

Additionaly, for the standard AlmaLinux8 MariaDB 10.3.35, the timeout is 300 seconds, that should be fine as well.
It appears that in this case, MariaDB 10.3.35 is only being installed after the startup of the services, which is not the expected behavior. I suspect that this may lead to issues during service startup since we trying to start an old version of MariaDB.
Therefore, I propose moving the installation of the expected MariaDB version 10.3 before starting Plesk services to address this problem.

Related PR is #143

I've merged the changes, and believe the problem should be resolved now. Please feel free to reopen the ticket if it's not.


Use raise instead of flunk

What was the end-user problem that led to this PR?
This PR doesn't fix any end-user problem.
This PR fixes a developer problem.
RSpec doesn't provide flunk method. It's a method name in test-unit and minitest.
What was your diagnosis of the problem?
3edfddbf7ebe4b942e1e2b6d693afd582e9a8147 may use flunk accidentally.
What is your fix for the problem, implemented in this PR?
My fix uses raise instead of flunk.
Why did you choose this fix out of the possible options?
I don't have another option.

I've cherry-picked the commit in #7526 to fix unrelated CI failure.

@bundlerbot merge


Chain.children not returns its children in registered order

Version
chainer==5.0.0b4
Problem
In the documentation of chainer.Chain, children is explained as follows:
    The child link is accessible via :meth:`children`
    generator, which returns a generator running through the children in
    registered order.

However, Chain.children returns its children sorted by their names.
    def children(self):
        d = self.__dict__
        for name in sorted(self._children):
            yield d[name]

Cause
I think the documentation is wrong.

I confirmed that this phenomenon is from chainer==v5.0.0b4.
import chainer
import chainer.links as L

print(chainer.__version__)

class Model(chainer.Chain):
    def __init__(self):
        super(Model, self).__init__()
        with self.init_scope():
            self.a3 = L.Linear(1, 1)
            self.a1 = L.Linear(1, 1)
            self.a2 = L.Linear(1, 1)

model = Model()
for link in model.children():
    print(link.name)

5.0.0b4
a1
a2
a3

5.0.0b3
a3
a1
a2

Thanks for reporting. As you suggested, we should update the docs.
It seems we failed to remove the sentence "in registered order" in #2657.
In and after Chainer v2, we are using set to keep children, which does not guarantee the order.


Errors in a Vector result only in an Execution failed message and no details.

If the user enters some invalid code within a Vector, this results in only an "Execution failed" and no way to find the error.

The IDE converts the execution error cause by Decimal.nan into PanicSentinel according to its documentation:

This tracing is enabled by the active intervention of the runtime instrumentation, and does
not function in textual mode.

This tracing isn't going to happen in the CLI - only IdExecutionInstrument does the conversion.
However PanicSentinel isn't a warning - e.g. it doesn't get propagated further...

What is the desired behavior? Shall we treat PanicSentinel as a Warning and propagate it like that? I can easily achieve

with following few lines change:
diff --git engine/runtime-instrument-id-execution/src/main/java/org/enso/interpreter/instrument/IdExecutionInstrument.java engine/runtime-instrument-id-execution/src/main/java/org/enso/interpreter/instrument/IdExecutionInstrument.java
index c868592d51..a710420955 100644
--- engine/runtime-instrument-id-execution/src/main/java/org/enso/interpreter/instrument/IdExecutionInstrument.java
+++ engine/runtime-instrument-id-execution/src/main/java/org/enso/interpreter/instrument/IdExecutionInstrument.java
@@ -244,7 +248,10 @@ public class IdExecutionInstrument extends TruffleInstrument implements IdExecut
       if (exception instanceof TailCallException) {
         onTailCallReturn(exception, Function.ArgumentsHelper.getState(frame.getArguments()));
       } else if (exception instanceof PanicException panicException) {
-        onReturnValue(frame, new PanicSentinel(panicException, context.getInstrumentedNode()));
+        var ctx = EnsoContext.get(context.getInstrumentedNode());
+        var sentinel = new PanicSentinel(panicException, context.getInstrumentedNode());
+        var warning = Warning.create(ctx, panicException.getMessage(), context.getInstrumentedNode());
+        onReturnValue(frame, WithWarnings.appendTo(ctx, sentinel, warning));
       } else if (exception instanceof AbstractTruffleException) {
         onReturnValue(frame, exception);
       }

Would you consider that a fix, @jdunkerley?

The #7385 pull request shows a conservative approach that just turns PanicSentinel into an object that has WarningsLibrary.hasWarnings(...). The more radical change is shown at jtulach/NoPanicSentinel branch - it removes PanicSentinel altogether - but then we'd have to fix a lot of Runtime*Tests. CCing @4e6


add example flag and ant compile

You will need to create a pull request on the IntelliJ Community repository. This repository will automatically be updated when changes to the java-decompiler (fernflower) occur.

well, thanks


tech: Extracts build phase steps into standalone scripts

📝 Summary of Changes
Takes the scripts in BuildPhases (specifically for building Rust dependencies) and moves them into a couple new bash script files that can be reused and referenced to, along with option to be used in CI pipelines.
Cleans up some existing scripts.
🔨 How To Test

[ ] CI passes

Closing due to changes in branch configuration while this PR was opened.


Support for logout function

Added support for Passport logout POST function.

Thanks for the PR @harmonmi. We're actually building these client libraries from a templating system we created. But I'll get that API into our template so that this method will be available.

Pushed a rebuild of the client, see d8b4192. Thanks for pointing out our missing API.


Add RVF support

This MR request adds RVF support (#33).

lgtm!

@edersondisouza : I have added support to Gstreamer (link). Feel free to take a look.

@edersondisouza : I have added support to Gstreamer (link). Feel free to take a look.

Great! I'll take a look in the next few days =D


feat: support explicit segment colors

In our experience at Janelia, it is helpful for proofreading tools to have explicit control over the colors of bodies in the segmentation.  This code adds that capability to Neuroglancer.  The display state of a layer now can include a map like the following, making one body purple and another green:
"segmentColors": { "193125660": "#ff00ff", "206326381": “#00ff00” }
The implementation uses a new Uint64Map class that is quite similar to the Uint64Set class.  I added some unit tests for both of those classes.
For slice rendering, there is a new SegmentStatedColorShaderManager class that uses a HashMapShaderManager and adds shader code to find a segment’s explicit color if present.  For mesh rendering, the getObjectColor function now checks for the segment’s explicit color.

@googlebot I signed it!

Overall this looks great!  The only change I'd like to see is with the color parsing.

Thanks!


WinAppDriver resize window

Hello,
Is there a way to resize the window using WinAppDriver? I do not want to maximize the window or use the fullScreen feature but I want to set my window to a particular size.
Width = 1845
Height = 1096
driver.Manage().Window.Size.Width does not let me set these values.
Is there a way to achieve this with WinAppDriver usig C#?

I ran into the same issue, fortunately the application we are testing takes it height and width from the registry as a binary value. Which we set in  [ClassInitialize]
    RegistryKey WindowSizeKey;
    WindowSizeKey = Registry.CurrentUser.CreateSubKey("SOFTWARE\\Location");
        WindowSizeKey.SetValue("Visible", 0, RegistryValueKind.DWord);
        WindowSizeKey.SetValue("Rect", new byte[]
        {
            0x05, 0x02, 0x00, 0x00,
            0x0F, 0x07, 0x00, 0x00,
            0x31, 0x03, 0x00, 0x00,
            0x3B, 0x08, 0x00, 0x00
        },

Hopefully this helps

This is how you can set the size:
driver.Manage().Window.Size = new Size(1845, 1096);

This is how you can set the size:
driver.Manage().Window.Size = new Size(1845, 1096);

Thanks @anunay1 Worked like a charm!

when I use a WindowsDriver session and Appium:
Run this line:
session.Manage().Window.Size = new Size(1000,1000);
Get below message:
Bad parameters: BadParametersError: Parameters were incorrect. We wanted {"required":["x","y","width","height"]} and you sent ["x","y"]

how to resize the window on java selenium?

still appium supports winappdriver(for java)


__typename is required to be set explicitly.

Describe the bug
__typename is required to be set explicitly. I'm actually not sure if this is the intended behavior.
To Reproduce
Steps to reproduce the behavior:
_1. I have just set everything up as per the documentation.
Expected behavior
I think the most intuitive approach would be for the __typename property to be inferred based on the type itself.

Hello @lbenevento !
Yes, totally agree with you 🙂
In v0.5, __typename will be optional by default for types, except for union members and interface implementing types.
For now in v0.4, here's how you can make __typename optional:
const config: CodegenConfig = {
  schema: "**/schema.graphql",
  generates: {
    "src/schema": defineConfig({
      typesPluginsConfig: {
        nonOptionalTypename: false, // Note that in v0.5, this will be set automatically
      },
    }),
  },
};

v0.5 is just around the corner 🙂 I'm adding a few features upstream in typescript-resolvers so we are just waiting for this PR to be merged.
Let me know if you have any other questions!

Thank you, I did read the documentation but I didn't quite catch that option! I'll use that while waiting for v0.5 to ship.
Awesome work! 😁


It is always in a verified state when adding one custom environment if using 'https://maneg.com' as the ARM endpoint

Storage Explorer Version: 1.34.0-dev (98)
Build Number: 20240425.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Sonoma 14.4.1(Apple M1 Pro)
Architecture: x64/x64/arm64
How Found: Ad-hoc testing
Regression From: Previous release (1.30.2)
Steps to Reproduce

Launch Storage Explorer
Open the connect dialog -> Select 'Subscription -> Custom Environment:' -> Click 'Manage custom environments...'.
Click 'Add' -> Type a valid environment name.
Type 'https://maneg.com' into ARM endpoint field -> Click 'Add'.
Check whether an error message appears.

Expected Experience
An error message appears.

Actual Experience
It is always in a verified state.

@v-kellyluo This does not reproduce for me at all using the same build or on our main branch. Clicking Add does require network communication. Perhaps there was a network issue? Does this still reproduce for you?

Hi @craxal, we waited more than 10 minutes, this issue still reproduces on our side, and it doesn't reproduce in 1.30.2 on the same machine.

@v-kellyluo I still don't reproduce. Can you provide app logs? Test machine?

@craxal , here is the logs, and I will share the test machine in Teams.
2024-04-27_172306.zip

There's a "TypeError: Converting circular structure to JSON" error getting thrown that isn't being caught.
Here's what I think is happening:

The management endpoint is passed to the function for adding custom environments.
The management is passed to the identity library which calls into a custom HTTP client to send a request for the management endpoint configuration.
Because the endpoint does not exist, the HTTP request throws an error.
The custom HTTP client handles errors by catching them, logging them, then rethrowing the error. The error is passed to our ExceptionSerializer.serialize() function. This, I think, is what is throwing, because it's trying to serialize a property (issuerCertificate) that isn't JSON-serializable (circular or self-referencing).

This is...a problem. I don't think we ever anticipated any error or error-like objects to contain circular references. We'll need to handle this somehow.
Considering this only happens when the management endpoint doesn't exist, and we don't accept them until we've gotten positive confirmation that it does, I don't think a fix for this is urgent.

Verified this issue on the build 20240705.1. Fixed. So, we are going to close it.

I'm still concerned about serializing and deserializing objects with circular references. I will create a work item to address that: #8048.


[stevo]: set regnext return width to be same as input

Chisel couldn't find bitwidth when I tried to use
Reverse(ShiftRegister(value, 2))
because Reverse calls getWidth. This fixes that issue. If a change to RegNext isn't desirable, I'll request a change to ShiftRegister instead.

This changes the semantics of RegNext.  Previously, something like the following
val foo = RegNext(x)
when (bar) { foo := y }

would result in width(foo)=max(width(x), width(y)), but this change will force width(foo)=width(x).
Changing ShiftRegister to use Reg(t = value, next = value) seems like a better solution to me.


Improve the display on bluegenes-GO-term-visualizer

I will like to work on the blue-genes-GO-term-visualizer CSS display.

It will look better if the box is centralized rather than it being on the far left.
2.Also the input field for the RESULT LENGTH is too long considering it just accepts two digit numbers

Please can I work on them?

I think that's a great idea. The filter panel could definitely be made more compact.
Try giving it a new look and we'll see whether we prefer it when you create the PR.

Good morning,
I'm done with the contribution but running 'git push -u origin
' (for me my branch name is dev) produced showed the test
failing, the error message is attached.
Thank you.
On Mon, 16 Mar 2020 at 14:32, Herald<EMAIL_ADDRESS>wrote:

Assigned #18
https://github.com/intermine/bluegenes-GO-term-visualizer/issues/18 to
@egbobright https://github.com/egbobright.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/intermine/bluegenes-GO-term-visualizer/issues/18#event-3133198401,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3J2YNGYI4ASGKGIJHVHQ3RHYS6HANCNFSM4LKP6R4Q
.

Hello,
I modified the filter panel as suggested. I've sent a new commit on issue
#19. While I think your suggestion on making it more compatible is perfect,
I also think leaving out the h4 header and making the apply filter button
inline with the p, selection and input tag will not be a great idea.
Kindly check my commit and let me know.
Thank you.
On Monday, March 23, 2020, Bright Egbo<EMAIL_ADDRESS>wrote:

Good morning,
I'm done with the contribution but running 'git push -u origin
' (for me my branch name is dev) produced showed the test
failing, the error message is attached.
Thank you.
On Mon, 16 Mar 2020 at 14:32, Herald<EMAIL_ADDRESS>wrote:

Assigned #18
https://github.com/intermine/bluegenes-GO-term-visualizer/issues/18 to
@egbobright https://github.com/egbobright.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/intermine/bluegenes-GO-term-visualizer/
issues/18#event-3133198401>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/
AK3J2YNGYI4ASGKGIJHVHQ3RHYS6HANCNFSM4LKP6R4Q>
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/intermine/bluegenes-GO-term-visualizer/issues/18#issuecomment-602434725,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3J2YLHJM2AS4WPBSXHKW3RI4HKTANCNFSM4LKP6R4Q
.

Fixed with #19


feat(cli): address command to read addresses of stored keys

What
Add address sub command for identity.
For example,
soroban config identity address test-identity

For identities that are seed phrases there is an hd-path option. E.g.
soroban config identity address test-identity --hd-path 1

The default is zero.
Why
Tests can now deterministically generate the same addresses from a single seed phrase. As shown in the tests provided.
Known limitations
Might be confusing for new users. Though I found it very useful for writing tests for the examples.

Though I found it very useful for writing tests for the examples.

Yeah, the idea came up because in the example-dapp we invoke the contract with initialize(owner: Address, ...), and it's nice to use an identity for that owner address.


data['disabled'] should NOT have additional properties

Error: data['disabled'] should NOT have additional properties
I'm having this error running extract-text-webpack-plugin with webpack. How to fix it?
0 info it worked if it ends with ok
1 verbose cli [ 'C:\\Program Files\\nodejs\\node.exe',
1 verbose cli   'C:\\Users\\cgone\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'dev' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'predev', 'dev', 'postdev' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for predev, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: C:\Users\cgone\AppData\Roaming\npm\node_modules\npm\bin\node-gyp-bin;C:\Www\Projects\pack3\node_modules\.bin;C:\Program Files (x86)\Intel\iCLS Client\;C:\Program Files\Intel\iCLS Client\;C:\windows\system32;C:\windows;C:\windows\System32\Wbem;C:\windows\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;c:\Program Files\Intel\WiFi\bin\;c:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files\Git\cmd;C:\Program Files\nodejs\;C:\Users\cgone\AppData\Local\Microsoft\WindowsApps;C:\Program Files (x86)\Microsoft VS Code\bin;C:\Users\cgone\AppData\Roaming\npm
10 verbose lifecycle<EMAIL_ADDRESS>CWD: C:\Www\Projects\pack3
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c', 'webpack -d --watch' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec dev script
14 verbose stack Error<EMAIL_ADDRESS>dev: `webpack -d --watch`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (C:\Users\cgone\AppData\Roaming\npm\node_modules\npm\lib\utils\lifecycle.js:279:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:194:7)
14 verbose stack     at ChildProcess.<anonymous> (C:\Users\cgone\AppData\Roaming\npm\node_modules\npm\lib\utils\spawn.js:40:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:194:7)
14 verbose stack     at maybeClose (internal/child_process.js:899:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd C:\Www\Projects\pack3
17 verbose Windows_NT 10.0.14393
18 verbose argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\cgone\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "run" "dev"
19 verbose node v7.9.0
20 verbose npm  v4.5.0
21 error code ELIFECYCLE
22 error errno 1
23 error<EMAIL_ADDRESS>dev: `webpack -d --watch`
23 error Exit status 1
24 error Failed at the<EMAIL_ADDRESS>dev script 'webpack -d --watch'.
24 error Make sure you have the latest version of node.js and npm installed.
24 error If you do, this is most likely a problem with the pack3 package,
24 error not with npm itself.
24 error Tell the author that this fails on your system:
24 error     webpack -d --watch
24 error You can get information on how to open an issue for this project with:
24 error     npm bugs pack3
24 error Or if that isn't available, you can get their info via:
24 error     npm owner ls pack3
24 error There is likely additional logging output above.
25 verbose exit [ 1, true ]

and my webpack.config.js is:
var path = require('path');
var HtmlWebpackPlugin = require('html-webpack-plugin');
const ExtractTextPlugin = require("extract-text-webpack-plugin");

module.exports = {
    entry: './src/main.js',
    output: {
        path: path.resolve(__dirname, 'dist'),
        filename: 'bundle.min.js'
    },
    module: {
        rules: [{
            test: /\.scss$/,
            use: ExtractTextPlugin.extract({
                fallback: 'style-loader',
                use: ['css-loader', 'sass-loader'],
                publicPath: "/dist"
            })
        }]
    },
    plugins: [
        new HtmlWebpackPlugin({
            title: 'Custom template',
            minify: {
                  collapseWhitespace: true
            },
            hash: true,
            template: './src/template.html',
        }),
        new ExtractTextPlugin({
            filename: "style.css",
            disabled: false,
            allChunks: true
        })
    ]
}

Package.json versions:
"devDependencies": {
    "css-loader": "^0.28.0",
    "extract-text-webpack-plugin": "^2.1.0",
    "html-webpack-plugin": "^2.28.0",
    "node-sass": "^4.5.2",
    "sass-loader": "^6.0.3",
    "style-loader": "^0.16.1",
    "webpack": "^2.4.1"
  }

NPM -v // => 4.5.0
NODE -v => 7.9.0

@cgonen From a first glace this looks like a Options Validation Error, could you remove the disable from ETWP options?

@michael-ciniawsky Thank you, that worked :-).

@michael-ciniawsky Thank you, that worked 👍

https://github.com/webpack-contrib/extract-text-webpack-plugin/blob/master/schema/plugin-schema.json#L10
It's mispelled in your config disabled => disable and that's the reason why the options validator is complaining 😛


Migrate to manifest V3

In the Chrome extension store, there is a notice that "Simply migrate to Manifest V3 and enjoy faster auditing services and new APIs and features."
details: https://developer.chrome.com/docs/extensions/mv3/intro/mv3-migration/

You can absolutely implement manifest v3 for your extension, currently there is a validation of the manifest version in the build process but it will be removed soon. One thing to be aware of is that

it is not a standard across browsers (standardisation is underway)
the APIs may differ in different browsers (Firefox's implementation is still under development)
the browser polyfill does not support the new APIs yet (issue)

It is nice of you to reply to my question, I think there is nothing needs to be done by this framework but left a link in README.md to indicate this issue.
You can close this as you wish.

Yeah absolutely, I will add a section in the readme for manifest v3.

it is about to end of life about manifest v2 https://developer.chrome.com/docs/extensions/mv3/mv2-sunset/

Yes, unfortunately there is still a lot of work that needs to be done before Blazor can be used in MV3 extensions.
At least 2 issues in Chromium that I am aware of:

Support loading wasm in extensions manifest v3
implement extension service worker

Standardization of web extensions are still underway and it will take some time to get implemented in all the browsers.


sitemap templates: remove unneeded space

Merged, but for the future, it would be good if you could start your commit messge title with a capital letter ("Remove ...")


Snowflake query doesn't return result when lob column is selected

Description
When I run a select query to snowflake where table has a lob kind of column and if this column is used as an output column then query doesn't return result and just goes on and on.  if I remove lob column from select then result returned within a second or two.
if table1 has no lob columns then "select * from table1" returns result.
if table2 has a lob column then "select * from table2" doesn't return result.
if I select explicit column and not selecting lob column then it does return result.
DBeaver Version
Community Edition 22.2.4
Operating System
macOS Monterey 12.6.1
Database and driver
Snowflake on AWS
snowflake-jdbc:release 3.13.6
Steps to reproduce
No response
Additional context
No response

Hello @mnt1979,
Can you please provide sample DDL of such a table?

PAYLOAD is a CLOB like column in snowflake and defined as VARCHAR max.

I guess that the problem is that that column is just too big. Can you verify that it has data in it?
During the execution of that statement, is the amount of memory that DBeaver consumes spikes?

I tried running queries with limit 1 and limit 10, both returned result within couple seconds.  As soon as I ran another query with limit 25, it has not returned any result so far after 1700 seconds and there was no major memory spike in activity monitor.

Thanks for explaining. We'll look into that.

Hello @Smart-Bird-ADM
Could you please check the latest DBeaver version? We added some changes for the significant content reading.
Maybe the issue is not reproducible anymore.

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.


Development: Remove moved to the Lecture microservice features from the Artemis microservice

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[ ] * This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added @PreAuthorize and checked the course groups for all new REST Calls (security).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.
[ ] * not relevant

Motivation and Context

This PR is the final one for the migration of the Lecture microservice.
Description

It removes the features related to lectures from Artemis which were moved to the Lecture microservice. Code review should be fast as there are only ~100 lines added and the deleted files already exist in the Lecture microsercvice /lecture.
Since code that was covered by tests is removed from Artemis there is a slight drop in the test coverage which required the change of the coverage rules.
Steps for Testing

Prerequisites:

Test server 6: https://artemistest6.ase.in.tum.de/ - it is already deployed there
1 Instructor (artemis_test_user_12 or you can create another instructor account using the server administration)
1 Student (artemis_test_user_1 or you can create another student account using the server administration)

Log in to Artemis on https://artemistest6.ase.in.tum.de/
Navigate to Course Management
Open/Create a course and create a lecture => lecture should be successfully created
Edit the lecture => the lecture should be successfully updated
Add attachments and lecture units of all types to the lecture => should work fine
Navigate to course overview (as a strudent) and the lecture you created => the lecture should be there including the units and the attachments you added
Edit the lecture units => should work fine
Open the lecture and check that the exercise is removed from the lecture units list
Go back to Course Management as an instructor and delete lecture unit(s) => should work fine
Delete a lecture => the lecture should be successfully deleted

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

Screenshots

Thanks for testing @nairaarnez. It looks like there is an issue connecting to Bitbucket (due to wrong credentials) which is the reason why programming exercises cannot be created. Will check it and let you know when it's fixed

@nairaarnez indeed there were server configuration issues. Thanks to @Mtze for fixing them.
If you can, could you please retest, please?

@Marvin-Vitcu could you also see the exercises from the Student perspective when you tested this PR?

Due to increase maintenance, we do not pursue the split into multiple applications as part of micro services. In fact, @sleiss is trying out a different, less disruptive approach as part of his master thesis.
Therefore, I close this PR.
@MerlinMehmed thanks for your effort, at the moment it's not necessary any more to keep the develop-microservices branch up-to-date.


make ModHooks.OnSceneChanged obsolete / change when it gets triggered

https://github.com/hk-modding/api/blob/master/Assembly-CSharp/ModHooks.cs#L2071-L2092
this could be changed to be done like
UnityEngine.SceneManagement.SceneManager.ActiveSceneChanged += (prev, next) => {
    ModHooks.OnSceneChanged(next.name);
};

after talks, it was determined that unity's activescenechanged and this modhook are for 2 entirely different use cases


Got error when importing dace

Hello
I am new with dace and I want to use it to perform acceleration on a certain iterative solver. However, when I try to import dace I get the following error log:
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\pcg_lib.py", line 3, in <module>
    import dace
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\__init__.py", line 7, in <module>
    from .frontend.python.interface import *
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\frontend\python\interface.py", line 8, in <module>
    from dace.frontend.python import parser, ndloop, tasklet_runner
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\frontend\python\parser.py", line 14, in <module>
    from dace.frontend.python import (newast, common as pycommon, cached_program, preprocessing)
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\frontend\python\newast.py", line 16, in <module>
    from dace import data, dtypes, subsets, symbolic, sdfg as sd
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\sdfg\__init__.py", line 2, in <module>
    from dace.sdfg.sdfg import SDFG, InterstateEdge
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\sdfg\sdfg.py", line 25, in <module>
    from dace.sdfg.scope import ScopeTree
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\sdfg\scope.py", line 8, in <module>
    from dace.sdfg import nodes as nd
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\sdfg\nodes.py", line 797, in <module>
    MapEntry = indirect_properties(Map, lambda obj: obj.map)(MapEntry)
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\properties.py", line 451, in indirection
    indirect_property(cls, indirect_function, prop, override)
  File "c:\Users\anoir\Desktop\CG_accelerator\pcg\venv\lib\site-packages\dace\properties.py", line 417, in indirect_property
    prop_indirect = copy.copy(prop)
  File "C:\Users\anoir\AppData\Local\Programs\Python\Python36\lib\copy.py", line 88, in copy
    return copier(x)
TypeError: __copy__() takes 1 positional argument but 2 were given
(venv) PS C:\Users\anoir\Desktop\CG_accelerator\pcg>

Please know that I am using the VS Code with the SDFGs plugin on Win10 64-bit
What could be the problem ?
Thank you

Thanks for reporting! If you try with Python 3.7 or newer, do you get the same error?

yes, it works now. Even for version 3.10


Enhancement/RFC/Proposal: Add support for Rubberduck folder Navigation

Is your proposal related to a problem?
At the moment importing the whole project is a bit of a chore. One of the pain points I noticed was reconciling whether I had imported all files from all sub directories.
Describe the solution you'd like
To aid in reconciling file imports and to aid in general project navigation, I'd like to add support for Rubberduck folder navidations.
This allows Rubberduck users to more easily check that the structure of the project import matches that of the repository. It also generally improves the project navigation experience when using the Rubberduck project navigator.
Describe alternatives you've considered
Ultimately a refined import process for end users should be considered. The proposed solution would act as a temporary measure mainly for the benefit of project maintainers.
Additional context
N/A

TBH you're right that just adding a filter to search for *. does make it fairly simple to import all the files. not sure why I was trying to import each one individually 🙃
That said, I don't believe there to be any downside to adding support for Rubberduck folder navigation and it would add a better UX for those that use it, so I will submit a PR for this feat.


Missing LICENSE file

What is the license associated with the project? I've found a bug and would like to contribute a fix but need to know the license first.
I notice that your package is a fork of @yefim's package so I'm mentioning him here in hopes that he will indicate the license associated with his original repo.

Added MIT licence

Thanks!


SignInFragment. Retain EditTexts values.

Values of FirstName and LastInitial edittexts within SignInFragment were not retained after screen rotation. Also there was related to the issue code that saved the values but didn't retain them.

I signed it!

sorry there's a mistake in code, will make another pull request.

yep, just a mistype, should be ==


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.145

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.145


[AutoRelease] t2-network-2023-07-20-27041(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/2985
Live test fail, detailed info is in pipeline log(search keyword FAILED)!!!
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1031
BuildTargetingString
azure-mgmt-network
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/2985


TapTargetView is rendered behind custom view

Version used: 1.7.0
Stack trace: N/A
Android version: 6.0.1 and 4.4.0
I've applied a TapTargetView onmy my custom view for Cryptogram, but noticed that the rendering of text appeared on top of it. This is the code I'm working with:
TapTargetView.showFor(
        CryptogramActivity.this,
        TapTarget.forView(mCryptogramView,
                "Hyphenation",
                "Watch out, this is a hyphen! The word continues on the next line.")
                 .titleTextColor(R.color.white)
                 .descriptionTextColor(R.color.white)
                 .tintTarget(false)
                 .transparentTarget(false),
        new TapTargetView.Listener() {
            @Override
            public void onTargetClick(TapTargetView view) {
                PrefsUtils.setHighlightedHyphenation(true);
            }
        });

Which yields:

I had expected the overlay to appear on top of my custom view, as it appears to for the Toolbar and the rest of my activity.

Interesting... Are you setting the elevation on your own view? Can you share details / code about your view? Are you adding your view programmatically?

No, there are no elevations set. Furthermore, the problem exists on pre-Lollipop devices.
The custom view is included in the layout, no frills there. It's a fairly simple extension of TextView that draws puzzle characters on a canvas. The problematic commit can be found here:
https://github.com/pflammertsma/cryptogram/tree/3524420f29ee7891c6dfe6ccf365574dcfbb8a9b

I should point out that the issue no longer occurs when using TapTarget.forBounds(); it only applies to TapTarget.forView() on the view in question.

Interesting! Thanks. It's great that the app is open source, I'll take a look tonight!

Ultimately the issue doesn't affect me as per chance TapTarget.forBounds() was what I needed, but it certainly is an interesting bug!

I see what the problem is: You're targeting a very large view (the entire cryptogram), and so, it is re-drawing your targeted view (otherwise it would be obscured), and since its large, it goes outside the bounds of the target circle.
So this seems to be the case of using the wrong target type. You do want to use specific bounds as you already figured out. Targeting a view is for smaller types of views, like FABs.

Gotcha. Perhaps a utility method that allows specifying a boundary or
offset within a view would make it easier for applications like this, but
I'm perfectly happy with the solution in place now. Thanks again!
On Sat, Feb 4, 2017, 04:52 Hilal Alsibai<EMAIL_ADDRESS>wrote:

I see what the problem is: You're targeting a very large view (the entire
cryptogram), and so, it is re-drawing your targeted view (otherwise it
would be obscured), and since its large, it goes outside the bounds of the
target circle.
So this seems to be the case of using the wrong target type. You do want
to use specific bounds as you already figured out. Targeting a view is for
smaller types of views, like FABs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KeepSafe/TapTargetView/issues/126#issuecomment-277416414,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARdY9-_uR87Pa2y9-CVn8tcjDlJcJNrks5rY_X-gaJpZM4L2r-a
.


how to read  request header map ?

Feature request

Use case

Hi @jilieryuyi,
I am not sure I fully understand your question.
Are you asking how to add the headers of the HTTP request to the produced span?
This can be achieved by modifying the HTTP instrumentor for example:
https://github.com/keyval-dev/opentelemetry-go-instrumentation/blob/master/pkg/instrumentors/bpf/net/http/server/probe.go
to read the Header field of the http.Request struct and to add the map contents into the produced span.
Notice that because this is a new field that none of the other instrumentors use, it needs to be tracked by offsets-tracker:
https://github.com/keyval-dev/offsets-tracker/blob/19f19f58020255b5f12585055076cefbc637c0c1/main.go#L18


Standalone install problem with "brew install gcc --with-glibc -v"

Linux version 2.6.39-200.24.1.el6uek.x86_64<EMAIL_ADDRESS>                                                 (gcc version 4.4.6 20110731 (Red Hat 4.4.6-3) (GCC) ) #1 SMP Sat Jun 23 02:39:0                                                 7 EDT 2012
Brew doesn't grab binary, but instead builds gcc.