CSS format issue

Details

Read the Docs project URL:
http://fusionpbx-documentation.readthedocs.io/en/latest/
Build URL (if applicable):
Read the Docs username (if applicable):
https://github.com/Len-PGH/fusionpbx-docs.git
https://github.com/fusionpbx/fusionpbx-docs
https://github.com/LEN-PGH/fusionpbx-docs
Using theme https://github.com/rtfd/sphinx_rtd_theme

Expected Result
A description of what you wanted to happen
Want the build to build like it normally does
Actual Result
A description of what actually happened
CSS looks to be broken.
http://docs.fusionpbx.com
http://docs.techlacom.com
Happens on both sites.  Last build with no issue was Wednesday the 13th.

I think this is because of an update in theme, I'll try to see if I can reproduce this (also you can try to specify the latest version of the theme).

Where do I find the version of the theme and sphinx?

Where do I find the version of the theme and sphinx?

They will be in the build output.

Looks like it is using the latest version of the theme (0.4.0) with Sphinx 1.7.4.

Seems like you are using a custom theme
https://github.com/Len-PGH/fusionpbx-docs/blob/d44f3545d85eedf9a4df8e43ef37cdeeaff36ecc/source/conf.py#L39
https://github.com/Len-PGH/fusionpbx-docs/tree/master/source/_themes/sphinx_rtd_theme

Thanks for the tip @stsewd. So what happened is that:

The latest sphinx_rtd_theme is installed but not used because you have duplicated an old version of the theme into your project directory.
Older versions of the theme relied on Read the Docs always using the same CSS (bundled with readthedocs.org) rather than the CSS that is bundled with the theme. This is necessary to allow the theme to be updated independently. Previously, whether you were using the 0.2.5 theme or the 0.3.1 theme, you always go the same CSS.
So the version detection that checks whether to include the CSS finds the latest theme. However, your copied theme overrides the change to use the CSS bundled with the theme.

I see a couple solutions:

Remove most or all of the duplicated theme from your project and rebuild. Only override the bits you need. This is probably the best solution.
Pin to a previous version of the theme.

Theme has been the same since starting to use read the docs. I'm happy to do whatever needs done but trying to understand why this just happened this week with constant builds  and after 2 years?

@Len-PGH that is explained above

Older versions of the theme relied on Read the Docs always using the same CSS (bundled with readthedocs.org) rather than the CSS that is bundled with the theme.

RTD update that in the latest deploy.

I see a couple solutions:

Another solution is to update your duplicated theme although I suspect you can just remove it.

trying to understand why this just happened this week

We made a change this week to increase the default version of the RTD theme used with Read the Docs. If there wasn't an overridden version of the theme in your project, this would have been transparent to you. The newest version of the theme bundles CSS a little differently and in the build process we detect the theme version to decide whether to Read the Docs should include some global CSS (the old, suboptimal way) or rely on the theme's CSS. This version detection failed on your project because it has the latest theme but you have overridden the theme with an old version that isn't installed.
Compare the layout of the theme duplicated in your project vs what is in the theme now. There are a number of improvements we made in the theme but those changes relied on upgrading the CSS independently of Read the Docs.

Thank you for the help. I just moved the theme folder and ran a build and css is back.  I really appreciate everyone's help!


feat(build-env): add libpam to tdp-builder

Apache Ranger pom.xml has a profile enabled if the lib pam headers are found on the host/. This profile enables the module unixauthpam.
This module simply compile a .c file, and the resulting binary is included in the distributed .tar.gz
This binary can then be used as an auth provider for the unixauthservice of the ranger-usersync. This binary uses Linux PAM as an authentication provider.
While this module is really not mandatory, I think it is worth bundling. Also it is built this way by other distros.

Some more details regarding the module unixauthpam.
By default, ranger-usersync exposes on port 5151 an Authentication Service. One can contact the port using TCP ( secured by TLS by default in TDP) and send a login attempt like this "LOGIN: ". The service then challenges the authentication, and send back "OK" if working, "ERROR" if not.
The default authentication backend is unixauthnative, which simply checks the credentials against the file /etc/shadow found on the host.
It is possible to modify the authentication backend using the configuration ranger.usersync.passwordvalidator.path. The service simply executes the file, and send the credentials to the process through STDIN.
unixauthpam is simply another authprovider that uses PAM modules, namely ranger-remote (which can be linked to module login). Using PAM instead of /etc/shadow enables the use of SSSD.
I wont go much further of why this can be useful, the purpose of this PR is simply to enable the build and packaging of the code given by Apache Ranger into TDP bundles.


Specific arguments for test discovery and unexpected exit code 2

We have a rather large python repository with multiple submodules. Testing the complete suite is handled by Bazel, but we want users to be able to run individual tests locally in docker as well. So far, we've only been able to do this using the command line or PyCharm.
After fiddling around with VSCode for a day, I got a working setup which required two minor adjustments in vscode-python.
Test discover phase only works with --noconftest, but this should not be passed during the actual tests.
When running pytest --collect-only we get an error saying Defining 'pytest_plugins' in a non-top-level conftest is no longer supported:. This error only happens during the discover phase or when we run all tests at once. Since in practise we always run tests per module, we want to pass the argument --noconftest. However, there is no separate config for pytest and the discover phase.
In this PR I've hardcoded the intended argument. However, I would like to understand if it's possible to make this a custom configuration exclusively for the discover phase that can be set in settings.json. Happy to make the adjustments.
When pytest --collect-only fails with some errors, pytest exists with exit code is 2, not 1.
Even though the pytest docs say that when Tests were collected and run but some of the tests failed the error code should be 1, we experience an exit code 2 (see below). The tests are collected just fine, and the errors can be safely ignored (at least for discovery).
logs
!!!!!!!!!!!!!!!!!! Interrupted: 233 errors during collection !!!!!!!!!!!!!!!!!!!
================== 1559 tests collected, 233 errors in 29.28s ==================

Traceback (most recent call last):
  File "/root/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/testing_tools/run_adapter.py", line 22, in <module>
    main(tool, cmd, subargs, toolargs)
  File "/root/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/testing_tools/adapter/__main__.py", line 100, in main
    parents, result = run(toolargs, **subargs)
  File "/root/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/testing_tools/adapter/pytest/_discovery.py", line 47, in discover
    raise Exception("pytest discovery failed (exit code {})".format(ec))
Exception: pytest discovery failed (exit code 2)

I'd be happy to take some input on how to best proceed. The implementation is purely for illustrative reasons and I'm happy to adjust.

Hi @Braamling, thanks for contributing! A few items in regards to this PR. So first we are in the process of rewriting how testing works in VS Code to increase functionality. We have released this rewrite tp all of the insiders and some of stable but you can try it yourself by adding "python.experiments.optInto": ["pythonTestAdapter"] to your user settings. This rewrite is not fully active but we will move towards this being the only way we run / discover tests. Due to this switch over we now will be using a pytest plugin to handle discovery and execution. You can see the plugin here. Both discovery and execution logic is in the file (due to design constraints we could not break it apart). This means the file you edited will soon be deprecated as well as potentially the issues you encountered.
Secondly, we are working on a proposal for how arguments should look with the new design to make it more functional. We are planning on breaking out discovery and execution args into separate settings so this should be the fix to your problem with --noconftest. Here is the issue with that discussion: https://github.com/microsoft/vscode-python/issues/21845, please add your thoughts to the discussion so we can consider your use case!
Finally thank you for pointing out this other error code, could you check if you are still seeing it when you enable the rewrite? If so I may want to reach out to the pytest team for any misalignment in docs since our intent is keep as close to the docs from pytest as we can.
Thanks and let me know your thoughts on the above.

@eleanorjboyd, thank you for your fast and complete answer! I'll definitely add my thoughts to https://github.com/microsoft/vscode-python/issues/21845, very happy to see this on the planning already.
However, I'm still struggling to get the tests to show up in discovery, even with the new testing adapter.
Running pytest --collect-only --noconftest results in 2116 tests collected, 119 errors in 52.08s, while python3 -m pytest -p vscode_pytest --collect-only . --noconftest results in Interrupted: 197 errors during collection. and Subprocess exited unsuccessfully with exit code 2 and signal null. without any tests showing up in the testing UI. Trying to run ImportError: Error importing plugin "vscode_pytest": No module named 'vscode_pytest' manually in the VSCode terminal results in ImportError: Error importing plugin "vscode_pytest": No module named 'vscode_pytest', and I'm not sure how to get this to work in the terminal.
Given that pytest is able to collect plenty of tests, I would still hope we can display them in discovery :) This could still be due to the error code, but I haven't figured out how to bypass this in the new adapter. Any ideas here how I can easily test this?

By the way, I understand a PR is not the place for these types of discussions. Would you prefer me to open an issue to continue the error code conversation?

Hi! Yes an issue or discussion would be great if you want to create one that would be extremely helpful! I will close this PR since I do not think we need any code changes. In the meantime, I will investigate and get back to you with any questions, thanks!

when you create your issue if you could switch to verbose logging (done through the command palette) and then add in the logs from both the "python" output and "python test output" that would be great. Thanks


Enable context isolation

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
This PR turns on Context Isolation for increased security between the loaded MM Webapp and the users underlying OS.
Issue link
https://mattermost.atlassian.net/browse/MM-13609
Test Cases
Mac: A local build (npm run build) on a Mac is sufficient to test this PR.
Windows: A package needs to be generated locally (npm run package:windows) and then installed from the resulting *setup*.exe in the ./releases/ folder for notifications to work properly.
Once running:

make sure you are logged into the same mm server with two different users, one within the running Desktop app.
with the user not running in the Desktop app, at-mention the user running in the Desktop app in a team/channel other than the one currently open in the Desktop app.

Expected: an OS-level notification popup should show and when clicked, should focus the Desktop app if not already focused and take you to the team/channel the at-mention was posted in.
Additional Notes
To verify context isolation is on:

