New line from query data

I'm looking for a way to generate a new line using the values returned from a query.
For example:
Using the template:

Test

Test2

{{_value}}

where _value =
Test3

Test4

we get:
Test
Test2

Test3

Test4

Is there any mechanism to have this turn into:
Test
Test2
Test3
Test4

The query value doesn't have to use
 tags, I was just wondering if there is any support (or planned support) for some sort of new line to be available?

Thank you for your question (and your patience)!
I learned something new about Handlebars while looking into this. It turns out that Handlebars escape any HTML within a double brace expression, {{_value}}. If you instead use three braces, {{{_value}}}, instead, it works:

Let me know if that solves the issue you're having.

Thank you for your question (and your patience)!
I learned something new about Handlebars while looking into this. It turns out that Handlebars escape any HTML within a double brace expression, {{_value}}. If you instead use three braces, {{{_value}}}, instead, it works:

Let me know if that solves the issue you're having.


No bttv emtoes

There is no bttv emotes under button, but yesterday there were these emoticons.

Unfortunately it's not something I can do to fix--this is a BTTV issue if it's affecting more than you. The API for adding emotes to the emote menu haven't changed.
@Night I'll let you handle this one.

They work for me.. possible they failed to load them from the API so they weren't loaded in BetterTTV at all.

Fair enough--closing this.
@Marcin44 you can trying clearing cache and reloading to see if it helps. Other than that, can't really help--sorry!


Updated the repo URL in the package

Hi @skttl, I'm finally playing around with the Grid in v8, (it been a while) ... so, of course, I install DTGE! 😄
Once the package is installed, I noticed that the URL was still pointing at our UMCO repo, so I've updated it to point at your repo.
I've also put your name first in the authors list. :pray:
...and I've changed the copyright to be Umbraco Community for 2020 onwards.
If all that is cool?

Definitely is - thanks! :)


Add contains_non_numerical_values and similar methods to the tables class

Is your feature request related to a problem?
A method that checks if a table has non_numerical values could be useful.
Desired solution
The implementation should be fairly simple by using the remove_columns_with_non_numerical_values method.
Similar methods should be implemented for

contains_duplicate_rows
contains_rows_with_missing_values
contains_rows_with_outliers

too.
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response

We won't work on this. There are already methods to remove rows/columns using these criteria, as well as Table.summarize_statistics and various other methods, also on Column, to simply check them. Because of this, I don't think these methods are worth adding, particularly since the interface of Table is already fairly large.


etcd ACI signature should be armored instead of binary

According to ACI spec, Image archives SHOULD be signed using PGP, the format MUST be ascii-armored detached signature mode.
Up to etcd v2.3.6, .aci.asc file is a binary signature (despite the extension). As per https://github.com/coreos/rkt/issues/2581 rkt is currently a bit loose on that, but we plan to start warning on non-armored signatures.
Would you mind moving to armor signatures for ACIs? I think the "binary signature in .asc file" is also a bit confusing, but I'm not sure if it's there on purpose; would you like a small amendent/PR against your release guide?

@lucab
The release guide is here: https://github.com/coreos/etcd/blob/master/Documentation/dev-internal/release.md.
It would be helpful if you can send a PR.


Authorization and authentication issue for Google cloud machine learning engine.

My code.  I attached the Json file as well with service account details. I am getting authorization error.
.Net 4.6.1
Google Cloud Machine Learning API 1.30
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Google.Apis.Auth.OAuth2;
using Google.Apis.Services;
using System.IO;
using Google.Apis.CloudMachineLearningEngine.v1.Data;
using Google.Apis.CloudMachineLearningEngine.v1;
using System.Net.Http;
using System.Threading;
namespace ConsoleApp4
{
class Program
{
static void Main(string[] args)
{
        GoogleCredential cd;

        using (var stream = new FileStream("Really.json", FileMode.Open, FileAccess.Read))
        {
            cd = GoogleCredential.FromStream(stream);

            var service = new CloudMachineLearningEngineService(new BaseClientService.Initializer() {
            HttpClientInitializer=cd
            });

            if (cd.IsCreateScopedRequired)
            {
                cd.CreateScoped(CloudMachineLearningEngineService.Scope.CloudPlatform);
            }
            var list=service.Projects.Models.List("projects/currexchange-186113").Execute();

            //2nd method
            var completeUri = "projects/currexchange-186113/models/trainedcprr";
            string req = @"{""instances"":[{""CBRate"":9,""recievercurrency"":""HKD"",""sendingcurrency"":""EUR""}]}";

            var bodydummy = new GoogleCloudMlV1PredictRequest { HttpBody = new GoogleApiHttpBody() };
            var b = service.Projects.Predict(bodydummy, completeUri);
            Uri uri = b.CreateRequest().RequestUri;

            HttpContent content = new StringContent(req, Encoding.UTF8, "application/json");
            HttpResponseMessage predictResponse = service.HttpClient.PostAsync(uri, content).Result;

            //first method
            var httpBody = new GoogleApiHttpBody();
            httpBody.Data = req;
            httpBody.ContentType = "application/json";
            var body = new GoogleCloudMlV1PredictRequest()
            {
                HttpBody = httpBody
            };
            var a = service.Projects.Predict(body
               , completeUri);
            a.ModifyRequest = httpRequestMessage =>
            {
                httpRequestMessage.Content = new StringContent(req, Encoding.UTF8, "application/json");
            };
            var res = a.Execute();

        }
    }
}

}
Really.zip

Ah - I see the problem, I think. You're ignoring the return value of CreateScoped. Just change this:
cd.CreateScoped(CloudMachineLearningEngineService.Scope.CloudPlatform);

to this:
cd = cd.CreateScoped(CloudMachineLearningEngineService.Scope.CloudPlatform);

... and move the service construction to after that line.
Note that everything below your "var list" line is irrelevant to this issue - I would suggest you edit your post to remove it, to keep things as brief and clear as possible.

@jskeet Thank u so much. . its working now, I didnt assign changed credential with scope..


ConPTY inserts unexpected new line character

Microsoft Windows [Version 10.0.17763.316] ( also reproducible on 10.0.18841.1000 )

What you're doing and what's happening:
ConPTY MiniTerm sample receives extra new line character to display tmux bottom status bar.

What's wrong / what should be happening instead:
ConPTY should process input VT sequences without inserting new line character.
The screenshot of the same input, but inside MSYS2 terminal emulator:

Steps to reproduce:

Build console CPP app to read input data from file and send to standard output
main.txt
File with input data input.txt to be placed in the same directory with console app executable from previous step
Build adjusted version of MiniTerm https://github.com/MaxRis/console/commit/82f36fd39b4fc7e12752d226dd5d70c0b3159824 , run it, launch executable from first step.

Additional info:

Similar issue is reproducible with ConEmu. Extra new line is added after tmux status line:

Following conclusion made on top of following observations might be important:
Changing SGR command length has straightforward impact on position where the extra new line character is generated by ConPTY

ConPTY might has an issue to process SGR command and count symbols related to the SGR command as output terminal emulator buffer symbols. Since the length of line is 130 symbols in above case, ConPTY inserts new line character every time when total count of SGR command symbols and visible output buffer symbols exceeds 130 symbols.
Here it's simplified input data used to generate above output with SGR command of different length
input.txt

Hey, about this "ConsolePrinter": is it turning on ENABLE_VIRTUAL_TERMINAL_PROCESSING?
If it isn't, something fun is happening with your VT sequences. When VT processing is disabled, those VT sequences are being treated as raw characters to display in the console. Since those characters are being displayed, they're counted towards the width of the output.
When you're connected over ConPTY, two strange (but by-design) things happen:

Those raw escape sequences are line-wrapped, because you didn't tell conhost that it should process them
The ConPTY consumer receives those sequences and then parses them. Since they may have been broken by the line wrapping ro any other console screen buffer manipulation, they will be rendered incorrectly.

In short, if you're not setting ENABLE_VIRTUAL_TERMINAL_OUTPUT but you are in fact using virtual terminal output, this is by design.

Hey @DHowett-MSFT thank you for looking into this!
ENABLE_VIRTUAL_TERMINAL_PROCESSING is set for MiniTerm.exe https://github.com/Microsoft/console/blob/master/samples/ConPTY/MiniTerm/MiniTerm/Terminal.cs#L36 and I have believed that the console instance and custom set attributes are shared with launched CMD.exe and further with ConsolePrinter.exe process. Does it make any sense here?
ConsolePrinter doesn't modify console mode and just writes to standard output:
#include <iostream>
#include <fstream>
#include <string>

void main() {
	std::ifstream dataFile("input.txt");
	if (dataFile.is_open()) {
		std::string dataLine;
		while (std::getline(dataFile, dataLine)) {
			std::cout << dataLine;
		}
	}
}

Ah, so the code in MiniTerm is enabling VT processing for the terminal itself. Since ConsolePrinter is on the other side:
+-------+                      +-------+
| Mini  | <----------(pipe)--- | pty   |
|  Term | -----------(pipe)--> |  host | - vt processing OFF.
+-------+                      +-------+
  |   ^                           |  ^
  v   |                           v  |
+-------+                      +---------+
| con   |                      | Console |
|  host | - VT processing on.  | Printer |
+-------+                      +---------+
  |   ^
  v   |
/ - - - +
: user  :
:       :
+ - - - /

it needs to enable virtual terminal processing for its console host.

@DHowett-MSFT thanks! will give it a try!

@DHowett-MSFT thank you for finding out the source of issue!
If I'm extending ConsolePrinter with something like following:
        HANDLE hOutput = GetStdHandle(STD_OUTPUT_HANDLE);
	DWORD dwMode;
	GetConsoleMode(hOutput, &dwMode);
	dwMode |= ENABLE_PROCESSED_OUTPUT | ENABLE_VIRTUAL_TERMINAL_PROCESSING;
	if (!SetConsoleMode(hOutput, dwMode)) {
		exit(1);
	}

Everything is rendered correctly, even under ConEmu! Thanks!

Ah, so the code in MiniTerm is enabling VT processing for the terminal itself. Since ConsolePrinter is on the other side:
+-------+                      +-------+
| Mini  | <----------(pipe)--- | pty   |
|  Term | -----------(pipe)--> |  host | - vt processing OFF.
+-------+                      +-------+
  |   ^                           |  ^
  v   |                           v  |
+-------+                      +---------+
| con   |                      | Console |
|  host | - VT processing on.  | Printer |
+-------+                      +---------+
  |   ^
  v   |
/ - - - +
: user  :
:       :
+ - - - /

it needs to enable virtual terminal processing for its console host.

These are the kinds of notes that belong on Docs.

