version is not working properly on prerelease

Describe the bug
I think version is not bump prerelease versions correctly & does not display the correct current version as well.
To Reproduce

I have started the tag with v0.2.2.
I commit a some changes that does not in the convention.
Then, I used convco to  get next prerelease:

echo "version=$(convco version --bump --prerelease rc)" >> "$GITHUB_OUTPUT"
# version  `v0.2.2-rc.1`

I tag the commit with the given new prelease tag. Also, I run version

# Current version is not correct, should be v0.2.2-rc.1
convco version # v0.2.2

I add a commit fix: test
I run the command again on step 3

# version `v0.2.3-rc.1`

I repeated step-3 & step-4 with a new commit fix: test1

# Result again is same
# version `v0.2.3-rc.1`

Also run the. version again.
# Current version is still not correct, should be v0.2.3-rc.1
convco version # v0.2.2

Expected behavior
I was excepting to bump a version as v0.2.2-rc.2, v0.2.2-rc.3 not as a new version & not bumping at all.
Also current version should change as well.
System (please complete the following information):

OS: MacOS, Ubuntu
Version: 13.4.1, 22.04

Additional context

I have to test this out.
I think there is a bug that when there is a tag v0.2.2 the bump pre-release should always be 0.2.3-rc.1 or 0.3.0-rc1.
I am not sure what should be the behavior if there are no fix/feat or breaking changes.

Is there a way to add/introduce custom config so that parse other commits used as patch/minor/major?

Additional types are not mandated by the Conventional Commits specification, and have no implicit effect in Semantic Versioning (unless they include a BREAKING CHANGE)

Not implemented as not in the spec.

Sorry to answer late, but i don't have much free time.
I see indeed that it does not do what is expected.

I think that in this case the issue is that there is no commit that increments the version by convention.
Maybe in this case, there should exit with an error, or it should bump to the next patch release?

For point 7 I cannot reproduce.
Which version of convco do you use @kaankoken ?
#!/bin/env sh
set -x
rm -rf .git
convco version
git init .
convco version
git commit --allow-empty -m 'hotfix: correct ..'
convco version
git tag v0.2.2
convco version
git commit --allow-empty -m 'chore: release'
git commit --allow-empty -m 'perf: use precompiled binary'
convco version --bump --prerelease rc
git tag "v$(convco version --bump --prerelease rc)"
convco version
git commit --allow-empty -m 'fix: test'
convco version --bump --prerelease rc
git tag "v$(convco version --bump --prerelease rc)"
git commit --allow-empty -m 'fix: test'
convco version

The last command prints 0.2.3-rc.1 as expected.

I think this has been fixed. @kaankoken can you re-try your use case ?


Is there a way to configure a LogIn With 2 Factor Auth ?

It was really nice to configure and deploy vaultwarden with your chart. Thanks a lot.
Is there a way to configure 2FA when logging in?

Found it. When logged in, in Account Settings myvault.mydomain.com/#/settings/two-factor
And for Organizations you can force that all user need to set their 2FA to access items


chore(master): release 0.5.0-alpha

:robot: I have created a release beep boop
0.5.0-alpha (2024-01-08)
Features

