cannot restart

PS C:\Users\Admin\Desktop\owofarmbot-main> npm install
up to date, audited 99 packages in 1s
7 packages are looking for funding
run npm fund for details
2 moderate severity vulnerabilities
Some issues need review, and may require choosing
a different dependency.
Run npm audit for details.
PS C:\Users\Admin\Desktop\owofarmbot-main>

PS C:\Users\Admin\Desktop\owofarmbot-main> npm audit
npm audit report
request  *
Severity: moderate
Server-Side Request Forgery in Request - https://github.com/advisories/GHSA-p8p7-x288-28g6
Depends on vulnerable versions of tough-cookie
No fix available
node_modules/request
tough-cookie  <4.1.3
Severity: moderate
tough-cookie Prototype Pollution vulnerability - https://github.com/advisories/GHSA-72xf-g2v4-qvf3
No fix available
node_modules/tough-cookie
2 moderate severity vulnerabilities
Some issues need review, and may require choosing
a different dependency.
PS C:\Users\Admin\Desktop\owofarmbot-main>

7 packages are looking for funding
run npm fund for details
PS C:\Users\Admin\Desktop\owofarmbot-main> npm fund
<EMAIL_ADDRESS>+-- https://github.com/chalk/chalk?sponsor=1
| | --<EMAIL_ADDRESS>| -- https://github.com/chalk/ansi-styles?sponsor=1
|     --<EMAIL_ADDRESS>+-- https://github.com/sponsors/sindresorhus |   --<EMAIL_ADDRESS><EMAIL_ADDRESS>+-- https://github.com/sponsors/RubenVerborgh
|   --<EMAIL_ADDRESS>+-- https://github.com/sponsors/feross |   --<EMAIL_ADDRESS>-- https://github.com/sponsors/epoberezkin     --<EMAIL_ADDRESS>PS C:\Users\Admin\Desktop\owofarmbot-main>

never mind it works again. Thank you for the code

then close the issue!!!


Switch to 1-to-1 result struct mapping

Related to:

https://github.com/test-network-function/cnf-certification-test/pull/1675
https://github.com/test-network-function/cnf-certification-test/pull/1666

Depends-On: https://github.com/test-network-function/cnf-certification-test/pull/1675

I merged this because the 'Depends-On' action wasn't doing what I wanted it to do.


Ryan/text to cad blender addon

This is a first pass at my blog post for the Blender addon using Text-to-CAD. Any and all suggestions welcome as writing is not my strong suit.

nice looks good, get @franknoirot to help w an image!


mysql insert get number of affected rows

Hi, I have a question. How can I get number of affected rows while using knex insert?
I'm using mysql 5.5.41

@George0406
Hi!
Can you please describe your problem or provide some example? I can't quite understand the problem.
For me, it's quite obvious
var  items = [...];
knex.insert(items).returning('*').then(function (rows) {
    console.log(rows.length); // = rows affected
    console.log(items.length); // = rows affected
})

@coockoo
HI!
thanks for your reply!!!!  But it seems doesn't work in mysql
I always get [0] when print out the rows.length
so I wonder to know are there any ways to get the rows affected

Yeah. This issue affects me too. Here's how I tried it:
var items = [...];
knex('my_table').insert(items).asCallback(function(err, result) {
    console.log(result);
});

But console.log(result) only outputs [ 0 ].

I wanted to drop in to point out it's is confusing in the knex docs.
http://knexjs.org/#Builder-insert

// Returns [2] in "mysql", "sqlite"; [2, 3] in "postgresql"
knex.insert([{title: 'Great Gatsby'}, {title: 'Fahrenheit 451'}], ['id']).into('books')
Outputs:
insert into \books` (`title`) values ('Great Gatsby'), ('Fahrenheit 451')`

It's very easy to miss that the example includes the ['id'] array for the returning param. I confused myself by reading that as the record count. It's also confusing when the first example says that MySQL returns something even without the returning param or helper.

// Returns [1] in "mysql", "sqlite", "oracle"; [] in "postgresql" unless the 'returning' parameter is set.
knex('books').insert({title: 'Slaughterhouse Five'})

I don't find this to be the case. But I suppose that might be because my table doesn't have an auto-increment numeric primary key.
I suppose not technically related to this issue, but maybe someone will read this and save themselves some head-scratching.

Mysql doesn't return number of affected rows

But I get the number of affected rows:
MySQL [oh_my_db]> desc users;
+-------+---------------------+------+-----+---------+----------------+
| Field | Type                | Null | Key | Default | Extra          |
+-------+---------------------+------+-----+---------+----------------+
| id    | bigint(20) unsigned | NO   | PRI | NULL    | auto_increment |
| name  | varchar(255)        | YES  |     | NULL    |                |
| age   | int(11)             | YES  |     | NULL    |                |
+-------+---------------------+------+-----+---------+----------------+
3 rows in set (0.001 sec)

MySQL [oh_my_db]> insert into users  (name, age) values ('f', 18), ('j', 19);
Query OK, 2 rows affected (0.004 sec) <------------------------------------------------------here
Records: 2  Duplicates: 0  Warnings: 0

Is it possible for knex to return the affected number?

https://github.com/knex/knex/pull/4677


Player Kicked From Server When Selecting Destination (Space Elevator)

I'm having an issue with Space Elevators in SMP. Every time a friend of mine tries to select a destination (presses the Select Destination button) inside of the Space Elevator GUI it kicks him from the server with the message "A fatal error has occured, the connection has been terminated". The elevator is fully powered and the chips are linked. This occurs for other players as well, but it only kicks the person who presses the button. He's able to rejoin afterwards just fine, but nothing in the GUI has changed.
This is occuring in the alpha of Advanced Rocketry 1.2.0 for 1.7.10.
I assume something is going wrong while trying to send packets to the server/client.

Can you get the stacktrace from the server/client log?

Possibly fixed with #636

I'm not currently in a position to get the stacktrace from his client, but the server seems to say nothing. I'll test if it still happens later and if needed will provide a client stacktrace.

The issue is no longer present in the latest alpha. Thanks a lot :D.


[ReducerProtocol] - Support Non-Dependencies Environment in ReducerProtocol

Improvement to support Non-Dependencies Environment inside ReducerProtocol implementation.
We need to find a way how we can do mock on the fly since it only possible when using Dependencies, not with non-dependencies Environment.
✅ Solution: We are improve our TestStore to have capability to hold our Environment, and giving access to developer to change it on the fly
let testStore = TestStore(
    initialState: "",
    environment: AuthenticatorService.mock,
    reducer: Authenticator.init(authenticatorService:)
)

/// here we can mock our environment
////
testStore.environment.getAuthResult = {
    .success("success")
}

The reducer implementation will be like this:
struct Authenticator: ReducerProtocol {
    typealias State = String
    typealias Action = Void

    /// ✅ here our Environment
    ///
    internal var authenticatorService: AuthenticatorService

    func reduce(into state: inout String, action: Void) -> Effect<Void> {
        switch authenticatorService.getAuthResult() {
        case let .success(successMessage):
            state = successMessage
        case let .failure(failureData):
            state = failureData.message
        }
        return .none
    }
}

For mocking Example we can do it like this:
Bootstrap.mock(
    for: Authenticator(authenticatorService: .mockFailed) // you can mock directly the Environment
) { currentReducer in
    ... your mocking Dependencies logic
}

@coderabbitai review


added ultimaBomba sheet only ita

Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

added ultimaBombaSheet

Hello @FrancescoCalise same game correct? https://www.serpentarium.net/l-ultima-bomba

Hello @FrancescoCalise same game correct? https://www.serpentarium.net/l-ultima-bomba

yes, I asked the author before publishing it

hi @Reonin, how can add this game in list of the game?

No worries I already added it but in the future write it somewhere in the title of the pull request


Add Wasbs support to Wasb Underfs

Azure Blob Storage over https is currently unsupported by the wasb underfile system; however, the version of azure-hadoop  used in the underfs supports the https scheme (wasbs).
This PR merely changes the accepted SCHEME from wasb:// to accepted SCHEMES including both wasb:// and wasbs://
wasbs support was tested by running runUfsTests in a k8s installation. All tests were passed successfully.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8155/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8157/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8158/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8159/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8162/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8205/
Test PASSed.

alluxio-bot, merge this please


🤔 常见问题

链接

What is the app backend? Is it connected to any server ?

What is the app backend? Is it connected to any server ?

@yendiscart This is just a local client, with no server connection, the database is sqfile.


Unicode decoding error?

