NullPointerException from ObjectExtractor.fillPath

getJavaColor() can return null, so the dead assignment at https://github.com/tabulapdf/tabula-java/blob/master/src/main/java/technology/tabula/ObjectExtractor.java#L283 can cause an NPE. I think this should just be commented out or deleted entirely.

Good catch, thanks.


InstallChartStore

Closed by #4653 which was merged successfully but due some glitch status didn't update.


MobilenetV2 bringup in TTNN

MobilenetV2 model running on device implemented in TTNN

mobilenetv2_pytorch_ref.gv.pdf

Since TTNN doesn't support conv with groups>1, I am facing error for mobilenetv2 ttnn implementation
AssertionError: Only convs with groups == 1 supported

I tried the workaround done in 6580, but it won't work here due to the following reasons:

We can't multiply groups with input_channels, because weights dimension depends on input channel and group, which will cause weights dimension mismatch.

In the PR  (6580) the shape of the input buffers is altered as
input shape = input_batch*input_channel*groups*input_height*input_width, but we can't alter the size of the previous layer's output.

@keerthana-r-mcw can we try to fall back the convs with group>1 to host for now? and then create a github issue with unit tests and share the issue link with me. cc: @mbahnasTT and @saichandax

Hi,
I am attaching the link for the unit test of the conv case throwing the error.
Add Unit tests for MobilenetV2 ops
To run the case with issue use the following command
pytest tests/ttnn/unit_tests/operations/test_conv2d.py::test_mobilenetv2_conv_ttnn_issue_case

WIP PR: 7485


Enable nothunk tests by setting check-tvar-invariant flag for strict-stm

Description

Add a flag checkTVarInvariant to ./nix/ouroboros-network.nix
haskellPackagesWithTVarCheck points to strict-stm package with "checktvarinvariant" flag enabled.
nix-build -A tvar-invariant-checks.tests runs the tests with "checktvarinvariant" flag turned on
Github action "Strict NoThunk Test" should run the tests with above flag turned on

Closes input-output-hk/ouroboros-consensus#604
Checklist

Branch

[ ] Commit sequence broadly makes sense
[ ] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] If this branch changes Consensus and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[ ] Self-reviewed the diff
[ ] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[ ] Reviewer requested

