Will not open under large resolution

Which operating system does the issue occur on?
Lubuntu, X11 using i3 as a wm.
I just changed my monitor to a CRG90 and it seems its not liking it one bit. I thought you should know :)

Could you provide more information about this? Does it crash or not?
if it crashes what's the output? I'm using 4k screen without any issues on X11/Wayland.

Could you go into a bit more detail please? If it doesn't start, there's likely a crash dump in stderr, posting that would be very helpful.
Alacritty has been tested at far greater densities than what you'd get on that monitor with a readable font size, so it's unlikely that this would cause it to not start.

Well, it kinda opens  but it doesn't, it opens but you cant see anything in it, then after a bit it crashes with this error:
thread 'main' panicked at 'assertion failed: line < self.num_lines()', alacritty_terminal/src/grid/mod.rs:393:9
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

This is my xorg config:
# nvidia-xconfig: X configuration file generated by nvidia-xconfig
# nvidia-xconfig:  version 440.26

Section "ServerLayout"
    Identifier     "Layout0"
    Screen      0  "Screen0" 0 0
    InputDevice    "Keyboard0" "CoreKeyboard"
    InputDevice    "Mouse0" "CorePointer"
    Option         "Xinerama" "0"
EndSection

Section "Files"
EndSection

Section "InputDevice"

    # generated from default
    Identifier     "Mouse0"
    Driver         "mouse"
    Option         "Protocol" "auto"
    Option         "Device" "/dev/psaux"
    Option         "Emulate3Buttons" "no"
    Option         "ZAxisMapping" "4 5"
EndSection

Section "InputDevice"

    # generated from default
    Identifier     "Keyboard0"
    Driver         "kbd"
EndSection

Section "Monitor"
    Identifier     "Monitor0"
    VendorName     "Unknown"
    ModelName      "Samsung C49RG9x"
    HorizSync       190.0 - 190.0
    VertRefresh     48.0 - 120.0
    Option         "DPMS"
EndSection

Section "Device"
    Identifier     "Device0"
    Driver         "nvidia"
    VendorName     "NVIDIA Corporation"
    BoardName      "GeForce GTX 1070"
EndSection

Section "Screen"
    Identifier     "Screen0"
    Device         "Device0"
    Monitor        "Monitor0"
    DefaultDepth    24
    Option         "Stereo" "0"
    Option         "nvidiaXineramaInfoOrder" "DFP-5"
    Option         "metamodes" "5120x1440 +0+0; nvidia-auto-select +0+0"
    Option         "SLI" "Off"
    Option         "MultiGPU" "Off"
    Option         "BaseMosaic" "off"
    SubSection     "Display"
        Depth       24
    EndSubSection
EndSection

@chrisduerr That's great news!!!

Could you provide the same output, but with RUST_BACKTRACE=1 together with the output of alacritty --version to check which exact hash you're running? Sounds like you ran into one of our sanity checks, hopefully should be a simple fix.

alacritty 0.3.3 (e0a2865)

@vanor89 Do you have a rust toolchain installed? Getting the backtrace in a debug build on the latest master would be helpful.

I sure do, but I don't know how to do what you are suggesting though, I'm by no means a Rust user, just have rustup for building stuff when needed, can you please give me more instruction on how to do that? sorry about that

No worries, thanks for the cooperation.
Running RUST_BACKTRACE=1 cargo run after cloning should be enough, assuming that your rust version is at least 1.36.0 (you can check by running cargo --version). If your rust version isn't 1.36.0 or higher, you can update by running rustup update.

Oh durr, of course that's how. gotcha, its updating right now, will post asap

Got a couple warnings and an error!
warning: use of deprecated item 'std::sync::ONCE_INIT': the `new` function is now preferred
  --> alacritty/src/cli.rs:78:21
   |
78 |             .author(crate_authors!("\n"))
   |                     ^^^^^^^^^^^^^^^^^^^^
   |
   = note: `#[warn(deprecated)]` on by default
   = note: this warning originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: use of deprecated item 'std::sync::ONCE_INIT': the `new` function is now preferred
  --> alacritty/src/cli.rs:78:21
   |
78 |             .author(crate_authors!("\n"))
   |                     ^^^^^^^^^^^^^^^^^^^^
   |
   = note: this warning originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

    Finished dev [unoptimized + debuginfo] target(s) in 1m 31s
     Running `target/debug/alacritty`
Alacritty encountered an unrecoverable error:

        Error creating GL context; Received multiple errors. Errors: `[OsError("eglCreateWindowSurface failed"), OsError("Couldn\'t setup vsync: expected interval `1` but got `0`")]`

@vanor89 Do you have vsync disabled in your driver? Which GPU + driver are you using?

I'm using nvidia-driver-440 and judging from the result of glxgears, I have it enabled.
➜ glxgears
Running synchronized to the vertical refresh.  The framerate should be
approximately the same as the monitor refresh rate.
602 frames in 5.0 seconds = 120.288 FPS
599 frames in 5.0 seconds = 119.760 FPS
599 frames in 5.0 seconds = 119.761 FPS
598 frames in 5.0 seconds = 119.559 FPS
599 frames in 5.0 seconds = 119.772 FPS
600 frames in 5.0 seconds = 119.964 FPS
600 frames in 5.0 seconds = 119.958 FPS
600 frames in 5.0 seconds = 119.966 FPS
600 frames in 5.0 seconds = 119.933 FPS
600 frames in 5.0 seconds = 119.996 FPS
600 frames in 5.0 seconds = 119.965 FPS
600 frames in 5.0 seconds = 119.963 FPS
600 frames in 5.0 seconds = 119.959 FPS
600 frames in 5.0 seconds = 119.957 FPS
600 frames in 5.0 seconds = 119.980 FPS

Anything like gsync or other driver configuration enabled? This is the first time I've seen the error Couldn't setup vsync: expected interval '1' but got '0'.

Nope, should they be enabled?

I mean it's definitely worth a shot to enable it, but generally it should be disabled.
If you're running i3 on X11 and not sway, it shouldn't be a problem either to run nvidia proprietary drivers.
Which GPU are you using?

@vanor89 Could you try running without any Xorg configuration?

@kchibisov how can I do that?
@chrisduerr, GTX 1070 Strix, I forgot to mention, it runs with no issue in lower resolutions, if I swap to any lower resolution, alacritty runs with no issue.

@vanor89 just temporary remove all .conf files from /etc/X11/ iirc.

Yeah it runs fine, I had to relog b/c I dont know how to restart it but it opened in a lower resolution, and in that resolution I can run alacritty just fine, if I toggle to the higher resolution it continues to fail

if I toggle to the higher resolution it continues to fail

How are you toggling it? Are you enabling everything again?
Could you try to change just the resolution, using xrandr? It's important to note that this issue is different from the original problem.

Well I was opening the nvidia settings and changing only the there, not using xrandr, let me try that

Also, just to clarify, when you change just the resolution, you get the vsync error, not the initial error, right?

sry guys I have work to do right now I'll update you guys as soon as possible

sry guys I have work to do right now I'll update you guys as soon as possible

No worries, thanks for all the quick feedback. Just let us know if you've got more time to provide additional information. There's usually people available on #alacritty on freenode too if you want more immediate discussions.


Wrong version

You should follow the rules http://semver.org
error "vkbeautify" is wrong version: expected "0.99.1", got "0.99.01"

Executed "npm i --save vkbeautify", no warning on output, package.json looks fine:
"dependencies": {
    "vkbeautify": "^0.99.1"
},

Output of "npm view vkbeautify", also looks correct.

Check installed package.json version

"0.99.1".
Feel free to submit a PR with your requested correction. 👍

Or download it directly https://registry.npmjs.org/vkbeautify/-/vkbeautify-0.99.1.tgz
 "version": "0.99.01",
but https://www.npmjs.com/package/vkbeautify displays it as v0.99.1

But you should republish the package :fountain:


Minor bug spotted in a Matlab test function

Hi,
In v.0.9.13, there is a minor bug in vl_test_pattern.m. It calls vlfeat_root.m twice but this function appears to have been renamed to vl_root.m. Other m-files ought to be checked for the same/similar bugs due to file name changes.

