Refactor pianoteq.

Remove fetching binaries using provided login, because it's utterly inconvenient and exposes the password. Instead, require the user to manually download the inputs.
Also, add hash for Pianoteq STAGE 8.1.3.
Description of changes

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
[ ] 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

[ ] Fits CONTRIBUTING.md.

Remove fetching binaries using provided login, because it's utterly inconvenient

I went to great lengths to implement this because I find it utterly convenient :slightly_smiling_face:

exposes the password

This is done via environment variables just like fetchFromGitHub does (in fact I copied it from there), I don't see the problem?

require the user to manually download the inputs.

I can see how some people would prefer this to having Nix downloading the binaries for them.
Instead of removing functionality I propose refactoring a bit to make the functions in here public (namely: mkPianoteq, fetchPianoteqTrial, fetchPianoteqWithLogin) so users can then easily build their own package in a couple lines instead of depending on nixpkgs being always up to date or needing to submit a PR for some version in particular.

This is done via environment variables just like fetchFromGitHub does (in fact I copied it from there), I don't see the problem?

Sure, but how do you pass those variables? The only viable solution I found was to tweak nix-daemon systemd unit, which is also stored in /nix/store.

Instead of removing functionality I propose refactoring a bit to make the functions in here public (namely: mkPianoteq, fetchPianoteqTrial, fetchPianoteqWithLogin) so users can then easily build their own package in a couple lines instead of depending on nixpkgs being always up to date or needing to submit a PR for some version in particular.

Not sure I follow how this would work. Could you provide an interface of what you want to see? For example, what should I put into configuration.nix in your version of things?

Sure, but how do you pass those variables? The only viable solution I found was to tweak nix-daemon systemd unit, which is also stored in /nix/store.

nix.envVars = {
  NIX_MODARTT_USERNAME = "...";
  NIX_MODARTT_PASSWORD = "...";
};

Now that you mention it, it took me a while to figure this out too. I don't think this is well documented.
In practice I put those values in a gitignored file, but you can probably use some secret-management solution too.

Not sure I follow how this would work. Could you provide an interface of what you want to see? For example, what should I put into configuration.nix in your version of things?

Pretty much what you wrote in this PR:
environment.systemPackages = [
...
  (pkgs.pianoteq.mkPianoteq rec {
    name = "stage-8";
    mainProgram = "Pianoteq 8 STAGE";
    startupWMClass = "Pianoteq STAGE";
    version = "8.1.3";
    src = requireFile {
      name = "pianoteq_stage_linux_v813.7z";
      url = "https://www.modartt.com/download";
      sha256 = "1x70hl15ql8i7g78mmhsfn5z1w5y41lxi9w8fnw779wl7dywj1a6";
    };
  })
];

We just need to put mkPianoteq and the other functions in the final set to enable this I think.

nix.envVars = {
NIX_MODARTT_USERNAME = "...";
NIX_MODARTT_PASSWORD = "...";
};

Yeah, no. If your entire configuration.nix is stored in /nix/store (and it is), this will be too. I can imagine making a kind of /secrets directory, having the login there, and then setting nix.settings.extra-sandbox-paths to include that directory. But this is just a world of pain to manage.

Pretty much what you wrote in this PR:
environment.systemPackages = [
...
(pkgs.pianoteq.mkPianoteq rec {
name = "stage-8";
mainProgram = "Pianoteq 8 STAGE";
startupWMClass = "Pianoteq STAGE";
version = "8.1.3";
src = requireFile {
name = "pianoteq_stage_linux_v813.7z";
url = "https://www.modartt.com/download";
sha256 = "1x70hl15ql8i7g78mmhsfn5z1w5y41lxi9w8fnw779wl7dywj1a6";
};
})
];
We just need to put mkPianoteq and the other functions in the final set to enable this I think.

Ok, will do. Should I also drop all those hardcoded Pianoteq versions in the meanwhile? Does anyone really need or want Pianoteq 6 here?

Yeah, no. If your entire configuration.nix is stored in /nix/store (and it is), this will be too. I can imagine making a kind of /secrets directory, having the credentials in a file there, and then setting nix.settings.extra-sandbox-paths to include that directory. But this is just a world of pain to manage.
As far as I gather, none of those work from inside nixpkgs. I.e. building a nixpkgs package using those isn't advisable.

Yeah secrets management is a pain in NixOS.
Like I said I just copied that environment variable thing from fetchFromGitHub so maybe try asking on https://discourse.nixos.org/ how people manage private repositories with that?
Personally I wouldn't want my private github credentials in the nix store but I don't think it's a big deal for some Modartt credentials on my personal laptop.

Should I also drop all those hardcoded Pianoteq versions in the meanwhile? Does anyone really need or want Pianoteq 6 here?

Let's not delete anything... People have contributed the version of Pianoteq that they own. Personally I own and use Pianoteq 6.

Are there any other packages in the nix store that have to solve similar problems?

I also think there is another problem to point out here:
If something goes wrong with the credentials, for whatever authentication reason, then it's possible for your entire configuration to stop building.
There are some different use cases to keep in mind too.

Downloading and installing on the same computer. This is likely the most common usage, and in this case, the secrets will only need to be on one machine.
Downloading and installing on different computers. This might happen to someone who, say, uses a raspberry pi to use pianoteq, but wants to install it from their laptop.

In case number 2, I think it makes sense to only have the secrets available in the build phase and not the installation phase. I.E. not storing the secrets in the nix store.

Are there any other packages in the nix store that have to solve similar problems?
In this case, the problem being, we can't even download the binary without credentials.

Yes like I said I roughly copied the solution here from fetchFromGitHub.
See also:

https://github.com/NixOS/nixpkgs/blob/26a7cb60c57b9802da3eca617945feb04ef05fb5/pkgs/build-support/fetchurl/default.nix#L81
https://nixos.org/manual/nix/stable/command-ref/conf-file.html?highlight=netrc#conf-netrc-file
https://nixos.org/manual/nix/stable/language/advanced-attributes#adv-attr-impureEnvVars

It might be possible to use a netrc file to provide the credentials instead (though it seems to be designed for HTTP basic auth)

If something goes wrong with the credentials, for whatever authentication reason, then it's possible for your entire configuration to stop building.

Hmm tbh I don't see how this is an issue? I mean it's just like with any other package? If it fails to build you fix it :slightly_smiling_face:

Downloading and installing on different computers. This might happen to someone who, say, uses a raspberry pi to use pianoteq, but wants to install it from their laptop.

This is an interesting scenario :+1:
For the scope of this PR I'd recommend sticking to exporting functions so people can use requireFile when necessary.

I decided in the end to just copy some code from this derivation into my personal flake, so there's no point in keeping this PR open.


Delegating CORS to nginx

I am trying to delegate all cors realted settings to nginx. Here is what nginx.conf looks like:
user nginx;
worker_processes 1;

error_log /var/log/nginx/error.log;
pid /var/run/nginx.pid;

events {
  # max_clients = worker_processes * worker_connections / 4
  worker_connections 1024;
}

