Docker erlerobot/gym-gazebo no display.

Hi there! Sorry to bother you. I am trying to run Erlecopter from this repository https://github.com/erlerobot/gym-gazebo#usage with Docker. I'm following the steps but gazebo is not being displayed when I am issuing the command gzclient in another terminal.
The error log is below, I'm getting this in my terminal verbose after I run the erlecopter program.
Your help would be appreciated.

[2019-04-04 05:25:02,940] Making new env: GazeboErleCopterHover-v0
/usr/local/lib/python2.7/dist-packages/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
warnings.warn('Matplotlib is building the font cache using fc-list. This may take a moment.')
##############################################################################
Load Erle-Copter parameters in MavProxy console (sim_vehicle.sh):
MAV> param load /usr/local/gym/gym-gazebo/gym_gazebo/envs/installation/../assets/params/Erle-Copter.param
Then, press  here to launch Gazebo...
##############################################################################Warning: This program is an suid-root program or is being run by the root user.
The full text of the error or warning message cannot be safely formatted
in this environment. You may get a more descriptive message by running the
program as a non-root user or by removing the suid bit on the executable.
xterm: Xt error: Can't open display: %s
xterm: DISPLAY is not set
Roscore launched!
Unable to register with master node [http://localhost:11311]: master may not be running yet. Will keep trying.
... logging to /root/.ros/log/06019c26-569a-11e9-a579-0242ac110003/roslaunch-adcf198a9e82-566.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.
started roslaunch server http://adcf198a9e82:43213/
ros_comm version 1.11.20
SUMMARY
PARAMETERS

/rosdistro: indigo
/rosversion: 1.11.20

NODES
auto-starting new master
process[master]: started with pid [580]
ROS_MASTER_URI=http://adcf198a9e82:11311/
setting /run_id to 06019c26-569a-11e9-a579-0242ac110003
Gazebo launched!
Taking off in in 10s
... logging to /root/.ros/log/06019c26-569a-11e9-a579-0242ac110003/roslaunch-adcf198a9e82-598.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.
process[rosout-1]: started with pid [607]
started core service [/rosout]
Taking off in in 9s
Taking off in in 8s
started roslaunch server http://adcf198a9e82:43087/
SUMMARY
CLEAR PARAMETERS

/mavros/

PARAMETERS

/mavros/cmd/use_comp_id_system_control: False
/mavros/conn/heartbeat_rate: 1.0
/mavros/conn/system_time_rate: 0.0
/mavros/conn/timeout: 10.0
/mavros/conn/timesync_rate: 0.0
/mavros/fcu_url: udp://<IP_ADDRESS>:5...
/mavros/gcs_url<EMAIL_ADDRESS>/mavros/global_position/frame_id: fcu
/mavros/global_position/rot_covariance: 99999.0
/mavros/global_position/tf/child_frame_id: fcu_utm
/mavros/global_position/tf/frame_id: local_origin
/mavros/global_position/tf/send: False
/mavros/image/frame_id: px4flow
/mavros/imu/angular_velocity_stdev: 0.000349065850399
/mavros/imu/frame_id: fcu
/mavros/imu/linear_acceleration_stdev: 0.0003
/mavros/imu/magnetic_stdev: 0.0
/mavros/imu/orientation_stdev: 1.0
/mavros/local_position/frame_id: fcu
/mavros/local_position/tf/child_frame_id: base_link
/mavros/local_position/tf/frame_id: world
/mavros/local_position/tf/send: True
/mavros/mission/pull_after_gcs: True
/mavros/mocap/use_pose: True
/mavros/mocap/use_tf: False
/mavros/plugin_blacklist: ['actuator_contro...
/mavros/plugin_whitelist: []
/mavros/px4flow/frame_id: px4flow
/mavros/px4flow/ranger_fov: 0.0
/mavros/px4flow/ranger_max_range: 5.0
/mavros/px4flow/ranger_min_range: 0.3
/mavros/safety_area/p1/x: 1.0
/mavros/safety_area/p1/y: 1.0
/mavros/safety_area/p1/z: 1.0
/mavros/safety_area/p2/x: -1.0
/mavros/safety_area/p2/y: -1.0
/mavros/safety_area/p2/z: -1.0
/mavros/setpoint_accel/send_force: False
/mavros/setpoint_attitude/reverse_throttle: False
/mavros/setpoint_attitude/tf/child_frame_id: attitude
/mavros/setpoint_attitude/tf/frame_id: local_origin
/mavros/setpoint_attitude/tf/listen: False
/mavros/setpoint_attitude/tf/rate_limit: 10.0
/mavros/setpoint_position/tf/child_frame_id: setpoint
/mavros/setpoint_position/tf/frame_id: local_origin
/mavros/setpoint_position/tf/listen: False
/mavros/setpoint_position/tf/rate_limit: 50.0
/mavros/startup_px4_usb_quirk: False
/mavros/sys/disable_diag: False
/mavros/sys/min_voltage: 10.0
/mavros/target_component_id: 1
/mavros/target_system_id: 1
/mavros/tdr_radio/low_rssi: 40
/mavros/time/time_ref_source: fcu
/mavros/time/timesync_avg_alpha: 0.6
/mavros/vibration/frame_id: vibration
/mavros/vision_pose/tf/child_frame_id: vision
/mavros/vision_pose/tf/frame_id: local_origin
/mavros/vision_pose/tf/listen: False
/mavros/vision_pose/tf/rate_limit: 10.0
/mavros/vision_speed/listen_twist: False
/robot_description: <?xml version="1....
/rosdistro: indigo
/rosversion: 1.11.20
/tf_prefix:
/use_sim_time: True

NODES
/
gazebo (gazebo_ros/gzserver)
gazebo_gui (gazebo_ros/gzclient)
mavros (mavros/mavros_node)
spawn_erlecopter (gazebo_ros/spawn_model)
ROS_MASTER_URI=http://localhost:11311
core service [/rosout] found
Taking off in in 7s
process[mavros-1]: started with pid [641]
[ INFO] [1554355518.249245223]: FCU URL: udp://<IP_ADDRESS>:5760@
[ INFO] [1554355518.249463006]: udp0: Bind address: <IP_ADDRESS>:5760
[ INFO] [1554355518.249574731]: GCS URL: udp://<IP_ADDRESS>@<IP_ADDRESS>:14550
[ INFO] [1554355518.249638660]: udp1: Bind address: <IP_ADDRESS>:14555
[ INFO] [1554355518.249685024]: udp1: Remote address: <IP_ADDRESS>:14550
[ INFO] [1554355518.288473239]: Plugin 3dr_radio loaded and initialized
[ INFO] [1554355518.288516178]: Plugin actuator_control blacklisted
[ INFO] [1554355518.288528105]: Plugin altitude blacklisted
[ INFO] [1554355518.337454962]: Plugin cam_imu_sync loaded and initialized
[ INFO] [1554355518.343459639]: Plugin command loaded and initialized
[ INFO] [1554355518.343489420]: Plugin distance_sensor blacklisted
[ INFO] [1554355518.343500343]: Plugin ftp blacklisted
[ INFO] [1554355518.357003524]: Plugin global_position loaded and initialized
[ INFO] [1554355518.357099950]: Plugin image_pub blacklisted
[ INFO] [1554355518.366608114]: Plugin imu_pub loaded and initialized
[ INFO] [1554355518.373478765]: Plugin local_position loaded and initialized
[ INFO] [1554355518.374211261]: Plugin manual_control loaded and initialized
[ INFO] [1554355518.374242254]: Plugin mocap_pose_estimate blacklisted
[ INFO] [1554355518.377045458]: Plugin param loaded and initialized
[ INFO] [1554355518.377071709]: Plugin px4flow blacklisted
[ INFO] [1554355518.382834179]: Plugin rc_io loaded and initialized
[ INFO] [1554355518.382874973]: Plugin safety_area blacklisted
[ INFO] [1554355518.387554138]: Plugin setpoint_accel loaded and initialized
[ INFO] [1554355518.387582674]: Plugin setpoint_attitude blacklisted
[ INFO] [1554355518.395534883]: Plugin setpoint_position loaded and initialized
[ INFO] [1554355518.408307968]: Plugin setpoint_raw loaded and initialized
[ INFO] [1554355518.414679777]: Plugin setpoint_velocity loaded and initialized
[ INFO] [1554355518.422357215]: Plugin sys_status loaded and initialized
[ INFO] [1554355518.427603841]: Plugin sys_time loaded and initialized
[ INFO] [1554355518.429169485]: Plugin vfr_hud loaded and initialized
[ INFO] [1554355518.429239398]: Plugin vibration blacklisted
[ INFO] [1554355518.430103422]: Plugin vision_pose_estimate blacklisted
[ INFO] [1554355518.430129469]: Plugin vision_speed_estimate blacklisted
[ INFO] [1554355518.437731457]: Plugin waypoint loaded and initialized
[ INFO] [1554355518.437789760]: Built-in SIMD instructions: SSE, SSE2
[ INFO] [1554355518.437803385]: Built-in MAVLink package version: 2016.5.20
[ INFO] [1554355518.437814125]: Built-in MAVLink dialect: ardupilotmega
[ INFO] [1554355518.437824419]: MAVROS started. MY ID 1.240, TARGET ID 1.1
process[spawn_erlecopter-2]: started with pid [642]
Taking off in in 6s
spawn_model script started
process[gazebo-3]: started with pid [857]
[INFO] [WallTime:<PHONE_NUMBER>.897111] [0.000000] Loading model xml from ros parameter
[INFO] [WallTime:<PHONE_NUMBER>.902594] [0.000000] Waiting for service /gazebo/spawn_urdf_model
[ INFO] [1554355519.099669756]: Finished loading Gazebo ROS API Plugin.
[ INFO] [1554355519.112393714]: waitForService: Service [/gazebo/set_physics_properties] has not been advertised, waiting...
process[gazebo_gui-4]: started with pid [871]
[ INFO] [1554355519.439188153]: Model name:      erlecopter
[ INFO] [1554355519.439223990]: Nb motor servos: 4
[ INFO] [1554355519.462579971]: ARI: Services declared !
[ INFO] [1554355519.462645022]: ARI: Binding to listening port from ArduPilot...
[ INFO] [1554355519.462685978]: ARI: SUCCESS in binding to port from ArduPilot
[ INFO] [1554355519.462727328]: ARI: Connecting send port to ArduPilot...
[ INFO] [1554355519.462759020]: ARI: Opened ArduPilot fdm socket
[ INFO] [1554355519.462801535]: ARI: Initialization finished
[ INFO] [1554355519.462905371]: ARI: Starting listening loop for ArduPilot messages
[INFO] [WallTime:<PHONE_NUMBER>.510479] [0.000000] Calling service /gazebo/spawn_urdf_model
[gazebo_gui-4] process has died [pid 871, exit code 1, cmd /usr/local/gym/gym-gazebo/gym_gazebo/envs/installation/catkin_ws/src/gazebo_ros_pkgs/gazebo_ros/scripts/gzclient __name:=gazebo_gui _log:=/root/.ros/log/06019c26-569a-11e9-a579-0242ac110003/gazebo_gui-4.log].
log file: /root/.ros/log/06019c26-569a-11e9-a579-0242ac110003/gazebo_gui-4*.log
Taking off in in 5s
[ INFO] [1554355519.698108873]: Laser Plugin (robotNamespace = /), Info: Using the 'robotNamespace' param: '/'
[ INFO] [1554355519.698150434]: Starting Laser Plugin (ns = /)!
[ INFO] [1554355519.701875497]: Laser Plugin (ns = /)  <tf_prefix>, set to ""
[INFO] [WallTime:<PHONE_NUMBER>.706095] [0.000000] Spawn status: SpawnModel: Successfully spawned model
[spawn_erlecopter-2] process has finished cleanly
log file: /root/.ros/log/06019c26-569a-11e9-a579-0242ac110003/spawn_erlecopter-2*.log
Taking off in in 4s
Taking off in in 3s
Taking off in in 2s
Taking off in in 1s
Waiting for mavros...

Im afraid we dont offer support for the previous version of gym-gazebo. Anyway, we have already solved this issue in gym-gazebo2, so checking the docker usage instructions might help you. You will just need to replicate our work in gymgazebo1.


kairos-agent upgrade on custom images fails with missing /etc/os-release keys

We built a custom image for RHEL 8 with the steps outlined here
https://kairos.io/docs/reference/build-from-scratch/
The resulting image has an /etc/os-release as below.
NAME="Red Hat Enterprise Linux"
VERSION="8.7 (Ootpa)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="8.7"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Red Hat Enterprise Linux 8.7 (Ootpa)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:8::baseos"
HOME_URL="https://www.redhat.com/"
DOCUMENTATION_URL="https://access.redhat.com/documentation/red_hat_enterprise_linux/8/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 8"
REDHAT_BUGZILLA_PRODUCT_VERSION=8.7
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="8.7"

This image is able to be deployed successfully but when trying and upgrade via kairos-agent upgrade in Kairos 2.2.0 the upgrade fails with
Cant find IMAGE_REPO key under /etc/os-release

I think the problem is that the following step:
https://github.com/kairos-io/kairos/blob/9b3887a6b8e95d9eb0959f3ffbdc05723c4e19c5/Earthfile#L111
is run when building the base images. So when using BYOI it is missing, or at least I couldn't find it on AuroraBuild/osbuilder

I think either osbuilder offer a mechanism how to do this (and then it can be used on the main factory as well), or it should be part of the documentation for BYOI. Having said that, it will be the job of the user doing BYOI to define all or most of the values that go there, e.g. keep their own versioning because at the end of the day, they can decide on the cadence of release and it doesn't fit that they keep the same versioning of our releases. Alternatively we could do something like what we do with the k3s version, and when building the BYOI the final version would be something along the lines of KAIROS_VERSION+OWN_VERSION

@venkatnsrinivasan we added functionality on the osbuilder to update the attributes in the os-release. The docs have been updated to reflect this
# Set the Kairos arguments in os-release file to identify your Kairos image
FROM quay.io/kairos/osbuilder-tools:latest as osbuilder
RUN zypper install -y gettext
RUN mkdir /workspace
COPY --from=base /etc/os-release /workspace/os-release
# You should change the following values according to your own versioning and other details
RUN OS_NAME=kairos-core-fedora \
  OS_VERSION=v9.9.9 \
  OS_ID="kairos" \
  OS_NAME=kairos-core-fedora \
  BUG_REPORT_URL="https://github.com/YOUR_ORG/YOUR_PROJECT/issues" \
  HOME_URL="https://github.com/YOUR_ORG/YOUR_PROJECT" \
  OS_REPO="quay.io/YOUR_ORG/core-fedora" \
  OS_LABEL="latest" \
  GITHUB_REPO="YOUR_ORG/YOUR_PROJECT" \
  VARIANT="core" \
  FLAVOR="fedora" \
  /update-os-release.sh

FROM base
COPY --from=osbuilder /workspace/os-release /etc/os-release

from https://kairos.io/docs/reference/build-from-scratch/

@mauromorales I guess we can close this now?

It still needs a bump in the agent to check the two items Dimitris added but it comes together with the yip bump so I'm running some tests in a branch

@mudler ⬆


OCTOPUS-496: Update variables.tf

@Prajyot-Parab this updates the variables.tf to the latest default.
many thanks, Paul

@prb112 Please sign your commit. See Details link in the DCO row above.

Fixed... apologies, I made the change via the web editor.


Fix text clipping in Tree items not working with negative values

Before this, Trees with short widths could fail to clip texts:

Thanks!


Latitude and longitude should be picked from map on land registraion

For register land on frontend, users should be able to add longitude and latitude with a map picker.

Hello. Id love to take on this task. ETA 48hrs
I’m an experienced software engineer, with experience in both FE, BE and mobile development. First time contributor

Would love to contribute to this one...as I am working on something similar that has to do with map integrations for live tracking.

@MarcusDavidG ? Any updates. We need to free this for other contributors.

@fishonamos Sorry about the delay, raising a PR tonight.

Can I work on this. ETA 48hrs


Data getting truncated uploading dataset from daf to kylo - Nifi polling

Daf - Kylo -  Nifi is polling every 10 sec
I uploaded a file of 46 MB - 00:31 sec to be uploaded -> the dataset was truncated.
The Nifi time-slot has to be increased.

I set the "minimum file age" property to "5 sec" in the "GetFile" Nifi Processor which avoids the file truncation, during the ingestion flows.


Fix backend not running migrations in review env

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #9 (5e1ff5c) into main (c7941f9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   96.20%   96.20%
=======================================
  Files          24       24
  Lines         474      474
=======================================
  Hits          456      456
  Misses         18       18

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 c7941f9...5e1ff5c. Read the comment docs.


Bitrate support

There's currently no way of reading bitrate information for audio files.
Was that simply missed or is there a reason it's not implemented for audiotags (i.e. the underlying metadata crates don't expose that kind of information)?

The information would be provided for MP4 and FLAC files, but it would not be available in ID3 tags.
That's different from the duration, which can be both determined by the file's contents with mp4ameta and metaflac, and taken from the ID3 tag with id3.
This crate focuses on the tag of the file, not any other contents that could be used to gather additional information on the properties. Attempting to add it into here would lead to an empty implementation for ID3, which is not desirable.

Alright, thanks for the clarification.
Is there any other crate that exposes audio bitrate info that you're aware of?

Yep, I maintain one :smile:. https://github.com/Serial-ATA/lofty-rs

Thanks a lot!


OpenGL ES 1.1: viewDidLoad gets called twice??

I wrote a small example for iOS that simply initializes OpenGL ES 1.1 and clears the screen without using GLKit.  The example I have written initializes the ViewController once didFinishLaunchingWithOptions is called at the beginning of the application's initialization, and this is where OpenGL ES is initialized.  Afterwards, this gets called again in -(UIView*)view (see UIViewController.mm line 944), causing the initialization to be called twice.  Then when the rendering loop begins, the code crashes within the OpenGL ES functions.
Source: https://www.dropbox.com/s/of669lyp7jt4g9w/XCGLES.zip?dl=0
So far, I have not found a workaround for this.
v-brflem

Thanks for the feedback, we'll look into this.


Including initial conditions in equations

Is it possible to include initial conditions in the equations (in some sense, these are essentially parameters, right?). E.g something like:
si_ode = @ODEmodel(
    X1'(t) = p - k1*X1(t) + X1(0),
    X2'(t) = X1(t) -  X2(t),
    y1(t) = X2(t)
)

The reason I specifically am wondering about this is that, when reaction system have conservation laws (such as X1 + X2 = C), this can be used to eliminate a variable, e.g. X2. However, this introduces a new parameter C, which can be expressed X1(0) + X2(0) = C. Also, if X2 (which is eliminated) happens to be a measured quantity, this would need to be replaced with something like X1(0) + X2(0) - X1(t).

But you can just express C as C = X1(t) + X2(t) and express everywhere X2 with C - X1. Am I missing something?

Yes, ish. The problem is that then in the output you get identifiability for C, and not for X1(0) and X2(0). Although I guess I can do a transformation on the output as well to put in identifiability for X2(0) given what I know of C and `X1(0). However, StructuralIdentfiabiltiy does not given the output in a pure symbol => symbol form, but the keys in the disc is a bit weird, so I am not sure how to generate these.

I think that one can use the funcs_to_check argument which allows to provide custom functions to evaluate identifiability for. Say, you had a system
ode = @ODEmodel(
    x1'(t) = x1(t)^2 + x2(t)^2,
    x2'(t) = -x1(t)^2 - x2(t)^2,
    y(t) = x1(t) - x2(t)
)

Here you can set C = x1(t) + x2(t) and transform
ode2 = @ODEmodel(
    x1'(t) = x1(t)^2 + (c - x1(t))^2,
    y(t) = 2 * x1(t) - c
)

If you wanted to assess identifiability of x1, x2, you now call assess_identifiabilit(ode2, funcs_to_check = [x1, c - x1]).

That's a good point, would it be possible to name the output of these things (so that it looks like c - x1 is x2, here my c is a parameter that the user has never seen and would not necessarily know what it is about)? Unfortunatley, and this is a side thing, I was not able to get funcs_to_check to work for my ModelingToolkit type input.

Indeed, there should be a code for renaming. Does the function you have for reducing wrt first integrals return you the correspondence between old and new variables?
Sorry to hear about the issue with funds_to_check! Could you tell me what is the problem you have? The following code works fine on my computer
using StructuralIdentifiability, ModelingToolkit

@parameters r1, r2, c1, c2, beta1, beta2, chi1, chi2
@variables t, x1(t), x2(t), y(t), u(t)

D = Differential(t)
eqs = [
    D(x1) ~ r1 * x1 * (1 - c1 * x1) + beta1 * x1 * x2 / (chi1 + x2) + u,
    D(x2) ~ r2 * x2 * (1 - c2 * x2) + beta2 * x1 * x2 / (chi2 + x1),
]

measured_quantities = [y ~ x1]

ode_mtk = ODESystem(eqs, t, name = :mutualist)
assess_identifiability(ode_mtk, measured_quantities = measured_quantities, funcs_to_check = [c1 + c2])

I think it might also be related to going through Catalyst. I've forgotten the exact detailed but remembered that I looked at options and decided supporting it was not going to be with the coding mess it'd produce. I will have a look again and message you if I can narrow it down properly.

See above. I meant to implement it that was, however, there were some issues and I decided it would be a really big mess. That was last week, forgotten what it was. Might make a try to move back to MTK (which would be preferably, but also depends on MTK keeping up to date with SI).

I don't think you ever stated how funcs_to_check was failing going through Catalyst.

Sorry, not sure I fully understand.
The input to funcs_to_check has to be of the same type as the model. Given that we use SI-type odes, funcs_to_check must have SI-type expressions (which we cannot generate, hence we cannot support it).
If we used MTK-type systems internally, funcs_to_check could take MTK-style inputs, which we could generate, and we could support it. Ideally I'd want to support funcs_to_check, but at the time I decided it was no worth it/possible since we then would have to use MTK internally. If we can use MTK internally, I would want to do this.

You said:

If we used MTK-type systems internally, funcs_to_check could take MTK-style inputs, which we could generate, and we could support it. Ideally I'd want to support funcs_to_check, but at the time I decided it was no worth it/possible since we then would have to use MTK internally. If we can use MTK internally, I would want to do this.

and

The obvious answer is to not do the preprocess_ode conversion but instead use the MTK system. I think I had some good reason for this as well (which currently eludes me, but when tell you when I recall).

I'm still missing why we can't use the MTK internally.

"The obvious answer is to not do the preprocess_ode conversion but instead use the MTK system. I think I had some good reason for this as well (which currently eludes me, but when tell you when I recall)."
As in:

Using MTK internally seems like a good idea.
I looked at implementing it, but it had implications which made it not work out well, so I didn't.
I don't remember what these reasons where, but when I do, I will elaborate.

I am working stuff again trying MTK, hoping that I will remember the problems when/if I encounter them. This PR depends on https://github.com/SciML/Catalyst.jl/pull/713 in any case, and we won't have to worry about merging this one until that one gets merged.

No worries, I was just trying to understand what the limitation in using MTK is.
As for the other PR, we need to settle the issue I raised about where to do (optional) expansion. I still prefer doing it as a system transformation at the ReactionSystem level rather than trying to support doing it within every system conversion function.

Yeah, me and Gleb had discussions at like 3 different places, didn't realise first that it was pretty impossible to follow that from an outsider.
For the other point, my general feeling is that I fully agree. I think that PR is better than what we currently got, but if you'd rather go directly to something else, I am happy to skip it. Since that issue is not a big hurry, I think next time we meet we just quickly go through so that we are on the same page, I am certain I know what you mean, and I know how we want it implemented. Then I can make a new PR to implement that.


No captcha in data-export for the first time

When a newly registered user tries to export data, the captcha doesn't show up and he is able to export in the desired format without the captcha.
Steps to reproduce:

Register a new user.
Click on Request Data Export.
No captcha appears.

That's happening for all users and it's intentional. CAPTCHA is only appearing when the user demands another export <5min after the initial one.


fix(#100): update eo2js up to EO 0.40.0

Closes: #100

PR-Codex overview
This PR focuses on refactoring the eo2js project by renaming and reorganizing various elements, particularly transitioning from float to number types, updating related tests, and enhancing data handling.
Detailed summary

Updated version in eo-version.txt.
Deleted multiple obsolete files related to float.
Renamed float references to number in various files.
Enhanced data handling in dataized.js.
Improved error handling in try.js.
Added new functions for file operations in fs.
Implemented regex pattern matching in regex$pattern$match$matched-from-index.js.
Updated tests to reflect changes from float to number.

The following files were skipped due to too many changes: eo2js-runtime/src/objects/org/eolang/fs/dir$tmpfile$touch.js, eo2js-runtime/test/objects/org/eolang/cage.test.js, eo2js-runtime/test/runtime/package.test.js, eo2js-runtime/src/objects/org/eolang/txt/sscanf.js, eo2js-runtime/src/runtime/data.js, eo2js-runtime/test/runtime/dataized.test.js, eo2js-runtime/src/objects/org/eolang/txt/sprintf.js, eo2js-runtime/test/runtime/bytes-of.test.js, eo2js-runtime/src/runtime/bytes-of.js

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here.

@rultor merge

@maxonfjvipon Oops, I failed.  OomKillDisable discarded.
WARNING: The requested image's platform (linux/arm64/v8) does not match the detected host platform (linux/amd64/v3) and no specific platform was requested
exec /bin/bash: exec format error
container 0c0fbf6067a29ab5973ca05ed84848541844dfcb1ec836dcf2f21d1d97fab127 is dead
Tue Oct 29 22:13:26 UTC 2024

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here.

@rultor merge

@maxonfjvipon Oops, I failed. You can see the full log here (spent 23min).
  ✔ test "tuple_with" should work (39ms)
  ✔ test "throws_on_wrong_tuple_at" should work
  ✔ test "tuple_fluent_with" should work
  ✔ test "tuple_fluent_with_indented" should work (94ms)
  ✔ test "gets_lengths_of_empty_tuple" should work
  ✔ test "gets_lengths_of_empty_tuple_without_additional_obj" should work
  ✔ test "creates_empty_tuple_with_number" should work
  ✔ test "at_fast_with_first_element" should work
  ✔ test "at_fast_with_last_element" should work
  ✔ test "tuple_empty_fluent_with_indented_keyword" should work
  ✔ test "tuple_with_negative_index_gets_last" should work
  ✔ test "tuple_with_negative_index_gets_first" should work
  ✔ test "throws_on_out_of_tuple_bounds_with_negative_index" should work
  ✔ test "while_dataizes_only_first_cycle" should work (38ms)
  ✔ test "simple_while_with_false_first" should work
  ✔ test "simple_bool_expression_via_malloc_in_while" should work (69ms)
  ✔ test "last_while_dataization_object" should work (141ms)
  ✔ test "last_while_dataization_object_with_false_condition" should work
  ✔ test "while_simple_iteration" should work (535ms)
  ✔ test "iterating_tuple_with_while_using_internal_iterator" should work (710ms)
  ✔ test "iterating_tuple_with_while_using_external_iterator" should work (533ms)
  ✔ test "iterating_tuple_with_while_without_body_multiple" should work (564ms)
  ✔ test "iterating_tuple_with_while_without_body_single" should work (94ms)

  520 passing (27s)

  \u001b[32m  ✓\u001b[0m\u001b[90m should execute all eo-runtime tests\u001b[0m\u001b[31m (608886ms)\u001b[0m

\u001b[0m  integration test\u001b[0m
Recompiling stylesheets...
Recompiled 0 stylesheets
npm WARN deprecated<EMAIL_ADDRESS>This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm WARN deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
  \u001b[31m  1) should execute simple unit test\u001b[0m
  \u001b[31m  2) should dataize simple program\u001b[0m

\u001b[92m \u001b[0m\u001b[32m 15 passing\u001b[0m\u001b[90m (16m)\u001b[0m
\u001b[31m  2 failing\u001b[0m

\u001b[0m  1) integration test
       should execute simple unit test:
\u001b[0m\u001b[31m     Error: Timeout of 100000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/r/repo/eo2js/test/it/simple.test.js)\u001b[0m\u001b[90m
      at Context.<anonymous> (test/it/simple.test.js:31:5)
      at process.processImmediate (node:internal/timers:476:21)
\u001b[0m
\u001b[0m  2) integration test
       should dataize simple program:
\u001b[0m\u001b[31m     Error: Timeout of 100000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/r/repo/eo2js/test/it/simple.test.js)\u001b[0m\u001b[90m
      at Context.<anonymous> (test/it/simple.test.js:36:5)
      at process.processImmediate (node:internal/timers:476:21)
\u001b[0m

\u001b[33mWarning: \u0007 Use --force to continue.\u001b[39m

\u001b[31mAborted due to warnings.\u001b[39m
container c5662cdcdfd92ed5ff660be21aaf2914dd7ce4ed51db90672fa640d1ff12cf9d is dead
Wed Oct 30 10:32:54 UTC 2024

@rultor merge

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here.

@rultor merge

@maxonfjvipon Done! FYI, the full log is here (took me 23min).


Closing worker with reason: worker recycled

hi there
I have compose the webssh for docker with command docker-compose up.
All works if in my laptop browser I connect directly to the host where docker runs. (https://myapp.com)
now I connect to a server to authenticate user (https://myauthserver.com) vs the LDAP, and if user is allowed it redirect to FQDN of my docker (https://myapp.com). but once I try to trigger the ssh connectivity, I got an error, session is closed with Closing worker with reason: worker recycled.
in some blog I found to add the --origin parameter, putting "*"  or the DNS name, or to increase the delay. but nothing.
here my last version of Docker file:
FROM python:3.9
EXPOSE 8888
RUN pip install webssh
CMD wssh  --ssladdress='<IP_ADDRESS>' --sslport=443 --certfile='/PATHTOCERT/certificate.crt' --keyfile=''/PATHTOCERT/certificate.key' --origin='https://myauthserver.com' --delay=6 --redirect=true
I put the certificate on /PATHTOCERT/
then I started the docker as below:
docker run --name webssh-test -d -v /PATHTOCERT/:/PATHTOCERT/  -p443:443 webssh-master-web
so if I run browser to go directly to the host where docker run, all works:
here the log
[I 231103 15:21:20 web:2344] 200 GET / (MYIP) 1.02ms
[I 231103 15:21:25 handler:452] Connecting to XXXXXXXXXX:22
[I 231103 15:21:25 transport:1893] Connected (version 2.0, client OpenSSH_8.0)
[I 231103 15:21:25 handler:86] Trying password authentication
[I 231103 15:21:26 transport:1893] Auth banner: b'Authorized uses only. All activity may be monitored and reported.\n'
[I 231103 15:21:26 transport:1893] Authentication (password) successful!
[W 231103 15:21:28 handler:446] Could not detect the default encoding.
[I 231103 15:21:28 web:2344] 200 POST / (MYIP) 2879.68ms
[I 231103 15:21:28 web:2344] 101 GET /ws?id=YYYYYYYYYYYYYYYYYYYYYYYYYY (MYIP) 0.80ms
[I 231103 15:21:28 handler:537] Connected from XXXXXXXXXX:54996
but If use an intermediate server to authenticate user versus LDAP, then it redirect to docker web:
[I 231103 15:22:29 web:2344] 200 GET / (AuthenticationServerIP) 1.01ms
[I 231103 15:22:37 handler:452] Connecting to XXXXXXXXXX:22
[I 231103 15:22:37 transport:1893] Connected (version 2.0, client OpenSSH_8.0)
[I 231103 15:22:37 handler:86] Trying password authentication
[I 231103 15:22:37 transport:1893] Auth banner: b'Authorized uses only. All activity may be monitored and reported.\n'
[I 231103 15:22:37 transport:1893] Authentication (password) successful!
[I 231103 15:22:37 web:2344] 200 POST / (AuthenticationServerIP) 755.35ms
[W 231103 15:22:41 web:2344] 400 GET /ws?id=YYYYYYYYYYYYYYYYYYYYYYYYYY  (AuthenticationServerIP) 0.72ms
[W 231103 15:22:43 worker:33] Recycling worker YYYYYYYYYYYYYYYYYYYYYYYYYY
[I 231103 15:22:43 worker:123] Closing worker YYYYYYYYYYYYYYYYYYYYYYYYYY   with reason: worker recycled
[I 231103 15:22:43 worker:131] Connection to XXXXXXXXXX:22 lost
what I am missing? which parameter shall I add to wssh ?

Hello, @jasthema, I have the same problem, have you fix it ?

@Disk-MTH , not solved yet :-(

I met as well

Have you solved this problem?

Not solved, I discontinued the project. It seems a problem of proxy redirect and I guess the docker image is not compatible with it. I had similar problem with other containers behind proxy redirect authentication. I had to create a firewall rule to limit access.

Not solved, I discontinued the project. It seems a problem of proxy redirect and I guess the docker image is not compatible with it. I had similar problem with other containers behind proxy redirect authentication. I had to create a firewall rule to limit access.
What other projects have you changed to subsequently?

Not solved, I discontinued the project. It seems a problem of proxy redirect and I guess the docker image is not compatible with it. I had similar problem with other containers behind proxy redirect authentication. I had to create a firewall rule to limit access.

What other projects have you changed to subsequently?


Tests for TreeSet and TreeMap were not modified at all, so semantics have not changed. Test for RedBlackTree slightly modified because function signatures have changed to allow the wrapper type.
RedBlackTree is now more amenable to MultiMaps and MultiSets if we need them later on

@dhillondeep  nice catch lol I didn't realise I did that


Bug: Timing Controls Currently Do Nothing

We'll need a way to set a time in the Arena and enforce that time across web socket requests. The best path is most likely a separate thread on Arena that has control over the WebSocket connected to each client, and can 1) monitor and 2) forcibly shutdown a client.

We'll also need a timer that is accessible via the global state! Clients will need to be equipped with a way to check how much time they have remaining.

Both issues were fixed as of commit https://github.com/pashneal/splendor-tournament/commit/3ed9189a768694748e87065e783c96519c7d0cc5


[20.09] mozjpeg: 3.3.1 -> 4.0.3

Motivation for this change
https://nvd.nist.gov/vuln/detail/CVE-2018-14498
Backport of #119117. Attempted patching but was foiled by yet another person running an auto-formatter :roll_eyes:
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119267 at 8e93a29d run on aarch64-linux 1

3 packages built successfully:

avocode
jpeg-archive
mozjpeg


v1.0.0

features

Output the difference of struct to map
Write README

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


I need help

Describe your problem here.
I wanted to move some things from my engine to this psych engine for android
And even if I manage to compile, it gives me error once I am in android.
Specifically this:

And if I add the file by editing the apk, when I compile and open the app, it does nothing.
I feel that it is a problem of the dependencies that I have to compile.
I have the ones from Psych Engine 0.7.1h, but maybe not the correct ones for android.
Which ones should I have? Since the original Psych Engine provided in Releases does open for me, so it can't be so much my phone's fault.
Are you modding a build from source or with Lua?
Source
What is your build target?
Neko, HashLink, or other build system
Did you edit anything in this build? If so, mention or summarize your changes.
Several things, but the most important would be the Modcharting Tools (from TheZoroForce240). Since that is what matters to me  Otherwise some text or image added.

look like you change lib.so,it cant change code

look like you change lib.so,it cant change code

When I first compiled, on the Luajit theme I had to move a folder out of the normal path where it would be so that the comp8laci would be completed.
But... What can I do so that if it loads? What dependencies do I need?

apk cannot be modified by adding lib.so,you must change source code at github

Yes.. I managed to compile the game through github, but I want to know how to do it locally because it takes a long time

it cant change code at locally,my version only need 6 to 10 mins pack Engine is faster than before

if you want change source code faster,extend hscript load to other state

I don't mean to change, I mean to compile the game, I see that I'm not making my point.
I want to compile the game from my PC, without relying on Github

Configure the source code environment and package it on your own computer (it's a hassle, but it only takes 30 seconds to package)

I don't have a tutorial on this because I don't even have a computer lol


Remove vcs() method that doesn't seem to exist for VcsMappings

Signed-off-by: Mike Kobit<EMAIL_ADDRESS>Context
Trying to try out source dependencies adapting from https://docs.gradle.org/current/javadoc/org/gradle/vcs/VcsMappings.html in the kotlin-dsl but the Javadoc seems to point to a method vcs that was removed or is unclear
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Thanks @mkobit.


Bug: Keep serialisation check enabled by default in batches.

Serialization is currently disabled by default. Please have it enabled.

Description
When a new batch is created - serialisation check is disabled by default.

Acceptance Criteria
Please keep the serialisation check enabled by default.
Environment
Google Chrome (latest) on
MacOS - Catalina
Prerequisites
Reproduction steps

Click on "Add Product" and Enter the product details
Click on "Add ePI" and upload the xml file downloaded from the dailymed website
Click on "save product"
Move to the Batches section and click on "Add Batch"


[RFE] Point out churning start date in indexed documents

Is your feature request related to a problem? Please describe.
To improve benchmark's result analysis, we need to be able to rule out the collected metrics during churning stage.
Describe the solution you'd like
Several changes will be needed:

As part of the jobSummary metric we currently generate

https://github.com/kube-burner/kube-burner/blob/69cbf3c694879f140d1a6acbf38f130a73b6f0fd/pkg/burner/metadata.go#L33-L40
Add a new field to this document to indicate the churning stage start date.

In the Prometheus/measurement/alerts based indexed results, add an extra (churning: false/true?) field to indicate they were collected during the churning stage or not

Once we've these fields we should be able to rule out metrics of churning stage of a benchmark.
Additional context
This feature popped up in today's meeting as analyzing metrics for benchmarks with churning can be challenging since their results can be misleading, ie the duration of namespace removal has lot of variation.
cc: @kube-burner/ocp-perfscale @kube-burner/ocp-perfscale-telco

Just to check if my understanding is correct from an implementation perspective, so we want to keep track of start and end times of all the churn cycles and mark all the Prometheus data-points that fall in between those intervals as churn metrics. Does that sound good?
Also along with the start time of churn, do we also want to publish the end time of churn in the jobSummary? Because benchmark end time is not same as churn end time as we have some sleep delay here.

Just to check if my understanding is correct from an implementation perspective, so we want to keep track of start and end times of all the churn cycles and mark all the Prometheus data-points that fall in between those intervals as churn metrics. Does that sound good? Also along with the start time of churn, do we also want to publish the end time of churn in the jobSummary? Because benchmark end time is not same as churn end time as we have some sleep delay here.

correct, adding an extra field churning: true/false to the indexed documents falling between churning/not-churning intervals should be the correct way.
wrt the second question, churningDelay is something that happens during the churning stage, so I think we should include it.
Churning start/end dates should be taken before/after calling this function:
https://github.com/kube-burner/kube-burner/blob/69cbf3c694879f140d1a6acbf38f130a73b6f0fd/pkg/burner/job.go#L155-L157


Migrated ImageGen to async programming. Converted ImageGen to context m…

…anager. Added quiet flag to be used as module without cluttering"

Okay, I have reverted ImagenGen changes and added async to a new class called ImageGenAsync, is that good?

Sorry for the slow response. My email got very cluttered

I'm currently working with the update, sorry.


changes to make pp3 user friendly, robust and less segfault

These changs include:

add pic16(l)f1823 to device database and verified.
add a help msg
add some check to ensure user set up args correctly (at least, '-c' and '-t' are essential args ) and less segfault
set a proper init value of sleep time to avoid some 'rx fail' issue.
add pp3 makefile for linux and support install it globally.

I try to keep same code-style, but suggest a re-construction  of pp3 codes :-)

Hello,
thank you for your major contribution. Please give me some time to digest it.
Regarding the code style: it started as quick hack, where spaghetti is perhaps acceptable - and then grown "organically". To be honest, I'd love to see it reworked into something more sane, but at this time I can't see myself doing this.


Centos 7 Setup Issues

Followed the instructions, however when i click on the go to next step on the install page, the request seems to occur however, nothing changes within the window.

Also:
The stylesheet http://localhost/midas/index.php/core/public/css/install/install.css was not loaded because its MIME type, "text/html", is not "text/css".
Use of getPreventDefault() is deprecated.  Use defaultPrevented instead. <-- Jquery error

I think the second issue is an issue with you server. Midas does not adjust the MIME type. Ignore the deprecation warning.

Ok thanks. I still can't get the /midas/index.php/install page to advance with the current version from GIT. Seems to be something wrong with the POST form, is JQuery ajax used for this?


Update plugin jni version to released 22.08.0 [databricks]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>as title. I will trigger the build after we release spark-rapids-jni 22.08.0

build

build


Export data: No file in directory found

What did I do
I tried to export the data from a recent session to the file system in json format.
What happened
Export dialog shows up and after presseing confirm the file seems to be written to my home directory.
Expected
I can see the file in my home directory, but it is not there! My home directory is empty.
Environment

OS: LinuxMint 21.1
MQTTX version: v1.9.6

More detail

I started the mqttx under my user, so I would not expect any write issues. It is interesting that previous written files are shown up in the dialog, but not on my ssd.

Hi, @Jambola77 . I have tested the export data feature of mqttx on both Ubuntu(Ubuntu 22.04.3 Gnome) and Mint(Linux Mint 21.2 Cinnamon), and they both work fine, generating files with correct content. Attached are the screenshots.

I need more information from you to reproduce this issue. Firstly, did you install mqttx via deb/snap/flatpak/appimage? Secondly, are you using any other desktop environment, such as Mate? Finally, is it because of root permissions that you can't see these files as a normal user? Try to see if you can export the data to another folder?
Thank you!

Hi Ni00, I think I use flathub installation. Syslog says: Started app-flatpak-com.emqx.MQTTX-4645.scope.

The file should be there, but it is not.

I also tried to save under /tmp with the same results.

I have now searched my whole disk and it was found here:
/run/user/1000/.flatpak/com.emqx.MQTTX/tmp/Broker.json

This seems to be caused by Flatpak. I will test it later because I installed it using the .deb method before, so I may not have noticed this issue.

This issue is related to file system permissions after packaging with Flatpak. The files are generated in a unified directory for Flatpak, rather than in the user directory. I will wait for the 2.0 version update and test if this problem still exists. If there is an issue, it may be necessary to configure packaging parameters. Please refer to: https://docs.flatpak.org/en/latest/sandbox-permissions.html#filesystem-access


Weird message with Import-Module

Environment
Operating System: Win 10
VenafiTppPS version: 2.0.1
PowerShell version: 5.1

Steps to reproduce
Load a new PS shell
Attempt to run script which:
1* Checks to make sure VenafiTppPS in installed and if not installs it from PS Gallery
2* Does an Import-Module VenafiTppPS
3* Does a Get-Module VenafiTppPS
4* Makes a connection to TPP and establishes a New-TppSession
Expected behavior
Module loads with no errors
Actual behavior
New-Object : Cannot find type [TppSession]: verify that the assembly containing this type is loaded.
At line:32 char:16

    return & $origNewObject @psBoundParameters

           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidType: (:) [New-Object], PSArgumentException
FullyQualifiedErrorId : TypeNotFound,Microsoft.PowerShell.Commands.NewObjectCommand

Screenshots

As per @wilddev65, this is isolated to 1 machine and can be closed.


docs: fix grammar in changelog

Description
Three changes to this sentence:

"since" should refer to an effect of the action
the double loading happens because "a and b" load it. "but" indicates opposition, but the two both need to happen for there to be double loading, so there's no opposition
use an em dash to separate example

Additional context
The rest of the changelog reads well to me

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[X] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the Commit Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Thanks a lot Ben! 🙏🏼


ci: Use GITHUB_OUTPUT envvar instead of set-output command

save-state and set-output commands used in GitHub Actions are deprecated and GitHub recommends using environment files.
This PR updates the usage of ::set-output to "$GITHUB_OUTPUT"
Instructions for envvar usage from GitHub docs:
https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-output-parameter

Thanks, this has now been fixed.


Extend 'Log' keyword functionality

For now Log keywords adds entries into Test Execution Errors only when level argument is set to ERROR or WARN. I think that:

Test Execution Errors section could be renamed to Test Execution Messages and

Additional argument, for example summary could be added to add possibility to log other severities into Test Execution Messages, e.g:

Log     some message     summary=True

Could you clarify the use case for this functionality?

Just enhancing Log wouldn't be enough. We'd first of all need an API that allows this and should expose it also to libraries.

Are you interested to implement this feature or sponsor the development if someone else would do it?

I have similar need. I was looking for NOTE level logging to log Python package version on top of the result.html.

Logged messages aren't shown in report at all at the moment. Been thinking that showing messages in the execution errors section (that this issue proposes to be changed to test execution messages) in report somehow would be a good idea. That would require its own issue, though.
At the moment the best way to get messages shown in report is probably using test suite metadata. See the User Guide for details.

My questions, most importantly the use case, was never answered. Closing.
For getting info shown at the top of reports, I recommend suite metadata.


Watchlist Test Cases

Test Cases

Test
Description

Verify Deployment
Verify from the main deploy ARM template /Deploy/azuredeploy.json

Test Module with no API Permissions Granted
Make sure the module returns

Verify GrantPermissions.ps1
* Ensure that all necessary permissions are granted by this powershell script for the module to work correctly

Verify Return Schema
* Does the return schema work in a Parse JSON step? * Are descriptions of the properties provided

Return Schema Consistency
Is the return schema consistent in style with other modules?

Return Property Clarity
Are the returned properties clear to understand the data they represent

Core Functionality
Is the core functionality working when appropriate entity data is provided?

Bad Entities
* If entities are not provided, how does the module respond? * If the entities provided cannot be found (ie. bad UPN)Expectation is the module would still return successfully and the issues would be represented in the returned data

Verify accuracy of readme.md for module
Ensure that the readme is still accurate (input/output/descrition/etc)

Remove deployment link from individual module readme.md
We don't want people deploying from the module, but rather use the /Deploy/ wrapper

#147 falls into that testing I guess

#167 as well


checkstyle: ensure overloaded methods are grouped together #229

Fixes #229
Ready for review.

@chao1995 you can request a peer review from one of the team member you think is suitable to review this.

Also, you can add the label toReview :-)

I tested it and it seems to be working most of the time.
However, I was able to break this rule if there are other class/interface declarations in the middle of the two methods. Not sure whether we can fix this or not (and whether we should be concerned about it):
@@ -101,14 +101,18 @@ public class ModelManager extends ComponentManager implements Model {
    private void updateFilteredPersonList(Expression expression) {
        filteredPersons.setPredicate(expression::satisfies);
    }

    //========== Inner classes/interfaces used for filtering =================================================

    interface Expression {
        boolean satisfies(ReadOnlyPerson person);
        String toString();
    }

+   public void updateFilteredPersonList(String keyword, boolean someRandomBool) {
+       // do nothing
+   }
+
    private class PredicateExpression implements Expression {

        private final Qualifier qualifier;

However, I was able to break this rule if there are other class/interface declarations in the middle of the two methods. Not sure whether we can fix this or not:

I think we don't need to be worried about it. I sounds like a loophole in checkstyle which could be fixed in a future version.

@m133225 for your inputs

@yamgent Nice spot! Thanks!
As @damithc said, this is a potential loophole in checkstyle, though we can add "InnerTypeLast" rule to help this specific case, so the violating overloaded method should not appear after the inner interface.
However, "InnerTypeLast" is not specified in Google Java code style, so I think for this PR's concern, we just don't add the "InnerTypeLast" first (maybe we can add in a separate PR if necessary), but add more info in the commit message documenting this potential loophole in checkstyle.

@m133225 @yamgent Thanks for the review! Updated commit message.

@chao1995 can rebase branch while waiting for @m133225 's approval?

@damithc Rebased.

Some comments on the commit message:

Check overloaded methods are grouped together in checkstyle

The rationale for this change is missing. Why do we need this extra checkstyle rule?

Note that there is a potential loophole in checkstyle
OverloadMethodsDeclarationOrder module which permits out-of-order
overloaded methods exist when they are separated by a nested class

This reads a bit strangely. Maybe remove exist?

or interface.
e.g. the overloaded updateFilteredPersonList in ModelManager.java
will pass.

I think we can rephrase this more clearly as:
e.g. the following code in ModelManager.java still passes the check, even
though there is a nested interface between the two `updateFilteredPersonList()`
method definitions:

private void updateFilteredPersonList(Expression expression) {
filteredPersons.setPredicate(expression::satisfies);
}
//========== Inner classes/interfaces ==================
interface Expression {
boolean satisfies(ReadOnlyPerson person);
String toString();
}
public void updateFilteredPersonList(String keyword, boolean someRandomBool) {
// do nothing
}

Indent the code by 4 spaces to make it clear that it is a code block.

@chao1995 Agree with the changes that @pyokagan has suggested. Do remember to update it!

v1
@chao1995 submitted v1 for review.

[v1 1/1] Check overloaded methods are grouped together in checkstyle

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/251/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

@m133225 Updated!

@chao1995 do a rebase  while we wait for @pyokagan to have another look?

Looks good overall.
Just a small nit that I think @chao1995 missed out from https://github.com/se-edu/addressbook-level4/pull/251#issuecomment-273757966 :

Indent the code by 4 spaces to make it clear that it is a code block.

I've fixed that up on my end.
Also, the commit message wasn't wrapped at 72 characters. I fixed that up on my end too.


As a technical contact, I want guidance so I can choose whether I need a submission period for a Pay for Pull opportunity.

During the UX research it was established that the technical contacts within the client programs did not understand if they needed to create a fixed submission period during which bids or work would be collected; in accordance with current procurement practices and policy.
Created by the UX team
DOD

Content that informs the user of correct procedure.
Design changes (wireframes)

Connect with Loren/Rommel

I suggest that the program owner will also want this guidance.


Travis CI: Linker error after updating cpt

Description of Problem, Request, or Question
Recent changes to my project (YoutubeListDownloader) made Travis-CI to fail: undefined reference for sqlite3. But only for the test package. Since I didn't change anything that could make the linker fail I searched for what has changed since the last CI job and I could only find cpt and conan itself. So I tried the exact same revision that was working in the past (cpt version 0.25.1) again it it failed with cpt version 0.29.1. When I run the build locally with recent conan version 1.18.2 it works fine so I guess the problem comes from cpt.
Logs of both builds (old and new) are attached.
Environment Details

Conan Package Tools Version: 0.29.1
Compiler+version: gcc-7.2 and gcc-8.0.1
Docker image: lasote/conangcc7 and  lasote/conangcc8
Conan version: conan 1.18.2
Python version: python 3.6.7

Steps to reproduce (Include if Applicable)
Build logs (Include if Available)
old.txt
new.txt

Hi @Linux13524 !
Lasote's docker images are deprecated. Please, update them to conanio/gcc7 and conanio/gcc8

Hi @uilianries
Thanks for the tip! I tried with conanio images, too. Same linker error...

@Linux13524  Okay, i'll take a look, thanks for reporting!

Hi @Linux13524 !
Your sqlite3 was built with an incompatible libc version.
From your log:
    sqlite3/3.27.2@bincrafters/stable from 'upload_repo' - Downloaded
It's been downloaded from your repo, not from Bincrafters repository. You need to rebuild with a compatible version, you can use Conan Docker images, OR you can we Bincrafters version.
Regards!

Hi @uilianries
Thanks! I deleted the incompatible packages and now it works again.
I cannot download from Bincrafters since I need packages for android and they are not provided by Bincrafters.
Can you tell me what the correct way is to keep the packages up-to-date with conan?
Why is the libc version not stored in the package if it can break builds when changed?

I cannot download from Bincrafters since I need packages for android and they are not provided by Bincrafters.

Can you tell me what the correct way is to keep the packages up-to-date with conan?

You can use the update argument:
conan install -u

https://docs.conan.io/en/latest/reference/commands/consumer/install.html

Why is the libc version not stored in the package if it can break builds when changed?

This topic is but in summary, it's complex to be done. You can follow some related discussions and put your ideas there, it will be appreciated:
https://github.com/conan-io/conan/issues/3972
https://github.com/conan-io/conan/issues/796
Anyway, you can customize your package settings, adding glibc there if you want. Here you can find more info how to do it:
https://docs.conan.io/en/latest/extending/custom_settings.html

@uilianries Thanks a lot for explanation!


Update to webdriver 5

Webdriver 5 breaking changes here: https://github.com/webdriverio/webdriverio/blob/master/CHANGELOG.md#boom-breaking-change

@codebytere can you help with that?

There seems to be a work in progress https://github.com/electron/spectron/issues/349.  So can we have a single issue tracking point?

CC @eostrom

Indeed, definitely sorry I missed it.


add mbedtls for windows (since they build it now)

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

This PR also manually pins to a version of mbedtls, which may be for the best since different versions are not binary compatible. (r-nanonext was failing because it was taking mbedtls=3.5 while nng was built against 3.3.)

Pending review, this should be ready to go.

It looks ok also for me, I'm merging, thanks @brendanf


fix: it was impossible to remove array or object attributes when setting the JS prop to null.

[x] Bug
[ ] Feature

Requirements

[x] Read the contribution guidelines.
[ ] Wrote tests.
[ ] Updated docs and upgrade instructions, if necessary.

Rationale
It is impossible for prop.serialize to return null if it simply returns JSON.stringify(val), because a string can not be coerced to null, therefore doing something like this.someProp = null does not remove the target attribute.
Implementation
Checks if val is null, and if it is just returns null.

Is this still an issue in skatejs/element?

Seems like it is, here:
https://github.com/skatejs/skatejs/blob/359057e333e3cb6697156d736202ed69786bf21c/packages/element/src/props.ts#L18


[AutoRelease] t2-containerservice-2022-10-25-58187(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3292
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=982

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


Make the launching easier

I have some suggestions

Add the Godot icon for the Godot menu and its Launch button
Make the launch button save the current scene before launch?

Replace the debug button(the one that have C# text thingy) with launch button with the Godot Engine icon:
https://docs.microsoft.com/en-us/visualstudio/extensibility/adding-icons-to-menu-commands?view=vs-2022

The scene need to auto save when you click it, it is very necessary


Ft user login 161049231

What does this PR do?
It creates a login page for the admin and the store attendant.
Description of the tasks completed

Created a feature branch and a UI directory
Added HTML for a login page
Added CSS for the login page
Added two images

How can you test this?
Clone this repository to your local machine and run the login.html file to view the user interface
What are the relevant pivotal tracker stories
#161049231

Looks good

beautiful...

One more thing
thats not way to name a PR
refer here https://github.com/andela/bestpractices/wiki/Git-naming-conventions-and-best-practices

nice

Simple and stylish design
You may also consider adding a recover password link

Simple and stylish design
You may also consider adding a recover password link

Thanks.

Good work, I like the colors an positioning, maybe you can make the form occupy more space, because is the main thing on the page.

This is very good.


Retrieve the tokens spent on the request.

I couldn't retrieve the tokens spent on the request. How can I obtain this information in the SDK?

there are four oauth auth examples in examples folder，are you finding it？

Yes, of course. But I don't understand how this relates to my question? I need to obtain the number of spent coze tokens in the conversation using the method coze.chat.create_and_poll(), for example. Previously, I received this in the "conversation.chat.completed" event with data['usage']['token_count']. I couldn't find similar data in the SDK.

fixed by https://github.com/coze-dev/coze-py/pull/82

it's amaising


Working with pixel size larger than 30m

Hi, I wonder if there is any option to work with pixels of much larger dimension?
I want to use the output of your simulation as an input for an optimization model. I need data of an area on about 61km by size (square area) that is going to be composed of 600 grid cells. I wish to have each cell (2.4km x 2.4km) to be directly obtained form your simulation model.
Is there anyway to have aggregated data for larger areas to be simulated without a requirement of such granularity?

Apologies for the late reply, but this would be a feature that we would require you to make into a PR and we can review and merge in.
The size of the data is currently directly dependent on the LANDFIRE API, and that only goes up to 90m resolution at its coarsest. If you wanted to make cells of much larger size, this would have to be interpolated after getting the data from LANDFIRE.
This is doable, but we don't currently have the development capacity to put towards this capability.
If you wanted to include this functionality, I would recommend putting a scaling factor in the config that is then propagated to an internal method (say named _upscale_area) of the LandFireLatLongBox class and put into the _make_data method.
Any further specifics could be discussed in a PR.

Update, it looks like landfire-python actual does support resampling, so this might be easier than anticipated. Still not on our radar for now, but definitely makes this a lighter load:
https://landfire-python.firesci.io/en/latest/usage.html#resampling-data


Entry point is not parsed correctly when entryPointStrategy is packages

Search terms
entryPointStrategy, packages
Expected Behavior
The package.json should be parsed correctly
Actual Behavior
An error appeared:
error Unable to find any entry points. Make sure TypeDoc can find your tsconfig

Steps to reproduce the bug

My document is placed in packages/documents;
Other packages are placed in packages/a, packages/b, packages/c;
Entry point is setted to:

{
  entryPoints: ['../a', '../b', '../c']
}

The package parsing call stack is:
a. getEntryPointsForPackages , With the following code:

    const expandedPackages = expandPackages(
        logger,
        ".",
        packageGlobPaths,
        exclude
    );

The second parameter packageJsonDir is ., this may assume that the package we want to generate documentation for is a subdirectory of the current directory, but in some cases this assumption does not hold, the following call stack proofed this:
b. expandPackages: It is to trying to parse a glob pattern:
        const globbedPackageJsonPaths = glob(
            resolve(packageJsonDir, workspace, "package.json"),
            resolve(packageJsonDir)
        );

The first parameter of this glob is:
ROOT_PATH_OF_THE_PROJECT\packages\a\package.json

The second parameter of this glob is:
ROOT_PATH_OF_THE_PROJECT\packages\documents

The home-made glob function simply won't return anything, and caused the parsing work failed.
Environment

Typedoc version: typedoc@npm:0.23.11
TypeScript version: typescript@npm:4.6.4
Node.js version: v16.15.0
OS: Windows 11

The second parameter packageJsonDir is ., this may assume that the package we want to generate documentation for is a subdirectory of the current directory, but in some cases this assumption does not hold

Knew this would come bite me at some point... yep, that was an assumption that was made.

Hi there,
I just came across this issue and think I'm missing something about how its being used.
In my monorepo I have my docs generation in a package as well as the source I'm documenting, and feel like I have a setup similar to above
packages/docs/docs-build
packages/framework/common
packages/support/lib1
packages/support/lib2

{
  entryPointStrategy: 'packages',
  entryPoints: [
    '../../framework/common-internal',
    '../../libraries/support-react',
    '../../libraries/support-angular',
  ]
}

I'm getting the following error when trying to build  ( using Docusaurus, but the issue seems to be in Typedoc, see below
[ERROR] Error: ENOENT: no such file or directory, scandir 'C:/Projects/Project1/packages/docs/docs-build/Projects/UHSA/Cockpit/packages'
    at Object.readdirSync (node:fs:1380:3)
    at glob (C:\Projects\Project1\node_modules\.pnpm\typedoc@0.23.12_typescript@4.7.4\node_modules\typedoc\dist\lib\utils\fs.js:140:32)
    at C:\Projects\Project1\node_modules\.pnpm\typedoc@0.23.12_typescript@4.7.4\node_modules\typedoc\dist\lib\utils\package-manifest.js:89:55
    at Array.flatMap (<anonymous>)
    at expandPackages (C:\Projects\Project1\node_modules\.pnpm\typedoc@0.23.12_typescript@4.7.4\node_modules\typedoc\dist\lib\utils\package-manifest.js:88:23)
    at getEntryPointsForPackages (C:\Projects\Project1\node_modules\.pnpm\typedoc@0.23.12_typescript@4.7.4\node_modules\typedoc\dist\lib\utils\entry-point.js:216:68)
    at getEntryPoints (C:\Projects\Project1\node_modules\.pnpm\typedoc@0.23.12_typescript@4.7.4\node_modules\typedoc\dist\lib\utils\entry-point.js:45:22)
    at Application.getEntryPoints (C:\Projects\Project1\node_modules\.pnpm\typedoc@0.23.12_typescript@4.7.4\node_modules\typedoc\dist\lib\application.js:118:49)
    at Application.convert (C:\Projects\Project1\node_modules\.pnpm\typedoc@0.23.12_typescript@4.7.4\node_modules\typedoc\dist\lib\application.js:134:34)
    at Object.loadContent (C:\Projects\Project1\node_modules\.pnpm\docusaurus-plugin-typedoc@0.17.5_hllrrslxckewhjmudyghxy3nxi\node_modules\docusaurus-plugin-typedoc\dist\plugin.js:49:37)

It seems like whats happening is the information passed to expandPackages in getEntryPointsForPackages seems wrong?
In the example above, deriveRootDir() returns Projects/Project1/packages (no drive letter) and passes in my packageGlobPattern which has been fully resolved by the Typedoc options parsing, which results in the glob function being called with an incorrect root
glob(
"'C:\\Projects\\Project1\\packages\\framework\\common-internal\\package.json",
"'C:\\Projects\\Project1\\packages\\docs\\docs-build\\Projects\\Project1\\packages'
", undefined)

This causes the glob function to explode as the root doesnt exist.
Am I missing something here?

Well... that certainly doesn't look right.


Accessibility Audit - Carbon Website: Patterns - Empty states

This epic is tied to a larger "release" that is meant to track the progress of the Accessibility Audit for the Carbon Website. We will audit all of the pages of the carbon website based on AVT 1, AVT 2, & AVT3 - which can be found here.

AVT 1: will audit the site against the IBM Accessibility Checker - browser extension

AVT 2: will audit the site against accessible keyboard interactions

AVT 3: will audit the site against screen readers

Legend:

🌶 fully audited - on all three AVT levels

:one: 🔍 AVT 1 audited
:two: 🔍 AVT 2 audited
:three: 🔍 AVT 3 audited

:boom: issues and documentation - on all three AVT levels are complete

:one: 🚨 AVT 1 issues submitted
:two: 🚨 AVT 2 issues submitted
:three: 🚨 AVT 3 issues submitted

:sparkles: tickets are triaged

Current status of this page:

Patterns - Empty states - 2 issues - no new issues found:

1⃣ 🔍 AVT 1 audited & 🚨 AVT 1 issues submitted

Closing due to inactivity. Will reopen if/when ready to be worked on again.


Shortcut to run script cause pop up window.

Description
Whenever I use the shortcut command (control + return) on Mac OS to run my script It caused a pop up window, and once I closed that popup, a second window opens up.

DBeaver Version
DBeaver Enterprise 24.2.0
Operating System
MacOS Sequoia 15.0
Database and driver
No response
Steps to reproduce

Place the cursor at the end of the script
Do control + enter

Additional context
No response

I have faced the issue too.
Mac OSX 15
Version <IP_ADDRESS>405051837

This is a "feature" introduced in macOS Sequoia. Please see https://github.com/dbeaver/dbeaver/issues/35109#issuecomment-2306254608

This is a "feature" introduced in macOS Sequoia. Please see #35109 (comment)

Thank you. As you said, it is a feature of macOS Sequoia.


Enable beta features

Hello,
Is it possible to enable javascript es 6 features like modules ?
How do you handle the requested import path ?
Thank you in advance for your help.

Sorry for the late response @Alphapage . You can enable experimental features by passing the JsRuntimeAttributeEnableExperimentalFeatures flag to JsCreateRuntime. Module is special as it needs a bit extra work on the host side, see https://github.com/Microsoft/ChakraCore/pull/1254.

No problem. Thank you for the info.
I just need to find an example on how to implement modules or to dig into some tests cases.


[8.x] Allow seeding the database when refreshing

When using the RefreshDatabase trait along with the seed method, the seeds are ran inside of the database transaction that gets rolled back at the end of the test, so you need to re-run them for every single test.
If you want to use all the seeds for every test this is unnecessarily slow -- it would be a lot faster to run the seeds once when the database is migrated, rather than running the same seeds for every test.
This PR adds the ability to enable the --seed flag for migrate:fresh, which runs the seeds once for the entire test suite.
It's impossible to accomplish this in your own code without overriding the RefreshDatabase::refreshTestDatabase method -- you would need to run the seeders between the migrate:fresh call and the beginDatabaseTransaction call, and those calls both happen in the refreshTestDatabase method.
I made this PR to the master branch since it's technically a breaking change if you have defined a seed property or shouldSeed method in your feature tests.
No tests because I didn't see any existing tests for the RefreshDatabase trait but I'm happy to add some if needed.

Use the DatabaseTransactions trait instead of RefreshDatabase. Then run php artisan migrate:fresh --seed && phpunit.

Hey @taylorotwell, that's what we are doing now. I was hoping we could use the RefreshDatabase trait instead because it makes things a lot simpler (don't need to use a .env.testing just for the migrate:fresh call, don't need to run 2 commands every time, etc).

@matt-allan I'll talk this over with Taylor today. I can see your use case.


Adding Hang Man Game

Description
Hang Man Game in Python
Please assign it to me under GSSOC'23
Thank you
Screenshots
No response
Additional information
No response

kindly assign me this issue under gssoc'23


Updated redux ref.

Summary
Updates redux ref.
Ticket Link
n/a
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] All new/modified APIs include changes to mattermost-redux (please link)
[ ] Has UI changes
[ ] Includes text changes and localization file updates

Device Information
none
Screenshots
n/a

@mkraft @enahum Wouldn't it include all the changes from redux till now? and in that case why the PR?
coudnt it be a cherrypick from master?

@sudheerDev this PR is for master and it can include all of the mm-redux master code, the one for the 1.13 branch has the commit from the mm-redux 5.4 branch

Got confused both look good 👍 .


Breaking changes in Redis-Py 3+

If you install the latest version of Redis Python Client (3.1.0 at this time) you will receive this error when trying to run the sample code and use the rank_member function for the first time:
AttributeError: 'str' object has no attribute 'items'
This is because of breaking changes that occurred between Redis Client v2.x to v3.x
https://github.com/andymccurdy/redis-py#upgrading-from-redis-py-2x-to-30
This specific issue can be solved as follows, but I'm not sure if other parts may have broken:
https://stackoverflow.com/questions/53553009/not-able-to-insert-data-using-zaddsorted-set-in-redis-using-python
As a super quick fix, you can use redis python client v2.10.6
To install that version of redis run the following:
pip install redis==2.10.6

I sent a PR #62 that adds support for redis-py 3+, it's pending review, but for those who need to use this lib with redis-py 3+ immediately, here's a workaround:
$ pip install git+git://github.com/YuriHeupa/leaderboard-python.git@ccf3b284db35512950909aed8d1d3436557d3efb

This command installs the changes from my PR, which supports newest version of redis-py.


RDF patching

Patching the JSON-LD data graphs may become important as we incorporate approaches to do "update" indexing.
Some references for JSON-LD "diff" optons include

http://jsonpatch.com/
https://www.w3.org/2001/sw/wiki/How_to_diff_RDF

https://www.w3.org/TR/ldpatch/


Support guest authors without an avatar image

Support being able to add a guest author with a default avatar image. If guest authors do not have an account or image in their linkedin or GitHub it does not render correctly on blog posts since there is no image for the person. Can we have a default avatar image that gets shown in these scenarios?

Per docs in BLOGGING.md this should be possible already. The socials section of the the TOML file is optional and the image is drawn from the static/images/team folder. It seems like as long as there's a TOML file for the guest author and an associated image, this should just work. Closing for now unless we encounter this as a problem in the future.


Using listBlobsByHierarchy api gets duplicate directory by chance

Describe the bug
When using listBlobsByHierarchy api to get what the directories and files are under a directory,  a directory name sometimes appears twice in the result.
Code Snippet
String containerName = commonFSConf.get(CommonFSParameter.FS_BUCKET);
BlobContainerClient containerClient = blobServiceClient.getBlobContainerClient(containerName);
ArrayList objList = new ArrayList();
try {
getContainerClient();
PagedIterable blobItems = containerClient.listBlobsByHierarchy("enos/streaming/");
for (BlobItem blobItem : blobItems) {
PathStatus pathStatus = null;
// dir
if (blobItem.getName().endsWith("/")) {
pathStatus = new PathStatus(blobItem.getName(), true);
}
// file
else {
BlobItemProperties blobItemProperties = blobItem.getProperties();
pathStatus = new PathStatus(blobItem.getName(), blobItemProperties.getLastModified().toInstant(),
blobItemProperties.getETag(), blobItemProperties.getContentLength(), false);
}
objList.add(pathStatus);
}
for(PathStatus p :objList){
System.out.println(p.getKey());
}
Expected behavior
From the result, you will find "enos/streaming/o15596179429341/" appears twice.When using restful api doing same thing, the bug also appear.But the bug appears by chance.I guess because this query sent two times. So the result appears duplicate directory.
Screenshots
The result is as follows:
enos/streaming/o15452840095831
enos/streaming/o15453595539601
enos/streaming/o15488302255821
enos/streaming/o15555734875001
enos/streaming/o15596179429341
enos/streaming/o15452840095831/
enos/streaming/o15453595539601/
enos/streaming/o15488302255821/
enos/streaming/o15555734875001/
enos/streaming/o15596179429341/
enos/streaming/o15628308016831
enos/streaming/o15638787142121
enos/streaming/o15640381907881
enos/streaming/o15706178455521
enos/streaming/o15741352005031
enos/streaming/o15869202502361
enos/streaming/o15596179429341/
enos/streaming/o15628308016831/
enos/streaming/o15638787142121/
enos/streaming/o15640381907881/
enos/streaming/o15706178455521/
enos/streaming/o15741352005031/
enos/streaming/o15869202502361/

Setup (please complete the following information):

OS: mac os
IDE :IntelliJ
Version of the Library used: jdk1.8.0_241, azure-storage-blob:12.6.0

@SJTUChenJi  Thank you for your question. This can happen if you have a blob with a name that is also a prefix for other blobs. For example if you have a blob named "foo/" and also a blob named "foo/bar". You will get one result for the blob itself and another result for the prefix element. I suspect if you examine the xml response you will see one entry that is from a prefix element and another that is from a blob element. The best way to handle this if you don't want to see both is to keep a cache of your listing results as you process them and eliminate any duplicates you encounter in iterating.

Thank you for your reply. You think the reason is there are a blob and a prefix with the same name. But on Microsoft Azure Storage Explorer, there are a blob named "enos/streaming/o15596179429341" and a blob named "enos/streaming/o15596179429341/.streamsets-spark-streaming". So a similar example should be I have a blob named "foo" and also a blob named "foo/bar". Another confusing point is other blobs are similiar with "enos/streaming/o15596179429341", but only prefix named "enos/streaming/o15596179429341/" appears twice. And if you use restful api to send same request, xml response will be divided into two parts by "NextMarker" though there are only about twenty blobs and blobprefixs. Blobprefix named "enos/streaming/o15596179429341/" appears at the end of first xml response and also at the beginning of second xml response.
Maybe it's not a bug.But I am so curious why this happens. Thank you so much and look forward to your reply.

@SJTUChenJi One easy way to test this theory with the sdk, is to print out alongside each path whether it was a blob or a prefix. It looks like you have this available in your PathStatus object if I'm correctly deducing the meaning of the boolean you pass in. For this experiment, I would recommend as well that instead of checking for a "/" at the end of the name, you use the isPrefix() method on the BlobItem.
As for the format of the results you're seeing, we can't really read too much into what the service gives back for these APIs. The service only guarantees a maximum value will be returned, but no minimum, so even though you requested 5000, it may only return 10 and a continuation token for the next 10. This is affected by a whole host of things from whether or not there are snapshots that it's iterating over and skipping, if there are a bunch of blobs it's collapsing into one prefix, or even other traffic to the tenant. The fact that you're seeing these duplicate entries split across pages I would suspect is due to the fact that there are a lot of blobs with the same prefix, but we can't really say for sure.

Ok. Thank you for your explanation. Maybe the blobs are special causing the same prefix appears twice.

@SJTUChenJi I think it is unlikely there is something special about the blob. Were you able to use the isPrefix method and print whether each entry is a blob or a prefix?

please reactivate if you are still hitting this issue after trying the suggestion from Rick.

please reactivate if you are still hitting this issue after trying the suggestion from Rick.


📚 Make a beautiful README.md

The README.md right now is rather ordinary and deserves some refreshment.
Provide a nice new version of the first page of our project

[ ] I suggest adding a GIF demonstration on how the bot works, would be really nice.
[ ] Also broaden the table of languages, adding the top 20 most popular languages for us to consider in the future
[ ] Rephrase and delete unnecessary wording
[ ] Add useful link (f.e. contributing guide)

Hi, I am a first-time GitHub contributor, Can I give this issue a try or another issue? Thank you!

Just wanted to update, I am working on: Also broaden the table of languages, adding the top 20 most popular languages for us to consider in the future
Rephrase and delete unnecessary wording
I have added more popular language. I am still working on the gif part. Will push to forked branch first, when its done.

Cool, I just created the 1st pull request, waiting on how it is.


Improve Shareable code quality

Shareable needs some work. From the audit:

Doesn’t check if _required <= len(_owners) for instance, that would be a bummer. What if _required were like MAX - 1?
I have a general concern about the difference between owners, _owners, and owner in Ownable.sol. I recommend “Owners” be renamed. In general we do not recomment single character differences in variable names, although a preceding underscore is not uncommon in Solidity code.
Line 34: “this contract only has six types of events”. . . actually only two.
Line 61: Why is ownerIndex keyed by addresses hashed to uints? Why not use the addresses
directly, so ownerIndex is less obscure, and so there’s stronger typing?
Line 62: Do not love ++i) ... owners[2+ i]. Makes me do math, which is not what I want to do.
I want to not have to do math.
Move execute function from MultisigWallet to Shareable.

@maraoz I'll get working on this.

@maraoz I think it'd be simpler if the Shareable contract creator passed themselves in as an owner in the owners argument if the want be one instead of automatically adding the msg.sender in the constructor.
Also I'm a little confused why there should be a mapping and an array for keeping track of owners.
It seems like:
mapping(address => bool) ownerIndex;
Should be enough to keep track of owners.  As opposed to:
mapping(address => uint) ownerIndex; and address[256] owners;

Closed by #328.


[ChannelTabs] Request: Add middle-click functionality

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
Could be, but not really.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Pressing the middle-click on the mouse opens the channel, server in a new tab, and not the website for channels
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
I made a feature request, it is up to you, to make it a reality or not.
Additional context
Add any other context or screenshots about the feature request here.
Edit: This should be togglable, just in case.

Surprised this hasn't been implemented. Would really appreciate if anyone can add it in a pull request


Extra Python testing

Adding  a handful of very simple Python tests to the codebase. For right now, these do little more than fix some behaviours in place.
A few minor fixes in the main source have been made too.

@Harsha-Nori @slundberg ?

Ping @Harsha-Nori @Nking92 @slundberg

Thanks @Nking92  and @Harsha-Nori . Have made some updates to address your comments.

I have removed the other type annotations I added @Harsha-Nori


Show reset "x" during search

Current State
When a user is searching, he may click off of the searchbox to do something else. Especially if using edit_after_submit="current/option/concat", the search result stays in the searchbox. It would be nice if the "x" reset option is still visible even when a search is happening.
Request
It would be awesome to have the reset button visible even when an item is not selected (ie, just a search has been performed)

Ideally it is also visible even during a search:

But I understand that this may have some more downsides or be tougher to implement.

Sorry I'm not able to be more helpful on this one, I don't know TS/JS very well.

Hey, I've made this part of 0.1.17:
st_searchbox(
    search_function=search,
    style_overrides=StyleOverrides(
        clear={"clearable": "always"},
    ),
)

if you set it to clearable (other options are never / after-submit) this will show as long as you have something in the search box regardless of any selected options. hope that helps!

This is amazing! Thanks so much!
Closed by #61


Requirements, server and performance

Thank you for great example!
Could you give some clues on a few missing questions:

How the requirements for processing code are defined?
The requirements.txt in the project root only lists CDK libraries.
What virtual server have you tested the demo and which performance (say, images per second) did you achieved?
We're working on the similar project and wanted to check what resources to use.

Thank you for having a look at the code example!
Regarding your questions:

As you mentioned, the requirements.txt file contains CDK related libraries to deploy the necessary AWS cloud infrastructure for the exercise. The libraries for the processing/runtime code are located in the  src_batch_job/environment.yaml file. That file is used by conda to define a conda environment and install python dependencies. If you would like to add libraries trough pip in the conda environment file, you can take a look at the conda documentation about using pip in an environment and an example conda environment file which lists pip libraries. Alternatively, you can create a  requirements.txt file in src_batch_job directory and install dependencies by modifying src_batch_job/Dockerfile and invoking the pip install -r requirements.txt command at docker image build-time.
Since performance might vary greatly depending on the individual use-case and implementation details (e.g. sizes of individual images being processed, execution time-limit, the computational complexity and memory footprint of the model being used, the chosen instance types, etc.) an in-depth performance analysis was outside of scope for this blog-post. In order to reduce costs of readers going trough this exercise, the current code is currently set to maximum 4 instances running at the same time with an 30 minutes instance execution time limit. You can change those limits in the CDK stack code (e.g. max_vcpus / vcpus = 32 / 8 = 4 instances, and instance timeout = 30 min) as a starting point to measure performance metrics you are interested in and which are relevant to your use-case.

Thank you a lot!
(another question goes to another issue)


Added user agent suffix

This PR implements support for adding a suffix to the user agent header, which can be used for monitoring purposes to find requests from a particular client/App.

Can one of the admins verify this patch?

/azp run python - cosmos - tests


[bug] .md files can't be previewed for minting

Describe the bug
md files cannot be previewed
Reproduction
Upload a .md text file for minting.
Press the preview button
Look at the browser console for an error.

Expected behavior
See md file contents on preview
Platform and versions
Ubuntu
Google Chrome

Console output
No response
Additional context
No response

This depends on the ipfs gateway. Works on a custom gateway: https://teia.rocks/objkt/765277

Once the OBJKT was minted, I could view it fine. It was just the preview that didn't work.

Looks like this was never supported, I started adding it in aac2c2e6b358cea95971e9990ecf46ec6fe826dc

Solved as stated before so this issue will be closed when #146 is merged, but styling is wonky (as in always was)