fix repeating rest calls

Description

Some refactoring to get rid of duplicate code and multiple identical rest calls

Type of change

[ X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix a feature that knowingly causes existing functionality to not work as expected)

Test and Reproduce
Samples page. clicking around.
Impacted Areas in Application
List general components of the application that this PR will affect:
Which pages?

[ ] Login Screen
[ ] Single Upload Page
[ ] Batch Upload Page
[ x] All Projects Page
[ ] Sample Details Page
[ ] Report Page

Specific components

Testing Script:
###Login Page

Successfully login to the application and get taken to the project page

###Single Upload

Using an AWS path that doesn't exist throws an error
Succesfully uploaded files from a folder (use test host)
Uploading a duplicate file throws an error
Pipeline gets initiated successfully

###Batch Upload

Using an AWS path that doesn't exist throws an error
Succesfully uploaded files from a folder (use test host)
Navigate to batch upload settings page after upload
Host selection from previous page still selected
Uploading a duplicate file throws a warning
Pipeline gets initiated successfully

###All Project Page

Switch between projects
Search for a sample
Navigate through different pages of samples
Select Upload new sample button
Select sample to see generated report

###Sample Details Page

Information is consistent from Project Page
Notes are editable and save successfully
Can navigate to report page

###Report Page

Data loads succesfully
Collapse and Uncollapse button works
Clicking on the taxonomy name gives you total reads for that taxon
Category filter works
Genus Search works
Filters work as expected
Disabling Filters works
Filtering by column works

Checklist:

[ ] I have run through the testing script to make sure current functionality is unchanged
[ ] I have done relevant tests that prove my fix is effective or that my feature works
[ ] I have spent time testing out edge cases for my feature
[ ] I have updated the test script or pull request template if necessary
[ ] New and existing unit tests pass locally with my changes

looks gr8


Add teardown of E2201 - RODRET

Some more info here: https://fcc.report/FCC-ID/FHO-E2201
The part underneath the shield can, as already guessed here, is labelled MG21 - https://www.silabs.com/wireless/zigbee/efr32mg21-series-2-socs

Thank you!

I have not looking under the shield and also not trying read its ID with SDW probe but it shall being possible then the OTA file is Silabs format and i dont IKEA have going for one MFR32MG24.
Also its very likely debug locked so not possible reading the internal flash and RAM but we was fixing that with Symfonisk II with Styrbars bootloader and flashing the OTA update file on one clean module.
Great work done !!


Database Native > Incorrect password entry > returns Connection Error > GSS-API Authentication Exception error > Unable to obtain Principal Name for Authentication

Description
When users enters incorrect password on Database Native authentication, seems the driver is failing and reporting a "Connection Error" with "GSS-API Authentication Error" indicating "Unable to obtain Principal Name for Authentication"
This probably shouldn't be going to Kerberos on a Windows install.
Seen this for a few terms now, it should just say "Incorrect Password"
Windows 10 & 11
MariaDB 11.5.2 & <IP_ADDRESS>
DBeaver EE <IP_ADDRESS>406100905
Connection error
Unable to obtain Prinipal Name for authentication
GSS-API authentication exception
Unable to obtain Principal Name for authentication
DBeaver Version
Enterprise Edition <IP_ADDRESS>406100905
Operating System
Windows 10 and 11
Database and driver
MariaDB 11.5.2 and <IP_ADDRESS>
Driver Settings

Settings:
Driver Type indicates: MySQL (PRO)
Class Name: org.mariadb.jdbc.Driver
ID: mariaDB
Description: MariaDB JDBC driver
Libraries:
MariaDB
drivers/mariadb/LICENSE.txt - C:\Users\bradv\AppData\Local\DBeaverEE\drivers\mariadb\LICENSE.txt
drivers/mariadb - C:\Users\bradv\AppData\Local\DBeaverEE\drivers\mariadb
Local Client
ID/Path/Name: both 11.4 and 11.5 pointing to C:\Program Files\MariaDB 11.5 ...doesn't seem right.
Product Name: MySQL/MariaDB
Driver License
MariaDB JDBC driver is distributed under the GNU Lesser General Public License (LGPL). Text of the license can be obtained from http://www.gnu.org/copyleft/lesser.html.

Steps to reproduce
If password is incorrect on Test Connection:

The following error message gets produced:

...and it should just indicate: Password is Incorrect or Incorrect Password.
Additional context
Just a local install of MariaDB, on Windows, connecting to localhost, no tunnels.  Using 3307 for 11.4, and 3306 for 11.5, the issue predates those port configs.  No SSH or SSL.  If password is correct, the Test Connection yields:

...so the driver might have some issue, as it should definitely not be reporting such a confusing error message like this.

DBeaver gets this error message directly from the driver. We can't handle it and show another message because it might be confusing in other situations when this message is sent, but the password is correct. At the same time, DBeaver doesn't know if your password is correct.
I agree with you, that this error is confusing. And unfortunately, we can do nothing here.


BTLH distribution map

Hi @klprudic, @ezylstra,  and @jcoliver!
I'm working on creating a map with the Broad-tailed distribution containing HMN's monitoring sites.
In the script BTLH_data_exploration, I could read the polygon and make the first map I wanted (map1).
I want to create a second map based on map1 that contains HMN's monitoring locations, but I had some trouble. I added the points, but the distribution colors (breeding and resident) have changed, and there are many colors in the legend. Also, I couldn't add the labels for each location to the map; I tried geom_label and geom_text, but I couldn't make them work.
Any help is appreciated, but we can also go through the code tomorrow. I wanted to try writing an issue and see how it works.
Thanks!

Added code to overlay banding station locations on range map and retain the appropriate aesthetics in 74ceb1d


Propagate options

-
-

Issue
--

Dependencies
--

Decisions
ensure* and related methods should receive options and propagate them. build receives the owner and init expects it

Closing this as using options in singletons is an anti-pattern

The correct fix for the random name in group id for singleton https://github.com/ripe-tech/ripe-bus-api-js/commit/c2fcf779aba91dd785b079ef720dab60837b0799

@joao-conde you can configure it with an env var

The correct fix for the random name in group id for singleton c2fcf77

Was going to do it here but was still dealing with the error which I see you dealt with


Add full beginners course 2024 Brno autumn Tuesday

@encukou @messa @brabemi @frenzymadness Can you review and merge this Brno autumn Tuesday course? Thanks.


[Task - EE]: Change return type for Create Permission Group to RoleViewDTO

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks

Change return type for Create Permission Group to RoleViewDTO

@nsarupr can I work on this issue?


Add ContributingGuidelines.md

The Contributing Guideline documentation will contain set of guidelines that will help contributors during the contribution process.
File type markdown.

@ShwetKhatri2001 could you please assign this issue to me under GSSoC 23
Thank you!

Go ahead @shanvijha30.

@ShwetKhatri2001 Could assign this issue to me.

Added PR #26
Please review it.


Solax update 0.1.0

Description:
Update version of Solax integration pypi package:

fix .../site-packages/solax/solax.py:78 RuntimeWarning: coroutine 'sleep' was never awaited
expose more inverter data, such as serial number and firmware version

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc. (N/A - none added)

Hi! I still can't get my solax integration to work. SolaxClound and "local" login works with the solax app. Can anyone confirm that the API is changed? Or am I just unlycky/stupid? If I use the IP of the inverter in my local network I get a time out. It seems the Rest API is gone. My pocket wifi is version 2.33.7. Help!
/Reik

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. Thanks!


Implement auto page reload

index page should have possibility for auto refresh by either selecting a "refresh" dropdown or hardcoding it into the page itself
Autorefresh SHOULD NOT be implemented for any other page

As a first implementation, make it autoreload every one minute for just the main page. No controls.
Second step: Make a proposal how several values could be selected and activated (30s, 1 min, 10 min, hourly)


add links to unlinked documents; move making-release-notes.md to devel

I noticed that some documents in the docs/ dir are unlinked. So they may easily be overlooked by users.
By the way, I think downward_api.md and container-environment.md talk about the same thing, so indeed, downward_api.md has already been overlooked by us. :)

This broke the build (yes, we have tests that cover our doc :) ). Fix here #10099. There is a precommit hook that will catch this. You can read how to set it up here https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/devel/development.md#hooks

I'm sorry for the problems.
Thanks @mikedanese for fixing it! And for the hint about precommit hooks.


Strobe Light Drivers

There are currently no drivers to control the strobe light on the real board.

The strobe light control is on the same pin as the fault light control, so the strobe light can be controlled by the lights drivers.


A few improvements to vEOS and vMX

Enabling the eAPI (HTTP based) on vEOS.
Opening up the ports 80 and 443 fon vEOS (required when enabling the eAPI).
On newer vMX versions (at least 17.2R1.13 which is the one I am currently
using), the start up might be affected by some error messages, as the image
comes with a pre-defined DHCP configuration. Removing this configuration, the
image starts without any issues, so I am adding the changes for vMX to disable
the DHCP configs, as the management interface needs a statically configured
address.

This is my very first contribution, hope this is going to be useful if accepted. Thanks again for creating this @plajjan - very nice project! Let me know your thoughts.


#144

Merge checklist

[ ] TravisCI tests passed
[ ] Documentation

Codecov Report

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

@@            Coverage Diff            @@
##             master     #152   +/-   ##
=========================================
  Coverage          ?   25.29%
=========================================
  Files             ?        4
  Lines             ?      597
  Branches          ?        0
=========================================
  Hits              ?      151
  Misses            ?      446
  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 e76759a...a49f080. Read the comment docs.


Sample code on azure websites - google authentication

Hi Linda, This is a nice article and very good implementation. Have you ever tried deploying this to any azure websites? I cant get it to work pass this point.
var credential = GoogleWebAuthorizationBroker.AuthorizeAsync(new ClientSecrets { ClientId = clientId, ClientSecret = clientSecret }
, scopes
, userName
, CancellationToken.None
, credentialPersistanceStore).Result;
and the logs weren't very helpful.

I don't have access to Azure so no i haven't tried any of this on Azure.   I can do a little research I know someone that might have done this.

Thanks for your response Linda. You can signup for Microsoft azure websites. They have a free subscription where you can host up to 10 azure websites as of now.
http://azure.microsoft.com/en-us/pricing/details/websites/

I didn't know you could get a free one.  This just got bumped to the top of my list.  It would be nice to have some samples that are verified to work on Azure.