from the View menu, choose Developer Tools for Current Server to open up the developer tools panel.
in the Console tab of the resulting panel, click on the dropdown near the top left of the panel that says top. You should see a second item called Electron Isolated Context, indicating that context isolation is on (see pic below).

@deanwhillier Is it possible to write a unit test for this?

Unfortunately I don't have much experience with testing Electron features like this. Looking at what was done for nodeIntegration in security_test.js as an example on how to accomplish, it looks like the testing has been disabled due to a chromedriver issue?
Thoughts?

@deanwhillier Well... it's not ideal that the security tests are disabled. Can you take a look?

Created a ticket to investigate: https://mattermost.atlassian.net/projects/MM/issues/MM-16883

@crspeller Are a lack of unit tests blocking on getting this in? We could test it by hand for now if there's something preventing us from implementing unit tests

@deanwhillier Actually you have unit test failures anyway.

npm run test works fine locally, errors seem to suggest that Chome is crashing when run by CI. 🤷♂

Maybe that's the chromedriver failing? 😛

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

nudge @deanwhillier although I know you've been busy with accessibility stuff :)

@cpanato, @DSchalla, this is another that Chrome is crashing on when running tests.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Added pipeline fixes from @Willyfrog, tests are now running again and passing!


Unable to access gRPC service though Yarp Gateway in .Net Core

I have created a gRPC service in Asp.net core Web API using nuget packages - Grpc.AspNetCore 2.57.0, Grpc.Tools 2.64.0, Google.Protobuf 3.27.2.
I am able to access it successfully through my .Net Core Web application. However, when I try to access it through Yarp Gateway, I am getting 404 error. Can someone please help me to get rid of this issue?
Exception Details : Grpc.Core.RpcException: Status(StatusCode="Unimplemented", Detail="Bad gRPC response. HTTP status code: 404")
Below is the code snippet:
gRPC service proto file =>
syntax = "proto3";
option csharp_namespace = "GrpcService.CatalogAPI";
package CatalogGrpcService;

import "google/protobuf/empty.proto";

service CatalogGrpcAPI{
    rpc GetAllCatalogs (google.protobuf.Empty) returns (CatalogListResponse);
}

// A Catalog resource.
message CatalogModel {
  string catalogId = 1;
  string name = 2;
  string description = 3;
  float price = 4;
}

// Response message to GetAllCatalogs method.
message CatalogListResponse {
    repeated CatalogModel catalogs = 1;
}

Aspnet Core Web Application client =>
var grpcChannel = GrpcChannel.ForAddress("https://localhost:7005");  // Works well with this direct URL of the gRPC service
//var grpcChannel = GrpcChannel.ForAddress("https://localhost:7700");  // Gives error with this https URL of Yarp Api Gateway
//var grpcChannel = GrpcChannel.ForAddress("http://localhost:5000");  //  Gives error with this http URL of Yarp Api Gateway

var client = new CatalogGrpcAPI.CatalogGrpcAPIClient(grpcChannel);
var response = await client.GetAllCatalogsAsync(requestModel);

Yarp Gateway config =>
"Kestrel": {
  "Endpoints": {
    "http": {
      "Url": "http://localhost:5000",
      "Protocols": "Http2"
    },
    "https": {
      "Url": "https://localhost:7700"
    }
  }
},
"Yarp": {
  "Routes": {
    "catalog-route": {
      "ClusterId": "catalogCluster",
      "Match": {
        "Path": "/CatalogGrpcService.CatalogGrpcAPI/{**catch-all}"
      },
      "Transforms": [ { "PathPattern": "{**catch-all}" } ]
    }
  },
  "Clusters": {
    "catalogCluster": {
      "HttpRequest": {
        "Version": "2",
        "VersionPolicy": "RequestVersionExact"
      },
      "Destinations": {
        "catalogCluster/destination1": {
          "Address": "https://localhost:7005/"
        }
      }
    }
  }
}

It's probably this part:
"Match": {
  "Path": "/CatalogGrpcService.CatalogGrpcAPI/{**catch-all}"
},
"Transforms": [ { "PathPattern": "{**catch-all}" } ]

You're removing part of the url before forwarding. If it works when you access the server directly then you don't need to remove this. Get rid of the transform.

Triage: Seems to be answered, closing. Feel free to reopen if there is more informaiton available.


Dataset exists (409) / dataset still in use (DELETE --> 400)

python2:

Still in use: precise-truck-742:system_tests_160041019_1473898532
Exists: precise-truck-742:system_tests_160041019_1473898532

then python3:

Still in use: precise-truck-742:system_tests_160041019_1473898730
Exists: precise-truck-742:system_tests_160041019_1473898730

From https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/160041019
Two proposed fixes:

Make sure the 400 is caught and retried for "still in use"
Use dataset IDs unique to each test (if possible)

Big :+1:  for avoiding shared dataset IDs.

@tseaver It's unclear if #2323 (double Michael Jordan) actually will fix the 400 "still in use". But it's not worth leaving this open forever.

Because testcases no longer share dataset names, there isn't going to be a case where the eventual-consistency window for one testcase deleting its dataset overlaps the creation during setup of another testcase.

But the clean-up of each dataset will be a delete and the "still in use" on DELETE is where the issue came from.

@tseaver Let's leave it closed until the "still in use" shows up again in the wild?

Hmm, I just kicked in #2333 to address it.

Oh gotcha. SGTM.


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3104

Questions ?
Please refer to the Backport tool documentation


SSL Client Certificate only for specific routes or controllers

Is there support currently or future planned support to allow for more granular SSL Client Certificates for example only for specific routes/controllers in a asp.net core kestrel hosted server? The issue is in wanting to host a combination of an asp.net mvc app and web api within the same process/service and applying client certificate requirements to the web api and not the mvc portion; as an example. This is possible in IIS by creating two virtual directories and then setting SSL settings to Accept on one folder and Ignore on the other. But IIS is not an option in this case; just mentioning as an example of where this does work.
Alternatively would it be possible to instantiate two WebHostBuilder() within a single process so that one binding can do client certificates and the other not? Would two WebHostBuilder in a single process/service be threadsafe/concurrency issues?

SSL negotiation is per connection, not per request, including client certs. The underlying SslStream only allows negotiating the client cert during the initial handshake. You can set ClientCertificateMode to AllowCertificate so it's optional, but you can't re-negotiate.
You could create two WebHostBuilders, but Kestrel can't share ports between instances to enable vdirs so that wouldn't help you.
Note we don't recommend exposing Kestrel directly to the internet, so you could use vdirs in IIS before proxying to Kestrel.  You could also do most of this with WebListener, it can do port sharing and vdirs, as well as delayed client cert negotiation.

This is for a server application to be hosted within a customers environment. I was hoping to allow it to support both Windows and Linux installations hence asp.net core on kestrel vs. WebListener etc...
I think creating two WebHostBuilders should work for my purposes as I am ok to have the web browser asp.net mvc site on standard (80/443) and then a separate port for essentially software agents  communicating to the same server over web api on a different port. In general it is safe to have two WebHostBuilders running within the same process?
BTW; I have seen it mentioned a few times about not allowing Kestrel directly to the internet. I am not planning to do that but was wondering why that is? Is it more of a performance/load concern or is there some security concern that should also be taken into consideration that might equally apply to internal networks? thank you so much; love everything you all are doing with .net core related endeavors!

Multiple WebHostBuilders in an app should be fine.
Kestrel is immature and doesn't have defenses against common DOS attacks yet.

Makes sense - I will look into packaging to leverage Apache/Kestrel for Linux and WebListener for Windows depending which OS customers install on. thank you!

@Tratcher just wondering if any of your guidance here has maybe changed in some way with the release of asp.net core 2.0? I went down the path of hosting multiple WebHostBuilders originally but it seems that becomes problematic in wanting to host the app in azure etc there is not an easy way to configure an azure web app to work properly with two webhostbuilders app running as console app. Wondering what asp.net core 2.0 might change here and alternatively if I just split my solution into two different projects what method do you recommend for having one project web app communicate the other project web app?

I presume in this scenario you are using renegotiation  (or would like to) to challenge the user if they go to specific controller and haven't authenticated with the correct type of certificate for that controller?
Or does the user only initiate with one and you just want to check it?

I essentially have one WebHost that is a frontend of sorts, set of web apis that are called by a angular interface. I have a second WebHost that is a backend that is SignalR with a lot of agents connected to it. The frontend WebHost needs to be able to call a SignalR call against the backend to signal the agents. Having this in the same process works because I can access the signalr interface from either WebHost. If I split my 2 WebHost into separate projects/processes then I would need the frontend WebHost to somehow call the second backend WebHost. I would assume the two options for this is to add a web api to the backend WebHost and create a secure way to call that from the frontend WebHost. Or alternatively the frontend could write to some shared SQL table that the backend polls on some interval. I had been trying to keep two WebHost within the same process to avoid this type of IPC scenario. So far two WebHost has been working fine in a single process. Just wondering if there is anything in asp.net core 2.0 that might make sense to change the scenario of how I am doing this. thank you much

Nothing has changed here for 2.0.

One option by the way is loopback. If one service is coming in via the loopback address you can ignore the authentication.

@Drawaes that's not a good idea... you should never make assumptions about the caller.

It's a pretty common practice. Say you have ngnix handing off locally. Or internally you have authenticated users coming through a termiñating proxy. IP address limiting/identification  is often used.

Isn't good for shared hosting though

Shared hosting? What's that? ;)
Let me be more clear you should never rely on an IP address if it's a public IP or you don't control the entire network. But say you have a local docker isolated virtual network and you service bridges to an outside network, then yeah I have no issue using that for a certain level of trust.
Systems like this have been through several external independent Pen testing/auditing cycles at financial institutions as never been an issue.


Add support for the smokemotes ffz addon

Describe the solution you'd like
Enable smokemotes integration to get gif emotes - while not having to pay for bttv: https://www.smokey.gg/tutorials/using-smokemotes-with-frankerfacez/
Add-on code: https://github.com/FrankerFaceZ/Add-Ons/tree/master/src/smokemotes
Describe alternatives you've considered
Paying for bttv

