The steps to obtain absolute depth on custom dataset

Hi! I look through some discussions in the MiDaS repo's issue and summarize the steps to obtain absolute depth from the estimated dense inverse depth. Am I right?
Step 0: Run SfM to get some sparse 3D points with correct absolute depth, e.g. (x1,y1,d1), ..., (xn,yn,dn)
Step 1: Inverse the 3rd dimension to get 3D points with correct inverse depth, e.g. (x1,y1,1/d1), ..., (xn,yn,1/dn)
Step 2: Run DPT model to estimate the dense inverse depth map D
Step 3: Compute scale S and shift T to Align D with {(x1,y1,1/d1), ..., (xn,yn,1/dn)}
Step 4: Output 1/(SxD+T) as the depth

The steps are correct to align the estimates to the SfM construction, but SfM is unable to recover absolute depth too. So the aligned depth maps would have a consistent scale for the given scene as opposed to an arbitrary scale per image before the alignment, but there would still be a missing global scale to get absolute metric measurements.
These slides give a good overview about SfM ambiguities: https://slazebni.cs.illinois.edu/spring19/lec17_sfm.pdf. Slide 7 shows the relevant issue.

Got it. Thanks for your kind reply!
Another question is, in EVALUATION.md I notice that when evaluate on KITTI, the argument absolute_depth is specified and the prediction is scaled by 256. There are no more post-processing steps to compute the scale and shift because the  dpt_hybrid_kitti-cb926ef4.pt model is trained (or finetuned) specifically on KITTI?
https://github.com/isl-org/DPT/blob/f43ef9e08d70a752195028a51be5e1aff227b913/run_monodepth.py#L165
https://github.com/isl-org/DPT/blob/f43ef9e08d70a752195028a51be5e1aff227b913/run_monodepth.py#L166
https://github.com/isl-org/DPT/blob/f43ef9e08d70a752195028a51be5e1aff227b913/util/io.py#L180
https://github.com/isl-org/DPT/blob/f43ef9e08d70a752195028a51be5e1aff227b913/util/io.py#L181
If I want to evaluate the dpt_large model on KITTI, do I still need to follow the above step0~step4 to convert the inverse-depth map?

Yes to both questions.
A word of caution for evaluating the large model this way: when evaluating the existing large model, which doesn't estimate absolute depth, the numbers are not directly comparable anymore to the numbers in Table 3 since the alignment step will "remove" part of the error. The numbers will be only comparable to Table 1 (or Table 11 in the MiDaS paper) where we did the alignment for all methods to have a fair comparison.

@07hyx06

Another question is, in EVALUATION.md I notice that when evaluate on KITTI, the argument absolute_depth is specified and the prediction is scaled by 256.   There are no more post-processing steps to compute the scale and shift ...

Additionally there are used invert=True, scale and shift parameters which depend on the model-weights and dataset (or for real cases - depend on model-weights, camera intrinsics and unit of measurement for depth):

when you use Kitti-weights: https://github.com/isl-org/DPT/blob/f43ef9e08d70a752195028a51be5e1aff227b913/run_monodepth.py#L53-L65

or NYU-weights: https://github.com/isl-org/DPT/blob/f43ef9e08d70a752195028a51be5e1aff227b913/run_monodepth.py#L68-L80

@ranftlr @AlexeyAB Thanks for your help!

Hi! I did some experiments on the flower dataset these days. Can you give me some help to improve the alignment result?
I run COLMAP with the default configuration to obtain the camera parameters and sparse 3D points (I have already converted them to the camera coordinate). The goal is to align the estimation of DPT to the SfM scale and get a dense depth map for every image.
Denote the depth map outputted by DPT as D, with shape of [h,w]; the collection of sparse 3D points as {[x_i,y_i,d_i]}. Firstly I extract the corresponding value of {[x_i,y_i]} from D and get {[D_i]}. Then I simply compute a scale and shift to align {[D_i]} and {[1/d_i]} by np.linalg.lstsq. The fitting result as shown in the figure below. The blue points are (D_i, scale * D_i + shift) and the orange points are (D_i, d_i).

I use the aligned inverse depth map, combined with the SfM scaled camera parameters, to warp a source image to the target viewpoint, the results (warped vs target image) are shown below:

It seems that some pixels are misaligned between the warped image and the target image. Is it a reasonable result? Can I do something to improve the fitting process?

As the results of the model are not perfect a residual error is expected. How much error, will likely vary per image.
Here are some works that try to address the consistency issue:
https://roxanneluo.github.io/Consistent-Video-Depth-Estimation/
https://robust-cvd.github.io/
These works tackle the case of dynamic objects in the reconstruction. If you expect no independently moving objects in the scene, you can also directly for MVS which will lead to consistent results out of the box.

It seems that some pixels are misaligned between the warped image and the target image. Is it a reasonable result? Can I do something to improve the fitting process?

@07hyx06 Have you found a solution to this problem?


Currency filter without cents

Is it possible to use currency filter without displaying the cents?

No, but you can pipe it through another custom filter that just strips off the cents.
Also, please ask questions like this at vuejs/Discussion or the gitter channel.


Protect against deep payload objects + improve obfuscated token

Description
In previous PR (#22), we protected the obfuscator from big objects by counting the leaf attributes. But this does not protect against iterating on deeply nested properties. This PR adds a guard against this.
The tokens used for obfuscation have been improved as well.

"string" -> "<redacted_string>"
"number" -> "<redacted_number>"
"boolean -> "<redacted_boolean>"
"undefined" -> "<undefined>"

Bump version to 2.25.1
Testing

[x] This change adds test coverage for new/changed/fixed functionality

Checklist

[x] I have added documentation for new/changed functionality in this PR or in auth0.com/docs
[x] All active GitHub checks for tests, formatting, and security are passing
[x] The correct base branch is being used, if not master

@radekk please review ☝
Thanks!

LGTM 👍


Fails when using with Vagrant NFS in another project

I've got an error:
[INFO] Configure NFS ...

 !!! Sudo will be necessary for editing /etc/exports !!!
exports:5: /Users conflicts with existing export /Users/adam/Projects/my-project

The directory path model for Docker Machine is in conflict with Vagrant's. It's not really a problem that this script can address (other than making a better warning message). I solved this by reprovisioning my Vagrant VM to use the Docker model. Now the entire /Users directory gets shared into the Vagrant VM. I then created a shell provisioner in my Vagrantfile that creates a symlink to keep all of my paths the same as they were before. If you don't care about the paths being exactly the same, you can just adapt to the new paths.

Hey @kusmierz ,
In addition to @silvamerica (btw thanks for the explanation) this script has /Users as hardcoded shared directory. If you want your docker-machine box to share a different directory, you'll (currently) have to change that manually in the script file. However, keep in mind that you will not be able to use commands like 'docker run -v $(pwd):/src' ... because your host path will not match your virtual machine path. (This is I guess one of the reasons that docker-machine by default shares /Users)
Hope this helps.


[oadp-1.4] remove restartPolicy for OCP 4.16

Why the changes were made
restartPolicy is no longer accepted in OCP 4.16, and it's not really needed
How to test the changes made
cd tests/e2e/sample_apps and deploy on 4.16

/lgtm


Fix wpa_supplicant systemd unit name

This seems to have been updated in nixos since Feb 2021. This ensures that the wpa_supplicant unit gets the correct options added.

LGTM


@conda-forge-admin, please add bot automerge

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/pywebp-feedstock#4.


did:ethr doesn't support the spec it references

The did-ethr resolver works great on etherium address DIDs (ones whose hex string is 40 chars long), but returns with invalidDid for public key hex DIDs (ones whose hex string is 66 chars long).  The spec that it ( https://github.com/spruceid/ssi/blob/6d82e3ef4f94c7c3292b4688e36a494bbcb24a07/did-ethr/src/lib.rs#L27 ) references is https://github.com/decentralized-identity/ethr-did-resolver/blob/master/doc/did-method-spec.md -- partial support wouldn't necessarily be a problem, except that I'm unable to override the resolver (I'm using didkit-cli or equivalently didkit-http) to use a custom DID resolver for did:ethr (such as universal resolver).
I think this ticket belongs to both ssi and didkit repos, since there are resolution paths for both of them.  Though obviously the better solution is to have ssi support the whole did:ethr spec.

Looking through the code, the problem is that anything passed to didkit for --did-resolver just gets tacked onto a Vec that's populated by DID_METHODS, and so presumably this means that if a particular DID method has a hit with any of these built-in resolvers (in particular ssi crate's did_ethr), it gets handled fully by that resolver.  And in the case of did:ethr, the resolver is only partially implemented, so it reports an error and then quits, instead of falling through to another resolver.
A good solution would be to allow for DID-method-specific resolver override.  Using a hypothetical new form for didkit-cli's did-resolve command,
didkit did-resolve --did-resolver --did-resolver "did:ethr https://resolver.identity.foundation/1.0/identifiers/ did:ethr:ropsten:0x0392196025ba090066590125ff3e9173e09bf537ca9582abb4fea861d1f652c35e"

where depending on how CLI args are implemented (I'm not familiar enough with the particular CLI parser), overrides for multiple DID methods could be specified either by repeated use of the --did-resolver arg, or by passing a single comma-separated list of did:method URL values.
An easier but more brutish way would be to clarify in the --help message that the existing --did-resolver always acts as a fallback (this fact is pretty vague currently), and add a new CLI arg such as --did-resolver-override which simply prepends the given HTTPDIDResolver value to the Vec of DID resolvers in ResolverOptions::to_resolver, so that it gets tried before any of the built-in resolvers.

For reference, the disparity in didkit vs universal resolver I'm encountering can be seen using the following commands:
Not working:
didkit did-resolve --with-metadata did:ethr:ropsten:0x0392196025ba090066590125ff3e9173e09bf537ca9582abb4fea861d1f652c35e

Working:
curl -X GET https://resolver.identity.foundation/1.0/identifiers/did:ethr:ropsten:0x0392196025ba090066590125ff3e9173e09bf537ca9582abb4fea861d1f652c35e

Hi @vdods, thanks for opening this and suggesting ways to fix it.
I agree with the idea of adding DID-method-specific resolver options.
As a quick fix for using a different resolver, I've added the override option in #209. It can be used with the URIs you mention, as follows (-R as alias for --did-resolver-override):
didkit did-resolve --with-metadata -R https://resolver.identity.foundation/1.0/identifiers/ did:ethr:ropsten:0x0392196025ba090066590125ff3e9173e09bf537ca9582abb4fea861d1f652c35e

This is merged. Please give it a spin and let us know if it works for you.

This works great, thanks for getting to it so quick!

Appreciate your patience! I will close this for now, and we'll keep working on https://github.com/spruceid/ssi/issues/287 for a permafix. Thanks again for the reports.


Publish Lexakai code quality article

Publish existing draft article on medium.com
https://medium.com/@jonathanlocke/lexakai-code-quality-assessment-9ae37a6436cd

https://medium.com/@jonathanlocke/lexakai-code-quality-assessment-9ae37a6436cd


Change camera position

I see that zooming and rotating is possible out of the box. How would one be able to pan the object? Or otherwise, how can one best change the camera position manually?
Thanks!

Not sure if I understand well what you want to do, but can't you achieve that by moving the object? By changing its position?


Add napari CLI to launch a viewer for input images

Description
napari *.tif

now launches a napari viewer with all the images concatenated along the z axis.
Lots of edge cases are not taken into account here but thought I'd get this started!
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

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

In the file napari/main.py:

Line 27:61: W292 no newline at end of file

This would now make file io something part of this repo, which I'm not 100% sure about curious how others feel? I think if we do support this we should also look into supporting drag-and-drop of .tif into an already open viewer to easily add images.
I had previously thought that file io would all be in a separate repo and a plugin but I'm open to this.
If I just type in napari does it open an empty viewer?

We could make all io optional, with a helpful error message if no io library is available.
To clarify the motivation for this PR, it's the tweet posted by @freeman-lab:
https://twitter.com/vallens/status/1105957771252400128
=)
Coordinating this over multiple repos would be challenging, I think.

If I just type in napari does it open an empty viewer?

No, it will demand an input argument, but it will do so nicely. =) (ie it will print the usage string, not crash)

