npm warning for dependency on<EMAIL_ADDRESS>but I have 4.17.0 installed

myuserid@localhost ~]$ sudo npm install -g vue-eslint-parser
/usr/lib
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
myuserid@localhost ~]$ eslint --version
v4.17.0

Thank you for this issue.
It's npm matter.
npm handles the global registry and project-local registries as isolated. Your eslint command is in the global (npm install -g eslint), but vue-eslint-parser is in a project-local (without -g flag). So the UNMET PEER DEPENDENCY warning is right.

A side note: We (ESLint core team) recommend that people don't install eslint to global.

You're making an incorrect assumption. Both eslint and vue-eslint-parser were installed globally, as you can clearly see in the command line in my original post. So the warning is incorrect.
I'm aware of the recommendation regarding global vs project local installations but it makes no sense to me so I don't follow it. I'm using these tools through emacs which has to find them in my path. I'm not interested in having to add each project directory to my path or having a different editor configuration for each project . This is how I have worked with every other emacs plugin for many years and I see no reason to change.

I got it.
However, in that case, it's npm's matter. I recommend opening an issue on https://github.com/npm/npm.


ENH: get_die now outputs the unmodified die position from the prober api

Removed custom die position mappings.
get_die() now outputs the direct response from the prober api.

Thanks, looks nice! Merged


Policies for Triliovault for Kubernetes

Creating policies to

install Triliovault for Kubernetes
create backup for a specified namespace using Triliovault for Kubernetes
create backup of namespace having a label using Triliovault for Kubernetes and kyverno

The policy syntax looks good to me. I'll leave it to @gparvin to finish off the review.


STORM-3006-updated-DRPC-docs

Extended DRPCClient initialization with DRPCClient config (which is now mandatory parameter) and added storm.thrift.transport: "org.apache.storm.security.auth.plain.PlainSaslTransportPlugin" in storm.yaml example so it matches DRPCClient config.

The docs are not identical for master, so I will manually make some updates to master, which is allowed without code review.


enhance logging framework

How to categorize this PR?

/area usability
/kind enhancement
/priority 3
What this PR does / why we need it:
Adds a small helper function.
Release note:

NONE

cc @achimweigel @robertgraeff


video test(s) are failing

Description
After the dependency upgrades in #8919 one or more tests under video fail in Java 11 (sponge) and Java 17 (sponge).
Based on the errors it looks like the tests take too long and exhaust resources in the testing environment.

Sometimes both tests (Java 11 and Java 17) fails with the following error:
#
# video
#

ERROR: Aborting VM command due to timeout of 21600 seconds

[ID: 1940114] Command finished after 21611 secs, exit value: 1


Add YouDeal Token

Add YouDeal Token

ERROR! json file 0xfaaf0Ecfb3246902de6A2C77c5B114DA555695A3.json's links every field must be an url
npm ERR! Test failed.  See above for more details.

Sorry about that. I just fixed the CI failed at the master branch.
Please make a new commit to trigger the CI again.


Master issue to track all pull requests

Module
Branch
Related Issue(s)
PR Link

PHP Class
rt/feature/php-api-stories
3064

Gutenberg blocks (latest stories)
rt/feature/gutenberg-latest-stories
3064

Gutenberg blocks (selected stories)
rt/feature/gutenberg-selected-stories
3064

Customizer Options
rt/feature/customizer-options-stories
-

Shortcode API
rt/feature/shortcode-api-stories
-

Sidebar Widget (classic)
rt/feature/sidebar-widget-stories
-

TinyMCE Button
rt/feature/tnymce-stories
-

Further to our discussion and as per the related separate issues being maintained, closing this issue now.


pkgs.xorriso: 1.4.4 -> 1.4.6

Motivation for this change
Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

See http://lists.gnu.org/archive/html/info-gnu/2016-09/msg00010.html
for release announcement

Pushed to master (ea1afcd9f468f9f8813dba93d167bf2e558c797a). Thanks.


eyetracker: record continuously?

maybe we could switch to have a continuous recording of the eyetracker and not just have it "ON" during a block, no?

sure it is definetly a valid alternative, though what is it the gain during the analysis?
I will open an issue in CPP_PTB  to add the feature to send 'messages' (tags) during the recording and then we can edit here in the main script

[ ] run the script in LLN to check for any problem (e.g. dealys)

[ ] compare the output with and without continuos recording

@marcobarilari
the associated PR has been merged but there are unticked boxes so we can't close that yet, right?

now we can :)


Remove 3rd Bullet Point from "Check Validity" screen (Addresses cwa-documentation issue 721) (COMMUNITY)

Description
This PR removes the 3rd Bullet Point from the "Check Validity" screen & the result screens, as it is out of date.
Link to Jira

GitHub Issue: https://github.com/corona-warn-app/cwa-documentation/issues/721

Internal Tracking ID: EXPOSUREAPP-10566

Screenshots

Additional Information
Please let me know
a) If I should do this for all language files
b) If this change is OK for you. I'm still in the Swift learning phase, so please bear with me (-:
c) If I should change the numbers of the bullet points which are still there, I did not change it now because I wanted to avoid any confusion with the translation team

Internal Tracking ID: EXPOSUREAPP-10566

@Ein-Tim Thanks for proving a PR. PR forwarded into Internal Tracking ID: EXPOSUREAPP-10566

This can't go into 2.14 now, or am I wrong?

I've verified that this bullet point indeed can be removed, so please go ahead with the merge of this PR. Thanks for the help!

Awaiting feedback on which branch to target.

@naveeddotio
You merged this PR into release/2.14.x, after version 2.14.1 was already released in the App Store.
Will this change
a) be included in an upcoming hot fix release (version 2.14.2) or
b) be included in the next minor release (version 2.15.x)?
Thanks for your answer!
Tim

Btw @marcussc this PR is missing the respective milestone. Could you take a look at this, please?


Fix typo: change 'hiearchy' to 'hierarchy'

Noticed this when I was reading over the flow type definitions.

Thank you @wdhorton for fixing it! =)


IN-1158 Reindex foreign keys embedded in JSON

Purpose
IN-1158
Approach
Added new reindex function for special cases, such as rows needing a foreign key to update inside a JSON column.
Learning
n/a
Checklist

[X] I have performed a self-review of my own code
[ ] I have done an adhoc run against preprod (only needed for high complexity PRs)
[X] I have added relevant logging with appropriate levels to my code
[X] I have updated documentation where relevant
[X] I have added tests to prove my work
[ ] The product team have tested these changes

Closing as this is already covered by a business rule written by @devkokov


generator: Various changes

These are the generator related changes from #847
This is a big PR made up of a bunch of smaller changes to the way the generator works.
Best browsed in 'one commit at a time' mode.
The individual commits should explain what each is for.
Let me know if there are any questions or suggestions.

It's been 10 days, no objections. Merging.


Get different output on A10 and A800 using the same prompt

I have deployed Qwen using vllm 0.2.3 on A800 and A10(2 A10 card) .
I find out that I will get different output using the same prompt，but get same output on same gpu type.
What should I do to get same output on different type of gpu?
request
{
    "model": "Qwen",
    "messages": [
        {"role": "system", "content": "You are a helpful assistant."},
        {"role": "user", "content": "Hello, what is Apple"}
    ],
    "temperature": 0,
    "do_sample": false,
    "max_tokens": 512,
    "stop": ["<|endoftext|>", "<|im_start|>", "<|im_end|>"]
}

output on A800
{
    "id": "cmpl-78a74b93e2124cd1b045f1cef84c3926",
    "object": "chat.completion",
    "created": 6997154,
    "model": "Qwen",
    "choices": [
        {
            "index": 0,
            "message": {
                "role": "assistant",
                "content": "Apple is a technology company that designs, develops, and sells consumer electronics, computer software, and online services. It is headquartered in Cupertino, California, and was founded in 1976 by Steve Jobs, Steve Wozniak, and Ronald Wayne. Apple is known for its popular products such as the iPhone, iPad, Mac computer, and Apple Watch, as well as its operating systems, such as iOS and macOS. The company also offers a range of online services, including the App Store, Apple Music, and iCloud. Apple is one of the largest technology companies in the world and is known for its innovative products and design."
            },
            "finish_reason": "stop"
        }
    ],
    "usage": {
        "prompt_tokens": 24,
        "total_tokens": 154,
        "completion_tokens": 130
    }
}

output on A10
{
    "id": "cmpl-233386b91f2c495587327dc5f145f40a",
    "object": "chat.completion",
    "created": 2269045,
    "model": "Qwen",
    "choices": [
        {
            "index": 0,
            "message": {
                "role": "assistant",
                "content": "Apple is a technology company that designs, develops, and sells consumer electronics, computer software, and online services. It is headquartered in Cupertino, California, and is one of the largest technology companies in the world. Some of its popular products include the iPhone, iPad, Mac, Apple Watch, and Apple TV. It also operates the App Store, which is a digital marketplace for mobile apps."
            },
            "finish_reason": "stop"
        }
    ],
    "usage": {
        "prompt_tokens": 24,
        "total_tokens": 103,
        "completion_tokens": 79
    }
}

I have the same problem in A100 and A40, does anyone have a solution?


div in tables.html DataTables example contains css class typo

http://ironsummitmedia.github.io/startbootstrap-sb-admin-2/pages/tables.html line 392
The dataTable_wrapper class doesn't exist, so

should be

I'll check it out, thanks.

Neither class exists actually. I guess I planned to use that for something but never did. I just removed it in #136


[WIP] Author page

@Mefjus I fixed the problem with the avatar query. Basically, it was caused by 2 things. First of all, it requires the newest version of gatsby, and second, when we query images from contentful there is a specific interface for that (...GatsbyContentfulFluid). I found it in Image Gatsby Docs. So I think that now you have an open road to continue 💪🏻

pull request ready to review @Jozwiaczek


output coords are negative floats

The network is defined by:
class Net(nn.Module):

    def __init__(self, layers):
        super(Net, self).__init__()
        if layers == 18:
            model = models.resnet18(pretrained=True)
        elif layers == 34:
            model = models.resnet34(pretrained=True)
        # change the first layer to recieve five channel image
        model.conv1 = nn.Conv2d(5, 64, kernel_size=7, stride=2, padding=3,bias=True)
        # change the last layer to output 32 coordinates
        # model.fc=nn.Linear(512,32)
        # remove final two layers(fc, avepool)
        model = nn.Sequential(*(list(model.children())[:-2]))
        for param in model.parameters():
            param.requires_grad = True
        self.resnet = model

    def forward(self, x):

        pose_out = self.resnet(x)
        return pose_out