Does this one still "need triage"? 🤔

he removes that feature

he is removing that feature https://i.imgur.com/9rUzvyc.jpg


regarding - On-line (via webcam)

Hai @ekosman
My project is related to this topic ( Anomaly detection ). After successful completion of training, how can I test the model with live anomaly prediction?
thanks

Hi @Chuttyboy ,
Have you tried using video_demo.py?

Hai @ekosman
Yes, I tried AD_live_prediction.py. But it takes more time to extract frames.
Thanks

@ekosman @PHDPeter
[ WARN:0] global /io/opencv/modules/videoio/src/cap_v4l.cpp (887) open VIDEOIO(V4L2:/dev/video0): can't open camera by index
I am facing this error while running the below command
python AD_live_prediction.py --feature_extractor c3d.pickle --feature_method c3d
--ad_model exps/c3d/models/epoch_80000.pt --clip_length 32
How to resolve that ???
Thanks

@ekosman @PHDPeter
[ WARN:0] global /io/opencv/modules/videoio/src/cap_v4l.cpp (887) open VIDEOIO(V4L2:/dev/video0): can't open camera by index
I am facing this error while running the below command
python AD_live_prediction.py --feature_extractor c3d.pickle --feature_method c3d --ad_model exps/c3d/models/epoch_80000.pt --clip_length 32
How to resolve that ???
Thanks

Hi @suki1504
I tried to run the same command and it worked for me. Can you please try again with the current code version?


Fixing the CI release of Release Branch

We would like to fix the CI release of the Release Branch, e.g https://github.com/harvester/harvester/tree/v1.0
So the full-regression test can be performed on the tagged branch with specific versions.

Excepted ISO results:
harvester/harvester CI

master branch
ISO: https://releases.rancher.com/harvester/master/harvester-master-amd64.iso

v1.0 branch
ISO: https://releases.rancher.com/harvester/v1.0/harvester-v1.0-amd64.iso

v1.0 branch + v1.0.1 tag
ISO: https://releases.rancher.com/harvester/v1.0/harvester-v1.0.1-rc1-amd64.iso

harvester/harvester-installer CI

master branch
ISO: https://releases.rancher.com/harvester/master/harvester-master-amd64.iso

master branch + v1.0.1-rc3
ISO: only local file dist/artifacts/harvester-v1.0.1-rc3-amd64.iso

Verified this issue has been fixed.
mater-head

v1.0.1-rc1

v1.0-head

Test Information

Environment: qemu/KVM single node

Verify Steps:
Follow Test Plan in https://github.com/harvester/harvester/issues/1817#issuecomment-1069191043


Illumination artifacts at object edges

 Issue by benjamin-heasly
Friday Oct 24, 2014 at 14:39 GMT
Originally opened as https://github.com/DavidBrainard/RenderToolboxDevelop/issues/7

We may need to clean up some artifacts in our illumination images.  For example, see light blue artifacts at the edges of some objects:

These artifacts are probably caused by errors in our object pixel mask code (MakeRecipeObjectMasks).  This code is responsible for assigning which pixels are "looking at" which objects.  It also assigns which pixels are ambiguous and not looking at any particular object.  It seems to be making some wrong assignments, which messes up our illumination computations.
Note that the assignment errors have a "relative" quality.  For example, the pixel assignments work out fine for most of the door jamb.  But where the door jamb appears next to the ground or next to the roof of the building in the background, there are artifacts.
We might want to revise the object pixel mask code to have an "absolute" quality, so that the assignment for a given pixel is less sensitive to the pixel's spatial context.

 Comment by benjamin-heasly
Friday Oct 24, 2014 at 20:21 GMT

A subtle issue occurred to me:
We have to interpolate to smooth out the gap pixels between objects.  We currently do this one spectral plane at a time.  This might be the wrong way to do it.
For example, at a given pixel, say the median in spectral band 1 corresponds to the contribution of object A.  It might be the case that the median in spectral band 5 corresponds to a different objects, B and C.  This might  be the case if B and C happen to give the same contribution in band 5, and their union happens to outweigh object A in that band.  So the total spectrum that we assign to pixel in question might be a mix of objects A, B, and C, and might look different from A, B, and C.
I am not sure if this is really happening.  I'm also not sure if it's a cause for concern.  But I think it's not what we intend when we say "take the median illumination of nearby pixels".  I think the median should be taken with respect to complete spectra, and not separately for each spectral band.

 Comment by benjamin-heasly
Friday May 01, 2015 at 17:05 GMT

We've had success using the "shapeIndex" factoid to identify objects as well as object borders.
Here are some thoughts for ongoing discussion:

We now have two methods for identifying objects and borders: our original "Materials" methods, and the new "Factoid" method.
We use this information for two distinct purposes.

One purpose is to figure out which pixels belong to which object.  This is pretty well solved.
The other is to perform interpolation/smoothing near object borders, in order to remove artifacts in our illumination analysis.  This is more subtle because of dithering performed by the renderer during image reconstruction.  This might be solved, or we might need to fuss over it more.  It depends on how our next batch of renderings turns out.

It would be nice to always have "crisp" object borders, say 1 pixel wide all the way around.  But this might not actually be necessary.
For some analyses, we might want to know which object "owns" the border.  This might affect asymmetric things like false positives vs. false negatives during identification of a given object.  We can probably address this kind of question using either method, Materials or Factoids.

 Comment by benjamin-heasly
Friday Dec 19, 2014 at 15:41 GMT

We can extract various scene "factoids" using the Mitsuba "field" integrator:
https://github.com/DavidBrainard/RenderToolboxDevelop/blob/master/VirtualScenesToolbox/Utilities/RenderMitsubaFactoids.m
One of the factoids is called "shapeIndex".  This appears to give us an identifier for each object in the scene.  So it's an independent way to identify objects.  This might give us insight into our own object detection code.


Instadapp Safe App Request

Name / Description
Instadapp - DeFi Middleware and Asset Manager
Type

New addition

Compatible Networks
- Mainnet
- Polygon

Code for review
- https://github.com/InstaDApp

IPFS hash/App URL
- https://defi.instadapp.io/

Desired date
N/A

I'm closing this because the app's flow doesn't satisfy the requirements (doesn't connect automatically, you have to click Metamask to connect to a safe). The problems were reported to the Instadapp team.


Adding a CITATION file.

Hello and thank you for your package!
I am a PhD student and I am currently using your library for one of my research projects. I was wondering if you would be interested in creating a citation file so the library can be cited. Here is a very good blog post on why/who citing your software is good practice: https://www.software.ac.uk/blog/2016-10-06-encouraging-citation-software-introducing-citation-files
Let me know if you would like me to open a pull request :+1:

Sure, please do open a PR


[MariaDB 10.5] Fail to get row value by column name

Problem
I'm using sqlx version 0.5.9 with MariaDB 10.5 to test the support of RETURNING syntax on SQLx. Found something unusual... I can get value with column index (row.get::<i32, _>(0)), but not column name (row.get::<i32, _>("id")).
Is this the intended behaviour or it's a bug? Thanks!!
Source Code
use sqlx::{mysql::*, *};

#[async_std::main]
async fn main() -> Result<()> {
    let pool = MySqlPoolOptions::new()
        .max_connections(5)
        .connect("mysql://root:root@localhost/returning_tests")
        .await?;

    let mut query = sqlx::query("INSERT INTO `bakery` (`name`, `profit_margin`) VALUES (?, ?) RETURNING `id`, `name`, `profit_margin`");
    query = query.bind("Bakery Shop");
    query = query.bind(0.5);
    let row = query.fetch_one(&pool).await?;
    println!("{:#?}", row);

    // Working...
    row.get::<i32, _>(0);
    row.get::<String, _>(1);
    row.get::<f64, _>(2);

    // Not Working... panicked at 'called `Result::unwrap()` on an `Err` value: ColumnNotFound("id")
    row.get::<i32, _>("id");
    row.get::<String, _>("name");
    row.get::<f64, _>("profit_margin");

    Ok(())
}

Full Log
MySqlRow {
    row: Row {
        storage: b"\0\x0c\0\0\0\x0bBakery Shop\0\0\0\0\0\0\xe0?",
        values: [
            Some(
                1..5,
            ),
            Some(
                6..17,
            ),
            Some(
                17..25,
            ),
        ],
    },
    format: Binary,
    columns: [
        MySqlColumn {
            ordinal: 0,
            name: id,
            type_info: MySqlTypeInfo {
                type: Long,
                flags: NOT_NULL | PRIMARY_KEY | AUTO_INCREMENT,
                char_set: 63,
                max_size: Some(
                    11,
                ),
            },
            flags: Some(
                NOT_NULL | PRIMARY_KEY | AUTO_INCREMENT,
            ),
        },
        MySqlColumn {
            ordinal: 1,
            name: name,
            type_info: MySqlTypeInfo {
                type: VarString,
                flags: NOT_NULL | NO_DEFAULT_VALUE,
                char_set: 224,
                max_size: Some(
                    1020,
                ),
            },
            flags: Some(
                NOT_NULL | NO_DEFAULT_VALUE,
            ),
        },
        MySqlColumn {
            ordinal: 2,
            name: profit_margin,
            type_info: MySqlTypeInfo {
                type: Double,
                flags: NOT_NULL | NO_DEFAULT_VALUE,
                char_set: 63,
                max_size: Some(
                    22,
                ),
            },
            flags: Some(
                NOT_NULL | NO_DEFAULT_VALUE,
            ),
        },
    ],
    column_names: {},
}
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ColumnNotFound("id")', /Users/billy/.cargo/registry/src/github.com-1ecc6299db9ec823/sqlx-core-0.5.9/src/row.rs:73:37
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

That's interesting cause the names are clearly included in the column info there, they just haven't been extracted to the map for some reason.