Thanks. We are migrating the function from toolbox/test to a new `architecture' toolbox/xtest, so they are obsolete indeed.


Can we use the bypass parameter in the set function

This parameter solve us very big issue

Thank you very much! have a lovely day


Support 3-part paths in DuckDB

resolves #21
History
https://github.com/datafold/sqeleton/pull/22 was an initial attempt at unblocking data-diff --dbt for DuckDB. This is a second attempt.
Happy to iterate on this PR to fix things for DuckDB the proper way 🤗  Just let me know anything that needs to be addressed.
Enabling 3-part fully-qualified names (FQN)
This PR enables all of the following for DuckDB connections:

1-part paths
2-part paths
3-part paths

The latter is especially important for the following reasons:

DuckDB 0.7.0 adds support for attaching multiple databases ✨
It appears that prior to DuckDB 0.7.0, supplying a database/catalog in a 3-part path is allowed and handled gracefully

Implementation details
Both DuckDB and Redshift have lots of similarities with PostgreSQL. So for this implementation, I just copied-pasted two methods from redshift to duckdb:

select_table_schema
_normalize_table_path

Caveats

I only tested with with DuckDB 0.6.1 -- I didn't test this with 0.7.0.
3-part paths worked for me with duckcli 0.2.1 + DuckDB 0.6.1, even when I supplied a random identifier for the database
3-part paths worked for me with dbt-duckdb 1.3.4 + DuckDB 0.6.1, even when I supplied a random identifier for the database
I didn't thoroughly investigate why 3-part paths worked in all of these cases, but I suspect DuckDB 0.6.1 just ignores the database specifier if supplied.

See it in action
Wanna see it in action? Follow the simple instructions here.

Please base your PR on this recently merged PR - https://github.com/datafold/sqeleton/pull/18
Use a similar implementation.  (using lower() is probably wrong for duckdb)
Also add DuckDB to the simple test that it introduces.

@erezsh I took a look at https://github.com/datafold/sqeleton/pull/18 and tried to align with it, namely:

Copying the implementations of select_table_schema and _normalize_table_path from Postgres rather than Redshift
Added DuckDB to the simple test that it introduces

Happy to iterate further as-needed.

File "/home/runner/work/sqeleton/sqeleton/sqeleton/databases/base.py", line 503, in _query_cursor
    c.execute(sql_code)
duckdb.CatalogException: Catalog Error: Table with name duckdb does not exist!
Did you mean "duckdb_types"?

Thanks for sharing that error message from the CI checks @dlawin.
Are there any instructions I can follow for running the CI checks locally?

@dbeatty10 See https://github.com/datafold/data-diff/blob/master/CONTRIBUTING.md#development-setup

1 and 2 part drop statements are working, but seems that 3 part is not:
DROP TABLE "duckdb"."main"."tbl__pti4o" - > duckdb.CatalogException: Catalog Error: Table with name duckdb does not exist!

Hmm I think the issue here is that duckdb doesn't seem to support 3 part paths. Only .

It should. https://duckdb.org/docs/sql/statements/attach#name-qualification
Are you sure you're using the newest duckdb?

seems to be added in 0.7.0

I have not tested how 3-part paths for DELETE statements works for the latest DuckDB (0.7.0), but I did just bump pyproject.toml to 0.7.0.
Could you try re-running the workflow?
More detail

Hmm I think the issue here is that duckdb doesn't seem to support 3 part paths. Only <schema>.<object>

I just did some more testing with DuckDB 0.6 (which is not the newest version).
It does support 3-part paths for SELECT statements, but it does not support 3-part paths for DELETE statements.

Warning: poetry.lock is not consistent with pyproject.toml. You may be getting improper dependencies. Run poetry lock [--no-update] to fix it.

Looks like I need to generate a new lock file after updating pyproject.toml.

Warning: poetry.lock is not consistent with pyproject.toml. You may be getting improper dependencies. Run poetry lock [--no-update] to fix it.

Looks like I need to generate a new lock file after updating pyproject.toml.

Oh I beat you to it. Works for me locally and the tests are passing here


Adding custom instruction support to the hardware.

Hi,
I have implemented 2 custom instructions 1. mod 2. mac and I have also added support for them in the spike which seems to be working fine. Now I want to add the support in hardware . As per my understanding till now I believe adding to the files ibex_pkg.sv, ibex_decoder.sv and ibex_multdiv_fast.sv will be enough. Please suggest if there is any more file that I need to consider and also suggest if there is anything more that I need to consider for adding the support to the Ibex.

Hi,
I am trying a custom instruction and want to add the support in hardware to make it work.
int main()
{

    static const int array[10] = {1,2,3,4,5,6,7,8,9,10};

    int i;
    int sum=0;
    for(i=0; i < 9 ; i++){

    asm volatile
   (
    "mac   %[z], %[x], %[y]\n\t"
    : [z] "+r" (sum)
    : [x] "r" (array[i]), [y] "r" (array[i+1])
   );
        //sum+= multiply(array[i],array[i+1]);

        puts("sum=");
        putdec(sum);
        puts("\n");
   }

    return 0;
}

The instruction support has been added in both tool-chain and ISS (Spike). As far as the hardware is concerned I have modified two files:

<ibex_pkg.sv>: I have defined instruction pattern in such a way that if fall under OPCODE_OP category as  the instruction class in M.

typedef enum logic [2:0] {
  // Multiplier/divider/MAC
  MD_OP_MULL,
  MD_OP_MULH,
  MD_OP_DIV,
  MD_OP_REM,
  MD_OP_MAC
} md_op_e;

<ibex_decoder.sv .>
I have added this portion for the unque case(opcode):

{7'b000_1000, 3'b111}: begin // mac
              multdiv_operator_o    = MD_OP_MAC;
              multdiv_signed_mode_o = 2'b00;
              illegal_insn          = (RV32M == RV32MNone) ? 1'b1 : 1'b0;
            end

I have added this portion for the unique case (opcode_alu):
{7'b000_1000, 3'b111}: begin // mac
              alu_operator_o = ALU_ADD;
              mult_sel_o      = (RV32M == RV32MNone) ? 1'b0 : 1'b1;
            end

Please suggest whether this is the correct way to add the instruction in decoder module.
Regarding<ibex_multdiv_fast.sv> file I can see mac operation is taking place internally with accumuator and summand signals. I am not being able to find the link that how to connect my instruction to that functionality. Now in disassembly it is showing mul and add instruction but I want this operation to show only mac instruction in disassembly . The instruction pattern of mac differs from other M class instruction in  the 7 bit MSB function field.

Hi @subhajit26 that looks like a reasonable way to do things in the decoder, though can't offhand say if it will achieve exactly what you need. Note that for a multiply accumulate you will require a new data path from the multiple unit to the alu to perform the add so further work will be required.
I can't comment on the disassembly side of things. I'm not hugely familiar with the tool internals there plus that's not really an Ibex related issue.

Hi Greg,
"Note that for a multiply accumulate you will require a new data path from the multiply unit to the alu to perform the add so further work will be required."
Paying heed to what you said I was trying to implement the custom mac instruction only to the fsm for RV32MFast.
ALBH: begin
          // al*bh<<16
          mult_op_a = op_a_i[`OP_L];
          mult_op_b = op_b_i[`OP_H];
          sign_a    = 1'b0;
          sign_b    = signed_mode_i[1] & op_b_i[31];
          // result of AL*BL (in imd_val_q_i[0]) always unsigned with no carry
          accum     = {18'b0, imd_val_q_i[0][31:16]};
          //if (operator_i == MD_OP_MULL) begin
          if (operator_i == MD_OP_MAC) begin
            mac_res_d = {2'b0, mac_res[`OP_L], imd_val_q_i[0][`OP_L]};
          end else begin
            // MD_OP_MULH
            mac_res_d = mac_res;
          end
          mult_state_d = AHBL;

        end

        AHBL: begin
          // ah*bl<<16
          mult_op_a = op_a_i[`OP_H];
          mult_op_b = op_b_i[`OP_L];
          sign_a    = signed_mode_i[0] & op_a_i[31];
          sign_b    = 1'b0;
          //if (operator_i == MD_OP_MULL) begin
          if (operator_i == MD_OP_MAC) begin
            accum        = {18'b0, imd_val_q_i[0][31:16]};
            mac_res_d    = {2'b0, mac_res[15:0], imd_val_q_i[0][15:0]};
            mult_valid   = 1'b1;

            // Note no state transition will occur if mult_hold is set
            mult_state_d = ALBL;
            mult_hold    = ~multdiv_ready_id_i;
          end else begin
            accum        = imd_val_q_i[0];
            mac_res_d    = mac_res;
            mult_state_d = AHBH;

          end
        end

#include <stdio.h>
#include "simple_system_common.h"
int main()
{

    int a=3;
    int b=4;
    int sum = 1;

    asm volatile
   (
    "mac   %[z], %[x], %[y]\n\t"
    : [z] "+r" (sum)
    : [x] "r" (a), [y] "r" (b)
   );

        puts("sum=");
        putdec(sum);
        puts("\n");

    return 0;
}

My only concern is that my code is giving me the multiplication for all the values but it is not adding up when I am using MD_OP_MAC  whereas when I am using MD_OP_MULL it is giving the result of mac operation.
It seems like MD_OP-MAC is only giving me the mac_res_d: the intermediary value. Can you please point out the line where it is performing the accumulation for RV32MFast  FSM. I am bit confused about how the accumulator is functioning in the FSM.

Hi there! I'm going to close this because it's been dormant for a while and it's not really clear what we can best do to help you. If you're still working on this and have a more specific question, feel free to re-open or to open a new issue.


[corev2] Review final surface area for core-client, core-https, and core-xml.

For each of the new packages, I'd like the 3 assignees to take a look at the public surface and confirm that it looks like what we expected (no breaking changes required!) before we 1.0 these packages.
@azure/core-client

[ ] @bterlson
[x] @chradek
[x] @jeremymeng

@azure/core-https

[ ] @bterlson
[x] @chradek
[x] @jeremymeng

@azure/core-xml

[ ] @bterlson
[x] @chradek
[x] @jeremymeng

I will collect feedback from the linked APIviews into this issue for tracking purposes once everyone has had a chance to provide feedback. Please tick your username when you're done.
Work to be done:

Fix typo of "expontentialRetryPolicyName" (should be exponential instead)

External Dependencies
core-https
"@opentelemetry/api": "^0.10.2",
"form-data": "^3.0.0",
"https-proxy-agent": "^5.0.0",
"uuid": "^8.3.0"

core-client
"@opentelemetry/api": "^0.10.2"

core-xml
"xml2js": "^0.4.19"


Colors in mermaid import

Awesome plugin!!
I use some colors and classes in my diagrams, but that format is not imported in Excalidraw
Example of class:
classDef premise fill:#2563EB, stroke:#000000

Have you found anything that does this well?


Vulnerability roundup 104: mysql-5.7.27: 1 advisory [5.9]

search, files

[ ] CVE-2021-2356 CVSSv3=5.9 (nixos-21.05, nixos-unstable)

CVE details
CVE-2021-2356
Vulnerability in the MySQL Server product of Oracle MySQL (component: Server: Replication). Supported versions that are affected are 5.7.34 and prior and 8.0.25 and prior. Difficult to exploit vulnerability allows low privileged attacker with network access via multiple protocols to compromise MySQL Server. Successful attacks of this vulnerability can result in unauthorized ability to cause a hang or frequently repeatable crash (complete DOS) of MySQL Server as well as unauthorized update, insert or delete access to some of MySQL Server accessible data. CVSS 3.1 Base Score 5.9 (Integrity and Availability impacts). CVSS Vector: (CVSS:3.1/AV:N/AC:H/PR:L/UI:N/S:U/C:N/I:L/A:H).

Scanned versions: nixos-21.05: 2262d7863a6; nixos-unstable: 1905f5f2e55.

See also: #124653, #113473, #93991, #90914

We no longer support nixos-21.05, but this has been fixed in the currently supported NixOS versions:

nixos-22.05: mysql-8.0.29, mysql-5.7.39
nixos-unstable: mysql-5.7.39, mysql-8.0.30


[REQ] Online G-Colab version

Hi there, this software is very interesting.
We wanna suggest you to build a Google-Colab notebook version in order to let users to try it online.
Check out (and, whly not, ask for help/collaboration) those other interesting virtual notebooks:

MiXLab by @shirooo39;
Codemaster by @mohitjoshi155

Hope that inspires !

Hello, this has been added

Thanks a lot for your work !

Google Colab's UI seems to have its own file upload interface so I just added a screenshot and a form where you can specify the input parameters

Google Colab's UI seems to have its own file upload interface

True, but users may don't know WHERE to load files exactly.
Thanks anyway.


Detect Hack files by content

e.g. vscode is unaware that https://github.com/hhvm/hhast/blob/master/bin/hhast-lint is a Hack file

Hmm, I'm thinking looking for #!.*hhvm.* in the first line will be the easiest fix. Thoughts?

https://github.com/PranayAgarwal/vscode-hack/commit/fc59bab43ec8d2cf2d12e90ee24d869fbece2a71


Module supports using the Deferred Type to execute lookups on the remote node

As a Puppet operator, I want to be able to leverage Puppet's new Differed Type, so that I can use the remote node's identity to retrieve secrets from Conjur, without having those secrets transferred to the master.
GIVEN a Puppet master and an un-configured node which is enrolled in Conjur and have permission to access a credential
WHEN the module is run against the node
THEN credential are placed on the node
AND that credential does not flow through the master
Developer Notes
Overview and example of how this can be done in the upcoming Puppet 6 release: https://gist.github.com/turbodog/06d3fecef403bfefd9c8174ede4d9174

Adding note from #20
To follow Puppet 6 best practices for secrets retrieval, we will update the Puppet integration to use deferred actions by default.

Note: this would be a breaking change for current Puppet integration users that conjurize their agents (a method of bootstrapping host identity that is not documented for Conjur OSS / DAP). These users would have to upgrade to another identity bootstrapping mechanism to use the new version with this change.

Additional notes from the original definition of #20
With the arrival of Puppet 6's deferred data types, we can move credential retrieval off the Puppet master and onto the node being configured. Deferred data types reduce the security challenges by eliminating the Puppet Master as a conduit for credential delivery.
This effort will address the following areas:

Credentials delivered through the deferred data type, falling back to the current approach for Puppet version < 6.
Puppet module can be run on the following operating systems:

Windows
Red Hat Linux
CentOS
Ubuntu
Debian

README is updated to include examples for the deferred data type

References

Docs on deferred actions
Puppet 6 best practices for secrets retrieval

Related issues

[ ] Module supports using the Deferred Type to execute lookups on the remote node - parent of #13
[ ] Module supports a variety of common Operating Systems - parent of #23

There is an initial implementation of this in #179. This issue will remain open until a more complete implementation is done.

Per the new code that is now on the main brach, method signature has to change to accommodate the Deferred use.
New signatures:

Using agent-side identity:

$dbpass = Sensitive(Deferred(conjur::secret, ['production/postgres/password']))

Using server-provided configuration:

$sslcert = @("EOT")
-----BEGIN CERTIFICATE-----
...
-----END CERTIFICATE-----
|-EOT

$dbpass = Sensitive(Deferred(conjur::secret, ['production/postgres/password',
  "https://my.conjur.org",
  "myaccount",
  "host/myhost",
  Sensitive("2z9mndg1950gcx1mcrs6w18bwnp028dqkmc34vj8gh2p500ny1qk8n"),
  $sslcert
]))

The longer signature may also change again depending on if #184 gets implemented.

Released with v3.0.0rc3. Closing.


bug: there is no pagination for slides

Describe the bug
There is no pagination for the slides on home page.
To Reproduce
https://github.com/user-attachments/assets/cc0ee071-12c6-4543-942e-5ecf7e3f325a
Expected behavior
https://github.com/user-attachments/assets/b53eaa71-3667-426d-b49e-32f1a64807a6
Info (please complete the following information):

Browser [e.g. chrome, safari]
Version [e.g. 22]

Hey! @toufiqfarhan0 I want to contribute on this one

Hey! @toufiqfarhan0 I want to contribute on this one

I have already raised an PR. #610


What permissions are needed for creating a new project scoped feed

I am a project administrator and also build administrator.  But I do not have an option to Create a new project scoped feed in spite of being a project administrator and a build administrator.  What permissions are required to enable Create New Feed option?  This page does not provide this information either

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

ID: 0e77774e-012a-45cc-d566-7170d86003cd
Version Independent ID: 778dafea-bf98-3292-f451-d82ce8db5bbc
Content: Secure and share packages using feed permissions - Azure Artifacts
Content Source: docs/artifacts/feeds/feed-permissions.md
Product: devops
Technology: devops-artifacts
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@srikcgaa2 -- here are a couple of options where you might consider asking your question:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

@WilliamAntonRohm
Thanks for the suggestions about the Support Bot and Stack Overflow.  It
would be great if this page can also provide this information requested so
that it is clear how the Create Feed permissions are controlled as well.
On Fri, Apr 10, 2020 at 9:19 PM William Anton Rohm<EMAIL_ADDRESS>wrote:

@srikcgaa2 https://github.com/srikcgaa2 -- here are a couple of options
where you might consider asking your question:

Azure DevOps Support Bot
https://azuredevopsvirtualagent.azurewebsites.net/
Azure DevOps on Stack Overflow
https://stackoverflow.com/questions/tagged/azure-devops

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-devops-docs/issues/7874#issuecomment-612089082,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF6FQN7YH2BHT64FS7BNOOTRL45X5ANCNFSM4MEQANDQ
.

@srikcgaa2 Thank you for your feedback. All feeds created through the web user interface are project scoped feeds.
Please follow the below quickstarts to create a new feed:

NuGet

NPM

Maven

Python

Universal

The issue reported was not resolved.  The issue was that the documentation is missing details about permissions needed to creat a new Feed and where to set such permissions for additional members of the team.  This was not addressed but now issue is closed.

@srikcgaa2 -- thank you for clarifying.
@ramiMSFT -- please look into this issue.

@srikcgaa2 -- thank you for clarifying.
@ramiMSFT -- please look into this issue.

@srikcgaa2 by default, all users in the same organization have permissions to create feeds. Check with your Organization owner or Project Collection Administrators group. Azure Artifacts settings allow to specify who can create and administer feeds.
See Azure Artifacts settings for more details.

It's very important to understand the difference between feed, project, and project collection administrators.
A project administrator on the other hand has permissions to administer all aspects of teams and project (delete team project, update project visibility, manage test environments etc.).
Project Collection Administrators are granted all collection-level permissions to manage resources for projects and project collections (add and delete projects, trigger events, manage build resources and audit streams etc.).


Caller ID for Yap

We would like the Twilio number we bought that Yap uses to call Helpline volunteers to show up with our choice of text such as "NA Helpline"

You should contact Twilio and ask them if it's possible to do this.

Sure, will do.
From: Danny Gershman<EMAIL_ADDRESS>Sent: Monday, February 22, 2021 4:26 PM
To: bmlt-enabled/yap
Cc: Paul Rinear; Author
Subject: Re: [bmlt-enabled/yap] Caller ID for Yap (#512)
You should contact Twilio and ask them if it's possible to do this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/bmlt-enabled/yap/issues/512#issuecomment-783685465, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADVAGHMIRLSAV7U7QCKVOUTTALDW5ANCNFSM4YBH3T3A.

Twilio can do this. Contact support and have them change the CNAM. 15
characters max.
Steve
On 2/22/2021 3:08 PM, Redguy1959 wrote:

We would like the Twilio number we bought that Yap uses to call
Helpline volunteers to show up with our choice of text such as "NA
Helpline"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bmlt-enabled/yap/issues/512, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHRR4UEVZFRRUXTQUEXWY6DTALBUVANCNFSM4YBH3T3A.

Thanks!
From: Steve Scranton<EMAIL_ADDRESS>Sent: Monday, February 22, 2021 5:35 PM
To: bmlt-enabled/yap
Cc: Paul Rinear; Author
Subject: Re: [bmlt-enabled/yap] Caller ID for Yap (#512)
Twilio can do this. Contact support and have them change the CNAM. 15
characters max.
Steve
On 2/22/2021 3:08 PM, Redguy1959 wrote:

We would like the Twilio number we bought that Yap uses to call
Helpline volunteers to show up with our choice of text such as "NA
Helpline"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bmlt-enabled/yap/issues/512, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHRR4UEVZFRRUXTQUEXWY6DTALBUVANCNFSM4YBH3T3A.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/bmlt-enabled/yap/issues/512#issuecomment-783725185, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADVAGHLCAQ5XVE6GPCJARJDTALL2DANCNFSM4YBH3T3A.


how to actually use it?

I am guessing that we need to add it to our .prettierrc? Here is mine below
{
  "semi": true,
  "tabWidth": 2,
  "useTabs": false,
  "singleQuote": true,
  "printWidth": 160,
  "trailingComma": "all",
  "plugins": ["prettier-plugin-sort-json"]
}

in VSCode, object like these:
const test = {
  b: 1,
  z: 2,
  a: 11,
}

are not sorted on save

From Prettier Offical, it says:

Plugins are automatically loaded if you have them installed in the same node_modules directory where prettier is located. Plugin package names must start with @prettier/plugin- or prettier-plugin- or @/prettier-plugin- to be registered.

I've also tried using this plugin on TS project, but it does not seem to work. All it does was sort .prettierc config.

It’s meant for JSON files, not for JavaScript/TypeScript. That’s why it doesn’t work on const test = ... above. See https://github.com/leo-buneev/eslint-plugin-sort-keys-fix instead.

Thanks @alifsabrani and @lensbart ! Together you have correctly answered the question.
I will update the README to include clearer instructions. I'll leave this issue open to track that work.

I’d think that having JSON in the package name would be sufficiently explanatory 😅

The README has been updated in #97. Please make a new issue if you find anything else confusing. Thanks!


Non-blocking UI during scene loading

When starting with a small scene and then programmatically adding a few hundred model tags with remote assets cause the UI to block sporadically during the loading phase of those (pure XML) assets.
This is probably caused by computationally heavy callbacks (after XHR response) blocking the UI thread for the length of several frames. This breaks the user experience when continuously interacting with the scene from the beginning.
This might be solved by a two step approach:

minimize computational effort within XHR callbacks

add a global task queue to resource manager
when XHR returns, the callback just adds a new task to this queue
in each frame, a small number of tasks is taken from the queue and actually executed

perform actual computation asynchronously using other than UI thread

instead of executing the tasks in each frame yourself, manage a pool of workers that
picks regularly a task from the queue
and executes each task without affecting the UI thread

Requests are still handled on the UI thread, but with the integration of the Fetch API they're now queued and processed between frames. This should improve the interactivity of the scene during loading.


release 1.0.1

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of Changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Current dependencies on/for this PR:

dev

PR #23   👈

This stack of pull requests is managed by Graphite.


Update primary site helm chart

Updates the primary site helm chart and enables passing of configuration information via env vars.

@defunctzombie they are the default in the app. I can update the comment to indicate that. Do you mean have them always defined? I feel like that would be weird for non-AWS users, browsing their env vars or something.

I meant they should be the defaults in the app. If they are the defaults then as a user my feedback is that I am confused if I need to set these or not. The comment makes me think its pretty important for me to tune these since it says
these parameters may be useful for tuning performance of object copies.

Based on what you've said I now find these comments somewhat misleading since they don't tell me when should tune these.


Merge forc-projects-experimental with forc-projects in fuel-gauge

Based off of comments in #1832, we merge forc-projects-experimental and forc-projects within fuel-gauge. This work should no longer be considered experimental.
This should include the following changes:

Merge the forc project directories
Have a version prefix inside forc-projects (i.e. v0 and v1)
Integrate this with the getFuelGaugeForcProject utility function and incorporate it in the experimental tests

This should probably go after:

#2017


main 分支<EMAIL_ADDRESS>build报错

><EMAIL_ADDRESS>build
> tsc && cross-env NODE_OPTIONS=--max-old-space-size=6096 vite build

vite v2.9.18 building for production...
transforming...
Module level directives cause errors when bundled, 'use client' was ignored.
Module level directives cause errors when bundled, 'use client' was ignored.
✓ 9474 modules transformed.
[vite]: Rollup failed to resolve import "#minpath" from<EMAIL_ADDRESS>This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
`build.rollupOptions.external`
error during build:
Error: [vite]: Rollup failed to resolve import "#minpath" from<EMAIL_ADDRESS>This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
`build.rollupOptions.external`
    at onRollupWarning (/var/jenkins_home/workspace/fe/node_modules/vite/dist/node/chunks/dep-0a035c79.js:41797:19)
    at onwarn (/var/jenkins_home/workspace/fe/node_modules/vite/dist/node/chunks/dep-0a035c79.js:41613:13)
    at Object.onwarn (/var/jenkins_home/workspace/fe/node_modules/vite/node_modules/rollup/dist/shared/rollup.js:23216:13)
    at ModuleLoader.handleResolveId (/var/jenkins_home/workspace/fe/node_modules/vite/node_modules/rollup/dist/shared/rollup.js:22466:26)
    at /var/jenkins_home/workspace/fe/node_modules/vite/node_modules/rollup/dist/shared/rollup.js:22427:26
Build step 'Execute shell' marked build as failure
Finished: FAILURE

我本地清理了如下目录依然报错
rm -rf $WORKSPACE/node_modules
rm -rf $WORKSPACE/package-lock.json
npm cache clean --force

rm -rf $WORKSPACE/package-lock.json
这是 react-markdown-preview 包依赖的  vfile 包近期升级导致的问题，基于我们的 package-lock.json 去安装依赖不会报错。

按上面方法解决了，谢谢

已解决


Can this be used with any autogressive model?

Hey @jy-yuan, thank you for the awesome paper and code. Is the method/code only applicable for llama or can it be used with any autoregressive model? If it's the latter,  are there instructions on how to quantize the kvcache of an arbitrary transformer model?

Thank you for interesting in our work. Yes, it can be applied to any autoregressive model!
You can think KiVi as another implementation of KVCache. You can copy-paste the code here and modify it for the attention module in different models like MistralAttention.
Here we would like to note two important things we found when we extend KiVi to different models.

Transformer Package Version: Please double check the transformer package version. We test our implementation in 4.35.2. Yet in version >= 4.36, the KVCache data structure is changed.

Attention Implementation Variants: Please double check what is the attention mechanism used in the model (Multi-Head/Multi-Query/Group-Query). Currently we only release the CUDA and triton code for supporting Multi-Head Attention. For Multi-Query/Group-Query,  it needs to change the low-level implementation a little bit. We will release this part soon.

Stay tuned for further developments!


walkReluctance to 2

Summary of changes
no ticket decreasing walkReluctance param to improve trip plan suggestions.

didn't improve suggestions closing for now


SSI include fails with pagespeed in more complex scenarios

I ran into some issues with SSI support when doing testing with pagespeed. Using Nginx 1.13.9 and ngx_pagespeed <IP_ADDRESS>-0, built together using the provided automated script. First I'll describe some scenarios where I managed to trigger problems with pagespeed and at the end you'll find a couple crash stack traces from another installation.
Two scenarios:

trying to include a page that does not exists and the error page is redirected to another server (another server config on the same Nginx instance)
including a file with "virtual" directive, so that the included file triggers another location configuration block than the original request (see example below for better explanation)

Here is my nginx config (relevant parts):
http {
  pagespeed on;
  pagespeed FileCachePath "/some/path/nginx_pagespeed/ps_cache";
  pagespeed EnableFilters combine_css,combine_javascript;

  server {
    listen       8080;
    server_name  localhost;
    root   /path/to/webserver/root;

    location / {
      index  index.html;
      ssi on;
    }

    location /ssi {
      root /path/to/webserver/root;
    }

    error_page 404 = @notfound;
    location @notfound {
      proxy_pass http://localhost:6666;
    }
  }

  server {
    listen 6666;
    server_name failfailfail;
    root   /path/to/webserver/root;

    error_page 404 /404.html;
    location /404.html {
        internal;
    }
  }
}

For the first scenario, I have the following HTML file in the webserver root directory:

  <title>SSI!</title>

  From SSI:

The request with pagespeed enabled looks like this:

From SSI:
‹%ŽAÃ ï¼bû%GËOÈ@¸)…Š:móû¢ø8«ÕìRÖçÎŽ²„Ä -º¯MñhGMä-päB±¥ójæ‰—û2òÉpæu+õíE>ÒOäÐâ¡Ð,x…Mð oÔ!×†(6qŠy\ðft]ú“ÁTš

When I disable the error page redirection or pagespeed, the response correctly displays a 404 error under the "From SSI:" line. This problem is somewhat scary, because the output looks like random memory contents.
The second scenario uses the following HTML file:

  <title>SSI!</title>

  From SSI:

In this case the response only contains the line "From SSI:", nothing gets included. No errors in the Nginx error.log either. If I disable pagespeed it works, and interestingly, it also works if I add "ssi on;" directive to the "location /ssi" block. This should not be necessary and as mentioned, everything works correctly without pagespeed.
Then the background: I started looking at this issue after receiving some core dumps where a module that I'm developing was suspected of crashing Nginx. What I found out was that it was pagespeed that was crashing and its context structure stored in the request object seemed totally corrupted. Everything else in the dump seemed correct, so it is very likely a pagespeed bug. Unfortunately I'm not allowed to include the full dumps or configuration here because they come from a customer, but here are the two stack traces that I have:
First a segfault:
nginx!net_instaweb::(anonymous namespace)::in_place::ps_in_place_body_filter(ngx_http_request_s*, ngx_chain_s*)+f8 /usr/lib/nginx/ngx_pagespeed/src/ngx_pagespeed.cc:2547
nginx!net_instaweb::ps_base_fetch::ps_base_fetch_filter(ngx_http_request_s*, ngx_chain_s*)+48 /usr/lib/nginx/ngx_pagespeed/src/ngx_pagespeed.cc:259
nginx!net_instaweb::(anonymous namespace)::html_rewrite::ps_html_rewrite_body_filter(ngx_http_request_s*, ngx_chain_s*)+71 /usr/lib/nginx/ngx_pagespeed/src/ngx_pagespeed.cc:2381
nginx!ngx_http_postpone_filter+5e /home/downloads/nginx-setup/nginx/src/http/ngx_http_postpone_filter_module.c:82
nginx!ngx_http_ssi_output+7c /home/downloads/nginx-setup/nginx/src/http/modules/ngx_http_ssi_filter_module.c:936
nginx!ngx_http_ssi_body_filter+142c /home/downloads/nginx-setup/nginx/src/http/modules/ngx_http_ssi_filter_module.c:805
nginx!ngx_http_charset_body_filter+42 /home/downloads/nginx-setup/nginx/src/http/modules/ngx_http_charset_filter_module.c:557
nginx!ngx_http_sub_body_filter+30 /home/downloads/nginx-setup/nginx/src/http/modules/ngx_http_sub_filter_module.c:299
nginx!ngx_output_chain+745 /home/downloads/nginx-setup/nginx/src/core/ngx_output_chain.c:214
nginx!ngx_http_copy_filter+130 /home/downloads/nginx-setup/nginx/src/http/ngx_http_copy_filter_module.c:154
nginx!ngx_http_range_body_filter+3b /home/downloads/nginx-setup/nginx/src/http/modules/ngx_http_range_filter_module.c:627

And a SIGABRT after an internal pagespeed sanity check fails:
#0  0x00007f31c23e11f7 in raise ()
#1  0x00007f31c23e28e8 in abort ()
#2  0x00000000004e4239 in base::debug::BreakDebugger() ()
#3  0x00000000004b5184 in (anonymous namespace)::LogMessageHandler (severity=4, file=<optimized out>, line=<optimized out>, message_start=<optimized out>, str=...) at /usr/lib/nginx/ngx_pagespeed/src/log_message_handler.cc:87
#4  0x00000000004e6d30 in logging::LogMessage::~LogMessage() ()
#5  0x00000000004bb051 in net_instaweb::(anonymous namespace)::ps_send_to_pagespeed (r=r@entry=0x50bed70, ctx=ctx@entry=0x402e510, cfg_s=cfg_s@entry=0x32dcee8, in=in@entry=0x3ccdc40) at /usr/lib/nginx/ngx_pagespeed/src/ngx_pagespeed.cc:2107
#6  0x00000000004bb78c in net_instaweb::(anonymous namespace)::html_rewrite::ps_html_rewrite_body_filter (r=<optimized out>, in=0x3ccdc40) at /usr/lib/nginx/ngx_pagespeed/src/ngx_pagespeed.cc:2394
#7  0x000000000048637c in ngx_http_postpone_filter (r=<optimized out>, in=<optimized out>) at src/http/ngx_http_postpone_filter_module.c:82
#8  0x0000000000486844 in ngx_http_ssi_output (r=r@entry=0x4d566e0, ctx=ctx@entry=0x3ccd508) at src/http/modules/ngx_http_ssi_filter_module.c:934
#9  0x00000000004899bc in ngx_http_ssi_body_filter (r=<optimized out>, in=<optimized out>) at src/http/modules/ngx_http_ssi_filter_module.c:805
#10 0x000000000048abbb in ngx_http_charset_body_filter (r=0x4d566e0, in=0x3ccd900) at src/http/modules/ngx_http_charset_filter_module.c:557
#11 0x000000000048c419 in ngx_http_sub_body_filter (r=0x4d566e0, in=0x3ccd900) at src/http/modules/ngx_http_sub_filter_module.c:299
#12 0x0000000000433965 in ngx_output_chain (ctx=ctx@entry=0x3ccd898, in=in@entry=0x3ccd7d8) at src/core/ngx_output_chain.c:214
#13 0x000000000048ead8 in ngx_http_copy_filter (r=0x4d566e0, in=0x3ccd7d8) at src/http/ngx_http_copy_filter_module.c:152
#14 0x0000000000484d61 in ngx_http_range_body_filter (r=0x4d566e0, in=<optimized out>) at src/http/modules/ngx_http_range_filter_module.c:627

Note that these are NOT dumps from the above scenarios, the test cases were simply some problems I managed to find when performing different kinds of tests. The customer is using Nginx 1.12.1 and pagespeed version <IP_ADDRESS>-0.

The segfault probably happens when dereferencing the recorder here:
https://github.com/apache/incubator-pagespeed-ngx/blob/f3c64ca8a251f05f228ad32947d7c68c89308f6a/src/ngx_pagespeed.cc#L2547
In the second trace, this CHECK fires:
https://github.com/apache/incubator-pagespeed-ngx/blob/f3c64ca8a251f05f228ad32947d7c68c89308f6a/src/ngx_pagespeed.cc#L2107

Detailed error logs at the debug level for these scenarios would be helpful in analysing this more -- if you are able to provide them (anonimized).
I do have a suspicion. ngx_pagespeed may need to ensure it maintains a single request context when nginx restarts the requests in this scenario to function properly. That may be quite tricky, but I did do a PoC for that at one point which was quite hacky and passed the context around in the request headers internally. It seems doable, but I really wouldn't be happy to merge it that way. But perhaps we can create an internal nginx parameter or something like that to persist our context across restarted requests.
One other upside, outside of this scenario, is that doing so may save us per-request cache-lookups in common configuration scenarios.


Best performance weights

This code saves the generator and discriminator weights on every epoch during training of the model. Is the weight of last epoch is best? How can we check the performance of the trained model? What should be the loss value of generator and discriminator for best performance ?

Hi, the model obtained from the last epoch does not necessarily mean the best model, considering that the model may be overfitting.
According to my actual experience in GAN training, you can see the convergence of the generator and discriminator through the loss, but more importantly, you need to evaluate the performance of the model using inference on the test data, where you can calculate various metrics for the selection of the optimal model.

Hello sir,
Thank you very much for your response. I am new in machine learning and
still learning about GAN from basic. Now I am clear about the evaluation of
model. And I have one more query. Is the performance more better using
LeakyReLU activation instead of PReLU? The SEGAN original paper uses PReLU
in generator part.
Thank you.
On Wed, 31 Jul 2024 at 21:05 Hao Zhao @.***> wrote:

Hi, the model obtained from the last epoch does not necessarily mean the
best model, considering that the model may be overfitting.
According to my actual experience in GAN training, you can see the
convergence of the generator and discriminator through the loss, but more
importantly, you need to evaluate the performance of the model using
inference on the test data, where you can calculate various metrics for the
selection of the optimal model.
—
Reply to this email directly, view it on GitHub
https://github.com/haozh7109/SEGAN-TensorFlow2/issues/1#issuecomment-2260776946,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKKOZMJACGKUBG6OCCGE5JDZPD6C3AVCNFSM6AAAAABLYESY3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRQG43TMOJUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I haven't specifically compared Prelu and leaky relu in SEGAN. But maybe parameterized relu gives better results, or introduces instability because it comes from training. You can run enough experiments to compare the results.

Thank you for your suggestion. Can you suggest any evaluation tool? ITU
said PESQ is also outdated. POLQA is licensed. Is there any other option?
On Thu, Aug 1, 2024 at 2:53 PM Hao Zhao @.***> wrote:

I haven't specifically compared Prelu and leaky relu in SEGAN. But maybe
parameterized relu gives better results, or introduces instability because
it comes from training. You can run enough experiments to compare the
results.
—
Reply to this email directly, view it on GitHub
https://github.com/haozh7109/SEGAN-TensorFlow2/issues/1#issuecomment-2262505867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKKOZMJHWMPZ3PGCZ7KWM33ZPH3KFAVCNFSM6AAAAABLYESY3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRSGUYDKOBWG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


How can I freeze first row?

Like this:
worksheet.views = [
{state: 'frozen', ySplit: 1}
];

I tried the newest code, and found the 'views' is moved to options, but the document is not updated accordingly, so anyone use the newest version should try below code:
workbook.addWorksheet(sheetName, {
views: [
{state: 'frozen', ySplit: 1}
]
});

What if i want freeze not first but only second row.
When i use ySplit: 2 will be frozen first and second row but i want only second not first.
Is this possible?

What if i want freeze not first but only second row.
When i use ySplit: 2 will be frozen first and second row but i want only second not first.
Is this possible?

I have same question, how to freeze selected cells

What if i want freeze not first but only second row.
When i use ySplit: 2 will be frozen first and second row but i want only second not first.
Is this possible?

I have same question, how to freeze selected cells

Has anyone been able to find a solution? I'm trying this as well, and even when setting up a template that I am filling out through my code I can't find a solution or viable work-around.

What if i want freeze not first but only second row.
When i use ySplit: 2 will be frozen first and second row but i want only second not first.
Is this possible?

I have same question, how to freeze selected cells

Has anyone been able to find a solution? I'm trying this as well, and even when setting up a template that I am filling out through my code I can't find a solution or viable work-around.

didn't find anything useful yet

Any update on freezing other rows than the first? I've been searching for a while now.

@gergoh I never found a solution. Luckily for me, I was able to convince my team that we needed to find another design to go with.

@gergoh I never found a solution. Luckily for me, I was able to convince my team that we needed to find another design to go with.

Seems like, I need to do the same thing. ExcelJS seems more and more limited as I work with it. It's really unfortunate.

@gergoh honestly from the research I did while working on this at the time, it's more of a limitation of Excel itself. If I recall correctly, Excel is capable of freezing the first row (or multiple rows, given that there are no unfrozen rows between the first row and the end of the frozen rows) or freezing columns (though I did not look much into this part of the feature since I had no use for it). If there is ever an update to Excel to allow for more dynamic freezing of rows, then I wouldn't be surprised to see an update to ExcelJS to account for it. It seems right now the biggest roadblock is the default software that the package is using, if that makes sense.

I'm trying both
worksheet.views = [ {state: 'frozen', ySplit: 1} ];
and
workbook.addWorksheet(sheetName, { views: [ {state: 'frozen', ySplit: 1} ] });
but it doesn't work.
I also get the "We found a problem with...Do you want to recover as much.." warning when opening up the spreadsheet.
Anyone getting this to work? All I want to do is freeze the 7th row. I can do this in Excel itself


add sylius 1.8

Hello,
I suggest to add Sylius 1.8 which has recently been released.

Closed in favor of another PR with another approach.


Great bq_scripts refactor

[ ] Type annotations for all of the code
[ ] All missing Docstrings
[ ] Factor out all BigQuery Schemas
[ ] Factor out all SQL queries
[ ] Factor out NumpyEncoder from dump_ingest_info
[ ] Clean up commented code

During PR #101 the following comment is highlighted:
"Unrelated to this PR, but we should never simply catch a broad exception, print the error message, and move on. Let's fix this during the great big refactor"

Another comment: make sure to get rid of print statements and establish an appropriate logger.


[gardening] Add a missing const to a const method.

Found by inspection.

@swift-ci smoke test and merge


Fix: complete_action needs branch_name and ref

look like 2c89d4d9c4edb1467535fcce52a7b1d3c6d1d448 forgot to update it

Thanks for all the PRs this evening!


Category search does search for my whole string

I typed "apple"
I got the results for "app"
The "MEDIA" tab is OK, though, showing pictures of apples.
Adding another letter then removing it solves the problem

Difficult to reproduce.

The search works fine most of the time. I think this is a very rare issue. Maybe due to Searchview it is showing wrong sometimes.

I have not seen this for a while, I will reopen if I observe it again.

Hi Nicolas,
I tried to reproduce this issue to figure out what is the reason and I found below reason for this issue.
When I want to search apple  but before finishing the writing apple loader starts to fetch the data from the server with just two or three letters (because of little dealy in my typing) it displays wrong results.
For example

I typed app  and loader started to fetch the data because of little delay in typing le
Before finishing the loader(which fetching the data for app) I completed the text and wrote **apple **.
Now the results will be wrong.

I wanted to work on this issue.
Could you please assign it to me?
Regards,
Deepa

Hi Deepa, Feel free to pick the issue. It is yours :-)

Thanks Ashish :)

Hi Ashish,
Below are the steps to reproduce it.
For example I want to get the result for Apple

Type app and let the loader starts
Here is the screen shot

Once loader starts immediately type remaining letters for example "le" in my case.

Now see the result. The result is not for Apple

@DeepaMalani Thanks for reproducing! Now please run the app in debug mode and see why the previous search is not aborted when the user type another letter :-)

Hi Nicolas,
I ran the app in debugging mode and figured out that the previous search will be aborted if the API call to fetch the results haven't started for the previous search, but once the API call starts we can not send the latest search text midway to API.
The solution which I think will work is once the API call starts let it finish to fetch ing the data and after that send the latest search query to API.

Thanks for the investigation!
Are you working on implementing the fix? 🙂
On Sun, 1 Dec 2019, 16:13 Deepa Malani<EMAIL_ADDRESS>wrote:

Hi Nicolas,
I ran the app in debugging mode and figured out that the previous search
will be aborted if the API call to fetch the results haven't started for
the previous search, but once the API call starts we can not send the
latest search text midway to API.
The solution which I think will work is once the API call starts let it
finish to fetch ing the data and after that send the latest search query to
API.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/issues/1650?email_source=notifications&email_token=AAAYKBWIDLG4QKTBX2NUN4LQWRHLDA5CNFSM4FGIIDKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFR2NKQ#issuecomment-560178858,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYKBSTNZCGAP3F7HBWGETQWRHLDANCNFSM4FGIIDKA
.

Hi Nicolas,
I have worked on this issue and already have submitted a pull request for this issue (#3200 ).

@nicolas-raoul Is this issue still exists now? I tried several times but can't reproduce it.

It is very hard to reproduce.
I haven't seen it for a long time, so I will close for now, but please anyone comment if you get this issue, thanks!


chore(package): update nyc to version 10.3.2

Coverage remained the same at 86.325% when pulling 02a9bac618a2af60d64acc0a039815d34b748bb1 on greenkeeper/nyc-10.3.2 into e2232412bd06daf0cb08327d767de672e1fde7b7 on master.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #95   +/-   ##
=======================================
  Coverage   86.32%   86.32%
=======================================
  Files          16       16
  Lines         117      117
=======================================
  Hits          101      101
  Misses         16       16

Continue to review full report at Codecov.

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


style(server): linter fix get imageFqn

Signed-off-by: Youngone Lee<EMAIL_ADDRESS>@petermetz ping for review, I think I should have fixed both the getClass and imageFqn warnings at once but I didn't know if I could have done that until I asked Mr.Ty this morning. Please excuse the amount of PRs!

@Leeyoungone It's okay, you'll know for next time so no biggie!


SshUtils.ps1 missing from repo

System Details
N/A
Issue Description
From commit 57aa9c1eab1ab1957a7805e821a746a34be8b1d5 src/SshUtil.ps1 seems to be missing from the repo. There apears to be a bad forced merge as the file has disappeared (with no record of it being deleted in the later commits).

Ah - new day, fresh eyes.
I can see the commit where the SSH utils were removed now. It makes more sense.
When I couln't see the commit it looked like the refactor where they'd been moved to src/SshUtils.ps1 had been forced out.
Sorry! Thanks for your help.


Docs on authentication don't contain samples of Web API projects

The documentations in the Authentication section don't contain samples of Web API projects.
Most of the content are in the scope of Web App.

Duplicate of #10064


Update answers to the choice of data type field.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #674
Description
Choice data type is added value[X] (e.g valueQuantity ...)
Answer is updated to the nested field of choice data type  (e.g Quantity.value, Quantity.code ...)
Alternative(s) considered
No
Type
Choose one: (Bug fix)
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read How to Contribute
[x] I have read the Developer's guide
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

Codecov Report

Merging #676 (8f575ea) into master (e1b48f2) will decrease coverage by 0.12%.
The diff coverage is n/a.

:exclamation: Current head 8f575ea differs from pull request most recent head 92d8635. Consider uploading reports for the commit 92d8635 to get more accurate results

@@             Coverage Diff              @@
##             master     #676      +/-   ##
============================================
- Coverage     53.92%   53.79%   -0.13%
+ Complexity      173      172       -1
============================================
  Files            44       44
  Lines           816      816
  Branches        117      117
============================================
- Hits            440      439       -1
  Misses          309      309
- Partials         67       68       +1

Impacted Files
Coverage Δ

...va/com/google/android/fhir/db/impl/DatabaseImpl.kt
35.55% <0.00%> (-2.23%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM. Please merge the changes once builds pass.


[FR] Add support for valkey

Realizing it's early days, but now that the Linux Foundation officially launched valkey, my first thought was to replace unbound's redis backend with it.
Any plans/roadmap on the horizon?

Unbound uses the libhiredis library for the cachedb module. That is a C client interface. It seems to be in the valkey repo at valkey/deps/hiredis.
I would think that it would work straight away. Since it is a copy of the externally developed hiredis library and that is what unbound uses, I guess that would work without needing changes in Unbound. Unbound has support to be able to support different backends for the cachedb module. It can be configured with the backend: "redis" option in the cachedb: section of the config.


Revert zcashd Orchard module structure changes

This PR reverts the Orchard module structure changes from #22.
It is based on https://github.com/zcash/zcash/pull/5373
We should update it and merge as part of https://github.com/ZcashFoundation/zebra/issues/2982
Closes #26.

Obsoleted by #29.


test-bad-hex-rule-1: add rule with incomplete hex

Add a rule with incomplete hex, for example "|22 2 22|" which
should result in a parse error.
Ticket #5201.

Suricata PR was merged, could you retry CI ?

Merged.


sendBuilt query parameter added to send simJSBuiltInfo to the kiosk

Kiosk games take awhile to load because they are getting compiled each launch. To decrease load time for the kiosk, we want to just send the game build instead of compiling each time in the runner. The addition of the sendBuilt query parameter provides this functionality.
The other part of this change can be found in pxt-arcade in this PR:  https://github.com/microsoft/pxt-arcade/pull/5175

LGTM, the changes outside run.html aren't necessary though / two minor nits in run.html~

I was getting in-editor and runtime errors when I didn't have the other locations were not defined. Are they something I can ignore?


fix(header): search should be always visible in PC UI

https://github.com/Magickbase/ckb-explorer-public-issues/issues/472
https://github.com/Magickbase/ckb-explorer-public-issues/issues/473

The layout logic has changed, which is likely unexpected.
New:

Original:

The layout logic has changed, which is likely unexpected.
New:

Original:

This is the reason here, but why? I didn't find it in the design draft

The layout logic has changed, which is likely unexpected.
New:
image
Original:
image
Uploading image.png…
This is the reason here, but why? I didn't find it in the design draft

Images are broken

https://github.com/Magickbase/ckb-explorer-frontend/blob/db52e40231355f2b8e6db8fb109b0e6a306855de/src/index.css#L46

https://github.com/Magickbase/ckb-explorer-frontend/blob/db52e40231355f2b8e6db8fb109b0e6a306855de/src/index.css#L46

Which comment is this replied to

Conflicted

Notice the layout at 0:22 and 0:33
0:22, items in the header are connected to each other
0:33, search field in the dropdown is connected to the border, but a span is added when the width is smaller

The search box switching is weird.
Whether to switch to the top only when the lower search box is completely blocked, or to hide the top search box when the search box is about to appear on the page.
https://github.com/Magickbase/ckb-explorer-frontend/assets/25676576/9a8044f4-a190-4cc7-973f-3facd670a0f3

Notice the layout at 0:22 and 0:33 0:22, items in the header are connected to each other 0:33, search field in the dropdown is connected to the border, but a span is added when the width is smaller

for 0:22, different from the design draft.

there is an extra item in the search component, but not found in desgin draft.
I found that the size between 1024px and 1920px is not listed in the design draft, so I just make them to auto adjust

Changing the resolution will also cause copywriting overlap.
At 1024px * 764px, the search box will suddenly change size, but the default information in the search is gone, and the cursor will not appear when clicking the search box.
In Screenshot 2, it is found that the width of the top bar has changed.

2、The search box size on the mobile phone is abnormal.

This problem still exists.


Add ability to pass logger instance to frameworks

https://github.com/iterative/dvclive/pull/302 followup to do the same for other loggers

it simplifies the Keras test

Next steps:

MMCV requires a separate PR in their repo
Lighting logger looks very different from the others + some cleanup can be done. But it already has a way to pass a live instance via experiment parameter
Update docs

Codecov Report
Base: 94.88% // Head: 95.01% // Increases project coverage by +0.12% :tada:

Coverage data is based on head (c8367e7) compared to base (129c8fa).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #318      +/-   ##
==========================================
+ Coverage   94.88%   95.01%   +0.12%
==========================================
  Files          36       36
  Lines        1702     1726      +24
  Branches      189      145      -44
==========================================
+ Hits         1615     1640      +25
  Misses         64       64
+ Partials       23       22       -1

Impacted Files
Coverage Δ

src/dvclive/catalyst.py
100.00% <100.00%> (ø)

src/dvclive/fastai.py
93.33% <100.00%> (+0.47%)
:arrow_up:

src/dvclive/huggingface.py
100.00% <100.00%> (ø)

src/dvclive/lgbm.py
100.00% <100.00%> (ø)

src/dvclive/xgb.py
94.11% <100.00%> (+0.36%)
:arrow_up:

tests/test_catalyst.py
95.12% <100.00%> (+0.67%)
:arrow_up:

tests/test_fastai.py
100.00% <100.00%> (ø)

tests/test_huggingface.py
100.00% <100.00%> (ø)

tests/test_keras.py
100.00% <100.00%> (ø)

tests/test_lgbm.py
100.00% <100.00%> (ø)

... and 2 more

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

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


"Story behind this image" in hero banner

Excellent use of Sloth imagery, but can we just host them remotely rather than put them in rizzo?

We can, and we have the slothstronaut image in the static-asset bucket but I couldn't for the life of me point to it. I should probably have asked sooner, but do you know what the url would be if it's at the root of the bucket and is called slothstronaut.jpg?

http://assets.staticlp.com/slothstronaut.jpg

Looks solid, just a couple of thoughts

Looks good +1


Added Bytesafe to Overview.md

Added https://bytesafe.dev/ to the list of products supporting private feeds

Docs Build status updates of commit 71affc3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/hosting-packages/Overview.md
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

docs/hosting-packages/Overview.md
:white_check_mark:Succeeded
View

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


Include CI Linter check on copyright header of files

LINT should check for correct copyright header
https://github.com/eris-ltd/eris-db/issues/359

duplicate of #407 which states then enforce the presentation...


Isometry

Summary
Details and comments

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Raban Iten:x: rabanitenRaban Iten seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


parse/unparse functions prevent plugin from functioning

I've been trying to get up and running with Snakemake 8 on Azure, and independently encountered the error fixed by Pull Request #8.
After fixing that error, trying to launch Snakemake using the plugin yields the following error:
(sn-dev) Jons-MacBook-Pro:snakemake-tutorial jgs286$ snakemake --profile Profiles/azure
Using profile Profiles/azure for setting default command line arguments.
Traceback (most recent call last):
  File "/Users/jgs286/Dropbox/Documents/Consulting/Development/Snakemake/snakemake/snakemake/cli.py", line 2047, in main
    success = args_to_api(args, parser)
              ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/jgs286/Dropbox/Documents/Consulting/Development/Snakemake/snakemake/snakemake/cli.py", line 1779, in args_to_api
    name: StoragePluginRegistry().get_plugin(name).get_settings(args)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/jgs286/mambaforge/envs/sn-dev/lib/python3.12/site-packages/snakemake_interface_common/plugin_registry/plugin.py", line 249, in get_settings
    extract_values(item, thefield, name, tag=tag)
  File "/Users/jgs286/mambaforge/envs/sn-dev/lib/python3.12/site-packages/snakemake_interface_common/plugin_registry/plugin.py", line 233, in extract_values
    value = parse_func(value)
            ^^^^^^^^^^^^^^^^^
TypeError: 'ellipsis' object is not callable

Commenting out those lines from init.py allowed Snakemake to run, but I was not able to connect to my test Azure Blob:
(sn-dev) Jons-MacBook-Pro:snakemake-tutorial jgs286$ snakemake --profile Profiles/azure
Using profile Profiles/azure for setting default command line arguments.
WorkflowError:
Error applying default storage provider azure. Make sure to provide a valid --default-storage-prefix (see https://snakemake.github.io/snakemake-plugin-catalog/plugins/storage/{provider}.html). {validation_res}

Not sure if that's due to an error in my specification, but this is what I was using for my profile:
use-conda: True
cores: 1
default-storage-provider: azure
default-storage-prefix: snakemake-tutorial
storage-azure-endpoint-url: https://bioinfotest.blob.core.windows.net
storage-azure-sas-token: "se=MYTOKEN"

Out of curiosity, has anyone actually used this plugin yet? I'd really love to help with documentation / tutorial if I can get it running.

Hi @tanaes!
I was wondering if you managed to find a solution for this? I'm currently running into the exact same situation.
Command executed:
snakemake -n --default-storage-provider azure --default-storage-prefix $AZ_DEFAULT_STORAGE_PREFIX \
  --storage-azure-endpoint-url $AZ_STORAGE_ENDPOINT_URL \
  --storage-azure-access-key $AZ_STORAGE_ACCESS_KEY

Output:
Traceback (most recent call last):
  File "/home/ubuntu/mambaforge/envs/snakemake-tutorial/lib/python3.12/site-packages/snakemake/cli.py", line 2091, in main
    success = args_to_api(args, parser)
              ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/mambaforge/envs/snakemake-tutorial/lib/python3.12/site-packages/snakemake/cli.py", line 1812, in args_to_api
    name: StoragePluginRegistry().get_plugin(name).get_settings(args)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/mambaforge/envs/snakemake-tutorial/lib/python3.12/site-packages/snakemake_interface_common/plugin_registry/plugin.py", line 264, in get_settings
    extract_values(item, thefield, name, tag=tag)
  File "/home/ubuntu/mambaforge/envs/snakemake-tutorial/lib/python3.12/site-packages/snakemake_interface_common/plugin_registry/plugin.py", line 243, in extract_values
    value = parse_func(value)
            ^^^^^^^^^^^^^^^^^
TypeError: 'ellipsis' object is not callable

As a side-note, if this plugin is not working, then what are the ways in which pipelines can be executed on Azure while also accessing data stored on Azure? Thanks!

Same issue. @jakevc any suggestions on  the fix?

I ended up giving up and switching to Nextflow for cloud compute.

Just releasing a fix now.. sorry for the delay


Add GET /admin/users/:username/keys resource to obtain SSH public keys

It would be nice to have Gitea as an alternative to GitHub
The resource should be available with out authentication.
http://manpages.ubuntu.com/manpages/bionic/man1/ssh-import-id.1.html

But as far as i can tell gitea does not akt as keyserver, does gh?!?

Response should be similar to described here https://developer.github.com/v3/users/keys/

This is implemented already at https://gitea.com/techknowlogick.keys (by appending .keys to a users profile)

API is also available: https://try.gitea.io/api/swagger#/user/userListGPGKeys

Tested.
Works correctly.


Node-RED support

Describe the bug
WebContainer is a great environment to use Node-RED, Low code programming tool because Katacoda or GitPod environments are limited in terms of the time and lost flow that users developed. But there is some problem to show the development UI.
Link to the blitz that caused the error
https://stackblitz.com/edit/node-lwdex7
Steps to reproduce

Type the following commands on the terminal.

npm install node-red
node node_modules/node-red/red.js

Click the "Open in New Window" button to show Node-RED flow editor.
Positions of some UI components are incorrect. Especially, deploy button.

Expected behavior
Users can see in-corrupted UI in the Node-RED flow editor.
Screenshots

Platform
Browser name  = Chrome
Full version  = 96.0.4664.110
Major version = 96
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.110 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 54248970,
  "usedJSHeapSize": 52273526,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = a3339ebb87d0fe73c7995fe67420660e0dc95de2

Additional context
There is another problem with the http request node. When the node accesses external REST API on the Internet, the Node-RED process is crashed. I think that WebContainer doesn't support got module.

Hi!
I got Node-RED working by using another start command than you did. I could also click Deploy and test a simple timestamp and I got a successful message in the debug component.
My start command was:
./node_modules/.bin/node-red
Does that help for you @zuhito ?

@joelml could you link to you a project that @zuhito could fork by chance?
@zuhito the project you linked to doesn't seem to have any files besides the basic node files. If the problem persists could you link to a new starter?


WASM build doesn't work

due to changes regarding array allocation in go and js, our build using go1.13 is broken. We need to fix it in EngoEngine/gl as well.

Note that go1.14 includes further breaking change :-) (js.Value is no longer comparable)

Yeah, I suppose that's what happens when you try to support stuff when the api can still change :P

Closed by #701


Updated helm collector to leverage helm client for go

Example of output object:
[
  {
    "name": "happy-panda",
    "namespace": "test",
    "status": "deployed",
    "chart": "wordpress",
    "history": [
      {
        "lastDeployment": "2021-08-04T17:57:05.275284338+02:00",
        "description": "Install complete",
        "status": "deployed",
        "revision": 1,
        "appVersion": "5.8.0"
      }
    ]
  }
]

Other remarks:

This new function lists & get history of all HR from all namespaces
For the time being, there is no possibility to filter by namespace or chart name

Thank you @arnaud-tincelin for making these changes. Has this code been tested. If yes, could you please share the image you used to test so that I can test it on my end in order to make sure that these code fixes work on the distros for our mvp: CAPZ, openshift, kind, and aks-engine? Thanks!

Thank you @arnaud-tincelin for making these changes. Has this code been tested. If yes, could you please share the image you used to test so that I can test it on my end in order to make sure that these code fixes work on the distros for our mvp: CAPZ, openshift, kind, and aks-engine? Thanks!

@sophsoph321 yes the code has been tested against an AKS cluster. If you wish to test this code I suggest you run the test along the collector.  To do so, go to the collector directory (aks-periscope/pkg/collector) and run go test -v -run TestHelmCollector
Please note that the test is expecting to find at least 1 helm release in the targeted cluster.
Of course we will release a docker image later on.

💡 This looks good, to me but I need to test it against my local image. I will deep dive into the code sometime tonight.

💡 Thanks @sophsoph321 and @arnaud-tincelin  - fyi @sophsoph321 - you can test is using this image docker.io/tatsat/helm-client if it helps. Thanks.
I will merge this soon, I have tested with AKS cluster and all looks good. 🙏


use Vagrant to build identical dev environment.

This is so awesome! I would like to create another repo to use this. Can you wait me to create it and send pull request to that repo again?

Please send PR to https://github.com/Keanux/keanux-vagrant

ok :)


[Software Request] Music Production

There are already some apps for video editing and graphical design.
How about some music software next?
some DAWs for example

FL Studio 21
Ableton Live 11
Bitwig
Reaper

and Free VST Plugins

Vital by Matt Tytel
OTT, Dimension expander by Xfer Records

just to name a few

@eye-wave we mostly like to only support Open Source application. Yes there are some that are not Open Source on the list but the number of those apps is pretty small.


Merge compiling to Lambda(undefined, undefined)

Hey all,
First of all just want to say this looks like an excellent tool with lots of promise!
Currently when using Merge in some of my queries, I'm getting the migration query output as Lambda(undefined, undefined) in a UDF.
For example, the following UDF definition:
import faunadb from 'faunadb';

const q = faunadb.query;
const { Query, Lambda, CreateFunction, Var, Let, Merge, Select } = q;

const ReturnWithId = doc => {
  return Let(
    { document: doc },
    Merge(Select('data', Var('document')), {
      id: Select(['ref', 'id'], Var('document')),
    })
  );
};

export const ReturnWithIDUDF = CreateFunction({
  name: 'return_with_id',
  body: Query(Lambda(['doc'], ReturnWithId(Var('doc')))),
});

export default ReturnWithIDUDF;

Compiles to:
CreateFunction({
  name: "return_with_id",
  body: Query(Lambda(["doc"], Let([{
    document: Var("doc")
  }], Lambda(undefined, undefined))))
}

Am I missing something or is this a bug?
Keep up the great work!

Definitely a bug, probably in the representation conversation of strings to FQL in the driver or vice versa, will take a look.

Determined the issue, talking with engineering to take a look.

Should be fixed when pairing it with the 4.1.2 fauna driver.


[feat] Add an option to save the generated jest.config.js

I want to be able to run jest tests from within my IDE (IDEA). It is far easier to just run jest directly than through tsdx test.
What I did was hacked a console.log() into node_modules/tsdx/lib/index.js, output the generated jest config and saved that in a jest.config.js file. Had to edit it a bit to remove the absolute paths (🤷♂ why those are there), but it works perfectly.
So, it would be nice to have something like the eslintconfig save option, but for the jest config.

I was just thinking further. Why not just generate a jest.config.js when creating the project?
Is there a good reason why tsdx test exists?

Providing optional configuration with a file like tsdx.config.js would be a fairly useful idea.

@lookfirst I totally agree. I'm working in WebStorm and it has good integration with Jest but it will only work with jest.config.js present because otherwise, typescript will not compile during tests.
I ended up creating jest.config.js with
const {createJestConfig} = require('tsdx/dist/createJestConfig');
const {paths} = require('tsdx/dist/constants');

process.env.BABEL_ENV = 'test';
process.env.NODE_ENV = 'test';

module.exports = createJestConfig(undefined, paths.appRoot);

It works although it's far from ideal solution as createJestConfig checks if jest.config.js file is present and might create a recursive loop
It would be awesome to add similar functionality to yarn lint --write-file

Sorry to be an ass and not share... here is what I'm using...
module.exports = {
  transform: {
    '.(ts|tsx)': 'ts-jest'
  },
  transformIgnorePatterns: [ '[/\\\\]node_modules[/\\\\].+\\.(js|jsx)$' ],
  moduleFileExtensions: [ 'ts', 'tsx', 'js', 'jsx', 'json', 'node' ],
  collectCoverageFrom: [ 'src/**/*.{ts,tsx}' ],
  testMatch: [ '<rootDir>/**/*.(spec|test).{ts,tsx}' ],
  rootDir: '.',
  setupFilesAfterEnv: ['./jest.setup.js'],
};

Yes, tsdx test makes sense so we can
upgrade jest for all of our users

Yes, tsdx test makes sense so we can upgrade jest for all of our users

That answer doesn't make much sense to me because:

The way tests were run was recently changed, which was a breaking change to go from non-watch to watch. I had to lookup the documentation, which was a tad bit frustrating. In other words, the upgrade path here isn't always clear.
We are talking about the difference between running jest vs. tsdx test... at least for my project, they are exactly the same. What needs to be upgraded?
It complicates tsdx for really no actual benefit today. If this is intended for upgrades, I'd consider this a premature optimization.

Oh hey, look at that... watch mode has been reverted.
Get rid of tsdx test and all this goes away.

I think a TSDX Jest preset would be helpful (so would an ESLint one and a Babel one) as all these integrations then become one-liners. Something I mentioned in https://github.com/jaredpalmer/tsdx/issues/389#issuecomment-568866380

@lookfirst it's not a premature operation, that's the sole existence of this platform... Remove configuration and allow for a simpler dependency upgrade strategy a lá create-react-app. It's mentioned in the first paragraph of the README. I'd say you have a valid reason for wanting an easier way to opt-out though.
I'm interested in that as well, but it's important to acknowledge were going against the grain of his gola.

@kylemh I said optimization and so far, tsdx test has proven to complicate things and not remove any configuration whatsoever.

It removes a large portion of what I would need to define in my Jest config

... and in my case the command does nothing productive, because I use an IDE that needs the file.
The whole point of this issue is to allow the file to be generated automatically. If the file is generated automatically, you don't need the command. 🥇
Large is relative... my file is is just a few simple lines.

If the file is generated automatically, there’s no way to support backwards compatibility for future changes to the Jest config from within TSDX.
Again, I’m not disagreeing saying we don’t need a way to help you out, but the solution you’re offering of just exporting the config file by default is a bad idea for most users who aren’t editing the config and don’t use a JetBrains editor, because it ruins the ability for the team to adjust the config and ship the update as a non-breaking change.

There is always a way, it is code, not rocket science. Updating the file is exporting it again and merging the diff. That is what revision control is for.

Wow, just going through recent commits, I noticed another bug related to jest... #499... this wouldn't exist if tsdx test didn't exist. I can fully understand the desire to make things 'simple', but so far, I think the complications outweigh the benefits. tsdx test-generate outputs jest.config.js and bam... tsdx steps out of the way. No more issues.

(39 PRs and counting)

Co-founded Apache Jakarta. One of the first Apache Members. Been contributing to hundreds of open source projects across multiple groups and languages for 25+ years now. And, who cares?

You are free to suggest changes, point out bugs, be critical of a feature, etc but some of your comments go further than that and can be overbearing -- I don't believe that's warranted or productive.

Oh come on, no I am not. Just because you don't agree with the direction or suggestion to really simplify things, doesn't mean it is overbearing.

Having to manually update is a poor experience.

Agreed. Automate it. I gave that as an example to move the discussion forward, not as an actual solution, which should definitely be automated.

That's similar to a breaking change. TSDX users shouldn't have to do that.

Agreed. tsdx test has already given us one breaking change. Along with several issues of confusing behavior. How much more do you need?

https://github.com/jaredpalmer/tsdx#npm-test-or-yarn-test
Runs in interactive mode?

This is getting too heated folks.

tsdx test will stay
We should make it pleasant to use your own jest setup if you want to

Copying from something Jared said in #118 almost a year ago:

I think someone tried to solve this in #75, but this generated absolute paths to tsdx's jest plugins.

So some of the requirements are currently:

Fixing any path issues. A preset might handle that without much fuss.

Create a preset

We'll need to migrate to a monorepo to set-up presets (and possibly handle some monorepo-related issues when doing so)
Ensure the preset merges your config correctly (it might break current tsdx test usage)

Read .babelrc configs, c.f. #583
Use TSDX's built-in Babel plugin that's used for build, c.f. #383. The plugin is also intertwined with some runtime config so that one's a good bit of work and may not be fully possible.

Just listing all those so everyone knows the status; that's why there's a hold-up on this, sorry about that 😓
In the meantime, I did get tsdx test to integrate with VSCode's debugger with my config in https://github.com/jaredpalmer/tsdx/issues/577#issuecomment-600866819 . Can use that as another workaround too.

@agilgur5 gave u publish rights. Go ahead a cut a release

I just figured out a workaround for IntelliJ IDEA that allows you to use tsdx test using a custom Jest runtime configuration. You can modify your project's Jest template directly, so those settings will be inherited by all new unit test runs.
The configuration needs to look like this:

If you have a jest.config.js file with some custom overrides, specify that under "Configuration file"
Select the node_modules/tsdx directory as "Jest package"
Set your project's root directory as "Working directory"
Add test to "Jest options"

With those settings it's possible to start tests from within the IDE.


Fix SipHash init order.

Setting the SipHash hash size and setting its key is done with two
independent functions...  and yet, the internals depend on both.
Unfortunately, the function to change the size wasn't adapted for the
possibility that the key was set first, with a different hash size.
This changes the hash setting function to fix the internal values
(which is easy, fortunately) according to the hash size.
evpmac.txt value for digestsize:8 is also corrected.

Merged.
master:
425036130dfb3cfbef5937772f7526ce60133264 Fix SipHash init order.
1.1.1:
61e78e7ace6c5d65910379556d7da7d23492291c Fix SipHash init order.


goxel: don't save imgui.ini to working directory

Motivation for this change
Before, goxel would save a file imgui.ini to the current working directory.
imgui.ini
[Window][Debug##Default]
Pos=60,60
Size=400,400
Collapsed=0

[Window][Goxel]
Pos=0,0
Size=959,353
Collapsed=0

This is to save window configuration, presumably to restore them on next execution.
Is this behaviour worth littering CWD?
An alternative would be to change upstream to honor XDG conventions.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg  build goxel

Is this behaviour worth littering CWD?
An alternative would be to change upstream to honor XDG conventions.

May be worth filing an upstream issue.

Success on aarch64-linux (full log)
Attempted: goxel
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/gm1jh2rgbh47y9xba4dignnl2k9phc0b-goxel-0.8.0/bin/goxel
shrinking RPATHs of ELF executables and libraries in /nix/store/gm1jh2rgbh47y9xba4dignnl2k9phc0b-goxel-0.8.0
shrinking /nix/store/gm1jh2rgbh47y9xba4dignnl2k9phc0b-goxel-0.8.0/bin/.goxel-wrapped
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/gm1jh2rgbh47y9xba4dignnl2k9phc0b-goxel-0.8.0/bin
patching script interpreter paths in /nix/store/gm1jh2rgbh47y9xba4dignnl2k9phc0b-goxel-0.8.0
checking for references to /build in /nix/store/gm1jh2rgbh47y9xba4dignnl2k9phc0b-goxel-0.8.0...
/nix/store/gm1jh2rgbh47y9xba4dignnl2k9phc0b-goxel-0.8.0


Support prefixes in search: eq, ne, gt, lt, ge, le

Is your feature request related to a problem? Please describe.
https://www.hl7.org/fhir/search.html#prefix
Describe the solution you'd like
Design and implement the change to the search API
Describe alternatives you've considered
NA
Additional context
NA
Would you like to work on the issue?
No

#436 & #419 add these prefixes for number and date. Is this what you had in mind or it is something else?

Most of these are supported for Date and Number ( see #436 and #419 ) I'll add them for quantity types.

Update, working on this. Will open a pr soon.

@epicadk, thanks for the update. Is there an ETA you have for completing the code changes for this?

I'm still searching for a library/any other solution that would help convert units to UCUM base units. I can get the rest of it done in around a week i.e. search on quantity types without canonical matches.

@epicadk , were you able to move ahead with this issue? Any ETA that you can provide for completion?

Sorry, slipped my mind, I'll open a pr that adds basic functionality by tomorrow.

Thanks @epicadk. By adding basic functionality, are there any changes that are being excluded from the PR for now or would it cover the entire scope mentioned as part of this issue? I assigning it to a reviewer in the meantime.

Yup, all functionality has been added except for date approximate as discussed here. https://github.com/google/android-fhir/pull/419#discussion_r644688966. Will discuss with @jingtang10 and get that done as well. After that , it should not take more than a day to complete.

FYI https://github.com/google/android-fhir/issues/568

Re-opened because still need to address #568.

Re-opened because still need to address #568. We could also close this and just have #568. @Tarun-Bhardwaj

@epicadk , thanks Aditya. Have assigned #568 to you. Let's keep #474 open until #568 is addressed as they both are closely linked. CC @jingtang10


Include traditional language model inference endpoints in server

What behavior of the library made you think about the improvement?
Currently, we only have Guided generation as an endpoint in the server. This is not ideal as I need a separate inference server with basic completions.
How would you like it to behave?
I would like to be able to pass no JSON structure or guidance to the generate function. I should also have control over several things like temperature, stop tokens etc. I honestly believe we could copy code directly from the VLLM inference server endpoint. Ideally we could use the Open AI compliant server if we wanted.

Have you tried no passing schema to the API? It should default to just generating text. Also we copied the code from vLLM so you should be able to pass any argument you can pass to vLLM.

Great, I haven't actually tried but given this is the case I'll go ahead and get this deployed asap, because this is awesome! Great work to this team

I'll close this for now, reopen if it doesn't work for some reason.


Memory is not properly provided for cluster execution

Our Slurm cluster enforces memory usage with cgroups. I added --mem=32G as a default to the snakemake profile. This is of course not good enough as it is too high for certain steps and too low for the assembly. If I increase it for assembly then the number of jobs running in parallel is too low.
The resolution would be to set proper memory resources for all rules.

Try setting --mem-per-cpu=8G as your default instead.  Our thread requests were designed with this assumption in mind.

Thanks for the hint. Alas, I don't have nodes that have 48*8=384 GB of free RAM. I'll try using 32 threads for hifiasm for now...


[tool_crash] RPCError

Command
flutter run

Steps to Reproduce

...
...
...

Logs
RPCError
#0      new _OutstandingRequest (package:vm_service/src/vm_service.dart:261:34)
#1      VmService._call.<anonymous closure> (package:vm_service/src/vm_service.dart:1778:25)
#2      VmService._call (package:vm_service/src/vm_service.dart:1790:8)
#3      VmService.callServiceExtension (package:vm_service/src/vm_service.dart:1729:14)
#4      FlutterVmService._checkedCallServiceExtension (package:flutter_tools/src/vmservice.dart:874:28)
#5      FlutterVmService.invokeFlutterExtensionRpcRaw (package:flutter_tools/src/vmservice.dart:894:49)
#6      FlutterVmService._flutterToggle (package:flutter_tools/src/vmservice.dart:684:41)
#7      FlutterVmService.flutterToggleWidgetInspector (package:flutter_tools/src/vmservice.dart:711:9)
#8      ResidentHandlers.debugToggleWidgetInspector (package:flutter_tools/src/resident_runner.dart:868:33)
<asynchronous suspension>
#9      TerminalHandler.processTerminalInput (package:flutter_tools/src/resident_runner.dart:1836:7)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.24.2, on Microsoft Windows [Version 10.0.22631.4169], locale en-US)
    • Flutter version 3.24.2 on channel stable at C:\Users\Dee\AppData\Local\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4cf269e36d (7 days ago), 2024-09-03 14:30:00 -0700
    • Engine revision a6bd3f1de1
    • Dart version 3.5.2
    • DevTools version 2.37.2

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\Dee\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio1\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11572160)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop Windows apps (Visual Studio Build Tools 2019 16.11.36)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools
    • Visual Studio Build Tools 2019 version 16.11.34902.97
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (version 2023.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2023.3)
    • Android Studio at C:\Program Files\Android\Android Studio1
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = C:\Program Files\Android\Android Studio1
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11572160)

[✓] VS Code, 64-bit edition (version 1.93.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.94.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22631.4169]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 128.0.6613.121
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 128.0.2739.67

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 4cf269e36de2573851eaef3c763994f8f9be494d

Hi @OncleDee, what platform(s) are you seeing this issue on? How often does this crash occur for you? Can you provide the output of flutter doctor -v which captures the crash?

Looking at the stack trace, it looks like this happened after toggling the widget inspector via the CLI. I'm curious to whether the widget inspector was toggled after the app exited (or the device was disconnected).


Maintenance/620/fix x11 release mode compilation

[x] Tested on all platforms changed
[x] Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users
n/a Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
n/a Created an example program if it would help users understand this functionality

Fixes #620 (tested it fails compilation before the fix, and works after)

By "Rust greater than 1.30" you mean "Rust 2018 (>= 1.30)"?

oh yes :stuck_out_tongue_closed_eyes:

Thanks!