Unable to import as a .csv

Hi Jonathan,
I'm having trouble uploading a csv file. I can't get past the error "File type (application/vnd.ms-excel) is not supported." even when uploading blank .csv files.

Also, I attempted to upload a .txt file and was given a server error:

Best of luck with this project!

Hey Alan,
Thanks for reporting this!
Do you mind sending the file(s) you  were trying to upload to<EMAIL_ADDRESS> Server log is showing something about a 'NULL byte', my guess is the file has binary content.

Hi Jonathan,
I forgot to add, when I download your template csv and try to import that unedited, I get the same file type error as above.
I can send you what I was planning to import. Thanks!

Sent bg readings csv in mail there. Cheers

Hey Alan,
Thanks for sending over the file.  I actually got a different error when uploading it, which was a bug (I wasn't handling mmol/L unit properly during import).  I just pushed a fix for that and your file now works for me in Chrome and Firefox.
Do you mind giving it another shot?  If you're still getting the same error, can you tell me which browser you are using?
Thanks!

Hi Jonathan,
Logged out, logged back in and tried uploading the exact file I sent to you, still getting the file type error.
Firefox 37.0.2, I also tried in chrome awhile ago but was having the same issue.
Thanks!

Tried wiping the cache for your site too just incase, but same issue.

Ah interesting, must be a Windows thing.  I explicitly allowed that file type and pushed the changes, can you give it another shot?
Thanks!

Thanks Jonathan,
I'm now getting a file type error for application/binary for the same file and for when I download and try import the sample file provided.

Wow, that's weird.  I added that content type just now, let's give it one more shot!

That worked, thanks for your time Jonathan!


Addresses issue 37

Simple fix for issue 37

Looks good to me. Tested locally and seems fine, I assume cluster test went fine too?

Yep. Cluster test worked


Default "serverpod create" project tries to connect to the wrong port

I tried creating a default project using serverpod create <podname>, then started the server in docker (it starts on port 8080), then ran the flutter code using flutter run.
The app fails when you try to send the inputted name to the server, with:
flutter: Failed call: example.hello
flutter: SocketException: Connection refused (OS Error: Connection refused, errno = 111),
        address = localhost, port = 56396

The Flutter code tries to connect to port 56396 (or other random port). This happens despite the code stating it is supposed to be connecting to port 8080:
var client = Client('http://localhost:8080/')
  ..connectivityMonitor = FlutterConnectivityMonitor();

@lukehutch I have faced a similar issue. But hope you aren't running this on the mobile platform. If it is in mac, then you need to allow client connection in x_flutter\macos\Runner\DebugProfile.entitlements and also x_flutter\macos\Runner\Release.entitlements.
<plist version="1.0">
<dict>

	<key>com.apple.security.app-sandbox</key>
	<true/>
	...
++      <key>com.apple.security.network.client</key>
++      <true/>

</dict>
</plist>

I tried running both the Linux client and the Web client -- both have the same problem.
The server is not OS-specific, I think?

@lukehutch Have you successfully started the server by running dart bin/main.dart in the server directory?
The way the socket works when it opens a connection to a server is that it also opens a new port for the client, which it uses to receive data. That port will be opened on a random port number and is what you see in your logs. It still tries to connect to the correct port.


Some issues when install RMF on Nvidia Jetson Xavier NX

Describe the bug
When I build rmf_core on nvidia jetson xavier nx and hit some build error on rmf_traffic_ros2.
Workspace information:

Operating system and version: Nvidia SD card image (https://developer.nvidia.com/jetson-nx-developer-kit-sd-card-image)
ROS distribution:foxy
ROS installation method: [from source]
RMF installation method: [from source]
RMF binaries version, if installed from binaries: NA
RMF source versions, if installed from source: master

rmf_demos branch or commit hash: NA
rmf_core branch or commit hash: master
rmf_schedule_visualizaer branch or commit hash: NA
traffic_editor branch or commit hash: NA

Steps to Reproduce

the gcc version is 7.5 which is not suitable for ROS and RMF. The workaround is to install gcc 8 separately.
yaml-cpp version 0.5.2 which fails rmf_traffic_ros2. The workaround is to install yaml-cpp from source separately.
after the workaround 1 and 2, when run "CXX=g++-8 colcon build --cmake-args -DCMAKE_BUILD_TYPE=RELEASE", still hit the build error.

Expected behavior
I expect the build successfully.
Actual behavior
I expect the build successfully but the output error as below:
Starting >>> ament_cmake_catch2
Starting >>> rmf_cmake_uncrustify
Starting >>> rmf_utils
Starting >>> rmf_dispenser_msgs
Starting >>> rmf_fleet_msgs
Starting >>> rmf_traffic_msgs
Finished <<< rmf_cmake_uncrustify [3.50s]
Starting >>> rmf_door_msgs
Finished <<< ament_cmake_catch2 [3.97s]
Starting >>> rmf_lift_msgs
Finished <<< rmf_utils [4.51s]
Starting >>> rmf_traffic
Finished <<< rmf_dispenser_msgs [13.6s]
Starting >>> rmf_task_msgs
Finished <<< rmf_lift_msgs [10.9s]
Starting >>> rmf_ingestor_msgs
Finished <<< rmf_door_msgs [14.3s]
Starting >>> rmf_charger_msgs
Finished <<< rmf_traffic [14.9s]
Starting >>> rmf_battery
Finished <<< rmf_fleet_msgs [21.0s]
Starting >>> rmf_workcell_msgs
Finished <<< rmf_battery [5.35s]
Starting >>> rmf_task
Finished <<< rmf_ingestor_msgs [12.1s]
Finished <<< rmf_charger_msgs [10.7s]
Finished <<< rmf_task [5.58s]
Finished <<< rmf_workcell_msgs [13.1s]
Finished <<< rmf_task_msgs [23.5s]
Finished <<< rmf_traffic_msgs [46.8s]
Starting >>> rmf_traffic_ros2
--- stderr: rmf_traffic_ros2
librmf_traffic_ros2.so: undefined reference to `std::filesystem::__cxx11::path::parent_path() const'
librmf_traffic_ros2.so: undefined reference to `vtable for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::__cxx11::path::_M_split_cmpts()'
librmf_traffic_ros2.so: undefined reference to `typeinfo for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node_data::convert_to_map(std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node::m_amount'
librmf_traffic_ros2.so: undefined reference to `YAML::BadConversion::~BadConversion()'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::absolute(std::filesystem::__cxx11::path const&)'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node_data::push_back(YAML::detail::node&, std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::create_directories(std::filesystem::__cxx11::path const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::status(std::filesystem::__cxx11::path const&)'
collect2: error: ld returned 1 exit status
librmf_traffic_ros2.so: undefined reference to `std::filesystem::__cxx11::path::parent_path() const'
librmf_traffic_ros2.so: undefined reference to `vtable for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::__cxx11::path::_M_split_cmpts()'
librmf_traffic_ros2.so: undefined referencemake[2]: *** [rmf_traffic_schedule] Error 1
 to `typeinfo for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node_data::convert_to_map(std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node::m_amount'
librmf_traffic_ros2.so: undefined reference to `YAML::BadConversion::~BadConversion()'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::absolute(std::filesystem::__cxx11::path const&)'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node_data::push_back(YAML::detail::node&, std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::create_directories(std::filesystem::__cxx11::path const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::status(std::filesystem::__cxx11::path const&)'
make[1]: *** [CMakeFiles/rmf_traffic_schedule.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
CMakeFiles/test_rmf_traffic_ros2.dir/test/unit/test_ParticipantRegistry.cpp.o: In function `____C_A_T_C_H____T_E_S_T____9()':
test_ParticipantRegistry.cpp:(.text+0x1a20): undefined reference to `std::filesystem::__cxx11::path::_M_split_cmpts()'
test_ParticipantRegistry.cpp:(.text+0x1a28): undefined reference to `std::filesystem::status(std::filesystem::__cxx11::path const&)'
CMakeFiles/test_rmf_traffic_ros2.dir/test/unit/test_ParticipantRegistry.cpp.o: In function `YAML::Node YAML::Node::operator[]<char [5]>(char const (&) [5])':
test_ParticipantRegistry.cpp:(.text._ZN4YAML4NodeixIA5_cEES0_RKT_[_ZN4YAML4NodeixIA5_cEES0_RKT_]+0x9f8): undefined reference to `YAML::detail::node_data::convert_to_map(std::shared_ptr<YAML::detail::memory_holder> const&)'
CMakeFiles/test_rmf_traffic_ros2.dir/test/unit/test_ParticipantRegistry.cpp.o: In function `YAML::Node YAML::Node::operator[]<char [10]>(char const (&) [10])':
test_ParticipantRegistry.cpp:(.text._ZN4YAML4NodeixIA10_cEES0_RKT_[_ZN4YAML4NodeixIA10_cEES0_RKT_]+0xcd8): undefined reference to `YAML::detail::node_data::convert_to_map(std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::__cxx11::path::parent_path() const'
librmf_traffic_ros2.so: undefined reference to `vtable for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `typeinfo for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node::m_amount'
librmf_traffic_ros2.so: undefined reference to `YAML::BadConversion::~BadConversion()'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::absolute(std::filesystem::__cxx11::path const&)'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node_data::push_back(YAML::detail::node&, std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystemcollect2: error: ld returned 1 exit status
::create_directories(std::filesystem::__cxx11::path const&)'
collect2: error: ld returned 1 exit status
make[2]: *** [rmf_traffic_blockade] Error 1
make[1]: *** [CMakeFiles/rmf_traffic_blockade.dir/all] Error 2
make[2]: *** [test_rmf_traffic_ros2] Error 1
make[1]: *** [CMakeFiles/test_rmf_traffic_ros2.dir/all] Error 2
librmf_traffic_ros2.so: undefined reference to `std::filesystem::__cxx11::path::parent_path() const'
librmf_traffic_ros2.so: undefined reference to `vtable for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::__cxx11::path::_M_split_cmpts()'
librmf_traffic_ros2.so: undefined reference to `typeinfo for YAML::BadConversion'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node_data::convert_to_map(std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node::m_amount'
librmf_traffic_ros2.so: undefined reference to `YAML::BadConversion::~BadConversion()'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::absolute(std::filesystem::__cxx11::path const&)'
librmf_traffic_ros2.so: undefined reference to `YAML::detail::node_data::push_back(YAML::detail::node&, std::shared_ptr<YAML::detail::memory_holder> const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::create_directories(std::filesystem::__cxx11::path const&)'
librmf_traffic_ros2.so: undefined reference to `std::filesystem::status(std::filesystem::__cxx11::path const&)'
collect2: error: ld returned 1 exit status
make[2]: *** [participant_node] Error 1
make[1]: *** [CMakeFiles/participant_node.dir/all] Error 2
make: *** [all] Error 2
---
Failed   <<< rmf_traffic_ros2 [3.95s, exited with code 2]

Summary: 15 packages finished [53.1s]
  1 package failed: rmf_traffic_ros2
  1 package had stderr output: rmf_traffic_ros2
  2 packages not processed

Screenshots
NA.
Additional context
NA.

There appears to be a linking issue with libyaml-cpp on your platform and possibly std::filesystem as well. Without having access to an nvidia jetson xavier nx, it will be quite difficult for us to identify why those linking issues exist. You'll need to check how libyaml-cpp is installed and whether its cmake config-file (or find-module) on your system is correctly importing its targets.
For reference, the config-file for libyaml-cpp on my platform is located at
/usr/lib/x86_64-linux-gnu/cmake/yaml-cpp/yaml-cpp-config.cmake

Hi @winnerineast
I have a Xavier NX lying around and I tried to compile rmf, here are my findings:

The libyamlcpp-dev package available in the repositories is out of date. I actually had compile errors like ‘class YAML::Node’ has no member named ‘Mark’. I managed to fix that by manually upgrading libyamlcpp-dev and libyaml-cpp to the version on focal (i.e. downloading the .deb packages from here and installing them through dpkg -i). This is not quite an ideal solution but it seems to work, until they upgrade to Ubuntu 20.04.
I managed to reproduce your issues with std::filesystem. It seems g++8 is not linking properly, I installed g++9 through these instructions and that got rid of the linking issue as well.

Basically it seems most of the issues arise from the ecosystem being stuck to Ubuntu 18.04 and should be solved once they provide Ubuntu 20.04 images (or if you wanted you could just upgrade manually).

Since the issue is understood and needs to be corrected on the platform rather than in any RMF package, I'm going to close this ticket.


Add a simple page for evaluations

Closes #237
Code Changes

[x] Add a view which displays evaluations in a table
[ ] Style table to match ethyca look n feel

Steps to Confirm

[ ] Visit page and verify evaluations are served - http://localhost:8080/view/evaluations

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
[ ] Documentation Updated

Description Of Changes

Add a new router endpoint which displays evaluations list
Style this page?

@ThomasLaPiana This PR has a very simple table with some static html which returns our current evaluations. This is what we discussed in planning but right now it is as bare bones as it gets. At the very least we should have some styling on this table. Is there any fides based CSS we could use? For now we could have a static endpoint for a css document

A couple other things I was thinking of adding onto this PR:

Evaluations need a date tied to them
Evaluation id needs to be reworked, something like edbb727a_5853_40b1_9613_c856272962bd_1638886066 is alot
Evaluation details might need a model rework otherwise we would have no choice but to print a string like this "Declaration (Share Political Opinions) of System (customer_data_sharing_system) failed Rule (reject_targeted_marketing) from Policy (primary_privacy_policy)"


Remove hardcoded "cirros" and "cubswin:)" password

Should not depend on the default user credentials used in OpenStack Cirros VMs. Ideally, the user should be prompted for this or a config file should have this.

Fixed via pull request https://github.com/CiscoZeus/don/pull/21


slight tearing on macOS while scrolling

I'm noticing some tearing while scrolling on macOS.
Perhaps this is an SDL issue, since even if the frame rate is low, it shouldn't tear if vsync is enabled. I didn't see anything related to vsync in the lite source code, but I'm not very familiar with SDL. Is vsync enabled by default?
macOS 10.15.7 (19H2), SDL 2.0.12
Any thoughts? Happy to help! 🙂

!! How on earth did you get a macOS version?!!

I can't reproduce this error on macOS 10.14.4
@KnowsCount I'll be putting in a MR to add macOS support soon

@KnowsCount macOS version available here.

cheers mate!

@KnowsCount macOS version available here.

maybe do you can merge lite-macos with the main branch of original lite?


downloadFile returns wrong FilePath with prefix "file: "

Version: 1.1.3
Device: Pixel 6
OS: Android 13
FileDownloader.downloadFile() function returns a File object with wrong file path(includes prefix file:).
This is due to this change. DownloadManager.COLUMN_LOCAL_URI  returns file path including "file://"

Hello @plabon,
Thank you for profiling this issue, I'll check it and upload a fix shortly,
If this stops you please use the version flutter_file_downloader: 1.1.2 unless you need the notifications management feature.
Thank you for helping in making this plugin better

Hello @plabon,
I fixed the issue and uploaded a Prerelease version,
I will be glad to hear from you about your case's results
please update your pub spec.yaml to flutter_file_downloader: ^1.1.4-dev.1 to test it out.
Thanks in advance

Hey @abdallah-odeh,
Thanks for providing a fix quickly. I have tested the dev version and its working fine.
You can release this fix.
Thanks

Hello @plabon,
I will release it to production and let you know, thanks

The version is now published on production flutter_file_downloader: ^1.1.4
please feel free to reopen this issue if you faced any further related issues


[Bug] Dashboard V2 issues

Hey everyone, I've just merged the new dashboard v2 into the main branch, this means anyone using the default dashboardUrl can now test and use the new version without having to do anything.
To use the new version, just navigate to /v2.0/ path in your instance like this https://demo.r2explorer.dev/v2.0/.
This Github Issue is meant to centralize all problems related to the new dashboard, there are definitely a lot of issue and there isn't yet feature parity, but that will hopefully get solved in the coming weeks.
Bug:

[x] Cannot download files
[x] Logo is not loading properly

Feature parity:

[x] Cannot open emails
[x] Cannot rename files

I’ve noticed you can’t drag and drop to upload on V2, and clicking the upload link doesn’t seem to work (it immediately says “1/1 file 100%”) - is this a known issue with a planned fix?

Hey @grstnhbr thanks for reporting the issue, I rarely use drag and drop, and i must have skipped that during test, i will try to push a fix today

Thank you!! It’s super useful to our workflow and we absolutely love this
project!
AJ Gerstenhaber
(512) 797-3101
atxfilmlab.com
On Mon, Mar 18, 2024 at 4:24 AM Gabriel Massadas <
@.***> wrote:

Hey @grstnhbr thanks for reporting the issue, I rarely use drag and drop,
and i must have skipped that during test, i will try to push a fix today
—Reply to this email directly, view it on GitHub, or
DuckDuckGo removed one tracker. More
https://duckduckgo.com/-hGysyZFupsqvAi0nzhKiHW-qw9F3OqfyEC7vdAtR-vuWAHE3eEp3Zzqey9XyPZVHR82JZouUBlSZSYAFi-uuPOKUoxXPOtr3zx2oAbaPFgKqt5GBYqZnJrnbDcwsAyTYmWgAbTCADKdDiZNWgJrXxLQBg0ix2uwgqByd3L_uNK31_PNU_DjYtHeuaBFaYr7W5wRT0TpsHLSyGo0h9-JIxFPXplhojY4j3Ubmwatl50wPLFEaLAkLGVimDgIPOV2cr3rXO7LxAXHU6z4YaT684BfFMMgDPEh8UpCFXxOHCGIwtNLUMD6WO-6A7VchploSSJzm1U2rpRxYvtOXwPRiTH6J_GoaWsjaVhqrKZB41V6dMcP5foBspS_h_Kcmqbj46UhZRSPbemSvYLrVSqzeZMr94vV4X-NtgSqlbK8r1-uHoRI7DdhRLbGSaZW-Q9wNfo3OCVe0elLxz1z3dNNWyz2xA4H55ssD0968gSXFZvuP8mAtdU0OLr0pchRzikDXoAA
Report Spam
https://duckduckgo.com/-hGysyZFupsqvAi0nzhKiHW-qw9F3OqfyEC7vdAtR-vuWAHE3eEp3Zzqey9XyPZVHR82JZouUBlSZSYAFi-uuPOKUoxXPOtr3zx2oAbaPFgKqt5GBYqZnJrnbDcwsAyTYmWgAbTCADKdDiZNWgJrXxLQBg0ix2uwgqByd3L_uNK31_PNU_DjYtHeuaBFaYr7W5wRT0TpsHLSyGo0h9-JIxFPXplhojY4j3Ubmwatl50wPLFEaLAkLGVimDgIPOV2cr3rXO7LxAXHU6z4YaT684BfFMMgDPEh8UpCFXxOHCGIwtNLUMD6WO-6A7VchploSSJzm1U2rpRxYvtOXwPRiTH6J_GoaWsjaVhqrKZB41V6dMcP5foBspS_h_Kcmqbj46UhZRSPbemSvYLrVSqzeZMr94vV4X-NtgSqlbK8r1-uHoRI7DdhRLbGSaZW-Q9wNfo3OCVe0elLxz1z3dNNWyz2xA4H55ssD0968gSXFZvuP8mAtdU0OLr0pchRzikDXoAA
Hey @grstnhbr https://github.com/grstnhbr thanks for reporting the
issue, I rarely use drag and drop, and i must have skipped that during
test, i will try to push a fix today
—
Reply to this email directly, view it on GitHub
https://github.com/G4brym/R2-Explorer/issues/58#issuecomment-2003311410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ3JOYPVBQIK2WIHQS3PSWTYY2XELAVCNFSM6AAAAABB2NATDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBTGMYTCNBRGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@grstnhbr i've just pushed the fix to drag and drop, it should be working fine now,
I didn't understood the other problem clicking the upload link doesn’t seem to work what upload link?

Thank you so much!! Sorry for not being more clear. The issue may be with
multipart upload but I can't find any error logs or anything to indicate
what may be happening.
Whenever I upload a large file (150mb+) using the UI, the UI always says
"100% complete" while it spins, and then it either spins forever (uploads
usually take 3-5 minutes if not faster), and sometimes sometimes the upload
silently fails. Here's the load UI that it always has:
I can upload these files to the same r2 bucket using rsync from my local
machine without issue, so I don't think it is a cloudflare problem.
[image: Screenshot 2024-03-18 at 5.24.19 PM.png]
On Mon, Mar 18, 2024 at 4:26 PM Gabriel Massadas <
@.***> wrote:

@grstnhbr i've just pushed the fix to drag and drop, it should be working
fine now, I didn't understood the other problem clicking the upload link
doesn’t seem to work what upload link? —Reply to this
DuckDuckGo removed one tracker. More
https://duckduckgo.com/-hGysyZFupsqvAi0nzhKiHW-qw9B1bM_WFhjfeDLMjziZvic5s51PZer7HsqjoebEs0fKAShMpsAAx_fVHnFKU4jln2975YzWAttFiQNVWcjCs1MxkV7vhuQWAZBsTLYANJpCBFGhxsmpQk9r4FgAwaRa7XQSVg5O7l_3Glb4_nuofO7kt6APmJ7sAqpbFdqsAmGp9YovZo-bT4UvzjM2UVluPgxBjSqyULcIG2qZu9L33djC1X6ZVTNRuOn4yEGdKrQ9JWbFc75Sav-fLK05BPJMMwFMfLwrSkCvicGEMxhaaWoaHUsd90J0q5DRLQsmTnNoptfQjixfacvgejNEP0T8NQ0tZm0pDVWUyj5qrU8Io_S_QjZQl_L-UZFU3Hx0piygke-_Y41x6IFdjhsRZQd2meJ99K7M5zxOWhek8_nB0IsdhO4olNjLNsjfI-4Gv0TnBqnavPF93xd0SJnwChTQgzL6WLDD9_StIUlyW7zg_5kLX1NDiqxJXIYc4ZA06
Report Spam
https://duckduckgo.com/-hGysyZFupsqvAi0nzhKiHW-qw9B1bM_WFhjfeDLMjziZvic5s51PZer7HsqjoebEs0fKAShMpsAAx_fVHnFKU4jln2975YzWAttFiQNVWcjCs1MxkV7vhuQWAZBsTLYANJpCBFGhxsmpQk9r4FgAwaRa7XQSVg5O7l_3Glb4_nuofO7kt6APmJ7sAqpbFdqsAmGp9YovZo-bT4UvzjM2UVluPgxBjSqyULcIG2qZu9L33djC1X6ZVTNRuOn4yEGdKrQ9JWbFc75Sav-fLK05BPJMMwFMfLwrSkCvicGEMxhaaWoaHUsd90J0q5DRLQsmTnNoptfQjixfacvgejNEP0T8NQ0tZm0pDVWUyj5qrU8Io_S_QjZQl_L-UZFU3Hx0piygke-_Y41x6IFdjhsRZQd2meJ99K7M5zxOWhek8_nB0IsdhO4olNjLNsjfI-4Gv0TnBqnavPF93xd0SJnwChTQgzL6WLDD9_StIUlyW7zg_5kLX1NDiqxJXIYc4ZA06
@grstnhbr https://github.com/grstnhbr i've just pushed the fix to drag
and drop, it should be working fine now,
I didn't understood the other problem clicking the upload link doesn’t
seem to work what upload link?
—
Reply to this email directly, view it on GitHub
https://github.com/G4brym/R2-Explorer/issues/58#issuecomment-2005025416,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ3JOYK54FN677RWF6VQDTTYY5LXZAVCNFSM6AAAAABB2NATDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBVGAZDKNBRGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
AJ Gerstenhaber
(512) 797-3101
atxfilmlab.com

@grstnhbr i've just pushed a new update, can you try now

Hello, and first of all, thank you for this wonderful project.
I'm having the same issue with the uploads (tested with upload and drag&drop). I'm having trouble uploading PDFs. I have tested a few of them with different sizes, 20Kb, 500Kb, 3Mb, 5Mb and no luck.
When I do the same operation from the Cloudflare R2 console, I have no issue uploading any size PDF.
I have tested with a txt, mp4, rpm, csv and with no issues.
When doing further testing I noticed the same problem with bmp, jpg, and png files.  And in all these cases, when uploading from the Cloudflare R2 console, there is no problem.
Many thanks,
Ignacio


Create goading-helper

This plugin contains the following features for use with the new goading potions.
Radius overlay to show aggression range.
Reminder overlay to remind user to sip potion when they have it in inventory and aren't under effect
Notification on potion expiry
Tick timer to display when the aggression check happens

If you need to make additional changes, don't open new PRs, just push to the existing branch.


Releasing Synthetic-Easy?

Hey, are there also plans to release the Synthetic-easy dataset that you used for your testing results in table 2 and the fine-tuning results in table 6? How big is that dataset?
Thanks!

Hello.  Thanks for your advice. At the moment, there are no plans to release the Synthetic-easy dataset. However, we're happy to share some information for your reference.
Regarding Table 2, the total size of dataset is on the scale of millions, which is generated following Table 1. As for Table 6, the training set consists of data in the hundreds of thousands. For training details of Table 6, please refer to Section 7 in the appendix.

Thanks, closing!


Haystack integration

Details
Introduces a new Haystack integration
Issues
Resolves #850
Testing
Documentation

@jverre
Few additions from my end:

Fixed the integration test to setup the environment variable for haystack tracing BEFORE the haystack import (otherwise only the names and metadata were logged).
Extended fake backend emulator to work support UpdateSpanMessage and UpdateTraceMessage (but it's still not recommended to use them), added a new fixture for fake backend - fake_backend_without_batching which must be used when an Opik client with disabled batching is used.
Updated the library integration test to check inputs, outputs, usage and model name.
Fixed some smaller issues, refactored the code.

P.S. current version only handles usage logging correctly for the openai provider. In case some other is used, usage will not be logged (but we can extend it later when we need it).


Messari price source is unreliable

Hi,
I just want to inform you that I often see this stacktrace in my logs:
2022-01-07 13:24:13,931 ERROR Job "update_price_tickers" raised exception
 Traceback (most recent call last):
  File "/opt/venvs/cypherpunkpay/lib/python3.8/site-packages/cypherpunkpay/jobs/job.py", line 13, in __call__
    self.job()
  File "/opt/venvs/cypherpunkpay/lib/python3.8/site-packages/cypherpunkpay/jobs/job_adder.py", line 17, in <l>
    lambda: self._app.price_tickers().update(),
  File "/opt/venvs/cypherpunkpay/lib/python3.8/site-packages/cypherpunkpay/prices/price_tickers.py", line 54,>
    self.update_coin(coin)
  File "/opt/venvs/cypherpunkpay/lib/python3.8/site-packages/cypherpunkpay/prices/price_tickers.py", line 61,>
    MessariCoinPriceSource(self._http_client).get(coin, 'usd')
  File "/opt/venvs/cypherpunkpay/lib/python3.8/site-packages/cypherpunkpay/prices/messari_coin_price_source.p>
    return Decimal(parsed_json['data']['market_data']['price_usd'])
KeyError: 'data'

They probably respond with an error object which has no data key. Overall I think the price source http requests should check the response status code before assuming its json. Logging warnings with at least the http status code may also help in the future to detect no longer working price sources.
And since I mentioned logging 😃 Could you consider decreasing the stats logging interval for production environments?
10 seconds is rather spammy and the stats are not that important in the log in my opinion. More interesting are charge status changes or price source and callback errors.

Thanks for reporting, will investigate.

Do you still have issues with Messari price source? I gave it some time and it looks fine in my logs now (no Messari exceptions).

The HTTP status code is verified by underlying HTTP client and JSON parsing is skipped if not OK. This is intentionally not logged because temporary connection failures are very often over Tor. Instead, I myself verify price sources fairly reguraly by running network tests. I do give them a slack time before disabling though.

Regarding the interval I will consider decreasing it. My motivation for 10s was to evidence the daemon is live and healthy.

Indeed the last time I see it in the logs is 2022-01-08 01:27:13,129 maybe they just had problems for a couple of days.
The call to json will not check the http status code, it blindly tries to parse the body as json. See here Response.json.
And base_http_client.py#get_accepting_linkability also does not check the http status code.
You could add a call to requests raise_for_status in get_accepting_linkability to change that. Then the traceback would have been silently swallowed by the try/except block in the messari price source.
But I am not really a fan of that, I would prefer a single log line with level warning and at least the status code.
Because I still assume they answered with a json error object and a non 200 status code, but without logging we will never know or notice.
What happens when all price sources silently fail? Will the last potential very out of date price be used or are there safeguards?

You are correct regarding the status codes.
When all price sources fail then the last (potentially out of date) price will be used.

Logging status codes >=400 is done.

Protection against stale prices is done (awaiting deb release).

Looks good, thank you very much.
I also like the new theme, but in my opinion the colorful qrcode is too much. With just a black and white qrcode it looks better, but in the end it's a matter of taste.
And you should probably improve the readability of a:hover black is barely visible. Maybe switch to something white?
Do you maintain a changelog somewhere? Because reading commit messages is not possible for everyone. Maybe on the website or as github releases?


Asusctl 5.0.10

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please squash the commits. Also, supergfxd version didn't change, but the hash did? Do we know what happened there?

I guess it's because Cargo.lock changed?

I see, upstream updated the commit without updating the version: https://gitlab.com/asus-linux/asusctl/-/commit/e3636ed8ce7fe1cbd587bad4aab3425236d6836a#14a5af756bb04b4a26c80cf95acf32f0f8b9f30f_3289_3286
Anyway, please squash the commits.

I mistakenly deleted that branch, should I open another pr? I can't reopen this one

I squashed these commits


Add Documenter docs

Includes new docstrings in src and a section on writing pullbacks.
Documenter deploy key will need to be added to Enzyme.jl repo settings.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@110030f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #72   +/-   ##
=========================================
  Coverage          ?   74.29%
=========================================
  Files             ?       10
  Lines             ?     1097
  Branches          ?        0
=========================================
  Hits              ?      815
  Misses            ?      282
  Partials          ?        0

Continue to review full report at Codecov.

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

Thank you Oliver!

Whoa, wait, doc generation was failing now, hadn't found out why yet! :-)

This should fix it, I hope: #76


How to install for a specific coq version

Hi,
I want to use this plugin for this coq (and coq-serapi) versions:
coq                     8.11.0        pinned to version 8.11.0
coq-serapi              8.11.0+0.11.0 pinned to version 8.11.0+0.11.0 at git+https://github.com/ejgallego/coq-serapi.git#v8.16

Oh I don't see a release for 8.11, maybe we missed it? There were a couple of fixes that landed in the release for 8.12 that are worth getting in, so if you could move to ≥8.12, that would be best.

No response, I'm closing this issue. Feel free to reopen.


CircleCI crashes with ./gradlew failed with exit code 1

Hi there - Anyone had any luck getting this working ?
Doing tns build android results in CircleCI Fastlane failure when building Android for me
I've tried various combinations  (with & without --env.aot & "enableIvy": false / true) but have no hair left now.
The build runs fine locally.
Background
NS Angular project
tns platform add android
Copying template files...
Platform android successfully added. v6.5.1
yaml generated by NS-remote-builds: https://gist.github.com/kakembo/2fce085522acfc8c0d4444075c797b76
CircleCI fastlane output:
test-output0.txt
test-output1.txt
Package.json:
https://gist.github.com/kakembo/a12cfdb02a2e456bbe8b4b2db63fa9ee
Any help very gratefully received.
Thanks,
Andy

I am getting the same error, also runs fine locally.

@kakembo please post if you got it working.

Sorry - No joy. The good people at CircleCI gave me some hope here: https://support.circleci.com/hc/en-us/requests/73983 - I hope that helps you. I gave up and moved to other tech. Let me know how you get on though.

Let me know if you can't see the CircleCI support ticket @garyffh  & I'll copy it here.

Thanks  @kakembo
If you could copy the support ticket.

adding  resource_class: medium+ to docker:
fixed this for me.


Palette Switches to one with "Sticky" and Arrow Icons Only--Can't Switch

Hello, so when I used "split-screen" (Great Feature!!!) and pulled off a tab to a different window (another Great Feature!) the default archimate palette changes to one which has 11 different colored square "sticky" options, arrows, block and image icon.

This is GREAT for collaboration and discussion... but the screens only have one or the other and can't swtich palettes. I can even make different tabs have one or the other by pulling the tabs on an off once the states have changed to the other palette--just can't change them once there, to get the normal pallete back, I needed to close the view and reload it.
BK

Can you provide steps to reproduce, I don't follow what you mean?

Good morning @Phillipus, so, in 4.10, if I have multiple views open. Switch between tabs, I get the standard palette.  If I drag a tab to split windows (vertical or horizontal), I sometimes get the pictured alternate palette.
It's actually quite useful in collaboration and discussions, it really is!  But it's not easy to switch back.   It also appears to save okay.  So, I take it that this was not intended behavior?

That palette is for a Canvas View. If you select the Canvas View. that will be the palette you get.

Yep--was about to post that.   Thanks--yeah, I had both types open with an original image of an architecture in the canvas view.  Great stuff!!


Simplify brew install command in README

Brew now supports directly tapping while installing
The homebrew prefix while specifying tap is unnecessary (in fact, the docs mention that taps exclude that prefix)

Didn't know brew added this. Thank you.


dropdown menu of nav bar

I would need to have my navigation bar contains dropdown menu.  Bootstrap provides such easy code for creating dropdown, I am very surprised that MDL doesn't has this feature for designer.  It should be a very common UI feature, and not sure why it's absent in MDL.  Does anyone know the solution or can find related code at somewhere ?  Thank you for your help.

As stated in our issue template:

The MDL core team has actively begun work on the next major version of MDL, dubbed MDL v2. Because we are a small team that's hyper-focused on delivering the best Material Design Library possible for the web, it is highly unlikely that we will be actively working on new features or making backwards-incompatible changes for MDL as it currently exists.

If there is a non-breaking feature you would like to see implemented in mdl-1.x and are willing to contribute, we'd be happy to offer assistance with you doing so. But we will not personally be actively working on said features.

We will be implementing this in v2 (#4475), but we don't have the time and the bandwidth to work on it for v1. In the meantime, this looks it could be an adequate solution: https://www.npmjs.com/package/getmdl-select. Sorry we can't be of more help here.


Multiple components in one bot project won't work

Describe the bug
When I have 2 custom components added to my bot project, only one ends up running.
Version
v2.1.1
Browser

[ ] Electron distribution
Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Follow instructions outlined here (https://docs.microsoft.com/en-us/composer/how-to-create-custom-actions) minus the schema portions
Repeat instructions to create another component
Make sure both components are present in the settings/appsettings.json components array.
Run bot locally (Composer or Emulator)
Confirm only one middleware is executing

Expected behavior
I expected both middleware to be working one after the other as they both worked individually.
Screenshots

components array containing 2 components
Additional context
First middleware created was the custom typing middleware for fixing a bug that occurred when you used the built-in ShowTypingMiddleware. Tested and worked on its own. Second middleware created was the sentiment analysis middleware for retrieving sentiment for each message and conditionally executing some other logic (this logic has not been implemented for now it just sends a message to the user of what the sentiment was). After adding the second middleware, only the second middleware executes (no more typing activities being sent in the traces). If I comment out the second middleware, the first will start working again.
If I was to try to guess what's happening, I'd assume that the typing middleware does not send the typing activities anymore because it executes after the sentiment analysis middleware (it does not meet the minimum delay time in order for typing activities to be sent). However, attempts to fix it by reordering the declaration of the components (I read in a microsoft doc that the middleware pipeline is arranged in order of adding the middleware) have not fixed the issue.
I have also tried adding both of the middleware classes to one component and then just having a single component in the components array but that did not solve the issue either.

The problem was that I was sending a message activity in my second middleware which basically prevented the typing middleware from being able to send its activities because it was dependent on at least a 500ms delay and the time between the two middleware is likely less than that.


fixed JFXSpinner's memory leak problem

fixed issue https://github.com/sshahine/JFoenix/issues/1181

Thank you @RationalityFrontline for the fix 👍


Download results button occasionally disabled

If you go to the visualisation tab and update the query or filters, the download button will be disabled.
It used to be that updating the query would always take you to the results tab as well, so this is a recent phenomenon. This happens because the frontend has not requested results, and thus the resultCount is 0.
This is a minor issue, but it's not great UI to disable a button when there is no reason why it cannot be executed. And while users can still download by switching to the results tab, they have no way of knowing that.
Could be resolved by:

Making a search request (with size = 0 for speed) to probe the total number of results
Implementing the layout change I propose in #674, to move the download button into the results tab.

Suggested to implement https://github.com/UUDigitalHumanitieslab/I-analyzer/issues/1310 instead.


Can not access to container console

Question:
How can I access to container console?
when I try to exec login with this container , it is always show me the wrong info below,
docker exec -it b2d76b3d56c8 /bin/bash
oci runtime error: exec failed: container_linux.go:265: starting container process caused "exec: "/bin/bash": stat /bin/bash: no such file or directory"
I want to exec into the container and add CA files and run the command " sudo /usr/sbin/update-ca-certificates"
thanks ~

The Portainer docker image is based on scratch (actually portainer/base), there is no way to exec into this container.
If you want to achieve something similar, you'll need to build your own image based on something else like alpine for example.
Why do you need to execute this command? We already bundle the certs in the image, but might need to refresh it (https://github.com/portainer/docker-images/blob/master/base/Dockerfile).

@deviantony  got it , thanks so much for the comment ~ let's me close this case now .


Add -y to add-apt-repository

This is useful when running this action via https://github.com/nektos/act to avoid add-apt-repository prompting for confirmation

Thx and thanks for testing this, feel free to ask if you get stuck. Btw. you can also run the scripts on a normal Debian testing system without the PPA.

@esteve btw. your current job is missing this: https://gitlab.com/autowarefoundation/autoware.auto/autoware_auto_msgs/-/commit/a85138f3f4adcece4fe46bef4ef344dde4a5923d

Good catch, thanks!

@esteve Do you have it working with act already? I am currently setting up something with this as well, but am currently stuck on sbuild failing to setup the chroot environment.

Not yet, I made some progress, but had to add a couple of hacks (missing /home/runner or not being writable)

Just for completeness, I don't think it will work in ACT as sbuild needs user namespaces. On the other hand it should be easy to adopt the scripts to run on any Ubuntu 22.04 or Debian testing.


abort should not call remaining remove hooks.

When we intend to stop observing, I don't think we intend to call remove hooks, because in this case it doesn't necessarily mean that the elements were removed, just that we want to stop observing.
As another example, MutationObserver doesn't trigger changes when disconnected.

I could see either way, but this was intentional in the original implementation of abort(). I mostly use it in tests as a way to teardown the result. So it's not something I can easily change without considering it a backwards incompatible change.


Muslimblogpost

blog post finished

@csgreer6 Thanks for working on this, Caroline. You did the hard part of getting the text and images ready.
It looks like you based this branch on the Shakers blog post branch as well, so it has some changes that we don't want here. In the future, you will want to checkout the main branch, then create a new branch. No worries; it's an easy mistake to make, and I do it at least once a week. :-) I might not merge in this specific pull request, but I've got everything I need and can pull in the right files.


obinskit: fix hash and add necessary referer header

Description of changes
i noticed i couldn't build the package because curl couldn't download it, and figured out it needed this header
Things done

Built on platform(s)

[x] x86_64-linux
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[x] Fits CONTRIBUTING.md.

@theotheroracle Do you know why the hash changed in the first place?

@theotheroracle Do you know why the hash changed in the first place?

no i'm not sure

i fixed the commit name

i fixed the commit message

No. Please read https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions.


Command-line interface for training

[x] Add a --load argument to training, to be able to (re)train a model in different sessions (need to write curr_nb_epochs somewhere).

Finished with #27


Update current gitlab merge request event payload

Gitlab Merge Request event has been updated at an unknown time.
I have updated the current interface to work with the version we have
now, 12th of November 2019.
I have also updated the testdata with the official event from docs:
https://docs.gitlab.com/ee/user/project/integrations/webhooks.html#merge-request-events

This fixes #98.


error on ng serve

after cloning and running npm install, when i use ng serve i receive this error:
ERROR in node_modules/@angular/flex-layout/core/typings/observable-media/observable-media.d.ts(1,22): error TS2305: Module '"/Users/nik/Documents/GitHub/angular-file-upload/client/node_modules/rxjs/Rx"' has no exported member 'Subscribable'.
Any idea why?

Did you tried this  ?


Error when knitting "Writing Vignettes"

Quitting from lines 155-167 (writing_vignettes.Rmd)
Error in dataset_vignette(vs1, display_vars = vars(USUBJID, VSTESTCD,  :
  could not find function "dataset_vignette"
Calls: <Anonymous> ... handle -> withCallingHandlers -> withVisible -> eval -> eval
Execution halted

Error has been resolved after re-installing the latest version of {admiral} which contains the dataset_vignette() function.


move to updateOne

closes #57
i havent added any tests, tell me if you want me to do this pr "better" :)

Thanks @johannordin


[Do not merge] Windows alternate user support for execute resources

Description
w.r.t https://github.com/chef/chef/pull/4297
Also includes fixing of review comments, bugs and specs from https://github.com/chef/chef/pull/5722
Issues Resolved
https://github.com/chef/chef/issues/2850
Check List

[x] New functionality includes tests
[x] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

This replaces https://github.com/chef/chef/pull/4297. We originally started by combining the remote_file support in https://github.com/chef/chef/pull/4299 again, but realized there's a use case we need to fix before we release any new properties. It'd be significant progress to get this feature merged.

tl;dr is that the code looks okay except the validation really needs to be moved directly to the resource, would probably prefer not to create the additional mixin as well.

Great work on this @NimishaS!

Thanks @mwrock. Yes I still need to fix the specs for mixin references.

@NimishaS the validations that you put in after_created of the resource should go in define_resource_requirements in the provider. The requirements class used here allows you to specify which actions (or all) that the requirements apply to as well as avoiding a failure in why-run mode when we would not be able to verify the requirements in why-run mode (typically due to a prior resource creating them).

@btm , I got review comments by @lamont-granquist to move the validations in after_created of the resources https://github.com/chef/chef/pull/5764#discussion_r98286129. Previously I had called the validation method in define_resource_requirements.

@btm , @lamont-granquist , please let me know where should the validations be kept. I would also like understand the difference between the purpose of after_created and define_resource_requirements method.

To help understand, here's a wall of text. :)
I figured after_created was a new pattern that I didn't understand, then I looked at the git history and saw that I added it 5 years ago as part of the original chef_gem implementation. Heh. I don't see a new pattern in use in the code. In theory starting to use after_created for validating arguments passed to properties by the user that wouldn't be affected by the state of the system would raise an error earlier, which would be good. after_created naturally exists to run code in the compilation phase tied to resource creation, but we use it sparingly.
By being in the provider, define_resource_requirements is well timed for the kind of validation that might not pass when chef-client starts but would later after prior resources have changed the state of the system. This is why it also has provisions for avoiding raising an error when in why-run mode. Because we're not actually making changes to the system in why-run mode, we have to ignore some of those failures for why-run to work.
Currently all of the property validation is done in define_resource_requirements. You can see a recent example of this in the launchd provider where we ensure that the 'type' property is either 'daemon' or 'agent'.
I chose that example to further muddy the waters of this discussion though. The property feature replaces set_or_return. This is new and we're still not used to using it. I was thinking there may be a good feature in property to use for static validation. In the launchd resource we are overloading the coerce option to not only downcase but to duplicate the same validation that is in the provider. Obviously the validation is in coerce and not in the type so we can use downcase in that block to catch all capitalization variations of the argument. If we passed an array of strings like [ "daemon", "agent" ] to type we would miss an argument like "SomethingElse" and fail messily later.
From the perspective of using property, being guaranteed that coerce runs before type (aka checking :is in Chef::Mixin::ParamsValidate) would be nice and allow something concise like this:
property :type, [ "daemon", "agent" ], default: "daemon", coerce: proc { |type| type.downcase }

So. Using define_resource_requirements for validating properties that depend on system state that could change between when the run starts and the provider runs seems to continue to best. Using property going forward for validation when the property the user provides can be validated irrespective of system state would be a good path, but we seem likely to make a habit of overloading coerce to achieve this.
There's an ordering problem if you need to compare one property with another though. Perhaps @lamont-granquist was thinking about using after_created as a compromise between the ordering problem and waiting until the provider action with define_resource_requirements.
There might be some validation that could be moved up if we overload coerce though, like:
property :domain, String, coerce: windows_only!
property :password, String, sensitive: true, coerce: windows_only!

def windows_only!
  unless Chef::Platform.windows?
    raise Exceptions::UnsupportedPlatform, "Values for `domain` and `password` are only supported on the Windows platform"
  end
end

@lamont-granquist ?

so coerce: windows_only! reads wrong to me because you're not doing any coercions, we should implement some other keyword for that which doesn't have the coercion side effect and doesn't read as poorly.  that should be fairly straight forward since it's just a keyword that acts like a coercion without coercing anything.
i agree with you about d_r_r being a better place for validations which depend on system state, but it didn't seem to me like any of these validations depended on system state and it was all static analysis of the resource properties.  most of the conditional logic was based on windows? being true or false and that can't change during the run (and chefspec should get it correct--if it doesn't that should get fixed).  but the complexity was that the validation required looking at more than one property at a time.  so if you just failed if someone set a username without setting a password on windows and did validation in the property then this would fail:
execute "whatever" do
   username "Administrator"
   password "foobar"
end

when the username is set, the password hasn't been set yet, which would fail validation then, so we have to lazy that check until the end block in order to be sure that the password has been supplied.  that is what moving the check to after_created accomplishes.
if there's a check in there which depends on system state, which might change during running, and can't be evaluated at declaration time then i missed it on my read-through.
so, i think after_created is still the most appropriate place to put the validations.  i like the idea of using the properties api, but abusing the coerce keyword makes me very twitchy.  i really like the effect, but i'd like to see that done right and extend the properties api to support it properly.

Another way of saying the same thing is to look at define_resource_requirements:
  def define_resource_requirements
    [...]
    requirements.assert(:all_actions) do |a|
      a.assertion { validate_identity(new_resource.user, new_resource.password, new_resource.domain) }
    end
    [...]
  end

That is code-smell because there's no a.failure_message or a.whyrun in that block.  All the raising is done by the assertion as a side effect.  Really it should just be written:
  def define_resource_requirements
    validate_identity!(new_resource.user, new_resource.password, new_resource.domain)
  end

But that just highlights the fact that user, password and domain are being validated statically and there's no system state that is being inspected -- or else there would have needed to be an assertion with a whyrun statement so that chef would output "assuming user 'foo' would have been created" or something.  I can see that we do have requirements which are added with no whyrun statement but those are really just syntactic sugar for raise <something> unless <assertion>.  the ones that i see it does make sense to lazy the assertion since it does inspect the underlying system its just not likely that chef would ever fix the issue during a run (e.g. the check for hive_exists? in registry_key -- you could throw an "assuming hive would have been created" but i believe nobody ever creates new hives and those are predefined on windows -- but since it evaluates that state of the node then you want to lazy that onto the provider so it doesn't troll chefspec running on unix).

And also putting the the user+password+domain validation on the resource should get the chefspec case correct.
    If you move the validation to d-r-r then it always passed.
Of course this assumes Chef::Platform.windows? is correctly checking node data, if it ain't then that's busted, but that indicates we should change that to inspect node[:platform].
Aaaand that's busted:
module ChefConfig

  def self.windows?
    if RUBY_PLATFORM =~ /mswin|mingw|windows/
      true
    else
      false
    end
  end

end

should probably use node[:os] == "windows" instead of Chef::Platform.windows? and we need to fix that.

@NimishaS in summary, let's leave everything in after_created for now because in property the other properties we want to inspect may not be set yet, and define_resource_requirements runs later in the run than we need to. Please do change Chef::Platform.windows? to node[:os] == "windows" as @lamont-granquist indicated in his last message.
@lamont-granquist I agree we should add a validation option for validating the input in a block, and then go back and clean up the launchd resource where it is doing that in the coerce block.

yeah i cut this issue to work on fixing Chef::Platform.windows? and making it all more usable, but its going to be more than a quick fix:
https://github.com/chef/chef/issues/5771
for now use node["platform_family"] == "windows"  (probably use platform_family over os since that is what chef-sugar has been using since forever, and i'm fairly strongly inclined to just copy that, but it probably doesn't matter at all)

@btm , @lamont-granquist , please review the PR.

Very cool to see all the commentary on this -- thx for joining in @lamont-granquist  and @mwrock. Sorry I wasn't able to shepherd this through all the way (still a little too busy these days), but grateful that we could get the feature to users!


Can't open a channel

User Story
Can't open a channel as everytime i try to deposit TTT it shows " Your maximum usable amount is 0.0 TTT"
Acceptance criteria
Tasks

Hey @Prajjwal-Arya did you mint also some tokens for the TTT token network?

Yes, i have already mint some tokens and i have tried refreshing but still it was showing the same thing.

@Prajjwal-Arya if you write to the mint function for this contract you will get the correct TTT tokens: https://goerli.etherscan.io/token/0x59105441977ecD9d805A4f5b060E34676F50F806#writeContract

We updated the TTT token address that was wrong in the instructions #1041
If you go to the previous screen when you selected the token does it show the minted amount?

No, it still shows 0.0 .

@Prajjwal-Arya that is weird at least after minting you should be able to see the amount there. any screenshots or more information you could provide?

@Prajjwal-Arya we did a ton of minting & channel opening this week, and couldn't reproduce your issue. There's also some optimizations to minting and channel opening. Please, test current version, and reopen if your issue persists. Thanks for the interest and reporting.


Feat: Using SocketIO ID generator for shorter ids

Using socket io id format instead of UUID

[ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Using SocketIO id generator for ids means ids are 20 digits long instead of 36 - both are still a 128bit number, but the base62 encoded scheme is shorter and more manageable
Instead of ids like this:

You get ids like this:

To run this PR locally, use the following command:
docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:fbd5e8e-nikolaik   --name openhands-app-fbd5e8e   docker.all-hands.dev/all-hands-ai/openhands:fbd5e8e

In retrospect, I wonder if this could present problems on Windows, where the file system is case insensitive

I think this will cause problems in windows environments, so I am closing it.


Fix copy text when TextEdit is readonly

Readonly TextEdit doesn't check any keys because if statement jump over this code.
Fix #25916

Thanks!


Review 'cell proliferation' branch

As discussed at an ontology call:
"Most ['cell proliferation' terms] have logical def, e.g. 'epithelial cell proliferation' is_a 'cell proliferation' that acts_on_population_of 'epithelial cell'. Did we have plans for a TG template? (Not in Jira.)
Note: this is a case where we have a semi-random distribution of terms in the ontology vs annotation extensions. Should we just make terms for all AE usage and retrofit AEs?
Should we have a template?
Yes. Paola will request it.
DONE: https://github.com/geneontology/termgenie/issues/85
Many are in AEs - should all those go into the ontology?
Melanie:  Isn't this going to cause bloat?  Shouldn't we be pushing in the other direction?
David: Doing this would hide the annotation from term enrichment (only inferred parent classes would be visible)
Paola: Enrichment to such specific terms would be rare.
[THIS IS THE OBJECT OF THIS TICKET:]
Decision: once the template is ready, create terms for children of 'cell proliferation' that have abundant annotation extensions.
Then look at the structure under 'cell proliferation': if it's too flat, see if we can create some intermediate terms (using the template).
Paola can do it (not as a top priority though)."
(For reference, minutes here:
http://wiki.geneontology.org/index.php/Ontology_meeting_2015-10-29#.27cell_proliferation.27_terms)

I think this is really low priority, and didn't stem from a curator's request, so I'll close it. The idea could be revisited in the future if there is interest and resources, but the same might apply for other branches in GO that have annotation extensions.


FE/t display flickers between ~9,000 FE/t and 0 FE/t

Describe the bug
When looking to see how much power is being used by a particular flux point, it flickers between 0 and ~9000 FE/t. I am looking at a point connected to a mekanism cable > netherite sluice in the Ocean block mod pack.
I have seen them do this before but I don't remember where.
To Reproduce
Use Open block mod pack.
setup a netherite sluice to run.
connect mekanism cable from sluice to flux point.
watch the output flicker in the FLux point UI.
Expected behavior
Display a consistent output value.
Possible Solution
Average the output values over the last second (20 ticks)
3 sec video clip
https://user-images.githubusercontent.com/6530707/143943302-38a2e64f-2399-46af-9034-5931fa0cebff.mp4
.

Leave this as is please or add a toggle, it helped me diagnose an unstable delivery issue with Powah!
Averaged transfer rate is shown on the graph, I like the instantaneous readout


#2241 implement left, right, full outer joins (+ limit control)

Description
I've implemented all kinds of joins in join pop-up. (inner, left outer, right outer, full outer)
Additionally, I fixed some limit control on sampling stage.
Related Issue : https://github.com/metatron-app/metatron-discovery/issues/2241
How Has This Been Tested?

Import A.txt and B.txt

Create a dataflow and add 2 datasets above.

Select join as command and click "Add" button.

Set parameters like this:

Check every join types with "Show result" button.

Click "Join" button to check if the result of dataset is as expected.

Need additional checks?
Yes, please.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

run build
deploy to 3

run build
deploy to 3

run build
deploy to 3

@ksparknot Still we have some problems with UI.

Choosing a right predicate does not set the same named left predicate.
When edit a join rule, if I change the join type and clicke "Show result", UI does not send any request, and shows a wrong result (like, all null rows)
When edit a join rule, the list of join target does not contains the current join target. This might be OK, but after selecting another target, I cannot change back because the list of join target always exclude the first join target when entered the join pop-up.

It might be better to report these problems independently, but there are too many.
I wish we solve these problems with this PR.

run build
deploy to 2

@ksparknot Still we have some problems with UI.

Choosing a right predicate does not set the same named left predicate.
When edit a join rule, if I change the join type and clicke "Show result", UI does not send any request, and shows a wrong result (like, all null rows)
When edit a join rule, the list of join target does not contains the current join target. This might be OK, but after selecting another target, I cannot change back because the list of join target always exclude the first join target when entered the join pop-up.

It might be better to report these problems independently, but there are too many.
I wish we solve these problems with this PR.

When you select left predicate, the right predicate is set.
If the left predicate changes again when the right predicate is selected,
You will never be able to use columns with different names with join keys.

So, I have an idea.
Choosing the right predicate changes the left predicate,
However, if the left predicate is already selected, it should not be affected by the right predicate.
But I'm worried that it might look like a bug.
I will apply it anyway.

I could not reproduce it.

Solved

run build
deploy to 2

run build
deploy to 2

run build
deploy to 2

@ksparknot @jeongeunkwon @eldorado0402 @AnnieHwang
I've fixed a bug and re-deployed on testbed2.
Self-test done. Thank you.

run build
deploy to 2


Impressive paper! Wondering the code release!

Hi,
I am super curious about this differentiable rendering work of parametric geometry! This work is pretty interesting and impressive. Could you please release the code and share the details?
Thank you so much!
Lan.

Hi,
thanks! I expect the code to be released next week. I'll leave this issue open until the release.

Thank you!

Hi,
Still waiting~
Thanks

A first version is now released.


Bug fix & enhancement to setup.py

#3: corrected url string to point to correct project
#4: Added long_description change

Submitted 2 separate pull requests to fix error.


Slack Message Showing version as a channel (private channel)

Hello!
I loved this plugin for semantic-release, I was able to quickly get started.
Not sure if the issue below is related to this plugin or a problem with slack itself, but slack tries to generate a channel link in the version number since it contains a hashtag.
Would you have any ideas how I can prevent that from happening?
Thanks in advance!!!

Thanks for the positive feedback! ❤
In the image, it looks like the version is 1.3.0-beta.2. Does the version contain a hashtag somehow? Or is it only that this plugin adds a hashtag?
In other words: do you have a special config where you add # to the version? Or do you use pretty default configurations of semantic-release?

Update
It looks like we add a hashtag in this place of the code:
https://github.com/juliuscc/semantic-release-slack-bot/blob/9f4c735b5bb8b77404819b9260afeb604d936f9b/lib/success.js#L68-L78
I don't know the reason for that though. It doesn't make sense to me but I would imagine that the symbol is added for a reason. I don't want to just remove it if I break something else.

Update 2
The # was there from the beginning. It's included in the first commit that posts to slack.
https://github.com/juliuscc/semantic-release-slack-bot/commit/d541e59c84dca0c3e7d0d741361383c24bcfa225
I don't know why no one else hasn't noticed it yet, including me. Could it be that # means something else in slack-markdown? In markdown, a # means heading but only if it's at the beginning of a line.

@juliuscc in slack # is used to reference channels. I'd guess no one has ever noticed that because they are probably slack admins, so slack can figure there is no channel named 1.2.0 (or whatever is the version number) and just ignores the #, but since I'm not admin and the slack from my company has private channels, slack will mask it as a "private channel" rather than ignoring it.

@juliuscc I was able to solve it by overriding the template and removing the hashtag:
{
  "plugins": [
    "@semantic-release/commit-analyzer",
    "@semantic-release/release-notes-generator",
    "@semantic-release/npm",
    "@semantic-release/github",
    [
      "semantic-release-slack-bot",
      {
        "packageName": "My Project",
        "notifyOnSuccess": true,
        "notifyOnFail": true,
        "onSuccessTemplate": {
          "text": "A new version of $package_name has been released!",
          "blocks": [
            {
              "type": "section",
              "text": {
                "type": "mrkdwn",
                "text": "A new version of `$package_name` has been released!\nCurrent version is *$npm_package_version*"
              }
            },
            {
              "type": "section",
              "text": {
                "type": "mrkdwn",
                "text": "$release_notes"
              }
            }
          ],
          "attachments": [
            {
              "color": "#2cbe4e",
              "blocks": [
                {
                  "type": "context",
                  "elements": [
                    {
                      "type": "mrkdwn",
                      "text": ":package: *<$repo_url|$repo_path>:*   <$repo_url/releases/tag/v$npm_package_version|v$npm_package_version>"
                    }
                  ]
                }
              ]
            }
          ]
        },
        "markdownReleaseNotes": true,
        "slackWebhook": "https://hooks.slack.com/services/{WEBHOOK_KEY}",
      }
    ]
  ]
}

Perfect! Could you provide a screenshot of how it looks? If it looks good then I think we should create a PR for removing the hashtag.

@juliuscc here is a screenshot:

Thanks for being so helpful! It's great that you took the time to investigate and share a screenshot of the changes.

:tada: This issue has been resolved in version 3.5.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Roadmap] D-Tree Malawi Core application

Legend
:heavy_check_mark: -- Completed
:construction: -- Work in Progress
:x: -- Not Done
:books: -- Research & development
:no_entry: -- Blocked
Apps to be delivered

CHW app
Site Supervisor
Expert Tracer app
Program manager app

Features to implement

User management

Authenticate users :x: -- Not Done & - :books: -- Research & development :construction

Sync

Sync Data from HAPI
Sync configs  :books: -- Research & development -  :x: -- Not Done & - :books: -- Research & development :construction
Configurable app management  -  :x: -- Not Done & - :books: -- Research & development :construction

Patient List

View Patient List of all enrolled patients  -  :x: -- Not Done & - :books: -- Research & development :construction
Configurable Patient List  -  :x: -- Not Done & - :books: -- Research & development :construction

Patient Profile

View  Patient Profile  -  :x: -- Not Done & - :books: -- Research & development :construction
Configurable Patient Profile  -  :x: -- Not Done & - :books: -- Research & development :construction

Questionnaire management

View Questionnaire  -  :x: -- Not Done & - :books: -- Research & development :construction
View QuestionnaireResponse  -  :x: -- Not Done & - :books: -- Research & development :construction
Edit QuestionnaireResponse  -  :x: -- Not Done & - :books: -- Research & development :construction

Multi-language support

Multilanguage support for Questionnaire  -  :x: -- Not Done & - :books: -- Research & development :construction
Multilanguage support for Questionnaire items -  :x: -- Not Done & - :books: -- Research & development :construction

To confirm is this can be configurable via Quest


Any hint for trainning own data?

This is intended to run our model (built from a checkpoint), but it's using basic tensorflow resnet, afaik. I would start with something like this:
https://github.com/tensorflow/models/tree/master/resnet
and general tensorflow tutorials if you are interested in building models from scratch. In our experience, it works out better to start with something over nothing (eg a model provided by google even if it's trained on non medical entities). I'll leave other comments to @mchen17

@vsoch Thanks. I wonder if there will be some basic descriptions e.g. how to train own bone-age dataset, which model you used for extracting the bone features.


Implement transport.TxBuffer

I haven't tested this yet, but it should resolve issue #22.

I've identified a race condition between Write and TxBufferLen that makes this unusable as it stands. Will require some extra work.

The last commit should fix the race condition. Still need to test this properly. If anyone feels up for it, feel free to check it out and report back 🙂 I'll probably look more into this in a couple of weeks.

I've done extensive testing with VARAFM after the last commit, and everything appears to be working as expected now. Pat is able to correctly indicate outbound message transfer progress 🎉


Draft Pull Request for implementation of Null Geodesics & Radiative Transfer

Draft in progress. More commits incoming.

Fixes: #105, #106
Potentially fixes/addresses: #103, #113, #122, #141

@shreyasbapat

Hello @JeS24! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/einsteinpy/integrators/runge_kutta.py:

Line 118:53: W291 trailing whitespace
Line 122:22: E261 at least two spaces before inline comment
Line 140:121: E501 line too long (127 > 120 characters)
Line 143:1: W293 blank line contains whitespace
Line 149:1: W293 blank line contains whitespace
Line 162:13: W291 trailing whitespace
Line 164:13: W291 trailing whitespace
Line 166:13: W291 trailing whitespace
Line 168:13: W291 trailing whitespace
Line 170:13: W291 trailing whitespace
Line 172:13: W291 trailing whitespace
Line 176:1: W293 blank line contains whitespace
Line 181:1: W293 blank line contains whitespace
Line 184:16: E222 multiple spaces after operator
Line 184:23: E261 at least two spaces before inline comment
Line 187:13: E221 multiple spaces before operator
Line 188:13: E221 multiple spaces before operator
Line 188:48: E225 missing whitespace around operator
Line 189:13: E221 multiple spaces before operator

In the file src/einsteinpy/metric/kerr.py:

Line 64:1: W293 blank line contains whitespace
Line 121:21: E261 at least two spaces before inline comment
Line 156:65: E261 at least two spaces before inline comment
Line 194:21: E261 at least two spaces before inline comment
Line 225:65: E261 at least two spaces before inline comment

In the file src/einsteinpy/utils/kerr_utils.py:

Line 599:1: W293 blank line contains whitespace
Line 600:6: E114 indentation is not a multiple of four (comment)
Line 654:46: W292 no newline at end of file
Line 654:46: W292 no newline at end of file
Line 654:46: W292 no newline at end of file

CI checks will probably not pass, at the moment.

Adaptive Mesh Refinement?

Makes sense. I will start with adding KS Coordinates

Adaptive Mesh Refinement?

This is potentially an advancement in that (AMR) direction. The proposed integrator uses adaptive meshing near problematic areas of the spacetime (coordinate singularities). This should be extensible to #103.

Makes sense. I will start with adding KS Coordinates

coordinates should not see major changes. Am committing the likely changes now.


Diagram update needed

Image still has ‘Data Box Edge’; update to Azure Stack Edge requested.

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

ID: 6a98cba6-7267-3b45-8403-ff4138c1fec0
Version Independent ID: e0ef9a1f-1caa-ed3a-257d-13a6e6136bd7
Content: Choose an Azure solution for data transfer
Content Source: articles/storage/common/storage-choose-data-transfer-solution.md
Service: storage
Sub-service: blobs
GitHub Login: @alkohli
Microsoft Alias: alkohli

@Git-PBradz Thanks for your question. We are checking on this and will respond to you soon.

@Git-PBradz Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@Git-PBradz Thank you for your feedback! The diagram is updated in this PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/131456
I will go ahead and close this issue. Thanks!
#please-close


Support for HRESULT error checking

Should use SUCCEEDED or FAILED macros. Not same as comparing to 0 like winerror does

HRESULT can be emulated with nonnegative + winerror so no need for separate annotation.


Gradio doesn't seem to be up?

Hi. I was trying to test how this works, but the Gradio link seems to be down.

There seems to be a problem with the wget command in Huggingface space. I don't know how to fix it yet, but you can run it locally.

Now, it's available


livecheck PageMatch should allow CSS selector/xpath queries rather than only a regex

Provide a detailed description of the proposed feature
Since Nokogiri is already available, why not make use of it’s HTML.parse and at_css [selector] methods?
Given for example, something like:

  Version

  1.2

it can be awkward to write a regular expression to find the version match, whereas instead, we could have something like:
livecheck do
  url :homepage
  strategy :page_match
  selector "h2.version + #latest"
end

which would actually be a method that looks something like this:
module Livecheck
  def selector(selector) # "h2.version + #latest"
    require "nokogiri"
    doc = Nokogiri::HTML.parse url # :homepage
    version = doc&.at_css selector # "\n  1.2"
    /(\d+(?:\.\d+)+)/.match version&.text&.squish # 1.2
  end
end

giving us the 1.2 version match. Another example would be, given:
Download

we could do:
repo = "github.com/user/example"

livecheck do
  url :homepage
  strategy :page_match
  selector "a[href*='#{repo}'][href$='.tar.gz']"
end

What is the motivation for the feature?
Regular Expressions can be an awkward way to parse HTML.
How will the feature be relevant to at least 90% of Homebrew users?
Many formulae/Casks rely on livecheck blocks. Making them easier/faster to write and get working helps contributors like me, which in turn benefits users.
What alternatives to the feature have been considered?
Alternatively, we could allow using XPath queries, making use of the at_xpath method.
Given the example above:

  Version

  1.2

we could do:
livecheck do
  url :homepage
  strategy :page_match
  xpath "//h2[@class='version']/following::div[@id='latest']" #/text()
end

Although these are less well-known/user-friendly than CSS selectors, they are not mutually exclusive approaches and both are already available with Nokogiri.

CC @samford and @nandahkrishna for thoughts.

I updated the title as this would not apply to HeaderMatch (i.e., it only checks HTTP headers, not body content like HTML).
I've explored using nokogiri within livecheck before, primarily in relation to exceptional situations where a regex may not be the easiest solution (e.g., technically feasible but potentially complex). For what it's worth, nokogiri is in our dependency tree but we don't directly use it anywhere in brew at the moment (we previously used it for XML parsing but we replaced it with REXML and removed nokogiri from the Gemfile).
One thing that's important to mention is that livecheck block regexes serve two purposes: matching text containing version information and using capture groups to extract [version] strings.
From my perspective, a CSS selector can help to match specific elements but the complexity lies in how the version information is extracted. A regex is flexible enough to be able to accommodate a variety of situations and PageMatch simply uses the first capture group from each match (anything more complex requires a strategy block). I don't see us being able to create one approach that would work for all Nokogiri elements in a similar fashion. For example, sometimes we want to work with an element attribute (e.g., href), other times we want to parse the inner text, etc.
For me, this idea has previously taken the shape of an Html strategy (using nokogiri internally) that could use a CSS selector to determine which elements are matched, with the logic for extracting version strings defined in a strategy block. A regex wouldn't be explicitly required but could be passed into a strategy block (e.g., to extract the version from text), if provided.
As an example, one situation that we've faced before where this could potentially be beneficial is when we want to match the inner text of an element without having to work around nested tags:

  Version 1.2.3

A somewhat robust regex matching elements with the version class could look like:
regex(/<[^>]+?class=["'][^"']*?version[^"']*?["'][^>]*?>\s*Version\s+?.*?(\d+(?:\.\d+)+)\s*</im)

Though this would work, the amount of boilerplate just to accommodate ambiguity (e.g., handling other classes before/after version, whitespace, etc.) makes for a pretty gnarly regex. Granted, we would likely just use />\s*Version\s+?.*?(\d+(?:\.\d+)+)\s*</i for the sake of brevity, if we could get away with it in this context.
An alternative implementation using an Html strategy could look like:
livecheck do
  url :homepage
  selector ".version"
  regex(/v?(\d+(?:\.\d+)+)/i)
  strategy :html do |elements, regex|
    elements.map { |element| element.text[regex, 1] }
  end
end

The strategy would take care of parsing the HTML, selecting the elements, and passing them to the strategy block. It ends up being more sane/readable but it may raise the barrier to entry rather than lowering it (i.e., requiring knowledge of CSS selector syntax in addition to Nokogiri's API). If we make some common regexes available within livecheck as constants (I have some local work in the pipeline that I need to wrap up), then we could sometimes omit the #regex call in this particular situation but inevitably we would end up needing #selector, #regex, and a strategy block in some situations.
As another example, let's say we want to match versions from href attributes, which is a very common situation. A regex approach typically looks like:
livecheck do
  url "https://www.example.com/downloads"
  regex(/href=.*?example[._-]v?(\d+(?:\.\d+)+)\.t/i)
end

An Html alternative may look like:
livecheck do
  url "https://www.example.com/downloads"
  selector "a"
  regex(/example[._-]v?(\d+(?:\.\d+)+)\.t/i)
  strategy :html do |elements, regex|
    elements.map do |element|
      url = element.attr("href")
      next if url.blank?

      url[regex, 1]
    end
  end
end

Since this is common, maybe it would just be its own strategy that handles the above logic internally:
livecheck do
  url "https://www.example.com/downloads"
  regex(/example[._-]v?(\d+(?:\.\d+)+)\.t/i)
  strategy :html_links
end

In the end, the only difference compared to the regex-only example is that we could omit href=.*? from the start of the regex, so it doesn't seem worth the additional overhead in common cases like this. [It's maybe worth mentioning that we could technically parse versions from some URLs using Version#parse but it may not be useful, as we would need to restrict matching to appropriate URLs using #selector or #regex anyway.]

All that said, I don't see this as necessarily easier/faster in practice and may just be different (i.e., it trades one type of complexity for another). I'll admit that most contributors probably wouldn't want to touch the lengthy regex above with a ten foot pole but regexes like that are uncommon and it was a fairly extreme example for the sake of discussion.
This could be useful for certain less common situations but I haven't introduced an Html strategy yet because of some misgivings. I'm not convinced that this would be something that we would want to adopt as a default in place of PageMatch and I've been concerned that making it available could fracture the livecheck block ecosystem if we're not careful (i.e., some may use an Html strategy out of preference, even when it's not necessary).
I've put a lot of effort into establishing some sort of standardization of livecheck blocks over time (more so in homebrew/core, as I'm still working on cleaning up homebrew/cask livecheck blocks) and I worry that this could make things worse by reducing overall consistency. I support the general idea of trying to make it easier to write livecheck blocks but this feels like a lateral move rather than a step forward.
Granted, this is just my perspective and I could be overlooking things, so I would be interested in other viewpoints.


Use given params, if any, in Client.make_call

Thanks for sharing this.  I had to make just one small change to Client.make_call so that it would use the params argument, since that's not handled in the URL format string.

Thanks for finding this bug! I've fixed it via the QUERY_STR param in 1b5b745.


driver_lxc: pass flags to shiftfs mount

Link: https://discuss.linuxcontainers.org/t/mount-home-directory-read-only-permission-denied
Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

thanks @brauner

@brauner looks like the tests are failing...

@brauner looks like the tests are failing...

oO weird

Wow, there's a slew of bugs.

Testsuite passed

Thanks, @tomponline!

Thanks for fixing it :)


Fibers

Do you have a schedule for using fibers?
#850

@ziaratban Excellent question!
Fibers are in fact supported in ReactPHP by using the new async package: https://github.com/reactphp/async. Because Ratchet builds on top of ReactPHP, this means it also supports fibers just fine 🚀
I hope this helps :+1:


feat(debian): auto-add an Exec arg (field code) in the .desktop file

This change adds an {exec_arg} field to the default main.desktop template. This field is populated by handlebars with a sane default value, which is based on whether deep_link_protocols or file_associations in the Config struct have been specified.
This allows an installed Debian package to be invoked by other applications with URLs or files as arguments, as expected.
The main.desktop template previously did not automatically include an Exec field code even if deep_link_protocols or file_associations were set, which could potentially confuse new users who incorrectly expected that the Debian package would automatically handle links and file associations upon installation. This change now ensures their expectations are met.

oh, huh, I just noticed that when making changes on my mac, the schema.json files were auto-updated, which didn't happen on my linux box (where I originally made this PR).
Should those schema files be committed, or are they auto-generated by some CI procedure?

Thanks for your contributions


Show ion intensity under cursor

Resolves #281

Includes three unrelated minor fixes:

Correctly calculate the change of offset when zooming in/out of ion images
Fix an issue in the width of DatasetList which made Firefox always show it as only one column
Fix the component name of MainImageHeader. This only showed up in the dev tools, but it was confusing.

Btw, in Chromium the datasets page also has only one column


Polymer new_element should add default style

This issue was originally filed by @zoechi

What steps will clearly show the issue / need for enhancement?
1.
pub run polymer:new_element some-element -o example
2.
3.
What is the current output?
the created Polymer element HTML file contains a style tag with a comment but no actual content.
What would you like to see instead?
I think the style should contain the following content by default
      :host {
        display: block;
      }
Without this style the default is 'display: inline;' which is not the expected behavior most of the time and often causes troubles for people new to styling Polymer elements.
I think it's easier to delete it if someone wants a different behavior than finding the cause for the unexpected behavior. It also demonstrates that this can be changed.
What version of the product are you using? On what operating system?
Polymer 0.12.1
Please provide any additional information below.

This issue has been moved to dart-lang/polymer-dart#274.


Style of pandasgui overwriting style of PyQt5 Gui

When I open pandasgui from my PyQt5 MainWindow the style of pandasgui overwrites the style of the MainWindow.
I tried to work around it by using the block setting and apply my old style to the app after I close the pandasgui, but I get "QCoreApplication::exec: The event loop is already running" when I open it blocked.
Is there another workaround? Can I somehow apply my chosen style to pandasgui while opening the gui?

I stopped the GUI from inheriting existing QApplication instances which should fix this. Install the latest changes with pip install git+https://github.com/adamerose/pandasgui.git and let me know if it's not fixed.

no, did not fix it. It still overwrites my style while opening and i can't block the further execution.
And now the app doesn't exit properly in vscode. I have to terminate Python over the task manager.
However when executing the file from the explorer it closes normally on exit.
(the latest release works just fine)
I think an easy "fix" could be, if i could set the style with the settings option. like:
settings={'style':'windowsvista'}

Okay, made a new commit adding a style option in the settings.
from pandasgui.datasets import pokemon
from pandasgui import show

show(pokemon, settings={'style': 'Fusion'})
show(pokemon, settings={'style': 'Windows'})
show(pokemon, settings={'style': 'windowsvista'})

Also please check that it exits properly in vscode again, I reverted back to using existing QApplication instance

yes, everything seems to works as expected, style option works and it exits properly.
Thank you


Unable to install: Office Pro Plus 2019 with Visio 2019

Hello guys!
I would like some help regarding the Office Installation. I successfully installed office 2019 (including word,excel,powerpoint,access) in my pc but I am not able to install Visio 2019 [different installation].

Does anyone know what should I do?
Thanks a lot for your help.

Sorry you're running into this. Steps for installing Visio are here. If you're following those and still getting an error message, I'd recommend following up with support or our support forums:


BUG: Explicitly give Python library path to ITK Python targets for MSVC

If Python library directory is not given, ITK will not find the debug Python
library when linking. It is required as this library addition is hardcoded in
pyconfig.h in the Python source code.
See https://github.com/python/cpython/blob/master/PC/pyconfig.h#L280
In theory this is only required for the debug build. But since we don't
know at configuration time if this is going to be a debug build or not,
we would need to use generator expressions which would give an empty
path if the build type is not debug. However, this is interpreted as
a relative path by CMake and is not allowed (CMP0081).
Since adding this extra path for non-debug builds is not breaking anything
we add this extra path in all cases.
Suggested-by: Jean-Christophe Fillion-Robin<EMAIL_ADDRESS>

Tests succeeded in Python build #444


"External changes" message after every single change

As of a few hours there's a constant message about a external change having been introduced. I realise that due to changes at Google, collaboration has temporarily become impossible. However, this message comes up even though I'm the only one making any changes. And yes, I checked whether I had the drawing open on any other system or tab (this was not the case). Draw.IO has become impossible to be used, as every little drag, text manipulation or other edit pulls up that window.

If you refresh you should find the dialog has been disabled.


Download CSV from HTML report?

I see that in the HTML report, in the left "gear" menu there's a function to export the data as JSON.
Is there a way to invoke the equivalent functionality, but as CSV?
For what it's worth, I think the customer to whom it's useful to present the nice HTML view is more likely to be able to wrangle CSV (in Excel) than JSON.  At least it is in my case at hand :-).

I agree, having a csv option is useful. Let me take a look at it, it think it shouldn't be to difficult to generate this from the JSON feed, though the only concern is that the JSON object is not completely flat.

Thanks for your response! A couple of suggestions:

Part of the impetus for a CSV export is to be able to browse a panel's data all at once, instead of 10 rows at a time. So a convenient UX for that desire could be a pop up view of only that one panel, expanded to show all rows (and including the same sort/filter features of the current panel).

There certainly are separate motivations to obtain CSVs, for import into Excel or other tools. If non-flatness of the complete page's data presents an obstacle, might it make sense to have CSV download as a feature of the individual panel rather than the page?