termusic: 0.7.10 -> 0.7.11

Description of changes
Change log
Most notable change is that the actual player is split of into a termusi-server that is able to run in the background and comes as its own binary and the tui/launcher termusic binary.
This split made it necessary to switch the fetcher to pull directly the tarball from github, as on crates.io those are two distinct and independent crates.
The one thing where I am not sure is, if darwin would need the openssl dep explicitly as well.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] 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.

Thanks for the review.
The requested change totally makes sense.
May I just ask what "DRY" refers to? I couldn't find the meaning in the linked issue or the PR the discussion arose from.
Sorry for the messed title, I was sure I put it right in the first place. However, now it's correct, right?

Thanks for the review and patience with my noob mistakes :pray:

Thanks for the review and patience with my noob mistakes pray

It's perfectly fine, we are here for helping newcomers :)

Sure, will do in a minute.
What I don't really get is, what caused the cargo-hash to change?

Sure, will do in a minute.
What I don't really get is, what caused the cargo-hash to change?

I don't really know. Maybe someone doing rust might help here (@figsoda maybe?).

@GrahamcOfBorg build termusic

@GrahamcOfBorg build termusic

@GrahamcOfBorg build termusic

I copied the dependency from the atuin package as I am pretty sure it builds well on Darwin.

@GrahamcOfBorg build termusic

@GrahamcOfBorg build termusic

@GrahamcOfBorg build termusic

Looks like it works now.
When everything is ok, I'll squash/merge the PR.

:tada: thanks again @drupol for your welcoming and kind support on my first contribution!
This is the sustainable way to ensure quality in OSS and the foundation for great communities.

Welcome to the NixOS community :)
Keep PRs coming!

I will, and the next will probably be linux only :grin:

Sorry for the late response. This is awesome work, @VuiMuich! Also, thank you to @ius and especially @drupol for helping with this. Thanks everyone, prost! :beers:

Sorry for the late response. This is awesome work, @VuiMuich! Also, thank you to @ius and especially @drupol for helping with this. Thanks everyone, prost! :beers:

Much appreciated, it was a great experience for a first contribution on such a huge project!
Prost 🍻


[resolves #733] Provide unit tests for PackageScanClassResolver

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/517/
Test PASSed.


Haxe + OpenFl Could not initialize SDL: No available video device.

Issue description
I am trying to do some basic development work with Haxe and OpenFl. I can get OpenFl projects and samples to compile, but I can't get them to run. I get this error:
Could not initialize SDL: No available video device.
Could not create SDL window: No available video device.
Could not create SDL renderer: Invalid window.

I asked about this on the OpenFl forums. I found I didn't quite have all the libraries installed, so I installed them. The libraries in question are libX11, libXext, libXinerama, libXi and libxrandr. Installing these libraries did not seem to help.
I then asked on the #nixos irc channel and was guided to set LD_DEBUG=libs in the environment. This was the output.
Looking at the printout, it looks like ld is unable to find the libraries I had installed, and it looked in the wrong places. I am not sure how to coerce ld into looking for the libraries in the correct locations.
Steps to reproduce
nix-env -iA nixos.haxe
haxelib install openfl
haxelib run openfl setup
haxelib run openfl create HelloTriangle
haxelib run openfl test linux

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 4.14.40, NixOS, 18.03.132336.ef74cafd3e5 (Impala)`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.0.1`
 - channels(root): `"nixos-18.03.132336.ef74cafd3e5, nixos-unstable-18.09pre140099.f53d22edd68"`
 - channels(desttinghim): `"nixpkgs-18.09pre140705.090b7cc8f1b, nixos-18.03-18.03.132405.ee80654b526"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs`

This works for me with default.nix below:
{ pkgs ? import <nixpkgs> {} }:
with pkgs;
stdenv.mkDerivation {
  name = "haxeapp";

  # export paths to be able to run tests
  LD_LIBRARY_PATH = with xorg; "${libX11}/lib:${libXext}/lib:${libXinerama}/lib:${libXi}/lib:${libXrandr}/lib:${libglvnd}/lib";

  buildInputs = [ stdenv pkgconfig libglvnd ] ++
    (with xorg; [ libX11 libXext libXinerama libXi libXrandr ]);
}

As an option you can simply run nix-shell to test it:
$ nix-shell
$ haxelib run openfl test linux

Of course, you should use wrapProgram for executables to create final nix derivations (packages).

same issue here, hoping to see a fix for this soon
Link: ApplicationMain
AL lib: (EE) ALCplaybackOSS_open: Could not open /dev/dsp: No such file or directory
Could not initialize SDL: No available video device.
Could not create SDL window: No available video device.


[docs] Describe the Curiously Recursive Inlinable Switch Pattern

Adds a section to the Standard Library Programmer's Manual for resilience, along with the first entry about CRISP.
https://twitter.com/AirspeedSwift/status/1020449828717785089
https://twitter.com/UINT_MIN/status/1020503397357760512

@swift-ci Please smoke test

@swift-ci Please smoke test and merge

@swift-ci Please smoke test

@swift-ci Please smoke test Linux


Convert usr/local/share/env to use alphanumeric wildcard importing

This allows any number of env files to be added and imported, avoiding assumptions in image heirarchy setup

will be best testing this

I missed committing the test.sh change, updated now


cli: make multicast subscriber list exportable

SubscriberData used for listing information need to have exportable data fields. When list command is executed with --output flag, unmarshalling of data can only happen for exported types.

/test

/test

Runtime Failure : https://github.com/cilium/cilium/issues/31855

Ginkgo test : https://github.com/cilium/cilium/issues/30934

/ci-clustermesh

/test

@harsimran-pabla if this is a bug in a stable version, we should also add a release note and applicable backport labels. If it's only a bug in the main branch (without ever hitting stable), then we don't need to call it out in the release notes and can set release-note/misc.


[5.9] Add singletonIf method to Container

This PR adds singletonIf method similar to bindIf.
I'm trying to overwrite singleton binded in deferred service provider of 3rd party package inside of my package, but no luck, because my package loading earlier.
Without this PR in 3rd party package it could be solved this way:
if (!$this->app->bound(ConfigRetrieverInterface::class)) {
    $this->app->singleton(ConfigRetrieverInterface::class, function () {
        return new ConfigRetriever();
    });
}

With this PR:
$this->app->singletonIf(ConfigRetrieverInterface::class, function () {
    return new ConfigRetriever();
});

Bindif takes a third argument that makes it a singleton, but I guess it makes sense to have this as well.

Yeah, I know, but since we have separate singleton method I've thought that it would be better to have singletonIf.


touchdevelop generated hexfile not importing into PXT

Repro:

Get script /xjuzjz from www.microbit.co.uk and compile to hex file
Drag and drop hex file to codethemicrobit.com

I did some investigation (David Whale was asking about this). The file is converted correctly, but the .ts file in the project is not overridden. Also, the import should not create main.blocks file, or at least take the user to the .ts file after import.

Fixed.


Safebrowsing/OCSP doesn't work with Parallels

The computer seems to be just okay, but I observe these errors in the logs.
Logs are in NextCloud

The issue is probably caused by the fact that I had an IPv6 DNS server with link-local address configured in the system.
Even when we fix this, I suggest using a fallback address.

Not problem of IPv6 DNS server with link-local address. This is problem of Parallels DNS.

Fix still needed for QA.


Devel

Baby steps towards Python 3 compatibility while retaining Python 2 compatibility. Please use this PR instead of #179 since this also fixes most exceptions for Py3 compatibility and resolves a conflict from upstream.

Too many things are broken for this to be merged into the main devel branch


Rename types to type

Coverage remained the same at 89.04% when pulling 0e103e26190d6fe4d30601b1171c929e980d57b1 on briansteffens:rename-services-to-types into 90f0559f5de8e77678f6edcbb9bb728a02e31a69 on linode:master.


Migration requirements make it hard to rollback Horizon

Changes introduced in https://github.com/stellar/go/pull/808 make it hard to rollback Horizon in case of errors. I'm wondering if we should remove checking for extra migrations as they always be backward compatible with the previous version.
CC: @ire-and-curses @tomquisel

Would you add some details about the situation that makes it hard to rollback?

Sure. I deployed Horizon 0.17.0 release candidate to horizon-testnet.stellar.org. Migrations were performed during deploy. Then I noticed #908 and decided to rollback to 0.16.0. It didn't start because it saw extra migrations (from 0.17.0). I had to manually remove a row in gorp_migrations to make it start. I couldn't just run db migrate down 1 because 0.16.0 has no knowledge about 0.17.0 migrations.

RIght. Yeah, the expectation was that you would run db migrate down  using a version of the code equal to or later to the DB. So I think if you had done this before rolling back to 0.16.0 it would have worked.
I can see how this is an inconvenience though. If you're sure the DB will always be backwards compatible, then I agree we could remove the check. I guess I'm worried that after running the older version of the code, the new columns will be in an inconsistent state if you moved back to the more recent version, unless you removed and re-applied the migration.
Alternatively, I could add a force option to allow you to run an older version of the code with a newer DB. In this case the user is taking responsibility for the cosequences.

Another option would be to add migration file to gorp_migrations table. Then if you want to migrate down using older version Horizon knows how to do it.
Let's move it to backlog for now.

It happened to me again when reverting 0.19.0rc in testnet. We should prioritize this, @ire-and-curses @tomquisel.

I support fixing this however you think is best (force option, full disable, or add migration file to that table).

I'm not 100% sure yet (let's discuss this) but it seems the best option is to save migrations SQL in gorp_migrations table. Then we can create a new command like horizon db migrate sync that will sync a DB to the latest migration in a current version (migrating down using the SQL code if necessary). This will automatically revert schema to a good one when there's a need for a rollback.

We discussed this again internally.

Now that our deploy has changed to a two cluster blue/green approach, this scenario isn't likely to be common
Also noting that there is a correct workflow (migrate down before reverting to the earlier binary), it just isn't intuitive
Solutions to store all relevant info in e.g. gorp_migrations might involve hacking a dependency, which would be a bit :hankey:

@brahman81 do you still feel this is an issue to prioritise?

