Value prop changes are not reflected in the display

Currently the value prop only sets the initial value. However, I would like value to instead behave like it would for a standard input field, where changes in the value prop would be reflected in the display.
In this manner it would be behaving similar to a standard React controlled component https://reactjs.org/docs/forms.html#controlled-components

Hey guys, I opened a PR aiming to fix this.
Also took the opportunity to move from componentWillReceiveProps to getDerivedStateFromProps, to make the component future-proof. Feel free to give some feedback!

@thmsmontana please fix tests there

This is a pretty big bummer. There doesn't seem to be any way a the moment to programmatically set the values or clear them out.

Will this be fixed any time soon? 👀

any updates?

use key property,when key was changed then render immediate,such as :


Feature vector for image similarity measure

For image similarity measurement, we tried to get the feature vectors using the method model.extract_features(img). However, comparing to nasnet_large for example, not only the feature vector from the EfficientNet is huge (87808 dimension total, 1792x7x7), the L2 distance of two such huge vectors from two similar images doesn't reflect the similarity as well as that from feature vectors of nasnet_large (only about 4000 dimension). Is there any other way to get feature vectors , maybe from a difference layer, that can be better suited for similarity measure of two images? Thanks!

That feature vector still has spacial information, so you might try averaging over the spacial dimensions (e.g. use torch.mean to make the 1792x7x7-dim vector to a 1792-dim vector). Let me know if this works any better!

Great! Keep me updated.
On Mon, Aug 26, 2019, 8:53 PM drjtan<EMAIL_ADDRESS>wrote:

yes, averaging works. However, I am not sure if the feature vectors
obtained this way are orientational invariant. Will do some testing...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lukemelas/EfficientNet-PyTorch/issues/68?email_source=notifications&email_token=ADFQ4MG5FMKSUXVABSRQGFTQGR3J5A5CNFSM4IPK2L22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5GDXQY#issuecomment-525089731,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADFQ4MA5BRMLH3E7BDYW2HLQGR3J5ANCNFSM4IPK2L2Q
.

yes, averaging works. However, I am not sure if the feature vectors obtained this way are orientational invariant. Will do some testing...

Can you share your test results? I am also working on image similarity. thank you very much!

waiting for results

Not orientationally invariant, unfortunately. In our application, we are interested in the dl feature of the object in the image, and  what we did was to apply a PCA to align the object to a pre-determined orientation, that seems to solve the problem.

how did you use PCA to align to a pre-determined orientation? I am currently tackling the same problem, I built and autoencoder out of efficientnet_b7, the encoder (effnet) extracts the features and the decoder (effnet transposed) reconstructs the image to compute the loss and teach the encoder what the best features are to represent an image, after the encoder is trained I use it to extract features of an image database and a single query image, and then use NearestNeighbor algo to search for similar images, and I too was failing with getting similar images, was getting better results computing hashes(wavelet hash, dhash, etc.)

in our application, the objects of interest are all rigid objects, which means they have well defined contours. Aligning contour using PCA is quick and easy.


Pin karma, karma-mocha dev dependencies

CC @anoonan @ccnixon @ladanazita

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #14   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          34     34
=====================================
  Hits           34     34

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67921a9...42ddaf6. Read the comment docs.


Issue when turning on or off with wireless switch

Hello, I don't know if it's a bug or a problem in my configuration. After several problems with my bedside lamp 2 with homkit, I have decided to try your plugin.
Apparently everything works fine, maybe a little slow when interacting with the lamp.
The problem I have is that when I turn on the lamp with a wireless switch, I have to click twice, to turn it off or to turn it on, if it is on and I press the switch in the hombidge log, another one appears just like the previous state. and if I press again, at that moment it turns off and the same thing happens in the opposite case.
Could you tell me if there is any way to solve it?
Thank you very much for your work.

Hi, sounds like your switch isn't configured properly.
What switch is it and how have you integrated it into homebridge/homekit?
Is there any automation/scene which's toggles the state for the switch?

I use several and have tried it on all of them. I have several philips dimmers and also several ikea on/off dimmers.
They are integrated into home kit through deconz and the homebride Hue plugin.
With other lights it works normally, it only has that behavior with the lamp and the plugin, since if I add it with homekit with the code it works normally

.

Again, is there any automation/scene which's toggles the state for the switch?
What is the "logic" after you've pressed the switch?
And does the lamp work normally if you control it via homekit or do you need to click there also twice?
The plugin just do what homebridge says, and the log clearly shows that false/true was received twice each time. So the issue can only be within the switch or the automation afterwards.

Sorry, I forgot to answer that question.
I have it configured with the shortcuts option, in this way:
-If the lamp is inactive.
-turn on the lamp.
-otherwise.
-Turn off the lamp.
-end.
The funny thing is that I have more things like this and it only happens here. but it doesn't matter because I can change the settings. I was only commenting on it in case that issue had been documented in other cases.
I just tried with siri and it works fine and from the home app too, so I understand that the problem comes when it interacts with the shortcut.

Hmm, I had something similar with another device and I could solve it with this workaround.
For the bslamps I have also another automation which toggles on/off with a simple scene. Scenes automatically toggles the lamp without the need to check the power state :)

I've just been doing tests with what it says in that article.
when I call the shortcut the first time, the lamp turns on but in homekit it is not notified on.
I give it a second time and it already notifies in homekit on.
I give it a third time and it turns off but homekit does not notify it off.
and I give it the fourth time and it is already notified in homekit as if it is turned off.
It's curious. Could it be that some configuration data of the lamp is not correct in homebridge?
something that I did not do well when doing bslamp pair?

Oh wait the message comes after the pairing and not inside homebridge... the problem is that your user (pi or whatever) don't have write rights in this directory. Quick fix - try the pairing again with sudo. The written file is just needed to unpair the lamp again.

