Show diagnostics for failed/skipped tests run with pytest

For #120

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[x] Unit tests & system/integration tests are added/updated
[x] Test plan is updated as appropriate
[x] package-lock.json has been regenerated by running npm install (if dependencies have changed)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: SalmonMode sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks a ton for the feedback! I really appreciate it, and have been working on this over the past few days to get in those fixes.

Once a broken test is fixed, the problems do not go away, they are still displayed in the problems window

Thanks for the examples screenshots in #120! The parametrization stuff tipped me off. Turns out, it was because the name of the test itself is technically changing because the params changed. So I added a check in the stripStaleDiagnosticMessages function to remove any diagnostics that no longer had any associated tests.

I quite like the way you have displayed the errors in the problems window.
Please could you try to ensure they are displayed in one line as opposed to having line breaks in there. (see here where there seems to be some line breaks) https://github.com/Microsoft/vscode-python/issues/120#issuecomment-449177085

Thanks! I wasn't sure how people would feel about that, but I wanted something convenient to jump to each point in the stack trace. I ended up just stripping out anything coming after the first new line, because pytest can dive pretty deep when it describes what variables were what, and I wasn't sure where else I could put them.
I was hoping there would be some title text for the diagnostic I could leave them in, but it doesn't look like VS Code supports that yet. Maybe I should make a feature request for that 🤔

I merged the master with your changes and #3770

I wasn't sure what to do with this info, but I appreciate it! 😄 Should I rebase there or something for this PR?


[BUG] shell task launch two datax processes

Version:
1.1.0
工作流添加了一个shell节点，shell节点执行python脚本启动datax，
工作流启动后，shell节点在服务器上启动了2个datax进程

You can look at the worker, how many times the task was submitted, sudo-u tenant sh xxx.command, and it is to be clean before running the task, for example, if you start, delete the process instance, after that, the worker There may be previous tasks on
你可以看worker，任务提交了多少次，sudo -u 租户 sh xxx.command，还有就是在跑任务之前要保证之是干净的，比如如果你启动了，之后把流程实例删除了，之后，worker上可能会有以前的任务

You can look at the worker, how many times the task was submitted, sudo-u tenant sh xxx.command, and it is to be clean before running the task, for example, if you start, delete the process instance, after that, the worker There may be previous tasks on
你可以看worker，任务提交了多少次，sudo -u 租户 sh xxx.command，还有就是在跑任务之前要保证之是干净的，比如如果你启动了，之后把流程实例删除了，之后，worker上可能会有以前的任务

这个流程今天是第一次跑任务，而且只有这一个流程在跑

Did you resolved?

Don't reply for a long time, it will be close.


fixing the spi port for the edison board

This fixes the BoardDefaults for the apa-102 example to use the correct one for the Edison board.

Due to the vast number of changes since the original change, including removing the Edison from samples, I'm going to close this PR.


Update ScaleOut lab

Update the Scale out lab to add exercises that demonstrate:

When custom hosting, how to use of the api from NServiceBus.Distributor.MSMQ
Using the feature profiles using NServiceBus.Host for the distributor and the workers.
how to distribute load (i.e. workers that process a command) and also how to distribute an event being processed (a subscriber that is behind a distributor)
how to decommission a worker

Relates to issue:
https://github.com/Particular/Housekeeping/issues/12

I don't think we have plans at the moment to change the manuals. Last update was to convert the existing scale out to v5 guidelines. Closing this issue.


openshot-qt: unbreak

Motivation for this change
Unbreak openshot-qt.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

4 packages built:

libsForQt5.libopenshot
libsForQt512.libopenshot
libsForQt514.libopenshot
openshot-qt

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

4 packages built:

libsForQt5.libopenshot
libsForQt512.libopenshot
libsForQt514.libopenshot
openshot-qt

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108689 run on x86_64-linux 1

4 packages built:

libsForQt5.libopenshot (libsForQt515.libopenshot)
libsForQt512.libopenshot
libsForQt514.libopenshot
openshot-qt

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108689 run on x86_64-linux 1

4 packages built:

libsForQt5.libopenshot (libsForQt515.libopenshot)
libsForQt512.libopenshot
libsForQt514.libopenshot
openshot-qt


DOC: Fixed typo in description of --bundle-timestamp

Coverage remained the same at 87.133% when pulling eca992f2bfefb71a3a645bd8bbb5ecca9deeb0ca on bundle-date-is-timestamp into b5585370e9d38088858db084b0e1da6892ad78dd on master.

lgtm


How to use the MIOpen fused kernels