Closing, feel free to re-open if you feel strongly.


Runtime error while finetuning  Sizes of tensors must match except in dimension 1. Expected size 24 but got size 23

Error
 line 1130, in _call_impl     return forward_call(*input, **kwargs)   File "user/ControlNet/cldm/cldm.py", line 41, in forward     h = torch.cat([h, hs.pop() + control.pop()], dim=1) RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 24 but got size 23 for tensor number 1 in the list.
Was finetuning with source and target images of a dimensions 640x360.
Any thoughts?

I fixed it by setting the width and height the same, so instead of 640x360 i used 512x512. Anyone know why it works? 😅

I have the same issue as yours. I tried to make inputs squared but it still exists. I finally solved it by using 512x512 resolution. I guess the one-pixel mismatch is due to different sampling in SD and ControlNet.

I fixed it by setting the width and height the same, so instead of 640x360 i used 512x512. Anyone know why it works? 😅

I think image dimensions should be the multiple of 64.


Remove Smartdown code

We've now converted the final Smartdown flow (pay-leave-for-parents) to Ruby so we can remove a lot of the Smartdown related code.
cc @dsingleton

I've found a couple of other bits of code you might want to consider removing:

https://github.com/alphagov/smart-answers/blob/master/bin/smartdown
https://github.com/alphagov/smart-answers/blob/master/app/assets/stylesheets/smart_answers.scss#L63-L119
https://github.com/alphagov/smart-answers/blob/master/lib/smart_answer/flow.rb#L42-L45 (and related code)

I assume you've intentionally not removed documentation relating to Smartdown. I think we should update the docs to make it clear that Smartdown flows are deprecated, but as long as it's on our list of things to do, it doesn't necessarily need to be part of this PR.
Otherwise this looks good to me. Good work! :smile:

Taking a look at this now.

The transition status was exclusively for Smart Answer -> Smartdown conversion, and can be removed. Up to you if it's in this PR, or as a follow up.

Other bits i've spotted:

There's some CSS that might be smartdown related, smartdown/multiple-questions CSS it would be worth investigating (and this might be better for custom to do rather than you)
There's still bin/smartdown

But broadly this looks :+1: I'd be happy merging as is and the other bits in follow up PRs.

OK, so finished looking. Still happy for this to be merged as is, but it would be good to address the bits @floehopper and I raised - in particular updating/removing documentation.
We should also update the README on the smartdown repo itself, to avoid confusion there. Let us know if you don't have access to that repo

I've found a couple of other bits of code you might want to consider removing:
https://github.com/alphagov/smart-answers/blob/master/bin/smartdown

Removed in 0cf3da159e33c091fe0eaedff9ef370f8dddecc3

I've found a couple of other bits of code you might want to consider removing:
https://github.com/alphagov/smart-answers/blob/master/lib/smart_answer/flow.rb#L42-L45 (and related code)

Removed in 793f09128b56e4c110d1de390e893b81c9f66e6c.

The transition status was exclusively for Smart Answer -> Smartdown conversion, and can be removed. Up to you if it's in this PR, or as a follow up.

Removed in 793f09128b56e4c110d1de390e893b81c9f66e6c.

The CSS you've both mentioned as being Smartdown specific actually appears to affect Ruby Smart Answers too. See the screenshots below for before/after removing this CSS:
Before

After

I think I'm going to remove the comment that talks about this being Smartdown specific instead of trying to understand which bits of this CSS I can remove.

We should also update the README on the smartdown repo itself, to avoid confusion there. Let us know if you don't have access to that repo.

@dsingleton: Are there any examples of how we've updated similar READMEs in the past? Do we have a standard way of saying a project is no longer under active development/not being used?

I've chosen not to remove the documentation as part of this PR. I've created a Trello card to capture that task.

OK, I'm going to rebase this on master and get it merged.

I'm going to close this pull request and open a new one with the rewritten commits. I wonder whether this will preserve more history than if I force push over this branch.


Parrot Bebop Drone 2

Can this library use on Parrot Bebop Drone 2? Thanks.

Is the same SDK for Bebop 1 and Bebop 2, so yes.
There are only a little changes on the MAVLINK file but is not used on this library.
Regards

How to change that?

Hi,
Most of the API is the same but the video. I couldn't find enough info to implement V2's video (see http://developer.parrot.com/blog/2016/ARSDK-3-8-release/ ). If anybody has time to implement, or even find more detailed technical info, I'd be willing to know!
Regards

Has anyone been able to implement the V2's video?


v-if is not reevaluated for nested components.

Given a setup like following, a container and an avatar components, v-if is not reevaluated after the root data is updated.
A fiddle that demonstrates the problem in 0.12.1: http://jsfiddle.net/91fwjo1j/
The console outputs, [Vue warn]: Error when evaluating expression "profile.image" despite the v-if in the template tag. The DOM updates after the warning as expected.
The same code working in 0.11.10: http://jsfiddle.net/c0xht555/1/
Thank you.

What do you mean by not re-evaluated? I can't see the difference between the fiddles except for the warning.
The warning was simply suppressed in 0.11 - I decided to make it more explicit in 0.12.

The v-if wrapper prevents "signed-in only" content from appearing.
Consider the following,
// The initial on load application state.
root.data = {
  profile: null
}

// User authenticates. UI now updates to reflect this.
root.profile = {
  username: "foobar"
  image: "http://example.com/foo.png"
}

// User signs out. UI now updates to reflect this.
// ....[Vue warn]: Error when evaluating expression "profile.image"

The warning implies that something was done incorrectly. Is the approach of replacing a logged out user's profile object with null incorrect, or is the warning misleading?
Thank you for your help! :)

This is due to some internal changes relating the order in which prop changes are synced. In practice this should not affect how your app behaves, you can safely ignore those warnings. The next release will restore the old behavior and no longer throw warnings in this case.


Benchmark failed to run

Describe the bug

ujust benchmark

Instead I got this error:
aborting: temp-path '.' must be readable and writeable
error: Recipe `benchmark` failed on line 55 with exit code 1

What did you expect to happen?
I expected the benchmark to run for 1 minute.
Output of rpm-ostree status
State: idle
AutomaticUpdates: stage; rpm-ostreed-automatic.timer: no runs since boot
Deployments:
● ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx-nvidia:latest
                   Digest: sha256:d926ea236354a21020716b1ae4461a91533f37cce4741f22a6f71663bfc35ca9
                  Version: 39.20240319.0 (2024-03-19T16:56:24Z)

  ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx-nvidia:latest
                   Digest: sha256:029a6a0e16d3becf5a2a509e829da95bc5d577e23a42c0404aca60a4bfb36b09
                  Version: 39.20240305.0 (2024-03-05T04:54:42Z)

Extra information or context
No response

Running the stress command manually works so it must be something else.

Running just benchmark works but running ujust benchmark does not

The problem seems to be that the --justfile /usr/share/ublue-os/justfile argument in the definition of ujust changes the working directory to  /usr/share/ublue-os, which does not have the permissions necessary for stress-ng. This is the expected behavior of just and it is stated here in Just Programmer's Manual

just normally executes recipes with the current directory set to the directory that contains the justfile. This can be disabled using the [no-cd] attribute.

We have two options.

Disable the change of directory using the [no-cd] attribute. This will execute ujust benchmark in the current directory.
Change the directory to /tmp within the recipe of ujust benchmark, which would execute it in a safe directory with the necessary permissions. Personally, I prefer this option.

I would be glad to submit a pull-request to fix this, if I get some input on the preferred option!
Note: setting --temp-path in stress-ng is not sufficient because stress-ng will continue to use the current directory for I/O tests.


$200 - make a youtube video about screenpipe and get > 5000 views

$200 - make a youtube video about screenpipe and get > 5000 views
here's how you can get more views usually:

show face with camera
sound is clear
screenpipe app used
minimum length 5-15 minutes
show website if more for non technical people, otherwise the repository
share on different places
have already a channel with subscribers

this is repeatable bounty, you will be rewarded through @algora-pbc when commenting your video link matching the criterias
ideas of videos:

non technical: presentation of the product, use cases, etc.
technical: explain how to build pipes, build one and show the value ...

/bounty 200

/attempt #410

/attempt #410
Sounds like fun to me! If you or Matt want to come on stream sometime and chat about how you made it happen, I did that once with the Sweep guys and it went pretty well. I'm smol on yt but I keep tryin. :)


Update Guweb version to 1.3.0

Update Lists

No more guweb API (just use gulag API only)
Profile Customisation (Banner & Background Profile)

Change a bit UI of Profile & Leaderboard Page
Redesign Settings page
Bug fixed
More cleaning code

@Gusb3ll @JKBGL @cmyui Go check and approve this please if it's working well same as main branch

Changes look nice though :)


Update blogpagestyle.css

Updated the css for blog page!
Related Issue
["None”]
Description
[Updated the CSS for the blog page and improved the readability of the CSS code]
Type of PR

Feature enhancement
Documentation update

Screenshots / videos (if applicable)
[None]
Checklist:

[ X] I have performed a self-review of my code
[ X] I have read and followed the Contribution Guidelines.
[X ] I have tested the changes thoroughly before submitting this pull request.
[X ] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X ] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

Add screenshot. What is your issue number.


Pluto crashes under heavy load

pluto ver 0.9.4
Machine - 2 CPU, 4 GB Memory
Docker container limit to 1.5 CPU and 1GB Memory
Steps to reproduce:

Using load test script (flags -n 200 -s 8) for two different rulesets simultaneously
-> Server will eventually crash with following message

/opt/PLUTO/server/routes/processFile.js:276
console.log('child process exited with code ' + code.toString());
^
TypeError: Cannot read property 'toString' of null
at ChildProcess.proc.on (/opt/PLUTO/server/routes/processFile.js:276:70)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)

Fixed on 0.9.5 (as part of the graceful server shutdown), this specific fix has be hotfixed onto 0.9.4 under branch v0.9.4-1, tagged at 0.9.4-1

Confirmed this no longer occurs in pluto ver 0.9.4-1


Error occurred on trying to return to MP menu after song