I've figured out so far that the problem is that the COM_STMT_PREPARE_OK response tells us that there are 0 columns in the result, which is decidedly wrong. I'm gonna go ahead and open a bug on the MariaDB issue tracker for that.
To save on allocations, we use this response to populate the column_names map, cache it in the prepared statement object, and then share it with the MySqlRow using an Arc. So that's why the map is empty, which breaks column lookup by name.
As a workaround, we can check if this map is empty when decoding the Row message and if so, populate it directly. This does mean that the map can't be amortized, although maybe we could memoize it at least for the current execution.

This does mean that until it is fixed in MariaDB, the query!() family of macros will think an INSERT ... RETURNING query returns 0 columns.

Opened a ticket against MariaDB: https://jira.mariadb.org/browse/MDEV-27013

Thanks!! @abonander

I've figured out so far that the problem is that the COM_STMT_PREPARE_OK response tells us that there are 0 columns in the result, which is decidedly wrong. I'm gonna go ahead and open a bug on the MariaDB issue tracker for that.

The issue seems stale. Is there any way the community of SQLX can pressure MariaDB folks? Where can I voice that the issue is important for me?

As a workaround, we can check if this map is empty when decoding the Row message and if so, populate it directly. This does mean that the map can't be amortized, although maybe we could memoize it at least for the current execution.

What about this possible workaround @abonander ?

What about this possible workaround @abonander ?

That won't fix the query!() macros since they don't actually execute the query at compile time, but it would fix the errors at runtime.


Function to compute velocity from positions

Let's discuss the API for this function.
def velocity_from_positions(
    x: xr.DataArray,
    y: xr.DataArray,
    time: xr.DataArray,
    coords: str='spherical', # where x is lon and y is lat, otherwise can be 'cartesian' where x is eating and y is northing
    order: int=1, # can also be 2 for centered difference; we can discuss if higher orders may be desired
) -> Tuple(xr.DataArray, xr.DataArray)

I wonder if we shouldn't use coords to avoid confusion with the Xarray special coords. Perhaps coord_system or coordinate_system?
Until there is native support for ragged arrays in Xarray, the user should be careful to not run this on multiple consecutive trajectories. It will be easy for this function to detect that (e.g. jump in time and space) and issue a warning.

sure, coord_system could work. in addition or instead of order we could have schemes? (forward, backward, centered)

I assume what you mean by forward and backward is the same as upstream and downstream in space in the context of advection. I considered this initially but realized it's redundant. Here's why:
Upstream vs downstream is the same calculation for dx: x[1:] - x[:-1]. The only difference is where dx is evaluated--at x[:-1] (upstream/backward) or x[1:] (downstream/forward). But if the function returns only the velocity and not the position of that velocity, it's up to the user to decide.
And if the user doesn't need to do anything about advection, then it doesn't matter--the position of the resultant velocity is at centerpoints (x[1:] + x[:-1]) / 2.
Is that what you meant or am I writing about something different?

For now, let's follow latlon2uv.m in jLab. Have a look.

latlon2uv

Difference algorithm

By default, LATLON2UV uses a first central difference algorithm which
is defined to be the average of a forward and a backward difference.

CV=LATLON2UV(...,'forward') instead uses a first forward difference.
Velocity is computed based on the great circle distance and bearing
from each point to the next point.

CV=LATLON2UV(...,'backward') instead uses a first backward difference.
Velocity is computed based on the great circle distance and bearing
from each point to the previous point.  This is equivalent to the first
forward difference computed backward in time.

The first forward and first backward difference can be thought of as
giving the Cartesian velocities of a departing and arriving particle,
respectively.

Note that acceleration is not output when the 'forward' or 'backward'
algorithms are specified, as acceleration is computed as a part of the
first central difference operation.

Let's discuss this more.
In the case of a 1st order difference, do you think the size of dx should be the same as the size of x, or less by 1?

dx definitively should be the same size as x. See how the end points are handled in latlon2uv.m on line 261-263.

OK, in my mind I'd think the opposite, but at the same time, I don't know of an established convention either way. We can certainly make the result the same size as the inputs, and patch the endpoints, but we should clearly document what we do there, since it may not be expected by everybody.

I just want to echo Shane's preference that length does not change when we compute velocity.  Although it might be mathematically weird, it makes a big headache if length changes when we take the finite difference.
So the way I handle this in latlon2uv is to take the first forward difference, and first backward difference (both of which are well defined on the sphere) and average them.  Then at the first point and last point I use only the first forward and first backward difference, respectively, so those points are treated specially.
Probably it could be done a little more nicely by taking the time to write down the first central difference on the sphere. Unlike the Cartesian case, I don't think the average of the forward and backward difference will end up being identical to the central difference.
You've no doubt thought of this, but it's worth mentioning that we don't want to work with irregularly spaced data in time, so it should be built in that time must progress uniformly.

I just want to echo Shane's preference that length does not change when we compute velocity. Although it might be mathematically weird, it makes a big headache if length changes when we take the finite difference.

I'm not surprised at all. :) I don't think it's even mathematically weird, but more from what I'm used to with numerical Python APIs. Specifically, np.diff() by default returns one point less, so it turns out that that's what I'm used to expect. But it can certainly be nice for many users for end-points to be taken care of by the library. I already took that approach in #64.

You've no doubt thought of this, but it's worth mentioning that we don't want to work with irregularly spaced data in time, so it should be built in that time must progress uniformly.

As I understand it, it's only problematic for centered differencing. For forward and backward, the user gets exactly what they ask for, even for irregular dt. But the interpretation must be careful then. If anything, I'd raise a user warning in case of irregular dt, but not error out. Would you error out?

I think the function returns the velocity in meter per unit of time input, correct? If yes we need to update the help. Can the function be made aware of the unit of the time and automatically return the velocity in m/s? with an option for that behavior to be disabled?

I opened #72 to discuss this further. Let's open new issues whenever new problems and questions arise, so they're easier to find. I'll close this one.


"Teișani", "Bughea de Sus", "Olteni", "Ștubeiu" and "Valea Stâlpului" villages are listed in the drop-down menu

The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Type "Teișani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Teișani, PRAHOVA (Teișani)".

Step 2
Type "Bughea de Sus" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Bughea de Sus, PRAHOVA (Teișani)".

Step 3
Type "Olteni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Olteni, PRAHOVA (Teișani)".

Step 4
Type "Ștubeiu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Ștubeiu, PRAHOVA (Teișani)".

Step 5
Type "Valea Stâlpului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Stâlpului, PRAHOVA (Teișani)".


Search not working when initial load fail retry.

Load application when network is not available
Connect network and let application load data
Try searching for a task

Current behavior: Search result wont show up.
Expected behavior: Search result should show up.

Fixed!


updated npm install declaration in documentation

"Help us understand your motivation by explaining why you decided to make this change." -- I find it is important to ensure that all commands in a repository readme run without any modifications necessary. This facilitates allowing beginners to use your library / framework.

@singularitysoft how is that different to not having quotes? 🤔


Fix how we query service uninstall status

@cruhland Yes. I'll cherry pick it after we merge this.

@cruhland I am going to merge this but I won't cherry pick for DC/OS 1.10. This fix is not critical.


rkt: run not working under openvz

Originally reported in https://github.com/coreos/rkt/issues/996#issuecomment-110566837 by @Arcnor, reproducing here:

This is without --debug:
[root@vps01 rkt-v0.5.6]# ./rkt run coreos.com/etcd:v2.0.4
rkt: searching for app image coreos.com/etcd:v2.0.4
rkt: found image in local store, skipping fetching from https://github.com/coreos/etcd/releases/download/v2.0.4/etcd-v2.0.4-linux-amd64.aci
Container rkt-ecc08f13-d7c0-4791-a819-0bed37f72a5f failed with error code 1.

With --debug
[root@vps01 rkt-v0.5.6]# ./rkt --debug run coreos.com/etcd:v2.0.4
rkt: fetching image from file:///root/rkt-v0.5.6/stage1.aci
rkt: searching for app image coreos.com/etcd:v2.0.4
rkt: found image in local store, skipping fetching from https://github.com/coreos/etcd/releases/download/v2.0.4/etcd-v2.0.4-linux-amd64.aci
2015/06/10 18:13:23 Preparing stage1
2015/06/10 18:13:24 Loading image sha512-1eba37d9b344b33d272181e176da111ef2fdd4958b88ba4071e56db9ac07cf62
2015/06/10 18:13:25 Writing pod manifest
2015/06/10 18:13:25 Setting up stage1
2015/06/10 18:13:25 Wrote filesystem to /var/lib/rkt/pods/run/e5e77b85-a453-4600-9663-38ff9a474a47
2015/06/10 18:13:25 Pivoting to filesystem /var/lib/rkt/pods/run/e5e77b85-a453-4600-9663-38ff9a474a47
2015/06/10 18:13:25 Execing /init
Spawning container rkt-e5e77b85-a453-4600-9663-38ff9a474a47 on /var/lib/rkt/pods/run/e5e77b85-a453-4600-9663-38ff9a474a47/stage1/rootfs.
Press ^] three times within 1s to kill container.
Container rkt-e5e77b85-a453-4600-9663-38ff9a474a47 failed with error code 1.

I've tried running systemd-nspawn with the stage1 directory that gets created on that SHA1, and I get up to Starting CoreOS, so I guess systemd-nspawn works (although I get a warning about having to disable audit, not sure if it's related)

Output of systemd-nspawn:
[root@vps01 e5e77b85-a453-4600-9663-38ff9a474a47]# systemd-nspawn -b -D /var/lib/rkt/pods/run/e5e77b85-a453-4600-9663-38ff9a474a47/stage1/rootfs
The kernel auditing subsystem is known to be incompatible with containers.
Please make sure to turn off auditing with 'audit=0' on the kernel command
line before using systemd-nspawn. Sleeping for 5s...
Spawning namespace container on /var/lib/rkt/pods/run/e5e77b85-a453-4600-9663-38ff9a474a47/stage1/rootfs (console is /dev/pts/1).
Init process in the container running as PID 12042.
/etc/localtime is not a symlink, not updating container timezone.
systemd 215 running in system mode. (-PAM -AUDIT -SELINUX +IMA -SYSVINIT +LIBCRYPTSETUP -GCRYPT -ACL -XZ +SECCOMP -APPARMOR)
Detected virtualization 'openvz'.
Detected architecture 'x86-64'.

