Cannot install plugin in ElasticSearch (elasticsearch-5.3.2)

Hi,
When I try to install the .ZIP on your project into elasticsearch-5.3.2, I get this error:
./elasticsearch-plugin install file:///apps/elasticsearch-5.3.2/bin/elasticsearch-arrayformat-<IP_ADDRESS>-plugin.zip
-> Downloading file:///apps/elasticsearch-5.3.2/bin/elasticsearch-arrayformat-<IP_ADDRESS>-plugin.zip
[=================================================] 100%
ERROR: elasticsearch directory is missing in the plugin zip
I think that this is because site plugins are no longer witking with ES 5+.  So how can I install your plugin on ES 5.3.2?
Thanks,
Jim

Hi,
I was able to get the plugin installed.  I had to install on an older ES 2.3.5 instance and I had install it "manually" by creating a dir under the /plugins/arrayformat.  In that arrayformat dir, I put the jar from the ZIP and I manually made a plugin-descriptor.properties, configured for "site=true".
However, searches do not seem to work.
For example, this search works:
curl -XGET '<IP_ADDRESS>:9200/bank/account/_search?q=account_number:25&pretty'
{
"took" : 6,
"timed_out" : false,
"_shards" : {
"total" : 5,
"successful" : 5,
"failed" : 0
},
"hits" : {
"total" : 1,
"max_score" : 5.5900564,
"hits" : [ {
"_index" : "bank",
"_type" : "account",
"_id" : "25",
"_score" : 5.5900564,
"_source" : {
"account_number" : 25,
"balance" : 40540,
"firstname" : "Virginia",
"lastname" : "Ayala",
"age" : 39,
"gender" : "F",
"address" : "171 Putnam Avenue",
"employer" : "Filodyne",
"email"<EMAIL_ADDRESS>"city" : "Nicholson",
"state" : "PA"
}
} ]
}
}
but if I use arrayformat, I get:
curl -XGET '<IP_ADDRESS>:9200/bank/account/_search_arrayformat?q=account_number:25&pretty'
{
"_index" : "bank",
"_type" : "account",
"_id" : "_search_arrayformat",
"found" : false
}
What is wrong with this?  Why isn't it showing the results?

This plugin has been upgraded to ElasticSearch 7.16 and the repo with the update is here: https://github.com/ramsoft-inc/ElasticSearch-Array-Plugin


Custom docker build fails

Hi,
I am trying to build a docker image by following the tutorial given on the readme.
I initially tried working inside a container with docker run -it osrf/ros:humble-desktop-full and following the steps in the installation instructions. Everything builds, and I am able to launch the ros2 launch ur_moveit_config ur_moveit.launch.py ur_type:=ur5e launch_rviz:=true launch file, but I do not have any graphics support in this case, and cannot verify that it indeed works.
I tried to create a custom docker image with the dockerfile as shown below:
FROM osrf/ros:humble-desktop-full

RUN mkdir -p /home/ros2_ws/src && cd /home/ros2_ws && colcon build

RUN sudo apt-get update && sudo apt-get install libxml2 libpoco-dev libasio-dev xterm gdb -y

RUN cd /home/ros2_ws && \
    git clone https://github.com/UniversalRobots/Universal_Robots_ROS2_Driver.git src/Universal_Robots_ROS2_Driver && \
    vcs import src --skip-existing --input src/Universal_Robots_ROS2_Driver/Universal_Robots_ROS2_Driver.${ROS_DISTRO}.repos && \
    rosdep update && rosdep install --ignore-src --from-paths src -y -r

RUN cd /home/ros2_ws && \
    ls -R src && \
    . install/setup.sh && \
    colcon build --mixin release

COPY startup_ur.sh /startup_ur.sh
RUN chmod +x /startup_ur.sh
ENTRYPOINT ["/startup_ur.sh"]
CMD ["/bin/bash"]