[WARNING   -- 13:38:22] Socket exception occurred! System.InvalidOperationException: Collection was modified; enumeration operation may not execute.    at System.ThrowHelper.ThrowInvalidOperationException_InvalidOperation_EnumFailedVersion()    at ServerHub.Rooms.RoomsController.GetRoomsListInBytes() in C:\Users\andru_000\Source\Repos\BeatSaberMultiplayer\ServerHub\Rooms\RoomsController.cs:line 175    at ServerHub.Data.Client.ProcessPacket(BasePacket packet) in C:\Users\andru_000\Source\Repos\BeatSaberMultiplayer\ServerHub\Data\Client.cs:line 310    at ServerHub.Data.Client.ReceiveCallback(IAsyncResult ar) in C:\Users\andru_000\Source\Repos\BeatSaberMultiplayer\ServerHub\Data\Client.cs:line 246
After I was kicked from my own room from a server hub being run locally

The same happened to me. I restarted the server and haven't seen it happen again though.


feat(ul): Add Unordered List component

A Unordered List component <Ul /> for list content using <li />.
This PR resolves #52.

Can you run yarn lint --fix to fix formatting?

Hm, for some reason it's not formatting correctly. I'm going to push a fix on the formatting.

Hm, for some reason it's not formatting correctly. I'm going to push a fix on the formatting.

Where exactly did you find formatter error?

Great stuff @marceloF5, thanks a lot for putting this together! 🎉


Stx balance does not take into account stacked stx

Describe the bug
This is shown:

But this differs from the available balance as seen in the explorer

Transaction ID
n/a
To Reproduce
Steps to reproduce the behavior:

Go to explorer
Click on sandbox
Login with stacked account
See error 97,9... stx balance

Expected behavior
I expect to see the available balance less than 1 stx.
Screenshots
See above.
Desktop (please complete the following information):
Windows 10, Chrome/Firefox
Additional context
Perhaps this can be resolved together with this issue for the Web wallet
https://github.com/blockstack/stacks-wallet-web/issues/1273

I'm closing since it appears the account balance and total balance across the two screenshots shows the same value (97.999145 STX)?


Partner - Profile page not loading after login.

Profile page not loading for a Partner

This is related to partner user data on dev. There are only 3 partner profiles on dev now

I cannot create a partner profile again for user robertmurer+partner1@gmail


Is it possible to select the output device ?

Hi,
Sorry for posting this here but I would like to know if it's possible to select the output device on which the player will play. Is there any parameter to select the device ?
Thanks !

Hey!
Not possible at the moment no, as https://github.com/YellowInnovation/node-rodio/blob/master/native/src/lib.rs#L23 shows, we pick the device's default output for now.
Of course this can be changed.
Thanks!

Thanks for your answer and your work :)


labhub.py: Add reassign message

This will generate a message that issue is already assigned to you, if both user and assignee are same.
Closes https://github.com/coala/corobo/issues/324

ack b9c4ba3

@gitmate-bot rebase

@rultor merge

@rultor merge

@meetmangukiya OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@meetmangukiya Done! FYI, the full log is here (took me 2min)


Update Composer dependencies (2021-01-22-00-10)

Visual regression test passed!
View the visual regression test report

Visual regression test passed!
View the visual regression test report


Fixed position of Navbar

Description
I've made the navbar sticky/fixed on all the pages so that the user should not have to scroll up again and again to go to different pages on the website.
Related Issues or Pull Requests
Fixes: #149
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[ ] I've edited the README.md and link to my code.

@chetan-rns @Aasthajain123 @avinashkranjan Please review my PR

Thanks @Saatvik21 LGTM!

Seems like npm run lint is failing because of missing "parser": "babel-eslint" in .eslintrc.json

Seems like npm run lint is failing because of missing "parser": "babel-eslint" in .eslintrc.json

Can you elaborate what should I do now?


[RPA/feature-501] Header

Description
Create header for RB landing page
Screenshots:

Types of changes

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

I agree to the terms of the RunBotics Contributor License Agreement.

Since we are using scss variables please use camel case in your scss properties


improved SAS expriy check logic

Packages impacted by this PR
@azure/microsoft-playwright-testing
Issues associated with this PR
Improved Sas uri expiry logic.
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Task lists not rendering as expected when converting from Djot to HTML

When making a task list in Djot and converting it via djot.js the task list items are not rendered. It turns out that for this to work an standalone HTML-document must be presented.
Steps to reproduce:
Make a task list like.