class CoordRegressionNetwork(nn.Module):
    def __init__(self, n_locations, layers):
        super(CoordRegressionNetwork, self).__init__()
        self.resnet = Net(layers)
        self.hm_conv = nn.Conv2d(512, n_locations, kernel_size=1, bias=False)

    def forward(self, images):
        # 1. Run the images through our Resnet
        resnet_out = self.resnet(images)
        # 2. Use a 1x1 conv to get one unnormalized heatmap per location
        unnormalized_heatmaps = self.hm_conv(resnet_out)
        # 3. Normalize the heatmaps
        heatmaps = dsntnn.flat_softmax(unnormalized_heatmaps)
        # 4. Calculate the coordinates
        coords = dsntnn.dsnt(heatmaps)

        return coords, heatmaps

And the training codes are as follows:
for i, data in enumerate(tqdm(train_dataloader)):
            # training
            images, poses = data['image'], data['pose']
            images, poses = images.to(device), poses.to(device)
            coords, heatmaps = net(images)

            # Per-location euclidean losses
            euc_losses = dsntnn.euclidean_losses(coords, poses)
            # Per-location regularization losses
            reg_losses = dsntnn.js_reg_losses(heatmaps, poses, sigma_t=1.0)
            # Combine losses into an overall loss
            loss = dsntnn.average_loss(euc_losses + reg_losses)

            optimizer.zero_grad()
            loss.backward()
            optimizer.step()

            train_loss_epoch.append(loss.item())