How do you feel about adding drag / drop support into the viewer in the future? If so then having io in the repo makes sense (as does supporting opening an empty viewer with no images to start with).
The question then might be how far do we go in supporting input formats - i.e. can I drag and drop a zarr file too? Or data for a shapes layer / markers layer?

For the image reading do you think we should use imageio instead of skimage.io?
I guess you sort of answered the no to multiple repos which is fine for now, though I could see including imageio / skimage.io in a separate napari-io repo that would also include zarr / dask as a dependency for loading big files, and any code for doing io on our shapes / markers layers as one way to go, otherwise all those things might need to be dependencies of this main repo, but they aren't used for anything other than io. Could still make sense to keep it all in one place for now and then factor out later if needed

To summarize an in person discussion (which unfortunately @jni couldn't join!) there was general consensus that the CLI is super cool but should maybe live in a separate repo / package called napari-cli. The suggested design is that napari-cli depends on napari as a requirement, and additionally depends on a few io libraries. It will give us a place to play with the UX around file format handling. But it won't require us adding significant new dependencies to napari, or require keeping the cli and the library in perfect sync.
Here's a diagram we drew showing this relationship, and how it could be a model for a future standalone napari-app similarly consuming the library. The diagram also shows how the io stuff could get modularized out into a napari-io library that napari-cli and napari-app depend on.

Thoughts @jni ? I know you have some aversion to having too many repos =p

@freeman-lab yes, it is indeed unfortunate that I wasn't there to nip this conversation at the bud! =P
More seriously, I actually don't mind that structure, I think it's a good idea, as long as we can figure out the packaging so that pip install napari[all] brings down napari and napari-cli together. I want it to be super easy to install the napari cli viewer.
Also I might call it napari-command.
Side note: I don't want us to start another io package — we should contribute whatever we need to one of:

skimage io
imageio
PIMS
zarr/n5

How about napari-command-line?
I agree we can use this repo to figure out exactly what io support we need and then see if we can integrate those changes into existing repos where it makes sense. We might still end-up with some shared loading code between the app and the cli but we can try and keep it at a minimum

@sofroniewn well, it's a command. The command line takes commands and executes them. =)


feat: Make tour examples to support path-based routing #1875

A relative URL with / in the beginning replaces the entire path name of the base URL.
What we want here is to appended a relative URL to the full base URL and this can be achieved by removing the leading slash.
https://user-images.githubusercontent.com/23740173/234636257-ad5ee447-fff2-46b2-a066-3e60bfe1f4b1.mov
Closes #1875

please resolve the conflicts and we are good to merge

@mturoci , done  ✅


Added prop to createToaster() to pause all timers on hover

See issue #997.
A createToaster() prop to pause all timers on any toast hover :
const toaster = createToaster({
  pauseAllOnHover: true // false by default
})

Let me know if you'd want this feature, and I'll update the doc accordingly.
:)

Maybe a group option, similar to createTooltip could make more sense... Or, instead of a pauseAllOnHover, maybe a hover prop, with threeavailable options pause, pause-all and null, the default being pause.

Maybe a group option, similar to createTooltip could make more sense... Or, instead of a pauseAllOnHover, maybe a hover prop, with threeavailable options pause, pause-all and null, the default being pause.

hover prop makes sense to me. I'll update and finish the PR :smile:


Extension causes high cpu load

Issue Type: Performance
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.17763
VSCode version: 1.33.1

:warning: Make sure to attach this file from your home-directory:
:warning:C:\Users\RiverSaid\ritwickdey.LiveServer-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

C:\Users\RiverSaid\ritwickdey.LiveServer-unresponsive.cpuprofile.txt


Anyone have tips for downloading more than tens of posts from Hashtags

Without an account I can't download any posts from a hashtag. With an account It blocks you (429 error) at <100 posts per hashtag. I tried using the instaloader get_all_posts function with long sleep in the loop as I iterate over it. Like this:
L = instaloader.Instaloader()
hashtag = Hashtag.from_name(L.context, "python")
for i,post in enumerate(hashtag.get_posts()):
   L.download_post(post, target="#"+hashtag.name)
   time.sleep(1)

but this still occurs.
Are others experiencing this issue for hashtags? Does anyone know a way around this?

When using Instaloader as a python module, downloading of comments and geotags is enabled by default which requires additional requests. If you do not want to download them while fetching hashtag posts they can be disabled. For example, change the first line to the following:
L = instaloader.Instaloader(download_geotags=False, download_comments=False)

Have a look at our documentation for a more complete description of the parameters that are used with the Instaloader Class along with its default values.

Hi! Thanks for the reply. I turned off all extra downloading, and my timeout is like 6 seconds between posts. I always get 429 eventually. I thinking Instagram may have just blocked this type of scraping permanently. If you have any suggestions I would love to hear them.


Load client.js from https://utteranc.es is a mandatory?

I don't feel safe to load an external java script to my blog. (I need to add CSP rules in order to use utterances).
Is it possible to use the tool in standalone? It is not enough to just download client.js in my website.

That's a valid concern. The script is necessary because the utteranc.es iframe posts messages to the host page instructing it to resize the iframe.
https://stackoverflow.com/q/9975810/725866

I download client.js, utterances.html, utterances.3c7bd773.js and utterances.css. I changed URL path in the code and it seems it can be working. But the problem is about CORS api.utteranc.es. The token is sent from my website to api.utteranc.es and of course it is blocked by the browser. Do you think it is possible to fix this?


Labels are being dropped between the get-sli and the get-sli.done keptn events

Your issue may already be reported! Please search on the issue tracker before creating a new one.
Environment

Keptn Version (keptn version): 0.6.1
Kubernetes Cloud Provider (e.g., GKE, AKS): EKS

Affected Component

[x] SLI keptn services

Describe the bug

Labels are being dropped between the get-sli and the get-sli.done keptn events.
To Reproduce

Add a label to a start evaluation event and then review the sli done event.
Expected behavior
The labels in the event are not dropped.
Current behavior
The labels are being dropped between the get-sli and the get-sli.done events. The jira-service listens for the sh.keptn.internal.event.get-sli.done and will use the label with the JIRA issue # to then use that to update the right ticket who made the request for the performance as a self-service.
I pass labels (with the JIRA ticket #) in the sh.keptn.event.start-evaluation event
I see the labels ok in the sh.keptn.internal.event.get-sli event
But in the sh.keptn.internal.event.get-sli.done event, the labels show ("labels": null,)
If this is still up, here is an example:
https://bridge.keptn.jahn-keptn-quality-gate.alliances.dynatracelabs.com/project/bitbucket-demo/simplenodeservice/b5edb095-65ec-4f33-aec0-a6a7e4427ce0/e9e7fdbe-84fb-4e23-8a0a-75361b942611
Screenshots
Here is data from one keptn-context flow...see the  ("labels": null,) in the last image.

Additional context

This issue is for the prometheus-sli-service and the dynatrace-sli-service. Both don't add the labels from a sh.keptn.internal.event.get-sli to a sh.keptn.internal.event.get-sli.done.


libxslt: Depends on python

i was following the standalone instructions and i had a linking error while building libxslt
/bla/bla/.linuxbrew/bin/ld: cannot find -lpython2.6

it seems one either needs to keep LD_LIBRARY_PATH or install homebrew's python before actually trying to install libxslt. Installing python solved the issue for me.
p.s. I am on SuSE Linux Enterprise Desktop

Thanks for the info. I don't want to add a dependency on Python, because I don't want to force the user to install Python if they already have it installed. This is an issue though, and similar issues have come up related to Python dependencies.

i just though that perhaps this could be mentioned in the instructions so that users are aware of that dependency.

These issues and Google are the documentation, so far. The Wiki is publicly editable. You could, if you like, start a page title "Dependencies"


Add missing await

The build is broken on the analyze step because this test has an unawaited future.

lol, beaten to it by @jonahwilliams :)


Simplify a glob path mapping with prefix into a glob

Related: #1906
Motivation:
When a user builds a Route with pathWithPrefix(), the following
route:
pathWithPrefix("/prefix", "glob:foo")

can be simplified into:
path("glob:/prefix/**/foo")

only except that the ** must not be captured.
Modifications:

Simplify a glob path mapping with prefix into a simple glob path
mapping, not into PrefixPathMapping.
Add numGroupsToSkip option to GlobPathMapping so that we can skip
the ** group we added implicitly.
Miscellaneous:

Address the comments which were not addressed in #1906

Javadoc fixes
Removal of redundant public modifiers
Do not insert between prefix and regex when generating a span
name for PrefixPathMapping.

Result:

Much cleaner Brave span names and meter tags for the route with a
prefix and a glob.
Potentially better performance for the route with a prefix and a glob.

@adriancole PTAL :wink:

https://github.com/line/armeria/blob/master/core/src/main/java/com/linecorp/armeria/server/Route.java#L93
Could you update this please?

Could you update this please?

@minwoox Done. :sweat_smile:


Legend for colored images drawn with rdMolDraw2D.PrepareAndDrawMolecule?

Is it supported to add legend for each color to add some customized description for each color?

This is not currently possible and I think the API to allow it would be quite tricky


Add WSGI and ASGI dispatchers

We should allow clients to trivially plug into WSGI and ASGI apps.
This will allow:

Using httpcore as a test client.
Stubbing out third-party-services during tests, rather than making live network requests.

We should support the following:
def hello_world(environ, start_response):
    ...

client = Client(app=hello_world)

And...
async def hello_world(scope, receive, send):
    ...

client = Client(app=hello_world)

This is easy enough for us to do, since we've already got a dispatch=... interface, so we'll want something like this:
if app is not None:
    if is_coroutine(app):
        dispatch = ASGIDispatcher(app)
    else:
        dispatch = WSGIDispatcher(app)

We'll then need ASGIDispatcher and WSGIDispatcher implementations, which implement Dispatcher.send().

Related #79

Done.


Comments are broken

Can't comment on any level, instead it just shows a single ghost comment. (No failed popup)

This is a known issue, and is being worked on.

Fixed by disabling error reporting in getGJComments.php. (error_reporting(E_ALL & ~E_WARNING ~E_DEPRECATED))


Pop up an error in 'DEBUG CONSOLE' window after getting the URL for JS

Repro Steps:

Create a new project with JavaScript language and v2 runtime->'Create New Function' with the new created folder and HTTP trigger template.
Debug -> Then check the 'DEBUG CONSOLE' window.

Expect:
No error display.
Actual:
Pop up an error in 'DEBUG CONSOLE' window after getting the URL for JS whatever the nodejs version is v8.12.0 or v10.8.0.

Closing as won't fix. That warning is expected and shouldn't actually stop you from debugging. Re-open if it blocks you from debugging.


[Travis] Use build-for-testing and test-without-building build actions on Xcode 8

This is workaround for timeout of testing on simulator.
It seems this workaround works: https://travis-ci.org/norio-nomura/Nimble/builds/151695213
https://github.com/ikesyo/Himotoki/pull/133
https://speakerdeck.com/ikesyo/playing-around-with-xcode-8?slide=9

Neat. Better then previous workarounds that force open the Simulator.


Create Dashboard skeleton application with Events as example

Closes: #...
Checklist

[ ] I have written tests
[ ] I have provided documentation

Changelog

A bullet list of what you've done to make the life of the reviewer easier.

How to test
A consise description of how to test the newly implemented functionality.

Please provide an iamge


Using Mixed .NET Versions for Client and Server Applications

Is your feature request related to a problem? Please describe.
I am attempting to have a .NET Core-built client connect to and use services registered on a .NET Framework-built server.
Out of the box, trying to do this would cause errors when the client makes a call on a proxy object while the client and server are using two different versions of the same Type.
For example, registering and invoking an Action on a proxy would send a message containing information for a version 6.0 Action Type, referenced from System.Private.CoreLib.dll, to the server which only knows about version 4.0 referenced from mscorlib.dll. This causes any Type.GetType() calls in RemotingSession to return null and fail. This is with using a .NET Core 6.0 client and a .NET Framework 4.8 server.
Describe the solution you'd like
I'd like support for at least a .NET Core client to work properly with a .NET Framework server. Ideally, any configuration of client(s) and server(s) using .NET Standard 2.0-supported .NET versions would work properly.
Describe alternatives you've considered
Additional context
I was able to create a one-way (client -> server) implementation of this feature by adding an additional step in ProcessRpcMessage in RemotingSession. It works by using the method and parameters passed in the client message to search for the equivalent method in the registered service on the server. The details of the message's method and its parameters are replaced with the server-equivalent method and parameter information. There is also additional searching and matching for overloaded methods.
I tested this with the HelloWorld example by added/changing the following methods and switching HelloWorld.Server to use .NET Framework 4.8. Additional changes to its .csproj may be needed to avoid build errors.
New Methods in RemotingSession:
        /// <summary>
        /// Converts argument types to those that can be used by the server.
        /// This is needed in the case where the server and client apps are built with different
        /// .NET platforms. For example, when server is of .NET Framework and client is of .NET Core,
        /// the string type is in System.Private.CoreLib for the client, but it's mscorlib for the server
        /// </summary>
        /// <param name="callMessage">Contains the parameters that are sent from the client and may need to be converted.
        /// </param>
        private void ConvertToServerTypes(MethodCallMessage callMessage)
        {
            var serverMethod = GrabServerMethod(callMessage);
            if (serverMethod == null) return;

            var serverParameters = serverMethod.GetParameters().Select(x => x.ParameterType).ToList();
            var messageParameters = callMessage.Parameters;

            for (int i = 0; i < messageParameters.Length; i++)
            {
                string localAssemblyQualifiedName = serverParameters[i].AssemblyQualifiedName;

                messageParameters[i].ParameterTypeName = localAssemblyQualifiedName;
                if (messageParameters[i].Value is RemoteDelegateInfo oldValue)
                {
                    messageParameters[i].Value = new RemoteDelegateInfo(oldValue.HandlerKey, localAssemblyQualifiedName);
                }
            }
        }

        /// <summary>
        /// Grabs the server-equivalent of the method called by the client
        /// </summary>
        /// <param name="message">Contains the method and parameters that are sent from the client.
        /// </param>
        private MethodInfo GrabServerMethod(MethodCallMessage message)
        {
            var localMethods = _server.ServiceRegistry.GetService(message.ServiceName).GetType().GetMethods().Where(x => x.Name == message.MethodName);

            if (localMethods.Count() == 1) return localMethods.First();

            var messageParameters = message.Parameters.Select(x => x.ParameterTypeName.Split(',')[0]);
            foreach (var method in localMethods)
            {
                var methodParameters = method.GetParameters().Select(x => x.ParameterType.FullName);
                if (messageParameters.SequenceEqual(methodParameters))
                {
                    return method;
                }
            }
            return null;
        }

Additional step in ProcessRpcMessage:
        /// <summary>
        /// Calls a method on a server side service.
        /// </summary>
        /// <param name="request">RPC message from client</param>
        /// <returns>Task which provides the serialized response message containing the method result asynchronously</returns>
        /// <exception cref="MissingMethodException">Thrown if specified method in request doesn't exist</exception>
        private void ProcessRpcMessage(WireMessage request)
        {
            var sharedSecret =
                MessageEncryption
                    ? SessionId.ToByteArray()
                    : null;

            var decryptedRawMessage =
                _server.MessageEncryptionManager.GetDecryptedMessageData(
                    message: request,
                    serializer: _server.Serializer,
                    sharedSecret: sharedSecret,
                    sendersPublicKeyBlob: _clientPublicKeyBlob,
                    sendersPublicKeySize: _keyPair?.KeySize ?? 0);

            var callMessage =
                _server.Serializer
                    .Deserialize<MethodCallMessage>(decryptedRawMessage);

            ConvertToServerTypes(callMessage);

            ServerRpcContext serverRpcContext =
                new ServerRpcContext
                {
                    UniqueCallKey =
                        request.UniqueCallKey == null
                            ? Guid.Empty
                            : new Guid(request.UniqueCallKey),
                    ServiceInstance = null,
                    MethodCallMessage = callMessage,
                    Session = this
                };
        ...

I think it would help if RemotingServer fires a type mapping event whenever a type needs to be loaded from a message. Then you could convert client types to server types at a central location in your CoreRemoting application, without touching the infrastructure code.
I'll implement this soon.

Also hit this when I was using a netcore client with a netfx server. I think the issue here is the TypeNameHandling on the BSON serialization, am I right? If so, this workaround might help: https://github.com/JamesNK/Newtonsoft.Json/issues/1378#issuecomment-889265856 . I'll try it out soon.

The latest version <IP_ADDRESS> of CoreRemoting (https://github.com/theRainbird/CoreRemoting/releases/tag/<IP_ADDRESS>) adds support for cross framework serialization.
This is done by forwarding types to another assembly.
To forward .NET Core types to .NET Framework types, add the following line to your program.cs:
CrossFrameworkSerialization.RedirectPrivateCoreLibToMscorlib();
To forward .NET Framework types to .NET Core types, use this line:
CrossFrameworkSerialization.RedirectMscorlibToPrivateCoreLib();
Use it on server AND client depending on the framework type they are running on.
For your custom assemblies, use:
CrossFrameworkSerialization.RedirectAssembly("FromCustomAssembly", "ToCustomAssembly");
Replace "FromCustomAssembly" and "ToCustomAssembly" with the real assembly names.

Great work @theRainbird . I'll check it out (and the other issue) soon. Just need to find time.


upgrade versions to work with Java 9 & 10

small changes to project.clj

The diff on this is pretty massive, are you able to open a branch which contains just what is needed for Java 9/10?


Remove deprecated useIR option

Fixes #2075
Option is already true by default and was removed in Kotlin 1.7.
Removed it in the Compose code so it no longer throws a java.lang.NoSuchMethodError.

Here you can see it was removed:
https://kotlinlang.org/docs/compatibility-guide-16.html#deprecate-useir-compiler-option
https://youtrack.jetbrains.com/issue/KT-48847

Here you can see it was removed:
https://kotlinlang.org/docs/compatibility-guide-16.html#deprecate-useir-compiler-option
https://youtrack.jetbrains.com/issue/KT-48847

Thanks for merging the PR Igor! Any chance you could publish a new build? That would be a bit easier for me instead of using a maven local build.

You can use 1.2.0-alpha01-dev703. It is not in the list of releases, because some tests were failed.

because some tests were failed.

These tests was regarding popup's. It seems they don't work in this release.

Okay, thanks for the heads up, Igor. Apart from the popups all is working fine with Kotlin 1.7.0-RC2 with the build you mentioned, just like my maven local build.


Equation numbering

When https://github.com/QuantEcon/sphinxcontrib-jupyter/issues/136 is resolved, we need to implement whatever changes come out of it.
Thanks @PooyaFa for finding this, and also @qirui-ubc's PR https://github.com/QuantEcon/lecture-source-jl/pull/353

Closing this since the numbering is not stochastic and the internal equation hyperlinks work on the webpage.


Brackets LivePreviwe Error

Hi,
Am using Brackets Editor i am working on it perfectly  first time execution after that i reload  browser and write a code in editor it displays error message like Live Preview Error.

@purnachandranaidu  Could you share code or files for whom this issue is occurring.

Hi,
I Learning Selenium Webdriver  but i have doubts please give Sessions and guidance of my project.
i post my project URL :
http://<IP_ADDRESS>/REGEN/viewer/aqua.html?resource=Library%3A%2F%2FREGEN%2FLAYOUT%2FREGEN3.ApplicationDefinition
above URL how to close and open widgets Ex:
i open the SearchWidget and after that how to open POI Search,
driver.findElement(By.xpath("//div[@title='Search-Industrial Area, Plot / Find Nearby']//img")).click();
driver.switchTo().frame(0);
Same code i wrote for POI Search just i change @title='POI Search' it's not woking
pls give me sussions
Thanks
Purna


Fix ondemand pipeline using wrong branch for setup scripts

Description
on demand action is using release branch scripts to run e2e tests, wich is wrong
How can this be tested?
Run ondemand piplines and check if they use the correct branch

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.06%. Comparing base (080fbb3) to head (d0b5862).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3826   +/-   ##
=======================================
  Coverage   65.06%   65.06%
=======================================
  Files         397      397
  Lines       21414    21414
=======================================
  Hits        13932    13932
  Misses       6143     6143
  Partials     1339     1339

Flag
Coverage Δ

unittests
65.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Make shared stacks visible again

Describe changes
I synced the /stacks and /projects//stacks endpoints to use the same logic. Also made it so that the user filter can not be used to get someone elses private stacks.
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

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

@fa9r I know this is yet another reason to have one endpoint instead of two for each functionality, looking forward to when we get there :)


feat: in translated docs, show if the English version is newer

Is your feature request related to a problem? Please describe.
With Arabic and Russian docs translations underway, we should start thinking about how to integrate the docs well.
One common issue with docs translations is that they become out of date as the English version gets more updates. Explicitly showing that could give people a warning that there might be newer information, and maybe also motivate more people to contribute to translations.
Describe the solution you'd like to see
A component that shows up on translated docs if the English version of this page is newer than the translated version.
Something like “This translation was last updated on (date). Since then there have been (number) revisions to the English version, with the newest one being on (date). Click here to see the English version of this page. Or click here to help us update the translated version” (I’m sure there is a better way to write this)
Of course this component would ideally be translated into each of the languages we support in the docs.
Describe alternate solutions
Not doing this
Additional information
I don’t think it’s super urgent as all the incoming translations will be new for the time being, but it would be an awesome thing to have.

I don’t think it’s super urgent as all the incoming translations will be new for the time being, but it would be an awesome thing to have.

yes i think it's a crucial thing to have.
we also need to make sure we have maintainers willing to review and update docs in the respective language - although i dont think that'll be a problem in this community.

we also need to make sure we have maintainers willing to review and update docs.
I'm ready for that.


How to send request with async-await style?

As per the docs, here's
final signOutReq = GLogoutReq((b) => b..vars.deviceToken = param);
client.request(signOutReq).listen((response)=>{
  // Do something
});

how to do it like:
final signOutReq = GLogoutReq((b) => b..vars.deviceToken = param);
await final result = await client.request(signOutReq);

I'm asking because the current app I am making is currently based on clean architecture datasource - repo - usecase. Thanks

duplicate of https://github.com/gql-dart/ferry/issues/217.
If you are only interested in the first result of a Dart Stream, you can use the first property which provides a Future.
Please direct these kinds of general questions to Stack Overflow or the Discord.


Disqus Dark Mode - Text Invisible

When the dark mode toggle is enabled Disqus comments aren’t visible, appears Disqus has a dark/light mode but it needs to be manually enabled. Not sure if this is fixable with a CSS override but thought it was worth mentioning as Disqus is Ghosts recommendation for comments.
Update
Just noticed it only happens setting changing the mode while on a post page, if the page is loaded in dark mode comments will work, when switched to light they will become the same colour as the background.
Example below. Thanks!

Hey @Slethen,
According to Disqus,

A light or dark color scheme is automatically selected based on your site's stylesheets.
The light scheme is loaded when the text color Disqus inherits from your site has >= 50% gray contrast: between color: #000000 and color: #787878
The dark scheme is loaded in all other instances.

That means when your site is in dark mode, Disqus will choose its dark mode automatically.
If you're on a single post, and change your site theme from light to dark, Disqus' script would require a refresh to calculate the colors.


Refactor Config loading and Tests

config loading can now be tested
multiple test config files can be placed in ./testfiles
other refactors and improvements

ref #10
closes #7
closes #6

Very nice! Lots of improvements here :)