- [ ] This is not a task well done.
- [ ] - [x] This task is done now.
- [ ] ```
Try to convert via djot like

djot -f djot -t html list.dj > list-converted.html

Open the HTML-document and you will find that the task list items did not render as expected.

This was tested on: Version 0.2.0

https://djot.net/playground/?text=-+[+]+unchecked
-+[X]+checked

Yes, we just generate

which will require some CSS to render properly. At the very least, we should write the appropriate CSS, put it in the documentation, and put it on the playground's style sheet!

Pandoc does:

<input type="checkbox" disabled="" />unchecked

<input type="checkbox" disabled="" checked="" />checked

with
    ul.task-list{list-style: none;}
    ul.task-list li input[type="checkbox"] {
      width: 0.8em;
      margin: 0 0.8em 0.2em -1.6em;
      vertical-align: middle;
    }

I think my original intent was to do a pure CSS implementation, but I have no idea what I had in mind specifically.

Proof of concept for pure CSS approach:
<!DOCTYPE html>
<meta charset="UTF-8">

simple item with no p tag

two

paragraphs

some
code

OK, I've updated the playground with this CSS:
ul.task-list {
  list-style: none;
  padding-left: 0.5em;
}
ul.task-list > li {
  margin-left: 1.5em;
  padding-left: 0;
}
li.checked:before {
  content: "☒";
  margin-left: -1.5em;
  float: left;
}
li.unchecked:before {
  content: "☐";
  margin-left: -1.5em;
  float: left;
}

Nice, glad to see it works there, hopefully it can be included in the CLI version of Djot.
Sendt fra min Mac Mini via Apple Mail

Den 9. jan. 2023 kl. 18.38 skrev John MacFarlane @.***>:
OK, I've updated the playground with this CSS:
ul.task-list {
list-style: none;
padding-left: 0.5em;
}
ul.task-list > li {
margin-left: 1.5em;
padding-left: 0;
}
li.checked:before {
content: "☒";
margin-left: -1.5em;
float: left;
}
li.unchecked:before {
content: "☐";
margin-left: -1.5em;
float: left;
}
—
Reply to this email directly, view it on GitHub https://github.com/jgm/djot.js/issues/31#issuecomment-1375998633, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGGLXRT5NIMPX7LOBSJHJEDWRREIPANCNFSM6AAAAAATVMXSJ4.
You are receiving this because you authored the thread.


Gen 3 Doom Desire and Future Sight bugs

Modifying the move does not modify the side condition that actually causes the damage unfortunately, so what we were getting was a special, 140 BP Steel-type damage from Doom Desire and a 100 BP Psychic-type damage from Future Sight. This is just a quick fix for the most part, (there are probably more efficient ways of doing this) as I am not the most proficient person when it comes to Java and especially PS! Java. All I did was add and edit the side condition that causes the damage to fit what the moves did in Gen 3.

Considering that your build has failed not once, but twice, have you tested the code yet?
Also please run npm test before trying to push new commits, please. You might have to use npm install to get the modules you need

As far as I'm aware, this isn't even correct. Doom Desire isn't affected by weaknesses/resistances in early generations.


Doc: batch_flatten example added

batch_dot() seems to have an example and batch_flatten() doesn't.
Adding example to tensorflow.keras.backend.py --> batch_flatten()

Hi Vijay, this seems like a good change. Can you re-open and sign the CLA? Thanks

@lamberta Thanks, will do!


Make playing field public

Making playing boolean field public and add property observers to set
when synchronizing with a given playback state.

good call, thanks for the pr!


Include the link of the App Store documentation

In this github repository there is no link to the IDS App Store Manual and Documentation.
Please include in the Readme.md the link of the documentation.
(https://international-data-spaces-association.github.io/IDS-AppStore/)

Yes, it is linked now on the ReadMe file.
Thanks for this hint.


auto beautifying php file sometimes make the file all gone

run beautify directly is ok for php file.
but if turn 'Beautify On Save' on PHP and try to save many tims, sometimes it make the file empty.
I didn't try other types of file, so I am not sure if the same would happen to others.

but if turn 'Beautify On Save' on PHP and try to save many tims, sometimes it make the file empty.

Yeah.. I am not surprised. There is a lot of room for improvement in this respect: making sure corruptions do not occur.
Atom's Text Editors and Atom Beautify have a very special relationship and sometimes when you beautify a file quickly like this, it ends up beautifying an empty text editor. To elaborate, Atom Beautify's PHP beautification is asynchronous and so making the changes to your file after completing the beautification, it will be in a state of flux, where Atom Beautify is trying to write to the file and maybe the file is temporarily empty. If you re-run beautify when it's in this nondeterministic state it can load the file as if it were empty, beautify it (beautify the empty file), and then overwrite the file with an empty beautification result (because the input file it thought was empty).
Long story short: try not to save and/or beautify so quickly in succession. Beautify On Save is currently the worst offender, because it has to work around Atom in some cases and everything is async and without conflict resolution for speedy-cases like this.
Please let me know if any other bugs occur. Thanks.

Just wanted to say I'm also seeing this problem with scss files (csscomb) when beautify on save is being used. Thanks for the explanation, Glavin.

I've seen this happen on pretty much kind of file I use regularly (JS, LESS, and HTML). Had to disable the "beautify on save" option and switched to manual operation until a solution can be found.

Unfortunately, I do not have much time after school and work and spending time with family to debug this issue and publish a fix. Maybe over the holidays in December I can find time. If someone finds a solution and can submit a Pull Request. I will happily review and merge it. Thank you!


Link onClick handlers supported in 3.0.0?

I'm using 3.0.0, and onClick handlers on Link components don't seem to work at all. Is this expected behavior?

Nope, definitely not. I haven't heard of anything like this. Can you share an example?

Actually, when I tried to isolate a dead simple example, it worked fine. So it's got to be something I'm misunderstanding about my own application in the few cases I was trying. As long as I know it should work, then I can sort out what I'm doing wrong in these other cases. No need for you to debug it. Thanks!


Doesn't work for me.

<TextBox PlaceholderText="placeholderString" HorizontalAlignment="Center" TextAlignment="Center" Height="26" Margin="117,151,117,0" Padding="3" TextWrapping="Wrap" Name="Username"  VerticalAlignment="Top" Width="137" Grid.Column="1"/>
This produces invalid markup error.

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

ID: f014ca2e-97fa-b8cb-241f-f2e4dd182b67
Version Independent ID: ff9618c2-a0e1-1980-1e2c-c3576dd00adb
Content: TextBox.PlaceholderText Property (Windows.UI.Xaml.Controls) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.controls.textbox.yml&version=GBlive&_a=contents
Product: uwp
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

@Tobi696 the XAML you provided works for me in XAML Studio.  What error are you getting in Visual Studio?  If you're on the latest VS version and getting this error still, it'd be good to use the Report a Problem link in the dropdown next to the Quick Launch box at the top-right of the IDE:


Affix loot table entries do not check champion conditions

I was trying to get tier 3 champions to drop a random affix item. There is a condition set for the champion, but the entry does not seem to be aware of this, and added an affix drop to every entity instead. Is this a vanilla limitation?

No this is actually just me discarding the conditions too early, oops


Commented out factorygirls that don't work

The binding.pry is fixed in the other pull request I have

:+1:


General RPC

[x] Protocol (flatbuffer schema etc. for RPC)
[x] JavaScript API for sending RPCs
[x] Runtime (Go) support for responding to RPCs

@dlespiau Do you think it's worth porting e.g., std/fs to use this RPC (and thereby removing some of the specialised protocol)?


Preview not working using list mode

Which HeadlessPreview version are you using? (Please write the exact version, example: 2.0.0)
2.2.0
Which Umbraco version are you using? (Please write the exact version, example: 10.1.0)
13.4.1
Describe the bug
The preview button is not replaced on when list mode is activated
To Reproduce
Create a parent and some child nodes, activate list mode on parent.
Expected behavior
The preview button should be replaced.
Screenshots

Additional context
It looks like it works if you remove "isLoading" from this line of code:
https://github.com/jesperweber/Our.Umbraco.HeadlessPreview/blob/b89b504cdb8a99c7c3d5a9091e8eb2492d585667/src/Our.Umbraco.HeadlessPreview/App_Plugins/Our.Umbraco.HeadlessPreview/js/headlessPreview.service.js#L47

Hi @CimplexAB, thanks for reporting this, and yes I can reproduce the issue as well.
I don't think the issue is specific to list mode pages though, but if the getPreviewMode endpoint is to slow, the if statement you mentioned is evaluated before the loading is complete and the headlessPreview doesn't trigger.
I'll create a fix for it :-)

Here's a PR with a fix. You can copy the headlessPreview.service.js file to your installation to see if it works.
https://github.com/jesperweber/Our.Umbraco.HeadlessPreview/pull/5

Great :-)
A new version 2.2.1 with the fix has now been released. Once again, thanks for reporting and help with debug and test 🙏


That use does not support absolute paths should be documented or supported

@rdw20170120 questions why the use builtin does not allow absolute paths. It is also unclear to me why use does not support absolute paths. The reason (security?) absolute imports are not supported should be documented or the machanism should be modified to allow absolute paths.

Augmenting the unit tests was straightforward and added coverage of an unrelated statement (when no module specs were matched). As usual for this type of change improving the documentation is the hard part.

Absolute paths are not portable between machines. Module references should always be relative in some sense - relative imports are relative to directory of the module itself, non-relative imports are relative to the module search paths.

While @xiaq makes an important point, I would like to describe a crucial use case and practice of which he seems to be unaware.
In my projects and repositories, I solve this portability problem using absolute paths anchored to a variable, controlled by my project.  In other words, my absolute paths are dynamic.
For example, when I work on one of my projects, my initial script defines an environment variable to hold the absolute path of the project repository root directory.  This is the first step of my "project activation" script.  This would look something like (in Bash) export ProjectRoot=${PWD}.  My projects typically involve a handful of such environment variables to manage ALL the file and directory references in my scripts and applications.  So, one of my scripts might call another script as something like (in Bash) source ${ProjectRoot}/DIR/SCRIPT.bash.
If a script references another nearby script, I use a dynamic absolute path for the reference.  For example, in the calling script (and all my scripts) I define a local variable holding the absolute path of the script's directory as something like (in Bash) local MyDir=$(absolute_path_of $(dirname $0))--where absolute_path_of is a function that finds the canonical absolute filesystem reference of the argument.  I reference the called script using that local variable to anchor its absolute path as something like (in Bash) source ${MyDir}/DIR/SCRIPT.bash.
As a consequence of these best practices, my projects are completely portable.  Someone can clone the project repository to any path on any machine, and everything is guaranteed to still work.  On the rare occasion that my project needs to reference something outside the project repository, I handle that explicitly via environment variables defined in machine-specific configuration.
I avoid relative paths religiously.  The Elvish-supported syntax of use ./DIR/MODULE or use ../DIR/MODULE is almost completely useless to me.  I would never invoke a script that way from within another script.  There are several reasons.

It can be confusing to the user, since it is not always clear to the user when the path is relative to the calling script and when it is relative to the current working directory.  I avoid such confusions.  I avoid forcing the current working directory just to make the script work because I do not want to limit my scripts or my users.  Therefore, my scripts work as designed regardless of the current working directory.
When troubleshooting script execution, all my traces and error messages show the absolute paths.  The user is never forced to attempt to interpret relative paths on-the-fly.
A code base with many relative paths is very fragile, particularly in the face of refactoring.  I prefer the freedom to restructure my code base without manually editing numerous relative paths.  That is wasted labor, and prone to error.
I observe that absolute paths anchored by a variable communicates my intentions FAR better.  A relative path might get the job done, but it does not tell the reader anything about WHY the reference is relative in that particular case.  Using a variable to anchor an absolute path encourages me to communicate the intention with a meaningful variable name.  That facilitates refactoring and extension by clearly showing reasons why some things should move when others should not.  Absolute paths anchored to variables make explicit the relationships that relative paths only imply and therefore leave hidden.

So, if you decline to make use work with absolute paths, that would force me to break a fundamental practice and it would affect all of my code.  I might be able to work around it, but it would likely be an ugly hack that I would avoid.  Elvish would then be useless to me.  Of course, it is certainly your right as the author and maintainer to implement as you please.  I would respect your decision, and move on.

I do have a workspace mechanism in mind for the module system. The current module searching behavior is akin to having a single workspace for each user.

Ohhh, that could be very cool...

Absolute paths are not portable between machines.

In the general case that is true. For specific users it is false. It also begs the question why builtin:cd allows a string literal or var holding a string. While use only supports a string literal. Why should I have to write this:
# A bunch of statements that ultimately set `$project-dir` to an absolute path name.
put var cwd = $pwd
cd $project-dir
use ./elv-module
cd $pwd

Rather than this:
# A bunch of statements that ultimately set `$project-dir` to an absolute path name.
use $project-dir/elv-module

In both of the above scenarios I assume we're talking about a project that supports Elvish via its use command. Such a project will, by definition, use relative imports for everything other than those imports meant to be performed by the user of the project. Whether the user first does cd $project-dir followed by use ./something is 100% equivalent to use $project-dir/something. Except that the latter is simpler and less error prone.

Why should I have to write this....

/Face-palm on my part
One scenario is if the use'd (i.e., imported) Elvish modules want to use artifacts relative to their own location (by which I do not mean the Elvish use command). In other words, they want to use filesystem based relative paths rather than absolute paths based on a possibly un-exported $project-dir variable. How best to reconcile these scenarios is TBD.

@krader1961 said:

Such a project will, by definition, use relative imports for everything other than those imports meant to be performed by the user of the project.

Before that, I said:

I avoid relative paths religiously. The Elvish-supported syntax of use ./DIR/MODULE or use ../DIR/MODULE is almost completely useless to me. I would never invoke a script that way from within another script.

So I must wholeheartedly disagree.  Using absolute or relative paths is a design decision that should be left to the user.  I cannot imagine EVER needing to override the user's design decision by specifically writing code that forbids one or the other, with ONE exception:  a relative path should be forbidden if there exists no clear design choice regarding the anchor to which it is relative.  In that case, your code has no anchor against which to resolve a provided relative path, so it must forbid relative paths in favor of an absolute path (resolved by the caller).  In most cases, though, there is a clear "current" SOMETHING (e.g., working directory, executing script) that can serve as the obvious anchor.
Why on Earth would you forbid absolute paths?  What is the point?  Why would you care?  You need an absolute path to cache the module anyway.  How does forbidding absolute paths serve anybody?
As for the question of how a module will reference other filesystem artifacts relative to the imported module, that is trivial.  A module can get its own location via src[name], from which it can compute relative references as desired.  I already do this in my modules as a routine step.  However, I ALWAYS render the result as an absolute path.
If the imported module wants to reference filesystem artifacts relative to some other anchor (e.g., $project-dir), well then the anchor needs to be published (exported, or something similar).  That is why my activation script publishes just such an anchor variable as its first step.

@krader1961 the reason for not allowing variables is that use will be getting compile-time checks in future - for example, building a list of valid symbols in that module and statically checking access to them (access to symbols in modules are not statically checked today; they should be), and the compiler has no knowledge of the value of variables. The "project root" use case is a valid one and I plan to address it via a dedicated mechanism.
Not responding to every point in detail, but use is supposed to be an opinionated mechanism for the module system, not for arbitrary code execution concerns - use eval for the latter.

Thanks, @xiaq, for sharing your intentions.  A separate dedicated mechanism makes sense.

@krader1961 cd /tmp; use ./x only works in interactive mode and is probably a misfeature.
The module system is a bit underdocumented in general; I'll revamp that section of the doc after the work on the workspace/project root mechanism is implemented.


Expose status also as Gauge.

If degraded, log probe results as severe.
Due to the way Quarkus initialises things, the metric is only available after the first call to the status endpoint.
$ curl localhost:8080/api/policies/v1.0/status
{"engine":"RESTEASY004655: Unable to invoke request: org.apache.http.conn.HttpHostConnectException: Connect to localhost:8084 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused (Connection refused)","notifications":"Unknown error, status code 401"}

Logs then:
2020-11-11 11:12:40,595 SEVERE [StatusEndpoint] (vert.x-worker-thread-4) Status reports: engine=RESTEASY004655: Unable to invoke request: org.apache.http.conn.HttpHostConnectException: Connect to localhost:8084 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused (Connection refused); backend-db=interface javax.enterprise.context.RequestScoped; notifications=Unknown error, status code 401
Metric:
$ curl  localhost:8080/metrics/application/status_isDegraded
# HELP application_status_isDegraded Returns 0 if good, value > 0 for number of entries in the status message
# TYPE application_status_isDegraded gauge
application_status_isDegraded 3.0

Do you know if that metrics will get scraped ?

It is scraped from the prometheus we use for Grafana as far as I can tell, as this just scapes everything.
Not sure if this is the desired instance though.


Unsubscribing

https://www.rfc-editor.org/rfc/rfc6121.html#section-3.3.1

@mattn: Have you seen this PR?

Thank you


Pass metadata in completion replies for jupyterlab-lsp

Is duplicating the completion code from Ammonite here temporary, i.e. the same changes are to be merged backed into Ammonite, or permanent? If it's the latter maybe after this we can add the scalapy completion here as well.

@kiendang I'm not sure yet, but the ability to add the ScalaPy completions here would be a good argument to add it here


Fix test with symfony/config:^7.2

Following https://github.com/symfony/symfony/pull/58490#issuecomment-2453496578. Until the Symfony PR, null wasn't supported in boolean nodes and defaulted to true.

Thank you. Your change makes our test suite green again, which is great of course. But, are we sure this subtle change in behavior won't break stuff downstream?

I had a look while creating the PR, and it seems the null possibility was actually handled, even if not possible until now : https://github.com/symfony/monolog-bundle/blob/ed0e4a287282971d110fd8c99d9ac391559a43ce/DependencyInjection/MonologExtension.php#L180

Thank you @alexandre-daubois.


Gekidou cold start

Summary
Improve cold start by not importing the emojis during app initialization and also reduced the size of the emoji.json file by removing props that are not used around the code base.
NONE

When we say 'improve cold start' - how much in terms of ms did it improve ?

When we say 'improve cold start' - how much in terms of ms did it improve ?

Up to two seconds


Global variables for kusto kernel?

Is it possible to have a Global variable that can be passed to all queries in a Kusto kernel enabled notebook?
This will make it much easier for Run-all scenarios.

Not currently, but this is a great ask that we've thought a bit about. We'll need a story for this but I don't have a concrete timeline at the moment, unfortunately.


Optimize Rectangle class memory management and performance,  file: vertex_instructions.pyx (part of the issue #8664)

This commit overhauls the Rectangle class by implementing direct memory management in C. Changes include:

Direct allocation and deallocation of vertex memory using malloc and free, respectively, to minimize Python's overhead.
Update vertices directly using pointer arithmetic, enhancing the update operations efficiency.
Integrated these changes into the property setters for  and , ensuring immediate updates on vertex data without intermediary Python object manipulation.

These improvements lead to a more efficient use of memory and a performance increase of approximately 5% in rendering operations, which is particularly beneficial for applications requiring dynamic graphic updates.

Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

I teste results runining 100 instances of 10000 rectangues creations several times, and it gets an average of 5% improvements.
before:

after:

Hi @tuliogv,
From a first glance seems something is missing there.
Is some commit missing, or you're still working on it? (If it's the latter one, please mark it as WIP via [WIP] in PR name)

Thank you for checking on the pull request. I am indeed still working on it, specifically on refactoring the classes. My intention is not to rush through this process as others might also be working on related issues. To coordinate our efforts effectively, I've suggested that we mark the classes each one of us is refactoring. This way, we can test each one thoroughly. Once all the classes are refactored, we could proceed with a general merge. However, if there are no other contributors currently working on this, I can complete the refactoring of all the classes and update the PR accordingly. Please let me know how you would like to proceed!
Best regards,
Tulio

As per today, no contributors are working on vertex instructions refactoring.
So, feel free to make all the appropriate changes (but remind to add [WIP] in the PR title, and remove it when is ready for a review)

I'm still working on the changes and i'm also trying to test every thing properly, this is going to take a while...


Example incorrect

Hello,
The example is the same as "CopyRows" method.
It would be interesting if you give some more information, for example... is this the correct method if I want to transfer data between 2 table extensions of the same table? if so, an example will be great, I haven't seen anything about transfer data between table extensions.
Best regards!!!

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

ID: bb97af30-8a43-648c-75fe-b62c86f425e1
Version Independent ID: bb97af30-8a43-648c-75fe-b62c86f425e1
Content: DataTransfer.CopyFields() Method - Business Central
Content Source: dev-itpro/developer/methods-auto/datatransfer/datatransfer-copyfields-method.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for the feedback. We had the wrong code example in the CopyFields method. You can read more about datatransfer in this  article: https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-data-transfer. The article will be updated with the next doc refresh.


Updated Upstream (Paper)

Upstream has released updates that appear to apply and compile correctly
Paper Changes:
976c6d425 [CI-SKIP] [Auto] Rebuild Patches
51deec726 Drop unneeded portal patch (#5566)
07a18c457 [Auto] Updated Upstream (CraftBukkit)
6733d7875 [Auto] Updated Upstream (Bukkit)

Thanks, but we don't need a PR every time something happens upstream. We got it covered ^_^

Keep in mind you don't need to refork Purpur everytime you want to send in a PR. The best way to PR something in the future is by creating a new branch and adding your changes there.


Move drag area to MainPage

Resolved / Related Issues
Items resolved / related issues by this PR.

On main + Windows 10 I cannot drag the window by the titlebar, this PR fixes it.
Closes #5476

Details of Changes
Add details of changes here.

Moved the drag area to MainPage

Validation
How did you test these changes?

[x] Built and ran the app

@gave92 I'm getting an issue when building the app in debug mode.

Mmm can't reproduce that.. @winston-de could you check if it's the same for you?

@yaichenbaum could you check if the latest commit fixes the issue?

@yaichenbaum could you check if the latest commit fixes the issue?

It's working fine now, lgtm!


Use pytest-httpbin

Currently the tester needs to manually set up httpbin before running the tests.
pytest-httpbin does that automatically.

Thanks for the suggestion. I did look into pytest-httpbin while working on #169, but for reproducibility across a local machine, CI builds, and other environments, it's hard to beat containers. I would really prefer to stick with Docker for any services we need to test against. For example, pytest-httpbin uses a different web server than httpbin.org or the httpbin container, so it could potentially have different behavior locally vs. in a GitHub Actions runner.
Httpbin is already included in docker-compose.yml, and running those containers is required for everything else under tests/integration. If you want to just run unit tests (which don't hit any services), you can run pytest tests/unit. There's more info in the contributing guide.
Are you in an environment where it's difficult or impractical to run containers?

I am building and testing requests-cache in a network-isolated-VM https://build.opensuse.org/package/show/home:jayvdb:branches:devel:languages:python/python-requests-cache
I would prefer to be running as many tests as possible to ensure compatibility with the other dependencies, which are constantly being modified triggering new builds each time.
I can install httpbin onto the worker and run it, so I'll do that if using pytest-httpbin is desirable.

Gotcha, that makes sense. Since there might be others out there with a similar setup, we could probably find a compromise that would make that easier without making pytest-httpbin the default.
Maybe something like adding a wrapper fixture that autouses the pytest_httpbin.httpbin_both fixture if:

httpbin isn't already running on localhost:80 and
pytest-httpbin is installed

I'm open to other suggestions, though.

That sounds great.  Ideally all of https://repology.org/project/python:requests-cache/versions are in the same boat, but many are likely happy to just run unit tests.
IMO the pytest-httpbin support would be considered experimental, i.e. those of us that are using it are expected to keep it working.
I would be happy to have an explicit envvar _USE_PYTEST_HTTPBIN to trigger using pytest-httpbin.  That avoids the need to check localhost:80.

Sounds good. There's also an HTTPBIN_URL envvar you could set, if that would work:
https://github.com/reclosedev/requests-cache/blob/f6a8f325b6cb1b4c4fab77c3860df2842ec5aaa2/tests/conftest.py#L26-L29

Is this something you'd like to submit a PR for? Otherwise, I can probably get around to this within the next week or two.
Also, could you point me to some docs on how packaging python libraries as RPMs (and other system packages) works? I'd like to get a better idea of what happens in between publishing a release on PyPi and getting that published to a package repo for openSUSE (or another distro).

I'd like to, but doubt I will have time.
https://build.opensuse.org/request/show/884522 is my request to upgrade this package in openSUSE.
I first needed to get the url-normalize dependency accepted https://build.opensuse.org/request/show/884161
That will be the first hurdle for distros, and you can see that there isnt many distros across it yet at https://repology.org/project/python:url-normalize/versions . It doesnt provide tests in https://files.pythonhosted.org/packages/ec/ea/780a38c99fef750897158c0afb83b979def3b379aaac28b31538d24c4e8f/url-normalize-1.4.3.tar.gz , so packagers usually look to the git repo for a versioned tarball with tests, but then https://github.com/niksite/url-normalize doesnt have a setup.py , so they need to use poetry in their build scripts, and most distros havent done that yet, c.f. https://repology.org/project/python:poetry/versions / https://repology.org/project/poetry/versions / https://repology.org/project/poetry-core/versions .

Okay, that's good to know. Sorry for the hassle with adding the extra dependency!
Besides making tests easier to run, is there anything else than can be done on this end to make the RPM packaging process easier?
Also, just a heads up, I plan to get a small patch release out sometime this week (0.6.1). If I have time, I'll try to get some changes for pytest-httpbin included in that.

requests-cache/__init__.py shouldnt have a hashbang.
I am also concerns that the tests mention redis and mongo and other deps, and those tests pass when I havent included those services.  It would be helpful to be able to test against a real redis, as all distros have that packaged, and the mock redis responses could have been from a different version to the one currently available.  mongo isnt in openSUSE Tumbleweed IIRC, so that doesnt interest me.
Adding new deps is good.  But the main problem is url-normalize doesnt include tests in their sdist.  That complicates setting up the dependency tree for distros.

requests-cache/__init__.py shouldnt have a hashbang.

Good catch, I'll remove that.

I am also concerns that the tests mention redis and mongo and other deps, and those tests pass when I havent included those services.

I haven't yet gotten around to refactoring those tests, but it is on my todo list (#212). One small improvement would be to explicitly mark them with pytest.mark.skip if the optional dependencies aren't installed, so it would at least be reflected in the pytest output. Or do you think it would be better to just let them fail instead?

It would be helpful to be able to test against a real redis

Sure, I think all that would take is adding another environment variable to set the Redis host, if something other than redis://localhost (and other connection args, if needed)

the mock redis responses could have been from a different version to the one currently available.

There aren't any mock responses for the Redis tests currently. Tests are run against this image locally, and against this image via this GitHub Action for CI jobs. We could test against multiple Redis versions (currently just 6.x) in GitHub Actions, if you think that would be valuable.

But the main problem is url-normalize doesnt include tests in their sdist

The issue there may be that poetry doesn't seem to let you include tests without also installing them as a package; see https://github.com/python-poetry/poetry/issues/2513 . I can't think of any good solutions for that off the top of my head.

If the missing deps are python, tests should fail if they are missing, otherwise packagers will often see the build was green and not look at the logs.  Non-Python and especially non-free optional deps should skip if missing, but failing is also ok.  packagers/testers can always use pytest args to skip tests they dont want to run.
The default redis port is able to be used, so using redis://localhost is not a problem.
@dephell is a better tool for creating the sdist, but it does have a few bugs that occasionally make it incompatible with poetry - the maintainer is quite responsive.

tests should fail if they are missing, otherwise packagers will often see the build was green and not look at the logs.

That makes sense. I'll update that when I get around to #212.

The default redis port is able to be used, so using redis://localhost is not a problem.

In that case, you should be able to pip install redis and then run pytest tests/integration/test_redis.py in that environment without any other changes needed.

Done that - the redis tests pass.
These are the ones which were running previously and concerned me as no redis was installed.  How could they pass, unless they were mocked..?
tests/unit/test_thread_safety.py::test_caching_with_threads[redis-..] PASSED [ ..%]

Hmm, tests fail for me both locally and in GitHub actions (logs here) if Redis isn't available.

Oh, I see, that's in tests/unit/test_thread_safety.py. You're right, those (and all the other tests under tests/unit) aren't testing actual backend connections.
The tests under tests/integration are run against real databases.

...But taking a closer look at that particular test module (test_thread_safety.py) did make me realize that those tests are incorrect and aren't actually testing what they're supposed to. I'll get that fixed soon.

@jayvdb Changes for pytest-httpbin are in the master branch now, as well as a 'fail fast' check for DynamoDb, MongoDb, and Redis tests. Give that another try for me when you get the chance.

USE_PYTEST_HTTPBIN=true pythonX.Y -m pytest -k 'not (dynamodb or gridfs or mongodb)' WFM.
Thank you!

Great! These changes are now in requests-cache 0.6.1.

Looks like I had made a mistake in MANIFEST.in, and tests didn't correctly get included in the sdist this time. I pushed out 0.6.2 to fix that.


fix bug

mask = tf.cast(mask, tf.float32) should add before maxpooling if using for GPU training in tensorflow-gpu version 1.11, otherwise there will be some warning occur.

Thank you for your bug report @bei-startdt. Actually, the variable mask is created by tf.random_uniform() in the function _bernoulli(), and the type of return value is defined by dtype=tf.float32.
I tested the code in tensorflow-gpu 1.11.0, it seems that the code is working fine. Could you give more information about your code & output message.

It's my fault. Close please.


0043 - Multiply Strings

Description of the Problem
Given two non-negative integers num1 and num2 represented as strings, return the product of num1 and num2, also represented as a string.
Example 1:
Input: num1 = "2", num2 = "3"
Output: "6"
Example 2:
Input: num1 = "123", num2 = "456"
Output: "56088"
Note:
The length of both num1 and num2 is < 110.
Both num1 and num2 contain only digits 0-9.
Both num1 and num2 do not contain any leading zero, except the number 0 itself.
You must not use any built-in BigInteger library or convert the inputs to integer directly.
Code
class Solution(object):
    def multiply(self, num1, num2):
        """
        :type num1: str
        :type num2: str
        :rtype: str
        """

Link To The LeetCode Problem
LeetCode

I would like to be assigned to this

I would Like to do this task.


A way to change config location.

Is your feature request related to a problem? Please describe.
I can't find a way to change where wakatime stores its configuration files.
By default it stores everything in $HOME. I want to change its location, but can't find a way to do it.
Describe the solution you'd like to see
An environment variable would be very useful.
Describe alternatives you've considered
None so far, and symlinks is not a solution.
Additional context
OS: linux
Intergrations used: vim

You can use $WAKATIME_HOME to customize the config file location.


Mark eligible recipes as primary

Summary
will be useful for distinguishing on catalog pages and cli commands
Test Plan
n/a
Checklist

[ ]  No images hosted from dockerhub
[ ]  Are dashboards present to understand the health of the application.  There must be at least 1 of these

[ ]  all databases should have dashboards
[ ]  ideally also have at least cpu/mem utilization dashboards for webserver tier of the app
[ ]  you can use plural from-grafana to convert a grafana dashboard found via google to our CRD

[ ]  Are scaling runbooks present

[ ]  all databases must have scaling runbooks
[ ]  you can use the charts in pluralsh/module-library to accelerate this

[ ]  do you need to add config overlays?

[ ]  inputing secrets
[ ]  configuring autoscaling

[ ]  If there’s a web-facing component to the app, we need to support OIDC authentication and setting up private networks if no authentication option is viable
[ ]  All major clouds must be supported

[ ]  Azure
[ ]  AWS
[ ]  GCP

The two line strings were the results of python's yaml dump so I'd assume its properly formatted, I'll fix the redis stuff


Workbook.SaveCopyAs Method (Excel) returns Run-time error '1004'

It is Office 2016 Mac OS edition issue.
Private Sub Workbook_BeforeSave(ByVal SaveAsUI As Boolean, Cancel As Boolean)
    Dim FilenameStr As String
    FilenameStr = Sheet1.Range("F11").Value
    FilenameStr = ActiveWorkbook.Path & "/" & FilenameStr & "_" & Sheet1.Range("I5").Value

Or this as an alternative for the previous line
    FilenameStr = FilenameStr & "_" & Sheet1.Range("I5").Value

At this point I get the FilenameStr like this
/Users/myusername/somedir/123456789_20130531-002
    ActiveWorkbook.SaveCopyAs (FilenameStr & ".xls")
    Cancel = True
End Sub
```
This is the error I'm getting
**Run-time error '1004': Cannot access read-only document 123456789_20130531-002.xls**