Routine cleanup of stale PRs targeting Consensus components (which nowadays live in https://github.com/IntersectMBO/ouroboros-consensus).
No work is lost, see https://stackoverflow.com/a/17954767.


Support for linux/arm64/v8 devices such as (m1 mac)

Feature Description
Hey, please provide support for the arm64 devices, as after the recent changes, the new image with latest tag doesn't support m1 or m2 chips as it was doing in the previous release.
Suggested Solution (optional)
No response
Already existing or connected issues / PRs (optional)
No response

Hi @anuragjanghala,
I assume you were the one opening the forum question here, right?
There's actually a bit of misunderstanding: the operator has NOT supported arm64 at all, in any version. So the issue you encountered has a different cause.
The v0.0.9rc2 build has broken Docker manifests, due to an issue in underlying CI dependency. So the problem you encountered with v0.0.9rc2 is not related to ARM architecture. Please try to pull these images:
docker pull ghcr.io/grafana/operator:controller-v0.0.9rc2-buildx-fix
docker pull ghcr.io/grafana/operator:latest

These should have fixed manifests.
OTOH, I find it quite curious that previous versions of the operator have worked for you and other ARM users (since nobody had expressed an issue with it before). This implies that ARM-based devices that people use can actually run existing images out-of-the-box. You can check what image you've de facto pulled by running this command:
docker inspect ghcr.io/grafana/operator:latest | grep -A1 'Architecture'

I suspect that you'll see Linux amd64 in output, and your Docker setup must be doing some hidden magic with architectures. I'd appreciate feedback about that :slightly_smiling_face: It'd be good to understand better what exactly happens on ARM devices.

I have a related issue i.e. I tried to install the operator from the bundle:
https://raw.githubusercontent.com/grafana/k6-operator/main/bundle.yaml
but it fails to start on an arm64-node with following error:
k6-operator-controller-manager-5ddf587fdb-zk24j kube-rbac-proxy k6-operator-controller-manager-5ddf587fdb-zk24j manager [k6-operator-controller-manager-5ddf587fdb-zk24j manager] exec /manager: exec format error
The image, which is being used is this one:
ghcr.io/grafana/k6-operator:controller-v0.0.11rc2
If I pin the workload to run on an amd64-node it works without issues.

@pasi-romo-idealo, thanks for feedback! Can you clarify which arm64-node? I.e. is it a cloud cluster, etc.

Hi, yes we use AWS EC2-instances (Graviton) with arm-architecture.

Ciao
Hope this help ... i had the same issue on oke cluster based on arm
so, i've downloaded the repository and changed a couple of stuff on the dockerfiles
docker buildx build --platform linux/arm64 . -t ... Dockerfile.controller
with this change on the Dockerfile
# Build
RUN CGO_ENABLED=0 GOOS=linux GOARCH=arm64 GO111MODULE=on go build -a -o manager main.go

docker buildx build --platform linux/arm64 . -t Dockerfile.runner
docker buildx build --platform linux/arm64 . -t Dockerfile.starter
with this change on Dockerfile
FROM curlimages/curl:8.4.0
all the images are presente here
lgirardi/k6-operator-arm:0.0.11a
lgirardi/k6-runner-arm:0.0.11a
lgirardi/k6-starter-arm:0.0.11a
the bundle.yaml is using the operator
image: lgirardi/k6-operator-arm:0.0.11a
  name: manager
  ...

the K6 is the following
kind: K6
metadata:
  name: k6-www-k8s-it
  namespace: k6-jobs
spec:
  parallelism: 2
  script:
    configMap:
      name: cfm-test-k8s-it
      file: test-k8s-it.js
  arguments: --out cloud
  runner:
    image: lgirardi/k6-runner-arm:0.0.11a
  starter:
    image: lgirardi/k6-starter-arm:0.0.11a

so after that everything is working as expected
k6-jobs              k6-www-k8s-it-2-gz9sl                                        0/1     Completed   0             12m
k6-jobs              k6-www-k8s-it-initializer-n4k2g                              0/1     Completed   0             12m
k6-jobs              k6-www-k8s-it-starter-5bvf6                                  0/1     Completed   0             12m
k6-operator-system   k6-operator-controller-manager-7cfd6c7777-pbbrv              2/2     Running     0             47m```

Grazie mille :blush:
For now I have been using just the amd64-based image since our clusters supports also such nodes. I will definitely give this a go, since because of this other issue:
https://github.com/grafana/k6-operator/issues/171
I anyway need to create a custom image for me for now.
For this specific issue the optimal solution would of course be if the official images were already built as multi-arch i.e. we would not need to build our own images.

@lorenzogirardi please feel free to open PR with the addition of ARM! :smile:
I agree that multi-arch images would be great if they work. AFAIK, k6 has added ARM only very recently (probably this PR) and there were some issues the team had to face & solve there.

I love arm64

With the help of @audacioustux, we have a first attempt at ARM merged :tada:
The ARM image has been built and is available under tag controller-arm-test:
https://github.com/grafana/k6-operator/pkgs/container/k6-operator/198006404?tag=controller-arm-test
Similarly with runner-arm-test and starter-arm-test tags.
Please try it out on your ARM setups! If there are any issues, we should resolve them ASAP. (I'll wait a bit for feedback before closing this issue.)

I tried the controller-image and got it running succesfully on an ARM64-node :tada: Due to this other open issue (https://github.com/grafana/k6-operator/issues/171) I was not able to test the whole functionality.

is this going to be available on dockerhub?

Closing this issue now as as no trouble was reported :slightly_smiling_face:
@Israphel, currently it's available as latest in ghcr.io/grafana/k6-operator. No, Docker Hub is not planned at this point of time. ARM will be part of v0.0.15 release: coming up soon, hopefully.


npm install -g phonegap@latest never ending download

Why it's keep downloading?  it's already exceed 150MB! meanwhile the source is only 100KB.
Then I stopped it before it draining my internet's kuota.
Please help.
I'm Using windows 8.1 Pro
64 bit,
this is the first nodeJS that I install (never install other from npm).

Additional info:

Hi @shofwan,
It looks like you are running everything correctly. npm install -g phonegap@latest will also install all of PhoneGap's npm dependencies (declared in package.json). This is typically ~100MB, so it's unfortunate about your bandwidth but there is no way around it.
The latest version of npm<EMAIL_ADDRESS>tries to reuse npm modules shared between packages. This could reduce the amount that you need to download.
@devgeeks mentioned to me that<EMAIL_ADDRESS>installs uses 112MB to install PhoneGap, while<EMAIL_ADDRESS>uses 95MB.

Is the latest version can be found here? latest.
according to that it's<EMAIL_ADDRESS>right?

Okay actually my npm is still 2.15.5.
Where can I found<EMAIL_ADDRESS>?

Ok I found it. Thank you.

The problem persists.
Phonegap installs with no ends. Then I try to copy phonegap files manually from my friends computer into my node_modules directory. And then this happen:

Could this be the root cause of the problem?
My npm's path: C:\Users\Lenovo\AppData\Roaming\npm\node_modules


[Feature Request] Predefine selected Select-Options

Existing Component
Yes
Component Name
Select
Description
Using the select-option, i want to predefine options, that are already selected.

Nevermind, that's what the v-model is for... sorry...


⚠ Bofrost.de has degraded performance

In f490acb, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1283 ms

Resolved: Bofrost.de performance has improved in ea05496 after 6 minutes.


.Net project appears incompletely committed (and incomplete!)

AssemblyInfo.cs appears in the .csproj but is not in the repo.  Is this expected?
Also, there appear to be utilities but the main body of the project is missing.  How's the port going, and do you want a hand?

@Ozzard working on it...
If you can help on python for some underlying issues will be helpful.
Let me know if you can contribute

@Ozzard this is implemented now.


Upgrade to event-listener v3.0.0

Testing to make sure that event-listener v3.0.0 actually works before we release smol-rs/event-listener#58

Now that event-listener v3.0.0 is released, this can now be merged


Clarify the overall structure of a naming rule

If you're not familiar with the overall structure of a rule this page can be a bit confusing. Attempt to illustrate the various parts using the example we already had.

@jasonmalinowski I resolved the merge conflicts in this file. Can you see if the changes still convey what you were trying to convey? I think this is ready to merge.


Formatting a region with a template strips all trailing whitespace from the template string

@zhengbli already has fix for this, reassigning to him


Gobblin Yarn overwrites job start date.

The GobblinHelixJobLauncher.waitForJobCompletion() method overwrite the job start time.  This results in all instances of the job being recorded to the history database with the same start time.

Closing as this was fixed a while ago


ANSI color codes working inconsistently/incorrectly in terminal

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.69.2
OS Version: macOS 12.1

Steps to Reproduce:

Clone & install this Node.js repo
Run npm run start in both VSCode and your OS's built-in terminal

This is a screenshot from VSCode:

This is a screenshot from iTerm2:

As you can see, VSCode isn't rendering the text as blue when it is on the red background (or any other color for that matter), but it is rendering the text as blue when it is on the black background. In iTerm2 and Terminal, everything renders as expected.
This is the case across all themes, from what I can tell.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.69.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Sorry, that was a typo — I've updated with the correct version. Issue still persists.

This is very interesting — when printing ANSI codes raw in the terminal, some of the text color is poking through but it is very muted (as if the background color is actually overlaying the text or the text is transparent).

Using a color picker in the center of one of the Xs on each line, this is what the actual  colors are rendering as:

MAGENTA TEXT -> rgb(212, 113, 217)
MAGENTA TEXT / GREEN BG -> rgb(79, 41, 81)
GREEN BG -> rgb(6, 63, 41)

/duplicate #146168

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Screen display updates only a partial region matching smaller secondary output

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit 3c964a9fdc220250a85b1c498e5b6fad9390272f dirty (keybinds: Add dispatcher for xmonad/qtile-style workspace switching (4439)).
Date: Tue Jan 16 02:30:46 2024
Tag: v0.34.0-48-g3c964a9f

flags: (if any)

System Information:
System name: Linux
Node name: milkbar
Release: 6.7.0-arch3-1
Version: #1 SMP PREEMPT_DYNAMIC Sat, 13 Jan 2024 14:37:14 +0000

GPU information:
0b:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 10 [Radeon RX 5600 OEM/5600 XT / 5700/5700 XT] [1002:731f] (rev c1) (prog-if 00 [VGA controller])
0c:00.0 VGA compatible controller [0300]: NVIDIA Corporation TU104 [GeForce RTX 2070 SUPER] [10de:1e84] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://bugs.archlinux.org/"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Bug
Description
I currently have two outputs, one 1920x1080 and the other 1440x900. Occasionally, seemingly randomly because I haven't worked out the steps to reproduce it yet, the 1920x1080 output (the primary one) will begin updating only a seemingly 1440x900 region of the screen starting from the top left corner. A bar along the right and bottom of the screen will become 'stuck'.
Fortunately I have discovered that simply opening and closing a terminal on the output will resolve the issue (I assume it forces a redraw of some kind).
The issue has been present for several weeks now (I build master daily), I'd attempt a bisection but without being able to reproduce it, and with it being as rare as it is, I am unable to at this time. Hopefully someone else has more information.
How to reproduce
No known way to reproduce at this time.
Crash reports, logs, images, videos
No response

System/Version info
```
Hyprland, built from branch  at commit 8d31c84483be7cc6f909d73c6ecd4c0b4bdc3360  ().
Date: 2024-01-10
Tag:
flags: (if any)
System Information:
System name: Linux
Node name: noire
Release: 6.1.71
Version: #1-NixOS SMP PREEMPT_DYNAMIC Fri Jan  5 14:18:41 UTC 2024
GPU information:
26:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Ellesmere [Radeon RX 470/480/570/570X/580/580X/590] [1002:67df] (rev e7) (prog-if 00 [VGA controller])
os-release: BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"
BUILD_ID="24.05.20240108.317484b"
DOCUMENTATION_URL="https://nixos.org/learn.html"
HOME_URL="https://nixos.org/"
ID=nixos
LOGO="nix-snowflake"
NAME=NixOS
PRETTY_NAME="NixOS 24.05 (Uakari)"
SUPPORT_URL="https://nixos.org/community.html"
VERSION="24.05 (Uakari)"
VERSION_CODENAME=uakari
VERSION_ID="24.05"
plugins:
</summary>

I have encountered the same situation as well, but in my case it is a 3840×2160 and a 1920x1080.
I do not have exact description or repro steps either, but the terminal opening and closing fixes it for me as well.

It just happened while I was writing this and doing a flake build of a debug build of hyprland.
I was fiddling with kitty and VSCode in a window group, but now I cannot replicate it

Update, thought this might have been fixed because I hadn't seen it in weeks, but it just happened to me again on 3d9ca63

ya i get this every now and again as well. Sometimes if i mess around with the windows it comes back but not always

guys I have monitors with different resolutions, this doesn't happen to me.
Can one of you check with the latest hyprland-git version??

I've got it with a very recent build of hyprland, my 2560x1440 monitor does this with my 1920x1080.

Opening and closing a window (I used kitty) fixed it for me (at least for now).


[LIBCLOUD-692] Run tests in a docker container

Corresponding issue:
https://issues.apache.org/jira/browse/LIBCLOUD-692

Thanks - the change looks good to me.
Can you please also add some documentation on how to use docker to run the tests to docs/testing.rst.

Added some docs (ccc53c5202f5aae7fd0e5ad3bf64d26ed3a4681a) and merged changes into trunk.
Thanks.


[C++][Compute] Add ExecNode(s) for vectorized take with computed indices

Describe the enhancement requested
I don't think the following is possible with any combination of the current set of ExecNodes and I'm curious (1) if there's interest in such functionality and (2) how it could be done. Something very close can currently be constructed using an AggregateNode with a UDF but even with that I haven't found a way to express this type of computation generally.
Assuming you have a table like this a variable number of attributes per combination of date and company. Note that a different company has the highest value of attribteA for each date:

date
company
attributeA
...
attribute N

1
A
3
...
...

2
B
2
...
...

3
C
1
...
...

1
A
2
...
...

2
B
3
...
...

3
C
1
...
...

1
A
1
...
...

2
B
2
...
...

3
C
3
...
...

and you wanted to be able create a plan to group by date and filter the company or companies according to some arbitrary computation on values in any column or columns in attributeA...attributeN and you want everything as a list. For example, imagine attributeA above is employee head count and you want to find the companies top 1/3 percentile in terms of head count by day.
You'd want this table as a result:

date
company
attributeA
...
attribute N

1
[A]
[3]
[...]
[...]

2
[B]
[3]
[...]
[...]

3
[C]
[3]
[...]
[...]

A key part of the result is that company and all attribute columns would be lists that contain arrays (1) all of the same length and (2) the arrays can be length 0 to G where G is the size of each date grouping.
Component(s)
C++

For me there is a bit of disconnect between the issue title and description. The title mentions a "take" kernel, but "take" is an existing vector kernel (to take values from a table/array based on pre-computed indices). I don't you can use the "take" kernel at the moment in an Aceor plan context (there is a FilterNode that does a take under the hood, but that's selecting rows based on a predicate expression, not based on pre-computed indices). But I assume this is not the kind of "take" you mean here?
My understanding is that you rather want some scalar take-like kernel that works on a list of elements and takes certain values from that list? So essentially like the existing "list_element(lists, index)" kernel, but then allowing multiple indices instead of only a single index?
The "list_element" almost works in the case you only want to select one element of the list:
import pyarrow as pa
from pyarrow.acero import Declaration, TableSourceNodeOptions, ProjectNodeOptions
import pyarrow.compute as pc

table = pa.table({'a': [[1, 2, 3], [1, 2, 3], [1, 2, 3]], 'b': [0, 2, 1]})

decl = Declaration.from_sequence([
    Declaration("table_source", TableSourceNodeOptions(table)),
    Declaration("project", ProjectNodeOptions([pc.list_element(pc.field("a"), pc.field("b"))]))
])
decl.to_table()

The only reason this doesn't work is because the "list_element" kernel isn't yet implemented for the variant where the second argument (the index) is an array instead of a scalar (i.e. constant index for all rows).
You could have something similar for a potential "list_take" kernel.
This does assume you first have computed those lists and indices. So that means that you always first materialize the full list in a groupby step, and only in a second step then take a subset of the list. I am not sure there would be a way to integrate that in a single groupby call (without using a custom UDF for the groupby aggregation).

Thanks @jorisvandenbossche for diving into this.

My understanding is that you rather want some scalar take-like kernel that works on a list of elements and takes certain values from that list?

Yes. And the indices would be generated at execution time and would act as a sort of mask.
I attempted to do something similar to what I want with DuckDB almost got there except everything should lists.

Setup
create table ex (date INTEGER, symbol VARCHAR, value INTEGER);
insert into ex values (1, 'a', 3);
insert into ex values (1, 'b', 2);
insert into ex values (1, 'c', 1);
insert into ex values (2, 'a', 2);
insert into ex values (2, 'b', 3);
insert into ex values (2, 'c', 1);
insert into ex values (3, 'a', 1);
insert into ex values (3, 'b', 2);
insert into ex values (3, 'c', 3);

D select * from ex;
┌───────┬─────────┬───────┐
│ date  │ symbol  │ value │
│ int32 │ varchar │ int32 │
├───────┼─────────┼───────┤
│     1 │ a       │     3 │
│     1 │ b       │     2 │
│     1 │ c       │     1 │
│     2 │ a       │     2 │
│     2 │ b       │     3 │
│     2 │ c       │     1 │
│     3 │ a       │     1 │
│     3 │ b       │     2 │
│     3 │ c       │     3 │
└───────┴─────────┴───────┘