https://pypi.org/project/iconv/ displays the author as "Martin v. LÃ¶wis"
:(

That's how it was provided to PyPI in the metadata, unfortunately there's not much we can do about that. You might want to inform the owner of the project instead. Thanks for the report though!

I'm admittedly very uninformed as to the guts of Python packaging, but the PKG-INFO and setup.py files in the archive both seem to have a correct UTF-8 encoding of "Martin v. Löwis".
>>> line = open('PKG-INFO', 'rb').readlines()[5]
>>> line.decode('utf-8')
'Author: Martin v. Löwis\n'
>>> line.decode('iso-8859-1')
'Author: Martin v. LÃ¶wis\n'

What's the correct way to use UTF-8 in package metadata?

(...oh no. is this still part of Python 2's ascii legacy?)

Warehouse/PyPI doesn't inspect the uploaded files, it gets sent alongside the file upload as form encoded data, so if whatever uploaded the file didn't properly send it, then you'll get that.

Ah, right. Thanks @dstufft!


Issue while doing handshaking over BLE

Hi,
i am trying to establish a connection between client and server running on python-mbedtls over BLE.
I am doing certificate authentication (one-way authentication). During handshake, client is able to send CLIENT_HELLO message and server is also able to receive it and sends SERVER_HELLO.
Since i am doing communication over BLE there is some delay in sending these messages as a result after receiving this SERVER_HELLO, client is not able to switch to CLIENT_HELLO state to send the message again to server for next steps.
If I do the communication over localhost without BLE, I am able to do handshaking and send data from Client to Server and vice-versa.
I want to do communication over BLE only which will result in some delay, what should i do to resolve it? Can you provide some insights please.

My guess would be you need to increase the handshake timeout. I’m not sure what BLE means either but you should know that TLS is very expensive for the processor. So if that’s related to embedded somehow, you may have a lot more bad surprises.

Hi Thank you for your response for the earlier issue.
I have one more question, I want to validate the server certificates at client side, So I enabled validate_certificates as True in client side while creating DTLSConfiguration but I receive errors: mbedtls.exceptions.TLSError: TLSError([0x2700] 'X509 - Certificate verification failed, e.g. CRL,
CA or signature check failed').
I have used the same CA certificate created by server in client.
Can you please help what I might be doing wrong in the process.


⚠ Bofrost.lu has degraded performance

In ed9a141, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1322 ms

Resolved: Bofrost.lu performance has improved in 6aac055 after 14 minutes.


[bugfix]: fix kafka custom op import error on non-linux system

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.杨熙 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


1.16.4 update request

Hi, the FakePlayer Plugin is not working on 1.16.4 Spigot and Paper server-client, please update to the latest version, thank you so much for that.
On 1.16.4 servers, I can only open the GUI, but when I click "Add fake players", nothing happened.
This is really a great plugin when it is combined with "HoldEntities" because players can move those fake players to somewhere so that they don't need to AFK by themselves.

#14 duplicate.


Updating README with minor improvements

This is to update README with below changes:

Adding a brief description about secure vault
Changing the word 'secure_vault' to 'secure vault'
Remove a duplicate line AS README file.

Closed this p/r to update the README bit more. Will fix it and reopen it.

I have refactored the modules README files. Please review and merge the p/r.

Thanks for the PR! This is now merged.


Unable to launch ur_bringup test_joint_trajectory_controller.launch.py

Issue
I am trying to start using ROS for work with a UR3e and running into difficulties in the initial set-up. My end goal here is to do most of my development in a simulated environment without the actual UR3e or teach pendant.
I can successfully launch the fake hardware in rviz; however, receive warning when I attempt to test the joint trajectory controllers, and no movement in rviz.
[publisher_joint_trajectory_controller-1] [WARN] [1649259061.755820186] [publisher_scaled_joint_trajectory_controller]: Start configuration is not within configured limits!
There are also some errors appearing in the initial bringup logging.
Thank you for any help here!
System
ROS: Galactic
Description:	Ubuntu 20.04.3 LTS
Release:	20.04
Codename:	focal
Traceback
Initial bringup
$ ros2 launch ur_bringup ur3e.launch.py robot_ip:=yyy.yyy.yyy.yyy use_fake_hardware:=true launch_rviz:=true
[INFO] [launch]: All log files can be found below
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [controller_stopper_node-2]: process started with pid [46555]
[INFO] [ros2_control_node-1]: process started with pid [46553]
[INFO] [robot_state_publisher-3]: process started with pid [46557]
[INFO] [rviz2-4]: process started with pid [46559]
[INFO] [spawner-5]: process started with pid [46561]
[INFO] [spawner-6]: process started with pid [46563]
[INFO] [spawner-7]: process started with pid [46565]
[INFO] [spawner-8]: process started with pid [46567]
[INFO] [spawner-9]: process started with pid [46569]
[INFO] [spawner-10]: process started with pid [46599]
[ros2_control_node-1] [INFO] [1649258887.015716145] [controller_manager]: update rate is 500 Hz
[robot_state_publisher-3] Link base_link had 2 children
[robot_state_publisher-3] Link base had 0 children
[robot_state_publisher-3] Link base_link_inertia had 1 children
[robot_state_publisher-3] Link shoulder_link had 1 children
[robot_state_publisher-3] Link upper_arm_link had 1 children
[robot_state_publisher-3] Link forearm_link had 1 children
[robot_state_publisher-3] Link wrist_1_link had 1 children
[robot_state_publisher-3] Link wrist_2_link had 1 children
[robot_state_publisher-3] Link wrist_3_link had 2 children
[robot_state_publisher-3] Link flange had 1 children
[robot_state_publisher-3] Link tool0 had 0 children
[robot_state_publisher-3] Link ft_frame had 0 children
[robot_state_publisher-3] [INFO] [1649258887.011347097] [robot_state_publisher]: got segment base
[robot_state_publisher-3] [INFO] [1649258887.011476578] [robot_state_publisher]: got segment base_link
[robot_state_publisher-3] [INFO] [1649258887.011493380] [robot_state_publisher]: got segment base_link_inertia
[robot_state_publisher-3] [INFO] [1649258887.011504660] [robot_state_publisher]: got segment flange
[robot_state_publisher-3] [INFO] [1649258887.011515457] [robot_state_publisher]: got segment forearm_link
[robot_state_publisher-3] [INFO] [1649258887.011525952] [robot_state_publisher]: got segment ft_frame
[robot_state_publisher-3] [INFO] [1649258887.011536636] [robot_state_publisher]: got segment shoulder_link
[robot_state_publisher-3] [INFO] [1649258887.011547622] [robot_state_publisher]: got segment tool0
[robot_state_publisher-3] [INFO] [1649258887.011557997] [robot_state_publisher]: got segment upper_arm_link
[robot_state_publisher-3] [INFO] [1649258887.011569186] [robot_state_publisher]: got segment world
[robot_state_publisher-3] [INFO] [1649258887.011579601] [robot_state_publisher]: got segment wrist_1_link
[robot_state_publisher-3] [INFO] [1649258887.011590258] [robot_state_publisher]: got segment wrist_2_link
[robot_state_publisher-3] [INFO] [1649258887.011600483] [robot_state_publisher]: got segment wrist_3_link
[controller_stopper_node-2] [INFO] [1649258887.002884336] [Controller stopper]: Waiting for switch controller service to come up on controller_manager/switch_controller
[controller_stopper_node-2] [INFO] [1649258887.022690319] [Controller stopper]: Service available
[controller_stopper_node-2] [INFO] [1649258887.022722883] [Controller stopper]: Waiting for list controllers service to come up on controller_manager/list_controllers
[controller_stopper_node-2] [INFO] [1649258887.022741685] [Controller stopper]: Service available
[ros2_control_node-1] [INFO] [1649258887.199332130] [controller_manager]: Loading controller 'joint_state_broadcaster'
[ros2_control_node-1] [INFO] [1649258887.213178053] [controller_manager]: Loading controller 'force_torque_sensor_broadcaster'
[spawner-5] [INFO] [1649258887.222247026] [spawner_joint_state_broadcaster]: Loaded joint_state_broadcaster
[ros2_control_node-1] [INFO] [1649258887.226668299] [controller_manager]: Configuring controller 'joint_state_broadcaster'
[ros2_control_node-1] [INFO] [1649258887.226751305] [joint_state_broadcaster]: 'joints' or 'interfaces' parameter is empty. All available state interfaces will be published
[spawner-8] [INFO] [1649258887.233244449] [spawner_force_torque_sensor_broadcaster]: Loaded force_torque_sensor_broadcaster
[ros2_control_node-1] [INFO] [1649258887.235361508] [controller_manager]: Configuring controller 'force_torque_sensor_broadcaster'
[spawner-5] [INFO] [1649258887.235378573] [spawner_joint_state_broadcaster]: Configured and started joint_state_broadcaster
[ros2_control_node-1] [INFO] [1649258887.249301979] [controller_manager]: Loading controller 'speed_scaling_state_broadcaster'
[ros2_control_node-1] [INFO] [1649258887.253924090] [controller_manager]: Loading controller 'forward_position_controller'
[ros2_control_node-1] [INFO] [1649258887.259754168] [controller_manager]: Loading controller 'scaled_joint_trajectory_controller'
[spawner-7] [INFO] [1649258887.263419763] [spawner_speed_scaling_state_broadcaster]: Loaded speed_scaling_state_broadcaster
[ros2_control_node-1] [INFO] [1649258887.266044966] [controller_manager]: Configuring controller 'speed_scaling_state_broadcaster'
[ros2_control_node-1] [INFO] [1649258887.266094597] [speed_scaling_state_broadcaster]: Publisher rate set to : 100.0 Hz
[spawner-9] [INFO] [1649258887.269844480] [spawner_forward_position_controller]: Loaded forward_position_controller
[spawner-8] [INFO] [1649258887.270556695] [spawner_force_torque_sensor_broadcaster]: Configured and started force_torque_sensor_broadcaster
[spawner-10] [INFO] [1649258887.276227188] [spawner_scaled_joint_trajectory_controller]: Loaded scaled_joint_trajectory_controller
[ros2_control_node-1] [INFO] [1649258887.288475702] [controller_manager]: Configuring controller 'scaled_joint_trajectory_controller'
[ros2_control_node-1] [INFO] [1649258887.288582609] [scaled_joint_trajectory_controller]: Command interfaces are [position] and and state interfaces are [position velocity].
[ros2_control_node-1] [INFO] [1649258887.289098153] [scaled_joint_trajectory_controller]: Controller state will be published at 100.00 Hz.
[ros2_control_node-1] [INFO] [1649258887.289581246] [scaled_joint_trajectory_controller]: Action status changes will be monitored at 20.00 Hz.
[ros2_control_node-1] [ERROR] [1649258887.298834812] [controller_manager]: Can't activate controller 'scaled_joint_trajectory_controller': State interface with key 'speed_scaling/speed_scaling_factor' does not exist
[spawner-10] [INFO] [1649258887.301726628] [spawner_scaled_joint_trajectory_controller]: Configured and started scaled_joint_trajectory_controller
[ros2_control_node-1] [INFO] [1649258887.328747328] [controller_manager]: Loading controller 'io_and_status_controller'
[spawner-6] [INFO] [1649258887.341482580] [spawner_io_and_status_controller]: Loaded io_and_status_controller
[ros2_control_node-1] [INFO] [1649258887.344648942] [controller_manager]: Configuring controller 'io_and_status_controller'
[ros2_control_node-1] [ERROR] [1649258887.354752839] [controller_manager]: Can't activate controller 'io_and_status_controller': Command interface with 'gpio/standard_digital_output_cmd_0' does not exist
[spawner-6] [INFO] [1649258887.357376278] [spawner_io_and_status_controller]: Configured and started io_and_status_controller
[ros2_control_node-1] [INFO] [1649258887.377924304] [controller_manager]: Configuring controller 'forward_position_controller'
[ros2_control_node-1] [INFO] [1649258887.378478028] [forward_position_controller]: configure successful
[ros2_control_node-1] [ERROR] [1649258887.383622846] [controller_manager]: Can't activate controller 'speed_scaling_state_broadcaster': State interface with key 'speed_scaling/speed_scaling_factor' does not exist
[spawner-7] [INFO] [1649258887.386400758] [spawner_speed_scaling_state_broadcaster]: Configured and started speed_scaling_state_broadcaster
[INFO] [spawner-8]: process has finished cleanly [pid 46567]
[INFO] [spawner-10]: process has finished cleanly [pid 46599]
[INFO] [spawner-5]: process has finished cleanly [pid 46561]
[INFO] [spawner-7]: process has finished cleanly [pid 46565]
[INFO] [spawner-6]: process has finished cleanly [pid 46563]
[INFO] [spawner-9]: process has finished cleanly [pid 46569]
[rviz2-4] [INFO] [1649258887.633581781] [rviz2]: Stereo is NOT SUPPORTED
[rviz2-4] [INFO] [1649258887.633734401] [rviz2]: OpenGl version: 4.6 (GLSL 4.6)
[rviz2-4] [INFO] [1649258887.734804806] [rviz2]: Stereo is NOT SUPPORTED

List controllers
$ ros2 control list_controllers
joint_state_broadcaster[joint_state_broadcaster/JointStateBroadcaster] active
force_torque_sensor_broadcaster[ur_controllers/ForceTorqueStateBroadcaster] active
speed_scaling_state_broadcaster[ur_controllers/SpeedScalingStateBroadcaster] inactive
forward_position_controller[position_controllers/JointGroupPositionController] inactive
scaled_joint_trajectory_controller[ur_controllers/ScaledJointTrajectoryController] inactive
io_and_status_controller[ur_controllers/GPIOController] inactive

Test controllers
$ ros2 launch ur_bringup test_scaled_joint_trajectory_controller.launch.py
[INFO] [launch]: All log files can be found below /home/pmm/.ros/log/2022-04-06-11-30-55-264755-ROS-Workstation-46809
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [publisher_joint_trajectory_controller-1]: process started with pid [46811]
[publisher_joint_trajectory_controller-1] [INFO] [1649259055.753190440] [publisher_scaled_joint_trajectory_controller]: Publishing 4 goals on topic "/scaled_joint_trajectory_controller/joint_trajectory" every 6 s
[publisher_joint_trajectory_controller-1] [WARN] [1649259055.754890324] [publisher_scaled_joint_trajectory_controller]: Starting point limits exceeded for joint shoulder_lift_joint !
[publisher_joint_trajectory_controller-1] [WARN] [1649259055.755152894] [publisher_scaled_joint_trajectory_controller]: Starting point limits exceeded for joint wrist_1_joint !
[publisher_joint_trajectory_controller-1] [WARN] [1649259061.755820186] [publisher_scaled_joint_trajectory_controller]: Start configuration is not within configured limits!

Echo joint_states
$ ros2 topic echo joint_states
---
header:
  stamp:
    sec:<PHONE_NUMBER>
    nanosec: 975929673
  frame_id: ''
name:
- elbow_joint
- shoulder_lift_joint
- shoulder_pan_joint
- wrist_1_joint
- wrist_2_joint
- wrist_3_joint
position:
- 0.0
- 0.0
- 0.0
- 0.0
- 0.0
- 0.0
velocity:
- 0.0
- 0.0
- 0.0
- 0.0
- 0.0
- 0.0
effort:
- 0.0
- 0.0
- 0.0
- 0.0
- 0.0
- 0.0
---

Hello, you can edit the =Universal_Robots_ROS2_Driver/ur_bringup/config/test_goal_publishers_config.yaml file and make the starting pose check false.
I assume you are not using melodic as the ros_info indicates ros2.

I'll take a look there, and have fixed the ros version to galactic, my mistake!

Setting check_starting_point: false gets rid of the warning; however, the robot still does not move in RViz. No other info is printed  to either log, so I presume it has something to do with the errors in the first call.
[ros2_control_node-1] [ERROR] [1649258887.298834812] [controller_manager]: Can't activate controller 'scaled_joint_trajectory_controller': State interface with key 'speed_scaling/speed_scaling_factor' does not exist

[ros2_control_node-1] [ERROR] [1649258887.383622846] [controller_manager]: Can't activate controller 'speed_scaling_state_broadcaster': State interface with key 'speed_scaling/speed_scaling_factor' does not exist

In the xacro file Universal_Robots_ROS2_Description/urdf/ur.ros2_control.xacro you will see that these will not be present in the description if use_fake_hardware:=true, you can verify that by running ros2 control list_hardware_interfaces, the result will only show position and velocity interface for the joints.
you can replace the scaled_joint_trajectory_controller with joint_trajectory_controller in the launch file and it should work (i guess).

Unfortunately that didn't work. I tried both passing it as a parameter initial_joint_controller:=joint_trajectory_controller and changing the launch file defaults directly then rebuilding with colcon.
The same 3 errors persist in the logs.
It's worth noting I also can't execute plans made with MoveIt, but expect it's probably worth starting here.

I'll have another look at fake_hardware in combination with the test motions.
Meanwhile: another valid approach would be to use URSim for simulating the robot. This would give you the benefits of having the same interfaces available (not only regarding control) as the real robot. To switch to a real robot all you would have to do is changing the robot's IP address.
I usually launch my URSim using
docker run --rm -it -p 5900:5900 -p 6080:6080 -v /home/mauch/.ursim/urcaps:/urcaps -v /home/mauch/.ursim/programs:/ursim/programs --name ursim universalrobots/ursim_e-series

and then connect the driver to that using
ros2 launch ur_bringup ur5e.launch.py robot_ip:=<IP_ADDRESS>

See the Dockerfile's documentation for why I chose the parameters like that.

This was very helpful! Following the setup of the External Control URCap using VNC viewer, I was able to get a mirrored visualization between URSim and RViz, and get Moveit to plan and execute movement.
However, neither the test_joint_trajectory_controller.launch.py nor the test_scaled_joint_trajectory_controller.launch.py seemed to have any effect. They reported the same warnings posted in the original issue. I'm happy to consider this issue resolved, but am still confused by the controller tests.

I assume that you are referring to the Start configuration is not within configured limits! errors.
The test files expect the robot to be close to a hardcoded starting pose. This is rather important, as moving to the first pose of the test motion might otherwise lead to catastrophic failures depending on the robot's mounting pose and obstacles within its workspace.
As mentioned above you can disable the starting pose check inside the test_goal_publishers_config.yaml file. Alternatively, you can move the robot to the configured starting pose [0, -90°, 0, -90°, 0, 0] manually, then start the external_control on the teach pendant and then execute the test_scaled_joint_trajectory_controller.launch.py file.

Thank you! Setting the initial position and ensuring that the external control was running on the teach pendant solved it.


SVG image support in preview

Please Select if your Request is Either something new or an Enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applys to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[x] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[ ] Misc (Choose this if your not sure)

Is your feature request related to a problem? Please describe
It's currently not possible to include SVG images and have them shown in the preview. This is probably more a missing feature or bug in the markdown renderer, so if this should be reported elsewhere, please let me know.
Reproduce

Create a SVG file in assets/images/.
Add an image in a note ![Test](/assets/images/test.svg).
View the markdown preview.

Result

Describe the solution you'd like
Support for SVG images in the preview.

Yeah, we should make sure this is in scope for the NextJS based preview we're working on #643. Putting on the backlog.

Hey, @fschrempf. Dendron has switched away from the MPE and is using its own component to render the preview and what you are describing seems to work right now. Could you double check if you are still have the same problem?
![](https://upload.wikimedia.org/wikipedia/commons/8/84/Example.svg)

@aleksey-rowan Thanks for the heads-up! I can confirm it works!


@getify/proper-arrows/names should follow func-names schema

and support as-needed and never
afaik async functions would behave the same as sync functions when it comes to inferred names
PS: I don't see why eslint's func-names shouldn't support asyncs, so if a PR gets accepted upstream, it's probably better than having a separate rule doing what func-names does but for async functions

You're talking about "async functions", but this plugin focuses on "arrow functions". These are obviously quite separate.

But assuming you meant "arrow function" instead of "async function"...
This rule intentionally diverges from the built-in "names" rule in both behavior and configuration schema, because the concerns of function names are different for arrow functions than for other functions.
For example, the "always" mode wouldn't make any sense for arrow functions, because arrow functions don't get names (syntactically) at all, and that would have the effect of meaning that no arrow-function could be used. Similarly, the "never" mode wouldn't make any sense, because they never get them anyway, so that would be moot.
The "as-needed" mode takes into account the idea of inferred names, so that's why this plugin mimics (most closely) the "as-needed" behavior; that's the only mode of the 3 that makes sense for arrow functions. Moreover, the built-in "as-needed" behavior does not properly account for all inferences, specifically the inferences from a "named-export declaration"; our plugin does it correctly.

PS: I don't see why eslint's func-names shouldn't support asyncs, so if a PR gets accepted upstream, it's probably better than having a separate rule doing what func-names does but for async functions

Again, assuming you meant "arrows" instead of "asyncs" here, yes, I started by thinking this should just be in the core rule. I asked them to do so, and they basically refused (or just didn't care).
Later, as I reflected on things, I realized the behaviors I wanted to do differently with the application of this rule when applied to arrow functions, as detailed above, so I decided it was actually best to keep it separate anyway.

once again thanks for the well-written reply! Signed: another "me too!" person
PS: yes, sed /async/arrow/g 😞


Add ParamRefs to VAPB policy details page

Renames the "Related resources" table to "Parameter resources" for ValdiatingAdmissionPolicyBindings, and displays the resources when provided by search.
Refs:

https://issues.redhat.com/browse/ACM-15376

/cc @yiraeChristineKim

/retest


KubeJS issue still not fixed

server.log
maybe something is wrong with my instance

Hey i figured out the issue, NuclearCraft Neoteric breaks the event.remove() method of  ServerEvents.recipes, which will only work when using an "id" type to remove the recipe.

 You won't find any mentions of ServerEvents in Nuclearcraft? How it suppose to be broken by NC?

@igentuman idk either, but i know it happens


Add connection_id and connection_alias in logs

What problem does this PR solve?
Issue Number: close #8345
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @SeaRise @JaySon-Huang

/hold

/unhold

/run-tests


Add Kurviger Liberty and remove old Openmapsurfer Tiles

This PR adds the Kurviger Liberty mapstyle to GH Maps and removes the old Openmapsurfer tiles that are not available anymore.

Thx!


Newlines appended to conversions

If I have a file that contains only the characters and (so literally 61 6e 64 hex) and I convert to Markdown (using both CommonMark.CommonMarkConverter.Convert and the CommonMark.Console app with that binary in a file called test.md and a command line of CommonMark.Console.exe test.md --out out.html), I receive the following output:
3c 70 3e 61 6e 64 3c 2f 70 3e 0d 0a 0d 0a
aka

and
\r\n\r\n
I'd expect:
3c 70 3e 61 6e 64 3c 2f 70 3e
that is,

and

without newlines.

One \r\n should be there (the parser implies a newline in the source even if there is none), however the second one might be a bug. I will see if the second one can be removed.

Yep, just checked try.commonmark.org, and the JavaScript reference implementation does add a new line. When textarea.val() is "and", the result is "
and
\n". Okay, actually it's '
and
\n', but that's beside the point.
Pain point -- I'm inserting a preprocessor for Multimarkdown tables and MarkUpDown multiline tables, and if I convert each cell's contents "on their own" so that they can contain Markdown, I get that extra newline, which makes the resulting table code parser unfriendly if there's whitespace and an empty line, but a Trim() with each convert is easy enough.
Thanks! Glad there was at least one valid improvement from the issue report!


Add publish workflows

Add a workflow to publish an image to ghcr.io

Signed-off-by: Wade Barnes<EMAIL_ADDRESS>

Some test releases and workflow runs can be found here;

https://github.com/WadeBarnes/indy-tails-server/releases
https://github.com/WadeBarnes/indy-tails-server/actions
https://github.com/WadeBarnes/indy-tails-server/pkgs/container/tails-server/versions

resolves: https://github.com/bcgov/indy-tails-server/issues/29


Gradio gui

Updated the GUI to use the gradio library
GUI now correctly uses multicore if selected
analyze.py writes the files in utf-8

LGTM


Support for supplying NATOutgoingAddress to Felix

Description
This is a PR required for addressing https://github.com/projectcalico/calico/issues/2222. Addressing the issue also requires a change in Felix which I will link below. We chose to implement this by allowing users to specify the IP address being used rather than using the BGP address by default because it seemed more flexible. For our use case we are using the Kubernetes downward API to supply this address. I've tested this by setting/unsetting FELIX_NATOUTGOINGADDRESS in a calico node container using our modified felix.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

OK, to follow up on my previous comment...  I see now that:

You did the Felix change first, using the downward API to set the config in the environment, and that gives you a complete working solution.
You then added this PR, following @fasaxc's request based on a general desire we have for the data model to be in sync with Felix configuration options.  But you're not relying on the data model for your own solution, hence not picking up that the PR here is incomplete.
In addition, there are considerations that if we're going to have data model for this, perhaps there are better options.

Let me think through a bit more what the upshot of all that is.    Of course the new field is in FelixConfiguration, not IPPool, and there is no per-field conversion code needed for FelixConfiguration.
Also, regarding @caseydavenport 's comment that I pointed to... given that these PRs already Just Work for you, with the downward API, I think we should bank them as they are, and NATOutgoingCIDR or NATOutgoingInterface can be added by further enhancements later if there is sufficient motivation for them.
So, this PR is good to go and I'll now merge it.

@maxstr We'll also need a docs PR to add the new field at https://github.com/projectcalico/calico/blob/master/master/reference/felix/configuration.md and https://github.com/projectcalico/calico/blob/master/master/reference/resources/felixconfig.md.  Would you like to do that?  I'm happy to do it if you prefer.  It's just a matter of credit where credit's due, I think, but we do need it in the next day or two, in time for the upcoming Calico 3.8 release.

Thanks for the merge and the review! Will try and get you a PR by EOD for the doc updates.

Was there ever a docs PR merged for this feature?

Nvm, I see that it is present in the reference material for v3.8.


[Sema] Record invalid protocol witnesses to avoid re-diagnosis.

This is a little risky because previously a recorded witness would only be empty if the requirement was optional, but it's a relatively non-intrusive way to improve diagnostics. The conformance is still marked invalid, which is the important part.
Groundwork for rdar://problem/28543037.

@swift-ci Please test


Cut a release?

@cablehead is it possible you can put a new release on Pypi? I need the added token support from the last commit and I prefer not to have a github dependency.
Thanks!

👍

Sorry for the delay :/ Doing some triaging today and will cut a release in couple of hours!

Awesome thanks!

New release pushed to pypi as v0.6.1. includes #115 Lemme know if you see any issues.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4384

Questions ?
Please refer to the Backport tool documentation


Fix descriptions not showing for theme properties

Supersedes #75552

The trailing underscore was meaningful, as it ensures this is not the entire name.

Sorry I didn't see it. it has been corrected

Thanks!

Cherry-picked for 4.0.3.


Dropdown responsive alignment

components: DropdownMenu
reactstrap version 9.0.0-0
import method umd/csj/es
react version 17.0.1
bootstrap version 5.0.0-beta3

What is happening?
Trying to implement Responsive Alignment for DropdownMenu using dropdown-menu-*-start/end and unable to get it to work
What should be happening?

When using dropdown-menu-end dropdown-menu-lg-start the dropdown menu should open to the right by default but should shift to the left when breakpoint reaches lg or higher.

Steps to reproduce issue

Update the DropdownMenu component in any example of reactstrap dropdown documentation and use
<DropdownMenu className='dropdown-menu-end dropdown-menu-lg-start'>
The Dropdown Menu should open on the left when operating on lg or higher breakpoints.
However, the menu keeps opening up on the right always.

I believe responsive alignment requires that popper be disabled, but I don't think that's currently possible.

@kyletsang i am not sure if there is a change in behavior but even bootstrap 5 is using popper for placement of the menu and in the bootstrap 4.6 version, I was able to make responsive alignment work by replacing the
DropdownMenu component with the html from bootstrap documentation (plus the onclick handler on button and conditionally adding show to toggle the menu visibility) and using the dropdown-menu-lg-left/right classes on the menu. I have tried using the same approach here with dropdown-menu-start/end but this time it does not work

import {useState} from 'react';
import {ButtonDropdown, DropdownToggle} from 'reactstrap';
import classnames from 'classnames';

export default function DemoDropdownResponsiveAlignment() {
  const [dropdownOpen1, setDropdownOpen1] = useState(false);
  const [dropdownOpen2, setDropdownOpen2] = useState(false);

  function toggleDropdown1() {
    setDropdownOpen1(!dropdownOpen1);
  }

  function toggleDropdown2() {
    setDropdownOpen2(!dropdownOpen2);
  }

  return (
    <>
      <ButtonDropdown isOpen={dropdownOpen1} toggle={toggleDropdown1}>
        <DropdownToggle caret>
          Left-aligned but right aligned when large screen
        </DropdownToggle>
        <div className={classnames('dropdown-menu', 'dropdown-menu-lg-right', {show: dropdownOpen1})}>
          <button className="dropdown-item" type="button">Action</button>
          <button className="dropdown-item" type="button">Another action</button>
          <button className="dropdown-item" type="button">Something else here</button>
        </div>
      </ButtonDropdown>
      <ButtonDropdown isOpen={dropdownOpen2} toggle={toggleDropdown2}>
        <DropdownToggle caret>
          Right-aligned but left aligned when large screen
        </DropdownToggle>
        <div
          className={classnames('dropdown-menu', 'dropdown-menu-right', 'dropdown-menu-lg-left', {show: dropdownOpen2})}
        >
          <button className="dropdown-item" type="button">Action</button>
          <button className="dropdown-item" type="button">Another action</button>
          <button className="dropdown-item" type="button">Something else here</button>
        </div>
      </ButtonDropdown>
    </>
  );
}

This code was working as expected for the responsive alignment for the following version numbers:
bootstrap: 4.6.0
react: 17.0.1
reactstrap: 8.9.0
However if I replace dropdown-menu-right with dropdown-menu-end and dropdown-menu-lg-left with dropdown-menu-lg-start - I was expecting this to work (to your point there has been no change in the behavior for responsive alignment) but this does not work and I am trying to figure out what changed.

@kyletsang never mind I found the culprit. data-bs-popper is the attribute that needs to be toggled when you are using the dropdown-menu. I inspected the code on the bootstrap site and found that attribute being toggled everytime I toggled  and toggling that in my react code gives the responsive alignment that I am looking for. Having said that - I do feel like there should be something in reactstrap that will allow us to use responsive alignment without having to use the hack that you mentioned.
Thanks for all the information. I will be closing this issue now.


phonon.min.js:2 Uncaught TypeError: Cannot read property 'style' of null(…)

I get this error when click in button to open side panel. In my case this error happening when using 2° option to create pages (see Navigator). The side panel example works fine, but not when page is defined in external file.
Phonon 1.3.2
Android 6
Google Chrome 54.0.2840.71

Hi @John-Henrique
Can you put an example of your code ?

Hi @John-Henrique,
Can you please include the non minified file (phonon.js), so that I can read the code ;).

@John-Henrique  @Qathom phonon.js but also implementation (where you call init()) and html file where you include the tags!
Lets catch this null's exceptions!

Using phonon.js, the error is marked in line 4439
return typeof settings.element.style[cache.vendor+'Transform'] !== 'undefined';
To test, this are content of:
index.html
principal.html
app.js
The error happening only when click in "more" button (small screens)

John,

Where is the 'more' button?
Would you be able to post js/contatos.js?

I would mean button to open side panel
<button class="btn icon icon-menu pull-left show-for-phone-only" data-side-panel-id="side-panel-example"></button>
When I click him crash

I tested again this time I get side panel from example path and edit, side panel works only when using
`

Side Panels

</div>

this way not working (external file)`
@Qathom any idea?

Hi @John-Henrique,
Your given error settings.element.style[cache.vendor+'Transfor concerns Snap.JS (the library used for the swipe gesture with side panels (as @andreujuanc already said).
I can't reproduce your issue..
Can you please copy/paste your settings with app.on(..)?
Did you put your side panel tag in the body element?

@Qathom yes, see 4 iteraction on top https://github.com/quark-dev/Phonon-Framework/issues/167#issuecomment-257192869

Hi @John-Henrique,
I think that your error is here: <div class="side-panel side-panel-left" data-expose-aside="left" data-disable="right" data-page="home" id="side-panel-example">. More precisely with the data-page attribute. Your default page is principal and not home.
However, I'll add a warning message in order to avoid this issue.

lol really, you is right


Big clean up

Noteworthy change: Halloween button fancy keyframe animation.
See #145 and #146

The contact form needs some margin, what you have done with the contributors buttons is good, but the rest of the stuff is not aesthetic, or need some more tweaks before I can merge.

Also you can push the image. So I expect another PR for that.

I think this merge is not going to be a good one. Try to merge, and if it's all messed up as heck, give me the merged .css and I should be able to do a much better job with that than Github's unexpectly bare-bonded merge thingy.

Sudz123, I do not have any way to push the image. When attempting to upload a new file into images/halloween, I get this from Github:
"Uploads are disabled.
File uploads require push access to this repository."
That dialog is why I asked you to grab the image at that URL in the file and push the image on your side.

Umm, I think you need to add a new file to the local repo and then do a git add followed by a git commit just as adding some new code, anyways if this is not working I will add the image. However I'd prefer not merging this if it's in a bad shape. Since, finally this project is attaining some stability. What I would suggest is if re-doing all this through patches is too much of a pain simply submit one PR with all the linked changes(This is a special case, I'm making an exception). If it runs well on my local machine I will merge that one PR. You can work further on it then.

I see. So the issue with me being unable to push an image is that I'm using github through the browser instead of via Git Gui or the equivalent. I'll be trying that in a bit.
Indeed. Normally I would make a big single pull, via the local repo set-up. I wanted to familiarize myself with github through the browser, and didn't quite expect to make so much changes. I expected a few cute lines of code there and there, but... I was horrified by what was happening on the HTML and CSS font and got carried away.

Alright so here is the deal, I tried to work things around but this is how the page would look after I merge your PR :

I really think it would be a bad idea to merge this, unless I am building your code wrong locally :/

Nah, go ahead! I can redo the CSS witchery with very little effort on my part.
Halloween button is not animated with the halloween picture, and the title in the header isn't animated. That should take me minutes to fix.  I can stay awake for a bit longer and get all of this straightened out in like, ten minutes tops.
I attempted to push the image from a local repo I made, but sadly, got this error:
"Authentication failed. You may not have permission to access the repository or the repository may have been archived. Open options and verify that you're signed in with an account that has permission to access this repository."
I double checked, I'm signed in my account on my desktop git for sure.

Hmm. Now that I've thought about it for a bit, I'm taking the wrong approach here for sure.
#145 and #146 should be fine to merge. This... You're right, lets not merge this one. I'll do a more careful merge after sleeping.

All right. I'm up now. I'm focusing my efforts on getting my local repo to work with Github. So I can submit one big PR instead of three PRs. Sadly, I'm having more trouble with this than expected.
With Git Gui, I was able to make a local clone of Happy_Hacking, and confirmed that it's pointing to this repo using a HTTPS link.
With Git Gui, I did...

Remote > Fetch From > Origin
Add image to src/images/halloween/ dir
Stage Changes, Commit note, Commit
Merge > Merge Local, image is merged without any issues.
Push

Error I'm getting with Git Gui:
'remote: Permission to sudz123/Happy_Hacking.git denied to DillonSimeone.
fatal: unable to access 'https://github.com/sudz123/Happy_Hacking.git/': The requested URL returned error: 403
Pushing to https://github.com/sudz123/Happy_Hacking.git'
I've tried making a new branch then pushing that, only to get the same error.
As you can see here,
https://stackoverflow.com/questions/22666828/no-pull-in-git-gui
What I did should had worked.
I attempted to use Git bash using this tutorial, https://yangsu.github.io/pull-request-tutorial/, and had the same degree of success as I did with Git Gui:

I'm reading documentations on this right now.

It appears that Git Bash is able to detect the difference between master and my current branch. I'm still getting the strange 403 issue.

I think I see the issue here. I need to make a fork of your project, then clone that locally and make my changes. Then a public pull request should be possible.

Eureka! I've got it!
See #164.
Most excellent. If not for the conflicts, I wouldn't had learned all of this. OK! I should be able to make a single PR for the HTML, CSS and JS in one go now!

Great, I am happy that you have learnt your lesson :P . Since this repo is meant for new comers to get used to git and open sourcing, many contributors com up with the same questions. Get all your answers here through practice and once Hacktoberfest ends, put these skills to use in a good project. Happy Hacking 👯

Heck, yeah! I'm thankful for Hacktoberfest getting me to stop avoiding Github!
I just made a big PR, see #175. Closing this and #146 and #145.


Generate and Publish API javadoc to javadoc.io

We need to generate the api reference docs and automatically publish to gh-page, or the corresponding language standard (docs.rs on rust, pkg dev in go, javadoc.io etc)

This is already being done automatically. See https://tbd54566975.github.io/web5-kt/docs/htmlMultiModule/index.html
Closing.

But it's not being published yet to javadoc.io... I'd prefer to keep it opened until we (OSP) have a chance to review the way we are publishing. WDYT?

Sounds good! I renamed it to make it more clear what this task is about.


remove lint-staged from package

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
15

Relevant Lines:
15

💛  - Coveralls


API documentation editor not working

Steps

Visit https://staging.apinf.io
Login as admin
Navigate to 'Settings'
Enable 'API documentation editor'
Enter details of 'Host'  (eg. http://editor.swagger.io/)
Click on 'Save' button
Add API
Go to 'My APIs' and select above added API
Go to 'Documentation' and click on 'Manage' button
Click on 'Open API documentation editor'

Finding
Click on 'Open API documentation editor' displays empty page
Screenshots

This is a DevOps issue and is fixed now. It can be accessed at https://editor.apinf.io/
Thanks to @shaliko for fixing this.

@bajiat , based on above comment, verified this issue on https://staging.apinf.io v 0.40.0 and it is reproducible.
Screenshot:


Added Subdomain enumeration and network scanning tool

Add changes in Alphabetical Order

Where to add this changes ? As I have just created the directory .
Do I have to change in README file of subdomain

Please reply


Error when encoding map with metadata on symbol key

transit-clj errors when trying to encode a map with a symbol key with metadata on it:
(require 'cognitect.transit)
(in-ns 'cognitect.transit)

(import [java.io File ByteArrayInputStream ByteArrayOutputStream OutputStreamWriter])

(def out (ByteArrayOutputStream. 2000))
(def w (writer out :json {:transform write-meta}))

;; meta on val works
(write w {'key (with-meta 'val {:foo 'bar})})

;; but writing a map with metadata on the key errors
(write w {(with-meta 'key {:foo 'bar}) 'val})

;; Execution error at com.cognitect.transit.impl.AbstractEmitter/emitEncoded (AbstractEmitter.java:1).
;;   Cannot be used as a map key cognitect.transit.WithMeta@5f483ec3

;; writing the same with a second `{}` key works:
(write w {(with-meta 'key {:foo 'bar}) 'val {} 'val2})

I think the issue is that transit should pick a cmap for a symbol with metadata on it.
The same works with transit-cljs:
(require [cognitect.transit :as t]
(def w (t/writer :json {:transform t/write-meta}))
(cognitect.transit/write w {(with-meta 'key {:foo 'bar}) 'val})
;; => "["^ ",["~#with-meta",["~$key",["^ ","~:foo","~$bar"]]],"~$val"]"

I just ran into this issue using transit-clj in babashka sending maps with symbol keys. It appears that symbols contain line / column metadata by default.

@djblue symbols no longer contains line / column metadata in bb anymore.

A workaround is to choose a different (longer than 1 character) tag for clojure.lang.Symbol and override the read and write handler for this:
(ns workaround
  (:require [cognitect.transit :as t])
  (:import [java.io ByteArrayOutputStream]))

(def out (ByteArrayOutputStream. 2000))

(def w (t/writer out :json
                 {:handlers {clojure.lang.Symbol (t/write-handler "$$" t/nsed-name)}
                  :transform t/write-meta}))

(t/write w {(with-meta 'SYMBOL {:foo 'bar}) 'val})

(def ba (.toByteArray out))

(def transit (String. ba))

(prn transit)
;;=> "[\"~#cmap\",[[\"~#with-meta\",[[\"~#$$\",\"SYMBOL\"],[\"^ \",\"~:foo\",[\"^2\",\"bar\"]]]],[\"^2\",\"val\"]]]"

(require '[clojure.java.io :as io])

(def r (t/reader (io/input-stream ba) :json
                 {:handlers (t/read-handler-map {"$$" (t/read-handler symbol)})}))

(prn (map meta (keys (t/read r))))
;;=> ({:foo bar})

I'm not entirely sure what a proper solution for this is, but I noticed there are several tag.length() == 1 checks in https://github.com/cognitect/transit-java/blob/master/src/main/java/com/cognitect/transit/impl/AbstractEmitter.java so a tag length of 1 seems to have a magic meaning.

The map with symbols having metadata passes a stringableKeys check but those symbols need WithMeta encoding meaning that they're not useable in "map" contexts. Maybe symbols with metadata should fail a stringableKeys check. More investigation needed for the right approach here, but it explains the failure and why the $$ workaround works.

I've got another workaround here which surfaces another interesting bit. When using a conditional tag that returns a custom tag when there's metadata on the symbol, but a single dollar when there's not, the transit appears normal and the special tag doesn't even appear.
I'm not sure why this workaround works, but the interesting bit is that it preserves the original transit format which can be read without any modifications on consumers.
(ns workaround
  (:require [cognitect.transit :as t]
            [clojure.java.io :as io])
  (:import [java.io ByteArrayOutputStream]))

(defn obj->str [v]
  (let [out (ByteArrayOutputStream. 2000)
        w (t/writer out :json
                    {:handlers {clojure.lang.Symbol
                                (t/write-handler (fn [v]
                                                   (if (meta v)
                                                     "<SPECIAL_TAG>"
                                                     "$"))
                                                t/nsed-name)}
                     :transform t/write-meta})]
    (t/write w v)
    (String. (.toByteArray out))))

(defn str->obj [s]
  (let [rdr (t/reader (io/input-stream (.getBytes s)) :json
                      #_{:handlers (t/read-handler-map {"$$" (t/read-handler symbol)})})
        v (t/read rdr)]
    v))

(let [map-w-symbol-meta {(with-meta 'SYMBOL {:foo 'bar}) 'val}
      transit-w-meta (obj->str map-w-symbol-meta)
      v (str->obj transit-w-meta)]
  (prn transit-w-meta)
  ;;=> "[\"~#cmap\",[[\"~#with-meta\",[[\"~#$$\",\"SYMBOL\"],[\"^ \",\"~:foo\",[\"^2\",\"bar\"]]]],[\"^2\",\"val\"]]]"

  (prn v) ;;=> {SYMBOL val}
  (prn (map meta (keys v))) ;;=> ({:foo bar})
  )

(let [map-wo-symbol-meta {'SYMBOL 'val}
      transit-w-meta (obj->str map-wo-symbol-meta)
      v (str->obj transit-w-meta)]
  (prn transit-w-meta)
  ;;=> "[\"^ \",\"~$SYMBOL\",\"~$val\"]"
  (prn v) ;;=> {SYMBOL val}
  (prn (map meta (keys v))) ;;=> (nil)
  )

We have a fix for this prepped that you can try, it uses a git dep for transit-java that requires prepping (there is no change in transit-clj):
In your deps.edn:
{:deps
 {com.cognitect/transit-clj {:mvn/version "1.0.324"}
  com.cognitect/transit-java {:git/url "https://github.com/cognitect/transit-java.git" :git/sha "f84ca6290b7a0489a8671bc96bd76ede8b391415"}}}

And then you will need to prep with:
clj -X:deps prep

And then you should be able to test this fix. Please let us know what you find!

@puredanger @fogus looks good to me running first tests on the repl, thank you! Will also plug this into our CI and report back if we encounter any other issues.

Released in transit-java 1.0.362 and transit-clj 1.0.329


Photographer profile

Codecov Report

Merging #35 (4a5a8e6) into master (1a83d0a) will increase coverage by 0.93%.
The diff coverage is 97.22%.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
+ Coverage   92.41%   93.35%   +0.93%
==========================================
  Files          25       33       +8
  Lines         277      346      +69
  Branches        6       11       +5
==========================================
+ Hits          256      323      +67
- Misses         21       23       +2

Impacted Files
Coverage Δ

...tructure/User/Repository/PhotographerRepository.ts
35.71% <0.00%> (-5.96%)
:arrow_down:

...c/Application/User/Command/UpdateProfileCommand.ts
100.00% <100.00%> (ø)

...cation/User/Command/UpdateProfileCommandHandler.ts
100.00% <100.00%> (ø)

...Application/User/Query/GetPhotographerByIdQuery.ts
100.00% <100.00%> (ø)

...tion/User/Query/GetPhotographerByIdQueryHandler.ts
100.00% <100.00%> (ø)

...pi/src/Application/User/Query/LoginQueryHandler.ts
100.00% <100.00%> (ø)

api/src/Application/User/View/PhotographerView.ts
100.00% <100.00%> (ø)

...omain/User/Exception/EmailAlreadyExistException.ts
100.00% <100.00%> (ø)

...in/User/Exception/PhotographerNotFoundException.ts
100.00% <100.00%> (ø)

api/src/Domain/User/Photographer.entity.ts
100.00% <100.00%> (ø)

... and 10 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #35 (4a5a8e6) into master (1a83d0a) will increase coverage by 0.93%.
The diff coverage is 97.22%.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
+ Coverage   92.41%   93.35%   +0.93%
==========================================
  Files          25       33       +8
  Lines         277      346      +69
  Branches        6       11       +5
==========================================
+ Hits          256      323      +67
- Misses         21       23       +2

Impacted Files
Coverage Δ

...tructure/User/Repository/PhotographerRepository.ts
35.71% <0.00%> (-5.96%)
:arrow_down:

...c/Application/User/Command/UpdateProfileCommand.ts
100.00% <100.00%> (ø)

...cation/User/Command/UpdateProfileCommandHandler.ts
100.00% <100.00%> (ø)

...Application/User/Query/GetPhotographerByIdQuery.ts
100.00% <100.00%> (ø)

...tion/User/Query/GetPhotographerByIdQueryHandler.ts
100.00% <100.00%> (ø)

...pi/src/Application/User/Query/LoginQueryHandler.ts
100.00% <100.00%> (ø)

api/src/Application/User/View/PhotographerView.ts
100.00% <100.00%> (ø)

...omain/User/Exception/EmailAlreadyExistException.ts
100.00% <100.00%> (ø)

...in/User/Exception/PhotographerNotFoundException.ts
100.00% <100.00%> (ø)

api/src/Domain/User/Photographer.entity.ts
100.00% <100.00%> (ø)

... and 10 more

Continue to review full report at Codecov.

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


feat!: parse colors

Instead of storing raw hex strings, create a Color struct and implement parsing and dumping for that.
BREAKING CHANGE: Color fields are not Strings, but Color values now.

Curiously, there doesn't seem to be a standard crate for representing colors. All libraries that I browsed through seem to be rolling their own.

Curiously, there doesn't seem to be a standard crate for representing colors. All libraries that I browsed through seem to be rolling their own.

Good thing that the implementation is straightforward! I don't mind having one less dependency for the project.


render called twice whit dynamic import

Version
2.6.10
Reproduction link
https://codesandbox.io/s/rendercalledtwice-rfrbt
Steps to reproduce
I want Vue can render the template by follow the json file which I defined.
so I start trying trough by render methods with dynamic import component,
then seen this happen. it show the same template double times when I using the ssr mode
so I try to reproduction it at spa mode and it still called twice even thought it looks fine at spa mode
What is expected?
render only called once even using dynamic import
What is actually happening?
it called twice
trouble with double template in ssr mode

This is expected, it renders once and then again once the dynamic component is loaded. This allows partially rendering a template, it's a feature.

it render once on server side
and then twice on client side
total is three times

it render once on server side
and then twice on client side
total is three times

This is expected, it renders once and then again once the dynamic component is loaded. This allows partially rendering a template, it's a feature.

oh ! I understand what you mean for now. but is any possible to discriminate twice render ?


XML Parsing Error: not well-formed when canceling an import

When canceling an import preview an error is thrown in Firefox Console.
To Reproduce
Steps to reproduce the behavior:

Use the Clipboard importer to import some text
Click Next and see the preview rows.
Click Start Over button and notice the error in Firefox Console

Current Results
XML Parsing Error: not well-formed
Location: http://<IP_ADDRESS>:3333/command/core/cancel-importing-job?jobID=4
Line Number 1, Column 1:

Expected Behavior
No XML Parsing Error
Screenshots

Versions

Operating System: Windows 10
Browser Version: Firefox latest
JRE or JDK Version: JDK8
OpenRefine: openrefine-3.4-beta-388-g1dcc832

Datasets

Additional context

Also happens for
XML Parsing Error: not well-formed
Location: http://<IP_ADDRESS>:3333/command/core/recon-judge-similar-cells?project=2519405186300
Line Number 1, Column 1: recon-judge-similar-cells:1:1

when clicking on the "Match All Identical Cells" button during Reconciling a cell value.

And happens when clicking on the Undo blue link on the yellow banner that pops up after you perform an operation:

XML Parsing Error: not well-formed
Location: http://<IP_ADDRESS>:3333/command/core/undo-redo?undoID=1595977412132&project=2519405186300
Line Number 1, Column 1: undo-redo:1:1

Transpose -> Columnize by Key/Value
XML Parsing Error: not well-formed
Location: http://<IP_ADDRESS>:3333/command/core/key-value-columnize?keyColumnName=item+-+batters+-+batter&valueColumnName=item+-+type&noteColumnName=item+-+batters+-+batter+-+id&project=2519405186300
Line Number 1, Column 1: key-value-columnize:1:1

This is just because the HTTP responses from the backend lack a Content-Type: application/json header (many commands return JSON as plain text).


Error

Hi ,
yarn run v1.22.19
$ node src/index.js
file:///root/NWTIMERBOT/node_modules/node-fetch/src/index.js:108
reject(new FetchError(request to ${request.url} failed, reason: ${error.message}, 'system', error));
^
FetchError: request to https://worldtimeapi.org/api/ip failed, reason: read ECONNRESET
at ClientRequest. (file:///root/NWTIMERBOT/node_modules/node-fetch/src/index.js:108:11)
at ClientRequest.emit (node:events:513:28)
at TLSSocket.socketErrorListener (node:_http_client:481:9)
at TLSSocket.emit (node:events:513:28)
at emitErrorNT (node:internal/streams/destroy:151:8)
at emitErrorCloseNT (node:internal/streams/destroy:116:3)
at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
type: 'system',
errno: 'ECONNRESET',
code: 'ECONNRESET',
erroredSysCall: 'read'
}
Node.js v18.7.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error message above

This error comes from the configuration of your proxy... I add some tests to prevent crashes if the application cannot call the api to get the time from the internet.

I invite you to download/clone this changes.

Thank you !

I invite you to download/clone this changes.

can i get fix too ? @BigChicChicken


Fixed all Places I expected/found errors

Hi I am back, I went through the typing cause I encountered more errors and others I found in it, so I fixed it should work

Yeah

thanks


Make the local readme-files MD and transform them to txt in NuGet

Is your feature request related to a problem? Please describe.
In addition to our main README.md (in git root) we have one README.txt per project. This is because they will get bundled with the NuGet package and be shown to the user on package install in Visual Studio. While support for packing *.MD files is planned, it's currently not supported, so they are currently in *.TXT (even though we've used the MD syntax). As @nikolaykrondev pointed out, it does not look optimal when viewed at GitHub, there *.MD would be better.
What area is it related to
All
Describe the solution you'd like
Optimally we would store them as *.MD but "rename" them when doing the packaging so they are bundled as .TXT. When NuGet supports *.MD we can just remove the rename step.

Note: I did give this a quick try when I first added the Readme files, but something (can't remember what) blocked me at that time and didn't want to get stuck on it. But it should be doable.


Remove Flow

Flow bindings are generated because projects downstream were not yet converted to TypeScript. Recently, the last project was migrated, so this generation should be removed.

Probably this could be done as well for all our JS libs:

https://github.com/freckle/exhaustive-js
https://github.com/freckle/parser-js
https://github.com/freckle/maybe-js
https://github.com/freckle/non-empty-js
https://github.com/freckle/query-params-js
https://github.com/freckle/resource-status-js


Fixed the search of System properties

Mapped system properties to retrieve them in any language
Fixed delete button to empty the new combobox

Oh, good.  I was going to start on this today.  Thank you!


Flexbox .list-group implementation

Requires the pull-right and pull-left flex change from PR #18976
Resolves #18978

@rosskevin This also has merge conflicts.

Hi @rosskevin!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

Changes must be made to the original Sass or Less source code file(s), not just the compiled CSS file(s).

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)


feat: Implement raw string literals

Description
Problem
Resolves #3475
Summary
As in the title: implements raw string literals r#"foo"#.
Like Rust, we allow up to 255 hashes surrounding the doublequotes.
The grammar for raw string is like Rust's
RAW_STRING_LITERAL = 'r' RAW_STRING_CONTENT SUFFIX?

RAW_STRING_CONTENT = '"' ANY* (non-greedy) '"' | '#' RAW_STRING_CONTENT '#'

Not sure about Rust's IsolatedCR*.
Additional Context
Needed soon for JSON tests.
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

hey
review? @jfecher
cc @kevaundray
cc @grasshopper47

Can you kindly add some Noir tests as well, for completion's sake?
This (I think) is where they're stored: https://github.com/noir-lang/noir/tree/master/tooling/nargo_cli/tests
Otherwise it LTGM and many thanks for this quick job!!

How is it possible these checks are failing?
RequestError [HttpError]: Resource not accessible by integration
Error: Unhandled error: HttpError: Resource not accessible by integration
    at /home/runner/work/_actions/actions/github-script/v3/dist/index.js:2591:23
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  status: 403,
  headers: {
    'access-control-allow-origin': '*',
    'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset',
    connection: 'close',
    'content-encoding': 'gzip',
    'content-security-policy': "default-src 'none'",
    'content-type': 'application/json; charset=utf-8',
    date: 'Fri, 24 Nov [20](https://github.com/noir-lang/noir/actions/runs/6980598368/job/18996157744?pr=3556#step:3:21)23 12:16:42 GMT',
    'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
    server: 'GitHub.com',
    'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
    'transfer-encoding': 'chunked',
    vary: 'Accept-Encoding, Accept, X-Requested-With',
    'x-accepted-github-permissions': 'issues=write; pull_requests=write',
    'x-content-type-options': 'nosniff',
    'x-frame-options': 'deny',
    'x-github-api-version-selected': '20[22](https://github.com/noir-lang/noir/actions/runs/6980598368/job/18996157744?pr=3556#step:3:23)-11-28',
    'x-github-media-type': 'github.v3',
    'x-github-request-id': '782A:9398:6CEE58:E3153D:6560942A',
    'x-ratelimit-limit': '15000',
    'x-ratelimit-remaining': '14933',
    'x-ratelimit-reset': '1700828258',
    'x-ratelimit-resource': 'core',
    'x-ratelimit-used': '67',
    'x-xss-protection': '0'
  },
  request: {
    method: 'POST',
    url: 'https://api.github.com/repos/noir-lang/noir/issues/3556/labels',
    headers: {
      accept: 'application/vnd.github.-preview+json',
      'user-agent': 'actions/github-script octokit-core.js/3.2.1 Node.js/16.20.2 (linux; x64)',
      authorization: 'token [REDACTED]',
      'content-type': 'application/json; charset=utf-8'
    },
    body: '{"labels":["documentation"]}',
    request: { agent: [Agent], hook: [Function: bound bound register] }
  },
  documentation_url: 'https://docs.github.com/rest/issues/labels#add-labels-to-an-issue'
}

No idea why formatting is failing

Done. One check is failing.

That's just because you're developing from a fork. We can ignore that check as it's not relevant.

Done.
Fixed a small bug in the most recent commit.

Done.
Fixed a small bug in the most recent commit.

You need to update the documentation checkbox in the description to included in this PR


ZAP Tool generates not compiling code

Hi,
I'm trying to generate a cluster with:

Binary Input
Percentage Remaining
Identify

The generated code for the binary input is not compiling. It also looks very strange:
In callback.h:
/** @brief Binary Input (Basic) Cluster Init
 *
 * Cluster Init
 *
 * @param endpoint    Endpoint that is being initialized
 */
void emberAfBinaryInput(Basic) ClusterInitCallback(chip::EndpointId endpoint);

and in gen_config.h:
// Use this macro to check if the server side of the Binary Input (Basic) cluster is included
#define ZCL_USING_BINARY_INPUT_BASIC_CLUSTER_SERVER
#define EMBER_AF_PLUGIN_BINARY_INPUT_(_BASIC_) _SERVER
#define EMBER_AF_PLUGIN_BINARY_INPUT_(_BASIC_)

Some compiler error output related to this code snippets:
../src/gen/gen_config.h:48: warning: "EMBER_AF_PLUGIN_BINARY_INPUT_" redefined
   48 | #define EMBER_AF_PLUGIN_BINARY_INPUT_(_BASIC_)
      |
../src/gen/gen_config.h:47: note: this is the location of the previous definition
   47 | #define EMBER_AF_PLUGIN_BINARY_INPUT_(_BASIC_) _SERVER

and
In file included from ../src/gen/call-command-handler.cpp:25:
../src/gen/callback.h:56:30: error: variable or field 'emberAfBinaryInput' declared void
   56 | void emberAfBinaryInput(Basic) ClusterInitCallback(chip::EndpointId endpoint);
      |                              ^
../src/gen/callback.h:56:25: error: 'Basic' was not declared in this scope
   56 | void emberAfBinaryInput(Basic) ClusterInitCallback(chip::EndpointId endpoint);

Here is also the used file for the generation: config.zap.txt

It seems that the problem is caused by the name property set to Binary Input (Basic) - please note the parenthesis.
During the generation of the code the asCamelCased method is used, and as a result, the electron's toCamelCase method is called here. As you can see below, this method does not remove ( or ) characters which result in building errors afterwards.

The quick workaround might be to change the name property. But for the future, I wonder if we should ignore parenthesis - @tecimovic.

We should fix this to ignore the parenthesis, I'll talk to @tecimovic about this fix

I'll take a look. Thanks for debugging this issue already, @LuDuda !

Ampersand (&) should also be on the ignore list. Example which is not yet in any of the XMLs:
#define EMBER_AF_PLUGIN_LUMINAIRE_DIAGNOSTICS_ &_MAINTENANCE

PR has been merged. Please close the issue after retesting.


Introduce Helm chart tests in the CI pipeline

We currently have nothing to prevent regressions in the Helm charts from landing in the main branch. This is quite extensive work, but it would be worthwhile.

partly implemented in #438


Linux workaround

Closes #353.

Can we please not work/make changes to the old CLI? Gives me a lot of overhead porting it.

One line of code is overhead? The "old" CLI is still in use so how else can I solve this issue?

Just add it and I will do it if the new cli is delivered...


Chat: don't overwrite full messages w/ cached ones

Waiting until after the release to merge this
@mmaxim this stops us overwriting although while i was trying to test this and i think you don't send a full until cached replies. i put in a delay and full doesn't beat it so i had to test it very artificiallly

@keybase/react-hackers this bookkeeps where the messages come from to ensure we never overwrite a full load with a cached load. full should be the most accurate load so the flow should be cached -> full and never full -> cached

this was speculative, closing


[SHIRO-530] INI parser does not properly handled backslashes at end o…

…f values

Do not skip escape characters for the value (new behaviour demanded by SHIRO-530).
rearrange and comment tests to reflect old and new, desired behaviour.

Signed-off-by: Benjamin Marwell<EMAIL_ADDRESS>Alternative to https://github.com/apache/shiro/pull/209

@bmhm sounds good!

@bdemers can you review please just before I merge it?

retest this please

I think the complexity is that we can have some regex or URI in the value, so we have to deal correctly with the backslash

@bmhm Thanks for the great work ;)


官网说的mmdet2.4支持mmcv1.2的，为什么实际不支持，报的错自相矛盾呀

File "/mmdet/mmdetection-2.4.0/mmdetection-2.4.0/mmdet/init.py", line 25, in
f'MMCV=={mmcv.version} is used but incompatible. '
AssertionError: MMCV==1.2.0 is used but incompatible. Please install mmcv>=1.1.1, <=1.2.
报的错自相矛盾呀

Mmdet2.4 supports MMCV1.2, why not？
MMCV==1.2.0 is used but incompatible. Please install mmcv>=1.1.1, <=1.2.

File "/mmdet/mmdetection-2.4.0/mmdetection-2.4.0/mmdet/init.py", line 25, in
f'MMCV=={mmcv.version} is used but incompatible. '
AssertionError: MMCV==1.2.0 is used but incompatible. Please install mmcv>=1.1.1, <=1.2.


Avoiding fatal() printout when getting weights branch names

The printouts now look like this when running on data:
[first occurence]
Trying to access incorrect event-level branch: genWeight, it's probably fine for data if this is a gen weight branch.
[first occurence] CutFlowManager failed to get gen weights from branch genWeight
[first occurence] NanoEventProcessor failed to get gen weight from branch: genWeight


Crash: json: cannot unmarshal object into Go

2020/03/20 15:30:04 json: cannot unmarshal object into Go struct field Attachment.messages.attachments.blocks of type slack.Block

Version: 2ee21247add4e50660e8b3b90b285814d21d8ad6
Installation method: go get -u github.com/erroneousboat/slack-term

Related: https://github.com/slack-go/slack/issues/645

Hi @lunemec @erroneousboat
Can I try solving this ? (Crash: json: cannot unmarshal object into Go #237)

Gathering from the issue on slack-go, there has been a merge of a pull request that might resolve the issue. I've updated the code to use the newest version of slack-go on master. Check out master to see if the problem persists or not.

Encountered the same issue. Will test 0.5.1 once released.
json: cannot unmarshal object into Go struct field Attachment.messages.attachments.blocks of type slack.Block

@erroneousboat , I was using the 0.5.0 binary version and had the same bug.
I just installed slack-term from master with the go method.
No more crashes.

I had to do go install github.com/erroneaousboat/slack-term@master then ~/go/bin/slack-term to get it to work


ctrl+P stopped working!!

We have written the needed data into your clipboard because it was too large to send. Please paste.

What do you mean by stopped working. Can you please provide necessary information to reproduce the issue on our side to fix it?


Fix wrong HTML Imports loaded callback in 'using'

HTMLImports.whenImportsReady was renamed to whenReady.

+1

+1 !!!!!

Thanks for the PR here! We recently made a major change to the master branch of this repo with the 0.9 release - would you mind re-submitting a PR here rebasing from the latest master? Thanks!


fix: add https protocol

What's in this PR:

add "protocol = https" to sitemaps

Codecov Report

Merging #763 (3a51973) into master (db7aa3c) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #763      +/-   ##
==========================================
- Coverage   80.27%   80.25%   -0.02%
==========================================
  Files          94       94
  Lines        3447     3449       +2
  Branches      510      510
==========================================
+ Hits         2767     2768       +1
+ Misses        588      587       -1
- Partials       92       94       +2

Impacted Files
Coverage Δ

...ictionary/management/commands/convertndstopanes.py
26.31% <ø> (ø)

CreeDictionary/CreeDictionary/paradigm/panes.py
62.96% <ø> (ø)

CreeDictionary/CreeDictionary/sitemaps.py
100.00% <100.00%> (ø)

CreeDictionary/CreeDictionary/paradigm/filler.py
89.66% <0.00%> (-0.42%)
:arrow_down:

...ranslate/management/commands/translatewordforms.py
73.17% <0.00%> (ø)

Continue to review full report at Codecov.

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


[d&d] Reordering in filter/group by with drag and drop will be blocked when the tooltip showed

Reordering in 'FILTERS'/'GROUP BY' with drag and drop will be blocked when the tooltip showed.
Expected results
Reordering in 'FILTERS'/'GROUP BY' with drag and drop will  not be blocked
Actual results
Reordering in 'FILTERS'/'GROUP BY' with drag and drop will be blocked when tooltip is showed in the way, user need to drag in a direction to avoid blocking by the tooltip
Screenshots
https://user-images.githubusercontent.com/81597121/127755037-06ef6cba-a6c6-49a3-812d-47e1a5a9a2d6.mov
How to reproduce the bug

Enable drag and drop
In Filter or Group-by, drop multiple
wait for tooltip and do drag and drop
See error

Environment
(please complete the following information):

superset version: master
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

For tooltip with long name, it is blocking the window of the above filter and hard to move filter up with d&d

Thanks for filing, this is awesome!! @jinghua-qa

We should never show a tooltip that repeats the exact same text as the component. I noticed this is many places in Explore. I also would advocate for info bubbles (the little i with a circle around it as the trigger point for the tooltip when it provides extra information)

@mistercrunch technically difficult if we wanna show long text, as the panel is resizable. we can remove behind feature flag UX_Beta though.

@rusackas @michael-s-molina I renamed this issue, since user can always resize the panel to show all text, i am totally fine with removing tooltip. let's put it behind UX_BETA.

In that case, should we also remove the limit of 33% max width for the resizable panels?

In that case, should we also remove the limit of 33% max width for the resizable panels?

let's do it. but it will increase the QA workload. or increase the max to 40%/50%? @michael-s-molina need some experiment.

https://user-images.githubusercontent.com/67837651/127862880-94ebfaa1-ef35-465d-a24a-6a288399bf24.mov


[SOA Bot] A mechanism for evaluating proposed orders

A function that takes in the current orders for a bot and proposed orders for it from other bots. It then evals the proposed orders and modifies its orders if it likes any of them.

Closed in favor of https://github.com/ALLAN-DIP/baseline_bots/issues/28


Move Java baseline to 17

TSIA

There is a problem using Java 17 with old dependencies grep4j and lombok. I will look at it

There is a problem using Java 17 with old dependencies grep4j and lombok. I will look at it

okay, maybe you can create PR as I don't really want to test this manually. Can you bump these old dependencies?

Changed my mind again, I am not able to remove this dependency without considerable effort, so closing and staying on JDK 11.


Ambassador agent labels

Description
Add deploymentLabels and podLabels to the ambassador-agent deployment.
Related Issues
N/A
Testing
Manual test.
Checklist

[N/A] Does my change need to be backported to a previous release?

What backport versions were discussed with the Maintainers in the Issue?

[N/A ] I made sure to update CHANGELOG.md.
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[X] This is unlikely to impact how Ambassador performs at scale.

Only adding some extra labels on the deployment, should not impact performance of the deployment itself.

[ ] My change is adequately tested.
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[N/A] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.

[N/A] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

@jnieuwen - Thanks for the PR. The Agent code was extracted to its own component and now has its own chart so can you take a look here to see if that meets your needs and if not submit a PR there?
https://github.com/datawire/ambassador-agent/tree/master/helm/ambassador-agent
We would like to move away from Agent specific work in this repository and maintaining duplicate templates. So, our idea would be either to remove the templating in the emissary-ingress chart. We can then either sub-chart it the Agent or not allowing it to evolve at a faster cadence independent of emissary-ingress releases.

Hi @LanceEa - that also might work for our use case. Thanks for the reply. From my point the PR can be closed.

@jnieuwen - Sounds good, going to go ahead and close this PR.


Training Infrastructure

Hi, great work! I'd like to know which compute infrastructure was used to train SEA-RAFT. More specifically, how many GPUs have been used to train SEA-RAFT, and how long did full training (w/ pre-training on TartanAir) take? Thanks for the help :)

We trained SEA-RAFT on 8 L40s. I do not remember the full training time but I believe it can be done in <1 week. Specifically, the ablation setting (540 x 960 input, 12k steps) takes about <1.5 days for SEA-RAFT(M).

Amazing, thanks for the details!

What could be done to train the model further? Is the problem the availability of great datasets?
I find that any optical flow model has quite a bit of drift over time. Is that a lack of good real-world training data?

Could one of the checkpoints be trained further on this dataset for better results?
https://github.com/google-research/kubric


Create class to orchestrate fontwr.json

This PR closes the issue #6 .

This was huge! Thank you!!


F.Cu/F.Mask layer option not working

KiBuzzard 1.2.0 works great with different inputs and fonts. Thanks.
An example output on the F.Silkscreen layer:

I tried to use the F.Cu/F.Mask option but after clicking OK nothing happens. No entry in the log and the window seems to stop working (no preview updates).
I expected something like this:

Can someone confirm that this is(n't) working? Thanks.
KiCad 6.0.7 @Win10

It currently doesn't work the way you're describing. Although that output does look better. for the inverted options, IMO.
F.Cu/F.Mask option currently should create a label with the same contents on both The copper and mask layers.


Support to f4 address and general improvements for filecoin integration

📝 Description
We have added support to new address type 4, which will be released on mainnet next year. Besides, we apply some general improvements in order to avoid possible bug in certain scenarios.
❓ Context

Impacted projects: common
Linked resource(s): ``

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

@haammar-ledger did you have the chance to see this PR?


single file mounts not respected by lstat and readlink in container

Description of problem:
Using -v to bind mount a file into a container does not appear to modify the behavior of lstat or readlink if the target path on the container already exists and is a symbolic link.
This causes programs that use these system calls to continue seeing the file present in the container instead of the file on the host system.
In our case this caused Java processes to incorrectly resolve the timezone.  An example of this behavior is provided below.

How reproducible:
We noticed this on our Java app server containers, which derive from the official CentOS7 base image.  Recently that image was updated so that /etc/localtime sym links to ../usr/share/zoneinfo/UTC
Our docker run commands use -v /etc/localtime:/etc/localtime to ensure that containers inherit the timezone of the VM they run on.  We noticed this failed to work with the latest CentOS 7 image.  After digging a bit, it appears that the JVM uses lstat and readlink to resolve /etc/localtime and this leads to unexpected results when combined with -v
Steps to Reproduce:
I have uploaded an image with Java 8 and a small Java program which demonstrates the issue.  This issue shouldn't be specific to Java, but it was the simplest way for me to demonstrate the precise symptom.

docker pull coopernurse/centos7-java
set timezone on host to something other than UTC
docker run -v /etc/localtime:/etc/localtime coopernurse/centos7-java /bin/bash -c '/bin/date && java Hello'

Actual Results:
Fri Jun 26 16:05:22 PDT 2015
Fri Jun 26 23:05:22 UTC 2015

Expected Results:
Fri Jun 26 16:05:22 PDT 2015
Fri Jun 26 16:05:22 PDT 2015

Additional info:
strace shows that /bin/date is simply calling open('/etc/localtime') whereas the JVM is calling lstat and then readlink
readlink still reports the original sym link, despite the -v bind mount:
$ docker run -v /etc/localtime:/etc/localtime coopernurse/centos7-java readlink /etc/localtime
../usr/share/zoneinfo/UTC

More strace output:
docker run -it --privileged coopernurse/centos7-java
[root@7befc66fc860 /]# strace -f java Hello 2>&1 | grep time
[pid    78] futex(0x7f1918009354, FUTEX_WAIT_BITSET_PRIVATE, 1, {5920, 570927155}, ffffffff) = -1 ETIMEDOUT (Connection timed out)
[pid    78] futex(0x7f1918009354, FUTEX_WAIT_BITSET_PRIVATE, 1, {5920, 572068364}, ffffffff) = -1 ETIMEDOUT (Connection timed out)
[pid    78] futex(0x7f1918009354, FUTEX_WAIT_BITSET_PRIVATE, 1, {5920, 573228882}, ffffffff) = -1 ETIMEDOUT (Connection timed out)
[pid    78] read(3, "\312\376\272\276\0\0\0004\0\v\1\0\3()V\1\0\31RuntimeVisibl"..., 201) = 201
[pid    78] open("/etc/localtime", O_RDONLY|O_CLOEXEC) = 4
[pid    78] open("/etc/timezone", O_RDONLY) = -1 ENOENT (No such file or directory)
[pid    78] lstat("/etc/localtime", {st_mode=S_IFLNK|0777, st_size=25, ...}) = 0
[pid    78] readlink("/etc/localtime", "../usr/share/zoneinfo/UTC", 4096) = 25
[pid    87] <... futex resumed> )       = -1 ETIMEDOUT (Connection timed out)

vs
[root@7befc66fc860 /]# strace -f /bin/date 2>&1 | grep time
open("/etc/localtime", O_RDONLY|O_CLOEXEC) = 3

Hello.java source:
[root@e868f7d70484 /]# cat Hello.java
import java.util.Date;

public class Hello {

    public static void main(String argv[]) {
        System.out.println(new Date());
    }

}

The way this ends up getting mounted is the file ends up being mounted on the path the symlink is pointing to. The symlink is resolved before the mount.
Closing since we can't really change this behavior.

I just created an new issue may related.
https://github.com/docker/docker/issues/16481


Track status is reset after seekTo (Expo)

Describe the Bug
After calling seekTo with any value, the track starts playing again
Code To Reproduce
TrackPlayer.seekTo(15);

Environment Info:
react-native-track-player version: ^3.2.0-d23c3fb990aaf9d6783881fabe21e059b62b0782
Real device? Real device
What OS are you running? Android 12

Im having same issue on Android in project where we don't use expo - i think this is general issue

By the way, the notification does not have the ability to seek, could this be a problem?
P.S. The capabilities are set correctly
    await TrackPlayer.updateOptions({
      progressUpdateEventInterval: 1,
      ratingType: RatingType.Heart,
      capabilities: [0, 3, 5, 9, 10, 12],
      android: {
        appKilledPlaybackBehavior:
          AppKilledPlaybackBehavior.StopPlaybackAndRemoveNotification,
      },
    });

+1, having this issue as well on Android

I'm using TrackPlayer.seekTo(15); in a production app on 3.2 with no issues.
If you can provide a reproducible bug then we can look into it.
Cheers
Brad

Having the same issue

I am having an issue that may or may not be related...when I call seekTo the playbackState event is fired with State.Ready. This seems to be a new behavior since we upgrading from v2 to v3. Just throwing this out there in case it is breaking people's existing setups (ex. having some logic that resets the player to 0 when State.Ready event is fired).


Refactor how transformers are structured; remove @transformer decorator

Closes #27

Codecov Report

Merging #36 (a0d2b4b) into main (435c77f) will decrease coverage by 0.45%.
The diff coverage is 99.42%.

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   94.26%   93.80%   -0.46%
==========================================
  Files           6       11       +5
  Lines         366      339      -27
==========================================
- Hits          345      318      -27
  Misses         21       21

Impacted Files
Coverage Δ

robotidy/decorators.py
100.00% <ø> (ø)

robotidy/transformers/ReplaceRunKeywordIf.py
98.30% <98.30%> (ø)

robotidy/transformers/AssignmentNormalizer.py
100.00% <100.00%> (ø)

robotidy/transformers/DiscardEmptySections.py
100.00% <100.00%> (ø)

...obotidy/transformers/NormalizeSectionHeaderName.py
100.00% <100.00%> (ø)

robotidy/transformers/NormalizeSettingName.py
100.00% <100.00%> (ø)

robotidy/transformers/__init__.py
100.00% <100.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #36 (a0d2b4b) into main (435c77f) will decrease coverage by 0.45%.
The diff coverage is 99.42%.

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   94.26%   93.80%   -0.46%
==========================================
  Files           6       11       +5
  Lines         366      339      -27
==========================================
- Hits          345      318      -27
  Misses         21       21

Impacted Files
Coverage Δ

robotidy/decorators.py
100.00% <ø> (ø)

robotidy/transformers/ReplaceRunKeywordIf.py
98.30% <98.30%> (ø)

robotidy/transformers/AssignmentNormalizer.py
100.00% <100.00%> (ø)

robotidy/transformers/DiscardEmptySections.py
100.00% <100.00%> (ø)

...obotidy/transformers/NormalizeSectionHeaderName.py
100.00% <100.00%> (ø)

robotidy/transformers/NormalizeSettingName.py
100.00% <100.00%> (ø)

robotidy/transformers/__init__.py
100.00% <100.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

Basically it moves transformers from transformers.py file (which was getting quite long) to this structure:

ClassName.py / class ClassName syntax is similiar how we're creating listeners for Robot Framework (so it should be easier to use).
Removed @transformer decorator as it's not needed anymore.

Basically it moves transformers from transformers.py file (which was getting quite long) to this structure:

ClassName.py / class ClassName syntax is similiar how we're creating listeners for Robot Framework (so it should be easier to use).
Removed @transformer decorator as it's not needed anymore.