Owner ID Issue

When putting the owner id in it comes up as <@!297203777331658763> starting with an ! instead of a 1. If the 1 isn't in the front the bot will crash and not open how do I solve this?

Your owner ID is simply<PHONE_NUMBER>31658763


Better approach to use std::variant

🦟 Bug fix
Fixes #
Summary
Using exception for a flow control is not a very good approach in a different points of view, like from a code structure, from a performance perspective (https://godbolt.org/z/s883vGr9f)
I'd propose to use std::holds_alternative as a more simple way to improve this part of code.
Also improved some typos
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@mergifyio backport gz-rendering8


MACCSKeys() generates 167 binary values?

Description:
I'm new to analyzing chemical structures, and I'm trying to generate MACCS keys with rdkit. Based on my understanding MACCS keys should have 166 binary values but when I tried it with MACCSkeys.GenMACCSKeys(), it returned a variable with 167 binary elements. Based on the source code (MACCSkeys.py), it looks like index 0 should just be ignored, because it's really setting values to positions 1 to 166... I'm guessing for ease of use. Please correct me if I'm wrong. Thank you!

RDKit Version: 2017.09.3
Platform: Python 3.6.4 on Red Hat Enterprise Linux Server release 6.6 (Santiago)

>>> from rdkit import Chem
>>> from rdkit.Chem import MACCSkeys
>>> key = MACCSkeys.GenMACCSKeys(Chem.MolFromSmiles('C1(=C(SSC1=O)Cl)Cl'))
>>> len(key)
167

Thanks for the verification!


Don't mangle ScrollTimeline and ViewTimeline class names to satisfy wpt.

@johannesodland @bramus This is an alternative to #200 now that we can use terser and configure exactly which classnames it shouldn't mangle. This addresses item 2 from #219.

Nice 😀🙏🏻

Here's a size comparison of the different options:

keep_classnames
size

false
51490 bytes

/(Scroll|View)Timeline/
51705 bytes

true
51949 bytes

Given the relatively minimal change to code size, it's probably fine to keep all classnames in the interest of correctness.


Bare bones evaluator which allows queries over striped data

This probably doesn't even work yet but just wanted to give an overview of the design in a piecemeal way.
I will hook it up to the corpus / well typed tests when #549 gets merged. For the moment I'm going to focus on pushing it all the way through to the repl.
! @amosr @tranma

looks plausible 👍

looks plausible 👍

(only barely!)


rafactor: extract EndpointResolverMiddleware from Service clients to AWSClientRuntime

Describe the feature
Currently EndpointResolverMiddleware is part of service client which as good amount of duplication. This was intentional due to EndpointResolver and EndpointParams being service specific. I couldn't able to generalize the middleware due to limitations with Swift type system.
Use Case
Reduce code
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Completed with https://github.com/awslabs/aws-sdk-swift/pull/1537.


fix: update all dependencies

BREAKING CHANGE: Something probably broke, but I do not know what
Closes #23
cc @benlesh, there's something busted in the tests. I'm afraid I haven't used or maintained this package in years and I don't have bandwidth to invest any more time into it. If you want you can figure out what's missing from this and I'll be happy to release an up-to-date version. Thanks!

Ah. What do you use now?

I copy/paste/modify. For example:

Exercise: https://github.com/kentcdodds/react-fundamentals/blob/main/src/exercise/05.js
Final: https://github.com/kentcdodds/react-fundamentals/blob/main/src/final/05.js

Split guide was pretty neat, but it was also pretty complicated. I think this is simpler.


VS Code not opening in Ubuntu 20.04

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.58.1
OS Version: Ubuntu 20.04

Recently, I boot my laptop and launched VS Code (from the terminal). I even launched Emulator from within the editor which I used for Flutter.
Thereafter, I closed VS Code and tried opening another project, only to see nothing.
I disabled all extensions but still, same.
When I code . --verbose, I got :
/usr/lib/x86_64-linux-gnu/gio/modules/libgvfsdbus.so: undefined symbol: g_date_time_format_iso8601
Failed to load module: /usr/lib/x86_64-linux-gnu/gio/modules/libgvfsdbus.so
(code:289448): Gtk-WARNING **: 17:25:40.420: Theme parsing error: gtk.css:1511:23: 'font-feature-settings' is not a valid property name
(code:289448): Gtk-WARNING **: 17:25:40.425: Theme parsing error: gtk.css:3437:25: 'font-feature-settings' is not a valid property name
(code:289448): Gtk-WARNING **: 17:25:40.426: Theme parsing error: gtk.css:3899:23: 'font-feature-settings' is not a valid property name
[289480:0802/172540.557655:ERROR:angle_platform_impl.cc(43)] Display.cpp:832 (initialize): ANGLE Display::initialize error 12289: glXQueryExtensionsString returned NULL
[289480:0802/172540.558269:ERROR:gl_surface_egl.cc(774)] EGL Driver message (Critical) eglInitialize: glXQueryExtensionsString returned NULL
[289480:0802/172540.558633:ERROR:gl_surface_egl.cc(1329)] eglInitialize OpenGL failed with error EGL_NOT_INITIALIZED, trying next display type
[289480:0802/172540.561106:ERROR:angle_platform_impl.cc(43)] Display.cpp:832 (initialize): ANGLE Display::initialize error 12289: glXQueryExtensionsString returned NULL
[289480:0802/172540.561743:ERROR:gl_surface_egl.cc(774)] EGL Driver message (Critical) eglInitialize: glXQueryExtensionsString returned NULL
[289480:0802/172540.562131:ERROR:gl_surface_egl.cc(1329)] eglInitialize OpenGLES failed with error EGL_NOT_INITIALIZED
[289480:0802/172540.562467:ERROR:gl_initializer_linux_x11.cc(182)] GLSurfaceEGL::InitializeOneOff failed.
[289480:0802/172540.567492:ERROR:viz_main_impl.cc(150)] Exiting GPU process due to errors during initialization
libva error: dlopen of /usr/lib/x86_64-linux-gnu/dri/iHD_drv_video.so failed: /snap/core/current/lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /usr/lib/x86_64-linux-gnu/dri/iHD_drv_video.so)
libva error: dlopen of /usr/lib/x86_64-linux-gnu/dri/i965_drv_video.so failed: /snap/core/current/lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.27' not found (required by /usr/lib/x86_64-linux-gnu/dri/i965_drv_video.so)
[main 2021-08-02T16:25:40.656Z] [File Watcher (node.js)] Failed to watch /home/unknown/.config/Code/User for changes using fs.watch() (Error: ENOSPC: System limit for number of file watchers reached, watch '/home/unknown/.config/Code/User')
[main 2021-08-02T16:25:40.661Z] [File Watcher (node.js)] Failed to watch /home/unknown/.config/Code/User/settings.json for changes using fs.watch() (Error: ENOSPC: System limit for number of file watchers reached, watch '/home/unknown/.config/Code/User/settings.json')
[main 2021-08-02T16:25:40.693Z] Starting VS Code
[main 2021-08-02T16:25:40.694Z] from: /snap/code/70/usr/share/code/resources/app
[main 2021-08-02T16:25:40.694Z] args: {
  _: [ '/home/unknown/Documents/Flutter/sign_up_login_ui' ],
  diff: false,
  add: false,
  goto: false,
  'new-window': false,
  'reuse-window': false,
  wait: false,
  help: false,
  'list-extensions': false,
  'show-versions': false,
  version: false,
  verbose: true,
  status: false,
  'prof-startup': false,
  'no-cached-data': false,
  'prof-v8-extensions': false,
  'disable-extensions': false,
  'disable-gpu': false,
  telemetry: false,
  debugRenderer: false,
  logExtensionHostCommunication: false,
  'skip-release-notes': false,
  'skip-welcome': false,
  'disable-telemetry': false,
  'disable-updates': false,
  'disable-keytar': false,
  'disable-workspace-trust': false,
  'disable-crash-reporter': false,
  'crash-reporter-id': 'cb38fdc7-3d56-46f5-afab-ae608927e29a',
  'skip-add-to-recently-opened': false,
  'unity-launch': false,
  'open-url': false,
  'file-write': false,
  'file-chmod': false,
  'driver-verbose': false,
  force: false,
  'do-not-sync': false,
  trace: false,
  'force-user-env': false,
  'force-disable-user-env': false,
  'open-devtools': false,
  __sandbox: false,
  'no-proxy-server': false,
  'no-sandbox': false,
  nolazy: false,
  'force-renderer-accessibility': false,
  'ignore-certificate-errors': false,
  'allow-insecure-localhost': false,
  logsPath: '/home/unknown/.config/Code/logs/20210802T172540'
}
[main 2021-08-02T16:25:40.697Z] Resolving machine identifier...
[main 2021-08-02T16:25:40.697Z] Resolved machine identifier: d592eb5ccfcc126764b2eb0f48054c950b09e251eb8a3ca0e06c7fbacef7fe7c
[main 2021-08-02T16:25:40.698Z] Main->SharedProcess#connect
[main 2021-08-02T16:25:40.744Z] update#setState idle
[main 2021-08-02T16:25:40.751Z] Error: ENOSPC: System limit for number of file watchers reached, watch '/snap/code'
    at FSWatcher.<computed> (internal/original-fs/watchers.js:218:26)
    at Object.watch (original-fs.js:1525:34)
    at new y (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:58:20816)
    at i._createInstance (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:1371)
    at i._createServiceInstance (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:3772)
    at i._createServiceInstanceWithOwner (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:3288)
    at i._createAndCacheServiceInstance (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:2963)
    at i._safeCreateAndCacheServiceInstance (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:2161)
    at i._getOrCreateServiceInstance (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:1882)
    at Object.get (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:450)
    at Pe.initChannels (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:59:63218)
    at /snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:59:58764
    at invokeFunction (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:29:319)
    at Pe.startup (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:59:58740)
    at async se.startup (/snap/code/70/usr/share/code/resources/app/out/vs/code/electron-main/main.js:61:1899)
