How to make it do not black the screen light when playing video?

As the title

Freshwrapper supports XScreenSaver and freedesktop.org compatible screensavers, but doesn't have DPMS support. Do you have DPMS enabled? (What is output of xset q command?)

If I recall correctly, you are using Mint 17 (or 17.1). Its cinnamon-screensaver doesn't fully implement fd.o screensaver dbus interface, SimulateUserActivity call have no effect there.
Fortunately, it's easy enough to use Cinnamon-specific interface, which is implemented in 891bce3b8c60bcc9aed54aab775c717a421ce906.
Is it working now?

Screensaver also isn't prevented here.  In e6a2dd0485b0136739614bf94d4ebd0b4ff0e7d4 I've added internal X screen saver timer resetting, it should prevent monitor from switching off.
To simplify testing, DPMS timeouts can be changed by calling xset dpms 30 60 90. That command will make monitor to go to standby mode after 30 seconds of inactivity, to suspend mode after 60 seconds, and turn off after 90 seconds. Setting value to 0 disables corresponding timer.

@i-rinat It's working now, but only if the video is in fullscreen mode. If not in fullscreen, it still enters power saving mode.

@bootkiller, try code from tmp-log-update-activity branch. Launch browser from terminal and try to notice, whenever lines "ppb_flash_update_activity called" printed to stdout, for regular and fullscreen modes. I think they should appear once per circa 10 seconds.

@i-rinat Messages are printed on both modes.

Flash calls that method on a regular basis. And I forward those calls to XScreenSaver and dbus-based interfaces of other screen savers. I asked you to try instrumented code to find out whenever Flash still calls that method. It calls, it seems.
Meanwhile I tried to install Plasma 5 in Arch VM, and (org.freedesktop.ScreenSaver, /org/freedesktop/ScreenSaver, org.freedesktop.ScreenSaver.SimulateUserActivity) method doesn't work. I also tried org.kde.screensaver interface and all related paths, they all aren't working.
However I found that there is workaround: to call both SimulateUserActivity() and GetSessionIdleTime().

If you want to test dbus interface, here is a script that should prevent screen saver from launching:
while true; do qdbus org.freedesktop.ScreenSaver /org/freedesktop/ScreenSaver org.freedesktop.ScreenSaver.SimulateUserActivity; date; sleep 10; done

@bootkiller, I've added workaround for Plasma 5 screen locker in 540cb83aeef4b4d189850918c02fc94bcf4c2575.
Line quirk_plasma5_screensaver = 1 needs to be added to ~/.config/freshwrapper.conf in order to enable workaround code.

fixes in my laptop

@i-rinat, everything is working now, thanks.

OK, great.

Plasma's screen locker maintainer speaking here: please use Inhibit instead of SimulateUserActivity.

please use Inhibit instead of SimulateUserActivity.

Are there any reasons to prefer one over another?
Main obstacle on using Inhibit method is to understand when to call UnInhibit. PPAPI doesn't have equivalent of those methods. Instead it has a method PPB_Flash_13_0.UpdateActivity which should be called from time to time to inhibit screen saver, just like SimulateUserActivity. Converting to Inhibit/UnInhibit will require additional care, some magic timeout to detect whenever calls of PPB_Flash_13_0.UpdateActivity stopped or not.

Inhibit tells the screen locker infrastructure that an application wants to inhibit the lock screen. Ideally that also allows to show in the UI that the screen is inhibited and why. This is the way how it is meant to be used. With SimulateUserActivity you need to constantly call it, keeping both your application as well as the lock screen application busy with handling that. You don't know the idle interval used by the lock screen, so you need to use a small interval. And as you noticed, this is not how it's supposed to be used and it broke. Nobody tests whether the SimulateUserActivity works and it's only provided for legacy reasons.
Overall: if you are able to go to inhibit, please do so. You win a better integration in all desktops and you can in addition also inhibit the power management, so that the system doesn't suspend while the application is playing.

At the moment all I have is periodical calls from PepperFlash. There are no explicit "start inhibiting" and "stop inhibiting" calls in PPAPI, there is only SimulateUserActivity equivalent. Deducing stop time from periodic calls would be an error-prone procedure (as one must to either start a thread that does estimations, of throw another bunch of asynchronous tasks into event loop). I think it's better to rely on screen saver itself.
Why can't be SimulateUserActivity fixed in Plasma 5 screen locker itself? Doesn't look like it's not implemented there at all.

Also doesn't https://github.com/i-rinat/freshplayerplugin/issues/164#issuecomment-93294103 help to make it working?

There are no explicit "start inhibiting" and "stop inhibiting" calls in PPAPI, there is only SimulateUserActivity equivalent

That's bad. I can promise you that this won't work on Wayland. E.g. the system might go to suspend, because it's not inhibited and on Wayland SimulateUserActivity will only affect the lock screen and not any other applications.

Why can't be SimulateUserActivity fixed in Plasma 5 screen locker itself? Doesn't look like it's not implemented there at all.

It is implemented, see: https://quickgit.kde.org/?p=kscreenlocker.git&a=blob&h=dc216fb06ccb63e3a3fbce138bd052a93c2495bc&hb=687272c11e4d51ea014779eb6b7e767ef11f0996&f=interface.cpp line 157.
KIdleTime is windowing system specific and depends on what the windowing system provides. It's quite possible that the used implementation just cannot provide it.
I couldn't find what the SimulateUserActivity should do, but what I found were discussions about removing it from the DBus interface. Please consider it as deprecated and not supported by all DEs (I'm not sure whether GNOME has support for it).
I can only recommend you to go for inhibit.

I can promise you that this won't work on Wayland.

This whole NPAPI thing is tied to X server with rusty nails :smile:, so I'd not worry about Wayland incompatibility.

will only affect the lock screen

From interface name org.freedesktop.ScreenSaver I'd expect it to control screen saver (only), so that's fine.

It is implemented

Then why did you ever noticed this issue? Back when I tested Plasma 5 screen locker, it had a strange behavior. It ignored SimulateUserActivity calls when they were by themselves. But if one calls GetSessionIdleTime after SimulateUserActivity, latter began to work as expected. There is code that does so in freshwrapper. One needs to add quirk_plasma5_screensaver = 1 line to configuration file to enable that code.

I couldn't find what the SimulateUserActivity should do, but what I found were discussions about removing it from the DBus interface.

Could you please provide a link to such discussions (or maybe links)? I'd like to read them too.

I can only recommend you to go for inhibit.
It's quite possible that the used implementation just cannot provide it.

This is still looks weird for me. Assuming I have nothing other than periodic calls, each denoting screen saver shouldn't fire up, how I'm expected to convert it to Inhibit/UnInhibit calls if implementation cannot do the same? Either implementation can't do it because it's not possible, and then I can't do it too for the same reason. Or implementation can do it, and I can too.
I never wrote a screen saver, but here is how I would do it. There would be some counter, which is initialized to 0. Also there would be a thread which runs infinite cycle with small workload of incrementing the counter and subsequent sleep(1);. When counter reaches certain value (say, 15*60 for 15 minutes timeout), screen locks. Every user activity resets the counter to its initial value (0). Every call to SimulateUserActivity resets counter to its initial value (0). Call to Inhibit sets increment to 0. Call to UnInhibit sets increment to 1. (It's also worth to handle cases where multiple clients call Inhibit/UnInhibit with proper "depth" checking).
See? Both ways of temporary screen saver disable, fit straightforward implementation just fine. It's also easy to go from one to another if you, say, implement a standalone video player. You know when video starts, you know when it pauses, you know when it stops. Freshwrapper knows nothing. There is no reliable way to discern a video from some background animation. They're all just swf.

I'm not sure whether GNOME has support for it

All gnome-screensaver, cinnamon-screensaver, mate-screensaver, kde-screensaver (KDE4), Plasma 5 screenlocker have that method. Either on some specific interface, or at org.freedesktop.ScreenSaver. XScreenSaver have no D-Bus interface, but it also have a way to postpone activation.
From list above, the only difference is Plasma 5. But SimulateUserActivity also works there, if followed by GetSessionIdleTime.

This whole NPAPI thing is tied to X server with rusty nails :smile:, so I'd not worry about Wayland incompatibility.

It matters as soon as the lock screen is Wayland powered. What the client speaks is irrelevant.

Then why did you ever noticed this issue?

I don't use your application, sorry :-) I was just pointed here and thought I provide some feedback to help you integrate better.

Could you please provide a link to such discussions (or maybe links)?
the thread: http://lists.freedesktop.org/pipermail/xdg/2012-November/012576.html

how I'm expected to convert it to Inhibit/UnInhibit calls if implementation cannot do the same?

I don't know the API you are using. But I honestly cannot believe that there is nothing you could use for turning it into Inhibit/Uninhibit. Why do I think that SimulateUserActivity should not do it: because it misses context for the lock screen. It is a way to allow malicious applications from stopping the lock screen from showing up ever. On the other hand with inhibition we have more information we can provide and even give the user a choice to ignore inhibtion. That's just not possible with SimulateUserActivity.

Also there would be a thread which runs infinite cycle with small workload of incrementing the counter and subsequent sleep(1);

erm no :-) Of course a lock screen is not using an infinite cycle, but rather sleeps till it gets the idle event. This is one of the things I don't like about your SimulateUserActivity approach: it wakes up the screen locker. It should not do anything till it needs to do something.

I don't use your application, sorry :-)

Oh. I did a wrong assumption.

But I honestly cannot believe that there is nothing you could use for turning it into Inhibit/Uninhibit.

Of course it's possible. But I don't see why there is a need to reinvent the wheel, when all screen savers already have that.

Why do I think that SimulateUserActivity should not do it: because it misses context for the lock screen. It is a way to allow malicious applications from stopping the lock screen from showing up ever.

Doesn't look like a reasonable protection unless there is an external authority which can confirm application name. Malicious app could send various randomly-generated names, or send names of well-known applications.

This is one of the things I don't like about your SimulateUserActivity approach: it wakes up the screen locker. It should not do anything till it needs to do something.

So it creates an timeout and waits, that's great. But won't it be woken by real user activity: key presses, mouse movements? Why D-Bus call makes it worse?

Malicious app could send various randomly-generated names, or send names of well-known applications.