Hi, I found that some fused kernels like conv+bias+activation has been implemented in MIOpen. How can I use this feature in AMDMIGraphx. I tested a simple case for classification, but there are no fused kernels in ~/.config/*.txt after running. Now the CBAinfer kernel may not support the MIOPEN_FIND_ENFORCE=3 mode in MIOpen.
Other question is that if I want to add some new OPs for inferencing in AMDMIGraphx, how should I do, is there some docs?

Closed, answer provided.

Closed, answer provided.


IPv6 addresses with more than two groups set cause address collision

Consider the following topo:
---
name: c

mgmt:
  network: clab-mgmt
  ipv4_subnet: <IP_ADDRESS>/16
  ipv6_subnet: fd1a:b128:5901::/32

topology:
  defaults:
    kind: linux
    image: alpine

  nodes:
    l1:
      mgmt_ipv4: <IP_ADDRESS>
      mgmt_ipv6: fd1a:b128:5901::101
    l2:
      mgmt_ipv4: <IP_ADDRESS>
      mgmt_ipv6: fd1a:b128:5902::101

Nodes l1 and l2 has different IPV6 addresses set, but they are not parsed either by clab or docker and cause address collision:
❯ clab dep -c -t test.clab.yml
INFO[0000] Containerlab v0.38.0 started
INFO[0000] Parsing & checking topology file: test.clab.yml
INFO[0000] Removing /tmp/clab-c directory...
INFO[0000] Creating lab directory: /tmp/clab-c
INFO[0000] Creating docker network: Name="clab-mgmt", IPv4Subnet="<IP_ADDRESS>/16", IPv6Subnet="fd1a:b128:5901::/32", MTU="1450"
INFO[0000] Creating container: "l2"
INFO[0000] Creating container: "l1"
ERRO[0001] failed deploy phase for node "l2": Error response from daemon: Address already in use
ERRO[0001] failed to run postdeploy task for node l2: failed to Statfs "": no such file or directory

This is because the address that gets configured for the first node only has two groups set:
❯ docker inspect clab-mgmt
[
    {
        "Name": "clab-mgmt",
        "Id": "b301768abab95b091bd22f7d2720900efaaccc17c9c3e1441e068c5f5a36a408",
        "Created": "2023-03-21T15:00:49.717101046+01:00",
        "Scope": "local",
        "Driver": "bridge",
        "EnableIPv6": true,
        "IPAM": {
            "Driver": "default",
            "Options": null,
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/16"
                },
                {
                    "Subnet": "fd1a:b128:5901::/32"
                }
            ]
        },
        "Internal": false,
        "Attachable": false,
        "Ingress": false,
        "ConfigFrom": {
            "Network": ""
        },
        "ConfigOnly": false,
        "Containers": {
            "1027b8dbe0df2b768d1283160aa1827f6d4128e740bb793a3c634fd8def76093": {
                "Name": "clab-c-l1",
                "EndpointID": "08f0a9da0dc45321e5c3b903f95c26b0516e1af78dc958406299368c8fc4a35f",
                "MacAddress": "02:42:ac:1e:01:01",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": "fd1a:b128::101/32"
            }
        },
        "Options": {
            "com.docker.network.driver.mtu": "1450"
        },
        "Labels": {
            "containerlab": ""
        }
    }
]

Reported to docker
https://github.com/moby/moby/issues/45402

Thanks @steiler for digging all the way to moby, as explained in https://github.com/moby/moby/issues/45402#issuecomment-1533717536 this is an implementation detail in moby that truncates makes an address  fd1a:b128:a:b:c:d:e:f to be truncated as fd1a:b128::c:d:e:f.
We have to make a note in the docs about it for ipv6 addresses


xdg-open spawn error

We are using email-templates in an AWS lambda function. The template files are hosted in an S3 bucket. In the lambda, we download the template to a local temp folder in the lambda.
When I try to call the send method, it errors out.
When using v5.0.5, it gives this error object:
"error": {
"code": "EACCES",
"errno": "EACCES",
"syscall": "spawn"
}
When I tried using v6.0.0, it gave this error:
spawn xdg-open ENOENT
The options being passed in are:
{
"template": "/tmp/0BaZ4XBBo/default-test",
"message": {
"to"<EMAIL_ADDRESS>"subject": "Test",
"headers": {
"X-SES-CONFIGURATION-SET": "Engagement"
},
"ref": [
"a reference"
],
"template": "default-test"
},
"locals": {
"message": {
"kpins": [
{
"kpin": "1234",
"id": "123444"
}
]
}
}
}

PR welcome that would fix this, I don't really use nor support Lambda


Added Annotated Text and Resume Parsing Functionality

Pull Request Title
Added Annotated Text and Resume Parsing Functionality
Related Issue
None
Description
This pull request introduces several key changes to the application. The changes made include:

Added the ability to display annotated text in the application.
Implemented a feature that allows users to view extracted keywords.
Enabled users to parse resumes within the app.

However, it's important to note that I'm encountering a couple of issues:

The application crashes frequently, specifically from backend port 8000, especially when there are changes in the backend code.
I have also made modifications to the implementation of the annotated text library.

Screenshots demonstrating these changes will be attached to the pull request.
Type

[ ] Bug Fix
[x] Feature Enhancement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Other (please specify):

Proposed Changes

Added annotated text feature.
Introduced functionality to view extracted keywords.
Enabled parsing of resumes.
Made modifications to the annotated text library.

Screenshots / Code Snippets (if applicable)
Screenshots will be provided with the pull request.
How to Test
To test the changes in this pull request, follow these steps:

Clone this branch.
Run the application.
Check the functionality related to annotated text, keyword extraction, and resume parsing.
Pay attention to any crashes occurring from backend port 8000, especially after making changes to the backend code.

Checklist

[ ] The code compiles successfully without any errors or warnings
[ ] The changes have been tested and verified
[ ] The documentation has been updated (if applicable)
[ ] The changes follow the project's coding guidelines and best practices
[ ] The commit messages are descriptive and follow the project's guidelines
[ ] All tests (if applicable) pass successfully
[ ] This pull request has been linked to the related issue (if applicable)

Additional Information
Feel free to provide any additional information
https://github.com/srbhr/Resume-Matcher/assets/110348311/9ced5232-758c-4ed0-a0fc-84864aab310d
about the pull request, especially regarding the issues encountered and the changes made to the annotated text library implementation.

Really awesome work @Devasy23


Add ResourceNavigationLinks list on subnets + record unit test for it

Add ResourceNavigationLinks list on subnets + record unit test for it

Can one of the admins verify this patch?

Hi @AhmedSalama-MS, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (ahsalam). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Leave a comment


'ssh: Could not resolve hostname localhost:4: Temporary failure in name resolution' when 'Fine tunning' in run.sh

i found a problem when running run.sh to 'Fine tunning', which is as follow:
ssh: Could not resolve hostname localhost:4: Temporary failure in name resolution
It seems horovod led to this problem, how can I solve this problem? i will appreciate it for any advice.

answer: 升级open mpi包  sudo apt-get install openmpi-bin openmpi-doc libopenmpi-dev


Add support for blocks in list items.

This PR adds support for src and dynamic blocks in list items. I got a report here about this. I tested on emacs orgmode, and it properly supports these blocks in list items.
I created an anonymous node for this for the sake of DRY, but if you think it needs to be done differently, let me know. I set it up to go only after new line.

Is there any reason why not to use _element here?

Not that I know of. I just wasn't aware I could use it. I'll update.

Well actually, seems I can't use it because of conflicts:
Trying this:
    itemtext: $ => seq(
      repeat1($._textelement),
      repeat(seq(
        $._nl,
        optional($._nl),
        choice(repeat1($._element), $.list)
      )),
    ),

And getting this:
Unresolved conflict for symbol sequence:

  _liststart  bullet  paragraph_repeat1  '
'  list  •  '
'  …

Possible interpretations:

  1:  _liststart  bullet  paragraph_repeat1  '
'  (_element  list)  •  '
'  …
  2:  _liststart  bullet  paragraph_repeat1  (itemtext_repeat2  '
'  list)  •  '
'  …

Possible resolutions:

  1:  Specify a higher precedence in `_element` than in the other rules.
  2:  Specify a higher precedence in `itemtext_repeat2` than in the other rules.
  3:  Add a conflict for these rules: `_element`, `itemtext`

Okay. Some rearranging is necessary, the nodes in _element have newlines associated already, and lines of text (repeat1($._textelement)) do not. I'd test a few things out but I'm not at a computer.
What we want here is very similar overall to the body node, basically. Without the standalone directives, which simplifies parsing. Should we just use the paragraph node for separate bodies of text within the list item? Since we're adding the rest of the elements. I.e we might see (itemtext (paragraph)) for a simple list item, `(itemtext (paragraph) (paragraph) (dynamic_block (directive)) (paragraph)) for something that looks like:
-  paragraph 1

   Paragraph 2
   #+NAME: A
   #+begin: a
   b
   #+end:

    Paragraph 3

This was too much phone effort. :S
What do you think?

This was too much phone effort. :S
Don't do this to yourself :D

Yeah, I think we could easily have a paragraph in there. From my tests, list item can contain anything that a headline body can, so I think that should be the way to go.

Just wanted to update on why this isn't done yet:
So, the _listitemend node that's matched by the external scanner expects to be just before a newline, which means that matching the element nodes as written (which include the newline chars) won't work right. For example, your PR here will match the following as a single list instead of two lists:
- A
  #+begin: a
  b
  #+end:

- B

There are at least three ways to resolve this.
My first attempt was to rewrite the scanner to start looking at column=0, which complicates things because the extras includes whitespace and is matched before the external scanner, so I'm not sure how to resolve it with this strategy. I can't use token.immediate with an external scanner, so I don't think this is easily resolved, but there might be a few other tricks we can pull with zero-width nodes that I would like to try before throwing out this idea.
The next way to resolve it is change from using 'listitemend' to move newline matching logic that's currently in the external scanner into the grammar, and only scan for indentation changes. This might be the route to settle on, but I'm not excited about it.
The next way to resolve it is not add the elements to the list parser at all, and inject an org parser into the itemtext. This seems like a bit extra to me, and it would be simple but a last resort.
For the short term, it might be simpler to solve why the parser is throwing an error in your example and resolve that, so I can sink into solving this problem when I have a day to invest into it.

I hoped this issue was fixed with the latest changes, but it seems it's still broken:
This:
* TODO Test
  - Test
    #+BEGIN_SRC
    Content
    #+END_SRC

Gives this:
(document [0, 0] - [5, 0]
  (ERROR [0, 0] - [5, 0]
    (headline [0, 0] - [1, 0]
      stars: (stars [0, 0] - [0, 1])
      item: (item [0, 2] - [0, 11]
        (expr [0, 2] - [0, 6])
        (expr [0, 7] - [0, 11])))
    (bullet [1, 2] - [1, 3])
    (expr [1, 4] - [1, 8])
    (ERROR [2, 4] - [2, 15])
    (expr [3, 4] - [3, 11])))

Is it still complicated to add a support for this? I know previous version caused some issues.

Is it still complicated to add a support for this? I know previous version caused some issues.

Unfortunately yes. I think the best solution until I can invest some time on that problem will be to inject an org parser across (itemtext), which is a bit sloppy feeling but I don't have a better solution
That said, it really shouldn't be an error, so I'll see if there's a solution to that.

Should be resolved in v0.2.2, commit 9a595e5. I removed (itemtext) to do so breaking previous compatibility, and parsing descriptions was a nightmare, so I'm moving that to queries. But it seems to be working for me. Checkboxes are trivial via queries and were added to the highlights.scm


Docker container

Might be useful to have a simple docker script to make containers out of biotool.
Will need one for each language implementation.

Will not do


feat: include and exclude in copy info

related: https://github.com/dagger/dagger/issues/2711
Then we could get readable image history.

cc @helderco

While the code makes sense, I cannot visualise the improvement.
Can you share a simple example @morlay we can run to see the behaviour before & after this change? I can see this helping future maintainers too.
Alternatively, it's OK to wait for @helderco & @sipsma which may have the context and can share.

it could be found in logs when #Copy used.
For final build history display, have to wait for this
https://github.com/moby/buildkit/pull/2932


add fullscreen priority, simplify telemetry descriptions

OPS Build status updates of commit 93694df:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Azure/portaldocs, working directory: W:\yclj-s, target branch: dev, target branch commit: 5a5a543c5b2ba5be80f3a4b3110c7b216e1036c3, source branch: ewinner/designpatterns, source branch commit: .

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.

OPS Build status updates of commit 51a9ee4:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Publish service's config file: .openpublishing.publish.config.json is not found. Exception:

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.


Linux axis range incorrect

Joystick axes on Linux seem to rest at 0.5 instead of 0, full range being 0 to 1 instead of -1 to 1.

Note that this issue is not presenting on web, so solutions should take care to except browsers.

Fixed in #119

Confirmed fixed, gj


Update index.jsx

Pull Request Type

[ ] ✨ feat
[ ] 🐛 fix
[ ] ♻ refactor
[ ] 💄 style
[ ] 🔨 chore
[ ] 📝 docs

Relevant Issues

resolves #xxx
What is in this change?

Additional Information

Developer Validations

[ ] I ran yarn lint from the root of the repo & committed changes
[ ] Relevant documentation has been updated
[ ] I have tested my code functionality
[ ] Docker build succeeds locally

Please do not push PRs back to the main repo that are just you replacing our branding with yours.


Update AVS device SDK to version 3.0.0

99% of the changes in this PR are due to updates in the 'alexa' fork, where the version has changed from v1.26.0 to v3.0.0.
To review the changes related to the XMOS devices, I recommend to use the comparison below:
https://github.com/alexa/avs-device-sdk/compare/master...lucianomartin:avs-device-sdk:feature/alexa-v3.0.0?expand=1
Most of the files are ported from the 'xmos' version v<IP_ADDRESS>, for example the HID and GPIO keyword detector files. I would pay special attention to the pi.sh and setup.sh scripts.
The XVF3615-UA and XVF3615-INT need special handling of the KWD classes.
This PR removes all the code related to the following defines/features (see https://github.com/xmos/vocalfusion-avs-setup/issues/69):

PI_HAT_CTRL: used for XVF3610
sales demo for XVF3510
SENSORY_OP_POINT and other Sensory KWD features
XMOS_AVS_TESTS: this feature requires a KWD on the host, such as Sensory

I tested the setups for:
- XVF3610-INT
- XVF3610-UA
- XVF3615-INT
- XVF3615-UA
Part of https://github.com/xmos/vocalfusion-avs-setup/issues/69
Closes #85

Given the difficulties of finding a specific file in the review, I will list my comments here.  I have only reviewed the files in the comparison provided by @lucianomartin.  Thanks to him for making this comparison available.  Files I do not mention are okay.
In GPIOKeywordDetector.h, remove the sentence on lines 1 & 2 that applies the XMOS Public license.  Also modify the XMOS copyright to state Modifications  (c) 2021-2022 XMOS Limited.  Please make this set of changes in each file that mentions the XMOS Public license and the Apache license.  These two license are incompatible with one another according to @tomblackie-xmos.
In KeywordDetector.cmake, lines 121 - 123 mention PI_HAT_CTRL, but the description of the PR says that all code related to PI_HAT_CTRL has been removed.

See comment

@mbanth, the requested changes have been included in https://github.com/xmos/avs-device-sdk/pull/87/commits/c436a358405e1d5fe80cf19da160bd0d2aee7c3c


[Candidate 3v3 Nocky Mountains ZH v2] Oil Derricks near ocean have greatly different terrain space around

Observed behaviour
Oil Derricks near ocean have greatly different terrain space around.
Expected behavior
Oil Derricks have about the same space around them to offer fair conditions.
Screenshots

Oil Derrick also clips into terrain.

Space looks still a bit different but looks acceptable to my noob eyes.


Binaries for amd64 downloaded on armv7

Environment information

py-solc-x Version: 1.1.0
solc Version: all
Python Version: 3.9.5
OS: linux running on a raspberry pi 4

What was wrong?
When trying to compile my contracts, I get "Downloaded binary would not execute..."
https://github.com/iamdefinitelyahuman/py-solc-x/blob/7bed05bad9811d8ee36177b200c6cdce91e5ff70/solcx/install.py#L636
How can it be fixed?
Check the system architecture instead of assuming amd64.
https://github.com/iamdefinitelyahuman/py-solc-x/blob/7bed05bad9811d8ee36177b200c6cdce91e5ff70/solcx/install.py#L41

armv7 isn't going to be compatible because it is little endian and solc compiler wants big endian.


Allow product's conf.py to inject version information when building docs

This allows the product to inject the version information before this file is even loaded. This in turn allows us to use this version for other purposes. This was not possible before because e.g. for the iso product the version would be inaccurate (INMANTA_DONT_DISCOVER_VERSION -> 1.0.0) until after this file had finished loading. By allowing version injection, the version becomes accurate at load-time.
See inmanta/inmanta-service-orchestrator#410

Processing this pull request

Merged into branches master in c1c13f1e915d3e03758ce25da814a765a7357941

Processing #6341.

Processing #6340.


Validate boolean config values - Closes #2045

Codecov Report

Merging #2465 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2465      +/-   ##
============================================
+ Coverage      83.2%   83.22%   +0.01%
- Complexity     2182     2184       +2
============================================
  Files           359      359
  Lines          6206     6210       +4
  Branches       1138     1139       +1
============================================
+ Hits           5164     5168       +4
  Misses          467      467
  Partials        575      575

Impacted Files
Coverage Δ
Complexity Δ

...rturbosch/detekt/rules/complexity/ComplexMethod.kt
94.44% <ø> (ø)
16 <0> (ø)
:arrow_down:

...itlab/arturbosch/detekt/api/internal/BaseConfig.kt
88.46% <100%> (+2.09%)
12 <1> (+2)
: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 1f44b4b...924f1f1. Read the comment docs.


make the auto screen size as an option for small screens in debian buster

Hi, my laptop is small, the default option of wechat makes everything too big, and could you please make it as an option that user can set manually? thank you very much!

os:
mmc@debian ~ $ screenfetch
         _,met$$$$$gg.           xxx@debian
      ,g$$$$$$$$$$$$$$$P.        OS: Debian 10 buster
    ,g$$P""       """Y$$.".      Kernel: x86_64 Linux 4.19.0-8-amd64
   ,$$P'              `$$$.      Uptime: 1h 15m
  ',$$P       ,ggs.     `$$b:    Packages: 1672
  `d$$'     ,$P"'   .    $$$     Shell: zsh 5.7.1
   $$P      d$'     ,    $$P     Resolution: 1366x768
   $$:      $$.   -    ,d$$'     DE: XFCE
   $$\;      Y$b._   _,d$P'      WM: Xfwm4
   Y$$.    `.`"Y$$$$P"'          WM Theme: JacWin10
   `$$b      "-.__               GTK Theme: Windows10 [GTK2]
    `Y$$                         Icon Theme: Faenza
     `Y$$.                       Font: 文泉驿等宽微米黑 10
       `$$b.                     CPU: Intel Core i5-3320M @ 4x 3.3GHz [54.0°C]
         `Y$$b.                  GPU: intel
            `"Y$b._              RAM: 5093MiB / 15741MiB
                `""""

We have a new feature to set DPI before you start.
You can try to set DOCHAT_DPI=96 to see if it helps?
DOCHAT_DPI=96 dochat.sh

See: https://github.com/huan/docker-wechat#dochat_dpi

my screen is small, and this is the default dpi, with -e DOCHAT_DPI=96, chatting dialog size seems to be ok, but the fonts will be bad.
most fonts seems to be ok, some fonts seems to be not good.

after i uncheck the auto screen size setting from setting page

the screen seems to be like this

when i run docker with dpi

i think maybe this problem also affect the fonts display.

Understood.
If we can get to know the registry key and value of that option, then we can be very easy to disable it by config an environment variable.

https://github.com/bestwu/docker-wechat
this one, with the default command, the fonts seems to be  fine on my laptop.
here is the screenshot for the fonts

the option "auto screen size" is also checked by default, but screensize is not so big as your docker, it seems as big as your docker command with dpi=96.

That's by design: DoChat v0.5 are using a default DPI=120, which is 125% larger than the default setting 96 DPI.
And I found that in Ubuntu, it seems that the font size can follow the DPI setting and will be resized as expected.
I believe there are other settings that need to be set under the Debian distribution.

ok, i see, thank you very much!

Is there anyone known the following DPI environment variables?

GDK_SCALE
GDK_DPI_SCALE

Will they be helpful for our DoChat? If so, how to use them?


Planner active message issue.

What is your Quests version / build number (do not say "latest")?
4.0.0-rc.2 of spigot
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
Having such a planner for example:
planner:
start: 3:10:2020:16:52:0:Europe/Paris
end: 3:10:2020:16:53:0:Europe/Paris
repeat: 86400
cooldown: 120
override: false
The way the issue work is basically. Quest starts at 16:52 then we finish it for 1 seconds for example and there is 120 seconds cooldown of the quest. But quest ends at 16:53. Basically after 1 minute quest is gone from planner and goes into 1 day repeat timer right?
This is where issues comes in "test wil lbe active in . "
This is what kind of message i'm getting after that where plnTooEarly: "<quest> will be active in <time>."  the placeholder  is not getting replaced properly and there is nothing in there at all. No errors either.
https://prnt.sc/vcvrt1 - This is a picture of how it looks like ingame.
If applicable, can you provide an example quest from quests.yml using Pastebin.com?

Thanks for the report. I believe this happens due to margin of error which is covered briefly in #1244
For the time being, I must suggest that you simply wait a few seconds before completing the quest, and delay your players accordingly if necessary. In a moment, I will update the plugin to display an erroneous message should this occur again, but it will be purely cosmetic. If waiting any number of seconds does not resolve the issue, feel free to reply with a new screenshot.

https://ci.codemc.io/job/PikaMug/job/Quests/159/

https://prnt.sc/vd7ox7 - Picture of the messages.
https://pastebin.com/Krra29Rs - List of errors (Something new that's occuring on dev builds now, but not on official spigot release)

I will look into the messages further. The list of errors is related to #1439 and I suggest following that ticket for updates.


Add definitions for common HomeLocationStruct and types - see connect…

Fixes https://github.com/project-chip/connectedhomeip/issues/33567
Adds the

Common Home Location Structure XML definion
Common Area Namespace XML definition

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


JMC thinks [-1] is a string slice

summon::entity.Pos = summon::objectives[-1].Pos;

raises
Unexpected token (Token(token_type=<TokenType.KEYWORD: 'Keyword'>, line=175, col=56, string='.Pos', _macro_length=0, quote='')) at line 175 col 56.

This is actually caused by a misalignment of merging path tokens (AKA. I just missed a +1 but that took half an hr to find).
Should be fixed in https://github.com/WingedSeal/jmc/commit/0393bed68d91ded9fba05a0488c2620805af9796


Windows support

This PR enables installation on Windows. The Eyelink Developers Kit is required and it must be modified slightly. Instructions are added to the Readme.md

Hi cboulay, thanks for taking the time to work on this. Unfortunately I don'thave access to a windows machine to test this. Looks good from a quick glance though. Thanks a lot!


navigate like routing

Hi,
There is any way to route a page using the router link or any other way ??
Like => this.router.navigateByUrl(['/redirecturl']);
Not using location.href="redirectUrl";
please update and thanks,

@kishorekumarvajravel you can change pages however you see fit. I would hide the tour before transitioning and then show it again once you are fully transitioned. You may need to leverage beforeShowPromise for this.

Hii @rwwagner90 ,
I tried this.router.navigateByUrl(['/redirecturl']); . But doesn't work that way.
message in browser console-
ERROR TypeError: Cannot read properties of undefined ( reading 'navigateByUrl')
console.log works but this doesn't which make me think if while executing shepherd service is not able to refer to navigateByUrl ( I did import this in the component where i use the tour but still the problem persists).

@yc-98 sounds like you are not binding your scope right and this.router is undefined.

You'll need to bind the function to the component scope.


Stacked Themes : Bug Fixes | New Features

1- Fixed Platform Checking error when app is run on web as platform isn't supported on web.
2- Added navigation bar color changer
3- Fixed overlay colors ( status bar and navigation bar colors )  when app is resumed

Nice PR. I like the feature.
Please can you run flutter test and post the results below so I can see if all the tests are still passing.
Also can you bump the patch version and then add the changes into the change log.

Actually I am not very expert but I will surely try. Thanks for such motivating comment.


Read Customer Points

Cosumed: 1.5 hour
Lines of code by file:

src/main/java/es/upm/miw/betca_tpv_core/domain/persistence/CustomerPointsPersistence.java


Add support for custom id's

Could it be autoincrementing and id: ID! by default but allow to use anything as ID and then do not care about increments and anything else. For example email could be marked as unique and that will be enough to have it as ID

What order do we choose?
If our model has:
id: ID!

and
@db.primary
email: String

which should we choose as the primary key used in database generation and mapping?
I think we should prefer the @db.primary field over id: ID! field. This is an explicit request by the user to make this the primary key so it should be treated as such.
^^ @wtrocki

Fully agree. Explicit over implicit case

This is now merged in Graphback, but graphql-migrations does not have this support yet (it is restricted to id: ID!. Closing this issue however as this is related to mapping and not migrations.


Make instructions in PR template a comment

Description

There are some instructions in the PR template that should be been enclosed in a comment, but which were not.
Motivation and Context

Several people mentioned that the instructions were cluttering pull requests.
How Has This Been Tested?

No new tests, just a change to the PR template
Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[x] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

The continuous integration build process will run make checks to confirm that tests pass and that code quality meets minimum standards. You may optionally run this locally as PRs will not be accepted until they pass.
Signed-off-by: Sheehan Anderson<EMAIL_ADDRESS>

LGTM


rocksdb : collection 100GB size : "Out of memory: Kill process" failures

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[ ] 3.0
[ ] 3.1
[V ] 3.2
[ ] 3.3
[ ] self-compiled devel branch

Mode:

[ ] Cluster
[V ] Single-Server

Storage-Engine:

[ ] mmfiles
[V ] rocksdb

On this operating system:

[ ] DCOS on

[ ] AWS
[ ] Azure
[ ] own infrastructure

[ ] Linux

[ ] Debian .deb
[ ] Ubuntu .deb
[ ] SUSE   .rpm
[ V] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[ ] MacOS, version:

this is an AQL-related issue:
[ ] I'm using graph features
I'm issuing AQL via:

[ ] web interface with this browser:     running on this OS:
[ ] arangosh
[ ] this Driver:

I've run db._explain("<my aql query>") and it didn't shed more light on this.
The AQL query in question is:
The issue can be reproduced using this dataset:
Please provide a way to create the dataset to run the above query on; either by a gist with an arangodump, or `db.collection.save({my: "values"}) statements. If it can be reproduced with one of the ArangoDB example datasets, it's a plus.
Foxx
this is a web interface-related issue:
I'm using the web interface with this browser:     running on this OS:

[ ] authentication is enabled?
[ ] using the cluster?
[ ] _system database?

These are the steps to reproduce:

open the browser on http://<IP_ADDRESS>:8529
log in as ...
use database [ ] _system [ ] other:
click to ...
...

The following problem occurs: [Screenshot?]
Instead I would be expecting:
this is an installation-related issue:
Describe which steps you carried out, what you expected to happen and what actually happened.
Single host, Centos 7.3, rpm installation, ArangoDB 3.2.9, RocksDB engine.
The host is a VMWare machine, 8 CPUs, 32G RAM, 32G swap.
The main in-use collection has 309M documents , and it's on-disk size  is about 116GB.
That collection is used heavily by another application host - mostly for read operations, and eventual writes.
Sometimes , the Arangodb fails and restarts. This happens when both RAM and swap memory are totally used (almost no free left).
No messages in arangod.log when the db processes went down, only afterwards regular startup messages.
In /var/log/messages following messages appear (cut) :
...
Jan 22 20:43:16 vm-skarndby5 kernel: Scheduler invoked oom-killer: gfp_mask=0x200da, order=0, oom_score_adj=0
...
Jan 22 20:43:18 vm-skarndby5 kernel: Out of memory: Kill process 3887 (arangod) score 977 or sacrifice child
Jan 22 20:43:18 vm-skarndby5 kernel: Killed process 3887 (arangod) total-vm:462874388kB, anon-rss:31277956kB, file-rss:0kB, shmem-rss:0kB
Jan 22 20:43:20 vm-skarndby5 systemd: arangodb3.service: main process exited, code=killed, status=9/KILL
Jan 22 20:43:20 vm-skarndby5 systemd: Unit arangodb3.service entered failed state.
Jan 22 20:43:20 vm-skarndby5 systemd: arangodb3.service failed.
Jan 22 20:43:25 vm-skarndby5 systemd: arangodb3.service holdoff time over, scheduling restart.
Jan 22 20:43:25 vm-skarndby5 systemd: Starting ArangoDB database server...
Jan 22 20:43:26 vm-skarndby5 systemd: Started ArangoDB database server.
...
My questions are :
Q1. This is a RocksDB engine configuration. As i learned from Arango documentation - RocksDB does allow to use DBs which are larger than the RAM. (And our DB is expected to grow till 1TB in future).
So, why do the failures happen ?
Q2. What is the proper way to avoid "Out of memory. Kill process" failures in ArangoDB with RocksDB engine ?
Q3: Are there any best practices for arangod.conf settings for a large RocksDB database ?
Q4: Are there any best practices for a Linux host hardware/software settings for a ArangoDB sized like ours ?
Best regards and looking forward your assistance,
Avi Vainshtein

Hi,
there have been more recent discussions on growing memory usage here:
https://github.com/arangodb/arangodb/issues/11577
resulting in two more configuration option being exposed with ArangoDB 3.7:
https://www.arangodb.com/2020/08/arangodb-3-7-a-big-step-forward-for-multi-model/
hence I'm closing this now.


chore(main): release 1.0.10

:robot: I have created a release beep boop
1.0.10 (2023-01-31)
Docs

contributor: contrib-readme-action has updated readme (53b52a2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/TinyWaste/releases/tag/v1.0.10 :sunflower:


daemon service does not work in windows10

Attempting to start windows service results in a long pause and the following error returned:
Error:  The service did not respond to the start or control request in a timely fashion.

See my pull request #96
Restructuring your service as i've done with the myservice example should make it work on windows.

I update to golang.org/x/sys v0.0.0-20200905004654-be1d3432aa8f , but it still desn't work.

I updated to golang.org/x/sys v0.0.0-20200905004654-be1d3432aa8f, but it still does't work.

Updating the dependency was just a side-effect, not at all necessary. The important thing is to restructure your code as i have done in my pull request.
If you clink on the PR link (#97) and then on "Files changed" you'll see what modifications i have done to examples/myservice.go to make it work.

@filippo-veneri I just looked at the code on your PR and I couldn't help but wonder why and how it "works"? Unless I missed it, I don't see any place where you are calling (daemon.Daemon).Start(). You seem to have implemented your own start method which does something else. At what point does it actually start the daemon (windows service). Could this explain why you are not getting this issue?
Appreciate your contribution on this :-)

This is also related to #68

It is a bit convoluted. On Windows the (daemon.Daemon).Install() interface method refers to the (daemon.windowsRecord).Install() concrete method. Assuming your service executable is named myservice.exe, this method creates the windows service using the path of myservice.exe as the service executable.
When the windows service is started, myservice.exe is executed without parameters, resulting in service.Run(mysvc) (defined in myservice.go) being called, which, in this case, corresponds to (daemon.windowsRecord).Run(e Executable). This method then calls svc.Run(windows.name, &serviceHandler{executable: e}) which in turn calls (daemon.serviceHandler).Execute(...), which is the main loop of the windows service.
In response to the windows service state changes this loop calls the MyService.Start() and MyService.Stop() methods. Note that MyService implements the daemon.Executable interface.

Note that MyService implements the daemon.Executable interface.

@filippo-veneri is correct. The example service does not work on the master branch. Your code needs to implement the daemon.Executable interface and be passed into daemon.Run(MyService) instead of the structure on the master branch. Look at their PR for a proper example https://github.com/takama/daemon/blob/c329ab930e326908bf8d91a622481e6d124343e4/examples/myservice.go#L131


Could not create index

Could not create index. Is your cluster ok?
TransportError(400, u'index_already_exists_exception', u'already exists')
Indices=30
documents=25
Client=3

@sharminakteranee can you share your run command?


Auto update desktop apps

Auto update desktop applications to latest version when possible.

Implemented with Tauri updater.


chore(ci): github webhook test

Description
coolify github webhook 요청을 위한 테스트용 PR 입니다.

Additional context

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines
[ ] Read the Contributing Guidelines and follow the Commit Convention
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

coolify trigger를 위해서 닫았다 열게요 :)


feat(additionnalRef): implement additionnalRef

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


No response to IPv6 unicast reachability check NSs

Our BNG under test has a feature where it tests IPoE subscriber host connectivity by using unicast IPv6 Neighbour Solicitations sent to the access interface address. If no NAs are received the BNG declares host connectivity is lost and tears down the IPoE session. These checks can be sent periodically or be event triggered.
In our use-case, each bngblaster session receives an IA_PD via DHCPv6 and has only a locally generated fe80::/10 link local address for the access interface between it and the BNG. No DHCPv6 IA_NA is assigned.
IPv6 NSs sent from the gateway address to the network interface are responded to successfully by bngblaster. However, NSs sent from the BNG to the bngblaster access interfaces (such as the packets fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 in the output below) appear in the pcap generated by bngblaster but are not responded to and the BNG brings the session down.  The corresponding IPv4 host connectivity check which is an ARP request to the access interface address (assigned by DHCPv4) works fine.  We are running bngblaster 0.7.8.
Expected behaviour is an IPv6 NA reply is sent by bngblaster from a source of an address assigned to the access interface from which the NS was received and with a destination of the source address of the received NS.
$ bngblaster -v
Version: DEV
Compiler: GNU (9.4.0)
GIT:
  REF: main
  SHA: 0abdeaf2eed9fc3b78b90c2dadf92548dfd7dccf
IO Modes: packet_mmap_raw (default), packet_mmap, raw

Jun 19 19:29:21.047988 Resolve network interfaces
Jun 19 19:29:21.048162 All network interfaces resolved
Jun 19 19:29:21.075023 IPv6 (ID: 1) DHCPv6 IA_PD prefix 2001:db8:3400:1800::/56
Jun 19 19:29:26.057986 IPv4 (ID: 1) address <IP_ADDRESS>
Jun 19 19:29:26.060320 ALL SESSIONS ESTABLISHED

$ tshark -r ipv6-no-na.pcap icmpv6
    2   0.000000 2001:db8:2016::2 → ff02::1:ff00:1 ICMPv6 86 Neighbor Solicitation for 2001:db8:2016::1 from 00:0c:29:de:e3:ef
    4   0.001257 2001:db8:2016::1 → 2001:db8:2016::2 ICMPv6 86 Neighbor Advertisement 2001:db8:2016::1 (rtr, sol, ovr) is at f0:0d:be:ef:5a:ba
    8   1.001388 2001:db8:2016::2 → ff02::1:ff00:1 ICMPv6 86 Neighbor Solicitation for 2001:db8:2016::1 from 00:0c:29:de:e3:ef
   10   1.002855 2001:db8:2016::1 → 2001:db8:2016::2 ICMPv6 86 Neighbor Advertisement 2001:db8:2016::1 (rtr, sol, ovr) is at f0:0d:be:ef:5a:ba
   12   1.028243 fe80::ffff:ffff:ff00:1 → ff02::2      ICMPv6 70 Router Solicitation
   13   1.119500 fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 ICMPv6 82 Router Advertisement from f0:0d:be:ef:5b:32
   15   5.058367 2001:db8:2016::1 → 2001:db8:2016::2 ICMPv6 86 Neighbor Solicitation for 2001:db8:2016::2 from f0:0d:be:ef:5a:ba
   16   5.059570 2001:db8:2016::2 → 2001:db8:2016::1 ICMPv6 86 Neighbor Advertisement 2001:db8:2016::2 (sol, ovr) is at 00:0c:29:de:e3:ef
   26  60.659329 fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 ICMPv6 90 Neighbor Solicitation for fe80::ffff:ffff:ff00:1 from f0:0d:be:ef:5b:32
   31  90.659055 fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 ICMPv6 90 Neighbor Solicitation for fe80::ffff:ffff:ff00:1 from f0:0d:be:ef:5b:32
   34 120.658560 fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 ICMPv6 90 Neighbor Solicitation for fe80::ffff:ffff:ff00:1 from f0:0d:be:ef:5b:32
   37 150.658771 fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 ICMPv6 90 Neighbor Solicitation for fe80::ffff:ffff:ff00:1 from f0:0d:be:ef:5b:32

IPv4 connectivity check:
   86 416.664069 f0:0d:be:ef:5b:32 → 02:00:00:00:00:01 ARP 60 Who has <IP_ADDRESS>? Tell <IP_ADDRESS>
   87 416.664144 02:00:00:00:00:01 → f0:0d:be:ef:5b:32 ARP 50 <IP_ADDRESS> is at 02:00:00:00:00:01

Config:
{
    "interfaces": {
        "qdisc-bypass": true,
        "io-mode": "packet_mmap_raw",
         "network": {
         "interface": "ens160.2016",
         "address": "<IP_ADDRESS>",
         "gateway": "<IP_ADDRESS>",
        "capture-include-streams": false,
         "address-ipv6": "2001:db8:2016::2",
         "gateway-ipv6": "2001:db8:2016::1",
         "tx-interval": 1,
        "rx-interval": 1,
        "io-slots": 4096
        },
        "access": [
        {
            "interface": "ens224f1",
            "type": "ipoe",
            "qinq": "false",
            "outer-vlan-min": 2016,
            "outer-vlan-max": 2048,
            "inner-vlan-min": 2,
            "inner-vlan-max": 4094,
            "ipv4": true,
            "ipv6": true,
            "i1_start": "500",
            "i2_start": "1000",
            "vlan-mode": "1:1"
                    }
     ]
    },
    "sessions": {
        "count": 1,
        "session-time": 0,
        "max-outstanding": 16000,
        "start-rate": 1000,
        "stop-rate": 1000
    },
            "ipv4": true,
            "ipv6": true,
            "i1_start": "500",
            "i2_start": "1000",
            "vlan-mode": "1:1"
        }
     ]
    },
    "sessions": {
        "count": 1,
        "session-time": 0,
        "max-outstanding": 16000,
        "start-rate": 1000,
        "stop-rate": 1000
    },
    "ipoe": {
            "ipv4": true,
            "arp-timeout": 5,
            "arp-interval": 300,
                    "ipv6": true
    },
    "access-line": {
        "agent-remote-id": "ABCDEFGH{session-global}",
        "agent-circuit-id": "<IP_ADDRESS>/<IP_ADDRESS> eth 0:{session-global}"
    },
    "dhcp": {
        "broadcast": true,
        "enable": true
    },
    "dhcpv6": {
        "enable": true,
        "rapid-commit": true
    },
   "session-traffic": {
        "ipv4-pps": 0,
        "ipv6pd-pps": 0
    }
}

A small amount of debugging verified that the condition at
https://github.com/rtbrick/bngblaster/blob/c75a723bbe0a7bd4f96a00340b462e95144a3b8c/code/bngblaster/src/bbl_rx.c#L576
is passing when the IPv6 NSs are received.

This issues should be fixed in latest dev branch. Could you please verify!

Hi Christian,
Thank you very much for looking at this so quickly.
NAs are now being sent and accepted by the BNG for the host connectivity check.
However they are being sourced from :: rather than the access interface address to which the NS was sent?
  526  68.597866 fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 ICMPv6 90 Neighbor Solicitation for fe80::ffff:ffff:ff00:1 from f8:ba:e6:b4:5b:32
  527  68.599035           :: → fe80::167b:acff:feac:8ebf ICMPv6 94 Neighbor Advertisement fe80::ffff:ffff:ff00:1 (sol, ovr) is at 80:61:5f:11:b6:4c

Version: DEV
Compiler: GNU (9.4.0)
GIT:
REF: dev
SHA: c0f52186bfa768892db81e9051b880ff290e04ca
IO Modes: packet_mmap_raw (default), packet_mmap, raw

Thanks for you fast response, could you try again with latest dev!

That's working as expected now, thanks very much.
   14   5.021175 2001:db8:2016::1 → 2001:db8:2016::2 ICMPv6 86 Neighbor Solicitation for 2001:db8:2016::2 from f0:0d:be:ef:5a:ba
   15   5.022308 2001:db8:2016::2 → 2001:db8:2016::1 ICMPv6 86 Neighbor Advertisement 2001:db8:2016::2 (sol, ovr) is at 00:0c:29:de:e3:ef
  1276 160.564731 fe80::167b:acff:feac:8ebf → fe80::ffff:ffff:ff00:1 ICMPv6 90 Neighbor Solicitation for fe80::ffff:ffff:ff00:1 from f0:0d:be:ef:5b:32
  1277 160.565894 fe80::ffff:ffff:ff00:1 → fe80::167b:acff:feac:8ebf ICMPv6 94 Neighbor Advertisement fe80::ffff:ffff:ff00:1 (sol, ovr) is at 80:61:5f:11:b6:4c

$ bngblaster -v
Version: DEV
Compiler: GNU (9.4.0)
GIT:
 REF: dev
 SHA: 29f984ecd3580f0b47c88eecc4e501310bcc21b7


MAFA dataset has not been accessible!

The link to the MAFA dataset was generating a 404 error for the past month or so! Now that the link is not giving you a 404 error it takes you to a Chinese website (https://pan.baidu.com/s/1qiflOq65yMNNTXyt8020GA).
When I log in with the 4fz6 pass-code it asks me to download and install the latest version of the Network Disk Client. Which is also in Chinese.
Could you, after the 3rd attempt, finally fix this please?!

I could download at my end. In Issue 3 we have Google Drive link. Let me know if that works.


Suggestion: sorting by video length

UPDATE: Oct 5, 2024
This extension is no longer available in the Chrome extension store.
YouTube has revoked the Chrome extension's access to the YouTube API. YouTube audited the extension and required several changes that I am unable to, or will not, make (including changing the name of the extension and user interface to not use the word "YouTube"). I guess we'd have to call it "Playlist Sorter for That Really Popular Video Sharing Service". No thanks.
I'll leave the code here for now, in case anyone wants to run it locally, or make their own attempt with YouTube's auditors.


Support minLine attribute in the style for multilineTextInputEditText.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1518
Description
Clear and concise code change description.
Alternative(s) considered

Create new attribute for multiline edit text.
Add minLine attribute to the style so that it can be overridden in the application.

Type
Choose one: Feature
Screenshots (if applicable)
Checklist

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

@shelaghm
As an example, the minimum number of lines in the textInputEditText is customized to six; by default, it is three. Any changes made to this screen are for demonstration purposes only.

@santosh-pingle this looks good to me


Added bedrock cohere embed model requirements

Description:

Added missing requirements for bedrock cohere embed model.
Split functions to support needs for titan models.
Added options for input type with validation check.

Issues this addresses: #2060

@QuinnGT thank you so much!

Awesome, thanks for the quick review @HenryHengZJ . Once you get done dealing with the fun upgrade to the latest langchain version, I'll add in the missing params added in https://github.com/langchain-ai/langchainjs/releases/tag/0.1.28


Expose adamax and adamaxw

Issue #429

Thanks! Could you also add these symbols to the Sphinx docs?

Could you also add these symbols to the Sphinx
Done!  Please let me know if there is anything else that needs to be changed.


Add Getting Started Section  to README

README Update
Please add a getting started section to the README so folks know how to start with this project.

Adding Getting Started section in README file.


Error integration SWAG

I followed the following guide:
https://www.openappsec.io/post/how-to-deploy-open-appsec-on-a-docker-swag-linux-server
I used a virtual machine to compile the libraries, if I try to activate swag the system gives me this error:
nginx: [emerg] dlopen() "/usr/lib/nginx/modules/ngx_cp_attachment_module.so" failed (Error relocating /usr/lib/libosrc_nginx_attachment_util.so: __libc_single_threaded: symbol not found) in /etc/nginx/nginx.conf:1

Hi @nekromantik,
We are excited to introduce the integration of open-appsec WAF with Docker SWAG! You can find full information in our docs and blog
https://www.openappsec.io/post/announcing-open-appsec-waf-integration-with-docker-swag-secure-web-application-gateway
https://docs.openappsec.io/integrations/docker-swag
We'll be happy if you could test it out and let us know what you think!

I installed Docker Compose according to your instructions, but I keep getting this error:
docker "swag-attachmet" logs:
nginx: [emerg] module "/usr/lib/nginx/modules/ngx_cp_attachment_module.so" is not binary compatible in /etc/nginx/nginx.conf:1

Hi, we're on it! Thank you for reporting this!

Hi @zichichi,
A fix has been deployed, please make sure to pull the latest image to get the version with the fix.
Thank you for involvement!


poetry 1.4.1 shows WheelFileValidationError

this is related to github.com/python-poetry /poetry/issues/7691

after the issue is closed on poetry and furo, this workaround should be retrieved.

closed in #626


spatialDLPFC data download

Currently no images are being downloaded. Full resolution images were not found.
High resolution is theoretically available and could be downloaded at a later point.
closes #95

Looks OK.


sinon in npm registry broken with node 0.6.5

node.js finally removed require.paths, the sinon (and sinon-nodeunit) in the NPM registry still have it.  Git clone directly into the node_modules folder works fine, please republish.
Here's the output
uncaught undefined: Error: require.paths is removed. Use node_modules folders, or the NODE_PATH environment variable instead.
at Function. (module.js:376:11)
at Object. (/Users/shawn/Developer/node/nodeproj1/node_modules/sinon-nodeunit/deps/sinon/lib/sinon.js:208:12)

Are you having problems with Sinon or sinon-nodeunit specifically? sinon-nodeunit bundles old versions of both nodeunit and sinon, and I'm thinking of deprecating the package altogether. Sinon in npm does not have this flaw.

Yeah, I think it's sinon-nodeunit.  I thought I saw the same problem with sinon (looked like an update hadn't been published to NPM in a while), but maybe I was mistaken.
Just to be clear, the issue isn't node unit it's that node 0.6.5 has removed require.paths from the API.

I know the issue. Sinon had this issue, but it was solved long ago. I recommend you use vanilla nodeunit and Sinon for now, I don't think I'll update sinon-nodeunit anymore, especially now that buster.js is out in beta (http://busterjs.org)

Got it, thx!


Optional styles

Coverage decreased (-0.02%) to 38.923% when pulling 78d3435a9aecd930a3c86c7fcbfc61a9ac44e871 on optional_styles into b886d8a38d34c55dc17e3d3bda07729983f49fc8 on master.


Can Connect local, but not from outside network

Servus from germany, again...
i am using your ActionServer and ActionClient. Both are fully functionally, if i initialize the Server with server.Start(999) and connect the client via client.Connect("<IP_ADDRESS>", 999, TimeSpan.FromSeconds(1000));.
But if i want to connect to a specific (public) ip address outside of the clients network, it throws an exception, because it's unable to find the server. Any Ideas why? Did i miss something?

Hello,
Here are some things that might help.
Did you port forward the server? Maybe check with an online port check tool (while the server is running)
Because you are using port 999, is the server run with root (or admin) privileges? This is needed for servers which use a port < 1024, you might want to use another port.
Are you giving a public ip to the .Connect() function? This should be the ip of the server. (curl or visit api.ipify.org on the server)

Hello,
Here are some things that might help.
Did you port forward the server? Maybe check with an online port check tool (while the server is running)
Because you are using port 999, is the server run with root (or admin) privileges? This is needed for servers which use a port < 1024, you might want to use another port.
Are you giving a public ip to the .Connect() function? This should be the ip of the server. (curl or visit api.ipify.org on the server)

Thank you very much! i got it now :)

Thank you very much! i got it now :)


Suspicious Average

We have an application with 5 projects.  I installed coverlet and ran the coverage.  We currently have 100% coverage (project is just starting).  Here are the results:
+------------------------------------------------+--------+--------+--------+
| Module                                         | Line   | Branch | Method |
+------------------------------------------------+--------+--------+--------+
| Project 1                                       | 100%   | 100%   | 100%   |
+------------------------------------------------+--------+--------+--------+
| Project 2                                       | 100%   | 100%   | 100%   |
+------------------------------------------------+--------+--------+--------+
| Project 3                                       | 100%   | 100%   | 100%   |
+------------------------------------------------+--------+--------+--------+
| Project 4                                       | 100%   | 100%   | 100%   |
+------------------------------------------------+--------+--------+--------+
| Project 5                                       | 100%   | 100%   | 100%   |
+------------------------------------------------+--------+--------+--------+
+---------+--------+--------+--------+
|                 | Line       | Branch | Method |
+---------+--------+--------+--------+
| Total        | 100%     | 100%    | 100%   |
+---------+--------+--------+--------+
| Average  | 20%       | 20%      | 20%    |
+---------+--------+--------+--------+
This doesn't seem right to me.  It's 100% coverage across the board for each project, but the average is only 20% across the board.  I understand how it is getting it (100 / 5), but I don't think the average is an accurate reflection.
It seems that the average should also be 100%.

@bmccord I think this might be a side effect of how Coverlet displays results. For example, if the code contains no branches, Coverlet will display 100% instead of 0%. However, when calculating the total and average it'll use the 0 value.

@tonerdo Is that the way it should function or is this considered a bug?

I was also wondering if excluded projects had something to do with this?

Hi guys. I see this bug still opened, is this right?

Same issue for me.
+-------------------------------+------+--------+--------+
| Module                        | Line | Branch | Method |
+-------------------------------+------+--------+--------+
| GardenDynamics.Infrastructure | 100% | 100%   | 100%   |
+-------------------------------+------+--------+--------+
| GardenDynamics.Domain         | 100% | 100%   | 100%   |
+-------------------------------+------+--------+--------+
| GardenDynamics.Application    | 100% | 100%   | 100%   |
+-------------------------------+------+--------+--------+
+---------+-------------------+-------------------+-------------------+
|         | Line              | Branch            | Method            |
+---------+-------------------+-------------------+-------------------+
| Total   | 100%              | 100%              | 100%              |
+---------+-------------------+-------------------+-------------------+
| Average | 33.3333333333333% | 33.3333333333333% | 33.3333333333333% |
+---------+-------------------+-------------------+-------------------+

@bmccord I think this might be a side effect of how Coverlet displays results. For example, if the code contains no branches, Coverlet will display 100% instead of 0%. However, when calculating the total and average it'll use the 0 value.

Hi @tonerdo @MarcoRossignoli
I did a fork just to check how Average is working.
Could you take a look at: https://github.com/dlplenin/coverlet
I did minor changes to calculate avegare based on average from all projects. Im not sure if it is ok only to modify CalculateLineCoverage(Modules modules) functions.
If it is right, only a change in CoverageResult.cs is needed (just to keep same logic, because it switch-case (ThresholdStatistic.Average) was working well):
//#line: 204
//foreach (var module in Modules)
//{
//    line += summary.CalculateLineCoverage(module.Value).Percent;
//    branch += summary.CalculateBranchCoverage(module.Value).Percent;
//    method += summary.CalculateMethodCoverage(module.Value).Percent;
//}

line = summary.CalculateLineCoverage(Modules).AveragePercent;
branch = summary.CalculateBranchCoverage(Modules).AveragePercent;
method = summary.CalculateMethodCoverage(Modules).AveragePercent;
.
.
.
//if ((line / numModules) < threshold)
 if (line < threshold)

Or maybe is better just to copy current logic from CoverageResult.cs to Program.cs:
//#line:175
                //these lines **
                double line = 0;
                double branch = 0;
                double method = 0;

                foreach (var _module in result.Modules)
                {
                    var linePercent = summary.CalculateLineCoverage(_module.Value).Percent;
                    var branchPercent = summary.CalculateBranchCoverage(_module.Value).Percent;
                    var methodPercent = summary.CalculateMethodCoverage(_module.Value).Percent;

                    //and these***
                    line += linePercent;
                    branch += branchPercent;
                    method += methodPercent;

                    coverageTable.AddRow(Path.GetFileNameWithoutExtension(_module.Key), $"{linePercent.ToString("0.00")}%", $"{branchPercent.ToString("0.00")}%", $"{methodPercent.ToString("0.00")}%");
                }

Thanks!

Diego feel free to open a PR with your changes we'll discuss solutions directly there otherwise there is a risk to fill issue thread and will be less clear what is the problem.
Thanks for your effort here!


Inserting CSS

Advanced JS
Some JS modders insert CSS via JS.
There is already an example of this in “Installing Mods.”
I can do this one.

@tiosgz How do I add myself as the assignee?

Advanced JS

I would consider it more ‘tips & tricks’.
Don’t forget to include the upsides & downsides.

How do I add myself as the assignee?

You don’t.

upsides & downsides.

Do you have any to share now, so I can put them in the first version?

+ It’s all in one file
- The CSS is harder to customize
- It can make the mod harder to orient in

👍
in addition, the code is a bit harder to write and must be minified.

must be minified.

Who said that?

Doesn’t it need to be in one line, unless you use special escape characters?

The backtick string is nothing special. You just have to be careful about chars that have special meaning, but that’s for all forms (another downside)

I haven’t done this before, but isn’t a backtick required on every line?

Nope.
const css = `
  body {
    background: blue;
  }
`; // => "\n  body {\n    background: blue;\n  }\n"
`${4 + 2}`; // => "6"
`\${4 + 2}`; // => "${4 + 2}"

Oh, good. I guess I’ve never had to do this before.
I can add this link to the guide:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals
I’m thinking that using the template expression should not be encouraged over CSS variables, what do you think?
Or I guess the CSS variable could be controlled by an expression, that is fine.

I can add this link to the guide:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals

I don’t see why 🤷

I’m thinking that using the template expression should not be encouraged over CSS variables, what do you think?
Or I guess the CSS variable could be controlled by an expression, that is fine. Like,
:root {
      —var : ${var}
}

Depends on the situation. Sometimes CSS vars are even the only possible way. If you’re gonna add some config, --var: ${config_option} is good; otherwise you can just keep it only in CSS. Also, --var, not -var.

--var, not -var.

I’m on mobile, it autocorrects everything.

I can add this link to the guide:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals

I don’t see why 🤷

Well, so the user can see more about this kind of variable.

Well, so the user can see more about this kind of variable.

Do as you like.

Come on code3z. You can’t write guides concerning Javascript knowing nothing about the language. What’s the point when you need every little piece of information from someone else? I also don’t see it as useful writing a section about inserting CSS in Javascript. That’s basic knowledge (which shouldn’t be part of a modding guide) and you don’t even need it when modding Vivaldi, because CSS is really easy to run native through experiments. Requesting closure of the issue. I’m gonna look at the other open issues too when I have time. Time shouldn’t be wasted on information that’s available form professional sources.

Time shouldn’t be wasted on information that’s available form professional sources.

There are definitely going to be things in the guide that are available from other places, but they don't address modding specifically and aren't in one place.
The CSS-in-JS is meant for small amounts of CSS where asking people to install two files doesn't make sense.

- It can make the mod harder to orient in

What do you mean by this? Harder to read / debug the code?

I would consider it more ‘tips & tricks’.

So what folder structure?


Add Resend

What is the vendor's name?
Resend
What is the vendor's pricing site?
https://resend.com/pricing
What is the base pricing? Use the lowest tier that looks sane for a small business customer, not free or personal tiers.
$20 per user/month
What is the minimum pricing for SSO support?
"Custom"
Does this pricing info come from a quote or other non-public source?
no
Are there any caveats we should list in the footnotes?
no

Hey @neilberkman ,
as this repository was not maintained for a long time, I decided to start a new initiative under https://ssotax.org/. The data is already way more up2date than in this repository! Moreover I integrated feedback from the community like a Friends of SSO page.
Feel free to contribute there. I will merge changes quickly, I promise!


Use newer handsoap version to fix bugs in VMwareWebService

Require the newer version of handsoap to fix two issues in VMwareWebService:
Memory from XML response payloads not being freed: https://github.com/ManageIQ/handsoap/pull/6
XML Documents with excesive depth cause XML parsing to fail: https://github.com/ManageIQ/handsoap/pull/5

cc @Fryguy

Euwe Backport details:
$ git log
commit 347ed0139a67ce0c893e770568f0e436ec072753
Author: Jason Frey<EMAIL_ADDRESS>Date:   Thu Oct 6 17:26:09 2016 -0400

    Merge pull request #11756 from agrare/vmware/infra/use_new_handsoap

    Use newer handsoap version to fix bugs in VMwareWebService
    (cherry picked from commit c46ce5e3fd36497e5aff34e364924c19f0f01857)


Texter with WaterdogPE

Describe the bug
Doesn't work on WaterdogPE, kicks the player without any errors on PocketMine-MP console but has errors on the WaterdogPE console logs
To Reproduce
Steps to reproduce the behavior:

Go to a WaterdogPE server with PocketMine-MP downstream that has the Texter plugin
Make a floating text

Expected behavior
To work
Screenshots

Server (please complete the following information):

OS: Linux
PMMP: 5.4.0
Plugins: AntiServerStop, BetterBan, Chat4Staff, DummyItemsBlocks, EasyEdit, EpicCustomAlerts, Fireworks, FormAPI, FormImagesFix, Hacks, HumanNPC, HypixelBow, MOTDShuffle, MultiWorld, RankSystem, ScoreHud, TransferAPI, theSpawn, Texter

Hi @ClydeplayzYT590,
What is the version of WaterdogPE?

v2.0.0

@fuyutsuki the issue should be fixed in 431d69c
it work's with and without WaterdogPE


Review ATAG2.0 guidelines

While not a high priority, we definitely should be aware of the ATAG 2.0 guidelines and ensure that Symphony does it's best to adhere to them wherever possible. I'm not sure of the current state of Symphony against such guidelines (I imagine we are doing better than most), but it'd be worthwhile reviewing.
As a side note, here in a Australia it will be required that all government websites adhere to WCAG 2.0 standards by the end of 2012 (Level A (Single A) by 31 December 2012, Level AA (Double A) by 31 December 2014) and we have noticed in a lot of tenders the requirement for authoring tools to comply with this as well.

During my talk in Cologne I mentioned accessibility as an issue and suggested to tackle it in Symphony 2.4. While I was implicitly reffering to WCAG, I do understand that ATAG guidelines are important too for Symphony. I'd like to be part of an accessibility review and I'd be glad to help solve some of the emerging issues, so count me in for this task.

I think they are very much related, one of the principles of ATAG is for the app to support WCAG, so it's almost two birds with one stone.
ATAG focuses on two principles:

Making the authoring tool interface accessible
Support the production of accessible content

The first is the core's job (to a degree), the second is more a of global push through all extensions I feel. One of the more interesting things I noticed while skimming the document was the reference of attempting to adhere to platform accessibility guidelines, which is interesting and perhaps relevant considering the UI work occurring now?
While we are talking about accessibility, we should look at using WAI ARIA as well, which from what I know (limited knowledge at this stage) can help screenreaders understand the page semantics more by the use of various role attributes.
All very interesting stuff, even if it's relatively behind the scenes and a usual user wouldn't know the difference.

While we are talking about accessibility, we should look at using WAI ARIA as well, which from what I know (limited knowledge at this stage) can help screenreaders understand the page semantics more by the use of various role attributes.

This page might be useful: http://www.html5accessibility.com/tests/landmarks.html


Race condition in async removal after destroy

With enqueue: true, an after_destroy callback is added which enqueues a MeiliSearch::Rails::MSJob intended to remove the destroyed record from the Meilisearch index:
https://github.com/meilisearch/meilisearch-rails/blob/b9b5efb219c8926551da1caf7566e8d1b2071c9f/lib/meilisearch-rails.rb#L443-L444
https://github.com/meilisearch/meilisearch-rails/blob/b9b5efb219c8926551da1caf7566e8d1b2071c9f/lib/meilisearch-rails.rb#L881-L889
https://github.com/meilisearch/meilisearch-rails/blob/b9b5efb219c8926551da1caf7566e8d1b2071c9f/lib/meilisearch-rails.rb#L367-L384
Line 372 above passes the destroyed record as an argument to the job. Active Job transparently serializes the record to a Global ID on enqueue and looks up the record from the Global ID when the job is performed.
The enqueued background job may be performed after the transaction wrapping the record’s destruction has committed. In this case, Active Job won’t be able to find the record in the DB from its Global ID, and the MSJob will fail with an ActiveJob::DeserializationError.
From a quick look, it seems that the plugin’s tests don’t catch this because they only assert that a job is enqueued with the correct arguments. They might have caught it if they exercised the end-to-end destroy→enqueue→perform flow using ActiveJob::TestHelper’s perform_enqueued_jobs or similar.
Suggested fix:

Replace the after_destroy with an after_destroy_commit callback.
Update the job to take a model and record ID as arguments. (Alternatively, add a new job to avoid breaking changes to the existing background job’s signature.)
Attempt to look up the record. If it exists, index it. If it doesn't, delete it from the index.def perform(model, id)
  if record = model.unscoped.find_by(id: id)
    record.ms_index!
  else
    model.ms_remove_document_from_index_by_id!(id)
  end
end

Further reading: elasticsearch-model’s documentation includes an example Sidekiq job implementing an alternative approach (taking an operation name argument rather than using the record’s existence to decide what to do).
History:

algolia/algoliasearch-rails#75
algolia/algoliasearch-rails#359
algolia/algoliasearch-rails#369
algolia/algoliasearch-rails#422

Hi @georgeclaghorn, thanks for your full debugging and plan to fix the bug!
I want to do it, but unfortunately, I can't now. The integrations team is changing how we work, so it could take a while until we have enough time to progress in issues like this.
Although, I would love to review a PR if you're willing to contribute! ❤


Update Screenshot Example

It's needed to add package main to the screenshot example. Otherwise, we get example.go:2:1: expected 'package', found 'import'

I don't see it :)


⚠ Clare Page Photography has degraded performance

In f12dc89, Clare Page Photography (https://clarepage.clicki.digital/) experienced degraded performance:

HTTP code: 200
Response time: 10851 ms

Resolved: Clare Page Photography performance has improved in 280f799.


tapes.only doesn't execute nested tests

var tapes = require('tapes');

tapes.only('main', function (t) {
  t.is(true, true);

  t.test('nested', function (t) {
    t.is(true, false);
    t.end();
  });

  t.end();
});

TAP version 13
# main
ok 1 should be equal

1..1
# tests 1
# pass  1

# ok

@scottcorgan this is more important

@scottcorgan , ping

@scottcorgan , thoughts?

It looks like it is a bug. Would you be willing to open a PR?

I think it's a difficult bug cause you use tape.only. It really executes one test only. So I think we have to avoid of using tape.only and emulate it. Or to remove this feature.


after #365 we cannot use dynamic content inside Button component

<script>
 import { Button } from '@ubeac/svelte'
 import {onMount} from 'svelte'

 let count = 0

 onMount(() => {
   setInterval(() => count = count + 1, 1000)
 })
</script>
<Button>
 Button {count}
</Button>

Also there is an error when we unmount buttons from page

#365 was not a good Idea and we need to find a better way


After deploying a MultiCluster app, the corresponding app at the project level shows no pods

rancher/rancher:master 1/25
After deploying a multicluster app, the corresponding catalog app at the project level shows 0 pods running, and no ingress link.
To reproduce:

Deploy a multicluster app
View the catalog apps at the project level dashboard

Expected:

Apps show the correct number of pods, and any ingress links

Actual:

Apps show 0 pods running, though the workloads are present and running within the cluster.

rancher/rancher:master 1/31
Pod count is showing correctly in project apps.


[RFC] Deprecate on_before_accelerator_backend_setup hook

Proposed refactor
Deprecate on_before_accelerator_backend_setup hook in v1.6 for removal in v1.8. Redirect users to the setup hook.
Motivation
I only see two external uses of this hook and 3 internal uses in https://www.grep.app/search?q=on_before_accelerator_backend_setup
Users can implement the setup hook instead.
Additional context
This hook was originally added in #5377
cc @justusschock @awaelchli @akihironitta @rohitgr7 @tchaton @carmocca @borda @ninginthecloud @daniellepintz

+1 - this hook is also confusing following the strategy refactor, since the Accelerator no longer directly controls the backend setup

This hook was introduced to enable freezing model weights prior to creating the optimizers.
Now that accelerator has been refactored, I believe setup would be a better replacement and we could depreciate this and move the base fine-tuning callback to rely on setup.

I would like to work on this!.

@daniellepintz can you please point me to examples/steps on how to achieve this?.

I agree with deprecating this, the hook is really specific. Using it doesn't paint a good picture design-wise so I think it'll be good to clean up, now that it's not necessary.

Thanks @krishnakalyan3! Please see https://github.com/PyTorchLightning/pytorch-lightning/pull/10940 as an example. Lmk if you have any questions!

Strictly speaking, setup() is not a direct replacement/alternative for this deprecated hook.
the sequence of hook calls is this:
        self._data_connector.prepare_data()
        self._call_callback_hooks("on_before_accelerator_backend_setup")
        self.strategy.setup_environment()
        self._call_setup_hook()

Just saying.
But nobody uses this hook specifically to run right before setup_environment.


Please add anchor tags to your subheadings

Please add anchor tags to all sections of your documentation. People often want to link to a specific part of documentation which is not possible without anchor tags.
e.g. I wanted to document where I'd pulled the URL for the CRD:
https://cert-manager.io/docs/installation/kubernetes/#installing-with-helm
I wanted an anchor to:
"Option 1: installing CRDs with kubectl"
However, the best available was about a page up from that section (https://cert-manager.io/docs/installation/kubernetes/#steps). Surely these can be autogenerated for subheadings?

I also wish we had a subanchors! GitHub READMEs already have them and it is very useful:

I'm interested in implementing that, I'll give it a go.
/priority backlog
/kind documentation
/assign

Looks like this anchor that shows on headings should also work on any header tag
,
... as long as a div id is given, which is already the case for all heading tags.
It looks like the subheading you are referring to is actually using a  tag:

Option 1: installing CRDs with kubectl

A fix could be to change it to:

Option 1: installing CRDs with kubectl

which is, in Markdown:
#### Option 1: installing CRDs with `kubectl`

Would you like to try to make the change and open a PR?

/unassign

Neat. Ok I'll take a shot as soon as I get home 👍

Thank you for looking into that btw. I hadn't realised I had stumbled onto the only section that didn't have anchors on a heading like section. Just my luck :)
Also apologies for the directness of my original request. I was being lazy when I wrote it. But rereading just now, I think it sounded demanding, which was not the tone I intended.


[Resource Handling] Make sure all contents in original README are included in reference doc. @backwind1233

linked pr:https://github.com/microsoft/spring-cloud-azure/pull/795


Convert the RegexBuilder overloads to use variadic generics

This change switches all the RegexBuilder overload-based "variadics" to use proper variadic generics. As far as our tests show, this is largely compatible with the existing API, though there is one exception, detailed below. That said, I'm a bit worried about changes in output type resolution for edge cases, particularly with capture groups nested into the various structures.

The incompatibility is for composed regexes with more than 10 capture groups, which is the current limit of our overloads in RegexBuilder. Right now, when a regex with a large number of capture groups is included in a builder-syntax closure, its captures are dropped from the output type of the resulting regex. Though this is definitely a bug in the current system, fixing it represents a source break that may or may not be visible to clients.
let regexWithTooManyCaptures = #/(a)(b)(c)(d)(e)(f)(g)(h)(i)(j)(k)(l)(m)/#
let dslWithTooManyCaptures = Regex {
  Capture(OneOrMore(.word))
  ":"
  regexWithTooManyCaptures
  ":"
  TryCapture<(Substring, Int)>(OneOrMore(.word)) { Int($0) }
  #/:(\d+):/#
}

The dslWithTooManyCaptures regex currently has an output type of (Substring, Substring, Int, Substring) because the captures in regexWithTooManyCaptures are left out of the resulting output type. With this change, the output type becomes (Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Substring, Int, Substring). If a client matches and only accesses output.3, the value of the captured value will silently change with this change in API. Trying to access a capture group with a different type (e.g, output.2 is currently Int, but would change to Substring) or using the output in a typed context (e.g passing the tuple as an initializer's parameters) would result in a compilation failure.
(Note that when converted to Regex<AnyRegexOutput>, the full set of captures is available in both the current and new versions of the API.)

@swift-ci Please test

@swift-ci Please test


Remove useless code

Prototyped arrays always have default to an empty array (unless defaulting to another array explicitly)

@dunglas note that the default value for autodiscover.directories does not ensure that these prefixes exist in the final config. The default value is used only when none of the config file is configuring the node. It is not merged with the user-provided config. I don't know whether it is what you expect

Thank you!
I'll change this behavior soon to disable autodiscovery by default and use a regex like src/Bundle/*/Controller. It will prevent the bundle to try to register controllers from third-party bundles while keeping current Symfony directory structure.