select
    date,
    list(symbol) as symbol_list,
    list(value) as value_list,
    max(value) as value_max,
    list_position(value_list, value_max) as idx,
    symbol_list[idx] as symbol_out,
    value_list[idx] as value_out
from ex
group by date;

┌───────┬─────────────┬────────────┬───────────┬───────┬────────────┬───────────┐
│ date  │ symbol_list │ value_list │ value_max │  idx  │ symbol_out │ value_out │
│ int32 │  varchar[]  │  int32[]   │   int32   │ int32 │  varchar   │   int32   │
├───────┼─────────────┼────────────┼───────────┼───────┼────────────┼───────────┤
│     1 │ [a, b, c]   │ [3, 2, 1]  │         3 │     1 │ a          │         3 │
│     2 │ [a, b, c]   │ [2, 3, 1]  │         3 │     2 │ b          │         3 │
│     3 │ [a, b, c]   │ [1, 2, 3]  │         3 │     3 │ c          │         3 │
└───────┴─────────────┴────────────┴───────────┴───────┴────────────┴───────────┘


Add collections.Append

For test and docs, make a note about #5188 and types. Support both one and many.

Note to self.  With Go 1.11 it would be cool if this just worked:
{{ $pages := nil }}
{{ range .Pages }}
  {{ if eq .Title "foo" }}
  {{ $pages = $pages | append  . }}
  {{ end }}
{{ end }}
{{ with $pages }}
{{ range .ByDate }}
{{ end }}
{{ end }}

Would this work on any slice or just page collections?

any slice or just page collections?

Any slice.


pylast fails import when using Auto Py To Exe

What did you do?
I'm trying to build an exe of my finished application, that has the scrobbling feature thanks to your library.
I'm using auto-py-to-exe 2.7.6 for this.
What did you expect to happen?
Being able to freeze my script generating a .exe using auto-py-to-exe or pyInstaller
Every other external libraries imported by my main script (more than 8) are included in the final exe. But for some reason the pylast library is the only one not bundled when compiling the exe.
What actually happened?
When runing the generated .exe, the application crash leaving this error:
pkg_resources.DistributionNotFound: The 'pylast' distribution was not found and is required by the application

What versions are you using?

OS: Windows 10
Python:  Python 3.8.2
pylast: 3.3.0

Example code
Even a simple script like this, when compiled by auto-py-to-exe, fails to import the pylast library. And when running the generated .exe the app crash with the above mentionated error.
import pylast
print("DONE!")

I'm not familiar with auto-py-to-exe, and can't think of anything in pylast that is unusual for a Python package.
This might be a question for auto-py-to-exe.
I see at https://github.com/brentvollebregt/auto-py-to-exe says Python 3.8 is not supported, see https://github.com/pyinstaller/pyinstaller/issues/4311.
Do you get the same thing with 3.7?
https://github.com/brentvollebregt/auto-py-to-exe/issues/71 also has a pkg_resources.DistributionNotFound. Do any of the suggestions in https://github.com/brentvollebregt/auto-py-to-exe/issues/71#issuecomment-533736044 help?

Did you read my post on Issues When Using auto-py-to-exe? This is linked at the bottom of the UI after every build and in the README of this repository; I highly recommend you give it a read.
In the post I discuss the error of modules not being found. This is very similar to your issue here so you could try to add scipy as a hidden import.

There's also links to similar issues at pyinstaller that are worth checking out.

Thanks a lot for your response! It definitely put me in the right track and finally managed to make it work.
I was able to do it using just PyInstaller.  And still using Python 3.8
I needed to create this extra file in a hooks directory (at the same level that the main script):
myprogram/hooks/hook-pylast.py
from PyInstaller.utils.hooks import copy_metadata
datas = copy_metadata('pylast')

and then in the spec file (created the first time you run pyinstaller against your main_script_of_my_program.py), add this folder path to the hookspath argument in the Analysis part:
myprogram/main_script_of_my_program.spec
#more stuff at top