That's true. But we can expose it in the user interface and if there is a "VLC", which is not running at all, the user will notice. Also KDBus will provide us more information about the peer.
At the moment this is all theoretical. As you point out there is no real security. This will change, I'm working on protecting the lock screen against internal attacks on Wayland. With Wayland we will not be able to provide SimulateUserActivity.

So it creates an timeout and waits, that's great. But won't it be woken by real user activity: key presses, mouse movements?

 The lock screen doesn't get woken up, unless the idle time is reached to activate.

Why D-Bus call makes it worse?

Because it wakes up the idle application.

I was just pointed here and thought I provide some feedback to help you integrate better.

Thanks for information.
Although I'm not going to change approach at the moment, it could change in the near future. And information you provided helps to understand situation better.

Also KDBus will provide us more information about the peer.

With information about peer available, explicit name statement loses all security-related benefit. Why to bother with parameters if you can just test who the sender is?

 The lock screen doesn't get woken up, unless the idle time is reached to activate.

What wakes lock screen then? Doesn't it monitor user activity?

Because it wakes up the idle application.

I'm confused. Am I right that screen locker handles D-Bus calls, but user activity monitored by some other application?

Doesn't it monitor user activity?

It's using XSyncAlarm, see http://www.x.org/releases/X11R7.7/doc/xextproto/sync.html - which means it gets an event from the windowing system once the idle time is reached.

Am I right that screen locker handles D-Bus calls, but user activity monitored by some other application?

yes, user activity is monitored by Xserver.

xextproto/sync

Oh... I was unaware of its existence. Thanks!
I wonder what else X server have...


Save repetitions in CrossEntropyResult and make it into a JSON serializable dataclass

@XiaoMiQC

Mypy error same as #2769

I just made mypy ignore the error.


Reduce E2Es dependency on CI environment (2)

Follow-up of #3435
What this PR does / why we need it:
Some code of the E2Es assume the environment is GitHub, because it is referring to GitHub-specific variables. This PR focuses on the kserve/custom-model-grpc container image, so that no Python code of the E2Es using this image is referencing the github_sha variable.
Also, a small improvement on the get_isvc_endpoint utility function is done to use the schema in the endpoint specified in the status of the InferenceService, rather than hard-coding to plain-text HTTP. This adds compatibility for CI environments where KServe ConfigMap has been configured with urlScheme: https for the Ingress.
Release note:

NONE

@yuzisun @terrytangyuan Would you like to review? or assign reviewers?


React 17 compatibility

React 17 has been released and has no breaking changes, so it would be nice to have this package dependencies updated.
I've submited a pull request with the needed updates so you can accept it if you think it's ok .

Published in new version 4.3.0


chore(package): update sinon to version 2.0.0

Closes #43
https://greenkeeper.io/

Coverage remained the same at 100.0% when pulling 03b3a435349c9c077b1ea034a5b23e98773291aa on greenkeeper/sinon-2.0.0 into a0225eb9ffc094a90d6d9a6f06fa680c04a46390 on master.


Feature: Make Home Assistant API optional

Is your feature request related to a problem? Please describe.
I use HASS.Agent for the sensors and commands, and have no need for the Home Assistant API which requires an unnecessary connection to HA.
Describe the solution you'd like
Similar to local API, please make Home Assistant API optional as well.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Might be me missing a coffee or two today but could you please explain what you mean by "Home Assistant API"?

Hi, I mean Home Assistant API in the context of the picture below. We connect to the Home Assistant API by configuring the Server URI + API token. However, this is not needed where we only want to use MQTT for sensors and commands.

Thanks!

Oh, that's what you've meant :D
That's definitely doable but of course it would also mean that notifications, quick actions and passing the token to HA web requests would fail. I'll give it a thought and put it either in 2.1.(1) or 2.(2).0.


Disable CAdvisor Json Metrics completed

As kubernetes/kubernetes#99236 is merged.
part of https://github.com/kubernetes/enhancements/issues/2129

LGTM

/assign @tengqm

/assign @tengqm

/lgtm
/approve


error received while creating octopus Project

Error while creating service endpoint: The URL specified for parameter XXXXXX.octopus.app is not valid. Parameter name: endpoint.Url Error while creating build definition: The pipeline is not valid. A task is missing. The pipeline references a task called '179fac12-2402-486e-80cf-5a6a8571f7c0'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace: https://marketplace.visualstudio.com. (Task version 3.*, job 'Phase_1', step 'OctopusPack1'.)
AB#3575

Hi @muralikrishna188,
Thank you for reaching us.
The template requires Octopus extension, which should be installed and enabled in your organization before provisioning the project. If the extension is installed and it is still throwing the issue, please go back to the pipeline and try to add the octopus task manually. In some cases, these extensions are taking longer than expected time to install and enable.
The URL and the key which you enter at the time of provisioning the template will be validated at the time of create a service end point of type Octopus. This could be one of the issue. Please make sure that you are entering the right URL.
Let us know if you need any help.
Thanks.

Hi @muralikrishna188,
Thank you for reaching us.
The template requires Octopus extension, which should be installed and enabled in your organization before provisioning the project. If the extension is installed and it is still throwing the issue, please go back to the pipeline and try to add the octopus task manually. In some cases, these extensions are taking longer than expected time to install and enable.
The URL and the key which you enter at the time of provisioning the template will be validated at the time of create a service end point of type Octopus. This could be one of the issue. Please make sure that you are entering the right URL.
Let us know if you need any help.
Thanks.


Removed inline from lock/unlock

Removed inline from lock()/unlock() because it might get optimized away as discussed on Slack.

Caused undefined symbol error on unlock because unlock is not referenced in cpp file. Other fix could be calling unlock from destructor but might harm perf by a nanosecond or so.


addition of topology of seq_region

Added attribute for topology of seq_region in response to user request

we already have a 'is_circular' flag on some endpoints (overlap and lookup I believe), can we use the same attribute or does it have to be called topology?
also, the assembly_info test case needs updating to reflect that additional field
although travis is not showing this because of the missing JSON library, which I cannot reproduce locally
I know Json.pm is used for the eg lookup, do we need to add it in the cpan file?

On 09/06/15 09:56, Magali Ruffier wrote:

we already have a 'is_circular' flag on some endpoints (overlap and
lookup I believe), can we use the same attribute or does it have to be
called topology?

Sure, consistency makes sense - I thought topology was more obvious, but
as long as the information is there, thats all I'm after...
--
Dan Staines, PhD               Ensembl Genomes Technical Coordinator
EMBL-EBI                       Tel: +44-(0)1223-492507
Wellcome Trust Genome Campus   Fax: +44-(0)1223-494468
Cambridge CB10 1SD, UK         http://www.ensemblgenomes.org/

the Json dependency error has been fixed, the CHI library was updated to use a different JSON library instead and we were not importing it any more
I will close this ticket and let you re-submit a request with updated is_circular flag and updated test case


Fix Typo

Signed-off-by: ansijain<EMAIL_ADDRESS>
What this PR does / why we need it: Typo fix distint -> distinct
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm
/ok-to-test

/retest

/approve

Seems test failing are not related to the change
@stu-gott @maya-r


Fix Clippy warnings for rustc version 1.65

Two warnings have been introduced, and both are from the category: needless_borrow
This commit is to remove the needless borrow
error: this expression borrows a value the compiler would automatically borrow
   --> src/back/backend_handler.rs:101:40
    |
101 |                 unwrap_or_else_return!((&app).as_ref().ok_or(ResponseStatus::NotAuthenticated));
    |                                        ^^^^^^ help: change this to: `app`
    |
    = note: `-D clippy::needless-borrow` implied by `-D clippy::all`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

error: the borrowed expression implements the required traits
   --> src/key_info_managers/on_disk_manager/mod.rs:398:56
    |