Hey, about this "ConsolePrinter": is it turning on ENABLE_VIRTUAL_TERMINAL_PROCESSING?
If it isn't, something fun is happening with your VT sequences. When VT processing is disabled, those VT sequences are being treated as raw characters to display in the console. Since those characters are being displayed, they're counted towards the width of the output.
When you're connected over ConPTY, two strange (but by-design) things happen:

Those raw escape sequences are line-wrapped, because you didn't tell conhost that it should process them
The ConPTY consumer receives those sequences and then parses them. Since they may have been broken by the line wrapping ro any other console screen buffer manipulation, they will be rendered incorrectly.

In short, if you're not setting ENABLE_VIRTUAL_TERMINAL_OUTPUT but you are in fact using virtual terminal output, this is by design.

This is the kind of information that belongs on Docs, along with all of the PseudoConsole stuff.

Please file a docs bug at https://github.com/MicrosoftDocs/Console-Docs. Feel free to make a branch and add suggested text to a pull request if you have a good idea of what you think it should say.

Inspired from @DHowett-MSFT's ascii art. Does ConPTY work with Duplex Named Pipe? As following:
+-------+                      +-------+
| Mini  |                      | pty   |
|  Term | <-------(pipe)-----> |  host | - vt processing OFF.
+-------+                      +-------+
    ^                              ^
    |                              |
    v                              v
+-------+                      +---------+
| con   |                      | Console |
|  host | - VT processing on.  | Printer |
+-------+                      +---------+
    ^
    |
    v
/ - - - +
: user  :
:       :
+ - - - /

Give it a shot!


Add notifications about PRs to Slack

Description
Adds notifications about PRs to Slack
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have tested this code
[ ] I have updated the README and other relevant documents (guides...)
[ ] I have added sufficient documentation both in code, as well as in the READMEs

It should notify about comments also

It looks like we don't need this after all.


Fix MetroWindow dialog behavior

If you have a MetroWindow and want to display this from an other window by using:
otherWindow.ShowDialog();

There is one issues with the dialog behavior:
If you click on the parent window it doesn't react as spectated, but you shouldn't be able to switch back to the parent window by using the Taskswitcher.

Do you mean it doesn't review focus ?

It does receive the focus, but you can switch to the parent window by using Alt + TAB and that's not the dialog behavior I expect.

@moser96 maybe you forgot to set the Owner property

This is the actual behavior , try messagebox.show
And it will also have the same behavior
If you want your behavior you can do windowinstance.Show()
And implement your own functions to get the result

@punker76  THX, I totally forgot it! Now it works as expected :smile:


Improved Autognome Armored Casing AC calculation

Is your feature request related to a problem? Please describe.
Each time I run the importer on my Autognome character I find that my AC always returns to my "natural" unarmored state of 13 + dex despite also having armor equipped. I am currently solving this by changing system.attributes.ac.formula from 10 + 3 + @abilities.dex.mod to max(10 + 3, @attributes.ac.armor) + @abilities.dex.mod, but each time I run the import it is overridden
Describe the solution you'd like
I'd like the importer to take into account whether my Autognome is wearing armour, and use the higher of the two values to calculate the AC
Describe alternatives you've considered
If there is a way to prevent this being overridden when importing by unchecking some of the boxes, please let me know
Additional context

Thanks, this fix will be in the next release.


Add threshold

I would like to be able to not trigger onStart unless the user has dragged more than 20 pixels (this should be configurable).

Happy to accept a PR for this.
On May 14, 2015 3:09 PM, "Stephen Demjanenko"<EMAIL_ADDRESS>wrote:

I would like to be able to not trigger onStart unless the user has dragged
more than 20 pixels (this should be configurable).
—
Reply to this email directly or view it on GitHub
https://github.com/mzabriskie/react-draggable/issues/54.

See the implementation of the grid property for some hints on how to do this. If you return false from your onDrag handler, the element won't move.


Consider JSON as pass-thru

JSON mapping is typically an application-level concern (JSON-P, JSON-B) because all transformations and bindings are reflected on application or client-level. Having support directly in the driver comes with several drawbacks as drivers need to either commit to a single library or provide support for multiple implementations (Jackso, GSON, Apache Johnzon). This isn't typically the thing you want to deal with.
Also, most JSON libraries work with blocking streams (InputStream, OutputStream) that is not Reactive-friendly.
Another approach would be to provide an extensible codec registry and optional codecs that users can register.

Ok, I will remove JSON support and type JSON will be treated as a VARCHAR.

JSON support has removed. See StringConverter in io.github.mirromutth.r2dbc.mysql.converter.


Nix NSS name service (libnss_8.so) is incompatible with Debian/Ubuntu

Describe the bug
I'm unable to retrieve my user details with getent in a Nix standalone installation on Ubuntu.
Steps To Reproduce
$ /nix/store/k20hh5k7b4ll7ll5j0x72z3d6f39zazq-getent-glibc-2.37-8/bin/getent passwd <user>

$ /usr/bin/getent passwd <user>
<user>:x:<uid>:<gid>:<name>,:/home/<user>:/bin/bash

I do have nscd running:
sudo systemctl status nscd
● nscd.service - Name Service Cache Daemon
     Loaded: loaded (/lib/systemd/system/nscd.service; enabled; vendor preset: enabled)
     Active: active (running) since Thu 2023-04-20 10:49:44 AEST; 6 days ago
   Main PID: 851 (nscd)
      Tasks: 8 (limit: 38131)
     Memory: 1.1M
     CGroup: /system.slice/nscd.service
             └─851 /usr/sbin/nscd

Apr 20 10:49:44 gillman-tv systemd[1]: Starting Name Service Cache Daemon...
Apr 20 10:49:44 gillman-tv nscd[851]: 851 monitoring file `/etc/services` (1)
Apr 20 10:49:44 gillman-tv nscd[851]: 851 monitoring directory `/etc` (2)
Apr 20 10:49:44 gillman-tv systemd[1]: Started Name Service Cache Daemon.