I converted keypoint groundtruth into float(-1,1), but all the predicted coords are negative floats:
tensor([[[-0.1286, -0.0830],
         [-0.1169, -0.0810],
         [-0.1205, -0.1476],
         ...,
         [-0.1767, -0.3881],
         [-0.1970, -0.2403],
         [-0.3226, -0.3909]],

        [[-0.0694, -0.0165],
         [-0.0744, -0.0288],
         [-0.1027, -0.0873],
         ...,
         [-0.0766, -0.3926],
         [-0.1146, -0.2482],
         [-0.0907, -0.1812]],

        [[-0.4647, -0.3639],
         [-0.4430, -0.3409],
         [-0.2485, -0.2339],
         ...,
         [-0.2906, -0.4541],
         [-0.3648, -0.3034],
         [-0.4190, -0.3880]],

and the heatmap seems strange:

the visualization results:

I merged dsntnn module into my previous single person pose estimator, MobilePose-pytorch


Augment keys of all metric types in the same way

It is unexpected that the common key prefix for some metrics types is
different. This makes it unnecessary difficult to find the metrics in
the database. This commit unifies the prefix for all metrics.
The prefix of each key is now service name, type prefix, hostname. All
of them are optional. service name is added if it is available and not
set as a label because the configuration flag EnableServiceLabel is
set. type prefix is added if it is enabled with the configuration flag
EnableTypePrefix. hostname is added if it is available and not set
as a label because EnableHostnameLabel is set.
The behaviour of EmitKeys, IncrCounter, IncrCounterWithLabels,
AddSample, AddSampleWithLabels, MeasureSince and
MeasureSinceWithLabels changes with this commit. For EmitKey the
hostname will be added (details see above) but the service name will no
longer be added when EnableServiceLabel is set to true. For the
other types the hostname will be added if it is available and
EnableHostnameLabel is set to false.

This PR contains the changes of #20 and #76.

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.


[Questions 2.2] Main Loop iterations and Physics

Hello!
I'm trying to make some fancy stuff with genetic algorithm. Basically I have a scene that creates 12 SceneTrees and initiate the same given PackedScene in it. These 12 simulations are iterated by the main scene using the MainLoop's iteration function.
I do that in order to be able to speed up or slow down at will the simulations. Sadly, it seems that the physics aren't affected by this. For example, if I speed every 8 times, things handled by my scripts will be sped up 8 times, but an object falling due to gravity is going to move the exact same way instead of moving 8 time faster.
I could micro manage this by increase by 8 the gravity. But this feels bad.
Is there a way to do what I want to do? Am I using MainLoop properly? Is there separate physics server with different paces for each scene trees?
Thanks!

This is Github issues. This isn't for Q&A - there's a section on the website for that.
Regardless, for the sake of having the physics be deterministic (which is especially important from evolutionary/genetic algorithms), the physics server runs at a fixed tick-rate (by default at 60fps).
You can change this under the Project settings under Physics.
To have your scripts run in sync with physics (which is sensible for anything which you want to be consistent and deterministic, i.e simulation stuff instead of visuals), but the section of code you want to run in the physics loop by using fixed_process() instead of process().
However, increasing or decreasing the physics tickrate is also going to fundamentally change the behavior of the simulation, as all the physics is still scaled by the delta-time, just by a fixed delta-time.
I'm unsure of the best way to increase the physics tickrate without modifying the delta-time; it may require some more integral changes than GDScript allows access to.

I've taken a look and it looks like the best method would be to use set_use_custom_integrator and add a custom integrator for the relevant rigid bodies to scale velocity by a constant instead of by the deltatime.
This would allow you to modify the physics tickrate without changing the behaviour of the simulation, so doubling the tickrate would deterministically double the sim speed (assuming you're running on your own machine - floats make cross-platform determinism less easy)

Hello and thank you for your answer!
I am sorry if I have used the issues instead of the Q&A.
Using OS:set_iterations_per_second and OS:set_time_scale together works, I've not noticed rounding problems yet.
The only thing I reproach to that solution is that it's not working independently on the scene trees and instead affect all the application. I would have wished scene trees to be more independent on these aspects.
Using integrate_forces would force coding the speed-up in the simulations, while I want it to be completely unaware.

As far as I can tell there's no real way to change the timescale or tickrate on a per-scene basis, it can only be done on the engine level.
Per-scene tickrate and timescale seem like potentially useful features, albeit admittedly sound more useful for simulations rather than games (where separate scenes are rarely going to be run in parallel, or especially at different timescales).
It may be a good addition to make to the engine but could be a fair amount of work,

You can always run multiple Godot instances and add some code to sync them.
I don't know if headless Godot currently builds, but it will do eventually and you could take some advantage from it.

@Causeless Note that I am using SceneTrees and not just Scenes, therefore a separated MainLoop, a separated server would have been logical.
@RandomShaper By doing so I wouldn't be able to render the viewport in a sprite I guess.

You could run those instances with settings for a borderless, 1x1 window with viewport stretch. Or something like that.
Just throwing out ideas. :)

Closing as the discussion stalled (and also, it would be better suited on the Q&A or the subreddit/forums :)).


Please add ALB X-Amzn-Trace-Id as first class span in a trace

When using an amazon load balancer that adds X-Amzn-Trace-Id please use this to create a first client span to the trace.
https://docs.aws.amazon.com/elasticloadbalancing/latest/application/load-balancer-request-tracing.html

What is the existing level of support for this trace id in Envoy?

@douglas-reid Can we reopen this ticket?


Fix imports in README.md

Nowadays, model.ModelAdmin doesn't exist, and I also reorder imports as suggested in Python's PEP8.

Thanks

👍


Tabs don't like being narrow

[x] I am on the latest Hyper.app version

[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: 10.12.2

Hyper.app version: 1.1.0

Link of a Gist with the contents of your .hyper.js: default

Relevent information from devtools (CMD+SHIFT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A

The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue

Default tabs don't like being narrow.

Compared to hyper-tabs-enhanced plugin

There's also no way to see the full title, so adding HTML title is a good solution to see it on hover ✌

I'll make a PR for this asap ✌


Dynamic batching of embed/rerank on single gaudi without wrapper

Description
Add dynamic batching of embed/rerank on single gaudi without wrapper
depends on https://github.com/opea-project/GenAIComps/pull/774
Issues
n/a
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)
[ ] Others (enhancement, documentation, validation, etc.)

Dependencies
None
Tests
NA

@chensuyue @lvliang-intel Maybe we can just close this PR and I will keep the branch for reproduction in the future? I do not think there are any appropriate names to describe this. Also I think this feature is not mature enough for this release.


find/captures disagree about match when using $ or \z

I believe I have found a bug in this library. I've tried to condense it down into the simplest possible test case which reproduces the problem:
#[test]
fn test_end_of_text() {
    let re = regex!(r"a(b*(X|\z))?");
    let text = "abcbX";
    let ma = re.find(text).unwrap().as_str();
    let mb = re.captures(text).unwrap().get(0).unwrap().as_str();
    assert_eq!("a", ma);
    assert_eq!(ma, mb);
}

I would expect this test case to pass. The first assert_eq! does, but the second assert_eq! reports that re.captures( actually returned "ab" instead of just "a". It seems like (b*(X|\z))? matches the b following the a and returns it as part of the full capture group, even though the following character c doesn't match X|\z. Interestingly, this only seems to happen when using captures, and not find.

I turned this into a real regression test according to this crate's existing style:
// See: https://github.com/rust-lang/regex/issues/334
#[test]
fn regression_end_of_text_mismatch() {
    let re = regex!(r"a(b*(X|$))?");
    let text = text!("abcbX");
    let ma = re.find(text).unwrap();
    let mb = re.captures(text).unwrap().get(0).unwrap();
    assert_eq!(text!("a"), match_text!(ma));
    assert_eq!(match_text!(ma), match_text!(mb));
}

Another thing I noted while doing this is that, while not in multi-line mode, the same test fails regardless of whether I'm using $ or \z.


Issue with Saving Sessions in SurrealDB: Missing Sessions Table

Hello,
I'm having trouble using the AxumSessionAuth crate with SurrealDB. I followed the example from the GitHub repo, adding simple_logger and logs crates. In the main function, I initialized the logger like this:
simple_logger::init_with_level(log::Level::Debug).expect("Couldn't initialize logging");

When I run the program and navigate to /, I see the following log output:
2024-10-22T17:10:18.917Z INFO  [axum_session::service] Session 4b6f3ce7-c71f-40e4-be64-69caf9c0fd48 did not exist in Database. So it was Recreated.
2024-10-22T17:10:18.917Z DEBUG [axum_session_auth::service] loading user id: 1 from load_user
2024-10-22T17:10:18.919Z INFO  [axum_session::service] Session id 4b6f3ce7-c71f-40e4-be64-69caf9c0fd48: was saved to the database.

Then I visit /login:
2024-10-22T17:10:27.334Z DEBUG [axum_session_auth::service] user id: 1 found in cache
2024-10-22T17:10:27.336Z INFO  [axum_session::service] Session id f1f21a2c-2841-4879-b852-2b89dfa032dc: was saved to the database.

And after navigating to /perm:
2024-10-22T17:10:41.093Z DEBUG [axum_session_auth::service] loading user id: 2 from load_user

Now, every time I refresh any page, I repeatedly see:
2024-10-22T17:10:42.947Z DEBUG [axum_session_auth::service] user id: 2 found in cache
2024-10-22T17:10:43.850Z DEBUG [axum_session_auth::service] user id: 2 found in cache
2024-10-22T17:10:44.386Z DEBUG [axum_session_auth::service] user id: 2 found in cache
2024-10-22T17:10:51.104Z DEBUG [axum_session_auth::service] user id: 2 found in cache

In the main function, the SessionConfig is created like this:
let session_config = SessionConfig::default().with_table_name("test_table");`

However, upon checking SurrealDB, I don't see any table named test_table for storing sessions—only the users and user_permissions tables are present. In the past, using AxumSessionAuth with MySQL, a session table would be automatically created.
Am I missing something here? How can I ensure sessions are saved to the database properly?
Thank you for your help!

I created a project based on the sqlx-example, and after switching from SQLite to MySQL (since I'm more comfortable with it), adding logging, and running the server, I observed the following behavior:
2024-10-22T20:21:24.366Z DEBUG [sqlx::query] summary="CREATE TABLE IF NOT …" db.statement="\n\nCREATE TABLE IF NOT EXISTS test_table (\n  id VARCHAR(128) NOT NULL PRIMARY KEY,\n  expires BIGINT NULL,\n  session TEXT NOT NULL\n)\n" rows_affected=0 rows_returned=0 elapsed=113.39µs elapsed_secs=0.00011339

As expected, the session table was created successfully. Navigating to /, I then saw:
2024-10-22T20:27:38.429Z DEBUG [sqlx::query] summary="INSERT INTO test_table (id, …" db.statement="\n\nINSERT INTO\n  test_table (id, session, expires)\nSELECT\n  ?,\n  ?,\n  ? ON DUPLICATE KEY\nUPDATE\n  expires =\nVALUES\n(expires),\n  session =\nVALUES\n(session)\n" rows_affected=2 rows_returned=0 elapsed=5.531823ms elapsed_secs=0.005531823
2024-10-22T20:27:38.429Z INFO  [axum_session::service] Session id 31b957e5-3cac-4246-9fcd-1770cb1b7e0f: was saved to the database.

The sessions were correctly stored in the database, and after restarting the server, I remained logged in, confirming session persistence.
However, this behavior contrasts with the SurrealDB example, where no session table was created and session persistence did not work. It seems something is preventing the session table from being created in the SurrealDB case.

surreal is a bit different. it creates tables differently than how normal sql does it.  in surrealdb
https://github.com/AscendingCreations/AxumSession/blob/main/databases/surreal/src/lib.rs#L114
this creates and adds a row to the table.
https://github.com/AscendingCreations/AxumSession/blob/main/databases/surreal/src/lib.rs#L129
would retrieve that Row if the session cache no longer contained that data but it is located in the session still.
if you run the load command on surrealdb it should display whats within that table. If it does then it means the table exists. just it might not Exist the same way you would expect like in SQL.

Thank you for pointing out those functions. I've done some debugging in the perm() function, where I now call the load with the session ID as follows everytime that I go to /perm:
let session_id = &auth.session.get_session_id();
auth.session
    .clone()
    .get_store()
    .client
    .as_ref()
    .unwrap()
    .load(&session_id.inner(), "test_table")
    .await
    .unwrap();

I also modified the store and load functions to add logging. After running cargo run and navigating through /, /login, and /perm, the following output was generated:
     Running `target/debug/axum_auth`
2024-10-23T13:52:56.887Z DEBUG [tungstenite::handshake::client] Client handshake done.

--> load called with:
   id: fafb2d28-e75e-4fb2-8f68-1aceba774e00
   table: test_table
--> Response1: Response { client: Surreal { router: OnceLock(Router { sender: Sender { .. }, last_id: 6, features: {LiveQueries} }), engine: PhantomData<surrealdb::api::engine::any::Any> }, results: {0: (Stats { execution_time: Some(59.481µs) }, Ok(Array(Array([]))))}, live_queries: {} }
--> Response3: Ok(None)
--> Response2: None
--> load done

2024-10-23T13:52:58.716Z INFO  [axum_session::service] Session fafb2d28-e75e-4fb2-8f68-1aceba774e00 did not exist in Database. So it was Recreated.
2024-10-23T13:52:58.716Z DEBUG [axum_session_auth::service] loading user id: 1 from load_user

--> store called with:
   id:fafb2d28-e75e-4fb2-8f68-1aceba774e00
   session: {"data":{},"longterm":false}
   table: test_table
--> Response: Response { client: Surreal { router: OnceLock(Router { sender: Sender { .. }, last_id: 9, features: {LiveQueries} }), engine: PhantomData<surrealdb::api::engine::any::Any> }, results: {0: (Stats { execution_time: Some(19.256µs) }, Ok(Array(Array([]))))}, live_queries: {} }
--> store done

2024-10-23T13:52:58.719Z INFO  [axum_session::service] Session id fafb2d28-e75e-4fb2-8f68-1aceba774e00: was saved to the database.
2024-10-23T13:53:00.405Z DEBUG [axum_session_auth::service] user id: 1 found in cache

--> store called with:
   id:c9224f46-9cf5-4d1a-920f-78852753d317
   session: {"data":{"user_auth_session_id":"2"},"longterm":false}
   table: test_table
--> Response: Response { client: Surreal { router: OnceLock(Router { sender: Sender { .. }, last_id: 12, features: {LiveQueries} }), engine: PhantomData<surrealdb::api::engine::any::Any> }, results: {0: (Stats { execution_time: Some(41.617µs) }, Ok(Array(Array([]))))}, live_queries: {} }
--> store done

2024-10-23T13:53:00.407Z INFO  [axum_session::service] Session id c9224f46-9cf5-4d1a-920f-78852753d317: was saved to the database.
2024-10-23T13:53:01.369Z DEBUG [axum_session_auth::service] loading user id: 2 from load_user

--> load called with:
   id: c9224f46-9cf5-4d1a-920f-78852753d317
   table: test_table
--> Response1: Response { client: Surreal { router: OnceLock(Router { sender: Sender { .. }, last_id: 15, features: {LiveQueries} }), engine: PhantomData<surrealdb::api::engine::any::Any> }, results: {0: (Stats { execution_time: Some(47.96µs) }, Ok(Array(Array([]))))}, live_queries: {} }
--> Response2: Ok(None)
--> Response3: None
--> load done

As we can see, the store response confirms that nothing is being saved. This is further verified by the subsequent load call, which returns None.
I have submitted a PR that introduces a potential fix for this issue. After applying the fix, here is the new output for the load function:
--> load called with:
   id: 2dc919bb-8e4a-45ee-88a4-276085c23adf
   table: test_table
--> Response1: Response { client: Surreal { router: OnceLock(Router { sender: Sender { .. }, last_id: 51, features: {LiveQueries} }), engine: PhantomData<surrealdb::api::engine::any::Any> }, results: {0: (Stats { execution_time: Some(94.648µs) }, Ok(Array(Array([Object(Object({"sessionstore": Strand(Strand("{\"data\":{\"user_auth_session_id\":\"2\"},\"longterm\":false}"))}))]))))}, live_queries: {} }
--> Response2: Ok(Some("{\"data\":{\"user_auth_session_id\":\"2\"},\"longterm\":false}"))
--> Response3: Some("{\"data\":{\"user_auth_session_id\":\"2\"},\"longterm\":false}")
--> load done

With this change, everything appears to be working as expected.


Accuracy

Hi,
How can I report accuracy along F1-score?

Hi, by removing the _BIO at the end of the dataset key, you can change from F1 to accuracy.
Outputting both at the same time is out of the box not implemented. You would need to update the respective code so that the F1 score as well as accuracy is computed.
Other option is to do this at the end, when the model is finished with training, that you compute accuracy on the labeled test set.
Best regards
Nils Reimers

How can I print the predictions made for the test set? If we want to analyse it further?

You don't need to print the predicts. As mentioned, the error is already in the gold labels.
You can inspect the gold labels by inspecting the mappings variable. I will use Train_Chunking.py as an example.
In line 59, add:
print(mappings['your_label_key'] # I.e., for Train_Chunking.py, set it to chunk_BIO

This should print you the mappings for your labels.
I expect that one of the label will not follow the BIO encoding scheme, i.e., it will be different than O, B-* or I-*
Next you need to find out why this label is in your dataset.
Best
Nils Reimers

Sorry, previous comment was supposed to be for another issue.
See RunModel.py and RunModel_CoNLL.py how to get the predictions. You can pipe the output to a file to analyze it further.
Best regards
Nils Reimers


The example does not support openai after the version is upgraded

Would you consider upgrading the source code to solve the call problem of the new version of openai？

Can you describe the problem you're experiencing more precisely? I'm not familiar with the breaking changes, but I would be happy to look into it if you can help me get started.


Informative errors

Currently if there is a problem with the request to github, the reported error is cryptic because the code assumes success. Check responses and provide a message to help the user understand what the problem is.

Hi,
Do you just want to check the HTTP status codes using something like http://docs.python-requests.org/en/latest/user/quickstart/#response-status-codes
Adding a Response.raise_for_status() in GistAPI.send() is one way I can think of solving this.

Yes, it is going to involve checking the returned status. However, raise_for_status is not really the way to go. The idea should be to isolate the caller from the particular transport mechanism used (in this case HTTP) and give them information that is semantically meaningful. So the status codes need to be checked from within GistAPI and appropriate exceptions raised.

An example of where this is an issue is when the token is invalid and the response is something like,
1450282075.615 ERROR[gist] string indices must be integers


[explorer] Top traded tokens API

One very important page in the block explorer will be the "Top traded tokens". Actually, our idea is to include them in the home page.
It will show the volumes and prices for the most traded tokens. It will allow the user to quickly understand what they can do in Gnosis Protocol, and get an idea of the expected volumes and prices for the active tokens.
Details about this page can be found in the spec: https://docs.google.com/document/d/1at2Vb8iZMxAKxUkBMR_HaMn6UjYhTD7dE9mxy_xaJP0/edit#
API
In this issue I would like to discuss a suitable API for supporting thing use case. In the spec document there's already a detailed proposal which we could use for iterating and shaping the API.

Would you prefer to discuss the API in this issue or on the document you linked?

Whatever works best. I thought it was easier in the doc, then we could make it into a swagger format PR


multiple select - move

would be nice if we could select multiple items to move them to other board

this!


Wireframe Shader not good on Spheres

Overview
The MRTK Wireframe Shader has artifacts when colliding with other geometries. In Editor and in Build
Expected behavior
The rendered sphere should look round
Actual behavior
The rendered sphere looks edgy

Steps to reproduce
Add a sphere GO
Add a Material with Wireframe Shader from MRTK
Add a cube or plane GO that intersects with the sphere
Unity editor version
testd with 19.3
Mixed Reality Toolkit release version
2.4

I just started using the MRTK wireframe shader and noticed this happening on my objects, as well as other z-fighting issues.
Line 42 in MRTK_Wireframe.shader: Offset 50, 100, seems to be hardcoding the depth offset values. If I comment it out, then it appears to fix this issue and the z-fighting issues I was having with my objects.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


SVG Graphics Don't Load Properly

Attempting to use the "Launcher icon generator" > "Foreground" > "Image" option to load an SVG file causes this to happen:

The file loaded is a stock icon, downloaded from the Noun Project.  This screenshot shows the generated icon's web version, but all versions have the same distorted graphic.
I'm using Chrome on Ubuntu Linux, if it matters.

This is caused when the <svg> has no width and height set, but does have a viewBox set. I don't see an immediate quick fix, but a workaround is to edit the SVG to include width and height.

Bug for reference:
https://bugs.chromium.org/p/chromium/issues/detail?id=475009

Workaround confirmed to work.
For those unfamiliar with editing SVG graphics, the procedure is:

Open the SVG in a text editor.
  Locate the attribute "viewBox" in the parent element, which should have a series of four numbers set as its value.
Add a "width" attribute to the parent element.  Set its value to the third number in the "viewBox" attribute.
Add a "height" attribute to the parent element.  Set its value to the fourth number in the "viewBox" attribute.
Save your changes.

The SVG should now render correctly in Android Asset Studio.

Fixed probably a while ago


Add Fergal O'Connor to contributors list

Thanks for the tutorial, much appreciated

@fergaljamesoconnor, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Add _ext.py for Python codegen

We currently rely on .ovverrides to inject code at runtime.
Sometimes it is much cleaner to inject code at compile-time. For instance:
setattr(TextLogLevelType, "CRITICAL", "CRITICAL")
TextLogLevelType.CRITICAL.__doc__ = """ Designates catastrophic failures. """

setattr(TextLogLevelType, "ERROR", "ERROR")
TextLogLevelType.ERROR.__doc__ = """ Designates very serious errors. """

setattr(TextLogLevelType, "WARN", "WARN")
TextLogLevelType.WARN.__doc__ = """ Designates hazardous situations. """

setattr(TextLogLevelType, "INFO", "INFO")
TextLogLevelType.INFO.__doc__ = """ Designates useful information. """

setattr(TextLogLevelType, "DEBUG", "DEBUG")
TextLogLevelType.DEBUG.__doc__ = """ Designates lower priority information. """

setattr(TextLogLevelType, "TRACE", "TRACE")
TextLogLevelType.TRACE.__doc__ = """ Designates very low priority, often extremely verbose, information. """

is pretty ugly, and I'm not even sure it works. It would be much nicer to output this code:
class TextLogLevelType(BaseDelegatingExtensionType):
    CRITICAL: Final = "CRITICAL"
    """ Designates catastrophic failures. """

    ERROR: Final = "ERROR"
    """ Designates very serious errors. """

    WARN: Final = "WARN"
    """ Designates hazardous situations. """

    INFO: Final = "INFO"
    """ Designates useful information. """

    DEBUG: Final = "DEBUG"
    """ Designates lower priority information. """

    TRACE: Final = "TRACE"
    """ Designates very low priority, often extremely verbose, information. """

    # codegen code here

We also want this in order to implement Boxes2D::from_xywh etc

@jleibs says we could be using "mixin classes" for this


Update implementation for ContextValue

SpongeAPI | SpongeCommon
This updates the implementation, borrowing the implementation of previously NamedCauseFilter.

Closed by @Zidane , See API PR.

@ryantheleach this PR is still needed as the ContextValue filter is still not implemented. The api pr had adjacent changes which is what @Zidane was rejecting.


Step Form ForwardRef Error

Describe the bug
ForwardRef error
To Reproduce
Steps to reproduce the behavior:

Go to Advanced Forms -> Step Form
Click Next button
You will see an error in the console

Expected behavior
No errors
Screenshots

fixed in pr #196


Fix Project number as per the invoice

Describe the task
Currently, project numbers are different than we have set up while sending the old invoices. We need to fix that numbers.
Expected behavior
All the project numbers should match with this sheet.
References
Specify any reference material / links

Hi @Shivam-Samant before merging code for generating invoice number we need implement this feature. Since client  id and project-id are not  correctly mapped.

Thanks @Shivam-Samant  I will  take care of this task
@pankaj-ag assigning this task to you as per the discussion with @mohitsharma-22


Wait until deploy completes

Related to #26
Example usage:
    rollout_check: [bodega-server, bodega-server-envoy]
    rollout_timeout: 300

Example output:
$ timeout -t 300 kubectl rollout status deploy bodega-server --namespace pubp-dev
Waiting for rollout to finish: 2 old replicas are pending termination...
Waiting for rollout to finish: 2 old replicas are pending termination...
Waiting for rollout to finish: 2 old replicas are pending termination...
Waiting for rollout to finish: 2 old replicas are pending termination...
deployment "bodega-server" successfully rolled out

There are no checks for deamonset or statefulset, but it would be easy to add this functionality.

Updated code.
Example usage:
    wait_deployments: [bodega-server, bodega-server-envoy]
    wait_seconds: 300

Example output:
[1/2] Waiting till rollout completes for bodega-server

$ timeout -t 300 kubectl rollout status deployment bodega-server --namespace pubp-dev
deployment "bodega-server" successfully rolled out

[2/2] Waiting till rollout completes for bodega-server-envoy

$ timeout -t 300 kubectl rollout status deployment bodega-server-envoy --namespace pubp-dev
deployment "bodega-server-envoy" successfully rolled out

Sure, updated


Sharing Implementation Ideas

Hello,
[First, I want to mention that English is not my first language, so I've used AI translation for this message]
About one or two month ago, I noticed that the project had become relatively inactive. Then I decided to create a similar library, improve some aspects of the implementation.
Now that the project is under new management, I'm really happy to see its progress. I'd love to share some of the ideas and improvements I developed in my own implementation. If you find them useful, I would be glad to contribute to the future development of the project.
I’m not sure if it’s good, but for convenient, here is my implementation: realme, And I think the value part and the adaptor part is more flexible and extensible than now.
I'd love to discuss these ideas with you . If you're open to it, I'd be happy to contribute my effort.
Looking forward to your feedback!
Thank you so much.

Input is welcome! Note that I'm not likely to invest a lot on implementation and design work but more be a steward to keep things going, so moving this forward will depend a lot on you and input from others.
I'd recommend opening individual issues or discussions for what you propose so we can have more focused conversations.  As such, I'm going to go ahead and close this for now.  If there is a reason to keep it open, let us know!


Authorization

Is there a way to do authorization with this as well? I see that there is data validation, but I don't see any authorization. It seems like the Breeze way is to do 'Save Interception'. Is that supported?

We believe that authorization is a different concern, and it depends on the applications. We solved it simply by denying access to certain routes for certain roles. You don't have to depend on the library for this purpose - you shouldn't IMHO. You are not limited to use any of the awesome features provided by Doctrine framework - e.g: lifecycle events. Let's do it the PHP way!
P.S: if you have any better suggestion or solution, please comment. We welcome pull requests too :-)

Lifecycle events on Doctrine should work in my situation. I'll give that a shot. Thanks!


Fix sets and Java threading

Automated message from Dropbox CLA bot
@izacus, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here and update the thread so we can consider merging your code.

Uh, wrong repo.


Add multiple and closeOnSelect options.

Property Binding Usage:
[multiple]="true"
[closeOnSelect]="false"

Thank you for this PR. With version 1.0.0 you can choose from all available Select2 options. Currently version 1 is in beta, but should be released in few days. You can check out implementations in this demo.


Light engine

Basically ported https://github.com/PaperMC/Starlight without the support of conditionally full opaque blocks (for example, piston heads).
Before my PR for MinestomDataGenerator is accepted, my fork is being used as a dependency within this PR.
Seems to work pretty well, but don't bother to message me for implementation details.

@RinesThaix do you mind asking the paper team if this is fine ?

Are we using @Nullable? I thought that @Nullable was the default for functions.

Closed, is stale and has licensing problems as discussed in Discord


map

@stephencelis is there a better/nicer way to perform this kind of operation:
public func mapMiddleware<A, B>(
  _ f: @escaping (A) -> B
  )
  -> (@escaping Middleware<StatusLineOpen, ResponseEnded, B, Data>)
  -> Middleware<StatusLineOpen, ResponseEnded, A, Data> {

    return { middleware in
      return { $0.map(f) |> middleware }
    }
}

Hey @emilwojtaszek! Sorry for the delay. We don't monitor this repo's issues/PRs as closely as some of our other open source. An operation we find helpful is filterMap, which we use all over the @pointfreeco code base: https://github.com/pointfreeco/pointfreeco/search?utf8=✓&q=filterMap&type=
It allows for a failable transformation into nil with a failover middleware.
@mbrandonw and I are hoping to explore some ergonomic improvements to the HTTP pipeline soon!


The guide is very out of date

Hi.
The latest version of VS 2019 Community Edition does not create Azure App Services, SQL DB servers or databases as the guide in this document describes. I did my best to try and follow it anyhow but ended up with an application that won't run in Azure.
Please try and find time to update the guide as this seems to be a good starting point for this type of application.
Thanks.

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

ID: 72d855c6-51fd-a1da-5c44-8b44658c0211
Version Independent ID: 15c87b07-e89e-2b0c-6337-3940a5259101
Content: Tutorial: ASP.NET app with Azure SQL Database - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-dotnet-sqldatabase.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@MartinNr5 while creating an Azure SQL instance has changed, it is labeled that this feature is being deprecated. Perhaps being replaced by something else; I'm not sure.

We'll update the docs as appropriate. In the meantime, the changes for creating an App Service has been broken up into a multi-wizard layout but it isn't that far off. For instance, once select Publish from context menu of the project, you're opted to select Azure as a destination, followed by app service type (e.g. Windows/Linux, Container Registry, Virtual Machine).

After selecting your target, you'll select your resource group, app service plan, and app service plan to create your publish profile.

@MartinNr5 I have a pending PR that revises the tutorial with the latest Visual Studio 2019 workflows. Thanks for your feedback.


Add Remaining built-in constraints

I'll pick this up. Will start with the tags constraint: so talks cannot be scheduled in the same session unless they share a tag.

including no tag? i.e. Any untagged talks can be scheduled in the same session.

Yup. I'm thinking we can handle that generically: no tag is a tag.

@meatballs currently the unavailability maps speakers to slots. Would you be averse to changing that to map from events to slots?
So this would have:
 21 class Event(NamedTuple):
 22     name: str
 23     event_type: EventType
 24     duration: int
 25     roles: Dict[Role, Person]
 26     tags: List[str]
 27     unavailability: List[Slot]

The unavailability list for any given event would by default be empty.
This would just make things a little bit more generic and also addresses our chat that unavailability for an event in a given slot need not be about the speaker (as a committee we could decide that we don't want a particular event to happen in a particular room).

we don't want a particular event to happen in a particular room

I think that's a separate issue
The main issue to deal with for unavailability is that we will straight away have speakers telling us that they cannot be at the conference for particular days. That's what I mean by 'unavailability.'

So, I don't see how it could be mapped to an Event - anyone who can't make Friday still won't be able to make Friday regardless of what we've scheduled that day.

This is just about how the data is stored and doesn't actually change anything.

The main issue to deal with for unavailability is that we will straight away have speakers telling us that they cannot be at the conference for particular days. That's what I mean by 'unavailability.'

When speakers say they are unavailable for a given day that in effect translates to the events that "belong" to that speaker being unavailable on the slots on a given day. So this doesn't change anything from that point of view.
By allowing for events to be more generically unavailable on a given slot it just makes things a bit more flexible.
This branch has those changes and the availability constraint programmed in: https://github.com/drvinceknight/ConferenceScheduler/tree/2-availability
(It's forked off my work on #5, will wait for that to go in before opening the PR)

When speakers say they are unavailable for a given day that in effect translates to the events that "belong" to that speaker being unavailable on the slots on a given day.

It also translates to sessions that they might chair, speakers they might be mentoring, workshops they might be assisting at and other things we haven't even thought of.

True and similarly we can have events that the committee doesn't want to
schedule in a given slot even if the speaker is available (for some
reason). Perhaps we need unavailability in both authors and events... I'm
at an all day thing, I'll open the PR when I get a chance and we can
discuss there? :)
On Tue, 2 May 2017, 09:03 Owen Campbell<EMAIL_ADDRESS>wrote:

When speakers say they are unavailable for a given day that in effect
translates to the events that "belong" to that speaker being unavailable on
the slots on a given day.
It also translates to sessions that they might chair, speakers they might
be mentoring, workshops they might be assisting at and other things we
haven't even thought of.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PyconUK/ConferenceScheduler/issues/2#issuecomment-298542274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACCGWj9sLG9sEsVkiBhX1XjUt1hW4pAPks5r1uNngaJpZM4NNLmj
.

True and similarly we can have events that the committee doesn't want to
schedule in a given slot even if the speaker is available

But that's a different issue - one that I think should be covered by an ad-hoc constraint in the 'script' for that year. Here, we're just trying to deal with the built-in availability constraints.

Cool. I'll tweak, push and PR when I get a moment (might be later today).
On Tue, 2 May 2017, 09:14 Owen Campbell<EMAIL_ADDRESS>wrote:

True and similarly we can have events that the committee doesn't want to
schedule in a given slot even if the speaker is available
But that's a different issue - one that I think should be covered by an
ad-hoc constraint in the 'script' for that year. Here, we're just trying to
deal with the built-in availability constraints.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PyconUK/ConferenceScheduler/issues/2#issuecomment-298546576,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACCGWojXvdoobK0rBITT3Zkl6VHWD6adks5r1uXIgaJpZM4NNLmj
.

duration - only scheduling events in slots with a matching duration

I'll pick this up next.

suitability - not scheduling events in rooms which are unsuitable for that type of event

I think this is covered: we can mark an event as unavailable for a given slot. We can pre process whether or not a given event is suitable for a given slot when we get the data in (and just add that to the event unavailability).

duration - only scheduling events in slots with a matching duration

Actually, come to think of it: that's also already covered by the same logic: we can simply include this all in a data input validation:

If an event has duration 30 mins then all the slots with duration < 30 mins should be marked as an unavailability for that event.
If an event has a particular type then all the slots that cannot cater for that type should be marked as an unavailability for that event.


Add guc gp_enable_explain_motion_detail to show sender time when Expl…

Show motion sender time when Explain Analyse.
When guc set, Explain Analyse result will be:
actual time=<receive_start>..<receive_end>/<sender_start>..<sender_end>
gp_enable_explain_motion_detail defaults to off.

fix #ISSUE_Number

Change logs
QEs will now send Motion Sender instruments to QD.
QD will collect two instruments(sender and receiver) to MotionPlanState->instruments[].
Add guc gp_enable_explain_motion_detail to show sender time.
Add test cases.
Why are the changes needed?
In plan on QD, Motion is a single node, so we need an array of two to collect both sender and receiver instruments.
Add guc to show this information only when needed.
Does this PR introduce any user-facing change?
Yes, the guc gp_enable_explain_motion_detail default to off. Show sending time by:
set gp_enable_explain_motion_detail to on
in client.
How was this patch tested?
Added icw test case and passed in local.
Contributor's Checklist

 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.

Close the PR since no response for a long time.


[SPARK-15674][SQL] Deprecates "CREATE TEMPORARY TABLE USING...", uses "CREAT TEMPORARY VIEW USING..." instead

What changes were proposed in this pull request?
The current implementation of "CREATE TEMPORARY TABLE USING..." is actually creating a temporary VIEW behind the scene. We should just use "CREATE TEMPORARY VIEW USING..." instead.
Example:
scala> spark.sql("CREATE temporary view  my_tab7 (c1: String, c2: String)  USING org.apache.spark.sql.execution.datasources.csv.CSVFileFormat  OPTIONS (PATH '/Users/seanzhong/csv/cars.csv')")
scala> spark.sql("select c1, c2 from my_tab7").show()
+----+-----+
|  c1|   c2|
+----+-----+
|year| make|
|2012|Tesla|
...

It NOW prints a deprecation warning if "CREATE TEMPORARY TABLE USING..." is used.
scala> spark.sql("CREATE temporary table  my_tab7 (c1: String, c2: String)  USING org.apache.spark.sql.execution.datasources.csv.CSVFileFormat  OPTIONS (PATH '/Users/seanzhong/csv/cars.csv')")
16/05/31 10:39:27 WARN SparkStrategies$DDLStrategy: CREATE TEMPORARY TABLE tableName USING... is deprecated, please use CREATE TEMPORARY VIEW viewName USING... instead

How was this patch tested?
Unit test.

Test build #59662 has started for PR 13414 at commit 94d66c2.

So I am not sure I understand this one. Why should we deprecate this in favour of creating a view? A create temp table ... using statement describes the access to a physical storage; which in my book is a table.
Could you elaborate on why we need this?

Test build #59663 has started for PR 13414 at commit c2a29b3.

Test build #59662 has finished for PR 13414 at commit 94d66c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class CreateTempViewUsing(

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59662/
Test PASSed.

Test build #59663 has finished for PR 13414 at commit c2a29b3.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class CreateTempViewUsing(

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59663/
Test PASSed.

Merged build finished. Test PASSed.

@hvanhovell

create temp table ... using statement describes the access to a physical storage; which in my book is a table.

We still allow create table using..., what we deprecate is "create temporary table using...". We still treats external data source as tables. To wrap the external data source as a table, we can use "create table using..." .
temporary views and temporary tables are intermediate layers between user and the actual table:
User --> Temporary view/table --> External data source table (Can be wrapped by create table using...)

Currently, in our implementation, we don't support temporary table, we only supports temporary view. The difference is that:

Temporary view is backed by a SQL string, which acts like a pointer. So every time we use the temporary view, the SQL is RE-executed again, which will asks data from the original data source.
Temporary table is supposed to execute the SQL for ONLY once, and store the result in a temporary HDFS directory. Every time you use the temporary table, we are actually using the data in the temporary HDFS directory directly, without bothering the original data source.

@hvanhovell
I updated the description, please check whether it makes more sense now.

@clockfly the description is getting there. IIUC the problem we are solving is the following:
CREATE TEMPORARY TABLE ... USING ... allows us to create a temporary (session bound) connection to a (potentially) permanent data store. When the session finishes, the table definition (connection) is dropped, but the data is not. This is more-or-less the behavior you expect with a TEMPORARY EXTERNAL table (do we have those?), and this actually violates the common definition of a temporary table in which both the table definition and the data are session bound.
Using CREATE TEMPORARY VIEW ... USING ... accomplishes two things:

It doesn't make assumption about the underlying data (it can both be permanent or session bound).
It doesn't allow user to write to the datasource.

I do have a couple of issues with this:

Using CREATE TEMPORARY TABLE ... USING ... should still be allowed to use if you are using an actual session local temporary table. We could detect these by checking if a schema is defined (the location is also an issue). How do we deal with this use case?
I would support creating a CREATE TEMPORARY EXTERNAL TABLE ... USING ... to retain the current behavior.

What do you think?

@hvanhovell Probably we can talk more face to face next week.

@clockfly this looks pretty good. I have left some (minor) comments.

Test build #60079 has started for PR 13414 at commit e521310.

@hvanhovell  Thanks for the review.
Updated.

Test build #60081 has started for PR 13414 at commit 127c309.

LGTM pending Jenkins

Test build #60079 has finished for PR 13414 at commit e521310.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60079/
Test PASSed.

Test build #60081 has finished for PR 13414 at commit 127c309.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60081/
Test PASSed.

Merged build finished. Test PASSed.

Thanks! Merging to master/2.0


Fix debug mode

Fixes https://github.com/gruntwork-io/terragrunt/issues/1502, where the Debug flag wasn't being propagated across the Clone call.

Thanks for review! Merging this now.

Thanks for review! Merging this now.


API Key

I have an API key from FRED which works, but where do i put it within the code so that i can run package tests?

@oganator You may set the API Key as an environment variable or leverage the helper methods on the client. The following test provides examples:

https://github.com/nswekosk/fred_go_toolkit/blob/master/fred_toolkit_v1_test.go

Anything else I can help you with?

Closing issue due to lack of a response.


[social network] socket issue with wrk

I'm unable to get statistics out of the wrk tool:
I ran the following on a swarm setup (<IP_ADDRESS> is the address where the frontend is reachable using the browser):
root@virtual-158-0-2:/tmp/dsb/socialNetwork/wrk2# ./wrk -D exp -t 1 -c 1 -d 10 -L -s ./scripts/social-network/compose-post.lua http://<IP_ADDRESS>:8080/wrk2-api/post/compose -R  1
./scripts/social-network/compose-post.lua: ./scripts/social-network/compose-post.lua:2: attempt to index global 'socket' (a nil value)
./scripts/social-network/compose-post.lua: ./scripts/social-network/compose-post.lua:2: attempt to index global 'socket' (a nil value)
Running 10s test @ http://<IP_ADDRESS>:8080/wrk2-api/post/compose
  1 threads and 1 connections
  Thread calibration: mean lat.: 0.963ms, rate sampling interval: 10ms
  Thread Stats   Avg      Stdev     99%   +/- Stdev
    Latency     -nanus    -nanus   0.00us    0.00%
    Req/Sec       -nan      -nan   0.00      0.00%
  Latency Distribution (HdrHistogram - Recorded Latency)
 50.000%    0.00us
 75.000%    0.00us
 90.000%    0.00us
 99.000%    0.00us
 99.900%    0.00us
 99.990%    0.00us
 99.999%    0.00us
100.000%    0.00us

  Detailed Percentile spectrum:
       Value   Percentile   TotalCount 1/(1-Percentile)

       0.000     1.000000            0          inf
#[Mean    =         -nan, StdDeviation   =         -nan]
#[Max     =        0.000, Total count    =            0]
#[Buckets =           27, SubBuckets     =         2048]
----------------------------------------------------------
  17 requests in 10.00s, 3.65KB read
  Non-2xx or 3xx responses: 17
Requests/sec:      1.70
Transfer/sec:     373.88B

the warning about the nil socket sounds weird to me though.
Originally posted by @msimonin in https://github.com/delimitrou/DeathStarBench/issues/182#issuecomment-1091439733

I found a solution to the same problem for now, simply by replacing require "socket" with socket = require "socket". I think this might be due to the version of lua or something, but I know very little about lua, so this is just a guess.

@gavinlichn any insight here ?

I found a solution to the same problem for now, simply by replacing require "socket" with socket = require "socket".

This solution worked for me too, thanks @Nick-LCY.
@msimonin and @balamurugan15 : can you share the output of these commands:
lua
socket = require("socket")
print(socket._VERSION)

Here's what I see on my system:
$ lua
Lua 5.1.5  Copyright (C) 1994-2012 Lua.org, PUC-Rio
> socket = require("socket")
> print(socket._VERSION)
LuaSocket 3.0-rc1

thanks  @dmorse76, settting socket=require "socket" does solve the issue wil nil values but ...
the benchmark is still non-functionnal
root@virtual-158-0-2:/tmp/dsb/socialNetwork/wrk2# ./wrk -D exp -t 1 -c 1 -d 10 -L -s ./scripts/social-network/compose-post.lua http://<IP_ADDRESS>:8080/wrk2-api/post/compose -R  1
Running 10s test @ http://<IP_ADDRESS>:8080/wrk2-api/post/compose
  1 threads and 1 connections
  Thread calibration: mean lat.: 12.998ms, rate sampling interval: 23ms
  Thread Stats   Avg      Stdev     99%   +/- Stdev
    Latency     -nanus    -nanus   0.00us    0.00%
    Req/Sec       -nan      -nan   0.00      0.00%
  Latency Distribution (HdrHistogram - Recorded Latency)
 50.000%    0.00us
 75.000%    0.00us
 90.000%    0.00us
 99.000%    0.00us
 99.900%    0.00us
 99.990%    0.00us
 99.999%    0.00us
100.000%    0.00us

  Detailed Percentile spectrum:
       Value   Percentile   TotalCount 1/(1-Percentile)

       0.000     1.000000            0          inf
#[Mean    =         -nan, StdDeviation   =         -nan]
#[Max     =        0.000, Total count    =            0]
#[Buckets =           27, SubBuckets     =         2048]
----------------------------------------------------------
  17 requests in 10.00s, 3.57KB read
Requests/sec:      1.70
Transfer/sec:     365.40B

root@virtual-158-0-2:/tmp/dsb/socialNetwork/wrk2# dpkg -l | grep lua
ii  liblua5.1-0:amd64               5.1.5-8.1+b3                   amd64        Shared library for the Lua interpreter version 5.1
ii  liblua5.1-0-dev:amd64           5.1.5-8.1+b3                   amd64        Development files for the Lua language version 5.1
ii  lua-any                         27                             all          helper script for shebang lines in Lua scripts
ii  lua-bit32:amd64                 5.3.0-3                        amd64        Backport of the Lua 5.2 bit32 library to Lua 5.1
ii  lua-filesystem:amd64            1.8.0-1                        amd64        luafilesystem library for the Lua language
ii  lua-json                        1.3.4-2                        all          JSON decoder/encoder for Lua
ii  lua-lpeg:amd64                  1.0.2-1                        amd64        LPeg library for the Lua language
ii  lua-posix:amd64                 33.4.0-3+b1                    amd64        posix library for the Lua language
ii  lua-sec:amd64                   1.0-1                          amd64        SSL socket library for the Lua language
ii  lua-socket:amd64                3.0~rc1+git+ac3201d-4          amd64        TCP/UDP socket library for the Lua language
ii  lua-term                        0.07-0.1+b1                    amd64        Lua module for manipulating a terminal
ii  lua5.1                          5.1.5-8.1+b3                   amd64        Simple, extensible, embeddable programming language
ii  luarocks                        2.4.2+dfsg-1.1                 all          deployment and management system for Lua modules

root@virtual-158-0-2:/tmp/dsb/socialNetwork/wrk2# lua
Lua 5.1.5  Copyright (C) 1994-2012 Lua.org, PUC-Rio
> socket=require("socket")
> print(socket._VERSION)
LuaSocket 3.0-rc1

@msimonin When I try your command line parameters, I get almost exactly the same output.  The -nan values indicate that some of your values need to be adjusted.  Please try -t 5 -c 5 instead and let me know if the output is better, here's what I get:
#[Mean    =     1234.639, StdDeviation   =      339.985]
#[Max     =     1633.280, Total count    =         9365]
#[Buckets =           27, SubBuckets     =         2048]
----------------------------------------------------------
  11708 requests in 10.00s, 2.40MB read
Requests/sec:   1170.78
Transfer/sec:    245.82KB

Also keep in mind that compose-post is very "heavy" vs. the other scripts (details in #59).

Hey :) That looks indeed better with -t 5 -c 5.
#[Mean    =     2369.672, StdDeviation   =      770.661]
#[Max     =     3100.672, Total count    =         3774]
#[Buckets =           27, SubBuckets     =         2048]

Do you know why -t 1 -c 1 are problematic ?

Hey :) That looks indeed better with -t 5 -c 5.

Great!  So your setup is fine.

Do you know why -t 1 -c 1 are problematic ?

I think 1 thread, 1 connection, at a rate of 1 request/second is just way too low to accurately calculate latency and throughput.  I've also seen socket timeout warnings when the duration is too low.  Generally with wrk, a good rule of thumb is to try larger values (just not so large that they overwhelm your system!).

Thanks a lot @dmorse76 for all of this !
Maybe let's see if @balamurugan15 is happy with that too :) before closing

Just a simple quick question,
What does Total count ,buckets and SubBuckets signify here ?
#[Mean    =        0.453, StdDeviation   =        1.883] #[Max    =       35.360, Total count    =          431] #[Buckets =           27, SubBuckets     =         2048]


118 frontend   rework edituserprofile page to allow mutliple location entry

Changed form to handle new inputs for country, state and city
Reworked setting the location logic to allow user to set multiple locations
Created a way for users to remove a location
Created functionality for auto-populated suggestions as the user types
Refactored rendering of location on Profile Page to handle new data structure and render in correct format
https://github.com/crystaljobe/change-mate/assets/141180011/71b229d1-bc4b-4594-959c-2d7a6cee153f

Hey Shawn, looks good to me . Just two minor requests -
Since we have a lot of useStates now it's getting hard to remember which one does what, so would you just mind to add  comments the ones you added or changed to specify what each state is for or when to use it. That way no one uses the wrong states in the wrong way.
The other thing is the rendering of the multi locations on the profile page card component is looking a little wonky, not a high priority or anything but if you don't want to work on that right now would you mind to just add an issue under the "backlog" iteration.
Thanks!

Added comments to useStates
Fixed spacing issue on User Profile Location Display

Beautiful, thank you for doing that!


Dependencies: Remove duplicate packages installed & upgrade sb

@tmeasday I suspect the problem you described here: https://chromaticqa.slack.com/archives/C051TQR6QLC/p1696295480474949 is possibly already fixed in latest storybook.
I upgraded and cleaned the lockfile, and it seems to not happen on my machine when I run storybook on this branch.
Would be care to verify? perhaps I'm missing something obvious.

📦 Published PR as canary version: 0.0.106--canary.131.be04536.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

We have updated to 8.0 deps. So, I'm closing this.


Replace implementation with Aircompressor code

The core compression and decompression code has been replaced with the latest code from Aircompressor. The existing interfaces have been retained where possible. The notable exception is the deprecated SnappyInputStream and SnappyOutputStream output stream implementations have been removed. Any existing data in these formats should be converted to the specification defined framed formats using an older version of this library.

@Marcono1234 this PR brings iq80 snappy in line with aircompressor

This also drops support for big endian CPUs (which would currently be using SlowMemory), right?
Maybe another alternative would be to always SlowMemory, and never UnsafeMemory. This would probably keep the same functionality, but maybe this performance decrease is not acceptable. Though on the other hand this at least solves the issue, and users are encouraged to migrate to Aircompressor anyway.
Or alternatively, fix the missing / incorrect checks which cause the issue. But I will probably check again then, just to be safe that there is no other issue.
(Though of course in the end it is your decision, so consider everything I wrote above only as suggestion please.)

Honestly there are so few modern shipping systems that are big endian (PPC and s390x), I don't think it is worth the effort.  If someone really complained loudly, I could likely add it back.


build-zfs-module: Update builder to use f38

CoreOS stable is now fedora 38, updating the default BUILDER_VERSION ARG value.

Yeah, definitely auto-matable, but this relates to a bigger picture question of whether it'd make sense to have quay.io/fedora/fedora-coreos:38 too...


Add HTML report

Addresses #44

[x] Summary plots

[x] relative free energy distribution
[x] cumulative distribution

[ ] RUN info

[x] RUN number
[x] DDG
[x] DDG 95% CI
[x] work distributions (thumbnail for top 20, links to png and pdf files)
[x] convergence plot (thumbnail, links)
[ ] SMILES? Molecule thumbnail?

Sample output

Jinja templates! Yay!

Wow, this is amazing!
The most important modifications, in order, are:

Add links to download compounds.{pdf,csv,sdf} and proteins.pdb generated by the code in postprocess.py, since these will be important for inspection
Add the molecule title to the rows (in addition to the RUN) since this is the most critical information for the chemists
Until we change to solving for absolute free energies (#52), we should display the free energy difference as "ΔΔG (kcal/mol)" and show "X ± Y" where Y is one standard error, or "X [W, Z]" where [W, Z] represents the 95% CI
If you can insert images in the table, but if so, you can use this idiom to generate images of the molecules from the SMILES strings using the openeye-toolkit conda package installed from the openeye channel. I believe this is already installed (along with the license file) in the perses environment. (Ask if you need a local copy!)
If we can add links to each row to the SDF and PDB files for molecule and protein in structures/, that would be awesome

Later, we can make this fancy and interactive!

This is awesome!!!

I'd personally prefer to have kcal/mol everywhere, including the table, but I'm happy to be ignored
Molecule thumbnail would be great, I can write a function for you if you want me to generate a .pdf of the ligand structure
3 decimal places might be overkill

Maybe I'm stupid but what's UB? upper bound?

We should eliminate that column and just have the X ± Y or X [Y, Z] 95% CI format.

I'd personally prefer to have kcal/mol everywhere, including the table, but I'm happy to be ignored

Let's use kcal/mol in all the plots for now. The chemists think in these units.
Later, after we integrate DiffNet (#52), we can also generate versions that show the Kd (in M, for Molar) on a log scale, so that we can show whether we expect pM, nM, µM, etc.

3 decimal places might be overkill

The appropriate way to represent uncertainty in X ± Y format is to report one sig fig of the standard error Y and then report X to the same number of decimal places, e.g.:
1.234 ± 0.002
1.23 ± 0.04
1.2 ± 0.3
5 ± 1

but not
1.234 ± 0.023
1.23 ± 0.17

For the 95% CI, we just need enough sig figs for the CI numbers to be distinct in how they are represented:
1.23 [1.20, 1.24]

but not
1.231 [1.201, 1.247

There's a useful package that helps with this:
https://pythonhosted.org/uncertainties/
but pint or unyt may have similar capabilities.

Whoops, we should also get the SMILES string in each row.

@jchodera @hannahbrucemacdonald

Add the molecule title to the rows (in addition to the RUN) since this is the most critical information for the chemists

you can use this idiom to generate images of the molecules from the SMILES

Molecule thumbnail would be great

Dumb question: each RUN includes 2 sets of title and smiles (start and end). Do we want to display one or the other or both in the table?

@hannahbrucemacdonald

what's UB? upper bound?

Yes, I originally had the idea to sort by upper bound of the CI to avoid bad estimates floating to the top (but this doesn't seem to be too much an issue). I'm in the process of changing this to standard error format (X ± Y).

Also, this seems silly, but can we change the font to something sans serif? Maybe one of the free Google fonts (instructions on including them are here)?
Type specimens are here. Roboto is pretty common, or Alata if we want to be extra special.

Erm, so at the moment when we are sorting by RUNS I think we only want the ‘old’ molecule, but when we move to more complicated networks of graphs I think we would want to not report on a RUN but have a row for each ligand (so only one ligand shown still)
Maybe we could chat about getting the more complicated graph analysis (DiffNet) into the pipeline soon?
Ha I was surprised that John hadn’t commented on the font... 😂

Erm, so at the moment when we are sorting by RUNS I think we only want the ‘old’ molecule, but when we move to more complicated networks of graphs I think we would want to not report on a RUN but have a row for each ligand (so only one ligand shown still)

Yes! So after #32 -> #52 get integrated, we will want to split up the table into three tables: (1) compounds, (2) molecules, and (3) transformations, matching the new JSON format.
Compounds will have

Compound ID (CID)
compound image
reference SMILES for the compound
predicted absolute free energy (ΔG = -7.23 [-7.29, -7.20] kcal/mol)
predicted affinity (Kd = 4.9 [2.3, 10.4] µM)
link to structure snapshots?

Molecules will have

Molecule ID (MID)
Compound ID (CID)
molecule image
SMILES for the specific molecule state
predicted absolute free energy (ΔG = -7.23 [-7.29, -7.20] kcal/mol)
predicted affinity (Kd = 4.9 [2.3, 10.4] µM)
link to structure snapshots?

Transformations will have

RUN ID
old/new MIDs
predicted relative free energy (ΔΔG = -1.23 [-1.29, -1.20] kcal/mol)
work plots
convergence plots
link to structure snapshots?

Maybe we could chat about getting the more complicated graph analysis (DiffNet) into the pipeline soon?

Can you set that up @hannahbrucemacdonald ? Happy to join if it would help!

Looking great!
The only issue I spotted was a logic error in sig figs: -0.8 ± 0.08 isn't correct since it should have one more sig fig in the -0.8 quantity.

Let's also show the whole CDF so we get a sense for the fraction that score well!
Also, great font!

Let's also show the whole CDF so we get a sense for the fraction that score well!

This should be the case already (the sample was run on my local machine with a small fraction of the runs)


[NEW ALGORITHM] Parallel Graph Algorithms

Issue will be closed if:

You mention more than one algorithm. You can create a separate issue for each algorithm once the current one is completed.
You propose an algorithm that is already present or has been mentioned in a previous issue.
You create a new issue without completing your previous issue.

Note: These actions will be taken seriously. Failure to follow the guidelines may result in the immediate closure of your issue.

Name:
Parallel Graph Algorithms - Graph
About:
Algorithms optimized for graph processing on large-scale data (such as social networks), which can process millions of nodes and edges in parallel.

Labels:
new algorithm, gssoc-ext, hacktoberfest, level1

Assignees:

[ ] Contributor in GSSoC-ext
[ ] Want to work on it

Do not create unnecessary PR until you complete the previous one.

@pankaj-bind okk 1 PR at a time now can i create new PR for this my old PRs are merged.


No library on current platform error from godot (Windows)

Hello.
i love your notification plugin.
it took me minutes to set up for the 1st time
and it working out of the box in both Android and iOS platform!
i have small issue with it,
im using Windows and once i run godot, i have this issue:

also, when i run the project inside Windows, i get this error:

normally, those errors does not do anything since the notification does not support windows,
but is it possible for the errors NOT to show when im on windows?
can i modify the script somehow? or wait for the patch?

Seems it is mainly a problem with Godot, you can go to settings GDNative and disable it while you use Windows, it won't throw a warning when you test it, but it will thow a warning once you open up your project. (not bad, only happens once)
Other not so perfect solution is that you can drop an empty file ".gdignore" which avoids Godot to import it so it doesn't show both warnings.
As far as I understood, it is only the iOS part that throws the error, so you would only need to enable it back once you export it to that platform.
It probably throws the error in other platforms that aren't iOS, like Android aswell, couldn't find another fix, creating an empty GDNative does the same, couldn't find an "ignore OS" code for it.
https://imgur.com/sWdquNX


LOVE quick-explorer, can we get some same functionalities in the search pane?

Hya
I LOVE quick-explorer so much, been waiting for commands like Quick Explorer: Go to next file in folderfor years!
I have a question. Any chance we can we get some same functionalities in the search pane, ie have the command Quick Explorer: Go to next file in folder work on search results?

thx so much for considering!
Z

I'm not sure what you mean here.  If you arrow down through those files you can nav to them already, and you can also use the QE commands to do what they do.  So I don't understand what you want that's different from that.  (If you want commands that navigate within the search, that wouldn't be folder navigation, for example.)

thx so much for you quick response @pjeby !
let me try and explain with some short clips :)
so if i do a search and try to use keyboard shorcuts you can see it either fails and focuses on the editing note are and if after many tries using the mouse i can actually focus the search pane it just goes up and down without showing/opening the notes contents:
https://share.cleanshot.com/qv7CH9ZY
as opposed to that with youer epic plugin i can use the binded keys for  Quick Explorer: Go to next to go up and done and get the notes to show/open, ie:
https://share.cleanshot.com/11F37T0K
i guess im looking for the same functionality in the search results, does that make sense
thanks again for all your amazing work, this is by far the most useful plugin i discovered this year!!
z

QE isn't going to get "next/previous in search" commands, so I'd suggest using hover previews, either the built in one or Hover Editor.  That approach won't work with the keyboard (at least not yet), but it's better than nothing, and it's what I currently do in order to move through search results or backlinks or anything else.

thx for the link and your consideration!!


Undo button

The buttons for "add face" and "remove face" are WAY too close, I've once again deleted a face that I was going to tag.
I cant find a way to see in the logs what is changed in the database in order to undo the action, I think an small log with like the last 5 changes to the database listed and an option to "restore" would be great.

I would like to add onto this feature request, or maybe before an undo button is just to add a tab to the "Labeled" and "Inferred" in the "Face Recognition" dashboard a "Deleted" to show all faces which have been deleted.


Perform the Design & Tech Assessment for the Self-Service Checkout adopter

hahnrob created the following on Jun 13:
As the IBM DDS architect, I need to perform a technical assessment of the Self-Service Commerce site so that I can understand what they need to properly implement IDL in a timely fashion.
Additional information

Self-Service Checkout Information Box Note
Self-Service Checkout Questionnaire

Acceptance criteria

[x] Information necessary for an accurate assessment of components by the adopter is collected
[x] Technical information is used to schedule the adoption
[x] Identify the next step tasks and the prioritization based on input from the adopter

Original issue: https://github.ibm.com/webstandards/digital-design/issues/1020

On Jul 16, hahnrob commented:
Initial questionnaire and overview meeting scheduled Thursday, July 25.


[BUG] Error in display of SVG when creating reports - Weasyprint will need to be upgraded

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
When creating a report with an SVG embedded in it via a link, there is a display error. It's our logo, and half of it is missing.
When I run Weasyprint 59.0 with the same source code on a different computer, the PDF looks like it should.
I believe this is caused by a bug in Weasyprint )https://github.com/Kozea/WeasyPrint/issues/1864), which has been fixed in version 59.0.
At the moment, we can not update the version of Weasyprint to 59.0 because of an incompatibility with django-weasyprint, as described here:
https://github.com/fdemmer/django-weasyprint/issues/71
Until this PR is merged in django-weasyprint, weasyprint can not be updated. For now, there is not InvenTree can do.
Workarounds:

Do no use <use ..:> tags in SVG
Convert SVG to another format, eg PNG

Steps to Reproduce

Make SVG with <use ...> tags
Use SVG in report template, eg with:

@top-left {
                width : 100%;
                height : 17mm;
                background-image: url("Logo.svg");
}

Create a report in InvenTree
Look for parts of SVG with <use ...>  in the PDF vain

Expected behaviour
Same as with 59.0
Deployment Method

[ ] Docker
[ ] Bare metal

Version Information
Current master has:
weasyprint==54.3
Bug remains with
weasyprint==58.0
Relevant log output
No response

Thanks for the report! We are pinned to 54.3 due to problems between django-weasyprint and newer weasyprint that popped up.
How much of an impact has this error? I personally do not use svgs that are affected but sounds like something pretty severe. Do you have any idea how widely used use is?
From a quick glance at django-weasyprint it seems like a monkey patch would be pretty simple/small.

In my limited understanding, use is probably applied more often than necessary by drawing programs.
The impact is big if the SVG is affected, but since there is a workaround, it can be handled. Since there seems to be a fix on the way for django-weasyprint, I would wait for that and then check if the other problems are solved.
The issues with merging PDF pages should be fixed in the newest Weasyprint. The "good reason to update" may be this issue with SVGs.
Relevant issues, for reference:
https://github.com/inventree/InvenTree/issues/3607
https://github.com/inventree/InvenTree/issues/3528

Still relevant, but the required PR is not merged yet (not much happening in django-weasyprint?

The relevant PR is now merged. Unclear when the updated version will be released.

not stale


Update attachment.ex to include .csv mime type

add the .csv mime type

This has already been included. Thanks!


Getting quorum loss issue as we rotate the orderer TLS certs in fabric 2.2

Hi Team,
I have 3 orderer nodes running on cloud, in k8s. From the cli I create a new certificate tls as well as MSP and store in the separate folder, with all Orderer still pointing to old certs. Then from cli, I fetch the config , encode, and ord0 tls cert, decode it and then push the update for syschannel as well as for application channel. Both the channel update operation return success for ord0. However, when I perform this for ord1 I am getting quorum loss issue.
2022-08-03 14:08:18.584 UTC [channelCmd] InitCmdFactory -> INFO 001 Endorser and orderer connections initialized
Error: got unexpected status: BAD_REQUEST -- error applying config update to existing channel 'syschannel': consensus metadata update for channel config update is invalid: 2 out of 3 nodes are alive, configuration will result in quorum loss

The real issue here is that you have one orderer down out of three, and when you try to rotate the certificate of that orderer, the orderer validation logic doesn't take into account that you're rotating the certificate of the orderer that is down, and wrongly decides it will result in quorum loss.
@shivdeep-singh-ibm do you want to take a look at this?

@yacovm  Sure.

https://discord.com/channels/905194001349627914/945037581500940298/1004380841973125150

The real issue here is that you have one orderer down out of three, and when you try to rotate the certificate of that orderer, the orderer validation logic doesn't take into account that you're rotating the certificate of the orderer that is down, and wrongly decides it will result in quorum loss.
@shivdeep-singh-ibm do you want to take a look at this?

I think this diagnosis is incorrect since the orderer validation code has logic in place which takes into account only alive orderers, to take a decision about quorum loss.
However, as seen from logs, where first orderer was not able to join the quorum as it din't find its own cert in the consenters, the probable causes may be:
a) orderer 1 dint receive the new config block which has rotated certs and it is pointing to rotated certs.
b) orderer 1 received the new config block which has rotated certs but it is pointing to original certs.

Please check the steps I am performing to arrive at this issue.
I have a network of 3 orderer(0,1,2) hosted via kubernetes on GCP. All the certificates are generated and kept in secret and configmap provided by kubernetes which is getting mapped to respective orderers. I create new msp and tls certs from cli. All orderer are still pointed to old certs only. I create a cli having core_peer_address as orderer0 and its msp. Post which I fetch config block decode it, add orderer0 new tls cert in consenter, encode it and published to channel. It gets successfully executed. I also perform same operation for application channel as well which also return success.
After this I replace Orderer0 msp and its TLS cert (in  secret and configmap provided by kubernetes). Now, I boot cli for orderer1 perform same operation. But once I publish channel update, it return quorum loss issue.
Please revert if you have any concern. Or can you share the steps to rotate the certs. It is affecting my production network. And network certificates would be expiring soon.

I have a network of 3 orderer(0,1,2) hosted via kubernetes on GCP. All the certificates are generated and kept in secret and configmap provided by kubernetes which is getting mapped to respective orderers. I create new msp and tls certs of all peers and orderers from cli. All orderer are still pointed to old certs only. I create a cli having core_peer_address as orderer0 and its msp. Post which I fetch config block decode it, add orderer0 new tls cert in consenter, encode it and published to channel. It gets successfully executed. I also perform same operation for application channel as well which also returns success.

Also after doing config update , wait for sometime, for orderer0, to get the config blocks. After this step, we assume that the config update has been published, but we need to be sure. For debugging, can you fetch the config the channel config, decode it and check whether your new tls cert is present, for orderer0? We want to make sure that the config update has been committed to orderer0.

Please find the observation. After updating syschannel and application channel for ord0(It works fine). I replaced ord0 msp and tls cert. After that when I fetch syschannel config block  from ord1 I am getting new tls cert of ord0. But when I am updating syschannel, getting same issue
2022-08-11 08:40:40.849 UTC 0001 INFO [channelCmd] InitCmdFactory -> Endorser and orderer connections initialized
Error: got unexpected status: BAD_REQUEST -- error applying config update to existing channel 'syschannel': consensus metadata update for channel config update is invalid: 2 out of 3 nodes are alive, configuration will result in quorum loss

getting the below error in ord1
2022-08-11 08:52:48.193 UTC 01d7 ERRO [comm.tls] ClientHandshake -> Client TLS handshake failed after 1.089742ms with error: public key of server certificate presented by orderer0-hlf-ord.svc.cluster.local:7050 doesn't match the expected public key remoteaddress=<IP_ADDRESS>:7050

getting the below error in ord1
2022-08-11 08:52:48.193 UTC 01d7 ERRO [comm.tls] ClientHandshake -> Client TLS handshake failed after 1.089742ms with error: public key of server certificate presented by orderer0-hlf-ord.svc.cluster.local:7050 doesn't match the expected public key remoteaddress=<IP_ADDRESS>:7050

Can you pull a config block from orderer0 and check the tls keys in it?

Okay, for this. from ord0 once I push the channel update to syschannel and application channel, I cannot fetch the config block, It gives error Service Unavailable.
From ord1, I can fetch the config block. In this config block, I do see new tls cert of ord0.

Okay, for this. from ord0 once I push the channel update to syschannel and application channel, I cannot fetch the config block, It gives error Service Unavailable.

So your orderer0 is has some problem or some step may be missing.  However you can go through this resource to rotate certs for fabric

I reckon unless ord1 and ord2 also submit the channel update ord0 will not be able to fetch config block. I have seen the video you shared, after performing all the steps. It gives me quorum issue on ord1.

Thanks for the help.


<EMAIL_ADDRESS>Could not find com.facebook.react:react-native

Description
@cortinico Essentially, I believe, I encountered the same issue with<EMAIL_ADDRESS>as #35909 and #35945. As I mentioned in #35909 (I think you overlooked my message in the thread there), I also encounter this problem when I create a standard scaffolding for a library using
$ npx create-react-native-library@latest react-native-awesome-module

With Turbo module with backward compat, Java and Objective-C; then installing NPM dependencies, copying in Gradle (thus, looking into Android build), and running ./gradlew lint ⇐ I guess, this should do as a repro?
Version
0.71.1
Output of npx react-native info
N/A
Steps to reproduce
N/A
Snack, code example, screenshot, or link to a repository
N/A

I also encounter this problem when I create a standard scaffolding for a library using

So let's try to keep things separate. The problem you faced with create-react-native-library should be reported here:
https://github.com/callstack/react-native-builder-bob/issues
Anyway I'm going to answer here as I already have an answer for you:

copying in Gradle (thus, looking into Android build), and running ./gradlew lint ⇐ I guess, this should do as a repro?

You're not supposed to copy gradlew there and you're also not supposed to run ./gradlew lint from that folder. The reason being that inside the ./android folder you just have a library, but not a full Gradle build (previously it was probably working for you by chance, but you're not supposed to invoke Gradle from there anyway. You're missing crucial files like settings.gradle and other files there).
You should be invoking gradlew from the ./example/android folder instead (or generally from the app project, not from the library project). Specifically create-react-native-library creates for you an example app in the ./example folder that you can use to test your library.
So to invoke lint in your case you will have to:
npx create-react-native-library@latest react-native-awesome-module && cd react-native-awesome-module && yarn && cd example/android && ./gradlew lint

(Note that this will run lint on both the app and the library module. If you want to lint only the library you'll have to ./gradlew react-native-awesome-module:lint

Thanks @cortinico ! It turns out I misread the new architecture migration guide when upgrading my library to<EMAIL_ADDRESS>and the new architecture, and when it says to configure gradle plugin for the app, I also added it to the library, and somehow it worked fine prior to<EMAIL_ADDRESS>

Probably duplicate of https://github.com/callstack/react-native-builder-bob/issues/361, fixed by https://github.com/callstack/react-native-builder-bob/pull/362
Try add + back
   // For < 0.71, this will be from the local maven repo
   // For > 0.71, this will be replaced by `com.facebook.react:react-android:$version` by react gradle plugin
   //noinspection GradleDynamicVersion
-  implementation "com.facebook.react:react-native"
+  implementation "com.facebook.react:react-native:+"


Unicode characters for identifier names

The current name generators are a bit lack-luster. I propose a new identifier name generator which basically generates random unicode sequences.
Most unicode is valid for variable names. See:
See: https://mothereff.in/js-variables#äöüß字漢字八
I think this would be a great addition, making code even more unreadable.

Right now i prefer existing identifier name generators


Integration for more sensor values

Hi,
I'm not using any humidity sensors or anything like that, just had a thought seeing this repo:
Maybe you could integrate some more data points for the script to look at like air quality in general (CO₂ levels, VOCs, PM₁₀ etc.) to trigger a reminder for venting.
If you could make those inputs optional, people can get the most out of their sensors, since there are many out there that got more than just temperature and humidity built in, but they don't get used too much.
As I said, I don't use any sensors (yet) and just starting out with HA, but that's an idea that instantly came to mind after seeing this.
Cheers

Interesting idea. Yes, one can probably derrive a correlation between humidity and co2 (or other measurements which drop during venting). The problem with this is, that i personally do not own any of those gas sensors so i can not collect data to model this correlation. Furthermore most of those sensor a probably not designed to work in very humid environments.


Dead code in core.py

https://github.com/pypa/pipenv/blob/2549656dc09e132d8ba2fa6327c939f5f9a951b7/pipenv/core.py#L1216
Is this line dead code?

Looks like it. Here's the changeset that removes a bunch of similar sys.exit() calls, but seems to have missed one. https://github.com/pypa/pipenv/commit/b57240dd45f69859fba3244dc9f642702a700e21