[main 2021-08-02T16:25:40.751Z] Lifecycle#kill()

I have no idea what it means.
If I should restart my PC and try again, VS code will work. And If I close it, and tried opening it up again, it will not come up.

Seems this is not VS code issue.
Running echo fs.inotify.max_user_watches=524288 | sudo tee -a /etc/sysctl.conf && sudo sysctl -p fixed it.
Ref


Add Note to the readme to must using the init only one time in the app

That's great, thanks @mahmoudalaa97!
I've also added a version to throw a dart exception if you call it a second time so that a native code exception is not thrown.


Long distance range settings

Required Info

Camera Model
D415/D435

Operating System & Version
Linux (Ubuntu 16

Kernel Version (Linux Only)
4.14.13

Platform
x86_64

SDK Version
2.15

Language
C++

Hello,
I would like to know which are the proper settings for measuring accurate distances between 3.5m and 6m. I played with Depth Quality tool without success. In that range I get variations for the same surface higher than 40 cm.
If you can also tell me some tips about how to tune the settings depending on the distance range is fine.
Thanks  in advance,
Jose Gomez

Pages 9 and 13 to 16 of the beautifully illustrated camera tuning guide give tips on accuracy over distance.  The guide as a whole is well worth reading too.
https://realsense.intel.com/wp-content/uploads/sites/63/BKM-For-Tuning-D435-and-D415-Cameras-Webinar_Rev3.pdf

Page 13 of the guide has a formula for calculating RMS error.

As the RMS / Depth chart on that page suggests, a high amount of RMS error at distances up to 6 m may be unavoidable if using the D435 camera model.  The D415 has less RMS error over distance.
Rather than use disparity shift to change MinZ and MaxZ, I would be more inclined in this case to change the depth units scale, as described on page 21.

[Realsense Customer Engineering Team Comment]
Hi jgomezlopez,
Do you still need help on this one?
Thanks!


Can this module be used without the client need to set proxy configuration on browsers?

Hi
Thanks for dveloping this great module and sorry for such an amatuer question.
I've successfuly implemented this module and get http 200 response when I use curl on my client (windows CMD or the proxy erver).
Is there anyway to use this proxy on browsers or third-party apps without the need of configuring proxy for them? something like non-Connect nginx http forward proxy which can be used as a url like this:
<proxy server address or IP>:<proxy port>/<[proxy_connect_address]>
I can't access the disired address when trying it like this.
Thanks in advance.

Is there anyway to use this proxy on browsers or third-party apps without the need of configuring proxy for them?

Its unable to do this.
In order to activate the tunnel, communication is necessary between the client and server. The client must send a CONNECT request to the server to ask it to open the tunnel. If you haven't configured the client with the CONNECT proxying setting, it will not automatically send a CONNECT request to indicate that it wants to use the CONNECT tunnel.

If you're looking for information on how to configure your browser to use an HTTPS proxy, you can check out this link for guidance: https://github.com/chobits/ngx_http_proxy_connect_module/issues/22#issuecomment-346941271

I am closing this issue. Please feel free to reopen it if you have any further questions.


Exporting a styled component as default leads to Prop className did not match.-Warning

Version
2.2.2
babel-plugin-styled-components: 1.3.0

Reproduction
Exporting a styled component as default leads to Prop className did not match.-Warning

https://www.webpackbin.com/bins/-KeeZCr0xKfutOfOujxN
https://codesandbox.io/s/rkmNRByE4
Steps to reproduce
Create a JS-file, e.g. named "Flex.js" in which you directly export a styled component:
import styled from 'styled-components'

export default styled.div`
    display: inline-flex;
`

Expected Behavior
No warning in browser-console
Actual Behavior
Warning in the form of Warning: Prop 'className' did not match. Server: "Flex-s1rf88eb-0 dMObJs" Client: "Flex___default-s1rf88eb-0 bunKfG"

Good catch, mind copying this bug report over to https://github.com/styled-components/babel-plugin-styled-components?
Thanks!


feat: upgrade Framework7 to v2.0.1

Resolve #17

@clarkdo nice! Ready to tag a new release?

/cc @pi0

@robsontenorio I'll have to make some local tests before next release. Will do it today.


Fatal error: Uncaught exception 'RuntimeException' with message 'Curve curve

Hi,
Issue is already raised at https://github.com/web-push-libs/web-push-php/issues/14
I tried to send push using V4, but code throwing new error messages. Please find error message below.

Warning: hex2bin(): Hexadecimal input string must have an even length in C:\xxxxxx\web-push-php-example\vendor\minishlink\web-push\src\Utils.php on line 60

Warning: hex2bin(): Hexadecimal input string must have an even length in C:\xxxxxxxx\web-push-php-example\vendor\minishlink\web-push\src\Utils.php on line 61

Fatal error: Uncaught TypeError: bin2hex() expects parameter 1 to be string, boolean given in C:\xxxxxx\web-push-php-example\vendor\minishlink\web-push\src\Encryption.php on line 94

TypeError: bin2hex() expects parameter 1 to be string, boolean given in C:\xxxxx\web-push-php-example\vendor\minishlink\web-push\src\Encryption.php on line 94

i am facing the same issue. in earlier version, in more than 70% of the cases, the error was curve point error which is issue number #14  and now it is the above mentioned issue. it does not come for all subscribers but is throwing the error for more than 70% of the cases. Please help.

Please fill in the issue template. Also provide your php.ini

Hi Minishlink,
I already fill the issue template, but don't know where to check it.
Sending it again below manually.

Yes , I have read the README entirely
Yes, I have verified in the issues that my problem hasn't already been resolved
I am tried on both php 5.6 and php 7.1
installed and enabled these PHP extensions : gmp, mbstring, curl and openssl
experiencing problems with : Chrome, Firefox
Features used : VAPID Support
Issue in php 7 is already mentioned above.
Issue in php 5.6 is raised at https://github.com/web-push-libs/web-push-php/issues/14

@Minishlink any update on the issue so far?

Same issue here unable to solve it

If that helps someone, in my case it was an error in my code, nothing lib related. This comment actually pointer me to the right direction:

Please make sure that the user public key and user auth token are base 64 encoded (in the MIME specification RFC2045).
A frequent mistake is to encode in URL-safe base 64.
https://github.com/web-push-libs/web-push-php/issues/12#issuecomment-217502223


Add compatibility with browser environments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

94 of 103 (91.26%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-2.03%) to 97.778%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

index.js
11
20
55.0%

Files with Coverage Reduction
New Missed Lines
%

index.js
1
97.79%

Totals

Change from base Build<PHONE_NUMBER>:
-2.03%

Covered Lines:
558

Relevant Lines:
568

💛  - Coveralls


[Query] thresholds and time sequencing in read query #8

The threshold for edge1 (transfer) and the threshold for edge2 ( withdraw ) should not be the same.
The timing of transfer and withdrawal is not significant as long as they are within the range of the inquiry time window.

Have talked with @tian-zhong , such a time sequencing design exists in some scenarios like finding exact transfer traces. We need to reconsider the query cases in transaction workload if they aim to find transfer traces. If not, we may need to add more queries that can be representative.


Scale the logo of generated events properly

Fixes #1277
Changes:

Scales the logo height of the events generated.

Approach

There were alternatives ways to do this. One of them was to scale the logo to a fixed dimension during the generation process. The disadvantage of that approach was that the event logo comes in different size and shapes. So resizing them to a fixed size will change its aspect ratio and it will appear stretched and pixelated. A much simpler approach was to fix the max-height of the logo displayed on the site. The width of the logo will adjust automatically according to the height of the logo. This way, the height of the sub -navbar doesn't increase too much and the logo doesn't look stretched or pixelated.

Screenshots for the change:

Test Server:
http://peaceful-earth-19050.herokuapp.com
@aayusharora @geekyd Please review. Thanks!!

@Princu7 Have you tried this for other events?

@geekyd Yep. Generated app for the FOSSASIA Summit and it looked decent. Please test it. Thanks :)

