Expose load_transformed() and TransformedModule.

To allow external programs to use the bundler transform phase to
generate a module graph.

Also includes a small fix for compiling against the spack module here: https://github.com/swc-project/swc/pull/1770/files#diff-887764dffae8a5dff11fe5af51313928fda3627c6bfa1a3d7fb28c51017917f5.

I think this is more suitable for swc_ecma_loader. (at /ecmascript/loader/)
The bundler can depend on it, and other crates can depend on it in the future.
Are you ok with it?

@kdy1, You mean move TransformedModule and dependent structs to ecmascript/loader?
Sure happy to do that if my program can still compile and run :+1:

Yes. I meant it and thank you in advance!

The problem with moving to ecmascript/loader is that there are issues resolving the Helpers.
These are the compiler errors I have after attempting to move the structs:
   Compiling swc_ecma_loader v0.6.0 (/home/muji/git/consensys/swc/ecmascript/loader)
error: cannot find derive macro `Is` in this scope
  --> ecmascript/loader/src/module.rs:48:24
   |
48 | #[derive(Debug, Clone, Is)]
   |                        ^^

error[E0433]: failed to resolve: use of undeclared crate or module `swc_ecma_transforms`
  --> ecmascript/loader/src/module.rs:23:26
   |
23 |     pub swc_helpers: Lrc<swc_ecma_transforms::helpers::Helpers>,
   |                          ^^^^^^^^^^^^^^^^^^^ use of undeclared crate or module `swc_ecma_transforms`

error[E0412]: cannot find type `Helpers` in this scope
  --> ecmascript/loader/src/module.rs:21:22
   |