398 |                     let mut key_info_file = File::open(&key_name_file_path).with_context(|| {
    |                                                        ^^^^^^^^^^^^^^^^^^^ help: change this to: `key_name_file_path`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

error: could not compile `parsec-service` due to 2 previous errors
warning: build failed, waiting for other jobs to finish...
error: could not compile `parsec-service` due to 2 previous errors

Signed-off-by: Mohamed Omar Asaker<EMAIL_ADDRESS>

Firstly, why do we set 777 if we need write permissions? Secondly, I'm not sure the new rust deployment procedure would require write permissions for everyone to the rust registry.

Why do we set 777? we need read and write permissions (not sure about the execute, but I believe we need it as well).
For the second question, it seems so (recently, it fails when parsec-client-2 tries to run cargo command)

Why do we set 777? we need read and write permissions (not sure about the execute, but I believe we need it as well).
For the second question, it seems so (recently, it fails when parsec-client-2 tries to run cargo command)

I think if we need to change permissions for the rust registry to by open for everyone then we do something wrong with either rust installation or build procedure. We need to fix that instead of adding a work-around.  Using "su" for running cargo also looks strange.
For example, a similar issue was fixed without  setting 777 permissions:
https://github.com/vibbits/rust-node-ci/issues/1

Why do we set 777? we need read and write permissions (not sure about the execute, but I believe we need it as well).
For the second question, it seems so (recently, it fails when parsec-client-2 tries to run cargo command)

I think if we need to change permissions for the rust registry to by open for everyone then we do something wrong with either rust installation or build procedure. We need to fix that instead of adding a work-around. Using "su" for running cargo also looks strange.
For example, a similar issue was fixed without setting 777 permissions: vibbits/rust-node-ci#1

I guess the benefit of using a common registry is not to download and build the same stuff again. By having a separate registry, we will lose this benefit. (as we are running the same test but for different users)
Not sure why su looks strange. Could you elaborate?

I guess the benefit of using a common registry is not to download and build the same stuff again. By having a separate registry, we will lose this benefit. (as we are running the same test but for different users)
Not sure why su looks strange. Could you elaborate?

But, instead of using 777 permissions I would prefer to see a common group created for parsec-client-1 and parsec-client-2 with using the group to set permissions.

I guess the benefit of using a common registry is not to download and build the same stuff again. By having a separate registry, we will lose this benefit. (as we are running the same test but for different users)
Not sure why su looks strange. Could you elaborate?

 But, instead of using 777 permissions I would prefer to see a common group created for parsec-client-1 and parsec-client-2 with using the group to set permissions.

Yea, it's mentioned in the slack discussion here.
I don't mind either approach, but the thing is, having a common group might (with unmask) allow users to access each other files (through group permission) might not be a good idea.  I will take a look at the tests closely.
Also, what is the reason you don't prefer granting permissions to the registry during testing?
@ionut-arm, Could you give us your insights on the common group?

I don't mind either approach, but the thing is, having a common group might (with unmask) allow users to access each other files (through group permission) might not be a good idea. I will take a look at the tests closely.
Also, what is the reason you don't prefer granting permissions to the registry during testing?

My reason is that for me permissions 777 always look like a hack saying "we didn't care to think about defining correct permissions". It's not a good example of using Parsec even if it's only in a test container
Parsec book recommends to use parsec-clients group for client applications that wish to use the Parsec service: https://parallaxsecond.github.io/parsec-book/getting_started/installation_options.html
So, instead of using 777 on the registry in ci.sh I would prefer to see in the docker image

parsec-clients group is created
parsec-client-1 and parsec-client-2 added into the group
the registry permissions set to parsec-cleints group ownership and 660 (or 664) if required

I think, if possible, using the group approach might be better. I think the tests we have which require multiple users are less about how the users can mess with each other, and more about showing that we can - in the Parsec service - facilitate user ID-based authentication. What the users can then do to each other outside of the service isn't really a concern (unless, of course, that has some implication on the actual tests).

Hey @anta5010, @ionut-arm
I did the changes and enabled the docker build, and everything is passing now.
Please check it out and if no further comments I'll push the docker and then disable the docker build in this PR.


Heads-Up: Title change to reflect changes to TOC

Docs Build status updates of commit c1d2403:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/mrlearning-base-ch1.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Small padding in the end

Why there is a small padding of around 16 dp in the end of the fab.. the view /text and icons are not aligned in center

Version 2.0.5 doesn't suffer from this issue. I went back to that version and the padding went away. I guess we have to wait for a new release for a fix.

check the latest version. If the problem still appears I will reopen the issue.


Update README.md - nanominer support announced

Nanominer will support etchash

Thanks. Please let me know once you have a release, so we can add instructions here.


PhysicsDirectSpaceState2D get_rest_info returned dictionary should also contain a field for the actual collision object

Godot version
3.3.2
System information
Windows 10
Issue description
Currently, the get_rest_info method in Physics2DDirectSpaceState returns a dictionary containing the following fields:

collider_id: The colliding object's ID.
linear_velocity: The colliding object's velocity Vector2.
metadata: The intersecting shape's metadata
normal: The object's surface normal at the intersection point.
point: The intersection point.
rid: The intersecting object's RID.
shape: The shape index of the colliding shape.

The problem is I cannot get the actual collision object from this information. (As far as I know) Both the "collider_id" and "RID" can´t be used anywhere to retrieve the actual object.
The intersect_point, intersect_ray and intersect_shape functions all return a collider (the actual object I am looking for) but get_rest_info does not.
I want to use get_rest_info in combination with cast_motion for a custom projectile system. Without the actual collider object, there is no point for me to use this function. (A projectile needs to interact with the object it collided with for applying damage etc. ) So currently I am using the collide_shape function (returns an array of collision points) with the intersect_ray function to get the results I need.
By looking at the class (Line: 481) responsible for those functions it looks like it should be very easy to add a collider field to the returned dictionary.
An alternative solution to my problem would be to add a function to the Physics2DServer to get the actual object from a RID or the collider_id.
I did not know if I should open a feature proposal or a bug report for this. If it's in the wrong place here I am happy to open a feature proposal instead.
Steps to reproduce
There are no steps to reproduce.
Minimal reproduction project
No response

var collider = instance_from_id(collider_id)

var collider = instance_from_id(collider_id)

@ICatRegister where does this function exist?

@ICatRegister Thank you very much :) It works.
I found it. It is global function in GDScript for anyone having the same issue.
I think it still would be nice to have to collider field in the returned dictionary or at least mention the "instance_from_id" function in the documentation.


Add SNAPSHOT publishing support to Publish-MavenPackages.ps1

Our release partners sometimes desire to publish SNAPSHOT packages through the partner release pipeline (this is something that we explicitly DO NOT support in our unified pipelines model, but we don't want to necessarily block it for partner pipelines. This issue is to track implementing SNAPSHOT publishing support in Publish-MavenPackages.ps1.

/cc @weshaggard

The account was already authorized, but the password contained a caret, ^, that was being scrubbed by maven before the http auth header was formed.   The password was rotated and I've verified snapshot publishing is possible on the account.   I still need to update the yaml and powershelll scripts to enable snapshot publishing through the pipeline.


Use Anki tags to customize card render

If the current note's tags are available within the DOM on render, then I can support variations of fill-blanks, as well as other layouts of the same information, without needing to create duplicate notes for associating to different card templates.
Examples:

If you add the show-logging tag, then render logs can be shown in the card body.
If you remove the show-choices tag, then the multiple choices for the cloze are not shown.
If you remove the show-source-file and show-source-line tags, that section will be hidden.
If you remove the show-prologue tag, the preceding sentence will be hidden, and similarly for show-epilogue.

After looking at the anatomy of a rendered card in the web browser version, it does not look like the tags are available in any of the DOM elements by default.
However, I can use the {{Tags}} variable to render the tags. The tags are rendered as a single plain string, tags delimited by spaces. This should work for me as long as the tags are placed in an identified container, and any quizgen tags never contain spaces.

Render control tags
Tags are an organizational concept already present in Anki, and the Quiz card generator (quizgen) includes them in its notes export in order to later quickly isolate notes within Anki that came from quizgen, and among them from what source texts they are. However, in addition to organizational tags, quizgen also uses card render control tags.
Render control tags serve as dynamic options to change how a note's cards are rendered.
Below is a list of supported render control tags:


Adding nutrition to uncountable list

What use case do you have for this word? Laravel has intentionally missed out less common words.

I'm building a API for diabetics where there is a table of nutrition information. Many apps in the IoT era will deal with this (wearable devices, smart phone apps, etc).

Probably more common than plankton. :D

;)


No clear install instructions

I'm looking to use this tool to help speed up simple let's plays videos that normally take a long time to edit when all I'm doing is removing silent periods. I would love to use this tool but I am having a hard time getting it to work. Please in the read me file (or another clearly marked file) can you leave install instructions on how to set this up
Thanks :D
p.s: I'm on Windows 10 if anyone can help me figure out what I need to do :)

install python from the official site.(check the OS and
using cmd locate the jumpcutter folder and run the command
pip install -r requirements.txt
enjoy using the thing

I have followed the steps you have stated above put when I try to run the jumpcitter,py file, cmd flashes up and then disappears again. Is there a specific way to start the program or a step I may have missed? (This is my first time working with python files)

you don't run the .py file.

I tried on windows 10 too and it didn't work

I tried on windows 10 too and it didn't work

you're on github, (on the internet for that matter) the reply "it didn't work" doesn't say anything. Be specific and don't expect people to read your mind from across the world

... Have you ever used python before?
On Sun, May 5, 2019, 4:13 AM CompressedWizard<EMAIL_ADDRESS>wrote:

I tried on windows 10 too and it didn't work
you're on github, (on the internet for that matter) the reply "it didn't
work" doesn't say anything. Be specific and don't expect people to read
your mind from across the world
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/carykh/jumpcutter/issues/93#issuecomment-489416268,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKVJJ3OD2XTNJSVESLOSC63PT26LXANCNFSM4HHNJIZA
.

I'm looking to use this tool to help speed up simple let's plays videos that normally take a long time to edit when all I'm doing is removing silent periods. I would love to use this tool but I am having a hard time getting it to work. Please in the read me file (or another clearly marked file) can you leave install instructions on how to set this up
Thanks :D
p.s: I'm on Windows 10 if anyone can help me figure out what I need to do :)
made this if that helps you just run the scripts
https://github.com/Aaronlikesmath/Easy-Carykh-jumpcutter/releases/tag/V1.0


Added: support to select picker for ready for pickup order ( #23egwz8 ).

Will be done differently. Closing


yarn lint-staged Parsing error: The keyword 'const' is reserved

📜 Description
yarn lint-staged throw error on scripts folder.
👍 Expected behavior
It should not through error. either it should be ignored or pass the lint.
👎 Actual Behavior with Screenshots
it should not throw error
👟 Reproduction steps
run following command
yarn lint-staged
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

@lalit774, any chance you could include the full error? Also details about the files? I'm assuming you are running this against the Backstage repo itself, is that correct?
Currently if I run yarn lint-staged I just get "No staged files found." as output

@awanlin yes, I am running against backstage repo. I noticed this issue for stage file for scripts folder on backstage root and i had following files. for time being, I have added script folder in .eslintignore and it works.
check-docs-quality.js
prepare-release.js
techdocs-cli.js

Thanks, the tricky thing with this is that you need changes to be able to really trigger it. I did a fresh clone of master as well as my own fork updated to latest and all I get is when I run yarn lint-staged is the "No staged files found." as output like before.
Would you be able to provide more details on what you are working on and what the changes are? That could help figure out what's going on but like I said, without the changes it's going to be pretty hard

Agree with Awanlin here, there's currently not enough information to confirm that this is actually a bug and something that needs fixing in the main repository. I can edit said script, stage it and run the commands without problems. Please submit enough information for us to understand and reproduce the problem you're seeing.

OK, this helps, so the command you ran was yarn test and then? Sorry, just can't tell this part from the image
As for documentation on linting there is some here, I found this by doing a search for lint: https://backstage.io/docs/local-dev/cli-build-system/#linting

Wanna highlight that it's this that makes the scripts lint in the main repo: https://github.com/backstage/backstage/blob/master/scripts/.eslintrc.js. In general you'll need either an ESLint config or ignore for any code in the repo. We don't ship the scripts folder in new apps, but feel free to copy ours 😁

Gonna close this since adding .eslinrc* is the solution to this issue.


Add uuidv3

Is your feature request related to a problem? Please describe.
There is currently no way of generating repeatable/reproducible UUIDs. They are always random, even if a constant random seed is provided.
Describe the solution you'd like
Add internet().uuidv3() which is based on a parameter we can control. We could simply generate random bytes using the random service (which in turn uses the constant random seed). The output would be of the same structure as the existing uuid() method. It would merely be uuid v3 instead of v4.
This has been discussed in #278. Created a new issue to give this issue/solution a bit more "exposure/visibility".

PR is merged.


Create interactive Google Assistant action

Along the same lines as an Amazon Skill, let's try to create an interactive Google Assistant action with Liturgical facts!
Getting started information can be found here: https://codelabs.developers.google.com/codelabs/actions-builder-1/#0

this will need to be done in it's own repository, not in the main API repository


refactor: Use named collections for dictionaries

This allows us to change admin passwords and keep
credentials out of the migrations.

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate.

@Ian2012 Even though your pull request wasn’t merged, please take a moment  to answer a two question survey so we can improve your experience in the future.


Error: Command not found: create

I'm trying to learn how to persist data from a container (the application is a web scraper I've written). When I follow the instructions in the user guide, docker gives me an error message:
$ sudo docker create -v /archive --name archive maltrieve
Error: Command not found: create
Error: Command not found: -v
Usage: docker [OPTIONS] COMMAND [arg...]
 -H=[unix:///var/run/docker.sock]: tcp://host:port to bind/connect to or unix://path/to/socket to use

This system runs Ubuntu 14.04 LTS.
$ sudo docker version
Client version: 1.0.1
Client API version: 1.12
Go version (client): go1.2.1
Git commit (client): 990021a
Server version: 1.0.1
Server API version: 1.12
Go version (server): go1.2.1
Git commit (server): 990021a

Looks like you are using an older version of docker; v1.0.1. The create command was added in docker v1.3.0.
You can find instructions here to upgrade your version of docker to the latest version.

Thanks, that worked great!


Hello world demo screenshots

Once the demo site has been created, we can add screenshots into our documentation
This issue is complete when:

[ ] There are screenshots of demo site in the README.md file in this repo

Blocked until the starter has been redesigned.


Using other RequestCultureProviders with UrlCulturePovider

Hi I want to use the UrlCultureProvider combined with QueryString, Cookies,AcceptLanguage Header Culture providers,
If I add UrlCulture at the end the functionality is not working , If i add it to the start the QueryString is not working .Do you have any suggestions on this?
options.RequestCultureProviders = new List()
{
new QueryStringRequestCultureProvider(),
new CookieRequestCultureProvider(),
new AcceptLanguageHeaderRequestCultureProvider(),
new UrlCultureProvider(options.SupportedCultures)
};

I don't think they are compatible, not as the UrlCultureProvider is at the moment at least.
The UrlCultureProvider redirects to default culture url if the start of your url doesn't contain a valid culture.
So for the cultures en & fi with en being default culture.
So in the case of example.com/?culture=fi I think it would still assume you're on english culture.
What kind of use case are you going for? You want example.com to be based on cookies or browser settings?

@haestflod I want the example.com to open in fi but url needs to be example.com and not example.com/es/. But this is not working as i cannot give the controllername in spanish.
Then i want the request culture provider to take cookies then accept headers in that order. and combined it with UrlCulture providers.

@dkadd This line: https://github.com/haestflod/asp.net-core-mvc-localized-routing/blob/master/localization/Localization/UrlCultureProvider.cs#L64
It defaults to always return a culture so the url culture provider always finds one.
I don't know what the "not found" value is, maybe null or empty string. But I think that's what you replace if you want it to accept cookies & accept headers.


[CI] Fix example fmt

my observation is if we don't put --all it won't fmt the entity crates

my observation is if we don't put --all it won't fmt the entity crates

Indeed!


FEATURE: enable support for conditional inclusing of each tag

Opening this in https://github.com/HackerHappyHour/tagging-strategy/issues/42 as well

allow the use of ::true or ::false after each tag to specify whether or not that tag should be produced.
This will allow the use of expressions to determine whether or not a tag should be produced.
Here's a real example: you may want to produce the {{ major }} pattern under most conditions, but it doesn't
make sense to do so in the given tag is a pre-release.  You could do something like:
-
  with:
    tags: |
      {{major}}::${{ github.event.action != 'prerelease' }}
      {{major}}.{{minor}}.{{patch}}

closed in favor of #30

I'm closing this since I implemented it with the latest release of HackerHappyHour/tagging-strategy, and it's getting to the point with this action IMO where it's too tightly coupled to the workflow event, with too much indirect control and output to really be capable of the flexibility I need.  Will open a separate issue with a proposal to redesign ghaction-docker-meta tagging abilities.


[debugging-the-compiler] Explain how one can use -R to set a breakpoint at a relative assembly offset.

Specifically, lldb resolves the breakpoint normally and then adjusts the
breakpoint address according to the value passed to -R. This is really useful
with -n (function name) and -M (method name) breakpoints across multiple-runs of
the debugger.

@swift-ci smoke test and merge


"UpdateLiveStats" inventory info

I would like to have info on how many Pokeballs/Great Balls/Razz Berry we have right now.
But it's not in "UpdateLiveStats", I think.
Also, is there a way that this info can have a line break?

#3950

In addition to this, i'm not sure if i'm not just missing it but would we be able to get distance to egg hatch as a live stat?

@TheCannings you would be better off making a new feature request as this is just waiting on being merged and will be closed as soon as it is

ahh thanks @k4n30 new to all this git jazz, have created it #4150


Clientlib WARN messages for ACS AEM Commons clientlibs

In testing #1165, I see the following WARN messages on a clean 6.3 instance + ACS AEM Commons 3.11.0:
06.11.2017 13:03:03.869 *WARN* [sling-oak-observation-27] com.adobe.granite.ui.clientlibs.impl.FileBundle Referenced path in /etc/clientlibs/acs-commons/angularCoral/js.txt does not exist: /etc/clientlibs/acs-commons/angularCoral/js/select.js
06.11.2017 13:03:03.902 *WARN* [sling-oak-observation-27] com.adobe.granite.ui.clientlibs.impl.FileBundle Referenced path in /etc/clientlibs/acs-commons/coralui-wrapped/css.txt does not exist: /etc/clientlibs/granite/coralui/less/cui-wrapped.less
06.11.2017 13:03:14.208 *WARN* [0:0:0:0:0:0:0:1 [1509991394186] GET /content/we-retail/us/en.html HTTP/1.1] com.adobe.granite.ui.clientlibs.impl.LibraryCacheImpl Library /apps/acs-commons/components/utilities/errorpagehandler/preview/clientlibs specified embedded that could not be resolved: coralui
06.11.2017 13:03:14.208 *WARN* [0:0:0:0:0:0:0:1 [1509991394186] GET /content/we-retail/us/en.html HTTP/1.1] com.adobe.granite.ui.clientlibs.impl.LibraryCacheImpl Library /apps/acs-commons/components/utilities/exporters/users-to-csv/clientlibs specified dependencies that could not be resolved: acs-commons.angularjs.v1.3
06.11.2017 13:03:14.208 *WARN* [0:0:0:0:0:0:0:1 [1509991394186] GET /content/we-retail/us/en.html HTTP/1.1] com.adobe.granite.ui.clientlibs.impl.LibraryCacheImpl Library /apps/acs-commons/components/utilities/quickly/clientlibs specified embedded that could not be resolved: acs-commons.json2

@davidjgonzalez can you take a look at this?

Sure thing


Add VS build tools installation

Description
Added Visual Studio build tools installation for Windows 2016/2019 images with vs_buildtools.exe installer. Minor changes for current VS installation, testing and documentation.
This functionality creates an opportunity to install workloads corresponding to Visual Studio build tools.
Seems that vs_enterprise.exe can't install properly the Microsoft.VisualStudio.Workload.WebBuildTools workload, the installation process ends without errors, but the workload components are not installed. The vs_buildtools.exe looks like the proper way of installation for build tools workloads (as said in documentation, Microsoft docs link), Microsoft.VisualStudio.Workload.WebBuildTools in this case.
After the installation there will be 2 "instances" of VS - one is VS itself and another one is build tools instance:
PS C:\WINDOWS\system32> Get-VSSetupInstance  | Select-Object DisplayName,InstanceId

DisplayName                    InstanceId
-----------                    ----------
Visual Studio Build Tools 2019 c97bc318
Visual Studio Enterprise 2019  fd5d5002

with different subset of components installed:
PS C:\WINDOWS\system32> Get-VisualStudioComponents -VSInstallType VS

Package                                     Version
-------                                     -------
Microsoft.VisualStudio.Component.CoreEditor 16.1.28811.260
Microsoft.VisualStudio.Workload.CoreEditor  16.0.28315.86

PS C:\WINDOWS\system32> Get-VisualStudioComponents -VSInstallType BuildTools

Package                                               Version
-------                                               -------
Microsoft.Component.MSBuild                           16.5.29515.121
Microsoft.Net.Component.4.7.2.TargetingPack           16.7.30310.162
Microsoft.Net.Component.4.8.SDK                       16.4.29313.120
Microsoft.Net.ComponentGroup.DevelopmentPrerequisites 16.3.29207.166
Microsoft.VisualStudio.Component.CoreBuildTools       16.7.30310.162
Microsoft.VisualStudio.Component.NuGet.BuildTools     16.1.28829.92
Microsoft.VisualStudio.Component.Roslyn.Compiler      16.0.28714.129
Microsoft.VisualStudio.Component.TypeScript.3.9       16.0.30322.227
Microsoft.VisualStudio.Web.BuildTools.ComponentGroup  16.0.28516.191
Microsoft.VisualStudio.Workload.MSBuildTools          16.0.28516.191
Microsoft.VisualStudio.Workload.WebBuildTools         16.7.30310.162

Related issue:
https://github.com/actions/virtual-environments/issues/1470
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2016

/azp run windows2016, windows2019


TileMapEditor::_bucket_fill Check autotile coordinates only if autotile is selected

Fixes #48388.
Cherry-pickable for 3.3, 3.x.

Thanks!

Cherry-picked for 3.4.

Cherry-picked for 3.3.1.


Execute require inside MetaScript context

Now if we load module inside MetaScript and pass require to it's context so module can load another module, this require will be executed in the global context of all application, not in internal MetaScript context. We need special require implementation to be executed inside MetaScript context.

@AliusDieMorietur https://github.com/metarhia/metavm/pull/58 landed but I'll close this issue after internal module nested require and commonjs cache implemented. Also need more tests for permissions. I hope will do that next week.

Main feature we need is loading npm packages in sandboxes with support of sandboxing nesting


Printing too many output lines per epoch

Hello,
I just installed Keras with Theano backend.  I am using the example code from
https://github.com/fchollet/keras/blob/master/examples/imdb_lstm.py
The program can run but print so many lines per epoch:
Using Theano backend.
WARNING (theano.sandbox.cuda): The cuda backend is deprecated and will be removed in the next release.  Please switch to the gpuarray backend. You can get more information about how to switch at this URL:
https://github.com/Theano/Theano/wiki/Converting-to-the-new-gpu-back-end(gpuarray)
Using gpu device 0: GeForce GTX 1060 6GB (CNMeM is enabled with initial size: 82.5% of memory, cuDNN 5110)
Loading data...
25000 train sequences
25000 test sequences
Pad sequences (samples x time)
X_train shape: (25000L, 80L)
X_test shape: (25000L, 80L)
Build model...
Train...
Train on 25000 samples, validate on 25000 samples
Epoch 1/15
32/25000 [..............................] - ETA: 78s - loss: 0.6960 - acc: 0.4062
64/25000 [..............................] - ETA: 78s - loss: 0.6911 - acc: 0.4844
96/25000 [..............................] - ETA: 76s - loss: 0.6895 - acc: 0.5312
128/25000 [..............................] - ETA: 73s - loss: 0.6862 - acc: 0.5703
160/25000 [..............................] - ETA: 79s - loss: 0.6903 - acc: 0.5563
192/25000 [..............................] - ETA: 77s - loss: 0.6834 - acc: 0.5833
224/25000 [..............................] - ETA: 76s - loss: 0.6927 - acc: 0.5491
256/25000 [..............................] - ETA: 78s - loss: 0.6915 - acc: 0.5469
288/25000 [..............................] - ETA: 77s - loss: 0.6915 - acc: 0.5486
320/25000 [..............................] - ETA: 79s - loss: 0.6966 - acc: 0.5219
352/25000 [..............................] - ETA: 78s - loss: 0.6984 - acc: 0.5057
384/25000 [..............................] - ETA: 78s - loss: 0.6976 - acc: 0.5026
......
24832/25000 [============================>.] - ETA: 0s - loss: 0.5113 - acc: 0.7486
24864/25000 [============================>.] - ETA: 0s - loss: 0.5112 - acc: 0.7486
24896/25000 [============================>.] - ETA: 0s - loss: 0.5112 - acc: 0.7486
24928/25000 [============================>.] - ETA: 0s - loss: 0.5110 - acc: 0.7488
24960/25000 [============================>.] - ETA: 0s - loss: 0.5110 - acc: 0.7490
24992/25000 [============================>.] - ETA: 0s - loss: 0.5109 - acc: 0.7490
25000/25000 [==============================] - 101s - loss: 0.5109 - acc: 0.7491 - val_loss: 0.4067 - val_acc: 0.8239
Epoch 2/15
32/25000 [..............................] - ETA: 74s - loss: 0.1730 - acc: 0.9688
64/25000 [..............................] - ETA: 71s - loss: 0.3168 - acc: 0.8594
96/25000 [..............................] - ETA: 70s - loss: 0.3654 - acc: 0.8229
......
Is it normal? How can I fix it? Thanks!

Yes you can use different verbose settings in your model.fit method (https://keras.io/models/model/#fit).

@tpinetz Thank you for the suggestion.
Actually I tried these verbose settings but it did not work.

Is there a final resolution to this please? What is the suggested action?
Thank you.


Check for "edit-tables" permission

Like in https://github.com/simonw/datasette-configure-fts/issues/3

This permission should take the current database and table into account, so administrators can configure permission for specific users to edit tables only within a specific database (or limit them to editing specific columns).
Import to consider what this means for table renaming - if you only have permission to edit one specific table you should not be able to rename it. Refs #19.

I'm going to include a mechanism for setting permissions in the plugin configuration, using an "allow" block - and code similar to the default Datasette permissions code in https://github.com/simonw/datasette/blob/448d13ea6b15a9c5c9165a4f6049d073f1524a5e/datasette/default_permissions.py

For the first version I'm just going to have a global permission, with no resource checks.

Split out advanced permissions stuff into separate issues. This is good enough for the 0.3 milestone.


Institution Views Repayment Rate Letter

As a (User Type/Persona) I want (Feature/enhancement) So That (Value, why is this wanted, what is the user trying to accomplish)
Additional Context
As part of our integration work with EDW
Comments from Steph Williams
The repayment letter is tied to the Pan Canadian Designation Framework (Canada Student Loan Program implementation 2003). It is expected that Provinces and Territories who are part of the federal Canada Student Financial Assistance Program participate in providing this data to schools.  This particular report is of concern going forward as it is currently tied to SFAS program codes and SIMS is now a mandatory CIP code with an option of SFAS program code. There is great danger with data integrity in EDW creating a cross walk between an institutions CIP code and SFAS code without institution confirmation that the cross walk is correct  (last time this type of activity was done with out each institution reviewing the data was more than 30% incorrect). Because of the use of an SFAS program code being left to schools to choose, there is inconsistency and no way to truly know if the code has any relationship to the actual program and CIP code.
I believe these reports go to public and private institutions. These reports are connected to Private institution designation as they have more rigorous requirements.
The second report is data that is sent to the financial aid office and Institutional Research and Planning Department at each Public Institution.  These reports provide funding summary information within an institution and across the sector.
BCIT-British-Columbia-Institute-of-Technology.pdf
Acceptance Criteria

[ ] Given (Context), When (action carried out), Then (expected outcome)
[ ] Given (Context), When (action carried out), Then (expected outcome)

This was considered a duplicate ticket as part of 2x planning


[Feature Request] Possible to add VRC+ management menu?

Is it possible to add shortcuts to manage VRC+ subscription from VRCX?
Thanks!

VRCX Runs through the games API, VRCPlus subscriptions are run through the games internal SteamAPI so its possible

Which SteamID type does it use? i can figure out how to fetch it myself if you know how to get the link

REGKEY: Computer/HKEY_CURRENT_USER/SOFTWARE/Valve/Steam/ActiveProcess/ActiveUser
This is the Active Users Steam32 ID

Or you can get the current logged in username:
Computer/HKEY_CURRENT_USER/SOFTWARE/Valve/Steam/AutoLoginUser

Currently isn't possible but will be in the future once steam isn't required.


Add align to new line option to NormalizeSeparator

Relates #507

Codecov Report
Patch coverage: 100.00% and project coverage change: -2.00 :warning:

Comparison is base (8a21ef7) 96.95% compared to head (55d20cf) 94.96%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #532      +/-   ##
==========================================
- Coverage   96.95%   94.96%   -2.00%
==========================================
  Files          49       49
  Lines        4800     4825      +25
==========================================
- Hits         4654     4582      -72
- Misses        146      243      +97

Impacted Files
Coverage Δ

robotidy/transformers/NormalizeSeparators.py
100.00% <100.00%> (ø)

... and 9 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Updated with more details and explanation

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

LGTM! This adds some great detail, @pgaret , thanks for filling this in 👍 💻 🖱


⚠ Bofrost.lu has degraded performance

In c3e0c87, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 894 ms

Resolved: Bofrost.lu performance has improved in 70691b9 after 6 minutes.


Composable not ready to recieve viewmodel despite instruction to do so

(https://developer.android.com/codelabs/basic-android-kotlin-compose-add-repository?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-5-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-add-repository#5)
The problem is in step 6 of the "Load and display images from the internet" codelab
In instruction 8 of step 6 it says to "Update the code to pass variable viewModel as an argument to the MarsPhotosApp composable." However, the MarsPhotosApp composable has not been updated to accept a viewModel in the constructor. I checked through previous steps and am using the repo-starter but I could not find an instruction to update the composable anywhere. If however I am mistaken I am very sorry for wasting your time with a false report.
Versions
Android Studio Electric Eel | 2022.1.1 Patch 2
API 31

Duplicate of #38, #31, #29, #25, #20.


[BUG] Nested Function With Sort Doesn't Sort Inner Hits

What is the bug?
Sorting on nested fields does not sort inner_hits of response.
Dataset:
{"index":{"_id":"1"}}
{"message":[{"info":"c"},{"info":"a"}]}
{"index":{"_id":"2"}}
{"message":{"info":"d"}}

Mapping:
{
  "mappings": {
    "properties": {
      "message": {
        "type": "nested",
        "properties": {
          "info": {
            "type": "keyword",
            "index": "true"
          }
        }
      }
    }
  }
}

Query:
SELECT nested(message.info) FROM nested_objects ORDER BY nested(message.info) DESC;

Response:
{
    "schema": [
        {
            "name": "message.info",
            "type": "keyword"
        }
    ],
    "total": 3,
    "datarows": [
        [
            "d"
        ],
        [
            "c"
        ],
        [
            "a"
        ]
    ],
    "size": 3,
    "status": 200
}

How can one reproduce the bug?
Steps to reproduce the behavior:

Upload dataset with index mapping.
Run SQL query.
Verify Results.

What is the expected behavior?
The hits portion of the response is sorted, and the inner_hits portion is not sorted. The inner_hits could be sorted, but this would be two separate sorts and all returned data would not be one contiguous sort. Perhaps an in-memory solution may be the only fix here.
What is your host/environment?

OS: IOS
Version: 13.4.1
Plugins: SQL

Do you have any screenshots?
N/A
Do you have any additional context?
Legacy engine bug.

Probably, we shouldn't push down sort if nested is present. This is similar to #1764. Could be done after optimizer rework described in #1752 .


Feature: Start application, if not running

Error 6.1 - can't start activity
If the app is not started yet, droidbot can't start the application.
It looks like droidbot try to open it with:
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}

But nothing happen. I think it would be good to start the app automatically if not running. Or is that a Bug?
Can you reproduce this? Or did you need any further information?
Envirement:

Ubuntu 14.04 LTS, 64 bit
GIT: 36978bc
Sample: http://www.h-lemoine.de/ctf.hack.lu_2011/0xA1EA512A.apk - NoMalware

Output
hanno@dockerDroidBox:~/work/PycharmProjects/gsoc_droidbot$ python start.py -d emulator-5554 -a ~/mobileSamples/myApkTresor/0xA1EA512A.apk
Connecting to a device with serialno=emulator-5554 with a timeout of 60 secs...
Connected to device with serialno=emulator-5554
Actual device serialno=emulator-5554
INFO:TelnetConsole:telnet successfully initiated, the addr is (localhost:5554)
INFO:AppEnvManager:start deploying environment, policy is none
INFO:AppEnvManager:finish deploying environment, policy is none
INFO:AppEventManager:start sending events, policy is monkey
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}
INFO:Device:sending event: {'intent': 'am broadcast -a android.intent.action.BATTERY_CHANGED', 'type': 'intent'}
INFO:Device:sending event: {'event_type': 'key', 'name': 'MENU'}
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}
INFO:Device:sending event: {'intent': 'am broadcast -a android.intent.action.PICK_ACTIVITY', 'type': 'intent'}
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}
INFO:Device:sending event: {'intent': 'am broadcast -a android.intent.action.BOOT_COMPLETED', 'type': 'intent'}
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}
INFO:Device:sending event: {'intent': 'am broadcast -a android.intent.action.POWER_USAGE_SUMMARY', 'type': 'intent'}
INFO:Device:sending event: {'intent': u'am start com.ctf.hack_lu', 'type': 'intent'}
INFO:Device:sending event: {'intent': 'am broadcast -a android.intent.action.PACKAGE_ADDED', 'type': 'intent'}
^CTraceback (most recent call last):
  File "start.py", line 59, in <module>
    main()
  File "start.py", line 54, in main
    droidbot.start()
  File "/home/hanno/work/PycharmProjects/gsoc_droidbot/droidbot/droidbot.py", line 57, in start
    event_manager.start()
  File "/home/hanno/work/PycharmProjects/gsoc_droidbot/droidbot/app_event.py", line 596, in start
    self.event_factory.start(self)
  File "/home/hanno/work/PycharmProjects/gsoc_droidbot/droidbot/app_event.py", line 624, in start
    time.sleep(event_manager.duration)
