Setting to open first file automatically when clicking Terminal "file links" which have multiple results

Setting to select first file automatically when clicking Terminal "file links". The vast majority of the time it's the first in the list when it can't resolve it to one single file once I have clicked on a filename in the terminal. It would be nice if there was a setting to just go to the first file that vscode thinks it might be.

(apologies if I have missed an existing setting that allows this, I couldn't find one)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Roll engine 1a96087747ac..fffeb8fe34f4 (2 commits)

git@github.com:flutter/engine.git/compare/1a96087747ac...fffeb8fe34f4
git log 1a96087747ac..fffeb8fe34f4 --first-parent --oneline
2019-12-03<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from 6QxEZ... to OSk8h... (#14085)
2019-12-03<EMAIL_ADDRESS>Roll src/third_party/dart e4344a568f..a4d799c402 (7 commits) (#14083)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Trybots failed. These were the failed builds: https://github.com/flutter/flutter/runs/331877719

Commit queue failed; closing this roll.


Xpanse Runtime documentation modifications

Fixed eclipse-xpanse/xpanse#836

@WangLiNaruto the PR is opened on your fork instead of the actual project. Please have a look.


Expand or collapse a node on click

Hi
I have been using the tool to draw network-graph. Lately a need to collapse or expand a node by clicking on it has come up as a convenient feature as the graph gets bigger. However I was not able to figure out how to just do that. Does anyone has any idea ? A quick internet search tells me that its possible by directly using d3.js. I was wondering if anyone has done that using "netjsongraph.js" however. Thanks in advance for any pointers.

Similar to this mentioned by @gubi in #13 . For now version, it's hard to achieve because of less params in click function. We'll consider adding this feature in the future.

@KuTuGu That will be of great help. Waiting for this feature. Thanks in advance.


[CWS] fix model used for constants when checking policies using the security agent

What does this PR do?
https://github.com/DataDog/datadog-agent/pull/24680 introduced the capacity to use a security agent running on linux and check policies designed for windows. Sadly the constants used are not the correct ones, this PR fixes this.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Sketch out unique user interactions/flows for PNF Browser

Some of our operations in PNF Browser involve user process flows that are unique to us.
This issue is concerned with building a skeleton for some of these flows. This will make it easier to convey to the Polygon Web Wallet designers what it is we need.
List of Flows (more to be added as we think of them)

[x] Indicating possible transfer/withdraw values based on currently available commitments
[x] Instant Withdrawal / Finalise Withdrawal indicators
[ ] Displaying transactions that have reverted because of Rollbacks from valid challenges
[ ] Derive multiple addresses (pkds) from the same mnemonic
[ ] ~Importing mnemonic (and multiple mnemonics) for PNF instead of generating a new one.~

Closing this issue because the other tasks are pending web component integration


Location services turned off ("some of these settings are managed by your organization")

Describe the Bug
It appears that this tool fiddles with the registry in some bad way, and modifying various Windows system options becomes disallowed after de-bloating. In particular, I can no longer change my location settings: instead, I see a message that "some of these settings are managed by your organization" and the settings dialogues are greyed out.
How can I fix this?
How can this issue be reproduced?
Run de-bloat, check location settings in Windows 11
Expected behavior?
No response
Operating System
Windows 11
Operating System Version
22H2
Logs
No response

Same issue there 😢

To solve this problem, you can import an .reg file into the Windows registry:
Windows Registry Editor Version 5.00

[HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\Windows\LocationAndSensors]
"DisableLocation"=dword:00000000
"DisableWindowsLocationProvider"=dword:00000000
"DisableLocationScripting"=dword:00000000

Thanks a lot @cspencer49519

@cspencer49519 Do you know a similar fix for automatically installing Windows Defender updates? I believe this software also messed that up. It used to be automatic, but now I get a daily notification about these small security updates, and I have to open Settings and install it manually.

It appears that this tool fiddles with the registry in some bad way, and modifying various Windows system options becomes disallowed after de-bloating. In particular, I can no longer change my location settings: instead, I see a message that "some of these settings are managed by your organization" and the settings dialogues are greyed out.

This was happening due to ShutUp10 configs, I'll add a proper fix to that, and another option.

Do you know a similar fix for automatically installing Windows Defender updates? I believe this software also messed that up.

Yup, this was messed up, sorry, the default settings will wipe the previous changes on the registry, unless someone want to manually disable automatic updates.


Address FB 61611

What?
Add test to reproduce a scenario similar to the crash reported in FB 61535. Update the code to throw an unchecked exception so that  further diagnostic information may be obtained on the underlying cause of FB 61535.
Why?
FB 61535 caused a crash for MIA. The underlying cause of the crash is not clear and is likely masked by the fact Datastore.getDocumentsWithIds() returns null in case of failure, which then leads to an NPE elsewhere.
How?
A test has been added to cause a failure in Datastore.getDocumentsWithIds() such that the existing code would have returned null. Datastore.getDocumentsWithIds() has been updated to throw a DocumentException retaining the details of the exception that caused it. Because QueryResult.iterator() implements the Iterable interface, we can't pass a DocumentException through that interface, so we encapsulate the DocumentException in an unchecked exception UncheckedDocumentException so that it can be thrown by QueryResult.iterator(). For further discussion of this, see FB 61611.
reviewer @mikerhodes
reviewer @rhyshort

Other than the name change mike mentioned +1

+1

+1


Media queries incorrectly applied in CSS

What version of Remix are you using?
1.4.3
Steps to Reproduce

Create a CSS file that has a @media (min-width: 1024px) for example (the actual min-width) value doesn't matter
Add the CSS file trough the links function in root.tsx (issues seems to be the same for page-specific CSS files)
View the app in the browser (I'm using Chrome). When I open dev tools and enable the mobile view tools, and select a small device like iPhone 12 pro, my min-width media queries are applied even though te screen is much smaller, see the first screenshot. The behaviour is the same when viewing from a real device (second screenshot, iPhone XS).
The fact that I think this might be a bug is that when I simply resize my browser window by dragging the right side, the media query doe apply correctly (in this case don't apply when the screen is smaller than the min-width) value.

I might also miss something very trivial in my app that's causing this, but I'm kind of lost on what to do from there.
Additional info:

Yarn v1
node v14
Mac OS Monterey 12.3.1

Screenshots
Viewing app in dev tools mobile viewer, the screen size of the device should be 390 according to chrome, but a media query with min-width: 769px is applied as you can see in the dev tools

Viewing the app on a real mobile device, you can see the layout is still wrong like the first screenshot. This means that the issue also applies in a production build deployed on vercel and is not related to a dev-tools specific issue

When simply resizing the browser window by enlarging the dev tools screen for example, the media queries are correct and I get the layout I expect.

Code
The code that reproduces this can be found here: https://github.com/TimoWestland/salty-web.
I've temporarily made the repo public as I can imagine this is hard to figure out without code to reproduce it :)
I hope this is a correct bug report and this is the correct way of posting it. If not I'm sorry in advance and I'll gladly change/move the issue. If it's not a bug I would be really happy with a solution for my problem or even a work-around would be fine for me at this moment.
Expected Behavior
Media queries to apply/not apply the same for all scenario's (dev tools mobile viewer, real device, resizing browser window)
Actual Behavior
Media queries are applying in some scenario's where the screen size is smaller than the min-width value in the media query.

I knew this had to be something simple, I didn't add the viewport meta tag in my main App wrapper:
<meta name="viewport" content="width=device-width,initial-scale=1" />
Adding this fixed it.


Implications of Classifier-Free Guidance in Auto-regressive Models

Hi, thank you for the insightful work!
I have some concerns regarding the classifier-free guidance (CFG) in auto-regressive models.
CFG in this work is implemented as follows:
https://github.com/FoundationVision/VAR/blob/1ae51772d2622e2fd44a188564cf394b71f5562d/models/var.py#L191-L192
However, it's important to note that CFG in auto-regressive models differs fundamentally from that in diffusion models (as outlined in Section 4 of this blog). In essence, the guidance in diffusion models is not theoretically applicable to auto-regressive models.
I am curious if this difference yields any notable empirical results. Have you conducted any quantitative or qualitative studies on the impact of CFG on this auto-regressive model? I would greatly appreciate any insights or empirical findings you could share on this subject.

@fkcptlst in the Ablation Study section of the paper we tested the influence of CFG. We simply follow Google MUSE's CFG introduced in their paper. https://sander.ai/2022/05/26/guidance.html seems a thorough analysis on CFG. We'll check that later and maybe try some more implementations. Thank you for providing this!


Laravel 10.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 10.x.
Before merging, you need to:

Checkout the l10-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected a dependency for the laravel/framework. Unless you have a special requirement for depending on the entire Laravel framework, it's recommended to add dependencies for the specific illuminate packages instead.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/flex-env.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "sven/flex-env": "dev-l10-compatibility",
    }
}

Finally, run: composer update


RHMAP-4978 - Making JS SDK not init automatically and use fh.init instead

I've been trying different approaches to make to init the SDK manually and I think this one is the better way of doing it.
I'm reusing the existing init on feedhenry.js as it was going to be deprecated (talked with @wei-lee about it), but removed the first param (opts), so it only have two params, the success callback and the fail callback (as the other SDKs)
In case it fails, it returns the error
In case it succeed it doesn't return anything (as the other SDKs don't return anything neither)
I've moved the fhinit function on src/modules/waitForCloud.js from inside the ready function and made it available on the exports, and I call it on the feedhenry.js init.
I've added an error on src/modules/waitForCloud.js ready method to be called when you try to use the cloud, but cloud is not ready and is not being initialized at the moment (i.e. if you try to execute cloud functions before manually calling init).

LGTM

👀

👍

@jcesarmobile Is this change still required or can it be closed along with its JIRA task? https://issues.jboss.org/browse/RHMAP-4978

It's required

@jcesarmobile - I was thinking about doing the same then I saw that PR.
Really good job on this. This change will be really useful and also can resolve multiple problems end users are facing.  Any idea why this wasn't merged?

Sorry @wtrocki, I missed your comment.
I sent an email about it and we decided to focus on other problems with more priority and also we didn't want to introduce breaking changes by then. A change like this will require to update all the templates and might take a lot of time that we didn't have by that time.
I sent it by 3.17 time frame, maybe we should start planning this to be available in 3.19 or 3.20.
Can you answer on the mail I sent?

Cannot find that email.
In my opinion having sdk automatically mounting some routes, configurations etc. is anti pattern.
This change will be the one step forward for more cleaner SDK where developers decide what they can use and when. I fully understand that this change will include a lots of testing and updates. Just wanted to point that this is the way to go.


Do not assume sequence is monotonic

https://github.blog/changelog/2021-09-08-npm-couchdb-upgrade-will-reset-sequence-number/

yes sorry about that, we are still keeping up with updates that stacked during the downtime.
You can track the progress on our public dashboard: here or here


How to install CPython to a custom location?

Hi.  New to uv.  I would like to install CPython to my user specified local directory.  Is there any way to do it?

You can specify UV_PYTHON_INSTALL_DIR.


Upgrade to Clap 4.0

I started on this but gave up. The migration to Clap 3.0 itself wasn't too hard, but the completion scripts have changed enough that all just's custom completion patches now don't apply cleanly.
The upgrade to Clap 3.0 will have to wait until:

Someone fixes the completion scripts. (Which would be awesome!)
Clap implements dynamic completion support, which would allow calling back into the just binary to get variables and recipe names: https://github.com/clap-rs/clap/issues/1232
There is a critical vulnerability in Clap 2 or some feature we really want, at which point I'll just upgrade and the completion scripts will all break, and so we'll revert back to not having variable and recipe completion support.

Another point in favor of doing this is there's now a crate for generating man pages for clap 3.0 apps. It would be nice to retire the help2man generated man page, which is pretty crappy, and requires a feature just to massage the help output into the right format.

Someone fixes the completion scripts. (Which would be awesome!)

I've just been working on custom completion scripts for a protect I'm working on so could have a look at this while I still have an idea how it works if you want.
Is there a branch/patch somewhere for the code migration part?

Is there a branch/patch somewhere for the code migration part?

Yup! It's here: https://github.com/casey/just/tree/upgrade-clap
I started trying to fix the completion scripts, but I gave up quickly, and I'm not sure if the changes I made have the correct behavior.
If it's more than a little painful, then I'm not sure you should spend a ton of time on it. I think the current approach of patching the auto-generated completion scripts is probably unmaintainable in the long term, and we'll eventually want to phase it out in favor of something better: https://github.com/clap-rs/clap/issues/1232

Finally merged!


Fix typos in log messages

Hi @fbricon, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Hi @fbricon, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Add the MyViewCounts View Counter

It's not quite a badge, but a while back I made an application called MyViewCounts, a view counter to embed in READMEs that allows the greatest amount of customizability when compared with other view counters!
Here's the Website: https://myviewcounts.rayhanadev.repl.co/
Here's the Github Repo: https://github.com/RayhanADev/MyViewCounts
Here's a View Counter on my Profile: https://github.com/RayhanADev#custom-viewcounter
Let me know what you think!

Hi @RayhanADev,
Thanks for the offer, but I won't accept as it's not a badge, sorry.
I also think there already is a view count badge in this repo.
Regards!

Okay thanks!


QSphere not giving a circular shape plot

Information

Qiskit Terra version: 0.14.2
Python version: 3.8.5
Operating system: MacOS, Windows, Linux

What is the current behavior?

Steps to reproduce the problem
from qiskit.visualization import plot_state_qsphere
from qiskit.quantum_info import Statevector

sv = Statevector.from_label('0')

plot_state_qsphere(sv.data)

What is the expected behavior?
The Qsphere should have a circular (or spherical, whichever way you want to see it as) shape. The shape is oval. This happened when I updated my qiskit-terra version recently.
Suggested solutions
I think there's some issue with the plotting about the vertical axis. Please look into the matter.

Duplicate of #4770. Submitted pull request #4893 to fix it.


Consider releasing android builds through FDroid

https://f-droid.org/en/docs/Submitting_to_F-Droid_Quick_Start_Guide

Or https://f-droid.org/en/docs/Setup_an_F-Droid_App_Repo/

I opened (a very minimalistic) request for packaging issue here: https://gitlab.com/fdroid/rfp/issues/750

Sweet thanks @Bubu! We just got our first automated apk release https://github.com/textileio/textile-mobile/releases/tag/android_1.6.14_262
Since Android is still pretty beta, how is F-droid for distributing new releases? Should we wait for a bit more stability or just go for it?

We are waiting for your F-Droid release...
Bare in mind that F-Droid will not be able to build  when you rely on JDK or other non-free tools.
Still alpha? No worries!
https://f-droid.org/docs/Inclusion_Policy/?title=Inclusion_Policy
Doing it throught your own repo? Also an option!

In its current state, the app has a show-stopper:
    compile('com.crashlytics.sdk.android:crashlytics:2.9.5@aar') {
        transitive = true;
    }

That's not acceptable at F-Droid. If you really want that, we'd need a separate FOSS build flavor for F-Droid that comes without this.


Upgrade java-properties to 2.0

Description
This should drop the dependency on encoding (https://github.com/stackabletech/operator-rs/issues/567).
Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)

bors merge

bors r+


Moving to a reusable Recorder

Related to #11, #30.
Currently, each time a snapshot is exported there are multiple (arguably) unnecessary allocations.
Lets say we look at the http metrics exporter code. This a small snippet extracted from https://github.com/metrics-rs/metrics/blob/master/metrics-exporter-http/src/lib.rs#L70.
// This is the main logic for the async event loop for the http exporter.
service_fn(move |_| {
    let recorder3 = recorder2.clone();

    controller2
        .get_snapshot_async()
        .then(move |result| match result {
            Ok(snapshot) => {
                let mut r = recorder3.clone();
                snapshot.record(&mut r);
                let output = r.into();
                Ok(Response::new(Body::from(output)))
            }
            Err(e) => Err(e),
        })
})

The following is some kind of pseudo code representation meant to illustrate the allocations made by this logic.
Controller::get_snapshot_async() {
    MetricsRegistry::get_snapshot() {
        Vec::new(); // allocates to hold metrics.
        ValueHandle::snapshot() {
            for value in values {
                Counter -> (),
                Gauge -> (),
                Histogram -> AtomicWindowedHistogram::snapshot() {
                    Vec::new(); // allocates to hold values
                }
            }
        }
    }
}
Recorder::clone() {
    HashMap::new();
    Vec::new();
}
Recorder::into<String>() {
  String::new();
}

So each time we allocate a new Snapshot and Recorder which is quite expensive.
What I suggest is that we get rid of the Snapshot entity and move to a reusable Recorder.
The logic would become something like this:
service_fn(move |_| {
    // The recorder should keep track of its state internally, so that it can
    // only be written to once.
    recorder.reset();
    // Takes a snapshot of the metrics and write them in the recorder. This might
    // allocate memory if needed.
    match controller.snapshot(&mut recorder) {
        Ok(_) => {
            let output: String = recorder.export();
            Ok(Response::new(Body::from(output)))
        }
        Err(e) => Err(e),
    }
})

This way we could reuse the allocated memory from the Recorder and only allocate the output String.
Admittedly, I am not currently certain that this is actually possible with the futures 0.1 runtime, but I'll work on a PR.

I think it's definitely possible.
What I'd like to do is keep around the ability to take an independent snapshot, though, because it provides flexibility if users need to do something with the raw data.
Ideally, I'd like to see Controller itself implement Snapshot just like metrics_runtime::data::Snapshot does -- although we might want to rename the trait to something more generic, dunno -- so that exporters and "internal" code can use the optimized, fewer-allocations Controller-based approach, while leaving the existing approach for anybody who needs it.

But what would be the current advantage of the current approach?
We could simply add a trait to Recorder that adds the ability to iterate over its content. So a user that wants a raw snapshot would create a recorder and then iterate over its content.

Also, I'd like to do the snapshot work first before we play with/try and move to reusable recorders.  That one still needs testing/thought, and I think the "controller records directly to recorders" thing is good and achievable right now.

I don't think it's a valid use case to create a recorder that simply holds the data in raw form.  Snapshot already does that, and will do it with less overhead than the recorder interface, because it lets us keep histograms compressed up until the moment that they need to be recorded.

I think we can close this out now that #35 has landed.


[GLUTEN-6300][CH] Ensure same hash results from NaN

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Fixes: #6300
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
unit tests
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Better to add an ut.

Better to add an ut.

There have been an UT


Rendering: Provide some way to show multiple nested errors (or display related as nested)

Currently, the related errors of a given report will show just one after another, without any clear indication of them belonging tightly to the outer error.
I.e., in my project, right now I have errors like these:

Which are supposed to express that the two related errors are actually part of the outer error  -- clearly not quite the intended use for related.
I can get closer to a more intuitive visualization by storing the related errors in a simple struct ErrList(#[related] Vec<Report>):

However, this now requires me to have another error message be displayed, which often doesn't add any semantic value, and takes up a lot of space.
For cases like these, where I really want to show several siblings as nested within one outer error, I'd optimally want to have them be displayed roughly like this:

My proposal here would be to add a variant of "related" errors that explicitly declare them as nested, i.e.:
#[derive(Diagnostic, Error, Debug)]
#[error("{message}")]
#[diagnostic]
struct ManyErrors {
  message: String
  #[related(nested)]
  errors: Vec<Report>
}

This is a reasonable proposal! I’d be happy to take this change :)

Nice! I'd love to take a stab at this, but I'm a bit unsure on how to approach an implementation here:
If the idea is to actually have this as a property of the related attribute, then I don't know how to express that within the current structure of the Diagnostic trait.
Alternatively, I could just add a separate nested field to the Diagnostic trait, which would potentially feel cleaner, but is also a much larger change to the API.
A third, minimally invasive, approach would be to just make the way related errors are shown a global setting in the GraphicalReportHandler.
What do you think, @zkat? Or is there something else I'm missing?

Probably a configuration in the GraphicalReportHandler is probably best, esp since this is all about controlling how things look, rather than some fundamental property about the relationship between diagnostics


reflect: Call is slow

reflect.Value.Call is pretty slow.

In addition to always allocating a slice for its []reflect.Value result parameters, it
also does a lot of prep work on each call.

It would be nice to both avoid that allocation and also do the setup checks just once.

Maybe a new method to return some sort of 'Caller' type that makes repeated Calls
faster, with less paranoia and no allocation.

Or just speed up the checks and make a new Call method that also accepts a slice for the
result values.

how is it going

Looking at some internal fleet-wide CPU usage, I see reflect.Value.Call and reflect.Value.call show up pretty high in the list.
It might be time to optimize this.
In Go 1.8, there's now precedent in the reflect package for returning a worker func (reflect.Swapper) after doing the validation only once.
Investigate the top 2014 comment's Caller idea, and see how much CPU it can save.
/cc @dsnet

Maybe a new method to return some sort of 'Caller' type that makes repeated Calls faster, with less paranoia and no allocation.

Value.Interface() sort of does this, but you have to call the result like a real function, not using Value.Call(). If you can fit code into using Value.Interface(), then there's no allocation overhead (unless the frame pool is empty) and no validation. The compiler constructs the arguments frame and interprets the result frame and reflect just has to do some memory copying. (That said, it still does two copies of both the arguments and the results. I feel like it should be possible to get that down to one.)

Please, make it better.
html/template use reflect.Value.Call a lot of time and performance is poor.

For some concrete numbers, as of 1.9 on darwin/amd64, reflect.Value.Call is ~65-80x slower than other invocation types. Gist of benchmark code.
BenchmarkReflectMethodCall-4         	10000000	       144 ns/op
BenchmarkReflectOnceMethodCall-4     	10000000	       138 ns/op
BenchmarkStructMethodCall-4<PHONE_NUMBER>	         2.20 ns/op
BenchmarkInterfaceMethodCall-4<PHONE_NUMBER>	         2.14 ns/op
BenchmarkTypeSwitchMethodCall-4<PHONE_NUMBER>	         1.88 ns/op
BenchmarkTypeAssertionMethodCall-4<PHONE_NUMBER>	         1.83 ns/op

Implementation of reflection in golang does need improvements, current performance is too bad.

Cache function layout in a Caller reduce 50% reflect call overhead. https://gist.github.com/sillyousu/606e4874839456cc02335bd1c5045f27
update @bruth  's benchmark:
BenchmarkReflectCaller-8                20000000                64.6 ns/op
BenchmarkReflectMethodCall-8            10000000               135 ns/op
BenchmarkReflectOnceMethodCall-8        10000000               126 ns/op
BenchmarkStructMethodCall-8<PHONE_NUMBER>               1.67 ns/op
BenchmarkInterfaceMethodCall-8<PHONE_NUMBER>               2.41 ns/op
BenchmarkTypeSwitchMethodCall-8<PHONE_NUMBER>               1.23 ns/op
BenchmarkTypeAssertionMethodCall-8<PHONE_NUMBER>               1.42 ns/op

Any new suggestion for using Call()?  seem 4 years later still did not solve yet!

@hzmnet, passive aggressively complaining about how long something is taking is not helpful and generally considered poor etiquette in open source projects. This bug is marked "help wanted". If you're interested in working on this, please help. It is not anybody's current priority and probably won't be anytime soon.

It seems that java sets a buffer slice to avoid this problem.

Now that we have https://github.com/golang/go/issues/49340, which is a reasonable proposal which is just waiting for a prototype, should we close this issue as a duplicate? Is there any other part of Call which is slower than it needs to be besides the allocation for the results slice?

I updated benchmark by @bruth here and these are results on my machine:
BenchmarkReflectMethodByNameInterface-8   	 3859410	       317.2 ns/op
BenchmarkReflectMethodByName-8            	 5119252	       237.1 ns/op
BenchmarkReflectMethodCall-8              	 8402196	       129.5 ns/op
BenchmarkReflectOnceMethodCall-8          	 9686301	       131.1 ns/op
BenchmarkReflectCallInterface-8           	516733941	         2.164 ns/op
BenchmarkStructMethodCall-8               	472757470	         2.524 ns/op
BenchmarkInterfaceMethodCall-8            	544497948	         2.027 ns/op
BenchmarkTypeSwitchMethodCall-8           	437183308	         2.694 ns/op
BenchmarkTypeAssertionMethodCall-8        	428367258	         2.714 ns/op

I added method invocation of using MethodByName and then of calling .Call directly or of first type-casting through .Interface().(func()). I also added .Interface().(func()) on the method pointer itself. What is interesting to me:

BenchmarkInterfaceMethodCall (calling a method on the interface value) is faster than BenchmarkStructMethodCall (calling a method on a struct pointer). That is a surprise.
Type-casting through .Interface().(func()) removes overhead for method pointer, but adds overhead when method is obtained using MethodByName. It is even faster to call .Call on the method from MethodByName then of type-casting it. That is the biggest surprise to me. Doing:

	s := reflect.ValueOf(v)
	f := s.MethodByName("Inc").Interface().(func())
	for k := 0; k < n; k++ {
		f()
	}

I would expect and assume that f would behave and be close to performance of a regular function call. Especially as it looks like a regular function value.
So I think there are definitely some improvements needed here.


New customPath parameter with test

Hi Invertase team,
This is the same as https://github.com/invertase/dart_custom_lint/pull/187, but including a test on the new customPath parameters.
Please let me know if this test is sufficient. I added no test for the other methods where the customPath parameter has been integrated as well in the pull request of Matthias, because I did not found any existing test to extend for those other methods. I even found no call at all to the addYamlFileEdit() method.
Thanks in advance for the review.
Best Regards,
Laurent

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: matthiaslarisch:x: laurentschallYou have signed the CLA already but the status is still pending? Let us recheck it.

Sorry this flew under the radar.
LGTM, thanks!

You're welcome, thank you for the merge !

tldr; File(customPath).readAsStringSync() returns the original file contents even after a fix is applied.
I think this is problematic because the file doesn't get saved after making modifications. I'm trying to add entries to a json file. The first fix works fine, but subsequent fixes are still referencing an old version of the file. The changeBuilder seems to know about the latest file version, but I don't see a way to access the workspace to read the file.
A simplified version of this is:
final path = 'some/file/path.json';

// read file contents (on subsequent fixes, this returns the version before the first fix)
final fileContents = File(path).readAsStringSync();
final asJson = jsonDecode(fileContents) as Map<String, dynamic>;
final encoder = JsonEncoder.withIndent('  ');

if (!asJson.containsKey('foo')) {
  changeBuilder.addGenericFileEdit(
    (builder) {
      final lastBracketPosition = fileContents.lastIndexOf('}');

      builder.addSimpleReplacement(
        SourceRange(lastBracketPosition - 1, 2),
        ',\n  "foo": "bar"',
      );
    },
    customPath: path,
  );
}

if some/file.path.json starts as
{
  "one": "two"
}

applying a fix the first time results in:
{
  "one": "two",
  "foo": "bar",
}

applying the fix again results in:
{
  "one", "two",
  "foo": "bar"
}  "foo": "bar"
}

I noticed that the analyzer plugin gets file contents from ChangeWorkspace, but I don't see how to access that.

opened issue #296


support for ubuntu 16 on OH master

support for the OpenHumanoids/DRC build of Director in ubuntu 16.06

these changes are against the RL/director master (as of yesterday)

core modifications: use the same versions of qt (4), vtk-pre-compiled (7.1) and pythonqt, PCL (1.8) and PointCloudLibraryPlugin using in the superbuild.

uses the 'oh fork' of drake and ddDrakeModelOH.cpp - which is 1 year old.

Key changes are in: src/vtk/DRCFilters/CMakeLists.txt

minor API changes in C++ code.

Remaining issues are that
(1) map server has to be disabled because it fails to compile against into a Python library
(2) octomap has a related issue
(3) various calls to setInput() still need to change to setInputData() in python segmentation code.
(4) the 'Task Panel' pop up doesnt work. (Pose utility does work)
Proof:
https://www.dropbox.com/s/pjoje19c2a63uys/2017-08-director-16-04.mp4?dl=0

@patmarion I have been mostly successful in upgrading OH's fork of director to vtk-7.1 as mentioned here: issue on RL/director repo
I used RL/director's master and made the minimal modifications. Could you help me with a few of the remaining changes?
(Note: when I forked director to RobotPerception I broke the connection in github.)
FYI: @wxmerkt  and @VladimirIvan

Sure, I'm happy to help.  I haven't made an openhumanoids build in a while though.  What branch of openhumanoids are you building with?  Or, feel free to copy and paste and compiler errors (here or email) and I'll try to offer suggestions.  If there are any changes you'd like to move into director/master I can help with that too.
I am leaving for a camping trip tomorrow though and won't be back until Tuesday!

Probably easier if I just copy in the compile error here. This is the only compile error I haven't been able to resolve. It is triggered when I enable:
-DUSE_DRC_MAPS:BOOL=OFF
I guess the problem is in wrapping that library for Python:
[ 12%] Building CXX object src/vtk/DRCFilters/CMakeFiles/vtkDRCFiltersPythonD.dir/vtkLCMGLPropPython.cxx.o
[ 13%] Building CXX object src/vtk/DRCFilters/CMakeFiles/vtkDRCFiltersPythonD.dir/vtkCollectionsPython.cxx.o
[ 13%] Building CXX object src/vtk/DRCFilters/CMakeFiles/vtkDRCFiltersPythonD.dir/vtkDRCFiltersPythonInitImpl.cxx.o
make[4]: *** No rule to make target '/usr/lib/x86_64-linux-gnu/libpython2.7.soPythonD', needed by 'lib/libvtkDRCFiltersPythonD.so'. Stop.
make[4]: Leaving directory '/home/mfallon/rpg-navigation/software/director/distro/pods/drc/pod-build'
CMakeFiles/Makefile2:1050: recipe for target 'src/vtk/DRCFilters/CMakeFiles/vtkDRCFiltersPythonD.dir/all' failed
make[3]: *** [src/vtk/DRCFilters/CMakeFiles/vtkDRCFiltersPythonD.dir/all] Error 2
make[3]: Leaving directory '/home/mfallon/rpg-navigation/software/director/distro/pods/drc/pod-build'
Makefile:127: recipe for target 'all' failed
make[2]: *** [all] Error 2
make[2]: Leaving directory '/home/mfallon/rpg-navigation/software/director/distro/pods/drc/pod-build'
Makefile:26: recipe for target 'all' failed
make[1]: *** [all] Error 2
make[1]: Leaving directory '/home/mfallon/rpg-navigation/software/director/distro/pods/drc'
Makefile:16: recipe for target 'drc' failed
make: *** [drc] Error 2
The only change I made to cmake was for src/vtk/DRCFilters/CMakeLists.txt:
https://github.com/robotperception/director/pull/7/files#diff-10f186afafb41fd1b9f9f831b97e18e6
I wouldn't be confident I did this correctly.
Any ideas?

Can you edit src/vtk/DRCFilters/CMakeLists.txt and go to the bottom, line 210, right before the last block of lines.  Try adding:
message("VTK_LIBRARIES is:  ${VTK_LIBRARIES}")

set(VTK_LIBRARIES
  vtkWrappingPythonCore
  vtkGUISupportQt
  vtkInteractionWidgets
  vtkRenderingAnnotation
  vtkRenderingFreeType
  vtkInteractionStyle
  vtkIOXML
  vtkIOImage
  vtkIOGeometry)

My guess is that when USE_DRC_MAPS is enabled, and it causes a find_package(PCL) I wonder if PCL is setting the VTK_LIBRARIES variable to include other non-vtk libraries.

Then when you run make or cmake it will print out the value of VTK_LIBRARIES, and I'd like to see what it is.  Then I am re-setting VTK_LIBRARIES which might fix the problem.

It complied successfully when I removed all the libraries in the variable by setting: set(VTK_LIBRARIES)
(Note that vtkWrappingPythonCore and vtkGIISupportQt could not be found)
But libvtkWrappingPython27Core-7.1.so and libvtkGUISupportQt-7.1.so do exist
VTK_LIBRARIES is: vtkIOPLY;vtkCommonCore;vtksys;vtkCommonDataModel;vtkCommonMath;vtkCommonMisc;vtkCommonSystem;vtkCommonTransforms;vtkIOCore;vtkCommonExecutionModel;vtkzlib;vtklibxml2;vtkChartsCore;vtkCommonColor;vtkFiltersGeneral;vtkCommonComputationalGeometry;vtkFiltersCore;vtkInfovisCore;vtkFiltersExtraction;vtkFiltersStatistics;vtkImagingFourier;vtkImagingCore;vtkalglib;vtkRenderingContext2D;vtkRenderingCore;vtkFiltersGeometry;vtkFiltersSources;vtkRenderingFreeType;vtkfreetype;vtkFiltersHybrid;vtkImagingSources;vtkFiltersImaging;vtkImagingGeneral;vtkFiltersAMR;vtkIOXML;vtkIOXMLParser;vtkexpat;vtkParallelCore;vtkIOLegacy;vtkFiltersFlowPaths;vtkIOSQL;vtksqlite;vtkImagingMorphological;vtkIOExport;vtkIOImage;vtkDICOMParser;vtkmetaio;vtkjpeg;vtkpng;vtktiff;vtkRenderingGL2PS;vtkRenderingContextOpenGL;vtkRenderingOpenGL;vtkImagingHybrid;vtkgl2ps;vtkFiltersHyperTree;vtkFiltersVerdict;verdict;vtkIOAMR;vtkhdf5_hl;vtkhdf5;vtkInteractionImage;vtkImagingColor;vtkInteractionStyle;vtkInteractionWidgets;vtkFiltersModeling;vtkRenderingAnnotation;vtkRenderingVolume;vtkRenderingImage;vtkFiltersPython;/usr/lib/x86_64-linux-gnu/libpython2.7.so;vtkWrappingPythonCore;vtkWrappingTools;vtkGUISupportQt;vtkFiltersPoints;vtkGeovisCore;vtkInfovisLayout;vtkViewsCore;vtkproj4;vtkFiltersGeneric;vtkIOMINC;vtkNetCDF;vtkNetCDF_cxx;vtkViewsInfovis;vtkRenderingLabel;vtkFiltersSMP;vtkDomainsChemistry;vtkIOParallel;vtkFiltersParallel;vtkIOGeometry;vtkIONetCDF;vtkexoIIc;vtkjsoncpp;vtkFiltersSelection;vtkoggtheora;vtkFiltersParallelImaging;vtkIOMovie;vtkIOInfovis;vtkIOTecplotTable;vtkIOEnSight;vtkIOParallelXML;vtkIOExodus;vtkIOLSDyna;vtkViewsContext2D;vtkRenderingVolumeOpenGL;vtkRenderingLOD;vtkFiltersTexture;vtkImagingMath;vtkIOVideo;vtkImagingStencil;vtkFiltersProgrammable;vtkRenderingLIC;vtkIOImport;vtkImagingStatistics

@patmarion Could you please specify what the VTK_LIBRARIES should output?
Currently I have only the following ones:
VTK_LIBRARIES is:  vtkInteractionWidgets;vtkInteractionStyle;vtkRenderingFreeType;vtkRenderingCore;vtkRenderingOpenGL;vtkFiltersExtraction;vtkFiltersCore;vtkCommonCore
Director compiles, but segfaults. (that's without Maps' Server)

Earlier in src/app/CMakeLists.txt this code is called which initializes the VTK_LIBRARIES variable:
find_package(VTK REQUIRED COMPONENTS
  vtkWrappingPythonCore
  vtkGUISupportQt
  vtkInteractionWidgets
  vtkRenderingAnnotation
  vtkRenderingFreeType
  vtkInteractionStyle
  vtkIOXML
  vtkIOImage
  vtkIOGeometry
  )
include(${VTK_USE_FILE})

I suspect that later, when find_package(PCL REQUIRED) is called, the pcl cmake script overwrites the vtk variable.  It's possible that duplicating the find_package(VTK) afterward will reset things correctly.
If director segfaults, I guess it could be because there are multiple versions of vtk installed.  So director compiles and links against vtk7.1, but at runtime the python code import vtk finds the system installed vtk5.  One way to avoid this is to prepend the PYTHONPATH environment variable with the location of the vtk7 python files (using something like prefix/lib/python2.7/dist-packages).  Then try running: python -c "import vtk; print vtk.__file__" and make sure the correct location is dispalyed.  Another workaround is to apt-get remove the system vtk5.

@patmarion I did have VTK6.2 installed systemwide. I have removed it since, and trying to compile once again.
I have manually unpacked (i.e. ctrl+c and ctrl+v) the precompiled vtk7.1 into software/build:
http://patmarion.com/bottles/vtk7.1-qt4.8-python2.7-ubuntu16.04.tar.gz
And I'm trying to get director to use that, however, the following packages cannot be found from that compiled version of VTK 7.1:
/usr/bin/ld: cannot find -lvtkIOODBC
/usr/bin/ld: cannot find -lvtkRenderingParallelLIC
/usr/bin/ld: cannot find -lvtkParallelMPI
/usr/bin/ld: cannot find -lvtkIOParallelLSDyna
/usr/bin/ld: cannot find -lvtkParallelMPI4Py
/usr/bin/ld: cannot find -lvtkPythonInterpreter
/usr/bin/ld: cannot find -lvtkTestingGenericBridge
/usr/bin/ld: cannot find -lvtkIOGDAL
/usr/bin/ld: cannot find -lvtkInfovisBoostGraphAlgorithms
/usr/bin/ld: cannot find -lvtkIOVPIC
/usr/bin/ld: cannot find -lvtkIOParallelExodus
/usr/bin/ld: cannot find -lvtkRenderingParallel
/usr/bin/ld: cannot find -lvtkRenderingQt
/usr/bin/ld: cannot find -lvtkIOFFMPEG
/usr/bin/ld: cannot find -lvtkGUISupportQtOpenGL
/usr/bin/ld: cannot find -lvtkTestingIOSQL
/usr/bin/ld: cannot find -lvtkWrappingJava
/usr/bin/ld: cannot find -lvtkFiltersParallelMPI
/usr/bin/ld: cannot find -lvtkTestingRendering
/usr/bin/ld: cannot find -lvtkIOXdmf2
/usr/bin/ld: cannot find -lvtkIOPostgreSQL
/usr/bin/ld: cannot find -lvtkIOParallelNetCDF
/usr/bin/ld: cannot find -lvtkRenderingFreeTypeFontConfig
/usr/bin/ld: cannot find -lvtkGUISupportQtWebkit
/usr/bin/ld: cannot find -lvtkViewsQt
/usr/bin/ld: cannot find -lvtkRenderingExternal
/usr/bin/ld: cannot find -lvtkIOMPIImage
/usr/bin/ld: cannot find -lvtkFiltersReebGraph
/usr/bin/ld: cannot find -lvtkRenderingMatplotlib
/usr/bin/ld: cannot find -lvtkIOGeoJSON
/usr/bin/ld: cannot find -lvtkRenderingFreeTypeOpenGL
/usr/bin/ld: cannot find -lvtkIOMPIParallel
/usr/bin/ld: cannot find -lvtkFiltersParallelFlowPaths
/usr/bin/ld: cannot find -lvtkViewsGeovis
/usr/bin/ld: cannot find -lvtkLocalExample
/usr/bin/ld: cannot find -lvtkIOMySQL
/usr/bin/ld: cannot find -lvtkFiltersParallelStatistics
/usr/bin/ld: cannot find -lvtkFiltersParallelGeometry
/usr/bin/ld: cannot find -lvtkGUISupportQtSQL

as well as:
/usr/bin/ld: cannot find -lvtkftgl
/usr/bin/ld: cannot find -lvtkPythonInterpreter
/usr/bin/ld: cannot find -lvtkFiltersReebGraph
/usr/bin/ld: cannot find -lvtkLocalExample
/usr/bin/ld: cannot find -lvtkxdmf2

When checking the where python's vtk is pointing at (python -c "import vtk; print vtk.__file__") I get:
/home/gtinchev/rpg-navigation/software/build/lib/python2.7/site-packages/vtk/__init__.pyc

Which seems correct.
Do you have any suggestions where to get the aforementioned packages, if they're not included in the precompiled version of VTK7.1?

What is printing that error message?  Could you post this in the context of the build log?  Also can you post the CmakeCache.txt file?

I'm trying to build director using make drc -j6
Build log: https://pastebin.com/XwPB5L98
CMakeCache.txt: https://pastebin.com/iXNf1dWD
Thanks in advance for your help!

I think the problem is PCL.  You'll need to make sure that PCL is compiled with the same version of VTK 7.1.
PCL_DIR:PATH=/home/gtinchev/rpg-navigation/software/build/share/pcl-1.8
If you just rm the pcl build and re-run it then it might just work.  You can always help ensure it finds the correct version of VTK by setting this environment variable: export CMAKE_PREFIX_PATH=/home/gtinchev/rpg-navigation/software/build/share:$CMAKE_PREFIX_PATH but it might not be necessary.

You were correct, the PCL was build under the old version of VTK (6.2). I'm currently re-building PCL w/ VTK7.1 and will try again.
One thing I've noticed is that PCL is currently linking to Qt5 (system installed) - is that going to cause problems for director?

It might cause an issue, but it might be ok.  But you can apt-get install both qt4 and qt5 on ubuntu, they do not conflict at all.  Then you can build pcl with the system version of qt4, which is the same version that the packaged vtk7.1 uses.  Another alternative is to compile vtk from source using the system qt5, if you really want to use qt5.


recycler.collapse(id) and recycler.collapse(id) not working

recycler.collapse(id) and recycler.collapse(id) not working

take a look at this example of my custom API
https://github.com/ldmarz/le-angular-gantt/blob/master/demo.le/demo.controller.ts#L132

Yes

$scope.collapse = function (id) {
$scope.api.recycler.collapse(id)
}

and
 <button ng-click="collapse(1)"> collapseOne </button>
No effect.
But  <button ng-click="collapseAll()"> collpaseAll </button>
in same code working.


Missing documentation about how to use string-utf8

There should be an example how to create a string-utf8 object in Clarity.
For unsigned integers, you use prefix u like u42, and for string-utf8 as well: u"A wolf face 🐺" and (contact u"abc" string-var) and u"\u{1F384}"

did you see the recent merge that added this content: https://docs.blockstack.org/write-smart-contracts/values ?
If so, what's the delta you're looking for?

That is more for Web developer. I was looking for details for Clarity devs, like https://stackoverflow.com/questions/65186078/how-to-join-two-utf8-strings-together-in-clarity/65186079#65186079

Ah, so it sounds like you would want to have better examples for the types, not the JS clarity values. Maybe this page is best for that: https://docs.blockstack.org/references/language-types ?

Took an attempt at adding this to language types


Default userid is ignored

PS C:\Users\mystery\Downloads\LxRunOffline> .\LxRunOffline.exe config-uid -n ArchLinux
[WARNING] You are running LxRunOffline with administrator privileges. It may cause problems.
Do you want to continue? (y/n): y
1000
PS C:\Users\mystery\Downloads\LxRunOffline> bash
[root@DESKTOP-CHU61QE LxRunOffline]# id
uid=0(root) gid=0(root) groups=0(root)
[root@DESKTOP-CHU61QE LxRunOffline]# su mystery
[mystery@DESKTOP-CHU61QE LxRunOffline]$ id
uid=1000(mystery) gid=1000(mystery) groups=1000(mystery)
[mystery@DESKTOP-CHU61QE LxRunOffline]$

I'm using LxRunOffline 2.2.2. The same thing happens when launching bash or wsl directly or using a non-admin poweshell

I couldn't reproduce it on my machine. Please post the output of reg query HKCU\Software\Microsoft\Windows\Lxss /s so that I can diagnose the problem. Thanks.

I get the same problem: default UID (1000) is ignored. I tried running your command, and here is the output:
PS C:\> reg query HKCU\Software\Microsoft\Windows\CurrentVersion\Lxss /s

HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Lxss
    DefaultDistribution    REG_SZ    {19f44f69-5afa-43c4-baf5-e88349148f67}

HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Lxss\{19f44f69-5afa-43c4-baf5-e88349148f67}
    DistributionName    REG_SZ    Ubuntu
    State    REG_DWORD    0x1
    Version    REG_DWORD    0x1
    BasePath    REG_SZ    c:\tools\lxss
    DefaultUid    REG_DWORD    0x3e8

Seems to be more of a Windows problem:

After reboot bash no longer started, stating there are no distros installed. When I ran LxRunOffline:
> LxRunOffline.exe run -n Ubuntu
[ERROR] Win32 API returned an error: 0x800703FA.

I ran LxRunOffline.exe default -n Ubuntu again

Now default user is not ignored anymore

Registry values were all the same before and after I ran LxRunOffline.

Maybe related: Microsoft/WSL#2576.
Please try restarting the LxssManager service.

Closing as it seems to be Windows' fault.


fixing example for 2.7

Coverage remained the same at 87.987% when pulling 2b361450235050291b0b6c8edb67f85dfa927a88 on kailuowang-patch-1 into c1fdaae905f4e169e2289ffabd9fbdf3ee109caa on play2.7.


Code Helper using ~100% CPU

From @coreh on December 9, 2017 0:26

VSCode Version: 1.18.1
OS Version: 10.12.6

I'm getting "Code Helper" stuck at ~100% CPU

ps a~ $ ps aux | grep 65462
coreh            65462 100.0  1.7  3492052 281544   ??
A "Code Helper" process will start to use 100% CPU. Quitting VSCode does not terminate the Code Helper process, it needs to be killed from the Activity Monitor

Reproduces without extensions: Probably not (?)
(It doesn't trigger immediately, or consistently, but rather after several hours using the App, and I haven't been able to conclude if it will not trigger without extensions. However the ps output seems to indicate that the offending file is /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/typescript/out/utils/electronForkStart.js, so it's the built in TypeScript extension?)
Thanks in advance!
Copied from original issue: Microsoft/vscode#39949

From @bestvow on December 9, 2017 2:20
igore files such as
/**/node_modules

From @coreh on December 11, 2017 12:0
@bestvow Wouldn't that break type checks, autocomplete and "Jump to Definition" for the modules within node_modules?

From @bestvow on December 11, 2017 13:36
@coreh Would you like to show me your extensions?

@coreh Do you have a jsconfig or tsconfig? If so, can you please share it.
Also, please try downloading VS Code insiders and opening the workspace. Then, from the command line, run code-insiders --status. This should print some generic information about your workspace that can help us track down what is going wrong

Here's my tsconfig.json:
{
    "compilerOptions": {
        "outDir": "./build/",
        "sourceMap": true,
        "module": "commonjs",
        "target": "es5",
        "jsx": "react",
        "allowJs": true,
        "moduleResolution": "node",
        "strictNullChecks": true,
        "noImplicitAny": true,
        "baseUrl": ".",
        "downlevelIteration": true,
        "lib": [
            "dom",
            "dom.iterable",
            "es2016"
        ],
        "paths": {
            "common": [
                "client/common"
            ],
            "common/native": [
                "client/common/native"
            ],
            "components": [
                "client/components"
            ],
            "pages": [
                "client/pages"
            ],
            "state": [
                "client/state"
            ],
            "static": [
                "static"
            ],
            "react-phone-number-input": [
                "types/react-phone-number-input"
            ],
            "@yfuks/react-native-action-sheet": [
                "types/react-native-action-sheet"
            ],
            "exenv": [
                "types/exenv"
            ],
            "accept-language-parser": [
                "types/accept-language-parser"
            ],
            "react-masonry-infinite": [
                "types/react-masonry-infinite"
            ],
            "react-gateway": [
                "types/react-gateway"
            ],
            "slugify": [
                "types/slugify"
            ]
        },
        "skipLibCheck": true,
        "newLine": "LF"
    }
}

As soon as I get back home I'll install Insiders and try running that command. I must run that with my project already open, correct?

Uninstalling the TSlint plugin may help.
Now i get a quiet and cool mac : )


engine: make output Readline-friendly

The motivation behind this commit is to make Foreman respect
GNU Readline 1.
As Foreman is often used with Pry (or at least many people want to), and
Pry is dependent on Readline, using these tools together is very
problematic 2,3.
The problem is that when you start Pry by means of Foreman, Foreman
supresses output from Pry, so you don't see what you type. The output
can only be seen after pressing carriage return. Demonstration (a bit
hard to follow, but you may want to try it yourself; clone the Pry repo
and execute the same command):
http://showterm.io/3eb716461d6602ac90b09
Although this problem was reported by Pry users, any Readline
application is affected. This commit uses IRB for tests, because it also
depends on Readline, hence it suits for testing perfectly. Other
Readline dependent applications that I tested were GHCi, the Lua REPL,
Eshell (erlang shell).
Previously, Pry was using a trick, which allowed us to bypass this
problem 4. The fix was featured in Pry v<IP_ADDRESS>, but then it was
removed by accident. It's been readded on HEAD recently (not released
yet). However, the fix we currently have is still not perfect. Although
you can see your input immediately, you can't see your output now :D
Demo:
http://showterm.io/f648de27568d96a02f1b3
The fix breaks correct piping. Now, when you pipe Pry's output, it
doesn't output Readline's prompt and user's input. What's being piped is
only return values of executed expressions.
Demo:
http://showterm.io/3651389faf1fdc0d18211
To fix the missing output and pipe everything, including the user's
input, we need to set Readline.output correctly. However, if we do
that, we break minimal support for Foreman. So when I fixed Pry 5, I
figured it would be interesting to try to fix Foreman.
Now, it's time to talk about the changes here. I'm not very familiar
with Foreman, so take the code with a grain of salt. Basically, the
whole point of the fix is to read one character at a time and print it
immediately, instead of waiting for a newline character. This technique
allows us to prepend so-called markers (timestamps, if you wish) to a
Readline prompt and display the user's input. It works for all the
Readline apps that I mentioned above and doesn't break piping (that is,
tee).
Note. I had to remove $stdout.flush from the output method, because it
was causing thread deadlocks with the flush in #handle_io. The
deadlock appears when you use Foreman with Pry, pipe the output and
trigger SIGINT (control-c press). Example:
% foreman start -f Procfile | tee log
15:50:05 pryhere.1 | started with pid 26881
15:50:28 pryhere.1 | 1 pry(main)> <CTRL+C><CTRL+D>
15:50:49 pryhere.1 | 2 pry(main)> foreman/engine.rb:323:in
`synchronize': deadlock; recursive locking (ThreadError)
This is probably because of the race condition, when on SIGINT Foreman
prints stuff here 6, which flushes $stdout.

Foreman intentionally does not handle stdin on the terminal. If you are running multiple processes which of them should get the data from stdin?

That's a very good point. I just tested both IRB and Pry together in one Procfile and it is unusable in both cases, with and without this patch (in a slightly different manner, though). I cannot easily describe the behaviour, because it's unpredictable.
I assume if you add multiple applications that read stdin to Procfile, you know what you're doing. On the other hand, maybe it makes sense to read input sequentially from each application. So the first time I enter input for Pry, press Enter, then I enter input for IRB, then for Pry again.
Among Rubyists Pry is probably the only thing that reads from stdin. If you think this is still a bad idea, would you be so kind and drop a comment here explaining the situation, so I could close this issue and label it "WONTFIX" :)
P.S.: the Travis fail shouldn't be hard to fix, they just set their custom prompt.

I haven't read through this in detail nor researched this issue a lot, but I have a couple of comments.

Thanks @kyrylo for you work on this! This has been a long stading issue in byebug as well... In particular deivid-rodriguez/byebug#59 and deivid-rodriguez/byebug#63. If you would like to help with those, it would be really appreciated. Otherwise, I might have some questions when I get there!

@ddollar Does it make any sense to broadcast the input to every process? Is it possible at all?

I'd love to have support for pry when using foreman. Debugging is of course very useful when developing applications but foreman doesn't handle it well. Maybe this isn't the way to fix it, but I'd love to see something done about it.

I don't think it makes sense to send stdin to all processes since that will almost certainly never be what you want. Your typing is intended for one process.
My advice would be to not run irb or pry as part of your Procfile and instead use a separate shell to foreman run pry if you need an interactive console.

@ddollar Note that most people is not adding irb to their proc files, they're just dropping a binding.pry line in their code. What I don't understand is that foreman seems to work fine with pry or byebug (the input entered is correctly received and processed). The only problem is that the user input is not displayed while the user is typing, but after the command is processed, which make the console unusable. This patch seems to fix that, what's the problem with it?

+1

+1

+1

:+1:
@ddollar, any chance of reconsidering based on @deivid-rodriguez's message?  This is indeed an extremely common use case for pry and it's a shame to have to keep the main server process out of foreman in order to attach a debugger.

+1
I've been using pry.binding with foreman in the same way that @deivid-rodriguez mentioned. It'd certainly be nice to see what I type as I type it. I hope this'll be reconsidered.

Guys, have you actually tried this patch and verify it works? If not, could someone do it?

Yes, I have tested this patch and it does work for me.

Cool.

+1

For those looking for a workaround to using pry with foreman: pry-remote.  The biggest caveat for me is that tab completion doesn't work.

:+1:

I couldn't get pry-remote working, but I got this patch working by using kyrylo's branch of foreman. This is how:
gem uninstall foreman
git clone -b readline-support https://github.com/kyrylo/foreman.git
cd foreman
gem build foreman.gemspec
gem install foreman-0.77.0.gem

It's so great to be able to see what I'm typing again!

I strongly believe that all Ruby REPLs should read/write from /dev/tty the way LLDB and GDB do. If this happens via Readline, those apps will need to call the input= and output= writers with IO.console. The Python debugger does this also, and it drives me bananas! You can't even debug an app that reads from stdin this way.

It'd be great if we could reconsider this. The screwy interaction with pry is the only downside of foreman.

@drewish The problem is with pry, not foreman. Foreman has no way to demultiplex the standard I/O with multiple children. The solution is for only one process at a time to run pry against the terminal using IO.console (see above) or to directly run the command from the Procfile without forman while using pry. You can see a similar problem running better_errors against multiple web processes.


∯~∯💚<2024>💚LaTeSt!!FrEEApple iTunes Gift Cards Free Generator🎮 Up to $100 in FREE itunes gift codes in UNDER 5 minutes!

Get the gift of Apple? Here's how to use your gift card
You can use your Apple Gift card on your iPhone, Mac, iPad, in person or even on an Android phone. Here's how.
🌍📱👉 Link Ready - Get Link
KEY TAKEAWAYS
Apple Gift Cards can be redeemed for a wide range of Apple products and services, including apps, music, and books.
Redeeming an Apple Gift Card is easy using the native apps on iOS and iPadOS, and can also be done on a Mac through the App Store or on an Android device.
Apple Gift Cards can be used for online purchases at apple.com or at physical Apple Stores, but they cannot be used at third-party retailers and cannot be transferred to a bank account.
While Apple used to have a gift card line-up almost as varied as its product line, the company has moved to a simpler system: the Apple Gift Card. The company's card can be redeemed on anything from new music from iTunes to a new MacBook Pro or iPhone from the Apple Store. Don't worry if you've held on to an old iTunes Gift Card or Apple Store gift card -- they don't expire and can still be redeemed.
But with the Apple Gift Card's versatility comes a list of questions. How do you redeem an Apple Gift Card? What can you get with one? If you're an Android user gifted with one, can you still use it? Spoiler alert: yes, you can. We'll answer all of those questions and more below.
What can you buy with an Apple gift card?
While Apple used to offer several different types of gift cards, the company has largely moved to a single Apple Gift Card. This approach is great for consumers because these cards can be redeemed for anything from apps to physical products. Apple Gift Cards are a white card with a colorful logo on it and can be physical cards or emailed redemption codes.
the cards can also be redeemed on an iCloud+ subscription. They also work for some subscriptions in the App Store, but you may need to have a credit card on file to cover any costs once the balance of the card runs out.
When you redeem an Apple Gift Card, it's added to your account balance. In the U.S., it may also appear inside your [Apple Wallet]So, while you may redeem the card inside Apple Music, you can still use the remaining funds in the App Store or Apple Books, for example. However, once you add the balance to your account, you can't use the card in a physical Apple Store, unless you have [Apple Pay].
While Apple has moved to a single gift card, if you have an older App Store iTunes Gift Card or an Apple Store gift card, these have more limitations. An iTunes gift card can be redeemed for music and apps. An Apple Store gift card is redeemable for physical products online at [apple.com]
How to use an Apple gift card with an iPhone, iPad, or iPod Touch
Once you redeem an Apple Gift Card, it's added to your account balance, where you can use it across the App Store, iTunes, Apple Music, Apple TV, and Apple Books. You can redeem the card through several of Apple's native apps on iOS and iPadOS.
On an iPhone, iPad, or iPod Touch, open theApp Store, Apple Music, Apple TV, or Apple Books. (You can also use the Apple Store app, but there's one more step involved: click on your profile, then tap Apple Account Balance, then tap Redeem a gift card.)
Tap on your profile icon or your custom Memoji at the upper right corner to view your account. If you aren't already signed in, you'll need to sign in to your Apple ID at this point.
Tap Redeem Gift Card or Code.
Locate the 16-digit code on the back of your card -- on some physical gift cards, you'll need to scratch or peel off the label to show the redemption code.
If your 16-digit code has a black border around it, tap on Use Camera and use your device's camera to scan in the code following the on-screen instructions. If you have an older gift card without the box around the code or your camera is having trouble reading the numbers, tap on Enter Code Manually instead, then follow the instructions on the screen.
Tap Done.
How to use an Apple gift card on Mac
On your Mac, open the App Store.
Click on your name in the lower left corner to access your account.
Click on Redeem Gift Card.
Locate the 16-digit code on the back of the card -- you may need to scratch or peel off the protective coating. If the code is surrounded by a black border
Click the Use Camera button. If your code doesn't have a black box around it or your camera is having trouble reading the card.
Enter the code manually in the field.
Click Redeem.
How to use an Apple gift card at the Apple Store
Apple Gift cards can be used for physical products as well as digital ones. If you already redeemed the card using the method above and the balance is in your account, you can check out at apple.com with your account balance.
You can also redeem your gift card directly at checkout. If you're an Android user, you can use this option if you want to avoid creating an Apple ID. You'll need an Apple ID if you purchase something like a new iPhone, but you can also buy accessories like headphones or USB-C cables that don't require an account.
Can you use an Apple gift card on Android or PC?
Apple gift cards can still be redeemed if you are an Android or PC user, but the options are much more limited. On a Windows PC, you can redeem an Apple gift card to make purchases from the iTunes app. With an Android smartphone, these cards can be used from the Apple Music app.
While the products from apple.com are limited in compatibility with Windows and Android devices, an Apple Gift Card can be used to purchase some accessories, like earbuds from brands like Beats Pro, that work with non-Apple devices. Some Apple products, like AirPods, will also work with non-Apple devices, though features like Siri are incompatible.
To redeem an Apple gift card on iTunes purchases from a PC:
Open the iTunes app, or download it if you haven't already.
Using the menu bar at the top, click Account.
Click Redeem.
Follow the onscreen instructions and type in the 16-digit code on the back of your gift card.
If you use an Android smartphone, you can redeem the card on the Apple Music App:
Open the Apple Music App -- you'll need to download it if you haven't already.
Tap the three-dot menu button, then choose account. You'll need to log in to an Apple ID or create one if you haven't already.
Select Redeem Gift Card or Code.
Locate the 16-digit code on the back of your card and enter it into the field, then select Redeem.
Can I transfer my Apple gift card to my bank account?
No, like most gift cards, Apple Gift Cards are non-transferable and cannot be redeemed for cash except in locations where required by law.
Can I use an Apple gift card for Apple Pay?
Apple Gift Cards can not be used at third-party retailers, even after adding the balance to your account. While you can't shop at, say, Target or Best Buy, with a balance on your Apple Wallet, you can use your balance at physical Apple Stores using Apple Wallet.dsfgsz

How?


Add version numbering, make bind behavior optional, and add minified version

These are a few small things I've added and tested having worked with your excellent utility.

I've added basic version number so it's easier for webmasters to cache the files in the long term (e.g. making a folder for version 1.0.1 and know that clients can cache the file forever and pointing them to a new directory when the script is updated).
I've made the automatic bind behavior optional for people like me who use the plug-in with jQuery validator and run it as part of another bind event for an entire form.  The option is a bool the user can set after the callback, but the argument isn't required, so no users who depend on the auto-bind functionality will be affected.  (I tested it in the demo page and it worked perfectly without any changes to demo.js.)
I needed to toss in the .gitignore file to allow me to install coffee-script locally in the folder (my system doesn't like node...)
I've created a minified version using closure-compiler and it's basic compression level.  The file works as expected, and it is about half the size of the original.

All of these changes are simply suggestions: I totally understand if you don't want to officially implement any of them.  Hopefully, these changes will be useful.  And again, this is wonderful plug-in!  (In the next few weeks, I will get it onto our site as part of our current redesign, and your welcome to see it in action, when it's online--as you have mentioned in other threads liking to see it used.)
Thanks for all of the work you've put into this,
tmyoung

This is implemented in just released 1.0.


With proxy - No Response

Hi .. It takes forever to respond with using the proxy param.
Do you have any idea why ?
Thanks in advance.

Update package.


Double Check Session Connection Time

Set interval to replace connection?

I was referring to the maximum session time for a neo4j connection: iirc it's infinite but requires double checking


Pimp integrations

Fix eclipse integration styles (glsp css need to be loaded befor ours)
Dispatch ui extensions before model request (new UpdatePaletteItems action was needed, because those still needs to be loaded after the model request is sent)

Was already a lot better without the loading messages...


Bluetooth: Notification callback is called with incorrect connection reference

Describe the bug
Central connects to 2 peripherals and subscribes for notifications.
Notification callback is called with incorrect connection reference.
To Reproduce
Steps to reproduce the behavior:

Pull this commit 7c7dee105f42bafece900df44202689efc6d30c9
Run ~/zephyrproject/zephyr/tests/bluetooth/bsim_bt/bsim_test_conn_stress/tests_scripts/conn_stress.sh

Expected behavior

Notifications are sent in both directions between the central and connected peripherals
For each received notification packet, correct connection reference is reported by the notification callback function

Logs and console output
Please check the attached log
log.log

At line 548, Central received a notification with incorrect connection reference and incorrect destination address.
At line 563, the same notification packet was received again with the correct connection reference and correct destination address

Environment (please complete the following information):

OS: Ubuntu 22.04
Toolchain : Zephyr SDK 0.15.1
Commit SHA 7c7dee105f42bafece900df44202689efc6d30c9

After sprinkling some logs, it seems it is the controller that messes up the connection handle (last packet dumped in the log).

I got supervision timeout on first connection, either of the two below should get the test running with 2 active peripheral connections from the central.
Please try with this workaround: https://github.com/zephyrproject-rtos/zephyr/pull/51648/files
The open issue: https://github.com/zephyrproject-rtos/zephyr/issues/21995
Fix PR: https://github.com/zephyrproject-rtos/zephyr/pull/52012

There is also a Control Procedure failure with reason 0x28 instant passed due to a probable limitation in controller when it comes to using/updating the instant to a future event count when under large number of enqueued data PDUs to be transmitted.

Summary:

#21995 needs to be addressed for multiple connection support with Data Length Updates.
LLCP implementation needs investigation, to not fail with instant passed .

@kruithofa @thoh-ot @erbr-ot FYI.

@ahmedmoheb-nordic and @jori-nordic  Please try the changes in this PR https://github.com/zephyrproject-rtos/zephyr/pull/53095.
And with CONFIG_BT_LL_SW_LLCP_LEGACY=y in your test with the changes in the PR, connection disconnection due to instant passed should also be fixed.

@cvinayak You're right. Cherry-picking/applying the changes from 53095 and adding the config fixes the test.
Note that the patch alone doesn't make this particular issue disappear, only adding the LEGACY cfg fixes it.
Here's the branch: https://github.com/jori-nordic/zephyr/commits/reproduce-53007-fixed

@jori-nordic @ahmedmoheb-nordic Can you check if PR #52012 fixes this issue? You'll have to add the following lines to the prj.conf files:
CONFIG_BT_CTLR_ADVANCED_FEATURES=y
CONFIG_BT_CTLR_CONN_EVENT_LEN_UPDATE=y

@kruithofa Yes, applying changes in PR https://github.com/zephyrproject-rtos/zephyr/pull/52012 fixes the issue


• Java binary at: PATH_TO_JAVA_17\bin\java     X Cannot execute PATH_TO_JAVA_17\bin\java to determine the version

          Thanks for filing a separate issue. Looking at both `flutter doctor` outputs above, I notice Android Studio hasn't been installed yet. Maybe the issue can be solved if you install it. Can you try to see if it helps?

Originally posted by @huycozy in https://github.com/flutter/flutter/issues/135402#issuecomment-1733528695

see google.com for u question
https://www.baeldung.com/java-home-on-windows-mac-os-x-linux
for Mac run
echo $JAVA_HOME
source ~/.bash_profile
add:
export JAVA_HOME=/path/to/java_installation
you must know the java version you want to install. different IDEs, editors require different versions.

Hi @bone2020,
It seems you are asking for assistance on personal code/setting up your environment. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow with sufficient code sample and explanation.
Closing, as this isn't an issue with Flutter. If you disagree, please write in the comments and We will reopen it.
Thank you


[FEATURE REQ] Add request timeline to HttpClient and HttpTransportClient

See PR#7227. Consider renaming HttpClient as HttpGatewayClient.

Is this issue taken?

Completed by @simplynaveen20


[7.0.x] [RHDM-706] Liveness and Readiness probes for Decision Central OpenShi…

…ft Image deployment failed
Cherry-pick of #83
Signed-off-by: Jakub Schwan<EMAIL_ADDRESS>

Closing without merging. We will make no more changes to 7.0.x unless we are going to release something after 7.0.2.GA. Please re-submit this PR to the 7.1.x branch.


Add test for serializing/deserializing type without schema

This is now possible (albeit with a warning) thanks to #571.

Coverage remained the same at 93.676% when pulling d3719c7ce1cf3747e7a875f83da00e7a738c8081 on drdavella:tag-without-schema into b60c46a560887e893d49d90c3151ddbdc3902bd7 on spacetelescope:master.


enable debug mode

I changed the config doc settings to
WEBSOCKET_DEBUG=true
I get a message in the console to get ready for some serious flood of messages but other than that, I don't see any other changes.
I used to be able to see the PING/PONG traffic from the server  (before version 2.0) to the clients and now there are no extra messages.

Debug mode was more relevant to the prior java_websocket server than netty.  I'm sure there's a similar flag, but the output will not be the same.


Added My name

Related issue

What changes does this PR do?
Added my name.
Thanks EddieHub for this open source contribution❤
Screenshots (If any)
Questions

Also I'm not able to join the discord... Showing some error
Any one can help on to add me to the server

what is the issue you are facing ?

what is the issue you are facing ?

When I Clicked Accept invite , Its not Joining

try it in icognito mode and by disabling all extensions, also make sure u have a discord account and you are signed in. It would be preferable if you have Discord desktop installed.

http://discord.eddiehub.org

Oh sorry it didn't worked mate :(
Any one please @everyone @eddiejaoude  help to Add me up in discord

https://discord.gg/3F3sgdBM try using this


[React Native] Option to hide [Switch to Personal Wallet] button

Problem solved
Short description of the bug fixed or feature added
Changes made

[x] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[x] Manual tests: step by step instructions on how to test

@kien-ngo tested this in RN and it's working as expected. To be honest, I'm not sure this should live in the ConnectWallet component but in the smartWallet config and then inject it down to the UI. We can merge this for now though


[@lexical/website] Documentation: Added FAQ page around Lexical collaborative mode

Description

This adds FAQ page with to address common pitfalls around Yjs integration. This is a first PR in a series that shall improve documentation in this area.
Closes: #4442
Test plan
See Vercel bot preview

I think this is a great start. My overall feedback is to start even simpler when we write deeper docs about this, as the overall suggested architecture is unclear (what's the Yjs model vs the document, how does one map onto the other, etc)

I think the best supplement for this kind of documentation would be a working example that shows best practices, e.g. persistent yjs truth + database for indexing/snapshots. Possibly even have two models of the headless collab support since it's likely that people will want to use a SaaS collab channel like liveblocks.io or reflect.net so they may want a sync example that is a headless client rather than being directly coupled to a y-websocket-server service.
The helper functions here also seem like a great candidate to be included in @lexical/yjs (which also simplifies the docs and examples). Since @lexical/headless is so small I doubt adding it as a dependency would be a problem?

@acywatson @etrepum Thanks for feedback! I'm merging this as is for now and will do follow up later next week or sooner.
My plan is to make at least:

More docs
Dedicated Yjs example, based on WSS/WebRTC + storage on a file system + awareness
Extract Yjs from React package so it can be used effectively with or w/o React
Add collab mode to https://playground.lexical.dev/ (I have it working w/o server via WebRTC). Now it's only accessible when run locally
(stretch) Add public WSS server for https://playground.lexical.dev/ so it works cross browser


Established compatibility with the latest (<IP_ADDRESS>) dev-build of QuickShop-Hikari

Since there was a great refactoring when it comes to FQNs of events, a single occurrence had to be patched, in order to get the plugin back up and running.

Thanks for your contribution, I've long since run out of energy to deal with Hikari's frequently changing APIs

Oh yes, I totally understand where you're coming from on that! While I was still working on QuickShopSearch, I have been frustrated to have had to add a compatibility-layer for no good reason whatsoever.
I'm quite on and off about Minecraft as a whole myself, but since a friend of mine whose rather active, especially within the Chinese community, pointed out that a few people actually like and use your plugin, I thought I'd just provide them with a temporary release for the time being; the dev-build of QS-H is not out yet anyways. Always happy to help, :).


Bug/893 fix globals middleware and example logger middleware

Fixes #893

Generally, how to do logging with Grape comes up often, so thanks for looking into this.
Regarding this specific implementation, today the Globals middleware assigns env['grape.request'], which can give you env['grape.request'].params and the rest, no? So could you have written the logger you suggest here with just that? That's just for my understanding.
More importantly, if that middleware is not included today, I think we should delete it rather than extend it and move the suggested logger into a new gem called grape-logger-middleware or something like that. Would you like to do that?

@dblock +1. But then, should we remove @env['grape.request.params'], too? What's the difference between headers and params?

If the middleware is not currently used, then it doesn't matter how it changes (especially if we're going to delete it).

Yeah, I'd like to do that.  Currently the Globals middleware doesn't work at all, as written it will raise an exception every time because the request local is never set before it is used
So, yes, I will make this into a gem, but I would like to not have to set grape.request in the gem, since then any other gem would also be doing the same thing, and we all be up in each other's business.
I propose:

Globals becomes an optional standard (loaded only when requested) in grape.
Grape gem extensions will load it the middleware, and then utilize the preset 'env['grape.request']`, modifying and preserving it for each middleware in the stack.