Cool!  I'm going to try it with sudo.  Tomorrow I will give you an answer that in my country it is already too late.
My homebridge setup is via the homebridge raspberry pi image (https://github.com/homebridge/homebridge-raspbian-image)
I have been trying to find what user it uses but i cant figure it out.
I think that with sudo the problem will be solved, however tomorrow I will confirm.
Thank you very much for your attention.

hello,
The installation issue has been solved with the sudo command at the beginning.
The part of shortcuts in the switch, after giving it many turns I have managed to solve it in a very rudimentary way.
Adding in front of the shortcut 2 actions to get the status of the accessory and at the end another two. 4 in all. It's the only way for honekit to get the actual state of the lamp.
If I find another solution I will send it to you.
Thank you very much for your help.


Minor README.md typo correction.

"Q&A" (question and answer) should be "QA" (quality assurance).

Checklist

[X] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[X] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[X] I've read the Contribution Guidelines
[X] I've updated the documentation if necessary.

Motivation and Context

Provides better documentation accuracy.

N/A.

N/A, not a code change.
Description

Changes "Q&A" (question and answer) to "QA" (quality assurance).

I signed it!

👍


Debian 11: Network Mode Emulated VLAN prevents connections to Internet

Describe the issue
Debian 11.4.0 (arm64) can't connect to the Internet (an external network) when the network mode Emulated VLAN is selected. It works fine with the network mode Shared Network. Debian 10 from the Gallery utilizes the network mode Emulated VLAN without any problems.
Configuration

UTM Version: 3.2.4
macOS Version: 12.5.1
Mac Chip (Intel, M1, ...): M1 Pro

Debug log
debug.log
Upload VM
I can upload the entire .utm file if required.
config.plist.zip

@LukBukkit any luck making it work?
I have got the same issue with Ubuntu 22. Shared network internet works just fine, but Emulated VLAN internet does not work. I have switched to Emulated VLAN just to access it through ssh...

No, sadly I didn't get any further with the resolution of this issue. I just keep using the Shared Network mode.

Can you access it thought ssh?

I've deleted the VM a few days ago, but it was possible to access it via SSH. You have to find out its IP address. The command ip addr show should be helpful for this. Then, you can connect to the VM via SSH from your Mac with ssh user@ip.


Rewritten confirm parameter to data-confirm-ujs in adv_search

https://bugzilla.redhat.com/show_bug.cgi?id=1295400

@himdel please review

LGTM (though it will become superfluous after #6031).

@h-kataria can you review/verify/merge?  Thx, Dan

@skateman, i tried few screens that are changed in this PR but for some reason did not get confirmation pop-up when i pressed on Delete button on these screens, am i missing something. I restarted server and cleared my browser cache.
screenshots from the screens that i tried:

@h-kataria you're right, I was confused because of the remote_function

suggest closing this one in favor of #6031

@martinpovolny #6031 doesn't fixed this bug, now it's rebased @h-kataria

@skateman doesn't :confirm get converted to data-confirm in link_to? Pretty sure it does..

@skateman tested in UI now i do get a confirmation message when Delete is pressed but after filter is deleted, "Delete" button is still enabled and if i press it again i see an error on the background screen, i will open a separate BZ on that.

opened https://bugzilla.redhat.com/show_bug.cgi?id=1298645 to address issue in my comment above.


Move ILoggingBuilder to Microsoft.Extensions.Logging.Abstractions

I don't want to depend on Microsoft.Extensions.Logging. Please move ILoggingBuilder to Microsoft.Extensions.Logging.Abstractions.
Is there a really good reason why it is in Microsoft.Extensions.Logging?
From where I stand, it is an abstraction.

We didn't want to make Logging.Abstractions pull in DI.Abstractions.

OK I see. Don't recall I've much used them apart but sure that's reasonable.

I now think we should do this.

Its just interfaces you'd pull in.. And NullLogger...useful.

@davidfowl why? DI is not required for logging implementation. But then you are required to reference our implementation to write provider implementation. Maybe referencing DI is less evil

Maybe referencing DI is less evil

@pakrym yep

I prefer the current situation. At my company we use Microsoft.Extensions.Logging without Microsoft.Extensions.DependencyInjection - our libraries use the logging abstractions, and when used with ASP.NET Core they can plug into its own loggerfactory. ILoggerBuilder is more specific to the asp.net/DI way of doing things, so it seems fine for it to be in the concrete package - or even in a higher-level package which composes together Logging and DependencyInjection.

we also have our own internal concrete ILoggerFactory etc - the current design means that this works really well, with consumers of our libraries able to use various providers without needing Microsoft.Extensions.Logging itself, only the abstractions, but capable of interoperating with asp.net or similar libs if they like

Yea but depending on DI.Abstractions adds nothing but a single dependency and the benefit is that providers don't need to reference the concrete implementation of the LoggerFactory to make extension methods.

it wouldn't be a huge deal but it's a bit annoying. why do they need to reference concrete MEL - are they adding extensions to ILoggingBuilder as well as ILoggerFactory?

Could add Microsoft.Extensions.Logging.DependencyInjection.Abstractions - or the other way around - but that's on the path to the curse of dimensionality.

@davidfowl do you still think we should pursue this?


Falsy Lint/UnusedMethodArgument for specific Ruby 2.3 lonely operator usage

Expected behavior
Given the following code below that uses the Ruby 2.3 lonely operator I get a complaint about an unused argument.
def self.some_method(bar:)
  something&.some_method(bar) { something_else }
end

W: Unused method argument - bar

Note that it only fails to detect the keyword argument usage if a block is given.
def self.some_method(bar:)
  something&.some_method(bar)
end

The example above doesn't receive a warning from Rubocop.
Actual behavior
Don't give unused argument warnings as the arguments are actually used.
Steps to reproduce the problem
Check out the code snippets above or the patch below.
diff --git i/spec/rubocop/cop/lint/unused_method_argument_spec.rb w/spec/rubocop/cop/lint/unused_method_argument_spec.rb
index 38f1bf3..48e6203 100644
--- i/spec/rubocop/cop/lint/unused_method_argument_spec.rb
+++ w/spec/rubocop/cop/lint/unused_method_argument_spec.rb
@@ -69,6 +69,18 @@ describe RuboCop::Cop::Lint::UnusedMethodArgument, :config do
       end
     end

+    context 'when a required keyword argument is used in lonely operator', ruby: 2.3 do
+      let(:source) { <<-END }
+        def self.some_method(bar:)
+          something&.some_method(bar) { something_else }
+        end
+      END
+
+      it 'accepts' do
+        expect(cop.offenses).to be_empty
+      end
+    end
+
     context 'when an optional keyword argument is unused', ruby: 2 do
       let(:source) { <<-END }
         def self.some_method(foo, bar: 1)

RuboCop version
$ rubocop -V
0.40.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-darwin15)

This is a bug in parser, not rubocop. The AST returned by parser is completely wrong.

Oooh, but it does work with the parser gem at HEAD!

Aha, looks like it was fixed in f08db93. @whitequark... new minor release please!!!

<IP_ADDRESS> released.

Thank you, sir!

Thanks, folks!


Failing to compile build.gradle.kts does not fail the Gradle build

Given a build.gradle.kts:
import org.gradle.api.*

task<DefaultTask>("broken)

Running gradle -b build.gradle.kts help produces:

Error: Expecting '"' (/Users/sterling/gits/tmp/kotlin/build.gradle.kts (3:27))
Error: Expecting ')' (/Users/sterling/gits/tmp/kotlin/build.gradle.kts (3:27))
:help
Welcome to Gradle 3.3-20161028185232+0000.
To run a build, run gradle  ...
To see a list of available tasks, run gradle tasks
To see a list of command-line options, run gradle --help
To see more detail about a task, run gradle help --task
BUILD SUCCESSFUL
Total time: 1.459 secs

I would expect this build to fail.  If I look at gradle tasks, I see there's a task named broken).

And I should add, this isn't all types of failures.  This is just one case I found.  Other failures do cause build failures.

Curious :)
Thanks, @big-guy!

A few other weird examples with 0.7.0:

Example version as second argument in kotlinModule

plugins {
  `lifecycle-base`
  `build-dashboard`
  java
}

dependencies {
  compile(kotlinModule("stdlib", ))
}

tasks {
  "wrapper"(Wrapper::class) {
    distributionUrl = "https://repo.gradle.org/gradle/dist-snapshots/gradle-script-kotlin-3.4-20170117200919+0000-all.zip"
  }
}

Results in:
./gradlew build
:compileJava FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Could not resolve all dependencies for configuration ':compileClasspath'.
> Cannot resolve external dependency org.jetbrains.kotlin:kotlin-stdlib:1.1-M04 because no repositories are defined.
  Required by:
      project :

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.


xpath, jo, curl and zip - APIGEE and missed scriptApprovals

What do we use jo for?

What do we use jo for?

@bleggett - we were using 'jo' to construct a json object in the installer bash script.

@muditpurwar In the Apigee installer? That requirement is gone now, that's one of the reasons I asked for the script to be redone in Python, so we can rely on a stdlib to do stuff like this cleanly and maintainably.

@bleggett I had updated the docker file to remove the "jo".


[BUG] - Cannot upload pdf files because of tenacity.RetryError

Description
Having this error when trying to upload any pdf file. Using llama3.1 and able to chat with the model. But file upload shows this error:
File ...\kotaemon-main\install_dir\env\lib\site-packages\tenacity_init_.py, line 326, in iter raise retry_exc from fut.exception()
tenacity.RetryError: RetryError[<Future at 0x235baabbc40 state=finished raised AuthenticationError>]
Tried to troubleshoot with no luck.
Reproduction steps
Installed the latest version on 9/1/24
Used admin account to log in
Added llama3.1
Chat works fine
PDF upload:  v9.pdf: RetryError[<Future at 0x235b7316980 state=finished raised AuthenticationError>]

Screenshots
No response
Logs
No response
Browsers
Chrome
OS
Windows
Additional information
No response

Likely that you haven't set the embedding model in Index setting to ollama. See https://github.com/Cinnamon/kotaemon/blob/main/docs/local_model.md#use-local-models-for-rag

Likely that you haven't set the embedding model in Index setting to ollama. See https://github.com/Cinnamon/kotaemon/blob/main/docs/local_model.md#use-local-models-for-rag

This helped. Thank you So Much!!!


added filipino translation (with translation txt)

I added a filipino translation for the game

Thank you @Maggotron! (sorry for the delay, it's been a busy week)


Graphing the Binning Example in 02.07

In 02.07, graphing the histogram using [plt.plot(bins, counts, linestyle='steps')] pops up an error. It says that 'steps' is not a valid value for linestyles.

In 02.07, graphing the histogram using [plt.plot(bins, counts, linestyle='steps')] pops up an error. It says that 'steps' is not a valid value for linestyles


Convert if pix-fmt doesn't match

Hello,
I have the following in my config:
pix-fmt = yuv420p
However, it doesn't seem convert non-yuv420p videos unless I explicitly force a conversion. I do also have:
process-same-extensions = True
but it doesn't seem to help.

Post logs

(venv) root@17af96687a63:/usr/local/sma# ./manual.py -ai /data/media/Attack\ on\ Titan\ -\ S01E01\ -\ To\ You\,\ in\ 2000\ Years\ -\ The\ Fall\ of\ Shiganshina\,\ Part\ 1.mp4 -nd
Manual processor started.
Python 64-bit 3.6.9 (default, Jan 26 2021, 15:33:00)
[GCC 8.4.0].
Guessit version: 3.3.1.
/usr/local/sma/venv/bin/python3
Loading config file /usr/local/sma/config/autoProcess.ini.
No-delete enabled
Input Data
{
    "format": "mov,mp4,m4a,3gp,3g2,mj2",
    "format-fullname": "QuickTime / MOV",
    "video": {
        "index": 0,
        "codec": "h264",
        "bitrate": 6055685,
        "pix_fmt": "yuv420p10le",
        "profile": "high 10",
        "fps": 23.976023976023978,
        "framedata": {
            "pix_fmt": "yuv420p10le",
            "side_data_list": [
                {
                    "side_data_type": "H.26[45] User Data Unregistered SEI message"
                }
            ]
        },
        "dimensions": "1920x1080",
        "level": 4.0,
        "field_order": "unknown"
    },
    "audio": [
        {
            "index": 1,
            "codec": "aac",
            "bitrate": 254991,
            "channels": 2,
            "samplerate": 48000,
            "language": "eng",
            "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired"
        },
        {
            "index": 2,
            "codec": "ac3",
            "bitrate": 640000,
            "channels": 6,
            "samplerate": 48000,
            "language": "eng",
            "disposition": "+default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired"
        },
        {
            "index": 3,
            "codec": "aac",
            "bitrate": 191265,
            "channels": 2,
            "samplerate": 48000,
            "language": "jpn",
            "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired"
        }
    ],
    "subtitle": [
        {
            "index": 4,
            "codec": "mov_text",
            "bitrate": 66,
            "disposition": "+default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired",
            "language": "zxx"
        },
        {
            "index": 5,
            "codec": "mov_text",
            "bitrate": 117,
            "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired",
            "language": "eng"
        },
        {
            "index": 6,
            "codec": "mov_text",
            "bitrate": 64,
            "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired",
            "language": "eng"
        },
        {
            "index": 7,
            "codec": "mov_text",
            "bitrate": 61,
            "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired",
            "language": "eng"
        }
    ],
    "attachment": []
}
Reading video stream.
Video codec detected: h264.
Pix Fmt: yuv420p10le.
Profile: high 10.
Video codec parameters None.
Creating h264 video stream from source stream 0.
Reading audio streams.
Audio detected for stream 2 - ac3 eng 6 channel.
Creating aac audio stream source audio stream 2 [universal-audio].
Creating copy audio stream from source stream 2.
Audio detected for stream 1 - aac eng 2 channel.
Creating ac3 audio stream from source stream 1.
Audio detected for stream 3 - aac jpn 2 channel.
Default audio stream set to eng copy 6 channel stream [default-more-channels: True].
Reading subtitle streams.
Text-based subtitle detected for stream 4 - mov_text zxx.
Text-based subtitle detected for stream 5 - mov_text eng.
Creating copy subtitle stream from source stream 5.
Text-based subtitle detected for stream 6 - mov_text eng.
Creating copy subtitle stream from source stream 6.
Text-based subtitle detected for stream 7 - mov_text eng.
Creating copy subtitle stream from source stream 7.
Attempting to download subtitles.
Input and output extensions match and the file appears to have already been processed by SMA, enable force-convert to override [force-convert: False].
Bypassing conversion and setting outputfile to inputfile.
Matched TV episode as Attack on Titan (TMDB ID: 1429) S01E01
Processing Attack on Titan Season 01 Episode 01 - To You, in 2000 Years: The Fall of Shiganshina, Part 1
Tagging file: /data/media/Attack on Titan - S01E01 - To You, in 2000 Years - The Fall of Shiganshina, Part 1.mp4.
Trying to write tags.
Tags written successfully using mutagen.
Relocating MOOV atom to start of file.
QT FastStart did not run - perhaps moov atom was at the start already or file is in the wrong format.

Input and output extensions match and the file appears to have already been processed by SMA, enable force-convert to override [force-convert: False].

even though pix-fmt doesn't match? Does SMA do anything other than a codec/extension check for prior conversions?

It does full checks but if the extension is the same and the file was already created by SMA it will bypass FFMPEG checks. Force convert will still analyze all the parameters and may still remux if appropriate.

thanks!


Fix begin at zero issue with constant data

Fixes #1579
We apply the ticks.beginAtZero option before seeing if we need to expand the scale range if scale.min === scale.max

There appears to still be an issue if all y values are '0'.


Spell check

Spell check all files, and add pre-commit and github action spell check

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

5 of 5 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.499%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1505

Relevant Lines:
1663

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 6 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.499%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1505

Relevant Lines:
1663

💛  - Coveralls


Cross Plat and Mobile UI fixes

[x] Vendor prefixes for transition, appearance and animation.
[x] Add meta for mobile web app capable
[x] Prevent user scaling
[ ] Prevent overscrolling (ios)

Does this prevent to zoom the page?

On mobile yes. Doesn't affect desktop


Num.powInt overflow behavior

When Num.powInt overflows, it doesn't panic, but silently returns 0.
To replicate, enter Num.powInt (Num.toU8 2) 8 in the repl.

@lukewilliamboswell I wrote this sample program:
app [main] {
    cli: platform "https://github.com/roc-lang/basic-cli/releases/download/0.15.0/SlwdbJ-3GR7uBWQo6zlmYWNYOxnvo8r6YABXD-45UOw.tar.br",
}

import cli.Stdout

main : Task {} _
main =
    x : U8
    x = 2
    y = Num.powInt x 8
    Stdout.line! (Num.toStr y)

I again get an output of 0 when building with --optimize and then running the executable.
I am using Pop-OS! on an x86_64 machine

I can take this ticket


$sce, Strict Contextual Escaping not possible in custom filter

Hi
Firstly, great module.. thank you!
I have a filter that returns HTML. However, Strict Contextual Escaping appears to be ignored. See http://jsfiddle.net/howieweiner/fhk93vdr/3/
I've not had a chance to look at the code. Perhaps a display-format-html attribute is also required?
I've got round this issue by writing a TR tag into the table for now.
edit: just noticed this issue that has been rejected https://github.com/MoonStorm/trNgGrid/issues/84
I get your point about filters not outputting HTML. I take your point, though in my experience (as a Django dev) filters are also often used as formatters.
Anyway, what is your suggested approach to formatting cell data? Writing out TR tags will become repetitive in my app. Looking for some re-use

Something along these lines maybe? http://stackoverflow.com/questions/28410239/how-to-show-html-content-in-trnggrid-with-sce-trustashtml/28416977#28416977


Test fixes for slower systems

Couple fixes from investigating test failures on hppa, from the thread "Test failure E89 in Test_terminal_special_chars() line 11 (bwipe)".

James McCoy wrote:

Couple fixes from investigating test failures on hppa, from the thread "Test failure E89 in Test_terminal_special_chars() line 11 (bwipe)".

I already included the change to test_terminal.vim.
I hesitate including the other change.  It relaxes what the test is
checking, I'm not sure that is needed.
As with the other test failure, I'm wondering whether RunVimInTerminal()
picks up the right Vim executable.
--
Contrary to popular belief, Unix is user friendly.
It just happens to be selective about who it makes friends with.
-- Dave Parnas
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

It relaxes what the test is checking, I'm not sure that is needed.

Did you check the example failures I mentioned in the commit?  It's a timing issue where many more iterations are able to happen between the sub-vim being told what to do and the parent vim checking the result.

I'm wondering whether RunVimInTerminal() picks up the right Vim executable.

The only vim executable in the chroot is the one created during the build.


[Minor issue] on tuigreet: --remember doesn't work with the version in the repo

Under /etc/greetd/config.toml we have the line
command = "tuigreet --remember --time --issue --asterisks --cmd sway"

Just letting you know that the argument --remember doesn't do anything with the version of the package included in manjaro-sway repos. I installed 'yay -S greetd-tuigreet-git', reboot, and now show the latest logged user.
So updating it to the newest version should fix it.
Cheers.

Tested on yesterday's build.

yay -S greetd-tuigreet-bin also fixes it, so it's probably safest as it is an stable version.

guess the rebase on aur is just stuck, haven't been using passwort login for a long time (fprintd), so I had no chance to really test this.

updated greetd and tuigreet. would be interesting if it work for you now!

Thank you for the fast response!!
I've done
sudo pacman -Syyu
sudo pacman -S sudo pacman -S greetd greetd-tuigreet

Restarted the computer a couple times, and I can confirm it works as expected now!


Missing release tag for 1.0.2

Our dependency bot made a new PR that bumps our elevio npm package to 1.0.2, but i can't find what changes are included. Could you please point me in the right direction?
Thanks!

@kristian-puccio Saw that you've been the most active contributor to this. Do you have some insights into @eLysgaard's question?

Sorry about the delay! Life has been busy.
I've published the release notes here https://github.com/elevio/client-packages/releases/tag/1.0.2
Thanks!


When I try to train the model,an AssertionError occurs.What should I do to train this model successfully?

Traceback (most recent call last):
File "train_visdrone.py", line 19, in
args=(args,),
File "/home/golven/code/detectron2/detectron2/engine/launch.py", line 62, in launch
main_func(*args)
File "/home/golven/code/QueryDet-PyTorch/train_tools/visdrone_train.py", line 179, in start_train
return trainer.train()
File "/home/golven/code/QueryDet-PyTorch/apex_tools/apex_trainer.py", line 227, in train
self.run_step()
File "/home/golven/code/QueryDet-PyTorch/apex_tools/apex_trainer.py", line 249, in run_step
loss_dict = self.model(data)
File "/home/golven/.conda/envs/query-det/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/golven/code/QueryDet-PyTorch/models/retinanet/retinanet.py", line 197, in forward
losses = self.det_loss(gt_classes, gt_deltas, box_cls, box_delta, self.focal_loss_alpha, self.focal_loss_gamma, self.cls_weights, self.reg_weights)
File "/home/golven/code/QueryDet-PyTorch/models/retinanet/retinanet.py", line 278, in det_loss
assert len(cls_weights) == len(pred_logits)
AssertionError

Traceback (most recent call last): File "train_visdrone.py", line 19, in args=(args,), File "/home/golven/code/detectron2/detectron2/engine/launch.py", line 62, in launch main_func(*args) File "/home/golven/code/QueryDet-PyTorch/train_tools/visdrone_train.py", line 179, in start_train return trainer.train() File "/home/golven/code/QueryDet-PyTorch/apex_tools/apex_trainer.py", line 227, in train self.run_step() File "/home/golven/code/QueryDet-PyTorch/apex_tools/apex_trainer.py", line 249, in run_step loss_dict = self.model(data) File "/home/golven/.conda/envs/query-det/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in _call_impl result = self.forward(*input, **kwargs) File "/home/golven/code/QueryDet-PyTorch/models/retinanet/retinanet.py", line 197, in forward losses = self.det_loss(gt_classes, gt_deltas, box_cls, box_delta, self.focal_loss_alpha, self.focal_loss_gamma, self.cls_weights, self.reg_weights) File "/home/golven/code/QueryDet-PyTorch/models/retinanet/retinanet.py", line 278, in det_loss assert len(cls_weights) == len(pred_logits) AssertionError

Hello, I also have the same problem. How did you solve it?


think about using template engine

Description:
Since we are planning to allow more customization (#170) and also early to do high breaking changes. It would be nice if we give a templating engine a shot (looking into all options handlebars seems very nice for use case but open for other options).
The reason for this is while tackling #174, #175 and with #135 we started reinventing wheel and creating our own solutions like having our identifier like $$$ for using vars, creating utils like stringify for allowing objects in .args.mjs etc.
Looking at current .template.mjs I think we need the following features:

Allowing template file(currently called as .template.mjs) to accepts args from extensions (currently passed via `.args.mjs)
Easily able to pass objects(with syntax highlighting) helpful for config files.

Objects can access arbitrary variables as well (which will be present in final full template)
mentioning somehow that weather this object will be merge with default SE-2 config or replaced (for certain files)

Passing of code snippets.
For future: We need to start thinking about this on how to allow multiple extensions as well (not in initial iteration but just to have it in back of our mind)
Not that imp: In --dev we create .dev files showing what args were passed.
DX for creating extensions remains same or at least improved(like current utils we have are in --dev mode symlinks)

We kind of have the above things working with current templating logic and utils but this workaround breaks when certain new use case comes up for example this where stringify  util failed.
Let's try to research and tinker more with it and see if it better fits for us 🙌

Past discussions:

Search this quoted text in SE-2 core TG group(for some reason I wasn't able to copy link to message):
"I forked what Shiv made, and made some tweak in order to test point 2 (handlebars)." - Carlos creating v0.1 with hbs checkout video about it.
https://www.veed.io/view/24b6495f-32e7-4e31-84c4-f7c9ae922668?panel=share - Initial templating video of passing with args in handlebars.
Branch with handle bars : https://github.com/carletex/create-dapp-example/tree/feat/extenstion-templateConfig
"Testing an alternative to handlebars.... using a function." - Suggestion on using current version of templates instead of handlebars.
"then, for template files maybe do something different, where instead of the actual resulting file we create a special file like file.ext.dev.mjs" - Dani tinker on --dev mode so that we show what args were passed.
Dani's Initial Draft with templates literals (read RFC-extensions.md):
https://github.com/carletex/create-dapp-example/pull/2/files#diff-1c5bb9873a4d62b19f0844b00ade8d65295ffdd188d146cea2e9ce43e60e778a

I created a repo to tinker with handlebars https://github.com/rin-st/tinker-handlebars , see README how to use it.
Current thoughts regarding this:

Objects can access arbitrary variables as well (which will be present in final full template)

I didn't find a good way to do it with handlebars. I see two ways:

Use special characters for variables like we already do (maybe something simpler than $$$)
Pass an object with variables, where every variable will be sent as string https://github.com/rin-st/tinker-handlebars/blob/main/different-objects/data.ts#L13 . But it already requires a lot of code to handle that object and to merge with other objects (see the whole helpers file) . Also, it's very inconvenient for our objects. See base example here and assume we need to use it for all the networks and all kind of different vars (arrays, process.env, ${...} etc, that logic is not even work for now)

You can also play with that repo, but for now looks like it's not worth to use handlebars at all and our solution is much better


An alternative to "run" that allows specifying a path.

It would be nice if there was a way to specify a path to a file to run, rather than inlining the script. Something like this:
[[task]]
name = "generate:api-docs"
engine = "node"
run = "./path/to/my/script.js"

Inlining the scripts makes development of extensions a pain.

Yes, that would be:
[[task]]
name = "generate:api-docs"
run = "node path/to/my/script.js"


⚠ Bofrost.be has degraded performance

In c5028ad, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1514 ms

Resolved: Bofrost.be performance has improved in 001f21b after 20 minutes.


Inconsistent results

Hi, Thanks for your interesting works. When I trained the model, I found that it was inconsistent with the results in the paper, and the training loss was very large. The hyperparameter settings are all default, I just ran run_mlp4rec.py.

Hi, thanks for bringing this up! Do you mind copy the console output to this thread so that we can have a look what seems to be the problem?

Mon 19 Sep 2022 22:47:41 INFO  epoch 0 training [time: 1447.52s, train loss: 4454.5431]
Mon 19 Sep 2022 22:58:09 INFO  epoch 0 evaluating [time: 628.51s, valid_score: 0.097000]
Mon 19 Sep 2022 22:58:09 INFO  valid result:
recall@10 : 0.2717    mrr@10 : 0.097    ndcg@10 : 0.1373    hit@10 : 0.2717    precision@10 : 0.0272
Mon 19 Sep 2022 22:58:09 INFO  Saving current best: saved/MLP4Rec-Sep-19-2022_22-23-33.pth
Mon 19 Sep 2022 23:22:14 INFO  epoch 1 training [time: 1444.15s, train loss: 3994.5049]
Mon 19 Sep 2022 23:32:42 INFO  epoch 1 evaluating [time: 628.02s, valid_score: 0.179700]
Mon 19 Sep 2022 23:32:42 INFO  valid result:
recall@10 : 0.3995    mrr@10 : 0.1797    ndcg@10 : 0.2313    hit@10 : 0.3995    precision@10 : 0.04
Mon 19 Sep 2022 23:32:42 INFO  Saving current best: saved/MLP4Rec-Sep-19-2022_22-23-33.pth
Mon 19 Sep 2022 23:56:46 INFO  epoch 2 training [time: 1444.31s, train loss: 3687.4842]
Tue 20 Sep 2022 00:07:16 INFO  epoch 2 evaluating [time: 630.31s, valid_score: 0.221700]
Tue 20 Sep 2022 00:07:16 INFO  valid result:
recall@10 : 0.4495    mrr@10 : 0.2217    ndcg@10 : 0.2753    hit@10 : 0.4495    precision@10 : 0.045
Tue 20 Sep 2022 00:07:17 INFO  Saving current best: saved/MLP4Rec-Sep-19-2022_22-23-33.pth
Tue 20 Sep 2022 00:31:22 INFO  epoch 3 training [time: 1445.17s, train loss: 3423.5285]
Tue 20 Sep 2022 00:41:46 INFO  epoch 3 evaluating [time: 623.91s, valid_score: 0.234500]
Tue 20 Sep 2022 00:41:46 INFO  valid result:
recall@10 : 0.4621    mrr@10 : 0.2345    ndcg@10 : 0.2882    hit@10 : 0.4621    precision@10 : 0.0462
Tue 20 Sep 2022 00:41:46 INFO  Saving current best: saved/MLP4Rec-Sep-19-2022_22-23-33.pth
Tue 20 Sep 2022 01:05:49 INFO  epoch 4 training [time: 1443.04s, train loss: 3220.5126]
Tue 20 Sep 2022 01:16:13 INFO  epoch 4 evaluating [time: 624.33s, valid_score: 0.239300]
n_layers = 4
hidden_size = 128
hidden_dropout_prob = 0.5
hidden_act = gelu
layer_norm_eps = 1e-12
initializer_range = 0.02
selected_features = ['sales_rank', 'categories', 'price', 'brand']
pooling_mode = mean
loss_type = CE
MODEL_TYPE = ModelType.SEQUENTIAL
MODEL_INPUT_TYPE = InputType.POINTWISE
eval_type = EvaluatorType.RANKING
device = cuda
train_neg_sample_args = {'strategy': 'none'}
eval_neg_sample_args = {'strategy': 'by', 'by': 100, 'distribution': 'popularity'}
I am confused and looking forward to your reply.

Hi, the result seems to be in an acceptable range, which metric exactly do you think is inconsistent?

I'm a beginner for the recommendation task, just I think the training loss seems a bit big.

Hi, based on my experience, I would say this training loss is quite common with respect to this particular dataset. Once the model is converge, you should observe a similar result as our paper presented :)
I hope this answers your question, if you have any further question please free to contact me or open a new issue anytime.

Hi, thanks for bringing this up! Can you copy the console output into a
text file and send that to me?
sherryyin1 @.***>于2022年9月19日 周一下午10:39写道：

Hi, Thanks for your interesting works. When I trained the model, I found
that it was inconsistent with the results in the paper, and the training
loss was very large. The hyperparameter settings are all default, I just
ran run_mlp4rec.py.
—
Reply to this email directly, view it on GitHub
https://github.com/Li-Muyang/MLP4Rec/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO5PLTPAVI6MQUPWCX5CWLDV7BNHTANCNFSM6AAAAAAQQCFFGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Question: Editing GPX

Could the GPX be edited by a draw control for example https://github.com/Leaflet/Leaflet.draw.
I tried the following code but it did not appear to work. The GPX displays fine but the draw control cannot edit it?
var gpxLayer = new L.GPX(gpx, {async: true});
        gpxLayer.on('loaded', function(e) { map.fitBounds(e.target.getBounds());});
        map.addLayer(gpxLayer);

    var options = {
        position: 'topleft',
        draw : {
            polygon: false,
            circle: false,
            rectangle: false,
            marker: false,
            line: false
        },

        edit: {
            featureGroup: gpxLayer,
            remove: false
        }
    };

    var drawControl = new L.Control.Draw(options);
    map.addControl(drawControl);

I must say I've never tried this. If there is a way to make it work, keep in mind that all the metadata information calculated by leaflet-gpx is done when reading/parsing the original GPX file, so none of that information will be accurate once you start editing the features directly.


Add .gitattibute for contribution from Windows

PR Checklist

[x] Addresses an existing open issue: related to #000
[x] Steps in Contributing were taken

Overview
I have had issues for contributing: I am in a Windows environment and git converts line endings with CRLF (by default core.autocrlf is set to true); some script like test do not work if this conversion happens.
I have added .gitattribute to avoid this from happening.

Hey @DanielBertocci!
Thanks a lot for your PR, very appreciate you've looked at the Windows configuration
I'm looking at your PR now. For future engineers benefit, I will add some comments for what each config setting means

@DanielBertocci thanks a lot, merging it!

@Beraliv I've noticed that I missed a s in the file name. It should be .gitattributes...it worked on my machine because in attempts to understand why I had those issues I changed the default behavior of git.


Fix travis yml

the current .travis.yml differs only from that one of iuc-tools by

&& flake8 --exclude=.git .
--urls

follow up on the two diffs


feat(grafana): apply MAINTENANCE.md upstream v0.12.0

Closing this task request as completed.
Reference:

Task Request #122

please keep the 2017-present format in the license header, otherwise lgtm

I ran the command available in the makefile to add the license "make add-license" do I need to add a flag? or do something else?

please keep the 2017-present format in the license header, otherwise lgtm

done

I don't see the main grafana/kustomization.yaml file changed, where we put our own registry and grafana version. Did you open a PR on container-image-sync repo?
Additionally see some other comments below point_down

the grafana/kustomization.yaml file was updated with the new version

please keep the 2017-present format in the license header, otherwise lgtm


6245 missing fragment imports fix

Related #6245
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

@gilgardosh can you please rebase? And this is still a draft, or it's ready for a review?

@dotansimha rebased and ready for review

@gilgardosh could you help getting the alpha release package for testing? If I understand https://github.com/dotansimha/graphql-code-generator/pull/6290#issuecomment-877768244 correctly it's a matter of adding the changeset entry.

@ertrzyiks  Here you go:
<EMAIL_ADDRESS>


Not startinng in current directory for drives mounted by subst

Your Windows build number:  (Type ver at a Windows Command Prompt)

Microsoft Windows [Version 10.0.18362.388]

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

When current directory is in the drive mounted by subst, wsl starts at ~, instead of current directory.
D:\> subst f: d:\$drives\F
D:\> f:
F:\> wsl
user@pc:~$ pwd
/home/user

This is same as #2043#issuecomment-402105837. But #3246, in which #2043 is merged, seems to be a topic only for virtual disks and optical disks.
In this case, /mnt/f is not appeared. It is mentioned in #789.
dmesg saids:
[    0.003052]  Microsoft 4.4.0-18362.1-Microsoft 4.4.35
[    0.086162] init: (8) ERROR: CreateProcessParseCommon:869: Failed to translate F:\

I created /mnt/f and mounted /mnt/d/$drive/f like subst, or created symbolic link /mnt/f -> /mnt/d/$drive/f, but both were ineffective.
The same applies when the drive mounted by registry 'HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\DOS Devices'. I use this method.

What's wrong / what should be happening instead:

I think wsl should start with current directory even on subst drives.
It is desirable that subst drive be automounted, but it can also be manually mount or a symbolic link.
In Windows 10 1803 (April 2018 Update), it had worked with symbolic link.

I have a similar problem see issue https://github.com/microsoft/WSL/issues/6798. It has been rejected due to an outdated but supported Windows version. Interestingly my network share is mounted read only.


Genome coverage estimation

Refactoring of the preassembly QC code:

Genome coverage now calculated using expected genome size of the reference genome and total bases from 'seqtk'
Removed contamination filtering and assembly of contaminated sequencings. Currently workflow will raise an exception if input is deemed to be contaminated by MASH.
Kraken2 species classification removed. Validation of contamination no longer required since contamination filtering is removed
Reorganized the QC logic. PASS/FAIL of each QC tool now stored in a dictionary ("qcVerdicts")
Fixed MASH always identifying multiple species. PHiX should not appear in the filtered list of MASH hits.

Excellent! :boom: :guitar: :boom:


Replace statefulMapConcat with statefulMap + mapConcat in codebase

As the title suggests, statefulMapConcat is deprecated so it should replaced with statefulMap + mapConcat so that the 1.1.0 version of pekko-http isn't relying on deprecated pekko methods.

also found in pekko-connectors

also found in pekko-connectors

Thanks, issue created at https://github.com/apache/incubator-pekko-connectors/issues/307

https://github.com/apache/incubator-pekko-http/pull/311 was merged already but a quick search indicates that we might still have 1 or 2 remaining uses.

I did a check seems there are no usage now. @pjfanning , the only lefts are in scaladoc and benchmarks

This might have performance implications (mapConcat had performance issues in the past) so make sure to benchmark before doing these changes.

This was deprecated in https://github.com/apache/incubator-pekko/issues/601 - if we think deprecating the code was a mistake then we can reopen that discussion. If we think it was the right decision to deprecate statefulMapConcat, then we should start removing its usage.

I like the change from an API orthogonality perspective, just saying that it might not come for free. In the worst case, it could be possible to keep a copy in StreamUtils, but better if that is not needed.

I think this issue can be closed as the associated change has been one. in https://github.com/apache/incubator-pekko-http/pull/311.
Or we can do some dedicated implementation just for pekko-http usage in StreamUtils, without any additional allocations.

The current statefulMap or mapAccumlate implementation will need a (S, A) , which is a kind  of allocation, in JEP https://openjdk.org/jeps/461 @viktorklang @viktorklang-ora is using an object which reducing this allocation.

@jrudolph I will review the changes and try to come up with an allocation-free implementation in pekko-http.

I'm going to close this because we undeprecated statefulMapConcat. Feel free to reopen.


improve the style of the test case and model template

improve the style of the test case and model's template to make it more readable and containing guidelines.

Implemented and working as expected.


ffmpeg_4-full: disable glslang

abd83e2924047706a54beb65692cb6832e0e3740 broke glslang support in ffmpeg_4. It's only enabled in ffmepg-full and v4 is only kept for compat reasons, so let's simply disable this feature.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 267700 run on x86_64-linux 1

7 packages built:

ffmpeg_4-full
ffmpeg_4-full.bin
ffmpeg_4-full.data
ffmpeg_4-full.dev
ffmpeg_4-full.doc
ffmpeg_4-full.lib
ffmpeg_4-full.man


Cursor stays hidden or hides unnecessary

Hyprland Version
d9b74ff96b3e3f0341a386451b9a3375b6f05503
Bug or Regression?
Regression
Description
Since 0c74df4f9efa0efe6ff1e73c74999acc217f68bb I noticed two issues with cursor hiding.

With cursor_inactive_timeout = 2 opening a video from qBittorrent hides the cursor immediately and even if I move the cursor to another window, or close the video player the cursor stays hidden.

With cursor_inactive_timeout = 0 if I move the cursor to window that will hide the cursor, for example mpv, and I move the cursor to another window while it's visible, it will be hidden over the second window.

How to reproduce
(all floating windows)

cursor_inactive_timeout = 2
Open a video from a torrent in qBittorrent
The video will open (mpv) under the cursor, which will immediately be hidden
Move cursor back to qBtitorent, you'll see that the window gets the focus, but cursor stays hidden
Moving the cursor over waybar will make it reappear

cursor_inactive_timeout = 0
Open mpv on the left, a second window on the right (I used alacritty)
Move cursor over mpv, and move it to the other window before it disappears
Wait a sec and the cursor will disappear on top of the second window
(something it works, sometimes it doesn't, so try a few times moving it from one window to anoter)

Crash reports, logs, images, videos
No response

For me inside mpv, once cursor is hidden due to inactivity, it stays hidden (in full screen/floating/tiled ) even after moving cursor, I have to move it outside mvp to make it visible again.

check with above

d1b8a63a8ef1d7258d49854d2256fa9b99fd99bc obviously breaks cursor_inactive_timeout.

ok, but does it fix this issue?

@andresilva check on latest, should work

Now the cursor is never hidden at all.

No change for me, both issues still persist.
@andresilva Unfortunately, the never hiding issue is not new. You have to move around your cursor, to a window which changes the cursor, then back, and at some point it will start working again.
(Just like hidden cursor in games, not respecting the timeout (sometimes it takes much longer than the setting), the cursor repeatedly appears/hides in some applications without touching the mouse (while watching a video for example).)

No, it's a different issue, with a197fe3c11d5ac095bf8e2f491edf28bde9f9b49 the cursor literally never hides. And d1b8a63a8ef1d7258d49854d2256fa9b99fd99bc just regresses to the state it was before 0c74df4f9efa0efe6ff1e73c74999acc217f68bb, i.e. when you move focus with keyboard the cursor reappears.

done some testing, above seems to work for me

Fixed for me on https://github.com/hyprwm/Hyprland/commit/48ecb13b14d99241725a65582edf42cd51843e94, including the original issue described in OP (although I only tested with cursor_inactive_timeout = 2).

Actually I just tested with default settings, so cursor_inactive_timeout = 0 and it has both issues.
I forgot to tell that I'm on nVidia and I have WLR_NO_HARDWARE_CURSORS = 1.
https://github.com/hyprwm/Hyprland/assets/150595692/18670d1f-acb0-44be-a706-369ef54bc903

I fixed it by removing the check of m_bWindowRequestedCursorHide in CHyprRenderer::shouldRenderCursor(), and because the variable didn't have any other uses, I just removed it entirely.
I'm 90% sure that this breaks cursor hiding in other ways, but from my testing(which is just seeing if mpv with cursor-autohide=always actually hides the cursor and whether cursor_inactive_timeout works) I haven't experienced any of that.
If it changes anything, I'm also on NVIDIA with WLR_NO_HARDWARE_CURSORS = 1
diff --git a/src/managers/input/InputManager.cpp b/src/managers/input/InputManager.cpp
index db06de55..a3a649fe 100644
--- a/src/managers/input/InputManager.cpp
+++ b/src/managers/input/InputManager.cpp
@@ -483,11 +483,6 @@ void CInputManager::onMouseButton(wlr_pointer_button_event* e) {
 }

 void CInputManager::processMouseRequest(wlr_seat_pointer_request_set_cursor_event* e) {
-    if (!e->surface)
-        g_pHyprRenderer->m_bWindowRequestedCursorHide = true;
-    else
-        g_pHyprRenderer->m_bWindowRequestedCursorHide = false;
-
     if (!cursorImageUnlocked())
         return;

diff --git a/src/render/Renderer.cpp b/src/render/Renderer.cpp
index 3afbd551..c0a43056 100644
--- a/src/render/Renderer.cpp
+++ b/src/render/Renderer.cpp
@@ -2097,7 +2097,7 @@ void CHyprRenderer::setCursorHidden(bool hide) {
 }

 bool CHyprRenderer::shouldRenderCursor() {
-    return !m_bCursorHidden && !m_bWindowRequestedCursorHide && m_bCursorHasSurface;
+    return !m_bCursorHidden && m_bCursorHasSurface;
 }

 std::tuple<float, float, float> CHyprRenderer::getRenderTimes(CMonitor* pMonitor) {
diff --git a/src/render/Renderer.hpp b/src/render/Renderer.hpp
index 32815344..6f0d3c51 100644
--- a/src/render/Renderer.hpp
+++ b/src/render/Renderer.hpp
@@ -75,7 +75,6 @@ class CHyprRenderer {
     bool      beginRender(CMonitor* pMonitor, CRegion& damage, eRenderMode mode = RENDER_MODE_NORMAL, wlr_buffer* buffer = nullptr, CFramebuffer* fb = nullptr);
     void      endRender();

-    bool      m_bWindowRequestedCursorHide = false;
     bool      m_bBlockSurfaceFeedback      = false;
     bool      m_bRenderingSnapshot         = false;
     CWindow*  m_pLastScanout               = nullptr;

Fixed for me on 48ecb13 ~, including the original issue described in OP (although I only tested with cursor_inactive_timeout = 2).~ (the original issue in OP still persists).

am also facing the exact same issue any updates ?

No, it's a different issue, with a197fe3 the cursor literally never hides. And d1b8a63 just regresses to the state it was before 0c74df4, i.e. when you move focus with keyboard the cursor reappears.

i am also facing the exact same issue , any updates ?

For me inside mpv, once cursor is hidden due to inactivity, it stays hidden (in full screen/floating/tiled ) even after moving cursor, I have to move it outside mvp to make it visible again.

i am also facing the exact same issue , any updates ?

committed the patch, thanks dtop129!

lmk if it's fixed.

Seems to be fine now.


How to use Traffic Manager with the Bridge

Hello,
I would like to configure the Traffic Manager (TM) so that it works in deterministic mode. In deterministic mode, the TM will produce the same results and behaviors under the same conditions.
Is that possible with the bridge?
We tried the code below but the inclusion of the instructions related to the TM blocks the non-player characters.
Thanks a lot
`import carla
import random
import time
Connect to the client and retrieve the world object
client = carla.Client('localhost', 3000)
world = client.get_world()
#world = client.load_world('Town03')
If we configure the TM as in the code below the simulation doesn't start
Set up the simulator in synchronous mode
settings = world.get_settings()
settings.synchronous_mode = True # Enables synchronous mode
settings.fixed_delta_seconds = 0.05
world.apply_settings(settings)
Set up the TM in synchronous mode
traffic_manager = client.get_trafficmanager()
traffic_manager.set_synchronous_mode(True)
Set a seed so behaviour can be repeated if necessary
traffic_manager.set_random_device_seed(0)
random.seed(0)
We will aslo set up the spectator so we can see what we do
spectator = world.get_spectator()
spawn_points = world.get_map().get_spawn_points()`

Hi @epris I didn't try TM before, but we are using sync mode by default.
Could you provide some information, like which branch you are using now and where you add these code (in our Python script here? https://github.com/evshary/autoware_carla_launch/blob/1ee137b61dc37df0547a56ac9ee89fe842ba9033/script/run-bridge.sh#L17)

Hello
we set the bridge in sync mode in the file run_bridge.sh and now the Traffic Manager works. The code above is a piece of a script that creates a scenario with the spawn of several players. The problem is that by setting the seed (both random seed and Traffic Manager seed as in the code above) and repeating the same scenario several times, the behavior is not the same, in particular the behavior of the ego-vehicle is different for each run (the velocity of vehicle is different) while that of the other players is the same. For both the ego-vehicle and the other players, autopilot is set.
The bridge, Carla and the Traffic Manager are in sync mode

Hi @JessicaFerrara, if I didn't misunderstand your settings, you haven't run Autoware and just set ego-vehicle to autopilot mode, but the ego-vehicle doesn't have a consistent behavior.
Here is how I configured the ego-vehicle, perhaps you can take a look and tweak the configuration
https://github.com/evshary/zenoh_carla_bridge/blob/main/carla_agent/simulation/world.py#L83

Hello, I run Autoware, the bridge and the Carla server. I also run a file that creates a scenario and sets the autopilot to the ego-vehicle. In order to repeat the same scenario several times and have the same behaviour of the players, I have used the Traffic Manager with a seed, but the behaviour of the ego-vehicle is not consistent, in fact it happens that it blocks, i.e. the speed is 0 km/h, for no reason (there is no error). Below is the file i use
import time
import random
import carla
client = carla.Client('localhost', 3000)
#client.set_timeout(2.0)
world = client.get_world()
random.seed(9)
Set up the simulator in synchronous mode
settings = world.get_settings()
settings.synchronous_mode = True # Enables synchronous mode
settings.fixed_delta_seconds = 0.05
world.apply_settings(settings)
Set up the TM in synchronous mode
traffic_manager = client.get_trafficmanager()
traffic_manager.set_synchronous_mode(True)
Set a seed so behaviour can be repeated if necessary
traffic_manager.set_random_device_seed(9)
blueprint_library = world.get_blueprint_library()
vehicle = blueprint_library.find('vehicle.tesla.model3')
vehicle2 = blueprint_library.find('vehicle.ford.mustang')
walker = blueprint_library.find('walker.pedestrian.0001')
cyclist = blueprint_library.find('vehicle.bh.crossbike')
moto = blueprint_library.find('vehicle.kawasaki.ninja')
moto2 = blueprint_library.find('vehicle.yamaha.yzf')
truck = blueprint_library.find('vehicle.carlamotors.firetruck')
truck2 = blueprint_library.find('vehicle.tesla.cybertruck')
walker_controller_bp = blueprint_library.find('controller.ai.walker')
s2=carla.Transform(carla.Location(x=55.7, y=-206.6, z=0.3), carla.Rotation(pitch=0.000000, yaw=180.00, roll=0.000000))
s3=carla.Transform(carla.Location(x=34.7, y=-196.6, z=0.3), carla.Rotation(pitch=0.000000, yaw=0.00, roll=0.000000))
s4=carla.Transform(carla.Location(x=23.8, y=-193.6, z=0.3), carla.Rotation(pitch=0.000000, yaw=0.00, roll=0.000000))
s5=carla.Transform(carla.Location(x=-28.2, y=-198.4, z=0.3), carla.Rotation(pitch=0.000000, yaw=0.00, roll=0.000000))
s6=carla.Transform(carla.Location(x=-36.6, y=-194.9, z=0.3), carla.Rotation(pitch=0.000000, yaw=0.00, roll=0.000000))
s7=carla.Transform(carla.Location(x=62.92, y=-209.88, z=0.3), carla.Rotation(pitch=0.000000, yaw=-180.00, roll=0.000000))
s8=carla.Transform(carla.Location(x=44.8, y=-212.6, z=0.3), carla.Rotation(pitch=0.000000, yaw=180.00, roll=0.000000))
v2 = world.spawn_actor(vehicle, s2)
v3 = world.spawn_actor(vehicle, s3)
v4 = world.spawn_actor(truck, s4)
v5 = world.spawn_actor(moto, s5)
v6 = world.spawn_actor(vehicle, s6)
v7 = world.spawn_actor(cyclist, s7)
v8 = world.spawn_actor(walker, s8)
controller = world.try_spawn_actor(walker_controller_bp, carla.Transform(), attach_to=v8)
walker_control = carla.WalkerControl()
walker_control.speed = 2
walker_control.direction = carla.Vector3D(-1, 0, 0)
time.sleep(2)
vehicle_id = 109
ego_vehicle = world.get_actor(vehicle_id)
v7.set_autopilot(True)
time.sleep(3)
v2.set_autopilot(True)
v3.set_autopilot(True)
v4.set_autopilot(True)
v5.set_autopilot(True)
v6.set_autopilot(True)
v8.apply_control(walker_control)
ego_vehicle.set_autopilot(True)

Hi @JessicaFerrara Sorry in advance that I updated your code to make it easier to read.
I still don't quite understand what you want to achieve. Is your ego vehicle here controlled by Autoware or it's just another vehicle controlled by the traffic manager?

Hi @evshary, @JessicaFerrara and I are using the system in the following way: we run the Carla Server and the Bridge, then we run Autoware and eventually, we run the code shared by @JessicaFerrara.
The bridge spawns the ego vehicle, which is then controlled by the traffic manager with the autopilot.

Hi @epris, I can give it a try next week and see what will happen. However, what I can't get is why the ego vehicle is controlled by both Autoware and the traffic manager. I thought it was only controlled by Autoware. Is there any reason for that?

Hi @evshary, we run Autoware because we need to record the topics published by the different nodes. In particular, we are interested in topics related to perception and tracking. When we observe the behavior of the ego vehicle in Rviz, we see that it follows the same trajectory in Carla.

Hi @epris @JessicaFerrara could you please try the branch here?
https://github.com/evshary/zenoh_carla_bridge/tree/fix_update_value
I found that I always update the control message to Carla even when there is no message sent from Autoware. Perhaps this is why the strange behavior happens.


Move the Crates.io Team under the Dev Tools team

This RFC proposes moving the Crates.io Team under the Dev Tools Team. This change has very little impact outside of the removal of the Crates.io Council representative. Crates.io Team interests will be represented by the Dev Tools team representative.
Rendered

@rfcbot fcp merge
As there haven't been any objections or other discussion, I'm proposing to merge this!

Although I don't think there will be any objections, I would like to also have a clear approval from the devtools team as well. We don't have a formal process for adding teams, but I think just making checkboxes here should suffice. I'm checking @Manishearth's box since there is already an approval above.

[x] @Manishearth
[ ] @GuillaumeGomez
[ ] @oli-obk
[ ] @rbtcollins
[x] @ehuss
[ ] @calebcartwright

Thanks everyone! The @rust-lang/leadership-council has decided to accept this RFC.
@carols10cents can you follow up with updating the team repo whenever you have a chance? It should look something like https://github.com/rust-lang/team/pull/1228. I can help if needed.

@carols10cents can you follow up with updating the team repo whenever you have a chance? It should look something like rust-lang/team#1228. I can help if needed.

Done! https://github.com/rust-lang/team/pull/1439


fix(FX-3472): Re-focus search input on clear

The type of this PR is: Bugfix
This PR resolves FX-3472
Description
After searching and pressing one of the pills the keyboard is dismissed, but when pressing the X button it doesn't re-focus on the search bar
Before
https://user-images.githubusercontent.com/44819355/138253521-eb8c23d4-3ae8-45f6-9dc9-09a42c4db85e.mp4
After
https://user-images.githubusercontent.com/44819355/138253123-97d63dea-1dd5-4be0-9a10-7b38da37f68c.mp4
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[x] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 41d591ad7578cd4c2fe753141f8e74fee5677e45


How to set 'Updated time'  to 24hrs clock?

I don't like using AM and PM, my Macbook is displaying 24hrs and its faster to see if it matches.
It regularly falls behind so i need to check it fast.

Hi,
I just installed this and had the same issue. I guess the time format should be displayed as it's set in the os regional setting but I am not a developer :) Not a pretty solution, but works. Open the nightscout_osx_menubar.py as described in the readme and make a small change.
Line 88:
Original (am/pm)
"Updated %s" % datetime.now().strftime("%a %-I:%M %p"),
Updated (24H)
"Updated %s" % datetime.now().strftime("%a %-H:%M"),


payment_methods endpoint returns hasSubscription true for canceled subscriptions

General information

SDK version: 3.73.1
Environment: Sandbox
Browser and OS: Chrome 90.0.4430.212 (64-bit) on MacOS 10.15.7

Issue description
Hello there,
Regarding the /payment_methods endpoint, we noticed that hasSubscription:true is returned for cards that their subscriptions have been canceled. Are there any recent changes to the endpoint?
Is this maybe connected to https://github.com/braintree/braintree-web/issues/566?
Thank you in advance

I don't think it's related to #566
I think the logic we are using to populate the value just checks if there is a subscription object attached to the payment method at all, regardless of the status of the subscription.
We'll have to think about the best way to handle this situation. It's pretty clear cut that the subscription is tied to the payment method when the status is active and no longer tied to the payment method when canceled, but I'm not 100% sure on the situations for the other statuses.
My assumption here is that expired and past due subscription would also have hasSubscription: false while pending would have hasSubscription: true, but I want to check in with my team and the owners of the API before making those changes. Basically, we can only due that if canceled, expired, and past due are terminal states, indicating that there's no way for the subscription to be re-activated. Otherwise, we risk creating a breaking change.

What we observed is that on some cases hasSubscription:true is returned even if the state of the subscription is canceled.
Thanks a lot for the reply, looking forward for the solution that will be covering all the terminal states.

Hello @crookedneighbor, yes this is covers up our needs fully. Thank you very much!

Hello @crookedneighbor! Do we have any updates on this one? thank you very much

No update yet, we have a lot of things on our plate and this has not yet been prioritized. I would encourage you to reach out to our support team or your account manager to get more visibility on this improvement.

@crookedneighbor thanks a lot for the reply!


Vague

Section that requires update
https://github.com/cloudflare/cloudflare-docs/blob/production/products/registrar/src/content/transfer-instructions/namecheap.md
Step 4: Transfer to Cloudflare
Additional information
Quote: "In the Cloudflare transfer screen, input the...".
Problem: Where is this transfer screen? I can't find it.

Hey @the-wazz. I've updated the docs and it should be easier to understand now. Going to close this. Thank you for bringing this to our attention!


[R-package] Windows R package installation

Environment info
Operating System: windows 10 x64
CPU: i7-3770K
C++/Python/R version:  Microsoft R Open 3.4.0
Error Message:
Error in eval(ei, envir) : Cannot find lib_lightgbm.dll

removing 'C:/R/MRO-3.4.0/library/lightgbm'
restoring previous 'C:/R/MRO-3.4.0/library/lightgbm'

Steps to reproduce

R CMD INSTALL --build .  with use_precompile = FALSE, use_gpu = FALSE and use_mingw = FALSE

There is a missing / in the install.libs.R#L49, and it need to be lib_folder <- paste0(R_PACKAGE_SOURCE, "/src/Release/")

fixed in the latest PR.

Is this a bug of R which happens only in Windows 10? I think it might be worth reporting the missing slash issue in Windows 10 to the R team so it gets patched.

@Laurae2
I think the reason is different R version ( like Microsoft R Open) .

·* installing source package 'lightgbm' ...·
·** libs·
·installing via 'install.libs.R' to D:/R/soft/R-3.4.1/library/lightgbm·
·Warning: 运行命令'cmake  -DCMAKE_GENERATOR_PLATFORM=x64  ..'的状态是127·
·Warning: 运行命令'cmake --build . --target _lightgbm  --config Release'的状态是127·
·Error in eval(ei, envir) : Cannot find lib_lightgbm.dll·
·* removing 'D:/R/soft/R-3.4.1/library/lightgbm'·
·Installation failed: Command failed (1)·
·Warning message:··
·GitHub repo contains submodules, may not function as expected! ·
in R, is it only one wtep to install with sentence devtools::install_github("Microsoft/LightGBM", subdir = "R-package")? or i need to install git and cmake first?

refer to: https://github.com/Microsoft/LightGBM/tree/master/R-package#preparation

@guolinke  after my installation of git /cmake and rtools in win10(x64), all of them are set in env--path, but when i run devtools::install_github("Microsoft/LightGBM", subdir = "R-package"), it always ask me to install rtools again and again... and  same issues happened as above..
when i tried cd d:/R git clone --recursive https://github.com/Microsoft/LightGBM cd LightGBM/R-package ' R.exe CMD INSTALL --build .'  it shows:

installing to library 'D:/R/soft/R-3.4.1/library'

installing source package 'lightgbm' ...
** libs
installing via 'install.libs.R' to D:/R/soft/R-3.4.1/library/lightgbm
-- Building for: NMake Makefiles
CMake Error at CMakeLists.txt:8 (PROJECT):
Generator
NMake Makefiles
does not support platform specification, but platform
x64
was specified.

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "D:/R/LightGBM/R-package/src/build/CMakeFiles/CMakeOutput.log".
Warning: 'cmake  -DCMAKE_GENERATOR_PLATFORM=x64  ..'1
系统找不到指定的文件。
CMake Error: Generator: execution of make failed. Make command was: "nmake" "/NOLOGO" "_lightgbm"
Warning: 'cmake --build . --target _lightgbm  --config Release'1
Error in eval(ei, envir) : Cannot find lib_lightgbm.dll

removing 'D:/R/soft/R-3.4.1/library/lightgbm'

@zheguzai100 , please make sure that your system was x64 supported, as well as your build tools

@zheguzai100
you need the visual studio in windows. And you should run it by the windows CMD ( not mingw or others ).
BTW, what is NMake  in your logs ?

can't the R package run in rstudio instead of visual studio ?? and i run senetences in Windows powershell

Lightgbm is written by cpp, and visual studio is used for the compiling cpp codes in windows.
And can you make sure your R/Rtools is 64bit version?

I have very similar problem here. Let me list out my tool version first:
win10 (x64)
R version 3.4.1 (2017-06-30) -- "Single Candle" Platform: x86_64-w64-mingw32/x64 (64-bit)
Rtool: Rtools34.exe. (Rtools 2.12 and later include both 32 bit and 64 bit tools.)
Cmake: cmake-3.7.2-win64-x64
Visual Studio: Community 2017 (2), Vewrsion 15.2
This is my PATH:
"C:\\ProgramData\\Oracle\\Java\\javapath;C:\\Windows\\system32;C:\\Windows;C:\\Windows\\System32\\Wbem;C:\\Windows\\System32\\WindowsPowerShell\\v1.0\\;C:\\Program Files (x86)\\ATI Technologies\\ATI.ACE\\Core-Static;C:\\Program Files\\Git\\cmd;C:\\Program Files\\nodejs\\;C:\\Anaconda2;C:\\Anaconda2\\Scripts;C:\\Anaconda2\\Library\\bin;C:\\Users\\zhouzilu\\AppData\\Roaming\\npm;C:\\Program Files\\R\\R-3.4.1\\bin;C:\\Rtools\\bin;C:\\Rtools\\mingw_64\\bin;C:\\mingw64;C:\\Program Files\\cmake-3.7.2-win64-x64\bin"
When I trying to run devtools::install_github("Microsoft/LightGBM", subdir = "R-package"), it shows
Downloading GitHub repo Microsoft/LightGBM@master
from URL https://api.github.com/repos/Microsoft/LightGBM/zipball/master
Installing lightgbm
"C:/PROGRA~1/R/R-34~1.1/bin/x64/R" --no-site-file --no-environ --no-save
--no-restore --quiet CMD INSTALL
"C:/Users/zhouzilu/AppData/Local/Temp/Rtmp2xMwLa/devtools51c04da97060/Microsoft-LightGBM-bde7f7c/R-package"
--library="C:/Users/zhouzilu/Documents/R/win-library/3.4" --install-tests

installing source package 'lightgbm' ...
** libs

*** arch - i386
installing via 'install.libs.R' to C:/Users/zhouzilu/Documents/R/win-library/3.4/lightgbm
-- Building for: NMake Makefiles
CMake Error at CMakeLists.txt:8 (PROJECT):
Generator
NMake Makefiles

does not support platform specification, but platform
x64

was specified.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "C:/Users/zhouzilu/AppData/Local/Temp/Rtmp2xMwLa/devtools51c04da97060/Microsoft-LightGBM-bde7f7c/R-package/src/build/CMakeFiles/CMakeOutput.log".
Warning: running command 'cmake  -DCMAKE_GENERATOR_PLATFORM=x64  ..' had status 1
The system cannot find the file specified
CMake Error: Generator: execution of make failed. Make command was: "nmake" "/NOLOGO" "_lightgbm"
Warning: running command 'cmake --build . --target _lightgbm  --config Release' had status 1
Error in eval(ei, envir) : Cannot find lib_lightgbm.dll

removing 'C:/Users/zhouzilu/Documents/R/win-library/3.4/lightgbm'
Installation failed: Command failed (1)
Warning message:
GitHub repo contains submodules, may not function as expected!

I then try to install with
git clone --recursive https://github.com/Microsoft/LightGBM cd LightGBM/R-package R CMD INSTALL --build .
Same error here,

installing to library 'C:/Users/zhouzilu/Documents/R/win-library/3.4'
installing source package 'lightgbm' ...
** libs

*** arch - i386
installing via 'install.libs.R' to C:/Users/zhouzilu/Documents/R/win-library/3.4/lightgbm
-- Building for: NMake Makefiles
CMake Error at CMakeLists.txt:8 (PROJECT):
Generator
NMake Makefiles

does not support platform specification, but platform
x64

was specified.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "C:/Users/zhouzilu/Desktop/LightGBM/R-package/src/build/CMakeFiles/CMakeOutput.log".
Warning: running command 'cmake  -DCMAKE_GENERATOR_PLATFORM=x64  ..' had status 1
The system cannot find the file specified
CMake Error: Generator: execution of make failed. Make command was: "nmake" "/NOLOGO" "_lightgbm"
Warning: running command 'cmake --build . --target _lightgbm  --config Release' had status 1
Error in eval(ei, envir) : Cannot find lib_lightgbm.dll

removing 'C:/Users/zhouzilu/Documents/R/win-library/3.4/lightgbm'
I have also tried different CMake Version including 3.8.2 and 3.9.0-rc5. Also tried installer and ZIP, but in vain. Somebody helps me plz.

@zz327

when installing Rtools, can you only install the 64-bit version? from your log, it is built by 32-bit Rtools.

The default compiler is Visual Studio (or MS Build) in Windows. You need to install one.

@guolinke
Thank you for your answer.
I uninstall my Rtools and reinstall as you stated above. Also I have installed Visual Studio 2017 Community Version 15.2. In addtion, I add MSBuild as the extension. However, it shows the same problem as above. Should I add any thing related to the Visual Studio to my PATH? I am a little confused...
ttttyyyy

git ; cmake ; visual studio installed automatically
R 3.4.1 x64 ; rtools34 x64 ;
this morning, i try to install lightgbm,

asking me to install rtools again(but i'v installed and  set in system env --path )
‘devtools::install_github("Microsoft/LightGBM", subdir = "R-package")’
Downloading GitHub repo Microsoft/LightGBM@master
from URL https://api.github.com/repos/Microsoft/LightGBM/zipball/master
Installing lightgbm
"D:/R/soft/R-34~1.1/bin/x64/R" --no-site-file --no-environ --no-save --no-restore --quiet CMD INSTALL
"C:/Users/Jose/AppData/Local/Temp/RtmpawkL4J/devtools10741faf4377/Microsoft-LightGBM-13ce3dc/R-package"
--library="D:/R/soft/R-3.4.1/library" --install-tests

installing source package 'lightgbm' ...
** libs
installing via 'install.libs.R' to D:/R/soft/R-3.4.1/library/lightgbm
-- Building for: NMake Makefiles
CMake Error at CMakeLists.txt:8 (PROJECT):
Generator
NMake Makefiles
does not support platform specification, but platform
x64
was specified.

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "C:/Users/Jose/AppData/Local/Temp/RtmpawkL4J/devtools10741faf4377/Microsoft-LightGBM-13ce3dc/R-package/src/build/CMakeFiles/CMakeOutput.log".
Warning: 运行命令'cmake  -DCMAKE_GENERATOR_PLATFORM=x64  ..'的状态是1
绯荤粺鎵句笉鍒版寚瀹氱殑鏂囦欢銆?CMake Error: Generator: execution of make failed. Make command was: "nmake" "/NOLOGO" "_lightgbm"
Warning: 运行命令'cmake --build . --target _lightgbm  --config Release'的状态是1
Error in eval(ei, envir) : Cannot find lib_lightgbm.dll

removing 'D:/R/soft/R-3.4.1/library/lightgbm'
Installation failed: Command failed (1)
Warning message:
GitHub repo contains submodules, may not function as expected!

@zheguzai100 @zz327
I just test the installation based on your environment, it will fail if using cmake 3.7.2 . Can you upgrade your cmake to the latest version ?

@zz327
@guolinke
hi all, i tried another way and get the target
devtools::install_github("Laurae2/lgbdl")
library(lgbdl)
lgb.dl(commit = "master",
       compiler = "gcc",
       repo = "https://github.com/Microsoft/LightGBM",
       cores = 4)
but at  the beginning of lgb.dl, i saw a cmake error reported:
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage -- Configuring incomplete, errors occurred!
fortunately, it doesn't influence th eresult,  it worked when i run library(lightgbm)

@zheguzai100
I think your installation may actually fails, can you run some demos ?

@guolinke
does it mean fail or not?
`> require(lightgbm)
载入需要的程辑包：lightgbm
载入需要的程辑包：R6

bst <- lightgbm(data = train$data,

            label = train$label,

            num_leaves = 4,

            learning_rate = 1,

            nrounds = 2,

            objective = "binary")

[LightGBM] [Fatal] len of label is not same with #data
[LightGBM] [Warning] Only contain one class.
[LightGBM] [Info] Number of positive: 0, number of negative: 209258376
[LightGBM] [Info] Total Bins 112
[LightGBM] [Info] Number of data: 209258376, number of used features: 63
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements.
[1]:	train's binary_logloss:9.99201e-16
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements.
[2]:	train's binary_logloss:0 `

@guolinke
Thanks for your response! I downloaded the latest release Cmake 3.8.2 and modified my PATH.
-- Building for: NMake Makefiles
CMake Error at CMakeLists.txt:8 (PROJECT):
Generator
NMake Makefiles

does not support platform specification, but platform
x64

was specified.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "C:/Users/zhouzilu/AppData/Local/Temp/RtmpKimkeL/devtools76d43a9355f7/Microsoft-LightGBM-8dde231/R-package/src/build/CMakeFiles/CMakeOutput.log".
Warning: running command 'cmake  -DCMAKE_GENERATOR_PLATFORM=x64  ..' had status 1
The system cannot find the file specified
CMake Error: Generator: execution of make failed. Make command was: "nmake" "/NOLOGO" "_lightgbm"
Warning: running command 'cmake --build . --target _lightgbm  --config Release' had status 1
Error in eval(expr, envir, enclos) : Cannot find lib_lightgbm.dll

removing 'C:/Users/zhouzilu/Documents/R/win-library/3.3/lightgbm'
Error: Command failed (1)
In addition: Warning message:
GitHub repo contains submodules, may not function as expected!
Still same error... I have decided to use Python for now, but I will keep trying...

@zheguzai100 it is success. But your data only have one class, which cannot be trained by binary classification.

@zz327
cmake will have some cache files.
can you delete the LightGBM folder, then re-clone and try again ?

We load in the agaricus dataset
In this example, we are aiming to predict whether a mushroom is edible
data(agaricus.train, package = "lightgbm")
data(agaricus.test, package = "lightgbm")
train <- agaricus.train
test <- agaricus.test
The loaded data is stored in sparseMatrix, and label is a numeric vector in {0,1}
class(train$label)
[1] "numeric"
class(train$data)
[1] "dgCMatrix"
attr(,"package")
[1] "Matrix"
table(train$label)

0    1
3373 3140
why does it recognize only one class??

@guolinke
Tried, still the same error...
Is this maybe due to win10? Can you try uninstall and then reinstall CMake to see if it works?

I am using win10 as well.
restalling cmake still works.
Did you choose the visual c++ when installing visual studio 2017?

can you paste the contents of running 'cmake --help' ?
Also, from your log, it seems you have many R versions(i saw 3.3, 3.4, 3.4.1) .
Can you paste the latest installation log?

I seem to getting a similar error when installing in R:
`Build succeeded.
"C:\Users\Howard\AppData\Local\Temp\Rtmpeq7jnP\devtools3538ea06bf\Microsoft-LightGBM-43d5037\R-package\src\build_lightgbm.vcxproj" (default target) (1) ->
"C:\Users\Howard\AppData\Local\Temp\Rtmpeq7jnP\devtools3538ea06bf\Microsoft-LightGBM-43d5037\R-package\src\build\ZERO_CHECK.vcxproj" (default target) (2) ->
(PrepareForBuild target) ->
C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\Common7\IDE\VC\VCTargets\Microsoft.CppBuild.targets(395,5): warning MSB8029: The Intermediate directory or Output directory cannot reside under the Temporary directory as it could lead to issues with incremental build. [C:\Users\Howard\AppData\Local\Temp\Rtmpeq7jnP\devtools3538ea06bf\Microsoft-LightGBM-43d5037\R-package\src\build\ZERO_CHECK.vcxproj]
"C:\Users\Howard\AppData\Local\Temp\Rtmpeq7jnP\devtools3538ea06bf\Microsoft-LightGBM-43d5037\R-package\src\build_lightgbm.vcxproj" (default target) (1) ->
C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\Common7\IDE\VC\VCTargets\Microsoft.CppBuild.targets(395,5): warning MSB8029: The Intermediate directory or Output directory cannot reside under the Temporary directory as it could lead to issues with incremental build. [C:\Users\Howard\AppData\Local\Temp\Rtmpeq7jnP\devtools3538ea06bf\Microsoft-LightGBM-43d5037\R-package\src\build_lightgbm.vcxproj]
2 Warning(s)
0 Error(s)

Time Elapsed 00:00:41.75
Found library file: C:/Users/Howard/AppData/Local/Temp/Rtmpeq7jnP/devtools3538ea06bf/Microsoft-LightGBM-43d5037/R-package/src/Release/lib_lightgbm.dll to move to C:/Users/Howard/Documents/R/win-library/3.3/lightgbm/libs/x64** R
** data
** demo
** tests
** preparing package for lazy loading
** help
*** installing help indices
** building package indices
** testing if installed package can be loaded
*** arch - i386
Error: package or namespace load failed for 'lightgbm' in inDL(x, as.logical(local), as.logical(now), ...):
unable to load shared object 'C:/Users/Howard/Documents/R/win-library/3.3/lightgbm/libs/i386/lib_lightgbm.dll':
LoadLibrary failure:  %1 is not a valid Win32 application.
Error: loading failed
Execution halted
*** arch - x64
ERROR: loading failed for 'i386'

removing 'C:/Users/Howard/Documents/R/win-library/3.3/lightgbm'
Installation failed: Command failed (1)
Warning message:
GitHub repo contains submodules, may not function as expected!`

Refer to this issue: https://github.com/Microsoft/LightGBM/issues/675#issuecomment-313593182
you only can use 64 bit R and R tools.

had the same problem. Updating cmake to the latest version solves my problem


z shortcut adds duplicate entries to browser history

With the current implementation of #24227 under testing, each press of Z adds an entry to the browser history.
This can be identified since:

the back/forward list in the browser (eg. press on back/forward) shows multiple copies of the near narrow
to return to the previous narrow one needs to use back multiple times
pressing back, then forward, results in no change to the narrow
pressing Z can cause a slight but discernible update in the UI (varying with the deployed version)

Other than unnecessary actions in the browser and giving the impression of updating the view, this causes confusion on use of back/forward navigation, when most other narrowing operations add one entry to the list - including, for example, multiple presses of A to show the all-messages narrow.

In particular, Z should not add an entry to the browser history when it doesn't change the message view.

Created a PR.  Please take a look!

Created a PR please take a look.
Thanks!


Drop temporary carry, and use pure upstream methods

This was introduced in c55234cab294bc4e7041ed9072fd76e45d4d3c60. Now that we're on k8s 1.30 it should not be needed anymore.
Followup to #100
/assign @ibihim

/hold
for testing

Testing in https://github.com/openshift/kubernetes/pull/2023

/hold cancel
this is good to go


Gh 1234

Fix docker images

Codecov Report

Merging #1279 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1279   +/-   ##
=======================================
  Coverage   95.39%   95.39%
=======================================
  Files           2        2
  Lines         152      152
=======================================
  Hits          145      145
  Misses          7        7

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5038783...56038b2. Read the comment docs.


History details page is returning 502 error

Expected behaviour: I was expected to see the commit changes
Actual behaviour: Getting a 502 bad Gateway error
Example: http://flexget.com/_commit/5ed54d4a2ff02f71bbc4c770c921a19a8eb27f15/home
Steps to reproduce:

Step 1: Go the history page from flexget.com
Step 2: Click on "View" on any line

This is a bug in my partials update for reams https://github.com/scragg0x/realms-wiki/pull/170 which I have now fixed. I still need to deploy this fix to our server.

Rolled out the fix to the server, this should be working properly now.


[Mod]: Stock Alarm Clock Disabler

Mod Name
Stock Alarm Clock Disabler
Author(s)
UltraJohn
License
GNU General Public License v3.0
Homepage link
https://forum.kerbalspaceprogram.com/topic/219015-112x-stock-alarm-clock-disabler/
Download link
https://github.com/UltraJohn/StockAlarmClockDisabler/releases
Additional info
Compatible with KSP 1.12.x
Requires HarmonyKSP
Recommends: Kerbal Alarm Clock

Thanks for the submission!


Worker-side single point accessibility

This calculates accessibility on the worker for single-point requests, and returns it in the JSON block at the end of the binary travel time grid (alongside the scenario application warnings). As usual, axis order is [destinationPointSet][percentile][cutoff], and this should support up to ten destination point sets of varying dimensions. There should be 121 values in each innermost array, for each cutoff from 0 to 120. Array index 5 would be a cutoff of 5 minutes, so all opportunities reachable in less than (5 * 60) seconds. This is "less than" rather than <= for consistency with our old definition: cutoffs are applied on the worker at one-second resolution, but on the UI travel times were floored, so a travel time of 4 minutes represented all times up to but not including 5:00 minutes.
{
  accessibility: [
    [
      [6,7,8,...],
      [5,6,7,...],
      [4,5,6,...],
      [3,4,5,...],
      [2,3,4,...]
    ],
    [
      [17,19,21,...],
      [15,16,17,...],
      [13,14,15,...],
      [11,12,13,...],
      [9,10,11,...]
    ]
  ],
  scenarioApplicationWarnings: [],
  scenarioApplicationInfo: []
}

This is rough around the edges but can serve as a starting point to initiate work on the UI side. The main differences in UI-backend communication are the need to send a destinationPointSetId with single point requests (when one is selected) and the ability to use the accessibility property of the response instead of calculating the accessibility within the UI.
Due to the application of distance decay functions, we may want to make the shift to non-integer accessibility values (especially for cases where there are relatively few opportunities across the whole study area). So ideally, interpret the accessibility property as if it contained floating point numbers, even though it currently contains integers.
This also introduces a wrapper class for mapping between indexes of grids of different dimensions. This is necessary because we want to display isochrones for the full region even if an opportunity data set covers a smaller geographic area, and if a user requests a small search area we don't want to search out to the full extent of the opportunity datasets (which could be much larger).
This transform class is being applied in the same way for single point and regional tasks, so should also allow regional analyses with multiple grids of different sizes, though I haven't tested that yet.
Applying the distance decay function to every travel time at every destination at each of 120 cutoffs does slow down propagation, but optimizations of that process should not affect the external interface and can be deferred.

There is a chicken and egg problem with handling regional analyses: loadAndValidateDestinationPointSets uses getWebMercatorExtents(), but that method needs the dimensions of all the pointsets, which assumes they're already loaded. I'll need to rework this.

This is now in a usable state, but is still marked "draft" because it might need a couple of rounds of refactoring or optimization, which will be facilitated by having a UI branch that can communicate with it.

Actually @trevorgerhardt have you seen https://github.com/conveyal/analysis-ui/pull/1210 ? I believe that's what we need to do to enable worker-side accessibility calculation. At least requesting it, of course displaying it is another change.

@abyrd will this break if the UI sends a destinationPointSetIds array to older worker versions?

@abyrd will this break if the UI sends a destinationPointSetIds array to older worker versions?

The intent is that the UI can send the same structure to the backend no matter what worker is selected, and the backend will perform any renormalization (thus avoiding needing multiple ui-to-backend request versions). But there are some "work in progress" blocks of code in this PR, which I will clean up during testing against the corresponding front end PR conveyal/analysis-ui#1210 including testing that it works with both older and newer workers.
When that's confirmed and cleaned up I'll remove the draft PR status.

With v5.10.0-25-gb73b853 and the related front-end changes, the accessibility results differ depending on whether the isochrone is fetched before or after the opportunity dataset is selected (i.e., whether the accessibility result is calculated in the front-end or the worker). That's understandable given the conditional logic in the front-end to calculate accessibility when it receives a null accessibility result but could be disconcerting to users.

My understanding was that if the dataset is changed after accessibility is calculated on the worker, the results would become invalid and be grayed out until the request is sent again. If/when accessibility is being computed UI side (with an older worker, or if we choose to do so for the step decay function) then the accessibility chart could be updated, but I think this would just be perceived as a bonus (don't need to click the "compute" button to clear the grayed/invalidated results). I wouldn't expect this to confuse users.

To summarize our discussion, it appears there is an off-by-one difference in how the accessibility result array slots are interpreted.
In this example, the red curve is calculated by the front-end and the blue curve is calculated by the worker:

For consistency with the front-end (and I think the regional results), the single-point accessibility result entry with index 0 should correspond to a cutoff value of 60 seconds (i.e. the number of opportunities reachable in less than 60 seconds). The isochrone above, corresponding to the minimum value of the travel time slider (1), should correspond to all the places reachable in less than 60 seconds.

@ansoncfit if I follow the links above about comments, it just takes me to the top of the "files" page rather than a specific line of a file. Can you provide the filenames and line numbers?

@abyrd Both in TravelTimeReducer: Javadoc TODOs at lines 41-51, and comment about Grids at 239-241


KIC 3.0: Getting Started guides

Description
Write KIC Getting Started guide. The structure mirrors the Gateway guide.
Testing instructions
Preview link: ...
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

tested and validated the getting started.
getstartedkic.txt


Feat: Cloud Trial Lifecycle

Epic Link : https://mattermost.atlassian.net/browse/MM-50719

Just to note that I'm holding off on my review until the above changes are made (or not). :)

@lindalumitchell
this is up for review now 😄


Apps-chrisda to Master

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Added template property to fields created with builder

Support for formForBuilder of template attribute (#164).

TODO: add template parameter to formForBuilder docs.
http://bvaughn.github.io/angular-form-for/#/documentation/formForBuilder

LGTM. Had to merge in master b'c of dist conflicts.


Improve the VisibilityEnabler and VisibilityNotifier documentations

This closes #4803.

Thanks!

Cherry-picked for 3.2.2.