Store chat messages per server within the minecraft config directory instead of using front-end.

Currently we are storing messages in the browser localstorage. This has some disadvantages

Size limit
Webchat client needs to be active for the messages to be stored.

Instead we should just store them in a similar way that chatpatches does in a json file or if we want to be fancy do something like sqlite (performance might be tricky there).

Probably do #22 first properly so we don't need to try and deal with migrations.

Probably don't want to serialize to json files on every message. I know chat patches does it at least when the menu comes up. Other events to consider are when the player leaves a server and when minecraft is shutting down.
Need to make sure we have a reliable way to detect a server name which is also relatively unique. Specifically need to consider that a local game also is a server.

Thinkling about it more, SQLite probably will be less of a headache to make performant and generally is easier to handle. The database structure can be really simple to begin with, something along the lines of
CREATE TABLE IF NOT EXISTS messages (
  id INTEGER PRIMARY KEY AUTOINCREMENT,
  timestamp BIGINT NOT NULL,
  server TEXT NOT NULL,
  message_json TEXT NOT NULL,
  minecraft_version TEXT
)

With a nice index on the timestamp and server.  ORDER BY timestamp DESC LIMIT ?
Pruning the database in some form might be a consideration. I honestly don't know if it is truly needed, some people might play on crazy busy servers where it might be needed?