a = Analysis(['bandonautica.py'],
             pathex=['C:\\myprogram],
             binaries=[],
             datas=[],
             hiddenimports=[],
             hookspath=["hooks"], # <---------------- ADD PATH TO HOOKS FOLDER HERE!
             runtime_hooks=[],
             excludes=[],
             win_no_prefer_redirects=False,
             win_private_assemblies=False,
             cipher=block_cipher,
             noarchive=False)

#more stuff at bottom

Good to hear, and thank you for posting an update of what worked, it might help someone else!


fix(deps): update terraform ibm to latest for the deployable architecture solution

This PR contains the following updates:

Package
Type
Update
Change

ibm (source)
required_provider
patch
1.71.2 -> 1.71.3

Release Notes

IBM-Cloud/terraform-provider-ibm (ibm)
v1.71.3
Compare Source
Bugfixes

fix: HOT FIX to include properties from issue 5579 (5818)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.23.9 :tada:
The release is available on:

GitHub release
v1.23.9

Your semantic-release bot :package::rocket:


Initial charm

Overview
Create the initial cloudflared charm.
Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

[x] The charm style guide was applied
[x] The contributing guide was applied
[x] The changes are compliant with ISD054 - Managing Charm Complexity
[x] The documentation is generated using src-docs
[x] The documentation for charmhub is updated
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

Could the access of this repo please be updated? is-devops-leadership should be admin and is-charms maintainer


IRGen: Use mangled names to access opaque type associated types in Swift >=5.2.

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - b636a07070101f8bf1cc22734d6a2cd64242ddf7

@swift-ci Please test OS X

Build failed
Swift Test OS X Platform
Git Sha - b636a07070101f8bf1cc22734d6a2cd64242ddf7

@swift-ci Please test OS X


task : add the possibility to log timestamp (start/ end /elapsed) of every keyword

Hi,
is there a possibility to add the (start/ end /elapsed) time of every executed keyword (green/red) in the log.html
even it does not have logging or printing

this example here Should Start With do not have any timestamp (tho it is obvious to tell the time, but with accumulation you can lose track on the time)

What version are you one? I don't think have any special settings or whatever, and I can see Start/End/Elapsed

Yeah, this information is there already.


[Conversation Error] Improving error log on failed conversation grab

Description
Partial remediation to this thread
Risks & deploy
na, front

PMRR


Command to nuke all desktops except for one, and create x amount of desktops after

I work from the office, at home, and sometimes during business travel.  As you're well aware, OSX struggles remembering spaces to a monitor.  This wastes a few minutes each morning as I delete spaces and add new ones and bring the applications back to their rightful desktop spaces.  I thought, how could I automate this (please feel free to suggest better methods if you know)?
Physical setup:
[1 (monitor)] ... [2 (monitor)] ... [3 (laptop)]
Spaces setup (accordingly): Matching HP Monitors
[ Desktop: 1,2,3,4 ] ... [ Desktop: 5,6,7,8 ] ... [ Desktop 9 ]
At work: Matching LG Monitors
[ Desktop: 1,2,3,4 ] ... [ Desktop: 5,6,7,8 ] ... [ Desktop 9 ]
At home it loads up something like this (applications are mostly at the right spots but its moved a screen over):
[ Desktop: 1,2,3,4 ] ... [ Desktop: 9 ] ... [ Desktop 5,6,7,8 ]
So I have to juggle destroying and creating desktops and manually move the apps over every time...
Is there a way to nuke all the spaces through Chunkwm and create 8 fresh ones, and then run commands like the following to put them into their rightful spots?
chunkc tiling::rule --name "Some App Name" --desktop 1
chunkc tiling::rule --name "Some App Name 2" --desktop 2
chunkc tiling::rule --name "Some App Name 3" --desktop 3
...
chunkc tiling::rule --name "Some App Name" --desktop 9
Other:
System Preferences -> Display shows the same monitor defined as the main one at both home and office, same physical layout, same display arrangement.

I think I found a solution to my home and work desktop/monitor mixups as mentioned previously, but time will tell.

Focus each monitor.
Destroy desktops on each monitor (can't destroy the first of each); I blindly destroy an arbitrary 10 desktops in cases where I might have created more, or when desktops have merged between monitors.
Create 3 desktops on each monitor except for last "monitor" (my MacBook Pro).
Move any open apps to the right desktop

Monitor 1 (4 desktops):
[ 1 ] [ 2 ] [ 3 ] [ 4 ]
Monitor 2 (4 desktops):
[ 4 ] [ 5 ] [ 6 ] [ 7 ]
Monitor 3 (1 desktops):
[ 9 ]
chunkc tiling::monitor -f 1
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --create
chunkc tiling::desktop --create
chunkc tiling::desktop --create

chunkc tiling::monitor -f 2
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --create
chunkc tiling::desktop --create
chunkc tiling::desktop --create

chunkc tiling::monitor -f 3
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate
chunkc tiling::desktop --annihilate

chunkc tiling::rule --owner \"Vivaldi\" --desktop 1
chunkc tiling::rule --owner \"Firefox\" --desktop 2
chunkc tiling::rule --owner \"Adobe Illustrator CC 2019\" --desktop 3
chunkc tiling::rule --owner \"Unibox\" --desktop 4
chunkc tiling::rule --name "~/dev/x-project" --desktop 5
chunkc tiling::rule --name "~/dev/x-project-2" --desktop 6
chunkc tiling::rule --owner \"Photoshop\" --desktop 7
chunkc tiling::rule --owner \"Slack\" --desktop 8

chunkc tiling::rule --owner \"Brackets\" --desktop 9
chunkc tiling::rule --owner \"Gas Mask\" --desktop 9
chunkc tiling::rule --owner \"Terminal\" --desktop 9
chunkc tiling::rule --owner \"Grabit\" --desktop 9
chunkc tiling::rule --owner \"Google Chrome\" --desktop 9
chunkc tiling::rule --owner \"Excel\" --desktop 9
chunkc tiling::rule --owner \"Safari\" --desktop 9
chunkc tiling::rule --owner \"OneNote\" --desktop 9
chunkc tiling::rule --owner \"Reminders\" --desktop 9
chunkc tiling::rule --owner \"ClickUp\" --desktop 9

chunkc tiling::rule --owner \"PhpStorm\" --except "~/dev/" --state float &
chunkc tiling::rule --owner 'PhpStorm' --except '\[.+\]' --state float &
chunkc tiling::rule --owner \"Spotify\" --state float
chunkc tiling::rule --owner \"Discord\" --state float
chunkc tiling::rule --owner Finder --name Copy --state float
chunkc tiling::rule --owner \"App Store\" --state float
chunkc tiling::rule --owner \"Activity Monitor\" --state float
chunkc tiling::rule --name \"System Preferences\" --state float
chunkc tiling::rule --name \"Reminders\" --state float

@koekeishiya Would be nice if there was a one liner like --annihilate-all

If your $SHELL is zsh you can just use repeat, which makes this quite easy to implement yourself
repeat $(expr ${$(chunkc tiling::query --desktops-for-monitor $(chunkc tiling::query --monitor id)): -1} - 1) chunkc tiling::desktop --annihilate

With bash you need to adapt this slightly, I don't think there's repeat and I'm not so sure about expr and ${x: -1} working the same way without looking it up. If you're not on the first space of the desktop you might also have to put a sleep 0.2 in there every iteration or something like that.

@dominiklohmann
Thanks for the detailed writeup.  I have this all done through .chunkwmrc.  Is there a way to feed them those commands through there?  Reason I ask is because whenever I switch work environments I run it through this simple BitBar plugin

The .chunkwmrc file is just an executable file for bash, you can do whatever you want in it as long as it's valid bash syntax, including defining functions and using them.


Remove MXNet dependencies from core

Issue #, if available:
Description of changes:
This is #721 after rebase
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

E               RuntimeError: Failed downloading url https://autogluon.s3.amazonaws.com/numcomp/branin_200_data.pickle
@jwmueller This is one of the files you have replaced. Are you sure this went well?

8c8590230e3b:~ matthis$ wget https://autogluon.s3.amazonaws.com/numcomp/branin_200_data.pickle
--2020-10-22 16:00:08--  https://autogluon.s3.amazonaws.com/numcomp/branin_200_data.pickle
Resolving autogluon.s3.amazonaws.com (autogluon.s3.amazonaws.com)... <IP_ADDRESS>
Connecting to autogluon.s3.amazonaws.com (autogluon.s3.amazonaws.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 403 Forbidden
2020-10-22 16:00:10 ERROR 403: Forbidden.
@jwmueller This worked for me yesterday. Maybe some access rights changed?

Job PR-726-3 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-726/3/index.html

8c8590230e3b:~ matthis$ wget https://autogluon.s3.amazonaws.com/numcomp/branin_200_data.pickle
--2020-10-22 16:00:08-- https://autogluon.s3.amazonaws.com/numcomp/branin_200_data.pickle
Resolving autogluon.s3.amazonaws.com (autogluon.s3.amazonaws.com)... <IP_ADDRESS>
Connecting to autogluon.s3.amazonaws.com (autogluon.s3.amazonaws.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 403 Forbidden
2020-10-22 16:00:10 ERROR 403: Forbidden.
@jwmueller This worked for me yesterday. Maybe some access rights changed?

The link works again

Yes there was permissions issue, but it's been fixed now

Job PR-726-6 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-726/6/index.html

Job PR-726-7 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-726/7/index.html

What is missing here, is changes to Jenkinsfile, and also to how pip install etc works, such that mxnet is not required anymore when only core is installed.

What is missing here, is changes to Jenkinsfile, and also to how pip install etc works, such that mxnet is not required anymore when only core is installed.

I think that can be done in a follow-up PR

Job PR-726-8 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-726/8/index.html


view init webpack does not work

view init webpack myproject opens default editor and creates no files
However vue init 'webpack#1.0' myproject works well

it's vue, no view 😄


Problem acessing the /data/populate route

Hi Stenio, I have follow the instructions and successfully installed the app and the server. I can access all the routes except the /data/populate route. When I enter
http://localhost:3001/bon-appetit/api/v1/data/populate
in my browser I get the error:
Error: Not Found     at /home/sfuenmayor/Development/bon-appetit-server/src/app.js:12:15     at Layer.handle [as handle_request] (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/layer.js:95:5)     at trim_prefix (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:317:13)     at /home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:284:7     at Function.process_params (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:335:12)     at next (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:275:10)     at /home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:635:15     at next (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:260:14)     at /home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:635:15     at next (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:260:14)     at Function.handle (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:174:3)     at router (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:47:12)     at Layer.handle [as handle_request] (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/layer.js:95:5)     at trim_prefix (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:317:13)     at /home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:284:7     at Function.process_params (/home/sfuenmayor/Development/bon-appetit-server/node_modules/express/lib/router/index.js:335:12)

Sorry for my lack of knowledge in the subject but I guess that you can't make POST requests by entering the route in a web browser anyway I solved my issue and filled the database by running the following command in a terminal:
curl -X POST http://localhost:3001/bon-appetit/api/v1/data/populate


Creating an archive

Tell me if this module windows::Win32::Storage::Compression allows you to create an archive, or does it act like a compact command for cmd? If possible, maybe you have the opportunity to share an example.

The members of the windows::Win32::Storage::Compression module represent the Compression API. It exposes compression algorithms only. There is no support for packaging compressed data into archives.


[EL] add scene hass turn on

Because there hasn't been any activity on this PR for quite some time now, I've decided to close it for being stale.
Feel free to re-open this PR when you are ready to pick up work on it again 👍
../Frenck


Shift enter to move to next line

I am coming from a ide by intellij and they have a beautiful feature which is whenever you are in middle of a line and you press Shift+Enter you moved to next new line. However in godot editor we move to next line (not new) with the text on the right of the cursor ;
for example :
--->
Godot is a 2D and 3D(imagine my cursor here) game engine.
<----
and when i press Shift+Enter
This is what currently happens
---->
Godot is a 2D and 3D
game engine .
<-----
what i want
----->
Godot is a 2D and 3D game engine .
<----

Can you not just press end and then enter? It's exact same amount of keys being pressed?

yeah of course but just image while programming how many times you press end key and compare it with how many times you press Shift and Enter. i m not saying you are wrong ,you are right but these little things counts sometimes. 😃😃

This is already possible with Ctrl + Enter (and Shift + Ctrl + Enter to add a line above the cursor), like in Visual Studio Code.


Merging changes

Merges all changes made by different parties

The option does not show for me, so I added you as a collaborator on my fork


this request accesses system indices

Hello! I am a happy KNN-plugin user, but wanted to report this warning. A similar message appears on every KNN-search request to ElasticSearch for me. Can it be disabled? I do not think I accessed these system indices myself. Does the plugin access system indices internally?
odfe-node1   | [2021-07-27T00:57:23,558][DEPRECATION][o.e.d.c.m.IndexNameExpressionResolver] [odfe-node1] this request accesses system indices: [.kibana_1, .kibana_92668751_admin_1], but in a future major version, direct access to system indices will be prevented by default

Hi @snthibaud, no k-NN plugin does not access any system indices. Could be part of security plugin running on your cluster?
There is a dedicated GitHub issues about this deprecation warning spamming the logs https://github.com/opensearch-project/security/issues/1307#issuecomment-887791155. Please feel free to comment on the GitHub issue as well

@vamshin Thank you. I am now following that issue!


Opening large folder crashes

Starting after a clean install
$ prj --version
projectable 1.3.0
$ prj
Error: error during app runtime

Caused by:
    0: error starting filesystem refresh watcher
    1: OS file watch limit reached. about ["/home/thomas/.cache/yarn/v6/npm-@babel-runtime-7.17.2-66f68591605e59da47523c631416b18508779941-integrity/node_modules/@babel/runtime/helpers/esm"]

Maybe it is possible to limit folder nesting

Hmm. I believe this will be fixed after 52da9b4 is released! If not, let me know and I can look into the issue further. If you want to see if it works, you can install from HEAD, too

indeed now installed from main and it loads :ok_hand:
Maybe another problem, I was now scrolling through my folder and I have a binary file that is 27GB. Cat is trying to print it but ofcourse crashes :P Maybe there should be a file limit before 'cat'ing the file?

Ah good idea! I could try implementing a file size limit of sorts. I'm not sure what an appropriate size would be, though.

I have a csv file that is 1MB that resolves instantly so that could be a nice limit? projectable is mostly for programming / text files right?

Yeah. One megabyte sounds like a good limit to me, as well!


Best practices for Zoom

Do we need best practices for hosting Zoom meetups?

Got this advice from someone on the WTD Slack:
The WTD ATX meetup group did this:

Noted in the meetup event that meeting info would be sent out on the day of the event.
Closed RSVPs to the meetup 3 hours before start time.
Emailed all attendees via messaging function in meetup with Zoom join info.

This worked pretty well for us!


SVM-Level Account on NetApp 8.3

Hi,
I just tried the latest change which allows to use SVM-Level Accounts. We have a Ontap 8.3 but if i understand the docs correctly this should still work (mostly) !?
NetApp Release 8.3.2P9
Data ONTAP API version: 1.32

If i run this i get the following error from trident_ephemeral Pod:
time="2017-03-29T15:32:47Z" level=error msg="SVM xxxxxxxx has no assigned aggregates." backend= handler=AddBackend
When i enable debugging this is the last response i can see in the output:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE netapp SYSTEM "file:/etc/netapp_gx.dtd">
<netapp xmlns="http://www.netapp.com/filer/admin" version="1.32">
  <results status="passed">
    <attributes-list>
      <vserver-info>
        <allowed-protocols>
          <protocol>nfs</protocol>
        </allowed-protocols>
        <antivirus-on-access-policy>default</antivirus-on-access-policy>
        <comment/>
        <disallowed-protocols>
          <protocol>cifs</protocol>
          <protocol>fcp</protocol>
          <protocol>iscsi</protocol>
          <protocol>ndmp</protocol>
        </disallowed-protocols>
        <is-config-locked-for-changes>false</is-config-locked-for-changes>
        <is-repository-vserver>false</is-repository-vserver>
        <language>en_us.utf_8</language>
        <max-volumes>unlimited</max-volumes>
        <name-mapping-switch>
          <nmswitch>file</nmswitch>
        </name-mapping-switch>
        <name-server-switch>
          <nsswitch>file</nsswitch>
        </name-server-switch>
        <operational-state>running</operational-state>
        <quota-policy>default</quota-policy>
        <root-volume>xxxxxxxx_root</root-volume>
        <root-volume-aggregate>xxxxxxxxx_data_06</root-volume-aggregate>
        <root-volume-security-style>unix</root-volume-security-style>
        <snapshot-policy>SnapShot_DR</snapshot-policy>
        <state>running</state>
        <uuid>xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx</uuid>
        <volume-delete-retention-hours>12</volume-delete-retention-hours>
        <vserver-name>xxxxxxxx</vserver-name>
        <vserver-subtype>default</vserver-subtype>
        <vserver-type>data</vserver-type>
      </vserver-info>
    </attributes-list>
    <num-records>1</num-records>
  </results>
</netapp>

I'm not a Ontap expert so i might be wrong but doesn't this state that there is in fact an aggregate xxxxxxxxx_data_06 assigned to the VFiler?
Is this a bug or do i do something wrong here?
Thanks in advance!
regards
christian

Hi, Christian.  Doesn't look like a bug.  Please use either System Manager or the ONTAP CLI to assign one or more aggregates to the vserver.

Once you assign aggregates to the vserver, you should see the aggr-list attribute in the log output:

Thanks clintonk!


Add option for remote repository URL

Is your feature request related to a problem? Please describe.
When I use conventionnal commit with fix #issue_number, the url generated by git-cliff is pointing git-cliff repository instead of my current remote repository url.
Example :
### Features

- Use latest image orhunp/git-cliff ([#1](https://github.com/orhun/git-cliff/issues/1))

As you can see, the url is pointing to your repo instead of mine.
Describe the solution you'd like
Is it possible to add an option to specify the repo url I want to use ?
I know there is an option in the config file, but I think on option in the command line is better solution.
For example :
  --remote_repo_url <url repo> : the remote url for issue refs.

Describe alternatives you've considered
I use git-cliff inside Gitlab-CI so I use a sed command on the result to replace your repo url with mine.
PS: thanks a lot for your work !

Tried to implement the feature with #121

Oh, and I realized I made a mistake.
I assumed (and I have honestly no clue how I got to that conclusion) that issue short links are #i and pull requests @i.
But now I've realized they are both just #i.
My problem is now, how to decide which reference is an issue and which is a pull request.
That is necessary because the routes differ: ../issues/{i} vs. ../pull/{i}.
One option would be to match against more than just #i, e.g.:

#i for issues
pr #i | pull request #i | Pull Request #i for Pull Requests

Or is there a GitHub endpoint that redirects the i to the correct origin?

Solved my problem, and realized I can use ../issues/{i} for both.
GitHub redirects correctly to issues if #i is an issue and to PRs if it is a PR.

I had a look at this with fresh eyes and this is caused by the default config, as you mentioned:
https://github.com/orhun/git-cliff/blob/9a070b248d4ae0b58c9463f0627c87ca647c3023/config/cliff.toml#L39-L41
So you should be fine when you just change that line to point your own repository.
But still, it's annoying. So I pushed 8f77caf86a1e5dd23eda1b9e9b5a7a6606642b8a to comment this entry out.
So there is no need for overcomplicating this via #121. It is always good to refrain from adding code-hosting related things to git-cliff since it is not specifically designed to work with e.g. GitHub, GitLab, etc. These remote URLs might change any time depending on which platform you are using. git-cliff should only integrate well with git and it should be configurable based on user liking.
Hope this solves it! 🐻

You can now achieve this with the GitHub integration.


feat: Introduce remote scheme

This PR intends to introduce the new remote scheme concept in yazi. After this new feature introduced, users can use yazi to visit and manage files store in remote storage services like s3, gcs, webdav, dropbox and so on.
This PR just a start of this PR and not integrate with file operations yet.

cc @sxyazi, I'm not sure if this PR is the right size for merging. Could you first review the overall design?

Thanks for the PR!
I think we should create a new crate yazi-fs and move yazi-shared/src/fs/ into this new crate - all integration work can be done within this new crate.
As for the scheme configuration, I don't think it's reasonable to keep it in yazi.toml because people often host their ~/.config directory on Git services, which could lead to sensitive information (like usernames and passwords) being leaked. I think the state directory (~/.local/state) would be a better location for it.

I'm not sure if this PR is the right size for merging

Typically, I expect a complete feature to be merged. Perhaps we can create a separate branch for ongoing work, or you could just give me write access to your fork so I can participate in the development?

Perhaps we can create a separate branch for ongoing work, or you could just give me write access to your fork so I can participate in the development?

Yes, I have enabled Allow edits and access to secrets by maintainers. You can push to this branch directly.

I think we should create a new crate yazi-fs and move yazi-shared/src/fs/ into this new crate - all integration work can be done within this new crate.

Nice idea, I will do this.

As for the scheme configuration, I don't think it's reasonable to keep it in yazi.toml because people often host their ~/.config directory on Git services, which could lead to sensitive information (like usernames and passwords) being leaked. I think the state directory (~/.local/state) would be a better location for it.

DONE, please take a look.

I think we should create a new crate yazi-fs and move yazi-shared/src/fs/ into this new crate

I tried but failed. yazi-shared and yazi-fs will depend on each other.

DONE, please take a look.

Thanks, the config for remote services should be lazily initialized on use – I'll refactor it.

I tried but failed. yazi-shared and yazi-fs will depend on each other.

Why does yazi-shared need to depend on yazi-fs?

I'm trying to make yazi mys3://path/to/file works but find that there are many places need to hack with:

Yeah currently Yazi's filesystem isn't abstracted and is full of local fs API calls. I think the first step should be to abstract the common APIs into the yazi-fs crate and implement it in a cross-scheme way, so it can be used everywhere uniformly.


Master

reformats and fixes a typo in unregistered contributor emails

Repackaged as a cleaner PR for ease of merging.


Bug: onlyChanged is currently true for INSERT

Describe the bug
CI4 doesn't respect the default values of entity attributes on INSERT. Currently only the values that have changed are passed to the INSERT query. This means that if the attributes default value is 500, and I try to insert an entity after setting the attributes value to 500, no value is written to the database at all (the default value from the table would be used, as CI would not include the column in the INSERT).
So the following statement is not currently accurate:

CodeIgniter 4 version
v4.1.1
Expected behavior, and steps to reproduce if appropriate
INSERT of entity should include all values, changed and unchanged. INSERT currently only includes changed values.
To reproduce:

Create an entity class
Define an attribute with a default value:

namespace App\Entities;

use CodeIgniter\Entity;

class BugExample extends Entity
{
    protected $attributes = [
        'the_attribute' => 500,
    ];

    public function setTheAttribute(int $value)
    {
        $this->attributes['the_attribute'] = 0;
        return $this;
    }

    public function getTheAttribute(): int
    {
        return (int) $this->attributes['the_attribute'];
    }
}

Create an entity, set the attrib value to match the default, insert:

$bugExample = new BugExample();
$bugExample->setTheAttribute(500);
$this->insert($bugExample);

The insert query will not include the_attribute

The root cause:

Model insert() calls model->transformDataToArray($data, 'insert');
transformDataToArray() converts the entity object to an array using baseModel->objectToArray($data, true, true);. The 2nd parameter true refers to bool $onlyChanged - which means object to array conversion will only include changed properties.

The solution:

Change $data = $this->objectToArray($data, true, true); to $data = $this->objectToArray($data, ($type == 'update'), true); in BaseModel.php:1690. This will mean that all attributes will be included for INSERT and only changed attributes will be included for UPDATE.
I have tested this, INSERT and UPDATE, and received the expected outcome.

protected function transformDataToArray($data, string $type): array
{
    if (! in_array($type, ['insert', 'update'], true))
    {
        throw new InvalidArgumentException(sprintf('Invalid type "%s" used upon transforming data to array.', $type));
    }

    if (empty($data))
    {
        throw DataException::forEmptyDataset($type);
    }

    // If $data is using a custom class with public or protected
    // properties representing the collection elements, we need to grab
    // them as an array.
    if (is_object($data) && ! $data instanceof stdClass)
    {
        ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
        ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
        // THE LINE BELOW....
            $data = $this->objectToArray($data, true, true);
        // NEEDS TO BE CHANGED TO:
            $data = $this->objectToArray($data, ($type == 'update'), true);
        ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
        ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
    }

    // If it's still a stdClass, go ahead and convert to
    // an array so doProtectFields and other model methods
    // don't have to do special checks.
    if (is_object($data))
    {
        $data = (array) $data;
    }

    // If it's still empty here, means $data is no change or is empty object
    if (empty($data))
    {
        throw DataException::forEmptyDataset($type);
    }

    return $data;
}

Workaround options for those affected:

Set all attribute default values to null and set the default values in the database table, or:
Set all attribute default values to null and create a constructor in the entity class to set your default values:

    public function __construct()
    {
        parent::__construct();
        $this->setTheAttribute(500);
    }

Can you send a PR for this?

I can confirm that this is still and issue with 4.1.3 and it is also in the 4.2 branch.
The bug will also show if you read a row from the database, update a couple of fields and then write back a new row. Only the changed values are written. This is for INSERT only.
I would create a PR, but I don't know how to do it.
`	/**
* Transform data to array
*
* @param array|object|null $data Data
* @param string            $type Type of data (insert|update)
*
* @return array
*
* @throws DataException
* @throws InvalidArgumentException
* @throws ReflectionException
*/
protected function transformDataToArray($data, string $type): array
{
if (!in_array($type, ['insert', 'update'], true))
{
throw new InvalidArgumentException(sprintf('Invalid type "%s" used upon transforming data to array.', $type));
}
	if (empty($data))
	{
		throw DataException::forEmptyDataset($type);
	}

	// If $data is using a custom class with public or protected
	// properties representing the collection elements, we need to grab
	// them as an array.
	if (is_object($data) && !$data instanceof stdClass)
	{
		$data = $this->objectToArray($data, ($type == 'update'), true);
	}

	// If it's still a stdClass, go ahead and convert to
	// an array so doProtectFields and other model methods
	// don't have to do special checks.
	if (is_object($data))
	{
		$data = (array)$data;
	}

	// If it's still empty here, means $data is no change or is empty object
	if (empty($data))
	{
		throw DataException::forEmptyDataset($type);
	}

	return $data;
}

`
Which is the change proposed by @arbitmcdonald, I have tried it and it seems to work correctly. So thanks for that.
Getting this into the next release would be appreciated.

I don't know if this pull request is done correctly as I am not sure of the instructions and I am also not sure if I followed them correctly.
Please let me know if I need to do more to make this work.

Please change your base branch to direct here. In the edit, there's an option to compare between forks. You should select that instead.

Hi @paulbalandan,
I don't really know what I am doing and I don't know how to do what you suggest.
I have not seen an option that obviously says use a fork as a base.
if I am in the right place, the screen shot below shows all the options I have to change the base. The list of branches does not scroll.

If that's the case, then just open a new pull request.

You'll see the compare across forks option at the top. Click that. It will change to this:

For the base repo, chose this repo, and select the 4.2 branch. In the head repo, select your fork and PR branch.

Thanks for your help.
OK, I have created a new PR and I think this is what you need.
As before, if it is not, please let me know and I will try to fix it.

OK, a new PR has been created, fingers crossed this time.

Is that all I need to do and can consider this done?

As i said earlier in your PR, you need to write some tests for that change. Please take a look at tests/system/Models folder.


Eliminate NC_MAX_VARS|DIMS|ATTRS limits

Eliminate all checks for exceeding the NC_MAX_VARS, NC_MAX_DIMS, and NC_MAX_ATTRS defines during the output (write) of a netcdf classic file.   Various options for this have been discussed in #371 and earlier in #148.
Dennis suggested the addition of a new function which would return the number of dims, vars, and attrs; but I think that the capability already exists in the nc_inq_ndims()and similar functions, so I didn't add that function to this PR.
Other details related to this topic are discussed in the referenced issues.  If this PR is seen as too radical, I can modify as needed.

@nmhamster, @wkliao, @roblatham00, @edhartnett, @jedwards4b
Add others who were involved or mentioned in the original issue discussion.

@wkliao The setting of NC_MAX_VARS|DIMS|ATTRS to NC_MAX_INT is an interesting idea, but I think that it might cause more problems than it is intended to solve.  First, if any user code is dimensioning any array to NC_MAX_*, then the code will probably start failing due to memory allocation errors.  Although this is a good way to determine who is doing this, it probably isn't good from a backward-compatible aspect.
I also think that there are other areas in the code that would fail if the dims, vars, attrs count reached or exceeded NC_MAX_INT and setting NC_MAX_* to NC_MAX_INT might provide a false sense of security that as long as these limits don't trigger, then everything is OK.  In other words, if that is implemented, there are likely many other changes required to make these checks exhaustive.  For example, should the checks now be active in non-classic mode?

I note that NC_MAX_VARS is used in the dispatch code:
ed@mikado:~/tmp/netcdf-c/nc_test4$ find ../libdispatch -name '*.c'|xargs grep NC_MAX_VARS
../libdispatch/dvar.c:\returns ::NC_EMAXVARS NC_MAX_VARS exceeded
../libdispatch/dv2i.c:	rs = (size_t*)malloc(sizeof(size_t)*NC_MAX_VARS);
../libdispatch/derror.c:      case NC_EMAXVARS:	 return "NetCDF: NC_MAX_VARS exceeded";

So maybe this should be changed too, since we can no longer count on NC_MAX_VARS to be the limit of the number of vars.

That usage is in the version 2 interface code.  I think it could be eliminated relatively easily since it is only passed down to nc_inq_rec and the space is used to store nrvars which I think is the number of record variables.  It is only used internally, so I think you could query number of nrvars and then allocate that much space instead of just allocating NC_MAX_VARS.

I think this issue is likely to become academic soon. I have a new branch
that is intended to improve the speed of opening a file when processing
its metadata. I believe that it renders NC_MAX_VARS|DIMS|ATTRS moot.
[Actually, I think they already are moot in looking at the code].

They are moot if this pull request is applied ;-)

Not sure it is a good idea to remove error codes even if they are no longer used.
Ward- do you have an opinion on this? Might break user code that tests for this case.

SInce we are moving in this direction, I think we can do this pr.

diff --git a/libdispatch/dv2i.c b/libdispatch/dv2i.c
index 0110d35c..8481674c 100644
--- a/libdispatch/dv2i.c
+++ b/libdispatch/dv2i.c
@@ -1154,7 +1154,8 @@ ncrecinq(
        size_t *rs = NULL;
        int status = NC_NOERR;

-       rs = (size_t*)malloc(sizeof(size_t)*NC_MAX_VARS);
+       nc_inq_rec(ncid, &nrv, NULL, NULL);
+       rs = (size_t*)malloc(sizeof(size_t)*nrv);
        if(rs == NULL)
            return NC_ENOMEM;

Updating with Master then will merge.

Thank you very much!  My life is now complete... ;-)
Now I just have to get everyone to update to 4.5.1...


Enhancement: support authorization via auth proxy

What features would you like to see added?
Option FORWARD_AUTH_HEADER
More details
There are various authorizing reverse proxies. Here are two examples:
https://github.com/oauth2-proxy/oauth2-proxy
https://github.com/Jipok/Jauth
They take care of authentication, authorization, registration, etc. They are easy to support for the developer - all that is needed is to process the configured header(most often this is Remote-User or X-Forwarded-User) where the username is specified. They are also convenient for the user, since there is no need to remember/store extra login/password pairs, provide a single entry point for their own services and increase security.
I'm not good at coding in js, but I was able to implement a simple way that works for me:
Just one change in:
https://github.com/danny-avila/LibreChat/blob/dd8038b375901d52eb2d2db32ad43534909e90ba/api/server/controllers/AuthController.js#L72-L77
const refreshController = async (req, res) => {
  const refreshToken = req.headers.cookie ? cookies.parse(req.headers.cookie).refreshToken : null;

  // Handle Remote-User from auth proxy
  if (!refreshToken && process.env.FORWARD_AUTH_HEADER) {
    let forwardedUserName
    const headerName = process.env.FORWARD_AUTH_HEADER.toLowerCase()
    if (req.headers.hasOwnProperty(headerName)) {
      forwardedUserName = req.headers[headerName];
    } else {
      return res.status(500).send('FORWARD_AUTH_HEADER('+headerName+') not provided');
    }
    // If user doesn't exist, register them
    let user = await User.findOne({ username: forwardedUserName }, '_id').lean();
    if (!user) {
      //determine if this is the first registered user (not counting anonymous_user)
      const isFirstRegisteredUser = (await User.countDocuments({})) === 0;
      const newUser = await new User({
        provider: 'local',
        email: forwardedUserName +<EMAIL_ADDRESS>        password: crypto.randomBytes(Math.ceil(10)).toString('hex'),
        name: forwardedUserName,
        username: forwardedUserName,
        avatar: null,
        role: isFirstRegisteredUser ? 'ADMIN' : 'USER',
      });
      const salt = bcrypt.genSaltSync(10);
      const hash = bcrypt.hashSync(newUser.password, salt);
      newUser.password = hash;
      await newUser.save();
    }
    user = await User.findOne({ username: forwardedUserName }, '_id').lean();
    const token = await setAuthTokens(user._id, res);
    return res.status(200).send({ token, user });
  }

  if (!refreshToken) {
    return res.status(200).send('Refresh token not provided');
  }
...

I understand that this code hardly corresponds to the complex architecture of the project. But it works for me and I hope someone can implement it correctly. The changes that really need to be made are resetting the jwt token if it is expired/incorrect so that the code can issue a new one automatically instead of showing the user the login page.
Documentation example: https://support.getgrist.com/install/forwarded-headers/#forwarded-headers
Which components are impacted by your request?
Endpoints
Pictures
https://github.com/danny-avila/LibreChat/assets/25588359/1fcf11f6-b302-4884-86ed-48efc3c80575
Code of Conduct

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

:+1:
I am using LibreChat behind a Traefik reverse-proxy with Authelia for MFA. It would be great to see support for Auth Response Headers (Remote-User or Remote-Email).

Thanks, with this code I could deploy it behind a caddy server with caddy-security.
Just a note, for the code to work, you need to add the following imports:
const User = require('~/models/User');
const crypto = require('crypto');
const bcrypt = require('bcryptjs');


Ignore close action for archived workspace chats

Details

Ignore the close action that was originally generated for OldDot (more details here), since we're already displaying the footer when a chat report is closed.
Fixed Issues

$ https://github.com/Expensify/App/issues/13839
Tests

Create a Workspace chat (Go to Settings > Workspaces > New workspace)
Delete it
Open the archived workspace chat
Verify the message "You closed this report" doesn't appear (just the footer explaining why the workspace chate is no longer active).

[ ] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Same as Test steps.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Updated and re-tested on all platforms 👍🏽

How can I do that?

Ah go to Settings > Workspaces > New workspace, I updated the instructions.

Is there an issue? I don't see this message.

Oh when I tested I always saw that message, I tested again with a new account and the "no longer active" message is shown:
https://user-images.githubusercontent.com/6829422/210416020-5336d10d-1954-4d11-86e3-f2e09e753dbb.mov

There is another issue where workspace chat is renamed back to default name after deleting it.

Ah this is an interesting bug and I can't reproduce it constantly, seems it happens when the connection is intermittent. Have you reported those issues in #expensify-bugs?

I think we are good to merge this. All the issues that I have reported are unrelated to this PR. Let me know once you have reviewed the issues above.

Thanks, I think we're missing the PR reviewer checklist here.

Came across this bug as we pick N7 back up and I was running end-to-end tests Workspace Chats as we look to get the project over the line.

This workspace chat is very buggy. There are different bugs on different platforms.

I'd love to dig into that more with you. Which platforms? I've run tests on Web & iOS so far and it's working pretty good and consistent. I have a full written script if you're interested in taking a look.

There is another issue where workspace chat is renamed back to default name after deleting it.

What does this mean exactly, what's "default name"? The workspace chat does have a different chat title depending on the viewer of the chat as an FYI:
Admin POV = memberName (or memberPrimaryLogin)
Workspace Chat Owner POV = workspaceName

🚀 Deployed to staging by @mountiny in version: 1.2.55-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @AndrewGable in version: 1.2.55-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fix for Java Todolist sample that didn't work with the latest play version

The todo list java didn't work, the Build.scala should set a defaultJavaSettings and the application.conf should also declare an application.secret.

This sample application is now deleted.


fix: ask user to check wallet

I added this PR to mock-web3-provider to allow us to do integration test of the loading screen https://github.com/rsksmart/mock-web3-provider/pull/15

Now we can do something like
    cy.visit(`/`)
    cy.get('#login').click()
    cy.contains('MetaMask').click()
    cy.get('#loading').should('have.text', 'Please confirm in your wallet')

    provider.answerEnable(true)
    confirmInformationStep()
    // ... and so on

We can also see test what happens if provider.answerEnable(false)

Good idea of that mock-web3-provider PR

Added test but https://github.com/rsksmart/mock-web3-provider still published with old version


Base AJAX Component - Content Is not displaying in Publish

Hi
I have followed the steps and set slin:resourceSuperType and renamed my jsp to ajax.jsp. When I am authoring, the component's values are displaying in the author mode. But the same is not visible in the publish mode
Please let me know if this is a bug or I missed any configuration. Please find Screenshots below
Author view

Publish View

Apps

Thanks
Veena

@veenavikram07 did you include the acs-commons.components clientlib into your site's clientlib? If you dont then the JS doesnt exist to do the AJAXing.

No response. Closing...


Generate a Certifications when all exercises from a module have been approved

Duplicate of https://github.com/FrontEnd-ro/frontend.ro/issues/695. Closing


@conda-forge-admin,please add bot automerge

[x] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/botcity-maestro-sdk-feedstock#5.


Sync: Can't login using Github account

Testing #93771 on Windows
I don't see any option to login with Github

I think you need to hit either "Sign-in options" or "Back" to get to the screen that has this option, which is not intuitive, but we don't control this UI and there isn't a way to land on that screen


ROX-22476: update roxctl netpol connectivity diff with new out format option

ROX-22476: adding dot format to diff
Includes:

updating documentation with the new output option (dot) for this command roxctl netpol connectivity diff
updating bats tests - verify output in this format is generated

This can be closed as the duplicated #10663 was merged.


fix: icon maybe nil on winbar

Icon maybe nil, so it cannot simply concatenate

if config the devicon defaule option it always return an icon. but yes this patch is inneed.


Update Readme.md Rules Section

In this section, add one more rule at the end this way -

If you are creating a new issue, then please also comment on it mentioning that You want to work on it.

@saumyasrivastava08 This is to be assigned to you as compensation for the last issue, Shall I assign it to you?

No it's okay let it be i solved lots of issue
Get Outlook for Androidhttps://aka.ms/ghei36

From: Manvi Tyagi @.>
Sent: Friday, March 26, 2021 12:21:04 PM
To: Girl-Code-It/Opportunity-Calendar-Frontend @.>
Cc: Saumya Srivastava @.>; Mention @.>
Subject: Re: [Girl-Code-It/Opportunity-Calendar-Frontend] Update Readme.md Rules Section (#148)
@saumyasrivastava08https://github.com/saumyasrivastava08 This is to be assigned to you as compensation for the last issue, Shall I assign it to you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Girl-Code-It/Opportunity-Calendar-Frontend/issues/148#issuecomment-807982840, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANXDGXMKWPSFSY3YF7B7BCLTFQVFRANCNFSM4Z2Z3IEA.

I am working on this


suppress numexpr threads messages when importing runhouse

Current dependencies on/for this PR:

main

PR #435   👈

This stack of pull requests is managed by Graphite.


I619 data set fixes and features part2

incl.
any(), all(), set_item_texts(), new getitem/setitem behaviour in the DataSet indexer impl.

This PR resolves #619.


Create Gather_Resources.lua

This combines Grass, Slate, and Clay gathering into a single script.  It will look for the gather icons on either side of the screen and both normal and small icons.

I took a shot in the dark and tried @Cyzygy06 in Discord and it matched up to 'Darkfyre'.
I reached out to that player via Discord to see if they could send those _small images, referenced in the .lua file...

Successful contact with player. They will check on missing images later, when they get home.

Great, thanks!

I have submitted this on behalf of player, with permission.  See https://github.com/DashingStrike/Automato-ATITD/commit/e6eb28947a4c3eeb4228d06b06f49fd3f3f48a85


Simplify Install Instructions

Is there a reason for the manual installation steps in the following docs for installing docker? I am thinking It might be better and easier to simply perform:
curl -sSL https://get.docker.com/ | sh
or for experimental:
curl -fsSL https://experimental.docker.com/ | sh

I don't think we need to add the experimental url, we don't use the experimental features, and as of docker 1.13 they're being moving into the main install under an experimental flag.

Fixed by #507


[VOTE] Balancer BIP 91 - Temporarily Adjust Optimism Fee Split

Snapshot
https://snapshot.org/#/balancer.eth/proposal/0x38bed268fe1536a6c8e9d5d5b3139bfba912b46507e2a72777cac691f58549d0
Option to vote for
Yes, let's do it
Authorization
(https://discord.com/channels/743271185751474307/842854176488161311/1033027082021785660)

Voted Yes on https://snapshot.org/#/balancer.eth/proposal/0x38bed268fe1536a6c8e9d5d5b3139bfba912b46507e2a72777cac691f58549d0


UI : Rename Add Elastic Search Reindex button item name

Describe your changes :

Issue : #8513
Rename Add Elastic Search Reindex Ingestion button  name to Add Elastic Search Reindex

Type of change :

[x] Improvement

Frontend Preview (Screenshots) :

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
@open-metadata/ui

Merge the main here

resolve conflict


:bug: Add missing ARCH variable to the Ubuntu KubeadmControlPlane

What this PR does / why we need it:
ARM support was added with https://github.com/syself/cluster-api-provider-hetzner/pull/766, but the ARCH variable was missing in the KubeadmControlPlane yaml file.
Which issue(s) this PR fixes:
Fixes #865.

Thanks for adding this! Please note that these templates are not production-ready and are for testing purposes only.


Release/0.1.0

Changelog
All notable changes to this project will be documented in this file.
The format is based on Keep a Changelog,
and this project adheres to Semantic Versioning.
[Unreleased]
Added

PODAAC-3504

As a developer, I want to create a Python package capable of merging granules together

PODAAC-3526

As a developer, I want to parallelize the granule merge service to improve performance

PODAAC-3604

As a developer, I want to create a deployment pipeline for the granule concatenation service

PODAAC-3607

As a developer, I want to create a Harmony service lib wrapper for the granule concatenation service

PODAAC-3663

As a developer, I want the subset_files data in my concatenated data to contain original filenames

PODAAC-3860

Created a UMM-S record for Concise
Utilize cmr-umm-updater Github Action for auto-publication of UMM-S changes/version bumps

Changed

Moved to GitHub.com!

Deprecated
Removed
Fixed
Security

Everything seems to be working now, and we have upgraded to the latest release of cmr-umm-updater, so this release is ready to go.


Minor updates to release notes from the od-pks team

Hey @neil-hickey thanks so much for this PR! Great to have more component versions documented. 👍


feat: Re-Add CODEOWNERS

A member of Node Sig will be required to merge a PR
What type of PR is this?

What this PR does / why we need it: Re-adds CODEOWNERS so that a member of Node Sig must approve a PR
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

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 79.036%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2247

Relevant Lines:
2843

💛  - Coveralls


Fix code examples

@sibprogrammer the changes looks good to me, mostly those are formatting change. Additionally if you are looking further for contributing in this repo then you can also update the example and docs according to: https://github.com/growthbook/examples/tree/main/flutter_example.


PHPFlasher in stateless API

Im using PHPFlasher in a symfony application however it adds session state when not in use. I am combining it with API Platform and my API itself it stateless so it should not hold any session data (causes errors).
How should i handle this (if even possible?)

Hi @pimjansen,
Thank you for bringing this issue to my attention. Indeed, it's problematic when PHPFlasher introduces session state in stateless APIs.
As a temporary solution until a suitable configuration option is implemented, consider overriding the flasher.storage service. To do this, replace the current implementation with an instance of ArrayBag.
Here are the relevant links:

flasher.storage service definition
ArrayBag class

This adjustment should prevent PHPFlasher from maintaining a session state when it's not in use.

I wish it was but it hits Flasher\Symfony\Http\Request

Hey @pimjansen 👋,
I have just released a new version, v1.14.0, which defaults to using ArrayBag when sessions are disabled. Could you please upgrade to this version and let me know if it resolves your issue?
If the problem persists, would you be able to share a minimal reproducible example project on GitHub? This would be greatly helpful for me to further investigate and address the issue.

Thanks for the quick responses to start with. Will test it out tomorrow when in office! Will update with the results

@yoeunes still the same error is there. I do not have a public repo for this issue but technically it is just a symfony application with a controller that uses flasher fine. Further it has API platform installed with a stateless API.
See screenshot for the trace

Hey @pimjansen,
I've just released a new version v1.14.1 to fix the error with stateless routes.
Could you please update to this new version and test it in your application? I would appreciate if you could confirm whether this version solves your issue. If you encounter any further problems, don't hesitate to let me know.

Still the same outcome:
Session was used while the request was declared stateless.
The problem is there since you are checking the actual request which has maybe a session active but it doesnt mean the endpoint should actually use session itself. What you can do is check the actual firewall
/** @var FirewallConfig $firewallConfig */
$firewallConfig = $this->firewallMap->getFirewallConfig($request);
if (!$firewallConfig->isStateless()) {

}

You can get there by injecting FirewallMapInterface $firewallMap,

Hi @pimjansen,
Thank you for your suggestion. I tried reproducing the issue with Symfony versions 6.0 and 6.3, but I was unable to encounter the same problem. I inspected the stateless routes carefully and found that they were operating as expected, even when no session was present.
I'm puzzled as to why we are seeing different behaviors. There might be a specific configuration or context in your application that is causing this issue to occur. I'd need more details or ideally a minimal reproduction case that I could run on my own.
Would it be possible for you to provide a sample project or more details about the specific conditions under which the issue occurs? This would greatly help me in identifying the cause of the issue.
Here are the screenshots from my tests:

Confirmed. It happens when you have a session in the application and use the swagger spec (openapi) to make the request. When that happens there a cookie active so the session is sent.
For me this is fine though as is. The changes you did are always good for fail safe use while sessions are not active though.

Hey Pim! 👋
Great solution! 🎉 It's awesome to see that you found this elegant solution without requiring any updates to PHPFlasher. 👍 Thank you so much for sharing it!

Hey Pim! 👋
Great solution! 🎉 It's awesome to see that you found this elegant solution without requiring any updates to PHPFlasher. 👍 Thank you so much for sharing it!

Well it is actually a decoration of the php flasher symfony lib where the session listener is located.
The main thing is when something is invoked on the existing domain it will always reuse the existing sessionid cookie by default even though the endpoint is stateless.
This was also the reason why you could not reproduce it since the endpoint had no stateful auth on itself. So i guess its fair to say if the url is stateless there is never need to set a session since it breaks on next request anyways


[FLINK-16395] [legal] Add OkHttp to NOTICE of statefun-flink-distribution

cc @igalshilman

Thanks! Merging ...


⚠ Bofrost.at has degraded performance

In 36353f8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 879 ms

Resolved: Bofrost.at performance has improved in 3f05fc6 after 46 minutes.


Why still supporting V1?

Hi, I can't understand why you still support API v1. I think that removing it could help the server and cut down the load.
Thanks

@Naramsim if we remove it 70% of our traffic and integrations die :)
Good API services should maintain their services for as long as possible. It's not hurting us to keep it and it's actually not part of the load problem either, that's solely on the V2 part.


"Assembly not supported" for .NET 3.5 test project when extension directory contains .NET 4+ assembly

We are having problem with the test adapter in our TFS build environment. Here are the versions used:

NUnit version 2.6.2
NUnit test adapter version <IP_ADDRESS>
Visual Studio 2012 update 3
TFS 2012 update 1

Here are the steps to reproduce:

Create a .NET 3.5 class library project with at lease one NUNit test. In this example it is called "aaa.bbb.Tests.dll"
Configure your build environment according to this guide: http://www.mytechfinds.com/articles/software-testing/6-test-automation/72-running-nunit-tests-from-team-foundation-server-2012-continuous-integration-build

Basically you copy the test adapter assemblies to a folder reachable by the build controller and edit the "test assembly file specification" in the build definition to match the name of your NUNit test project.

Queue a build, and wait for it to finish.

Result: You should see correct test results in the build report.
However, the folder specified in "version control path to custom assemblies", where the test adapter assemblies are copied, is used for more than just test adapters. For example you place assemblies that contains workflow activities for customized build workflows. And here comes the strange part:

Create a new .NET 4.0 class library project in visual studio. Name it something that comes before "nunit" in the alphabet, for example "abc.dll". In our case it contains workflow activities, but we can reproduce the problem with just the the standard "Class1.cs".
Build the .NET 4.0 project and check in the resulting "abc.dll" to the "version control path to custom assemblies" folder. That is the same folder as the NUNit test adapter assemblies were copied earlier.
Queue another build.

Result: No tests are run and the build report says:
Assembly not supported: C:\Builds\5\aaa\bbb\bin\aaa.bbb.Tests.dll
No test is available in C:\Builds\5\aaa\bbb\bin\aaa.bbb.Tests.dll. Make sure that installed test discoverers & executors, platform & framework version settings are appropriate and try again.
We have found the following ways to make it work again:

Remove the "abc.dll" from the custom assemblies folder.
Rename "abc.dll" to "xyz.dll" (!)
Change the target framework version of "abc.dll" to .NET 3.5.

We can also reproduce the problem locally in Visual Studio, using the same project setup as above. This is a strange thing to do, but much easier for troubleshooting.

Install the latest NUnit test adapter for Visual Studio.
Open the .NET 3.5 class library project containing NUnit tests.
Open the Test Explorer window and build the project. Your tests should appear.
Right click some of the tests and select "Run selected tests".

Result: Your tests should run successfully.
Now look up the Visual Studio extensions folder, where the test adapter is installed. On my computer it is C:\Users\username\AppData\Local\Microsoft\VisualStudio\11.0\Extensions\do0tbn4e.opv.

Copy the "abc.dll" to the test adapter folder you found above.
Restart Visual Studio, open and build the NUnit project

Result: The tests are discovered in the Test Explorer window as before, but when you right click a test and select "Run selected tests" you get the following error message in Output/Tests:
NUnit <IP_ADDRESS> executing tests is started
Assembly not supported: c:\Projects\aaa\bbb\bin\Debug\aaa.bbb.Tests.dll
NUnit <IP_ADDRESS> executing tests is finished
We find it very strange that an unreferenced .NET 4.0 assembly in the same folder as the test adapter can disturb the test execution. And we have really no choice to alter our .NET versions. The NUnit test projects need to be 3.5 for legacy reasons and the workflow activity assemblies needs to be .NET 4.0, or actually .NET 4.5, to reference the TFS 2012 assemblies.
More discussion at https://bugs.launchpad.net/nunit-vs-adapter/+bug/1209192

@OsirisTerje Seems like we would fix this if confirmed. Do you disagree?

Agree


Entity metrics feature date

Hello!  :)
Is there any idea of when the metrics entity feature will be available, as described here? I am very interested in trying it out!
Thank you

@cvrnogueira Thanks for asking! We don't have this on our short-term roadmap (https://datahubproject.io/docs/roadmap) mainly because we didn't find any open source metrics framework / tool that would be good to build an integration with.
Is there a specific tool or framework where you are defining your metrics?

Hello @shirshanka , thank you for your reply! Actually I have not found an open source framework either! Today the metrics I am studying as use cases are mostly documented in static documents and being defined based on the queries that are build to run this metrics. We can chat more about this tomorrow!


Intents. Added address is saved in whitelist although done button wasn't pressed

https://github.com/warden-protocol/wardenprotocol/assets/59832414/008c1456-8223-4e9f-9904-add8e3c4bcfd

not a bug


V13 to main

-urls to env file
-update dependencies
-@ to (at) in TG->Discord bridged messages

Codecov Report

Merging #204 (1db4f1f) into main (b3fcc51) will decrease coverage by 0.15%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main     #204      +/-   ##
==========================================
- Coverage   82.40%   82.25%   -0.16%
==========================================
  Files          56       56
  Lines        1472     1499      +27
  Branches      204      230      +26
==========================================
+ Hits         1213     1233      +20
- Misses        220      227       +7
  Partials       39       39

Impacted Files
Coverage Δ

src/discordBot/services/service.js
58.33% <33.33%> (-0.72%)
:arrow_down:

src/discordBot/commands/student/join.js
90.69% <0.00%> (-1.62%)
:arrow_down:

src/server/routes/discordAuth.js
37.50% <0.00%> (-1.21%)
:arrow_down:

src/discordBot/commands/faculty/edit_course.js
66.21% <0.00%> (-0.91%)
:arrow_down:

src/discordBot/commands/faculty/create_course.js
100.00% <0.00%> (ø)

src/discordBot/commands/student/help.js
91.13% <0.00%> (+0.11%)
:arrow_up:

src/discordBot/commands/faculty/lock_chat.js
96.55% <0.00%> (+0.12%)
:arrow_up:

src/discordBot/commands/faculty/unlock_chat.js
96.42% <0.00%> (+0.13%)
:arrow_up:

src/discordBot/commands/faculty/hide_course.js
96.29% <0.00%> (+0.14%)
:arrow_up:

... and 4 more

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 b3fcc51...1db4f1f. Read the comment docs.


Implement symlink function

The symlink function should take as it's arguments a file path (to a configuration file) and a second file path to the location to create a symlink at.

If empty, create a symlink
If a symlink exists, overwrite and log
If a non-symlink exists, do not overwrite and log failure (should this immediately exit the script?)

Maybe this is better implemented as link_file and link_dir to give users better control of the ln -h option.
This may create confusion when users are linking directories (such as a vim-pathogen config) as this would fail on a second run if they were to use link_file. When merely replacing old symlinks, I don't think there's a danger to using ln -shf instead of ln -sf

While implementing these functions I found it necessary to resolve #26 as well.