每日5篇 (91e791b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mafqla/yalis-blog/releases/tag/v0.5.0-alpha :sunflower:


chore(release): 2.24.0

Version bump in package.json and package-lock.json for release 2.24.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[5.5] Add more uncountable nouns

Sorry, but your list is wrong, there are quite a few words that are indeed countable.

I shouldn't make PR's late at night 😅   Thanks, @jmarcher and @GrahamCampbell. I removed those words.

@manniL This is a breaking change and should go to the master branch. The pluralizer is used for getting the table Eloquent models use.

@m1guelpf Referring to other PRs #21324, #21306, #21236, this isn't considered as a breaking change.

@manniL See discussion from https://github.com/laravel/framework/pull/21324#issuecomment-331607085

Please send to master. There have been enough potentially breaking additions for people to deal with.


Many transitions not work well

For example,
const notificationTransitions = {
  enter: variant.value.transitionEnterActiveClass,
  enterFrom: variant.value.transitionEnterFromClass,
  enterTo: variant.value.transitionEnterToClass,
  leave: variant.value.transitionLeaveActiveClass,
  leaveFrom: variant.value.transitionLeaveFromClass,
  leaveTo: variant.value.transitionLeaveToClass,
}

should be:
const notificationTransitions = {
  enterClass: variant.value.transitionEnterActiveClass,
  enterFromClass: variant.value.transitionEnterFromClass,
  enterToClass: variant.value.transitionEnterToClass,
  leaveClass: variant.value.transitionLeaveActiveClass,
  leaveFromClass: variant.value.transitionLeaveFromClass,
  leaveToClass: variant.value.transitionLeaveToClass,
}

you should check the component Transition is from '@headlessui/vue' or 'vue'

Where don't these transitions work?

Notification

This issue has now been resolved with the latest release of Nuxt UI Vue v1.0.1-beta.3.


Remove Ex 2 from Lab 21 - in the wrong Learning path and repeated in Lab 24

Module: 04
Lab/Demo: 21
Task:
Step:
Description of issue
Remove Ex 2 from Lab 21 - in the wrong Learning path and repeated in Lab 24

I have no idea how that duplication of lab existed this long.  Thanks for the pointer.  It should be fixed now.


Beta tech debt

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: # (issue, if applicable)
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[x] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[x] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Summary by Sourcery
Refactor the AuroSkeleton component to use a new utility class for component tag renaming and update the component registration process to utilize RuntimeUtils. Update CSS custom properties to align with container-related variables. Enhance documentation to reflect these changes and remove outdated sections. Add a new ESLint configuration and clean up deprecated scripts.
Enhancements:

Introduce a new utility class, AuroLibraryRuntimeUtils, to handle component tag renaming in the AuroSkeleton component.
Refactor the component registration process to use RuntimeUtils for registering custom elements, replacing the previous registerComponent function.
Update CSS custom properties to use container-related variables instead of background-related ones for the AuroSkeleton component.

Build:

Add a new ESLint configuration file, eslint.config.mjs, to manage linting rules and configurations.

Documentation:

Remove outdated documentation regarding CSS custom property fallbacks from the README.
Update documentation to reflect changes in the component registration process using RuntimeUtils.

Chores:

Remove deprecated Husky script with a warning message for future failures in version 10.0.0.
Delete unused scripts: generateDocs.mjs, postCss.mjs, and staticStyles-template.js.

:tada: This PR is included in version 3.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create .deb packages

The script "curl -sSL https://experimental.docker.com/ | sh" install almost everything to test Docker with YubiKey, except notary, which is vital for tests.

@mhalano in the interim we have notary binaries available in the releases section of this repo. We accept pull requests if you happen to be a master of debian packaging (the team here is running Arch Linux and Macs).

In case it's useful; The install script on experimental.docker.com (and get.docker.com) can be found in the docker/docker repository, here https://github.com/docker/docker/blob/master/hack/install.sh
Scripts for building the deb packages for docker-engine can also be found in the docker/docker repository, e.g. https://github.com/docker/docker/blob/master/hack/make/build-deb, https://github.com/docker/docker/blob/master/hack/make/release-deb
(And for yum/RPMs; https://github.com/docker/docker/blob/master/hack/make/build-rpm)

Hi people!
This issue is still valid? Debian/Ubuntu repositories already have a package for Notary tool.

hey @mhalano, we're not responsible for those packages. I'd like to get deb packaging defined here in the notary repo to provide a consolidated packaging experience.
The blocker has been the maintainers (myself included) don't have experience setting up deb packaging and it hasn't been high enough priority for us to get round to learning it.


feat: add parameter prettyPrint=false to REST API requests

Closes #3434.

Did you verify that all the system tests affected by this change still pass?
bigquery, datastore, debugger, language, logging, pubsub, speech, storage, trace, translate, vision.

I've tested all those libraries, with and without generated prettyPrint parameter. Although there were problems, I found no errors related to this parameter.
Some successful test cases need hours to complete, some tests are skipped due to missing configuration, some tests throw a 403 error due to missing permissions, some tests throw a 400 error (not related to prettyPrint), some tests fail due to unexpected response values.
Kokoro CI does not run system tests. Are they really used in some CI scenario?

They are run daily, though it appears they are failing.


ref(*): stepping down as maintainer

It's been an absolute pleasure and a privilege to be able
to contribute to OSM. I won't be having the necessary cycles
to contribute to the project going onwards, I think there's plenty
of better candidates to take on the maintainer's role already.
Best of luck to OSM team,
Eduard

Description:

Testing done:

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?

Is this a breaking change?

It was a pleasure having you involved @eduser25!!! Thanks for all you efforts.

@eduser25 given the nature of this PR, it seems that your Github account may have been hacked 👀


The Select component cannot use the "react-select-plus" component

Today, the Select component is using either rendering an input or rendering a react-select-plus component. This can't be. RAF allows the user to select the component they want to use. Today, for lists there are the "Lookup" and the "Select" component. We cannot arbitrarily render a "Lookup" when the user have explicitly chosen a "Select" just because the url prop is set.

Besides, React components, as far as I know, cannot return false on render.

Please do not take this. I'm taking it.

If you want URL loading functionality, use the Lookup component. The Select component renders a simple HTML input. Unless we find a way to asynchronously loading options in a HTML input, the Select will remain not compatible with loading data from URLs.


Prefix string for enum types

For many error enums you might want a common message to be printed before the individual message for each variant. A possible syntax could be:
#[derive(Debug, Error)]
#[error(prefix="Invalid syntax: ")]
enum ParseError {
    #[error("unmatched left bracket")]
    UnmatchedLeftBracket,
    #[error("unmatched right bracket")]
    UnmatchedRightBracket,
}

This would return "Invalid syntax: unmatched left bracket" for the display of the first case.

I would prefer not to build this into this library -- but it would be fine if someone wanted to maintain a fork for this.


Fix 3.6 CI build error due to cryptography

Code build and tests succeed for Python versions 3.6-3.9 locally using tox.  But running on Travis in CI generates this build error for Python 3.6. Some basic attempts at suggested fixes failed. Have disabled Python 3.6 Travis integration for now in .travis.yml.
Reference:

https://stackoverflow.com/questions/59441794/error-could-not-build-wheels-for-cryptography-which-use-pep-517-and-cannot-be-i
https://stackoverflow.com/questions/59200737/what-does-failed-building-wheel-for-cryptography-for-pyopenssl-in-digitalocean-m

Planning to cut over to GH Actions for CI builds and testing #124


hostname in cloud-init does not work

On Create Virtual Machine wizard, choose a template and tick the "Use cloud init", fill a hostname and ssh keys for the VM and create it. After the VM is up, check its hostname, it's not the hostname specified by cloud-init.
VM yaml:
apiVersion: kubevirt.io/v1alpha2
kind: VirtualMachine
metadata:
  annotations:
    description: ff
  selfLink: /apis/kubevirt.io/v1alpha2/namespaces/default/virtualmachines/ff
  resourceVersion: '694788'
  name: ff
  uid: 7058742b-d802-11e8-a4cd-fa163e0953ea
  creationTimestamp: '2018-10-25T03:02:42Z'
  generation: 1
  namespace: default
  labels:
    kubevirt-vm: vm-ff
    kubevirt.io/os: fedora27
spec:
  running: true
  template:
    metadata:
      creationTimestamp: null
      labels:
        kubevirt-vm: vm-ff
        kubevirt.io/os: fedora27
    spec:
      domain:
        cpu:
          cores: 2
        devices:
          disks:
            - disk:
                bus: virtio
              name: registrydisk
              volumeName: registryvolume
            - disk:
                bus: virtio
              name: cloudinitdisk
              volumeName: cloudinitvolume
          interfaces:
            - bridge: {}
              name: eth0
        machine:
          type: ''
        resources:
          requests:
            memory: 2G
      networks:
        - name: eth0
          pod: {}
      terminationGracePeriodSeconds: 0
      volumes:
        - name: registryvolume
          registryDisk:
            image: 'kubevirt/fedora-cloud-registry-disk-demo:latest'
        - cloudInitNoCloud:
            userData: |
              #cloud-config
              users:
                - name: root
                  ssh-authorized-keys: >-
                    ssh-rsa
                    AAAAB3NzaC1yc2EAAAADAQABAAABAQC2Xm5/nJ4bV/KmUp6fClT955enMd1N3XYth1ZrT7qqk5OcA6KWHbBU8fbONI2XwVgIMENX+DQHXroYNjqwRZTQ5trptVGIJ8CgBpvco4Vt7LpTBOqBMag75llu4a/xuQx4rKqlKed3IWePHCDzdqNyS9H3IqCOSDCNYfihvpBRnRTICHJqUkhMIbrACSPOcPM813kdcQco5htmXx9JHPnZzHxa3WloR+xBQLsZjBOoFbzC7geafUr3m3qcORGNJzWwtzQhKSnyIGh3KfejQkkPJzsUIxgWHFNOL/im10QoTDRylQLHnvgfxLbDzwaghvTT6XDb4RwVMXvuMlRiSvdB
<EMAIL_ADDRESS>              hostname: ff28
          name: cloudinitvolume
status:
  created: true
  ready: true

After the VM is up, login get:
ssh <IP_ADDRESS>

The authenticity of host '<IP_ADDRESS> (<IP_ADDRESS>)' can't be established.
ECDSA key fingerprint is SHA256:fSRuuER0LgpJZTzsP9a+l5zQJ2h0iGj46D4ATEldLms.
ECDSA key fingerprint is MD5:40:a7:a2:79:25:1c:44:05:e9:ba:9a:b7:46:44:f3:d9.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
[root@ff ~]# hostname
ff.localdomain

and this is the cloud-init part of the VM:
         Starting Login Service...
[  OK  ] Started Login Service.
[   89.719632] IPv6: eth0: IPv6 duplicate address fe80::858:aff:fe82:34 detected!
         Starting Hostname Service...
[  OK  ] Started Hostname Service.
[  OK  ] Started LSB: Bring up/down networking.
         Starting Initial cloud-init job (metadata service crawler)...
[  OK  ] Reached target Network.
[  122.634581] cloud-init[753]: Cloud-init v. 0.7.9 running 'init' at Thu, 25 Oct 2018 03:21:24 +0000. Up 115.08 seconds.
[  122.659927] cloud-init[753]: ci-info: ++++++++++++++++++++++++++++++Net device info++++++++++++++++++++++++++++++
[  122.674245] cloud-init[753]: ci-info: +--------+------+-------------+---------------+-------+-------------------+
[  122.683487] cloud-init[753]: ci-info: | Device |  Up  |   Address   |      Mask     | Scope |     Hw-Address    |
[  122.691414] cloud-init[753]: ci-info: +--------+------+-------------+---------------+-------+-------------------+
[  122.702902] cloud-init[753]: ci-info: | eth0:  | True | <IP_ADDRESS> | <IP_ADDRESS> |   .   | 0a:58:0a:82:00:34 |
[  122.715639] cloud-init[753]: ci-info: | eth0:  | True |      .      |       .       |   d   | 0a:58:0a:82:00:34 |
[  122.720532] cloud-init[753]: ci-info: |  lo:   | True |  <IP_ADDRESS>  |   <IP_ADDRESS>   |   .   |         .         |
[  122.728533] cloud-init[753]: ci-info: |  lo:   | True |      .      |       .       |   d   |         .         |
[  122.738896] cloud-init[753]: ci-info: +--------+------+-------------+---------------+-------+-------------------+
[  122.744839] cloud-init[753]: ci-info: ++++++++++++++++++++++++++++Route IPv4 info+++++++++++++++++++++++++++++
[  122.756424] cloud-init[753]: ci-info: +-------+-------------+------------+---------------+-----------+-------+
[  122.771622] cloud-init[753]: ci-info: | Route | Destination |  Gateway   |    Genmask    | Interface | Flags |
[  122.779763] cloud-init[753]: ci-info: +-------+-------------+------------+---------------+-----------+-------+
[  122.787746] cloud-init[753]: ci-info: |   0   |   <IP_ADDRESS>   | <IP_ADDRESS> |    <IP_ADDRESS>    |    eth0   |   UG  |
[  122.794809] cloud-init[753]: ci-info: |   1   |  <IP_ADDRESS> |  <IP_ADDRESS>   |  <IP_ADDRESS>  |    eth0   |   U   |
[  122.806863] cloud-init[753]: ci-info: |   2   |  <IP_ADDRESS> |  <IP_ADDRESS>   | <IP_ADDRESS> |    eth0   |   U   |
[  122.814887] cloud-init[753]: ci-info: |   3   |  <IP_ADDRESS>  |  <IP_ADDRESS>   |   <IP_ADDRESS>   |    eth0   |   U   |
[  122.829977] cloud-init[753]: ci-info: +-------+-------------+------------+---------------+-----------+-------+
[  OK  ] Started Initial cloud-init job (metadata service crawler).
         Starting OpenSSH server daemon...
[  OK  ] Reached target Cloud-config availability.
[  OK  ] Reached target Network is Online.
         Starting Apply the settings specified in cloud-config...
         Starting Permit User Sessions...
[  OK  ] Started Permit User Sessions.
[  OK  ] Started Getty on tty1.
[  OK  ] Started Serial Getty on ttyS0.
[  OK  ] Reached target Login Prompts.
[  OK  ] Started Command Scheduler.
[  OK  ] Started OpenSSH server daemon.
[  OK  ] Reached target Multi-User System.
         Starting Update UTMP about System Runlevel Changes...
[  OK  ] Started Update UTMP about System Runlevel Changes.

Fedora 27 (Cloud Edition)
Kernel 4.13.9-300.fc27.x86_64 on an x86_64 (ttyS0)

It seems that this is issue with cloud-init itself. See https://bugzilla.redhat.com/show_bug.cgi?id=1542657 which says that cloud-init 0.7.9 fails to set hostname and from the cloud-init logs you posted, we can see that 0.7.9 is used
Cloud-init v. 0.7.9 running 'init' at Thu, 25 Oct 2018 03:21:24 +0000. Up 115.08 seconds.

Good found. I'd like to test it against a newer cloud-init version, but I'm able to do it because of some other blockers. I will test it once I can.
Feel free to close the issue.

Feel free to close the issue.

nah, Id rather wait until you verify that it actually works with newer cloud init :)

Now the container image of fedora kubevirt/fedora-cloud-container-disk-demo:latest is updating to fedora 29 with cloud-init-17.1-7, test this issue with that image still see this issue.

Don't see this issue any more with kubevirt v0.13.3 and cloud-init-17.1-7.


Create zh_cn.json

Curseforge Credit: Godzmoline

I have no real way of knowing how correct this is, but putting it through google translate shows it's at least roughly correct.
I'll merge this now and it will be in the next release coming soon

@linzexin022 If you're still interested in creating chinese translations for this mod (and possibly our future mods) we've made a special translator role on our discord server with a channel to discuss needed translations for each update.
It's of course totally optional and we are grateful for your contribution either way.
Discord link: https://discord.gg/APPWa6s


Add definition for hacking (Fixes GH-206)

@MaxWofford: want to do the background image thing with this section that you suggested in hackclub/hackclub#749? I think it's a bit more fitting under our existing description of the organization.

we use it in its only most positive definition

This read funny to me.

@MaxWofford Oh, wtf... you're right. That's super awkward. I definitely meant to have that in a different order... like, "we only use it in its most positive definition". Proofreading skills not on point today. :stuck_out_tongue: I'll make a new PR.


Add customization options to plot viewer

Addresses the suggestions made by @psobolewskiPhD in https://github.com/Ikuyadeu/vscode-R/pull/620#issuecomment-846284374.
re 1.: It is now possible to set r.plot.customStyleOverwrites to a CSS file containing custom style overwrites. If specified, this file replaces html/httpgd/styleOverwrites.css. To check this, save e.g. the following css to a file and set r.plot.customStyleOverwrites to its absolute path:
.httpgd rect {
    stroke: none !important;
    fill: blue !important;
}

svg text {
    font-family: var(--vscode-editor-font-family) !important;
    color: blue !important;
}

.httpgd line, .httpgd polyline, .httpgd polygon, .httpgd path, .httpgd circle, .httpgd rect:not(:first-of-type) {
    stroke: blue !important;
}

re 2.: The command r.plot.togglePreviewPlots now cycles through mutlirow/scroll/hidden layouts. The setting r.plot.defaults.plotPreviewLayout can also be set accordingly.

I test it on macOS and the plot viewer tab is always empty and viewing in external browser works.
I observe the following errors in the WebView developer tools:

Thanks for catching that!

Thanks for the quick fix. It works well now.

I put a css file like the following
.httpgd rect {
    stroke: none !important;
    fill: none !important;
    color: blue;
}

svg text {
    font-family: var(--vscode-editor-font-family) !important;
    stroke: var(--vscode-foreground) !important;
    color: blue;
}

.httpgd line, .httpgd polyline, .httpgd polygon, .httpgd path, .httpgd circle, .httpgd rect:not(:first-of-type) {
    stroke: var(--vscode-foreground) !important;
    color: blue;
}

and use its path in r.plot.customStyleOverwrites but it seems nothing is changed. How could I check if it actually works?

For these elements, the colors are specified by the properties stroke and fill. In order to overwrite the values set inside the svg source, you need to append !important (as e.g. in the original styleOverwrites.css). The following example works for me:
.httpgd rect {
    stroke: none !important;
    fill: blue !important;
}

svg text {
    font-family: var(--vscode-editor-font-family) !important;
    color: blue !important;
}

.httpgd line, .httpgd polyline, .httpgd polygon, .httpgd path, .httpgd circle, .httpgd rect:not(:first-of-type) {
    stroke: blue !important;
}

Thanks for the example css. It works as designed.

re 2.: The command r.plot.togglePreviewPlots now cycles through mutlirow/scroll/hidden layouts. The setting r.plot.defaults.plotPreviewLayout can also be set accordingly.

The css override works perfectly for me.
I also set this setting, because I never found the thumbnails too useful.
It works, in that they are hidden, but the empty area beneath the plot is still present?

I mean with some care, it can be made smaller, but I sort of assumed it would be hidden.
Also, I thought that the paint palette would toggle to the VS Code theme, which matches my OS--dark after dark, light during the day.

I have vscode selected here:

I downgraded back to Release and it worked...

It works, in that they are hidden, but the empty area beneath the plot is still present?

That empty area is a feature, not a bug :D My intention was to mimic the scroll-behavior of text editor tabs, where you can scroll past the last line to position the thing you're looking at at the top of the screen. If you want to remove that space, you could set display: none for the div with id "placeholder" (I haven't tested this)