You'll want an index on (server, timestamp). We'll always be filtering by server, so using that as a prefix is a given. Then we can skip to relevant time frames using the second part of the index.

You'll want an index on (server, timestamp). We'll always be filtering by server, so using that as a prefix is a given. Then we can skip to relevant time frames using the second part of the index.

Yup that is what I was thinking as well.
Pruning is still something I am not entirely sure on, if it becomes an issue for some users we can always build it in later.
Something else that needs to be considered is when we send the history to the client. I am currently thinking it should be a client initiated action based on:

a server join event
Websocket (re)connect event if it didn't already already have the history.

For the latter it also means that on websocket connection we need to send the server (if the user is on a server) to the client.

The client should request history (traditionally called scrollback). Maybe request the last 100 lines. We can later allow for more scrollback if you scroll to the top of history.

That's something for later that I also considered. For now I just want to show a fixed amount of lines to get started.

Architecting now for client-requested history could still be nice. Otherwise the server needs to understand what the client has already rendered. What if the server restarts (Minecraft is reloaded)? It shouldn't dupe the scrollback in the client once the websocket connection is re-established.

I might have not worded it clearly, or you read over it, but it will be client requested ;) Just not with scrollback for now, just a single request for the latest x amount of messages.

Some more thoughts about server context switching I want to do as part of this issue.