http {
  include mime.types;
  default_type application/json;
  sendfile on;

  # backend applications
  upstream nodes {
    server <IP_ADDRESS>:8000;
    server <IP_ADDRESS>:8001;
    keepalive 64;
  }

  server {
    listen 80;
    server_name domain.com;

    # static resources
    location ~ ^/(robots.txt|humans.txt|favicon.ico) {
      root /usr/share/nginx/html;
      access_log off;
      expires max;
    }

    # everything else goes to backend node apps
    location / {
      proxy_pass http://nodes;

      proxy_redirect off;
      proxy_set_header X-Real-IP $remote_addr;
      proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
      proxy_set_header X-Forwarded-Proto $scheme;
      proxy_set_header Host $host;
      proxy_set_header X-NginX-Proxy true;
      proxy_set_header Connection "";
      proxy_http_version 1.1;

      if ($request_method = 'OPTIONS') {

        add_header Access-Control-Allow-Origin '*';
        add_header Access-Control-Allow-Credentials 'true';
        add_header Access-Control-Allow-Methods 'GET, POST, PUT, DELETE, OPTIONS, HEAD';

        #
        # Custom headers and headers various browsers *should* be OK with but aren't
        #

        add_header Access-Control-Allow-Headers 'DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';

        #
        # Tell client that this pre-flight info is valid for 20 days
        #

        add_header Access-Control-Max-Age 1728000;
        add_header Content-Type 'application/json charset=UTF-8';

        return 204;
      }

      if ($request_method = 'POST') {

        add_header Access-Control-Allow-Origin '*';
        add_header Access-Control-Allow-Credentials 'true';
        add_header Access-Control-Allow-Methods 'GET, POST, PUT, DELETE, OPTIONS, HEAD';
        add_header Access-Control-Allow-Headers 'DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';

      }

      if ($request_method = 'GET') {

        add_header Access-Control-Allow-Origin '*';
        add_header Access-Control-Allow-Credentials 'true';
        add_header Access-Control-Allow-Methods 'GET, POST, PUT, DELETE, OPTIONS, HEAD';
        add_header Access-Control-Allow-Headers 'DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';

      }
    }
  }

}

As you can see, I am setting cors headers in the nginx conf.
Any request to the server returns 2 values for the same header. For example:
Access-Control-Allow-Credentials:true
Access-Control-Allow-Credentials:
Access-Control-Allow-Headers:DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type
Access-Control-Allow-Headers:

After some debugging, I can safely guess that the empty headers are being set by sails. Is there a way to disable these headers from sails? As I just want sails to be my application and everything else to be handled by nginx

Try proxy_hide_header ?
-kevin
On Mar 13, 2015, at 15:54, Akshay Katyal<EMAIL_ADDRESS>wrote:
I am trying to delegate all cors realted settings to nginx. Here is what
nginx.conf looks like:
user nginx;
worker_processes 1;
error_log /var/log/nginx/error.log;
pid /var/run/nginx.pid;
events {
max_clients = worker_processes * worker_connections / 4
worker_connections 1024;
}
http {
include mime.types;
default_type application/json;
sendfile on;
backend applications
upstream nodes {
server <IP_ADDRESS>:8000;
server <IP_ADDRESS>:8001;
keepalive 64;
}
server {
listen 80;
server_name domain.com;
# static resources
location ~ ^/(robots.txt|humans.txt|favicon.ico) {
  root /usr/share/nginx/html;
  access_log off;
  expires max;
}

# everything else goes to backend node apps
location / {
  proxy_pass http://nodes;

  proxy_redirect off;
  proxy_set_header X-Real-IP $remote_addr;
  proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
  proxy_set_header X-Forwarded-Proto $scheme;
  proxy_set_header Host $host;
  proxy_set_header X-NginX-Proxy true;
  proxy_set_header Connection "";
  proxy_http_version 1.1;

  if ($request_method = 'OPTIONS') {

    add_header Access-Control-Allow-Origin '*';
    add_header Access-Control-Allow-Credentials 'true';
    add_header Access-Control-Allow-Methods 'GET, POST, PUT,

DELETE, OPTIONS, HEAD';
    #
    # Custom headers and headers various browsers *should* be OK

with but aren't
#
    add_header Access-Control-Allow-Headers

'DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';
    #
    # Tell client that this pre-flight info is valid for 20 days
    #

    add_header Access-Control-Max-Age 1728000;
    add_header Content-Type 'application/json charset=UTF-8';

    return 204;
  }

  if ($request_method = 'POST') {

    add_header Access-Control-Allow-Origin '*';
    add_header Access-Control-Allow-Credentials 'true';
    add_header Access-Control-Allow-Methods 'GET, POST, PUT,

DELETE, OPTIONS, HEAD';
add_header Access-Control-Allow-Headers
'DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';
  }

  if ($request_method = 'GET') {

    add_header Access-Control-Allow-Origin '*';
    add_header Access-Control-Allow-Credentials 'true';
    add_header Access-Control-Allow-Methods 'GET, POST, PUT,

DELETE, OPTIONS, HEAD';
add_header Access-Control-Allow-Headers
'DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';
  }
}

}
}
As you can see, I am setting cors headers in the nginx conf.
Any request to the server returns 2 values for the same header. For example:
Access-Control-Allow-Credentials:true
Access-Control-Allow-Credentials:
Access-Control-Allow-Headers:DNT,X-Mx-ReqToken,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type
Access-Control-Allow-Headers:
After some debugging, I can safely guess that the empty headers are being
set by sails. Is there a way to disable these headers from sails? As I just
want sails to be my application and everything else to be handled by nginx
—
Reply to this email directly or view it on GitHub
https://github.com/balderdashy/sails/issues/2749.

I added:
proxy_hide_header Access-Control-Allow-Origin;
proxy_hide_header Access-Control-Allow-Credentials;
proxy_hide_header Access-Control-Allow-Methods;
proxy_hide_header Access-Control-Allow-Headers;
proxy_hide_header Access-Control-Max-Age;

Now 2xx response code work find but no header is attached for any other response code. Any idea why that might be happening?

Haven't worked with nginx in a while, but it's highly possible the commands
only set headers for 2xx replies. May want to try the headers more module
On Fri, Mar 13, 2015 at 4:16 PM, Akshay Katyal<EMAIL_ADDRESS>wrote:

I added:
proxy_hide_header Access-Control-Allow-Origin;
proxy_hide_header Access-Control-Allow-Credentials;
proxy_hide_header Access-Control-Allow-Methods;
proxy_hide_header Access-Control-Allow-Headers;
proxy_hide_header Access-Control-Max-Age;
Now 2xx response code work find but no header is attached for any other
response code. Any idea why that might be happening?
—
Reply to this email directly or view it on GitHub
https://github.com/balderdashy/sails/issues/2749#issuecomment-79520078.

--
kevin

Got it working by using more headers module. Thanks :)

@MrDHat Can you post your response here?


Default Splat logging shouldn't suck

Console.WriteLine it or something on every platform that can Read Good And Do Other Stuff Good (aka not WinRT)

See also #123 plus this was discussed in Slack today again as a good idea (tm).
PR's welcome.


Dice Count Increment

I'd like to have a modifier to the rolled dice total, eg, 3d6 "+5"
Describe the solution you'd like
Add a slider below the dice count to select a value (0-10) to increment the final dice total
Additional context
common for RPG roles to have a modifier on the roll

I've written the code (an easy extension based upon existing code around the dicer slider) for your consideration; As I am a very infrequent contributor to projects, please let me know how best I can support your project.  Can you create a feature branch for my work?

