AWS EKS x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs

Hi,
I've tried to install Calico on an EKS cluster, but I keep getting x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs error, reason why I am reaching out for help :)
Installation Steps

Using Terraform, I deployed an EKS cluster v1.25 with an empty node-group (desired instances = 0)
Once the cluster deployed, I deleted the VPC cni using kubectl delete ds aws-node -n kube-system
I then applied the Calico manifest kubectl apply -f https://raw.githubusercontent.com/projectcalico/calico/v3.25.1/manifests/calico-vxlan.yaml
And ran the command kubectl -n kube-system set env daemonset/calico-node FELIX_AWSSRCDSTCHECK=Disable
I updated my existing node-group's desired instance to 1.

Issue
The calico-kube-controllers pod remains in "ContainerCreating" state and the kubectl describe pod calico-kube-controllers-xxxxxx -n kube-system returns the following:
Events:
  Type     Reason                  Age                    From               Message
  ----     ------                  ----                   ----               -------
  Normal   Scheduled               38m                    default-scheduler  Successfully assigned kube-system/calico-kube-controllers-xxxxxx-xxxx to ip-193-80-228-16.us-west-1.compute.internal
  Warning  FailedCreatePodSandBox  38m                    kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx": plugin type="calico" failed (add): error getting ClusterInformation: Get "https://<IP_ADDRESS>:443/apis/crd.projectcalico.org/v1/clusterinformations/default": x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs
  Normal   SandboxChanged          3m46s (x162 over 38m)  kubelet            Pod sandbox changed, it will be killed and re-created.

Note
My VPC does not have direct access to the internet. Egress traffic goes through an AWS Transit Gateway which is then routed to a proxy (in case this info helps).
Thanks

Get "https://<IP_ADDRESS>:443/apis/crd.projectcalico.org/v1/clusterinformations/default": x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs

This is suggesting that your API server is presenting Calico with a certificate that doesn't include an IP address subject names in order to verify that Calico is talking to the correct server.

Ok. But how to fix it?

You need to provide a certificate to your API server that includes its various IP addresses / domain names so that clients can verify it correctly.

Closing since it seems the issue is clarified. Feel free to re-open think otherwise.


[ONNX] Header files for ONNX_ML

This https://github.com/NervanaSystems/ngraph/pull/1738 should resolve this.

Closing this as PR#1738 meets my requirement.


z-ordering fix for sidebar list items

Sidebar list items don't set a positive z-index value, but they are designed to overlap the dividing line of the SidebarLayout component. If a user makes the content of a SidebarLayout position: relative, or otherwise elevates its z-ordering, the SidebarListItem will no longer overlap it properly. Setting a positive z-index value maintains the behavior. 100 was chosen arbitrarily... It might be nice to establish a guideline system for z-index values if we deal enough with them in the future.

Thus begins the z-index wars


[Feature Request]: the current version does not support incremental insertion, but it is very necessary！

Do you need to file an issue?

[X] I have searched the existing issues and this feature is not already filed.
[ ] My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
[ ] I believe this is a legitimate feature request, not just a question. If this is a question, please use the Discussions area.

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
supports incremental insert, no duplicated computation or data will be added
Additional context
No response

Please see #741


Travis: update CI matrix

This PR only updates the CI matrix to the latest generally available Ruby versions.
Source for version numbers: https://github.com/rbenv/ruby-build/tree/master/share/ruby-build

Oh, there's some package management tasks to take care of before this can pass.

Fixed in a similar way in #529, thanks! ❤


Merge to live!

Replace references to Outlook Online
Clean up OWA references and transition 'for' to 'on'
Updates based on feedback

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

File
Status
Preview URL
Details

docs/actionable-messages/adaptive-card.md
:white_check_mark:Succeeded

docs/actionable-messages/index.md
:white_check_mark:Succeeded

docs/actionable-messages/message-card-reference.md
:white_check_mark:Succeeded

docs/add-ins/activation-rules.md
:white_check_mark:Succeeded

docs/add-ins/add-and-remove-attachments-to-an-item-in-a-compose-form.md
:white_check_mark:Succeeded

docs/add-ins/add-in-commands-for-outlook.md
:white_check_mark:Succeeded

docs/add-ins/add-in-requirements.md
:white_check_mark:Succeeded

docs/add-ins/add-mobile-support.md
:white_check_mark:Succeeded

docs/add-ins/compare-outlook-add-in-support-in-outlook-for-mac.md
:white_check_mark:Succeeded

docs/add-ins/contextual-outlook-add-ins.md
:white_check_mark:Succeeded

docs/add-ins/index.md
:white_check_mark:Succeeded

docs/add-ins/limits-for-activation-and-javascript-api-for-outlook-add-ins.md
:white_check_mark:Succeeded

docs/add-ins/metadata-for-an-outlook-add-in.md
:white_check_mark:Succeeded

docs/add-ins/outlook-addin-design.md
:white_check_mark:Succeeded

docs/add-ins/outlook-mobile-addins.md
:white_check_mark:Succeeded

docs/add-ins/pinnable-taskpane.md
:white_check_mark:Succeeded

docs/add-ins/sideload-outlook-add-ins-for-testing.md
:white_check_mark:Succeeded

docs/add-ins/testing-and-tips.md
:white_check_mark:Succeeded

docs/add-ins/tips-for-handling-date-values-in-outlook-add-ins.md
:white_check_mark:Succeeded

docs/add-ins/toc.yml
:white_check_mark:Succeeded

docs/add-ins/troubleshoot-outlook-add-in-activation.md
:white_check_mark:Succeeded

docs/breadcrumb/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Turn off server utilities

The event submissions, the mod mail and the bug report should have an option to be turned off. + integrate with API (all the above mentioned improvements comes under the server utils endpoint)

Implemented


Structure of results.pth

Hello!
Very interesting paper, thanks!
I have a question about the ‘’’results.pth’’’. It’s a file with distilled images (tensors), but what about its structure? For example, for MNIST its ‘’’len’’’ 30, so it’s because we have 3 steps? And labels, it’s just tensor from 0 to 9, so the order is important, like the first 3 tensors are class 0, right?
And at the testing phase we use pretrained model which we got after distill basic?
Thanks!

Hi, 30 represents 10 steps repeated for 3 epochs. Each element of the list contains (data, label, lr).
In test phase, the model is randomly initialized / loaded, depending on the setting. distill_basic actually does not train models. It only distills images.

Thanks a lot for your reply!


Modify security group ingress rules to allow access only on specific ports

Description
In order to allow access only on specific ports of the cluster, this change modifies the tcp port access of security groups from alltcp to 80, 443, 9200, 5601 and 8443.
Another change that this PR address is removal of duplicate props securityGroupId. The required securityGroupId can be passed via serverAccessType and restrictServerAccessTo.
This change also fixes modification of the existing SG. The user needs to take care of port availability at their end if using existing SG.
Since this is a breaking change, bumping the version to next major.
Issues Resolved
related https://github.com/opensearch-project/opensearch-devops/issues/129
resolves https://github.com/opensearch-project/opensearch-cluster-cdk/issues/31
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Little confused here.
The purpose of the accepting securityGroupId parameter was that if the user wants to reuse an existing security group they can provide that and the access restrictions on the ec2 hosts will based on that. This security group will the one attached to ec2 instances.
It assumed that the provided SG would reside in the same VPC, but it is good to enforce a failure if only SG-id was provided and not VPC.
Coming to serverAccessType , the purpose of introducing these settings was to add rules to an existing security group. So when I provide serverAccessType=securityGroupId, it meant that I am adding a new rule in an existing security group (attached to ec2 instances) that restricts the access from resources behind the security group provided  in serverAccessType.
I would like to keep them both separated.

Little confused here. The purpose of the accepting securityGroupId parameter was that if the user wants to reuse an existing security group they can provide that and the access restrictions on the ec2 hosts will based on that. This security group will the one attached to ec2 instances. It assumed that the provided SG would reside in the same VPC, but it is good to enforce a failure if only SG-id was provided and not VPC.
Coming to serverAccessType , the purpose of introducing these settings was to add rules to an existing security group. So when I provide serverAccessType=securityGroupId, it meant that I am adding a new rule in an existing security group (attached to ec2 instances) that restricts the access from resources behind the security group provided in serverAccessType.
I would like to keep them both separated.

I see what you mean. I will update this PR to only reflect the specific port access instead of all ports for now. Will create another PR for security Group ID change and throwing an error if vpc id was not provided.


Recent Advances and Challenges in Task-oriented Dialog Systems (Survey Paper)

집현전 중급반 스터디

2022년 5월 15일 일요일 9시
김도은님, 이지현님, 권다영님, 이강희님, 최윤진님 발표
논문 링크: https://arxiv.org/abs/2003.07490

Abstract
Due to the significance and value in human-computer interaction and natural language processing, task-oriented dialog systems are attracting more and more attention in both academic and industrial communities. In this paper, we survey recent advances and challenges in task-oriented dialog systems. We also discuss three critical topics for task-oriented dialog systems: (1) improving data efficiency to facilitate dialog modeling in low-resource settings, (2) modeling multi-turn dynamics for dialog policy learning to achieve better task-completion performance, and (3) integrating domain ontology knowledge into the dialog model. Besides, we review the recent progresses in dialog evaluation and some widely-used corpora. We believe that this survey, though incomplete, can shed a light on future research in task-oriented dialog systems.

발표 링크: https://www.youtube.com/watch?v=qkVZdXL0wDU&list=PL2tRglRS_GqjCcBVM26gYNdoOnX12lLIh&index=7