@geekyd Thanks for the review. That is a better approach. Would change according to it :)

@geekyd @aayusharora I tried to follow the above approach. The issue is that the approach works well when the size of the logo is greater than 25px but when the size of the logo is below 25px, the UI suffers. eg: - on the FOSSASIA 16 event:

Please suggest. What could be the correct approach? Resize the logo using the generator?

@Princu7 One solution could be scaling the logo while generating the web app. Here is how we can solve the problem:

Get height & width of the logo in the API/ZIP.
Calculate the aspect ratio width:height of the logo.
Now scale it down while maintaining the aspect ratio.

By doing this we will have a common height & now we can use a fixed padding for the logo to place it on the navbar appropriately. @aayusharora Please share your views regarding this.

@geekyd sounds correct, that's the only way to maintain this.
@Princu7 please follow this. Thanks :)

@geekyd Thanks for suggesting the way. I tried to do it that way. The small problem is that by transforming the size of the logo during the generation, we are able to resize it to a constant height, say, 45px. And the width of the logo is then set according to the previous aspect ratio. Now in the event part, we have set the max-width of the logo to be 110px. This means that whenever the width of the logo will cross 110px, it will get trimmed and the height will automatically be set according to the aspect ratio.
This creates a problem in logo where the size of the width is much larger than 110px. For Example,  FOSSASIA 16. Here the size of the logo after resizing comes around to be 264 * 45. We have fixed height of 45px. But since we have max-width of 110 px applied to it, the height automatically adjusts to around 18px in accordance with the aspect ratio. And with padding applied on the anchor tag above, it looks misaligned.

One of the straight ways to solve the issue is to increase the max-width of the image. But in some cases, the width becomes a bit too large IMO. eg:-

I am trying to do some changes and see which looks best. Suggestions would be welcome. Thanks @aayusharora @geekyd

Ok. I tried many different settings and it seemed to work the best. Here are the screenshots of different events:-

Facebook Developer Conference

Mozilla All Hands

Fossasia Summit

Open Tech Summit

@aayusharora @geekyd Please review. Thanks!!

@aayusharora
Sorry, this is taking longer than usual. Yes. Earlier, I was doing the same thing:

Resized the logo image to a fixed height of 45px and set its width according to the initial aspect ratio of the image. The height will always be 45px and only the width of the logo will vary after this process. Then applied a padding of 5px on the anchor tag above the image tag.

The following property was applied on the image previously
max-height: 45px; max-width: 110px;
The problem with it appeared when the width of the logo was much higher than 110px. The exact size of FA logo after resizing was 264 * 45. Now since the max-width set was 110px, the height was automatically adjusted to 18px (in accordance with the aspect ratio) and it looked like this:-

To make it look better, we have to adjust the height properly and in order to do that,  we had to increase the max-width to around 220px which was a bit too wide for a logo:-

So, I went with the following setting,
height: 45px; max-width: 160px;
We know that the height of the logo will surely be 45px and its width may vary (due to resizing step in the generation step). The height is hard-coded but it is because we are sure that the height of the logo will be 45px due to resizing done in the generation step. The constraint is applied on the max-width of the image to ensure that it doesn't become too large in the case of some logos like FA. The logo of FA is still not perfect but looks better now (it has shrunk a bit due to the less width offered) and other logos are appearing perfectly. If we don't set height rigidly, then when the width is very large like in the case of FA logo, the height will shrink and it will become misaligned like shown in the previous screenshots.
I hope I was able to convey what I was thinking and doing. Thanks for the guidance. Really appreciate it. There might be a way to solve this using some javascript but please suggest any alternative suggestion. Thank you so much :)

@Princu7 Why do we need to apply max-height & max-width when we are resizing the image while generation?

@geekyd During generation, we are only fixing the height of the image to 45px and its width is set according to the initial aspect ratio of the image. Now, this width may be quite large and not suitable to display like in the FA logo, where the width comes out to be 264px. That's why I was applying only the max-width property not the max-height property since the height is fixed but the width may vary.

@Princu7 Can you please check my review comment regarding this.

@aayusharora I am really sorry. Yes, I read that but didn't totally understand what you were saying. Can you please elaborate. Thank you so much :)

@aayusharora @geekyd So, I redid this using a different approach using dynamic padding of anchor tags above the images. The algorithm is:

Declare variable padding = 5px
Get the width, height and aspect ratio of the image.
Set the height to 45px and calculate the width according to the aspect ratio. If the width <= 110px, then directly resize the image and no change in padding is necessary
If the width > 110px, then make width constant to 110px and calculate height according to the aspect ratio. It will surely come less than 45px. Subtract the difference = (45 - height), divide it by 2 and add it to the padding variable.
Apply padding variable on the anchor tag. Now every logo should be displayed nicely and we have fixed the height of the navigation bar = 55px for all cases.

Here are the screenshots:-

FOSSASIA 2016

Mozilla All Hands

Facebook Developer Conference

Open Tech Summit

@aayusharora Please review this. It have been opened for a long time now. Thanks

Travis issues will get fixed after #1359.

Travis passing and fixed conflicts. @mariobehling @aayusharora @geekyd Please review and merge. Thanks :)

@Princu7 This seems a nice algorithm, this can be more practical when you remove the hardcoded values 110px and 45px and calculate them using jQuery

@Princu7  I am merging this right now, do the changes in next PR.


[FEATURE] Automate release notes

Is your feature request related to a problem?
Manually creating release notes is prone to error
What solution would you like?
Add automation to generate release notes

automation suggestion: https://github.com/opensearch-project/opensearch-plugins/blob/main/RELEASE_NOTES.md#automation

Copying from side-channel discussion: I haven't gotten the Github action to work consistently on repos where it's been implemented for a while, eventually I resorted to writing a script for it: https://github.com/Swiddis/opensearch-utils/tree/main/release
Not quite as convenient as the promise of an action that lets you just click "Generate notes", but it's been working for me for the 2.13 release.


Any plan on releasing part2 code?

Thank you for sharing the part one code! This is amazing work. Is there any plan on posting the SVR part?
Thanks.

Please find the implementation here.


Mean device traffic/wifi traffic/wifi usage graph does not make sense

If you are looking at a specific device it is possible display the "WiFi traffic" or the "WiFi usage" graph.
These graphs are actually only showing traffic from the Access Point or to the Access Point itself.
I was running iperf tests on the SSID configured on this AP for about 4 hours. the graph did not show any bigger value (always about 0.01Mbps/1 MB).
At last I started an iperf server/client on the AP itself, then I could see some correct values in these graphs.
Actually I don't think it makes sense for the customer to see this value (normally there is only traffic between CT Server and the AP which is about 20MB per day)

The wireless stats are wireless stats, not bandwidth stats. That's why they're called WiFi traffic or usage.
We think there's a reporting issue with some devices at the moment that we're working on.


fix a typo

@iglesias

Would be nice if you can also fix indentation starting in line 100. The first level of indentation of the method has two spaces, which does not correspond to the first level of the other methods in the same file.

Merging. Travis failures not caused by this change.

Hi guys,
such patches should be labelled properly. Rather than "fix a typo", it should be "add forgotten return 0 statement"
It is impossible to sort things later otherwise, or track down bugs from commit logs in the buildbot

You are definitely right. Thanks for the heads up!


LFS-955: Polish the look of form questions and answers

Improved horizontal alignment indentation, vertical spacing, vocabulary input placeholders, answer instructions message. See subtasks of LFS-955 for details.

[] Answers should not be indented relatively to the question text

re-tested after the "dev" branch merge

retested - all good

Tested, love the improvements! No bugs found from me

Ah, nevermind, it's an old bug, triggered by how many options were already selected before re-editing.

Weird corner case (that should never happen in practice): minAnswers=3, maxAnswers=2, forces me to select 3 answers, but the controls get disabled after only 2 are selected. Should we enforce maxAnswers = max(maxAnswers, minAnswers)?

retested in 0.8.0 smoke test


README

README
User Story
As a bootcamp student, I want the prework notes to be structured on a web page so that I can easily find and read information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

All set


Testing latest pyproj with CI

Coverage decreased (-1.8%) to 57.864% when pulling d53bfd89f0e6f8bbdd6d762ce6ac341591d19ca6 on kalxas:test_pyproj into cee50fb871feab20ea332c6fb58952503c44455f on geopython:master.

Thanks :-)

Thanks, I also hit this recently :)

relates to https://github.com/geopython/OWSLib/issues/823


Fix README.rst 'compatibility' typo