Questions:

Are there any grape gem extensions doing similar things now?
How would the gem extension both to insert itself into the stack, and make sure it is inserted after the Globals middleware?

A while ago we got rid of preserving request across middleware and others. I think the best course of action is not to re-introduce Globals, but to implement request as Grape::Request.new(env) whenever you need it. It comes with params and everything else you may need and keeps middleware independent, which is how it's designed for.
I don't know about other grape middleware's out there, but I am sure they exist.
I think the consumer (developer) should insert the middleware manually and it should just be part of instructions.

Makes sense.  If I have time I will send a separate PR that removes Globals.
I will create a gem for a logger.  Would you consider a section in the readme for listing grape extensions?

We currently have this in the wiki: https://github.com/intridea/grape/wiki, people can add their own.

@dblock :+1:

If you don't think you'll get to the Globals removal, please open an issue? Appreciate it.


Can we pull template code into PHP File

Can we pull template code into PHP File, We are trying to create Joomla Module. Can you please help me.

The template file is loaded client side, PHP is server side. So I don't see the use case. Can you give more details?

As per joomla structure we create a view file default.php, we need to create all the HTML structure in this PHP file.   You can see following code. i want to load all HTML structure in the same PHP file. is it possible.
`
$document = & JFactory::getDocument();
$document->addStyleSheet(JURI::base() . 'modules/mod_xp_twitter/tmpl/css/jquery.socialfeed.css"  media="screen');
$document->addScript(JURI::base() . 'modules/mod_xp_twitter/tmpl/js/jquery.socialfeed.js');
//$document->addScript(JURI::base() . 'modules/mod_xp_twitter/tmpl/js/jquery.socialfeed.js');
//echo $xpertpress;
?>

`