Welcome to Linux!

Initializing machine ID from random generator.
[  OK  ] Created slice -.slice.
[  OK  ] Created slice system.slice.
         Starting Graceful exit watcher...
[  OK  ] Started Graceful exit watcher.
[  OK  ] Created slice system-prepare\x2dapp.slice.
         Starting Prepare minimum environment for chrooted applications...
[  OK  ] Reached target rkt apps target.
[  OK  ] Started Prepare minimum environment for chrooted applications.
         Starting coreos.com/etcd...
[  OK  ] Started coreos.com/etcd.

(although after this, I can't close it or kill it by Ctrl+C or anything :smile:)

@Acrnor in the last output you've pasted, is that the final line that prints? it looks like the app is actually started - does it respond to requests? (e.g. go to another terminal and curl localhost:4001/version)

I know about the Ctrl+], but for some reason it doesn't seem to work when running systemd-nspawn directly.

/cc @alban @iaguis

@jonboulle output:
[root@vps01 /]# curl localhost:4001/version
{"releaseVersion":"2.0.4","internalVersion":"2"}

OK, so nspawn itself works fine, good to know.
@yifan-gu I wonder if we should be printing the full nspawn invocation in --debug mode, thoughts?

@jonboulle Maybe add some debug level ?

@jonboulle Oh sorry, ignore above comment.
+1 to print the nspawn invocation in debug mode

This paste contains the last lines of the strace for rkt run ..., in case it's useful to find out why it fails: http://pastebin.com/AVEXP6Ec

@Arcnor to confirm, this is the release from GitHub? (i.e. not one you
built?)
On Wed, Jun 10, 2015 at 4:04 PM, Edu Garcia<EMAIL_ADDRESS>wrote:

This paste contains the last lines of the strace for rkt run ..., in case
it's useful to find out why it fails: http://pastebin.com/AVEXP6Ec
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/rkt/issues/1029#issuecomment-110941867.

This is 0.5.6 downloaded from the releases page on GH, not custom or anything.

 Does running the full strace strace (without -e open,write) work?

@iaguis I don't understand what you mean. The full strace is what I pasted before, but had to cut it because of pastebin's limits. Do you want me to run it again with strace -f -s512 rkt...? Or are you referring to something else?

Yes, we need -f to follow child processes when rkt forks and get the nspawn output and -s 512 to get the full parameters in the syscalls.
Usually, following the children makes strace not work properly with rkt if you don't filter syscalls with -e.

Here you go:
https://s3.amazonaws.com/uploads.hipchat.com/10804/158901/KI7CBy9WWmwwkMk/trace.zip

This seems to be the problem:
[pid  3580] prctl(PR_SET_SECCOMP, 0x2, 0x7f1aef5dcb30, 0x7f1aef5df0e0, 0x132480dd) = -1 EINVAL (Invalid argument)
[pid  3580] writev(2, [{"Failed to install seccomp audit filter: Invalid argument", 56}, {"\n", 1}], 2) = 57

This is where nspawn tries to do that: http://cgit.freedesktop.org/systemd/systemd/tree/src/nspawn/nspawn.c?id=ce30c8dcb41dfe9264f79f30c7f51c0e74576638#n3004
I'll look a bit more into it tomorrow, it's quite late in Berlin :)

Not sure if related to the warning about audit being broken and the need to disable it (which I can't do, because I don't control the kernel on the VPS). However, systemd-nspawn by itself seems to work, so not sure.
Good night :smile:

Your systemd-nspawn seems to be compiled with --disable-seccomp and that's why you get the audit warnings, seccomp's task is filtering audit system calls so they don't break containers.
systemd-nspawn in stage1 is compiled with seccomp enabled so it tries to install the audit filter. This is failing in OpenVZ for some reason.

@iaguis does this means it's a problem with the image I'm trying to run, or with Rocket itself? If it's a problem with the image, can I try with another one to confirm? (I don't know where is Rocket's "repository" to try this myself, if any)

It's a problem with rkt, more specifically with systemd-nspawn inside stage1. When the kernel doesn't set CONFIG_SECCOMP and systemd-nspawn is compiled with HAVE_SECCOMP, nspawn tries to enable the filtering and treats errors as failures.
It should instead log it and continue, you'll get the audit warning but at least you can start the container.
I've filed a PR upstream: systemd/systemd#182

Now that systemd/systemd#205 is merged you should be able to run rkt in OpenVZ when we update stage1 to the future systemd v221. You can make it work now if you build stage1 from the sources with the master branch.
Note though that running containers with audit enabled in the kernel without filtering the syscalls with seccomp can be problematic.

Thanks, I'll try to compile it myself.
In the meantime (and sorry if I seem a bit lost) if I manage to get my host provider to disable audit for the kernel (passing audit=0, although I very much doubt they will do such a thing) will that work without problems? I don't really understand what the auditing does, sorry :smile:

Yes.
audit lets you create create rules so you get logs whenever something happens (e.g. a file changes, a syscall is called...). The problem is that it breaks containers so either you disable it with audit=0 or you configure your kernel with CONFIG_SECCOMP so systemd-nspawn can filter out the audit syscalls when running containers.

Any news about this?

Well, I couldn't make my host change kernel parameters, as expected (it seems audit breaks containers, but you still need it enabled to have audit on the host itself, obviously)
I need to go the way of compiling everything myself, but I haven't had the time to do so, as it looks daunting :D. Hopefully tomorrow or this weekend :).

It's actually pretty easy. Make sure you have the systemd build deps and then run:
RKT_STAGE1_USR_FROM=src RKT_STAGE1_SYSTEMD_VER=master ./build

After that, your stage1.aci should have a systemd built from the current master branch.

I'm closing this. Please reopen if you still have problems :)


When will online payments be available in ArPaymentCreate?

Im using the SDK to allow our customers to pay invoices and noticed the TODO to add online payment which is documented in the API (ONLINECARDPAYMENT). I am writing a custom function to handle this for now, but I just wondered when this will be available within the SDK?

@dan2kx , sorry for the late response. Took a look at what you're referring and I can have this done soon. It'll be in upcoming JS SDK release.


Clean up the logic related to installCRDs.

We have already removed the installCRDs configuration item from the values.yaml file.
https://github.com/Kong/charts/commit/bf98a5cbae44af1f30c72f0d0d4f1910a38a0d8c
As a result, the logic related to  installCRDs will not work as expected.
https://github.com/Kong/charts/blob/5c74afe783d8b3905d7117b9836ddd1eb09b991e/charts/kong/templates/custom-resource-definitions.yaml#L1-L34
I will send a PR to clean up related logic

Closed per https://github.com/Kong/charts/pull/949#issuecomment-1816933641. If we want to remove this we need some path we've tested to confirm it does not delete CRDs coming from ancient installs.


Update README.md

I have added a few more recent challenges from QuillCTF and updated their links.

Folks, I would prefer to have links to individual solutions than each challenge which can be located from the main link.


v5 beta: react-query-persist-client import coming from "react.js"

Describe the bug
Build output is incorrect for @tanstack/react-query-persist-client PersistQueryClientProvider. Imports from "react.js" instead of "react"
- import * as React from "react.js";
+ import * as React from "react";

When using with Vite:
    node_modules/@tanstack/react-query-persist-client/build/modern/PersistQueryClientProvider.js:4:23:
      4 │ import * as React from "react.js";
        ╵                        ~~~~~~~~~~

Your minimal, reproducible example
https://stackblitz.com/edit/vitejs-vite-1tmnhf?file=src%2Fmain.tsx&terminal=dev
Steps to reproduce

Open minimal reproducible example
Start dev server with npm run dev (should start automatically)
View error in terminal

Expected behavior
Build output imports correct React location.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Firefox

Tanstack Query adapter
react-query
TanStack Query version
5.0.0-beta.0
TypeScript version
5.0.2
Additional context
No response

in the sourcecode (typescript) we always do:
import * as React from 'react'

as you can see here:
https://github.com/TanStack/query/blob/1d3ee672d2a1536e3463243813c91902aa97d973/packages/react-query-persist-client/src/PersistQueryClientProvider.tsx#L2
@lachlancollins does this have something to do with how things are bundled in beta ?

in the sourcecode (typescript) we always do:
import * as React from 'react'

as you can see here:
https://github.com/TanStack/query/blob/1d3ee672d2a1536e3463243813c91902aa97d973/packages/react-query-persist-client/src/PersistQueryClientProvider.tsx#L2
@lachlancollins does this have something to do with how things are bundled in beta ?

Oh yeah, it probably does. I'll have a look, it might be to do with the deps/devDeps/peerDeps affecting how esbuild recognises something as external/internal.


Upgrade Kafka to 3.2 and confluent platform to 7.2.2

Closes #105

@torbsto Good point. I'll change it to Kafka 3.1 in the docs.


[enhancement]: color prompt

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
input: a rough sketch where each zone has a flat color
prompt workflow: sample each flat color create a prompt text input, enter prompt for each color
output: an image that follows the prompt per color region
Alternatives
rolling dice a thousand times and hope you get lucky on masked canvas
Aditional Content
No response

Relevant research paper, for when we implement this
https://github.com/omerbt/MultiDiffusion

Something no model does well is directed flow of liquid.
As an example draw a thick wavy mask and ask for a flow of water.
The NN are not trained on understand flowing liquids perhaps. Maybe it's not possible with SD to teach it flow but I've seen NN guided by image gradient so maybe an extra layer to explain direction would help. Similar to flow map to describe liquids flowing in a shader.


feat: Flow Wallet support