Fixing README.rst 'compatibility' typo (issue #504)

I mentioned the issue in the description, but I don't know how to link it to the PR.


not adding objects to the html

I am trying to make the library work But I cannot get the code working. It seems that code works and compiles but not adding the mesh to the html file.
I have tried different versions of Unity and the same library you have in the video (2018.3.0f2)
Which version of AR.js you have implemented?
index.html.txt

You can try to put the object under the ImageTarget
e.g. ImageTarget > Cube, Sphere ...etc


Tests in blueprint.py are order-dependent

Describe the bug
If you comment out some of the blueprint tests, or change the order in which they are run, you may cause otherwise unrelated tests to fail. I believe this is caused because several tests set the camera orientation, but do not reset it. I will investigate further.

We should move to always reset at the start and we may have to update baselines due to this.

I wonder if this note from the documentation is related...
If you want to restrict the amount of parallelism used in running the test suite you can do so with the -n command line option. By default, the test suite will be run using all the cores on your system. We have found that on some systems, running more than one test at a time may result in failures. To work around this issue you can run one test at a time.
https://visit-sphinx-github-user-manual.readthedocs.io/en/develop/dev_manual/test_suite.html


Fix Curve2D/Curve3D baking

Fixes  #61086.

Looks like the midpoint map was expected to be sorted.
Regression from #60999.

Thanks!


COMP: Fix deprecation warning for PyEval_CallObject

M:\Dashboard\ITK\Wrapping\Generators\Python\PyUtils\itkPyCommand.cxx(117,31): warning C4996: 'PyEval_CallObjectWithKeywords': deprecated in 3.9 [M:\Dashboard\ITK-build\Wrapping\Generators\Python\PyUtils\ITKPyUtilsPython.vcxproj]
PR Checklist

[x] No API changes were made (or the changes have been approved)
[x] No major design changes were made (or the changes have been approved)

Mac Python has 47 test failures. Also, I am not quite sure about correctness of this patch.

/azp.run ITK.macOS.Python

/azp run ITK.macOS.Python

In coming up with this code, I looked at https://sceweb.sce.uhcl.edu/helm/WEBPAGE-Python/documentation/python_tutorial/ext/callingPython.html


Use correct memory orderings

hello, first of thank you for making this! Ive noticed you use moRelaxed everywhere, this is obviously not correct, as it leads to data races. One way to test this is with ThreadSanitizer, an example. An oversimplification, but a good starting point is to make all loads moAcquire and stores moRelease. Basically you need to do the publishing protocol. You can use moRelaxed when outside or inside a "critical section" (I think but that might also be an oversimplification), that is code between a load and a store.

@planetis-m thanks so much for the bug report! I can look into this further when I have some time in the coming weeks. If you want a fix sooner a pull request would be very welcome. If I end up doing this, I would appreciate your review on the changeset. Cheers!

Here is my opinion (so take it with a grain of salt), the "how to write a correct ringbuffer" are not really worth the effort. I mean 1 lost slot, doesn't justify having twice the required capacity. They also need uints because of wrap around, but that's not an issue, it's just missing from this implementation. The Wait-free ring buffer the "classic" approach can be slightly tweaked to hide that extra slot from the user and use any integer as capacity instead of powers of two. Here's a wip nim port. Hopefully it's not much trouble adjusting to this algorithm in the future.

Imo your project is worth pursuing as well, building on top of a spsc queue is a good design. so don't loose faith on it.

Imo your project is worth pursuing as well, building on top of a spsc queue is a good design. so don't loose faith on it.

Ha, don't worry, I haven't lost faith just because of a bug report :)
I've put plenty of time into this project. I don't have time at the moment to fully respond to your comments or look into this. If you have time to put together a pull request for me to review, it would certainly be appreciated.
Cheers!

@planetis-m I think I'm using memory ordering as per your suggestion now in https://github.com/elijahr/lockfreequeues/pull/7
nimble test also now runs with LLVM thread sanitization to check for (and fail on) data races.
If you have some time and gumption, I'd certainly appreciate your review of that pull request. If not no problem, I'll just merge it sometime soon and cut a new release.
Cheers!


Able to move regions among Datanodes in Metasrv

What problem does the new feature solve?
Better resources (system load or bandwidth) usage and efficiency for regions.
What does the feature do?
Some regions are best placed in other Datanodes. Metasrv should support the feature of moving regions around among Datanodes. We can reuse some of the region failover procedure here, with some enhancement. For example, DeactivateRegion state should retry closing the region. We cannot reuse the region failover procedure, because we want the region movement to be as smooth as possible -- while the region failover procedure introduces too much unavailable time between deactivate and activate.
Implementation challenges
When to do the re-placement of region, or by what the Metasrv decides to start moving a region? Achieving the most balanced resources usage of Datanodes is very challenging.

duplicated with https://github.com/GreptimeTeam/greptimedb/issues/2700


ibmcloud: wait condition for ssh deletion

Added wait condition for SSH key deletion
Fixes: #1303

Hi @stevenhorsman
After completion of e2e test cases resources like clusters, SSH keys and VPC's must be deleted completely. Sometimes the VPC are failed to delete because ssh keys are still in use.
time="2023-07-28T01:47:29Z" level=warning msg="Failed to delete vpc resources, err: SSH-key still in use by 1 instance(s). First instance ID: 0757_1ad6081d-xxxxxxx-xxxxx-xxxxx-xxxxxxxx."

In order to validate SSH key deletion, we added an extra loop for 5 minutes and breaks if its completely deleted
Implemented the same logic used for deletion of subnets - PR

Thanks for the extra info, but in both the VPC and ssh key case, shouldn't the behaviour be that if, after 5mins the delete still hasn't worked, we should error rather than just carrying on? We'd hope that the delete won't take that long, but if it does then we want to avoid just hitting the error later on?

Hi @stevenhorsman ,
Modified the logic little bit now,

If SSH key deleted completely, it returns nil
else it will return error directly