More info
Tracing through the files of each of the commands with strace:
In system shell:
$ strace  getent passwd gil2a4 2>&1 | grep open
openat(AT_FDCWD, "/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/usr/lib/locale/locale-archive", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/etc/nsswitch.conf", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/libnss_files.so.2", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/etc/passwd", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/libnss_db.so.2", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/libdb-5.3.so", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/libpthread.so.0", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/sys/devices/system/cpu/online", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/var/lib/misc/DB_CONFIG", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/var/lib/misc/passwd.db", O_RDONLY) = 3
openat(AT_FDCWD, "/var/lib/misc/passwd.db", O_RDONLY) = 3

In NIx shell:
$ strace  getent passwd gil2a4 2>&1 | grep open
openat(AT_FDCWD, "/nix/store/1n2l5law9g3b77hcfyp50vrhhssbrj5g-glibc-2.37-8/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/1n2l5law9g3b77hcfyp50vrhhssbrj5g-glibc-2.37-8/lib/glibc-hwcaps/x86-64-v3/libc.so.6", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/1n2l5law9g3b77hcfyp50vrhhssbrj5g-glibc-2.37-8/lib/glibc-hwcaps/x86-64-v2/libc.so.6", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/1n2l5law9g3b77hcfyp50vrhhssbrj5g-glibc-2.37-8/lib/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/nix/store/4isa2k1vj05g0hpc5wxjqacb0fq9dg08-glibc-locales-2.37-8/lib/locale/locale-archive", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/etc/nsswitch.conf", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/etc/passwd", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/nix/store/1n2l5law9g3b77hcfyp50vrhhssbrj5g-glibc-2.37-8/lib/libnss_db.so.2", O_RDONLY|O_CLOEXEC) = 3
openat(AT_FDCWD, "/var/db/passwd.db", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)

The issue seems to be that the passwd.db file location is hard-coded, on Debian systems it is at /var/lib/misc/passwd.db, but on other systems it is at /var/db/passwd.db.
I guess this is what nscd is supposed to solve, but it seems to be ignored. I'm not sure if nscd is actually monitoring /var/run/nscd/socket, it doesn't seem to be referenced above by strace.
nix-env --version output
nix-env (Nix) 2.13.3
nixpkgs https://nixos.org/channels/nixos-22.11
Additional context
I'm having this issue trying to get Emacs to find my home directory, https://github.com/NixOS/nixpkgs/issues/12335
Having nscd running is the proposed solution, but doesn't seem to be working for me.
Maybe relevant? https://github.com/NixOS/nixpkgs/issues/1868

Thanks @flokli
Yes, /var/run/nscd/socket exists.
FYI, I'm on Ubuntu not NixOS.
Using Ubuntu's getent passwd $USER
20:53:06.871 >>> process getent [3979723 -> 851] len 19(19)
0000  02 00 00 00 0b 00 00 00  07 00 00 00 70 61 73 73  ............pass
0010  77 64 00                                          wd.
20:53:06.871 >>> process getent [3979723 -> 851] len 19(19)
<redacted>
20:53:06.872 >>> process nscd [851 -> 3979723] len 36(36)
<redacted>

The actual output of that command, getent passwd $USER, was:
<user>:x:521549:521549:<name>,:/<home>:/bin/bash
And using nix shell nixpkgs#getent, getent passwd $USER
20:54:00.775 >>> process getent [3980326 -> 851] len 19(19)
0000  02 00 00 00 0b 00 00 00  07 00 00 00 70 61 73 73  ............pass
0010  77 64 00                                          wd.
20:54:00.776 >>> process getent [3980326 -> 851] len 19(19)
<redacted>
20:54:00.776 >>> process nscd [851 -> 3980326] len 36(36)
<redacted>

The actual output of that command, ``getent passwd $USER`, was blank.
To me, it looks like both native and Nix glibs are using the socket, but only the native one is getting the right information.

Unfortunately the data of the request and response was redacted.
Did you spot any differences in the initial handshake, or request sent from getent?

Yes sorry, I'm not sure exactly what the information is so I redacted it.
But the bytes in both the handshake and the response are the same.

I have no understanding of nscd, is the last line to response to a request that the original process would receive? I guess this would be consistent with the nscd daemon not running correctly and always giving a blank response (so a host error, not a Nix problem)?

(If not a Nixpkgs problem, apologies for using your time but thank you very much for teaching me some socket debugging tools - Google search didn't find me any useful tooks like sockdump)

I didn't fully dig into the responses, but if nscd is also not able to use the host NSS modules, it looks like there's some misconfiguration on the host.
glibc code falls back to trying to resolve locally (but can't for nix-built binaries), which might be why your /usr/bin/getent might work, but the nix-built getent does not.
Please make sure your nscd can properly read /etc/nsswitch.conf, and the nss modules specified in there are reachable via a dlopen of libnss_$name.so (so in one of the rpaths, or LD_LIBRARY_PATH - you should see what's going on by strace-ing nscd.

Thanks very much, your help is much appreciated. I'll close this now as I don't believe to to be a Nixpkgs issue.


docs(geojson): add export default to example

Does any other open PR do the same thing?

No
What issue is this PR fixing?
Help new developers to be able to copy/paste the GeoJson quickstart exemple. Exemple on StackOverflow.
How did you test this PR?

No

Hello @devpolo 👋
Any chance you can fix the formatting and make the CI green?


Delete declaration of InlineCache::CacheMissing -- never defined or used

ChakraCore and ChakraFull build-branch builds are complete and clean, so I have high confidence that this method declaration is dead code.


linux_5_2 not reproducible

Describe the bug
Building linux_5_2 locally on a 19.03 install (and I presume the same is true for 19.09 and master, but have not tested it yet) the resulting outputs differ from what is in the binary cache and result in a checksum mismatch when run with --check:
error: derivation '/nix/store/dcy1vplvfwjbckhvfrk8dqv973sm7mh3-linux-5.2.14.drv' may not be deterministic: output '/nix/store/im1y2sf411940gyn1x59rpfbw1k508b3-linux-5.2.14' differs from '/nix/store/im1y2sf411940gyn1x59rpfbw1k508b3-linux-5.2.14.check'

cc @grahamc due to previous convo on IRC.
To Reproduce
Steps to reproduce the behavior:

nix-build -k -K '<nixpkgs>' -A pkgs.linux_5_2 --check

Expected behavior
A local build without modifications of flags or config should result in the same outputs.
Additional context
I ran diffoscope on the two resulting store locations. Here is a shortened output, where I cut out ~39MB of .rodata. If needed, I can upload the full dump.
https://gist.github.com/d-goldin/12c7cd9998ed9d8f31ca5792a6b5aa69#file-shortened_diffoscope
I tried this for a few different patch versions of 5.2 and so far they all seem to be affected, or I missed something.
Metadata
$ nix run nixpkgs.nix-info -c nix-info -m
 - system: `"x86_64-linux"`
 - host os: `Linux 5.2.14, NixOS, 19.03.173506.2dfae8e22fd (Koi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.2.2`
 - channels(root): `"nixos-19.03.173506.2dfae8e22fd, home-manager-19.03"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: pkgs.linux_5_2
# a list of nixos modules affected by the problem
module:

This is also the case for linux_5_3 with very similar looking diffs.
It looks like it has to do with a new mechanism introduced somewhere in for v5 kernels: https://lwn.net/Articles/783581/

I think I identified the exact source of nondeterminism. Currently trying out a small patch locally.

Closing because for NixOS the issue is addressed. There is a little bit more work to be done upstream to support older environments, but that's not relevant for us.


ARK Server wipes DefaultGameUserSettings.ini settings when updated

when the server updates it wipes the DefaultGameUserSettings.ini file removing settings for added mods. this breaks the server meaning users have to start again. using the app_update 376030 validate option should only be used on first installation after that app_update 376030 should be used to update script

You shouldn’t be modifying that file. That is just the template that is copied to GameUserSettings.ini when a server is created. You should be putting your stuff in the latter file, which is in arkSE/376030/ShooterGame/Saved/Config/LinuxServer/ on Linux and arkSE\376030\ShooterGame\Saved\Config\WindowsServer\ on Windows

You shouldn’t be modifying that file. That is just the template that is copied to GameUserSettings.ini when a server is created. You should be putting your stuff in the latter file, which is in arkSE/376030/ShooterGame/Saved/Config/LinuxServer/ on Linux and arkSE\376030\ShooterGame\Saved\Config\WindowsServer\ on Windows

Thats where The line ActiveMods= goes to allow mods to pass through
On Fri, 8 Jan 2021, 10:26 Greelan<EMAIL_ADDRESS>wrote:

You shouldn’t be modifying that file. That is just the template that is
copied to GameUserSettings.ini when a server is created. You should be
putting your stuff in the latter file, which is in
arkSE/376030/ShooterGame/Saved/Config/LinuxServer/ on Linux and
arkSE\376030\ShooterGame\Saved\Config\WindowsServer\ on Windows
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756678225, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZENUFWHVM3YUDU4UHTSY3MUBANCNFSM4VZOVDSQ
.

Thats where The line ActiveMods= goes to allow mods to pass through
On Fri, 8 Jan 2021, 10:26 Greelan<EMAIL_ADDRESS>wrote:

You shouldn’t be modifying that file. That is just the template that is
copied to GameUserSettings.ini when a server is created. You should be
putting your stuff in the latter file, which is in
arkSE/376030/ShooterGame/Saved/Config/LinuxServer/ on Linux and
arkSE\376030\ShooterGame\Saved\Config\WindowsServer\ on Windows
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756678225, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZENUFWHVM3YUDU4UHTSY3MUBANCNFSM4VZOVDSQ
.

In GameUserSettings.ini, not DefaultGameUserSettings.ini which is in a different directory

In GameUserSettings.ini, not DefaultGameUserSettings.ini which is in a different directory

See the ARK wiki: https://ark.gamepedia.com/Server_Configuration#Configuration_Files
You shouldn't be touching DefaultGameUserSettings.ini (or DefaultGame.ini)

See the ARK wiki: https://ark.gamepedia.com/Server_Configuration#Configuration_Files
You shouldn't be touching DefaultGameUserSettings.ini (or DefaultGame.ini)

Thats the file i use to mod, in any instance using the validate method
wipes settings applied. Because it verifies it against the steam
installation
On Fri, 8 Jan 2021, 10:51 Greelan<EMAIL_ADDRESS>wrote:

See the ARK wiki:
https://ark.gamepedia.com/Server_Configuration#Configuration_Files
You shouldn't be touching DefaultGameUserSettings.ini (or DefaultGame.ini)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756689912, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZETSFSV26EPZMF7YJ3SY3PSTANCNFSM4VZOVDSQ
.

Thats the file i use to mod, in any instance using the validate method
wipes settings applied. Because it verifies it against the steam
installation
On Fri, 8 Jan 2021, 10:51 Greelan<EMAIL_ADDRESS>wrote:

See the ARK wiki:
https://ark.gamepedia.com/Server_Configuration#Configuration_Files
You shouldn't be touching DefaultGameUserSettings.ini (or DefaultGame.ini)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756689912, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZETSFSV26EPZMF7YJ3SY3PSTANCNFSM4VZOVDSQ
.

Closed as user error, Greenlan is right - you're modifying the 'default' files instead of making new ones.

Closed as user error, Greenlan is right - you're modifying the 'default' files instead of making new ones.

I would suggest you guys go make a private server via steamcmd and add mods
and see where the information goes. I ran into this issue with a auto
update system i created on server 2012 with the validate option set. If
either of you would like to see my setup i would gladly let you see my
server with the setup and how the validate setting effects mods and
corrupts the server instance. Or if there is a way to i can show you how it
effects it i would gladly do that
On Fri, 8 Jan 2021, 13:28 PhonicUK<EMAIL_ADDRESS>wrote:

Closed as user error, Greenlan is right - you're modifying the 'default'
files instead of making new ones.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756755653, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZFEBMRZRBJCKMF42VTSY4CBHANCNFSM4VZOVDSQ
.

I would suggest you guys go make a private server via steamcmd and add mods
and see where the information goes. I ran into this issue with a auto
update system i created on server 2012 with the validate option set. If
either of you would like to see my setup i would gladly let you see my
server with the setup and how the validate setting effects mods and
corrupts the server instance. Or if there is a way to i can show you how it
effects it i would gladly do that
On Fri, 8 Jan 2021, 13:28 PhonicUK<EMAIL_ADDRESS>wrote:

Closed as user error, Greenlan is right - you're modifying the 'default'
files instead of making new ones.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756755653, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZFEBMRZRBJCKMF42VTSY4CBHANCNFSM4VZOVDSQ
.

https://www.akliz.net/manage/knowledgebase/66/How-to-Install-ARK-Survival-Evolved-Mods-on-a-Server.html
here is the link on server mod installations
On Fri, Jan 8, 2021 at 1:33 PM Safe Dave<EMAIL_ADDRESS>wrote:

I would suggest you guys go make a private server via steamcmd and add
mods and see where the information goes. I ran into this issue with a auto
update system i created on server 2012 with the validate option set. If
either of you would like to see my setup i would gladly let you see my
server with the setup and how the validate setting effects mods and
corrupts the server instance. Or if there is a way to i can show you how it
effects it i would gladly do that
On Fri, 8 Jan 2021, 13:28 PhonicUK<EMAIL_ADDRESS>wrote:

Closed as user error, Greenlan is right - you're modifying the 'default'
files instead of making new ones.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756755653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZFEBMRZRBJCKMF42VTSY4CBHANCNFSM4VZOVDSQ
.

https://www.akliz.net/manage/knowledgebase/66/How-to-Install-ARK-Survival-Evolved-Mods-on-a-Server.html
here is the link on server mod installations
On Fri, Jan 8, 2021 at 1:33 PM Safe Dave<EMAIL_ADDRESS>wrote:

I would suggest you guys go make a private server via steamcmd and add
mods and see where the information goes. I ran into this issue with a auto
update system i created on server 2012 with the validate option set. If
either of you would like to see my setup i would gladly let you see my
server with the setup and how the validate setting effects mods and
corrupts the server instance. Or if there is a way to i can show you how it
effects it i would gladly do that
On Fri, 8 Jan 2021, 13:28 PhonicUK<EMAIL_ADDRESS>wrote:

Closed as user error, Greenlan is right - you're modifying the 'default'
files instead of making new ones.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756755653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZFEBMRZRBJCKMF42VTSY4CBHANCNFSM4VZOVDSQ
.

I have. And I wrote this wiki (https://github.com/CubeCoders/AMP/wiki/Adding-mods-to-an-ARK:-Survival-Evolved-server) to help document it for AMP users, particularly for download automation.
And the link you have provided confirms what I have been saying

I have. And I wrote this wiki (https://github.com/CubeCoders/AMP/wiki/Adding-mods-to-an-ARK:-Survival-Evolved-server) to help document it for AMP users, particularly for download automation.
And the link you have provided confirms what I have been saying

Alright maybe i was wrong gents. Not fully used to your systems yet. I
retract my post. I do apologise
On Fri, 8 Jan 2021, 13:39 Greelan<EMAIL_ADDRESS>wrote:

I have. And I wrote this wiki (
https://github.com/CubeCoders/AMP/wiki/Adding-mods-to-an-ARK:-Survival-Evolved-server)
to help document it for AMP users, particularly for download automation.
And the link you have provided confirms what I have been saying
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756760196, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZH46UD36X5DT7B6KD3SY4DIZANCNFSM4VZOVDSQ
.

Alright maybe i was wrong gents. Not fully used to your systems yet. I
retract my post. I do apologise
On Fri, 8 Jan 2021, 13:39 Greelan<EMAIL_ADDRESS>wrote:

I have. And I wrote this wiki (
https://github.com/CubeCoders/AMP/wiki/Adding-mods-to-an-ARK:-Survival-Evolved-server)
to help document it for AMP users, particularly for download automation.
And the link you have provided confirms what I have been saying
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CubeCoders/AMP/issues/387#issuecomment-756760196, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRCVZH46UD36X5DT7B6KD3SY4DIZANCNFSM4VZOVDSQ
.


dd does not delete trailing lines

If you are at the end of the file and you have a few empty lines, "dd" or forms thereof will not delete those extra lines.

+1
Disabling whitespace package doesn't help either.
I'll dig into this later today.
I'm fairly new to this package so it'd be awesome if any maintainer could give me some pointers.

+1

Closing this out in favor of #801 since that issue has more detail. Please subscribe to that one for updates.


feat: add installation by BT panel

Signed-off-by: killer<EMAIL_ADDRESS>

@oranges-eating the docs style is not good. Let's hear some suggestions from docs experts. @ErikQQY @Kininaru

@Kininaru we will have all:

Docker
BTPanel
Native way

Since Docusaurus provide so much docs features, we can fully utilize them to make our docs more pretty.
Here are some suggestions:

Use different headings to divide the tutorial into many sections to make the tutorial more methodical

I think put some space between different paragraphs would be great

We can specify different languages to highlight the syntax to make the code block more readable

Emphasize some important key word such as usr/local

Change some press key to specially highlight such as Esc and i

@ErikQQY How is it now

Looks great!!


Remote Cluster configured with AutoScalling. Pod not getting a scheduled in remote cluster

What happened:
Our Remote Cluster is created using Azure VM scale set with auto scaling features for GPU based nodes and default to 0 nodes.
I was able to install Liqo in both Local and remote cluster and peering between the cluster is successful. But when I describe the virtual node capability has 0 GPU’s.
How can I configure virtual node with arbitrary resources, as by default remote cluster doesn’t have static nodes.
What you expected to happen:
Should be able to configure virtual node with arbitrary resources so local cluster can schedule pods to remote cluster.
How to reproduce it (as minimally and precisely as possible):
Create remote cluster with 0 GPU worker nodes and try to schedule pod in local cluster with GPU.
Anything else we need to know?:
Environment:

Liqo version: 0.5
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration: onpremise and azure
Network plugin and version:
Install tools:
Others:

By default, liqo characterizes virtual nodes with an amount of resources that is a percentage of the free resources available in the target cluster. Still, it supports the possibility of delegating the computation of the resources offered to a given peer to an external component, which might operate based on any logic of interest (ranging from returning a fixed amount, as it might be suitable in your scenario, to performing complex computations for brokering use-cases).
Practically speaking, this requires developing a component implementing the server side of the resource reader gRPC interface, and configuring the liqo controller manager to delegate requests to it, rather that performing the default computation (i.e., setting the --external-monitor=endpoint flag). To this end, there is an ongoing effort to better document this liqo functionality, as well as provide a skeleton of the component based on a simple use-case, ready to be extended for more complex scenarios.

Hey @Sharathmk99,
I just wanted to say that we recently improved the Liqo functionality which enables the possibility of delegating to an external plugin the computation of the amount of resources offered to a given peer; specifically, Liqo v0.6.1 also includes better documentation about this feature.
In addition, we created a new repository (https://github.com/liqotech/liqo-resource-plugins/), serving both as a collection of already existing plugins, and as a guideline to develop new ones. Currently, it includes the fixed-resources plugin, which allows you to manually override the amount of resources offered with arbitrary values (e.g., accounting for autoscaling, as in your use-case).
I'm closing this issue for the moment, since I think the above addresses your requirements. Feel free to reopen it if you feel something is still missing, or not working as you would expect.


Admin can update an existing question

Blocked by #101
Done when
Admin can view the form for an existing question definition and submit it to update the question definition

wireframes : https://miro.com/app/board/o9J_lXULxwE=/?moveToWidget=3074457354353430975&cot=14

M1 Review - In progress


Update to ilib build 20150729-build-11.0-005-01

Update to ilib build 20150729-build-11.0-005-01. The ilib-webapp  part  is already submitted  to the Build. (master, drd4tv)

Fixed missing dependency in Currency.js
Fixed some closure compiler warnings
Calendar.newDateInstance() is now deprecated, as it creates a circular dependency. This call will go away in release 13.0 or 14.0. The alternate way of creating a date for a particular calendar is to call the date factory directly instead of having the calendar classes do it for you:

Before:
var d = cal.newDateInstance();
After:
var d = DateFactory({calendar: cal.getType()});
Souped up the jsdocs a bit

Fixed bug where creating a new ResBundle would throw an exception if there were no arguments to the constructor AND the current locale was a pseudo locale.

@aarontam @gouniLee - Could you check this? Current version of onDevice-ilib is 11.0.005 so we need to sync both of ilib.

@kyo504 Is this needed for pre.15.1?

If container uses onDevice ilib, we don't have to but if not, this needs to be merged pre.15.1

@JayCanuck, are we currently using the on-device iLib in the build (see @kyo504's note above)?

We are currently using on-device ilib with container, yes.

This has a circular dependency. This was fixed with https://github.com/enyojs/iLib/pull/34.  Please rebase to that commit.

I've added https://github.com/enyojs/iLib/pull/34 patch.
fyi, ilib-webapp for webOS TV, it's already applied with version "build 20150814-build-11.0-005-02" and merged to drd4tv_60.

We the fix for circular dependency added, we can probably remove enyo-ilib/lib/patch.js eh? @ryanjduffy

Ya. Let's open a separate ticket for that and we can revert that commit 328bce3d1c76348f93cc45ebefc63a5a71095bb7


Fastboot compatibility

Adding Fastboot compatibility.

I'm willing to accept this, but I wonder if @tomdale has better recommended approaches.  Looking up the service seems heavier than a typeof FASTBOOT !== 'undefined' check, which I've seen around a few places.

@runspired  The FastBoot service and isFasBoot is the official public API for determining if the application is in FastBoot mode.
window.FastBoot is exposed as a config object inside the server side context, but that's private API and could change. I'll defer to Tom though.

You can check out the work we've been doing with Ember Simple Auth to see how we are handling it over there.
https://github.com/simplabs/ember-simple-auth/pull/964

@arjansingh I see. At a minimum I'd like to know why the service is being looked up in such a non-ember way.

Specifically:
Why something like this?
fastboot: computed(function() {
   let owner = getOwner(this).lookup('service:fastboot');
}),

Instead of this?
fastboot: inject.service('fastboot')

Because if an application is not requiring ember-cli-fasboot as a dependency, the injection will throw an error since the service won't exist.  You can use Ember.inject.service but then you will have to make ember-cli-fasboot a required peer dependency for anyone using your addon.

@arjansingh thanks! This all makes much more sense now.
@anthonybalmeo thanks again for doing the legwork here :)

Believe me. I wish there was a cleaner way too. 😀 I'll let you know when I find one.

@runspired thanks!

Many months ago this was not an issue, and injecting a service does not actually call init() so wondering if there is more context anyone could provide (at the least for the knowledge sharing sake).
This seems superfluous I guess IMO... Fastboot doesn't have an event dispatcher and it seems odd it would invoke such a code path. Anyone care to explain more if they have time. Likely a misunderstanding on my part so figured I'd ask the obvious.

Can you create a release for this? Thanks!

@anthonybalmeo - window.Hammer is only invoked here when rendering in fastboot (within init) - https://github.com/runspired/ember-gestures/blob/31ce7c74bff5eadbfdfa7bcf4c4539ae85e310fa/addon/mixins/recognizers.js#L88
Hammer.js should not exist though anyway b/c of this guard I put in a while back within the build - https://github.com/runspired/ember-gestures/blob/31ce7c74bff5eadbfdfa7bcf4c4539ae85e310fa/index.js#L17-L24
To explain,  didInsertElement is not called when rendering in fastboot. However init is called (first link above). The guard should be placed there if we're going to guard the world for fastboot - however i propose that a better way to handle this is for ember-gestures to only be included in the broccoli tree when outputting a browser build for an app consuming this addon (@tomdale confirm/deny?), and for it to not be included when outputting a fastboot build.
ember-network is a good example of this sort of branching - https://github.com/tomdale/ember-network/blob/2826247c5dd2bb2537497af5ad7fecbd3feba6c9/index.js#L49-L55
This way of handling fastboot shouldn't be a problem given ember-hammertime handles the ast transform of the rendered html from the server thus still providing 'fastclick' support before the app is replaced by the browser rendered version. (worth mentioning here that ember-hammertime should be in fastboot and browser build, if the ember-network inspired 'at build time' option becomes appealing to ya'll)
anyway, my 2 cents - figured i'd mention this cause I saw it on my phone yesterday but didn't have the time to explain what I was trying to invoke with the questions i asked
@anthonybalmeo I am curious what you've found in practice and what stack trace you got, that'll confirm or deny my suspicions (and/or craziness LOL)
k, cheers folks, let me know if I can help further

@eriktrom I think modifying the build tree approach would definitely be the cleanest. I don't know enough about ember-gesture's dependency structure to do that would probably let you remove all of the in code checks from this PR.

On Hammer existing or not, the issue isn't specifically about that, but rather that the code that calls Hammer is being executed server-side, hence the need for the guards.

@anthonybalmeo - lets move this to #57 - gave the cleanup a shot


Don't include amqp connection options in logs

Currently, Sneakers::Worker#worker_trace logs the queue options, which can contain sensitive authentication information (username/password).
The connection details should be scrubbed (only display scheme://host:port/path) prior to being displayed, or omitted entirely (not ideal, as it does provide useful information)

@stevenkaras I think that was an encouragement for a PR if you're up for it :)

I created PR #101 to redact just the password. I think the not mentioning the amqp is going too far. I also think redacting the username is going to far. Usernames are in logs all the time.


Features/poet 26 add intelligence

POET-46 Integrated LUIS into the project.

Hi @sabiipoks , is this PR related to POET-46 or POET-26?
Also there are some merge conflicts, please solve that first. Thank you.

Hi @haolinj, it is POET-46. Also, the conflict has been resolved.

@haolinj, required changes have been made. Please review code for PR. Thank you.

@sabiipoks What is the status on this PR?

@haolinj, Dongxu has created another branch from this branch for restructuring and code cleaning. Should we close this PR?

If this PR is not valid anymore, yes please remove this PR.


Fixing sw-build -> package.json

R: @jeffposnick @addyosmani
Fixes #340
This fixes the problem - but it raises a worrying concern with sw-build and sw-cli being tied to the top level dependencies.

Waiting for the build to complete.


Add option to run Analyze in the background

It would be very nice if the plugin would run Analyze in the background while the tester can keep browsing with the "high privilege" account.
Willing to implement this if accepting PRs.

@Splinter0 Yep! Would love that. I was thinking of an "Auto-analyze" checkbox that we can add close to the "Analyze" button.
Authmatrix works by specifying which roles/users should have access to a resource.
What should the default behavior be? I guess all roles/users would be unchecked?

I think so, all would be unchecked and then based on what you check the status changes, perhaps it might be nice then to integrate with the findings tab so that it does not get confusing. What do you think?

Could be interesting. I would start with just the auto-analysis first, and add something for the findings later down the road if we find it confusing.

Closed via https://github.com/caido-community/authmatrix/pull/15


changed input-group type to input group append

Bootstrap has a button append class to prepend or append a button directly to an input field leaving no spaces between the two - that looks much better than the normal input-group-btn.
Before

After

Thanks!

Any particular reason this wasn't merged?

Sorry about that, it got mixed in with some other changes and it was something I found manually before looking at the pull request.


Unable to bind modifier keys alone

E.g. I cannot rebind 'Left Shift' or 'Shift' to 'Sprint', it instead produces S+Shift which is non-functional and still reacts as if Ctrl is bound to Sprint, the default.
*If I edit the config file to force the parameter I desire (left.shift, left.control), the game will assume the default binds instead.
Inputting an apparently invalid key name ("shift", neither left or right) crashes on start-up. Figures, there is no "shift" key, only left or right.

"Left Alt" -> "Alt" + "Left Alt"

"Left Control" -> "C" + "Left Control"

"Left Shift" -> "S" + "Left Shift"

I am updating tms to 1.21.1 and I tested this. I was not able to reproduce. Any other mods installed?

forgot to mention, i am on minecraft version 1.21, the version with more mod support.
i will do more testing soon to try and isolate.

Tested with Fabric API and Language Kotlin, on 1.21 and 1.21.1.
Visually, the key-binds still present in a confusing manner, however, they do perform as desired.
After more testing, the conflict is with the mod "Keybind Fix": https://github.com/fzzyhmstrs/kf/issues

I have this issue as well, but I have no other mods related to changing keybinds, afaik.
I many mods installed: https://gist.github.com/sleeplessKomodo/2c9a20e7c3fe1917a5495647d056bbae

Ok I tested it again. Unable to reproduce. Even with Keybind Fix mod.
https://modrinth.com/mod/too-many-shortcuts/version/o4fyQKNQ
I uploaded a new version with some depend updates. Try this.


Telemetry support

There exists the npm module vscode-extension-telemetry for that.

There doesn't seem to be a need for this.


Added compatibility logic

@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
3, because the PR involves changes across both backend and frontend, including logic for compatibility checks and UI updates. The complexity of the compatibility logic and ensuring it works correctly with the frontend requires careful review.

 🧪 Relevant tests
No

 🔍 Possible issues

Possible Bug: The compatibility check in routes.js only compares processor and motherboard sockets. This simplistic approach might not cover all aspects of compatibility, such as power requirements, physical dimensions, or RAM compatibility.

Performance Concern: The compatibility check logic retrieves each component from the database sequentially, which could lead to performance issues with increasing database size or concurrent requests.
 🔒 Security concerns
No

 Code feedback:
relevant fileroutes/routes.js
suggestion

Consider implementing more comprehensive compatibility checks beyond just the processor and motherboard sockets. For example, checking for power supply compatibility with other components could prevent overlooking potential issues. [important]

relevant lineif (processorSocket.specifications.socket === motherboardSocket.specifications.socketType) {
relevant fileroutes/routes.js
suggestion

To improve performance, consider fetching all required components in parallel using Promise.all instead of awaiting each findById call sequentially. This change could significantly reduce response times for the compatibility check endpoint. [important]

relevant lineconst processorSocket = await Processor.findById(processor);
relevant fileclient/frontEnd/src/Components/CompatibilityPage.jsx
suggestion

Implement error handling for the checkCompatibility function to manage and display errors more effectively. This could improve user experience by providing more specific error messages in case of failures. [medium]

relevant line} catch (error) {
relevant fileclient/frontEnd/src/Components/CompatibilityPage.jsx
suggestion

Use a unique identifier for the key prop in lists instead of the array index to improve React's re-rendering performance and prevent potential issues with list item reordering. [medium]

relevant line{component.name}

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.


How to get snapshot 6.pth.tar

Great job! I noticed that "snapshot 6.pth.tar" was used in "run_hand4whole.py", but I couldn't find it in https://github.com/mks0601/Hand4Whole_RELEASE. That only has the download link for "snapshot 12.pth.tar", I used it instead of "snapshot 6.pth.tar", does that have an impact on the result? Because my final SMPL-X model did not fit the video characters well, it could not meet the demonstration effect of the example, and it also affected the creation and animation of my final avatar.
How do I download snapshot 6.pth.tar or how do I achieve a better SMPL-X fitting model? Thank you so much!

https://github.com/mks0601/Hand4Whole_RELEASE?tab=readme-ov-file#quick-demo

Thank you for your reply.


chore(main): release 0.3.1

:robot: I have created a release beep boop
0.3.1 (2024-07-24)
Bug Fixes

Add workflow for Mintlify documentation updates (#295) (605ba7a)
file explorer and uploader (#291) (f91b837)
Remove old docs (#288) (f20f74c)
run_all button (#293) (e48ce3b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Zero-True/zero-true/releases/tag/v0.3.1 :sunflower:


60FPS and 1080 buttons for apprtc

I propose adding a couple of buttons for apprtc:

1080 mode (minWidth=1920, minHeight=1080)
60 FPS HD mode (maxFramRate=60, minWidth=1280, minHeight=720)

Yes, these can be activated via URL params but not sure how many do that and it would expose those features even more.
Also activating these requires a page reload or stopping current stream and request a new gum stream and attach it to the peerconnection. I vote for the latter.
If we want to save the selections we could use localStorage.
Note: To actually get 1080 and/or 60FPS will most likely require a capture device with a 1080 60fps capable camera connected to it hence the audience will be small but I think it's a small effort nonetheless to showcase smooth video using WebRTC!

FYI it seems that 1080 actually only gives around 10fps atm in M41.
So maybe just add 60fps@720 for now as that seems to work fine.

NVM if the machine and camera is capable of 60fps it actually deliver it....

That was easy ;).
On Wed, Jan 14, 2015 at 9:06 AM, Christoffer Jansson <
<EMAIL_ADDRESS>wrote:

Closed #372 https://github.com/GoogleChrome/webrtc/issues/372.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleChrome/webrtc/issues/372#event-218621187.

Really fast implementation! ;)

Thank you ;).
On Wed, Jan 14, 2015 at 3:59 PM, Christoffer Jansson <
<EMAIL_ADDRESS>wrote:

Really fast implementation! ;)
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleChrome/webrtc/issues/372#issuecomment-69938413.


Fix: Speed up docker build step

What
Another team has recommended trying this to speed up the build step
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
The standards in the Git Workflow document on Confluence should be followed
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

more work needed - killing for now!


Error for 3.6.1 sdk

With "@freeclimb/sdk": "^3.6.0" as a dependency in this directory's package.json, I ran yarn install and this error came up. Same result on both node versions, 20.11.0 and 18.16.0. Bumping down to version 3.6.0 of the sdk works fine, so it appears changes from this commit is causing an error?

Hi @umachaVail, we've released a new version for the NodeJS SDK that should resolve the issue in 3.6.1, the new version is 3.6.2, please give that version a try and let us know if we can close out the issue.

Hi @amahadaya yes the new version is working thank you


Shield for Azure DevOps shield is broken in readme screen.

Describe the bug

Expected behavior
For the shield to not be borked so frequently
Example of how to Reproduce
Load the readme in GitHub

@nicksterx you've set this up, might it have something to do with the server not waking up on time? Is there a work around, or should drop the badge and look into some sort of replacement?

This looks like an error with shields.io timing out when it is loading that badge. Looking into alternatives before we remove it.
https://github.com/badges/shields/issues/1568

https://badgen.net/ seems like it might be an alternative. But looking at other projects they don't have the test/coverage badges. I'm thinking we should drop them?


Would it be in the same area to ask to interpret Properties from Notes as an anchor in the plugin code ?
Right now, we can either create long code with all data being written inside the code, either having AI writting down the code for us. But it is still mandatory to have all the information used to build the timeline explicit there.
What I imagine would be to have some items being just a link to a note, and having needed information as Properties of this Note.
For instance :
`
Christmas at Mom's
StartDate : 2024-12-24 18:00:00
EndDate : 2024-12-25 02:00:00
TypeEvent : Date range
Section : Family events
ColorEvent : green
`
`
Christmas
StartDate : 2024-12-25 00:00:00
TypeEvent : Marker
`
And therefore, the chronos code would be :
`
- [[Christmas at Mom's]]
- [[Christmas]]
`

And it would pull from the Notes, all the attributes required to display related events.

Cherry on top : being able to configure  the keyword used in Notes properties, in the plugin parameters. For instance, if you need to have a property named StartEvent, I could declare an equivalent as "Date début événement". Just to avoid to redefine pre-existing Properties ^^

Maybe I am dreaming too much and it would not be as easy to build as it seems. But again, for the sake of building more easily big timelines, for people like me who have all the data already stored in Notes, it would be so great !

that's a cool concept - I was thinking it might be nice to add "Insert with AI" that can add things at the item level - which in theory could take a note as an input and aligns with your idea
For note linking, I am setting it up to recognize links in the description of an event, and format the event/point in a way that indicates it is a link. This will allow users to add links (both in and outside of obsidian) without making the event title messy


Support \cfrac

Much of this comes directly from #135, so credit to @kevinbarabash.
This was an easier PR to write than #135 since KaTeX now has well established methods for pt-to-em conversions and nulldelimiter.
As in #135, this PR does not support the LaTeX optional argument for numerator justification.  It supports a \cfrac #1 #2  syntax.
Fixes #85.

The screenshot below compares a \frac to a \cfrac.

@ronkok here are the updated screenshots:
Chrome

Firefox

Happy to see this work, which dates back to 2014, come to fruition! Thanks @ronkok and @kevinbarabash!

I can't believe it's been that long.

@OrkhanAlikhanov Great! Let us know if there are other sticking points you know of, or discover.

@edemaine Hey! I've removed MathJax and integrated KaTeX. Everything seems okay so far. Rendering is now super fast! Especially in android and ios apps. Thank you all!

@OrkhanAlikhanov Glad to hear it!!


README

README needs to be updated after all functions are working

README updated according to the latest version of the programme


Fallback to the built-in font renderer when font loading fails

After PR #9340 all glyphs are now re-mapped to a Private Use Area (PUA) which means that if a font fails to load, for whatever reason[1], all glyphs in the font will now render as Unicode glyph outlines.
This obviously doesn't look good, to say the least, and might be seen as a "regression" since previously many glyphs were left in their original positions which provided a slightly better fallback[2].
Hence this patch, which implements a general fallback to the PDF.js built-in font renderer for fonts that fail to load (i.e. are rejected by the sanitizer). One caveat here is that this only works for the Font Loading API, since it's easy to handle errors in that case[3].
The solution implemented in this patch does not in any way delay the loading of valid fonts, which was the problem with my previous attempt at a solution, and will only require a bit of extra work/waiting for those fonts that actually fail to load.
Please note: This patch doesn't fix any of the underlying PDF.js font conversion bugs that's responsible for creating corrupt font files, however it does improve rendering in a number of cases; refer to this possibly incomplete list:
Bug 1524888
Issue #10175
Issue #10232

[1] Usually because the PDF.js font conversion code wasn't able to parse the font file correctly.
[2] Glyphs fell back to some default font, which while not accurate was more useful than the current state.
[3] Furthermore I'm not sure how to implement this generally, assuming that's even possible, and don't really have time/interest to look into it either.

/botio test

/botio-linux preview

This is something that we would really benefit from. We currently have a hacky workaround in our application involving getting the document with {stopAtErrors: true} and attempting to render, catching any errors that occur and re-rendering with different options if we failed.
This is a much nicer solution.

Looks good! I forgot we had a cache of all the glyphs, that makes things much easier.

Thank you; I'd also forgotten all about the glyphCache, and somehow it popped into my head when I was doing the dishes the other day and everything just fell into place :-)

It would be great if we could get a follow up bug to add a test case.

The reason that I didn't add a test-case here, is that this is nothing more than a work-around. The end goal should still, as far as I'm concerned, be to fix the font bugs that cause us to generate a font file that the OpenType Sanitizer ends up rejecting.

I am also thinking, for testing, we may want a way to ensure that this path isn't allowed, unless the test specifically allows it (maybe a pref that defaults to enabled?).

In an ideal world, where the underlying PDF.js bugs have been fixed, I'm not sure what you'd actually test in that case!? (Unless you somehow corrupted the font file on the main-thread, prior to loading it.)

The reason that I didn't add a test-case here, is that this is nothing more than a work-around. The end goal should still, as far as I'm concerned, be to fix the font bugs that cause us to generate a font file that the OpenType Sanitizer ends up rejecting.

I agree, I'd still like to fix the underlying issue and for that specific PDF I have a fix in progress. Though it would still be nice to test this new code path and make sure it always works.

In an ideal world, where the underlying PDF.js bugs have been fixed, I'm not sure what you'd actually test in that case!? (Unless you somehow corrupted the font file on the main-thread, prior to loading it.)

I think we'd need a font that we intentionally never fix. Or an idea I like better is: during the test, the test framework could override the font loading api and intentionally make it fail. We could add an option to test_manifest.json for this.

Or an idea I like better is: during the test, the test framework could override the font loading api and intentionally make it fail. We could add an option to test_manifest.json for this.

Totally agree, that seems like the nicest/cleanest solution.


Create functionality to store the history of invoice email

Describe the task
Finance department sent email realted to invoice to client in following scenarios
Requesting payment for a project work, done by the company for the client
Reminder in case payment for the project work, not received from the client.
Payment confirmation in when, payment for the project work received from the client.
At present there is no implementation for checking history of invoice email sent to client.
Expected behavior
We require a functionality through which we can view the history of invoice email sent to client.

[x] #1960
[x] #1961
[x] #1962

References
Specify any reference material / links

We need to show the time format in this form


HCLtech stock price prediction using Linear Regression, Support Vector Machines (SVM)

🛠 Related Issue

Closes #445

📝 Description

[Provide a concise description of the changes. Include the purpose and any relevant context or details.]
Predicting HCLTech Stock Prices involves using various machine learning models like Linear Regression, SVM and k - nearest neighbours to analyze historical stock data and forecast future trends. Each model leverages different algorithms and methodologies to enhance prediction accuracy and reliability.
🔍 Type of PR

[x] 🐛 Bug fix
[x] ✨ Feature enhancement

✅ Checklist

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

@Akshat111111 please review this pull request and merge this under gssoc'24

@Akshat111111 hi i have added the correct model actually i had saved both of this code together then put it in different folders i by mistake put the hdfc in both the folders

@Akshat111111 i checked the images and dataset they are of hcltech only only the model i had made mistake


Set HF_SCRIPTS_VERSION to main

After renaming "master" to "main", the CI fails with
AssertionError: 'https://raw.githubusercontent.com/huggingface/datasets/main/datasets/_dummy/_dummy.py' not found in "Couldn't find a dataset script at /home/circleci/datasets/_dummy/_dummy.py or any data file in the same directory. Couldn't find '_dummy' on the Hugging Face Hub either: FileNotFoundError: Couldn't find file at https://raw.githubusercontent.com/huggingface/datasets/master/datasets/_dummy/_dummy.py"

This is because in the CI we were still using HF_SCRIPTS_VERSION=master. I changed it to "main"

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Link to Contributing file from Readme

Contributing.md is a step by step tutorial about how to contribute to GitHub. Need link to this file from the readme file. New GitHub contributor don't know which file to open to know how to contribute to a repository

@ratanparai I have created a pr for this https://github.com/OpenCodeFoundation/first-contributions/pull/33


Incorrect Modal Documentation

Hi,
in the example at https://v4-alpha.getbootstrap.com/components/modal/#modal-components the id specify in the button (#exampleModal) is not accorded to the real modal id (#myModal). "#exampleModal" should be changed  to "#myModal".

Best regards,
Cristian Romanello.

fixing in the beta version https://getbootstrap.com/docs/4.0/components/modal/#live-demo


PIP install ansible permissions should be validated before recommending using it

`: -->
Issue Type

Bug Report / Support Request

Your Environment

Vagrant 1.9.1
VirtualBox 5.1.12r112440
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides

Your OS
sw_vers
ProductName:	Mac OS X
ProductVersion:	10.12.1
BuildVersion:	16B2555

macOS (version)
Windows (version)
Linux (version)

Full console output

Summary
I tried installing Vagrant but unfortunately I my Python directory was owned by root so I kept errors with some of the Vagrant dependencies. I even used a new vagrant 1.9.1 dmg to upgrade vagrant but that was insufficient for ensuring PIP could install. Ideally there would some kind of validation step about PIP install permissions so that users could avoid getting trapped like I did.
This fixed it.
https://github.com/sindresorhus/weechat-notification-center/issues/1
sudo chown -R $USER /Library/Python/2.7/site-packages/pync did the trick.
This probably seems out of scope but I am big fan of guard rails.

@kieranacquia - For the Vagrant install, it should either be done through Homebrew (e.g. brew cask install vagrant), or via the DMG download. I haven't seen permissions issues installing Vagrant before.
For pip, if using macOS with nothing else installed, you can install Ansible via Pip with:
$ sudo easy_install pip
$ sudo pip install ansible

It's not ideal to use sudo in this case, so you can use the above change to install under your regular user account instead... but this discussion seems out of scope for Drupal VM itself, since there are at least four or five different ways you can install Ansible (depending on your system setup)—or you can just leave it off altogether and let it run inside the VM.


feat:support cose

Signed-off-by: zaihaoyin<EMAIL_ADDRESS>

Codecov Report

Merging #365 (cd96e82) into main (06b7b5b) will decrease coverage by 0.08%.
The diff coverage is 25.45%.

@@            Coverage Diff             @@
##             main     #365      +/-   ##
==========================================
- Coverage   30.62%   30.53%   -0.09%
==========================================
  Files          25       26       +1
  Lines        1610     1657      +47
==========================================
+ Hits          493      506      +13
- Misses       1104     1138      +34
  Partials       13       13

Impacted Files
Coverage Δ

cmd/notation/push.go
21.91% <0.00%> (-2.70%)
:arrow_down:

cmd/notation/verify.go
17.92% <0.00%> (-1.08%)
:arrow_down:

internal/cmd/flags.go
50.00% <0.00%> (-4.55%)
:arrow_down:

internal/cmd/options.go
0.00% <0.00%> (ø)

internal/cmd/signer.go
0.00% <0.00%> (ø)

internal/envelope/envelope.go
100.00% <100.00%> (ø)

pkg/configutil/once.go
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Edits to Reporting

Attempting to bring consistency of phrases, terms, and formatting to the Reporting section.
I've tried to use the same phrases for the tracking tools as was used in the source control section.  I've made an assumption that the bug reporting tool and archive MUST be publicly visible (which can obviously be changed).

Hmm.  We tried to separate out MUST and SHOULD into separate lines, and ask for the URL early, so that filling in the form would be easier (we can auto-fill some info with the URL).

the first criterion was more like a form question so I was trying to bring
that inline with the other criterion.  will have another try.
On Thu, 24 Sep 2015 at 04:29 David A. Wheeler<EMAIL_ADDRESS>wrote:

Hmm. We tried to separate out MUST and SHOULD into separate lines, and ask
for the URL early, so that filling in the form would be easier (we can
auto-fill some info with the URL).
—
Reply to this email directly or view it on GitHub
https://github.com/linuxfoundation/cii-best-practices-badge/pull/29#issuecomment-142689729
.

in conflict, will update these changes and resubmit


Fix podman network setup to detect disabled ipv6

If IPv6 is disabled on the host, creating a podman network with IPv6 enabled will lead to the bridge interface failing to go up.
Try hard to detect if IPv6 is disabled on the host before enabling it.
To help updating the configuration after enabling IPv6 on the host, also remove any existing uyuni network that doesn't have IPv6 and recreate if needed.

Please log which IP versions Will be uses. User should also force one or the other

Please log which IP versions Will be uses. User should also force one or the other

I'll log if IPv6 is not enabled, and that's it. The podman network will mimic what is available on the host. The less we let people tamper with the network definition the safer for us.


keep getting this error

i remove the files like it says to do and i still keep getting the same error. what do i do ?

just wait a couple of seconds / minutes and it will continue


Create simple API and automate its deployment to AWS

Deployment to AWS had complications, separating into other smaller issues for completion.


Optimize conversion to binned sparse matrix

The speed to create a binned representation with the help of sparse matrices is slow.
Especially in line 791
for inner_elem in filled:
    lil_mat[idx, inner_elem] += 1

the addition step takes a lot of time.

First optimized version:
feature/opt_bin_hist
changed to coo_matrix, the initialisation of the matrix and skipped the addition.

See here now https://github.com/NeuralEnsemble/elephant/pull/32


Merge master to 1.6.0

PR created while looking at #2378, which only shows 6 modified files.
This PR is created directly on DD's upstream branches, without any intermediary code pull.

Unsure why entrypoint_scripts/common/dojo-shared-resources.sh re-appears here, since both branches do have the proper changes in :thinking:

Resolving the conflicts here would merge to master, so we probably should hold off until all pending PR to be applied to 1.6.0 are done and we're ready to release.

Talked with @valentijnscholten, moving with sibling PR.


Multiprocessed dataset builder [WIP]

This PR adds the multiprocessing part of #2650 (but not the caching of already-computed arrow files). On the other side, loading of sharded arrow files still needs to be implemented (sharded parquet files can already be loaded).

I would also like to add a test, but am not sure whether it should go into test_builder (more natural imo) or test_load (which already contains a lot of the things I have to import to run my current testing setup). For reference, what I run to test that it works looks like:
import os
from pathlib import Path
import shutil

import datasets
from datasets.builder import DatasetBuilder
from datasets.features import Features, Value

DATASET_LOADING_SCRIPT_NAME = "__dummy_dataset1__"

DATASET_LOADING_SCRIPT_CODE = """
import os

import datasets
from datasets import DatasetInfo, Features, Split, SplitGenerator, Value

class __DummyDataset1__(datasets.GeneratorBasedBuilder):

    def _info(self) -> DatasetInfo:
        return DatasetInfo(features=Features({"text": Value("string")}))

    def _split_generators(self, dl_manager):
        return [
            SplitGenerator(Split.TRAIN, gen_kwargs={"filepaths": [os.path.join(dl_manager.manual_dir, "train1.txt"), os.path.join(dl_manager.manual_dir, "train2.txt")]}),
            SplitGenerator(Split.TEST, gen_kwargs={"filepaths": [os.path.join(dl_manager.manual_dir, "test.txt")]}),
        ]

    def _generate_examples(self, filepaths, **kwargs):
        idx = 0
        for filepath in filepaths:
            with open(filepath, "r", encoding="utf-8") as f:
                for line in f:
                    yield idx, {"text": line.strip()}
                    idx += 1
"""

def dataset_loading_script_dir(tmp_path):
    script_name = DATASET_LOADING_SCRIPT_NAME
    script_dir = tmp_path / script_name
    script_dir.mkdir()
    script_path = script_dir / f"{script_name}.py"
    with open(script_path, "w") as f:
        f.write(DATASET_LOADING_SCRIPT_CODE)
    return str(script_dir)

def data_dir(tmp_path):
    data_dir = tmp_path / "data_dir"
    data_dir.mkdir()
    with open(data_dir / "train1.txt", "w") as f:
        f.write("foo\n" * 10)
    with open(data_dir / "train2.txt", "w") as f:
        f.write("foo\n" * 10)
    with open(data_dir / "test.txt", "w") as f:
        f.write("bar\n" * 10)
    return str(data_dir)

def load_dataset_builder_multiprocessed(tmp_path):
    builder = datasets.load_dataset_builder(
        os.path.join(dataset_loading_script_dir(tmp_path), DATASET_LOADING_SCRIPT_NAME + ".py"),
        data_dir=data_dir(tmp_path),
    )
    assert isinstance(builder, DatasetBuilder)
    assert builder.name == DATASET_LOADING_SCRIPT_NAME
    assert builder.info.features == Features({"text": Value("string")})
    builder.download_and_prepare(tmp_path / "prepare_target", max_shard_size=500, num_proc=2)

if __name__ == "__main__":
    tmp_path = "tmp"
    if os.path.exists(tmp_path):
        raise FileExistsError(f"path {tmp_path} already exists")
    os.makedirs(tmp_path)
    try:
        load_dataset_builder_multiprocessed(Path(tmp_path))
    finally:
        # pass
        shutil.rmtree(tmp_path)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Nice ! I think the test can go in test_builder.py :)

I've added sharded arrow dataset loading. Two WIP items in the PR:

Order is not conserved (it seems like the sharded files are read in the wrong order)
the tqdm for preparing the splits is wrong (it compares against the size of the whole split rather than against the size of the multiprocessing shard, but I am not sure how to access the latter)

Also naming.filenames_for_dataset_split is not very elegant imo.
@lvwerra if you don't care about order, as I do, it's functional for now but I'd still quite like to get to the bottom of this.

I fixed the tqdm to be less misleading, but it can't tell where to stop. I am a bit hesitant to add a top-level tqdm (on the shard iterator) since for most intents it will do 0 -> N shards straight, but I am not sure what is the best way to present that info here.

I'm continuing the PR :)

Did a few changes:

make shards naming consistent:

use {builder_name}-{split_name}.{file_format} when there's only 1 shard
otherwise use {builder_name}-{split_name}-{shard_idx:05d}-of-{num_shards:05d}.{file_format}

update the reader to support reading several shards

added a new shard_lengths field in SplitInfo (FYI it is saved in dataset_info.json next to the shards as usual)

it's None when there's only 1 shard
otherwise it's a list of integers that correspond to the number of rows per shard

implemented partial reading to only memory map the required shards

e.g. when someone asks for a partial split like train[:10%]

align the sharding for beam datasets

no more combining into 1 big arrow file

added a tqdm bar

only one single bar, handled by the main process
gathers progress updates from other processes using iflatmap_unordered
shows the number of examples (even for datasets prepared by generating arrow tables)

tests
docs

Alright this is ready for review - sorry it ended up so big ^^'
If I can do anything to make it easier for your to review this PR @mariosasko let me know

Multiprocessing is disabled by default but we may show a warning to encourage users to pass num_proc if the dataset is split in many files. Let me know what you think

Hey, is this error seems to you guys natural?
>>> data = load_dataset('dataset_loaders/rfw2latentplay', num_proc=14)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/somewhere//mambaforge/envs/datasets/lib/python3.8/site-packages/datasets/load.py", line 1719, in load_dataset
    builder_instance = load_dataset_builder(
  File "/somewhere//mambaforge/envs/datasets/lib/python3.8/site-packages/datasets/load.py", line 1523, in load_dataset_builder
    builder_instance: DatasetBuilder = builder_cls(
  File "/somewhere//mambaforge/envs/datasets/lib/python3.8/site-packages/datasets/builder.py", line 1292, in __init__
    super().__init__(*args, **kwargs)
  File "/somewhere//mambaforge/envs/datasets/lib/python3.8/site-packages/datasets/builder.py", line 303, in __init__
    self.config, self.config_id = self._create_builder_config(
  File "/somewhere//mambaforge/envs/datasets/lib/python3.8/site-packages/datasets/builder.py", line 456, in _create_builder_config
    builder_config = self.BUILDER_CONFIG_CLASS(**config_kwargs)
TypeError: __init__() got an unexpected keyword argument 'num_proc'

Do we have some benchmarks to see the speed-up?

On my machine running load_dataset("oscar-corpus/OSCAR-2201", "br") (which is split in shards) I go from 2-3k examples per sec to 4-5k examples per sec with num_proc=2 😉

Hey, is this error seems to you guys natural?
The package built from 0d4e3907 commit tag, and here is the version displayed from the import ...

I don't know where you got the 0d4e3907 commit tag from, it doesn't seem to be in this PR. You should try installing from this PR, or wait for it to be merged on main

Splits vs Shards
Maybe it's a good idea to add some documentation on the sharding that can be achieved by passing list based arguments to the SplitGenerators gen_kwargs ...

Maybe it's a good idea to add some documentation on the sharding that can be achieved by passing list based arguments to the SplitGenerators gen_kwargs ...

This is part of this PR :) you can check the changes in docs/source/dataset_script.mdx

I took your comments into account @mariosasko thanks !
Let me know if it's good for you now ;)


Enter to submit response

Description
As an user, I would like to press enter to submit my response
Scope of Work

[ ] Link the enter button to form submission on the chatbot

Demo requirement

The user types in a response and presses enter
The response is sent to the server

This is for landing page, not chat interface

great QOL fix. 👍


Path of index.js on Win 10

Thanks for this very cool theme! You may wish to include this on the readme page: the path to the index.js file is tricky to find on Win 10:
C:\ProgramData\<username>\slack\app-2.9.0\resources\app.asar.unpacked\src\static\index.js

Do you know if there's an environment variable that gets the C:\ProgramData\<username> folder?

Or more specifically, does echo %APPDATA% give you that folder? Afaik it's been C:\Users\<user>\AppData up until now. Do you have something installed that's freezing or otherwise managing your user folder?
Also, are you on the new Creator's Fall update? I'm not if MS is moving stuff around all of a sudden.

Thanks for your reply. I'm on 1703 OS Build 15063.726 version of Win 10. echo %APPDATA% gives me this:
C:\Users\<username>\AppData\Roaming
And that location doesn't have the installation; it does have some Slack files, but not the JS file we need to  edit. I'm on a domain PC that has group policy but I'm also admin on this box.

echo "%LOCALAPPDATA%\slack"

Hi ! I have installed slack for Winodws 10 from microsoft Store. I can't find that path to index.js ... There is no such path as "C:\ProgramData<username>\slack\app-2.9.0\resources\app.asar.unpacked\src\static\index.js" on my computer.
echo "%LOCALAPPDATA%\slack" gives me path like this :
C:\Users<userName>\AppData\Local\slack
but there is no directory slack in that location ...
I'm using Windows 10 Pro 1709 (compilation nr 16299.309). I'm admin also on that machine.
Can someone help me find that file ?
Am I even able to get access to that file if I installed slack from Microsoft store ?

You just have to open the Windows Task Manager, find the Slack process, and right click on it to open file location. The path is something like C:\Program Files\WindowsApps\91750D7E.Slack_<IP_ADDRESS>_x64__8she8kybcnzg4\


DBZ-6256 Add log.mining.flush.table.name Oracle config option

https://issues.redhat.com/browse/DBZ-6256
Premise:
There is the potential when using multiple connector deployments on Oracle that there could be some level of lock contention with all connectors using the same table. This allows users to configure the flush table name themselves, reducing the lock contention across multiple connector deployments.
@jpechane its safe to back-port this to Debezium 2.2 at least, the code is completely backward compatible.

Applied to main (=2.3).


zod: try out pythonic names

Just drafting this so I can sleep on the idea
but let me know what you think @meatballs
I think, at the moment, I prefer this, since it's closer to python's typing names
z.typeddict is a bit verbose but hey... I didn't choose it
I've left the TypeScript names in as aliases for convenience
See #120 for context

It seems better to match the typing names, even if they're not what we might have chosen

merci
do you think remove the aliases?
Or just leave them in and not document them (whenever I get round to that)

The latter, I reckon

General convention is to convert camel case to snake, so the underscored option.

But maybe it can be like namedtuple?

There's plenty of stuff in the standard lib that was written well before pep8 existed and doesn't conform to it.
I don't think that justifies non-conformance today.

Ok - names changed - ready I think


docs(DeferObservable.ts, EmptyObservable.ts, FromEventObservable.ts, …

Description:
Related issue (if exists):
…FromObservable.ts, combineLates
Add output results for the examples in the documentation pertaining to the operators. Change
seqenceEqual.ts 'var code = Observable.from' to 'var code = Rx.Observable.from'. If the current
example is pasted into an editor and executed it will throw an error of 'Observable not defined'.
The addition of Rx. fixes this.
Issue: Examples do not have resulting output #2171

Coverage remained the same at 97.687% when pulling 8f8608a6d1111c87b72601686c5b7f6827c955d7 on jrroman:doc-additions into 89b506d20485b7e849f53cc34a7d35f16773d27c on ReactiveX:master.

Hi @jrroman. Have you tried generating the docs site (npm run build_docs if I'm not mistaken) after your changes? I'm afraid those // comments won't show up in the code block, because the documentation generator looks for adjacent lines. So to be safe the // comments should be after code, with no empty line in between.

Hi @staltz, I did run npm run build_docs and it looked like they appear in the code block with the associated example. I have attached a screenshot of my local, maybe I misunderstood something. If you would like I can move them directly adjacent so that there is no chance of error.

@staltz I have gone through and removed all spaces between the code and the comments. It now looks like the below screenshot. If this is what you mean, I can push these changes.

@jrroman I personally like it better with a space in between. But that's me...

Ok, cool. I was suggesting no lines in between just to make sure that they compiled as a code block. If it works even with an empty line in between, then sure, go ahead.

@blesh @staltz I cleaned the examples output up a bit just slight corrections with spacing and such. I kept the space between the example and the output per blesh's request.

Coverage remained the same at 97.687% when pulling 0fb50c74e77c5a5e0a46a658d35760c22aa43d19 on jrroman:doc-additions into 89b506d20485b7e849f53cc34a7d35f16773d27c on ReactiveX:master.

screenshot of the output from the last commit for reference.

LGTM

LGTM


Porting to use 3 bpp display

Hi,
I'm using a LPM013M126C display from Japan Displays and wondering how easy would it be to port Littlevgl to create some simple GUIs. The display uses 3 bpp colour depth (8 colours) but there's a 4 bit mode where basically one of the bits is just not used.
I see in lv_conf_templ.h that the options for Color depth are 1/8/16/32.
How easy would it be to port this library to use 3bpp?
Regards,
Joao

3-bit color is not directly supported in LittlevGL. @kisvegabor can probably explain in more depth, but you could have a look near the end of the discussion in #300.

More specifically, this comment how to describes how to handle special color formats: https://github.com/littlevgl/lvgl/issues/300#issuecomment-410434520

There has been no activity for 1 month so I will assume the issue is resolved. If you are still having problems feel free to reopen the issue and comment.

I'm using an identical display - using it with littlevgl is pretty simple, if you use 8bit colours and convert them to 3 bit (or 4) colours when updating your display. This might not be the most efficient use of the VDB but for our case it works pretty well. I have not looked into adding special colour formats at this point.


Samsung native browser

What classes are added to the html tag for say a Samsung Galaxy S6 mobile phone using the native Samsung Internet browser?

Looking at logs, "samsung" is there as being detected, but isn't being a class added to the html tag ...


When UW door and UW lever is placed on the same tile, lara can interact with the door infinite times

When UW door and UW lever is placed on the same tile, and lara interacts with UW door to open it, Lara can interact with it again because of the UW door, will insert videos soon.

https://github.com/MontyTRC89/TombEngine/pull/820 fixed here


Apply timezone transform only once

Fixes #1356 by applying timezone transform only once.

@all-contributors please add @LunarLanding for bugs


Anticheat for second door still isn't working properly

https://twitter.com/admittedlyhuman/status/566733448032030720
It's possible to go back the way one came if one moves slightly past the center line then backtracks.

dupe of #2, sorry.


/docs/user-guide/configmap need more details on ConfigMap updates

ConfigMap design doc (https://github.com/kubernetes/kubernetes/blob/master/docs/design/configmap.md) has a section "Consuming ConfigMap Updates" that describes if pods see changed ConfigMap values depending on how ConfigMap is consumed (via volume plugin or via environment variable).
I believe such information is valuable in the ConfigMap user guide as well.

Agreed.  It's pretty easy to add this section in.  Jeff, any advice?
Also, this document needs a top to bottom rewrite to conform to our templates.

No advice, looks like a useful addition.

/remove-lifecycle stale
/lifecycle frozen

/language en
@tengqm is this something you planning on working on ?

Also see #1817

/sig apps

/priority backlog
#16085 should have an nice high priority though

Also see #1911

In my opinion: fixed in #21452
/close


fix nashorn support

The recent improvements have broken some of the nashorn support.
Task: fix this!
After that fext.js is ready for integration into production.

This was a regression. See also:
https://github.com/glathoud/fext/issues/8
https://bugs.java.com/bugdatabase/view_bug.do?bug_id=JDK-8205600

Note about nashorn: utility limited in the end for build (but okay for testing) because no let/const support at the moment (2018-07), and let/const support needed for safe inlining.
Digging into this:
 jjs -v
nashorn 1.8.0_171

let, const:
http://openjdk.java.net/jeps/203
https://bugs.openjdk.java.net/browse/JDK-8046038

jjs --language=es6
adds support for let/const

...and now this:
http://openjdk.java.net/jeps/335
apparently Oracle is too small of a company to support Nashorn. Interesting...


tolerance of incomplete code

For now, we may simply refuse to format(by returning input as-is) broken inputs


glslang: add dev and lib outputs to prevent cross comp issues

Description of changes
Related #330057
Moved the headers, CMake files, and pkg-config files out of $out and into $dev. Hopefully this will prevent buildPlatforms.glslang which is built with libstdc++ linking with mesa's intel-clc which is built with libc++ in pkgsLLVM.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Drafting until the situation in https://github.com/NixOS/nixpkgs/pull/330785 is cleared up.

Can this be merged independently of https://github.com/NixOS/nixpkgs/pull/330785?

Yes

Bisect says commit 9b7f1a3913168bc9b9c070dbc9720dad1d06b49c glslang: add dev and lib outputs to prevent cross comp issues broke the build of glslang on staging:
$ nix build --no-link -f. -L glslang
...
glslang> evaling implicit 'postFixup' string hook
glslang> ln: failed to create symbolic link '/nix/store/vcfi5x5cdlxhfc69p7sp9g2l3q6cvb7w-glslang-14.3.0-dev/bin/glslangValidator': No such file or directory

What environment did you manage to get it to build?

pkgsLLVM.glslang, idk how it even built locally looking at the symlink creation. It's using dev instead of bin. Should be able to fix it tonight after work.

Interestingly at least on x86_64-linux pkgsLLVM.glslang seems to fail just the same in staging:
$ nix build --no-link -f. pkgsLLVM.glslang
...
evaling implicit 'postFixup' string hook
ln: failed to create symbolic link '/nix/store/hba5r0m6aj1bc29rclfg9xvssaqd4ah1-glslang-x86_64-unknown-linux-gnu-14.3.0-dev/bin/glslangValidator': No such file or directory

Yeah, it was a typo that wasn't caught. I just opened a PR to fix it. #332581

I could have sworn I at least built it before merging but apparently not. Sorry about that.
Thanks a bunch for the bisect @trofi, you are incredibly helpful!

Yeah, it's no problem. I thought it worked as well. At least we caught this in staging.


Prediction becomes empty, therefore the loss become nan.

I've tried to finetune the llm4decompile-6.7b model on my dataset and the result is impressive.
My own dataset looks like the following format
{'instruction': 'MY_CUSTOMIZE_QUESTION, 'input': '', 'output': 'MY_CUSTOMIZE_ANSWER}
and it will be formed like this
{{ bos }}
user: data[idx]['instuction']
{{ eos }}
assistant:
classificiation: data[idx]['output']
{{ eos }}

Everything works totally fine and the evaluation results is satisfied.
However, everything goes wrong when I try to fine-tune the 9B model.
I change the part of my code that loads the model from ‘llm4decompile-6.7b’ to ‘llm4decompile-9b’ while keeping everything else the same.
The model prediction becomes empty after a few steps update and the loss become nan due to the empty output.
The first step of model predictions:
Decoded Predictions: ['" on the  provided the followingE"s" section... "]

The few steps of model predictions:
Decoded Predictions: ['', '', '', '']

This question is really bothering me, and I hope someone can give me some advice. Any advice would be greatly appreciated.
package version:
accelerate==1.0.1
bitsandbytes==0.42.0
deepspeed==0.15.2
datasets==2.17.0
evaluate==0.4.1
gpustat==1.1.1
huggingface-hub==0.23.2
hydra-core==1.3.2
icecream==2.1.3
Jinja2==3.1.2
jsonlines==4.0.0
langchain==0.1.0
langchain-core==0.1.8
loguru==0.7.2
mlflow==2.9.0
openai==1.40.0
packaging>=23.2
pandas==2.0.3
peft==0.11.0
pendulum==2.1.2
pyarrow==14.0.0
pysnooper==1.2.0
PyYAML==6.0
retrying==1.3.4
scikit-learn==1.3.2
seaborn==0.13.0
tokenizers==0.20.3
torch==2.4.0
torchvision==0.19.0
vllm==0.6.3.post1
transformers==4.45.2
trl==0.11.0
wandb==0.16.0
flash-attn==2.6.2

The 9B model is based on (Yi-Coder)[https://github.com/01-ai/Yi-Coder], while the training script is from Deepseek-Coder. We did not test the 9B model for the script,  we recommend to use llama factory to tune the 9B model.


Support Symfony v7 components

A good idea! Any PRs we can start with? :)

@PhilinTv I opened a PR to allow Symfony 7 here: https://github.com/propelorm/Propel2/pull/1992

Resolved in the last release 😉