You can fork this repository, push your changes there and open a PR.

PR Created - Dice Count Increment #81

I am VERY much a rookie at ui layout, so it is likely that adjustments will be needed regarding the placement of the increment slider


When Cancel is selected on Tag editing flyout the flyout should close

Type of issue

[x ] Bug
[ ] New feature
[ ] Improvement

Description
When Cancel is selected on Tag editing flyout the flyout should close

https://github.com/Azure/pcs-remote-monitoring-webui/pull/637. PR is in review.


Update correct GCC email relay

Email relay for GCC was inaccurate, causing customer issues. Updating.

@LolaJacobsen - thanks!

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LolaJacobsen PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[AzureRmWebAppDeployment]Release annotations bug fix

Release annotations endpoint is returning a 201 response now as opposed to 200 response returned earlier.

Fix for the issue #8022 .


[QUESTION] What is the purpose of this repo?

Will all the runtime code be removed?

The purpose of this repo is to store the code for the EF6 and LINQ-To-SQL designers. Currently it lives in a branch on the ef6 repo, but the "master" branch of that repo had it (purposely) removed.
Yes, eventually we're planning to remove the runtime code and build it just off the NuGet versions of EF6, but some interdependency has built up over time so it's not quite as simple as just deleting the folders.


add activation price and reservation price for long term flexibility in PT demo

The activation price is the one to be used in the DE demo.
We will need both the activation price and the reservation price in the PT demo

@pierre-crucifix-n-side @ArnaudDebray  Can you check if this PR is ok for you ? Should those two prices be allowed simultaneously ? Is it ok to have only one price in the outputs of the tool ?

As discussed in meeting, we will postpone this PR until we've discussed it in more detail. Things to note that will affect this PR:

Changes should be based on the updated data structure after taking into account price curves etc. Meaning that the prices should be added to eg a new PricePoint data type.
Clarifications are needed on what an order are. If assets are not activated immediately, how should they be activated?
This will affect the trade data type as well - how?


0066-knife

An alternate lowercase f based on the f from Fifteen, as requested by @chrissimpkins
The contour is simply an extension of the already existing f, extending to the same height as the ascender ascends. The cutaway bits are at a 45º angle, the italics slightly more to adjust for the slant. That took some "very difficult" math to figure out ; )

Love it!  I am going to build a set with these glyphs.  Have you tried them yet?


Sub tree solving

Hey,
I was wondering how would one go about solving a game of NLTH on only a few subsets of a full game tree.
For example, how could I limit the poker engine used when training CFR; so that lets say,  the flop 2c2h2d always gets dealt. Another example would be to have a fixed starting hand.
What I am trying to do is this: find flop textures that are similar and group them together.  For that I would like to see how each hand acts on particular board and so on.
Another completly seperate thing I noticed. You have a DLL for evaluating hands, is it possible to get a source code for that?  Some variants of poker have smaller decks and rules.
For example  6+ No limit holdem, in this game players play with 36 card deck. And because of that hand rankings are different; Flush beats a full house etc.
Thank you,
Jonas

Hi Jonas,
Subtree solving with CFR is not directly supported (feel free to make a PR!). However, for sampling-based methods, this can be done with the RL interface by setting the state of the environment to a manually created state dictionary.
The source of the C++ handeval source code has purposefully not been released for exactly this reason. I particularly tried to make it hard to extend to 6+ NLH. Why? Because this library is for research and 6+ NLH does not really seem like an interesting research env, while it is played for a lot of money online. Hence, there is no gain but a lot of risk for non-scientific use. I hope you understand my reasoning here.
Best,
Eric


Optimize like function for generic patterns

Below benchmarks are with only % in the wildcard character set. When the wildcard character set also includes _, a larger improvement is seen (benchmark computation takes a long time and will be updated soon).
Without changes (Re2 matcher):
============================================================================
[...]arks/basic/LikeFunctionsBenchmark.cpp     relative  time/iter   iters/s
============================================================================
genericPattern                                            750.11ns     1.33M

With changes (recursive matching):
============================================================================
[...]arks/basic/LikeFunctionsBenchmark.cpp     relative  time/iter   iters/s
============================================================================
genericPattern                                            262.51ns     3.81M

re2 would perform better for large strings (although a recursive/backtracking search outperforms re2 for smaller strings) so using this approach would not be ideal.


Wrong Link for Erdtree's Favor +2

The current link is https://eldenring.wiki.fextralife.com/Erdtree's+Favor but it should be https://eldenring.wiki.fextralife.com/Erdtree's+Favor+2

Thanks!


UDS Listener merge, no shared SSL

#1363 + merge master.
Check CClient::UserPortCommand in src/ClientCommand.cpp, I STILL don't know about the "listports" entries.

Due to the lack of a decent IRC client for Android, I've chosen to continue to use IRCcloud and will abandon this.

Alright.


Fixed error: 'cp932' codec can't decode

Previously, if the code in the venv node contained Japanese characters, the following error occurred.
The error occurred even if the code was commented out.

This Pull Request resolved that error.
However, if the output contains Japanese characters, the characters are garbled.

The characters are also garbled when executed in the exec node.

This Pull Request only solves the error.
I will create a new issue for the garbled characters.


PROPOSAL: create adopters pages for dapr users

We should create adopters pages for dapr users and accepte the PR from users to add them into the user list.  This page should be a part of dapr website.
It can be two list in this page: one for adopted, one for planning to adapte.
We also should provide a template for this.

Good idea. Do you think the ADOPTERS file needs to be in the community repo or dapr/dapr repo?

Good idea. Do you think the ADOPTERS file needs to be in the community repo or dapr/dapr repo?

I would like to add adopters pages in our dapr website so that it will be included in the web site automately, but repo https://github.com/dapr/website is private....
So anyway we need to "copy" the content of adopters pages to web site.
For community repo or dapr/dapr repo, I found that now community repo is for dapr contributors, not for dapr end users. But I think the end users is also a important part of community, so I suggest to use community repo.

Good idea. Do you think the ADOPTERS file needs to be in the community repo or dapr/dapr repo?

I would like to add adopters pages in our dapr website so that it will be included in the web site automately, but repo https://github.com/dapr/website is private....
So anyway we need to "copy" the content of adopters pages to web site.
For community repo or dapr/dapr repo, I found that now community repo is for dapr contributors, not for dapr end users. But I think the end users are also a important part of community, so I suggest to use community repo.

Yes to creating this ADOPTERS.MD file

File is created.


fix behaviour of memoized functions when given no arguments

Fixes #791
We could add extra logic to handle caching of nullary functions, but this doesn't seem useful enough to justify the increase in code complexity.

There seems to be a tension between the need expressed in #613 to avoid depending on the function arity for memoization, and the need @tolmasky is expressing in #791 to memoize the expensive call to a nullary function.  An older version of this handled that by having memoize simply return once(fn) for nullary functions.  But that won't do if we want to be able to memoize on the actual arguments, regardless of arity.  I think we might need that extra logic...

I think we might need that extra logic...

I have something in mind. Let's see what I can do.

I've updated the pull request. The new implementation is correct more often in addition to being significantly simpler.

Almost LGTM.  This is very misleading, and took a little time to figure it out:
assert.strictEqual(f(), 'foobar');
assert.strictEqual(f(), 'foobar');
assert.strictEqual(f(), 'foobar');
assert.strictEqual(count, 3);