But in this case, the colcon build fails with the log:
Starting >>> forward_command_controller
Starting >>> joint_trajectory_controller
Starting >>> force_torque_sensor_broadcaster
Starting >>> joint_state_broadcaster
Starting >>> ur_dashboard_msgs
Starting >>> ur_description
Starting >>> ur_msgs
Starting >>> ros2_controllers_test_nodes
Starting >>> ur_client_library
Starting >>> diff_drive_controller
Starting >>> imu_sensor_broadcaster
Starting >>> tricycle_controller
Starting >>> gripper_controllers
Starting >>> rqt_joint_trajectory_controller
/usr/lib/python3/dist-packages/setuptools/dist.py:723: UserWarning: Usage of dash-separated 'script-dir' will not be supported in future versions. Please use the underscore name 'script_dir' instead
  warnings.warn(
/usr/lib/python3/dist-packages/setuptools/dist.py:723: UserWarning: Usage of dash-separated 'install-scripts' will not be supported in future versions. Please use the underscore name 'install_scripts' instead
  warnings.warn(
--- stderr: forward_command_controller
CMake Error at CMakeLists.txt:24 (find_package):
  By not providing "Findament_cmake.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "ament_cmake", but CMake did not find one.

  Could not find a package configuration file provided by "ament_cmake" with
  any of the following names:

    ament_cmakeConfig.cmake
    ament_cmake-config.cmake

  Add the installation prefix of "ament_cmake" to CMAKE_PREFIX_PATH or set
  "ament_cmake_DIR" to a directory containing one of the above files.  If
  "ament_cmake" provides a separate development package or SDK, be sure it
  has been installed.

---
Failed   <<< forward_command_controller [0.81s, exited with code 1]
--- stderr: joint_trajectory_controller
CMake Error at CMakeLists.txt:27 (find_package):
  By not providing "Findament_cmake.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "ament_cmake", but CMake did not find one.

  Could not find a package configuration file provided by "ament_cmake" with
  any of the following names:

    ament_cmakeConfig.cmake
    ament_cmake-config.cmake

  Add the installation prefix of "ament_cmake" to CMAKE_PREFIX_PATH or set
  "ament_cmake_DIR" to a directory containing one of the above files.  If
  "ament_cmake" provides a separate development package or SDK, be sure it
  has been installed.

---
Failed   <<< joint_trajectory_controller [0.80s, exited with code 1]
--- stderr: force_torque_sensor_broadcaster
CMake Error at CMakeLists.txt:24 (find_package):
  By not providing "Findament_cmake.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "ament_cmake", but CMake did not find one.

  Could not find a package configuration file provided by "ament_cmake" with
  any of the following names:

    ament_cmakeConfig.cmake
    ament_cmake-config.cmake

  Add the installation prefix of "ament_cmake" to CMAKE_PREFIX_PATH or set
  "ament_cmake_DIR" to a directory containing one of the above files.  If
  "ament_cmake" provides a separate development package or SDK, be sure it
  has been installed.

---
Failed   <<< force_torque_sensor_broadcaster [0.80s, exited with code 1]
--- stderr: joint_state_broadcaster
CMake Error at CMakeLists.txt:14 (find_package):
  By not providing "Findament_cmake.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "ament_cmake", but CMake did not find one.

  Could not find a package configuration file provided by "ament_cmake" with
  any of the following names:

    ament_cmakeConfig.cmake
    ament_cmake-config.cmake

  Add the installation prefix of "ament_cmake" to CMAKE_PREFIX_PATH or set
  "ament_cmake_DIR" to a directory containing one of the above files.  If
  "ament_cmake" provides a separate development package or SDK, be sure it
  has been installed.

---
Failed   <<< joint_state_broadcaster [0.79s, exited with code 1]
--- stderr: ur_dashboard_msgs
CMake Error at CMakeLists.txt:4 (find_package):
  By not providing "Findament_cmake.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "ament_cmake", but CMake did not find one.

  Could not find a package configuration file provided by "ament_cmake" with
  any of the following names:

    ament_cmakeConfig.cmake
    ament_cmake-config.cmake

  Add the installation prefix of "ament_cmake" to CMAKE_PREFIX_PATH or set
  "ament_cmake_DIR" to a directory containing one of the above files.  If
  "ament_cmake" provides a separate development package or SDK, be sure it
  has been installed.

---
Failed   <<< ur_dashboard_msgs [0.79s, exited with code 1]
--- stderr: ur_description
CMake Error at CMakeLists.txt:4 (find_package):
  By not providing "Findament_cmake.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "ament_cmake", but CMake did not find one.

  Could not find a package configuration file provided by "ament_cmake" with
  any of the following names:

    ament_cmakeConfig.cmake
    ament_cmake-config.cmake

  Add the installation prefix of "ament_cmake" to CMAKE_PREFIX_PATH or set
  "ament_cmake_DIR" to a directory containing one of the above files.  If
  "ament_cmake" provides a separate development package or SDK, be sure it
  has been installed.

---
Failed   <<< ur_description [0.78s, exited with code 1]
Aborted  <<< ur_msgs [0.79s]
Aborted  <<< tricycle_controller [0.77s]
Aborted  <<< imu_sensor_broadcaster [0.77s]
Aborted  <<< gripper_controllers [0.76s]
Aborted  <<< diff_drive_controller [0.78s]
Aborted  <<< ur_client_library [0.97s]
Aborted  <<< ros2_controllers_test_nodes [1.04s]
Aborted  <<< rqt_joint_trajectory_controller [1.01s]

Summary: 0 packages finished [1.37s]
  6 packages failed: force_torque_sensor_broadcaster forward_command_controller joint_state_broadcaster joint_trajectory_controller ur_dashboard_msgs ur_description
  8 packages aborted: diff_drive_controller gripper_controllers imu_sensor_broadcaster ros2_controllers_test_nodes rqt_joint_trajectory_controller tricycle_controller ur_client_library ur_msgs
  11 packages had stderr output: diff_drive_controller force_torque_sensor_broadcaster forward_command_controller gripper_controllers imu_sensor_broadcaster joint_state_broadcaster joint_trajectory_controller tricycle_controller ur_dashboard_msgs ur_description ur_msgs
  10 packages not processed

Would appreciate some help on how I can build my own image.
Thanks in advance !

This works for me:
FROM osrf/ros:humble-desktop-full

RUN mkdir -p /home/ros2_ws/src && cd /home/ros2_ws && colcon build

RUN sudo apt-get update && sudo apt-get install libxml2 libpoco-dev libasio-dev xterm gdb -y

RUN cd /home/ros2_ws && \
    git clone https://github.com/UniversalRobots/Universal_Robots_ROS2_Driver.git src/Universal_Robots_ROS2_Driver && \
    vcs import src --skip-existing --input src/Universal_Robots_ROS2_Driver/Universal_Robots_ROS2_Driver.${ROS_DISTRO}.repos && \
    rosdep update && rosdep install --ignore-src --from-paths src -y -r

RUN cd /home/ros2_ws && \
    . /opt/ros/humble/setup.sh \
    colcon build --mixin release

#COPY startup_ur.sh /startup_ur.sh
#RUN chmod +x /startup_ur.sh
#ENTRYPOINT ["/startup_ur.sh"]
CMD ["/bin/bash"]

Hi @fmauch,
Thanks for the help. I was able to build the docker image successfully with your edit. It seems to me that I forgot to source the correct setup file.
I'm closing this issue for now while I carry on with my testing.


Changing all team logo

We need to change all team logo, we will attach new logos.

only three logo which are devops, chomlex and backend (posinova) team logo will attached here, other logos, you can download in discord channel
Here is the devops logo

This issue is done, I am not sure what's happen it. I t does not close automatically


Filter

Add a chat filter to the bot so you can filter out certain things that may or may not be something you want said.
side note this should probly be something that is seperate and can be placed on any bot. Maybe i should make a different space for that for that.

Maybe twitch emoji support if you ever get both of those working?

what exactly do you mean? your talking about if i add twitch support to this bot? or something else?

Yea, if you add twitch.

I have a seperate bot i started to create. it has its problems as well. It doesn't work too badly. It does irc and twitch is basically irc if that helps you. Only thing is if i added this I'd be merging some stuff together if not the 2 bots. I'm not sure how id add the emojis to youtube or discord but I'm sure I'd have to start with the twitch api.

Just a thought really.

Its worth looking into. out of curriousity. which is more important. Features or fixing the crashing every 10 or whatever hours. I think its just the api timing out. I authenticate and i think that gets old and needs to be refreshed. Im just currious if thats more important that the bot translating the discord to youtubes chat.

Nah, discord to youtube would definitely be better.

K. when i get to it I'll look at that. It's gonna be a longer project than some things as i have to read through the api and understand what i did wrong. I dont even think i implemented all the code into this at all as before i implemented it i tested in a separate small program to just print say hi or something and it wouldnt. fun times right?

If you don't mind it sounds more like a weekend project when I don't have exams and stuff going on as much to take my focus away on. I may brush up on it soon because curiosity of why gets to me.  Tbh i kinda forgot about this issue.

Yea man, it's not super pressing - I was just wondering if you were still working on it or not, ya know. But thanks for looking into it when you can!

Sorry that it kinda slipped. I'll finish that soon.


nss: 3.68.4 -> 3.79.1

The 3.68.4 release was the last of the 3.68 ESR series and 3.79 is the
new ESR series.
https://firefox-source-docs.mozilla.org/security/nss/releases/nss_3_79.html
https://firefox-source-docs.mozilla.org/security/nss/releases/nss_3_79_1.html
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Cherry-picked to master for staging, nixpkgs-review reveals no new failures on macos 10.15.


Webhook not receiving events

webhook endpoint is not receiving any events any help please ?
@jibon57

Please use discussion to discuss first


[BUG] State issue with label graph tabs in the App

On fiftyone==0.7.0, I run the first few cells of the uniqueness tutorial in order to load a dataset with content in both the Labels and Scalars graph tabs in the App.
I then do the following:

Click Scalars to open the scalars graph tab
Click Hide ^
Click Labels to open the labels graph tab

At this point, the Labels tab is active but the Scalars graphs are still being shown. I have to click another tab and then click the Labels tab again in order to see the correct Labels graphs.
The issue: Labels tab is active but Scalars graphs are showing:

I am unable to reproduce on develop and 0.7.0. I tested in a notebook and browser, using the uniqueness tutorial.
Can you still produce this issue?

I am unable to reproduce on develop and 0.7.0. I tested in a notebook and browser, using the uniqueness tutorial.
Can you still produce this issue?

Hmm that's weird. I tried again in a colab notebook and it still happens. Then I tried loading the quickstart dataset instead and running the cell + procedure above in the same notebook and it did not happen for that dataset.
I tried with quickstart dataset locally (with App in web browser) and it did not happen there either.
I'm using Chrome, btw.

I marked as awaiting-more-evidence.

Thanks, I'll take another look before the next release.


Bug fix/SEARCH-373

Scope & Purpose
To fix iteration same field as nested and as normal - additional container for nested fields were created.

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[x] Tests

[ ] Regression tests
[x] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[x] Backport for 3.10: (Please link PR)
[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)

Related Information

[ ] Docs PR:
[x] Enterprise PR: https://github.com/arangodb/enterprise/pull/1066
[ ] GitHub issue / Jira ticket:
[ ] Design document:

Tests blue except known
=== shell_client_replication2_recovery_3 ===

Test "shell_client_replication2_recovery_3"
[FAILED]  tests/js/client/shell/transaction/replication2_recovery/shell-transaction-replication2-recovery.js
"test" failed: server unavailable for testing. Instance not healthy! {"setUpAllDuration":{"transactionReplication2Recovery":181},"testFailoverDuringTransaction":{"setUpDuration":67,"tearDownDuration":43,"status":true,"duration":9339},"teardownAllDuration":{"transactionReplication2Recovery":962},"duration":10592,"status":true,"failed":0,"total":1,"totalSetUp":248,"totalTearDown":1005}

Suites failed: 2 Tests Failed: 0


Error listing dataset files

This dataset with only directories in its root (there are files in the directories) returns an error when attempting to list the files:
> kaggle datasets files jonathanbesomi/rsna-miccai-png
max() arg is an empty sequence

The CSV display option does correctly handle no files. Though the dataset does indeed have files in it.
> kaggle datasets files jonathanbesomi/rsna-miccai-png --csv
name,size,creationDate

Reproducing in Python
from kaggle.api.kaggle_api_extended import KaggleApi

api = KaggleApi()
api.authenticate()

resp = api.process_response(api.datasets_list_files(
    owner_slug="jonathanbesomi", dataset_slug="rsna-miccai-png"))
print(resp)

python ./test.py
{'datasetFiles': [], 'errorMessage': None}

System info
> kaggle -v
Kaggle API 1.5.12

> python --version
Python 3.9.0

So is there a workaround for this issue, or some other way to list files?

I am facing the same issue . I want to download only train dataset from https://www.kaggle.com/paultimothymooney/chest-xray-pneumonia but unable to do so and getting the same exact error .

Same issue here, still not addressed. It seems kaggle files list will not list files under the sub-directories, which will lead to less or even no files listed by the api. Similar issue: https://github.com/Kaggle/kaggle-api/issues/386


Add rawValue property in MSGMessageBody to access the raw value.

FIX : #55, related #51

@sjoness, I updated the PR, I just follow the other switch-case indentations from other files.
Thanks,

Thanks for this @rogermolas


Fix proper generation/versioning metadata labels

Ref: https://github.com/nais/start.nais.io/blob/8f1b9fac37d7f5b2758ef997a4a0771a7ecdc48b/src/main/kotlin/io/nais/RequestResponseMapping.kt#L45
Suggestion that could help; https://stackoverflow.com/a/56716067 (thanks to this slack message)
Also: Maybe set these labels on all k8s resources created? Like alerts and the like?

Actually, it's just the "set on all resources" that was fixed. The TODO referenced is still valid.


Add shorter duration for stale PR

Duplicate Fabricbot rule for marking stale issue to mark PR as stale after 60 days of inactivities.
AB#2260

is that file generated by a tool? seems like lots of changes, so not really reviewable

Yes it is.

@sonalivdeshpande How did you get your recent PRs that updated the bot config to be small?

@sonalivdeshpande How did you get your recent PRs that updated the bot config to be small?

I had updated the required config on fabric bot portal, exported the new config file and replaced it with the file in .github folder 😄

oh... the intent got changed... let me fix that

@anthony-murphy @tylerbutler, fixed the indent and the diff is reasonable now.


Why do you have radar in the code?

There is nothing mentioned in the paper about radar, but why does the code has radar usage as default?
also,  in vip3d_resnet50_3frame.py, the default flag for 'use_radar' is set as False, but the data loader loads radar data.
We need explanation on this.

The dataloader loads both lidar and radar as inputs, but the model does not use them. You can set points = None and radar = None at the beggining of the forward function here during inference, and it will obtain the same evaluation results.


Stopped working on "Files changed" (review) tab

Hi, first of all thanks for a great add on!
I'm not sure when, or if it's just me, but recently the "GIF" button has only been appearing on the "Conversation"-tab editor, and is no longer available on the code-review editor :(
It would be really nice if this functionality could be restored :)

Thanks for raising this 🙌
I have just merged a fix for the button not appearing when editing a comment in the "Conversation" tab (https://github.com/N1ck/gifs-for-github/pull/41). This should be out in the next release but let me know if it's still a problem.
I have taken a brief look at fixing it within the code-review editor and it's not quite as trivial. I will have a go at this when I have some time 💪

Im still not seeing when I try to add a new comment on a line via the Files Changed tab

Still not seeing it 😢

Sorry that it took so long to get to this. I am releasing a new version now that I hope resolves the issue


FIX: Re-add fsr-checkxopts to Docker image

Fixes #286 in sMRIPrep.

In my tests of mounting the missing command to the container, fmriprep finished without errors.


Config from image-url branch applied to alt-1

Would be better to use a different branch name in the stackhpc repo next time.


Add db/schema.rb to .gitignore

The deploy script runs migrations, thereby generating a fresh schema.rb every time.
So do developers, when they run db:migrate after setting up the VM or migrating.
According to @pellaea, nothing uses the schema.rb version committed to github, and it causes many deploy conflicts, so let's get it out of the repo!

This is not the right thing.

Coverage: 93.524%. Remained the same when pulling fdc8276c3c875a20baf081df43cc9ad3dad50450 on nimmo-gitignore-schema into 9ce4fb90381b0d507997d107a712e14ec9d12271 on main.

The schema.rb is used to create initial environments.  If it gets removed from the repo there will be no way to build a new system.  I very much doubt the current migrations will do the right thing.  There has to be a way to get a schema.rb that is consistent.

Nathan, do you mean dev-startup needs it?  Does the test suite use it?
I'm totally fine with it staying in there, but if so we need to find a way
to keep it more stable.  It is constantly in conflict with my local copy.
For example, every time I run a db migration it tweaks the options on the
"create table" lines uselessly, so I'm constantly having to clobber my
local copy.
Another case I've seen a number of times is that a dev accidentally gets
their local db in a weird state, resulting in old superfluous tables.  This
gets written into their copy of the schema and accidentally added to PRs.
I thought maybe we could skip the whole thing by simply removing it from
the repo (but allowing rails to create it locally).  But if we somehow need
to get a copy to dev-startup, that doesn't make any sense, I agree.
On Thu, Mar 2, 2023 at 9:38 PM Nathan Wilson @.***>
wrote:

The schema.rb is used to create initial environments. If it gets removed
from the repo there will be no way to build a new system. I very much doubt
the current migrations will do the right thing. There has to be a way to
get a schema.rb that is consistent.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1385#issuecomment-1452871141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNLI3QFG3BXIDR4A263W2FKQ3ANCNFSM6AAAAAAVOCAMBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

This indicates that there is an issue with one or more of our local databases.  That's what needs to get fixed.  Can you provide a copy of the conflicting schema.rb?  That would help trace down what is wrong.

No, I don't have time for that now.  I've got multiple projects due in the
next few days.  Sorry.  I guess when it comes up again I'll re-open this.
It won't be long, I'm sure! :)
On Thu, Mar 2, 2023 at 9:55 PM Nathan Wilson @.***>
wrote:

This indicates that there is an issue with one or more of our local
databases. That's what needs to get fixed. Can you provide a copy of the
conflicting schema.rb? That would help trace down what is wrong.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1385#issuecomment-1452883045,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNPJPEJEN3ELPJVBBIDW2FMTPANCNFSM6AAAAAAVOCAMBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just in case you have a moment with your local environment running, all I need is for you to run: bundle exec rake db:schema:dump, send me db/schema.rb, and then do git checkout db/schema.rb.

Okay, I can do that...  Mine is currently identical to the one on main.
Maybe we can get Joe and Nimmo and you to do the same test?  At least track
down whose is different?
It is conceivable that when I recently tested Nimmo's new Vagrantfile --
and therefore reinstalled from scratch -- that fixed some long-standing
problem with configuration of my local db...   Ah, and if so, I bet the
production db has the same issue that I used to have...
On Thu, Mar 2, 2023 at 10:06 PM Nathan Wilson @.***>
wrote:

Just in case you have a moment with your local environment running, all I
need is for you to run: bundle exec rake db:schema:dump, send me
db/schema.rb, and then do git checkout db/schema.rb.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1385#issuecomment-1452891132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNKZKMOEBDRC4LU2JT3W2FNZXANCNFSM6AAAAAAVOCAMBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Interesting!  Mine is also identical to the one on main as is the one currently in production.

Nah, I just tried bundle exec rake db:schema:dump on production, and it
too is identical to what's currently in main.
On Thu, Mar 2, 2023 at 10:12 PM Jason Hollinger @.***> wrote:

Okay, I can do that...  Mine is currently identical to the one on main.
Maybe we can get Joe and Nimmo and you to do the same test?  At least track
down whose is different?
It is conceivable that when I recently tested Nimmo's new Vagrantfile --
and therefore reinstalled from scratch -- that fixed some long-standing
problem with configuration of my local db...   Ah, and if so, I bet the
production db has the same issue that I used to have...
On Thu, Mar 2, 2023 at 10:06 PM Nathan Wilson @.***>
wrote:

Just in case you have a moment with your local environment running, all I
need is for you to run: bundle exec rake db:schema:dump, send me
db/schema.rb, and then do git checkout db/schema.rb.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1385#issuecomment-1452891132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNKZKMOEBDRC4LU2JT3W2FNZXANCNFSM6AAAAAAVOCAMBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I agree with Nathan.
From schema.db:
db:schema:load. When creating a new database, bin/rails db:schema:load`
tends to
be faster and is potentially less error prone than running all of your
migrations from scratch. Old migrations may fail to apply correctly if
those
migrations use external dependencies or application code.

It's strongly recommended that you check this file into your version
control system.
I do sometimes see the first issue as Jason (conflicting options in the
schema). For me, it takes only one click to fix it locally.
I have seen old superfluous tables in my local db, and can have fixed that
locally. I haven't seen that in a while.
In response to Nathan's question, it was caused -- in my case -- by some
combination of switching between branches with different schemas and/or
using an older db snapshot and/or running migrations in the wrong branch at
the wrong time.
There's a short write-up at:
https://github.com/MushroomObserver/developer-startup/blob/main/DEVELOPER-WORKFLOW.md#loading-a-snapshot-of-the-live-database-optional
I don't remember those tables being added back to the repo, which to me
indicates that it's a rare event. It also can be fixed.

Joe

On Thu, Mar 2, 2023 at 7:06 PM Nathan Wilson @.***>
wrote:

Just in case you have a moment with your local environment running, all I
need is for you to run: bundle exec rake db:schema:dump, send me
db/schema.rb, and then do git checkout db/schema.rb.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1385#issuecomment-1452891132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFEMLPQKAQXCGSLJMNLW2FNZVANCNFSM6AAAAAAVOCAMBE
.
You are receiving this because your review was requested.Message ID:
@.***>

Yeah, I guess we just wait until it happens again and see if we can figure
out why it just happened... :(
On Thu, Mar 2, 2023 at 10:16 PM Nathan Wilson @.***>
wrote:

Interesting! Mine is also identical to the one on main as is the one
currently in production.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1385#issuecomment-1452898155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNIKGGXV5IXAIW5MKZTW2FPCHANCNFSM6AAAAAAVOCAMBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yeah, it should be rare, but it seems to happen way too often.  For
example, it just happened to Nimmo out of the blue on production while
deploying the latest main.  WTF?  What possible reason did it have for
suddenly messing with the schema??
On Thu, Mar 2, 2023 at 10:17 PM Jason Hollinger @.***> wrote:

Yeah, I guess we just wait until it happens again and see if we can figure
out why it just happened... :(
On Thu, Mar 2, 2023 at 10:16 PM Nathan Wilson @.***>
wrote:

Interesting! Mine is also identical to the one on main as is the one
currently in production.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1385#issuecomment-1452898155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNIKGGXV5IXAIW5MKZTW2FPCHANCNFSM6AAAAAAVOCAMBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'd have to see that example.  The way I look at it, any unexpected change to schema.rb indicates that something is wrong in someone's database and that needs to be hunted down.  You can actually look at that as one of the benefits of having the schema.rb checked into main.  There can be several sources of this problem from people not checking in the new schema.rb after running a migration (which is kind of harmless) to folks getting turned around after running migrations from different branch and doing the correct rollbacks.

Closing this... it'll be here to preserve the conversation for reference.


A vision for the future of full-stack python development

I'd love to be able to write something like this, so I'm working on a little framework that enables it.
@router.get(
    "/", response_model=Union[FastUI, List[Item]], response_model_exclude_none=True
)
async def list_items(
    request: Request,
    items_svc: ItemsService = Depends(ItemsService),
) -> Union[FastUI, List[Item]]:
    data = await items_svc.list_items()
    return await respond_to(
        request,
        {
            Format.json: (data),
            Format.html: (data, items_svc.render_ui_list),
        },
    )

I think it might not be worth the implementation work if FastUI and FastAPI were able to manage Content-Type and Accept headers to serve json responses and html responses to users.
Is something like this already supported? Does something like this already exist?
Thanks!

For more context, items_svc.render_ui_list returns a List[AnyComponent] and respond_to handles the response formatting.


Update Composer dependencies (2020-12-19-00-15)

Visual regression test passed!
View the visual regression test report


MacOS/AArch64 support

Given that Apple is planning to migrate all of their desktop offerings to AArch64 over the next 2 years, it seems like effort spent on porting wouldn't be wasted. I'm hoping that the existing Linux AArch64 support can be shared but I haven't taken a good look at the docs yet.
Opening this with lofty ambitions to take a look at this over the next few weeks but I'm under no illusions that this change is probably well beyond my current skill level. More realistically I guess it'd be good to just get this on the radar.

Thank you for your interest in async-profiler.
No doubts that macOS/ARM64 port will be a valuable addition to the project. I don't expect much difficulties in creating the initial port; however, supporting new platform in future is a continuous job that includes regular building, testing, publishing, etc.
In the mean time, I don't even have the suitable hardware to do this.
I'll make the port if someone gets me a new M1 MacBook Pro :wink:
Seriously, I think this would be a fair deal: permanently adding macOS/ARM64 to the list of async-profiler supported platforms for one-time material support of the open source project.

Sorry for the radio silence on this, you raise a valid point. I'm a bit snowed at the moment with other work but have done a bit of digging and at least for now it looks like GHA support for M1 runners is a while away so there goes that bright idea. Still won't have time to dedicate to this for now but that might change in the next few weeks.

This is critical for us as well, using it with the clojure wrapper clj-async-profiler 🙏🏻

macOS/ARM64 port has been pushed to the master. It will be included in the nearest release.

Here is a pre-release package of async-profiler for macOS/arm64.
Please try and provide the feedback.
async-profiler-2.1-ea-macos-aarch64.zip


Fix SignOut Issue

There is an issue with signout functionality

@hereisSwapnil Please assign me under gssoc.

@JrG-One Done

Hey! @hereisSwapnil and @JrG-One ,
I forked this repo maybe a week ago and was working on this issue. If @JrG-One has not yet started working on this issue please let me know ! Actually, the fix is about to complete.

@JrG-One are you working on the issue?

@hereisSwapnil actually i'll start working on it from tomorrow

@JrG-One are you working on the issue?

Yes i was working from the day i forked the repo !
In between i got busy with some stuff but now i am totally on it.

@hereisSwapnil can i create a pull request for this as i fixed this issue


LSTM input mapping

Hi,
i tried to find information in the documentation but didn't find anything related to that. I also looked in the LSTM papers but wasn't able to find the information. So maybe it's missing documentation or I'm just to dumb.
I would like to know how e.g. 30 inputs are mapped to 10 lstm units.
There are several possibilities which I can imagine and could make sense. But non of them fit to the kernel weights when I plot them.
e.g.
10 LSTM units have 40 kernel weights (10 [units]*4 [gates]) - which is ok according to the LSTM paper from 1997.
but how are these now mapped to my 30 inputs? Are they applied in a sliding window approach where the windows has the size 10?
BR

I think you confuse "LSTM cell" with LSTM layer argument named "units".
In Keras, LSTM(units=N) means that every cell of LSTM layer have N neurons (at every moment the hidden state is vector with size N).
The number of LSTM cells is determined automatically (if you feed LSTM with input tensor with shape (150, 300) then 150 LSTM cells will be applied).
This is LSTM layer containing 3 LSTM Cells (not units):

Best LSTM tutorial I found so far:
http://colah.github.io/posts/2015-08-Understanding-LSTMs/
Very detailed description of algorithm here:
http://deeplearning.net/tutorial/lstm.html
The original LSTM paper you can find here: http://deeplearning.cs.cmu.edu/pdfs/Hochreiter97_lstm.pdf
Keras LSTM implementation may also be helpful to understand how it works: https://github.com/fchollet/keras/blob/master/keras/layers/recurrent.py#L1122

Thanks for your answer.
I looked into the original paper and I marked the parts of the LSTM Layer (in green), i marked one LSTM unit (in red) and a single LSTM cell (in purple).
  Source: Hochreiter 1997 - Page 9
But like the picture shows each input goes into all gates of each unit. So thats the point I don't get.
When I model a LSTM layer with 10 units, I get a recurrent weight matrix with 40 weights.
But the number of weights don't change when I change the number of inputs (in keras).
That means that there needs to be a way to apply a "random" number of inputs to a static LSTM layer.
I had a look at the implementation in keras. and in line 1144 they just take the first x inputs (number of units) as x_i. And before that, they do a time_distributed_dense in line 1074, maybe thats the "magic" how to apply a random number of inputs to a fixed number of weights.
 Maybe I just miss something :-)
BR

@mahnerak I want to have one clarification regarding the expansion of step function.  Does the step function execute recursively calling itself again and again pass old computed values of h_t-1 and c_t-1 to the next call to get h_t,c_t and so on

@shaifugpt Right, every time it takes previous states h_t-1, c_t-1 (and also x_t-1) in order to compute new states h_t, c_t.
But actually it is not recursively calling itself. The backend functionK.rnn (same as theano.scan or tf.scan) makes it possible to apply the expression on sequences. (Low level-code is behind)
I don't know what's the main purpose of your question, so can't decide what to note in more detail.

Thanks @mahnerak
I want to use all the previous hidden states for computing the values of the h_t instead of just h_t-1.  For this I modified the step function in the LSTM to store the previous hidden state as: Here h_tm12 is defined to store all states.
var_exists = 'h_tm12' in locals() or 'h_tm12' in globals()
if(var_exists):
h_tm12=numpy.vstack((h_tm12,states[0]))
else:
h_tm12 = states[0]
But h_tm12 shows the same size as states[0]. I expected it to be timesteps x states [0]. I expect that the step function executes once for every timestep. But the size does does not seem to agree to that. This is why I wondered if or not it is running in a loop so that I can save all the previous states while training to use them


music player is broken! noooooooooooooooooooooooooooooooo!

I blame @develogix 's refactor.

@naterchrdsn, it was my fault... 💩


Default weights_initializer for tf.contrib.layers.convolution2d should be xavier_initializer_conv2d

Hi,
I notice that in tf.contrib.layers.convolution2d, tf.contrib.layers.convolution2d_in_plane, tf.contrib.layers.convolution2d_transpose, tf.contrib.layers.separable_convolution2d, the default weights_initializer is tf.contrib.layers.xavier_initializer, rather than tf.contrib.layers.xavier_initializer_conv2d. Is it better to use tf.contrib.layers.xavier_initializer_conv2d?
Thanks!

xavier_initializer_conv2d = xavier_initializer

Got it. Thanks!


Artifactory supermarket support

The extension should have a task to upload to an artifactory supermarket instance using basic authentication.

Hello,
Would the be possible to use the JFrog Artifactory extension in the VSTS Marketplace - https://marketplace.visualstudio.com/items?itemName=JFrog.jfrog-artifactory - to accomplish this.
If it does not do what you require could you add some specifics on what you require?
Thanks. Sorry about the delay in replying to you about this.

That might work, I'll take a look. At first glance it looks like you'll have to package the cookbook on your own.
Anyway, all I really need is a task which wraps knife-art. Maybe I should just make one for myself... Are the vsts-chef task definitions available somewhere?

That makes sense. I will setup and test a task that uses knife-art to upload a cookbook.
At the moment the VSTS Chef Extension code is not available publicly as it is in VSTS so we can take advantage of the pipelines there. I am looking at mirroring the code to here so people can see it - trouble with that is how to handle PRs.

We have just released a new version (1.3.9) of the extension that has two new tasks:

Install Gem
Execute Knife

The first will allow you to install knife-art and the second task will allow you to run the Artifcatory knife commands.
Hope this helps you with what you need to do.


Supported Device: Roomba 976

Roomba Model:
976
Did It Work
No

[2/26/2022, 11:50:05 AM] [iRobot] Failed to configure roomba: Roomba Woonkamer see below for details
[2/26/2022, 11:50:05 AM] [iRobot] No Roomba Found With Blid: 31F2841051519550

@mbnn the Homebridge UI is a bit misleading, where it reads "Install previous version" it should read "Change version" or similar:

then

I cannot control this. This is part of the built-in homebridge ui

Installed, thanks now it says: 3/1/2022, 5:54:58 PM] [iRobot] Initializing iRobotPlatform platform... 3/1/2022, 5:54:58 PM] [iRobot] Logging into iRobot...
3/1/2022, 5:55:29 PM] [iRobot] UDP Disabled 3/1/2022, 5:55:49 PM] [Homebridge UI] [homebridge-irobot] Terminating child process... 3/1/2022, 5:55:49 PM] [Homebridge UI] [homebridge-irobot] Child process ended 3/1/2022, 5:55:55 PM] [Homebridge UI] [homebridge-irobot] Terminating child process... 3/1/2022, 5:55:55 PM] [Homebridge UI] [homebridge-irobot] Child process ended
Thats it.

acording to this line:  [3/1/2022, 5:55:29 PM] [iRobot] UDP Disabled UDP is disabled on your router. To fix this you will have to manually specify your Roomba’s ip addresses in the config.

Got the IP manually in now, still nothing.
However, when using manual i have no idea where i should find the Blid and password of the device itself.
There is nothing under the tab 'devices'.

Please try the latest beta:


found a typo

Hi, I found a typo on your website.

@Makonike can you help fix it at: https://crowdin.com/project/casdoor-website ?

@Makonike can you help fix it at: https://crowdin.com/project/casdoor-website ?

done

@Makonike great, thanks!

@Makonike BTW, are you interested in: https://github.com/casbin/Talent2023 ?


fix(deps): allow pyarrow version 7+

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass

One exception: I couldn't test 3.7 because PyArrow doesn't provide a macos/arm64 wheel for 3.7, and I couldn't get the build working on my machine. PyArrow does claim to be compatible with 3.7 however.

[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #478 🦕

Your prerelease build fails because this installs Shapely 2.0a1
nox > python -m pip install --prefer-binary --pre --upgrade alembic geoalchemy2 google-api-core google-cloud-bigquery google-cloud-bigquery-storage sqlalchemy shapely google-cloud-core grpcio

and that version introduces breaking changes from 1.x for google-cloud-bigquery. The only reference to shapely I could find was in a google-cloud-bigquery extras definition, and that constrains shapely to "Shapely>=1.6.0, <2.0dev", which won't install the alpha release that's breaking this build.
How should this build failure be resolved?

Any updates on this PR?
As it stands, a user can't use snowflake-connector-python with sqlalchemy-bigquery 1.4.4
The conflict is caused by:
    sqlalchemy-bigquery 1.4.4 depends on pyarrow<7.0dev and >=3.0.0
    snowflake-connector-python[pandas] 2.8.0 depends on pyarrow<8.1.0 and >=8.0.0; extra == "pandas"

This PR has been open so long that a newer PyArrow version has been released, and this PR should be updated to include it. I can do that when I get back from vacation, but is anyone from this organization available to review and comment on how to resolve the build failure? I’m happy to puzzle through dependency trees if someone can tell me what it is I actually need to fix.
Requiring stale versions of popular packages is pathological to Python development because pip offers no mechanism to override conflicting dependencies, so this package is a bit of an albatross for anyone who needs to use it. This only gets worse as more of the PyArrow ecosystem drops support for old versions

@tswast ^^ Any help will be appreciated

@chalmerlowe and I discussed this. Since PyArrow releases so many major versions that don't actually break our usage, I think it's best if we just omit the top constraint on pyarrow across all of the BigQuery packages.

I think it's best if we just omit the top constraint on pyarrow across all of the BigQuery packages.

@willsthompson @tswast - FWI the accidental closure of this PR may have removed the “automerge” label.

20221128: there is a known Kokoro prerelease issue with parsing the sqlalchemy version number.
The prerelease version number is currently x.x.0b2.
The patch value of 0b2 cannot be converted to an integer and thus the prerelease test fails.
For the moment, I am ignoring this issue.


Regarding Custom Dataset

Hi. I have been doing research regarding point cloud object detection. However, all the architectures currently support only benchmark datasets and there is no proper guidance on how to prepare our own datasets. For example, I have some point clouds of patients and I want to detect some regions for medical purposes. However, I cannot find any guidance here or in other models as well to work with custom datasets. Will appreciate any guidance on this. Thanks!

Hi @pytholic. Were you able to train with your custom dataset ?

@Windson9 Not really. I ended up using some segmentation methods. It was long time ago so I do not remember all the details.
Have a look at his repo, it might be useful.

@Windson9 @pytholic I've implemented the training of the custom dataset, you could refer to this


allow ajax requests from 'self' same origin within inscriptions

Relating to discussion here:
https://github.com/ordinals/ord/issues/1082
Allow browser ajax requests to same origin /content/ path for flexibility and composability in the ord protocol.

This would allow inscriptions to request arbitrary content from the server, which is not guaranteed to be stable, and thus introduce mutability, which we don't want.

good point, we need to limit it to /content/ which are liimted to on-chain inscription data


update schema to include parallel property

Expected Behavior
To better support autocomplete, the workflows schema requires a definition for the parallel property in the step definition
Documented usage: https://cloud.google.com/workflows/docs/reference/syntax/parallel-steps

Merged!


babe: introduce a request-answering mechanic

For now, the only request type it answers is requesting the epoch that a child block will have.

I'm not totally sure I need this, given #7789, but maybe it's useful? Cheaper than going to runtime for sure.

I'm not totally sure I need this, given #7789, but maybe it's useful? Cheaper than going to runtime for sure.

any progress on this?

@gnunicorn pretty sure it's ready


Merge sample data endpoint from osints/dev

Description
For work on the integrations UI, we added a sample data endpoint to osints/dev. This PR does that merge. TODO: Add tests.
Issues Resolved
N/A
Check List

[x] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Cypress tests are available in the front-end, which will be in a future PR. Unit tests still aren't present, we can add it later.


Edit and  Update Sample Data

Based on this https://github.com/nus-cs2103-AY2223S2/forum/issues/303, we're not allowed to edit sample data unless there's an error! Closing this :)


Need link to submission form on home page and on country page (US as sample) to create a new country page or add to an existing page

As described. At present, there's no way to get a user from the home page or the current sample country page (United States) to the form where fields are completed and submitted to the different spreadsheets. Please add so we can have the initial submitters set up example pages.

I don't think we need a request-a-country page, people submit resources and we create a page for them. However we could create empty pages for all the countries? That would work well if we can add form links to the bottom of each page. I just want to avoid having a few pages, with some having content on and some not and no way to tell which is which.
The current build has the forms linked from a page called "Contribute":
https://labs.creativecommons.org/reversionary-rights/contribute/
Having links to add resources on each page is a very good idea, I'll see if we can pre-populate the country field in the form.

+1


Error reading SSH key even AWS_SSH_KEY env is set

What happened:
This is a bug in 1.19 and previous branches.
Test command:
ETCD_CERTIFICATE=/etc/kubernetes/pki/kube-apiserver/etcd-client.crt
ETCD_KEY=/etc/kubernetes/pki/kube-apiserver/etcd-client.key
KUBE_SSH_USER=ubuntu
AWS_SSH_KEY=/Users/jiaxin/.ssh/id_rsa
go run cmd/clusterloader.go --kubeconfig=/Users/jiaxin/.kube/config --testconfig=/Users/jiaxin/go/src/k8s.io/perf-tests/clusterloader2/testing/density/config.yaml --provider=aws --enable-prometheus-server --alsologtostderr --v=2 --report-dir=/tmp/pt/

Result:
E0209 23:19:24.425522   29323 clusterloader.go:221]   Status: Fail
E0209 23:19:24.425525   29323 clusterloader.go:223]   Errors: [measurement call TestMetrics - TestMetrics error: [action gather failed for EtcdMetrics measurement: unexpected error (code: 0) in ssh connection to master:
&errors.errorString{s:"error getting signer for provider aws: 'error reading SSH key /Users/jiaxin/.ssh: 'read /Users/jiaxin/.ssh: is a directory''"}]]

The problem is there's a bug that a new variable keyFile is recreated under case scope. It doesn't change value of keyFile defined earlier.
https://github.com/kubernetes/perf-tests/blob/5fb41ba7134696e897ad416289de0041141906b8/clusterloader2/pkg/measurement/util/ssh.go#L97
When it tries to check if argument is an absolute path,
https://github.com/kubernetes/perf-tests/blob/5fb41ba7134696e897ad416289de0041141906b8/clusterloader2/pkg/measurement/util/ssh.go#L117
it reads value which is empty and then it create $HOME/.ssh + '' as the new keyFile.
https://github.com/kubernetes/perf-tests/blob/5fb41ba7134696e897ad416289de0041141906b8/clusterloader2/pkg/measurement/util/ssh.go#L84
What you expected to happen:
keyFile assignment should work properly.
How to reproduce it (as minimally and precisely as possible):
Follow above command I attached.
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Network plugin and version (if this is a network-related bug):
Others:

release-1.19 has the fix now. I will cherry-pick changes to rest of the active branches.

/assign

This can be resolved once 1.18 cherry-pick is merged


Cartoview News App installation error

tried to install News app and the server went totally down !

Issue moved to cartologic/cartoview_news #1 via ZenHub


Time picker have to click OK 2 times

I just downloaded it today to try out - very nice - EXCEPT clicking OK on time picker is required TWICE to close the picker??  Why is that?

Because you need to select first hours and then minutes.
You can set switchOnClick to true during initialisation. After selecting hours/mintues it automatically continues so you don't have to click ok


Use .hidden on checkbox and radio toggle elements #987

Fixes issue #987
checkbox and radio remove both .hide and .hidden but only add .hidden.

From Bootstrap's docs:

.hide is available, but it does not always affect screen readers and is deprecated as of v3.0.1. Use .hidden or .sr-only instead.

This change updates Fuel UX to be compatible with this standard.

More is needed to bring us into compliance here, but I've created Issue #1002 to address these other occurrences.


fix(versions): update Activiti/example-runtime-bundle versions into master

UpdateBot pushed maven dependency: org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies to: 7.1.152

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.1.152


Update reactor error handler

Update reactor error handler
Improves the ability to record Reactor errors.

reactor.core.publisher.MonoOnErrorResume
reactor.core.publisher.FluxOnErrorResume
reactor.core.publisher.MonoOnErrorReturn
reactor.core.publisher.FluxOnErrorReturn

# Set whether to trace the onErrorComplete(), onErrorResume(), onErrorMap(), onErrorReturn() methods
profiler.reactor.trace.onError=false

The range of reactor error processing is too wide, adding functions that can only be applied to reactor-netty.
# Set whether to trace the transport error(connection timeout, unknown host, connection refused)
profiler.reactor-netty.client.trace.transport.error=true
# Set whether to trace the http error(response timeout, read/write timeout)
profiler.reactor-netty.client.trace.http.error=true

Applied to version 2.6.0.


[WIP] Eth 2.0 reference tests

PR Description
Adds the reference tests as a submodule and implement a test suite for BLS signatures from the reference files
Fixed Issue(s)

Fixes #23

Wow! This is super 🎉
I'll gladly fill out the actual BLS tests once this is merged (and BLS is finished...). There are some subtleties: (1) the tests are out of date with the spec, and (2) there is an issue with some of them that I've worked around for now. Since all this is fresh in my mind it would make sense for me to do this. You could raise an issue for it and assign to me.
I hard-coded some of the tests for now, and it was laborious, so this is welcome 😃

@benjaminion Sure, that makes sense! I'll clean it up [after I get home tonight] and ensure the right objects are being passed as arguments to the parameterised tests and remove the WIP so it can get merged. I'll leave the actual tests for you.

@benjaminion - Arguments are fixed - you can take over to implement the BLS tests. Jenkins is failing - it's not cloning with --recursive or calling submodule update - I'm not sure how to fix. Let me know if there's anything else I can do to help.

@vitorpy I recommend you drop a question in the Gitter about the Jenkins failure (I'm not your guy to sort that one out). Since this PR is still marked WIP I guess people are not keeping a close eye on it.

@vitorpy I will review once you are feel like you are done (just remove the WIP text from the title so I know to review).  thanks for all your hard work!!!

As per earlier conversation with @benjaminion , I'm leaving the BLS tests for him. I'll add a second PR for the shuffling test suite.


Add Undo command

Allows user to undo the most recent change from commands that modifies the address book (mainly add, delete, edit, clear).
Calling undo again right after an undo command will throw error message to result box (don't have time to implement addressBookHistory like the commandHistory). I also don't have redo command (granted, they can literally just scroll up to the past command and just press enter).
Let's say the user successfully enters delete 1 and then enters help, list, find .... Then entering undo will still work by undoing delete 1.

Will update UG if the feature is approved.


[BUG] Does not install

Describe the bug
neither the zip not the msi install
the zip fails with
"Unable to locate application file 'VAICOM PRO for DCS World.msi'.
See the setup log file located at 'C:\Users\dream\AppData\Local\Temp\VSD44E4.tmp\install.log' for more information."
D
the msi  fails with :
unable to write to D:\Program Files\VoiceAttack\Apps\VAICOMPRO
install.log

what does seem to work is manually copying the exe and  dlls to the directory having extracted the m from the zip file

Let me know if you need any data

We are unable to reproduce this and have only seen 2 reports, one was resolved due to a conflict with a background process on the users machine. Leaving upen for awareness.

Closing as we cannot replicate.


Prettify filenames, JSON, fix errors

In particular, Kim Mears was not showing up (#7), and there was a .json~ file.

This is great. Thanks a lot!


(PUP-5484) Include Cumulus platform in mount util helpers

@LuvCurves looks good. Not sure what's up with appveyor, but it doesn't matter for this acceptance test. Is this good to merge?

@joshcooper Yes, it's good to merge.  I've tested this locally.


Missing space in documentation text

Need a space between words 'Key' and 'dialog' on the following page https://learn.microsoft.com/en-us/power-apps/developer/data-platform/tutorial-write-plug-in#sign-the-plug-in

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

ID: ad2eb047-525a-63b7-2d6e-1536a374d688
Version Independent ID: 2b8e3e9d-c957-d43e-ceeb-c75da0857d2d
Content: Tutorial: Write and register a plug-in (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/tutorial-write-plug-in.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @divkamath
Microsoft Alias: jdaly

Thanks for your comment, @bambooch. This issue has been assigned to the doc author for review.

@bambooch Fixed. Thanks.


Fix z-stream release channel to stable-v1.11

CPaaS has a gated check where operator cannot
be released on a channel that already has a higher
version as its head. This is a follow up for v1.11.5 (Rapid reset CVE fix).

Wait for https://github.com/openshift/release/pull/44697

/hold

/cc @TrilokGeer

/test e2e-operator

/cc @lunarwhite @xingxingxia
for qe-approved label

/label docs-approved

/cc @davemulford
for px-approved label

/label px-approved

@thejasn @lunarwhite , understood the gate check issue, but not sure whether this fix will make new v1.11.x not shown in stable-v1 any more? Before, it shows.

Tried to pre-merge examine my above question with cluster-bot, CC @lunarwhite about this method:
catalog build openshift/cert-manager-operator#158 cert-manager-operator-bundle

Then used the catalog registry.build05.xx.xxxxxxxxx.org/ci-ln-xxxxxxx/pipeline:catalog from it. Got channel ci-chat-bot only as below screenshot, no stable-v1.11 channel, I think this is a config problem @thejasn @swghosh WDTY?

$ oc get packagemanifest -n openshift-marketplace -o yaml -l catalog=cm-pre-merge-158
        version: 1.11.5
      entries:
      - name: cert-manager-operator.v1.11.5
        version: 1.11.5
      name: ci-chat-bot
    defaultChannel: ci-chat-bot
    packageName: cert-manager-operator

Tried to pre-merge examine my above question with cluster-bot, CC @lunarwhite about this method:
catalog build openshift/cert-manager-operator#158 cert-manager-operator-bundle

Then used the catalog registry.build05.xx.xxxxxxxxx.org/ci-ln-xxxxxxx/pipeline:catalog from it. Got channel ci-chat-bot only as below screenshot, no stable-v1.11 channel, I think this is a config problem @thejasn @swghosh WDTY?
$ oc get packagemanifest -n openshift-marketplace -o yaml -l catalog=cm-pre-merge-158
        version: 1.11.5
      entries:
      - name: cert-manager-operator.v1.11.5
        version: 1.11.5
      name: ci-chat-bot
    defaultChannel: ci-chat-bot
    packageName: cert-manager-operator

The channel name is from cluster-bot. It does not read the channel name from the ci job config or the CSV, instead has a hard-coded default value. (ref)

@thejasn @lunarwhite , understood the gate check issue, but not sure whether this fix will make new v1.11.x not shown in stable-v1 any more? Before, it shows.

Yes, post this fix v1.11.5 will not show up on stable-v1 and any new v1.11.z+1 also will not show up on the stable-v1 channel anymore. This release's upgrade graph can be tested using opm alpha render-graph -p cert-manager-operator <opm-index-image-build-on-top-of-4.13-catalog-index>.

/assign @swghosh

/remove-hold
as https://github.com/openshift/release/pull/44697 is merged

/lgtm

/retitle [cert-manager-1.11] Fix z-stream release channel to stable-v1.11


Main page has broken links

Here is the main page:
https://huggingface.co/learn/audio-course/chapter0/introduction
It has internal links to units like this:
https://huggingface.co/learn/audio-course/chapter1
which end up at :
https://huggingface.co/docs/audio-course/main/en/chapter1
With a 404 error.
On the left menu bar, we see working links with:
https://huggingface.co/learn/audio-course/chapter1/introduction

i think this was an internal tooling (doc-building) issue but it should be fixed now?


JsSIP(0.7.4) - deprecated 'addStream()'

Good afternoon, I am developing a project in which I followed your repository, and when validating the audio, I received an error stating that there is a deprecated function 'addStream()' in the jsSIP version (0.7.4). Is there any possibility of providing an update or sending an example to better understand the functionality?

This library is out of date and deprecated. Please contact<EMAIL_ADDRESS>for the newer library version.


2.0.0 Modal content shift to right with container and scrollbar

Bug description:
Hello since the version 2.0.0, when i have a container with enough content to display a scrollbar and i click on a button to open a modal the content is shift to the right.
Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/9QQME0SJOiJ24v4I7dYk
Version of Angular, UIBS, and Bootstrap
Angular: 1.5.5
UIBS: 2.0.0
Bootstrap: 3.3.6

@Megaboot, please be respectful of our time and processes by searching both open and closed issues prior to entering any new issue.  This issue has been addressed a myriad of times and is most likely a CSS issue on your end.

@icfantv I'm sorry i can't find an issue related to me, i found https://github.com/angular-ui/bootstrap/issues/5749 which is similar to mine but with a fixed navbar. But we can also reproduce the issue with the plunk that @RobJacobs gave in is last comment if you update ui bootstrap to 2.0.0, because in the navbar it contains a container class.
I can't either find a documentation that speak about this breaking change because it was working fine in 1.3.3.


Issue on Windows - error: Failed to read C:/Program: open C:/Program: The system cannot find the file specified.. Exiting.

I'm getting this issue when trying to run Puppet on Windows:
Error: Execution of 'c:\Program Files\Filebeat\filebeat.exe -N -configtest -c C:/Program Files/Filebeat/conf.d/sccm_logs
.yml20170523-3620-lmle23' returned 1: Loading config file error: Failed to read C:/Program: open C:/Program: The system
cannot find the file specified.. Exiting.
Error: /Stage[main]/Profile::Sccm/Filebeat::Prospector[sccm_logs]/File[filebeat-sccm_logs]/content: change from {md5}f34
d4cf4c4b0610b984d6eb76ec292bf to {md5}76e01183a3974ae213b624ee870ade1c failed: Execution of 'c:\Program Files\Filebeat\f
ilebeat.exe -N -configtest -c C:/Program Files/Filebeat/conf.d/sccm_logs.yml20170523-3620-lmle23' returned 1: Loading co
nfig file error: Failed to read C:/Program: open C:/Program: The system cannot find the file specified.. Exiting.
It seems like the command should be ran with quotes around it because the space between Program and Files is breaking the command apart. Is this an issue with the module?

Which version of the module are you using? I thought that we had fixed this in version .0.10.1 as part of fixing #75.

I was using .0.10.0 - I'll try a newer version. Thank you!


Fix web backend to redirect DKG setup requests

I am absolutely not sure if this is correctly written !
But currently DKG setup requests are not redirected, they use the default proxy, the idea is to redirect them to the correct proxy (the one provided in the input json).
As the regex used to check if it's an initialisation request does not match the setup i.e const regex = /\/evoting\/services\/dkg\/actors$/ does not match with the setup endpoint: /api/evoting/services/dkg/actors/${ElectionID}

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 62.573%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
3103

Relevant Lines:
4959

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

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 62.533%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3101

Relevant Lines:
4959

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

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 62.533%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3101

Relevant Lines:
4959

💛  - Coveralls


Add ngrok usage to README

We provide the way to use ngrok when running Agora as a Validator on a home server.
After this, I will add the instructions for the Korean README.
Fixes #3241

Updated and applied it to the Korean README.
Will merge this PR with @hewison-chris's approval as I have checked the content with a spell check program once more.


tptp: 6.1.0 -> 6.3.0, and get rid of builderDefsPackage

TPTP contains one binary executable for 64-bit linux, but all other scripts (and the gigabytes of problem files) should work fine on other platforms, hence I've set platforms to platforms.all.
cc maintainer @7c6f434c

Thanks! We're down to 13 calls of builderDefsPackage.


Create git tag when running make-version script

This clears the way for automated changelog diffs. See #262.
An enhancement for this workflow could be to let make-version accept an argument that matches the npm version api -- namely patch, minor, or major.
We could then call npm version ${arg}, which would increment the package.json version, and create both a matching git tag and version commit.
Omitting the arg could just revert to the functionality covered by this PR. Let me know what you think.
For example:
$ npm run make-version -- patch
$ npm run make-version -- minor
$ npm run make-version -- major
$ npm run make-version

This change is

 Thanks!

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


(TEPHRA-117) Added the TransactionContext.removeTransactionAware method

Removal is only allowed if there is no active transaction

This is the prevent the TransactionAware being left in some unknown state

Two minor comments, otherwise looks good.

+1


Current track controls

Hello!
Sometimes (for various reasons), it is desirable to pause the batch operation (i.e. to not be bombarded by FL repeatedly opening up). Other times, there might be a desire to halt or postpone the rendering of a track (i.e. to fix an issue where there is an infinite loop in the rendering). It would be nice, therefore, if there were controls inside the app UI for pausing/resuming the operation, and also for controlling the current track (like described above).

Hey there! A function to pause is now fully implemented, as well as the option to quickly edit a project in Fruity Loops via context menu. Happy rendering!


Missing MEILI_SERVER_PROVIDER env var in the instance-options specification

The instance options specification file should mentions the MEILI_SERVER_PROVIDER env var
The telemetry policies specification file contains a typo -> system.server_pro**diver**

Closing this since this repo will be archived, so not relevant anymore


Cleanup codebase, create v1

Thanks for a great port of the Github library.
I'm offering this PR to cleanup a few things in the codebase, to make them a bit more friendly to what a node.js dev would expect in 2016. Here's a summary of the changes:

parse_reply renamed to parse.
String.prototype is no longer augmented, instead we use local functions for performing trim, chomp, etc.
The module is exported directly, meaning you get the whole module when you require('emailreplyparser').
Test dependencies added to package.json to make testing easier.
underscore replaced with lodash in the tests.
Code style cleaned up: use single quotes everywhere, add 'use strict' definitions, use triple equals where necessary.

I hope this is acceptable, if not then I'm happy to maintain my fork and publish to npm under a different name.

@fiznool Did you fork and publish to npm under a different name?

@CTassisF the changes are still in my fork at https://github.com/flowxo/emailreplyparser
I didn't publish anything to npm yet. If @mko doesn't reply in the next few weeks I'll look to do that. In the meantime you can depend on my fork directly. In your package.json:
"emailreplyparser": "github:flowxo/emailreplyparser#0834875


Does anyone have a list of the fingerprints for the public keys?

So I can see the public keys here, and also what looks like a hash, are these SHA1 hashes of the PEM or something? Or are they the SHA1 hashes of the serialized public keys?
https://github.com/tdlib/td/blob/d2b441f812b632baf244f701a3e45cb9c9f6f4ba/td/telegram/net/PublicRsaKeyShared.cpp#L56
I'm just looking for a precomputed list of the fingerprints for these public keys

This isn't a fingerprint. The fingerprint is calculated there: https://github.com/tdlib/td/blob/d2b441f812b632baf244f701a3e45cb9c9f6f4ba/td/mtproto/crypto.cpp#L78.


B024: ruff in pre-commit fails to recognise B024 with the presence of a class variable

List of keywords I searched for before creating this issue

B024

A minimal code snippet that reproduces the bug
from abc import ABC

class Foo(ABC):
    a: int
    def method(self) -> None:
        print("nothing")

The command you invoked
poetry run pre-commit run -a

The current Ruff settings
Relevant snippet of .pre-commit-config.yaml
  - repo: https://github.com/astral-sh/ruff/
    rev: 0.7.4
    hooks:
      # Run the linter.
      - id: ruff
        args:
          - --fix
          - --exit-non-zero-on-fix

Relevant snippet of pyproject.py
[tool.ruff]
line-length = 88
force-exclude = true # Recommended for pre-commit (https://github.com/charliermarsh/ruff#force-exclude)
# Exclude a variety of commonly ignored directories.
exclude = [
    ".bzr",
    ".direnv",
    ".eggs",
    ".git",
    ".git-rewrite",
    ".hg",
    ".mypy_cache",
    ".nox",
    ".pants.d",
    ".pytype",
    ".ruff_cache",
    ".svn",
    ".tox",
    ".venv",
    "__pypackages__",
    "_build",
    "buck-out",
    "build",
    "dist",
    "node_modules",
    "venv",
]
target-version = "py310"
# unsafe-fixes = true

[tool.ruff.lint.per-file-ignores]
"*ipynb*" = [
  "B018",  # https://docs.astral.sh/ruff/rules/useless-expression/
]

[tool.ruff.lint]
# Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`)  codes by default.
select = ["ALL"]
ignore = [
  "ANN001",  # https://docs.astral.sh/ruff/rules/missing-type-function-argument/
  "ANN201",  # https://docs.astral.sh/ruff/rules/missing-return-type-undocumented-public-function/
  "ANN202",  # https://docs.astral.sh/ruff/rules/missing-return-type-private-function/
  "ANN401",  # https://docs.astral.sh/ruff/rules/any-type/
  "ARG",     # https://docs.astral.sh/ruff/rules/#flake8-unused-arguments-arg
  "B007",    # https://docs.astral.sh/ruff/rules/unused-loop-control-variable/
  "B023",    # https://docs.astral.sh/ruff/rules/function-uses-loop-variable/
  "B905",    # https://docs.astral.sh/ruff/rules/zip-without-explicit-strict/
  "C408",    # https://docs.astral.sh/ruff/rules/unnecessary-collection-call/
  "C901",    # https://docs.astral.sh/ruff/rules/complex-structure/
  "D",       # https://docs.astral.sh/ruff/rules/#pydocstyle-d
  "DTZ001",  # https://docs.astral.sh/ruff/rules/call-datetime-without-tzinfo/
  "E501",    # https://docs.astral.sh/ruff/rules/line-too-long/ (handled by black)
  "E722",    # https://docs.astral.sh/ruff/rules/bare-except/
  "E731",    # https://docs.astral.sh/ruff/rules/lambda-assignment/
  "F841",    # https://docs.astral.sh/ruff/rules/unused-variable/
  "G",       # https://docs.astral.sh/ruff/rules/#flake8-logging-format-g
  "S101",    # https://docs.astral.sh/ruff/rules/assert/
  "S301",    # https://docs.astral.sh/ruff/rules/suspicious-pickle-usage/
  "S608",    # https://docs.astral.sh/ruff/rules/hardcoded-sql-expression/
  "SIM114",  # https://docs.astral.sh/ruff/rules/if-with-same-arms/
  "BLE", "COM", "EM", "ERA", "FIX", "FBT", "FLY", "INP",
  "N",
  "PD",
  "PERF", "PGH",
  "PLC", "PLR", "PLW",
  "PT",
  "PTH",
  "PYI", "RET",
  "SLF",
  "T201",
  "TCH", "TD", "TID", "TRY", "UP", "W"
]

The current Ruff version
0.7.4

Thank you for submitting this issue!
This behavior is intentional - it is assumed that an unassigned, annotated class variable is intended to be abstract.
But that has certainly been a point of discussion/refinement. See this flake8-bugbear issue and the issues linked therein. Let us know if you think it should be further refined!

Hi, thank you for the reply.
In such a case, the documentation needs to be updated. For example, in abstract-base-class-without-abstract-method, instead of

Checks for abstract classes without abstract methods.

we can write

Checks for abstract classes without abstract methods or class variables.

Please let me know if I can continue completing and implementing this idea.

That'd be great! I think the check only skips if the class variable is annotated and not assigned, though. So you may want to adjust the wording a bit, maybe:

Checks for abstract classes without abstract methods. Class variables which are annotated but not assigned are regarded as abstract.

I am working on the documentation. Meanwhile, upon checking the Python documentation, it turns out that only when subscribed by typing.ClassVar is an annotate variable considered a class variable. Copied from the above-mentioned documentation:
class Starship:
    stats: ClassVar[dict[str, int]] = {} # class variable
    damage: int = 10                     # instance variable

What I had as an example in the original post is therefore an instance variable, not a class variable. This contradicts the discussion in https://github.com/PyCQA/flake8-bugbear/issues/293, which only has to do with typing.ClassVar.

Hey @cmp0xff  - you'll want to open a PR for ruff not docs. The relevant code is here:
https://github.com/astral-sh/ruff/blob/3c52d2d1bd8042d98415f70ecc8d6402795756ef/crates/ruff_linter/src/rules/flake8_bugbear/rules/abstract_base_class.rs#L13C1-L19C67

Meanwhile, upon checking the Python documentation, it turns out that only when subscribed by typing.ClassVar is an annotate variable considered a class variable.

That's an interesting point... can you open a separate issue for this? For the present issue let's just change the documentation.
Thank you!


DRILL-7250: Query with CTE fails when its name matches to the table name without access

Removed catalogReader.getTable(tempNode.names) call since it does not resolve table names correctly using provided scope.
Reworked catalogReader.isValidSchema() method to follow logic on how Calcite determines whether the table is present: it uses schemas from CatalogReader and resolves specified schema among them.
Updated Calcite version to use the fix for CALCITE-3061.
Added unit test.

For problem description please see DRILL-7250.

+1, LGTM.


css paths to fonts and bower

Personally, I only ever use bower to source files that I'll package in a taskrunner (I'm using Grunt at present) to spit out packaged files. The ../../fonts reference in the CSS files destroys this for me because my scripts get served from /css so the path to the fonts ends up being invalid. This could be easily addressed for both the current users and those like myself by either dropping the css/** subfolders (just flatten file names and you keep plenty of organization though I could imagine that makes the less/sass jobs a bit more tedious) OR (and I don't love this, but it would work) putting the font folder inside the css folder and in either case, update the paths to use only ../fonts
Essentially, I'm asking you to consider updating your file structure to accommodate what I think is a typical pattern of serving css from /css and fonts from /fonts OR css from /css and fonts from /css/fonts.
Something you'd consider doing? Would you take a PR to do this (updating any less/sass jobs as well of course).

BTW, thank you for this project :)

The same problem with ../../fonts structure. It can't find correct path from npm modules. Like example, structure can be similar material-design-icons module.
For now i mix local and network fonts:
@import url(~material-design-icons/iconfont/material-icons.css);
@import url(https://fonts.googleapis.com/css?family=Roboto:400,300);

I'm not sure if I understand the problem, but maybe the following could be helpful for you?
I'm using SCSS to import the Roboto fontface to my project. I had to set the $roboto-font-path variable, else the fonts path in the generated CSS does not work in my project.
$roboto-font-path: "../lib/roboto-fontface/fonts";

@choffmeister:
Maybe it would be useful to mention the $roboto-font-path variable in the README?

@rainabba Sorry for the delay, I am very distracted currently. I do not plan to change the folder structure. The relative path ../../../fonts is correct as long as one serves the files as they are. If you relocate files, then you can just change the path variable, as @fraulyoner pointed out.
@fraulyoner Yes adding the hint for the font-path variables to the README is a good idea. Can you provide a PR?

No worries and thank you. The font-path idea mentioned by @fraulyoner only applies is one is processing CSS through an SCSS processor right? I don't presently use one and only know a little about them so I want to confirm what you guys might be taking for granted in that answer.

Will close this for now, since this is not a bug in this repo.

With SASS
$roboto-font-path: '~roboto-fontface/fonts';
@import '~roboto-fontface/css/roboto/sass/roboto-fontface.scss';


Update to accept genetic code 25

Update the list of genetic codes to match NCBI. Also, recognize genetic code 25 as bacterial. It is recognized by both Prokka and Prodigal. It needs to be updated here so results can be saved.
@scanon, @sychan

Part of an update to Prodigal so RAST can better support MAGs. Prodigal can be more flexible than the current version allows.


com.jayway.jsonpath.internal.JsonContext#renameKey(java.lang.String, java.lang.String, java.lang.String, com.jayway.jsonpath.Predicate...) doesn't rename all keys

For JsonContext created from:
{
  "oldName": {
    "oldName": {
      "oldName": {
        "otherName": "value"
      }
    }
  }
}

after calling the method:
com.jayway.jsonpath.internal.JsonContext#renameKey(java.lang.String, java.lang.String, java.lang.String, com.jayway.jsonpath.Predicate...) with parameters:

jsonPath == "$..[?(@.oldName)]"
oldKeyName == "oldName"
newKeyName == "newName"

we have result:
{
  "newName": {
    "newName": {
      "oldName": {
        "otherName": "value"
      }
    }
  }
}

with it seems to be wrong. One rename from "oldName" to "newName" was skipped.

@tgimbut #587 will fix your issue, please try it on :)


Implements s3 exporter

This PR implements S3 exporter using the already existing s3 encode stage.
Here is a sample of generated files in the bucket:
sample.txt
Rely on https://github.com/netobserv/flowlogs-pipeline/pull/442

/hold

Closing this as it's quite outdated


Sharing & Rearrange Slides

I am requesting two features.

I would like to rearrange the slides.
Often when making a plan, I end up having to delete slides to be able to rearrange them in the order I want. It would be nice to drag or re-order them in any way.

Sharing Issues

2.1 When sharing, the link is too long for discord. Discord is heavily used by raiding groups to communicate, so it would be pleasant to directly link raid plans on there. Right now, links have to be posted as text files.
2.2 When I share a raid plan, the link I give goes to a static version of the plan, with no way to update the plan without giving a new link.
It would be nice to have some way to give one link and have that show the latest version of the plan.
If this is not possible, then is it possible to label these plans with versions because right now it is hard to compare to see if the person is viewing an up-to-date plan or an old version.
Thanks for the work put into this project.

Rearranging would be possible to implement, but I just need to come up with a good UI for it first. My current best idea is a dialog window that shows thumbnails of each step (like the arena presets window) and lets you drag and drop, then confirm so it all ends up as one change in the undo order.
The reason the link is so long and goes to a static version of the plan is because the link is the plan. All of the data for the plan is encoded within the link. I don't store any data, nor do I want to store data (because then I have to pay for storage, maybe deal with GDPR, etc.). If you check the other issues here, you'll see some discussions on ways we might be able to use 3rd party sites for storage, but I've been spending my free time on other things recently and haven't taken the time to figure that out yet.
Current workarounds for sharing issues:

The smaller your plan is, the more likely it is to fit in a Discord link. If you're currently making one plan for a whole fight, try breaking it into separate plans per mechanic.
Stick a text field onto the first step and write a version number there.

I see, for the slide ui I think the ui for raidplan.io is solid as well.
They just have horizontal arrows the move the currently selected slide either 1 back or 1 forward.
It's crude but works extremely well.
As for the link, I understand. Np.
Lastly I think I read you saying that you prog blind. If so when selecting arenas/maps it should not show all phases. It should only show 1 at a time, so a player can not see how many phases there are.
Anyway Ty for all the good work.

I see, for the slide ui I think the ui for raidplan.io is solid as well.
They just have horizontal arrows the move the currently selected slide either 1 back or 1 forward.
It's crude but works extremely well.

That was my first idea, but I think it would be hard to tell what you're doing and would potentially insert many unnecessary steps into the undo order.

Lastly I think I read you saying that you prog blind. If so when selecting arenas/maps it should not show all phases. It should only show 1 at a time, so a player can not see how many phases there are.

So far I've just been assuming that everyone else is faster than my group. 🙂 I'll have to think of whether there's any good way to do that. I don't want to only show one thing at a time, because then if you have gotten to a particular phase, that makes it take longer to set up a new plan for that phase.
I could set it up so you have to tell the app how far you've gotten, and it only shows arena presets up to that point, but then how do I let you choose that without giving away how many phases exist? My best idea so far is just "next fight" and "next phase" buttons for each raid tier, but even that is tricky. For example, if I only show the buttons when there are still phases left, then you'll know when you're on the last phase (that I've made a preset for).
A simpler idea would be to give each preset a more generic name that doesn't indicate a particular phase and blur each one out until you've clicked on it once. That would still give you some idea of how many variations in the arena occur in a fight, but it wouldn't let you see what they look like until you choose to look.

I've implemented rearranging slides and hiding potential spoiler info in arena presets. I think the only remaining request here is for a way to have a single link that contains an up-to-date version of the plan, which would be covered by #6 since I have no plans to start storing any data myself.
Feel free to create a new issue if there's any other unresolved part of this that I missed.


self.users.update in slackclient.py crashes on TypeError

After running for a while, slackbot will inevitably crash with this error.
Traceback (most recent call last):
  File "./main.py", line 27, in <module>
    main()
  File "./main.py", line 21, in main
    slack.Bot().run()
  File "/home/pi/slackbot/venv/lib/python3.4/site-packages/slackbot/bot.py", line 37, in run
    self._dispatcher.loop()
  File "/home/pi/slackbot/venv/lib/python3.4/site-packages/slackbot/dispatcher.py", line 151, in loop
    self._client.parse_user_data(user)
  File "/home/pi/slackbot/venv/lib/python3.4/site-packages/slackbot/slackclient.py", line 78, in parse_user_data
    self.users.update({u['id']: u for u in user_data})
  File "/home/pi/slackbot/venv/lib/python3.4/site-packages/slackbot/slackclient.py", line 78, in <dictcomp>
    self.users.update({u['id']: u for u in user_data})
TypeError: 'NoneType' object is not subscriptable

Some system information:
$ python --version
Python 3.4.2
$ uname -a
Linux hostname 4.9.35-v7+ #1014 SMP Fri Jun 30 14:47:43 BST 2017 armv7l GNU/Linux
$ cat /etc/issue
Raspbian GNU/Linux 8 \n \l

It seems to me that when iterating through the user_data list, it is possible to come across a None object in the list, and that trying to access the 'id' key of the (what is presumed, falsely to be a) dict results in a TypeError since the element in user_data is not in fact a dict.
A naïve fix for this would be to change the line to read:
self.users.update({u['id']: u for u in user_data if u is not None})

but I expect that there might be other code that expects user_data to not have a None in it either, and so there needs to be better filtering on elements going into the user_data list in order to enforce that only proper/expected objects make their way in there.
Let me know if you want any more info, etc.
Also, thanks for making this! It's much better than the code that slack themselves put out for working with the RTM API.

Suggested label: Bug


Optimise perm filter

Replaced looping through array of words of a certain length with running a single includes or regExp.test on a single string that has all words 15 chars or less in it
Dramatically reduced cpu time for filtering permutations for joker and non joker hands

Nice work


Make it work with strict variables

Make sure all vars in params.pp are assigned values (even if just to undef) to the module works under strict_variables=true.
Fixes #76

thank you @robinbowes!


[BUG] Cannot find save handler 'redis'

Describe the bug
No connection to redis
which Branch / PHP Version are you using ?
php74
I tried
my php.ini config
session.save_handler = redis
session.save_path = "tcp://redis:6379"

or

session.save_path = "tcp://lamp-redis:6379"

or

session.save_path = "tcp://<IP_ADDRESS>:6379"

Im not able to reproduce this. Is this still an issue with the latest code base and a clean setup ?


return 200 OK with empty result if searching not found

Closing temporarily. Need to add few more things. Will reopen once ready


Possible to cause infinite loop?

I was just looking at the latest properties for Options and it seems it could infinite loop if both iteration_count and iterations_without_improvement are None. Sure, you would be silly to deliberately set them both to None, but I'm concerned it could quite easily happen unintentionally if you did something like this:
Options {
    iteration_count: NonZeroU64::new(get_value_from_args()),
    ..Default::default()
}

If get_value_from_args() returned 0 then you might have a problem.
Previously iteration_count wasn't an option so you were required to unwrap the NonZero. I think it was safer like this - you could always set it to NonZeroU64::MAX if you wanted to rely solely on iterations_without_improvement.

I think your analysis of the situation is spot on. Client code resorting to NonZeroU64::new with variable input can be a footgun I didn't realize when this feature was merged, and while such a mistake would be on the user's end, the fact that it is easy for client code to make this mistake suggests that a better API could help.
I think that dropping the Option here is a good way to go. For practical purposes, there is no difference between 264 - 1 iterations and potentially infinite iterations. In fact, I think it'd be good to drop the Option from both, so it is not possible to do too few iterations without improvement if a NonZero is constructed from a zero iterations_without_improvement number.


Replacing latest_release with current_path

Solving https://github.com/everzet/capifony/issues/551
I've written some tests also, but I'm unable to pass test at line 41

@everzet ping

Well, if tests don't pass, no chance to get this PR merged :/

@willdurand I'm not a Ruby expert, I'm trying to write ruby for first time. Please give me tiny help.
How to test that
put result, "#{current_path}/#{web_path}/#{maintenance_basename}.html", :mode => 0644

is called and file has been created?
Tried for File.exists? but it checks for file on local server, not the remote one.

@everzet @willdurand Finaly found a working solution. What now?

great! squash your commits and I'll be happy to merge your PR :)

@willdurand

thanks!


Register For the Hackweb3

added my repo file
Shivansh-firstgame
Please check and merge My PR and give me the badge of HACKWEB3

@aslams2020 Please check my PR and Merge it under the Gssoc _EXT and Hackweb3 label

@MastanSayyad @sanjay-kv @aslams2020 please check my PR and merge it under gssoc-ext and hackweb3 label

@Snehsk  please review and merge it

@Snehsk I change the name of the screenshot


Curation task results are not included in the output file or process output

Describe the bug
When running a curation task via the DSpace 7.0 UI, the output generated by the task process is being written to the dspace.log but not being captured in either the provided Output File or the listed Process Output. The only purpose of some curation tasks is to perform an action and provide a result. If the user cannot see the result, there is little point in allowing them to perform the task in the first place.
For example, if you run the profileformats (or "Profile Bitstream Formats") Curation Task, the Process Output will provide an output like this, which only indicates that the task ran, but doesn't provide the actual output the user needs:
2021-09-22 16:32:20.166 INFO curate - 57 @ The script has started
2021-09-22 16:32:20.555 INFO curate - 57 @ The script has completed

In contrast, the output captured in the dspace.log looks like this:
2021-09-22 16:32:20,517 INFO  unknown unknown org.dspace.curate.Curator @ Curation task: profileformats performed on: 123456789/5 with status: 0. Result: '     1 (K) Item-specific license agreed to upon submission
     1 (K) Joint Photographic Experts Group/JPEG File Interchange Format (JFIF)
'

To Reproduce
Perform any curation task that provides an output (which is most of them). In all cases you'll see the same two lines in the displayed output. Check the dspace.log to find the expected output.
Expected behavior
The results generated by the Curation Task being executed should be captured and available in the output file and process output.

Hi @tdonohue ,
we, as 4Science, can look at this issue. Feel free to assign it to us.
Thank you

Thanks @corrad82-4s !  Assigning to you and moving to the "in progress" column for 7.4.

this should take 4-8 hours with manual testing. Adding IT/Unit on this part of the code base could be very costly

@abollini : I don't expect IT/Unit tests in this situation, as they don't already exist.  So, manual testing is OK (and this might be best tested manually.  I'll add an estimate of 6hrs to the ticket.

A duplicate ticket to this one was just created in https://github.com/DSpace/DSpace/issues/8440.  Copying in some notes from @KevinVdV on his analysis (in case these notes are useful to the 4Science team):

The fix itself should be fairly simple, in theory it just boils down to changing the log.info on the following places:

https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/curate/Curator.java#L549
https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/curate/Curator.java#L565

We should also log the system.out calls in the output, just search for that in the Curator.java class.
I estimate the work to fix to be around a day we would then:

Ensure that the logs end up in the output
tests that use the "NoOpCurationTask" & verify that the "No operation performed on " is added to the response


Requesting insight to make brush tool work

Hi,
I'm working on making the brush tool from this branch to work.
It's able to export in CVAT format like this
<mask label="blah" occluded="0" source="manual" rle="17, 10, 27, 15, 23, 18, 20, 21, 18, 23, 16, 24, 15, 26, 13, 27, 13, 27, 12, 29, 10, 30, 9, 32, 7, 34, 5, 36, 3, 38, 2, 38, 2, 39, 1, 39, 1, 120, 1, 39, 1, 39, 1, 39, 2, 38, 2, 38, 3, 37, 3, 37, 3, 37, 4, 36, 4, 35, 6, 34, 7, 33, 7, 32, 9, 30, 10, 29, 13, 26, 15, 23, 18, 20, 22, 15, 12" left="79" top="304" width="40" height="40" z_order="0"> </mask>

But it seems like the RLE here, is relative to the left, top, width, height numbers, because when i create datum_annotation RLE, the output is very weird. To test it out, i draw a polygon with the same shape and at the same space as the brush. the locations for the brush mask doesn't match with the location of the polygon.
Can you shed some light on this please? Is there anything that can be done?

Hi @bsekachev,
Thanks for your response!
Could you give a small code snippet or approach or documentation that I could follow?
I'm not able to crack how to convert this particular list along with the 4 border points into realistic points.
If not that, something to see the logic on how to convert that list into PIL image?
Thanks!

@kkmehta03
Have you seen this code from the branch?
static rle2Mask(rle) {
            const [left, top, right, bottom] = rle.slice(-4);
            const width = right - left;
            const height = bottom - top;
            const decoded = Array(width * height).fill(0);
            const latestIdx = rle.length - 4;
            let decodedIdx = 0;
            let value = 0;
            for (let rleCountIdx = 0; rleCountIdx < latestIdx; rleCountIdx += 1) {
                let count = rle[rleCountIdx];
                while (count > 0) {
                    decoded[decodedIdx] = value;
                    decodedIdx++;
                    count--;
                }
                value = Math.abs(value - 1);
            }

            return [decoded, [left, top, right, bottom]];
        }

You can do approximately the same in Python with numpy array and then encode it to datumaro RLE using pycocotools.

yeah but this one isn't a single array right? it still has the left, top, right, bottom array separately

@kkmehta03
Let's imagine that you created this image. It is actually fragment, but, then you can place it to another image (which has the same size as dataset picture). Using left and top coordinates. Right?

Yes, exactly

thank you very much! i'll try this out and comment further