The file doesn't exist and has to be created, but everythig worked back in Office 2011.

Thanks for your comment. Your feedback is appreciated. This GitHub forum is specifically for suggested changes and additions to the VBA documentation. We recently updated the SaveAsUI topic, so check to see if this change addresses your issue. Also try setting Cancel = False. For other code issues, we suggest that you post your question to one of these forums:
https://social.msdn.microsoft.com/Forums/office/en-US/home?forum=isvvba
http://stackoverflow.com/questions/tagged/excel-vba


Why Node and TsLint path on server side

Hello,
I've tested this plugin for sonar and it seems quite interesting.
I've followed instructions on this page : https://github.com/Pablissimo/SonarTsPlugin
and I don't understand why node should be install on server and why tslint path must be fill as the rules are analyzed locally.
Can you explain me please ?

Node doesn't have to be installed on the server, just has to be on whatever is doing the analysis (which is itself sometimes the same box as the machine hosting SonarQube, depending on your setup).
Being able to set the config path at a server level lets you (in an enterprise setting) use a single reference analysis rule set for multiple projects, from a network share or some source controlled repository for example.

Ok thank you for the details according to my english level it was not clear on the documentation (sorry about that !).
Ok so if the ts path is not referenced at server side, tslint is searched locally inside the npm registry ?