Should probably only call f() once in there.
I'd seen CANON before.  This is a nice use, even of such a restricted subset of it.

Good catch, @CrossEye. I added count to that example after the fact, forgetting that the default argument values were filled in via recursive calls. I've removed the recursive calls so the final value of count is now 1.

:herb:

Oops. I forgot to push the update. I'll open a new pull request.


Add highlights for the week ending 2023-03-09

Please add your contributions by 14:00 CET, either as suggestion or as a commit.

nothing from honeybadger this week


Disable building the MatrixDiag and OneHot kernels for GPU on Windows.

This is a temporary workaround for issue #6509, while the root cause of the runtime errors is under investigation.

@gunan I didn't manage to get the GPU PR build started... here's the error: http://ci.tensorflow.org/job/tf-pr-win-cmake-gpu/5/console
21:59:28 c:\tf_jenkins\home\workspace\tf-pr-win-cmake-gpu>call tensorflow/tools/ci_build/windows/gpu/cmake/run_py.bat
21:59:28 'tensorflow' is not recognized as an internal or external command,
21:59:28 operable program or batch file.
21:59:28
21:59:28 c:\tf_jenkins\home\workspace\tf-pr-win-cmake-gpu>exit 1

Any ideas? I'll test it on my workstation tomorrow if not....

Got it to run now,
http://ci.tensorflow.org/job/tf-pr-win-cmake-gpu/7/
Apparently we need to provide PR number slightly differently.

Thanks for looking into that! Trying a new run, which should fix the failure: http://ci.tensorflow.org/job/tf-pr-win-cmake-gpu/8/

Once more without a silly typo this time: http://ci.tensorflow.org/job/tf-pr-win-cmake-gpu/9/

