[Feat]MOUSE CURSOR TRAIL EFFECT

Description
This  effect will bring :
1.Enhanced User Engagement: The effect can increase user engagement and interaction with the website, making the user experience more enjoyable and memorable.
2.Visual Interest: The trail of colors or shapes can add visual interest to the website, making it more appealing and attention-grabbing.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hello @Priyanshi0112, this is a very good feature, but we are converting the entire code into the Next.js framework for easier backend integration. Once the Next.js conversion is complete, I will let you know so you can work on that issue.

Please don't forget to give our repository a star and let me know once you've done it.


User signed out after login when IS is in different domain than JS client.

Hello,
We have implemented single sign on/out using your library on an Angular 6 application. Everything is working fine when both client JS and IS are running on the same domain (e.g. localhost). However, if they are running on different domains ( client js on localhost and IS on machine name), user manager throws addUserSignedOut event after signing callback redirect.
Some light?
Our Auth service implementation
`export class AuthService {
private manager: UserManager = new UserManager(this.getClientSettings());
private user: User = null;
constructor(
    @Inject('ROOT_URL') private rootUrl: string,
    @Inject('IDENTITY_URL') private identityUrl: string) {
    this.manager.getUser().then(user => {
        this.user = user;
    });
    this.subscribeEvents();
}

isLoggedIn(): boolean {
    return this.user != null && !this.user.expired;
}

getClaims(): any {
    return this.user.profile;
}

public subscribeEvents(): void {
    this.manager.events.addSilentRenewError(() => {
        console.log("error SilentRenew");
    });

    this.manager.events.addAccessTokenExpiring(() => {
        console.log("access token expiring");
    });

    this.manager.events.addAccessTokenExpired(() => {
        console.log("access token expired");
        this.logout();

    });

    this.manager.events.addUserSignedOut(() => {
        console.log("user signed out");
        location.reload();
    });
}

public refreshCallBack(): void {
    this.manager.signinSilentCallback()
        .then(() => console.log("Token has been refreshed") )
        .catch(() => console.log("Error refreshing the token") );
}

getAuthorizationHeaderValue(): string {
    return `${this.user.token_type} ${this.user.access_token}`;
}

startAuthentication(url: string): Promise<void> {
    return this.manager.signinRedirect({ state: url });
}

completeAuthentication(): Promise<string> {
    return this.manager.signinRedirectCallback().then(user => {
        this.user = user;
        return user.state;
    });
}

logout() {
    this.manager.removeUser().then(() => this.user = null);
}

startSignOut(): Promise<void> {
    return this.manager.signoutRedirect();
}

public completeSignOut(url?: string): Promise<any> {

    return this.manager.signoutRedirectCallback(url)
        .then(user => {
            window.localStorage.clear();
            this.logout();
            // This will clear the OpenID access tokens that is stored in the session
            return this.manager.clearStaleState();
        });
}

getClientSettings(): UserManagerSettings {
    return {
        authority: this.identityUrl,
        client_id: 'Next',
        redirect_uri: this.rootUrl + '/auth-callback',
        post_logout_redirect_uri: this.rootUrl + '/signout-callback',
        response_type: "id_token token",
        scope: "openid profile",
        filterProtocolClaims: true,
        loadUserInfo: true,
        automaticSilentRenew: true, // Activate automatic token refreshing so above flow will be performed.
        silent_redirect_uri: this.rootUrl +'/silent-refresh'
    };
}

}`

Ok, after some investigations I noticed that Identity Server was not keeping the cookie when the client called from other domain. So basically, it (IS) was issuing valid token after login, the client stored it and used it in the silent request call, however IdentityServer rejected the request since that Identity(token) didn't exist for it.
The following solution solved my issue:
IdentityServer/IdentityServer4#1983
Thanks.

I'm facing the same issue, can you elaborate a little bit more on the solution

Hi Leobel,
I followed this to solve it:
"Okay, so the root of my issue is that I was not using the
IdentityServer4.AspNetIdentity library for Identity integration. That
library adds this line of code now
https://github.com/IdentityServer/IdentityServer4.AspNetIdentity/blob/dev/src/IdentityServer4.AspNetIdentity/IdentityServerBuilderExtensions.cs#L48
to
turn off the SameSite for Aspnet Identity."
Can you try it?
El mié., 17 abr. 2019 15:06, Leobel Izquierdo<EMAIL_ADDRESS>escribió:

Ok, after some investigations I noticed that Identity Server was not
keeping the cookie when the client called from other domain. So basically,
it (IS) was issuing valid token after login, the client stored it and used
it in the silent request call, however IdentityServer rejected the request
since that Identity(token) didn't exist for it.
The following solution solved my issue:
IdentityServer/IdentityServer4#1983
https://github.com/IdentityServer/IdentityServer4/issues/1983
Thanks.
I'm facing the same issue, can you elaborate a little bit more on the
solution
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/IdentityModel/oidc-client-js/issues/729#issuecomment-484076241,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVxv_iZMnIEN4Z1a1h3vQZoXa7AvsZnJks5vhxxNgaJpZM4Z9XnO
.