add EVM on Flow to chains: https://developers.flow.com/evm/using
add support for Flow Wallet

the implementation is based on https://github.com/Outblock/flow-evm-rainbow/blob/main/component/config.ts#L18-L76

https://github.com/onflow/docs/issues/1005


⚠ Bofrost.fr has degraded performance

In 6848abe, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1877 ms

Resolved: Bofrost.fr performance has improved in 67af4b4 after 22 minutes.


0.2.1 - Update pubspec.yaml to include intl 0.19.0

Updated the following references.

Use intl 0.19.0

sdk version range updated

flutter_lints use 4.0.0

No functional changes.

@imanabu Thank you for your contribution


Fails to properly create necessary tables with Postgres

Note: This is just a template, so feel free to use/remove the unnecessary things
Description

Type: Bug

Bug
Expected Behavior
Running npm run sequelize db:migrate should create the relations in the DB.
After following this instructions here: https://github.com/reactGo/reactGo/blob/master/docs/databases.md and running npm run sequelize db:migrate it seems to run properly. The tables are created.
Actual Behavior
After following this instructions here: https://github.com/reactGo/reactGo/blob/master/docs/databases.md and running npm run sequelize db:migrate it seems to run properly. The tables are created.
However, when I try to select anything from them, it says that the relations don't exist.

Steps to Reproduce

Git clone the repo
npm install
Run:

createuser root
createdb react_webpack_node_development # or test/production
npm run sequelize db:migrate

Then
DB_TYPE="POSTGRES" npm start
Then go to localhost:3000
The router will throw an error Cannot GET /
and see the following output:
><EMAIL_ADDRESS>start /Users/parkerruhstaller/Sites/reactGo
> cross-env NODE_ENV=production node compiled/index.js

--------------------------
===> 😊  Starting Server . . .
===>  Environment: production
===>  Listening on port: 3000
===>  Using DB TYPE: POSTGRES
===> 🚦  Note: In order for authentication to work in production
===>           you will need a secure HTTPS connection
--------------------------
(node:29663) DeprecationWarning: Using Buffer without `new` will soon stop working. Use `new Buffer()`, or preferably `Buffer.from()`, `Buffer.allocUnsafe()` or `Buffer.alloc()` instead.
Failed to prune sessions: relation "session" does not exist
Executing (default): SELECT 1+1 AS result
Successfully connected to sequelize database
Failed to prune sessions: relation "session" does not exist

Has anyone got this working properly with postgres? I didn't try it with mongo and it doesn't work with no db.

For your query have you tried to add double quotations to delimit the identifier:
select * from "Users"
Given that you're getting a failed to prune sessions: relation "session" does not exist warning and you're able to connect to the database, it might not hurt to double check that the production database (react_webpack_node_production) has had the migration run on it as well.

We're cleaning the project and closing issues.
We didnt hear back from you about this so i'll assume it was resolved.
If you still have any issue, please comment here or open a new one


InterLap vs QuickSect

Hi @brentp, I started out using InterLap in a project, and am now looking to switch to QuickSect. In InterLap, the closest() method returns the nearest interval, which may be overlapping the query. On the other hand, QuickSect left() and right() methods return the nearest interval not overlapping the query.
For example, if I have a db with two intervals [(1, 10), (1, 20)], calling interlap.closest((15, 30)) returns [(1, 20)] whereas calling itree.left(Interval(15, 30)) returns [(1, 10)].
Is this difference between the two libraries intended? Is there anyway to replicate the behavior of InterLap in QuickSect?
Thanks!

hmm. that's probably an oversight on my part but interlap does lend itself more easily to this type of query. you could make your own closest function in quicksect by using find(15, 30) and then using left if that fails. Or you can use itree.left((30, 31)). Or, you could open a PR to provide a closest method for quicksect. There may be one in bx-python that you could borrow.
Sorry for the inconsistency, hope that helps.

Ok, I'll work on adding this functionality and submit a PR. Closing this issue.


Cost regarding the usage of s3-connector-for-pytorch

Dear AWS Labs Team,
I'm writing to inquire about the underlying mechanism used by the S3 connector. Specifically, does it rely on LIST requests to access objects within an S3 bucket?
If so, Is the following cost estimation associated with iterating over objects for neural network training correct? Let's consider a scenario with a bucket containing 100,000 objects and with standard cost of the LIST request being 0.0005/1k USD. If I train a network for 10 epochs (and in each epoch I do a full pass of the dataset) would the total cost for these requests be approximately 100k * 10 * 0.0005 USD / 1k = 0.5 USD?
Thank you for your time and assistance.
Best regards,

Hi Tomasso,
Thanks for reaching out and for your interest in our connector!
I need to research a bit the cost aspect and get back to you.
Thanks,
Diana.

Hi Tommaso,
Thanks a lot for your interest in using S3 Connector for PyTorch.
Caching is a interesting topic for the PyTorch connectors. Are you interested in caching object keys (to avoid lists) or object data? If former, you can always use S3MapDataset.from_objects() or S3IterableDataset.from_objects() methods those let you pass list of S3Keys you like to access. We would love to learn more about your use-case so we can understand how caching would benefit your workload?
You would not pay for data-transferred if your S3 bucket and compute is located in the same AWS region. There would still be request cost though, which is 0.0004 USD for 1000 requests for S3 Standard in us-east-1.
Please let us know if you have any follow-up questions.
Fuat.

Hi @TommasoBendinelli Is my answer above answered your questions? I am going to close this issue for now, please let us know if you have any other questions.


WithColumn() alias value is changed by other query

I have two queries in one table using the same alias name for a virtual column:
$expensePerMonth = ExpenseQuery::create()
    ->withColumn('MONTH(Expense.Date)', 'Month')
    ->withColumn('SUM(Expense.Amount)', 'Total')
    ->groupBy('Month')
    ->orderBy('Month')
    ->find();

echo '
'; var_dump($expensePerMonth->get(0)->getTotal()); echo '
';

$expensePerCategory = ExpenseQuery::create()
    ->withColumn('SUM(Expense.Amount)', 'Total')
    ->groupByCategoryId()
    ->find();

echo '
'; var_dump($expensePerMonth->get(0)->getTotal()); echo '
'; exit;

I'd expect this code outputs two times the same value, but instead it outputs this:
string(6) "477.00"
string(7) "5401.00"

When I change the name of one virtual column to aTable the script outputs two times the same value.

Is this still relevant or can it be closed?

I don't use propel anymore, so for me it's irrelevant.


FIX: enforcing contiguous array when saving output data.

Fixing BufferError: memoryview: underlying buffer is not C-contiguous when pyelastics writes output image data.

Thanks!


pushError loses context data

Description
Context data attached to a error via the context object gets lost in the backend when using Grafana Agent.
aro.api.pushError(error, {
  type: 'network',
  context: {
      message: 'React error boundary',
      componentStackTrace: {...}
    },
});

The issue was reported on the Faro community Slack.
I tested it with the Faro Receiver in cloud and the Grafana Agent.
Faro receiver works well, but when using the Agent the context attributes do not appear in the Loki log.
Steps to reproduce
I started the Demo app which uses the Grafana Agent and sent a custom error with context attributes attached.
The context attributes did not show up in the error log.
Then I started the fe-o11y plugin, which is using the Faro receiver, locally and sent a custom error with context attributes attached as well.
The context attributes showed up in the error log as excpected.
Expected behavior
Context attributes should appear in the Loki log.
Actual behavior
context attributes do not appear in the Loki log.
Environment

SDK version: 1.3.8
SDK instrumentations:
Device type:
Device name:
OS:
Browser:

Demo

Context

Having this exact same issue, any updates / fixes in sight?

Anyone looking at this?
We think we are running into the same issue.

Found the issue: https://github.com/grafana/alloy/pull/1413

The fix will be release with Alloy v1.4

Alloy v1.4 i already released. Closing this issue


Update PyPI

This common problem:
skpy.core.SkypeApiException: ('404 response from PUT https://db5-client-s.gateway.messenger.live.com/v1/users/ME/endpoints/<snip>/presenceDocs/messagingService', <Response [404]>)

Which several users have encountered, is not fixed in the PyPI version of SkPy.
Specifically, its conn.py file needs updating.
It is currently at this version:
https://github.com/OllieTerrance/SkPy/commit/741cffd2fe9abf1aec0c7db895dddc8d14346d38#diff-3f76e41bdeaab4f1f3989aa66219d780
But the fix was pushed in the following commit:
https://github.com/OllieTerrance/SkPy/commit/7b6be6e41238058b9ab644d908621456764fb6d6#diff-3f76e41bdeaab4f1f3989aa66219d780

Pushed 📦 v0.8.1 (GitHub, PyPI).
Apologies for the delay -- I was holding this pending some refactoring which I never got round to...

Thanks for the blazingly fast turnaround - the PyPI package is now working perfectly


updated readme file for docker-section after container build

Duplicate #404

Thanks @harsha14581, but closing as duplicates #404


Skip non-issues messages returned from CodeClimate

fixes #76

Thanks for opening this PR. This does look good, but we need to get the tests green. They're red on master, so it's not related to your change. I opened #78 to try and get them green, but ... did not succeed. Will plan to come back to this as soon as we can merge it

@a-ali @maxjacobson This PR is outdated. I'm using CodeClimate cli v0.71.1, and now the type is issue, not Issue, so this PR is currently making the linter missing all the issues reported by CodeClimate cli.

A note about the required CodeClimate cli version should be added to the README.md too, because depending on that, this package may malfunction.


How can I add validation error messages for ReactiveCheckbox as like ReactiveTextField

ReactiveTextField(
style: TextStyle(fontSize: 25),
formControl: store.fg.controls['name'],
validationMessages: (control) => {
ValidationMessage.required: 'Name Is Required',
ValidationMessage.minLength: 'Min Length is 2'
},
)