Well the build timed out before reaching the one_hot_op_test.py. :(
@gunan Any ideas for how to make it run longer? I also made a similar change to the MatrixDiag ops and diag_op_test.py has failed, but when I look in the workspace at the testing log, I don't see any errors.... Is it maybe failing because it's not using the GPU (e.g. because of some smarts put in to ensure that we're actually testing GPU implementations)?

ah, I have upped the timeout to 5 hours and restarted the build here:
http://ci.tensorflow.org/job/tf-pr-win-cmake-gpu/10/

Five hours ought to be enough for any test run! :P

@gunan Looks like one_hot_op_test.py and trace_op_test.py succeeded, but diag_op_test.py failed for mysterious reasons. 2 out of 3 isn't bad:
http://ci.tensorflow.org/job/tf-pr-win-cmake-gpu/10/console
Shall we merge this, since it should at least prevent ugly crashes?

Sure, sgtm.
The remaining issues are known to me so the tests are all OK.


Run mongodb monitoring and backup agents as non-root user

USER mongodb-mms-agent  in Dockerfile for both backup and monitoring agents.
Will need dir and file permission changes in entrypoint script and Dockerfile.

We need to modify the read permissions on the secrets and allow
the mongodb-mms-agent to read the credentials. Alternative approach is to change the readOnly permissions from mounted volumes /etc/mongod/cloud and /etc/mongod/ssl and make mongodb-mms-agent the owner of the above mentioned directories in the entrypoint script but I went with the earlier approach because changing readOnly on the volumes might be such a good idea.

@muawiakh Can this issue be close now?

Yes.


UnhandledPromiseRejectionWarning: RangeError: Invalid count value (in docker-compose on Mac)

Current behavior
Currently when using webpack-dev-server every build is logging the following when running inside a docker-compose container (the behavior is not seen outside the docker container):
Issues checking in progress...
(node:328) UnhandledPromiseRejectionWarning: RangeError: Invalid count value
    at String.repeat (<anonymous>)
    at highlightedLines.split.slice.map (/app/node_modules/fork-ts-checker-webpack-plugin/node_modules/@babel/code-frame/lib/index.js:121:135)
    at Array.map (<anonymous>)
    at Object.codeFrameColumns (/app/node_modules/fork-ts-checker-webpack-plugin/node_modules/@babel/code-frame/lib/index.js:108:65)
    at codeFrameFormatter (/app/node_modules/fork-ts-checker-webpack-plugin/lib/formatter/CodeFrameFormatter.js:16:34)
    at webpackFormatter (/app/node_modules/fork-ts-checker-webpack-plugin/lib/formatter/WebpackFormatter.js:21:58)
    at configuration.logger.issues.error.issues.map (/app/node_modules/fork-ts-checker-webpack-plugin/lib/hooks/tapDoneToAsyncGetIssues.js:65:69)
    at Array.map (<anonymous>)
    at Object.<anonymous> (/app/node_modules/fork-ts-checker-webpack-plugin/lib/hooks/tapDoneToAsyncGetIssues.js:65:54)
    at Generator.next (<anonymous>)
(node:328) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)

Expected behavior
Expected behavior would be getting notified of issues as normal
Steps to reproduce the issue
webpack.config.development.js:
module.exports = {
  context: `${process.cwd()}/client/`, // to automatically find tsconfig.json
  entry: './src/index.tsx',
  devtool: 'inline-source-map',
  plugins: [
    new ForkTsCheckerWebpackPlugin({
      eslint: {
        enabled: true,
        files: './src/**/*.{tsx,ts}',
        options: {
          extensions: ['.js', '.ts', '.tsx'],
          cache: true,
        },
      },
      typescript: {
        enabled: true,
        profile: false,
      },
    }),
    new ForkTsCheckerNotifierWebpackPlugin({
      title: 'TypeScript',
      excludeWarnings: false,
    }),
  ],
  module: {
    rules: [
      {
        test: /\.tsx?$/,
        use: {
          loader: 'ts-loader',
          options: {
            transpileOnly: true,
            getCustomTransformers: () => ({
              before: [tsImportPluginFactory({ style: 'css' })],
            }),
          },
        },
        exclude: /node_modules/,
      },
    ],
  },
  output: {
    filename: 'bundle.js',
    path: path.resolve(__dirname, 'build'),
    publicPath: '/',
  },
  devServer: {
    clientLogLevel: 'warning',
    open: true,
    historyApiFallback: true,
    stats: 'errors-only',
    host: '<IP_ADDRESS>',
    port: 8080,
  },
  watchOptions: {
    ignored: /node_modules/,
    aggregateTimeout: 300,
    poll: 500,
  },
};

Command being run: webpack-dev-server --progress --color --mode development --config ./client/webpack.config.development.js
Environment

fork-ts-checker-webpack-plugin: 5.2.0
fork-ts-checker-notifier-webpack-plugin: 3.0.0
typescript: 4.0.2
eslint: 7.9.0
webpack: 4.44.1
webpack-dev-server: 3.11.0
os: macOS Catalina 10.15.5

Docker versions:

This appears to have been solved by rebuilding dependencies, closing for now.

I am having this same issue. Its definitely a bug as you shouldn't have to rebuild your project for fork-ts-checker to not crash.

any progress?

I have experienced this error as well. Rebuilding didn't help.


Obsidian freezes with floating window and obsidian advanced slides preview upon starting

Describe the bug
Obsidian freezes when starting the app if a slide preview is open, and a floating window as well.
It's pretty reproducible for me, I need to manually edit workspace.json and delete the config around the floating window to get it running again.
Samples to Reproduce

Open slide preview for advanced slides
Open a floating window, so, a file in an additional obsidian window of the same instance
close obsidian
reopen obsidian
obsidian freezes

Expected behavior

Obsidian does not freeze when starting

I would love to further debug this, but I have no clue how to get more info out of the frozen Obsidian :sweat_smile:

I'm experiencing this too. It's far more reproducable if you have the slides preview in split mode.
That said it does happen in tab mode as well. Once it starts happening the only way to be able to use a slide preview again seems to be to clear workspace.json

Yes, I encountered that problem as well. Closing that window and then restarting obsidian resolves that problem.


adding to bom option.

Hi!
i want to blob data include bom.
because i created csv file include garbled characters Japanese.
i want to this code to be changed like this.
module.exports = function(data, filename, mime, bom=null) {
    if (bom !== null) {
        data = [bom, data]
    } else {
        data = [data]
    }
    var blob = new Blob(data, {type: mime || 'application/octet-stream'});
    ...
}

// using sample.
var fileDownload = require('react-file-download');
var bom = new Uint8Array([0xEF, 0xBB, 0xBF]);
var mime = 'application/csv; charset=utf-8;';
fileDownload(data, 'filename.csv', mime, bom);

thanks.

@lukaszflorczak @luccafort in release 0.4.6 now.


New package: XAMPPRocky.Tokei version 12.1.2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Resolves #111727

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/111759&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


how to fix it? ERROR: Could not find a version that satisfies the requirement torch==0.4.1 ERROR: No matching distribution found for torch==0.4.1

ERROR: Could not find a version that satisfies the requirement torch==0.4.1
ERROR: No matching distribution found for torch==0.4.1  how to fix it?

Try this:
pip install torch==0.4.1 -f https://download.pytorch.org/whl/torch_stable.html


Check if a subnet has upgrades available

The APM should add the ability to check if an upgrade is available. That function should be callable from the CLI.

Likely need #28 and #29 to be completed first.


Multiple objects

Hi. Is it possible to add multiple objects (target) to the timeline? Or you support just a single object per timeline?
Thanks!

@sasmaster: Other editors solve this by grouping the attributes of each object, so the object is represented by a group (that can be collapsed/expanded).
@zz85:
The editor is great, but it seems that this project isn't maintained anymore.


Jenkins v. 2.249.1 large left menu icons

Upgraded to Jenkins 2.249.1.

Disk Usage Icon is too big
Job Import plugin icon is too big
Attaching screenshot

I have the same problem

Theme doesn't work fine

Fixed that by search and replace 57.6px with 1.5rem.

@visit1985 That worked for some of the icons, but Job Import Plugin icon is still too big and manage jenkins icons are too small now after you change.


can we store attachments in db instead of keeping them in file-system

There are situations where saving attachments as blobs in db is better than saving them in file-system. Can this is be done in Simplity?

Yes.
Step 1 : Create table in your db
create a table named internal_attachments with two fields in it
field1 - attachment_id int primary key, autoIncrement (in oracle, just mention this as primary key)
field2 - attachment blob.
in oracle, create a sequence named internal_attachments_seq. No need to write trigger, simplity manages that
Step 2 : configure application.xml
set attachmentAssistantClass="org.simplity.kernel.db.DbAttachmentAssistant"
If you have been using file-storage, you can not just make this switch. existing attachments would not be available in the new structure.


fix attic data server url

Move the attic data server url setting to docker compose

@ann26 Do we want only the attic server in the docker file? If we're putting the attic server as an environment variable, should we do the default overpass url as well?

hi @smit1678 yes we can do that

LGTM


Fix merge causing includes to no longer work

I was prototyping a bit and noticed includes weren't working anymore. It looks like something broke with merging my changes with @AlmarAubel's changes.
All includes are loaded in line 398 so these lines are not needed anymore. Removing them seems to fix the problem.

Good catch @robert-dejong, thank you!


V4.1.0 Cannot create a second Character using Hero Mancer

After creating the first character using Hero Mancer when attempting to create a second character I get the error that has been attached.
Uncaught (in promise) Error: undefined. Failed to render Application "hero-mancer-app":
Cannot read properties of undefined (reading 'start')
[No packages detected]
    at #render (foundry-esm.js:44349:15)
    at async Semaphore._try (foundry-esm.js:6645:19)

Can confirm, same issue on my side aswell


responsive / adjustable dominode sizes

Dominodes currently have a fixed size, which makes the app unusable on smaller screens

Dominodes now scale to the current viewport width. There is a slight bug when the window is resized, but I will open another issue thread for that


[build] Cython files are not located at the fully qualified name location

@picca have this logs will compiling the debian package (relative to https://github.com/silx-kit/silx/issues/1012).
Warning: Extension name 'silx.io.specfile' does not match fully qualified name 'specfile' of 'silx/io/specfile/specfile.pyx'
Warning: Extension name 'silx.math.fit.functions' does not match fully qualified name 'functions' of 'silx/math/fit/functions/functions.pyx'
Warning: Extension name 'silx.math.fit.filters' does not match fully qualified name 'filters' of 'silx/math/fit/filters/filters.pyx'
Warning: Extension name 'silx.math.fit.peaks' does not match fully qualified name 'peaks' of 'silx/math/fit/peaks/peaks.pyx'
Warning: Extension name 'silx.math.chistogramnd' does not match fully qualified name 'chistogramnd' of 'silx/math/histogramnd/chistogramnd.pyx'
Warning: Extension name 'silx.math.chistogramnd_lut' does not match fully qualified name 'chistogramnd_lut' of 'silx/math/histogramnd/chistogramnd_lut.pyx'
Warning: Extension name 'silx.math.marchingcubes' does not match fully qualified name 'marchingcubes' of 'silx/math/marchingcubes/marchingcubes.pyx'
Warning: Extension name 'silx.math.combo' does not match fully qualified name 'combo' of 'silx/math/combo/combo.pyx'

It is maybe a warning from a special Cython version. @PiRK try to reprocude it without success.
It looks like we do not locate always Cython file at the final module location.
For example we expecte to have an ext module silx.io.specfile but the Cython file is located at silx/io/specfile/specfile.pyx. It would make sens to have a silx/io/specfile.pyx file.

Yes sire,
I have cython     | 0.25.2-1

Cython 0.21.1 on my machine.
We have a consistency issue anyway, because some of our pyx files are at the location of their fully qualified name (silx.image.bilinear ...) while some other are in a subfolder together with a C-library that they are wrapping (all the ones that show warnings on your machine).
I will try to fix this.


Enhanced cluster updates

Added parameter for cluster_type (basic | enhanced). Modified oke.tf worker node image selection to be dynamic type, support for OKE, Platform, and Custom image types.

Nice enhancement! Thank You @enschilling :)))


per window shaders like picom

Description
I'd like to have window shaders similar to picoms window-shader-fg-rule.
What i want to archive is color based transparency to turn the background color of my windows transparent without affecting visibility.
https://github.com/ikz87/picom-shaders/wiki/Shader-gallery#color-based-transparency

I dont see a point. Easily done with a plugin, anyways

@quantenzitrone I forked the hypr-darkwindow plugin here and made it do chromakey transparency. You can find the installation instructions here


Problem with angular-electron?