Hi Victor,
Yes I finally solve the issue by locking into the code of AddAspNetIdentity and notice exactly that this is the way to get the SameSite be affected when use AspNetIdentity:
builder.Services.ConfigureApplicationCookie(options =>             {                 ...                 // we need to disable to allow iframe for authorize requests                 options.Cookie.SameSite = SameSiteMode.None;             });

Keep in mind that some browsers started to reject insecure SameSite=None cookies, i.e Chrome version 85


Error establishing a database connection. When using Wordpress 6.2

Hi there and thanks for the amazing work you've done.
When using this in my .env:
#Site image
SITE_IMAGE=wordpress
SITE_VERSION=6.2
The WP cli doesn't work.
I'm don't know how to troubleshoot further.  I ran into the problem when trying to use the update site name script provided in the wordpress folder.
I may have tracked it down to the Wordpress version.  It works when I use 4.6 for the site version.
Any pointers on how to troubleshoot would be awesome!
Thanks again.
edit:
The output i get when trying to run wp.sh is:  (I replaced my domain with "site" in the following)
...:/home/root/server/sites/dev2.site.org/compose# ./wp.sh
/snap/docker/2746/lib/python3.6/site-packages/paramiko/transport.py:32: CryptographyDeprecationWarning: Python 3.6 is no longer supported by the Python core team. Therefore, support for it is deprecated in cryptography. The next release of cryptography (40.0) will be the last to support Python 3.6.
from cryptography.hazmat.backends import default_backend
Creating site-m0dff_wpcli_run ... done
Error: Error establishing a database connection.
ERROR: 1

I have found that it has something to do with the different syntax of the wp-config file.
If I replace the following lines with hard coded definitions instead of getenv_docker() methods, it works.
`define( 'DB_NAME', getenv_docker('WORDPRESS_DB_NAME', 'wordpress') );
/** Database username */
define( 'DB_USER', getenv_docker('WORDPRESS_DB_USER', 'example username') );
/** Database password */
define( 'DB_PASSWORD', getenv_docker('WORDPRESS_DB_PASSWORD', 'example password') );
/**

Docker image fallback values above are sourced from the official WordPress installation wizard:
https://github.com/WordPress/WordPress/blob/f9cc35ebad82753e9c86de322ea5c76a9001c7e2/wp-admin/setup-config.php#L216-L230
(However, using "example username" and "example password" in your database is strongly discouraged.  Please use strong, random credentials!)
*/

/** Database hostname */
define( 'DB_HOST', getenv_docker('WORDPRESS_DB_HOST', 'mysql') );
/** Database charset to use in creating database tables. */
define( 'DB_CHARSET', getenv_docker('WORDPRESS_DB_CHARSET', 'utf8') );
/** The database collate type. Don't change this if in doubt. */
define( 'DB_COLLATE', getenv_docker('WORDPRESS_DB_COLLATE', '') );`

As mentioned, I'm pretty new to all this. So I don't know how to implement a fix for this yet, but I did find people discussing what looks like the problem.
https://github.com/docker-library/docs/pull/1911

I chased this down to an issue with the tatemz/wp-cli image used in the https://github.com/evertramos/docker-wordpress repo.
I solved my issue by changing to the official wordpress:cli image and adding env variables to it.
See:
https://github.com/evertramos/docker-wordpress/pull/67


telegram 0.9.28

Thank you for the contribution. It was merged directly as da3977d8430170b59a5cd93a9ea95774b0351ee5 to keep commit history cleaner. Your contribution is still credited to you.


move away from using podman in operatork-sdk bundle validate cmd

Currently for ValidateOperatorBundle preflight is passing in podman as an argument, even though the bundle has already been pulled/extracted/etc. Since the bundle files are already on disk, we can use none as an argument.

/assign


Compile problem

Hi everyone, does anyone encounter this problem when compiling the code?
TypeError: forward() missing 1 required positional argument: 'dilation'
Thanks

Thank you for reporting this, we've identified the issue and will be fixing it.

Thanks, I will try it again.


kbfs service: Do not assume fuse is installed

Motivation for this change
Currently the kbfs service does not work unless fuse is installed as a system package.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Hmm, we've actually hardcoded fusermount in kbfs

https://github.com/NixOS/nixpkgs/blob/9995881b791e101e08012d1a180420c9cf0cd344/pkgs/tools/security/keybase/fix-paths-kbfs.patch#L10

But I think this might have had unintended side-effects.
I believe this needs to be suid, which it won't when it's in the /nix/store.
That's why the path here is at /run/wrappers.

Also, it's perfectly fine to assume this in NixOS because the following configuration will be applied on every system using NixOS

https://github.com/NixOS/nixpkgs/blob/c81e4f10c2fb8d63a0694a5c5661ed9ade050990/nixos/modules/security/wrappers/default.nix#L159

@worldofpeace thanks for the response and explanation.
It seems something is broken in my current setup. I'll close this PR for now while I investigate it.


added favicon and title changes

Coverage decreased (-0.02%) to 55.348% when pulling 2dc7a25366b00ea17d01b7d1bc91ad1f9d5c3ef9 on jordanrinke:new_ui into 98dc3aefa1120fbe004a4e3e42fb0de4eaf87df0 on supergiant:new_ui.


publish an npm package

hi , the plain-draggable is a very useful project for our work.
but there is a problem that we expect to install your code with npm.
so is there any plan to publish your work to npm?

i'm sorry that i haven't searh the right key word before . already found it . please close this issuse if you see it.

Hi @MAGICSCIENTIST, thank you for your comment.
So, could you close this issue if it was solved?

No reply came, then I close this abandoned issue.


expose forgejo

Closes #268
This is dirty, but modules from nixpkgs can be exported
If this is accepted, #288 can be modified to:
- modules.services.ntpd-rs = null;
+ modules.services.ntpd-rs = "${lib.modulePath}/services/networking/ntp/ntpd-rs.nix";

Upstream NixOS tests can also be re-exposed like this, lmk if this wanted or not

I've done a build of the overview site for this and it looks good. I agree that making the path to the nixpkgs module explicit is much better than using null, and that the same should be done for the other PRs for exposing nixpkgs projects on the overview (#288 and #289).

Upstream NixOS tests can also be re-exposed like this, lmk if this wanted or not

That's an interesting idea, though I'm trying to think of what advantages there would be from doing it. Even if these exposed tests did run on our CI for ngipkgs, any changes to the projects would make more sense to happen via PRs to nixpkgs which would then run on their CI first. Our overview site currently only lists packages, options, and examples, and I'm not sure how useful it would be to add tests here.

Also, if we are going to use the module paths from the nixpkgs input on all the nixpkgs exposure PRs, then this PR #305 should be merged first so that the inherit inputs; gets into the flake.nix first.


Updated readme

As discussed with @bob-davis

As discussed with @Bob-Davis

Ack - Publisher will be Red Hat, "for VS Code" will be in the name.


Privacy progress tile is incorrect

The user's private balance is higher than his total:

There's also an additional segment of the privacy bar that shouldn't be there at 100% privacy on both Linux and Windows 10:

Both problems were solved by restarting the client, so it's not an emergency.

Rebooting fixes it.

Similar issue here

Fixed itself after Wasabi restart.

@soosr this is a really common issue, any ideas? Some testing was done on this PR: https://github.com/WalletWasabi/WalletWasabi/pull/13186

@soosr this is a really common issue, any ideas?  So when you restart Wasabi, which one gets corrected?

The balance or the private amount is the wrong? So when you restart Wasabi, which one gets corrected?

The private amount is wrong. When you restart Wasabi, the private amount is lowered and matches the total balance.

Is there only 1 wallet logged in when the issue occurs?

Is there only 1 wallet logged in when the issue occurs?

I've experienced it with only 1 wallet and also with multiple wallets logged in.


problem with ubuntuterminate called after throwing an instance of 'std::runtime_error' thread failed with: thread failed with: small_delta

[P2] Table 5 scan took 41.0579 sec
[P2] Table 5 rewrite took 71.2529 sec, dropped 762436231 entries (17.7586 %)
[P2] Table 4 scan took 40.5733 sec
[P2] Table 4 rewrite took 70.9661 sec, dropped 829230521 entries (19.3107 %)
[P2] Table 3 scan took 52.752 sec
[P2] Table 3 rewrite took 70.1277 sec, dropped 855351726 entries (19.9169 %)
[P2] Table 2 scan took 52.6918 sec
[P2] Table 2 rewrite took 70.1359 sec, dropped 865697488 entries (20.157 %)
Phase 2 took 935.941 sec
Wrote plot header with 268 bytes
[P3-1] Table 2 took 187.732 sec, wrote<PHONE_NUMBER> right entries
terminate called after throwing an instance of 'std::runtime_error'
what():  thread failed with: thread failed with: small_delta >= 256<PHONE_NUMBER>5)
Aborted (core dumped)
anyone have the same problem? solution? thanks to all

edit new problem
Phase 1 took 2360.49 sec
[P2] max_table_size =<PHONE_NUMBER>
[P2] Table 7 scan took 18.5047 sec
[P2] Table 7 rewrite took 208.169 sec, dropped 0 entries (0 %)
[P2] Table 6 scan took 41.1214 sec
chia_plot: /home/marbvelous/chia-plotter/include/chia/bitfield_index.hpp:45: std::pair<long unsigned int, long unsigned int> bitfield_index::lookup(uint64_t, uint64_t) const: Assertion `bitfield_.get(pos) && bitfield_.get(pos + offset)' failed.
Aborted (core dumped)

I woke up today to notice the plotter had stopped at night and had the same error for thread failed.
First time seeing this and I've been using the plotter without issues for days.

Increase ram voltage to 1.45v and try again.

My RAM is rated at 1.35v

Mine also, but I set it to 1.45V.

That helped for this issue?
I prefer staying in the rated voltage as the warranty probably won't cover broken ram if it's been used at higher voltage it was meant for, no matter if the voltage was the reason for it breaking.
I rebooted my machine and the issue seems to have been temporary for me as I'm plotting successfully again.

yes that's a RAM issue

Mine also, but I set it to 1.45V.

Hello, I've been trying to solve this issue for days now.
I have two of those HX436C18FB3AK2/64 on my AsRock x570 Phantom Gaming 4 and 5900x, XMP gives me 3000 and 3600 profiles.
3600 works only in dual channel as it restarts and sets ram back to 2400mhz when I try to boot it.
But 3000Mhz sets to 1.35v and throw this and few other errors at me.
With 2400mhz there is no error since ram works on 1.2v, but as I have bought 3600Mhz ram, I would like it to work at least on 3000Mhz.
From what I've found online, I think it's not safe for me to increment voltage by myself (tried to the 1.365v but with no success) because probably some other parametres have to be changed along with voltage, and I am not an expert in that field.
Can you suggest to me what can I possibly do to speed up that clock since XMP is not working?

Mine also, but I set it to 1.45V.

Hello, I've been trying to solve this issue for days now.
I have two of those HX436C18FB3AK2/64 on my AsRock x570 Phantom Gaming 4 and 5900x, XMP gives me 3000 and 3600 profiles.
3600 works only in dual channel as it restarts and sets ram back to 2400mhz when I try to boot it.
But 3000Mhz sets to 1.35v and throw this and few other errors at me.
With 2400mhz there is no error since ram works on 1.2v, but as I have bought 3600Mhz ram, I would like it to work at least on 3000Mhz.
From what I've found online, I think it's not safe for me to increment voltage by myself (tried to the 1.365v but with no success) because probably some other parametres have to be changed along with voltage, and I am not an expert in that field.
Can you suggest to me what can I possibly do to speed up that clock since XMP is not working?

Sorry I am no expert overclocker, just set 1.45V to my memory.
I have Corsair Vengeance LPX 2x8GB 3200Mhz

yes that's a RAM issue

Any idea what is it related to? I have 4x32GB Corsair Vengeance LPX 3200MHz CL16 sticks with 16-20-20-38 latency.
I'm plotting with default 256 buckets. Do you think it might mean my system ran out of ram? Maybe I'll try with 512 buckets.

512 buckets did not help. I am now constantly getting this error... Can make one or two plots and it dies to this same error:
terminate called after throwing an instance of 'std::runtime_error'   what():  thread failed with: input not sorted Aborted (core dumped)

512 buckets did not help. I am now constantly getting this error... Can make one or two plots and it dies to this same error:
terminate called after throwing an instance of 'std::runtime_error' what(): thread failed with: input not sorted Aborted (core dumped)

I haven't been able to find voltage that doesn't crash plotter or the system if I increment it over 1.38v (since voltage of RAM is the only thing I was changing, didn't touch CPU voltage or latencies).
You have to run it in default (underclocked) mode or to find some settings for ram out of xmp.
I've found some, and now I have crashed on like every 15th plot, which is fine because I've got about 10% increased speed.

Increase ram voltage to 1.45v and try again.

How does one go about increasing the ram voltage in Ubuntu?

Increase ram voltage to 1.45v and try again.

How does one go about increasing the ram voltage in Ubuntu?

You increase it from bios, not from the OS
This is a good way to start: https://www.google.com/search?q=how+to+increase+ram+voltage


Tracker_Session & Tracker_Cookies record entries, but doesn't log user_id

After upgrading from 5.8 to 6.1 I had a bunch of errors with the Tracker.
Most of it some integrity constraints, but currently it doesn't log the user_id even if signed in.
Any idea how to fix this?

Tested it on production, and for some reason it records the client ip as local host (<IP_ADDRESS>).

I have:

Added protected $connection = 'mysql' in the User model. and referenced it in the Config.
Added Tracker::class to the $middlewareGroups for web and api.

I'm noticing that it stores the tracker_sessions and the tracker_cookies.
However even though the cookie_id fk in tracker_sessions links up with the id in tracker_cookies, it doesn't post the user_id in the tracker_sessions table.
I use web auth but I also have laravel passport installed but that shouldn't be an issue either. Anyone have any ideas?

You may return an example ID in the following function by
return 12345; // your example ID
https://github.com/antonioribeiro/tracker/blob/master/src/Services/Authentication.php
public function getCurrentUserId()
{
if ($this->check()) {
return $this->user()->{$this->config->get('authenticated_user_id_column')};
}
}
In my installation the Auth::check(); is not yet ready at the time the method is called. I don't know how the tracker is started.


Increase progress manually

I see that we can show progress bar using pjax:start and pjax:end,
but is possible to increase progress manually?
I know that I need to send from server this info, but how to catch this with pjax?
Thanks

I see this post:
http://www.dave-bond.com/blog/2010/01/JQuery-ajax-progress-HMTL5/
but not sure how to implement with pjax

@kamov try this http://github.hubspot.com/pace/docs/welcome/

Thanks, but this is not really file progress, but any ajax progress. I have already integrated this, but when file is uploaded, this is not show, until the end of file uploaded.

There isn't any way to show incremental page loading progress using pjax.


refactor: Refactoring config configuration

Description:
refactor:

Rewrite core package config already compilationResolveConfig logic
breakChange: add configResolved hook
Rewrite preview command

optimize:

Optimize the size of core and cli packages
Make full use of the common method of core package share to remove duplicate files and functions
Optimize server logic to make devserver logic more general

feat:

Add build cli options targetEnv e.g: node & browser

BREAKING CHANGE:

Related issue (if exists):

Develop specifications for config and configResolved return values

Develop specifications for config and configResolved return values
FarmJsPlugins

config hook

configResolved hook

VitePlugins

config hook

configResolved hook


Haskell builds do not always build/source test dependencies

Describe the bug
The Haskell build framework does not appear to appropriately source Haskell packages in the nix-shell environment when doCheck = false for the entire Haskell package set except a single packages (in this case, the package being developed.
See this repository for a reproduction; specific areas of interest are as follows:

the mkDerivation override disabling tests
the overlay that re-enables tests for the package-under-test

To Reproduce
Steps to reproduce the behavior:

Clone this repository
Invoke nix-shell
Observe the following:

hspec is neither built by Nix, nor included in the packages displayed by the shellHook call to ghc-pkg list
calling cabal test from within the shell will attempt to download dependencies from Hackage

Additionally, one can apply the following patch and observe the following:

hspec is built by Nix however it is still not displayed by the shellHook call to ghc-pkg list
calling cabal test still attempts to download the package from Hackage, even though we observed that it was compiled by Nix

patch

diff --git a/shell.nix b/shell.nix
index 31d485e..eaf8bf4 100644
--- a/shell.nix
+++ b/shell.nix
@@ -1,11 +1,10 @@
 let
-  inherit (import ./default.nix {}) haskellPkgs;
+  inherit (import ./default.nix {}) haskellPkgs pkgs;
 in

-haskellPkgs.shellFor {
-  packages = p: [ p.repro ];
-  nativeBuildInputs = [ haskellPkgs.cabal-install ];
-  shellHook = ''
-    ghc-pkg list
-  '';
+pkgs.mkShell {
+  buildInputs = [
+    (haskellPkgs.ghcWithPackages (p: [p.repro]))
+    pkgs.cabal-install
+  ];
 }

Expected behavior
Nix's Haskell build framework should support the following behavior:

disabling all tests in a package set while also selectively enabling tests for a subset of packages as desired
appropriately sourcing the transitive dependency tree of all enabled components in a Haskell project within the nix-shell environment's GHC pkgdb

Notify maintainers

ping @cdepillabout, as we spoke about this on Slack.
Metadata
 - system: `"x86_64-darwin"`
 - host os: `Darwin 19.6.0, macOS 10.15.7`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.4pre20201102_550e11f`
 - channels(jkachmar): `"darwin"`
 - channels(root): `"nixpkgs-21.03pre246624.cfed29bfcb2"`
 - nixpkgs: `/nix/store/s18y38xsw017icfnskgasfxy437ms267-13jxji98gdq08mzcxsk9gvm70cd5fkzr-darwin-stable-src`

NOTE: My personal Nix configuration is fairly involved; at the time of writing I've tested this with:

this nixpkgs-20.09 checkout
this nixpkgs-unstable checkout

cc @expipiplus1, who I think touched the shellFor stuff recently and may be able to spot the bug sooner than I'll be able to take a good look at this.

I tried an earlier nixpkgs commit (c565d7cc16f5409304b5f3815115d373e3632811, which was some arbitrary 20.03 release from February) along with GHC 8.6.5 (since that's what was in-tree at the time).
ghc-pkg list looks like it still doesn't display the transitive test dependencies and cabal test still tries to download from Hackage, so maybe I'm just misunderstanding how this is all meant to work together.

cc @expipiplus1, who I think touched the shellFor stuff recently and may...

Heart racing

Basically the culprit is this line https://github.com/NixOS/nixpkgs/blob/afe87caf9aa94b0fdc61ccd2c31e201afcceb9c3/pkgs/development/haskell-modules/make-package-set.nix#L442
This uses the mkDerivation which you've overridden here: https://github.com/jkachmar/haskell-nix-repro/blob/fbae41b936f9e789bc38783e05dbbcff2dcebe93/default.nix#L27 which forces the tests to be disabled, in this case for the fake package which shellFor has constructed.

A potential fix is having something which respects flags which have been explicitly set in your mkDerivation override, for example flipping the arguments to // :
          args: hprev.mkDerivation ({
            doCheck = false;
            doBenchmark = false;
            doHoogle = false;
            doHaddock = false;
            enableLibraryProfiling = false;
            enableExecutableProfiling = false;
          } // args);

and setting doCheck = true; in the call to shellFor in shell.nix. This is pretty nasty though because the hlib.doCheck is still being ignored. Not to mention that this doesn't stop anyone else from bumping into this.
I think it's quite safe to always pass doCheck to mkDerivation in make-package-set.nix as test dependencies won't make it into cabalDepsForSelected if that package has had tests disabled.
I've not tested this or anything so it could all be wrong!

Looking at similar code for doBenchmark (added by @cdepillabout) doBenchmark has to be specifically requested by the user.
Why isn't doBenchmark always turned on like I suggest doCheck is above, would it make any difference to always enable it?

I sent a PR with an example of getting this to work: https://github.com/jkachmar/haskell-nix-repro/pull/1.  I left some commentary there.

@expipiplus1 Thanks for the suggestion with the order of overriding args.  That turned out to make the difference.  I agree this is pretty tricky.

Why isn't doBenchmark always turned on like I suggest doCheck is above, would it make any difference to always enable it?

This is a good question.
I think it depends on how we want shellFor to be used.  If we want shellFor to be able to explicitly ignore deps for benchmarks and tests, then we really need shellFor to have the option to disable/enable doBenchmark and doCheck.
If we want shellFor to be dump and just take all the transitive deps of its packages, then we could just always set doCheck and doBenchmark to true.  But in this case, the user would always have to do something like this mkDerivation override to control  whether tests and benchmarks are enabled for all the packages in the Haskell package set.
I don't think there is a "correct" answer here, we just need to think about what would be easier for the most end-users.

But the test dependencies (same for bench depends) are included in the arguments to mkDerivation iff doCheck (or doBench) is true: https://github.com/nixos/nixpkgs/blob/4718ba1ab10d109e2383854c5c56c2c2340c00c1/pkgs/development/haskell-modules/generic-builder.nix#L538-L546
I do see the rationale for disabling this though, but I think there's no harm in enabling these both by default as they do nothing when nothing in the package set requests test or bench depends.
This of course ignores other effects of doCheck and doBench, but I don't think any of those (like enabling the checkPhase) affect the env derivation.

@expipiplus1 I'm sorry, I think we might be talking past each other.
In my previous post (https://github.com/NixOS/nixpkgs/issues/104349#issuecomment-730958996), I was trying to explain our two choices between providing arguments to shellFor to disable/enable tests and benchmarks, or just always having shellFor enable tests and benchmarks on the virtual Haskell package it creates.

I do see the rationale for disabling this though, but I think there's no harm in enabling these both by default as they do nothing when nothing in the package set requests test or bench depends.

I think you're talking here about the following:  Assuming we do provide arguments to shellFor to disable/enable tests and benchmarks (currently we only have the argument for benchmarks), do we let the default value be true or false?
I agree with you that defaulting them both to true make sense, for the reasons you give.
I defaulted doBenchmarks to false so that the hash wouldn't change for anyone currently using shellFor.  However, I don't feel strongly about this.  We could change it to true.  Ideally we would also add a Release Note about this.

I agree with that all! Sorry for the communication mishap!

I defaulted doBenchmarks to false so that the hash wouldn't change for anyone currently using shellFor

I had not considered this, makes sense!

defaulting these to true does feel slightly weird because: the action we want to tell mkDerivation isn't "enable the tests and benchmarks" but instead "make sure you include every dependency I give you". Setting doCheck and doBench to true accomplishes this, but only because of the indirect action.
Perhaps an alternative would be to, after zipping the attrs, just pile all the dependencies together in buildDepends? (Again it's not quite the right thing to do though, but maybe a little better?)

First off thanks for getting around to this so quickly, and I appreciate all of the elucidation in this thread (and elsewhere)!
Having slept on it, I think what I really want (and what might be more generally useful) is closer to what @expipiplus1 is saying (https://github.com/NixOS/nixpkgs/issues/104349#issuecomment-731154475).
Specifically, it seems like there should be some attributes for the fake package that's used to construct the shell environment which allow us to include these sorts of other dependencies irrespective to doCheck (etc.).

It seems reasonable that local developers would want Nix to manage all of their dependencies (i.e. cabal test, cabal bench, and cabal run shouldn't have to access Hackage at all).
However it also seems reasonable that the compilation/inclusion of these dependencies in the nix-shell's exposed pkgdb isn't coupled with whether or not the test/bench suite is actually run when the environment is constructed. It's often very useful to drop into a cabal repl (or even plain ghci) session to muck around with some dependencies that are used within the project, even if I don't intend to build it at all.

The counterargument to the behavior described above is that changing mkDerivation is cache-invalidating, and I imagine most people are either pulling from the Nix binary cache or some hosted build cache fed from a CI process (which can probably run test/benchmark suites without issue).
Does that make sense, and do you know if there's a way that the Haskell build framework could be updated to support this kind of exposure?

First off thanks for getting around to this so quickly, and I appreciate all of the elucidation in this thread (and elsewhere)!

Make sure to pass it on down the line :+1:

This flag passed to genericBuilder could also disable the building phases, as this derivation is only useful for an interactive shell, and the phases don't make much sense (especially with multiple packages in packages)

Whoops, sorry for not following up on this 😓 .
It sounds like the change @cdepillabout tried out in https://github.com/cdepillabout/nixpkgs/commit/47d19bef86481df4f52c2bf909c23940cf98973a addresses the core issue here, and it seems like this would be generally useful for Nix's Haskell package expressions.
If that's the case, what do you think the best path forward towards getting something like this merged into nixpkgs proper would be? Ideally, it would be nice if this could be backported to release-20.09 but I don't know anything about what that might involve (although I'd be willing to assist with this, as it's affecting a few projects at work).
If that's not the case, then is there anything that needs to be explored before something like it could be merged? I'd also be willing to help here (even if it's just messing around with a branch that has this change and trying to poke at the ergonomics of the API).

Thanks, especially for the quick response to my initial issue and for all of the discussion. Hope y'all had a happy new year!

Whoops, sorry for not following up on this 😓 .
It sounds like the change @cdepillabout tried out in https://github.com/cdepillabout/nixpkgs/commit/47d19bef86481df4f52c2bf909c23940cf98973a addresses the core issue here, and it seems like this would be generally useful for Nix's Haskell package expressions.
If that's the case, what do you think the best path forward towards getting something like this merged into nixpkgs proper would be? Ideally, it would be nice if this could be backported to release-20.09 but I don't know anything about what that might involve (although I'd be willing to assist with this, as it's affecting a few projects at work).
If that's not the case, then is there anything that needs to be explored before something like it could be merged? I'd also be willing to help here (even if it's just messing around with a branch that has this change and trying to poke at the ergonomics of the API).

Thanks, especially for the quick response to my initial issue and for all of the discussion. Hope y'all had a happy new year!

@jkachmar Do you want to try sending cdepillabout@47d19be as a PR to nixpkgs?  It would be great if you could add a little documentation about what is going on, and maybe give some motivation in a comment for why someone would want to use the additional option.
Once we merge it into the haskell-updates branch, I imagine it would be fine to also backport to 20.09 (assuming it wouldn't break anything for most people).

@jkachmar Do you want to try sending cdepillabout@47d19be as a PR to nixpkgs?  It would be great if you could add a little documentation about what is going on, and maybe give some motivation in a comment for why someone would want to use the additional option.
Once we merge it into the haskell-updates branch, I imagine it would be fine to also backport to 20.09 (assuming it wouldn't break anything for most people).


[1.40.x] Update project version to 1.40.0

Generated by build jenkins-KIE-kogito-1.40.x-release-kogito-serverless-operator-deploy-3: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/1.40.x/job/release/job/kogito-serverless-operator-deploy/3/.
Please do not merge, it shoud be merged automatically.

Openshift tests job #63 was: UNSTABLE
Possible explanation: This should be test failures
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/63/display/redirect
See console log:

Console Logs
openshift-network-diagnostics                      network-check-target-tzkft                                        1/1     Running     0             26dopenshift-network-diagnostics                      network-check-target-v26kv                                        1/1     Running     0             26dopenshift-network-operator                         network-operator-6b7968774f-mthnz                                 1/1     Running     0             26dopenshift-operator-lifecycle-manager               catalog-operator-7b5d974c6b-lznd7                                 1/1     Running     0             26dopenshift-operator-lifecycle-manager               collect-profiles-28123725-bvxlj                                   0/1     Completed   0             44mopenshift-operator-lifecycle-manager               collect-profiles-28123740-ksd5z                                   0/1     Completed   0             29mopenshift-operator-lifecycle-manager               collect-profiles-28123755-xgrkv                                   0/1     Completed   0             14mopenshift-operator-lifecycle-manager               olm-operator-f5b9db4c8-wm2w5                                      1/1     Running     1 (12d ago)   26dopenshift-operator-lifecycle-manager               package-server-manager-7f54df8479-nr24r                           1/1     Running     0             26dopenshift-operator-lifecycle-manager               packageserver-cccd79994-jzxcr                                     1/1     Running     0             26dopenshift-operator-lifecycle-manager               packageserver-cccd79994-qwcdp                                     1/1     Running     1 (12d ago)   26dopenshift-roks-metrics                             metrics-9fd6fd97f-n2d2v                                           1/1     Running     0             49dopenshift-roks-metrics                             push-gateway-7dd6b9c999-ffczz                                     1/1     Running     0             49dopenshift-service-ca-operator                      service-ca-operator-7c99c77499-8c6xd                              1/1     Running     0             26dopenshift-service-ca                               service-ca-5fd7645d7b-cg4pv                                       1/1     Running     0             26dteste                                              httpd-sample-7bdf74f5cd-b2rbf                                     1/1     Running     0             78dtigera-operator                                    tigera-operator-7cb9f67f44-7nwkl                                  1/1     Running     0             49d[Pipeline] }[Pipeline] // scriptPost stage[Pipeline] archiveArtifactsArchiving artifacts‘test/logs/**/*.log’ doesn’t match anything: ‘test’ exists but not ‘test/logs/**/*.log’No artifacts found that match the file pattern "test/logs/**/*.log". Configuration error?[Pipeline] junitRecording test resultsNone of the test reports contained any result[Checks API] No suitable checks publisher found.[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // timeout[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // withEnv[Pipeline] }Lock released on resource [Sonataflow Operator OpenShift tests https://c107-e.us-south.containers.cloud.ibm.com:30353][Pipeline] // lock[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/63/api/json[Pipeline] readJSON[Pipeline] shNo test report files were found. Configuration error?


fix(strategysheet): 修复趋势分析表多列头时叶子节点未和数值单元格对齐

👀 PR includes
🐛 Bugfix

[x] Solve the issue and close #0

📝 Description

#1364 重构后, 列头数值配置和 dataCell 解耦, 趋势分析表的虚拟数值列头命中了 textAlign: 'center', 由于 drawObjectText 这个方法使用 dataCell 的 textAlign 做的坐标计算, 一个是 right, 一个是 center 导致坐标计算错误

未兼容趋势分析表, 虚拟数值列头也应该命中 measureText 主题配置

🖼 Screenshot

Before
After

🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v1.18.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.16.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


DNS Server and port 53

Version
Microsoft Windows [Version 10.0.19044.1766]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.4.72
Distro Version
Ubuntu 2204
Other Software
Dnsmasq - configured to listen on <IP_ADDRESS> interface or <IP_ADDRESS> - default port 53
Repro Steps

Let dnsmasq listen on port 53 for dns lookups
Try in WSL 2 to make nslookup example.org <IP_ADDRESS> and see the correct result
Try in PowerShell to make nslookup example.org <IP_ADDRESS> - Server:  UnKnown

If you run a netstat -nab in PowerShell
UDP    <IP_ADDRESS>:53             *:*
SharedAccess
[svchost.exe]

It seems like a service occupying port 53 on Windows when running WSL. I have tried with a Windows restart - and the SharedAccess seems to start with the WSL start up.
Expected Behavior
You should be able to run nslookup within Windows and not only in WSL 2.
Actual Behavior
dnsmasq runs fine in WSL 2, but can not be reached within the Windows environment
Diagnostic Logs
WSL
nslookup foo.test <IP_ADDRESS>
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

Name:   foo.test
Address: <IP_ADDRESS>

PowerShell
nslookup foo.test <IP_ADDRESS>
Server:  UnKnown
Address:  <IP_ADDRESS>

*** UnKnown can't find foo.test: No response from server

Thank you for reporting this @boan-jfm.
UDP traffic can't be routed to WSL via <IP_ADDRESS> at this time, but TCP should work.
Can you try in TCP mode? What's the output of  nslookup -vc foo.test <IP_ADDRESS>

I did a netstat to see if there where some listing on TCP before testing - and it was:
 TCP    <IP_ADDRESS>:53           <IP_ADDRESS>:0              LISTENING
 TCP    [::1]:53               [::]:0                 LISTENING

nslookup -vc foo.test <IP_ADDRESS>
WSL response (immediately)
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

Name:   foo.test
Address: <IP_ADDRESS>

PowerShell response (very slow - maybe minutes)
Server:  localhost
Address:  <IP_ADDRESS>

Name:    foo.test
Address:  <IP_ADDRESS>

It seems to work with TCP lookup - but it is very slow, and you can't select whether a common client should use TCP or UDP?
I wanted to test some implementations and point my test device to the developer machine running dnsmasq in WSL, and let the "test" TLD point to the nginx server running on the same machine - but i think this requires UDP to be available?

DNS can happen over either TCP or UDP so I don't think UDP is needed here.
I'm surprised that the lookup takes that long though.
What if you try to use WSL2's ip address directly (both UDP and TCP) ? (You can get it by run ip a inside WSL2. The interface is eth0).

PowerShell
nslookup foo.test <IP_ADDRESS>
Server:  UnKnown
Address:  <IP_ADDRESS>

Name:    foo.test
Address:  <IP_ADDRESS>

nslookup -vc foo.test <IP_ADDRESS>
Server:  UnKnown
Address:  <IP_ADDRESS>

Name:    foo.test
Address:  <IP_ADDRESS>

nslookup -vc foo.test <IP_ADDRESS>
Server:  localhost
Address:  <IP_ADDRESS>

Name:    foo.test
Address:  <IP_ADDRESS>

nslookup foo.test <IP_ADDRESS>
Server:  UnKnown
Address:  <IP_ADDRESS>

*** UnKnown can't find foo.test: No response from server

Every lookup is fast now - I do not know if there is a cache - but seems to work with the WSL 2 IP address on eth0. The only lookup not working is the UDP against <IP_ADDRESS> - but I think my test devices should work when using the WSL 2 IP address. I will try that again - thank you.
Is there an explanation for why UDP is not working for <IP_ADDRESS>?

Okay - maybe I need some knowledge on how IP is mapped from Windows to WSL.

I have the pretty much the same need as @boan-jfm. I need to hit a service running in WSL from another device on my local network where that device uses a DNS name that can only be resolved by my local system. I was hoping to add my system as a secondary DNS server in my local network router configuration. The client on the device in question would only try UDP for the name resolution, so TCP is probably not an option.
If the DNS service is needed to have things working between host and guest (WSL), would it be possible to add to the configuration that the SharedAccess service module is using for resolution? As it stands, I can't run a DNS server listening on UDP port 53 nor can I add additional entries to the existing service. @OneBlue, would it be possible to add this functionality? Can this be done today via some undocumented config?
BTW, @boan-jfm , on my system, I don't see a listening service using TCP port 53, only UDP. Did I understand correctly that on your system you have something using TCP?

@asampal When I run dnsmasq in WSL2 it listen on UDP and TCP port 53 as default.

When I tried to install dnsmasq in WSL2 it actually mentioned a failure due to port 53 already being used as you can see here:

I wonder how it installed for you without any warnings. Perhaps it's due to differences in Windows and WSL kernel versions. I'm at:
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.45
MSRDC version: 1.2.3575
Direct3D version: 1.606.4
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22000.1165

I do not know. But @OneBlue did not mention the "SharedAccess" as an issue in his first comment - I mentioned it in the issue description as i thought it was blocking the DNS requests to WSL.
As far as I understand it, WSL 2 has its own closed environment (VM) - if you run ip a, you can see the internal ip address for the virtual network interface "eth0".  I have done some experiments where I need to map my physical WIFI adapter in Windows to this virtual network interface within WSL 2 to be able to make a nslookup request on the public IP. You can get some inspiration here.
In WSL 2 you can run netstat -lnp to see if there are any other services listing on port 53.

Where I said, "This should be expected given the SharedAccess service module which is tying up the port.", more precisely it would be the case if the WSL2 adapter that dnsmasq was trying to bind to was bridged to the host adapter.

Just a note: From the following comment https://github.com/microsoft/WSL/issues/8783#issuecomment-1233098476 I tried to run ethtool -K eth0 tx off without any changes in the results.

Is there any updates on the resolution of this issue ?


Add Spec.CloudProvider in KubeadmConfig

/kind feature
Describe the solution you'd like
An optional field in the KubeadmConfig Spec called CloudProvider could be useful to implement default logic to set the cloud-provider flags in multiple places.
This is an example of a configuration to setup the first controlplane instance:
apiVersion: bootstrap.cluster.x-k8s.io/v1alpha2
kind: KubeadmConfig
metadata:
  name: ${CLUSTER_NAME}-controlplane-0
spec:
  initConfiguration:
    nodeRegistration:
      name: '{{ ds.meta_data.hostname }}'
      kubeletExtraArgs:
        cloud-provider: aws
  clusterConfiguration:
    apiServer:
      extraArgs:
        cloud-provider: aws
    controllerManager:
      extraArgs:
        cloud-provider: aws
    certificatesDir: /etc/kubernetes/pki
    controlPlaneEndpoint: ""
    clusterName: ${CLUSTER_NAME}
    dns:
      type: ""
    imageRepository: ""
    kubernetesVersion: v1.14.4
    etcd: {}
    networking:
      dnsDomain: "cluster.local"
      podSubnet: "<IP_ADDRESS>/16"
      serviceSubnet: "<IP_ADDRESS>/12"

As you can see, the cloud-provider: aws option is repeated in multiple places. Bringing that up to a unique place (even in Cluster API, if we think it might benefit other bootstrap providers as well) would be beneficial to allow for better defaulting and simplifying the UX.

/milestone v0.1.x

/priority important-soon

/cc @fabriziopandini @chuckha @ncdc @detiber

I'm not necessarily sure this is worth doing, since we said that we want to require external cloud providers for beta.

Wouldn't we have to set the cloud-provider flag to external as well? When we're set to migrate to the external cloud providers, we'll default the proposed field, Spec.CloudProvider to external but still allow users to overwrite it.

@vincepri I'm not sure we would allow users to override it at that point.
One thing we discussed previously, which might make more sense than singling out cloud provider here was to let the infrastructure provider specify additional configuration that is merged with the kubeadm config.

That makes sense, I don't mind either way, after #156 the configuration is looking much better. This is definitely a nice to have 😊, we can always drop it after v1alpha2.


[Event extension request] Purch Post - OnPostInvoicePostingBufferOnAfterVATPostingSetupGet

We would like to extend the existing event OnPostInvoicePostingBufferOnAfterVATPostingSetupGet with additional paramters and an ishandled pattern. We would like to change the VAT caculation in case of reverse charge VAT. We cannot use the event at the start as hat one is outside of the loop.
From:
local procedure OnPostInvoicePostingBufferOnAfterVATPostingSetupGet(var VATPostingSetup: Record "VAT Posting Setup") // <>
To:
local procedure OnPostInvoicePostingBufferOnAfterVATPostingSetupGet(var VATPostingSetup: Record "VAT Posting Setup"; var TempInvoicePostBuffer: Record "Invoice Post. Buffer" temporary; var IsHandled: Boolean) // <>
VATPostingSetup.Get(TempInvoicePostBuffer."VAT Bus. Posting Group", TempInvoicePostBuffer."VAT Prod. Posting Group");
// <<start new event
IsHandled := false;
OnPostInvoicePostingBufferOnAfterVATPostingSetupGet(VATPostingSetup, TempInvoicePostBuffer, IsHandled);
if not IsHandled then begin
// >> Stop new event
 VATBaseAmount := TempInvoicePostBuffer."VAT Base Amount" * (1 - PurchHeader."VAT Base Discount %" / 100);
 VATBaseAmountACY := TempInvoicePostBuffer."VAT Base Amount (ACY)" * (1 - PurchHeader."VAT Base Discount %" / 100);

You add a begin and you do not specify where the "end" should be. If you are interested in "handling" all of the loop, you should use the OnBegin of this whole procedure instead...

We would only like to skip the code inside the Reverse charge VAT code block. For the other case statements we would like to use the standard microsoft code. We are not interested in handling the whole procedure.
            repeat
                case TempInvoicePostBuffer."VAT Calculation Type" of
                    TempInvoicePostBuffer."VAT Calculation Type"::"Reverse Charge VAT":
                        begin
                            VATPostingSetup.Get(TempInvoicePostBuffer."VAT Bus. Posting Group", TempInvoicePostBuffer."VAT Prod. Posting Group");
                            // Start new event
                            IsHandled := false;
                            OnPostInvoicePostingBufferOnAfterVATPostingSetupGet(VATPostingSetup, TempInvoicePostBuffer, IsHandled);
                            if not IsHandled then begin
                                // Stop new event
                                VATBaseAmount := TempInvoicePostBuffer."VAT Base Amount" * (1 - PurchHeader."VAT Base Discount %" / 100);
                                VATBaseAmountACY := TempInvoicePostBuffer."VAT Base Amount (ACY)" * (1 - PurchHeader."VAT Base Discount %" / 100);

                                if PurchHeader."Currency Code" <> '' then
                                    VATBaseAmount := CurrExchRate.ExchangeAmtLCYToFCY(
                                        PurchHeader.GetUseDate(), PurchHeader."Currency Code",
                                        VATBaseAmount, PurchHeader."Currency Factor");

                                VATAmount := VATBaseAmount * VATPostingSetup."VAT %" / 100;
                                VATAmountACY := VATBaseAmountACY * VATPostingSetup."VAT %" / 100;

                                TempInvoicePostBufferReverseCharge := TempInvoicePostBuffer;
                                if TempInvoicePostBufferReverseCharge.Find then begin
                                    VATAmountRemainder += VATAmount;
                                    TempInvoicePostBuffer."VAT Amount" := Round(VATAmountRemainder, CurrencyDocument."Amount Rounding Precision");
                                    VATAmountRemainder -= TempInvoicePostBuffer."VAT Amount";

                                    if PurchHeader."Currency Code" <> '' then
                                        TempInvoicePostBuffer."VAT Amount" := Round(
                                            CurrExchRate.ExchangeAmtFCYToLCY(
                                                PurchHeader.GetUseDate(), PurchHeader."Currency Code", TempInvoicePostBuffer."VAT Amount", PurchHeader."Currency Factor"));

                                    VATAmountACYRemainder += VATAmountACY;
                                    TempInvoicePostBuffer."VAT Amount (ACY)" := Round(VATAmountACYRemainder, Currency."Amount Rounding Precision");
                                    VATAmountACYRemainder -= TempInvoicePostBuffer."VAT Amount (ACY)";

                                    TempInvoicePostBuffer."VAT Base Amount" := Round(TempInvoicePostBuffer."VAT Base Amount" * (1 - PurchHeader."VAT Base Discount %" / 100));
                                    TempInvoicePostBuffer."VAT Base Amount (ACY)" := Round(TempInvoicePostBuffer."VAT Base Amount (ACY)" * (1 - PurchHeader."VAT Base Discount %" / 100));
                                end else begin
                                    if PurchHeader."Currency Code" <> '' then
                                        VATAmount := Round(
                                            CurrExchRate.ExchangeAmtFCYToLCY(PurchHeader.GetUseDate(), PurchHeader."Currency Code", VATAmount, PurchHeader."Currency Factor"))
                                    else
                                        VATAmount := Round(VATAmount);

                                    TempInvoicePostBuffer."VAT Amount" := VATAmount;
                                    TempInvoicePostBuffer."VAT Amount (ACY)" := Round(VATAmountACY, Currency."Amount Rounding Precision");

                                    TempInvoicePostBuffer."VAT Base Amount" := Round(TempInvoicePostBuffer."VAT Base Amount" * (1 - PurchHeader."VAT Base Discount %" / 100));
                                    TempInvoicePostBuffer."VAT Base Amount (ACY)" := Round(TempInvoicePostBuffer."VAT Base Amount (ACY)" * (1 - PurchHeader."VAT Base Discount %" / 100));
                                end;
                            end;
                            TempInvoicePostBuffer.Modify();
                        end;
                    TempInvoicePostBuffer."VAT Calculation Type"::"Sales Tax":

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 51064.


folaga_msgs: 0.0.3-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository folaga_msgs to 0.0.3-0:

upstream repository: https://github.com/revdevlev/folaga_msgs.git
release repository: https://github.com/revdevlev/folaga_msgs-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.23
previous version for package: null

folaga_msgs
* Initial commit
* Contributors: genexusdev

Holding to wait for sync...


Fix infinite recursion for Traceur by refactoring comment handler.

@fkling Thanks for quick reactions btw! I can already see my changes working on live website :)

It's easy to merge things quickly if the contributions are of high quality ;) I really appreciate your help!


No file named datasets/2_generate_vp_label_dist_culane.py

There is no file named 2_generate_vp_label_dist_culane.py in datasets folder.
And the script didn't create any file when I tried to run datasets/generate_vp_label_dist_culane.py

I Am confused too, there is not any file named  2_generate_vp_label_dist_culane.py in datasets folder.

Sorry, I was too busy in the past three months to pay attention to this code. And I will polish this code again in the coming week.


Pass different compiler flags to different compilers

Replaced by https://github.com/Werni2A/OpenOrCadParser/pull/122


Failed to link components.

I got an error when I tried to execute wasi-virt build/pong_s.wasm --allow-random -e PONG=demo -o virt.wasm  the virtualization components with reference to the documentation.
(base) root@PF2S3J70:~/wasmCloud/examples/rust/composition/pong# wasi-virt build/pong_s.wasm --allow-random -e PONG=demo -o virt.wasm
Error: Unable to compose virtualized adapter into component.
Make sure virtualizations are enabled and being used.

Caused by:
    no dependencies of component `build/pong_s.wasm` were found

WASI Virt is currently only supporting<EMAIL_ADDRESS>so this may be due to using a newer version.
I've posted a tracking issue in https://github.com/bytecodealliance/WASI-Virt/issues/110 to address this further.

Excuse me. Are there any ways for me to use it quickly now?

If you're able to use an older toolchain to generate<EMAIL_ADDRESS>components, then you could try it, otherwise I think you are blocked on the upgrade tracked in https://github.com/bytecodealliance/WASI-Virt/issues/110.


compilation_level: "ADVANCED" producing errors for both Java and JavaScript compilers

I just got around to using this plugin to shave off extra bytes from my .js output of an Elm project. I have installed Java 9 SKD as per the requirements as well as the google closure compiler from npm.
This configuration:
 new ClosureCompilerPlugin({
      compiler: {
        jar: './node_modules/google-closure-compiler/compiler.jar',
        language_in: 'ECMASCRIPT6',
        language_out: 'ECMASCRIPT5',
        compilation_level: 'ADVANCED',
        warning_level: 'QUIET'
      },
      jsCompiler: false,
      concurrency: 3
    })

produces these errors and won't compile my output .js file at all:
ERROR in app.js from Closure Compiler
The compiler is waiting for input via stdin.
stdin:1948: ERROR - variable _elm_lang$core$Set$toList is undeclared
                x = _elm_lang$core$Set$toList(x);
                    ^^^^^^^^^^^^^^^^^^^^^^^^^

stdin:5605: ERROR - variable main is undeclared
                view: function(model) { return main; },
                                               ^^^^

stdin:13254: ERROR - variable define is undeclared
if (typeof define === "function" && define['amd'])
           ^^^^^^

3 error(s), 0 warning(s)

However, if I switch the config up to use compilation_level: 'SIMPLE', it will all bundle just fine.

Going further, I tried flipping on the jsCompiler option with compilation_level: ADVANCED, which actually will compile, but when I load the output file in a web browser, I get this error in the console:
app.js:formatted:142 Uncaught TypeError: Cannot read property 'gd' of undefined
    at app.js:formatted:142
    at Object.<anonymous> (app.js:formatted:163)
    at g (app.js:formatted:80)
    at Object.<anonymous> (app.js:formatted:115)
    at g (app.js:formatted:80)
    at app.js:formatted:112
    at app.js:formatted:114

which references this line:
        T = N.Nc.gd(document.getElementById("main"));

Admittedly, the difference between SIMPLE and ADVANCED within the first configuration is 10k, but I'm at a loss for why I'm unable to use the compilation_level: ADVANCED, regardless of the  compiler, beit Java or JavaScript.
If anyone could shed some light on these errors, I would really appreciate it.
Here is the full repo, if anyone needs it.
Love this plugin regardless 👍

This is actually not an issue with this plugin. ADVANCED compilation with Google Closure Compiler requires you to use externs when referencing external JavaScript. E.g document. I recommend taking a look at the docs for ADVANCED compilation located here: https://developers.google.com/closure/compiler/docs/api-tutorial3

As @bansawbanchee pointed out you might want to add externs to your build. Closing it for now.


Rendering plot issue

This is due to some scale not working?? Still looking into it but maybe @jwildfire can help :gulp:

There was an undocumented setting called config.comparisonWidth that we needed to add to the widget init function.


Add support for types, segments, scopes

What has been added :
Types
The following types are set on symbols when possible:

char, unsigned char, int, unsigned int, long, unsigned long
far
array
struct and members

Segments
They are renamed based on related module name (which is based on C source filename)
Eg: "seg002" will be renamed to "SYSTEM" because it's defined in SYSTEM.C
Scopes
local variables and function parameters are renamed when possible.
Eg: "arg_2" will be renamed "counter"
Notes

If needed, I can make some changes / suggestion to the code before merge.
Best results are achieved by creating a new disassembly with auto-analysis turned off, then run python script and only later run analysis.
Otherwise IDA might recognise some symbols incorrectly (eg: "somevar1 dw, somevar2 dw" instead of "somevar dd").
Function to call is still "apply_tdinfo_symbols" while it's actually doing more than that.
Maybe it should be renamed (eg: "apply_tdinfo"), or to provide several functions (eg: "apply_tdinfo_scopes"), depending what user want to achieve.

Thank you very much for your PR, it is a great addition to the project!
I've merged it into a new branch, where I will make some changes and then ask you what you think about them (before merging everything into the main branch). This may take a while.

Thanks a lot for effort you put to improve it and make merge possible. I will take a look at code, do some tests and let you know.


Installation instructions do not work on Niagara anymore because of disabled old modules

The installation instructions of Narval must then be used on Niagara.
The installation page will need updating to take this into account.
Two additional modules are needed:
module load CCEnv
module load StdEnv/2020

Is this still a thing? I just updated deal.II and Lethe on Narval, using the version on the documentation website, and got no problem. I opened a PR to fix a typo on the page of the documentation, so let me know if I need to add this also.


Support for JSON API spec?

New Ember projects use JSONAPIAdapter by default and expect API responses to match the JSON API spec. But getting Rails to serialize models according to this format is pretty hard.
I always find this to be a pain point when starting new Ember/Rails projects. I've settled on using ActiveModelAdapter and ActiveModelSerializer instead, and I put together a boilerplate to make that a little easier.
However, boilerplates are a bad solution. I would much prefer it if using ember-cli-rails just worked out of the box, or came with a very easy configuration step to make it work, so that Rails and Ember can communicate with each other properly.

@philpee2 thanks for opening this issue and getting the discussion started.
This gem's main concern is integrating EmberCLI and Rails at the server/build level.
While I agree with your proposal in spirit, I think integration between Ember-Data and a Rails-based JSONAPI server is out of scope for this project.
However, I'd love to look over your proposed solution. If you'd like, please open a PR. Based on your implementation, we can decide what to do with the code.


Choosing video quality when downloading an anime episode

Describe your suggested feature
Hello,
I would like to ask if you could add an option to choose video quality when trying to download an episode of any anime because downloading the episodes always in Full HD (default by the app downloader) is storage consuming and if I am on data it would take a lot of time so adding a dialog to choose video quality when pressing the download button it would be a cool feature.. and thats my feature request.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to the latest version.
[X] I will fill out all of the requested information in this form.

Duplicate of #136


Migrate script-setup-uses-vars

https://eslint.vuejs.org/rules/script-setup-uses-vars

+1!
Any updates on this? Running into same problem

Made an issue on eslint-plugin-vue but it got closed. I have no idea how to handle this but if it were to be fixed I can communicate it to people there as well.

@Shinigami92 Made an issue on eslint-plugin-vue but it got closed. I have no idea how to handle this but if it were to be fixed I can communicate it to people there as well.

I already have implemented some rules. Please open a PR to support this rule and try to orient yourself on previous PRs.
This project became so huge that I cannot tackle it alone anymore and I'm also working heavily on other projects like @faker-js/faker.

@Blissful89 also, I'm very happy for every helping hand in this project. Sponsoring money would currently not help :slightly_smiling_face:
It's really just a problem that I do not have the time to work on every single thing I want to work on. But I would benefit myself from this rule!
Maybe I should make a shout-out via discussion to ask for helping hands in this repo :thinking:


Add Public Api Analyzer for Management Packages

Description
Enable Microsoft.CodeAnalysis.PublicApiAnalyzers on Steeltoe.Management.* Packages.
Lockdown Surface API area.
Resolves #1122
Quality checklist

[X] Your code complies with our Coding Style.
[X] You've updated unit and/or integration tests for your change, where applicable.
[ ] You've updated documentation for your change, where applicable.
If your change affects other repositories, such as Documentation, Samples and/or MainSite, add linked PRs here.
[X] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.
[X] You've added required license files and/or file headers (explaining where the code came from with proper attribution), where code is copied from StackOverflow, a blog, or OSS.

/azp run code-cleanup

/azp run cleanup-code

/azp run Steeltoe.All

/azp run cleanup-code

/azp run cleanup-code

/azp run cleanup-code

/azp run Steeltoe.All

/azp run  cleanup-code

/azp run  cleanup-code

/azp run  cleanup-code

/azp run  cleanup-code

@hananiel Please add #906 to the list of issues this PR closes. It links to a list of considerations for API review.

/azp run  cleanup-code

/azp run  cleanup-code

/azp run  cleanup-code

/azp run cleanup-code

/azp run  cleanup-code


examine abundance function models for sanity...

@eteq, I have created some custom Schechter function models using the astropy.modeling.model tools.  I want to use these for both fitting abundance functions and defining ones from the literature.
I have code on my fork of halotools on the sham_devel branch.
halotools.empirical_models.abundance_matching.schechter_functions.py
and
halotools.empirical_models.abundance_matching.premade_abundance_functions.py
If you or someone else who has worked with models could take a look at these, and see if I follow conventions and use them intelligently, that would be wonderful.
If you are feeling adventurous, I also coded up a custom fitter:
halotools.empirical_models.abundance_matching.abundance_fitter.py

Closing this issue since there are no longer any plans to include deconvolution abundance matching into halotools.


Renamed 'librocksdb-sys' to 'rocksdb-sys'.

Also added back error::Error impl.

I renamed the FFI to be librocksdb-sys so that we could upload it to crates.io, as rocksdb-sys was already taken.

It's ugly for sure, but it works.

Oh, sorry! I forgot about this.
(I presume the previous owner wasn’t willing to give up the name…)
I’ll let you put in the error::Error impl yourself then.

On 13 Nov 2016, at 23:46, Tyler Neely<EMAIL_ADDRESS>wrote:
It's ugly for sure, but it works.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/spacejam/rust-rocksdb/pull/95#issuecomment-260222541, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEF3K4v3XlVI5ijvucNEy19uOQ4bBTDks5q96FYgaJpZM4Kw1fP.

(And actually, it might be worth accepting this PR, and keeping the dir name as rocksdb-sys, and only change the crate name. But up to you of course.)

On 13 Nov 2016, at 23:46, Tyler Neely<EMAIL_ADDRESS>wrote:
It's ugly for sure, but it works.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/spacejam/rust-rocksdb/pull/95#issuecomment-260222541, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEF3K4v3XlVI5ijvucNEy19uOQ4bBTDks5q96FYgaJpZM4Kw1fP.

I'd like to keep them in-sync for now to minimize breakage due to forgotten naming. It's nice to keep all usage of a thing aligned to as few names as possible, to reduce effort to understand usage while doing future work on it. I appreciate this effort, but I'm going to close this PR now. Master now has the error::Error impl in it!

The problem wasn’t master having the Error impl in it, but rather the 0.5.0 branch.

On 20 Nov 2016, at 10:47, Tyler Neely<EMAIL_ADDRESS>wrote:
I'd like to keep them in-sync for now to minimize breakage due to forgotten naming. It's nice to keep all usage of a thing aligned to as few names as possible, to reduce effort to understand usage while doing future work on it. I appreciate this effort, but I'm going to close this PR now. Master now has the error::Error impl in it!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/spacejam/rust-rocksdb/pull/95#issuecomment-261770944, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEF3Jw7X6r4PwsoWgP_OTmFArpm6RkWks5rACVPgaJpZM4Kw1fP.

Oh, ignore me. You merged 0.5.0 into master. Probably worth remotely deleting the 0.5.0 branch then. :-)

On 20 Nov 2016, at 10:47, Tyler Neely<EMAIL_ADDRESS>wrote:
Closed #95 https://github.com/spacejam/rust-rocksdb/pull/95.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/spacejam/rust-rocksdb/pull/95#event-865576495, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEF3Jw7X6r4PwsoWgP_OTmFArpm6RkWks5rACVPgaJpZM4Kw1fP.


Style updates: collections page

testing notes
designs
desktop / tablet / mobile

-Tile size: In the Zeplin designs, the tiles are all the same size: they seem to conform to the size of the largest tile (the one with the most text), and the tiles with less text on them have more white space at the bottom to fill out the size. On the test site, Grammar and History are smaller and Original Bib and Poetry are bigger based on the amount of text. I think it looks better to standardize the size?

iPhone text margins: On iPhone the margins for the blurb ("The Archive builds...") don't line up with the left edge of the tiles like they do on Zeplin or on iPad. It looks kinda weird but not terrible--is it an easy fix to line it up?

Touch interaction: You'll see on the screenshot too that the tile highlight is melon. On iPad and iPhone if you touch one of the active tiles to scroll, it turns melon and stays melon even after you lift your finger. It doesn't bother me that much, but you tell me whether it's behaving how you want it to.

Page description text margins line up great with the card edges on tablet and mobile.
Re: touch interaction - the tiles still "stay clicked" (melon) until you touch something else on the page (it doesn't change when you lift your finger). This behavior is fine for me, if you're ok with it.
This is super nit-picky-- for mobile, the white space margins on the verbosest tile (Poetry) isn't equal on the bottom and top like it is on iPad and desktop for some reason; the margin is twice as big on the bottom. But no one would notice that but me--it's a difference of .5cm. It's fine.


ORDS 3.0.8 Not supported in OXAR

I am testing latest version of OXAR and downloaded the latest version of ORDS (3.0.8) and received the message:
This version of ORDS is not yet supprted in OXAR
Please report this at http://github.com/OraOpenSource/oxar/issues
To "make it work" I changed the condition in scripts/config_validation.sh to:
    elif [[ ! "${ORDS_MAJOR}.${ORDS_MINOR}.${ORDS_REVISION}" == "3.0.4" ]] \         && [[ ! "${ORDS_MAJOR}.${ORDS_MINOR}.${ORDS_REVISION}" == "3.0.5" ]] \         && [[ ! "${ORDS_MAJOR}.${ORDS_MINOR}.${ORDS_REVISION}" == "3.0.6" ]] \         && [[ ! "${ORDS_MAJOR}.${ORDS_MINOR}.${ORDS_REVISION}" == "3.0.7" ]] \         && [[ ! "${ORDS_MAJOR}.${ORDS_MINOR}.${ORDS_REVISION}" == "3.0.8" ]]; then
I don't see any "interesting" new features since 3.0.6 which was the last supported version:
Changes in 3.0.8
Issues Fixed in 3.0.8
The following changes and enhancements have been made since 3.0.7:
BUG:23640212 - Fix regression preventing the externally authenticated user identity being propagated to the database via the REMOTE_USER CGI header variable
New Features since 3.0.7
None.
Changes in 3.0.7
Issues Fixed in 3.0.7
The following changes and enhancements have been made since 3.0.6:
BUG:24507612 - Fix regression preventing updating of an Application Express based OAuth Client Registration
BUG:24440967 - Address Connection resource leak that may occur under load.
BUG:24344949 - Show 503 Service unavailable status when database reports ORA-00018 Max Sessions Exceeded
BUG:24344949 - Retry request after jittered delay when database reports ORA-00018 Max Sessions Exceeded
BUG:23732108, BUG: 23732138 - Reduce overhead in dispatching all requests, improving performance.
The following changes and enhancements have been made to the Simple Oracle Document Access (SODA) functionality:
BUG:24450332 - Proper handling of 500 and 503 errors from ORDS. Now SODA JSON responses are returned, instead of ORDS responses.
BUG:24308373 - Support versioning in SODA URIs (version "v1" is supported, and is the only supported version at this time).
New Features since 3.0.6
ENH:24347162 - When PL/SQL Gateway uses APEX_PUBLIC_USER automatically add WWV_FLOW_EPG_INCLUDE_MODULES.AUTHORIZE to the security.requestValidationFunction setting

I will report if ORDS 3.0.8 with OXAR downloaded on 2016-10-24 seems to be working..

There's also a version check in the ORDS script, as 3.0.4 had a different silent instal methodl. Tested with 3.0.8 and all working


Fix cross-media-measurement-api repo target to use version.

This was missed in #1055.

This change is


Adds completions to wp-cli plugin

Oh right! I missed that you also added completion! Sorry :grin:
@lmartins @joshmedeski can you try this?

Looks good here.


Create Database for teams

8 teams in a csv file format
First Line (Header):
Team Name,Wins,Losses,Win-Loss Percentage,Games Behind,Year
Add additional columns if  necessary. Not the main database so valid siguro na <8 teams()
Output
Number of lines: 1 + (8 * number of years of your choice)
SAME TEAM : Line 2 and Line 10
SAME TEAM : Line 3 and Line 11
.....
SAME TEAM : Line 10 and Line 18
if 2015 - 2023, Number of lines should be 73

@joeff1 2020 - 2024 dapat yung pagkakasunod nung year
2020
2020
...
2021
..
2022
2022..
..
..
..
2024


[spark] Add pyspark TextEmbedder and update ImageClassifier

Description
Brief description of what this PR is about
Add pyspark TextEmbedder

Codecov Report
Base: 72.08% // Head: 74.04% // Increases project coverage by +1.95% :tada:

Coverage data is based on head (58a8cf4) compared to base (bb5073f).
Patch coverage: 73.72% of modified lines in pull request are covered.

:exclamation: Current head 58a8cf4 differs from pull request most recent head 8908f80. Consider uploading reports for the commit 8908f80 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2414      +/-   ##
============================================
+ Coverage     72.08%   74.04%   +1.95%
- Complexity     5126     6789    +1663
============================================
  Files           473      669     +196
  Lines         21970    29599    +7629
  Branches       2351     3057     +706
============================================
+ Hits          15838    21916    +6078
- Misses         4925     6198    +1273
- Partials       1207     1485     +278

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
67.10% <0.00%> (+7.89%)
:arrow_up:

api/src/main/java/ai/djl/modality/nlp/Decoder.java
63.63% <ø> (ø)

... and 635 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Tuesday 8/13 Goals

Finish from yesterday:
[ ] Finish weight comparison / recap page
[ ] Finish global competition board
Today's tasks:
[ ] Display current ("active") workout page
[ ] Edit current workout and should update in database
[ ] Page that displays pre-set workouts
[ ] Page to see all exercises
If time:
[ ] If user clicks on a pre-set workout, it should become their current active workout

Reminder: CSS is our last priority BUT if you want to style, please focus on PHONE view


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_c23f0f74-af33-4baf-9b03-ea0573b20605/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_c23f0f74-af33-4baf-9b03-ea0573b20605.git

Docs Build status updates of commit 03f72d3:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-02-05-15-53-17 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-02-05-15-53-17

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


init scripts need to send SIGTERM not SIGHUP

1. Tell us something about your changes.
Change the init scripts to send SIGTERM instead of SIGHUP
Removed the now unnnecessary set the leds to black
This allows hyperiond to switch the LEDs and exit cleanly.
Also a minor change to pgrep to exactly match on the hyperiond executable
2. If this changes affect the .conf file. Please provide the changed section
nope
3. Reference a issue (optional)
#575

Did you tested both on a system?
Systemd was working for ma already with SPI, pwm not?

Only Hyperion.init.sh on OSMC.
-HUP is clearly wrong. It's not caught by Hyperiond and it dies immediately

On closer looking the systemd does NOT need fixing.
It refers to a reload option that hyperiond doesn't support
Lemme fix it before you merge

done!
btw, systemd was working for you because systemd defaults to sending a SIGTERM

Don´t get me wrong, we just want to ensure that we don´t screw up anything :+1:
Thank you

Testing ? What's that ?


req.url behavior changed to represent raw url path

Fastly implementation treats req.url as a raw value while falco automatically urldecodes it
This PR implements consistent req.url behavior with fastly.
It also fixes presentation of escaped string values in the debugger

Several, related issues fixed in this PR:

req.url (and req.url.path) in Fastly implementation provides raw version of url and url path while an interpreter returns url decoded versions. This PR makes interpreter behavior consistent with Fastly.
Also urldecode implementation essentially applies url decoding twice, once by applying url.PathUnescape and then by calling url.QueryUnescape. In %25 get yet another extra decoding. Fastly implementation is equivalent to a single application url.QueryUnescape. The confusion is likely caused by the fact that % escape is not decoded in quoted strings which is apparently fixed in PR-256
Fixed % escaped string representation in the debugger
testing.call_subroutine does not report an error when called with nonexistent function. The call is simply ignored.

NOTE: I am splitting this draft into smaller PRs so they are easier to handle


Added Question/Solution for LeetCode Question in Java

Hey there!, Thanks for submitting a PR to this repo
Please complete the following checklist for your PR to be approved. Mark the items by adding an x.
Example:  - [ ] Mark me as completed
To mark: - [x] Mark me as completed

[x] My Pull Request adheres to the Contribution guidelines.
[x] I have added readme.md and solution files in the appropriate folder.
[x] The solution I have given is my own and not plagiarized in any form.
[x] My solution passes all the test cases.
[x] I have properly indented and documented my solution using comments.

If any of the above is unmarked, please make the necessary changes and then mark them so that you're PR can be approved and merged.

@Vishruth-S


[SPA templates] React service-worker interferes with identity.

When publishing a react app with individual auth on, react includes a service-worker by default. This service-worker captures all the requests for identity/identity-server so they never make it to the server.

Seems like we are using an ancient version of react scripts. Updating it seems to fix it.

@javiercn Does this mean you checked-in updates to use the latest react and we can close this?

I updated the react scripts and got rid of the problem


Remove Ansible .retry files

These are created by Ansible when a playbook fails. They should not be
committed to version control.
Signed-off-by: Daniel Farrell<EMAIL_ADDRESS>

cc @shawn-hurley @jmrodri

Hi @mhrivnak,
Shows that we forget to merge it. Any objection?


Support getenvoy CLI on Windows

With the Alpha release of Envoy on Windows approaching, we think it would be useful for users to eventually be able to use the getenvoy CLI tool to fetch and run Envoy etc. once Envoy on Windows is GA. We are opening this issue is to start the conversation about the steps needed to get to Windows support in getenvoy.
As far as I can tell, the high-level steps I can glean from looking around the getenvoy repos are:

Augment https://github.com/tetratelabs/getenvoy-package to support building on Windows
Augment this repo to build the getenvoy CLI on Windows and have it fetch Envoy-Windows builds
Update documentation to add usage examples for Windows

getenvoy/Tetrate package distribution channels could also be augmented for Windows, distributing Envoy via chocolatey or some other package manager
If there is anything high-level that maintainers of getenvoy see is critical and missing, please let us know (and of course if you are willing to support getenvoy on Windows at all!)

We're getting close to a Beta release of Envoy on Windows as well, do the maintainers have any thoughts on this work?

@sunjayBhatia Sorry for being late on this, yes we definitely love to see Windows support of GetEnvoy! The outline above looks great to me.

Awesome, thanks @lizan, we will get some PRs going to hopefully incrementally get this going

we are also using GH actions consistently here, now, and would need to add windows to the matrix for that to work. Are you still interested in contributing?

It looks like windows is officially supported with the most recent version of envoy. We may be able to play a trick to test this by extracting the binary from the upstream docker image https://blog.envoyproxy.io/general-availability-of-envoy-on-windows-267e4544994a

added this issue to ask envoy to retain the binaries they already build https://github.com/envoyproxy/envoy/issues/16830

Sorry unfortunately internal company priorities changed for me so I'm light on time to contribute to this at the moment!

@sunjayBhatia no problem. Tetrate bought me a windows laptop so I can look into this personally.. and I am now. I hope by the end of the week things will be working even if using a temporary envoy binary tarball. Will make notes here with progress.

#309 will fix this, but it won't likely merge until next week. If anyone is impatient and has a windows environment with go 1.16.5 installed, you can install it on your own meanwhile!
GOBIN=$PWD go install  github.com/tetratelabs/func-e@windows-compile

func-e v0.6.0 supports windows

notes are here, though we need to complete packaging #316