The Azure e2e tests are new and have never passed (https://github.com/confidential-containers/cloud-api-adaptor/actions/workflows/azure-e2e-test.yml), so I will merge this without them.

@stevenhorsman e2e-test were failing due to quota exceeded issue while building podvm image. It is passing now for other PRs. Thanks


[Search v2.1] Use new query syntax in App

Implement this section of the design doc.

PR is merged

PR was merged

Summarizing payment on this issue:

Contributor: n/a internal
Contributor+: @rayane-djouah $250 via Upwork -- offer extended, please accept when you get a second! https://www.upwork.com/nx/wm/client/contracts?contractStatus=pending&contactPerson=36910897

Upwork job is here: https://www.upwork.com/ab/applicants/1823504324434878915/job-details

Thanks @rayane-djouah, all paid!


Source code

Hello @sail-sg, Could you update source codes of this paper "Inception Transformer". Thank you

Thank you so much.


fix the wrong path issue

Upgrade operator docs moved from /design to /user but docs have not been changed.

/assign @danielxlee


[ENG-4598] Try initializing GTM in a way that works runtime

Ticket: ENG-4598

Purpose
You can't just pull something into the index.html from runtime, it has to happen build time. So we'll try doing this the way raven works instead.
Summary of Changes

Initialize GTM in a way that works runtime.
Add environment variable for the gtm id
Types

Screenshot(s)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.473%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5711

Relevant Lines:
7667

💛  - Coveralls


Wait for ajax to load first

There's a plugin that waits for xhr to load https://github.com/jtassin/pending-xhr-puppeteer
Is this a reliable method to wait for all ajax content to load? Would it work with puppeteer-cluster?
I'm wondering if there's a more reliable method to wait for ajax to load? I prefer not waiting for a selector to load since I have many different sites, and this would require me to configure for each ajax site as discussed here: https://stackoverflow.com/questions/51066987/puppeteer-how-can-i-wait-for-ajax-request-and-process-the-result
Here's a sample page that takes a while to load:
https://cportal.lendingqb.com/Onlineapp2/GetRate.aspx?siteid=9f4ab128-33f9-4d14-9b11-18c96aef1fb3&sLPurposeT=0&aOccT=0&sProdSpT=0&sLAmtCalc=400000&sPurchPrice=300000&sSpZip=92708&sSpState=CA&sSpCounty=Orange&sProdImpound=1&sRLckdDays=45&sCreditScoreType1=760
Thanks in advance!

I recommend you ask general questions regarding puppeteer in the puppeteer repository or on stackoverflow.
I haven't tested the plugin, but I'm pretty sure it should work as it just works on the page. So put it in your task function and you should be fine.


Issue 4 command line power

Points 1 to 3 in issue #4

Example output:
Solar voltage: 14.07V
Solar current: 0.94A
Solar power: 13.93W
Load voltage: 13.27V
Load current: 0A
Load power: 0.0W
Battery voltage: 13.27V
Battery current: 1.03A
Battery power: 13.8W
Battery state of charge: 87%
Battery temperature: 20.89°C
Remote battery temperature: 0°C
Controller temperature: 23.01°C
...

I'll assume this works because you pasted the output, I don't have my charge controller with me right now.

This ticket can be closed.


Does Update()/UpdateAsync() refresh the cache of current DbContext?

Lets say I inserted some data by DbContext, after that I updated that data by Update() (it uses ExecuteSqlCommand, as my understanding), right after that I retrieved the data and it is came the same as inserted. Adding AsNoTracking() helps me to retrieve actual, fresh data.
So, does it work as expected? And do need I to use AsNoTracking() alsways for a such cases?

Hello @a-a-k ,
Update doesn't refresh the change tracker. It works as expected.
If the data is already loaded and you try to load it again in the Change Tracker, it will take the entity that has already been loaded, no matter change that's has been done on database or application side. When you specify AsNoTracking, it doesn't use the ChangeTracker, so you get fresh data.
Let me know if that explains correctly the current situation.
Best Regards,
Jonathan


Add new email domain: zsnr4.edu.pl

Add email domain of Zespół Szkół Nr 4 im. Mikołaja Kopernika w Dębicy  (@zsnr4.edu.pl) to the list

@lukasz26671 First stage of verification passed, official domain is zsnr4.edu.pl as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@lukasz26671 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


fix Fix a typo and remove useless member rowset in CommitTabletTxnInfo

Proposed changes
Fix a typo in https://github.com/apache/doris/pull/23078
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.48 seconds
stream load tsv:          540 seconds loaded<PHONE_NUMBER>9 Bytes, about 132 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.5 seconds inserted 10000000 Rows, about 338K ops/s
storage size:<PHONE_NUMBER>7 Bytes


Please add the following URL - graph.microsoftazure.us   Recent updates by the Azure Migrate Appliance now require this URL to be available prior to passing any prerequisite checks.

[Enter feedback here]

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

ID: 7a4a21a9-fcda-355f-6668-21cd3adf25af
Version Independent ID: 659ad43a-882f-1510-36df-dd1d994b6a09
Content: Azure Migrate appliance - Azure Migrate
Content Source: articles/migrate/migrate-appliance.md
Service: azure-migrate
GitHub Login: @Vikram1988
Microsoft Alias: vibansa

@pasnopik Thank you for your feedback. We will review it with content owner and update as appropriate.
@Vikram1988 FYI

@SadiqhAhmed-MSFT I have reviewed the PR and pushed the change to the docs. The change will start reflecting in 12 hours

Hi @pasnopik,
Thank you for bringing this to our attention.
The change has been updated in https://docs.microsoft.com/en-us/azure/migrate/migrate-appliance#government-cloud-urls.
Hence, closing this issue.
#please-close


Avoid unnecessary heap allocation when parsing frame headers.

Motivation:
As noted in #105, I spotted that we were heap-allocating an array
when parsing the frame header. That heap allocation isn't necessary, we
can just write directly into an Int, which should save us some heap spray.
Modifications:

Rewrote code to avoid heap allocating array and instead to use Int.
Elided some bounds checks by using an unsafe pointer, since I was
already here and it saved about 1% performance in a hot loop.

Result:
Fewer heap allocations on the hot code path.
Resolves #105

This has a fraction of one percent of a performance improvement in the real-world benchmark I'm running: just not really noticeable. Still worth avoiding the allocation, but just worth keeping it in perspective.


[#940] Preserve newlines in existing card files

How to test:
card = ModelCard.load("elonmuskceo/windows-linebreaks")
card.save("./toto.md")

$ wget https://huggingface.co/elonmuskceo/windows-linebreaks/raw/main/README.md

then diff the two files and check the diff

(would be cool to have a test for this i guess)

TY for this! I'll try to add a test for this and merge it into my branch.

This one is fine - tests failing for unrelated reason. merging into mine now


Implement command line option --version

This addresses issue #87. It is based on top of PR #100.
Still missing: The web interface should also display the version.

Okay, so I see currently in this branch the version v0.1.0-132-ge1deaab, but I think we want the additional --tags option, i.e.:
$ git describe --tags
v0.2.3-48-ge1deaab

I cannot say much more about the code, because I don't understand most of the lines your wrote (besides the obvious ones:).

I think we want the additional --tags option.

The v0.2.x tags were created as lightweight tags instead of annotated tags and are not used therefore. Citing man git-tag:

Annotated tags are meant for release while lightweight tags are meant for private or temporary object labels. For this reason, some git commands for naming objects (like git describe) will ignore lightweight tags by default.

We should create the next release with an annotated tag. Then the code will work as expected.

Hm... but why not use git describe --tags anyways as it fixes the situation we have at the moment?

Thank you @stweil for all the work!


Loop

How to display Names and Emails in different lists?
<!DOCTYPE html>

  <title>Loops</title>

Expected:

But:

How to get the expected result?

Only method I see is to put in two different arrays names and mails. So you can loop on each of them.

Yeah. but, why do we really need to build different arrays as we can't create and call multiple APIs for this approach or after getting from one API then creating multiple arrays for a large set, it doesn't seem to be right or scalable.

I really like lemonadejs but this is an example of limitation... :(

We are creating a alternative for that... it should be released soon

The user of an Array in multiple @loop is now possible using the unique='false'


Pluggable security configuration providers

This proposals replaces #48. It suggests pluggable mechanism for PodSecurityProviders and implementation which matches the Kubernetes restricted Pod Security Standards profile. This should make it easier for users to run under this profile.
In addition to the proposal, there is also a draft implementation which can be seen here. It currently doesn't have the docs, Javadoc and full test coverage, but it is fully functional under the restricted profile.

This proposal now has 5 binding +1 votes which is enough for approving. I will leave it open till tomorrow to see if anyone else has some more comments. Thanks.


Dependencies should be peer dependencies

For example, right now you pull in an outdated axios version.

Could You update the axios version number to 0.21.1 and upload to the npm repository please?
Thank you.

Ah well, time to vendor this module or fork it or find an alternative I guess...


Advertisement deletion for unjoin

Description
This PR adds the second part of unjoin process, when a cluster doesn't want to share his resources anymore.

Added enableBroadcaster flag in ClusterConfig
When the enableBroadcaster flag is set to false, the home cluster notifies to the foreign he wants to stop sharing by setting the advertisement status to DELETING.
The virtual kubelet on the foreign cluster is triggered:

it deletes all the offloaded resources
it deletes the Advertisement, causing the deletion of the virtual-kubelet deployment and virtual node through ownerReference.

Other features

the ClusterRole created by discovery modules has been updated to allow operations over Advertisement status
the remote watcher is closed when the adv is deleted
added periodic garbage collection in advertisement controller to remove expired adv
added createClusterConfigClient function to have a simple way to create a client for ClusterConfig resource

How Has This Been Tested?

[x] tested with kind
[x] ClusterConfig unit tests

[x] when we modify the SharingPercentage the advertisement is updated with the new quantity of shared resources
[x] when we disable Advertisement the Advertisement is deleted on foreign cluster

/rebase

/rebase


Cow: Watch jps[\W_]*+extensions

Cow requests the watch of the watch_keyword jps[\W_]*+extensions. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

jps[\W_]*+extensions has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by VLAZ in Charcoal HQ


What in the world.

mac:repo root#  DEBUG=True /bin/bash -x /opt/halyard/repo/meta/puppet-run
+ set -euo pipefail
+ [[ 0 != \0 ]]
++ python -c 'import os, sys; print(os.path.dirname(os.path.realpath(sys.argv[1])))' /opt/halyard/repo/meta/puppet-run
+ SCRIPT_DIR=/opt/halyard/repo/meta
+ . /opt/halyard/repo/meta/environment
++ OPT_PUPPET_BIN=/opt/puppetlabs/puppet/bin
++ ROOT_PATH=/opt/halyard
++ REPO_PATH=/opt/halyard/repo
++ ENV_PATH=/opt/halyard/environments
++ LOG_PATH=/opt/halyard/logs
++ : production
++ : True
++ :
++ export PATH=/opt/puppetlabs/puppet/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/puppetlabs/bin
++ PATH=/opt/puppetlabs/puppet/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/puppetlabs/bin
+ FLAGS=
+ '[' -n True ']'
+ FLAGS=' --debug'
+ '[' -n '' ']'
+ [[ -e /opt/halyard/environments ]]
+ [[ -e /opt/halyard/environments/production ]]
+ cd /opt/halyard/repo
++ git status -s
+ '[' -n ' M Gemfile.lock
 m meta' ']'
+ echo 'Repo is unclean: /opt/halyard/repo'
Repo is unclean: /opt/halyard/repo
+ which bundle
+ bundle check
+ bundle install
+ echo 'Failed to update bundle'
Failed to update bundle
+ [[ -e /opt/halyard/repo/preflight/main.sh ]]
+ librarian-puppet install --clean
/usr/local/bin/librarian-puppet:22:in `<main>': undefined method `activate_bin_path' for Gem:Module (NoMethodError)

librarian-puppet install --clean works manually as root so I think it's the env/PATH thing with ruby.

Hm. On a mac, it shouldn't be using /usr/local/bin's anything. It should be using /opt/puppetlabs or whatever. Can you dump out $PATH at at that point? And ls -la /opt/puppetlabs/puppet/bin?

This one can actually be closed, there was a nefarious brew install to /usr/local that I didn't expect to be there that after killing resolved this

haha


Remove in-coordinator Docs API implementation, update stargate.rest.enableV1

In Stargate V2, all HTTP APIs are moved to stand-alone services, out of coordinator.
Since then we have removed GraphQL implementation from Coordinator, and can now remove Document API as well, having first removed Integration Tests that rely on it.
This still leaves REST API,  and for that we have stargate.rest.enableV1 system property that was added earlier.
We need to change that a bit to allow for disabling RESTv2 part of API only, leaving RESTv1 as that is still needed for time being (for Stargate V2 deployments; new REST API only implements RESTv2 part of API).

Final removal on 2.1 branch in #2640.


Project name should be com.netflix.karyon

NetflixOSS » karyon » karyon-pull-requests #61 SUCCESS
This pull request looks good


Calling model_json_schema method in version 2.4 will serializes data (Field extra data).

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Calling model_json_schema method in version 2.4 will serializes data (Field extra data).
version 2.0 does not have this problem.
example code:
from pydantic import BaseModel, Field, version

class Faker(object):

    def __str__(self):
        return 1

class Demo(BaseModel):
    aaa: int = Field(example=Faker())

print(version.VERSION)
print(Demo.model_json_schema())

VERSION 2.0.3 output
2.0.3
{'properties': {'aaa': {'example': <__main__.Faker object at 0x7f7e3824d208>, 'title': 'Aaa', 'type': 'integer'}}, 'required': ['aaa'], 'title': 'Demo', 'type': 'object'}

VERSION 2.4.2 output
2.4.2

...

  File "/home/so1n/github/pait/.tox/py38-pydantic-vlast/lib/python3.8/site-packages/pydantic/json_schema.py", line 1221, in _named_required_fields_schema
    field_json_schema = self.generate_inner(field).copy()
  File "/home/so1n/github/pait/.tox/py38-pydantic-vlast/lib/python3.8/site-packages/pydantic/json_schema.py", line 549, in generate_inner
    json_schema = current_handler(schema)
  File "/home/so1n/github/pait/.tox/py38-pydantic-vlast/lib/python3.8/site-packages/pydantic/_internal/_schema_generation_shared.py", line 36, in __call__
    return self.handler(__core_schema)
  File "/home/so1n/github/pait/.tox/py38-pydantic-vlast/lib/python3.8/site-packages/pydantic/json_schema.py", line 541, in new_handler_func
    json_schema = js_modify_function(schema_or_field, current_handler)
  File "/home/so1n/github/pait/.tox/py38-pydantic-vlast/lib/python3.8/site-packages/pydantic/_internal/_generate_schema.py", line 965, in json_schema_update_func
    json_schema.update(to_jsonable_python(json_schema_extra))
pydantic_core._pydantic_core.PydanticSerializationError: Unable to serialize unknown type: <class '__main__.Faker'>

Example Code
from pydantic import BaseModel, Field, version

class Faker(object):

    def __str__(self):
        return 1

class Demo(BaseModel):
    aaa: int = Field(example=Faker())

print(version.VERSION)
print(Demo.model_json_schema())

Python, Pydantic & OS Version
2.4.2

After looking at the source code, I found a solution as follows:
from pydantic import BaseModel, Field, version

class Faker(object):
    value = 1

class Demo(BaseModel):
    aaa: int = Field(json_schema_extra=lambda v: v.update({"example": Faker().value}))

print(version.VERSION)
print(Demo.model_json_schema())

But the model_json_schema method returns a dict, so why serialize it?

Hi @so1n,
Thanks for your question. I believe the behavior in 2.4.2 is correct. model_json_schema returns a python dict of jsonable data, thus the issue with trying to serialize the Faker class.

@sydney-runkle  This is breaking change (like v1 to v2), but the documentation doesn't say anything about it.

@so1n,
Thanks for pointing that out. I'll add an update to the docs this week to add more detail to the docs 😄.


storage.list should return path also

storage list returns list of StorageInfo whereas storageinfo consist of name, size & path.
but currently storage.list returns only two things, name & size.

Fix in #349