Hello, I've got empty angular-electron app and wanted to add NFC to it. The problem is that when I copy the first example to HomeComponent I can't compile it.
ERROR in vendor.bundle.js from UglifyJs Unexpected token: name (ACR122Reader)
error comes out, any ideas?
Additionally this happens whenI try to run in dev mode.
I'm using the newest electron and nodejs.

Hi @drajvver,
I'll sum up (just for the evidence) what we discussed and found out during our chat:

According to the angular-electron documentation – Use NodeJS Native libraries:

If you need to use NodeJS native libraries, you MUST add it manually in the file webpack.config.js in root folder ...

So in this case you need to add "nfc-pcsc": "require('nfc-pcsc')" in the webpack.config.js externals section (for example below https://github.com/maximegris/angular-electron/blob/master/webpack.config.js#L199).

Don't forget that you need to rebuild the native modules for Electron.
You can do it for example this way (just note that's not the only solution). 🙂

I have created a PR https://github.com/drajvver/nfc-pcsc-angular-error/pull/1 with these changes in your not working example code.
It works for me and I hope you will be able to make it work too (just follow the instructions in the PR).
Hope it helps.

PS Don't forget to star ⭐  my library, if you find it useful. 😃 Thanks.

There is no webpack.config.js in recent versions of angular (as of 6.0).
So how can we configure nfc-pcsc in angular +6?


Improve layout for news article

Implement a concept of fluid design/grid for the news article template across screen sizes

We need to find a clever way to do this for the serializers. My initial idea was ofc to just use some styled component woo aka parent selector, but that didn't work server side. Seems like the serializers run before that's initialized... @TODO: Look more into this to avoid duplication of serializers.

Needs alignments: who owns the margins, the component or the container

My vote goes to the container implementing the component owning the margins around the component - since we may use the component in multiple places, each with different needs.

Let's take a look at the remaining issues on Monday (add the correct spacers)


CustomEntityProjectile as EntityProjectile

Make CustomEntityProjectile extending EntityProjectile so that it can be used in RangedAttackGoal#setProjectileGenerator.

On second thought, this poses a problem, since the tick() method of EntityProjectile is called before the custom tick method, which means that it might trigger a hit or an unstuck when we don't want it to

Related to issue #22


fix(afs): catch error when enabling persistence

Checklist

Docs included?: no
Test units included?: no
In a clean directory, npm install, npm run build, and npm test run successfully? yes

Description
Instead of getting an "Uncaught Promise" error in console, this will set the persistanceEnabled$ observable properly

Nice one, I was planning on doing that this evening. Thanks!


(PUP-4065) Warn once about illegal environment.conf setting

Before this because an environment.conf file is validated mulitple times there could be many logged warnings for ignored but invalid contents.
This changes the logging of these warning to use warn_once - there will now only be one logged message per environment.conf file for extra sections, and one for extraneous settings found.

CLA signed by all contributors.

@thallgren - This is 4.10.x - before internationalization...

I know, but I've been told that it's still desirable to do I18N on new or modified messages, otherwise there's a good chance we loose track of them (they get merged without getting I18N'ed).

@Magisus thoughts on ^?

I would say we should I18N the strings since they're warnings.

I will deal with the I18N when I merge this to 5.3.x


[Bug]:

Platform
iOS 16.7
Plugin
connectivity_plus
Version
4.0.2
Flutter SDK
3.13.6
Steps to reproduce
Just wasted 4 days becuase of this package. it's working on debug mode with no issues, but craches the app in release mode.
The package was working fine, but after I updated xCode to the latest version that supports iOS 17, the app started crashing immediatly after launching it on iOS versions before iOS 17, but it is working on iOS 17.
Code Sample
No response
Logs
No logs, because it is working on debug mode, but not working in release mode.

Flutter Doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.6, on macOS 14.0 23A344 darwin-arm64, locale en-MA)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.82.3)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

Same thing, simulator - ok 16.4 simulator - ок, 17 device - ok, 16.3.1 - crash here VVV

@raqeta  I solved this issue but instaling the latest package from github instead of the pub.dev package:
# connectivity_plus: ^4.0.2   connectivity_plus:     git:       url: git@github.com:fluttercommunity/plus_plugins.git       path: packages/connectivity_plus/connectivity_plus
Now this temporary solution is working fine, while waiting for the package team to solve this issue.

same issue

you are great 😎 such a quick response and fix, thank you

Just wasted 4 days becuase of this package. it's working on debug mode with no issues, but craches the app in release mode.
The package was working fine, but after I updated xCode to the latest version that supports iOS 17, the app started crashing immediatly after launching it on iOS versions before iOS 17, but it is working on iOS 17.

You could use search in already opened issues in the first place and we wouldn't have to hear stories about wasted 4 days.
There is literally a checkbox that you marked that you did search, while it is definitely a lie as it seems. Closing as another duplicate of other issues where it was explicitly mentioned that there will be a fix released on pub.dev and in the meantime you can use version from main.

I'm sorry to let you know that it's not a lie, and many people are affected by this issue.
As you can see the app started craching on 3 october, and I didn't fix the issue until yesterday, waiting for apple to accept the update.

@vbuberen I'm sorry to let you know that it's not a lie, and many people are affected by this issue.
As you can see the app started craching on 3 october, and I didn't fix the issue until yesterday, waiting for apple to accept the update.

I'm sorry to let you know that it's not a lie, and many people are affected by this issue.

Lie is not the issue, but that you tried to search the issue here in the repo before opening yours.
Doesn't matter as there will be a fix anyway.


fix(deps): update ember to 3.16.5

https://github.com/emberjs/ember.js/pull/18831

Did you do this with ember-cli-update?

I wanted to ask exactly the same question :sweat_smile:

I changed it manually. Running the cli now.

Done


Export all dates in one date file

Fetch available months
Fetch available dates for a month in JSON
Iterate over all availabe months and merge
Return as JSON
Return as date file (maybe a json converter, so it also works for just one month)

api should be simple, conversion is out of scope, represent the same functionality as the stine website


Moebius should use supervised connections

Today, Moebius starts a connection directly on the caller process:
https://github.com/robconery/moebius/blob/ba2bc6ad4e46af14e6946df293189212ca5f42c4/lib/moebius/runner.ex#L9-L30
This is must be avoided in Elixir. You want new process to always be part of a supervision tree, this way you can start :observer and access all processes holding database connections. Not only, you can do a simple Supervisor.count_children to know how many connections you have open at a given moment. In other words, supervision trees are extremely useful even if you don't have a restart strategy (i.e. all workers are temporary).
It is extremely important that we follow OTP conventions, otherwise many of the benefits we have been talking over and over about the VM and runtime will be lost (this was the disclaimer I was talking about on twitter).
Keep in mind this is not about pooling connections, handling transactions nor any of that. It is simply about following OTP principles. I thought the solution would be tiny but, since we want the connection to crash if the caller exits, we would need to either add this feature to potsgrex or support some sort of ownership model. Because I think this could be generally useful, I will talk to @fishcakez about making this part of db_connection so postgrex gets it for free soonish.

Adding a supervisor/use once pool, with the same pattern as :ssl, to db_connection is quite simple as @josevalim says. I will try to add it this week. Hopefully we could get postgrex using db_connection as soon as possible too, though I would like to add a per connection prepared query cache to db_connection first. It is hard to make good use of the prepare/execute pattern in postgrex master at present and it may change the API.