10 | pub struct TransformedModule {
   |                             - help: you might be missing a type parameter: `<Helpers>`
...
21 |     pub helpers: Lrc<Helpers>,
   |                      ^^^^^^^ not found in this scope

error: aborting due to 3 previous errors

I cannot depend upon swc_ecma_transforms in swc_ecma_loader as it creates a cyclic package dependency and the other Helpers in swc_bundler seem like they should be in that package to me.
So in order to do this refactoring we would need to decouple both Helpers from TransformedModule - maybe with a wrapper type in swc_bundler?
But obviously this is quite a lot of work. What do you think?

The problem of swc_ecma_transforms can be workarounded by depending on swc_ecam_transforms_base, while the helper of bundler going to other crate seems wrong to me.
But I think bundler should not expose such apis. I will think about this bit more.

The behavior of the loader implementation of bundler is too opinionated for public apis.
It modifies span hygiene data in a way that can be handled by the bundler afterward.
It's an implementation detail and can be changed anytime if I found a better way.


env var needing to be deleted in release chart

From release checklist, there is the following manual step:
Delete the `KUBERNETES_POD_TERMINATION_GRACE_PERIOD_SECONDS` env var from `workflow-$WORKFLOW_RELEASE_SHORT/tpl/deis-controller-rc.yaml`

We should automate this step in deisrel -- perhaps after inquiring as to the origins of exactly why it needs to be deleted and whether or not a different approach in chart itself might be a better option...

deisrel no longer manages charts, but that step is still there in the release guide.
The controller chart is now using deployments instead of rcs, so I wonder if it's stale.
@helgi any idea?

This still needs to be done in the deis-controller-deployment.rc for Helm classic Chart releases. That step was automated in deis/charts#363 and (un-)documented in deis/workflow#530.


fix issue #1075 - update dockerfile

Updated web-ui Dockerfile Node dependency to v12.
This fixes issue #1075.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1080   +/-   ##
=========================================
  Coverage     84.75%   84.75%
  Complexity     1056     1056
=========================================
  Files           176      176
  Lines          2821     2821
  Branches        460      460
=========================================
  Hits           2391     2391
  Misses          187      187
  Partials        243      243

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 be4ed04...e6c201a. Read the comment docs.

Let's update package.json   as well then.

Should I bump the version to 0.4.1 or 0.5.0?  Also, are you asking me to bump the node version to >=12?

FYI, please see issue #1075 for updates.  I don't think this PR is necessary anymore as the issue may have been user error on my part.


Adding classes and tests for parsing a topics json file

This change is

Reviewed 10 of 10 files at r1.
Review status: all files reviewed at latest revision, 9 unresolved discussions.

BaboonFramework/src/org/unc/lac/baboon/exceptions/BadTopicsJsonFormat.java, line 5 at r1 (raw file):

private static final long serialVersionUID = 1L;

use autogenerated serialVersionUID, it's a way to maintain version control of the serializable objects.
And if the class changes, compute it again.

From https://dzone.com/articles/dont-ignore-serialversionuid:
Declare a specific serialVersionUID, and update it whenever you make a change that breaks backward compatability.

BaboonFramework/src/org/unc/lac/baboon/exceptions/BadTopicsJsonFormat.java, line 7 at r1 (raw file):

public BadTopicsJsonFormat(String errorMessage) {
super(errorMessage);

add default constructor

BaboonFramework/src/org/unc/lac/baboon/exceptions/NoTopicsJsonFileException.java, line 3 at r1 (raw file):

package org.unc.lac.baboon.exceptions;

public class NoTopicsJsonFileException extends Exception {

same as above

BaboonFramework/src/org/unc/lac/baboon/topic/Topic.java, line 15 at r1 (raw file):

 */
public class Topic {
    public String name;

it's not a good practice to set all these fields as public.
It's best setting them as private and creating getters and setters with some restriction

BaboonFramework/src/org/unc/lac/baboon/utils/TopicsJsonParser.java, line 26 at r1 (raw file):

 */
public class TopicsJsonParser {
    static ObjectMapper objectMapper = new ObjectMapper();

don't omit private modifier

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 20 at r1 (raw file):

}

delete

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 38 at r1 (raw file):

"test/org/unc/lac/baboon/test/resources/topics.json"

define this as private final static String. You'll probably use it in newer tests

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 39 at r1 (raw file):

assertEquals(topics.size(),expectedNames.length);

arguments order is inverted, first is expected and second is actual. It's easier to debug id you respect it.

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 46 at r1 (raw file):

···

remove trailing spaces

Comments from Reviewable

Review status: 3 of 9 files reviewed at latest revision, 9 unresolved discussions.

BaboonFramework/src/org/unc/lac/baboon/exceptions/BadTopicsJsonFormat.java, line 5 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> > private static final long serialVersionUID = 1L;
>
> use autogenerated serialVersionUID, it's a way to maintain version control of the serializable objects.
> And if the class changes, compute it again.
>
> > From https://dzone.com/articles/dont-ignore-serialversionuid:
> > Declare a specific serialVersionUID, and update it whenever you make a change that breaks backward compatability.

Done.

BaboonFramework/src/org/unc/lac/baboon/exceptions/BadTopicsJsonFormat.java, line 7 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> > public BadTopicsJsonFormat(String errorMessage) {
> >         super(errorMessage);
>
> add default constructor

Done.

BaboonFramework/src/org/unc/lac/baboon/exceptions/NoTopicsJsonFileException.java, line 3 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> same as above

Done.

BaboonFramework/src/org/unc/lac/baboon/topic/Topic.java, line 15 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> it's not a good practice to set all these fields as public.
>
> It's best setting them as private and creating getters and setters with some restriction

Done.

BaboonFramework/src/org/unc/lac/baboon/utils/TopicsJsonParser.java, line 26 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> don't omit private modifier

Done.

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 20 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> > }
>
> delete

Done.

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 38 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> > "test/org/unc/lac/baboon/test/resources/topics.json"
>
> define this as private final static String. You'll probably use it in newer tests

Done.

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 39 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> > assertEquals(topics.size(),expectedNames.length);
>
> arguments order is inverted, first is expected and second is actual. It's easier to debug id you respect it.

Done.

BaboonFramework/test/org/unc/lac/baboon/test/cases/TopicsParserTest.java, line 46 at r1 (raw file):
Previously, airabinovich (Ariel Iván Rabinovich) wrote…
> > ···
>
> remove trailing spaces
>

Done.

Comments from Reviewable

Reviewed 3 of 7 files at r2.
Review status: 5 of 9 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 4 of 7 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Call assembly functions on Windows.

Coverage increased (+0.5%) to 82.667% when pulling 1736d2b98ebea26957086d5041285a6d9fe879bd on tdaede:windows_nasm_enable into 9f84d4d5eb90062fe4b835e9c60e4c6af822e53d on xiph:master.

In a very quick run on 20 frames rav1e was 44% faster with its assembly functions! The output it not bit exact however, is that expected?
rav1e0314 "E:\Morocco8K-8b-1920.y4m" -o NULL.ivf -l 20 --psnr
1920x1080 @ 19001/317 fps
encoded 20/20 frames, 0.099 fps, 1013.03 Kb/s, est. size: 0.04 MB, est. time: 0 s
Key Frames:      1    avg size:   18877 B
Inter:          19    avg size:    1283 B
Intra Only:      0    avg size:       0 B
Switch:          0    avg size:       0 B
Mean PSNR: Y: 47.7122  Cb: 51.6057  Cr: 50.7179  Avg: 50.0119

rav1e0314-asm "E:\Morocco8K-8b-1920.y4m" -o NULL.ivf -l 20 --psnr
1920x1080 @ 19001/317 fps
encoded 20/20 frames, 0.143 fps, 1008.80 Kb/s, est. size: 0.04 MB, est. time: 0 s
Key Frames:      1    avg size:   18877 B
Inter:          19    avg size:    1274 B
Intra Only:      0    avg size:       0 B
Switch:          0    avg size:       0 B
Mean PSNR: Y: 47.7231  Cb: 51.6010  Cr: 50.7246  Avg: 50.0163

uhhhh... no that's not expected, as we are supposed to have unit tests that catch that. Could you maybe try checking out previous version of individual files and narrow down the difference? (I'll also check Linux, as it's probably happening there too)

I can reproduce this on Linux. Dunno if that should block the patch (Windows now because the same as Linux/Mac) but we should still fix it.

The difference between RUst-native and assembly only seems to affect RDO, no decode mismatches have been witnessed. We can chase it down after this. The only thing blocking my approval is the comment I left on src/mc.rs.


name change - from mother board to demo board.

change silk text on the front to 'TinyTapeout demo board'
update schematic from 'motherboard' to 'demo board'

Changed for v2.2.3, fab run #2


Compatibility with Zarr v3b2

Closes #9277

Remaining upstream failures are https://github.com/zarr-developers/zarr-python/issues/2501 and that we seem to now have 4 new list_prefix calls (I've bumped the count, we can optimize later). :/


cmd/go: add go generate -exec flag like go run -exec and go test -exec

I sometimes use go run -exec to quickly check how long a program takes to run excluding the build time, for example:
$ /bin/time go run testdata/gen.go
7.53user 1.17system 0:01.51elapsed 573%CPU (0avgtext+0avgdata 601640maxresident)k
0inputs+384outputs (10major+131651minor)pagefaults 0swaps
$ go run -exec=/bin/time testdata/gen.go
6.96user 1.08system 0:01.21elapsed 661%CPU (0avgtext+0avgdata 492024maxresident)k
0inputs+224outputs (1major+134476minor)pagefaults 0swaps

Similarly, I have found go test -exec very useful on occasion, for example to test for a different OS like GOOS=windows go test -exec=wine, or to test under a different environment like with https://github.com/mvdan/dockexec.
I wish I could do something similar with go generate. For example, in one of my projects, go generate ./... in CI takes twelve seconds, and I'd like to dig into which of the steps is taking longer than I would expect. For this reason I'd love to be able to do go generate -exec=time ./... or something similar with a short script.

Related Issues and Documentation

cmd/go: generate: be more informative/verbose #67471
cmd/go: feature request: timing for each component built #16375 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I don't see any immediate problems with this idea


[rcore] Weird text shaking/changing

WARNING: Please, read this note carefully before submitting a new issue:
It is important to realise that this is NOT A SUPPORT FORUM, this is for reproducible BUGS with raylib ONLY.
There are lots of generous and helpful people ready to help you out on raylib Discord forum or raylib reddit.
Remember that asking for support questions here actively takes developer time away from improving raylib.

Please, before submitting a new issue verify and check:

[x] I tested it on latest raylib version from master branch
[x] I checked there is no similar issue already reported
[x] I checked the documentation on the wiki
[x] My code has no errors or misuse of raylib

Issue description
A weird text shaking happened when I'm writing a code to replicate "the Matrix" effect (a javascript example can be found here). Apologize for a vague title and not-detailt description because I tried lots of time but cannot find where exactly the problem is.
Environment
Provide your Platform, Operating System, OpenGL version, GPU details where you experienced the issue.
uname -a:
Darwin QZKago-Requiem.local 24.1.0 Darwin Kernel Version 24.1.0: Thu Oct 10 21:05:14 PDT 2024; root:xnu-11215.41.3~2/RELEASE_ARM64_T8103 arm64

OpenGL information (grabbed from raylib's log output):
INFO: GL: OpenGL device information:
INFO:     > Vendor:   Apple
INFO:     > Renderer: Apple M1
INFO:     > Version:  4.1 Metal - 89.3
INFO:     > GLSL:     4.10
INFO: GL: VAO extension detected, VAO functions loaded successfully
INFO: GL: NPOT textures extension detected, full NPOT textures supported
INFO: GL: DXT compressed textures supported
INFO: PLATFORM: DESKTOP (GLFW): Initialized successfully

the operating system is macOS 15.1.1 (SIP disabled, though I think that shouldn't matter?)
Raylib is grabbed from latest master branch using FetchContent in cmake.
Issue Screenshot

notice that I'm screenshoting from paused state (i.e. the code running here is just identical to just call BeginDrawing() & EndDrawing(), and nothing else. ). The screen is constantly switching these two states somehow.
Code Example
Due to the same reason described in the description, I'm sorry that I can't provide code that is small enough ( i guess? ) cuz I cannot even extract the code with problem.
#include <stdlib.h>
#include <time.h>
#include <string.h>
#include <raylib.h>

const int GRID_WIDTH = 25;
const int GRID_HEIGHT = 10;
const Color BACKGROUND_COLOR = CLITERAL(Color) {0x00, 0x00, 0x00, 0x11};

char str_pool[128][2];

void init_str_pool() {
    for (char ch = 33; ch < 127; ch++) {
        str_pool[ch-33][0] = ch;
        str_pool[ch-33][1] = 0;
    }
}

char *get_str(char ch) {
    return str_pool[ch-33];
}

char gen_char() {
    return rand() % (127-33) + 33;
}

int main(void) {
    init_str_pool();
    SetConfigFlags(FLAG_WINDOW_RESIZABLE | FLAG_MSAA_4X_HINT);
    InitWindow(800, 600, "RayTrix");
    SetTargetFPS(10);
    srand(time(NULL));

    bool reset = true;
    bool pause = false;
    int *ypos = nullptr;
    int ypos_count = 0;
    int screen_width = GetScreenWidth();
    int screen_height = GetScreenHeight();
    while(!WindowShouldClose()) {
        if (IsKeyPressed(KEY_SPACE)) {
            pause = !pause;
        }

        if (!ypos) {
            ypos_count = screen_width / GRID_WIDTH;
            ypos = malloc(ypos_count * sizeof(int));
            memset(ypos, 0x00, ypos_count * sizeof(int));
        }

        if (screen_height != GetScreenHeight() || screen_width != GetScreenWidth()) {
            screen_height = GetScreenHeight();
            screen_width = GetScreenWidth();
            reset = true;
        }

        BeginDrawing();

        if (reset) {
            ClearBackground(BLACK);
            reset = false;
            if (ypos) {
                free(ypos);
                ypos = nullptr;
            }
            continue;
        }

        if (!pause) {
            DrawRectangle(0, 0, screen_width, screen_height, BACKGROUND_COLOR);
            for (int i = 0; i < ypos_count; i++) {
                int x_coord = i * GRID_WIDTH;
                int y_coord = ypos[i];
                DrawText(get_str(gen_char()), x_coord, y_coord, 20, GREEN);
                if (y_coord > 100 + rand() % (screen_height*10 - 100)) {
                    ypos[i] = 0;
                } else {
                    ypos[i] = ypos[i] + GRID_HEIGHT;
                }
            }
        }
        EndDrawing();
    }
    if (ypos) {
        free(ypos);
    }
    CloseWindow();
}

Attempts to Figure Out
Here are some attempts I did trying to figure out the problem.
Firstly as you see the string is picked from a static array. At first it is actually generated in the loop but I'm afraid that some kind of a use-after-free may happen there, so I changed the code so that it will generate all the strings that will be used at start and store into a global array so that there shouldn't be any use-after-free problem anymore.
Secondly I'm thinking about that it might be a problem of drawing a rectangle with alpha value not being 0xFF. So I changed it to opaque temporarily but the problem still exists.
I tried to draw line of text before EndDrawing() and this line of text is just normal, but the rest is still changing between two states.
I changed get_str(gen_char()) to a constant "A", and the "A"s were just shaking up and down.
I fixed y_coord to always 0 and problem is still here.
I fixed y_coord to always 0 and changed get_str(...) to constant "A", and this time problem disappeared somehow.
Based on the last attempt, I guess that it might be a problem of generating y-coordinates & text, but weirdly when I printed generated coordinate and the text, they are normal. Nothing wrong with the coordinate/text generation.
Finally I added the pause functionality, to see what will happen if I pause the entire animating process (when pause is true, the entire body of the while loop will be identical to BeginDrawing() + EndDrawing() and nothing else, so nothing will update and nothing new will be drawn). This should make the screen shows only one static image, but it constantly switches between two states (as two screenshots shown above).

WARNING: Please, read this note carefully before submitting a new issue:
It is important to realise that this is NOT A SUPPORT FORUM, this is for reproducible BUGS with raylib ONLY.
There are lots of generous and helpful people ready to help you out on raylib Discord forum or raylib reddit.

@onion108 I'm afraid this issue is more related to user code than a raylib internal issue...

@onion108 I'm afraid this issue is more related to user code than a raylib internal issue...

Already figured out. Really apologize for disturbing

@onion108 No worries! Actually I took a quick look to the code trying to spot the issue but I couldn't find anything... the effect looks nice in the screenshots! 👍🙂


Add validation and test configuration

As proposed in https://github.com/puppetlabs/pdk/issues/333 this repository should serve as a default setup for control repositories generated by PDK.
Hence, we should add a Gemfile, .gitignore, .rubocop.yml (and optionally proposed CI configuration and other files?) that allow running validation and testing in a simple fashion via PDK (see below). Sample files are included in the Puppet Labs pdk-module-template repository.
Example Workflow (PDK)
# create control repo
$ pdk new control

# run static analysis tools
$ pdk validate

# run tests
$ pdk test unit

Would it make sense to run 'pdk convert' to this repository?  The unfortunate side effect is that the control-repo will get a whole bunch of other stuff that might not be desirable:
# On branch production
# Changes not staged for commit:
#   (use "git add <file>..." to update what will be committed)
#   (use "git checkout -- <file>..." to discard changes in working directory)
#
#       modified:   .gitignore
#
# Untracked files:
#   (use "git add <file>..." to include in what will be committed)
#
#       .devcontainer/
#       .fixtures.yml
#       .gitattributes
#       .gitlab-ci.yml
#       .pdkignore
#       .puppet-lint.rc
#       .rspec
#       .rubocop.yml
#       .sync.yml
#       .travis.yml
#       .vscode/
#       .yardopts
#       CHANGELOG.md
#       Gemfile
#       Rakefile
#       appveyor.yml
#       metadata.json
#       pdk.yaml
#       spec/
no changes added to commit (use "git add" and/or "git commit -a")

How about adding some simple spec test examples?  I'm stumped as to how to add spec testing to my control-repo as the manifest aren't in 'standard' locations.

@bschonec generally speaking when spec testing in a control-repo there are two paths to choose from:

Wire up the various pieces similarly to what's demonstrated in this blog post https://blog.example42.com/2017/07/24/spec-testing-control-repo/
Leverage onceover to do a lot of the heavy lifting for you https://www.puppet.com/blog/use-onceover-repository-testing

Personally, I prefer going the onceover route over the 'roll your own' approach. With that being said I've implemented it both ways before.
Going back to your previous comment, I do find value leveraging the pdk in this scenario as well.
Any files that you don't want the pdk to add in can always be removed by adding something along the lines of this (picking on appveyor):
appveyor.yml:
  delete: true

to a .sync.yml file before running a pdk convert or if this has already been done running pdk update.
If using the pdk I also recommend setting the PDK_FEATURE_FLAGS=controlrepo environment variable, and using the latest version of the pdk.
To leverage onceover a .sync.yml file might look similar to this:
Gemfile:
  optional:
    ':development':
      - gem: 'onceover'
Rakefile:
  requires:
    - require: 'onceover/rake_tasks'
      conditional: "Gem.loaded_specs.key? 'onceover'"
.fixtures.yml:
  unmanaged: true
spec/spec_helper.rb:
  delete: true
.gitignore:
  paths:
    - '.onceover'

We do also have a community slack: https://slack.puppet.com/
There's a testing channel which would be the best place for queries such as this, or if you'd prefer feel free to DM me :)


Produce a wheel for python 3.8

There is no wheel for python 3.8 which makes installing and running libraries which are dependent on pynvx difficult on MacOS due to the lack of an Nvidia GPU on Apple devices. Instead pip attempts to build pynvx which fails due to the lack of nvidia-toolkit.
Several people have faced this issue: https://stackoverflow.com/questions/61576609/problem-installing-fastai-on-macos-10-15

Added


New smartcast PR

I can't decide if this is a reasonable feature or not.

I've decided that I don't want to include this feature in ubelt. Its not robust enough.


Fullscreen doesn't work with Chrome

Hi, AlexxIT,
First of all, thanks for your great job with this component.
Fullscreen doesn't work for me with Chrome browser. Here is the error I have:
webrtc-camera.js?v2.0.2:257 Uncaught TypeError: this.requestFullScreen is not a function
    at HTMLElement.fullscreen.onclick (webrtc-camera.js?v2.0.2:257)
fullscreen.onclick @ webrtc-camera.js?v2.0.2:257

I'm ready to help debugging it further.

same issue here, also does not work in android app and firefox.
http://<IP_ADDRESS>:8123/webrtc/webrtc-camera.js?v2.0.2:257:56 Uncaught TypeError: this.requestFullScreen is not a function

Thanks @idoaflalo. I didn't find similar issues. If needed, I can close this issue.

Thanks @idoaflalo. Fixed in latest release


doc(tutorials): improve go instructions

The goal is to improve the instructions by adding how to init a mod with Go, creating the main function, and adding a note to create a new directory for the receiver. This is necessary because, in GoLang, we cannot have two files with the same function name in the same directory. We cannot create two files with the same func main(){} name in this case.
It can improve the tutorial's clarity and prevent the reader from having problems with the module and the package. Because you need to be inside a module to run go get.

@vit0rr Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@vit0rr Thank you for signing the Contributor License Agreement!


Add boot animation

Added boot animation with configurable wait times between each line.
Closes #20 ?

I was thinking if you could add this as well, I hope I'm not asking too much from you

Added loading animation


Fix #807 parse the userinfo with the golang tools

Note, this is now only fixed with Go version >= 1.4.2, see this bug in Go: https://github.com/golang/go/commit/07d86b1f2db73c6a0716aec4d4cd62dfacb04a21

Thanks your PR!

Note, this is now only fixed with Go version >= 1.4.2

I'm not sure what does this mean? Does this mean Gogs now requires Go 1.4.2?

No, it does not mean Gogs requires Go 1.4.2, but the bug #807 is still present if Gogs is compiled with an older Go

Anyway, I'm not sure why I wrote 1.4.2, I meant just Go 1.4

Thanks again!


Mandd/knapsack models

It is needed to develop Knapsack models that can be used coupled with RAVEN when
the desired optimization problem requires the use of specific models to generate
knapsack required parameters.
Ref. Issue #7

Tests are green on my local machine:

Documentation compiles as well without errors

This PR is good to be merged.


Daily Dev Merge

This pull request has been deployed to Vercel.

Latest commit:
da28606

✅ Preview:
https://hacktobered-demo-9ljwu64kl-ankitwww.vercel.app

🔍 Inspect:
https://vercel.com/ankitwww/hacktobered-demo/9HdU8nroMwLWMgCSxvsCY8VNEPko

View Workflow Logs


Screen tearing with DRI3 enabled on Intel

Issue description
There's screen tearing on Macbook Air 6,2 after PR (https://github.com/NixOS/nixpkgs/pull/18717) which enables DRI3 by default.
Problem seems to go away after explicitly enabling DRI2 in services.xserver.deviceSection:
      videoDrivers = [ "intel" ];
      deviceSection = ''
        Option "DRI" "2"
      '';
      useGlamor = true;

Is there a widespread improvement with tearing problems after enabling DRI3 by default?
Would it be better to default DRI2 or add a note somewhere instead (or ask upstream guys)?
If anyone feels like it -- please test if you see how things change if you use DRI2 and DRI3 and comment here.
Steps to reproduce

Update to the current nixos-unstable on MBA mid 2013.
Open  https://en.wikipedia.org/wiki/October_4 in firefox or qutebrowser and try scrolling.
Observe gruesome tearing.

Technical details

System: 17.03.git.adfcc2d (Gorilla)
Nix version: 1.11.4
Nixpkgs version: 17.03.git.adfcc2d

do you also have the problem using modsettings?
xserver = {
  videoDrivers = ["modsetting"];
}

@Mic92 I've tried different combinations (on/off; dri2/3, etc) of:
      videoDrivers = [ "modesetting" ];
      deviceSection = ''
        Option "DRI" "2"
        Option "TearFree" "true"
      '';
      useGlamor = true;

and tearing is present with all of them.

Thanks. This behavior is consistent, with what I heard of from T440s user.

I think there not much we could do hear then reconsidering DRI3.
This is clearly an upstream issue: https://wiki.archlinux.org/index.php/intel_graphics#DRI3_issues

I have never been able get rid of the tearing with DRI3. An option, which is what I'm currently using, is to run compton with the vsync option.

Agreed that this is an upstream issue, and should probably be raised there. Odd that you're experiencing it with both intel and modesetting. Is there good reason to believe that DRI2 is more stable in general? DRI3 certainly works fine for some users, is required for some features, is the default in other distros, and is easy enough to turn off, so considering the general shabby state of the intel driver the current state might be the best we can do.

@Mic92's fix worked on my machine! Yay! Thanks @Mic92!

(triage) Any updates?
It seems like there are already some related upstream issues (Upstream bug HowTo):

https://bugs.freedesktop.org/show_bug.cgi?id=97957 (+ duplicate)

Would be great if someone could help out over there :smile:
I personally wouldn't switch back to DRI2 but it would probably be worth adding a short comment to the NixOS documentation (like in the Arch Wiki).

If any xmonad users are experiencing these issues, know that compton is not mutually exclusive with xmonad, and that, at least in my case, adding services.compton.enable = true; to my configuration.nix completely solved this issue.

If any xmonad users are experiencing these issues, know that compton is not mutually exclusive with xmonad, and that, at least in my case, adding services.compton.enable = true; to my configuration.nix completely solved this issue.

Weird. I'm running on xmonad, and have compton enabled, and I'm still getting screen tearing. Did you enable compton's vsync?

@IvanMalison I'm also having screen tearing lately. Not sure why; I'll try enabling compton vsync though.

The following worked for me too (thanks to https://wiki.archlinux.org/index.php/intel_graphics#DRI3_issues and @Mic92's suggestion):
  videoDrivers = [ "intel" ];
  deviceSection = ''
    Option "DRI" "2"
    Option "TearFree" "true"
  '';

Are you still having tearing issues? I haven't used the intel igpu in a long while.
If tearing is still an issue we could mention enabling tearfree and ~compton~ picom in the NixOS manual, but forcing DRI2 in 2020 doesn't look like a good idea.

@rnhmjoj tearing was driving me mad when using XMonad. Picom did not solve the issue. I just applied all the workarounds suggested here, only enabling DRI 2 + TearFree worked (regardless of Picom being enabled). Interestingly, there seems to be no tearing on Wayland though. Sadly I'm too attached to my XMonad to give up XOrg just yet...

Oh... I was wrong, tearing was also fixed with:
  deviceSection = ''
    Option "DRI" "3"
    Option "TearFree" "true"
  '';

It does not work if I remove the DRI explicit line... weird?

Oops... no. I was wrong, tearing is back, had to revert to DRI 2.

Does DRI2 alone (without tearfree) have tearing?

Just removed TearFree and tearing seems gone still, only with DRI2.

Last thing before updating the manual, can you check this settings?
  deviceSection = ''
    Option "DRI" "3"
    Option "AccelMethod" "sna"
  '';

No tearing when using smplayer, but there is tearing and stutter when using Youtube on Firefox.

I also have extreme tearing with iGPU and none+xmonad setup. It happens with two different laptops. Tearing does not occur with the old xf86-video-intel driver, but it somehow breaks everything multi-monitor related.

@arximboldi, I realized you may have been trying to use AccelMethod and TearFree with modesetting, while this option are specific to xf86-video-intel. Can you check if you have videoDrivers = [ "intel" ]?

@rnhmjoj I have not explicitly put in the snippets, but I had videoDrivers = [ "intel" ] on all time through my experiments :)

I was just reding your snippet, maybe I should try to use modesetting driver instead which is more modern?

xf86-video-intel is considered legacy. It is still updated, but it does not have proper versioning. We only have bleeding edge git snapshots.

I actually just tried the modesetting driver and I get tearing with it, so I'll stick to intel + DRI 2 for now :)

Thanks @rnhmjoj for helping me investicate and documenting the results!

@arximboldi No problem, thank you for the collaboration.
It's quite sad that modesetting does still have tearing issues but at least we have found a working solution.

@rnhmjoj sadly since 20.09, using the intel driver causes OpenGL applications to crash. I am using the modesetting driver now as a workaround, but it causes screen tearing when watching videos in full screen :(

It looks like screen tearing is not solvable with modesetting driver. Tearing intensifies when X11 transform operations are applied (--scale, --scale-from etc).

That is sad. I was hoping this would be solvable in modesetting, overall it seems to perform better than intel, I've tried various options like PageFlip. However tearing just makes watching movies too annoying, so I'd like to go back to intel.
Are other people experiencing issues with intel on 20.09? Any clue on how to solve them?

sadly since 20.09, using the intel driver causes OpenGL applications to crash

Uhm, the xf86-video-intel package hasn't been updated since 20.03. The problem has probably been introduced by mesa xorg-server. Since I don't see any report of crashes in their issue tracker, it could very well be Nixpkgs has an outdated revision: you could try to override the package to use a more recent commit and see it that solves the crashes.
I can give you directions, if yu want to try.

Regarding TearFree for the newer modesetting driver:

Upstream issue: https://gitlab.freedesktop.org/xorg/xserver/-/issues/244
Upstream MR: https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/24

Rebased patches for xorg-server-1.20.9: https://gitlab.freedesktop.org/vliaskov/xserver/-/commit/18be004ad18b8a33e5854c08ab2ea023f34890b6

So there is some progress being made but I doubt the patches will be merged anytime soon. Though overriding xorg-server to apply the patches could be a viable solution for the meantime.
@rnhmjoj's idea also sounds good. The last xf86-video-intel release is currently 5 years old (list) but apparently there is still quite some activity in the master branch.
Other solutions could be using a compositor for X11 or switching to Wayland(+XWayland).

Thanks @primeos and @rnhmjoj!
@rnhmjoj I am trying overriding the xf86videointel package:
  nixpkgs.config.packageOverrides = pkgs: {
    xorg.xf86videointel = pkgs.xorg.xf86videointel.override {
      src = fetchGit {
        url = "https://gitlab.freedesktop.org/xorg/driver/xf86-video-intel.git";
        rev = "a511f22cdec56504913b457a2e60dafee8e2b570";
      };
    };
  };

But it is failing with:
error: attribute 'overrideScope'' missing, at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/pkgs/top-level/static.nix:286:10
(use '--show-trace' to show detailed location information)
building Nix...
error: attribute 'overrideScope'' missing, at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/pkgs/top-level/static.nix:286:10
(use '--show-trace' to show detailed location information)
building the system configuration...
error: anonymous function at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/pkgs/os-specific/linux/amdgpu-pro/default.nix:1:1 called without required argument 'libxcb', at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/lib/customisation.nix:69:16
(use '--show-trace' to show detailed location information)

@arximboldi override is for arguments of the package (ie options and dependencies), to change an attribute like src you have to use overrideAttrs, works similarly.

Sadly still not working @rnhmjoj, I get this error:
error: attribute 'overrideScope'' missing, at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/pkgs/top-level/static.nix:286:10
(use '--show-trace' to show detailed location information)
building Nix...
error: attribute 'overrideScope'' missing, at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/pkgs/top-level/static.nix:286:10
(use '--show-trace' to show detailed location information)
building the system configuration...
error: anonymous function at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/pkgs/os-specific/linux/amdgpu-pro/default.nix:1:1 called without required argument 'libxcb', at /nix/store/csgakpb8wgymlj3a4374347hkg23qa01-nixos-20.09.2161.214e71a78a9/nixos/lib/customisation.nix:69:16
(use '--show-trace' to show detailed location information)

Oh, right: it's because the override is replacing the whole xorg set with { xf86videointel = ...; }.
Try this:
{
  nixpkgs.overlays = lib.singleton (self: super:
  { xorg = super.xorg // {
      xf86videointel = super.xorg.xf86videointel.overrideAttrs (old: {
        src = fetchGit {
          url = "https://gitlab.freedesktop.org/xorg/driver/xf86-video-intel.git";
          rev = "a511f22cdec56504913b457a2e60dafee8e2b570";
        };
      });
    };
  });

}

Thank you so much for your help @rnhmjoj! :heart:
That has worked. I can now use the Intel driver again without OpenGL crashes nor tearing issues.
Looking forward to trying the modesetting driver again in the next release, hopefully the TearFree patch is in by then!

While just enabling picom (even with vSync=true) didn't work on its own for me, for some reason, some combination of these settings DID stop screen tearing for me:
  services.picom = {
    enable = true;
    vSync = true;
    backend = "xr_glx_hybrid";
    settings = {
      glx-no-stencil = true;
      glx-no-rebind-pixmap = true;
      unredir-if-possible = true;
      xrender-sync-fence = true;
    };
  };

I think the key here was backend = "xr_glx_hybrid". No other backend got rid of the screen tearing for me. I'm not sure if the extra options under settings matter. I don't know why this made the difference.

For ones using Xrandr transform operations like --rotate or --scale-from, I doubt utilization of a compositor will solve the tearing issue completely. I've seen a branch of modesetting driver that includes TearFree work that has been going on X11 repo. This is the commit URL: https://gitlab.freedesktop.org/vliaskov/xserver/-/commit/18be004ad18b8a33e5854c08ab2ea023f34890b6.
I don't have time to test it, yet alone break my xserver, meanwhile an override to X11 package with this branch may solve the issue.

@rmanne
Thank you! I can confirm this is working on Intel HD Graphics 520 with xmonad™ and modesetting/DRI3.


[FLINK-32201][runtime]Automatically determine if the shuffle descriptor needs to be offloaded by the blob server based on the number of ShuffleDescriptor edges.

What is the purpose of the change
Flink support distributes shuffle descriptors via the blob server to reduce JobManager overhead. But the default threshold to enable it is 1MB, which never reaches. Users need to set a proper value for this, but it requires advanced knowledge before configuring it.
I would like to enable this feature by the number of connections of a group of shuffle descriptors. For examples, a simple streaming job with two operators, each with 10,000 parallelism and connected via all-to-all distribution. In this job, we only get one set of shuffle descriptors, and this group has 10000 * 10000 connections. This means that JobManager needs to send this set of shuffle descriptors to 10000 tasks.
Since it's difficult for users to configure, I would like to give it a default value.
Brief change log

determine if the shuffle descriptor needs to be offloaded by the blob server based on the number of ShuffleDescriptor edges.

Verifying this change
This change added tests and can be verified as follows:

*Manually verified the change by running a cluster with 1 JobManagers and 2000 TaskManagers (10 slots per TaskManager), a streaming program with 20000 parallelism, and verifying that the task deploy was successful.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving):  (no)
The serializers:  (no)
The runtime per-record code paths (performance sensitive):  (no)
Anything that affects deployment or recovery: Task Deployment: (yes)
The S3 file system connector:  (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for your review. @Thesharing , your comments are totally correct.
I updated the PR. Please take a look at your free time.
There is a change beyond the comments. I removed the logic to exclude Unknown shuffle descriptors, as it also made the serialized shuffle descriptors larger (200KB for 10000).

@Thesharing, @wanglijie95 , I have rebase this to the master branch. Could you please take another look?

@wanglijie95 Thanks for your review, I have updated this PR, PTAL

@wanglijie95 ，Thanks for pointing out the problems. I have addressed them as you suggested.

We give this option a default value  2048 * 2048, which is good enough for most cases.
This means JobManager need to serialize and transport 2048 shuffle descriptors (about 30KB) to 2048 consumers (60MB in total).

@flinkbot run azure


Using the TailwindCSS project configuration

Clear and concise description of the problem
Currently, the Nuxt OG image uses the TailwindCSS CDN with a default configuration.
I've suggested adding an option to use the local configuration used in the project, and more specifically the one used on the [TailwindCSS Nuxt module] (https://tailwindcss.nuxtjs.org/tailwind/config).
With the current configuration, I can't use my custom configuration with custom colors.
Suggested solution
Add an option to retrieve the local configuration and pass it to the module.
Modify the file html.ts at line 122 to use the Tailwind project configuration.
Alternative
No response
Additional context
No response

Hey, thanks for this suggestion.
This is now supported with the v3. Your custom theme classes should just work if you're using Tailwind or UnoCSS.
Let me know if you have any issues.


[SR-6097] Add appropriate bounds to CountableClosedRange and CountableRange.

In Swift 4.1 and earlier, both CountableRange and CountableClosedRange required
their bound types to be Strideable with a Stride type that conforms to
SignedInteger. Those constraints were not present on the generic
typealiases that replaced these structs. Add them back now, which
fixes the GRDB source compatibility regression.
Fixes SR-6907 / rdar://problem/29066394.

Only the last commit is relative to this PR; the rest are part of https://github.com/apple/swift/pull/15497.

@swift-ci please test source compatibility

@swift-ci please smoke test and merge

@swift-ci please smoke test

@swift-ci please smoke test


CSI preview but NFS is supported already since 9 October

I have checked this Azure Update: https://azure.microsoft.com/es-es/updates/azure-files-premium-tier-is-now-available-in-more-regions-with-lrs-zrs-and-nfs-support/
Indicating NFS support , not preview, so CSI driver is not necessary in new AKS clusters for support NFS feature? What is the new documentation about this?
Thanks

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

ID: fc6adf61-d0eb-f95c-60a7-bd3e1b52b5a5
Version Independent ID: e1a7cc5d-2988-3c62-5169-48eac7c0fc47
Content: Use Container Storage Interface (CSI) drivers for Azure Files on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/azure-files-csi.md
Service: container-service
GitHub Login: @palma21
Microsoft Alias: mlearned

Thanks for the Comment. We are actively investigating and will get back to you soon.

As per the blog, Network File System (NFS 4.1) is also in public preview on premium tier


working code with failing unit test

Codecov Report

Merging #280 into master will decrease coverage by 1.07%.
The diff coverage is 90%.

@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
- Coverage     100%   98.92%   -1.08%
==========================================
  Files           5        6       +1
  Lines          83       93      +10
  Branches       18       18
==========================================
+ Hits           83       92       +9
- Misses          0        1       +1

Impacted Files
Coverage Δ

lib/bankHolidayChecker.js
90% <90%> (ø)

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 6806682...01776ad. Read the comment docs.


Break up SCSS and stylus into imports

Related: https://github.com/ianrose/typesettings/pull/43
Allows the dev to determine when and where typesettingsInit takes place rather than have those required styles be hardcoded in the _typesettings.scss or _typesettings.styl.
  html {
    font-size: ($base-font-size / 16px) * 100%;
    -webkit-text-size-adjust: 100%;
    -ms-text-size-adjust: 100%;
  }

  body {
    @include setLeading();
  }

@ianrose Nice, and thank you for "typesettings" :)


Update participant guide and add to header

See the participant guide https://ioos.github.io/ioos-code-sprint/resources/participant-guide/
https://github.com/ioos/ioos-code-sprint/edit/gh-pages/resources/participant-guide.md
We should update it for 2024 and link to the header of the 2024 website.
Should it be a yearly page??
cc: @mwengren

Added https://ioos.github.io/ioos-code-sprint/2024/participant-guide

@mwengren please review the page and close this issue if you're comfortable with it.

It looks good to me!  We can update if needed before next week.


cmd+shift+a opens terminal window with org4idea plugin enabled

This is very strange. I've never used an intellij product before, just downloaded WebStorm yesterday so I don't know what info you need nor how to get it. What I do know, is that when I enable org4idea, restart webstorm, wait a few seconds, then press cmd+shift+a I get this

I have reproduced this by disabling org4idea, restarting webstorm, then testing and this window never pops up. Then I re-enable org4idea, restart, test and the window always pops up.

Surprising no one, turns out this isn't the case. macOS seems to have a default keybinding cmd+shift+a that "Open man Page in Terminal". Go to System Preferences -> Keyboard -> Shortcuts -> Services. Turning that off seems to fix this (at least for now). I'm still very curious why this didn't happen all the time 😕


Fix for InternalAPIHelper Error in Unity 2023.2.15f1 and Later

This pull request addresses an issue that changed the signature of the GetDrawerTypeForType method in Unity 2023.2.15f1 and later versions, including Unity 2023.3.
These changes were causing errors in InternalAPIHelper.
This PR fix ensures compatibility and resolves the error.

I have confirmed that it works and have merged it. Thank you!


Repeated getInterpreters on activation

We were looking into performance improvements for registering notebook controllers for Python kernels and while debugging we found that

https://github.com/microsoft/vscode-python/blob/main/src/client/interpreter/interpreterService.ts#L139 InterpreterService#getInterpreters is invoked four times (on my machine particularly) and from the log, each time it returns the same three interpreters and each call takes one second.

Under the hood it's using fs to read and creating processes to find intepreters so maybe there is room for perf improvement to do some caching or moving some of the code to external module/binary.
Environment data

VS Code version: 1.59-insiders
Extension version (available under the Extensions sidebar): 2021.7.3-pre.1
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Conda, Pyenv
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pyenv, conda

Hey @rebornix, what version of the extension are you using?

Under the hood it's using fs to read and creating processes to find intepreters

I don't believe this is the expected behavior, and was not able to reproduce this. I believe https://github.com/microsoft/vscode-python/pull/16674 fixes the cause of why it takes one second despite caching.


Support libkrun (mostly for Virtio-GPU Venus), via krunkit

libkrun seems to have merged the PR for supporting Virtio-GPU Venus (https://docs.mesa3d.org/drivers/venus.html)

https://github.com/containers/libkrun/pull/174

QEMU still hasn't merged the patches for supporting Venus<EMAIL_ADDRESS>

lima should probably use https://github.com/containers/krunkit for implementing the libkrun driver.
Not sure if it should be a built-in driver, or we have to implement the plug-in system.

We can explore and finalise the approach for plug-in model. As i already see several experimental PR's for supporting different drivers.
This will help us in providing different options based on performance and usecases.


Fixed color config in hdd example

#610

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.605%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
329

Relevant Lines:
392

💛  - Coveralls


Progressive archery is incompatible

So I tried to install SolipIngen's progressive archery mod, but it seems like its not compatible... so, Im requesting that this mod be compatible with sinytra connector

Please submit a proper description as instructed in the README


[Question] When the Typescript 2.3 will be available in VS2017?

[When the Typescript 2.3 will be available in VS2017?

Now that the VS2017.2 is released, the installer for VS2015 should also work for VS2017. The site does not mention about this, though.

Am I supposed to do anything particular to make 2.3 work with VS17? I installed the VS15 package and restarted the IDE but that changed nothing.

Open Tools->Options->Text Editor->JavaScript/TypeScript->Intellisense and you should see "Use TypeScript version" which should be set as 2.3. This should be automatically set but it seems sometimes not.

You should install Typescript SDK for VS2017 from https://www.microsoft.com/en-us/download/confirmation.aspx?id=55258

I've installed the Typescript SDK, changed the Tools->Options->Text Editor->JavaScript/TypeScript>Intellisense to 2.3 version, but when I check the version of typescript in the About screen of VS, I still have 2.2.2 version

Try something specific with TS 2.3, for example:
class Component<P, S = object> { }

If this does not show error then you have TS 2.3.

this does not show error

Then you have TS2.3, regardless of what About page says 🎉

Thanks to all.
I don't know how that critical, but VS 2017 ignore the TypeScriptToolsVersion setting of csproj with ToolsVersion="14.0".

@ifle Visual Studio 2017 15.3 will respect your TypeScriptToolsVersion setting in Intellisense as well.

Forgot to mention, 15.3 also includes TypeScript 2.3 by default.


Array source

Hi! I'm trying to build a datatable with an array as a source. My goal is to allow the users to filter some values (select one or more employees from a list, date rangee and some other checkboxes) and perform some queries to gather data, do some business logic and present the results in the datatable.
I did not find any example in the docs about how to this and actually don't know if this is possible.

Hey. For now, only Doctrine ORM queries are supported, although I want it to be supported by default as well. Creating an array adapter would require adding a ProxyQuery class accepting arrays and handling pagination and sorting.

Ok, I'll wait. Thanks!

I'm trying to achieve same goal. Hope can this steps can help you @easis
In my case, I have a custom Doctrine query that obtains an array result (plain associative array).
// Controller
    public function customDatatableAction(Request $request, WorkOrderResultRepository $wor): Response
    {
        $data = $wor->batchAnalysis();
        /**
            [
              [
                "batchCode" => "2416",
                "totalWorkstationSeconds" => "815448"
               ],
               [...]
            ]
        */
        $batchAnalysisArrayProxyQuery = new ArrayProxyQuery(
            data: $data,
            page: $request->query->has('page_batch_analysis') ? (int) $request->query->get('page_batch_analysis') : 1
        );
        $dataTable = $this->createDataTable(BatchAnalysisDataTableType::class, $batchAnalysisArrayProxyQuery);
        $dataTable->handleRequest($request);
        if ($dataTable->isExporting()) {
            return $this->file($dataTable->export());
        }

        return<EMAIL_ADDRESS>[
            'batchAnalysisTable' => $dataTable->createView(),
        ]);
    }

Then I've created a custom ArrayProxyQuery with this code:
<?php

namespace App\Datatable\Query;

use App\Doctrine\Enum\SortOrderEnum;
use Kreyu\Bundle\DataTableBundle\Filter\Filter;
use Kreyu\Bundle\DataTableBundle\Filter\FilterData;
use Kreyu\Bundle\DataTableBundle\Pagination\PaginationData;
use Kreyu\Bundle\DataTableBundle\Query\ResultSet;
use Kreyu\Bundle\DataTableBundle\Sorting\SortingData;
use Kreyu\Bundle\DataTableBundle\Query\ProxyQueryInterface;
use Kreyu\Bundle\DataTableBundle\Query\ResultSetInterface;

final class ArrayProxyQuery implements ProxyQueryInterface
{
    public function __construct(
        private array $data,
        private readonly int $page = 1,
    ) {
    }

    public function sort(SortingData $sortingData): void
    {
        foreach ($sortingData->getColumns() as $column) {
            usort($this->data, function ($a, $b) use ($column) {
                if ($column->getDirection() === strtolower(SortOrderEnum::ASC->value)) {
                    return ($a[$column->getName()] < $b[$column->getName()]) ? -1 : 1;
                } else {
                    return ($a[$column->getName()] > $b[$column->getName()]) ? -1 : 1;
                }
            });
        }
    }

    public function paginate(PaginationData $paginationData): void
    {
        if ($this->page > 1) {
            $total = count($this->data);
            $limit = $paginationData->getPerPage();
            $totalPages = ceil($total / $limit);
            $page = max($this->page, 1);
            $page = min($page, $totalPages);
            $offset = ($page - 1) * $limit;
            if( $offset < 0 ) {
                $offset = 0;
            }
            $this->data = array_slice($this->data, $offset, $limit);
        }
    }

    public function getResult(): ResultSetInterface
    {
        return new ResultSet(
            iterator: new \ArrayIterator(iterator_to_array($this->data)),
            currentPageItemCount: $this->page,
            totalItemCount: count($this->data),
        );
    }

    public function filterDataBy(FilterData $filterData, Filter $filter): void
    {
        $this->data = array_filter($this->data, function ($item) use ($filterData, $filter) {
            return $item[$filter->getConfig()->getName()] === $filterData->getValue();
        });
    }
}

At last, a simple ArrayProxyQuery is now available in the 0.23 version of the bundle. Usage explained in the documentation.


godeps: bump docker2aci

docker2aci recently gained support for the v2 docker HTTP API. Bumping
the version used by rkt to pull that into here.

you hurt CI's feelings

Interesting:
=== RUN TestNonRootFetchRmGcImage
--- FAIL: TestNonRootFetchRmGcImage (4.60s)
	rkt_non_root_test.go:94: Running rkt install
	rkt_non_root_test.go:109: Running /home/runner/rkt/builds/build-rkt-src-v222/build-rkt-0.12.0+git/bin/rkt --dir=/tmp/datadir-258904928 --local-config=/tmp/localdir-763737919 --system-config=/tmp/systemdir-093519250 image gc
	rkt_non_root_test.go:114: Running /home/runner/rkt/builds/build-rkt-src-v222/build-rkt-0.12.0+git/bin/rkt --dir=/tmp/datadir-258904928 --local-config=/tmp/localdir-763737919 --system-config=/tmp/systemdir-093519250 image rm sha512-cf933df66addd9008fe28d50d97b01c1
	rkt_non_root_test.go:123: Running /home/runner/rkt/builds/build-rkt-src-v222/build-rkt-0.12.0+git/bin/rkt --dir=/tmp/datadir-258904928 --local-config=/tmp/localdir-763737919 --system-config=/tmp/systemdir-093519250 image rm sha512-cfb3d8cb872d7da7aae3265e3cba128e
	rkt_tests.go:281: didn't receive expected output "successfully removed": rkt: error removing aci for image ID "sha512-cfb3d8cb872d7da7aae3265e3cba128e": permission denied, are you root or the owner of the image?

		rkt: 1 image(s) cannot be removed

		rkt: error(s) found while removing images

	rkt_tests.go:125: rkt didn't terminate correctly: exit status 1
		Output:

I don't see why docker2aci would impact that test...

semaphore looks fishy. kicking it.

  error downloading ACI: Get https://github.com/coreos/etcd/releases/download/v2.1.2/etcd-v2.1.2-linux-amd64.aci: dial tcp: lookup github.com: no such host

hmph

Semaphore is happy now. LGTM

Sanity check, is this (V2) flow actually covered by one of the functional tests?

Sanity check, is this (V2) flow actually covered by one of the functional tests?

We fetch docker images from the docker hub in TestFetch so yes


[2.0.3] Debian package conflict between influxdb vs influxdb2

Steps to reproduce:
wget https://dl.influxdata.com/influxdb/releases/influxdb2_2.0.3_amd64.deb
sudo dpkg -i influxdb2_2.0.3_amd64.deb

Expected behavior:
Upgrade from 2.0.2 to 2.0.3 should be straight forward.
Actual behavior:
sudo dpkg -i influxdb2_2.0.3_amd64.deb
Selecting previously unselected package influxdb2.
dpkg: regarding influxdb2_2.0.3_amd64.deb containing influxdb2:
 influxdb2 conflicts with influxdb
  influxdb (version 2.0.2) is present and installed.

dpkg: error processing archive influxdb2_2.0.3_amd64.deb (--install):
 conflicting packages - not installing influxdb2
Errors were encountered while processing:
 influxdb2_2.0.3_amd64.deb

Indeed:
dpkg -l influxdb
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name           Version      Architecture Description
+++-==============-============-============-=================================
ii  influxdb       2.0.2        amd64        Distributed time-series database.

Environment info:

System info: Debian 10 - up to date
InfluxDB version: 2.0.2
Other relevant environment details: N/A

This is expected behavior as a result of changes to the packaging, and is mentioned in the release notes at https://docs.influxdata.com/influxdb/v2.0/reference/release-notes/influxdb/

Thanks @psteinbachs - I was to search for release notes to check this point :+1:

I could upgrade it successfully on a Ubuntu 20.04 with these steps:
# Backup data / you could avoid the param -t if you already have a proper influx config
$ influx backup /path/to/backup/directory -t [admin_admin]

# Back of data directories , just in case
$ sudo cp -a /var/lib/influxdb/.influxdbv2/ /path/to/backup/data_directory

# Stop service
$ sudo systemctl stop influxdb

# Remove previous version - Do not use param --purge
$ sudo dpkg -r influxdb

# Install new version
$ sudo dpkg -i influxdb2_2.0.3_amd64.deb

# Move data directories to new version location
$ sudo mv /var/lib/influxdb/.influxdbv2/engine /var/lib/influxdb/engine
$ sudo mv /var/lib/influxdb/.influxdbv2/influxd.bolt /var/lib/influxdb/influxd.bolt

# Start service
$ sudo systemctl sart influxdb

I could upgrade it successfully on a Ubuntu 20.04 with these steps:
# Backup data / you could avoid the param -t if you already have a proper influx config
$ influx backup /path/to/backup/directory -t [admin_admin]

# Back of data directories , just in case
$ sudo cp -a /var/lib/influxdb/.influxdbv2/ /path/to/backup/data_directory

# Stop service
$ sudo systemctl stop influxdb

# Remove previous version - Do not use param --purge
$ sudo dpkg -r influxdb

# Install new version
$ sudo dpkg -i influxdb2_2.0.3_amd64.deb

# Move data directories to new version location
$ sudo mv /var/lib/influxdb/.influxdbv2/engine /var/lib/influxdb/engine
$ sudo mv /var/lib/influxdb/.influxdbv2/influxd.bolt /var/lib/influxdb/influxd.bolt

# Start service
$ sudo systemctl sart influxdb


Download link issue

Trying to install beta .exe for win os. Loading till 60% and stops.

https://github.com/SadeghHayeri/GreenTunnel/releases/download/v1.3.2/Windows-Green.Tunnel.exe
This link worked for me.


Capacitors/ Resistors not appearing in assembly view

This is an issue in circuit-to-svg most likely
/bounty $15
https://github.com/user-attachments/assets/305ff112-c312-44ed-9877-bbba18b7985e

CC @AnasSarkiz

@techmannih  you can have the bounty if you fix the story 🫡

/tip $3 @techmannih


Custom Fields

I am not getting back custom fields using the "Query" method, however, it does respect a where clause on the custom field. But I need to get back custom fields to work with that data.

I see, records without values in the custom field don't come back at all - but they are coming back for records with values in the field. I guess that's easy enough to work around.


Update python-lzo to 1.11

Green! The trick was to update python_lzo to python-lzo, which is the package name on PyPI.

Copied to wheels.galaxyproject.org. Thanks @nsoranzo!


[Mega-Menu]

Problem
The current resources navigation menu which includes Topic, Medium and Languages is displayed as an unordered list. It doesn't make the resources page visually appealing.

Description

I would like to implement the mega-menu features with a dropdown for the current resource navbar to improve the look and feel of the navigation menu.
Implementation

[x ] I would be interested in implementing this feature.

#23 addresses the same issue and has been opened for discussion by the maintainers. It'd be great if you provide your inputs over there.


Hope ND4J can support bit operations on integers.

The bit operations on integers like and, or, xor, move, etc, are too important for general purpose computing applications. It would be great if ND4j supported these operations.

And/Or/Xor are already available at cpp level.

What else you'd like to have?

bit move operation.

You mean bit shift?

such as:  i>>2

Yes, that's bit shift. Okay, we'll add those too.

And/Or/Xor are already available at cpp level.

Not Java level yet though as far as I can see... easy to add though

Yes, that's bit shift. Okay, we'll add those too.

For import, we need and/or/xor/not, left/right shift...
https://www.tensorflow.org/api_docs/python/tf/bitwise
Plus one more while we're at it:
https://www.tensorflow.org/api_docs/python/tf/bitcast

Great! and when can I use them?

A special requirement is cyclic shift. such as: b0b1b2b3b4b5>>2, the result is b4b5b0b1b2b3; and b4b5b0b1b2b3<<2 is b0b1b2b3b4b5.
ndarrayX>>ndarrayY, cyclic shift ndarrayX[i]>>ndarrayY[i].
another requirement is ndarray map, such as ndarrayX.map(Map<Integer,Integer> ndaMap), the result is ndarrayY[i]=ndaMap.get(ndarrayX[i])

the cyclic shift operation  is very important for me.

Bit shift operations are related to sign. Does ND4J plan to support unsigned integers and unsigned short integer types?

Unsigned types were added soon after beta4 was released. They are fully supported on current master/snapshots.
https://github.com/eclipse/deeplearning4j/blob/master/nd4j/nd4j-buffer/src/main/java/org/nd4j/linalg/api/buffer/DataType.java#L32-L34

Look-up table method is a very common calculation method, so it is suggested that ND4J provide a method:
INDArray INDArray.map(Map<Integer,Integer> map);
INDArray resultArray=vector.map(mm);
for (int i;i<vector.shape()[0];++i){
    resultArray.putScalar(i, map.get(vector.getInt(i)));
}

@cqiaoYc That's just gather operation on 1d array, which we already have.
SameDiff implementation:
https://github.com/eclipse/deeplearning4j/blob/57efc2bf956af1140c00675aca7c8bd8a67f29bf/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/autodiff/samediff/ops/SDBaseOps.java#L579-L614
https://github.com/eclipse/deeplearning4j/blob/master/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ops/impl/shape/Gather.java
Works identically to TF gather:
https://www.tensorflow.org/api_docs/python/tf/gather?hl=en

@AlexDBlack Thank you! I I'm not familiar with sameDiff, I'll learn it.

@cqiaoYc You can use the operation normally for ND4J too, not just in SameDiff
(Though now that I look at it, there's no INDArray constructor for that op)
You can do something like:
INDArray inputVector = ...
INDArray indicesVector = ...
INDArray out = ...
Nd4j.exec(DynamicCustomOp.builder("gather")
    .addInputs(inputVector, indicesVector)
    .addOutputs(out)
    .addIntegerArguments(0)   //Axis - along dimension 0
.build());

@AlexDBlack Thank you!

Another requirement is to compute bit-based Hamming distances, that is, to compute how many bits of the corresponding integers in two ndarrays are different. This can be calculated by XOR, mapping and summation, so it is not very urgent.

shift_bits & cyclic_shift_bits implemented.


Please add netshoot kubect plugin to krew

Hi,
netshoot is such a timesaver and also your plugin, it would be really nice if you could add the plugin to krew which basically only needs writing a manifest file to add it to the krew plugin index.
https://krew.sigs.k8s.io/docs/developer-guide/distributing-with-krew/
i can also give it a shot because with all the plugins it's much nicer to have a system to automatically update all the plugins, and with hundrets of them already in krew it would be a huge benefit, because imho netshoot is the best debug image out there :)!

oh... ok they deem it a too thin wrapper over already existing primitives, seems they want to correct their one-line wrapper count a bit from the beginning of krew project...
yea i can understand them... so it's back at just using the base primitives, thx for the quick answer!


[Feature request] Support project group

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
When you have a lot of projects, it becomes very difficult to manage, and there is currently no search function. If you introduce a concept similar to folders or groups, allowing users to group some projects together, it would be very convenient.
Describe alternatives you've considered
No.
Additional context
No.

Thanks for your feature request @secsilm. I've taken this to the product and engineering team and we will add it to our queue for review.


Unable to create reminders with date within [[link]]

Thank you for this plugin— its incredibly useful. I tend to use dates within [[]] links wherever possible in order to ensure that tasks can also be back-linked to the right day. Is there a way to change settings in order to support this?

I'm glad you like.
Can you give me an example input and expected reminder?
For example,
input:
- [ ] foo (@2021-09-10)

expected reminder:

title: foo
due date: 2021/09/10

Sure— What I mean is
 - [  ]  foo (@[[2021-09-10]] 14:55)
The advantage of this is that is ensures that all dates can also be backlinked to the daily notes page

I'm also using the Kanban board with the option turned on to link dates to journal pages.  The resulting date format is
@[[YYYY-MM-DD]]
Because I've chosen YYYY-MM-DD as the date format within the Kanban settings.  I think time formats are set using the following formats
@@HH:MM
Currently this date format isn't recognised by reminder even with the enable kanban format toggle enabled.  I realise this might be tricky to implement generically as Kanban allows you to change not only the date format but also the date and time triggers (e.g. from @ and @@ to anything else).

@rushikb Thank you.  I will consider implementing it.
@dcoales I didn't know that! Current implementation uses @ for date and @@ for time trigger.  Also their format must be YYYY-MM-DD , HH:mm.
I will consider to use Kanban plugin's format in the setting.

Thank you very much.
On Sat, Sep 11, 2021 at 1:11 PM Yuhi Ishikura @.***>
wrote:

@rushikb https://github.com/rushikb Thank you. I will consider
implementing it.
@dcoales https://github.com/dcoales I didn't know that! Current
implementation uses @ for date and @@ for time trigger. Also their format
must be YYYY-MM-DD , HH:mm.
I will consider to use Kanban plugin's format in the setting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uphy/obsidian-reminder/issues/22#issuecomment-917396806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEV5W53WGEGINDBCZ4PGMG3UBNBNJANCNFSM5DYWLO3A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@rushikb Supported in v1.1.0.

@dcoales  Supported in v1.1.0.

Thank you! this is amazing!

Hi,
As part of this change, could Kanban dates in the format @[[YYYY-MM-DD]] be supported.  I have tried turning on both Kanban support and linking to dates in the options but this still doesn't recognise Kanban dates when they too are links to dates.
Many thanks,
David

I have tried turning on both Kanban support and linking to dates in the options but this still doesn't recognise Kanban dates when they too are links to dates.

It is not reproduced for me.  Maybe you need a restart.
If you have problems again, please open again.


Status Workflow Bar Converted to Old Workflow

Account name or name of person: Di
Any personally identifying information: I like long walks on the beach

Describe the bug
Submissions below are tests on the live webapp. The workflow status bar is different than the status bar that appears on other submissions. I'm unsure the cause of this behavior.
Examples:
Test 1: https://apply.opentech.fund/apply/submissions/4964/
Test 2: https://apply.opentech.fund/apply/submissions/4964/
To Reproduce
Steps to reproduce the behavior:

Go to Test 1: https://apply.opentech.fund/apply/submissions/4964/ and Test 2: https://apply.opentech.fund/apply/submissions/4964/

Expected behavior

Affected roles

Staff

Screenshots

@fourthletter Staff and applicants see the states differently. Both of these you have submitted.
Two me they both look exactly like all other proposals in that round.


Adding CUSTOM_CERTIFICATE environment variable

Adding a CUSTOM_CERTIFICATE environment variable for use in environments with self-signed certificates.

Already implemented on a separate PR


[SU-8] Easily share workspace with support

Currently, it's not obvious how to share a workspace with Terra Support. They need to know the Terra Support email. Sharing a workspace is frequently the first step after a user contacts Support, and currently Support has to spend time explaining how to share the workspace.
This adds a "Share with support" switch to the share workspace modal.
Potentially controversial... this removes Terra Support from the collaborators list in the share workspace modal, which prevents a user from seeing or modifying Support's level of access to the workspace. The plan is discourage granting the Support team higher than read access. In cases where they do need additional access, the user will be asked to add an individual Support team member to the workspace.
For cases where Support already has additional access to a workspace or additional access has been granted through an API request, etc. the "Share with support" switch has a tooltip that includes the level of access granted to Support.

@nawatts can you create a PR deployment for this?

Thanks for the suggestion on the tooltips. I think that prevented a lot of future confusion.


fix(tasks.js): Changing to 'url.fileURLToPath' instead of the 'url.UR…

Here's the fix that works on my end. Like I said, this is your repo so I don't mind you rejecting the pull request and fixing the issue yourself. This is mainly to point out what I did. Hope this helps :)
…L' class
The 'url.fileURLToPath' function provides the correct path for all platforms (including Windows).
This corrects the builderDir glob bug.
fdd3224

Thanks! If you don't mind, I'll try to get the CI passing on this PR so we can merge it :)

It looks like asset watching is not working on MacOS and Windows. Don't worry, that's certainly unrelated – I never ran the CI on anything but Linux before...

Alright, I have a macbook too so I can investigate this a bit. I'll check on Windows first, let you know what I find.

Alright, I have a macbook too so I can investigate this a bit. I'll check on Windows first, let you know what I find.

Wow, that'd be great!

I just merged my test branch into this one so we can continue here: Decided to consistently convert back slashes to forward slashes since gulp/glob wants it that way. Webpack seems to works with both. Now wondering why gulp.watch doesn't work...

Hmmm weird... I installed this fork on my npm and went through the normal steps of creating a project and running it. When I do npm start I get hit by:
File not found with singular glob: /C:/Users/.../jspsych-builder//assets/index.html

So there is some other place where the paths are bongled hahah. I'm gonna investigate that, perhaps it will help us with the gulp watch, perhaps not. However, I cannot user test the gulp watch issue until I've resolved this.

Did you npm link your local version?

So there is some other place where the paths are bongled hahah.

I hope not :see_no_evil: No idea where that might be...

Oh, and interestingly, the windows CI job doesn't have the problem: https://github.com/bjoluc/jspsych-builder/runs/4510036258?check_suite_focus=true

Hmmm wait, when I do jspsych run this works, npm start does not work. I have to admit I don't see what the difference is. I am doing npm install -g <local repo> and up until now it's been responding to my changes in the local repo. Perhaps I am missing something

Ahhhh ofc, npm start uses the local jspsych-builder in <project>/node_modules which is downloaded by the global jspsych-builder during init. This still has the old new URL(...) code hahaha

It works! Changes to the asset folders are reflected in .jspsych-builder/media folder. Also, just for sanity, I tried making a jsPsychImageKeyboardResponse with a reference to a non-existent image. When I added the image, it got copied in the watch folder and appeared on my screen.
So gulp watch works, but the test is not working. Or is it? You said this:

Oh, and interestingly, the windows CI job doesn't have the problem: https://github.com/bjoluc/jspsych-builder/runs/4510036258?check_suite_focus=true

Don't know if you were referring to this issue, but clicking on the link I see the same fail as on macos. Maybe I am misunderstanding you.
I'm going to look a bit at the tests to see if I can spot the issue. It would be great to get it working!

It works! Changes to the asset folders are reflected in .jspsych-builder/media folder.

Great, that's insanely helpful, thanks!

Perhaps you meant the npm start issue.

:+1:
Maybe it's related to shelljs and how I use it then.  I should probably have a few of these scripts and also check for proper errors and stuff. Don't know how to best go about this. Unit testing – while probably possible – doesn't seem too benifitial here, since most of jspsych-builder is just assembling configs for other tools.

Maybe it's related to shelljs and how I use it then.  I should probably have a few of these scripts and also check for proper errors and stuff. Don't know how to best go about this. Unit testing – while probably possible – doesn't seem too benifitial here, since most of jspsych-builder is just assembling configs for other tools.

Yeah exactly, I know what you mean.
Well, just to let you know I did some more testing (on Windows). So while running only half of the story.js script up until the server was running, I started node in the terminal and tested creating files with shelljs. What I found was that files and folders are reflected in the watch folder but not files within folders. Specifically, when I created the folder test within media/images (like you do in your original script) then that is reflected. However, when I create 1.txt within that test folder, it does not reflect it.
So perhaps gulp is not configured to reflect files within folders? Which is weird because the tests run ok on Linux...
But so actually, if gulp is actually not watching files within folders, then the tests actually worked out weed out a bug hahah. But I get what you mean, I also often feel like this too. But perhaps it is good to have it since you are working on a Linux (?). I don't know though, let's see if there's any validity to this "gulp watch not seeing files within folders" thing.

For now perhaps, we can change the tests so that it only checks 1st level changes in the asset folders. Then the checks can go through and original changes can be implemented. Leave it up to you to decide ofc :)

Just saw you started the checks with changes to the test script. Thanks for being so active with this bug fix and accepting my pull request. I really appreciate this tool and use it at work.

Thanks for the kind words and for all the help @Haffi921! GH somehow doesn't interpret the Co-authored-by in the commit msg, but folks should be smart enough to get that you didn't thank yourself in the message of a squash-merged PR :sweat_smile:


V0.9.0 memory leak

I am using V0.9.0 as a service, running on default configuration. In a remote server it was using 3GB RAM. Now i tried that version on my local computer it has been running for 2 hours and it is using 61MB RAM. Do we have a solution for this?

@rebelsowl does the leak also occur on v0.8.3?

In v0.8.0 it occured. Now I am trying on v0.8.3, I will let you know in a few hours

Thank you! If you can identify the version in which the leak began that would save a lot of time.

Another useful piece of data would be to go to <host>:9182/debug/pprof/heap, save the output file and share it here. That will give us counters on what memory is in use and where there are many allocations.

I have been running v8.3.0 for 4 hours.It looks like there is no memory problem.

I will run this version for a while after that I will switch to v9.0 then upload heap file.

Thanks @rebelsowl!
The 0.8.3 dump looks fine, as you say. Nothing unexpected. Let me know when you have one for 0.9 and I can compare.

I have installed v9 back on my local computer but I couldn't reproduce leak for 4-5 days, then started v9.0 on server, it's memory usage looks fine too.
Here is heap from server:
heap_prod.gz

Hi,
Same issue here.
It appears than when vmware collector is enabled, the scraping always timed out. I even tried a localhost browser http://localhost:9100/metrics -> timed out.
I have to uninstall with msiexec and then reinstall without vmware (it's  mssql server vm on vmware hypervisor with vmware agent installed).
This morning the wmi_exporter cosummed about 650MB on memory and do not respond. I have to uninstall and reinstall without vmware collector.
ENABLED_COLLECTORS=mssql,process,vmware,os,cpu,cs,logical_disk,net,service,system,tcp -> scraping timed out
ENABLED_COLLECTORS=mssql,process,os,cpu,cs,logical_disk,net,service,system,tcp -> no issue.

@carlpett
I'v got the same issue, on one server wmi_exporter consumes 3GB ram.
heap.zip
Running parameters:

"C:\Program Files\wmi_exporter\wmi_exporter.exe" --log.format logger:eventlog?name=wmi_exporter --collectors.enabled cpu,cs,iis,logical_disk,memory,net,os,system,tcp,vmware,textfile,netframework_clrexceptions,netframework_clrlocksandthreads,netframework_clrmemory,netframework_clrjit,service,process --telemetry.addr :9182  --collector.textfile.directory C:\custom_metrics\ --collector.process.processes-where="Name LIKE 'w3wp.exe' OR Name LIKE 'Fortis.%'" --collector.service.services-where="Name LIKE '%BackgroundJobs%'"

I also have healthy node with the same runnning settings, but it has much more metrics.
metrics_healthy.zip
metrics_sick.zip

Thanks @AntonSmolkov! Could you also upload the goroutine dump? You can find it here: <host>:9182/debug/pprof/goroutine
It looks like it could be related to #446

Unfortunately, can't do it yet.
This is production server, so i had to disable vmware collector and restart wmi_exporter yesterday.
After14 hours memory consumption is still ok (30MB).
BTW, i think it worth mentioning that this server has been under high CPU load for a while.
May be it somehow caused timeouts while gatherings data from wmi.


We should strongly consider formalizing the parser/compiler APIs into a system that would be consumable and pluggable at the various levels.
This may take the form of AST/Opcode API formalization or similar. The SpiderMonkey AST model comes to mind.

Released in 3.0.0


Umlaut vowels in the Zotero Short Title

The Zotero Guide tells us that "spaces, hyphens, diacritics and any other non-letter signs should be removed" from author names when creating a ZST. But it turns out that German names with an umlaut vowel (ö, ü, ä) have been treated inconsistently and, mostly, wrongly if we are to heed the letter of the guide. Should an exception be made to the rule that German names including one of these vowels must be represented as oe, ue and ae respectively? I worry that it may not always be straightforward to tell which name counts as German, and that non-European colleagues may have difficulty with this in general. It also bothers me a little that we could do the same with e.g. Hungarian names, where the practice of using vowel combinations was well established in the age of telegrams, although unlike in German, it went out of currency with personal computers, and diacritical marks are generally just dropped when they cannot be written. And Scandinavian?
Anyway, what we now have is quite inconsistent, e.g.

digraph: Buehler, Boehtlingk, Fluegel, Buehnemann, Hinueber, Huesken, KiefferPuelz, Lueders, Mueller (these are the most numerous)
dropped accent: Bohtlingk, Kolver, Luders (these are rare)
actual diaeresis in the ZST: Hinüber, Müller, Gräfe, Härtel, HauserSchäublin (these are definitely not correct by the ZST)

Any change to existing short titles of course requires checking the XML files for references.
Any ideas what to do? Just live with the inconsistency?

If Michaël confirms that having umlaut or other diacritical marks in ZST is not a problem, I guess we could adapt the ZG on this and just use in ZST names with umlaut or other diacritical marks.

Diacritics will not be an issue for the DHARMA app, but I do not know
whether the Zotero app performs the required operations. I will perform a
few tests to confirm that.
Le jeu. 13 juin 2024, 10:27, manufrancis @.***> a
écrit :

If Michaël confirms that having umlaut or other diacritical marks in ZST
is not a problem, I guess we could adapt the ZG on this and just use in ZST
names with umlaut or other diacritical marks.
—
Reply to this email directly, view it on GitHub
https://github.com/erc-dharma/project-documentation/issues/318#issuecomment-2164984472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYU73BZLE6HYS65L2KSCGTZHFJYRAVCNFSM6AAAAABI6KDRF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRUHE4DINBXGI
.
You are receiving this because you were assigned.Message ID:
@.***>

Hang on a moment. Are you suggesting to change the Zotero guide so that names in the ZST must include diacritical marks (and then, I assume, other special characters such as apostrophes) and be identical to the name field? Or that names in the ZST can optionally include diacritical marks?
The former would require that we revise hundreds of existing Zotero entries and any already existing references to them throughout the corpus. The latter would make our Zotero short titles inconsistent, making it more difficult to keep track of number suffixes and generally increasing the chance of human error. Neither would bring us any noticeable gain, and importantly, neither would solve the problem that at the moment, existing ZSTs such as Buehler contradict the ZG.

@danbalogh
Your are right. This was an intrepid suggestion, without noticeable gain.
I can just live with the inconsistency.

I am, like Dan, quite averse to the idea of allowing freedom on this point and am inclined to suggest we must go through all offending ZSTs and make them compliant with the ZG, rather than modify the ZG.
Looking at the names represented with digraph, I don't think many are directly relevant to our database. Some of the ZSTs represented with diaeresis may have been  used in our XML files but I don't expect they'll be very numerous.
If @michaelnmmeyer can list all offending ZST, and perhaps even automate their correction, I'll be happy to work on updating any XML files where wrong ZSTs have been used.

I don't think living with inconsistency is a good idea on this point but I am willing to do a lot of the work, and can find other people willing to help, so Manu doesn't need to lose any time with contributing to the cleaning process.

The problem with automation, or even just listing, is that there are different kinds of "offenders" which have to be dealt with in different ways. The digraphs ae, oe, ue (also with uppercase initial) can occur legitimately or illegitimately, and I don't think it's feasible to write an algorithm that would compare ZSTs containing one of these with the author (etc.) name fields to see if they are also present there. So human attention is needed to check all of these and keep only the illegitimate ones on the list. On the other hand, ä, ö, ü (and any other characters other than [a-zA-Z0-9_\+]) are illegitimate. The first three could be blindly corrected to a, o and u respectively, but the number block must then also be checked, since a ZST might potentially already exist with the umlaut-less vowel and the same number.
Then again, the actual replacement probably could and should be automated, and done at a single point of time in the entire corpus (as well as in the Zotero database), not piecemeal as each of us find time to correct our own files. What we need for that is a list with two columns, the first with existing incorrect ZSTs, and the second with the correct replacement. I guess it should be feasible to write an algorithm that would then go over that list row by row and do the replacements corpus-wide; I don't know if it is also possible to do this on our Zotero database, or if it would have to be done manually there.

I confirm that using diacritical marks in short titles would be OK (the Zotero app performs the necessary transformations).
Before editing existing entries: are people using short titles in other contexts than within DHARMA editions? I can guarantee referential integrity in this context (though this is not done for now), but if people are using short titles as references somewhere else, modifying them might break their work.

I repeat: we do not want diacritical marks in ZSTs.
We are talking about changing the existing ZSTs involving diacritics or digraphs to plain vowels, to conform to the ZG.
I am not aware of ZSTs being used anywhere other than in the XML editions.

We need a decision (and, I think, explicit confirmation of it from the PIs) here.
A. Have we agreed to stick to what the ZG says and record Short Titles with the diacritics simply removed, as in  Coedes for “Cœdès” (thus, also Buhler for "Bühler")? If yes, we'll need to start listing and correcting the existing entries that don't conform to this, as I outlined above.
If we don't want to do that, then we need a different decision - what shall that be?
B. Require oe, ue and ae for German Umlaut vowels and stick to just removing the diacritic for everything else? (Then we still need to list all existing ZSTs containing a character other than [a-zA-Z0-9_\+]) and check and correct manually.)
C. Keep the inconsistency?

For me it's definitely A. So indeed Cœdès becomes Coedes and Bühler becomes Buhler.
I am regularly merging items with two different ZST's for the same bibliographic entry, one of teh ZSTs not compliant, or correcting single entries with wrong ZSTs.
Please implement the necessary clarification and add any useful examples in ZG.
Can the listing be automated?

Annette has confirmed that she agrees with A.

I've added the clarification to the ZG as a suggestion.
On automation, see my suggestions above. We'll need @michaelnmmeyer to tell us if this is feasible.

I have just (manually) weeded out all cases of Hinueber and Hinüber in ZST fields. It would be nice of @michaelnmmeyer could confirm that no such culprits remain and that there are no cases of <bibl> in our xml files where ZSTs now need to be adjusted.
It is a somewhat tedious process to carry out manually if there are lots of items concerned, because ever time an offending tag has been removed, the search window automatically empties itself so that I had to type Hinueber again. And this I had to repeated about 50 times to catch all offending ZSTs and derivative tags.


🚧Update issue and PR templates

I have done the items below . . .

[X] I have updated the title without removing the 🚧 emoji.

Description
Update the issue and PR templates in the project
- | Bug Fixes           | https://github.com/KinsonDigital/Velaptor/labels/%F0%9F%90%9Bbug                     |
+ | Bug Fixes           | https://github.com/KinsonDigital/[TODO: add project name here]/labels/%F0%9F%90%9Bbug                     |

Acceptance Criteria
This issue is finished when:

[x] Issue templates updated
[x] PR templates updated

ToDo Items

[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[X] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[X] Issue linked to the correct project.
[X] Issue linked to the correct milestone.
[x] Draft pull request created and linked to this issue.

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
https://github.com/KinsonDigital/[TODO: add project name here]/labels/%F0%9F%90%9Bbug

Breaking Changes
https://github.com/KinsonDigital/[TODO: add project name here]/labels/%F0%9F%A7%A8breaking%20changes

Enhancement
https://github.com/KinsonDigital/[TODO: add project name here]/labels/enhancement

Workflow Changes
https://github.com/KinsonDigital/[TODO: add project name here]/labels/workflow

Code Doc Changes
https://github.com/KinsonDigital/[TODO: add project name here]/labels/%F0%9F%93%91documentation%2Fcode

Product Doc Changes
https://github.com/KinsonDigital/[TODO: add project name here]/labels/%F0%9F%93%9Ddocumentation%2Fproduct

Priority Type Labels

Priority Type
Label

Low Priority
https://github.com/KinsonDigital/[TODO: add project name here]/labels/low%20priority

Medium Priority
https://github.com/KinsonDigital/[TODO: add project name here]/labels/medium%20priority

High Priority
https://github.com/KinsonDigital/[TODO: add project name here]/labels/high%20priority

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Issue reopened.
Reason: It was discovered that this issue wasn't fully implemented.  Team decided to open this issue and create a new PR to finish the work.


fix invert on source and destination address

this should fix bug: https://github.com/crayfishx/puppet-firewalld/issues/10

@koweblomke  Can you explain a bit about what you are doing with the "NOT" notation... The documentation describes the rich rule structure as follows;
rule [family="rule family"]
    [ source address="address" [invert="True"] ]
    [ destination address="address" [invert="True"] ]
    [ element ]
    [ log [prefix="prefix text"] [level="log level"] [limit value="rate/duration"] ]
    [ audit ]
    [ action ]

I can't find the reference to using "NOT" in the address.....

I'm not shure about the documentation about the invert but I did some debugging and discovered puppet executed:
"/bin/firewall-cmd --permanent --zone=some_zone --list-rich-rules""
The result of that command shows that invert is not an attribute with value "True" but it returnes as NOT in between source and address or destination and address.
In the /etc/firewalld/zones/some_zone,xml the rich-rule does have a source or destination with an invert attribute set to "True"

@koweblomke I've done some digging on this, and found this:
https://github.com/t-woerner/firewalld/commit/7d3ae9f6881c5eb251f4f21da35b4cbe35527c51
My understanding is that 'invert=true' used to be the way to set this but they are moving towards "not address" (athough the API still accepts invert=true).  Given the "not" notation seems to be the way the firewalld project want to do it going forward it makes sense for this module to support it.
Judging by the comments in that commit, it's likely that in future versions may start returning "not" in lower case, rather than NOT, and in it's current form that would break the module - I'll try and get some clarification on that from someone on the firewalld project.  In the mean time, I'll merge this fix.... thanks.


Traceback Error

Traceback (most recent call last):
File "/home/alao/TikTok-Mass-Reporter/main.py", line 1, in
import tls_client, httpx, requests
File "/home/alao/TikTok-Mass-Reporter/venv/lib/python3.12/site-packages/tls_client/init.py", line 15, in
from .sessions import Session
File "/home/alao/TikTok-Mass-Reporter/venv/lib/python3.12/site-packages/tls_client/sessions.py", line 5, in
from .settings import ClientIdentifiers
File "/home/alao/TikTok-Mass-Reporter/venv/lib/python3.12/site-packages/tls_client/settings.py", line 1, in
from typing_extensions import Literal, TypeAlias
ModuleNotFoundError: No module named 'typing_extensions'
Appears to be an issue with the modules

pip install typing_extensions


Invitation

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Tests pass?
yes

License
MIT

This PR adds the possibility to register, and to send an invitation by email to someone who is not a member.
Register is public only if the "sign up enable" option is checked by the admin.
Anyway you can always register if you recieved an invitation.

Thanks @landru21


⚠ Bofrost.be has degraded performance

In ac09a46, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1650 ms

Resolved: Bofrost.be performance has improved in 04bf949 after 7 minutes.