Hi @Skanda-Kumar,
Thanks for using Reactive Forms and for the issue. Well actually in the native Flutter Checkbox there is no Validator function, which means that Flutter doesn't validate or show error messages for checkboxes.
On the other hand, Reactive Forms does allow you to validate a form control bound to a Checkbox (for example Validators.requiredTrue). But is up to you to show the message of the error. You can use for example a ReactiveStatusListenableBuilder for listening when the control becomes invalid and show the message.
ReactiveStatusListenableBuilder(
  builder: (BuildContext context, AbstractControl<dynamic> control, Widget child) {
    return control.invalid ? Text("required true") : Container();
  },
),

Could you please give me a use case example so I can better understand what you are trying to achieve?
Thanks.

Thanks for your quick response!
My requirement is like if you have a I Agree check box and in the formgroup if you give the formcontrol as required, then it should show the error message as like how it is showing for text field as shown in the image below on clicking the submit button.

How can I achieve this ?

Hi @Skanda-Kumar,
you can use the ReactiveStatusListenableBuilder to show/hide the error like this:
ReactiveStatusListenableBuilder(
  formControlName: 'control',
  builder: (BuildContext context, AbstractControl<dynamic> control, Widget child) {
    // if control is invalid and is touched then show the error
    return control.invalid && control.touched ? Text("required true") : Container();
  },
),

and in the onPressed of the button:
RaisedButton(
  child: Text('SUBMIT'),
  onPressed: () {
    if (form.valid) {
      //do something
    } else {
      // this will marks all children as touched and the errors will show up
      form.markAllAsTouched();
    }
  },
),

Of course, there is a second solution that is to disabled the Button and enabled just when the form is valid.
Please let me know if this answers responses your question.

This works for me. Thanks !!


sections: Extract widget component from Section and others

To ensure that widget generation is always handled the same way and new features are not added in a single code path, a Widget component is added to display the high level widget (Text, Group, Button, InfoMap or Question). There is one version for end components (Widget) and one for widgets that are part of a grouped object (InGroupWidget), with additional pathPrefix and different widgetStatus location.
Section templates in demo_survey are updated to use this new component.
Also type the SectionConfig and pass the sectionConfig to the Section widget instead of individual fields. This is a breaking change for surveys as section templates will also need to be updated to match the Section's props. See commit message for details.

This is pre-requisite work to bring the segment section template to evolution and reduce to a minimum what is the "template" part of the "template" as currently, there's a lot of common code between the templates and the Section component that are not exactly the same just because they did not evolve at the same pace, with bug fixes in one not matched in the other.

@samuel-duhaime Cette PR contient des breaking changes. Attention à la prochaine mise à jour d'Evolution dans les enquêtes. Je vais attendre avant de merger si jamais tu veux reviewer.


Some suggestions (multithreading capabilities and GPU integration)

Thank you for developing this project. However, I encountered a significant issue with the documentation. While there is online documentation available, I believe it could be more user-friendly. The quick start tutorial is helpful, but the web documentation lacks detailed information about the parameters for each function and explanations of what each function does.
Undoubtedly, this project is very impressive and noteworthy. The addition of multithreading capabilities and GPU integration could significantly enhance its performance, especially during large data calculations. I hope these features will be considered in the future.

Hi @HemlockPoison
Thank you for your feedback. I may not understand your suggestion completely but wanted to highlight that each function in the API includes detailed documentation, so not sure what you mean by "the web documentation lacks detailed information about the parameters for each function and explanations of what each function does."
Additionally, the functions that were used to find the modules are primarily identical to the R version, including function names, so you may refer to the original paper for more information on the methods.
As the sole developer, I am doing my best to maintain the package. I'll keep your suggestions, such as adding multithreading capabilities and GPU integration, in mind for the future. I also welcome anyone interested in contributing to these enhancements.

@nargesr I'm new to wgcna and trying this out. We have a roughly large dataset ~100k cells x 17k genes and I think it would benefit from some parallelism. Do you know which step during findModules may benefit the most from some parallelism? Would it be during calculating adjacency with corrcoef?

Hi @tuanpham96,
Thank you so much for expressing your interest in PyWGCNA. I would suggest you look at the R version of WGCNA to fully understand the method.
There are three functions during findModules() step that would benefit from doing some parallelism

pickSoftThreshold()
'adjacency()'
TOMsimilarity()

I believe TOMsimilarity() is the most time-consuming part.
Please let me know if you have any questions.
Best,
Narges


compute coordinates at full precision internally

Although it's fine to display coordinates at a mm precision, the value exposed internally should have full precision, in case some people may want to re-use that.

Also, if the numSlice parameter is not integer, this should be used to set the viewer at some exact coordinates, and then numSlice should be rounded.


[iOS] Portuguese (Portugal) compound character entry creates extra character

Steps to reproduce

Run the sample app in the iPad Air 13-inch emulator with iOS 17.5 with a Portuguese (Portugal) keyboard
Press ´ and then a

Expected results
Á is inserted
Actual results
´á is inserted
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(
    const MaterialApp(
      home: Scaffold(
        body: TextInputExample(),
      ),
    ),
  );
}

class TextInputExample extends StatelessWidget {
  const TextInputExample({super.key});