Also, I thought that the paint palette would toggle to the VS Code theme, which matches my OS--dark after dark, light during the day.
[...]
I have vscode selected here:

That menu is a bit misleading if you're using the CSS overwrite. The file you specify there completely replaces the normal style overwrites, which includes the color theme matching. In order to keep these, you can start by copying https://github.com/Ikuyadeu/vscode-R/blob/master/html/httpgd/styleOverwrites.css and modify that file (as you previously did in the installation directory).

Not sure I get the empty space bit, but I'll defer to your expertise 😄

That menu is a bit misleading if you're using the CSS overwrite. The file you specify there completely replaces the normal style overwrites, which includes the color theme matching. In order to keep these, you can start by copying https://github.com/Ikuyadeu/vscode-R/blob/master/html/httpgd/styleOverwrites.css and modify that file (as you previously did in the installation directory).

Hmm, I did copy that file and edit it—to get my Fira font preference and eliminate the outlines, etc. as I did before. So for svg text I still have     stroke: var(--vscode-foreground) !important;
I'll play around some more, maybe tomorrow.

Hmm, I did copy that file and edit it—to get my Fira font preference and eliminate the outlines, etc. as I did before. So for svg text I still have stroke: var(--vscode-foreground) !important;
I'll play around some more, maybe tomorrow.

I just realized that the proper way to color text is using the fill property, i.e.
svg text {
    fill: var(--vscode-foreground) !important;
}

in case that makes a difference here (the text looks way better this way than before, I always thought this was due to the dark color theme or somthing -.-). I'll make a new PR with this change, in which I also rename the config entry you mentioned above.
If you're not doing it already, make sure that the new CSS is loaded after you make changes (safest way to do this is to restart vscode, though toggling between styles should work as well). If you still cannot get it to work, please post an example of your CSS file.
If you find some CSS settings that look good for most plots, feel free to share them here. Even in case they're too "disruptive" for the default style, we can still add them as comments, to make it easier for other users to personalize the CSS file.


ARROW-9068: [C++][Dataset] Simplify partitioning interface

Format and Parse now operate on a whole path rather than on individual segments.
This will simplify building WritePlans but I've only made minimal changes there (to keep tests passing); follow up in https://issues.apache.org/jira/browse/ARROW-8382

Thanks Francois for the review! Merging on green.

Appveyor error is unrelated.


Apparent WER bug?

Love the package, thanks for all the hard work!
I came across what I think is a WER calculation bug:
import jiwer

jiwer.process_words('the cat went to the store', 'the car went to store front').wer
# 0.5

jiwer.process_words('the cat went to the store', 'the car went to green store').wer
# 0.3333333333333333

Given that WER = S + D + I / N, shouldn't the second example also have a WER of 0.5? i.e. one substitution (cat -> car), one deletion (the -> None), and one insertion (None -> green).

Thank you! Makes complete sense.


Audit the POST_TYPE constant's usage

Related to #12 - Potentially more abstract functions, opportunities for breaking certain pieces of functionality out into traits. Audit the POST_TYPE constant's usage and replace it with get_post_type().

Timber implements a non-static get_post_type() already, so we can't enforce a static method signature for it. It's too confusing to have both a Timber instance method and to then also require a static method that does mostly the same thing and is named similarly.
The compromise I've come up with is to have a protected static function _post_type() in Post that throws an exception if POST_TYPE is not defined. This allows for declaring CPTs that inherit directly from Post but in a way that doesn't necessarily require you to implement an abstract method. But if you want to use any post type-specific functionality, like custom admin columns, that method will get called internally and you will get a helpful error message telling you exactly what you need to do.
The next step is to make sure we're not referencing POST_TYPE directly anywhere it might not already be defined, and to instead call _post_type() to get the benefit of the error message.

I believe I've already taken care of all these references, but not to self to properly grep it once I'm in front of a computer again.
https://github.com/sitecrafting/conifer/search?utf8=✓&q=POST_TYPE&type=


LiveShare: No response in client when hovering the mouse pointer over an identifier.

VS Version

Steps to Reproduce

Input some codes

from decimal import *
d = Decimal(123)
d.conjugate

Hovering the mouse pointer over an identifier

Expected behavior
It displays a Quick Info tooltip.

Actural behavior
No response in client when hovering the mouse pointer over an identifier.

This issue also occurred when hovering over a call to that function.


External DHCP use with LXD bridge

Required information

Distribution: Fedora
Distribution version: 25
The output of "lxc info" or if that fails:

Kernel version:4.10.14-200.fc25.x86_64
LXC version:2.0.7
LXD version:2.13
Storage backend in use:lvm