I have talked to @fishcakez and we have decided to add an :owner feature to db_connection. This way a simple one for one supervisor will be enough as long as we mark the current process as the owner.

@josevalim @fishcakez I've submitted PR https://github.com/robconery/moebius/pull/38 that implements the Runner as a GenServer and adding it to the Supervisor. I assume that this is similar to what this issue is requesting. We've actually been working through & debating a possible broker for the connection pooling for some time which included adding the Runner to the supervision tree. Once @robconery reviews/merges the PR, I believe we can close this issue

@nurugger07 sorry, that is not what we are proposing. Your pull request makes all database requests go through a single GenServer which will very quickly become a bottleneck in your application. It also introduces serious concurrency issues: if a request issues a transaction, that transaction will apply to all other requests in the system, since you are effectively sharing the connection.
Even worse, if a process starts a transaction and exits due to a link, the connection in the GenServer will be forever inside the transaction and all other operations from all other processes will happen inside the transaction that won't be committed until there is a crash.
I have mentioned this to @robconery but please don't reinvent pooling or brokering. I know I wouldn't. I only have confidence on the Ecto pooling implementation because of @fishcakez. It is extremely hard to get it right once there are transactions because the database state now becomes shared with multiple processes. That's also why @fishcakez is working on extracting it, so others can leverage it.
If you believe you should do your own brokering, please write an extensive test suite considering different scenarios with and without transactions.
Sorry for being emphatic about this but this is extremely important. We talk about how Elixir is great for writing robust and maintainable applications. If we drop OTP principles or introduce race conditions that locks a connection in a transaction, it goes directly against what we have been saying.
Here is what I would do. I would wait until Postgrex is ported to db_connection. From there on, you will have two options:

Have no pool. Instead you would start a new connection every time there is a need by using a simple one for one supervisor, quite similar to how you use it today, the only difference is the connection process is supervised. To implement this without the issues I mentioned above, we need to add an :owner feature to db_connection + postgrex.

Use poolboy or sbroker (pools) with db_connection.

From previous discussions, it looks like you want to go with 1 which is the simplest approach. I will gladly review the code too.

It is extremely hard to get it right once there are transactions because the database state becomes shared with multiple processes.

With db_connection we actually move all transaction state to one process. For postgrex we will be able to check that the local transaction state is in sync with the postgres server process (using a protocol value we currently have to ignore) so that we should crash if there is a bug in the logic.

Well, there are still two processes, the connection and the one that started the transaction. They are still sharing some state. :) Anyway, two is better than three, so :+1:.

@josevalim @fishcakez Yes! We would prefer the connection management to be offloaded :) It's something we've said from the beginning. It feels like the natural solution to ask for a connection from Postgrex, more so than managing connections in Moebius. Thanks @fishcakez for the work on db_connection!

This is done now.


Vanilla-Extract (CSS in TS) / Nuxt - HMR issues with SSR enabled

Environment

Operating System: Windows_NT
Node Version:     v19.8.1
Nuxt Version:     3.5.3
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      modules, neonUi
Runtime Modules:  normalizedModule(), @unocss/nuxt<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://github.com/Hecatron-Forks/vanilla-extract-example
Describe the bug
Hi,
I've been trying to use vanilla extract with nuxt, It's basically a CSS in Typescript library.
https://github.com/vanilla-extract-css/vanilla-extract
If I try and use it directly within nuxt then it seems to not do HMR updates when SSR is enabled by default (the default hybrid setting)
I originally raised an issue here
https://github.com/vanilla-extract-css/vanilla-extract/issues/1118
but I'm starting to think this might be more nuxt related.
I keep thinking I'm missing a nitro or other vite specific setting within nuxt to get things working
Since I'm working on my own component library anyways, what I'm currently doing is
bundling anything component related into a seperate package using vite, running pnpm dev on it
Then import that bundle into a nuxt with a seperate pnpm dev instance which picks up on the bundle / any changes to the bundle
But it would be nice If we could use it directly within nuxt as well with working HMR
Additional context
No response
Logs
No response

Hello @Hecatron
I found that this issue is caused by when nuxt running in ssr mode, it would create two vite server with shared plugin context, so if the plugin would retrieve vite server instance in configureServer to use later, the plugin must handle both vite server instance to keep hmr work fine.
After checking vanilla extract's vite plugin code here, here's the point that it would lose one of the two vite server in nuxt.
That's the reason why the hmr is okay for ssr: false but not work in ssr: true.


Added tests directory to ESLint targets

Current dependencies on/for this PR:

main

PR #269   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: kTQ4mYD9rj6bF9rtCZGW


quiche::Connection is not Send

I have a http3 server that uses quiche, and handles every request by spawning a new tokio task. But it seems like Rust won't let me send the connection (Arc-ed) over to the tokio task:
284 |                     tokio::spawn(async {
    |                     ^^^^^^^^^^^^ `*mut quiche::tls::SSL` cannot be sent between threads safely
    |
   ::: github.com-1ecc6299db9ec823/tokio-0.2.13/src/task/spawn.rs:123:21
    |
123 |         T: Future + Send + 'static,
    |                     ---- required by this bound in `tokio::task::spawn::spawn`
    |
    = help: within `quiche::Connection`, the trait `std::marker::Send` is not implemented for `*mut quiche::tls::SSL`

My question is two-fold:
1).  Is there a way to share a connection between tokio tasks when handling a request?
2). If not, what is the recommended method/pattern for handling requests asynchronously, especially for POST request with body data stream?

I found a workaround by using a different Async runtime. Closing this issue.


Error training on new docs

Getting this error out of the gate when trying to train on a single new .rst file. Any thoughts?
Traceback (most recent call last):
  File "/home/pshriwise/soft/doc-gpt/docsgpt/scripts/ingest_rst.py", line 36, in <module>
    store = FAISS.from_texts(docs, OpenAIEmbeddings(), metadatas=metadatas)
  File "/home/pshriwise/.pyenv/versions/3.9.1/lib/python3.9/site-packages/langchain/vectorstores/faiss.py", line 192, in from_texts
    index = faiss.IndexFlatL2(len(embeddings[0]))
IndexError: list index out of range

@pshriwise Could you provide the .rst file that causes the error?

Sure thing! https://github.com/openmc-dev/openmc/blob/develop/docs/source/index.rst

@pshriwise Hey Patrick,
If you haven't already, create a folder named "scikit-learn" inside the "scripts" folder and toss your index.rst inside of it. That should fix your problem.
You can really name the folder anything you would like, just make sure it matches the path in the ingest_rst.py and ingest_rst_sphinx.py files:
ps = list(Path("<FolderName>").glob("**/*.rst"))

Let me know if that works for you so I can close the issue. :)

It did. Thanks! I appreciate the help!

No problem! Glad to help.


digitalocean/rsyslog_exporter link is dead

https://github.com/prometheus/docs/blame/master/content/docs/instrumenting/exporters.md#L73

@taotetek do we need to remove that exporter or does it exist somewhere else now?

I reached out to DO per email. I guess we'll have to remove it, but let's wait 1-2 days in case they get back to us.


doxity init: The "listener" argument must be of type Function. Received type undefined