  @override
  Widget build(BuildContext context) {
    return const Center(
      child: Padding(
        padding: EdgeInsets.all(8.0),
        child: TextField(
          autofocus: true,
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/user-attachments/assets/9fe51c8e-298a-419e-aef2-09d098d57a3c

Logs
Logs
No relevant logs.

Flutter Doctor output
Doctor output
$ flutter doctor -v
[✓] Flutter (Channel master, 3.25.0-1.0.pre.153, on macOS 15.0 24A335 darwin-arm64, locale en-BR)
    • Flutter version 3.25.0-1.0.pre.153 on channel master at /Users/angelosilvestre/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 17f63272a0 (5 weeks ago), 2024-08-27 19:30:17 +0000
    • Engine revision 7d751acc81
    • Dart version 3.6.0 (build 3.6.0-175.0.dev)
    • DevTools version 2.39.0-dev.15

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/angelosilvestre/Library/Android/sdk
    • Platform android-35, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 16.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16A242d
    ! CocoaPods 1.12.1 out of date (1.13.0 is recommended).
        CocoaPods is a package manager for iOS or macOS platform code.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/to/platform-plugins
      To update CocoaPods, see https://guides.cocoapods.org/using/getting-started.html#updating-cocoapods

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] VS Code (version 1.93.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.98.0

[✓] Connected device (4 available)
    • iPad Air 13-inch (M2) (mobile)  • 9D3D13AF-EA1C-47A8-AADF-DEBF72D2C99E • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-17-5 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 15.0 24A335 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 15.0 24A335 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 129.0.6668.90

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

This might be related to https://github.com/flutter/flutter/issues/156183

Reproducible using the code sample and steps provided above.

flutter doctor -v
[✓] Flutter (Channel stable, 3.24.3, on macOS 15.0 24A335 darwin-arm64, locale en-US)
    • Flutter version 3.24.3 on channel stable at /Users/deanli/dev/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2663184aa7 (3 weeks ago), 2024-09-11 16:27:48 -0500
    • Engine revision 36335019a8
    • Dart version 3.5.3
    • DevTools version 2.37.3

[!] Flutter (Channel master, 3.26.0-1.0.pre.353, on macOS 15.0 24A335 darwin-arm64, locale en-US)
    • Flutter version 3.26.0-1.0.pre.353 on channel master at /Users/deanli/dev/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 389846308a (7 hours ago), 2024-10-03 21:52:22 -0400
    • Engine revision 66d397dff8
    • Dart version 3.6.0 (build 3.6.0-317.0.dev)
    • DevTools version 2.40.0-dev.2

@angelosilvestre Is this also fixed in iOS 18 like https://github.com/flutter/flutter/issues/156183?

@justinmc No, it still happens in iOS 18.


Lazy Loading Remote Data white rectangle before items load

Try to click the link bellow and before the items for the MultiSelect are loaded, open the Dropdown.
https://cuppalabs.github.io/angular2-multiselect-dropdown/#/lazyloadingRemoteData
It will show a blank white rectangle that won't disappear when the items arrive.

@enoquemoises fixed in v4.5.0


CAMEL-17524 fix resource loading so it works in OSGi

Fix resource loading calls to pass in calling classes class loader so that they work as expected in OSGi enviroments.

[ x ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ x] Each commit in the pull request should have a meaningful subject line and body.
[ x] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[ x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ x] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/main/CONTRIBUTING.md

cherry picked to main


Port Watcher

Added project for server status watchers.
Currently available PortWatcher for scanning and ensuring opened ports for tcp connections.

Uses DNS to resolve host name
Fluent configuration like other watchers
Documented public API
Interface-based
Covered with tests
Added sample in console projec

Future, possible watchers:

Ping (ICMP porotocol)

New changes (fixes) was commited to this branch.

Thank you for the contribution! Port watcher is already available on NuGet :).

Thank you for the contribution!
Port watcher is already available on NuGet


Workaround for getFullyConverted returning multiple results.

It seems that getFullyConverted sometimes returns more than one result, which causes GlobalValueNumbering to produce a huge number of results.

Thanks, @kevinbackhouse!


Add At the Speed of Light x Rap God by Dimrain47

Song Name
At the Speed of Light x Rap God
Artist Name
Dimrain47
Source
Youtube
Youtube Link
qMw1TjgDnj33hAFs
Direct File Link
No response
Song ID
467339
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 7573669

Accept

accept
пн, 5 авг. 2024 г., 08:06 auto-nong[bot] @.***>:

Thanks for contributing!
Someone will soon review your request.
Here is the JSON result for the reviewer:
{
"name": "At the Speed of Light x Rap God",
"artist": "Dimrain47",
"source": "youtube",
"startOffset": 0,
"yt-id": "qMw1TjgDnj33hAFs",
"songs": [
467339
]
}
To accept the request, a reviewer needs to write "accept".
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/2383#issuecomment-2268181745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKKH6SAUJYEVVGCQJJT7B3TZP4B5TAVCNFSM6AAAAABL7RBP6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRYGE4DCNZUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

reject
invalid YouTube link


[BUG] fix Search-component adjustments

Describe the bug
We use the Search component in many places in the app.
In pr #652 , @Som401 created a clear button, it works great in Feeds.

However, we use this component in other places in the app , there this button does not work because we need to send the relevant prop to make it work.
When we write some text the size of search icon changes , change the style so it will not be affected

To Reproduce
We use the SearchInputBox in next components:
CTF.jsx --> https://dev.thecyberhub.org/#/ctf
Sidebar.jsx --> https://dev.thecyberhub.org/#/dashboard/chat
NoteSidebar.jsx --> https://dev.thecyberhub.org/#/dashboard/notes
Explore.jsx --> https://dev.thecyberhub.org/#/explore
Feeds.jsx --> here it is working
Expected Behavior

make it work
see that the search bar is not moving things around and the size of the search icon is stays the same.

Screenshot/ Video
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/00ee53ab-de67-434d-9601-8b1df47fde9d
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/82e0f12e-7a55-466b-bc86-9a401239142d
Code of Conduct

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

@ArkadiK94, I've been thinking about expanding the tag-filtering functionality beyond the feeds.jsx file. I believe it would be beneficial to move this functionality to the Search component file. This way, it can be easily accessed and utilized by other components throughout the project like the explore section.

@Som401 Great idea, so users could search multiple words, but wait with making changes in Feeds and FeedPosts files there is a pending pr which make some other changes in those files and I would like to minimize the conflicts that can happen.


Varying Success Status Codes

Running into an issue where we return a partial response from a certain RPC when the user has logged in but not completed MFA.  We return the response and client reacts appropriately to data in it but it would be really nice for our monitoring if we could pick up the status code instead of adding a custom metric to detect this scenario.
One potential solution I see here is a special twirp.StatusCodeError - this could be returned with a response from handlers to vary the http status while still returning success responses.  Codes could be limited to 2xx range if we want.
I'm happy to work on a PR for this just wanted to check if there was another facility I'm missing to do the same or a technical reason this hasn't been done yet before I dive in.

Hi. Apologies for the delay on a response.
We discussed this and came to the conclusion that because Twirp is not just a library at this point, but also a protocol that expects a status 200 for success, that making this change would not be easy. I think the custom metric is the correct approach at this point.


✋ [MEMBER] - clintonb

Why do you want to join?
I am considering transferring https://github.com/jazzband/django-waffle so that I have greater administrative support. I'd rather not return it to independent status.
Which project(s) are you looking to contribute to?
https://github.com/jazzband/django-waffle
Code of Conduct

[X] I agree to follow Django Commons's Code of Conduct

Thank you @clintonb for joining! You'll get an invite email from GitHub. You'll have one
week to accept that. If you don't mind, after accepting, can you set your
organization membership as public?
This helps Django Commons grow.


Surface Normal Offset

Great work
I was just wondering
Is there any way to offset an STL surface using numpy-stl ?
something like what Alec has done in the link below:
https://www.alecjacobson.com/weblog/?p=4708
Thank you

You should have all variables available to make the calculation: https://numpy-stl.readthedocs.io/en/latest/stl.html?highlight=v0#stl.base.BaseMesh
I would assume that you can use either mesh.vectors or mesh.points with mesh.normals to offset in the right direction.
How you actually want to do it can be tricky though... But I think this paper might be able to help a bit with that: http://www.cad-journal.net/files/vol_1/CAD_1(1-4)_2004_285-291.pdf

I have used the following code to call the STL file from my external folder:
 Folder = os.path.join('0001-STL/test.stl');
your_mesh = mesh.Mesh.from_file(Folder.stl)
Andand I successfully was able to get all the variables (your_mesh .vectors, your_mesh .points, your_mesh .normals, your_mesh .x, your_mesh .y , your_mesh .z ........ etc)
Now I wish to get the data of the offsetted file named for example (your_mesh _1)
How can I do that ?
Thank you so much.

You can copy a mesh like this:
your_mesh_1 = mesh.Mesh(your_mesh.data.copy())

Thanks for your reply
I was actually asking how to offset the stl surface and get the data of the your_mesh1 file.
for example if I use pymadcad as in this link:
where the inputs are ([points, ...], [triangles, ...])
and output is my_thick_mesh
from madcad import *

surface = Mesh([points, ...], [triangles, ...])
my_thick_mesh = thicken(surface, distance)

I would end up with a new data named my_thick_mesh for the offsetted part (thickened part)
My question is:
How can I use numpy-stl the same way I use pymadcad  where my inputs are ( your_mesh , and the distance) instead of  ([points, ...], [triangles, ...]) ,
and my output is the your_mesh1 data, instead of  my_thick_mesh
Thank you again

I haven't worked with pymadcad yet so I can't be sure, but it looks like pymadcad is already using this library to read stl files:
https://github.com/jimy-byerley/pymadcad/blob/b7757d15f8ae952f7b25460dc7c3674f6569d391/madcad/io.py#L155-L178
Copying that code, I believe you can do this:

# stlmesh is your numpy-stl Mesh instance
trinum = stlmesh.points.shape[0]
ptsbuff = stlmesh.points.reshape(trinum*3, 3).astype('f8')
pts = glm.array(ptsbuff).to_list()
faces = [(i, i+1, i+2)  for i in range(0, 3*trinum, 3)]
# This is the pymadcad Mesh
mesh = Mesh(pts, faces)
mesh.options['name'] = stlmesh.name

thick_mesh = thicken(mesh, distance)

I have seen this before. The problem is that pymadcad is not working on my machine. So I tried to see if numpy-stl has a specific function that works as same as thicken function.
Thank you

No, not really. This library was meant as a quick way of storing and loading STL files, performing operations on that mesh are largely outside of the scope of this library.

Thank you for clarification.


Eigen needs to display lot number for Live Auction

Issue:
In a Live Auction we don't provide the user with Lot number but rather provide lot position. This gets tricky because both - the auction catalog and the prediction web (and the live video feed) call out Lot number for the lot.
Solution:
We need to add actual Lot Number information

Related to #1755

@katarinabatina What do we want this to look like? It seems as though it would be easy to replace the lot progress with the lot number, but then we'd lose our indication of progress.

@katarinabatina @acjay bumping to next sprint. Def need this one before the mid-september phillips sale.

@ashfurrow lets move towards just displaying the actual lot number, not lot progress

@katarinabatina Sure, so remove the purple bar?

@ashfurrow no, the bar stays just remove the /x and only post the real lot number

I did that in my very first Eigen PR about a week ago: https://github.com/artsy/eigen/pull/1827 :) Probably should have linked it to this issue.

@acjay I don't understand your PR what was it that you did?

Just what you said. Whenever the new version hits the store, it'll have the assigned lot number, not the index anymore. And no total.
So if your on the 30th lot out of 200, and it's what Phillips calls Lot 35, the user sees Lot 35, and that's it.

@acjay ah cool, but the progress indicator still functions correct?

Yep, no change there.

@acjay rockin', just wanted to double check! (congrats on your first eigen PR 🎉  )

Cool! Are we good to close this issue then?

Sure, if it's pretty much hands-off for getting it to the Store at this point.

We've shipped this, right?

Yup!


undefined method `new' for "Rack::RawUpload":String

rails generate rich:install

when i run this line i got error ,
middleware/stack.rb:35:in `build': undefined method `new' for "Rack::RawUpload":String
Did you mean?  next (NoMethodError)

my gem file is
gem 'rails_admin', '~> 1.2'
gem 'rich'
gem 'carrierwave', '~> 1.0'

could you help me out from this.
thanks .

I'm having the same issue updating an old application.

same error. any solution?

same here, does anyone managed to solve this?

getting same error, any solution?

Same error, any solution to this?


Undo operation after changing a cell with linked formulae does not change the cell itself

Describe the bug
Undo operation after changing a cell with linked formulae does not change the cell itself
To Reproduce
Steps to reproduce the behavior:

Go to a cell on which other cell's formula is dependent
Change the value, the linked formula cell also changes
Perform 1 undo operation, the formula cell value revert but the original edited cell still remains unchanged
See error

Expected behavior
Both the cell and the formula cells should revert with the same 1 undo operation

Yes, this is a known bug. Right now it requires 2 undos

thakns for the update @zyc9012 , i will have a look at the pr too

I pushed a commit 42b3c22353556b3748278e03f0539dac22223552 which should resolve this issue

I think this can be closed now


Avoid using RLS

The Avoid using RLS section ought to be altered to clarify which approach should be the default one.
Currently it sounds like by default, people should avoid RLS. In my view, unless you've got extremely large data models or complicated RLS conditions, then the benefits of RLS far outweigh the disadvantages.
Maintaining multiple workspaces and reports, as well as inability to view everything at once (unless there's a "total" dataset, which would defeat the purpose of multiple small models) are all huge disadvantages. Not every company has such large data models that a single dataset with RLS will be too slow for people to use and refresh. For some reason, there's no mention of incremental refresh, which can address the data refresh responsiveness.
The documentation says "While it results in more models, they are smaller in size." Yes, they're smaller in size, but overall they're bigger than a single model, no doubt about that.

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

ID: 8e49a178-5b5e-ae62-b12c-be846eb6ce40
Version Independent ID: 8372b4a0-75e5-dc84-cdbf-e1c2d76331a7
Content: Row-level security (RLS) guidance in Power BI Desktop - Power BI
Content Source: powerbi-docs/guidance/rls-guidance.md
Service: powerbi
Sub-service: powerbi
GitHub Login: @paulinbar
Microsoft Alias: painbar

Hi, @DaniilMaslyuk -- thanks for your feedback! Assigning to the author for consideration.


Add more details into the root readme

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Can we add more details into the root readme? This is where a user lands when clicking "code repo" from the Learn pathway. Should have a bit more details to avoid confusion
Affected area/feature

the '../README.md#Prerequistes' in /backend/src/readme.md leads to nothing