Still can't get this to work with Azure.
var credential = GoogleWebAuthorizationBroker.AuthorizeAsync(
throws a [HttpListenerException (0x5): Access is denied] exception

This might help HttpListener Access Denied

Thank you very much for your suggestion. I don't think I can run netsh
command in azure environment...
On Tue, Jan 3, 2017 at 9:12 PM, Linda Lawton<EMAIL_ADDRESS>wrote:

This might help HttpListener Access Denied
http://stackoverflow.com/a/4115328/1841839
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LindaLawton/Google-Dotnet-Samples/issues/6#issuecomment-270083460,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKTh2ILjrU2y47Y4biUDCV1PEOv-hSYbks5rOh73gaJpZM4FDm01
.

Same issue as #16 and #19
I need to add samples for web Auth.  Currently i am only building for installed applications.


New release to fix inlined huge literals

@harendra-kumar I think #106 is worth a new release.

Sure. That would be a minor release, there is no change in the API.

There will be change in inlining, therefore I would suggest a major bump.


Add socialite authentication for okta

Luckily somebody created a provider for okta some days ago, so I was able to create this pull request without much effort. I only tested with php7.1 and oktapreview.com so far, since I'm not using okta in production it would be nice if someone else could give it a try.
Documentation still missing.

We will test this, need to spin up another Okta app and test instance for PHP 7.1
Thanks for the work!

Thank you @chaseconey for adding this provider to socialite.
And thanks @lommes for this pull request. I will merge in very soon for the next release. Only thing I'll probably change is to convert the SVG to be a full SVG rather than a png embedded in an svg.

Thanks again @lommes, Tested and all works perfectly. Now merged in ready for the next release.
Any changes I made afterwards can be seen here: 90bf13c1ab0642d70e4c101c4a7d30f9403ca6ae
Basically I just re-named the the env keys to align with the other login services and made the SVG fully vector based.


Fix pkg_search_module

Fixes:
CMake Error at CMakeLists.txt:21 (pkg_search_module):
  Unknown CMake command "pkg_search_module".

Hi, have you tested this package on Windows? Usually this line is not necessary for Linux. I've only heard of such problems on Windows. So I am curious about your development environment.

I had the issue on Ubuntu 22.04 in a Docker container

Thanks, I can finally reproduce this issue. I think this issue is not related to Docker. It's just that several Docker images I know have switched to Cyclone DDS. If the environment variable "RMW_IMPLEMENTATION=rmw_cyclonedds_cpp" is set, you will encounter this problem when compiling. As odd as it is, it is true. ROS2 developers have encountered quite a few strange problems with DDS. We'll take a closer look at what this is all about later.

Thanks for adding the directions parameter. While I agree @borongyuan's suggestion to use transmissions is more robust, I am having trouble getting it working.
@lukicdarkoo, I'm finding that after adding the directions parameter and setting the directions of each axis of my diff. drive robot (see below) the FORWARD / REVERSE movements are indeed correct, but the Z axis rotation is showing backwards. The physical robot's wheels are rotating the correct way, but when viewing on RViz2 I'm seeing the opposite direction. Wondering if you have any ideas about this.
...
      <xacro:if value="${enable_joint0}">
        <joint name="${joint0_name}">
          <param name="serial_number">${serial_number}</param>
          <param name="direction">1</param>
          <param name="axis">0</param>
          <param name="enable_watchdog">1</param>
          <param name="watchdog_timeout">0.1</param>
        </joint>
      </xacro:if>

      <xacro:if value="${enable_joint1}">
        <joint name="${joint1_name}">
          <param name="serial_number">${serial_number}</param>
          <param name="direction">-1</param>
          <param name="axis">1</param>
          <param name="enable_watchdog">1</param>
          <param name="watchdog_timeout">0.1</param>
        </joint>
      </xacro:if>
...

Thanks, I can finally reproduce this issue. I think this issue is not related to Docker. It's just that several Docker images I know have switched to Cyclone DDS. If the environment variable "RMW_IMPLEMENTATION=rmw_cyclonedds_cpp" is set, you will encounter this problem when compiling. As odd as it is, it is true. ROS2 developers have encountered quite a few strange problems with DDS. We'll take a closer look at what this is all about later.

Good catch, I was using Cyclone DDS but I didn't think it was worth of mentioning.
@avielbr I implemented reversing and it was working fine, but deleted it eventually as it makes the implementation ugly. I simply switched the wires on ODrive


Fast iterative Feature adversaries

It looks like there aren't any tests, can you add at least one test per new function / class?

Yeah. Thanks. This PR is unfinished. I'm resolving the issues.
On Thu, Oct 5, 2017 at 12:33 PM, Yen<EMAIL_ADDRESS>wrote:

@yenchenlin commented on this pull request.
In cleverhans/feat_advs.py
https://github.com/tensorflow/cleverhans/pull/283#discussion_r142990386:

If clipping is needed, reset all values outside of [clip_min, clip_max]

if (clip_min is not None) and (clip_max is not None):

   adv_x = tf.clip_by_value(adv_x, clip_min, clip_max)

return adv_x

+class FastFeatureGradient(Attack):
+

"""
This is similar to FGSM but done for the internal representations.
"""

def init(self, model, back='tf', sess=None):

   """

   Create a FastGradientMethod instance.

This should be Create a FastFeatureGradient instance, right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/cleverhans/pull/283#pullrequestreview-67436751,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG-A-c3Pn5yyR2bUEkkbMFJ6XE879lhfks5spQThgaJpZM4PlLap
.

Thanks for the comments. I have addressed the comments except for the one regarding writing tests. The example in imagenet_featadvs/ is supposed to be a very basic test which is not converging to 0 right now. Let me know if you have any ideas for unit tests and if you found any bugs through reviewing.

The tests are failing with "from utils_tf import clip_eta" getting an ImportError

@fartashf are you able to take a look at this PR in the next couple of days?

@npapernot thanks for the reminder. I fixed the issues and added a test. The tests fail for test_mnist_tutorial_tf.TestMNISTTutorialTF. I guess it's again due to a tight threshold.

Quoting the failure here:
Traceback (most recent call last):
File "/home/travis/build/tensorflow/cleverhans/tests_tf/test_mnist_tutorial_tf.py", line 27, in test_mnist_tutorial_tf
self.assertLess(report.train_clean_train_adv_eval, 0.036)
AssertionError: 0.036799999999999999 not less than 0.036

Since this changes FastGradientMethod it does seem possible the failure is real.
Notes for my future self / other people following the thread: this PR changes FastGradientMethod to call clip_eta rather than having its own version of the clip_eta code internally. This is a great refactor (thanks Fartash!) but I'm concerned that there's something subtly different between the two versions that we don't know about.
Fartash: are the tests failing on your machine too or just on Travis? It looks like they fail only for newer version of TensorFlow. Do they fail consistently or flakily? From one run of Travis it looks like we have 4 trials that all support the hypothesis that the tests fail consistently for TensorFlow > 1.1. Since they consistently passed before, this looks like something did meaningfully change in clip_eta.

My bad for mixing multiple PRs. This PR is actually not changing the FastGradientMethod. It only refactors BasicIterativeMethod to use clip_eta. This should not affect the tutorial. clip_eta was introduced in 47d85a6d73401d60f21be0653bf7bae78b4c459b and later was moved to utils_tf.py in 766add490bef3fc48393b4056973f62ace3e1f55.
Tests don't fail on my machine. I tested with Tensorflow 1.4.0 using python 2.7.14 and 3.6.3.

Are you sure the test only takes 19 seconds? The tests for this PR took up to 33 minutes to run but if I look at another currently open PR they're consistently under 26.
https://travis-ci.org/tensorflow/cleverhans/builds/314515713?utm_source=github_status&utm_medium=notification

This one is also just barely over 26 for only one of 4 runs: https://travis-ci.org/tensorflow/cleverhans/builds/314921361?utm_source=github_status&utm_medium=notification

Thanks, it does look a lot faster now.
Unfortunately, we now have two more examples of the tensorflow 1.1 tests failing, and by exactly the same amount of error as before:
Traceback (most recent call last):
File "/home/travis/build/tensorflow/cleverhans/tests_tf/test_mnist_tutorial_tf.py", line 27, in test_mnist_tutorial_tf
self.assertLess(report.train_clean_train_adv_eval, 0.036)
AssertionError: 0.036799999999999999 not less than 0.036
Since this isn't happening to pull requests in other branches, I do think there's something unusual about this branch.
One thing you might try is to rebase your branch on top of master and run the tests locally. If the rebase gets the tests to fail locally, that would give you some more convenient signal for debugging. You'll need to push -f back to the pull request after the merge.

I rebased and the tests still pass on my machine.
To make the PR simpler, I separated the part regarding clip_eta into a new PR #326. We will get a clue after that one is tested.

Thanks! Feel free to push -f the remainder of the code to this branch. If we're lucky, it will just pass.

The checks have passed. I think it is only because of randomness in the tests. The only change I made was moving part of the code to PR #326.

Thanks!


Give chat timestamp a fix width to fix message alignment issues

Fixes https://github.com/thelounge/lounge/issues/1581.
I gave 5px headroom to the timestamp width I get on my laptop. Tested with/without seconds, on Chrome/Safari/Firefox on Mac, Chrome on Android. Would be nice if someone on Windows and/or Linux could test, just to be sure.

Before
After

Yay for fixing this! But is there a way to fix the width relative to the font size? I can't get in and play around right now, however it seems like something around 5-6em would do unless I'm forgetting something important about CSS sizing…

That could work. What do you think, @xPaw?

@dgw, after chatting with @xPaw, we decided to go for pixels as a quick fix for #1581 because we're not 100% sure how that would behave under different fonts, sizes, etc.
That being said, if you or anyone wants to give it a try, we'd totally consider it in the next release while keeping the quickfix for this release.
I'm merging that now to be able to release v2.5.0-rc.2 right now, but feel free to object and we could totally discuss it before v2.5.0 happens if that seems unwise to you!

Eh, I don't think there's anything wrong with pixels unless font sizes are going to change a lot. This isn't something that would be worth holding back v2.5.0 stable for. If I (or anyone) get around to playing with it in time, fine; if not, also fine!


Research Papers Available?

Love your project! Super excited to explore it more with my data. I think it's a super exciting direction for the field.
In the docs you mention two pieces of research - Text-Grad 2.0 and Learn-to-Reason Few-shot In Context Learning. I'm trying to learn more about the general concepts and ideas behind this project. Are these papers available or will they be soon? If not, what is the best resource for understanding the methodology behind the code?

@liechtym Thank you. We are still working on benchmarking and more complicated system, will come! For now, you can check the use cases: https://adalflow.sylph.ai/use_cases/index.html where you get to see the optimizer in actions on various applications.


[stable/percona-xtradb-cluster] move files outside configmaps

Having the scripts and default config file embedded in the configmaps
makes them harder to work with / indent correctly / etc.
By moving them to external files and then having helm pull them in as
needed makes it easier to work with.

Thank you for contributing to kubernetes/charts. Before you submit this PR we'd like to
make sure you are aware of our technical requirements and best practices:

https://github.com/kubernetes/charts/blob/master/CONTRIBUTING.md#technical-requirements
https://github.com/kubernetes/helm/tree/master/docs/chart_best_practices

For a quick overview across what we will look at reviewing your PR, please read
our review guidelines:

https://github.com/kubernetes/charts/blob/master/REVIEW_GUIDELINES.md

Following our best practices right from the start will accelerate the review process and
help get your PR merged quicker.
When updates to your PR are requested, please add new commits and do not squash the
history. This will make it easier to identify new changes. The PR will be squashed
anyways when it is merged. Thanks.

/assign

/ok-to-test

/lgtm


How to unbind data-remote events

Hi everybody,
I need to unbind data-remote events. (basically I want the click to happen with no-ajax and open a new window).
ajax:before returning false does not work as it would cancel the click event which I still want to happen.
Any idea how to do the below?
if (certain condition) {
$(''a[date-remote=true] //disable/unbind rails click events on this selector?
}

I have been trying to use
$(document).undelegate('.selector', 'click.rails')

and have yet to get it to work. The code above
$(document).undelegate($.rails.linkClickSelector, 'click.rails');

works but it unbinds everything. Another I don't total get is that jQuery recomends using on rather then delegate so it might be a bug with the newer versions of jquery.

@newdark, I'm not in front of my computer, so I can't check and I don't remember off-hand, but I'm not sure it's possible to unbind a handler for a subset of the selectors that were bound.
Using delegate over bind won't cause any bugs, as they use the same underlying code, and the jquery core team has stated that delegate won't be going anywhere since it's only one line of code in the jquery codebase. I personally prefer delegate over on, as it's clearer with on's API which will bind with two arguments and delegate with three arguments (thus making it difficult to see at a glance what is actually happening with code that uses on). But that's basically the only reason we use delegate, so there's nothing more going on there.


Compute Task Pool panicked at attempt to add with overflow - github build

Hello!
I was using the block mesh library  directly from github using block-mesh = {git = "https://github.com/bonsairobo/block-mesh-rs"} in my Cargo.toml file.
Problem: i get the error thread 'Compute Task Pool (5)' panicked at 'attempt to add with overflow', /home/yairama/.cargo/git/checkouts/block-mesh-rs-d08695c2c9849c77/71aa5de/src/greedy/merge_strategy.rs:71:28 (the compute task pool  number varies for each build)
the error is generated by the greedy_quads funcion:
    greedy_quads(         &voxels,         &ChunkShape {},         [0; 3],         [CHUNK_SIZE_U32-1; 3],         &faces,         &mut buffer,     );
(ChunkSize is 32 and ChunkShape is ConstShape3u32<32,32,32>
** The error only apperas with cargo run command, when i use cargo run --release it runs perfectly  **
Also when i use the crate io build ( block-mesh = "0.2.0" in my Cargo.toml file ) it runs perfectly.

Nice find. I think that should be a wrapping_add. I'll cook up a fix real quick.

This should fix it!
https://github.com/bonsairobo/block-mesh-rs/commit/793c53e82c12b3f7e9502006664c6e81f8b1c78a


CGO Wrapper for adbc_driver_manager does not Implement all ADBC Methods

The following methods of ADBCConnection do not have implementations in the CGO wrapper for adbc_driver_manager:

GetObjects
GetTableSchema
GetTableTypes
Commit
Rollback
ReadPartition
SetSubstraitPlan
BindStream
GetParameterSchema
ExecutePartitions

I've opened a draft PR for GetObjects in #1290


jsx-indent rule fixer won't replace spaces with tabs and vice-versa

The existing implementation of jsx-indent rule autofixer won't replace spaces with tabs and vice-versa. The tests which will fail for this case are in tests/lib/rules/jsx-indent.js and commented out

the rule is now called "react/jsx-indent", but the issue still stands


fig 16, increment by 5 or 10

This is 2D-gaussian distributed node + 2 block SBM. Even though I increased the number of nodes by 5,  power goes almost zero to 1 suddenly. We should discuss it soon.


Is it possible to run commands after cloning or installation?

Hello again! I have a feature request to run a command after cloning or installing a plugin, similar to atclone with zinit, or tag hook-build of zplug.
I briefly looked at the antidote's documentation and not sure if it has any relevant features.
For me, I want to automatically add a symlink inside the plugin directory after it is installed, rather than doing it manually. Is there a viable way?
Related: https://github.com/DFurnes/purer/issues/12

I'll close this for now and track it here: https://github.com/mattmc3/antidote/issues/12

@mattmc3: None of the antigen family of plugin managers (of which I consider antidote to be a part of) have these kinds of hooks, however I'm not opposed to it, it's just not on any near term roadmap for antidote.

Sorry for bothering a closed issue. I am not sure "these kinds of hooks" you mentioned means the zsh hook functions or a more general hook function (https://github.com/mattmc3/antidote/issues/12) for plugin manager.
If you meant hook function for plugin manager, antigen did implement a antigen-add-hook, and I have used it to implement hooks that will be executed pre-apply & post-apply (https://github.com/LuckyWindsck/dotfiles/issues/1#issuecomment-1684739810).

@LuckyWindsck - it's not well documented, but a hook feature exists in the form of pre and post annotations. For example:
$ antidote bundle zsh-users/zsh-autosuggestions pre:func_do_before post:func_do_after
func_do_before
fpath+=( $ANTIDOTE_HOME/zsh-users/zsh-autosuggestions )
source $ANTIDOTE_HOME/zsh-users/zsh-autosuggestions/zsh-autosuggestions.plugin.zsh
func_do_after

It's up to you to write the pre/post functions (ie: func_do_before and func_do_after in this example). There aren't hooks for pre/post clone and pre/post update, however, though I'm not sure they're strictly useful. Never having needed or used antigen's hook feature myself, and with antibody not having anything like it, I didn't want to go too deep down this rabbit hole without a user who could properly describe what they needed from a feature like this. Let me know if the pre/post annotation is sufficient.


Add basic support for hoyoverse account

I try to implement some functionalities for hoyoverse account. Most things work well except for getting wish history by stoken since stoken is not used currently. Is stoken necessary? Cookie fields ltuid, ltoken and cookie_token are used for requesting API.
Moreover, could you add a field just like IsOversea in User.cs? I am poor at handling database and need help. There is no flag in user entity to determine whether it is an oversea user. For testing purpose, I use a bad trick to indentify those oversea accounts by checking if stoken==null.
For those who live in china mainland, you may need a proxy to access those oversea APIs properly otherwise a 403 forbbiden error would occur.

I believe there is stoken in hoyolab.
We need to somehow figure out how to get it and use it.

@Xhichn Hi, the property you requested has been added
in this commit https://github.com/DGP-Studio/Snap.Hutao/commit/a8a33542170a84610e178cbbe87a902bec267b1d

We can get stoken by login_ticket and user id by requesting such url, no DS is required
GET https://api-account-os.hoyolab.com/account/auth/api/getMultiTokenByLoginTicket?login_ticket=&uid=&token_types=3
login_ticket can be found in cookie after signing in account.hoyolab.com. I haven't found something like stoken V2 for now from the packets captured from Hoyolab App. Maybe there is difference between the stoken used in miHoyoBBS and HoyoLab.

@Xhichn  Hoyolab Version 2.28.0
[OSK2]: 599uqkwc0dlqu3h6epzjzfhgyyrd44ae
[OSLK2]: rk4xg2hakoi26nljpr099fv9fck1ah10

Maybe you guys can check on https://github.com/thesadru/genshin.py about hoyolab authentications

Notes
Cookie on account.hoyolab.com has login_ticket but has no login_uid. On the login page, users need to manually input their user id which is required for requesting other token.
No mid is found in cookie. Assign account_id to mid for user identification temporarily.
Functions should work:

User

Login from website
Add user from cookie input (stoken & stuid)
Refresh cookie token

Spiral Abyss

Get record
Upload

My Character

Get info from hoyolab
Get info from enka api
Get info from hoyolab dev plan
Add dev plan

Daily Note (Real-time note)

Sync from hoyolab

@Xhichn Thanks for your contribution!!!

@Xhichn Hi, I’ve merge this PR into our main branch and made several changes
Could you do some test to ensure all function you added still works after my refactor?

@Xhichn Hi, I’ve merge this PR into our main branch and made several changes
Could you do some test to ensure all function you added still works after my refactor?

@Xhichn Hi, I’ve merge this PR into our main branch and made several changes
Could you do some test to ensure all function you added still works after my refactor?

OK，I will try that later.

@Lightczx Most functions works well but Hoyolab user creation is broken. I will make a new pr soon.


[Question]: Does the fronend support LLM streaming?

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://docs.all-hands.dev/modules/usage/troubleshooting
[X] I have checked the existing issues.

Describe the bug
The assistant's reply is showing up in one piece rather than streaming out in the chatting box.
I have checked the doc: https://docs.all-hands.dev/modules/usage/intro and most of the source code without seeing anywhere to set "stream=True".
I thought this was a feature ( reply in non-stream style in frontend ) before finding this offical demo in which the words are streaming out. (https://github.com/OpenDevin/OpenDevin/assets/38853559/f592a192-e86c-4f48-ad31-d69282d5f6ac)
so, should the word streaming out in the chatting box in the frontend? or have I just missed some configuration? Thanks
Current OpenDevin version
0.8.0

Installation and Configuration
started with "make run"

Model and Agent
ollama:qwen2:latest
Operating System
centos7
Reproduction Steps
No response
Logs, Errors, Screenshots, and Additional Context
No response

Streaming the LLM output is not implemented, yet.

If I recall correctly, OD never had LLM streaming, so I'm not sure what videos you are refering to?

And just to make sure: this repo is "OpenDevin", not "Devin" 😁

If I recall correctly, OD never had LLM streaming, so I'm not sure what videos you are refering to?

This is the demo video I mention above, I found it from the doc page
And this is another video in youtube showing "streaming words"

And just to make sure: this repo is "OpenDevin", not "Devin" 😁

Oh yes, I am talking about "OpenDevin".

Oh, I see now! Wow, when I first tried OD myself, that was already gone.

Oh, I see now! Wow, when I first tried OD myself, that was already gone.

What do you mean by "already gone"? They have supported text stream in frontend once but not anymore?

That old video wasn't streaming the LLM output, it was a hard-coded experiment: it got the whole response from the LLM, but the frontend was typing it letter by letter like streaming would look like. It wasn't very stable either. We have eventually concluded it will be better to have real streaming. ☺

Thank you both @enyst @tobitege  for answering my questions and contributions to this project. Now I got the answer and this issue will be closed.


Fix 3055

Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues
Fix https://github.com/apache/skywalking/issues/3055

+1 for code scan. I also think, we should check all ContextManager#activeSpan in plugin, and decide whether we should @Deprecated or delete. Let's check that and talk.

As I know, ContextManager#activeSpan() is used for handleMethodException

I think it cannot be deleted. Because it is useful sometimes.

From the current failures, I am thinking about, is this right using ContextManager#activeSpan() even for logging error? It could cause an NPE too, right?

Not NPE. It throws 'No active span.' exception when ContextManager#activeSpan() called.

Not NPE. It throws 'No active span.' exception when ContextManager#activeSpan() called.

But the exception should not happen in agent, it costs too much.

I entirely agree with you. Maybe I have a mistake. I mean we need ContextManager#activeSpan() in other scenario.

I scanned the code and found only the two places, which are obvious usage errors.


upgrade nginx version to 1.16.0

merge with head repository

cannot be done easily, things changed a lot in nginxinc:master


Downloads (Outdated shell script)

Hi there :hugs:
I just installed Gitea after a while and thought the installation process got worse, so I decided to write a short issue:

The current implementation lets me decide between "gitea" "theme" and "dl.sh"

The third item here on the list is an installation script. Great, it could be the default download link, since this is what I guess 99% of all users want.
Sadly, its out of date, so far as I can see. No younger version as 1.4 is mentioned.

"gitea" leads me into an imho convoluted S3 tree and Gitea itself provides neat UI options for releases. Dog food..

The binary is this tree is not signed as executable and also nowhere described as something that has to be marked so. I guess loads of people cant scry this and dont know about this mechanism beforehand.

Thanks a lot for everything

Thank you for the ticket. dl.sh is a script that helped up bootstrap our S3 software archive by fetching all the files that existed on Github so we could self-host all of our generated assets, not an install script. For sake of clarity I have since removed that script as it was only used once, and isn't for the end user.
As this script has been deleted from our hosting, I will close this ticket.

Just for clarification: The binary is still undescribed, the installation process still a bit tiresome.
Thanks a lot


[STM32L4] Fix IAR section placement failed error

For some reason STACKHEAP block was placed in SRAM2 section which lead to
Error[Lp011]: section placement failed - unable to allocate space for sections/
block error. This patch modifies the STM32L4 linker script and places STACKHEAP
into SRAM1 section which was previously unused.
Change-Id: Ibe6ca52a9fe7af232a3eade2f6b1f2ce28c9bd49

Please sign http://developer.mbed.org/contributor_agreement/

Done

Isn't there nucleo that has the same issue?


no funds recipient

Current dependencies on/for this PR:

main

PR #147

PR #148   👈

This comment was auto-generated by Graphite.


Automation_ACC-264.1 (Level A)Kyma-dashboard_Input Field Provide a valid label for form fields.

1.1 Internal Message Details
Internal Message:
Test Plan:	ACC SAP Kyma runtime 2024
Message Priority:	Medium
Affected Check Point:	ACC-264.1
Affected UI:	Input Field
Short text:	Automation_ACC-264.1 (Level A)Kyma-dashboard_Input Field Provide a valid label for form fields.
1.2 Application Details
Application Name	Kyma-dashboard
Application URL:	https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview
User:	SSO
Technology:	SAP UI5
UI5 Version:	1.124.1
1.3 Test Environment Details:
GitHub	https://github.com/kyma-project/busola/issues
Web browser:	Google Chrome - Version 127.0.6533.73 (Official Build) (64-bit)
Operating System:	Windows 11 Enterprise Version 22H2
Screen Reader:	JAWS 2024.2405.83 ILM 64 bit for Windows
2. Screen Flow
•	Login into the application using the URL provided.
•	URL: https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview.
•	Activate ‘Extension’ item.
•	Activate ‘Create’ button.
3. Issue
Synopsis: Provide a valid label for form fields.
Observed Behavior:
‘Resource’ INPUT (role=combobox) field in ‘Extensions’ page, does not have a mechanism that allows an accessible name value to be calculated.
Code Snippet:

Expected Behavior:
Developers should define form labels using the label element whenever an on-screen label is present. The label element's value is then explicitly associated with the relevant form field using the for and id attributes. Use of the label element provides an added benefit as it increases the target area that users can click to focus the form field. In certain instances, however, it is impractical or 'undesirable' to use an explicit label for a form field; this is largely driven by the fact that the value between the opening and closing label tags must, by definition, be a block of displayed text.

Kindly refer to attached error and issue processing document for more details.
Regards,
Huda

Automation_ACC-264.1 (Level A)Kyma-dashboard_Input Field Provide a valid label for form fields.docx
Issue processing-09.2021 (1).docx


Adding in convergence testing

Trying to add in convergence tests into asy_peakbag.  Running tests to see if possible.
Linked to issue #21

While I'm here I'm gonna refactor the code to  put the MCMC class in its own subpackage and then use the new class to run the mcmc in epsilon.

I have added in some basic convergence testing following the emcee==3.0 docs.  It is still fairly rudimentary.
I have also refactored and made an mcmc subpackage.  I have also added a trick where asy_peakbag starts from the samples produced by epsilon (which is the equivalent of sampling the prior before turning the likelihood on).
Still todo:

[ ] Documentation.
[ ] Run docs examples.
[ ] Fix conflicts.
[ ] Update the tests.
[ ] Be awesome.

I'll work on this ASAP.


Issue 1775: Upgrade to Cassandra to 4.0-beta2

Pull Request Description
This pull request closes #1775
Acceptance Test

[ ] Building the code with mvn clean install -Dintegration.tests still works.
[ ] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[ ] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[ ] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[ ] No

Code Review And Pre-Merge Checklist

[ ] My code follows the coding convention of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.

@carlspring the conflicts is because of issues/1649 was rebased and it affected on this branch because it was based on old issues/1649 commits


Tests for Calculate version 2

Fixes #75.

Codecov Report

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

@@           Coverage Diff            @@
##             master     #85   +/-   ##
========================================
  Coverage          ?   62.1%
========================================
  Files             ?       2
  Lines             ?      95
  Branches          ?       0
========================================
  Hits              ?      59
  Misses            ?      36
  Partials          ?       0

Support for tests and continuous integration has been set up for Calculate. From now on, the writing of new tests to cover the full library will be carried on another feature branch since this is too much bloated already.


Backup of Managed Instance is not clear

It is not clear from the page how the backup of Managed Instance works. For example, "What is a SQL Database backup?" paragraph states that "SQL Database uses SQL Server technology to create full backups every week, differential backups every 12 hours, and transaction log backups every 5-10 minutes" However. on other microsoft pages I'have read that differential backup and transaction log backups are not supported for SQL Managed instance, only Full Backup.  Also, the same page it says that the long term retention also not automatically supported and there is a limit of 4TB to the backup size.  https://docs.microsoft.com/en-us/azure/sql-database/sql-database-managed-instance-transact-sql-information#backup
Please clarify

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

ID: 9492cedc-cea0-af9d-29a0-52cbca8c5e21
Version Independent ID: 969a8654-8f95-d5de-351d-73c8e02e5d3a
Content: Azure SQL Database automatic, geo-redundant backups
Content Source: articles/sql-database/sql-database-automated-backups.md
Service: sql-database
Sub-service: backup-restore
GitHub Login: @anosov1960
Microsoft Alias: sashan

@sternlu Thank you very much for the detailed feedback. I am working with the content team to have a PR submitted and approved that helps provide some clarity.

The Automated backups document (link) discusses the back-up process for two distinct deployment options for "SQL Database" in Azure. Azure SQL Database (Singleton and Elastics Pool) and Azure SQL Managed Instance. Azure SQL Database is provisioned as part of a federated cluster where you have been provisioned a single node serviced through a GW. Managed Instance is SQL Server provisioned on an Azure VM and is almost functionaly the same as SQL Server. Azure SQL Database has a subset of SQL Server functionality.
When the doc refers to SQL Database it is referring to Azure SQL Database PaaS but where applicable, references a specific deployment option. Please see Choose the right SQL Server option in Azure (link) for more information with regard to the specific deployments of SQL in Azure.
Please let us know if this is helpful. ~Mike

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


docstrings/sphinx: Support getting annotations/defaults/values from parent for "Receives" section

Is your feature request related to a problem? Please describe.
The goal is to have a common set of features between all parsers.
Describe the solution you'd like
The Sphinx parser should be able to retrieve annotations/defaults/values from the docstring parent for "Receives" sections.

Closing for cleanup, but if you'd like to work on this, just ask and I'll reopen!


Discussion: Why use ngrx/store instead of Redux?

Redux is extremely popular and currently has over 19,000 stars on github.  So why did you guys decide to use the less popular ngrx/store instead of Redux for state management?

Redux is really just a philosophy of state management which is behind ngrx/store and provides a nice api when used in conjunction with Angular. You can use either, the principles are the same.

I could open this up for discussion here but it's really best left to discuss on ngrx/store repo where discussion has already taken place. As mentioned there, some of the Redux (TM) toolilng is tied to React which is another reason ngrx/store is better suited here.


How to make the cells in a column have drop-down options?

How to make the cells in a column have drop-down options? Now can only make the specified cells have drop-down options

@alonglive  I don't understand your purpose, can you provide a screenshot😄 ?
If you want to add a drop-down list, https://github.com/exceljs/exceljs#data-validations-

I want to make the whole column have drop down
for example：Now i can only do like this
worksheet.getCell('A1').dataValidation = {
  type: 'list',
  allowBlank: true,
  formulae: ['"One,Two,Three,Four"']
}

I want to do this, but it's didn't work
worksheet.getColumn('A').dataValidation = {
  type: 'list',
  allowBlank: true,
  formulae: ['"One,Two,Three,Four"']
}

Exceljs does not support this feature, and may not support it in the future, maybe you need to use the loop setting, good luck😄.
  for (let i = 0; i < length; i++) {
    worksheet.getColumn(`A${i}`).dataValidation = {
      type: 'list',
      allowBlank: true,
      formulae: ['"One,Two,Three,Four"']
    }
  }

Please re-open if you still hava thoughts.

Hey, Can we pass the Dynamic dataset in formulae:
I am trying to call it but it is throwing an error while opening the excel:
`let adb = Company_Code_Data.map((item) => {
var fullname = [item.Company_Code].join(" ");
return fullname;
});
            let Company_Code_list = '\'\"'+adb.toString()+'\"\'';

            console.log(Company_Code_list);

            newSheet.getColumn('Company_Code').eachCell((cell) => {
                cell.protection = {
                    locked: false
                }
                cell.dataValidation = {
                    type: 'list',
                    allowBlank: false,
                    operator: 'equal',
                    formulae: [adb.toString()],
                    showErrorMessage: true,
                    errorStyle: 'error',
                    errorTitle: 'Invalid value',
                    error: 'The value must be C&W, Pro Unlimited, Concur, AWS, Azure, Legal, Other'
                }
                cell.fill = {
                    type: "pattern",
                    pattern: "solid",
                    fgColor: {
                        argb: "FF85c8ff"
                    }
                }
            }); `

@garvitjoshi You need to transform your array in a string with join let joineddropdownlist= '"'+adb.join(",")+'"'; and use joinedddropdownlist on formulae; formulae: [joineddropdownlist]. At least Worked for me


Add tests converting dab files

Resubmitting to work around conflict with tests.sh

@ngtuna Is it possible I could get a quick +1?

@cab105 👍 LGTM

Thanks!


[MIGRATION] Changes for AdvancedApplicationConfiguration component in application REST API

Describe:

In Application REST API,
the property named skipConsent under  AdvancedApplicationConfiguration, renamed to skipLoginConsent and skipLogoutConsent property is added to resolve issue https://github.com/wso2/product-is/issues/7935.

Changes are done in: https://github.com/wso2/identity-api-server/pull/127/

Affected when migrating to: 5.11.0

Tracked with https://github.com/wso2/product-is/issues/9642


fix: display max_fee as int

Closes #410
Introduced changes

display max_fee as int unless --hex-format is specified for it

Breaking changes

Checklist

[x] Linked relevant issue
[x] Updated relevant documentation
[x] Added relevant tests
[x] Performed self-review of the code
[x] Added changes to CHANGELOG.md

@THenry14 i see there is a --int-format flag in sncast, wouldn't it be confusing to use --hex-format along with that?

@THenry14 i see there is a --int-format flag in sncast, wouldn't it be confusing to use --hex-format along with that?

They should be mutually exclusive - if someone passes --int-format all outputs should be in int format; when --hex-format all outputs should be in hex format; when someone passes both flags we should error out (I believe clap has such feature) and if no flag is passed we output addresses/hashes as hex and fees as ints. Does that make sense?

when --int-format is not specified we are already using hex format.
i think the problem is that even when using hex format (i.e. --int-format is not specified) we want to print max_fee as int, and give an option for hex format (just for max_fee) inside sncast account create subcommand

note: --int-format that i mention is argument available on sncast and --hex-format i have added in this PR is on sncast account create subcommand

when --int-format is not specified we are already using hex format.

yes, and we want to have this default behavioour changed to display addresses/hashes as hex, but fees as int - not hex

i think the problem is that even when using hex format (i.e. --int-format is not specified) we want to print max_fee as int, and give an option for hex format (just for max_fee) inside sncast account create subcommand

note: --int-format that i mention is argument available on sncast and --hex-format i have added in this PR is on sncast account create subcommand

I suppose this flag should be global and not to an account subcommand exclusively - plus it should change every output value to hex, as I described in previous comment. This way, by default, we have all the addresses/hashes in hex and fees as int (as it probably should be), but if someone wishes so one could change the outputs to ints/hexes exclusively (be it for scripting, to avoid manual conversion for contracts input etc). I believe that should be a behaviour that would suit more usage scenarios, and it could even potentially be simpler to implement.
so, a tldr:

--int-format - show everything as int
--hex-format - show everything as hex
no flags - show addresses and hashes as hex, fees as int
both flags - error

Does that clarify things?

okay got it, thanks for the detailed summary!
This will be bigger change then i initially thought it to be as it would affect every part of sncast. So to test out things i would need to setup account contracts and stuff so i can run each and every subcommand.
for e.g. i was trying to run sncast account create subcommand by specifying my own keystore and account (that was created using starkli) but it was giving some errors. I also tried using katana as local devnet which was giving some other error.  (i didn't look to deeply into it)
is there any guide on setting up accounts and local devnet to test out this things?

okay got it, thanks for the detailed summary!
This will be bigger change then i initially thought it to be as it would affect every part of sncast. So to test out things i would need to setup account contracts and stuff so i can run each and every subcommand.
for e.g. i was trying to run sncast account create subcommand by specifying my own keystore and account (that was created using starkli) but it was giving some errors. I also tried using katana as local devnet which was giving some other error. (i didn't look to deeply into it)
is there any guide on setting up accounts and local devnet to test out this things?

I think just running the script prepare_for_tests.sh as detailed in contribution guidelines https://github.com/foundry-rs/starknet-foundry/blob/master/CONTRIBUTING.md should be enough to get you going. Make sure you have installed asdf (for scarb versioning) and starknet-devnet first, and you can then just run tests (eg cargo test  -p cast, you can also filter out tests as per cargo docs). Let me know if you run at any issues :)

i tried installing starknet-devnet via pip but it failed with some error. will the new starknet-devnet-rs work instead?

i tried installing starknet-devnet via pip but it failed with some error. will the new starknet-devnet-rs work instead?

Make sure you have libgmp3-dev (linux) or gmp (mac) installed, and then please try again. the new starknet-devnet-rs still has some missing features and issues - it could work for you for some simple tests, but most of our tests from the test suite will fail

i do have it installed, i think the problem is that i have newer python version then what starknet-devnet supports. i will try this on a different system in sometime.

i do have it installed, i think the problem is that i have newer python version then what starknet-devnet supports. i will try this on a different system in sometime.

I see. you can also try installing pyenv for python version management.

pyenv with virtualenv did the trick!

(rebased changes to master)

@lambda-0x please fix the issues found by clippy 🙏 after that, I think this PR is ready :+1:


dynamic dropdown for spesific address

Hello!
For example, I have country and city data,
in the country there are: id, name.
Then in the city there are: id, name, country_id.
how do i make the dropdown dynamic when i select a country with id=1 it will bring up the city city with country_id =1

Thankyou

<multiselect v-model="form.contry"                    :options="countries.map(type => type.id)"                   placeholder="Choose country"                   :custom-label="opt=>countries.find(x=>x.id==opt).name">   </multiselect>
<multiselect v-model="form.city"                    :options="cities.map(type => type.id)"                   placeholder="Choose city"                   :custom-label="opt=>cities.find(x=>x.id==opt).name">   </multiselect>

Hi there
To achieve this, in your city select, you would filter your options based form.country. For example, you could have a Vue Computed Property that filters your cities based on your form.country selection. Then, in your options prop of your city selector, you can pass the computed property.
Hope this helps


Error 503

For a few days now, when I try to log in, I get the following error: Service 503 temporarily unavailable and a CORS error.
Does it just happen to me? Any solutions?

What function call was this returned by?
I'm guessing the server was down.
Generic HTML definition of error 503:

The 503 Service Unavailable error is an HTTP status code that means the website's server is simply not available right now. Most of the time, it occurs because the server is too busy or because there's maintenance being performed on it.

yes, I thought so, but I still get that error for days.
I'm simply doing this, in a react hook:
React.useEffect(() => {
  const newConnection = new ewelink({
    email: "<myEmail>",
    password: "<myPass>",
    region: "eu"
  });
  newConnection.login();
  setConnection(newConnection);
}, []);

All this worked until few days ago.
Thanks,
Fakkio.

@Fakkio probably your browser is blocking the request to a different domain
try with a simple node script to confirm is not the server or your credentials

Ok, i'll try with a node script. I don't get why this did work until few days ago.

Hi, I confirm: in node it works like a charm! Any idea what can I do to make it work in the browser?

@Fakkio not sure if ewelink changed his CORS policy/config in the last few days or what ...
you can always make the requests to ewelink servers through your backend where CORS will not be a problem

@skydiver I think so, they must have changed CORS policy for some reason. Unfortunately, my back end is in C#, and I cannot use this library, neither I found one for that language.
For the moment I'll use https://cors-anywhere.herokuapp.com/ and then I'll decide what to do.
Thank you, I think we can close here.

@Fakkio how did you do that? Have the same problem in my react application.


[ServiceBus] Batch Delete test fixes

Summary
The focus of these changes is to adjust the approach of tests for the purge-related functionality of the batch delete operation to match the expected behavior of the service, based on discussions with the service team about the contract.

/azp run net - servicebus - tests

/azp run net - servicebus - tests

/azp run net - servicebus - tests


Add skip.DocsMsPublish in nightly build pipeline

Add variable of skipping docsms step for unified pipeline. So we can give people flexibility to skip the step which is not necessary.

@sima-zhu as part of this lets also make sure we disable this automatically on repo's that aren't the public repo so we don't accidently publish things from the private repo.


Improve samples for Event Hubs

Description

Remove repetitive samples(samples which are using the same functionality).
Added summary at the top of each sample.
Added comments.
Added in notes to re-use sendMessages sample.
Few other clean ups as discussed on https://github.com/Azure/azure-service-bus-node/issues/43#issuecomment-452404781

Related issue: #2634


Pre-compiled version for Godot 3.3

Could we get pre-compiled version for newly released Godot 3.3? While I can compile this myself for Windows and Linux, I just don't have hardware to do that for Mac - and I do feel some new features would benefit Mac users most.

@lekoder The new pre-compiles are coming soon together with a new update for GodotSteam itself.
If we don't encounter any unexpected problems they should arrive within the next 48 hours.

@SapphireMH is correct.  I am actively compiling Windows and Linux as we speak and will be pushing the GodotSteam 3.9 changes soon.

@SapphireMH is working through some compiling issues but other pre-compiles are up.

Tried to import the 3.9 template.tpz, got this
No version.txt found inside templates.

Huh, guess I'll need to add one.  Good catch!  Should have that updated in a few minutes (my internet is very slow at uploading).

Alright, the MacOS templates are up, please let me know if you encounter any issues with them!

It seems that linux-33-template-64bit is compiled with debug enabled.

  Haha, my bad.  I'll get it re-compiled today and uploaded.  I was wondering why it was so large this time around.


Evaluation involving outer when there are multiple outer

I did not try but I suspect this evaluation to fail
class A(x: String) {
  class AInner {
    def xx: String = {
      x
    }
  }
}

class B(x: String, y: String) extends A(x) {
  class BInner extends AInner {
    def yy: String = {
      y // evaluate x or y
    }
  }
}

Because class BInner has two outers, one of type A and another of type B.

It does not have 2 outers, when de-compiling the Binner class, we get the following
public class example.B$BInner extends example.A$AInner {
  private final example.B $outer;
  public final example.B example$B$BInner$$$outer();
  ...
}

So, evaluating y would work, but evaluating x would fail because it is in A and not in B. Need to use ReferenceType#visibleFields instead of ReferenceType#fieldByName


Able to create a space without any specified template

Able to create a space without any specified template
Steps to reproduce

create a space
don't choose any template

expected result : there should be some default space template incase user is not selecting one.
video link : https://drive.google.com/a/redhat.com/file/d/0B1qUq_laG6CObEVhMS03aDhDMms/view?usp=sharing

duplicates #390

Closing as it is a duplicate.


Undo after fix this file triggers prompt

Testing #98989

Open file with special new line chars and prompt enabled
Click fix
Undo

:bug: => prompt opens immediately

The prompt will appear the next time the file is opened, but not immediately after undoing.


Use Itamae#logger instead of Logger

Itamae v1.5 has a logger injectable from outside of itamae. So, can not use Logger on itamae v1.5.
https://github.com/itamae-kitchen/itamae/blob/master/CHANGELOG.md#v150

@fukuiretu Could you please check this pr? :bow:

@dex1t
PR Thank you, and reply Sorry slows down:bow:
Please wait so make sure now🙏

@dex1t
It merges the PR, is now available as v0.0.3 to RubyGems:bulb:
https://rubygems.org/gems/itamae-plugin-recipe-homebrew
Thank you very much, Also thank you now:bow:

Thank you for merging :sparkles:


main: check if local storage is mounted when --local flag is used (HMS-3792)

This PR is a follow up to https://github.com/osbuild/bootc-image-builder/pull/120 and addresses some of the comments on the PR.
The most notable change is switching to use local storage as the default. A helper function is introduced to check if the image is in the local store and, if not, copy it into the store.

Closing in favour of #293


Start consuming IIIF images from Loris

Type

✨ Feature

Needs

[ ] Platform team?

UI Requirements
n/a
Acceptance Criteria

[ ] Images coming from IIIF image server, not S3 bucket

Problem to be solved
Business requirement for using IIIF images

@jennpb Do we need to do this to hit our first milestone of an internal release via a feature switch?

@BethPF @pollecuttn No, I'll remove this from the R1 milestone.

This is done, but still wrapped in a feature flag with imgix fallback.
If platform team are happy, we should get rid of the feature flag and fallback code


Replace old build_ext with new cythonize(...) version.

Rather than redirect everyone to the one in Cython.Build, simply upgrade the Cython.Distutils version. In particular, this is the version picked up by setuptools.
Notably, however, the new one takes far fewer options (this configuration primarily moved into in-file directives). The old one is kept as Cython.Distutils.old_build_ext to ease migration; we should consider which, if any, additional options should be moved to the new one.

We'll need an extended test cycle for the 0.25 release anyway, so let's get this in and fix it in master.

This breaks https://github.com/zeromq/pyzmq

In order for this to be a viable replacement to cythonize(), it should support passing keywords to cythonize. For example, Sage passes the keywords nthreads, build_dir, force, aliases, compiler_directives.

I agree that this is not a replacement for calling a customized cythonize(), but should be a step forward. For better or for worse, the distutils way to customize thing is to subclass (e.g. to specifying the build_ext, it expects a class, not an instance).

And I'd want to think long and hard about what command line options to add, given that they're even harder to change.

I agree that this is not a replacement for calling a customized cythonize(), but should be a step forward.

And it serves as nice "documentation" on how to customize the build_ext command.

breaks pyyaml. See #1498


bug(hubble): periodic latency spike on HTTP APIs after upgrading to 1.10.9

What is the bug?

How can it be reproduced? (optional)
Monitor latency on HTTP APIs

Lines up perfectly with merkle trie queue size

Will be potentially fixed by #1797


add a method to show and hide a tab bar programmatically

I implemented a function mentioned at #395.
usage:
$ionicTabsDelegate.showTabBar(true); // show tab bar
$ionicTabsDelegate.showTabBar(false); // hide tab bar

Though I haven't written any comment or document for this function yet, I'm ready to do it if this function is in demand and my code is acceptable.

Thanks for the PR, I ended up merging a different one: 7dd620da8f8335bc340e8e49eda32b72856b7741


Make Cardano node version part of the Mithril network configuration

Issue
The ledger state used to fast bootstrap the Cradano nodes may change from one release of the node to another.
For example, a breaking change was introduced between 1.35.7 and 8.0.0: this means that a ledger state created with 1.35.7 would need to be recomputed (i.e. replay all blocks) when restored on a node running 8.0.0 (and reciprocally). This does not prevent Cardano nodes to be restored, but it introduces a delay that will limit the speed of restoration.
We want to provide a clear information about which versions of Cardano node are optimum for restoration for each Mithril network.
To do

[ ] Add a Cardano node version range field in the configuration matrix for each network of the documentation
[ ] Add a note/warning in the documentation to explain what happens if the Cardano node version used to restore is not in the range of the Mithril network
[ ] Add the Cardano node version in the command used displayed by the Mithril client restore command (at least in the documentation)
[ ] Create a check list of operations needed to properly orchestrate the upgrade of a Cardano node for Mithril

It seems not possible to get the Cardano node version using the Cardano CLI tool (which can just pop out its own version). See this Stack-Overflow post.
I will make the assumption the Cardano CLI is the same version as the node.

In the documentation the versions are declarative and will be updated manually.
In issue #948, you can assume that the version of the Cardano node is the version given by the Cardano cli with cardano-cli --version as they are deployed together.


Is this still being maintained?

Is this plugin still being maintained? It is the perfect tool for my application, however I could really do with being able to shorten the date ranges.

I've updated the code to add the following few features.
https://github.com/nileshgamit/jquery-schedule

Added support for touch screen
Added 3 options
singleDaySchedule: false, // true will allow only single-day schedule (e.g. a particular holiday might have some open hours for some business)
autoScroll: true, // false will display full scheduler without scroll
showTimeInGrid: true // false will hide time on table grid
Some css fix to fit the scheduler in mobile view (it's not responsive)

Great plugin. Thanks Thomas.


MudTab - Dense and width

Feature request type
Enhance component
Component name
MudTabs
Is your feature request related to a problem?
Tabs occupy too many space
Describe the solution you'd like
Support for IsDense property and tab width
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I'm having the same problem. Is there any workaround for this ?

I also apply this, to get rid of text in caps:
.mud-tab, .mud-button-label {text-transform: none;}

Still no support for Dense 😕 ?

Still no support for Dense 😕 ?


Add cluster update support

Description of changes:
Depends on https://github.com/aws-controllers-k8s/code-generator/pull/256
Adds hooks for checking cluster status, and gating update or delete operations unless cluster is in ACTIVE status. Adds custom update logic for the UpdateClusterVersion and UpdateClusterConfig operations.
Note:
Update operations are asynchronous for EKS clusters, but can only be made one at a time. At the end of any update call, we forcefully set the synced condition to false and requeue with a 15 second duration. This gives the server time to receive and start the async update request, start the update and set the cluster status to UPDATING before we requeue and call read one.
By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license.

/hold
Waiting for the code-generator change to be merged

/test eks-kind-e2e
Cleaned up the test account

@RedbackThomson interesting approach! :) I've left some (hopefully helpful!) suggestions of how you can do this without needing to dig into $SDKAPI.API.Operations or require aws-controllers-k8s/code-generator#256

Super solid suggestion. For some reason I totally forgot about that option. Much simpler, thank you!

/unhold

/lgtm


[BIOMAGE-2307]- Change legend default position to right

Description

Make default legend position to right (make it into a column)
Calculate number of rows available per column according to the height of the plot
Delete __test__/utils/metadataUtils.test.js is because it is a duplicate

Details
URL to issue
https://biomage.atlassian.net/browse/BIOMAGE-2307
Link to staging deployment URL (or set N/A)
https://ui-agi-ui148.scp-staging.biomage.net/
Links to any PRs or resources related to this PR
N/A
Integration test branch
master
Merge checklist
Your changes will be ready for merging after all of the steps below have been completed.
Code updates
Have best practices and ongoing refactors being observed in this PR

[ ] Migrated any selector / reducer used to the new format.

Manual/unit testing

[ ] Tested changes using InfraMock locally or no tests required for change, e.g. Kubernetes chart updates.
[ ] Validated that current unit tests for code work as expected and are sufficient for code coverage or no unit tests required for change, e.g. documentation update.
[ ] Unit tests written or no unit tests required for change, e.g. documentation update.

Integration testing
You must check the box below to run integration tests on the latest commit on your PR branch.
Integration tests have to pass before the PR can be merged. Without checking the box, your PR
will not pass the required status checks for merging.

[ ] Started end-to-end tests on the latest commit.

Documentation updates

[ ] Relevant Github READMEs updated or no GitHub README updates required.
[ ] Relevant Wiki pages created/updated or no Wiki updates required.

Optional

[ ] Staging environment is unstaged before merging.
[ ] Photo of a cute animal attached to this PR.

After toggling the legend button, the legend still hides the plot. Is this ok and what we agreed on?

Also, the text that shows at the top will benefit some rewording in my opinion. "The legend" is repeated too many times, also, by changing the settings under Legend > Toggle Legend. is not as precise as it could be, because the legend settings are under "Styling settings". How about we change the text to something like:
We have hidden the plot legend, because it is too large and it interferes with the display of the plot.
You can still display the plot legend by changing the Toggle Legend option to "Show" in Plot Styling Settings under Legend.

For steps 6&7 (integration and configure embedding), the controls menu should jump to under the plot when the plot width is increased. Otherwise it is unusable. See screenshot example from step 6.

In plots and tables, the frequency plot and categorical embedding plot both work well. Completely usable even with the legend on (with 194 samples).
BUT the following plots are broken:
-I tried to load the violin plot with several experiments including the 194 sample dataset but also my own small dataset - every time I got "it's not you, it's us".
-I also tried the dotplot and got this error, again with several experiments:

Question regarding the text Iva also commented on (below), should it not read
Plot styling > Legend > Toggle Legend instead of Legend > Toggle Legend

After toggling the legend button, the legend still hides the plot. Is this ok and what we agreed on?

That is ok. We've informed the users that we're hiding the legend by default so that the plot can be shown. If they enable the  legend manually, then they should expect that the legend interferes with the plot.

Another thing -- if we've selected the legend as shown, does it make sense to still show that yellow message?

I think this is fine. If you refresh the page or visit it again, the legend will be hidden again.

In plots and tables, the frequency plot and categorical embedding plot both work well. Completely usable even with the legend on (with 194 samples).
BUT the following plots are broken: -I tried to load the violin plot with several experiments including the 194 sample dataset but also my own small dataset - every time I got "it's not you, it's us". -I also tried the dotplot and got this error, again with several experiments:

I have no problem with the violin and dot plots:

Can you try again? Maybe the worker wasn't up to process the data?

@ivababukova @jildoubaarsma

Question regarding the text Iva also commented on (below), should it not read Plot styling > Legend > Toggle Legend instead of Legend > Toggle Legend

I'm changing the text per Iva's recommendation to
We have hidden the plot legend, because it is too large and it interferes with the display of the plot.
You can still display the plot legend by changing the value of "Toggle Legend" option in Plot Styling Settings under "Legend".

@jildoubaarsma

In Data Exploration it seems a bug is introduced with this change. If I do the following, I get screen shot below

unview louvain clusters (results in grey plot)
Select a gene from the gene list

I am not sure though if this is due to the UI changes, or a loading issue, because the behavior is different for different genes it seems.

I tried reproducing this, but the plot is showing the gene expression correctly. I did not make any changes to the plots in Data Exploration, so I think this might be due to a loading issue.

@vickymorrison

For steps 6&7 (integration and configure embedding), the controls menu should jump to under the plot when the plot width is increased. Otherwise it is unusable.

I've fixed this by setting a minimum width for the controls. If the control is squished smaller than 300 px, it will flow down instead.

Another thing -- if we've selected the legend as shown, does it make sense to still show that yellow message?

I think this is fine. If you refresh the page or visit it again, the legend will be hidden again.

No, this is what I meant @aerlaut. The user can make the legend hidden by clicking on the "Hide" button under "Legend">"Toggle Legend", they don't need to refresh the page. I meant that if the user updates the plot styling settings to show the legend, this yellow message still stays the same. To me, this seems like a wrong state, because the message says that the legend is hidden, when the legend is actually shown. We should either hide the message, or change it in my opinion.
About refreshing the page to fix a bug with the platform -- it is bad UX. Especially, in Data Processing it is super annoying for the user. This is because after page refresh they will not go to the right filter, they still need to remember where they were and do a few clicks to get to the right page. We should not count on something getting fixed after a page refresh.

@aerlaut I just checked and this message doesn't appear at all in any of the cases now. Even if the legend is hidden, the message does not appear.

Both the violin and dot plots now load which is good. Don't know what happened the other day (I definitely had workers assigned).
But if I toggle the legend on in the violin plot, and put the legend on the right, then the plot looks like this:

Can the number of legend rows be increased to make this look better?


Turn off logging to stderr in cmd to all dockerfiles

Description
Changes proposed in this pull request:

adjust dockerfile run option to have flag logtostderr=false
...
...

Related issue(s)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Revamped docs to add basic 101 tutorials

Tutorials now goes basic wide function -> basic algorithms -> advanced algorithms -> SIMD tuple/UDT
Added an advanced tutorial on manual loop

I didn't push the HTML changes as it's like 5k+ files. If this PR pass and is validated, I'll make a direct-to-develop push for all the changed HTML.

fma4 is not supported and its a temporary  amd feature fma3 is supported and is an intel and amd feature

tu dis g++11 or above et apres tu compiles avec g++10

fma4 is not supported and its a temporary amd feature fma3 is supported and is an intel and amd feature
tu dis g++11 or above et apres tu compiles avec g++10

Fixed

fma4 is not supported and its a temporary amd feature fma3 is supported and is an intel and amd feature
tu dis g++11 or above et apres tu compiles avec g++10

Fixed


*: Adds unithash to status output

Fixes https://github.com/giantswarm/giantswarm/issues/524

This PR adds a UnitHash field to the MachineState based on the Hash of units in fleet. This is returned for the Controller.GetStatus calls.

The CLI formicactl status <group> command was changed to take the UnitHash field into account for grouping as well. Added a test for this too.

core@core-01 ~ $ ./formicactl status demo
Group   Units  FDState   FCState   SAState  IP            Machine

demo@1  *      launched  launched  active   <IP_ADDRESS>  99f23dbedf2f4bdd9ed9f2600b5944c3
demo@2  *      launched  launched  active   <IP_ADDRESS>  4ea5734c9d6347829d9236502b79c6ca

core@core-01 ~ $ ./formicactl status demo -v
Group   Units                FDState   FCState   SAState  Hash                                      IP            Machine

demo@1  <EMAIL_ADDRESS>launched  launched  active   fa59254bb1fac86a10935d9aaf839fe0638fbaba  <IP_ADDRESS>  99f23dbedf2f4bdd9ed9f2600b5944c3
demo@2  <EMAIL_ADDRESS>launched  launched  active   fa59254bb1fac86a10935d9aaf839fe0638fbaba  <IP_ADDRESS>  4ea5734c9d6347829d9236502b79c6ca

Current coverage is 51.66%

Merging #46 into master will increase coverage by +0.08% as of 60173bc

Powered by Codecov. Updated on successful CI builds.

ping @giantswarm/cena

The current state of this PR does not fix the given issue IMO. The hash is not displayed when fetching the status. I have expected changes in the cli package. There are none. What do I miss?

What do I miss?

A hammer to hit me. You are correct.

ping @xh3b4sd I added the output.

LGTM

I noticed one thing. Sorry to being late though. We extended the status display but we did not test createStatus. See https://github.com/giantswarm/formica/pull/46/files#diff-84398aaa4cb21783dbbf26c2c9a4ceb6R142
Would you mind doing that?

Agreed. We added an issue for this in the planning.


[JSON-RPC] - RPC Gateway server using paritytech/jsonrpsee

JSON-RPC gateway server implementation to replace the Rest Gateway server

using JSON-RPC lib from paritytech/jsonrpsee
ported all endpoints from rest server,  endpoints mirroring GatewayAPI
endpoint methods are prefixed with namespace sui
list of methods : sui_getObjectInfo, sui_moveCall, sui_syncAccountState, sui_getTransactionsInRange, sui_getTotalTransactionNumber, sui_getOwnedObjects, sui_transferCoin, sui_splitCoin, sui_executeTransaction, sui_mergeCoins, sui_publish, sui_getRecentTransactions
New RPCGatewayClient implementation for the wallet cli

Running the server (*you will need to do a fresh genesis first)
./rpc_server
Running the server with CORS
ACCESS_CONTROL_ALLOW_ORIGIN=* ./rpc_server
Invoke method with curl
curl -X POST \
     -H 'Content-Type: application/json' \
     -d '{"jsonrpc":"2.0","id":1,"method":"sui_getOwnedObjects","params":["<< address >>"]}' \
     http://<IP_ADDRESS>:5001

Replace << address >> with one of the addresses in wallet.conf
You can also use the wallet cli with the RPC server, to do this modify wallet.conf's gateway section to
  "gateway": {
    "rpc": "http://<IP_ADDRESS>:5001"
  }

Todos:

OpenRPC doc
Add getTransaction endpoint from @stella3d 's branch https://github.com/MystenLabs/sui/tree/get-tx-route
Refactor/simplify GatewayAPI response objects
Documentation, tutorial etc
Remove rest gateway server

list of methods : sui_getObjectInfo, sui_moveCall, sui_syncAccountState, sui_getTransactionsInRange, sui_getTotalTransactionNumber, sui_getOwnedObjects, sui_transferCoin, sui_splitCoin, sui_executeTransaction, sui_mergeCoins, sui_publish, sui_getRecentTransactions

This is probably more of a matter of personal taste but seeing camelcase + underscores is very jarring. Is there a standard JSON-RPC way for naming methods that we could use?

list of methods : sui_getObjectInfo, sui_moveCall, sui_syncAccountState, sui_getTransactionsInRange, sui_getTotalTransactionNumber, sui_getOwnedObjects, sui_transferCoin, sui_splitCoin, sui_executeTransaction, sui_mergeCoins, sui_publish, sui_getRecentTransactions

This is probably more of a matter of personal taste but seeing camelcase + underscores is very jarring. Is there a standard JSON-RPC way for naming methods that we could use?

Ethereum's JSON-RPC uses similar naming convention <namespace>_<camelCaseMethodName>, I am open for any naming suggestion!

list of methods : sui_getObjectInfo, sui_moveCall, sui_syncAccountState, sui_getTransactionsInRange, sui_getTotalTransactionNumber, sui_getOwnedObjects, sui_transferCoin, sui_splitCoin, sui_executeTransaction, sui_mergeCoins, sui_publish, sui_getRecentTransactions

This is probably more of a matter of personal taste but seeing camelcase + underscores is very jarring. Is there a standard JSON-RPC way for naming methods that we could use?

Ethereum's JSON-RPC uses similar naming convention <namespace>_<camelCaseMethodName>, I am open for any naming suggestion!

Then feel free to ignore my comment on this :)

It would be super nice to auto generate the client code, so I really wonder about the support for things like that.

jsonrpsee supports client code generation in rust, we are using that in https://github.com/MystenLabs/sui/blob/main/sui/src/rpc_gateway_client.rs


When row name is set to first column, set row.names to 1, not "1".

Fix for https://github.com/Microsoft/RTVS/issues/1535

:+1:


Implement Apply, Delete and DeleteByName in kubehttp

Short description
As a Weave GitOps developer, I would like to use the go client to perform all kubernetes operations. Currently, the three operations that need to work across kinds are not implemented and are performed via calls to kubectl. In order to move away from kubectl, we will have to implement these operations via the dynamic kubernetes interface.

Maybe the kustomize-controller has some code we could borrow.


semantic_analyser.type_ctx get failed on aarch64

On aarch64, with bpftrace-0.11.2, kernel-5.10.1, gcc-10.2.1,glibc2.32,clang-10.0.1,llvm-10.0.1,bcc-0.16.0,
bpftrace_test/semantic_analyser.type_ctx get failed.
[ RUN      ] semantic_analyser.type_ctx
/root/rpmbuild/BUILD/bpftrace-0.11.2/tests/semantic_analyser.cpp:1651: Failure
Expected equality of these values:
  CreateInt8()
    Which is: int8
  assignment->var->type
    Which is: unsigned int8
/root/rpmbuild/BUILD/bpftrace-0.11.2/tests/semantic_analyser.cpp:1653: Failure
Expected equality of these values:
  CreateInt8()
    Which is: int8
  fieldaccess->type
    Which is: unsigned int8
/root/rpmbuild/BUILD/bpftrace-0.11.2/tests/semantic_analyser.cpp:1663: Failure
Expected equality of these values:
  CreateInt8()
    Which is: int8
  assignment->var->type
    Which is: unsigned int8
/root/rpmbuild/BUILD/bpftrace-0.11.2/tests/semantic_analyser.cpp:1665: Failure
Expected equality of these values:
  CreateInt8()
    Which is: int8
  fieldaccess->type
    Which is: unsigned int8
[  FAILED  ] semantic_analyser.type_ctx (18 ms)

I test on latest bpftrace, also have this failure.

The signedness of char depends on the compiler and the target platform: the defaults for ARM and PowerPC are typically unsigned, the defaults for x86 and x64 are typically signed.
so on aarch64 it failed as expected.


I have problem with cookies in httpc-test get requests after login but I can get result with curl command.

Hi @jeremychone ,
I am using your valuable tutorial about axume and I enjoyed your httpc-test crate as well.
But when I tried to add some new nested route beside api route in your tutorial I found myself in trouble with request headers without cookie.
I added new nested route similar to your tutorial for api:
    let private_files = private_file_routes()
        .route_layer(middleware::from_fn(web::mw_auth::mw_require_auth));

    let private_files = private_file_routes()
        .route_layer(middleware::from_fn(web::mw_auth::mw_require_auth));

    let routes_apis = web::routes_tickets::routes(mc.clone())
        .route_layer(middleware::from_fn(web::mw_auth::mw_require_auth));

    let routes_all = Router::new()
        .merge(routes_hello())
        .merge(web::routes_login::routes())
        .nest("/api", routes_apis)
        .nest("/pvf", private_files)
        .layer(middleware::map_response(main_response_mapper))
        .layer(middleware::from_fn_with_state(mc.clone(), web::mw_auth::mw_ctx_resolver, ))
        .layer(CookieManagerLayer::new())
        .merge(public_no_auth_routes())
        .fallback_service(public_no_auth_routes());

async fn handler_private_file(
    ctx: Ctx,
    Path(priavate_path): Path<String>,
) -> impl IntoResponse {
    println!("->> {:<12} - handler_private_file - {ctx:?} ", "CTX");
    println!("->> {:<12} - handler_private_file - {priavate_path:?} ", "HANDLER");

    Html(format!("bublic file requested: {priavate_path}"))
}

So when I try httpc-test:

    let hc = httpc_test::new_client("http://localhost:8080")?;

    hc.do_get("/public/lvl_01/lvl_02/public_target_at_several_level").await?.print().await?;
    hc.do_get("/public/target_at_public_root").await?.print().await?;

    // try to read private data before login:
    hc.do_get("/pvf/target_at_private_root_b4_login").await?.print().await?;

    let req_login = hc.do_post("/api/login",
            json!({
                "username": "demo1",
                "pwd": "welcome"
            }));
    req_login.await?.print().await?;

    // try to read private data after login:
    hc.do_get("/pvf/lvl_01/lvl_02/private_target_at_several_level").await?.print().await?;
    hc.do_get("/pvf/private_target_at_root_level").await?.print().await?;

    //Check if cookies still working for api route
    hc.do_get("/api/tickets").await?.print().await?;

and here is test result:
server side log:

->> LISTENING on <IP_ADDRESS>:8080

->> HANDLER      - handler_public_file - "lvl_01/lvl_02/public_target_at_several_level"
->> HANDLER      - handler_public_file - "target_at_public_root"

->> MIDDLEWARE   - mw_ctx_resolver
mw_ctx_resolver <<-- req.headers {"accept": "*/*", "host": "localhost:8080"}

mw_ctx_resolver <<-- cookies Cookies { inner: Mutex { data: Inner { headers: [], jar: None, changed: false } } }

mw_ctx_resolver -->> auth_token: None -

->> EXTRACTOR    - Ctx
->> EXTRACTOR    - Ctx
->> MIDDLEWARE   - mw_require_auth
->> INTO_RES     - AuthFailNoAuthTokenCookie
->> RES_MAPPER   - main_response_mapper
      ->> client_error_body: {"error":{"req_uuid":"018b38f2-29c0-7aaf-9cb4-298d80d1a604","type":"NO_AUTH"}}
      ->> log_request:
{"client_error_type":"NO_AUTH","error_type":"AuthFailNoAuthTokenCookie","req_method":"GET","req_path":"/pvf/target_at_private_root_b4_login","timestamp":"1697467476416","uuid":"018b38f2-29c0-7aaf-9cb4-298d80d1a604"}
      ->> server log line - 018b38f2-29c0-7aaf-9cb4-298d80d1a604 - Error: Some(AuthFailNoAuthTokenCookie)

->> MIDDLEWARE   - mw_ctx_resolver
mw_ctx_resolver <<-- req.headers {"content-type": "application/json", "accept": "*/*", "host": "localhost:8080", "content-length": "36"}

mw_ctx_resolver <<-- cookies Cookies { inner: Mutex { data: Inner { headers: [], jar: None, changed: false } } }

mw_ctx_resolver -->> auth_token: None -

->> EXTRACTOR    - Ctx
->> HANDLER      - api_login
->> RES_MAPPER   - main_response_mapper
      ->> log_request:
{"req_method":"POST","req_path":"/api/login","timestamp":"1697467476417","uuid":"018b38f2-29c1-7285-a666-c0decf982c41"}
      ->> server log line - 018b38f2-29c1-7285-a666-c0decf982c41 - Error: None

->> MIDDLEWARE   - mw_ctx_resolver
mw_ctx_resolver <<-- req.headers {"accept": "*/*", "host": "localhost:8080"}

mw_ctx_resolver <<-- cookies Cookies { inner: Mutex { data: Inner { headers: [], jar: None, changed: false } } }

mw_ctx_resolver -->> auth_token: None -

->> EXTRACTOR    - Ctx
->> EXTRACTOR    - Ctx
->> MIDDLEWARE   - mw_require_auth
->> INTO_RES     - AuthFailNoAuthTokenCookie
->> RES_MAPPER   - main_response_mapper
      ->> client_error_body: {"error":{"req_uuid":"018b38f2-29c3-78e3-a0e7-98f5a192cee2","type":"NO_AUTH"}}
      ->> log_request:
{"client_error_type":"NO_AUTH","error_type":"AuthFailNoAuthTokenCookie","req_method":"GET","req_path":"/pvf/lvl_01/lvl_02/private_target_at_several_level","timestamp":"1697467476419","uuid":"018b38f2-29c3-78e3-a0e7-98f5a192cee2"}
      ->> server log line - 018b38f2-29c3-78e3-a0e7-98f5a192cee2 - Error: Some(AuthFailNoAuthTokenCookie)

->> MIDDLEWARE   - mw_ctx_resolver
mw_ctx_resolver <<-- req.headers {"accept": "*/*", "host": "localhost:8080"}

mw_ctx_resolver <<-- cookies Cookies { inner: Mutex { data: Inner { headers: [], jar: None, changed: false } } }

mw_ctx_resolver -->> auth_token: None -

->> EXTRACTOR    - Ctx
->> EXTRACTOR    - Ctx
->> MIDDLEWARE   - mw_require_auth
->> INTO_RES     - AuthFailNoAuthTokenCookie
->> RES_MAPPER   - main_response_mapper
      ->> client_error_body: {"error":{"req_uuid":"018b38f2-29c4-71bd-a26c-584b775e198b","type":"NO_AUTH"}}
      ->> log_request:
{"client_error_type":"NO_AUTH","error_type":"AuthFailNoAuthTokenCookie","req_method":"GET","req_path":"/pvf/private_target_at_root_level","timestamp":"1697467476420","uuid":"018b38f2-29c4-71bd-a26c-584b775e198b"}
      ->> server log line - 018b38f2-29c4-71bd-a26c-584b775e198b - Error: Some(AuthFailNoAuthTokenCookie)

->> MIDDLEWARE   - mw_ctx_resolver
mw_ctx_resolver <<-- req.headers {"accept": "*/*", "cookie": "auth-token=user-1.exp.sign", "host": "localhost:8080"}

mw_ctx_resolver <<-- cookies Cookies { inner: Mutex { data: Inner { headers: ["auth-token=user-1.exp.sign"], jar: None, changed: false } } }

mw_ctx_resolver -->> auth_token: Some("user-1.exp.sign") -

->> EXTRACTOR    - Ctx
->> EXTRACTOR    - Ctx
->> MIDDLEWARE   - mw_require_auth
->> EXTRACTOR    - Ctx
->> HANDLER      - list_tickets
->> RES_MAPPER   - main_response_mapper
      ->> log_request:
{"req_method":"GET","req_path":"/api/tickets","timestamp":"1697467476424","user_id":1,"uuid":"018b38f2-29c8-7c01-bce9-4985a40472c0"}
      ->> server log line - 018b38f2-29c8-7c01-bce9-4985a40472c0 - Error: None

client side log:

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

running 1 test

=== Response for GET http://localhost:8080/public/lvl_01/lvl_02/public_target_at_several_level
=> Status         : 200 OK OK
=> Headers        :
   content-type: text/html; charset=utf-8
   content-length: 84
   date: Mon, 16 Oct 2023 14:44:36 GMT
=> Response Body  :
bublic file requested: lvl_01/lvl_02/public_target_at_several_level
===

=== Response for GET http://localhost:8080/public/target_at_public_root
=> Status         : 200 OK OK
=> Headers        :
   content-type: text/html; charset=utf-8
   content-length: 61
   date: Mon, 16 Oct 2023 14:44:36 GMT
=> Response Body  :
bublic file requested: target_at_public_root
===

=== Response for GET http://localhost:8080/pvf/target_at_private_root_b4_login
=> Status         : 403 Forbidden Forbidden
=> Headers        :
   content-type: application/json
   content-length: 78
   date: Mon, 16 Oct 2023 14:44:36 GMT
=> Response Body  :
{
  "error": {
    "req_uuid": "018b38f2-29c0-7aaf-9cb4-298d80d1a604",
    "type": "NO_AUTH"
  }
}
===

=== Response for POST http://localhost:8080/api/login
=> Status         : 200 OK OK
=> Headers        :
   content-type: application/json
   content-length: 27
   set-cookie: auth-token=user-1.exp.sign
   date: Mon, 16 Oct 2023 14:44:36 GMT
=> Response Cookies:
   auth-token: user-1.exp.sign
=> Client Cookies :
   auth-token: user-1.exp.sign
=> Response Body  :
{
  "result": {
    "success": true
  }
}
===

=== Response for GET http://localhost:8080/pvf/lvl_01/lvl_02/private_target_at_several_level
=> Status         : 403 Forbidden Forbidden
=> Headers        :
   content-type: application/json
   content-length: 78
   date: Mon, 16 Oct 2023 14:44:36 GMT
=> Client Cookies :
   auth-token: user-1.exp.sign
=> Response Body  :
{
  "error": {
    "req_uuid": "018b38f2-29c3-78e3-a0e7-98f5a192cee2",
    "type": "NO_AUTH"
  }
}
===

=== Response for GET http://localhost:8080/pvf/private_target_at_root_level
=> Status         : 403 Forbidden Forbidden
=> Headers        :
   content-type: application/json
   content-length: 78
   date: Mon, 16 Oct 2023 14:44:36 GMT
=> Client Cookies :
   auth-token: user-1.exp.sign
=> Response Body  :
{
  "error": {
    "req_uuid": "018b38f2-29c4-71bd-a26c-584b775e198b",
    "type": "NO_AUTH"
  }
}
===

=== Response for GET http://localhost:8080/api/tickets
=> Status         : 200 OK OK
=> Headers        :
   content-type: application/json
   content-length: 2
   date: Mon, 16 Oct 2023 14:44:36 GMT
=> Client Cookies :
   auth-token: user-1.exp.sign
=> Response Body  :
[]
===

.
test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.05s

now when I try same url with curl command including cookie I have correct result:
curl -v --cookie "auth-token=user-1.exp.sign" http://localhost:8080/pvf/lvl_01/lvl_02/public_target_at_several_level           17:57:09
*   Trying [::1]:8080...
* connect to ::1 port 8080 failed: Connection refused
*   Trying <IP_ADDRESS>:8080...
* Connected to localhost (<IP_ADDRESS>) port 8080
> GET /pvf/lvl_01/lvl_02/public_target_at_several_level HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/8.4.0
> Accept: */*
> Cookie: auth-token=user-1.exp.sign
>
< HTTP/1.1 200 OK
< content-type: text/html; charset=utf-8
< content-length: 84
< date: Mon, 16 Oct 2023 14:59:25 GMT
<
* Connection #0 to host localhost left intact
bublic file requested: lvl_01/lvl_02/public_target_at_several_level%

So please help me find why there is no cookie in httpc-test requests when I try the other route?

Ok, I think that was the issue, in the rust-axum-course I did a commit to fix it on the rust-axum-course repo:
https://github.com/jeremychone-channel/rust-axum-course/commit/0bcde6fd1e49e605e9352031538cceda9e4287eb
If you checkout the rust-axum-course repo, it should work. Let me know if that was the issue. Closing for now.

@jeremychone
Too many thanks,
I did same changes as you mentioned and now it's working in quick-dev test.