KeyboardInterrupt

Usually, apps can be started by calling am start <package_name>''. However, some apps require calling am start <package_name>/<main_activity_name>'' to start.
Use the later version of command line should solve this issue.
Does this work for you?

This alternative method to start the app is interesting, have you ever tried it?
http://stackoverflow.com/questions/4567904/how-to-start-an-application-using-android-adb-tools/25398877#25398877

Cool, this method doesn't require main activity name. I tried it just now and it works.

Now it works fine. I think you fix that in 866657a.


Plugin documentation not displayed on plugins.jenkins.io

The plugin documentation is not visible at https://plugins.jenkins.io/octopusdeploy/ like it is for other plugins.  The page on the plugins site includes a link to the  https://github.com/OctopusDeploy/octopus-jenkins-plugin page instead of embedding the documentation on the plugins.jenkins.io page as others plugins do.
If you would like the documentation to be visible on https://plugins.jenkins.io/octopusdeploy/ , I think that the <url> in the pom.xml file may need to be in the jenkinsci organization.
@halkeye have I understood correctly that having the authoritative plugin repository outside the jenkinsci organization may not be fully supported by plugins.jenkins.io?
The source code of the octopus deploy plugin in the jenkinsci organization https://github.com/jenkinsci/octopusdeploy-plugin is out of date compared to the repository that appears to be used to release new versions.  Could the contents of this repository be pushed to the jenkinsci organization repository so that open source contributors can see the current code that is the basis for releases?
I was surprised that the releases are being generated from a repository that is not in the jenkinsci GitHub organization.  That may be one of the reasons that the documentation on https://plugins.jenkins.io/octopusdeploy/ is not embedding the current documentation from https://github.com/OctopusDeploy/octopus-jenkins-plugin#readme .