When leaving a server we can probably leave the history in place.
When joining a server we clear the backlog. Even if it is the same server, once history playback is in place that will repopulate anyway. Timing might be a slight issue as we already do get messages when we join the server and those might not be part of the history playback.
We need to show the server name somewhere in the interface on joining. Also add it to the page title probably so people can spot it in their tab.
On leaving a server we might want to also show that somewhere.

Also need to deal with clients connecting when already connected to a server.


Implement cupy.put and cupy.place

Solves #1690. Reference pages and tests are also added.

@okuta Thank you for your comments! I updated them.

I set the default to mode='wrap' because mode='raise' (NumPy's default) accepts an index i such that -len(values) <= i < 0. I had checked this behavior of NumPy, and a unit test also check it.

@okuta Thanks for your suggestions, and sorry for late reply.
I rebased the commits (squashed) onto the reorganized core.pyx. The test is modified to check NumPy version, with some comments and link to NumPy's commit.

Thanks.
jenkins, test this please.

Jenkins CI test (for commit 7c9e25ce9d5760300e57f200114edb90edee0669, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

LGTM!


Add libraries to VSCode

Excelent tool, gcc and make are working as expected, but in VSCode I get errors about stdio.h and relatives libraries (VSCode complains about not finding them), how can I add w64devkit to the VSCode include path?, I alreay added the include and derivates folders but looks like VSCode is missing something...

Make sure you add the correct include/. There's an empty, vestigial
include/ just inside w64devkit — I should probably remove it before
packaging so that it doesn't mislead — and the real include/ is at
"x86_64-w64-mingw32/include" (or i686-… for 32-bit). GCC can print its
sysroot which will show you this path regardless of where you unzipped
w64devkit (useful in scripts):
echo "$(gcc -print-sysroot)/include"
(I'm surprised the VSCode "C/C++ extensions" can't find it on its own
given that it can trivially interrogate GCC for its location.)


Streetpass : Multiple runs

I had used this software about 9 months ago and, at the time, was able to run multiple executions of it to get a full board of street passes (back to back).
New laptop, setting it up, and I went to run this and it only works once. Hours delay. Then it works again. Can someone confirm for me that Nintendo has done something to put in place an 8 hour (I think I read that somewhere) delay between executions?

(I see something about the 8 hour delay on the main page for this script, but it has been so consistent over the last few days, that I am wondering if that delay is now built in)

And if I had waited 10 more minutes, I would have found the answer to my problem as I dug around. My network. Not your script. Sorry to bother you.


Fix incorrect OAuth message

As the title, close #18327

#18327

Before

After
(ignore the empty application name Authorize "" in the screenshot, that part is filled by a mock 😊)

Codecov Report

Merging #18332 (71187a5) into main (212e81f) will increase coverage by 0.01%.
The diff coverage is 63.72%.

@@            Coverage Diff             @@
##             main   #18332      +/-   ##
==========================================
+ Coverage   45.74%   45.76%   +0.01%
==========================================
  Files         831      831
  Lines       92178    92222      +44
==========================================
+ Hits        42171    42206      +35
- Misses      43249    43258       +9
  Partials     6758     6758

Impacted Files
Coverage Δ

models/issue_lock.go
0.00% <0.00%> (ø)

modules/structs/repo.go
80.76% <ø> (ø)

routers/api/v1/repo/issue_comment.go
37.82% <0.00%> (ø)

services/issue/status.go
41.66% <0.00%> (ø)

services/migrations/gitea_uploader.go
27.04% <0.00%> (-0.09%)
:arrow_down:

services/pull/merge.go
48.21% <0.00%> (-0.12%)
:arrow_down:

services/repository/files/upload.go
0.00% <0.00%> (ø)

routers/web/repo/issue.go
38.91% <16.66%> (+0.02%)
:arrow_up:

models/issue.go
58.84% <30.76%> (-0.35%)
:arrow_down:

models/migrate.go
42.96% <40.00%> (-9.57%)
:arrow_down:

... and 17 more

Continue to review full report at Codecov.

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


Please, could you add the ESP32 WROOM 32D board.  Thanks

https://forum.arduino.cc/t/cant-download-code-to-esp32-wroom-32d/659376

You can use ESP32 Wroom-32UE, it's the same pinout and almost exactly the same appearance.

Thanks a lot for your response.
I Will do that.
On Sun., Dec. 22, 2024, 07:07 thelastoutpostworkshop, <
@.***> wrote:

You can use ESP32 Wroom-32UE, it's the same pinout and almost exactly the
same appearance.
—
Reply to this email directly, view it on GitHub
https://github.com/thelastoutpostworkshop/gpio_viewer/issues/145#issuecomment-2558417636,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BN4Q4QNN3LVRSRBVPPACDFL2G2MPTAVCNFSM6AAAAABUBIVGZ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJYGQYTONRTGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Hyperlinks in comments should be clickable

Detect hyperlinks as text starting http:\\ or https:\\
Underline the hyperlink to give user clue it is clickable
I think copy VS code behaviour with a clickable tooltip or ctlr-click to follow

Use the editor we use for documentation panel to every comment box. It should handle links by itself - or we are just one plugin away.

This is blocked on #9935.


Allow duplicated path

I would like to have duplicated path in my routes definitons.
The Use case: I need to navigate from state A to state B while on the page, but for the state B I do not have an URL which works on the server, hence state B cannot be reached via typing an URL into the browser, but that's ok, state B should not be reachable by URL - it is a kind of "intermediate" state.
Do you think it is a feasible idea?
My patch would: via an option: on route definition: allow duplicate path, do not change browser URL, if it keeps the same, use the one not marked via option as target state when reaching the page via typing in the URL.
Maybe: use some (generated) hash value as browser URL for the intermediate state, so my server does not bother on page realod.

...and as this is my first post here: thank you for router5 - it's great!

What you are proposing is path-less routes. I don't how feasible it is, I've not thought about ways to implement it yet.

The "longest" one will match first. "longest" means number of params / segments / characters.

I won't support this: I'm trying to limit the amount of configuration and option to get set-up with router5. a forwardTo option was added recently (see #154), which prevents some states to be reached (and instead they are "forwarded" to another route): it might work for your case.


Possible issue with circular references in Components/Schema

I'm trying to access the Pets schema from the following document - https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v3.0/petstore.yaml
But I'm not entirely sure it's working correctly. If I access the node_data on the schema object it doesn't pull in any of the data from the referenced Pet schema object.
Is there a special way of accessing this data?

Hey - I've tried opening that file here and I've found there was a bug with the x-head header. I've sorted that now though and just putting out a new release.
I wasn't able to hit the problem you seem to be mentioning though. Would you be able to post a code sample.
My attempt was:
irb(main):018:0> d = Openapi3Parser.load_url("https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v3.0/petstore.yaml")
=> Openapi3Parser::Document(openapi_version: 3.0, root_source: Openapi3Parser::Source(input: https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v3.0/petstore.yaml))
irb(main):019:0> d.components.schemas["Pets"].items.node_data
=> {"required"=>Openapi3Parser::Node::Array(#/components/schemas/Pets/items/%24ref/required), "properties"=>Openapi3Parser::Node::Map(#/components/schemas/Pets/items/%24ref/properties), "title"=>nil, "multipleOf"=>nil, "maximum"=>nil, "exclusiveMaximum"=>false, "minimum"=>nil, "exclusiveMinimum"=>false, "maxLength"=>nil, "minLength"=>0, "pattern"=>nil, "maxItems"=>nil, "minItems"=>0, "uniqueItems"=>false, "maxProperties"=>nil, "minProperties"=>0, "enum"=>nil, "type"=>nil, "allOf"=>nil, "oneOf"=>nil, "anyOf"=>nil, "not"=>nil, "items"=>nil, "additionalProperties"=>false, "description"=>nil, "format"=>nil, "default"=>nil, "nullable"=>false, "discriminator"=>nil, "readOnly"=>false, "writeOnly"=>false, "xml"=>nil, "externalDocs"=>nil, "example"=>nil, "deprecated"=>false}
irb(main):020:0> d.components.schemas["Pets"].items.required.to_a
=> ["id", "name"]

I'm going to consider this resolved now. Thanks for raising the issue, and do feel free to re-open if this is still a concern.