Sorry, just deleted a comment about the tslint path because it was misleading - there are also some situations where you may want to ensure the same version of tslint is used for analysing all projects, in some enterprise settings.
You don't have to do this, you just can. Typically the path to tslint is auto-detected by the plugin in the node_modules folder, but you can override this in your sonar-project.properties file if it can't find it. Run the scanner with the -X flag and it'll tell you where it's searching for things and, if it goes wrong, give hints as to why.

OK thanks, i can close this issue !


test client-hc routing without internet requirement

For #1986
@easbar Do you know how we can make the parametrized testing working with the dropwizard extension? Maybe some junit5 trick :) ?
One important difference to note is that for graphhopper.com we have to stay backward compatible and so it is not required for the client to always include the vehicle (or profile).

Cool! Thanks :)

Do you know how we can make the parametrized testing working with the dropwizard extension? Maybe some junit5 trick :)

You mean parameterized at the class level? No, unfortunately not. It looks like this is still an open issue: https://github.com/junit-team/junit5/issues/878. Doing it at the test level is easy though, like I did here for the 'export' test: #2000. And if you put the parameter array in a class-wide constant it will only take one more line per test:
private static final String[] PARAMS = {"true,0","true,1000","false,-1"};
// ...
@ParameterizedTest // instead of @Test
@CsvSource(PARAMS)
public void myTest(boolean usePost, int maxUnzippedLength) { // have to add the parameters here

I would argue that this might be a bit more code, but at the same time it makes the tests more decoupled so we can adjust the paramaters easily for each test if needed. For example the gpx export test does not make use of max unzipped length (that's why I think we can move it to GraphHopperIT like I did in #2000).
Btw in some cases I would have liked to do the server configuration on the test level as well, I am still looking for a way to do this without creating a new test class.

You mean we cannot rely on the vehicle/profile always being set and instead should keep some tests where this is not the case?

I meant for 2 profiles (like here) the routing engine fails if no vehicle is specified which is not the case for graphhopper.com where we always assume car as default.

You mean parameterized at the class level? No, unfortunately not. It looks like this is still an open issue: junit-team/junit5#878.

Thanks, will try that!

Unfortunately we have to add more boilerplate as CsvSource requires a "String constant" and so PARAMS (String array that could be changed) is not working. Also the GraphHopperWeb instance has to be created in every test:
@ParameterizedTest(name = DISPLAY_NAME)
@CsvSource({"true,0", "true,1000", "false,-1"})
public void myTest(boolean usePost, int maxUnzippedLength) {
  GraphHopperWeb gh = new GraphHopperWeb(TestUtils.clientUrl(app, "/route")).
   setPostRequest(usePost).setMaxUnzippedLength(maxUnzippedLength);

Unfortunately we have to add more boilerplate as CsvSource requires a "String constant" and so PARAMS (String array that could be changed) is not working.

Oh you are right. We could still do @CsvSource(value ={VAL1, VAL2, VAL3}). Or we could use @EnumSource, which is actually even better:
    private enum TestParam {
        GET(true, 12),
        POST_MAX_UNZIPPED_0(false, 5),
        POST_MAX_UNZIPPED_1000(false, 5);

        public boolean usePost;
        public int maxUnzippedLength;
        TestParam(boolean usePost, int maxUnzippedLength) {
            this.usePost = usePost;
            this.maxUnzippedLength = maxUnzippedLength;
        }
    }
    @ParameterizedTest
    @EnumSource(value = TestParam.class)

Also the GraphHopperWeb instance has to be created in every test:

We can probably make this a class field and put the creation into a @BeforeEach (or @BeforeAll method?

Should profile=xy already work for POST requests?

Yes

And should we include the profile name in the exception?

Yes I only left this out because I was concerned about a multi-server setup, but now we should include this.

And should we include the profile name in the exception?

The error message above is from profile resolver, which lists the requested and available profile properties (since no profile was specified). I think in all these error messages we should recommend the user to use the profile parameter directly and list the available profiles.

btw: it seems this has improved the build times from ~4.5 to ~4 minutes. But unfortunately the memory issues can now be seen also on master: https://travis-ci.org/github/graphhopper/graphhopper/jobs/677183186

But unfortunately the memory issues can now be seen also on master

I also ran into this but not consistently. But increasing the limit by some 50mb should also be no problem?

But increasing the limit by some 50mb should also be no problem?

I do not like it much as in the past it uncovered some memory related problems (usually in GraphHopper itself ;)). Still as even increasing to 110MB helped I'll do it and report this problem to dropwizard.

Ok I already set to 150mb on master, but of course feel free to set it 110mb.

I changed to @EnumSource it seems nice to me. Still calling createGH for each test, so we can pass it the test parameters (but I think thats fine).
Open points (to be fixed somewhere else)

[ ] memory limit for tests increased
[ ] list available profiles when profile not found
[ ] (maybe) remove GET /route from client-hc entirely

I think we can merge here.

Thanks! Will merge.

(maybe) remove GET /route from client-hc entirely

The problem is then the GPX export again as we removed support for POST :/


Unable to set the value of the slope angle to more than 1 in the style editor

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
When creating a parallelogram/trapezoid, it is unable to set the value of the slope angle to more than 1 in the style editor.
To Reproduce
Steps to reproduce the behavior:

Create a parallelogram.
Click on the created parallelogram shape to open the style editor
Open the property list and find the "Slope Angle" item, change its value to 30
The value is not changed to 30 but clipped to 1.

Expected behavior
The value of the "Slope Angle" is changed to 30
Screenshots
https://github.com/jgraph/drawio/assets/38486514/6045de73-df29-4bcf-8e32-bc5af37d0366
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version: 21.3.2

Desktop

OS: Ubuntu 22.04
Browser: Firefox 114.0.1

Additional context
Current workaround: right click on the shape, click "Edit Style", and modify the "size" value.
Possible cause:
https://github.com/jgraph/drawio/blob/9d6532de36496e77d872d91b1947bb696607d623/src/main/webapp/js/diagramly/Editor.js#L4334-L4337

Still broken, 6 months later...

Related: the property editor in the user interface shows the default Slope Angle as 0.2, but the actual default Slope Angle appears to be 20.


Add Some Basic Tests

Fixes #140.

Code looks good! Should the base branch be main now?

We haven't moved adabot yet. Working on it. This PR is headed for the right destination.


Column A not read in IE8 for a remote file

Hi,
When I test the ajax example in IE8, the column A is not read. I did some tests on my side with two different files, and I have the same behavior.
Any idea why?
Thanks

Not sure why GH didn't automatically close the issue, oh well


Release v17.2.4

A little confused as to what it happening here :( Help!!

just re-releasing what @patflynn  already released + the extra bug fix commits. there are no files changed because the version I'm releasing matches exactly what he releaed

yeah it's a bit weird. I think we'll need to come up with a better strategy so that we don't have to retag things.


Select Random

Hi,
There is a function to select data like orderByRand in php ?

@oscariusbow would a shuffle method do it for you?
Model.all().shuffle()

?


Integrating soft delete with Azure Search soft delete

It would be useful to implement the soft delete of a blob in such a way that a separate property configuration is not required for tracking deletions on the datasource when an indexer is run in Azure Search.

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

ID: 78964956-a479-5fbe-710d-19e361542a36
Version Independent ID: 15e70532-6d39-47ed-6932-ef3eb88e33f0
Content: Soft delete for Azure Storage blobs (Preview)
Content Source: articles/storage/blobs/storage-blob-soft-delete.md
Service: storage
GitHub Login: @MichaelHauss
Microsoft Alias: mihauss

Can you elaborate on what you mean? Are you suggesting you would like soft delete to be on by default (no configuration required)? Not sure I quite understand the ask.

Right now, in Azure Search, even with soft deletes turned on on the blob, I still have to add a property to the MetaData of IsDeleted on the blob so that an indexer will remove that blob from the index on delete. It would be useful that if soft delete were turned on for the blob, that it would automatically be removed from the index when the indexer is run OR at least have a metadata property that represents the status of "deleted" that can be chosen as the Soft delete column on the datasource in Azure Search

When a blob is soft deleted, it is unreadable and not listed through existing APIs. Effectively, it is deleted.  The same behavior should occur in Azure Search independent of soft delete configuration. In fact, when soft delete is on, the soft deleted blob will have a number of new properties returned in List Blobs "For version 2017-07-29 and above, Deleted, DeletedTime and RemainingRetentionDays appear when this operation includes the include={deleted} parameter. These elements do not appear if this blob was not deleted. These elements appear for blob or snapshot that are deleted with DELETE operation when soft delete feature was enabled. Deleted element is set to true for blobs and snapshots that are soft deleted. Deleted-Time corresponds to time when the blob was deleted. RemainingRetentionDays indicates number of days after which soft deleted blob will be permanently deleted by blob service."

Upon further inspection, it would appear that a soft deleted blob is still recognized by the indexer as being hard deleted because it cannot access the metadata properties once the status is "deleted", so it doesn't really matter what properties there are, the two cannot be linked. It would be nice to have a way to somehow link the soft delete of a blob to it's datasource delete policy, so that it could actually be implemented without have to run a clean up job of blobs that have been removed from the index via the IsDeleted = true property.  In short, have a way to both soft delete a blob and tell the indexer to remove it from the index.

Okay, thanks.


Show group metadata within chat

Metadata events are generated when users join/leave the chat. These should be displayed.
Screenshots
quirk.x joins the chat

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Registering the device after inactive state is failing for android OS 4.4.2

I connected a device after that I performed some operations,
then after switching the network connection(WiFi) off and connecting back to the same network , the device is not getting connected to the server. ( Still in in active state).
I tried this with Android tab of OS 4.4.2

@Nirothipan Did you try syncing the device from the agent app?

This has occurred due to the change in IP address of the server as the WiFi connection got interrupted in between.


Markdown list format does not work in Desktop and Web application

Reply in a topic with text which should be formatted as list, but after message is posted no format applied:

@MironovDmitry I think the actual story here is that Zulip's markdown does not render mixed - and + prefixes as a markdown list.  It sounds like the be an issue where the library used for local echo doesn't agree with the authoritative backend markdown processor on this.

I think this classifies as a bug. Every implementation here supports mixed lists including python-markdown.

Hello.
what is strange, after clicking on Send button, the list renders correctly
with bullets, but then (in about a second or two) formatting disappeared.
On Tue, 3 Dec 2019 at 17:58, Rohitt Vashishtha<EMAIL_ADDRESS>wrote:

I think this classifies as a bug. Every implementation here
https://babelmark.github.io/?text=%2B+one
-+two supports mixed lists
including python-markdown.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/13447?email_source=notifications&email_token=ABWLXJSN2LPSVG6N7AJB4CTQWZXZRA5CNFSM4JR4N2VKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFZU7RY#issuecomment-561205191,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWLXJXLJ7FC7XF2K4QGEMLQWZXZRANCNFSM4JR4N2VA
.

That is what Tim meant in his comment. In our setup, we have a frontend library to render each message you send that immediately shows you what the message will look like - the local echo; and we also have a backend library that is the authoritative rendering that everyone sees. We replace the local echo that only you see with the final backend render when it is available.
Here, the local echo version is rendering the list as expected but the backend version isn't, which is why you see the list 'disappear' after we get back the rendered message from the server.

@timabbott should we close this?

Yes, this was resolved by #13482.


[v1.5.0-beta.0] Add option to check updates on the dev branch

The title says it all, what do you think guys?

Yes, please!

Done


How check end of path

I'm developing a race car sims, i need to check end of path (race start line), it's possibile with path creator object or i need a collider object to solve the problem?

Scanning through the source code, there is no events that you can handle when the path following has ended, reversed or looped.
However, if you just want to know if the follower has reached the end of the path, then you can check if the follower's transform position is equal to the path's last point.
Here is an example that will scale the follower by 10 (x, y, z) when it reaches the end of the path:
void Update()
{
    if (pathCreator != null)
    {
        distanceTravelled += speed * Time.deltaTime;

        transform.position = pathCreator.path.GetPointAtDistance(distanceTravelled, endOfPathInstruction);
        transform.rotation = pathCreator.path.GetRotationAtDistance(distanceTravelled, endOfPathInstruction);

        if (transform.position == pathCreator.path.GetPoint(pathCreator.path.NumPoints - 1))
        {
            // Put whatever code you want to run here when the follower reaches the end of the path
            transform.localScale = new Vector3(10, 10, 10);
        }
    }
}

To further this, you could create your own path follower script like the examples that only always uses the stop end of path instruction and fire a unity event to say it has reached the end of the path. Then you can add "actions" that should happen at the end of the path via the inspector as you may have done with other scripts/components etc.
using UnityEngine;
using UnityEngine.Events;

namespace PathCreation.Examples
{
    // Moves along a path at constant speed, will stop at the end and fire unity event.
    public class ToEndPathFollower : MonoBehaviour
    {
        public PathCreator pathCreator;
        public float speed = 5;
        public UnityEvent reachedEndOfPath;

        float distanceTravelled;

        void Start() {
            if (pathCreator != null)
            {
                // Subscribed to the pathUpdated event so that we're notified if the path changes during the game
                pathCreator.pathUpdated += OnPathChanged;
            }
        }

        void Update()
        {
            if (pathCreator != null)
            {
                distanceTravelled += speed * Time.deltaTime;

                transform.position = pathCreator.path.GetPointAtDistance(distanceTravelled, EndOfPathInstruction.Stop);
                transform.rotation = pathCreator.path.GetRotationAtDistance(distanceTravelled, EndOfPathInstruction.Stop);

                if (transform.position == pathCreator.path.GetPoint(pathCreator.path.NumPoints - 1))
                {
                    reachedEndOfPath.Invoke();
                }
            }
        }

        // If the path changes during the game, update the distance travelled so that the follower's position on the new path
        // is as close as possible to its position on the old path
        void OnPathChanged() {
            distanceTravelled = pathCreator.path.GetClosestDistanceAlongPath(transform.position);
        }
    }
}

thanks, ShanYu for your solution to get the end position of the pathCreator. I was trying to do a reverse travel from endpoint to start point for a liquid simulation using this PathCreation package from the unity store. But I couldn't get the endpoint position
using getPoint(1) from the documentation.  Luckily your pathCreator.path.NumPoints - 1 helps to solve my problem.
using UnityEngine;
using PathCreation;
public class NegativeFlow : MonoBehaviour
{
public PathCreator pathCreator;
public float speed = 5f;
public float trailTime = 5f;
public float distanceTravelled;
TrailRenderer trail;

[HideInInspector]
public bool isFlowing = false;
[HideInInspector]
public bool pathIsEnd = false;

public EndOfPathInstruction endPath;

private void Start()
{
    trail = GetComponent<TrailRenderer>();
    endPath = EndOfPathInstruction.Loop;
}

private void Update()
{
    if (isFlowing)
    {
        transform.position = pathCreator.path.GetPoint(pathCreator.path.NumPoints - 1);
        distanceTravelled -= speed * Time.deltaTime;
        trail.time = trailTime;
        transform.position = pathCreator.path.GetPointAtDistance(distanceTravelled, endPath);
        transform.rotation = pathCreator.path.GetRotationAtDistance(distanceTravelled, endPath);

        if (distanceTravelled <= pathCreator.path.length)
        {
            pathIsEnd = true;

        }

        if (CompareDistance(transform.position, pathCreator.path.GetPoint(0)))
        {
            endPath = EndOfPathInstruction.Stop;

        }
    }
    else
    {
        trail.time = 0;
        distanceTravelled = 0;
        transform.position = pathCreator.path.GetPoint(pathCreator.path.NumPoints - 1);
        endPath = EndOfPathInstruction.Loop;

        pathIsEnd = false;
    }

}

public bool CompareDistance(Vector3 liquid, Vector3 endPoint)
{
    return Vector3.SqrMagnitude(liquid - endPoint) < 0.0001f;
}

}

``

public float distanceTravelled;
TrailRenderer trail;
[HideInInspector]
public bool isFlowing = false;
[HideInInspector]
public bool pathIsEnd = false;
public EndOfPathInstruction endPath;
private void Start()
{
trail = GetComponent();
endPath = EndOfPathInstruction.Loop;

I see you refrenced to a Trail Renderer. I added it as a component, but now nothing moves anymore.
Do I really need this component to reverse the path after it ends?


ko apply always stop at some place and not continue

here is the output:
jw@cci-network-0003:~/git/src/github.com/julz/freeze-proxy$ ko apply --watch -f config/
2021/08/19 17:22:04 Using base gcr.io/distroless/static:latest for github.com/julz/freeze-proxy/cmd/sleeptalker
2021/08/19 17:22:04 Using base gcr.io/distroless/static:latest for github.com/julz/freeze-proxy/cmd/daemon
2021/08/19 17:22:04 Using base gcr.io/distroless/static:latest for github.com/julz/freeze-proxy/cmd/webhook
2021/08/19 17:22:04 Using base gcr.io/distroless/static:latest for github.com/julz/freeze-proxy/cmd/freezeproxy
2021/08/19 17:22:19 Building github.com/julz/freeze-proxy/cmd/webhook for linux/amd64
2021/08/19 17:22:19 Building github.com/julz/freeze-proxy/cmd/daemon for linux/amd64
2021/08/19 17:22:19 Building github.com/julz/freeze-proxy/cmd/sleeptalker for linux/amd64
2021/08/19 17:22:19 Building github.com/julz/freeze-proxy/cmd/freezeproxy for linux/amd64
2021/08/19 17:22:21 Loading ko.local/sleeptalker-b12b6347badbe02cd11cbb86d888f4d9:012e09d20b35bc3b7ad130296723f3631490a2555ad3d2dc225c95021d4e20fa
2021/08/19 17:22:21 Loading ko.local/freezeproxy-574709e712d1653288bca81e43a5807f:574a0cfae4dcc6a9e6a05539f2f704dfbde5c6e803a96f8a460d04d2ea8d7e8b
2021/08/19 17:22:23 Loading ko.local/daemon-fb121567bd6b51a004586ed25ffc0671:22b02b21c6ce815509d795062d36f29ebc8c3be0ed8a8c9ccd955083a34b95cf
2021/08/19 17:22:24 Loading ko.local/webhook-66fcf13d03b6d9aa3f54b8878581cf63:6fba08efc182c6ce1284fc86e683c32190f03877f5b73e17b4b56415f7e633b7
2021/08/19 17:22:33 Loaded ko.local/sleeptalker-b12b6347badbe02cd11cbb86d888f4d9:012e09d20b35bc3b7ad130296723f3631490a2555ad3d2dc225c95021d4e20fa
2021/08/19 17:22:33 Adding tag latest
2021/08/19 17:22:33 Added tag latest
2021/08/19 17:23:05 Loaded ko.local/freezeproxy-574709e712d1653288bca81e43a5807f:574a0cfae4dcc6a9e6a05539f2f704dfbde5c6e803a96f8a460d04d2ea8d7e8b
2021/08/19 17:23:05 Adding tag latest
2021/08/19 17:23:05 Added tag latest
2021/08/19 17:23:09 Loaded ko.local/daemon-fb121567bd6b51a004586ed25ffc0671:22b02b21c6ce815509d795062d36f29ebc8c3be0ed8a8c9ccd955083a34b95cf
2021/08/19 17:23:09 Adding tag latest
2021/08/19 17:23:09 Added tag latest
2021/08/19 17:23:09 Loaded ko.local/webhook-66fcf13d03b6d9aa3f54b8878581cf63:6fba08efc182c6ce1284fc86e683c32190f03877f5b73e17b4b56415f7e633b7
2021/08/19 17:23:09 Adding tag latest
2021/08/19 17:23:09 Added tag latest

I meet so many times about this situation, it just stopped, or how to open logs.
the log I see every img is ready.
Thanks a lot!

With --watch, ko doesn't exit, and watches for new file changes to rebuild/publish/update new images. Does it do what you expect if you remove --watch?

Yeah, it's this one. Thanks a lot


Permalink doesn't work

Installed today version 2.3.1 and copying a message's URL doesn't work. When pasting the URL e.g. into the browser I have still the text previously stored into the clipboard. It works on the Linux client.

@taitruong   Wrong project.  Please report issue to Cordova project.


Allow different triggers

Bootstrap's tooltip allows for different triggers for displaying the tooltips: http://getbootstrap.com/javascript/#tooltips-options
I think it can be very useful, especially when dealing with mobile devices. In fact, I need that feature for the project I'm working on now.
I can create a pull request if you think it will be good.
Thanks!

Would be a nice addition indeed! Feel free to file a PR. I guess we'll include the events click hover focus.

Sure. I'll see if 'manual' would be also easy enough to integrate.


Fix crashes for empty distro and fixes some react issue in component

React component returned list of elements instead of one element, as it should return only one.
Distro had unresolved exception when distro was not found in the system.

You're mentioning this is fixing some crashes but ... what are they?

You're mentioning this is fixing some crashes but ... what are they?

  <PageDetail label={t('Distribution')}>{distribution.name}</PageDetail>
  <PageDetail label={t('Distribution')}>{distribution?.name}</PageDetail>
  <PageDetail label={t('Repository URL')}>
    <CopyCell text={distribution.client_url} />
    <CopyCell text={distribution?.client_url} />

Already resolved in https://github.com/ansible/ansible-ui/pull/1453.


bigquery-test-fix - removed preview row

closing this since already stale for some time, please raise another one if still needed