is there any solution?

Can't you use template_html instead of template? In your example you use both parameters, but template overwrites template_html. Removing the line "template: "bower_comp...." will do.

i think istructions of social-feed script are really confusing.
If we take alook to the all parameters (the bottomo code example) all template options are used.
Alos the script in the demo is the Old one.
Who can manage to fix it?


ACT-692 Ability to automatically deploy from 'dev" to "staging" when a PR is merged

GIVEN I am a developer
AND I create a PR to merge dev with staging branches
AND once the PR is approved and merged
THEN  I would expect that the code on staging is automatically deployed to the staging environment.

#456 updated staging from dev was done via the automated ci/cd. Marking as done.


calendar icon show only

I want to show calendar icon only.
not show Input field.
Can you guide me?

You can use custom input or trigger element, see: https://vue3datepicker.com/slots/trigger-and-input/
In one of my projects I had to show the calendar in my own custom overlay, so I uses the inline option, see: https://vue3datepicker.com/props/modes/#inline

As @lupus78 said. Closing.


Fix server example in sdk to have timeout

Proposed changes
Change sdk server example to use http.Server so it has timeouts
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[x] I have run make install-tools and have attached any dependency changes to this pull request
[x] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[x] If applicable, I have updated any relevant documentation (README.md)
[ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (89fc334) 66.14% compared to head (752ad65) 66.14%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #518   +/-   ##
=======================================
  Coverage   66.14%   66.14%
=======================================
  Files         118      118
  Lines       13462    13462
=======================================
  Hits         8905     8905
  Misses       3957     3957
  Partials      600      600

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


support json type?

i get this exception,
[Doctrine\DBAL\DBALException]
Unknown database type json requested, Doctrine\DBAL\Platforms\MySQL57Platform may not support it.
Does not support mysql->json type？

Thank you for reporting this. It should be fixed in v2.0.0

Json fields still generated as : $table->text('data') instead of $table->json('data')


WIP: Add RuntimeHandler.RuntimeRoot

Hey there, the main intention of this PR is to resolve this TODO:
https://github.com/kubernetes-sigs/cri-o/blob/7a9445829c6165d62a75bcdab1843b264bb01f37/oci/oci_linux.go#L75-L80
My approach would be to introduce a RuntimeRoot for the RuntimeHandler and pass it around where it is necessary. This would also help in unit testing, since we now could use a temporarily run root directory for runc where we are privileged.
WDYT?
Todo:

[ ] add integration test with different run root for runc

Codecov Report

Merging #2175 into master will decrease coverage by 0.01%.
The diff coverage is 4%.

@@            Coverage Diff             @@
##           master    #2175      +/-   ##
==========================================
- Coverage   31.46%   31.44%   -0.02%
==========================================
  Files          76       76
  Lines        6770     6777       +7
==========================================
+ Hits         2130     2131       +1
- Misses       4522     4528       +6
  Partials      118      118

/test e2e_rhel

@saschagrunert @mrunalp I'm not clear on the need to differentiate this root path between runc and kata-runtime here. The factory can be initialized on a global path chosen by CRI-O, I don't think this needs to be specific to each runtime.
Also, I am not clear why CRI-O is calling into the libcontainer factory since runc should be responsible for this.
I think I'm looking for clarification here :slightly_smiling_face:

@saschagrunert @mrunalp I'm not clear on the need to differentiate this root path between runc and kata-runtime here. The factory can be initialized on a global path chosen by CRI-O, I don't think this needs to be specific to each runtime.

I'm not sure, since other runtimes (I know there is currently no real use case for that) could use different root directories which may clash with that one of runc. This is only hypothetical, so I would also be fine in just using a global config for all runtimes here. WDYT?

Also, I am not clear why CRI-O is calling into the libcontainer factory since runc should be responsible for this.
I think I'm looking for clarification here

I don't think that the current runc API provides what we would need here, but I guess with an additional API we could get rid of this code part completely.

/test e2e_rhel

/test e2e_rhel

After clarification from @mrunalp during the weekly CRI-O meeting, I realized the reason for this code and I think having this handled from RuntimeHandler.RuntimeRoot sounds good :+1:

/test e2e_rhel

LGTM

LGTM

/retest

/retest

/test e2e_fedora

/retest

Rebased on top of the latest master branch.

/lgtm

/test e2e_fedora
On Mon, Apr 8, 2019 at 10:37 AM OpenShift CI Robot<EMAIL_ADDRESS>wrote:

@saschagrunert https://github.com/saschagrunert: The following test
failed, say /retest to rerun them all:
Test name Commit Details Rerun command
ci/openshift-jenkins/e2e_fedora 296d80b
https://github.com/cri-o/cri-o/commit/296d80b6ac61b427583e1cfe06e74da73e7e875b
link
https://openshift-gce-devel.appspot.com/build/origin-federated-results/pr-logs/pull/cri-o_cri-o/2175/test_pull_request_crio_e2e_fedora/2320/ /test
e2e_fedora
Full PR test history
https://openshift-gce-devel.appspot.com/pr/cri-o_cri-o/2175. Your PR
dashboard https://openshift-gce-devel.appspot.com/pr/saschagrunert.
Please help us cut down on flakes by linking to
https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests
an open issue https://github.com/cri-o/cri-o/issues?q=is:issue+is:open
when you hit one in your PR.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cri-o/cri-o/pull/2175#issuecomment-480858942, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIqYJ8iyl8iYu-D0RdiwHtXclcuQMvNgks5ve1QXgaJpZM4cNMm7
.

/check-dco

/test e2e_fedora

/test e2e_rhel


Sports dataset on the test set how to get the results in the paper

Hello Dear Author:
Thank you for your contribution to the MOT community, I have some questions about how to get the effect of mix ocsort in the paper in the sports test set, do I need to use the weights XX_sports_mix.pth.tar or XX_sports_train.pth.tar of which yolox and mixformer you have provided for training or XX_sports_train. pth.tar?, and some specific hyperparameters, looking forward to your reply!

Hello,
You need to use the provided corresponding weights to get the results. For hyperparameters, you can just use the default values in https://github.com/MCG-NJU/MixSort/blob/main/tools/track_mixsort_oc.py


Capture node name of the worker node

Signed-off-by: Sai Sindhur Malleni<EMAIL_ADDRESS>

@rsevilla87 with current changes in https://github.com/cloud-bulldozer/benchmark-operator/pull/511 this no longer is required.

@rsevilla87 with current changes in https://github.com/cloud-bulldozer/benchmark-operator/pull/511 this no longer is required.


[WIP] automated screenshots

[ ] capture screenshots
[ ] decide where to upload screenshots. Current bucket is continuous-integration-screenshots, but we may want to upload coverage data as well
[ ] upload screenshots to s3
[ ] upload HTML page to s3
[ ] report screenshots via Danger

Merged #96


Indev focus fix

Previously, the following actions would cause undesireable effects (indev query not properly reseting, causing deleted objects' callbacks to be called).

Create an object objA in a group
Click objA.
Within the short_clicked callback:
a. Create a new object objB in the same group
b. Focus on objB object
c. Delete objA

Since the currently focused object is objB, deleting objA wouldn't trigger proc.reset_query, so other signals (like LV_EVENT_CLICKED) would be sent to the deleted objA's callback.
This PR explicitly stores the object pointer that indev is sending signals to, and lv_obj_del explicitly checks if this is the object being deleted to trigger proc.reset_query.
I only fixed this for the KEYPAD device since this is what I use and am most familiar with. Other devices (POINTER, ENCODER, BUTTON) may need a similar fix.
Other things:

I think lv_obj_del and delete_children should be refactored as they contain a lot of duplicate code.

Sorry that this includes #1069 , we can cherry pick if we want to clean this up.

Alright, I believe all comments have been addressed.

@BrianPugh Looks good, thank you.
I added the use of indev_obj_act for POINTERs too.
After some quick tests, all seems working.


Unable to login using Google Social Login with online access

Hi,
I'm able to login using Google as OAuth provider, but only with 'offline' access (providerData contains code). The 'online' access (providerData contains accessToken) fails in Stormpath's end, even though I'm supposedly doing things correctly. Could you verify that the online access in Stormpath's is working correctly?
I'm using JSON API method as described in https://docs.stormpath.com/nodejs/express/latest/social_login.html
Thanks,
Matti

Hey Matti,
I just tried Google Login with Express-Stormpath 3.1.2, and I'm confirming that it's working fine for me with the following permissions:

How are you generating the access token? I'm guessing you're using a Google SDK?

Hi Matti, can you also show us the scope string that you are passing to Google?  In our library, this is configured with the web.social.google.scope property.  I see that this new property is not documented, I've added this to our todo list in #380 - sorry about that!

Hi,
I should have opened this against the stormpath node.js module as I'm using getAccount(providerOptions, [options,] callback) function to implement Social Login as express-stormpath's implementation doesn't suit well for api-based social login (client's are mobile devices).
So I'm able to get this working:
var options = {   providerData: {     providerId: 'google',     **code**: 'abc1235'   } };
But not this one:
var options = {
providerData: {
providerId: 'google',
accessToken: 'abc1235'
}
};
This is can be a misconfiguration from my part, but I'd like you to confirm if the latter is confirmed as working with application.getAccount.
Thanks,
Matti

Hey Matti,
Yes, both of these should be valid. I work @ Stormpath on the iOS side. Can you tell us a little more about how you're doing mobile?
If you haven't already seen it, our iOS and Android SDKs will plug into Express-Stormpath and make mobile auth just work.
If you want to handle it manually -- can you please tell me which OS & client library you're using to get the Google Access Token?
Google has this thing called the ID Token which is different from the access token. If you see Google's documentation, there's both the ID Token and access token. Make sure you're sending the access token to Stormpath.

Hi edjiang,
I'm doing the Google OAuth on iOS side using GoogleSignIn version 3.0.0. Google's docs are bit outdated for v3.0.0, but I've understood that when I define

only GIDSignIn.sharedInstance().clientID in AppDelegate, then Google provides user.authentication.idToken as successful sign-in (which I can provide as accessToken for Stormpath's getAccount ?)
both GIDSignIn.sharedInstance().clientID and GIDSignIn.sharedInstance().serverClientID in AppDelegate, then Google provides user.serverAuthCode after successful sign-in (which I provide as code for Stormpath's getAccount and the offline login works okay)

Thanks,
Matti

Correction to the above: I'm actually POSTing providerData to /login endpoint as you noted.

Hmm I might have read Google's specs wrong, https://developers.google.com/identity/sign-in/ios/api/interface_g_i_d_authentication#instance-method-summary actually defines accessToken, idToken is meant for OpenId.
I'll test this out and close the issue with my regrets if this now works..

Yep, you should be using user.authentication.accessToken instead.
If you're doing this, I would suggest trying our iOS SDK! We actually implement all of this for you, saving quite a bit of work =]
https://github.com/stormpath/stormpath-sdk-ios

Thanks! (and sorry about this) I'll close the ticket.