Presentation Slides: https://drive.google.com/file/d/1Nqb5w2fPDlW0Gf_mxUf03SKZ0ZXoED2s/view?usp=sharing


Replaced duplicate homepage link with packages link

Title, this link could also be added to the about section.

Please amend the commit according to CONTRIBUTING.md


require() of ES Module not supported

The following error occurs when running node . with Node.js v19.7.0 on linux:
/home/ks/discord-wrapped/src/animations/wrap.js:6
const editly = require('editly');
               ^

Error [ERR_REQUIRE_ESM]: require() of ES Module /home/ks/discord-wrapped/node_modules/editly/index.js from /home/ks/discord-wrapped/src/animations/wrap.js not supported.
Instead change the require of index.js in /home/ks/discord-wrapped/src/animations/wrap.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/home/ks/discord-wrapped/src/animations/wrap.js:6:16)
    at /home/ks/discord-wrapped/index.js:3:15
    at Object.<anonymous> (/home/ks/discord-wrapped/index.js:5:3) {
  code: 'ERR_REQUIRE_ESM'
}

After changing src/animations/wrap.js
- const editly = require('editly');
+ const editly = import('editly');

another TypeError is thrown:

Processed 100% of Analytics data (708670779 / 708850519)...
Finished creating 1st frame
Finished creating 2nd frame
Finished creating 3rd frame
Finished creating 4th frame
Finished creating 5th frame
Finished creating 6th frame
Finished creating 7th frame
/home/ks/discord-wrapped/src/animations/wrap.js:393
        editly({
        ^

TypeError: editly is not a function
    at module.exports (/home/ks/discord-wrapped/src/animations/wrap.js:393:2)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async /home/ks/discord-wrapped/index.js:4:2

Possible causes:

https://github.com/sindresorhus/file-type/releases/tag/v17.0.0
https://github.com/mifi/editly/releases/tag/v0.13.0
https://github.com/Assassin-1234/discord-wrapped/commit/ac8597c525de91e0e34ec870ef190cf5aeebdf44

Don't just blindly update to a new major version with breaking changes!

my bad, looks like we updated to what I thought was a minor version of editly to fix a security vulnerability. Reverted back in https://github.com/Assassin-1234/discord-wrapped/commit/3c220edc5ca0fb606b664a5ec78efe0756bec565.

It looks like the same error still occurs and also FFmpeg is no longer being found in /usr/bin/
3c220ed does not actually change const editly = require('editly') which causes the problem.

It seems as changing package.json
- "editly": "^0.14.2",
+ "editly": "^0.12",

fixes the problem, but FFmpeg still isn't seen.

#11


Update README.md broken link

Description
Fix broken link to project structure documentation file.

Using the Github Wiki as main source of documentation is a good idea. Let's think about a little revamp of the README.md someday to fine-tune a little bit more the first contact with the codebase.


suggestion for #2271 with support for Build Configuration Release

Suggestion for #2271: Extend the header search paths. Supports the build configuration Release, too.
What are your opinions? Would a hint in the docs be a better solution?
The only argument for updating new-library is, that the file structure, which is described in #2271, is fairly common.

Since there's no progress, I have thought about the need for this PR. See #2271


fix: fixed data inconsistency on website

Signed-off-by: Maksudur Rahman Maateen<EMAIL_ADDRESS>

@maateen Would you please give more details about what problem this PR try to solve? A screenshot would be very helpful.

@nan-wang I have added the details.


Update Sweden.php to always use leading zero in day

Just adding missing zeroes to two dates for consistency.

Thanks!


ValueError: The current device_map had weights offloaded to the disk. Please provide an offload_folder for them. Alternatively, make sure you have safetensors installed if the model you are using offers the weights in this format.

when run: !xtuner chat internlm/internlm-7b --adapter internlm_7b_qlora_colorist/epoch_1_hf --prompt-template colorist
some errors happened:
2023-09-22 07:01:14.626725: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
2023-09-22 07:01:21.715312: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/xtuner/tools/chat.py", line 263, in
main()
File "/usr/local/lib/python3.10/dist-packages/xtuner/tools/chat.py", line 136, in main
model = AutoModelForCausalLM.from_pretrained(
File "/usr/local/lib/python3.10/dist-packages/transformers/models/auto/auto_factory.py", line 558, in from_pretrained
return model_class.from_pretrained(
File "/usr/local/lib/python3.10/dist-packages/transformers/modeling_utils.py", line 3187, in from_pretrained
) = cls._load_pretrained_model(
File "/usr/local/lib/python3.10/dist-packages/transformers/modeling_utils.py", line 3308, in _load_pretrained_model
raise ValueError(
ValueError: The current device_map had weights offloaded to the disk. Please provide an offload_folder for them. Alternatively, make sure you have safetensors installed if the model you are using offers the weights in this format.

We can add offload_folder for from_pretrained
This error is fixed on https://github.com/InternLM/xtuner/pull/140


Windows support?

I was using this in a windows flow and thought it was working but then noticed:
Run wget https://raw.githubusercontent.com/dineshsonachalam/markdown-autodocs/master/action.py

--2023-01-28 02:06:40--  https://raw.githubusercontent.com/dineshsonachalam/markdown-autodocs/master/action.py
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... 185.199.108.[13](https://github.com/csm10495/pyhtcc/actions/runs/4029633411/jobs/6927728105#step:6:15)3, [18](https://github.com/csm10495/pyhtcc/actions/runs/4029633411/jobs/6927728105#step:6:20)5.[19](https://github.com/csm10495/pyhtcc/actions/runs/4029633411/jobs/6927728105#step:6:21)9.109.133, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... [20](https://github.com/csm10495/pyhtcc/actions/runs/4029633411/jobs/6927728105#step:6:22)0 OK
Length: 3579 (3.5K) [text/plain]
Saving to: 'action.py'

     0K ...                                                   100% 10.1M=0s

20[23](https://github.com/csm10495/pyhtcc/actions/runs/4029633411/jobs/6927728105#step:6:25)-01-28 02:06:40 (10.1 MB/s) - 'action.py' saved [3579/3579]

Run python3 action.py -repo 'csm10495/pyhtcc' -access_token '***' -commit_author 'csm10495<EMAIL_ADDRESS>-commit_user_email<EMAIL_ADDRESS>-commit_message 'Apply automatic changes' -branch 'master' -output_file_paths '[./README.md]' -categories '[code-block,json-to-html-table,workflow-artifact-table,markdown]'
1.0.1[33](https://github.com/csm10495/pyhtcc/actions/runs/4029633411/jobs/6927728105#step:6:36): Pulling from dineshsonachalam/markdown-autodocs
image operating system "linux" cannot be used on this platform
Unable to find image 'dineshsonachalam/markdown-autodocs:1.0.133' locally
1.0.133: Pulling from dineshsonachalam/markdown-autodocs
docker: image operating system "linux" cannot be used on this platform.
See 'docker run --help'.
The system cannot find the file specified.
error: pathspec 'automatic' did not match any file(s) known to git
error: pathspec 'changes'' did not match any file(s) known to git
Everything up-to-date
git config user.name 'csm10495<EMAIL_ADDRESS>git config user.email<EMAIL_ADDRESS>git add ./README.md
git commit -m 'Apply automatic changes' ./README.md
git push origin master

Should this work on Windows? If not can we at least have it fail in a case like this (docker failure)? ... or make it work in Windows :) ?
Thanks!

Until/if its officially working again, I'm using this as a workaround:
      - uses: actions/setup-node@v3

      - name: Markdown autodocs
        shell: bash
        run: |
          npm install -g markdown-autodocs
          markdown-autodocs -c code-block -o README.md
          git commit README.md -m "Updating readme" || true
          git push origin master || true

Basically i setup node, install this module, use it, make commit/push. The || true accounts for if not changes were made, so nothing to commit/nothing to push.


flutter: 2.8.0 -> 2.10.0

Motivation for this change
New upstream release.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Tested by building one of my Android apps with flutter build appbundle.

Result of nixpkgs-review pr 158400 run on x86_64-linux 1

2 packages built:

flutter (flutterPackages.stable)
hover


2022-06-22 equity dashboard chart data update

Equity dashboard stats updates in this PR may be reviewed on staging - here.
After reviewing, if all looks well, approve BUT DO NOT MERGE this Pull Request.
If there are issues with the data:

Note concerns or issues here by commenting on this PR
Work with CDPH directly to resolve data issues
Alert the COVID19 site team in Slack (in the Equity page channel)

This was likely premature -- doing another pull.


Alacritty crashes on opening nvim or cat binary_file.jpg unless stdout is redirected

Which operating system does the issue occur on?
Fedora 28
If on linux, are you using X11 or Wayland?
X11, bspwm with sxhkd
I initially encountered this bug when launching alacritty with a sxhkd binding. Usually, alacritty would behave as expected, but following a strange systemd-related event where messages like this would be generated,
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:67
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:71
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:73
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:72
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:70
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:68
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:74
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:65
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:76
Jul 21 14:36:27 f /usr/libexec/gdm-x-session[2021]: (II) systemd-logind: got pause for 13:66

I would get (after restarting sxhkd to regain keyboard input) this bug in alacritty. Gnome-terminal (launched from alacritty) would open nvim fine. Also, running alacritty from alacritty would allow the spawned alacritty to open nvim, and it would print "Unhandled CSI" errors into the terminal used to spawn it. (https://github.com/jwilm/alacritty/issues/857#issuecomment-405226776).
Finally, I tried changing my sxhkd binding from alacritty to alacritty > /dev/null. This fixed the problem, and leads me to suspect that alacritty will crash if it needs to output these "unhandled CSI" errors without a stdout to print to.
It is, however, also possible that this is an sxhkd bug in spawning alacritty. And there is certainly some problem with systemd-logind creating these pause messages during normal usage (not at lid close time). Sorry for the big mess, hopefully this is not too complicated to fix.

Hmm, your guess that Alacritty will crash when it doesn't have an stdout to print to doesn't seem completely unlikely, however for debugging it would be nice to have a backtrace.
This command will provide a full backtrace while still printing to stdout:
RUST_BACKTRACE=1 ./target/debug/testing 2>&1 | tee /tmp/alacritty.backtrace
If you can get this to work (it might need some tweaking to be used with sxhkd), plase share the full backtrace so the exact error can be pinpointed more easily.

Thanks for that backtrace, would you mind trying the PR I've opened here? Hopefull that should fix this issue.

That seems to have fixed it! I guess we can close this issue when the PR is merged?

Thanks for the feedback! The issue will be automatically closed once the PR is merged.

Wait, i may have spoken too soon. sxhkd is being weird

If you still get crashes, please share the backtrace and I'll get on it.

With your PR branch, alacritty fails to launch unless stderr is redirected somewhere. I suspect I might not have noticed, had it not been for the config file errors from my scrollback config. I can't give you a traceback because stderr being redirected makes the bug disappear.

This is why I've tried using tee to redirect output and still print it.
RUST_BACKTRACE=1 ./target/debug/testing |& tee /tmp/alacritty.backtrace
This is pretty much the same command I've printed earlier, but I think there was a misconception. This does not redirect stdout but rather split stderr to print both to the file and to stdout.
You could also try the following:
RUST_BACKTRACE=1 ./target/debug/testing 2>&1 tee /tmp/alacritty.backtrace 1>&2
This will redirect stderr to stdout, then split it with tee and after splitting it to a file it will redirect the stderr back to stdout.

Pending our coredump generation, I found one place which emits stderr messages and appears to cause this bug (assuming my "stderr has nowhere to output and crashes the program" theory): https://github.com/jwilm/alacritty/blob/dbcb5885adb1f0b23c29838ef8dd837212322409/src/config.rs#L73
I tracked this down since I'm getting these config errors from using a scrollback config with your fix branch (which is a branch off master and hence doesn't have scrollback), and my sxhkd binded alacritty fails to launch if stderr is not redirected, but if i redirect it with /path/to/your/alacritty 2> /tmp/logfile then these config errors appear in the stderr file.
https://doc.rust-lang.org/std/macro.eprintln.html#panics says that eprintln! will panic if it can't output to stderr.
https://github.com/jwilm/alacritty/search?q=eprintln! shows that there are 4 uses of this macro in alacritty. If these had some error handling (or ignoring) then this specific class of crashes should go away.

Related, https://github.com/rust-lang/rust/issues/24821 has the language discussion where the maintainers seem to suggest that mature programs should just eventually ignore the error.
https://github.com/vincentdephily/emlop/pull/4/files is an example of a way to fix this, by using .unwrap_or(()) to suppress the panic.


List of parent functions in code editor

Suggestion for code editor:
"Alt + Insert" hotkey for a list with functions of parent class(es) which you could overide. It also would be handy if this list will be in order "parent -> grandparent" with separators and parent class names.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Buy Token page problem

Problems with dark mode design when you click "buy token" from wallet dropdown menu. See screenshot. Should just route to the exchange page.

Please just have the "buy token" link route to the Exchange page instead of this replica exchange page.

https://github.com/swaponline/MultiCurrencyWallet/pull/2984


Hello, if there is only one GPU, how  to use your code for training?

You can reduce the batch size to 2 and decrease the input size from 769 to 713. And, these operators will result in a performance drop.


[1.14.4] Please update dependency

Describe the bug

Forge - 1.14.4 - 28.2.1

Made a quick fix on GitHub, however I am away and can't make a build for CurseForge so if you would like to use it on the new version try building it yourself.

Ping me when you push to curseforge and I will add it to my 1.14.4 pack and push, thank you.

@ProsperCraft I have pushed the updated 1.14.4 version to CurseForge.


Check if app exists before removing it "resin app rm "

Check if the input is valid i.e. the specific app actually exists/

Fixed in the referenced PR.


Allow dynamic allocation of GPU memory

Hi again,
I thought it might be worth a separate ticket – when running on GPU, all available memory is allocated, but the Tensorflow model of BERT may not actually need it. This should be simple enough to configure – e.g. in the Java API, the following code did the trick for me (replacing this line):
        ConfigProto configProto = ConfigProto.newBuilder()
                .setAllowSoftPlacement(true)
                .setGpuOptions(GPUOptions.newBuilder()
                                .setAllowGrowth(true)
                                .build())
                .build();
        SavedModelBundle bundle = SavedModelBundle.loader(path.toString())
                .withTags("serve")
                .withConfigProto(configProto.toByteArray())
                .load();

        return new Bert(bundle, model, path.resolve("assets").resolve(VOCAB_FILE));

Similarly in the Python API, it should be possible to start the TF session with an appropriately configured ConfigProto.
Thanks

This sounds good to me. I'll add this for both Python and Java next time I do some work on this project, or feel free to send a PR.


KAFKA-4049: Fix transient failure in RegexSourceIntegrationTest

ping @dguy @enothereska @mjsax @ijuma @ewencp for reviews.

Jenkins failed on  org.apache.kafka.streams.integration.RegexSourceIntegrationTest.testRegexMatchesTopicsAWhenDeleted

Discussed with @hachikuji about this issue, and it may be the result of KAFKA-3949, but it is a bit hard to confirm as I have not been able to re-produce locally. So I'll hold on this PR until that issue is fixed to see if it does resolve the transient failure.

Waiting for consecutive jenkins builds on https://github.com/apache/kafka/pull/1762 now.

#1762 seems not the root cause of this failure, will continue investigating in this PR.

Triggered Jenkins build 5181 - 5185 for this commit.

Apache Jenkins builds are not from expected commit, using  https://jenkins.confluent.io/job/kafka-branch-builder/ instead, with consecutive 74 - 78

Consecutive tests in https://jenkins.confluent.io/job/kafka-branch-builder/ from 78 to 82. @ijuma mind taking a look again?

@guozhangwang it's worth noting that this test never failed in Confluent Jenkins (as far as I could see), only Apache Jenkins. Apache Jenkins runs the tests with fork=1 while Confluent uses a higher fork count. Usually this makes Apache Jenkins more stable, but it may be having the opposite effect in this case (or there's another reason why the test fails in Apache Jenkins). If you want to be confident that this change fixes the issue, we'd want to run it in Apache Jenkins.

This failure became quite rare at some point, last failure in trunk jdk7 build was a while back. Since this PR seems like an improvement anyway, LGTM (assuming the tests pass). If the test fails again, we can reopen the JIRA.

Transient failure in Jenkins is not relevant; created https://issues.apache.org/jira/browse/KAFKA-4075 for it.

Thanks @ijuma , merged to trunk.


Large results crash the browser

For our use case, we often have to run large queries and export the results to CSV. We would love to do that through pgweb, but currently when running a query that returns +10k rows, the browser slows down heavily, and with +100k rows, the tab just crashes.
This seems to be because there is no pagination on the results, and pgweb just adds those 100k rows to the DOM (or at least tries to) where here we mostly just want to export it.
Do you have any solution for this? Thanks
Edit: We have one workaround for now that might work which is to just truncate the displayed results to the first 1000 rows

You can try hitting the "Export" button before running the actual query.

@sosedoff I did not even think about it, I assumed we had to execute the query first.
My go skills are limited so the solution I implemented is really just truncating the result array before sending it back to the browser, but maybe some sort of pagination, or truncating but telling the user that the results are truncated would be a better option?

Yeah, you can type the query and hit export, it will not load anything in the browser. Regarding the pagination you mentioned, its true, theres a need for something like that, but i dont have any solutions since i use psql for big data dumps.

On Mar 23, 2017, at 9:48 AM, Tristan F<EMAIL_ADDRESS>wrote:
@sosedoff I did not even think about it, I assumed we had to execute the query first.
My go skills are limited so the solution I implemented is really just truncating the result array before sending it back to the browser, but maybe some sort of pagination, or truncating but telling the user that the results are truncated would be a better option?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Most of the results I've seen are paginated by default. Does your query have its own LIMIT or something on it?
That number of results is going to cause quite a massive result set. Unless we separate pagination into a front-end piece in certain contexts I don't know of any other way to solve this. The problem with doing pagination in the front-end though, is then you have literal DB pagination that is done and front-end level. Which is difficult from a user-interface perspective to handle right.
I think we could improve by perhaps getting the count of returned rows back, and on any calls that will try to display more than like 15k rows or something (we can play around to find a decent number that most systems can handle) then show an alert to the dev that things can be slow and if their goal is to export they could just export it directly without rendering it.

Quick update on this issue:

I've been working on getting https://clusterize.js.org/ integrated into pgweb. It offers a way better experiment on any large datasets.
Limiting the result dataset will likely result in faster page loads. We could add a limit to any custom query passed by a user, and maybe add more results to the table dynamically. Still experimenting.

@sosedoff Did you get anywhere with Clusterize.js? I'd be keen to have a crack at implementing it.

It has been a while and i lost all my work on clusterize.js, so no, i have not gotten anywhere with that. At this point im not even considering adding into the project simply because the whole frontend code needs to be refactored/rewritten, and that opens up doors for all sorts of projects (react? etc).


[BACKLOG-30048] Creating hard coded list of values for Spark tuning UI

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -f 'pom.xml' -pl 'plugins/engine-configuration/impl' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/engine-configuration/impl' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/engine-configuration/impl/src/main/java/org/pentaho/di/engine/configuration/impl/spark/SparkTunableProperties.java
  plugins/engine-configuration/impl/src/test/java/org/pentaho/di/engine/configuration/impl/spark/SparkTunablePropertiesTest.java

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -f 'pom.xml' -pl 'plugins/engine-configuration/impl' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/engine-configuration/impl' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/engine-configuration/impl/src/main/java/org/pentaho/di/engine/configuration/impl/spark/SparkTunableProperties.java
  plugins/engine-configuration/impl/src/test/java/org/pentaho/di/engine/configuration/impl/spark/SparkTunablePropertiesTest.java

Closing for a combined fix in #6799


Only show commands with permission to run in help

Adds new restrictions to the help command to only show commands/subcommands that the user can run. Also updates the subcommand count to reflect permissions.
Fixes #42.

@SamHaggans Pushed some of my changes. In order to avoid serialization of the command tree, I just made all of the class fields public instead of private, which we probably should have done earlier.


$CurrentContainer line doesn't work in custom template - is this a docker-gen bug?

I'm using an own template, and I recently tried to update it for docker-compose V2 with https://github.com/jwilder/nginx-proxy/blob/master/nginx.tmpl
However, when just adding those two lines in the beginning:
{{ $CurrentContainer := where $ "ID" .Docker.CurrentContainerID | first }}

# GENERATING CONTAINER: name: {{ $CurrentContainer.Name }},  id: {{ $CurrentContainer.ID }}

.. the result will start with:
# GENERATING CONTAINER: name: <no value>,  id: <no value>

Similarly, $CurrentContainer.Networks seems to be empty, so that I cannot adapt the loop at all.
I am using the latest docker-gen image (tag "latest"), so is there some obvious reason why this wouldn't be working? I assume this is most likely still my own error, but I cannot really figure out why it wouldn't work given the line is taken unmodified from https://github.com/jwilder/nginx-proxy/blob/master/nginx.tmpl and right at the beginning...

Hmm could it be https://github.com/jwilder/nginx-proxy/pull/459? docker-gen with -only-exposed  running in container with no exposed ports

Oh yea, most definitely, I remember that parameter - and I upgraded the image for years, but never the parameters.
I also know a lot of other people who didn't do this, whom I'm gonna notify about this so they don't run into the same issue :smile: thanks for the explanation.

No problem 👍


Minimalism in Github content

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://github.com/github/docs/commit/9b6f1ad283cb0cf56b8c07a9e8a6a8b19b6b6a2c
What part(s) of the article would you like to see updated?
I am suggesting some changes for content minimalism. In minimalism, we attempt to remove most obvious information like 'Enter your search query and press Enter'. Here, press enter is obvious and we can conveniently remove from user manual. This is the standard practice in technical documentation.
Additional information
No response

I made some changes only. I would like you to review and provide feedback. Do I need to submit pull request for you to see changes?

Dear fellow Americans,
     Today, I want to talk to you about one of the most important issues facing our country and the world today: sustainability. As we look at the state of our planet, we are faced with the harsh reality that our current way of living is not sustainable. We are using up resources at an alarming rate, polluting our air and water, and contributing to climate change.

     However, there is hope. We have the potential to come together as one conservative party, away from the self-implosive culture that has emerged from our two-party system, and work towards a more sustainable future. By doing so, we can not only protect our planet but also reestablish ourselves as the world's base currency.

     One area where we can start is by focusing on the Neo Arctic Region. As climate change continues, the arctic seas are evolving into cold-water trading ports. This presents both an opportunity and a challenge. We must ensure that this development is done sustainably, with a focus on protecting the environment and preserving this unique region for future generations.

To achieve this, we must learn to work together as one family. We must put aside our differences and come together with a shared vision for a sustainable future.
     This means setting aside our political affiliations and working towards common goals.

     As conservatives, we have a unique opportunity to lead on this issue. We can show the world that sustainability is not a partisan issue, but rather a shared responsibility that we all have as stewards of our planet.

     But we cannot do this alone. We need the support and participation of every American. We need to make sustainability a priority in our personal lives, in our businesses, and in our government policies. This means reducing our carbon footprint, conserving resources, and investing in renewable energy.

     In conclusion, let us come together as one conservative party and work towards a sustainable future. Let us embrace our responsibility as stewards of this planet and take action to protect it for future generations. Let us show the world that the United States is committed to being a leader in sustainability, and that we are ready to work together to achieve this goal.

Thank you.

Harmonize and debug terrorism.  "feeding all Ariens opium " - Islamic Mafia

@shusinghal Thanks for opening an issue! We always welcome issues and PRs with ideas for improvements! Feel free to submit one anytime ✨
However, taking a look at the linked commit, I'm not sure this PR would be accepted. Per our style guide, we don't use these sorts of symbols in our directions, unless they refer to an actual UI component. Please take a look through our style guide to see other parts of our content strategies 👍
I'm going to go ahead and close this issue, but we'd love to see other contributions from you. Please take a look at our help wanted section to find open issues you can work on. 💛
Thank you for your interest in improving GitHub Docs! :sparkling_heart:


Pylance crashed due to oom

Environment data

Language Server version: 2024.6.1
OS and version: macOS Sonoma 14.5 (23F79)
Python version (& distribution if applicable, e.g. Anaconda): 3.12.4 (global)

Log
2024-06-11 12:15:18.602 [info] [Info  - 12:15:18 PM] (3385398) Emptying type cache to avoid heap overflow. Used 4033MB out of 4144MB.
2024-06-11 12:39:07.022 [info]
<--- Last few GCs --->

[3385398:0x5b55260]  6420461 ms: Mark-Compact 4043.8 (4133.2) -> 4035.5 (4141.2) MB, 3314.61 / 0.00 ms  (average mu = 0.947, current mu = 0.104) allocation failure; scavenge might not succeed
[3385398:0x5b55260]  6425465 ms: Mark-Compact 4051.5 (4141.2) -> 4042.2 (4147.4) MB, 4901.01 / 0.00 ms  (average mu = 0.865, current mu = 0.021) allocation failure; scavenge might not succeed

<--- JS stacktrace --->

2024-06-11 12:39:07.023 [info] FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
----- Native stack trace -----

2024-06-11 12:39:07.035 [info]  1: 0xca5580 node::Abort() [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.035 [info]  2: 0xb781f9  [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.037 [info]  3: 0xeca4d0 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.038 [info]  4: 0xeca7b7 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.039 [info]  5: 0x10dc505  [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.039 [info]  6: 0x10f4388 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.040 [info]  7: 0x10ca4a1 v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.042 [info]  8: 0x10cb635 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.043 [info]  9: 0x10a8c86 v8::internal::Factory::NewFillerObject(int, v8::internal::AllocationAlignment, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.044 [info] 10: 0x1503a16 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.045 [info] 11: 0x193cef6  [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-11 12:39:07.416 [info] [Error - 12:39:07 PM] Server process exited with signal SIGABRT.

Thanks for the issue. This can happen if your workspace contains a lot of python files. Can you share a repro?
The workaround is what you discussed in the other issue, you set your own node executable which doesn't have a 4GB memory limit. There's more information here:
https://github.com/microsoft/pylance-release/blob/main/TROUBLESHOOTING.md#pylance-is-crashing

Additionally if you turn on full logging:
https://github.com/microsoft/pylance-release/blob/main/TROUBLESHOOTING.md#filing-an-issue
It should show all the files we're trying to analyze. We might get a clue as to what specifically is causing the OOM issue.

I increased the amount of memory available to Node with --max-old-space-size=8192 and it seems to be working at the moment.

Are you able to share your repository? Unless there's 10s of thousands of python files, we shouldn't be running out of memory even with 4GB. At times in the past there were some small tweaks we made that eliminated a lot of data we didn't always need to store, but it's hard to guess what that might be without a repro.

No, unfortunately I can't share the repository: the company policy denies it.

I increased memory limit to 8 Gb, but it still crashes:
<--- Last few GCs --->

[1255022:0x73283c0]   623693 ms: Mark-Compact 8073.2 (8227.9) -> 8063.8 (8235.4) MB, 5509.63 / 0.00 ms  (average mu = 0.914, current mu = 0.315) allocation failure; scavenge might not succeed
[1255022:0x73283c0]   633074 ms: Mark-Compact 8079.8 (8235.4) -> 8070.5 (8241.2) MB, 9291.35 / 0.00 ms  (average mu = 0.744, current mu = 0.010) allocation failure; scavenge might not succeed

<--- JS stacktrace --->

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
----- Native stack trace -----

2024-06-16 14:06:52.449 [info]  1: 0xca5580 node::Abort() [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.449 [info]  2: 0xb781f9  [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.450 [info]  3: 0xeca4d0 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.451 [info]  4: 0xeca7b7 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.452 [info]  5: 0x10dc505  [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.452 [info]  6: 0x10f4388 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.453 [info]  7: 0x10ca4a1 v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.454 [info]  8: 0x10cb635 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.455 [info]  9: 0x10a8c86 v8::internal::Factory::NewFillerObject(int, v8::internal::AllocationAlignment, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]

2024-06-16 14:06:52.460 [info] 10: 0x1503a16 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/home/ogstepanov/.vscode-server/cli/servers/Stable-89de5a8d4d6205e5b11647eb6a74844ca23d2573/server/node]
11: 0x7f8949e99ef6

2024-06-16 14:06:53.377 [info] [Error - 2:06:53 PM] Server process exited with signal SIGABRT.

Unfortunately I cannot provide neither snapshots of my workspace nor filenames, etc.  I can give cloc stats for Python files:
---------------------------------------------------------------------------------------
Language                             files          blank        comment           code
---------------------------------------------------------------------------------------
Python                                1387          27870           7098         170338

But if there were a way to provide high-level stats such as distribution of file sizes and amount of time needed to index them, I could send that. Also, if I could see hottest directories/files locally, I could decide whether I could exclude them from indexing.

Yeah we need to take a heap snapshot to figure out where the problem is. But without a repro taking a heap snapshot on a release build will mangle all of the types, making it all but impossible to figure out where the problem is.
I don't think it's the number of files though. I can open the root of my C: drive and pylance continues to work just fine. It's 100s of thousands of files.
I'm guessing it doesn't immediately crash either, making it hard for you to narrow down the problem.
Can you share your requirements.txt at least? I can try reproing with the same venv at least.

if you think it is related to indexing, you can set "python.analysis.indexing": false and see whether OOM stops.

Closing issue with no response. If you can respond with the asked for information, please reopen this issue.
Thanks.


TypeError: Cannot read property 'length' of undefined

I am trying to split a string into groups and I found the groupWith function helpful. However, my string is not always available. I am getting a TypeError in such case.
TypeError: Cannot read property 'length' of undefined
    at /path/ramda.js:1910:23
    at /path/ramda.js:466:28
    at f1 (/ramda.js:446:27)

Here is a simple test case to reproduce that. The function readStr first reads a string and sends it to groupWith. If the string is not available, it returns undefined. If undefined is passed to groupWith, I am getting the above error.
var str = readStr(params); // return undefined if the string is not available
console.log(R.groupWith(R.equals)(str));

I think if there is a default value of zero for len on line 1910 then I can still get the empty array from res on line 1908 without crashing.

We tend to avoid most parameter checking in Ramda, leaving to Sanctuary the role of being the more strict type-checking library.
Such a check seems fairly innocuous to me, though, if you want to create a PR for it.
It might not be as trivial as you suggest for several reasons.  A naive || 0 won't work as the field would still be tested for the property first and would throw the exception.  And dealing with potential undefined values here gets somewhat interesting when you add in currying.  You can probably find something that works; it just doesn't seem quite as simple as it first appears.

@CrossEye I notice from the API documentation that groupWith supports only arrays and strings. In my pull request, I check if list is either type and set it to zero if it is not. Please let me know how it looks to you and if you need further changes. Thank you.

I'm opposed to this change. If a function expects a value of a certain type it should not accept values of other types. Any -> Any functions are difficult to reason about.

I agree with @davidchambers ; i'd rather see it blow up than try to be "helpful" by providing a default.

I think the implementation should be consistent throughout the whole library. For example in line 2426 of the length function, instead of simply returning list.length and let it blow up, it checks whether the list is null and whether its length is a number. The same pattern should also apply here.

My answer is slightly different.  I don't care what type the input is.  It's something with a length property that responds to indexing; our implementation also requires it to respond to slice, but that's a detail.  So I don't want to test for Array/String.
But, I would have no problem with something that extended this as the original idea suggested, so that undefined simply returned an empty list.  I know this is something on which @davidchambers  and I have fundamental disagreements, and I think @buzzdecafe and I are also at least somewhat at odds.  To me, throwing exceptions should be reserved for the most dire of circumstances, when there is simply nothing left to do.  Here there is a reasonable option.  I'd prefer to go for that.


Cinder/NetApp plug-in enhancements for multiple NFS shares

Cinder/NetApp plug-in enhancements for multiple NFS shares:

Support multiple shares on the same NetApp back-end using different IPv4 addresses.
Remove NFS share IPv4 address (now specified in the share itself).

In our production OpenStack environments where are integrating Cinder with our NetApp filer using NFS as the storage back-end. We have two different NFS shares configured in Cinder, each one pointing to a different IPv4 adresss. With the change implemented in this pull request. this is how our /etc/cinder/shares_1.conf looks like:
cat /etc/cinder/shares_1.conf
<IP_ADDRESS>:/ESJC_OST2_NFS_cinder02
<IP_ADDRESS>:/ESJC_OST2_NFS_cinder01
Notice how each NFS share points to a different IPv4 address. Now, the IPv4 address is part of the CDOT share itself, not a common attribute to the entire NetApp back-end.
Since this change is not backwards compatible, I've bumped the release of the plug-in from 6.x to 7.x.

Thank you for contributing to openstack/fuel-plugin-cinder-netapp!
openstack/fuel-plugin-cinder-netapp uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


When reading data from DuckDB in to pandas Dataframe, the cases of column names are not consistently preserved.

Following the documents, I save a  pandas Dataframe with all column names in uppercase in  DuckDB.
Loading back the data into pandas dataFrame, I do get all column names in uppercase if I run:
df = conn.execute(select * from tablename).fetchdf()

However, all of the column names will be in lowercase if I run
df = conn.execute(select column_name1, column_name2 ... from tablename).fetchdf()

Thanks for the report! I can confirm this goes wrong here. As a temporary fix you can fix it by quoting the column names (e.g. df = conn.execute(select "A", "B" ... from tablename).fetchdf()), or by providing a quoted alias (e.g. df = conn.execute(select column_name1 AS "COLUMN_NAME_1" ... from tablename).fetchdf())


Full-chain test from ROOT file to PrimitiveArray.Float8

We should do this before adding all the other types (as I described in PR #1).

DONE!


Check message integrity.

Oh, cool – I didn't know this was a thing! Where'd you find it in the documentation?

I checked the headers and follow GitHub's guide: https://developer.github.com/webhooks/securing/ 😂
I just found this: https://developers.facebook.com/docs/graph-api/webhooks#receiveupdates

I can add tests and remove the rewind thingy if this looks good to merge. Later, you can decide what to respond in case that integrity fails, but probably it doesn't matter if an attacker receives a 200. I need to investigate more.

Cool! Looks good to me, so please do.

Done!

Coverage increased (+0.06%) to 99.233% when pulling 2efd225d4ed26366d72e34cfed008eef9ba539f8 on frodsan:feature/check-message-integrity into 8b62993869c7da1a1958a728a5198c617f8f152d on hyperoslo:master.

this is pretty and good 😉

:cake:


Filip/update readme 2

Codecov Report

Merging #202 (a741f95) into master (f4818d9) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a741f95 differs from pull request most recent head b62b0b5. Consider uploading reports for the commit b62b0b5 to get more accurate results

@@           Coverage Diff           @@
##           master     #202   +/-   ##
=======================================
  Coverage   26.10%   26.10%
=======================================
  Files          36       36
  Lines        2264     2264
=======================================
  Hits          591      591
  Misses       1673     1673

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 f4818d9...b62b0b5. Read the comment docs.

Fixed. There is also a task for trusted setups, which has much more context: https://github.com/webb-tools/arkworks-gadgets/issues/78


Documentation

first version of the documentation

Codecov Report

Merging #41 into master will increase coverage by 0.08%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
+ Coverage      93%   93.09%   +0.08%
==========================================
  Files          20       20
  Lines        1687     1694       +7
==========================================
+ Hits         1569     1577       +8
+ Misses        118      117       -1

Impacted Files
Coverage Δ

src/solver.jl
100% <ø> (ø)
:arrow_up:

src/table_log.jl
0% <0%> (ø)
:arrow_up:

src/model.jl
95.65% <100%> (+0.03%)
:arrow_up:

src/BnBTree.jl
90.03% <100%> (+0.4%)
:arrow_up:

test/runtests.jl
100% <100%> (ø)
:arrow_up:

src/bb_user_limits.jl
94.11% <0%> (+2.94%)
:arrow_up:

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 b7c5e60...35b6bff. Read the comment docs.


Import/Edit props (spotlights)?

Hey!
Wondering if there's a feature (or if you guys are dev-ing it rn) to import/export spotlights (or other props)? Especially since the node ref and positioning with relative coords is not very straight forward....
Cheers
MG

Hello, yeah currently there isn't this feature nor is it being worked on currently, although yes I'd like to have this feature in the future.


I found a broken link

https://docs.docker.com/develop/develop-images/dockerfile_best-practices/ is broken and returns a 404
This link is referenced in the left menu in
Develop with Docker ->
Develop your apps on Docker ->
Develop images ->
Best practices for writing Dockerfiles

Addressed by #7908 .


fmt/1157

The new Folio Infobase File has overlapping beginning-of-file sequences:

Only the "Folio" bit is being generated in the test signature.

Is this an error with PRONOM do you think @richardlehane?

it is a bit of an odd one, but I think intended: i.e. looks for two sequences "Folio" and "v200" which both need to be near the start of the file but can be in any order

confirmed with@Dclipsham . Failed testing DROID on this file
No results found for file: fmt-1157-signature-id-1539.nfo. Expected: fmt/1157

fmt/1157 skeleton suite hasn't created a conformant file - https://www.nationalarchives.gov.uk/PRONOM/Format/proFormatSearch.aspx?status=detailReport&id=1967&strPageToDisplay=signatures

file should contain string' v200' which equates to 0x76323030 but this is missing from the file available via zenodo for v95 https://zenodo.org/record/3269467#.Xc1KGDP7SUk

Thanks Jeremie, et al. I should have done something with this but forgot about it... Not sure I'll have a fix for a few weeks, but let me know if it does help you to have it sooner or if you can live without for this release.

Thanks Ross, it just came up as part of DROID testing. I've created a pair of files that contain the missing 'v200' so a fix certainly isn't urgent from our perspective (am happy to share these - GitHub Issues doesn't like the file type though for file upload - let me know if you want them..). As Richard points out we cant be certain of the order in which 'Folio' and 'v200' will appear so expressed them both as distinct BOFs with offset range set

Sounds good, if you zip them, the can be added here, and I can use them for reference in the fix!
I have of course excitedly being watching references to the suite in recent testing. Big changes. They look to be shaping up well.

Great - learning something new :) thank you!
fmt-1157-variants.zip

the latest release (v108) has a couple more signatures impacted by this

Wondering if I fixed this for container signatures and whether this will work here: https://github.com/exponential-decay/skeleton-container-test-suite-generator/pull/14

fmt/1062 (https://www.nationalarchives.gov.uk/PRONOM/Format/proFormatSearch.aspx?status=detailReport&id=1868&strPageToDisplay=signatures) also seems to be impacted by this issue


Hard reboots phone when mounting ISO

Nexus 6P 8.1 Oreo
Magisk v16
Selected Arch Linux ISO and every time I hit mount, the phone locks up and fully reboots

Are you running the stock rom? Also note that I won't support Magisk because of the way it screws with the system. Please try with regular root (yes, the one that modifies your system partition). If you can still reproduce that way, we can try further troubleshooting.

Yeah stock. Magisk is really the most common way nowadays. SuperSU and the like can't be trusted. If it's not working due to that, then I guess I won't use this software. Thanks.

Sorry, I can't help then. If anyone is able to reproduce this without magisk, I can reopen this and check things out with you.


RFC Deprecate extras.installer-name in composer.json

We have some code that uses a custom property in composer.json to determine the name of a module:
https://github.com/silverstripe/silverstripe-framework/blob/c278576996f73cf8be53924c0fe55dff86b5045e/src/Core/Manifest/Module.php#L137-L139
extras.installer-name is not a standard part of the composer.json schema - I can't find any indication that it's used by anyone other than us, and even we only use it in one place: silverstripe/assets composer.json
We should deprecate usage of this property, and stop using it ourselves in the one place it's used. It's just unnecessary complexity.
Alternatively, if there's a really good reason to keep it, that reason should be very clearly documented.

Yep, voting 👍

IIRC it’s a hangup from the SS3 days when things weren’t installed in vendor/, and that was used to allow specifying the folder name because things like alphabetical order used to be important for loading config things.
One thing to check: is it still used by YAML fragments and/or the moduleexists: thing?

I don't think we need an RFC for this. This seems obviously pointless.
I'd say let's add a deprecation notice now and set it up in such a way that we don't forget to remove it once we start work on CMS5.


Support LaTeX in markdown infobox

datavzrd should support LaTeX in the description for a table.

The easiest option would probably to use the showdown-katex extension for the showdown.js that we already use.


MRG: Add MGH montages

These are translations of standard_1020 to MGH 60- and 70-channel EEG cap locations using this gist, which includes transcriptions courtesy of @SherazKhan.
WIP for now because hopefully we can eventually turn this into a tutorial on how to localize EEG without dig. For example, we know that the ref was actually the nose, so we could use the Nasion for that, but I don't think our montage code provides support for setting loc[3:6] currently, which we might (?) need to do for forward modeling. (Seems like it shouldn't really matter since we do an avg ref proj anyway.)

Codecov Report

Merging #4326 into master will increase coverage by 3.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4326      +/-   ##
==========================================
+ Coverage   82.91%   86.06%   +3.14%
==========================================
  Files         346      346
  Lines       62897    62897
  Branches     9672     9672
==========================================
+ Hits        52150    54130    +1980
+ Misses       8064     6106    -1958
+ Partials     2683     2661      -22

Okay this should be good to go. I'll do the example/tutorial later. For now this can go in (test shows it works).

thx @Eric89GXL


discord video cannot be sent to line

Hi @Jj088456 ,
Discord video to Line has now only supported .mp4 files only, please make sure the video file you're syncing is in MP4 format, and also under 200MB.
I'll soon make it more format supported when I have time, any PR is also welcomed!

我是傳mp4格式也小於200MB，是用你做好的機器人!

還想請問 能夠多群同步嗎?

了解，可以請你上傳該影片給我嗎? 這樣方便我進行測試 謝謝
目前不支援喔！ 專案目前更新速度很慢，主要以維護為主。歡迎任何大神一起維護~


switch to @rossjillian controlnet model

the model supports all the controlnets in a single model
it is faster and has less variance in prediction time (see her writeup)

The work by @rossjillian to speed up controlnet is 🥇


[public report]: Can't review. Error execution reverted: LaborMarket::review: Not signaled

User Impacted
0xC4b8651f45596678755186F360dea4f05293175E
Discord ID (including number)
dinatim#3567
Workflow Impacted
Analyze
Feature Impacted
Score Submission
Issue description
I can't revie because of this error:
execution reverted: LaborMarket::review: Not signaled

Blocking
yes it is
Relevant log output
No response

Is there an update for this issue? still can't review


Handle 'weird' message formats

Issue: #473 and #474

[X ] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:
a) Make GeoHashCacheProvider async
b) Handle malformed JSON messages via CorruptMessageHandlers
c) Handle messages that are not served in the specified pipeline(s)
Test:
Use the test client to send a malformed message or a message with a non-described pipeline.

Looks very good now, waiting for build server, then I'll merge


ScriptJSLoader google is not defined

Hi,
I'm trying to use the ScriptJSLoader but I run into an issue for which I don not see a workaround yet. Maybe you can help. My code is as follows:


RangeError [ERR_INDEX_OUT_OF_RANGE]: Index out of range

I have the custom device and I want to get its indications. But when I try to subscribe to indications, I get this error:
RangeError [ERR_INDEX_OUT_OF_RANGE]: Index out of range
    at checkOffset (buffer.js:1015:11)
    at Buffer.readUInt16LE (buffer.js:1063:5)
    at Gatt.<anonymous> (/home/ns16/projects/my/noble/sample/node_modules/noble/lib/hci-socket/gatt.js:625:24)
    at Gatt.onAclStreamData (/home/ns16/projects/my/noble/sample/node_modules/noble/lib/hci-socket/gatt.js:132:26)
    at AclStream.emit (events.js:185:15)
    at AclStream.push (/home/ns16/projects/my/noble/sample/node_modules/noble/lib/hci-socket/acl-stream.js:35:10)
    at NobleBindings.onAclDataPkt (/home/ns16/projects/my/noble/sample/node_modules/noble/lib/hci-socket/bindings.js:286:15)
    at Hci.emit (events.js:180:13)
    at Hci.onSocketData (/home/ns16/projects/my/noble/sample/node_modules/noble/lib/hci-socket/hci.js:497:14)
    at BluetoothHciSocket.emit (events.js:180:13)

This is my code:
const noble = require('noble')

const CUSTOM_SERVICE_UUID = '2a6178017f4211e8b5670800200c9a66'
const ENABLING_MEASUREMENT_MODE_CHARACTERISTIC_UUID = '2a61c62a7f4211e8b5670800200c9a66'
const MEASURED_VALUES_CHARACTERISTIC_UUID = '2a62b0807f4211e8b5670800200c9a66'

noble.on('stateChange', state => {
  console.log(`State changed: ${state}`)
  if (state === 'poweredOn') {
    noble.startScanning()
  }
})

noble.on('discover', peripheral => {
  console.log(`Found device, name: ${peripheral.advertisement.localName}, uuid: ${peripheral.uuid}`)

  if (peripheral.advertisement.localName === 'DEVICE-1') {
    noble.stopScanning()

    peripheral.on('connect', () => console.log('Device connected'))
    peripheral.on('disconnect', () => console.log('Device disconnected'))

    peripheral.connect(error => {
      peripheral.discoverServices([CUSTOM_SERVICE_UUID], (error, services) => {
        console.log(`Found service, name: ${services[0].name}, uuid: ${services[0].uuid}, type: ${services[0].type}`)

        const customService = services[0]

        customService.discoverCharacteristics([ENABLING_MEASUREMENT_MODE_CHARACTERISTIC_UUID, MEASURED_VALUES_CHARACTERISTIC_UUID], (error, characteristics) => {
          console.log(`Found characteristic, name: ${characteristics[0].name}, uuid: ${characteristics[0].uuid}, type: ${characteristics[0].type}`)
          console.log(`Found characteristic, name: ${characteristics[1].name}, uuid: ${characteristics[1].uuid}, type: ${characteristics[1].type}`)

          const enablingMeasurementModeCharacteristic = characteristics[0]
          const measuredValuesCharacteristic = characteristics[1]

          enablingMeasurementModeCharacteristic.write(new Buffer([0x01]), true, error => {
            console.log('Measurement mode is enabled')

            setTimeout(() => {
              measuredValuesCharacteristic.on('data', data => {
                console.log(data)
              })

              measuredValuesCharacteristic.subscribe(error => { //Here is error!
                console.log('Measured values indication enabled')
              })
            }, 1000)
          })
        })
      })
    })
  }
})

The error arise in this line.
I debugged and finded out that the data variable each time containes the same buffer <Buffer 09 04 29 00>. The buffer has only four bytes, but above code tries to read fifth and sixth bytes, so the error arise.
I made a fork of this library and want to fix the error in it. How I can change the code correctly?
I fixed as follows:
var value = data.length >= 6 ? data.readUInt16LE(4) : 0x0000;

The error doesn't arise, but I don't get the indications. Is this right or wrong?

I am facing the same issue as well! Is there any info related to this !?


alt+space hotkey is not working

I am not able to get the hotkey feature to work. I have tried updating the hotkey to different combinations of alt+x or super+x or supre+space but none of them worked. This is a fresh install.
Expected Behavior
alt+space hotkey should toggle the zuzu app
Actual Behavior
alt+space hotkey does not do anything.
Steps to reproduce the behavior
prest alt + space hotkey.
Versions

Zazu: 0.3.2
OS: Ubuntu 16.04
GIT: 2.7.4

Please, include a link to gist of most applicable ~/.zazu/log/ file.
Here is the snippet from ~/.zazu/log/ file
{"level":"info","message":"plugins are loaded"}
{"version":"0.3.2","level":"debug","message":"app is ready"}
{"hotkey":"alt+space","level":"verbose","message":"registering zazu hotkey"}
{"plugin":"tinytacoteam/zazu-light-theme","level":"verbose","message":"loading package: tinytacoteam/zazu-light-theme"}
{"plugin":"tinytacoteam/zazu-light-theme","level":"info","message":"loading css for theme"}
{"plugin":"tinytacoteam/zazu-calculator","level":"verbose","message":"loading package: tinytacoteam/zazu-calculator"}
{"plugin":"tinytacoteam/zazu-file-finder","level":"verbose","message":"loading package: tinytacoteam/zazu-file-finder"}
{"plugin":"tinytacoteam/zazu-template","level":"verbose","message":"loading package: tinytacoteam/zazu-template"}
{"plugin":"tinytacoteam/zazu-system","level":"verbose","message":"loading package: tinytacoteam/zazu-system"}
{"plugin":"tinytacoteam/zazu-package-manager","level":"verbose","message":"loading package: tinytacoteam/zazu-package-manager"}
{"plugin":"tinytacoteam/zazu-fallback","level":"verbose","message":"loading package: tinytacoteam/zazu-fallback"}
{"plugin":"tinytacoteam/zazu-clipboard","level":"verbose","message":"loading package: tinytacoteam/zazu-clipboard"}
{"hotkey":"cmd+shift+v","level":"verbose","message":"registered a hotkey"}
{"level":"info","message":"plugins are loaded"}
{"level":"info","message":"opening about page"}

Based on the log provided, it looks like it's registering alt+space at the time, and it never detected you hitting that combination. Is it possible your os has reserved this combination?

Ubuntu uses Alt + Space for the window menu (minimize, maximize, always on top, etc.). Try ctrl+space.

If you just did a fresh install it could be that you need to log-out and in again before hotkeys are picked up correctly. No clue why, but it worked for me...
I've the Ctrl+Super+Space and it works like a charm.

Where can we edit hotkeys configuration? Anybody knows?


1 positional argument expected

In a example at pub.dev . you have not included positional argument.

Oh! So this was the reason some users were reporting that they were having difficulties.
Thanks for bringing this to my attention. Fixed!


Adblock

Is it possible to implement dnsmasq to filter ads when client uses the VPN? Would you guide me in that direction? Thank you!

https://github.com/pi-hole/pi-hole/wiki/OpenVPN-server:-Installation
use whatever VPN installation you choose and run pihole alongside it

Thank you @technicalpyro. Wouldn't that make the ip address of the server to be a public DNS that blocks ads? I want to avoid that.

no.  there are a few features that prevent this.  the first being firewall rules i myself have a droplet running pihole and openVPN it is not a public resolver.  dnsmasq allows three settings within this project
Listen on all interface and respond to all queries
listen only on the designated interface
listen on all interfaces but only respond if the device requesting is one hop away or less
The only thing i have that is public facing is the web interface but that can be easily disabled as well
also worth noting it can be a little tricky to set up in terms of the VPN server options and a couple other things but if following the wiki i pasted above has you struggling i am willing to help as much as i can


Language Support Problem

This library can be used just for english address, I have made some changes with that it'll support all languages, like. en, ur, gr, ru, chinese, japanese etc.

So sorry for the delay on this ... for some reason never saw this. Looking into it soon.

Great

On 24-Jul-2017, at 11:05 PM, Mike Bronner<EMAIL_ADDRESS>wrote:
So sorry for the delay on this ... for some reason never saw this. Looking into it soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/GeneaLabs/Phpgmaps/pull/5#issuecomment-317506426, or mute the thread https://github.com/notifications/unsubscribe-auth/ARXl18XzbIIZQHF0gWdDlqLi0BPI_kKSks5sRNz6gaJpZM4LUnr_.

Thank you so much
On Sat, 14 Oct 2017 at 5:23 PM, Mike Bronner<EMAIL_ADDRESS>wrote:

Merged #5 https://github.com/GeneaLabs/Phpgmaps/pull/5.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GeneaLabs/Phpgmaps/pull/5#event-1293381774, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARXl1y-JcKV5iRsrLNxZEZ5ZeAVSMKwdks5ssKfagaJpZM4LUnr_
.

--
M Shafique Qadri
Skype: mr.shafique.qadri
Profession: Software Engineer


Multiple small updates

Hi,
Following the issues that I had (issue #17), I update the file table_evaluator.py. Here are the changes:

Remove the numerical_encoding function: It was creating one-hot encoding for all the categorical variables. It thus created a sparse matrix. And I think the inconsistencies that I got were coming from this issue.
Update the function convert_numerical: I updated this function to simply deal with categories written as strings (since it seemed to be the main issue of the code). I'm using the factorize function from pandas to replace the string values with integers.
Use the convert_numerical function when needed: You had duplicate code when you needed to encode the dataset. I thus removed these parts and simply called the function convert_numerical instead.
Add PCA similarity score back: I simply added the PCA similarly score back. I don't know why you removed it. Maybe you might want to remove it again. But the results were interesting for me. =)

That's all the changes I've made to your code. It shouldn't have any issues with inconsistent results now. But if there are any issues, please let me know and I will have a look at it. =)

@glederrey Thanks for the PR. Some very good stuff. To answer some questions/raise some points.

Numerical encoding was using one-hot-encoding for categorical columns with more than 2 values, because for most/all distance measure, you introduce arbitrary distance between values if you just factorize. That's why we can only use one-hot encoding in that location. I think this only goes for row_distance, so if you re-add it for that function but use factorize for the others, I think it should be ok.

To clarify, the distance in your code is
nearest neighbor mean                    0.5580
nearest neighbor std                     0.3509

While on develop i get:
nearest neighbor mean                    0.5655
nearest neighbor std                       0.3726

Regarding PCA, one current issue is that both datasets fit their own PCA model. To actually compare the results, we should train PCA on one dataset and transform both with those values. Additionally, PCA gets really easy skewed by large numbers. So with large numbers in a dataset, the small values don't really matter anymore. That's why I removed it from the actual report output, because I didn't think it actually reflected the similarity that well. I think the plots can still offer some insights, but should also be considered very carefully.

Hope this helps. You raise some good points, so if you can address these two changes, I'll be happy to merge your PR.
Furthermore, thank you for your patience for my response :)

@Baukebrenninkmeijer Thanks a lot for the clarification.

I see the issue with the distance. The best would be to have a distinction between categorical and ordinal data types. Since the idea of distance works for ordinal data but it does not make sense for categorical data. But it's definitely something for the future =)
I see the issue with PCA. I was playing with something like this when doing my own evaluation. And it's true that it was not super consistent.

I will update it on my repo and make another pull request. =)

I made the update. I added a function numerical_encoding_one_hot that is the old numerical_encoding function. The new function is used in the function row_distance as requested. I also removed the PCA results. =)

@glederrey Indeed, a distinction between ordinal and categorical data would be the ideal solution. However, this is often hard to do or requires a lot of manual work, so for the most consistent yet easy approach I opted for this.

@Baukebrenninkmeijer Yeah, I'm currently investigating new validation techniques that will take this into account. But it's a lot more complicated than expected. =)
I just added a boolean parameter to make the KFold CV optional. =)

@glederrey I see that you've put the output in evaluate within the if verbose section. I think those should be enabled by default, since it shows useful information. Because it's not being returned by all_results, there's also not a non-verbose way of showing this information. We could add a toggle to hide it, but it doesn't qualify as verbose imo.
However, I'm also not looking forward to having a verbose, a hide_output and a notebook toggle, that feels like overkill. Ideally, I'd just want the output to be there by default.
What we could do is add the other values to all_results, and make sure everything is returned from the useful information. We can then have a toggle that says return_values or smthing like that. verbose really is for details during running, so that you can really track what's happening, not for just showing the results.

Thanks for the docstring addition btw :)

In my very specific case, I'm running your code on 725 different datasets. So, printing the results creates a lot of "noise". That's why I added it in the verbose. But I agree that the final results should not be in the verbose.
If the notebook output is already printing stuff, we could simply remove the prints. (I have no idea how it would work if launched from the terminal though) If not, then I will revert the code to its previous state. I agree that adding another flag to hide the output would be overkill. (There are always ways to redirect the sys.stdout to a file or whatever to hide it in the worst case)
However, I think it's important to output the results in a variables for better systematic results.
Let me know what you think and I will update the code accordingly. =)
Btw, I will have to cite your evaluator in my article. So, should I cite your master thesis? Or have you published an article following your master thesis?

@glederrey
The notebook section is really meant for notebook situation, and shouldn't be expected to be the default imo. I did also built in a function to automatically detect jupyter environments, so that could help a bit there.
725 datasets is indeed a lot, hahaha. I understand where you are coming from now. 😄
I think i'll have a look myself at returning all the needed information. Regarding hiding the output for now, could you just change that locally for now? I think I will eventually solve it with the return_values flag, as mentioned. That's fairly clean, and more often used in parallel with verbose.

Okay, I see. And I agree with you.
I will then revert back to printing by default and remove it locally =)
I'll do the commit tomorrow morning.

@glederrey regarding your other question:

Btw, I will have to cite your evaluator in my article. So, should I cite your master thesis? Or have you published an article following your master thesis?

I have unfortunately not followed up with anything, so you can cite my thesis :).


Can I just get the lower/upper bounds instead of the real distance?

It is included in another paper. https://link.springer.com/chapter/10.1007/978-3-030-20081-7_2


#3718 change multi series tooltip

Description
Line Chart tooltip missing cross-shelf content
Related Issue : #3718

How Has This Been Tested?

Create a line chart using measured values and dimension values.
Check that the tooltip on the chart shows the measure by dimension value.

Need additional checks?
Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 2

deploy to 1


Assertion `canonicalStart == max(trackReader->s, chunk->s)' failed.

Hello I am running flagger_v1.0.0.sif and run into the following error:
[2024-12-17 14:56:07] Parsing/Creating coverage chunks.
[2024-12-17 14:56:07] The given input file is not binary so chunks will be constructed from cov file.
[2024-12-17 14:56:07] Index file exists: figures/postcuration_qc/flagger/IMR90/coverage_file.cov.gz.index
[2024-12-17 14:56:07] Parsing index ...
[2024-12-17 14:56:07] Index is parsed from disk.
[2024-12-17 14:56:07] Parsing header info for ChunksCreator.
[2024-12-17 14:56:09] Truth tag was set to false (or not defined) so truth labels will not be parsed from file.
[2024-12-17 14:56:09] Prediction tag was set to false (or not defined) so prediction labels will not be parsed from file.
[2024-12-17 14:56:09] Creating empty chunks.
[2024-12-17 14:56:09] Created a thread pool with 16 threads for parsing chunks
[2024-12-17 14:56:09] Queued 277 jobs for the thread pool (no more than 16 jobs will be processed at a time)
hmm_flagger: submodules/chunk/chunk.c:425: Chunk_addTrack: Assertion `canonicalStart == max(trackReader->s, chunk->s)' failed.

The command I am running is:
singularity exec --bind /scratch/rranallo-benavidez/BJ_IMR90/figures/postcuration_qc/flagger/IMR90/ /tgen_labs/barthel/software/bin/flagger_v1.0.0.sif hmm_flagger --input figures/postcuration_qc/flagger/IMR90/coverage_file.cov.gz --outputDir figures/postcuration_qc/flagger/IMR90/hmm_flagger_outputs --alphaTsv /home/programs/config/alpha_optimum_trunc_exp_gaussian_w_4000_n_50.tsv --labelNames Err,Dup,Hap,Col --threads 16

Do you have any ideas for what could be the error? Thank you.

Hey @tbenavi1  Could you remove figures/postcuration_qc/flagger/IMR90/coverage_file.cov.gz.index and try again?

I deleted the index file and reran and got the same error. I then also deleted the coverage_file.cov.gz, and reran bam2cov which ran fine. Then I reran hmm_flagger and got the same error.


Adding/removing directory entries does not respect directory ownership

Describe the bug
Adding or removing entries from a directory requires that the user has write permissions to that directory. Trying to modify entries in a directory to which the user cannot write should raise PermissionError.
How To Reproduce
The following test cases should both succeed:
import os
import pytest
import pyfakefs.fake_filesystem

def test_fail_write_dir(fs):
    # Prereq: Create a directory at '/readonly-dir' that is owned by root and has
    # permissions 0o755 (rwxr-xr-x)
    rodir = "/readonly-dir"
    os.makedirs(rodir, exist_ok=True)
    st = os.stat(rodir)
    if st.st_uid != 0:
        os.chown(rodir, 0, 0)
        os.chmod(rodir, 0b111_101_101)
        st = os.stat(rodir)
    # Only modifyable by the root, but readable/traversable by all
    assert st.st_uid == 0
    assert st.st_mode & 0o777 == 0b111_101_101  # == 0o755
    # Try to add a new entry to the readonly subdirectory
    with pytest.raises(PermissionError):
        with open(f"{rodir}/file.txt", "w"):
            pass

def test_fail_fakefs_root_owned_dir():
    fs = pyfakefs.fake_filesystem.FakeFilesystem()
    fake_os = pyfakefs.fake_filesystem.FakeOsModule(fs)
    # Create a regular file
    fs.create_file("file.txt")
    # Create a directory that can only be written-to by root
    fake_os.mkdir("/readonly-dir", mode=0b111_101_101)
    fake_os.chown("/readonly-dir", 0, 0)
    # Creating a new entry should fail
    with pytest.raises(PermissionError):
        fake_os.link("/file.txt", "/readonly-dir/file.txt")

Your environment
Please run the following in the environment where the problem happened and
paste the output.
$ python -c "import platform; print(platform.platform())"
Linux-6.7.6-arch1-1-x86_64-with-glibc2.39
$ python -c "import sys; print('Python', sys.version)"
Python 3.11.7 (main, Jan 29 2024, 16:03:57) [GCC 13.2.1 20230801]
$ python -c "from pyfakefs import __version__; print('pyfakefs', __version__)"
pyfakefs 5.3.5
$ python -c "import pytest; print('pytest', pytest.__version__)"
pytest 7.2.0

If I understand correctly, the issue appears to be in FakeDirectory.add_entry, which starts with this guard:
if (
    not helpers.is_root()
    and not self.st_mode & helpers.PERM_WRITE
    and not self.filesystem.is_windows_fs
):
    raise OSError(errno.EACCES, "Permission Denied", self.path)

The check on st_mode checks that the S_IWUSR bit is set, but does not consider the case that st_uid != get_uid().
I'm trying to write code that is robust against "adverse" filesystem conditions, with non-writable directories being a common example.

Thanks! pyfakefs currently has only a rudimentary concept of user rights - it is currently mostly concerned with root vs. non-root users, so this (and your other issue) are not surprising.
Thanks for digging into this and providing the tests - I will have a go at this, but that may take a bit.


Add a prefix list real S3 e2e Rust test

A follow-up of https://github.com/neondatabase/neon/pull/3843#discussion_r1147757889 , that adds the first S3 integration tests

Welp, the test fails with
2023-03-28T13:01:09.905354Z  INFO Using s3 bucket 'ci-tests-s3' in region 'us-west-2' as a remote storage, prefix in bucket: 'Some("pagination_should_work_test_1680008469905/")', bucket endpoint: 'None'

...snip

Caused by:
    0: Error { code: "AccessDenied", message: "Access Denied", request_id: "WE44JPZ4VA7WTH6G", s3_extended_request_id: "6yYbnOwa8qBbeH3aD+qLU4P8DGcCFkd3ncRR9g/lM/C+E23Qeb5b0skZyjNSWp4rfJVVn3OrQFM=" }
    1: Error { code: "AccessDenied", message: "Access Denied", request_id: "WE44JPZ4VA7WTH6G", s3_extended_request_id: "6yYbnOwa8qBbeH3aD+qLU4P8DGcCFkd3ncRR9g/lM/C+E23Qeb5b0skZyjNSWp4rfJVVn3OrQFM=" }
    2: Error { code: "AccessDenied", message: "Access Denied", request_id: "WE44JPZ4VA7WTH6G", s3_extended_request_id: "6yYbnOwa8qBbeH3aD+qLU4P8DGcCFkd3ncRR9g/lM/C+E23Qeb5b0skZyjNSWp4rfJVVn3OrQFM=" }

https://github.com/neondatabase/neon/actions/runs/4543295204/jobs/8007804407#step:14:658
I know that the agents we use have an access key for the public github bucket, will try to use that.


Cannot set estimated difficulty while approving a kata

Really MINOR bug, but just to let you know, if I both set the estimated difficulty for a kata and approve it with a given official difficulty, I get this (which is easily solved by pressing F5 on your browser):

I'm pretty sure this is fixed, though estimated ranks are meaningless once they are approved so it shouldn't really matter.


S3 read only memory region and stat

@mihaimaruseac
This PR adds tf_read_only_memory_region, NewReadOnlyMemoryRegionFromFile and Stat ( because we need GetFileSize ).
Thank you for doing an import manually with the previous PR.
Could you please tell me why the it failed the internal import ?

I am not sure why it didn't work before. It's very likely that it is because the import happens at file level (so if a file changed, you need to rebase back on master), unlike git which is at change level

@mihaimaruseac
The CI failed again. Could you check it please ? Thank you

Will import manually.


Add CHRC

inforamtion.txt
hi . i would like to request you to add Charna Token (CHRC) to exchanging and reviewing that we meet all the requirement .
these are the information :
website : http://charnatoken.top
"code": "CHRC",
"issuer": "GBRPTWEZTUKYM6VJXLHXBFI23M2GSY3TCVIQSZKFQLMOJXH7VPDGKBDP"
I attached the icon of CHRC .
an other information please let us know
thank you

Can you make a pull request with the changes?

okay wait

done

can you help me please?
how to add CHRC ?