I was surprised that the releases are being generated from a repository that is not in the jenkinsci GitHub organization

They come from update center.

@halkeye have I understood correctly that having the authoritative plugin repository outside the jenkinsci organization may not be fully supported by plugins.jenkins.io?

only jenkinss and jenkins-infra are whitelisted

I was surprised that the releases are being generated from a repository that is not in the jenkinsci GitHub organization

They come from update center.

Thanks.  I should have been more specific.  I was surprised that the tags for the release are being written to a repository that is not in the jenkinsci organization and that the repository URL in the released pom.xml file points to a GitHub repository that is outside the jenkinsci organization.  I'm not sure there is anything wrong with that, but it did surprise me.

So let's close it? I don't see any reason to support non standard repos.
If the code isn't in standard repos, adoption can never happen. Fixes can't be applied.


Formatting removed on pasting formatted copied text

I have implemented for web.
When i paste any formatted text it remove all formatting and hyper links. Please help me.
you can try from thispage content

Same problem here, is there any quick fix to support pasting at least bullet points/list?

Same here, just copies the plain text

Thank you for the report, this issue has been open for a while and since there are no interactions with it, we had to close it
but if you still facing this issue, or other issues or you have any other questions, feel free to contact us anytime you want.


arbitrary_precision leads to issues with adjacently tagged enums