Issue description
It seems that there is no option to use external DHCP server when creating LXD bridge.
Steps to reproduce
The goal is to use external DHCP server to assign IP addresses to my containers:
lxc network create boza0 bridge.external_interfaces=enp0s31f6
enp0s31f6 interface is connected to network that have DHCP server in it. I didn't saw any LXD option (  ) that allows bridge to take it's IP from the external DHCP.
When I run on LXD Host:
dhclient boza0
The bridge interface takes IP address from external DHCP server and containers that have network interface "attached" to above bridge and configured to automatically obtain IP addresses also got their addresses without any issue.
The question is : Can we have (if we still don't) lxd network option that allows us to use external dhcp during bridge creation/configuration?
Thanks in advance!

Sorry my bad!
There is no need to assign IP to bridge.
Bellow steps works as a charm:
lxc network create boza0 bridge.external_interfaces=enp0s31f6 ipv4.address=none ipv4.dhcp=false ipv6.address=none ipv6.dhcp=false
lxc network attach boza0 test eth1
Container "test" is able to automatically take IP address from external DHCP server!


Why is the SqlDependency object unable to detect the changes in SQL Server table?

It's been 3 days that I have been trying to achieve this but couldn't.
I am using the SqlTableDependency nuget package to get notifications if a particular table's cell gets modified. I have enabled the service broker.
I have taken every necessary measure. Upon starting the application it creates a trigger, queue, contract, message type automatically on the table and safely disposes upon closing the app but it never returns the results. The database user has Db_Owner role and everything but still not working. Please help me. I am frustrated and tired of this. It's been 3 days.
I change the values in the edit mode of the table or using an update query.
I am following this link. It has all the necessary steps and also a demo video. https://github.com/christiandelbianco/monitor-table-change-with-sqltabledependency
Code:
namespace SqlDependency
{
    class Users
    {
        public string UserName { get; set; }
        public string Password { get; set; }
    }

    class Program
    {
        const string ConnectionString = @"Data Source=(local);Initial Catalog=HimHer; Integrated Security=True";

        static void Main(string[] args)
        {
            using(var dep = new SqlTableDependency<Users>(ConnectionString) )
            {
                dep.OnChanged += Dep_OnChanged;

                dep.Start();

                Console.WriteLine("Press a key to exit");
                Console.ReadKey();

                dep.Stop();
            }

            Console.WriteLine("Waiting For Changes ..... ");
            Console.ReadLine();
        }

        private static void Dep_OnChanged(object sender, RecordChangedEventArgs<Users> e)
        {
            var changedEntity = e.Entity;

            Console.WriteLine("DML operation: " + e.ChangeType);
            Console.WriteLine("ID: " + changedEntity.UserName);
            Console.WriteLine("Name: " + changedEntity.Password);
        }
    }

}

I need a copy of your database. Can you remove all tables present in it and send me?

Sure in a few minutes

HimHer.zip

check the database above

Man, I have spent days on it. but not working

the above file is in .bak format. I am also going to attach script file.
himher.zip

Have you checked ?

May be I was not clear.
I NEED the file .dbms not the script to create it?
Why, because most of times, this database was created time ago using SQL Server 2005 or 2008 (not R2) Instance.

And by the way, have you read "Note about Compatibility Level and Database Version for tracking record changes" section?

Yes, I checked. It's 2012 version.
dbi_version	706	SQL 2012
dbi_createVersion	706	SQL 2012

Really, I am very very frustrated. It's been 4 days that I am trying but nothing happens,

I need the DB file.

alright. sharing. .MDF ?

make a .bak file (buckup)

HimHer.zip

check this.

Ok I have already shared the .bak, but sharing again

HimHer.zip

Does not  work

Because I created it in 2012. maybe that's the reason?

or add .bak with filename

HimHer.zip
check this. (I have taken the backup in 2012 version of sql server)

now?

Restored your DB. Test SqlTableDependency.
Please check the video: https://streamable.com/no9vw

sure, let me check

thanks bro but still mine's not working.
Can you do me a favour and come on teamviewer ? to check my configurations and all. I will be very grateful

ID: 579 916 281
Pwd: 9129

Done.
It was only because I haven't been using sa as a login. Only I changed my database to sa and used that in the connection string. It worked.


MySQL: remove_foreign_key() Leaves The Index Created By add_foreign_key()

Adding FKs to InnoDB tables creates a constraint and an index. On MyISAM tables only  the index will be created. When the FK is removed these indexes are left behind since remove_fk() only drops the constraint.
I've written this fix: http://github.com/sshaw/foreigner/blob/master/lib/foreigner/connection_adapters/mysql_adapter.rb
This is arguably somewhat DB specific, which I know is something you're trying to avoid. SQL Server does not behave like MySQL (at least 2000),  I don't know how Postgre handles FKs.  Though the gem does rely on the transparency of engines supporting SQL standard syntax without actually implementing the given DDL commands so maybe it's OK.
I added the option :keep_index to avoid dropping indexes that existed before the constraint was added.
I also modified the arguments so one isn't forced to add the column name to the Hash
when specifying options, the Symbol can still be used.
Since the existing test case is broken I create an ad hoc one.

I've decided not to do anything about this. You can call remove_index, and it's a rare operation.
Doing it automatically will be too magical. Passing in the option saves very little over calling remove_index.


Too many arguments in call to ipinfo.NewClient

I am using the standard example code as stated in ipinfo/go.
But I get the following errors:
I am using:
Go Version: go 1.19
Router: Gin (https://github.com/gin-gonic/gin)
Code:
client := ipinfo.NewClient(nil, nil, token)
Error:
too many arguments in call to ipinfo.NewClient
	have (nil, nil, string)
	want (*http.Client)

And
Code:
info, err := client.GetIPInfo(net.ParseIP(ip_address))
Error:
client.GetIPInfo undefined (type *ipinfo.Client has no field or method GetIPInfo)
The function ipinfo.NewClient takes 1 argument instead of 3.
Is the documentation/readme outdated?

How are you importing the package? You're using v1 of the package which had that model, you need to be using the latest version.
See https://github.com/ipinfo/cli/blob/master/go.mod#L10 for how to import, as an example.


fix incorrect dislpay text in question block

Closes #132

PR-Codex overview
This PR focuses on improving text formatting in various components to prevent overflow.
Detailed summary

Added break-words class to Question component to prevent text overflow.
Added break-words class to CategoryRow component to prevent text overflow.
Added break-words class to Disclosure.Panel component to prevent text overflow.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report
Patch coverage has no change and project coverage change: +0.16 :tada:

Comparison is base (4a163a2) 71.08% compared to head (d4fd778) 71.24%.

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #134      +/-   ##
============================================
+ Coverage     71.08%   71.24%   +0.16%
  Complexity      172      172
============================================
  Files            78       78
  Lines          1231     1252      +21
  Branches        106      110       +4
============================================
+ Hits            875      892      +17
- Misses          258      259       +1
- Partials         98      101       +3

see 3 files with indirect coverage changes

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


OpenApi annotation cannot find correct annotations

Actual behavior (the bug)
When the OpenApiDocumentation is generated using annotations (following javalin guidelines) the first OpenApi annotation of the handler-parent-class is always used
Expected behavior
The method's specific annotation should be serialized
To Reproduce
run attached project
OpenApiBug.zip
produces given output swagger:

Additional context
io.javalin.core.util.ReflectionUtil
internal fun Any.resolveMethodReference(): Method? {
    val javaMethodReference = javaClass
            .getDeclaredMethodByName(methodReferenceReflectionMethodName)
            ?.parameters?.get(0)
            ?.parameterizedType as Class<*>?
    return javaMethodReference?.declaredMethods?.get(0)
}

returns FIRST method
used in io.javalin.plugin.openapi.dsl.extractDocumentation
private fun HandlerMetaInfo.getOpenApiAnnotationFromReference(): OpenApi? {
    return try {
        handler.lambdaMethod?.getAnnotation(OpenApi::class.java)
                ?: handler.lambdaField?.getAnnotation(OpenApi::class.java)
    } catch (e: NoSuchFieldException) {
        null
    } catch (e: Error) {
        return if (e::class.qualifiedName == "kotlin.reflect.jvm.internal.KotlinReflectionInternalError") {
            Logger.getGlobal()
                    .warning("Local functions, lambdas, anonymous functions and local variables with @OpenApi annotations are currently not supported. " +
                            "The annotation of the handler for \"$httpMethod $path\" will be ignored. To fix this, move the handler into a global function.")
            null
        } else {
            throw e
        }
    }
}

also method extractDocumentationWithPathScanning is never used, in io.javalin.plugin.openapi.dsl.extractDocumentation:
fun HandlerMetaInfo.extractDocumentation(options: CreateSchemaOptions): OpenApiDocumentation {
    val documentation = if (handler is DocumentedHandler) {
        handler.documentation
    } else {
        val openApiAnnotation: OpenApi? = getOpenApiAnnotationFromReference() ?: getOpenApiAnnotationFromHandler()
        openApiAnnotation?.asOpenApiDocumentation()
                ?: extractDocumentationWithPathScanning(options)
                ?: document()
    }

    options.default?.apply(documentation)

    return documentation
}

possible implementation that would work:
in fun HandlerMetaInfo.extractDocumentation(options: CreateSchemaOptions) find class and verify OpenApi-annotation using OpenApi.path and options.handlermetaInfo.path (logger warn people that they did not implement a OpenApi.path when they haven't)
possible work-arounds: implement API - interface class to hold the annotation and path-scan (as long as your first handler-method does not have an OpenApi-annotation, the package-scanning should kick into effect)
This whole thing is a bit catch-22 since Kotlin does not support nesting Annotations inside Annotations (so the annotated class has to be Java, for the OpenApi-annotation to do stuff like @OpenApi(responses = {@OpenApiResponse(...)})

Thanks for a thorough issue @MikkelHJuul. I'm still not as familiar with this plugins as I'd like, so I'm going to ping @TobiasWalle.

Thank you for reporting the bug @MikkelHJuul. I'm going to look into it today.

@MikkelHJuul I created a PR with the fix.
I improved the logic which matches the handlers based on your recommendation. The behaviour is now the following:
If the class has only one method => This method is used, at it has to be the lambda method
If the class has multiple methods and no OpenApi annotations => Returns null, as there is no documentation in this class
If the class has multiple methods and OpenApi annotations => First try to find the method by the path information in the annotation. If there are multiple matches, include also the method information in the search, to find the only handler that matches. If nothing was found, log the following warning:
Unfortunately it is not possible to match the @OpenApi annotations to the handler in <class>. Please add the `path` and the `method` information to the annotation, so the handler can be matched.
It is not necessary to activate path scanning for this, as I have access to the methods.
I also added two tests for this case. One with conflicts in path and one without the conflicts.
Does this resolve your issue?

@TobiasWalle thanks for the pr. It looks like it should fix the issue (reading the code). Unfortunately I don't have time to recompile/manual test ATM, and maybe I won't the next few weeks (Vacation). Anything you need me to do, and at a deadline to finally resolve this issue?

@MikkelHJuul I don't think so. If @tipsy doesn't find anything, my PR should be ready to merge. I think it is sufficient if you verify it again after the next release. Enjoy your vacation!


PublicSuffix::DomainNotAllowed: not allowed according to Registry policy

> PublicSuffix.parse('zapruse.com')
PublicSuffix::DomainNotAllowed: `zapruse.com` is not allowed according to Registry policy

Not allowed? I'm stumped, so I change a character for testing purposes:
> PublicSuffix.parse('zaprusi.com')
=> #<PublicSuffix::Domain:0x00000003c2be80 @tld="com", @sld="zaprusi", @trd=nil>

Then it works fine. Is there something I'm missing? What is this about registry policy, does it explain why zapbruse.com fails and zapbrusi.com doesn't?
Thanks

Which version are you using? It works fine to me
➜  publicsuffix-ruby git:(master) rake console
2.3.0 :001 >  PublicSuffix.parse('zapruse.com')
 => #<PublicSuffix::Domain:0x007f8fbd026b48 @tld="com", @sld="zapruse", @trd=nil>


Error code: Wsl/Service/CreateInstance/CreateVm/ConfigureNetworking/HNS/E_ACCESSDENIED

Windows Version
Microsoft Windows [Version 10.0.26100.1591]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
No response
Distro Version
Ubuntu
Other Software
No response
Repro Steps
C:\Windows\System32>wsl.exe
Expected Behavior
Wsl default distro is launched.
Actual Behavior
Kan ikke konfigurere nettverk (networkingMode Nat). Hvis du vil deaktivere nettverk, angir duwsl2.networkingMode=None i C:\Users\trond.wslconfig
Error code: Wsl/Service/CreateInstance/CreateVm/ConfigureNetworking/HNS/E_ACCESSDENIED
Error message is in norwegian but transelate to "can not configure network (networkingMode Nat) you can disable networking in .wslconfig"
Similar error message on every distro i try to start.
WSL was working a week ago and no configuration changes since then.
The system has been updated with "Win 11 version 24H2 KB5041865" and a HP bios update since last time wsl was working
Diagnostic Logs
WslLogs-2024-08-30_22-04-13.zip

+1, on arm64 with Version 24H2 (Build 26100.1591)
PS C:\Users\micro> wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.61
MSRDC version: 1.2.5326
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26091.1-240325-1447.ge-release
Windows version: 10.0.26100.1591

For now, locally, I'm working around this by setting the following in my .wslconfig.
Maybe it helps someone:
[wsl2]
networkingMode=mirrored

Same issue here with the exact same versions as above, and yes networkingMode=mirrored did fix it for me, thanks for the hint.

@martin-labsky FYI, for me switching back to nat mode worked again after working on mirrored. Might be worth the attempt, as mirrored mode is still somewhat in beta.

@maxboone Switching to mirrored and back to nat did the trick for me to.
Thank you

@thastad , thanks for reporting the issue. can you please collecting networking logs using instructions at https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#collect-wsl-logs-for-networking-issues ?
The script will create a zip with name starting with "WslNetworkingLogs"

@CatalinFetoiu logs attached
As you can see from the comments the issue is no longer present after creating .wslconfig  with networkingMode=mirrored run wsl and then switch back to networkingMode=nat
WslNetworkingLogs-2024-09-06_22-42-51.zip

Thanks @dingwen07! Restarting the Host Network Service (HNS) resolves the problem (temporarily) for me too.
The error code for this issue ends with HNS/E_ACCESSDENIED. I wonder if some recent Windows security patches have tightened access policies that affect WSL networking.

Switching between networkingModes did not help for me. However, restarting the Host Network Service (thanks @dingwen07) did it for me.
Starting WSL the first time after the restarting the service logs these messages:
wsl: Failed to create virtual network with address range: '<IP_ADDRESS>/20', created new network with range: '<IP_ADDRESS>/20', Zugriff verweigert
wsl: Failed to create network endpoint with address: '<IP_ADDRESS>', assigned new address: '<IP_ADDRESS>'

Not sure if that's normal or not.

I ran into this issue after updating from Windows 11 23H2 to Windows 11 24H2. Disabling "Virtual Machine Platform" in Windows features and then re-enabling it ended up fixing this issue for me.
Interestingly enough a new virtual network adapter appeared called "vEthernet (WSL (Hyper-V Firewall))" after I did this.

I ran into this issue after updating from Windows 11 23H2 to Windows 11 24H2. Disabling "Virtual Machine Platform" in Windows features and then re-enabling it ended up fixing this issue for me.
Interestingly enough a new virtual network adapter appeared called "vEthernet (WSL (Hyper-V Firewall))" after I did this.

Thank you @Mk-rbv
I had to disable both Virtual Machine Platform and Windows Hypervisor Platform, reboot, then enable again and reboot. WSL is working again.

Another case here after updating from 23H2 to 24H2, restarting the Host Network Service did not work for me. After disabling Virtual Machine Platform, rebooting, re-enabling it, and rebooting again WSL negotiated a new address in the reserved <IP_ADDRESS>/12 block and things are back to working.
wsl: Failed to create network endpoint with address: '172.XX.XXX.XX', assigned new address: '172.XX.XXX.XXX'
Windows Hypervisor Platform doesn't need to be enabled for WSL.

Any chance the issue is related to Hyper-V? I disabled the option and the issue disappears.

And renabled Hyper-V the issue appears again

nothing is working for me to fix the ubuntu after the H24h2 upgrade, even my system restore to H23h2 ain't working to fix that issue. I need help!

Any chance the issue is related to Hyper-V? If I disable the option, the issue disappears.  And renable Hyper-V the issue appears again

Hey, i had been facing the issue with Ubuntu ever since a week *ever since i accidentally upgraded to H24h2) and your suggestion helped me with it, just:

disabling "Hyper-V" and
enabling "Virtual Machine Platform" + "Windows Hypervisor Platform"
fixed it for me.

Been dreaded for a week, you helped me! Thanks a ton!

I only had these Hyper-V features enabled for shrinking WSL VHDs when I ran into the issue

Restarting "Host Network Service" fixed it for me.

I only had these Hyper-V features enabled for shrinking WSL VHDs when I ran into the issue, so it might specifically be the Services one. It'd be good for someone who's run into it since updating to 24H2 to grab their logs before resolving it.

I've been running into it since a cumulative update this morning (KB5044284). Not sure if it's entirely what you meant, but might still prove useful.
WslNetworkingLogs-2024-10-09_08-05-33.zip

Restarting "Host Network Service" fixed it for me.

Same here

As a note from a Nix Wsl user. if anyone facing the same issue and want internet to work with it:

create .wslconfig in your windows userdir. (type cd ~ in powershell and enter)
put the code below inside the config file:

[wsl2]
networkingMode=mirrored

put the code below in your configuration.nix:

wsl.usbip.snippetIpAddress = "<IP_ADDRESS>";

install usbipd with winget install usbipd
reboot windows for good measures

I also tried all the above workarounds, none of them worked :-(
However what worked for me (for some reason): I just installed another distro (in my case Ubuntu 24.04 LTS). The installation seems to fix the issue for me.
Hope it helps any of you guys, too :-)

After upgrade to Windows 24H2 I have this issue with wsl.
wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.65
MSRDC version: 1.2.5620
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.26100.2033

wsl: Access is denied.
Error code: CreateInstance/CreateVm/ConfigureNetworking/HNS/E_ACCESSDENIED
wsl: Failed to configure network (networkingMode Nat), falling back to no networking.

cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.4 LTS"

24H2 update broke my setup as well, before that I was using the mirrored mode without problems, including ipv6 and host dns.
Shutting down wsl and restarting Host Network Service works. If the network does not come up first time doing it, I try until it works.
In optional windows features, I have
Hyper-V, fully on
-> Hyper-V Management Tools, fully on
-> Hyper-V Platform, fully on
Windows Machine Platform, fully on
Windows Hypervisor Platform, fully on
Windows Subsystem for Linux, fully on
PS C:\Users\sorus> wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.65
MSRDC version: 1.2.5620
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.26100.2033

PS C:\Users\sorus> wsl --shutdown

Restarting Host Network Service on Services.msc, first time
PS C:\Users\sorus> wsl
wsl: Access is denied.
Error code: CreateInstance/CreateVm/ConfigureNetworking/HNS/E_ACCESSDENIED
wsl: Failed to configure network (networkingMode Nat), falling back to no networking.
Welcome to Ubuntu 24.04 LTS (GNU/Linux <IP_ADDRESS>-microsoft-standard-WSL2 x86_64)

 * Documentation:  https://help.ubuntu.com
 * Management:     https://landscape.canonical.com
 * Support:        https://ubuntu.com/pro

 System information as of Sun Oct 13 00:42:53 CEST 2024

  System load: 0.24                Memory usage: 4%   Processes:       100
  Usage of /:  4.5% of 1006.85GB   Swap usage:   0%   Users logged in: 0

 * Strictly confined Kubernetes makes edge and IoT secure. Learn how MicroK8s
   just raised the bar for easy, resilient and secure K8s cluster deployment.

   https://ubuntu.com/engage/secure-kubernetes-at-the-edge

Expanded Security Maintenance for Applications is not enabled.

171 updates can be applied immediately.
To see these additional updates run: apt list --upgradable

3 additional security updates can be applied with ESM Apps.
Learn more about enabling ESM Apps service at https://ubuntu.com/esm

This message is shown once a day. To disable it please create the
/home/user/.hushlogin file.

user@host:/mnt/c/Users/sorus$ ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/32 brd <IP_ADDRESS> scope global lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
user@host:/mnt/c/Users/sorus$ exit
logout
PS C:\Users\sorus> wsl --shutdown

Restarting Host Network Service on Services.msc, second time
PS C:\Users\sorus> wsl
wsl: Failed to create network endpoint with address: '<IP_ADDRESS>', assigned new address: '<IP_ADDRESS>'
user@host:/mnt/c/Users/sorus$ cat /etc/os-release
PRETTY_NAME="Ubuntu 24.04 LTS"
NAME="Ubuntu"
VERSION_ID="24.04"
VERSION="24.04 LTS (Noble Numbat)"
VERSION_CODENAME=noble
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=noble
LOGO=ubuntu-logo

user@host:/mnt/c/Users/sorus$ cat /etc/wsl.conf
[boot]
systemd=true

[wsl2]
networkingMode=mirrored
dnsTunneling=true
autoProxy=true
guiApplications=true
user@host:/mnt/c/Users/sorus$

It helped to create a .wslconfig in the root folder of a windows user with such content:
[wsl2]
networkingMode=mirrored

wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.65
MSRDC version: 1.2.5620
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.26120.1843

Happened to me today out of the blue. Latest Windows Update broke it, specifically "2024-10 Cumulative Update for Windows 11 Version 24H2 for x64-based Systems (KB5044284)"
networkingMode=mirrored fixes it for now

I had the same issue and mirrored networkingMode was fine, but only temporary - after some network issues, it stopped working at all
I've decided to try restarting Host Network Service and after that everything works fine (without mirrored)

There are two workarounds for this:

Switch to mirrored networking or virtio proxy networking

Add one of the following to %userprofile%.\wslconfig
[wsl2]
networkingMode=mirroed
#or
networkingMode=virtioProxy

Uninstall KB5041865 while the fix to Windows is rolled out.

This issue has been resolved.


[commitment] refactored the batch commitment

This commit cleans the batch commitment test to make it clearer. It also creates types in order to make it easier to create aggregated proofs and pass them to the verify API.
I also added a simpler unit test.

rebasing

Codecov Report

Merging #144 (0cc51cf) into master (9b526a3) will decrease coverage by 3.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #144      +/-   ##
==========================================
- Coverage   98.11%   94.96%   -3.16%
==========================================
  Files          68       68
  Lines       13387    13916     +529
==========================================
+ Hits        13135    13215      +80
- Misses        252      701     +449

Impacted Files
Coverage Δ

dlog/commitment/src/commitment.rs
93.50% <100.00%> (+2.65%)
:arrow_up:

dlog/commitment/src/combine.rs
21.35% <0.00%> (-77.01%)
:arrow_down:

dlog/commitment/src/srs.rs
69.07% <0.00%> (-21.47%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix search button not doing working

Codecov Report

Merging #72 (78abc66) into prep-release (e65f32a) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff              @@
##           prep-release      #72   +/-   ##
=============================================
  Coverage         24.44%   24.44%
=============================================
  Files                82       82
  Lines              4582     4582
=============================================
  Hits               1120     1120
  Misses             3462     3462

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


New release v0.6.1

Pull Request Test Coverage Report for Build 120

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

Totals

Change from base Build 118:
0.0%

Covered Lines:
129

Relevant Lines:
133

💛  - Coveralls

Will merge, release was discussed in Goerli PR [thread] and there was no objection to the idea.


We can reduce memory usage in the Poisson solver by a factor of 2.

We don't need to store both f and ϕ. When converting the f Fourier coefficients to ϕ coefficients, we can just overwrite the f array.
This will be great because the Poisson solver requires Complex{Float64} so we free up quite a bit of memory.
We can then get rid of fCC2 in StepperTemporaryFields, move fCC1 somewhere else like PoissonSolver or Model, and I think we don't need fC1 anyways. So we get rid of the StepperTemporaryFields struct, and end up storing zero intermediate calculations!

Also, the 'factors' that are stored as 3D arrays should really be stored as 1D arrays.


Setup Datastore in Playground project using Terraform - change main.tf file

Since ShareAnyCode required Datastore, there are couple of changes in this Pull Request, related terraform changes:

Datastore was added in App Engine and database type was defined
Setup Datastore

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

LGTM (internal review)

R: @damondouglas


runtime:  Add error message when running non GOAMD64=v1 on unsupported hardware

With #45453 we are introducing GOAMD64 to build binaries that are not supported on all amd64 machines.
This can lead to an illegal instruction exception when running such a binary.
We should make best effort to report when such a binary with GOAMD64 > v1 is run on unsupported hardware.
One way to check is to add some assembly early in the runtime startup like it is done to detect MMX on 386 and print a short error message then exit the go program:
https://github.com/golang/go/blob/c7543e5db95d80205d1a0be3b4e946fac244601c/src/runtime/asm_386.s#L156
Waiting for internal/cpu to initialise to then do the minimum hardware checks in go code is likely to late as by that time an instruction that was emitted by the go compiler could have been used that already is unsupported.
/cc @mdempsky @randall77

@martisch Is this fixed now? It looks like it, but I wanted to double check. Please close if so. Thanks!


Result form: Add missing period at the end of the Result date tooltip message.

Tiniest ticket ever. Please add a period after the word "appear" at the end of the Result date tooltip.

QA Verified:
Period is there now:

Pre-Deploy Check:

[x] Non-regressed
[x] Feature/fix can be found in release branch in staging environment, functioning as expected.

If above is checked, this feature/fix is DEPLOY READY.


X does not clear the search box results on 'element catalog' - intended behavior?

From @dman777 on August 23, 2015 18:0
On https://elements.polymer-project.org/:

I type in the search box with all elements.
I clear the search box by hitting the x
Although the search box clears visually, it does not really clear
I have to hit backspace to clear out the empty search box to get all results to show on a empty search box.

I'm not sure, maybe this is the intended behavior. Logically, I would expect the search results to clear(and all elements to show on all elements) when I clear the search box by hitting the x.
Copied from original issue: Polymer/polymer#2352

I'm not able to repo this on the latest version of the catalog. It's possible this was fix with the recent updates. Please reopen if you see otherwise


Crashlog after update from 2.92 - 3.11

tl;dr:
http://pastebin.com/ixHTm8tW
after updating from 2.92 - 3.11
long story:
I'm running small server for friend. We were using 2.92, until we decided to update rftools to 3.11.
I wasn't able to play, when i logged in, client seemed to not receive map properly from server, and I was disconnected after like 10 seconds.
I've tested it in SSP. I wasn't able to create new world with 3.11 or 3.10. I was able to load world generated without rftools.
For time being, I've downgraded to 3.02, and it seems to work fine, so it's not dead-end for me.
I'll provide You with all my logs, I hope that they'll help You somehow.
http://pabluuz.pl/filesystem/

Nvm. Sorry, it's not RFtools. Looks like problem is with some other mod.


Runtime Error for loading SNAP Module , while running car.py

Hello Team,
I'm facing issues in running the car.py example, I have windows 10 x64 OS.
The error log is:  Message=DLL load failed: The specified module could not be found.
Source= StackTrace:
File "D:\R_and_D\TICC\ticc-master\papercode\snap.py", line 20, in swig_import_helper
return importlib.import_module(
I suspect the SNAP module the native dependencies is not getting resolved correctly in runtime.
I have installed the snap by following the instruction mentioned in this site: http://snap.stanford.edu/
Snap.py: SNAP for Python
Installation of Snap.py on Windows 64-bit
But still, I'm facing the issues at runtime, can you please tell me how I can resolve this.
Thanks.

Hello Team,
I also met the same thing.
Thanks.

Hello Team,
I also met the same thing.
Thanks.


On-Hold Event

Hello Pedro, and thank u for this gallery.
I need to add 'on-hold' event for each image, and i modified the js files and added this code at the image's creation:
on-hold="{{photo.onHold}}"
And in the $scope.items, for each image, i have added an onHold variable with the correspondent callback function, and all works fine, and the on-hold tag are added to each image, but when i hold over an image the event doesn't work.
Can you help me please.
PD: Forgive my english

I have the same problem, someone has some information that can help.


setHasPlugin function catch a MODULE_NOT_FOUND error when use universalImport on server

Environment: Node v12.9.1
I got a problem when I use this plugin in @babel/register:
I want to use chunkName this plugin generate, but as what universal component did, only setHasBabelPlugin has been called, can generated chunkName been used.

This picture shows that require("babel-plugin-universal-import/universalImport") is required before require("react-universal-component"), So the setHasPlugin function catch an Error: {"code":"MODULE_NOT_FOUND","requireStack":[]}.  As a result, I can't have the right chunkName been involved in.
my @babel/register config below:
plugins: [
            require.resolve('@babel/plugin-syntax-dynamic-import'),
            [require.resolve('babel-plugin-universal-import'), {
                        babelServer: true
           }],
]

Move to loadable components. I don't have any other maintainers who are willing to support this project unfortunately

Got it, thanks anyway.


[FEA] Add support for orc.stripe.row.count config

Is your feature request related to a problem? Please describe.
I hope support for the orc.stripe.row.count parameter can be added, which would facilitate testing by allowing precise control over the number of stripes generated.
The only parameter related to ORC stripes that I found is orc.stripe.size, which is not very convenient to use.

@ustcfy we currently do not honor orc.stripe.row.count or orc.stripe.size. CUDF does have some options we can expose
https://github.com/rapidsai/cudf/blob/d1bad33caef34b8fa95543c7494780f2084ee603/cpp/include/cudf/io/orc.hpp#L41-L42
https://github.com/rapidsai/cudf/blob/d1bad33caef34b8fa95543c7494780f2084ee603/cpp/include/cudf/io/orc.hpp#L889-L911
But those do not behave the same way as what the Spark ORC writer configs do.
The size limit in CUDF is for pre-compressed data sizes. Spark/ORC's size limit I believe is post compression, checked periodically.
Beyond that the RAPIDS Accelerator will split the input data into batches at an arbitrary point (targeting about 1 GiB uncompressed by default). The CUDF ORC writer will also not produce stripes that span these batches.
Because of all of those differences we decided not to expose these configs. I would really like to understand your use case so that we can produce the correct solution. I am happy to expose these configs in a non-standard way, because they are not the same. But I am not sure that is what you really want.

I want to address this issue https://github.com/NVIDIA/spark-rapids/issues/11735. I need to generate multiple stripes in the test, but with the default configuration, the stripes seem too large. I feel that exposing the stripe_size_rows setting might make writing the test easier. Do you have any suggestions?🥺

Okay, then we can expose the size and row count configs, but lets do them as rapids specific configs for now. We can then decide if we want to honor the standard ORC ones. While we are at it we should do the same for parquet.


missing checkout in detect-changes job

Thanks, however, this isn't needed as this step only looks at the changes made in the PR.
Checking the contents of the repo is not required by this step.
Closing as not required, however, feel free to share your thoughts if you think it's needed.


Fix documentation syntax

https://github.com/cebel/pyuniprot/blob/6bb0045cdd71199d0713400ad35c4bdd08f550b1/src/pyuniprot/manager/query.py#L195-L215
each of these instances where it can be multiple things can should be written as
:param str or tuple[str] name: UniProt entry name(s) , etc.
If we ever get awesome and write Python 3.6+ only code, we can use real type annotations too :)

Can you please specify a reference to this style of documentation. Because this ends up with ugly Sphinx documentation as:
or tuple[str] name (str)

Found it
https://pythonhosted.org/an_example_pypi_project/sphinx.html#function-definitions
but it's different, should be:
    :param name: UniProt entry name(s)
    :type name: str or tuple[str]

See Sphinx doc...
http://www.sphinx-doc.org/en/stable/domains.html
:type point: tuple(float, float)
:type point: tuple[float, float]

tuple can use () or []. I like more () because more intuitive for tuples
Will change query docs now to:
:param name: UniProt entry name(s)
:type name: str or tuple(str) or None


rune:  the process of Entrypoint should be compatible with runc

rune can't process entrpoint format like --entrypoint "/usr/lib/jvm/java-11-alibaba-dragonwell/jre/bin/java --spring.profiles.active=test"

Both in runc and rune entrypoint should be a binary.


OutgoingMessage.prototype._headers is deprecated

Hi Tyler,
This really is not an important thing. I have started using the plugin from its first versions and I have always got this error
(node:64995) [DEP0066] DeprecationWarning: OutgoingMessage.prototype._headers is deprecated
Maybe it could be used instead OutgoingMessage.prototype.getHeader(), but I haven't found any reference in the code, maybe you could give me some hint.
Thanks

Hi @rodrigo-arias , thanks for pointing this out. I went searching for this myself a couple months ago and couldn't find it. I believe it's actually coming from within axios (which is being used to source nodes). https://github.com/axios/axios/issues/2469

Thank you for opening this issue. We recently moved this plugin over to the gatsbyjs monorepo and are trying to clean out this issue queue. We documented this bug and will track this to see if we can get an upstream solution.


Better support for strongly typed generic commands

There is some previous discussion in #1004
Commands with concrete types (e.g., Selector<String>) are nicely supported in druid, but commands with generic parameters (e.g. Selector<Vec<T>>) are not, because the internal mechanism has no way to distinguish between Selector<Vec<i32>> and Selector<Vec<String>>.
As discussed in #1004, the best current pattern is to use Selector<Any>, and then do the run-time type-checking yourself. It would be nice to have stronger compile-time safety.

So the main question about this that comes to my mind is: what do you want generic commands for?
And by generic I mean you literally want it to be generic, one command that you can pass different types as payload.
Because so far I've only had the need for 'pseudo-generic' commands, where I did not know the specific type of the command, but really would have wanted it to be a specific one.
An example for this was the SET_MENU command, which I wanted to take a WindowDesc<YourAppState> as payload, but I did not know what YourAppState actually is. So what I wanted was not a generic command, but rather one where I can type check at runtime, and that is a job for Box.
At this point it is probably best to just use some kind of utility that can create such box commands in a more type safe way. Either through a context API as we did in Druid with ctx.new_window, or you could use a more type safe wrapper around Box, which is what I first tried to do in order to solve the NEW_WINDOW command problem, but that is not as ergonomic as using context.
So I really can not find a good use case for this. If someone can come up with a practical example, I would be happy to add some better mechanics for it, but as far as I can tell it would not be worth it.

Is this still relevant?
To be specific, are there use cases where a generic Vec<T> parameter is needed, that someone has met in real code? Otherwise, this issue can probably be closed.

I think this issue still exists, but is very low priority. There are some places where we need to do the Any trick; I think one example is with the current mechanism for creating child windows?


Change the background colour of the transactions filters

Change the background of the transactions filters (type, category and date) to '#011019' when in dark theme.

Hi, I'd like to fix it , can you assign it to me?

@A-4-Atom yes please!

@ani1609 I've changed the background colors as requested, however I also noticed that the color of dropdown menu does not look good on dark theme (see image below) plus it's hard to see the text. If you want i can fix that.

@A-4-Atom please raise a new issue for this and leave a comment on the issue. i'll assign you the issue


[6718] Filter out any other sensitive info

Context
This PR just glazes a few bits on top of what's already in place to help filter out PII/sensitive info from logs/sentry.
Changes proposed in this pull request

Filter out TRNs from Sentry
Filter out Authorisation header from logs
Reduce Sentry's noise level so it only outputs [INFO] calls in the logger

 API: Filter out PII


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In 847b32f, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 6385 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in 845b42d after 11 minutes.


chore(master): release 1.2.57

:robot: I have created a release beep boop
1.2.57 (2023-09-13)
Miscellaneous

deps: update dependency phpstan/phpstan to v1.10.34 (31399b7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-symfony-serializer/releases/tag/v1.2.57 :sunflower:


Close unused project sets in Live

Describe the Issue
We want to reduce the amount of unused AWS accounts in Live to reduce our cost going forward. There are project sets in the "projects-to-close" folder that we can remove. Additionally there is an "Unclassified" account in "tnfhhm" that are no longer in use and can be removed.
Additional Context

We want to use this opportunity to train Abi and Bruce on the process of deleting accounts as we'll be going through each of the three deletion processes in this ticket.

Acceptance Criteria

[x] Close project sets in the "projects-to-close" folder
[ ] Close the "Unclassified" account in "tnfhhm"

https://github.com/bcgov-c/aws-ecf-live-workspaces-settings-stack/pull/294


[Feature] supports action "saveAsImage"

What problem does this feature solve?
Add an action "saveAsImage" for dispatchAction so that you can write code to trigger  the saveAsImage event of an echart instance.
What does the proposed API look like?
myChart.dispatchAction({
    type: 'saveAsImage',
    // saveAsImage options
    ...
})

Can this be done by chart.getDom().toDataURL()?

You sure can use current APIs like getDataURL and getConnectedDataURL to extract the url and save it as an image, which is exactly what I'm doing in my production codes right now. But from what I see there are at least 2 inconveniences:

I have to somehow extract and maintain the option I use both in defining toolbox.feature.saveAsImage and getDataURL
I have to define my own download method, which has a great chance to have less compatibility compare to the one echarts has already defined

What I wanna do is as simply as trigger the saveAsImage action, and at first I was like "How hard it is? We already have a saveAasImage button here!" . But turns out the work I have to put into is not that simple, to be fair.

Hey, anyone still interested in this feature? If no official team members reply within 24 hours, I'm gonna close this issue and related PR.

I concur that this is needed as any external trigger noted above using getDataURL() or getConnectedDataURL() would not have "saveAsImage.excludeComponents" applied, but rather save the image as shown on screen.


Update current features

This is intentionally going to master because we want to have an updated version of the README and cause the changes in qiita-recover-jobs made possible the deblur reprocessing and job recovery after power outage.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2695   +/-   ##
=======================================
  Coverage   94.32%   94.32%
=======================================
  Files         166      166
  Lines       19747    19747
=======================================
  Hits        18626    18626
  Misses       1121     1121

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 fd47ca8...a36b4f4. Read the comment docs.


Run NPM tests in parallel

What does this PR do?
Run NPM e2e tests in parallel.
The tests are independant so no issues are expected. It highly increase execution time for no extra resource consumption. From 1h+ to 20minutes

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge -c

/merge


Add support for S3 custom endpoint

Add support for non-AWS hosted S3-like services such as Digital Ocean spaces for instance.

I did not review the code itself but I built this PR and tested locally. It works as expected! Thank you :)

I think it's a side effect not strictly related to this PR. It just reveals it. I can create a separate issue if necessary.
The alternate S3 provider I use does not support server-side encryption and although I did not set the parameter structurizr.encryption property, I got this error message when creating a review:
[http-nio-8080-exec-3] AmazonWebServicesS3ReviewDao - com.amazonaws.services.s3.model.AmazonS3Exception: Server-side encryption is not supported. (Service: Amazon S3; Status Code: 501; Error Code: NotImplemented; Request ID: tx85efe787a92a4d3c80ba4-00645400a7; S3 Extended Request ID: null; Proxy: null), S3 Extended Request ID: null
com.structurizr.onpremises.component.review.ReviewComponentException: Could not create review

My properties file
structurizr.url=http://localhost:8080
structurizr.data=aws-s3
aws-s3.endpoint=s3.fr-par.scw.cloud
aws-s3.accessKeyId=<redacted>
aws-s3.secretAccessKey=<redacted>
aws-s3.region=fr-par
aws-s3.bucketName=dehy-test-structurizr

The file is uploaded to the S3 bucket but it fails just after and the review is not created.
I did not see any specific information in documentation (like "if storage is s3, encryption is activated")
I had not problem when creating a diagram workspace and importing JSON.

Thanks ... I will merge the PR and fix the server-side encryption issue (IIRC, this can be set on the bucket if needed, when it's created).

Thanks @francois-travais!


fix: config with name 'devnet' is not present

Issue being fixed or feature implemented

mn config setup devnet fullnode was failing with:
strophy@W540:~/Code/dashevo/mn-bootstrap$ mn setup devnet fullnode -v
[STARTED] Set configuration preset
[SUCCESS] Set configuration preset [0.0s]
[STARTED] Task without title.
[STARTED] Task without title.
[FAILED] Config with name 'devnet' is not present

What was done?

add devnet preset

How Has This Been Tested?

Setting up a generic node works, but cannot set seed nodes
Breaking Changes

Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

Should we just set this up as a palinka preset instead? I have a set of commands to mn config:set all required values for palinka, but I can't figure out how to pass a properly escaped JS array from terminal.


master

add cloneReadableStream.ts with new ES6 typescript syntax for Node 14 and 15

First of all thanks for the PR, I really appreciate it.
Could you briefly explain the changes and the problems they solve?

First of all thanks for the PR, I really appreciate it.
Could you briefly explain the changes and the problems they solve?

First of all, I would like to express my gratitude to you for your wonderful package.
This is my first PR - so I apologize for any possible mistakes :)
Your package works great on Node 14LTS, so there are no problems to solve.
But: the documentation says:
https://nodejs.org/dist/latest-v14.x/docs/api/util.html#util_util_inherits_constructor_superconstructor
"Usage of util.inherits () is discouraged. Please use the ES6 class and extends keywords to get language level inheritance support."
Actually, only for this reason, I decided to use ES6 syntax and rewrite the code a little.

Closing this due to #6.
Maybe we could work on #10 instead


Quantizing lm_head for gemma (and others)

System Info
TL;DR:

Quantization for the lm_head was fake-quantized, at least with int4-awq and int8_sq configurations. Model was Gemma-2b, Gemma-7b and Llama-2-7b. How can I make it "real-quantized" to be compressed? (like weights are quantized with int4?)
Using lm_head quantization for the Gemma model caused problems with the output, which has not happened in the Llama-2 model.

Environment

CPU architecture: x86_64
CPU/Host memory size: 251GB
GPU properties: NVIDIA A100 40GB x2
TensorRT-LLM branch: main-66ca337 (TensorRT-LLM version: 0.9.0.dev2024031900)
Container used: tensorrt-llm-release
Versions:

TensorRT: 9.3.0.post12.dev1
AMMO: 0.7.4
CUDA: 12.3.0
NVIDIA driver: 550.54.15
OS: Ubuntu 22.04

Who can help?
@Tracin
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
Fake-quantized lm_head

Modify the preset in the quantize_by_ammo.py file in the package to enable lm_head quantization.

# /usr/local/lib/python3.10/dist-packages/tensorrt_llm/quantization/quantize_by_ammo.py:92

def quant_cfg_choices():
    import ammo.torch.quantization as atq
    QUANT_CFG_CHOICES = {
        "int8_sq": atq.INT8_SMOOTHQUANT_CFG,
        "fp8": atq.FP8_DEFAULT_CFG,
        "w4a8_awq": atq.W4A8_AWQ_BETA_CFG,
        "int8_wo": EMPTY_CFG,
        "int4_wo": EMPTY_CFG,
        "full_prec": EMPTY_CFG,
        "int4_awq": { # Customized
            "quant_cfg": {
                "*weight_quantizer": {"num_bits": 4, "block_sizes": {-1: 128}, "enable": True},
                "*input_quantizer": {"enable": False},
                "*lm_head*": {"enable": True},
                "*output_layer*": {"enable": False},
                "default": {"enable": False},
            },
            "algorithm": {"method": "awq_lite", "alpha_step": 0.1},
        }
    }

You can reproduce the same result for modifying the int8_sq configuration.

quantize through examples/quantization/quantize.py file.

python3 ../quantization/quantize.py --model_dir ${HF_MODEL_PATH} \
--dtype float16 --qformat int4_awq --output_dir ${UNIFIED_CKPT_PATH} --tp_size 1

I found that the lm_head.weight was not quantized. For example, in the Gemma 7B model,
cd ${UNIFIED_CKPT_PATH}
python
>>> from safetensors import safe_open
>>> f = safe_open("rank0.safetensors", framework="pt", device=0)
>>> f.get_tensor("lm_head.weight").size()
torch.Size([256000, 3072])
>>> f.get_tensor("lm_head.weight").dtype
torch.float16
>>> for k in f.keys():
...   if 'lm_head' in k:
...     print(k)
...
lm_head.activation_scaling_factor
lm_head.prequant_scaling_factor
lm_head.weight
lm_head.weights_scaling_factor

The result was same for the Gemma 2B and Llama-2 7B model.
Result of quantized lm_head

build it

trtllm-build --checkpoint_dir ${UNIFIED_CKPT_PATH} --gemm_plugin float16 \
--gpt_attention_plugin float16 --lookup_plugin float16 --max_batch_size 8 \
--max_input_len 256 --max_output_len 256 --gather_all_token_logits \
--enable_xqa enable --context_fmha enable --output_dir ${ENGINE_PATH}

Test it with summarize.py

python3 ../summarize.py --test_trt_llm --engine_dir ${ENGINE_PATH} \
--max_input_length 256 --batch_size 8 --max_ite 5 --eval_ppl \
--vocab_file ${VOCAB_FILE_PATH}

The result from the Llama-2 was acceptable.
# lm_head quantized int4-AWQ llama-2 7B result
[04/02/2024-19:11:32] [TRT-LLM] [I] TensorRT-LLM (total latency: 5.553373098373413 sec)
[04/02/2024-19:11:32] [TRT-LLM] [I] TensorRT-LLM (total output tokens: 4000)
[04/02/2024-19:11:32] [TRT-LLM] [I] TensorRT-LLM (tokens per second:<PHONE_NUMBER>085213)
[04/02/2024-19:11:32] [TRT-LLM] [I] TensorRT-LLM beam 0 result
[04/02/2024-19:11:32] [TRT-LLM] [I]   rouge1 : 8.275318677554012
[04/02/2024-19:11:32] [TRT-LLM] [I]   rouge2 : 0.04273504273504273
[04/02/2024-19:11:32] [TRT-LLM] [I]   rougeL : 6.7506633186696865
[04/02/2024-19:11:32] [TRT-LLM] [I]   rougeLsum : 7.960074361569913
[04/02/2024-19:11:32] [TRT-LLM] [I]   Per-token perplexity: 6.3687220275402066

# lm_head not quantized int4-AWQ llama-2 7B result
[04/02/2024-19:11:02] [TRT-LLM] [I] TensorRT-LLM (total latency: 5.527586936950684 sec)
[04/02/2024-19:11:02] [TRT-LLM] [I] TensorRT-LLM (total output tokens: 4000)
[04/02/2024-19:11:02] [TRT-LLM] [I] TensorRT-LLM (tokens per second:<PHONE_NUMBER>830951)
[04/02/2024-19:11:02] [TRT-LLM] [I] TensorRT-LLM beam 0 result
[04/02/2024-19:11:02] [TRT-LLM] [I]   rouge1 : 5.290267273128518
[04/02/2024-19:11:02] [TRT-LLM] [I]   rouge2 : 0.6819251087961309
[04/02/2024-19:11:02] [TRT-LLM] [I]   rougeL : 4.785798335241817
[04/02/2024-19:11:02] [TRT-LLM] [I]   rougeLsum : 5.001144738190409
[04/02/2024-19:11:02] [TRT-LLM] [I]   Per-token perplexity: 5.8846661925315855

But the result from the Gemma-7B cannot even generate a single proper token.
# lm_head quantized int4-AWQ Gemma 7B result
[04/02/2024-15:09:58] [TRT-LLM] [I] TensorRT-LLM (total latency: 5.9773688316345215 sec)
[04/02/2024-15:09:58] [TRT-LLM] [I] TensorRT-LLM (total output tokens: 4000)
[04/02/2024-15:09:58] [TRT-LLM] [I] TensorRT-LLM (tokens per second:<PHONE_NUMBER>652237)
[04/02/2024-15:09:58] [TRT-LLM] [I] TensorRT-LLM beam 0 result
[04/02/2024-15:09:58] [TRT-LLM] [I]   rouge1 : 0.0
[04/02/2024-15:09:58] [TRT-LLM] [I]   rouge2 : 0.0
[04/02/2024-15:09:58] [TRT-LLM] [I]   rougeL : 0.0
[04/02/2024-15:09:58] [TRT-LLM] [I]   rougeLsum : 0.0
[04/02/2024-15:09:58] [TRT-LLM] [I]   Per-token perplexity: inf

# lm_head not quantized int4-AWQ Gemma 7B result
[04/02/2024-15:09:23] [TRT-LLM] [I] TensorRT-LLM (total latency: 5.9766669273376465 sec)
[04/02/2024-15:09:23] [TRT-LLM] [I] TensorRT-LLM (total output tokens: 3820)
[04/02/2024-15:09:23] [TRT-LLM] [I] TensorRT-LLM (tokens per second:<PHONE_NUMBER>07679)
[04/02/2024-15:09:23] [TRT-LLM] [I] TensorRT-LLM beam 0 result
[04/02/2024-15:09:23] [TRT-LLM] [I]   rouge1 : 15.085234543827761
[04/02/2024-15:09:23] [TRT-LLM] [I]   rouge2 : 3.085591159231036
[04/02/2024-15:09:23] [TRT-LLM] [I]   rougeL : 10.889472087599273
[04/02/2024-15:09:23] [TRT-LLM] [I]   rougeLsum : 12.451584690196892
[04/02/2024-15:09:23] [TRT-LLM] [I]   Per-token perplexity: 7.5976266145706175

Expected behavior
Mentioned above
actual behavior
Mentioned above
additional notes

If I apply the quantization for the lm_head, it was fake-quantized at least with the int4-weight only-AWQ and int8-smoothquant configuration. Models were Gemma-2b, Gemma-7b, Llama-2-7b.
Using lm_head quantization for the Gemma model caused problems with the output, which has not happened in the Llama-2 model.
+) How's lm_head and output_layer are different in the quantization configuration? What does output_layer mean in here?

@dasistwo If *lm_head*: enable doesn't work, I think probably AMMO  remove the support.
@RalphMao Do we have any approaches to quantize lm_head now?

I think quantizing lm_head would give some space to the memory, especially for the small models. As far as I see the trend of models recently released is using a huge vocabulary size including llama-3. @Tracin Why was the feature removed? Was the significant accuracy drop expected?


Add scancodes for macos. Fixes #42

By using the /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/System/Library/Frameworks/Carbon.framework/Versions/A/Frameworks/HIToolbox.framework/Versions/A/Headers/Events.h file created the mapping between the virtual keycodes and the scancodes for macos.
I did only a simple test for Delete and Backspace keys.

Thanks for doing this!


Make test case slightly more robust.

Reported in rdar://33876675 , rdar://33993056.

@swift-ci test and merge

@swift-ci test and merge

I tested (Ubuntu) Linux and Darwin. I really don't know about SystemZ.

@adrian-prantl my suggestions: Just do a which check in the lit.cfg file and add a feature. If the feature isn't set, skip the test.

Then there are no worries.

I got rid of the xxd dependency by using only head -c.

@swift-ci please test and merge

@swift-ci Please test and merge


[Coverity CID :190977]Integer handling issues in /lib/os/printk.c

Static code scan issues seen in File: /lib/os/printk.c
Category: Integer handling issues
Function: _vprintk
Component: Other
CID: 190977
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

false positive, the problem is spotted because in the architecture tested, long and int has the same size.


create_region fix for 2d extractor

Codecov Report
Attention: Patch coverage is 80.00000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 76.42%. Comparing base (244d74a) to head (1768cfe).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4413      +/-   ##
==========================================
- Coverage   81.52%   76.42%   -5.10%
==========================================
  Files         109      109
  Lines       53160    53162       +2
==========================================
- Hits        43339    40630    -2709
- Misses       9821    12532    +2711


If GameObject is disabled, copied hierarchy is also disabled

Hi, thanks for your awesome tools. I have just one problem that is easy reproduceable, but really annoying:
If I copy everything over from one avatar to the other, and there are disabled GameObjects, if those GameObjects have other Objects underneath them, they get copied, but get also disabled (but they are enabled in the source hierarchy).
That makes it really annoying if you use that for logic or similar things.
I hope it's not a limitation of the Tools, and just a bug :/
Cheers

tschaerni

Sounds like a bug. I might have overlooked this when I added code to disable game objects that were disabled at the source. It used to always copy everything as enabled. I'll try to fix this in the next update

Good to hear, I'm looking forward to your bugfix, because it's getting more and more annoying the more complex my Avatars become ;)
Cheers

tschaerni

Fixed. Will be out in the next update later today. Thanks!

Thanks!
You do a great job mate (:


Installation with homebrew does not work

First of all @vitobotta great project! Love it!
Now to the issue - I did first installation on M1 Mac and everthying went well, however later I changed to another Macbook and tried to reinstall it using brew:
brew install vitobotta/tap/hetzner_k3s
But start getting an error:
foo@MacBook-Pro bin % ./hetzner-k3s
dyld[47539]: Library not loaded: /opt/homebrew/opt/gmp/lib/libgmp.10.dylib
  Referenced from: <2FE6A1FB-FEEC-3024-8771-608C1B9EA9A1> /opt/homebrew/Cellar/hetzner_k3s/1.0.8/bin/hetzner-k3s
  Reason: tried: '/opt/homebrew/opt/gmp/lib/libgmp.10.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/opt/homebrew/opt/gmp/lib/libgmp.10.dylib' (no such file), '/opt/homebrew/opt/gmp/lib/libgmp.10.dylib' (no such file), '/usr/local/lib/libgmp.10.dylib' (no such file), '/usr/lib/libgmp.10.dylib' (no such file, not in dyld cache)
zsh: abort      ./hetzner-k3s
foo@MacBook-Pro bin %

I'm on
System Version: macOS 13.1 (22C65)
      Kernel Version: Darwin 22.2.0

HI, can you run brew doctor and check if there is a problem with Homebrew?

Thanks for quick reply @vitobotta
Apparently there is some node includes that are invalid:
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected header files:
  /usr/local/include/node/cppgc/common.h
  /usr/local/include/node/js_native_api.h
  /usr/local/include/node/js_native_api_types.h
  /usr/local/include/node/libplatform/libplatform-export.h
  /usr/local/include/node/libplatform/libplatform.h
  /usr/local/include/node/libplatform/v8-tracing.h
  /usr/local/include/node/node.h
  /usr/local/include/node/node_api.h
  /usr/local/include/node/node_api_types.h
  /usr/local/include/node/node_buffer.h
  /usr/local/include/node/node_object_wrap.h
  /usr/local/include/node/node_version.h
  /usr/local/include/node/openssl/aes.h
.... and more of these

Not sure how to fix it though

In these cases IMO it's easier to completely uninstall Homebrew and do a clean install, rather than trying to fix

@zubairov did you solve? It doesn't seem an issue with my tool so can we close this? :)

FYI - works great for me, no issues to report

Thanks. I'll close for now.


Hiding UINavigationBar

Is there a way to hide the navigation based on the current VC?
For example, I retrned 4 VCs : return [firstLevelVC, secondLevelVC, thirdLevelVC, fourthLevelVC] and I want the UINavigationBar to be present on all but the "thirdLevelVC"
Tried using changedToPageIndex, but that doesn't work.

Your best option seems to be utilising navigationBarDataForPageIndex. Simply return a standard UINavigationBar for all of your VC's except the third, where you can simply set visible to false.


Drop support for Node 0.10 and 0.12

I mentioned this before, the original plan was to support the latest three releases of Node.js and to follow along with the LTS schedule: https://github.com/nodejs/LTS
At the end of the year both 0.10 and 0.12 will have reached end of maintenance, so Karma can drop them safely.
This should happen in a new major release, aka 2.0.
Things to be done for this

Update docs
Update engines matcher in package.json
Update travis and appveyor configuration to move to 4, 6, 7 being tested.

Note that 0.10's maintenance has already ended (0.12 ends December 31st).

How soon should these changes be made? Is the intention to do a release of 2.0 soon after 0.12 support ends? Will there be an interim release beforehand?

There is no urgent rush to do this, so I would wait until January for a new 2.0 release. We can still do  regular minor and patch releases in the meantime as we need them.

We are ready for this :)

The latest versions of eslint, lodash, and log4js now require Node >=4. I think it would be a good idea to update soon if possible.

When should this be scheduled for?
Should this declared Node 4+ support change also include an internal change utilizing a Node 4+ feature such as a fat arrow or template literal in order to force code to break if the user is not on Node 4+?

Fixed by PR #2825


Status menu is flipped when using with another plugin

Describe the bug
Status menu is flipped when using with CompleteTimestamps v1.6.9 from DevilBro
To Reproduce
Steps to reproduce the behavior:

Disable all plugins
Enable CollapsibleUI v8.5.0 & CompleteTimestamps v1.6.9
Click on user bottom left to show status menu
See error

Expected behavior

Screenshots

Desktop (please complete the following information):

OS: Windows 11
Discord Version: Stable
Enabled Theme(s): Dracula Officialv1.0.0by Dracula /  same without
(OPTIONAL) Enabled Plugins: No other than the 2 mentioned here.

Thank you for the report - I will look into this.

Thank you !


pre update changes

[x] Rename inside the ConfigData record the variable of "token" to "discordToken"
[x] Add field of "websiteURL" for the bot website url
[x] Inside the RootController class change the redirection to the website url

Notify command

Requests a role to get notification when a pipeline fails

[ ] Create notify command (and dont forget to save the role in the db)

[ ] Change some security issues
[ ] Rename the "setup" command to "init" command
[ ] Rename the class "SetupCommand" to "InitCommand"

Unset command

[ ] Rename the command to "remove"
[ ] Rename the class to "RemoveCommand"

Status command

[ ] Create a status command that will replay with all the channels that the linked to the bot and the secret token

[ ] Change the description of the commands

Chnage the "remove" command to a two different commands

[ ] removeByChannel command
[ ] removeByToken command

[ ] Create command called "tokens" and the bot will replay with all the registered tokens inside the db


Mitigate CVE-2020-11996

Description
MItigate CVE-2020-11996. Bumps Apache Tomcat to 9.0.36 (see mitigation)

Change can be reverted with https://github.com/corona-warn-app/cwa-server/pull/574