No ./scripts directory is created, exits immediately with trace below.
Node: v10.16.2
npm: 6.9.0
events.js:55
throw new errors.ERR_INVALID_ARG_TYPE('listener', 'Function', listener);
^
TypeError [ERR_INVALID_ARG_TYPE]: The "listener" argument must be of type Function. Received type undefined
at checkListener (events.js:55:11)
at process.removeListener (events.js:318:7)
at Object. (/usr/local/lib/node_modules/@digix/doxity/node_modules/truffle-compile/index.js:8:9)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Module.require (internal/modules/cjs/loader.js:692:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object. (/usr/local/lib/node_modules/@digix/doxity/lib/compile/solc.js:76:23)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Module.require (internal/modules/cjs/loader.js:692:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object. (/usr/local/lib/node_modules/@digix/doxity/lib/compile/index.js:44:13)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)

@7-of-9 , have you solved this issue? We need a tool to generate solidity docs as well.

it would be nice to have support of typescript or flow types, static type checkers can catch problems earlier

@7-of-9 , have you solved this issue? We need a tool to generate solidity docs as well.

Did you manage to come up with the solution?

I am having the same issue, using truffle and the problem comes from the truffle-compiler code.

This is still happening.

Still happening. Any workaround?

Hey! I downgrade doxity to ^0.4.0 in the package.json and now its working.


⚠ Spryr has degraded performance

In d324af3, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3016 ms

Resolved: Spryr performance has improved in 9c8e112 after 7 minutes.


adapted documentation to match instance change

Signed-off-by: Felix Engelmann<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master     #411   +/-   ##
=======================================
  Coverage   96.46%   96.46%
=======================================
  Files          17       17
  Lines        1103     1103
  Branches      129      129
=======================================
  Hits         1064     1064
  Misses         18       18
  Partials       21       21

Continue to review full report at Codecov.

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

Hi @felix-engelmann, thanks proposing this.
There's currently a conflict in your PR, would you mind rebasing it on current master and fixing the conflict?

Hi @albertodonato, thanks for the reminder. It should be mergeable now.


Create explore-intro.html

Create the French version of explore-intro.html

Have you tried building it in local? I get a 404 when I try to access the page with a local build.

Have you tried building it in local? I get a 404 when I try to access the page with a local build. But it seems to work here: https://deploy-preview-37937--kubernetes-io-main-staging.netlify.app/fr/docs/tutorials/kubernetes-basics/explore/explore-intro/

No, I built it on GitHub. Should I try doing it locally? Or can I delete the branch?


Add Class JavaDoc for oneOf and anyOf DTO's

Add Class JavaDoc for oneOf and anyOf DTO's to describe the intention of the class and how to use it.

This will get obsolete with the support of the combination of properties, additionalProperties and the compositions for object types in #76 , #99 and #100 .


Create class for salary

Proposed definition by @seljaseppala zhengj2007 in issue #62
salary: no definition (parent: IAO_0000027 data item)

We have a class in OMRSE 'income data' which is subclass of 'data item'. It has no definition yet. Do you see a distinction between income data and salary as a data item that we could capture?

Is it fair to assume this is meant to be restricted to only monetary gains, as opposed to in non-monetary consumption opportunity (e.g., rations or goods acquired through bartering)?

Our usage is restricted to monetary gains (i.e. salary/ wages).

proposed label = monetary personal income data item
proposed definition = A data item that is about the sum of earnings for some person(s) from wages, salary, or tips; interest generated by capital; receipt of rental payments; or profit from entrepreneurial activities.
comment = We use the phrase ‘some person’ in the definition because this class can be about an individual's or a household's earnings. Similarly, we use 'personal income' in the label because the term is typically restricted to individuals and households. This class is also restricted to being about monetary forms of income.

This works for VEuPathDB/ClinEpi.
Something to consider though is that enumerated lists aren't the best for definitions as it is usually not possible to be exhaustive and comprehensive. How about:
"A data item that is about the sum of earnings for some person(s) from various forms of payment or profits." Then use 'examples of usage' to list: wages, salary, or tips; interest generated by capital; dividends earned; lottery, game, or contest monetary winnings; alimony; royalties; trusts; pensions; receipt of rental payments; unemployment payments; social security payments; worker's compensation; social security payments; or profit from entrepreneurial activities.

We are good to move forward with the proposed change.


Wetek Play error

I compiled latest LibreELEC from Github to test this new addon, but I get the following error in log file:
21:27:23.446 T:2736620448   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.OSError'>
                                            Error Contents: [Errno 38] Function not implemented
                                            Traceback (most recent call last):
                                              File "/storage/.kodi/addons/plugin.video.netflix/addon.py", line 7, in <module>
                                                from resources.lib.KodiHelper import KodiHelper
                                              File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/KodiHelper.py", line 11, in <module>
                                                from MSL import MSL
                                              File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/MSL.py", line 11, in <module>
                                                import random
                                              File "/usr/lib/python2.7/random.py", line 885, in <module>
                                              File "/usr/lib/python2.7/random.py", line 97, in __init__
                                              File "/usr/lib/python2.7/random.py", line 113, in seed
                                            OSError: [Errno 38] Function not implemented
                                            -->End of Python script error report<--
21:27:23.454 T:3017418848   ERROR: GetDirectory - Error getting plugin://plugin.video.netflix/

I am not really sure, but looking at the affected source from python, this error looks way strange, because a fallback is implemented: https://github.com/python/cpython/blob/2.7/Lib/random.py#L115

Post a link to your entire log file - what specific version of LibreELEC/Kodi is this? Are you building LibreELEC from master or libreelec-8.2?

Here the entire log file: http://paste.thedevstack.de/view/1408621a
But as I could find so far, this seems to be a bug in Python 2.7: https://archlinuxarm.org/forum/viewtopic.php?f=60&t=11670

Oh and I build LibreELEC from master

I am building latest master with Python 2.7.12 to test it again.
But I think I will open an issue in LibreELEC.tv, because the problem is not specific to this addon.
As described in archlinuxarm forum python2.7 -c "import random" on shell is enough to get the error.

@lookshe Is it okay to close this here? I also think that this should be moved to the LibreElec Bugtracker.

Yes it's fine. I already opened a bug repot (https://forum.libreelec.tv/thread/9671-wetek-play-python-2-7-3-bug-perhaps-arm-related/) and testing the mentioned patch.


Update GitHub Actions Checkout to v3

No functional change intended.

The tests are failing. I'll retry them in a few days.


Fixed the display of all the cards as specified

@VG-1 I have fixed the styling of all the cards and separated the youtube and spotify cards.
In addition, I have also fixed some code by adding necessary closing tags

Hey @Gunika2002 thanks for contributing.
Can you just send a video or gif of code working properly?
Will be happy to merge pr

sure @VG-1 Here you go
https://user-images.githubusercontent.com/90132300/197393569-6e955ce4-02b1-4c7e-8ca4-68298910e9c6.mp4

seems cool.
thanks for working on it.
I really appreciate it.


update 8.1b

Checklist:

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Updating from v7.8 to v8.1b upstream
@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

This recipe is using a compiler, which now requires adding a build dependence on {{ stdlib("c") }} as well. Note that this rule applies to each output of the recipe using a compiler. For further details, please see https://github.com/conda-forge/conda-forge.github.io/issues/2102.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

I'll restart the process