I see that there are a couple open issues around arbitrary_precision which are likely related. Leaving this here for people who ran into this particular issue.
When tag and content are reversed in an adjacently tagged repr, deserialization fails for floats:
use serde::Deserialize;
use serde_json::from_str;

fn main() {
    #[derive(Deserialize, Debug)]
    #[serde(tag = "t", content = "c")]
    enum Foo {
        A(u64),
        B(f64),
        C(bool),
    }
    println!("{:?}", from_str::<Foo>(r#"{"t":"A","c":1}"#));
    println!("{:?}", from_str::<Foo>(r#"{"t":"B","c":3.14}"#));
    println!("{:?}", from_str::<Foo>(r#"{"t":"C","c":true}"#));

    println!("{:?}", from_str::<Foo>(r#"{"c":1,"t":"A"}"#));
    println!("{:?}", from_str::<Foo>(r#"{"c":3.14,"t":"B"}"#));
    println!("{:?}", from_str::<Foo>(r#"{"c":true,"t":"C"}"#));
}

Output:
Ok(A(1))
Ok(B(3.14))
Ok(C(true))
Ok(A(1))
Err(Error("invalid type: map, expected f64", line: 1, column: 18))
Ok(C(true))

After researched this a couple of days, I don't think there is an easy way to fix this.
println!("{:?}", from_str::<Foo>(r#"{"c":3.14,"t":"B"}"#));

This is what happened. First, json deserializer will generate a value
ParserNumber::String(String) // where String is "3.14"

[1] Then since tag is not known at this point, it will be converted to Content
Content::Map(Vec<(Content, Content)>)
// where Vec only has one element (Content::Str("$serde_json::private::Number"), Content::String(String)
// where String value is "3.14"

[2] Then tag is readed, serde will try deserialize to f64 from Content Deserializer which will call this function internally
ContentDeserializer::deserialize_f64()
// only return Ok if contained Content is Number type
// but now Content is Map type which cause the
// Err(Error("invalid type: map, expected f64", line: 1, column: 18))

We can change [1] , so that when we convert to Content, Content::F64 is the final result. Then step [2] will succeed. But, this is a breaking change.
Or we can change [2], let Content::Map support converting  f64. But the map format is only used in serde_json, which means we will bring implement details of serde_json [arbitrary_precision] into serde. This is not a good method
Or We can change [1], so that auto implement of Deserialize will not use Content as the temporary buffer instead use a deserailzer related type(Value in this case) ?
Just some thoughts.

Or We can change [1], so that auto implement of Deserialize will not use Content as the temporary buffer instead use a deserailzer related type(Value in this case) ?

Yes this is the plan, it's blocked on associated type defaults at the moment. This is tracked in https://github.com/serde-rs/serde/issues/1183.


increasing memory every time I change the size of the app

How to solve the problem of increasing memory every time I change the size of the app?
Steps to Reproduce

the code sample : Basic code created when creating a New Project in VSCode (like an Empty Code)
Bug : The memory increases whenever you resize the width or height of the app with the mouse.(From 400M to 8G)
Test Env : Flutter, Windows10-64bit, VSCode, run Windows App

Expected results:
I want to solve this memory leak problem.
Actual results:
The memory increases whenever you resize the app with the mouse.
Thank you in advance.

Code sample

Logs

@wongeumhae Can you provide a recorded video for this? When recording the video, please also open Task Manager.
And please provide the output of flutter doctor -v as well.

Thank you for your attention.

recoded vedio
https://drive.google.com/file/d/1KsxY2UQemw92l1FgCzRNuy3hZ7kT9w9d/view?usp=share_link

doctor
[√] Flutter (Channel stable, 3.7.0, on Microsoft Windows [Version 10.0.19045.2486], locale en-US)
• Flutter version 3.7.0 on channel stable at C:\src\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision b06b8b2710 (7 days ago), 2023-01-23 16:55:55 -0800
• Engine revision b24591ed32
• Dart version 2.19.0
• DevTools version 2.20.1

[√] Windows Version (Installed version of Windows is version 10 or higher)
[X] Android toolchain - develop for Android devices
X Unable to locate Android SDK.
Install Android Studio from: https://developer.android.com/studio/index.html
On first launch it will assist you in installing the Android SDK components.
(or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
If the Android SDK has been installed to a custom location, please use
flutter config --android-sdk to update to that location.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.17)
• Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
• Visual Studio Community 2019 version 16.11.32630.194
• Windows 10 SDK version 10.0.19041.0
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
[√] VS Code (version 1.74.3)
• VS Code at C:\Users\happy\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.58.0
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.2486]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 109.0.5414.120
• Edge (web)        • edge    • web-javascript • Microsoft Edge 109.0.1518.70
[√] HTTP Host Availability
• All required HTTP hosts are available
! Doctor found issues in 2 categories.

@wongeumhae thanks for providing the recorded video.
I tried to reproduce this issue on my Windows machine but didn't see the same result. Verified on Flutter stable 3.7.0, Windows 10.0.19045.2486. I can see it growing a bit (several hundred KB or a few MB) after resizing the windows many times, but not as much as yours.

flutter doctor -v (stable & master)
[√] Flutter (Channel stable, 3.7.0, on Microsoft Windows [Version 10.0.19045.2486], locale en-US)
    • Flutter version 3.7.0 on channel stable at C:\WIP\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b06b8b2710 (3 days ago), 2023-01-23 16:55:55 -0800
    • Engine revision b24591ed32
    • Dart version 2.19.0
    • DevTools version 2.20.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

Checking Android licenses is taking an unexpectedly long time...[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\AndroidSDK
    • Platform android-32, build-tools 32.0.0
    • ANDROID_HOME = C:\AndroidSDK
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.4.4)
    • Visual Studio at D:\DOWNLOADWORK\VS2022
    • Visual Studio Community 2022 version 17.4.33213.308
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[!] Android Studio (version 2022.1)
    • Android Studio at C:\Program Files\Android\Android Studio Electric Eel 2022.1.1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    X Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[√] VS Code (version 1.74.3)
    • VS Code at C:\Users\ADMIN\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.56.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.2486]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 109.0.5414.120
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 108.0.1462.76

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.7.0-30.0.pre.7, on Microsoft Windows [Version 10.0.19045.2486], locale en-US)
    • Flutter version 3.7.0-30.0.pre.7 on channel master at C:\WIP\flutter_master
    ! Warning: `flutter` on your path resolves to C:\WIP\flutter\bin\flutter, which is not inside your current
      Flutter SDK checkout at C:\WIP\flutter_master. Consider adding C:\WIP\flutter_master\bin to the front of your
      path.
    ! Warning: `dart` on your path resolves to C:\WIP\flutter\bin\dart, which is not inside your current Flutter SDK
      checkout at C:\WIP\flutter_master. Consider adding C:\WIP\flutter_master\bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 15cd00f1ed (5 hours ago), 2023-01-29 21:58:05 -0500
    • Engine revision a7bb0e4108
    • Dart version 3.0.0 (build 3.0.0-179.0.dev)
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git"
      directly to perform update checks and upgrades.

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\AndroidSDK
    • Platform android-32, build-tools 32.0.0
    • ANDROID_HOME = C:\AndroidSDK
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.4.4)
    • Visual Studio at D:\DOWNLOADWORK\VS2022
    • Visual Studio Community 2022 version 17.4.33213.308
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[!] Android Studio (version 2022.1)
    • Android Studio at C:\Program Files\Android\Android Studio Electric Eel 2022.1.1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    X Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[√] VS Code (version 1.74.3)
    • VS Code at C:\Users\ADMIN\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.58.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.2486]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 109.0.5414.120
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 108.0.1462.76

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

Can you also check VSCode and dart memory as well?  (including dart process there inside VSCode). May be similar to https://github.com/flutter/flutter/issues/94147.
Also, can you confirm:

Does this issue always happen? Even after rebooting the system?
Does this issue happen on older Flutter versions (in case you have used old versions before)

windows version : 10Pro 10.0.19045 build 19045

The same happens on Flutter stable 3.7.0 and stable 3.3.10
flutter doctor -v after installing android studio
• No issues found!
windows app - release mode, debug mode
flutter reinstall
the other new project
DesktopPC, Laptop

In all cases, the same problem always occurs even after rebooting.
VSCODE's memory does not increase.

Thanks for providing more information. Labeling the issue for further insights.
Re-attaching the recorded video from your Google Drive link here:
https://user-images.githubusercontent.com/104349824/215939381-6561e02a-317a-458f-b98b-1d14dc953fa8.mp4

Hi @wongeumhae could you please attach the actual output of flutter doctor -v instead of just the success line (there's a lot of diagnostic value in the data there).
Also, can you verify that the video you showed is just the generic Flutter counter app created with flutter create with no modifications? That's what it looks like, but we'd like to be sure.
(Desktop Triage)

thanks for your help in advance.
[flutter source]
https://github.com/wongeumhae/flutter_memory_issue/blob/238d9f3fa6c6c482634a23eead93919a1acf6993/flutter doctor -v.txt
[flutter doctor -v]
https://github.com/wongeumhae/flutter_memory_issue.git

After upgrading Flutter, the symptoms disappeared.
Flutter 3.7.8
Dart 2.19.5


no nose -> runtests.py quits early

Just installed on MacOS from source (see #8353). Did not install nose.
python3 runtests.py -v
fails, reporting
!!!!!!!!!!!!!!!!!!! Interrupted: 2 errors during collection !!!!!!!!!!!!!!!!!!!!
without running any tests because
scipy/_lib/tests/test__gcutils.py and
scipy/io/matlab/tests/test_mio5_utils.py
try to import from nose.tools.
Installing nose fixes the problem.
Version of SciPy installed via pip doesn't have this problem. It collects over 14096 items without any errors.
Reproducing code example:
python3 runtests.py -v

or in Python console:
import scipy
scipy.test()

Error message:
Matts-MacBook-Pro:scipy matthaberland$ python3 runtests.py -v
Building, see build.log...
Build OK (0:00:05.530066 elapsed)
============================= test session starts ==============================
platform darwin -- Python 3.6.4, pytest-3.4.0, py-1.5.2, pluggy-0.6.0 -- /usr/local/opt/python3/bin/python3.6
cachedir: ../../../../../.pytest_cache
rootdir: /Users/matthaberland/Desktop/ScipyDev/scipy, inifile: pytest.ini
collected 13960 items / 2 errors

==================================== ERRORS ====================================
 ERROR collecting build/testenv/lib/python3.6/site-packages/scipy/_lib/tests/test__gcutils.py
ImportError while importing test module '/Users/matthaberland/Desktop/ScipyDev/scipy/build/testenv/lib/python3.6/site-packages/scipy/_lib/tests/test__gcutils.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
scipy/_lib/tests/test__gcutils.py:9: in <module>
    from nose.tools import assert_equal, raises
E   ModuleNotFoundError: No module named 'nose'
 ERROR collecting build/testenv/lib/python3.6/site-packages/scipy/io/matlab/tests/test_mio5_utils.py
ImportError while importing test module '/Users/matthaberland/Desktop/ScipyDev/scipy/build/testenv/lib/python3.6/site-packages/scipy/io/matlab/tests/test_mio5_utils.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
scipy/io/matlab/tests/test_mio5_utils.py:13: in <module>
    from nose.tools import (assert_true, assert_equal, assert_raises)
E   ModuleNotFoundError: No module named 'nose'
!!!!!!!!!!!!!!!!!!! Interrupted: 2 errors during collection !!!!!!!!!!!!!!!!!!!!
============================= 570 tests deselected =============================
=================== 570 deselected, 2 error in 8.97 seconds ====================

Scipy/Numpy/Python version information:
1.0.0.dev0+f2c31e3 1.14.0 sys.version_info(major=3, minor=6, micro=4, releaselevel='final', serial=0)

You are installing an old version of Scipy, the version printout you give above says 1.0.0.dev0+f2c31e3.
I think the above issue is not present in scipy 1.0.0 or current master.

@pv sorry for the trouble.
Re: the duplicate, I thought that I had failed to create the issue; I didn't notice it was already closed.
Re: the original issue, I thought that I had cloned a brand new fork of the main repo when I was making the video, so I thought it should have been the latest version. (I didn't; clicking Fork on github brought me to my existing fork.) And apparently I didn't know how to read version numbers.


bug: successful fee-grant and revoke-fee-grant show ERROR in logs and   "result": "RPC client error: sendRequest failed: http status 200 OK unmarshaling response: EOF" in CLI

Celestia Node version
v0.13.2-17-g6382dc13
OS
mac
Install tools
building on main at a point after https://github.com/celestiaorg/celestia-node/pull/3304, at 6382dc13b92d7b40077154aab823367500371cb8
Others
No response
Steps to reproduce it
make feegrant or revoke feegrant
Expected result
no ERROR in logs, because the transaction was successful
Actual result
ERROR in logs of celestia-node on successful fee-grant and in revoke-fee-grant and RPC client error in CLI, although http status 200 OK

see comment with logs for successful grant-fee tx
see comment with logs for successful revoke-grant-fee tx

Relevant log output
No response
Notes
No response

I can confirm this is not working and this mainly seems to be a serialization issue:
2024-04-19T21:47:45.267+0200    ERROR   rpc     go-jsonrpc@v0.3.1/handler.go:489        json: error calling MarshalJSON for type *types.Any: JSON marshal marshaling error for &Any{TypeUrl:/cosmos.tx.v1beta1.Tx,Value:[10 191 1 10 188 1 10 42 47 99 111 115 109 111 115 46 102 101 101 103 114 97 110 116 46 118 49 98 101 116 97 49 46 77 115 103 71 114 97 110 116 65 108 108 111 119 97 110 99 101 18 141 1 10 47 99 101 108 101 115 116 105 97 49 103 109 104 120 99 104 106 56 108 116 101 103 115 99 55 117 122 56 103 114 51 106 110 48 106 53 48 117 55 57 48 51 103 116 104 51 53 52 18 47 99 101 108 101 115 116 105 97 49 50 112 115 103 57 48 108 119 118 120 100 107 116 110 113 101 114 114 50 112 53 109 120 117 119 50 118 51 106 101 52 57 55 117 118 54 116 106 26 41 10 39 47 99 111 115 109 111 115 46 102 101 101 103 114 97 110 116 46 118 49 98 101 116 97 49 46 66 97 115 105 99 65 108 108 111 119 97 110 99 101 18 30 10 8 18 4 10 2 8 1 24 76 18 18 10 12 10 4 117 116 105 97 18 4 50 48 48 48 16 192 132 61 26 64 246 131 239 137 195 146 139 141 23 210 30 199 216 59 100 100 152 66 100 16 117 100 206 179 187 127 67 108 191 153 154 72 53 210 124 27 217 173 143 37 227 239 8 75 139 74 213 222 117 29 251 100 26 219 52 49 161 248 36 80 19 108 28 246],XXX_unrecognized:[]}, this is likely because amino is being used directly (instead of codec.LegacyAmino which is preferred) or UnpackInterfacesMessage is not defined for some type which contains a protobuf Any either directly or via one of its members. To see a stacktrace of where the error is coming from, set the var Debug = true in codec/types/compat.go

The feegrant and revoke actually are successful transactions that land on chain. The logging in node seems to be the issue on my end

Hmm, somewhere the JSON can't be (de)/serialized. So while the transactions seem to work the JSON rpc is broken.
Given that @vgonkivs tested it before in https://github.com/celestiaorg/celestia-node/pull/3304#issue-2231361008, I assume that one of the changes/commits that came after the testing somehow broke sth.

It's not working for me too on the latest main.

seems it panics after marshaling TX

I assume that one of the changes/commits that came after the testing somehow broke sth.

Yep! I can confirm that testing this on the PR itself results in successful result
Semantic version: v0.13.2-22-g9f5f4534 (log)

log of grant-fee, with no errors in the light node's logs
log of revoke-grant-fee, with no errors in the light node's logs

If the change that led to this has nothing to do with the feegrant PR, it could be that more tx or other responses can't be unmarshalled right now. I think it would be good if API responses would be tested.


docs.rs build failing for v0.16.0

docs.rs
build logs
Likely from building with --all-features flag.
[INFO] [stderr] error[E0252]: the name `tls` is defined multiple times
[INFO] [stderr]   --> /opt/rustwide/cargo-home/registry/src/index.crates.io-6f17d22bba15001f/rusoto_core-0.47.0/src/lib.rs:25:5
[INFO] [stderr]    |
[INFO] [stderr] 23 | use hyper_rustls as tls;
[INFO] [stderr]    |     ------------------- previous import of the module `tls` here
[INFO] [stderr] 24 | #[cfg(feature = "native-tls")]
[INFO] [stderr] 25 | use hyper_tls as tls;
[INFO] [stderr]    |     ^^^^^^^^^^^^^^^^ `tls` reimported here
[INFO] [stderr]    |
[INFO] [stderr]    = note: `tls` must be defined only once in the type namespace of this module
[INFO] [stderr] help: you can use `as` to change the binding name of the import
[INFO] [stderr]    |
[INFO] [stderr] 25 | use hyper_tls as other_tls;
[INFO] [stderr]    |     ~~~~~~~~~~~~~~~~~~~~~~
[INFO] [stderr]
[INFO] [stderr]     Checking lz4 v1.24.0
[INFO] [stderr]     Checking blake2 v0.10.6
[INFO] [stderr]     Checking sha2 v0.10.8
[INFO] [stderr]     Checking itertools v0.11.0
[INFO] [stderr]     Checking zstd v0.12.4
[INFO] [stderr]     Checking fastrand v2.0.1
[INFO] [stderr] error[E0599]: no function or associated item named `new` found for struct `hyper_rustls::HttpsConnector` in the current scope
[INFO] [stderr]    --> /opt/rustwide/cargo-home/registry/src/index.crates.io-6f17d22bba15001f/rusoto_core-0.47.0/src/request.rs:224:41
[INFO] [stderr]     |
[INFO] [stderr] 224 |         let connector = HttpsConnector::new();
[INFO] [stderr]     |                                         ^^^ function or associated item not found in `HttpsConnector<_>`
[INFO] [stderr]
[INFO] [stderr] error[E0599]: no function or associated item named `new` found for struct `hyper_rustls::HttpsConnector` in the current scope
[INFO] [stderr]    --> /opt/rustwide/cargo-home/registry/src/index.crates.io-6f17d22bba15001f/rusoto_core-0.47.0/src/request.rs:238:41
[INFO] [stderr]     |
[INFO] [stderr] 238 |         let connector = HttpsConnector::new();
[INFO] [stderr]     |                                         ^^^ function or associated item not found in `HttpsConnector<_>`
[INFO] [stderr]
[INFO] [stderr] Some errors have detailed explanations: E0252, E0599.
[INFO] [stderr] For more information about an error, try `rustc --explain E0252`.
[INFO] [stderr] error: could not compile `rusoto_core` (lib) due to 3 previous errors

This is already fixed #1693

Haha thanks, I only checked open issues before creating this 🫣


[Cattle] API collection /v1/settings or v1/accounts/ doesn't support filtering

The settings collection on the the api does not respect filtering.
Something like curl://<ip>:<port>/v1/settings?name=db.cattle.database will still return the entire collection.

As an example, if you run this:
curl http://localhost:8080/v1/accounts/1a1/credentials?kind=registrationToken

The filter is not applied. You still get back all the creds for the account:
{
  "type": "collection",
  "resourceType": "credential",
  "links": {
    "self": "http://localhost:8080/v1/accounts/1a1/credentials"
  },
  "createTypes": {
    "apiKey": "http://localhost:8080/v1/apikeys",
    "credential": "http://localhost:8080/v1/credentials",
    "registrationToken": "http://localhost:8080/v1/registrationtokens"
  },
  "actions": {},
  "data": [
    {
      "id": "1c1",
      "type": "apiKey",
      "links": {
        "self": "http://localhost:8080/v1/apikeys/1c1",
        "account": "http://localhost:8080/v1/apikeys/1c1/account",
        "instances": "http://localhost:8080/v1/apikeys/1c1/instances"
      },
      "actions": {
        "update": "http://localhost:8080/v1/apikeys/1c1/?action=update",
        "deactivate": "http://localhost:8080/v1/apikeys/1c1/?action=deactivate"
      },
      "accountId": "1a1",
      "created": "2015-02-06T23:35:42Z",
      "createdTS":<PHONE_NUMBER>000,
      "data": {
        "fields": {}
      },
      "description": null,
      "kind": "apiKey",
...
    },
    {
      "id": "1c52",
      "type": "registrationToken",
      "links": {
        "self": "http://localhost:8080/v1/registrationtokens/1c52",
        "account": "http://localhost:8080/v1/registrationtokens/1c52/account",
        "instances": "http://localhost:8080/v1/registrationtokens/1c52/instances",
        "registrationUrl": "http://localhost:8080/v1/scripts/..."
      }
       ...
    }
}


feat: support jx-requirements log url with a querystring

To use minio as log storage, query parameters are required for the log url e.g.
  storage:
  - name: logs
    url: s3://jx3?endpoint=minio.minio.svc.cluster.local:9000&disableSSL=true&s3ForcePathStyle=true&region=ignored
  - name: reports
    url: s3://jx3?endpoint=minio.minio.svc.cluster.local:9000&disableSSL=true&s3ForcePathStyle=true&region=ignored

The PR uses helms parseUrl and joinUrl to place the golang template in the path portion of the url, rather than appending, which ends up placing it as part of the querystring

Hi @cameronbraid. Thanks for your PR.
I'm waiting for a jenkins-x or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign garethjevans
You can assign the PR to them by writing /assign @garethjevans in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Example of how it fixes my visualiser :

/ok-to-test
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


BUG: automatic assignement of gat scorer even if scorer is passed

https://github.com/mne-tools/mne-python/pull/2856 had assigned scorer_ as a function of clf and predict_method even when the user had already defined it:
e.g.
gat = GeneralizationAcrossTime(scorer=my_scorer)
gat.fit(epochs)
gat.score(epochs)
gat.scorer_ != my_scorer

This is corrected, with an additional test.

cc @gmeade

travis is not happy

Is there a way to nosetest with previous sklearn versions?

Install a virtualenv is the most common way

Ok I'll try setup a virtualenv, thanks.
Travis passed, ready for review.

thx @kingjr


Merkle whitelist

See https://medium.com/@ItsCuzzo/using-merkle-trees-for-nft-whitelists-523b58ada3f9.

Construct Merkle tree for whitelist off-chain in Stargaze Tools.
Update whitelist contract with the Merkle root
Send Merkle proof to whitelist contract to verify whitelist inclusion

Devil's advocate: It's actually a benefit of Stargaze to store whitelists on-chain. You can't do this on Ethereum because of storage and gas constraints. But Stargaze doesn't have those issues.


Create a sample script for Girder item manipulation

To start, this can be a Python script utilizing the Girder Python Client. Eventually, we may want to put it into an iPython Notebook, but that can be done by users.
For the first iteration, the script should:

Connect to https://girder.neuro.emory.edu/ , and authenticate. User credentials (particularly password) can be read from standard input at runtime, or passed as command line arguments.
Create a new subfolder in the "DG TCGA" collection called  "GBM"
Get a list of all large_image items that currently live in "DG TCGA". If items are not already marked as large_image, find all items with SVS files, and mark them as large_image first.
Within the recently created GBM folder, create subfolders "PATIENT1", "PATIENT2", and "PATIENT3"
Move one of the large_image items to the recently created folder PATIENT1
Set the following metadata on the "PATIENT1" folder:
"PatientAge":  22
"PatientDeceased": True
Set the following metadata on the large_image item:
"SlideSource": "FromTCGA"
"SlideType": "DX"
Look for empty folders and delete them, as large_image items that are moved into the "PATIENT*" hierarchy will leave empty folders behind.
Find and delete any items containing a "README.TXT" file.

@brianhelba
Some clarifications needed
7 Within the collection delete folders that are completely empty or just when there are no more large images left(within the actual archive there are multiple txt files)
8  Within the collection, delete items named README.txt, or delete any item that contains a file named README.txt

@salamb On 7, let's delete all folders that are completely empty.
On 8, delete any items containing the file "README.txt". Note, as a practical matter, I expect there's only 1 file per item in this case, but we shouldn't rely on that fact.

@brianhelba
Is their a way in the girder client to get /resource/search under a specific collection or directory? I can only get it to search the entire database and i don't see such options in the interactive api. If not I will use a recursive search.

@salamb I think the recursive search is better anyway, since we're trying to write a clear example on how to browse and manipulate a Girder instance, as opposed to doing things in a more efficient but obscure way.

@brianhelba
If you have a series of nested folders with a single uploaded image in the leaf/last child folder. parent folders of the child folder have a size of 0. I found the issue #454 on girders repo. I was originally deleting all folders of size 0 which included GBM. Do you want me to delete all folders with just zero contents or recursively determine if the size is zero in the case sub folders actually have zero size .

@salamb Recursively check the size. Only folders which are truly empty should be deleted.

Closing due to age.