Intent to package: Snaps are universal Linux packages

Dear maintainer(s),
Snaps are universal Linux packages that can be installed and run on many GNU+Linux distributions with ease.  I've been following the Snapcrafters instructions and have built a working snap on the Snap Store, you may try it by running the following commands after installing snapd:
sudo snap install --channel=beta universal-update-utility
universal-update-utility -H
# sudo snap alias universal-update-utility uuu
# uuu -H

Update: Connecting to the raw-usb interface is no longer required as the store auto-connection is now active.
I'm also looking forward to working with you to make the packaging work upstream (if you're interested).  You may want to check out the current packaging recipe, the documentation, registering a developer account on the Snap Store, and get in touch with us to transfer the work under your control.
Thanks for your time and attention.

Thank you very much.  It think it may not be too late.  I just realize it is quite good features.
I just start learn snap.

I can prepare a PR to add the packaging recipe to the repo and initiate the ownership transfer process.

Thanks. I requested the new name "uuu" and wait for review.  universal-update-utility is too long.

I requested the new name "uuu" and wait for review. universal-update-utility is too long.

I've previously attempted to register 3-letter snap names for other snaps but was since been retracted with the following staff recommendation:

We're trying to limit the amount of 3-letter snap names because they are not very descriptive unless they're very high-profile software (git, vim). Would you consider using a longer more descriptive name for the snap? as you know we can still assign an alias so a "bfg" command will be available once installed, but naming the snap "big-fantastic-gun" or something more descriptive is preferred.

I assume uuu falls under this case and does not attempt it in the first place.

Thank you for your comments.

Feedback:
Three-letter and shorter snap names are too generic and nondescript and don't provide a good idea of what the snap actually is or does. For that reason, we prefer and suggest you pick a more descriptive name for your snap (universal-update-utility-uuu for example). Keep in mind that even if the snap has one of those names, a command called "uuu" can still be provided by the snap - this needs creating an "alias" but it's entirely feasible to do so.
Let me know if some of the suggested names are OK with you or if you want to pick a new one; in any case, you can simply register the new name and it should be automatically approved - only 3-letter or shorter names (well and a few other exceptions) require manual name review/approval."
Maybe we, can use universal-update-utility-uuu, or uuu-universal-update-utility, so user can use keywords uuu to search it.

Maybe we, can use universal-update-utility-uuu, or uuu-universal-update-utility, so user can use keywords uuu to search it

I've added search keywords to the universal-update-utility snap's listing page and can verify that the snap is now searchable from the store website and the CLI, does that work for you?

brlin@brlin-musicha-mz-530:~$ snap find uuu
Name                      Version          Publisher  Notes  Summary
universal-update-utility  1.4.95+pkg-021c  brlin      -      Freescale/NXP I.MX Chip image deploy tools
brlin@brlin-musicha-mz-530:~$ snap find nxp
Name                                    Version          Publisher    Notes  Summary
zephyr-crosstool-xtensa-nxp-imx-adsp    <IP_ADDRESS>         stephanosio  -      Zephyr Xtensa NXP i.MX ADSP Cross Compiler Toolchain
zephyr-crosstool-xtensa-nxp-imx8m-adsp  <IP_ADDRESS>         stephanosio  -      Zephyr Xtensa NXP i.MX 8M ADSP Cross Compiler Toolchain
universal-update-utility                1.4.95+pkg-021c  brlin        -      Freescale/NXP I.MX Chip image deploy tools

Also, I would like to request to add your Ubuntu One account e-mail address to the universal-update-utility snap's collaborator list so that you can access the current snap's backstage.

I also fixed the build failure and update the current snap to 1.4.95

My account is<EMAIL_ADDRESS>

Collaborator invited, please check your mail.

I've also added your GitHub account to the source repo's collaborator list, you may push code changes and trigger the automated snap building.

I can push code into your repo.
How to trigger snap build if change source in uuu repo?

How to trigger snap build if change source in uuu repo?

This is currently not possible if the snap recipe is in another repo, we can work around it by:

Merging the snap recipe to the uuu repo.
Transfer the snap's ownership to your developer account (probably not required with contributor access)
Setup snap building with the uuu repo at https://snapcraft.io/universal-update-utility/builds.

404 for link https://snapcraft.io/universal-update-utility/builds.

About 1.  Just need copy everything under snap into uuu repo?

404 for link https://snapcraft.io/universal-update-utility/builds.

Probably no permission as a snap collaborator, then the ownership transfer process must be done.  Here is the process:

Register an account on the snapcraft forum
Post a snap name transfer request under the store topic category of the Snapcraft Forum, you'll have to mention either your snap account id listed in https://dashboard.snapcraft.io/dev/account/ or the associated e-mail address. You may check out this example: Transfer brook - store - snapcraft.io
I can also make the post for you, in this case simply mention the aforementioned account details here.
After the name transfer is completed, the build page should be accessible.

About 1. Just need copy everything under snap into uuu repo?

I've filed a PR (#255) that adds the necessary files.

I've found more issues while using the uuu snap:

If the user has no write access to the target device node, one has to run the uuu snap command as root (the raw-usb interface connection is, unfortunately, not enough)
If the uuu snap command is run as root then due to the current implementation of the home interface's security confinement, the program can't access files under the user home directory (i.e. uuu scripts can't be read)

Refer Libusb application can't access device even after the raw-usb interface is connected - snapcraft - snapcraft.io for more info.
A dirty workaround is to make the user modify the permission of the target device mode manually(0666) and run uuu as the user instead of root.

pull request merged.
I created snapcraft account, email is<EMAIL_ADDRESS>please help post transfer request for me
About sudo access usb device problem.
1: put above udev run into /etc/udev/rules.d/70-uuu.rules
sudo sh -c "uuu -udev >> /etc/udev/rules.d/70-uuu.rules"
2: update udev rule
sudo udevadm control --reload
I am not sure how to integrate into snap

pull request merged.

Please merge #256 which fixes errors in the original PR.

I created snapcraft account, email is<EMAIL_ADDRESS>please help post transfer request for me

Please provide the snap account id listed in https://dashboard.snapcraft.io/dev/account/.

I am not sure how to integrate into snap

Unfortunately, it doesn't seem to be supported as of now.  You might want to add the instructions to the snap's launcher https://github.com/NXPmicro/mfgtools/blob/master/snap/local/launchers/universal-update-utility-launch so that the user will be notified of the workaround.

Snap account-id
NxFU2gRxjZ0LR5Ma7eNl05ZapIZtsxoK

Thanks!  The snap ownership transfer request is filed at https://forum.snapcraft.io/t/please-transfer-universal-update-utility-to-nxpfrankli/23120

Our license team require add your MIT license file under snap

Our license team require add your MIT license file under snap

I filed a new PR (#257) which should meet the requirement.

Thank very much

Sorry, license team review and find one more thing.
Please add copyright at yaml file
Copyright (c) 2017 Snapcrafters

One more question:
how to distinguish beta and stable package?  Most people should get stable package default.

Please add copyright at yaml file

Done. (#258)

how to distinguish beta and stable package? Most people should get stable package default.

The snap ecosystem features channels and tracks for this purpose.  The new build will always land in the edge channel and can be promoted to other channels via store backstage or the snapcraft promote command.
channels:
  latest/stable:    –
  latest/candidate: –
  latest/beta:      1.4.95+pkg-021c         2021-03-02 (37) 1MB -
  latest/edge:      1.4.95+pkg-021c         2021-03-02 (37) 1MB -

This snap implementation also integrates a selective-checkout script which, will help ensure that stable/candidate/beta snap revision will always be built for promotion, however, it relies on distinguishable naming of git tags so it doesn't for uuu.  Refer Selective-checkout: Check out the tagged release revision if it isn't promoted to the stable channel - doc - snapcraft.io for more details.

Also, I would like to request to add your Ubuntu One account e-mail address to the universal-update-utility snap's collaborator list so that you can access the current snap's backstage.

Please accept the snap collaboration invite in order to proceed with the ownership transfer according to the store staffs' request.

I've found more issues while using the uuu snap:
* If the uuu snap command is run as root then due to the current implementation of the home interface's security confinement, the program can't access files under the user home directory (i.e. uuu scripts can't be read)

I found that this is no longer the case as the home interface supports a read attribute which, when set to all, also allows reading non-hidden files in the home directories of all users as traditional file permissions allow.  However, setting this attribute will make the interface non-auto-connect and requires a manual interface connection(sudo snap connect universal-update-utility:home) or a store approval for autoconnect by default.  Refer Process for aliases, auto-connections and tracks - store-requests - snapcraft.io for details.
I've initiated the interface auto-connect request: Interface auto-connect request for the universal-update-utility snap - store-requests - snapcraft.io

accepted invite

accepted invite

I've updated the request's status.

I see.  https://snapcraft.io/universal-update-utility/builds
I have not seen edit button.

I have not seen edit button

To set up a new repo for snap building, disconnect the old repo and connect to the new one.

After I update selective-checkout part, get below error.
Staging selective-checkout

snapcraftctl stage
Sorry, an error occurred in Snapcraft:
[Errno 2] No such file or directory: '/build/universal-update-utility/parts/selective-checkout/install/scriptlets'
Traceback (most recent call last):
File "/snap/snapcraft/6120/bin/snapcraft", line 33, in
sys.exit(load_entry_point('snapcraft==4.5.4', 'console_scripts', 'snapcraft')())
File "/snap/snapcraft/6120/lib/python3.6/site-packages/click/core.py", line 829, in call
return self.main(*args, **kwargs)
File "/snap/snapcraft/6120/lib/python3.6/site-packages/click/core.py", line 782, in main
rv = self.invoke(ctx)
File "/snap/snapcraft/6120/lib/python3.6/site-packages/click/core.py", line 1259, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/snap/snapcraft/6120/lib/python3.6/site-packages/snapcraft/cli/_command.py", line 89, in invoke
return super().invoke(ctx)

After I update selective-checkout part, get below error.

Please revert 6cffef2 as the part isn't meant to be used like that.

This snap implementation also integrates a selective-checkout script which, will help ensure that stable/candidate/beta snap revision will always be built for promotion, however, it relies on distinguishable naming of git tags so it doesn't work for uuu. Refer Selective-checkout: Check out the tagged release revision if it isn't promoted to the stable channel - doc - snapcraft.io for more details.

Currently, the part will not do much else than always making the latest tagged release built at least once.

reverted. Do you why there are dirty at package name universal-update-utility_1.4.107-1-g8f92a02-**dirty_**s390x.snap?

Do you why there are dirty at package name universal-update-utility_1.4.107-1-g8f92a02-**dirty_**s390x.snap?

The selective-checkout script somehow determines the repo is modified, I'll check it out.

Do you why there are dirty at package name universal-update-utility_1.4.107-1-g8f92a02-**dirty_**s390x.snap?

I unable to reproduce the problem after fixing the selective-checkout part (#259), please verify.

Still have dirty v1.4.107-7-g1b9a4d1-dirty (66)
https://snapcraft.io/universal-update-utility/releases

Still have dirty v1.4.107-7-g1b9a4d1-dirty (66)

I still unable to reproduce the problem locally:
Pulling main
+ set -o nounset
+ snapcraftctl pull
+ /root/stage/scriptlets/selective-checkout --release-tag-pattern=uuu_[[:digit:]]+(\.[[:digit:]]+){2} --release-tag-prefix=uuu_
selective-checkout: The last tagged stable release(1.4.107) hasn't been promoted to the stable channel(1.4.107-1-g8f92a02-dirty) on the Snap Store yet, checking out 1.4.107.
Note: checking out 'uuu_1.4.107'.

However, after examining the build log (https://snapcraft.io/universal-update-utility/builds/1323496) I found it was the libusb submodule clone failure that causes the unclean state of the repo(#252):
Cloning into 'libusb'...
[06/Mar/2021:14:57:29 +0000] "CONNECT github.com:443 HTTP/1.1"<PHONE_NUMBER> "-" "git/2.7.4"
[06/Mar/2021:14:57:29 +0000] "CONNECT github.com:443 HTTP/1.1" 200 27358 "-" "git/2.7.4"
fatal: reference is not a tree: e782eeb2514266f6738e242cdcb18e3ae1ed06fa
Cloning into 'zlib'...
[06/Mar/2021:14:57:30 +0000] "CONNECT github.com:443 HTTP/1.1"<PHONE_NUMBER> "-" "git/2.7.4"
Submodule path 'zlib': checked out 'cacf7f1d4e3d44d871b605da3b647f07d718623f'
Unable to checkout 'e782eeb2514266f6738e242cdcb18e3ae1ed06fa' in submodule path 'libusb'
'git submodule update --init --recursive failed with exit code 1 (build may fail later)

snapcraft-universal-update-utility ../parts/main/src# git status
On branch master
Your branch is up-to-date with 'origin/master'.
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)
  (commit or discard the untracked or modified content in submodules)

        modified:   libusb (new commits, modified content)

no changes added to commit (use "git add" and/or "git commit -a")

It appears to be a deficiency/bug in Git <2.8 that makes Git unable to check-out the submodule hash:
git 2.7.4(ubuntu 16.04) - failure
git 2.17.1 (ubuntu 18.04) - success
git 2.25.1 (ubuntu 20.04) - success
git 2.27.0 - success
Possible solution:

Upgrade Git before checking out submodule
Migrate to a recent Ubuntu Core base(core18) that ships a recent Git distribution

Still have dirty v1.4.107-7-g1b9a4d1-dirty (66)

#262 PR should fix the problem.

I found that this is no longer the case as the home interface supports a read attribute which, when set to all, also allows reading non-hidden files in the home directories of all users as traditional file permissions allow. However, setting this attribute will make the interface non-auto-connect and requires a manual interface connection(sudo snap connect universal-update-utility:home) or a store approval for autoconnect by default. Refer Process for aliases, auto-connections and tracks - store-requests - snapcraft.io for details.
I've initiated the interface auto-connect request: Interface auto-connect request for the universal-update-utility snap - store-requests - snapcraft.io

Due to the snap ownership change and in order to accelerate the review process, I would like to ask for an additional confirmation that I am requesting the new interface assertion on behalf of you.

done


Add type guard support to the Filterable data structures

Do you want to request a feature or report a bug?
feature
What is the current behavior?
Example for Option
some<string | number>(42)
  .filter(t.string.is)
  .map(s => s.startsWith('i am a string'));

This does not work cause the the filter method does not support type guards.
What is the expected behavior?
As for th native Array.filter function it would be nice to have the same behaviour for all Filterable types.
Which versions of fp-ts, and which browser and OS are affected by this issue? Did this work in previous versions of fp-ts?
1.11.2
When accepted I would provide a pull request.

@mlegenhausen for Option there's an explicit refine method, but if you missed that I guess is not discoverable enough so we could deprecate it and add an overloading to filter

@gcanti ah thanks simply haven't seen it. Even after working on a daily basis with fp-ts I learn every day new hidden features in this library :smile:
You can better decide if a deprecation is appropriate. Is there any advantage over an overload filter method?
An overload would be nice cause it aligns better with the standard Array.filter method and other libraries filter method like in rxjs.

An overload would be nice cause it aligns better with the standard Array.filter method and other libraries filter method like in rxjs

I agree.
We can deprecate

Array

refine

Either

.prototype.refineOrElse
.prototype.refineOrElseL
fromRefinement

Option

prototype.refine
fromRefinement

and add support for refinements to

Array

takeWhile
span
dropWhile

Either

.prototype.filterOrElse
.prototype.filterOrElseL
fromPredicate

NonEmptyArray

.prototype.filter

Option

.prototype.filter
fromPredicate

ReaderTaskEither

fromPredicate

Record

filter

Set

filter
partition

StrMap

filter

TaskEither

.prototype.filterOrElse
.prototype.filterOrElseL
fromPredicate

Validation

fromPredicate


[feat] Bulk get/set config parameters

Ref: https://github.com/zwave-js/node-zwave-js/pull/2875

Yes this would be really nice, for example as I have 50+ units where I would like to change the temp/watt change reports to be less frequent to easen the network traffic.

I still dunno what could be the right way to implement this on the UI. BTW you already can do this by using a driver function :)

I'm not sure if this feature is for bulk setting different parameters, or to be able to set one parameter that may be split into multiple sub-values.... assuming the latter, I imagine that for the MQTT transports, at least, you could re-support the configuration parameter itself, rather than the individual bits and bytes the config files describe.
Example:
Today:
zwavejs2mqtt/NODE/112/0/26/65280/set
zwavejs2mqtt/NODE/112/0/26/16711680/set
zwavejs2mqtt/NODE/112/0/26/4278190080/set

New:
zwavejs2mqtt/NODE/112/0/26/set <-- sets the "raw" byte value.

I'm not sure if this feature is for bulk setting different parameters, or to be able to set one parameter that may be split into multiple sub-values

Both


Best way to add or remove items from List View

Hi,
Let's say I've loaded my app with 10 items in a list view. Now, let's assume that a new item needs to be added to the list. My questions are:

Do I have to use cloneWithRows with an array of all 11 items?
Does it render the whole list again?
Can I just add the 1 item to the list?
What about saving the scroll position?
What if I just want to remove an item from the list? Do I need to use cloneWithRows with the smaller array now or is there a way just to remove the 1 I want to remove?

I know it's not an 'issue' but I couldn't find a 100% sure answer on this.
Thanks!

Do I have to use cloneWithRows with an array of all 11 items?

Yes, exactly.

Does it render the whole list again?

Yes, though it only creates new react components for the rows that have changed and then applies the normal React reconciliation algorithm to the native views, whether that's dom or uiviews or android.

Can I just add the 1 item to the list?

Yes, create a new set of rows with the new item and pass everything to cloneWithRows.

What about saving the scroll position?

Scroll position is not affected when you append rows. The list will get shifted down if you prepend rows. Similar logic applies to removing rows. If you want to preserve the logical scroll position then you will need to calculate it from the contentOffset and height deltas of the modified rows.

What if I just want to remove an item from the list? Do I need to use cloneWithRows with the smaller array now or is there a way just to remove the 1 I want to remove?

You always call cloneWithRows -- for this case with the smaller array as you said.
In general it's best if questions about React Native are posted on Stack Overflow where there is a react-native tag to help categorize questions.

Thanks @ide

Are there any performance issues when a data source gets 100's of rows long? Or is all that React diffing overhead fast enough?

React diffing is not fast enough so that's why the data source requires rowHasChanged and components offer shouldComponentUpdate. That's how you tell React what rows are new. Then you can handle hundreds of rows no problem.


stat $: no such file or directory when attempting first release

I'm trying to use upload-rust-binary-action to upload my first release of a small CLI/GUI app and can't seem to make it succeed no matter what I do. Given my release.yml so far:
name: Release

permissions:
  contents: write

on:
  push:
    tags:
      - v[0-9]+.*

jobs:
  create-release:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: taiki-e/create-gh-release-action@v1
        with:
          draft: true
          changelog: CHANGELOG.md
          token: ${{ secrets.GITHUB_TOKEN }}

  upload-assets:
    needs: create-release
    strategy:
      matrix:
        target: [aarch64-apple-darwin, x86_64-apple-darwin,
                 x86_64-unknown-linux-gnu, x86_64-pc-windows-msvc]
        include:
            - target: aarch64-apple-darwin
              os: macos-latest
            - target: x86_64-apple-darwin
              os: macos-latest
            - target: x86_64-unknown-linux-gnu
              os: ubuntu-latest
            - target: x86_64-pc-windows-msvc
              os: windows-latest
              assets: target/wix/swmon-${{github.ref_name}}-x86_64.msi
    runs-on: ${{ matrix.os }}
    steps:
      - uses: actions/checkout@v4
      - name: Install packages (Linux)
        if: ${{ runner.os == 'Linux' && !matrix.apt-arch }}
        run: sudo apt-get update && sudo apt-get install libudev-dev
      - name: Install MSI (Windows)
        if: runner.os == 'Windows'
        run: |
            cargo install cargo-wix
            cargo wix -o target/wix/swmon-${{github.ref_name}}-x86_64.msi
      - uses: taiki-e/upload-rust-binary-action@v1
        with:
          # (required) Comma-separated list of binary names (non-extension portion of filename) to build and upload.
          # Note that glob pattern is not supported yet.
          bin: swmon,swmon-gui
          include: LICENSE.md,README.md,assets/swmon-gui_k8Sy5hg15P.png
          # (optional) Target triple, default is host triple.
          target: ${{ matrix.target }}
          tar: unix
          zip: windows
          archive: swmon-$target
          leading-dir: true
          asset: $${{ matrix.assets || '' }}
          # (required) GitHub token for uploading assets to GitHub Releases.
          token: ${{ secrets.GITHUB_TOKEN }}

The build consistently fails with:
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory
stat $: no such file or directory

Eventually the build times out and fails. What am I doing wrong? Even if this is user error, is it possible to improve the error message to explain what I'm doing wrong?

asset: $${{ matrix.assets || '' }}

This is treated as file name $ when matrix.assets is not set. You have to use ${{ matrix.assets || '' }} instead of $${{ matrix.assets || '' }}

Here is the expanded log of the failed job (https://github.com/cr1901/swmon/actions/runs/9248472961/job/25438823984#step:5:338).
(asset is set to $.)

is it possible to improve the error message to explain what I'm doing wrong?

In this case, it would probably be to check beforehand whether the path passed in asset or include is existent, but it might be okay to simply output the command to be executed.

I am about all CI-'ed out tonight... anyways that change worked. CI is now green, thanks for the help :D: https://github.com/cr1901/swmon/actions/runs/9249016102


Is it possible to reduce the file size for a PNG file without actually resizing the image and not reducing the quality significantly ?

I am pushing pictures taken by webcam to s3 bucket and want to process those or rather optimize those images so that their file size reduces but there is not much loss of quality and resolution. Is it possible to accomplish the same with this utility ?

Hi @mandeepm91 — Hm.. Sharp does have some parameters for controlling the quality of the output image, and also by default omits metadata (e.g. EXIF data) when writing a new image file, but I'm not sure this would cover what you're looking for.
However, incorporating such a feature into this project makes a lot of sense. Perhaps enabling the use of a tool like imagemin would be useful?

Hi @adieuadieu,
For now we switched to JPEGs because that solved the purpose but I think adding this feature might be useful. I can try implementing this in a few days using imagemin. Let me know if I can proceed.

@mandeepm91 Yes I'd welcome a PR. I haven't thought too much about how it would be best to implement the imagemin support. Perhaps extending the "outputs" support currently used for calling Sharp functions, but add a special-case "imagemin" item..


Missing 64-bit ARM binary in cardboard.jar?

On 64-bit ARM phones (specifically Mi 4i (ferrari)), cardboard activity crashes with
Caused by: java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/<app>/base.apk"],nativeLibraryDirectories=[/data/app/<app>/lib/arm64, /vendor/lib64, /system/lib64]]] couldn't find "libvrtoolkit.so"
	at java.lang.Runtime.loadLibrary(Runtime.java:366)
	at java.lang.System.loadLibrary(System.java:989)
	at com.google.vrtoolkit.cardboard.CardboardViewNativeImpl.<init>(CardboardViewNativeImpl.java:88)
	at com.google.vrtoolkit.cardboard.ImplementationSelector.createCardboardViewApi(ImplementationSelector.java:35)
	at com.google.vrtoolkit.cardboard.CardboardView.init(CardboardView.java:881)
	at com.google.vrtoolkit.cardboard.CardboardView.<init>(CardboardView.java:241)

I also found a post on Stack Overflow indicating the same issue with Intel processors:
http://stackoverflow.com/questions/32727395/cardboard-library-doesnt-load-on-intel-based-devices

Update: Downgrading to v0.5.4, which seems to be Java only, fixes the issue.

I am getting the same issue with Nexus 5X.
Caused by: java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/<app>/base.apk"],nativeLibraryDirectories=[/data/app/<app>/lib/arm64, /data/app/<app>/base.apk!/lib/arm64-v8a, /vendor/lib64, /system/lib64]]] couldn't find "libvrtoolkit.so"

s-gv, did you try the suggestion from jclova on Stack Overflow? Even though only armeabi-v7a binaries are currently shipped, they should work fine on arm64 phones.

Alternative architectures are now available. Instructions are at /libraries/native/README.md


Cannot insert data

I am indeed new to Vertica but I tried several options to insert data.
One of my latest attempts (from examples):
import * as dotenv from 'dotenv';

dotenv.config();

const {Client} = require('vertica-nodejs')

const verticaInsert = () => {
    const client = new Client()
    client.connect()
    client.query("INSERT INTO public.testing (call_id) VALUES (2)", (err: any, res: any) => {
        console.log(err)
        console.log(res.rows)
        client.query("SELECT * FROM public.testing", (err: any, res: any) => {
            console.log(err || res.rows[0])
            client.end()
        })
    })
}
verticaInsert();

Output seems okay:
root@d9de246fd6ff:/usr/src/app# ./node_modules/.bin/ts-node src/test_vertics.ts
null
[ { OUTPUT: 1 } ]
{ call_id: '100' } // Initial data

Configuration:
Node: v14.20.1
Vertica Analytic Database: v12.0.0-0
I can select the data as seen above but the data is inserted, is there something I am missing?

i think you forgot about “COMMIT;”


Explain the Behaviour of /submodel-elements

What?
/submodel-elements is under-specified for the different SerializationModifiers.
How?
Explain the target behaviour for the different SerializationModifier combinations and in particular how the pagination works.

[x] I have signed the required Developer Certificate of Origin (DCO) already.

Related issues
#185 discusses valueOnly serialization for /submodel-elements
#246  discusses path serialization for /submodel-elements
Both suggested solutions seem to be in line and propose that /submodel-elements$modifier returns the same as /submodel/$modifier but with pagination.


Add Electronic-Waste as member.

Links to PRs:

https://github.com/kubeflow/katib/pull/2285
https://github.com/kubeflow/katib/pull/2274

Pytest output:
======================================================= test session starts ========================================================
platform linux -- Python 3.8.10, pytest-8.1.1, pluggy-1.4.0
rootdir: /home/wangshao/internal-acls/github-orgs
collected 1 item

test_org_yaml.py .                                                                                [100%]

======================================================== 1 passed in 0.24s =========================================================

PTAL👀 @andreyvelich @tenzen-y

@andreyvelich Thank you for your recognition! I'll continue to contribute to Katib and other projects in Kubeflow since I really consider it a great opportunity for me to learn AI Infrastructure.


Support self-closing component tags

When trying to auto-close the div element

<my-component/>
resolves to

<my-component/></my-component>
It should close the div tag.
Self-closing tags are now part of Vue code styleguide. I believe this issue is similar to HTML syntax, resolved by this PR: https://github.com/sublimehq/Packages/pull/971.

Auto-completion is not handled by and cannot be changed by this package, it's simply using the same expand rules as HTML.

What's strange though is that in Sublime Text (Build 3143, the latest) self-closing tags work correctly in .html files, but not in .vue files (using Vue Syntax Highlight 3.2.3)

Actually this package does control autocompletion, indirectly, in the same way as that PR fixed it for basic HTML.
This commit seems to have (accidentally?) fixed this issue https://github.com/vuejs/vue-syntax-highlight/commit/17139a34b31a163374a083143418df077074b1e6

@Herteby 17139a3 should technically not affect 3143. Only 3153 onwards. Lemme know if you notice any odd behaviours.

Relevant issue (it should already be fixed) https://github.com/SublimeTextIssues/DefaultPackages/issues/176


rename Developer Guides to Guide

We are using these files to generated the sidenav entries in ng2.io, however, they don't seem to have a net effect on the sidenav in ng1.io
cc @chalin @kwalrath

As follow-up: note that "Developer Guides" isn't used in the sidenav anymore. Since the site refresh by Alex, we've been using "Guide".


Changing language forces menu focus and makes it hard to write multi-language messages

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary

If you use multiple languages you would normally swap between them via ALT + SHIFT.
However, because of an Electron bug (https://github.com/electron/electron/issues/22213), this focuses the menu and thus you need to either hit Escape like 15 times or manually click with the mouse in the message input.
As you can imagine, this becomes an EXTREME PAIN when your workflow requires language switch.
I literally need to slow down my entire flow to avoid and go around this bug.
Mattermost as well as https://github.com/RocketChat/Rocket.Chat.Electron/issues/1355, Skype, Slack and other Electron based applications also have this issue and there are a dozen of opened tickets about this in the Electron github issue tracker.
Environment

Operating System: Ubuntu 20.04 LTS
Mattermost Desktop App version:  Mattermost Version: 4.6.1
Mattermost Server version: Mattermost Version: 5.28.1

Steps to reproduce
Setup an additional input language and configure it to switch via  ALT + SHIFT.
Start typing and switch fast which is common practice since sometimes you write нещо на Български език :D where part of the message is in Bulgarian and part of it an emoji which is in English.
Expected behavior
Be able to write a message like a normal person without having to use the mouse to
Observed behavior
ALT + SHIFT causes menu focus and severely complicates writing and usage.

Possible fixes
Usually this issue goes away if you can set the application to work in full screen.
This hides the toolbar menu therefore it never gets focused.
However with the latest desktop version 5.28.1 of Mattermost there's a top toolbar which is always shown even in full screen and the workaround no longer work.
I would suggest making a simple checkbox in the App Options which is under File -> Settings that can hide the toolbar (which is something I believe used to be there?). This would be the easiest solution, because the guys from Electron don't seem to be interested in fixing this bug on their end.

Hi, while not directly addressing what you say, we do have a plan to hide the custom window in case of having only one server. You can see more details in the Jira ticket

Thank you very much for pointing me to the jira ticket! I've dropped a message there as well! :)

See also #1080.

you can try https://github.com/mattermost/desktop/releases/tag/v4.7.0-rc6 to see if that fixes the problem. it should be addressed there


Static Analyser on Inherited Generic Types

I am in a situation which feels like a defect with the static analysis.

This gives me an undefined method error: https://dart.dev/tools/diagnostic-messages#undefined_method
I have clearly mentioned that T extends the required interface that has a fromJson function.

Also, typecasting it twice might fix things...
                return (T as JsonCompatibleModel).fromJson(_snapshotData) as T?;

I am using:
Ubuntu 20.04 latest updates
➜  cloud-ktchn-pos-flutter-app git:(refacttors/rav-sep9) ✗ flutter --version
Flutter 2.5.0 • channel stable •
https://github.com/flutter/flutter.git
Framework • revision 4cc385b4b8 (3 days ago) • 2021-09-07
23:01:49 -0700
Engine • revision f0826da7ef
Tools • Dart 2.14.0

Flutter doctor:
➜  cloud-ktchn-pos-flutter-app git:(refacttors/rav-sep9) ✗ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.0, on Ubuntu 20.04.3 LTS
    5.4.0-84-lowlatency, locale en_GB.UTF-8)
[✓] Android toolchain - develop for Android devices (Android
    SDK version 30.0.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.2)
[✓] VS Code (version 1.60.0)
[✓] Connected device (1 available)

• No issues found!

Having to make double typecasts for something compiler has the opportunity to pick up just feels wierd

This has nothing to do with type inference, you simply cannot call static methods on type variables.
Dart static method invocations are resolved at compile-time. A call like T.fromJson cannot do that, so it's not a proper static call, it is not allowed and therefore doesn't work.
You need to pass the actual fromJson function as a parameter, along with the type, in order for this to be able to work.


wierd operation when running launch.py

when I followed the readme instructions to run python launch.py --config configs/test.yaml --train --gpu 0 system.prompt_processor.prompt="A boy with a beanie wearing a hoodie and joggers",  folders and files were made infinitely:
"./HumanGaussian/experiments/e1/A_boy_with_a_beanie_wearing_a_hoodie_and_joggers<EMAIL_ADDRESS>until No space left on device.

Hi, many thanks for your interest in our work:)
Could you please paste your configs here? It is indeed weird, as the saving folder structure should be like: ${exp_root_dir}/${name}/{text_prompt}@{running_time}, so that it's impossible for multiple different runs to be saved at the same location.
Best

this is configs/test.yaml
name: "e1"
tag: "${rmspace:${system.prompt_processor.prompt},_}"
exp_root_dir: "./experiments/"
seed: 0

data_type: "random-camera-datamodule"
data:
  batch_size: 8
  eval_camera_distance: 2.0
  camera_distance_range: [1.5, 2.0]
  light_sample_strategy: "dreamfusion3dgs"
  height: 1024
  width: 1024
  # resolution_milestones: [600]
  eval_height: 1024
  eval_width: 1024
  elevation_range: [-30, 30]

  enable_near_head_poses: true
  head_offset: 0.65
  head_camera_distance_range: [0.4, 0.6]
  head_prob: 0.25
  head_start_step: 1200
  head_end_step: 3600
  head_azimuth_range: [0, 180]

  enable_near_back_poses: true
  back_offset: 0.65
  back_camera_distance_range: [0.6, 0.8]
  back_prob: 0.20
  back_start_step: 1200
  back_end_step: 3600
  back_azimuth_range: [-180, 0]

system_type: "gaussiandreamer-system"
system:
  radius: ${data.eval_camera_distance}
  texture_structure_joint: true
  smplx_path: "./third/smplx_models"
  disable_hand_densification: false
  pts_num: 100000
  densify_prune_start_step: 300
  densify_prune_end_step: 2100
  densify_prune_interval: 300
  size_threshold: 20
  max_grad: 0.0002
  gender: 'neutral'
  prune_only_start_step: 2400
  prune_only_end_step: 3300
  prune_only_interval: 300
  prune_size_threshold: 0.008
  apose: true
  bg_white: false

  prompt_processor_type: "texture-structure-prompt-processor"
  prompt_processor:
    use_perp_neg: false
    pretrained_model_name_or_path: "stabilityai/stable-diffusion-2-base"
    negative_prompt: "shadow, dark face, colorful hands, eyeglass, glasses, (deformed iris, deformed pupils, semi-realistic, cgi, 3d, render, sketch, cartoon, drawing, anime:1.4), text, close up, cropped, out of frame, worst quality, low quality, jpeg artifacts, ugly, duplicate, morbid, mutilated, extra fingers, mutated hands, poorly drawn hands, poorly drawn face, mutation, deformed, blurry, dehydrated, bad anatomy, bad proportions, extra limbs, cloned face, disfigured, gross proportions, malformed limbs, missing arms, missing legs, extra arms, extra legs, fused fingers, too many fingers, long neck"
    prompt: ???

  guidance_type: "dual-branch-guidance"
  guidance:
    pretrained_model_name_or_path: "stabilityai/stable-diffusion-2-base"
    model_key: "./third/texture-structure_joint_model"
    vae_key: "stabilityai/sd-vae-ft-mse"
    guidance_scale: 7.5
    weighting_strategy: sds
    min_step_percent: 0.02
    max_step_percent: 0.98
    grad_clip: [0,1.5,2.0,1000]
    lw_depth: 0.5
    guidance_rescale: 0.75
    original_size: 1024
    target_size: 1024
    use_anpg: true
    enable_memory_efficient_attention: true
    grad_clip_pixel: true
    grad_clip_threshold: 1.0

  loggers:
    wandb:
      enable: false
      project: 'threestudio'
      name: None

  loss:
    lambda_sds: 1.
    lambda_sparsity: 1.
    lambda_opaque: 0.0
  optimizer:
    name: Adam
    args:
      lr: 0.001
      betas: [0.9, 0.99]
      eps: 1.e-15

trainer:
  max_steps: 3600
  log_every_n_steps: 1
  num_sanity_val_steps: 0
  val_check_interval: 100
  enable_progress_bar: true
  precision: 16-mixed

checkpoint:
  save_last: true # save at each validation time
  save_top_k: -1
  every_n_train_steps: ${trainer.max_steps}

Hi:
I think this could due to the miss of a .gitignore file, as mentioned in this issue. I've just fixed it and  could you please check if there are any further problems.
Best,

OK, thanks. it works.

sorry, the incessive reproduction issue seems not be solved. but after add .gitignore, the code runs smoothly util CUDA out of memory.  RTX 4090 24GB cannot support the generation of a single person?

Hi:
I think this could due to the miss of a .gitignore file, as mentioned in this issue. I've just fixed it and could you please check if there are any further problems.
Best,

How did you specifically modify the file? After I modified it, it still couldn't be solved.

As mentioned in here, not save the ckpts or output images to each run file can stop duplicating files for many time.


Add Oracle CodeOne 2019

Hello,
I would like to add Oracle CodeOne 2019 to the list, but I am not yet authorized to push to this repository and create a pull request for it. Is this correct and if so, could you please grant me access?
Kind regards,
Edwin Derks

Hello,
You should be able to create branch on the fork of the repository, and then to create a pull request here providing this branch.


Sam/15 16 nix package, build and test of 15.8 and 16.3

What kind of change does this PR introduce?
This makes all the necessary changes to package, build and test in the nix/pg_regress context our psql bundles with 15.8 and 16.3
We attempt to change no versions, and as much as possible to keep 1 pg_regress test suite, and one set of extensions and wrappers.
There will be a follow up PR to address the packer/ansible/gh action work for integration tests and releasing images.
It includes the changes to packer/ansible/gh actions to allow for a matrix build of pg versions on release.
Significant changes
This PR moves the source of truth for data about postgres major and semver back to ansible/vars.yml, which are then read at time of release in the ami-release gh action. The common-vars.nix.yml file is no longer under source control, but instead is generated duing the ci run for ami release or testinfra runs. The version is now controlled in this allows us to avoid making too many changes to our release and test tooling for AMI)
To bump the versions of postgres it is proposed to update these values that are now in anisble/vars.yml
These versions cover both release and testinfra
postgres_major:
  - "15"
  - "16"

# Full version strings for each major version
postgres_release:
  postgres15: "<IP_ADDRESS>-staging-5"
  postgres16: "<IP_ADDRESS>-staging-5"

How do these workflow changes impact the workflows which are tightly bound to the common-nix-vars.pkr.hcl file? Should we parse ansible/vars.yml outputs to ensure they continue working as intended?
i.e.
https://github.com/supabase/postgres/blob/develop/.github/workflows/publish-nix-pgupgrade-scripts.yml
https://github.com/supabase/postgres/blob/develop/.github/workflows/publish-nix-pgupgrade-bin-flake-version.yml
https://github.com/supabase/postgres/blob/develop/.github/workflows/test.yml

The current workflow generates a git tag for every AMI release - do these changes maintain this behaviour?

How do these workflow changes impact the workflows which are tightly bound to the common-nix-vars.pkr.hcl file? Should we parse ansible/vars.yml outputs to ensure they continue working as intended?
i.e.
https://github.com/supabase/postgres/blob/develop/.github/workflows/publish-nix-pgupgrade-scripts.yml https://github.com/supabase/postgres/blob/develop/.github/workflows/publish-nix-pgupgrade-bin-flake-version.yml https://github.com/supabase/postgres/blob/develop/.github/workflows/test.yml
The current workflow generates a git tag for every AMI release - do these changes maintain this behaviour?

Good point thank you!
I did adjust  https://github.com/supabase/postgres/blob/develop/.github/workflows/test.yml but I did not address
https://github.com/supabase/postgres/blob/develop/.github/workflows/publish-nix-pgupgrade-scripts.yml https://github.com/supabase/postgres/blob/develop/.github/workflows/publish-nix-pgupgrade-bin-flake-version.yml
...so I will take a look at those today
The general approach I was taking was to generate the hcl file at build time, and move the values to vars.yml

The current workflow generates a git tag for every AMI release - do these changes maintain this behaviour?

Yes they do as demonstrated when I ran a staging release they are visible at https://github.com/supabase/postgres/releases  (those with -staging-)

Going to address the docker work in a soon to be completed follow up PR to try to keep this work moving along, and break up into smaller chunks

@samrose that works great, thanks!
Only nit I'd have is that we'd want to preserve the overrides (the if [[ "${{ inputs.postgresVersion }}" != "" ]]; blocks) in order to deliver script/upgrade flake hash hotfixes for previous releases if required 😬

@pcnc ok https://github.com/supabase/postgres/pull/1266/commits/312e4d7922926870d7f5fa18aa1baa22116ce01f should take care of that if it looks good can you approve and we'll get it merged. Thank you!

Looks good, thanks! Approved.


Changing a address in checkout has no user feedback

PHP Version: -   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Expected behaviour:
Clicking the save button will close the modal and reload the checkout like before.
Actual behaviour:
You are annoyed that no feedback is coming.
Selecting from an existing address has also no more effect.
https://github.com/user-attachments/assets/05a02eba-085c-4ee2-b28f-af97d4b014fa
How to reproduce:
Open Checkout-confirm, change the address, press save.
Selecting from an existing address has also no more effect.

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39697
[created from NEXT-39697, comment 513969]

[public] This is a duplicate of https://github.com/shopware/shopware/issues/5524
[created from NEXT-39697, comment 516889]


HSEARCH-3764 Allow disabling conversion in the ID predicate

https://hibernate.atlassian.net/browse/HSEARCH-3764

Nice work. I have nothing to say about it.
Merged thanks


New version: eMClient.eMClient version 9.2.1553.0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.4 schema?

Microsoft Reviewers: Open in CodeFlow

Actually updates eMClient instead of having name looking like update for eMClient.

/azp run


All page title are using the wrong text style

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.22621.1702
Other software
OS Build Version: 10.0.22621.1702.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.16
Steps to reproduce the bug
All page title are using title

which is wrong
Expected result
All page title should be using the subtitle style
" Text="Subtitle" Style="{StaticResource SubtitleTextBlockStyle}"/> "
Please also refer to figma files
Actual result
Wrong page title style
Included System Information
CPU: AMD EPYC 7763 64-Core Processor
Physical Memory: 15.95GB (8.84GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_<IP_ADDRESS>_x64__8wekyb3d8bbwe

Duplicate of #567


hard before partial

https://github.com/nexlabs22/Nex-Dex-Contracts/blob/49154352d089cfc03c96e7399355d7f9d0b57ea3/contracts/Exchange.sol#L366-L367
https://github.com/nexlabs22/Nex-Dex-Contracts/blob/49154352d089cfc03c96e7399355d7f9d0b57ea3/contracts/Exchange.sol#L423-L424
Can you do the hard liquidate first? Similar to other ones.

https://github.com/nexlabs22/Nex-Dex-Contracts/blob/79297eea1a0bca6a66492cddc97945327c220a81/contracts/Exchange.sol#L423-L424


[WIP] Implementing nameof operator

This is a rebased version of #13, which was moved from https://visualfsharp.codeplex.com/SourceControl/network/forks/forki/fsharp/contribution/7698 (see discussion there)
https://fslang.uservoice.com/forums/245727-f-language/suggestions/5900625-add-nameof-operator-to-follow-c-vb-update suggests we implement a "nameof" operator which is something similar to the typeof operator.
This feature was "approved in principle" by @dsyme.
Related information:
C#/VB spec for nameof: https://roslyn.codeplex.com/discussions/570551
Mads showing the nameof operator: http://channel9.msdn.com/Events/Visual-Studio/Connect-event-2014/116 (at 5:35)

@forki put [WIP] in the title

btw, jenkins fails /cc @otawfik-ms  @TyOverby
java.io.IOException: remote file operation failed: d:\j\workspace\Microsoft_visualfsharp\debug_windows_nt_prtest at hudson.remoting.Channel@3da6bc31:Azure0128054658: java.nio.file.AccessDeniedException: d:\j\workspace\Microsoft_visualfsharp\debug_windows_nt_prtest\debug\net40\bin\FSharp.Core.dll

add gitignore of fsharp.core in tests ( see https://github.com/Microsoft/visualfsharp/pull/894#issuecomment-175119749 )

Nah, that should be done in separate PR and applied immediately. Annoying to the max.  /cc @otawfik-ms

thx for the feedback. pushed a new version

so appveyor is green. What's going on with Jenkins?

I think it's related to new jenkins worker images
java.io.IOException: remote file operation failed: d:\j\workspace\Microsoft_visualfsharp\debug_windows_nt_prtest at hudson.remoting.Channel@3da6bc31:Azure0128054658: java.nio.file.AccessDeniedException: d:\j\workspace\Microsoft_visualfsharp\debug_windows_nt_prtest\debug\net40\bin\FSharp.Core.dll
That's happening before git checkout, when it's trying to cleanup workspace , the file it's in use? /cc @mmitche @otawfik-ms
Maybe it's related to or fixed by #904 ?

@7sharp9 suggested to mainain a list of open issues.
I'm starting by adding/answering some open questions from @latkin (copied from codeplex):

This needs to handle cases where "nameof" already resolves to something user-defined, to avoid being a breaking change. Roslyn team took great pains to make sure no existing code is broken by nameof.

We have a test which shows that let nameof x = ... shadows the nameof functionality.

Is nameof treated as a "regular" function, i.e. what happens if I try x |> nameof  or let f = nameof ;; f x or similar.  Should this work, and if not, is there a reasonable error.

let f = nameof is forbidden, but x |> nameof works. We have tests for this.

Is the result considered a literal?

How can I verify this?

Can I use it in attributes?

Not yet. No idea how to tackle this.

Can I use it as a string pattern match case?

Yes. we have a test for this.

Does it work in quotations?

Yes. we have a test for this.

Does it resolve provided symbol names?  i.e. generated by type providers

No idea.

Various cases for type names - are these supported?

I don't understand that question.

Instance members of types? e.g. can I do nameof(System.String.Length)

Nope, but let s = "";; nameof(s.Length) works.

How are generics handled, including wildcard generics?

Things like let fullyGeneric x = x;; nameof(fullyGeneric) work.

Do we want the result to match the compiled name of stuff or the F#-code name of stuff?

Currently we use the CompiledName.

Have you tested out the IDE experience? If you hover the mouse over "nameof" do you get reasonable tooltip?

Tooltip: "Returns the name of the given symbol". And most important: refactoring works.

to use nameof in attributes

To support this, result should be considered a literal.

Yes, but it's only one of many problems to support this.

@forki maybe you can make a checkbox list as summary in the pr description, to show current status and open questions.
In the comments it's easy to lose focus

done

y |> nameof now gives compiler instead of runtime error. one edge case down.
Anybody knows if
Expr.Const(Const.String name, m, cenv.g.string_ty),
{ TotalSize = 1
FunctionSize = 1
HasEffect = false
MightMakeCriticalTailcall = false
Info = UnknownValue }
emits a literal?

Ok looked into ILSpy:
let myFunction parameter1 = nameof parameter1

is same as
let myFunction parameter1 = nameof "parameter1"

The goal is for nameof <expr> to be IL-equivalent to "expr" so you can do something like
type Person =
    { Name : string
      Age : int }
    member __.Update(fld : string, value : obj) =
        match fld with
        | nameof __.Name -> { __ with Name = string value }
        | nameof __.Age -> { __ with Age = int value }

This is what I meant by "supporting pattern matching".
Is the self-identifier supported? e.g. do member __.f() = nameof __ and member x.f() = nameof x return "__" and "x" respectively?
Requiring an expression as the argument means non-expression stuff won't work. e.g. things like below, which are all supported by C# nameof

nameof System.Diagnostics // namespaces
nameof System.String // types
nameof List // modules
let f<'t> (x : 't) = nameof 't  // type arguments

IMHO the goal with nameof is "just" to remove magic strings in the most common positions to make it refactor easier. I think we don't need to support every possible location to make it still useful.
Your sample is interesting. I will take a look, but this already works:
    type Person =
        { Name : string
          Age : int }
        member __.Update(fld : string, value : obj) =
            match fld with
            | x when x = nameof __.Name -> { __ with Name = string value }
            | x when x = nameof __.Age -> { __ with Age = value :?> int }
            | _ -> __

Regarding name of types. I did some initial work on a typenameof operator that works like typeof:

@enricosada I switched the machine affinity back.  I'll take a look at the issue and see what the issue is.
What is the status of the elevated nature of the tests you guys run...is that going away or staying?

@mmitche i am not from visualfsharp team  (from community).
@otawfik-ms it's doing some work with #894 (remove from gac), @TyOverby fixing tests with #888 (i think with fakesign too).
I think @otawfik-ms and @TyOverby can answer you questions

@mmitche i added #924 to track current jenkins build failures
Sry @forki for noise, i think we can delete these comments tomorrow

A bit late, I would like to ask if this function has to be called nameof like C# or if an operator could be used instead for F#. Might for example "@xxx" be used instead of "nameof(xxx)" ?

It's very similar to the already existing typeof. So I think we should use
the currently suggested naming.
On Feb 4, 2016 8:06 AM, "bluemmc"<EMAIL_ADDRESS>wrote:

A bit late, I would like to ask if this function has to be called nameof
like C# or if an operator could be used instead for F#. Might for example "
@xxx https://github.com/xxx" be used instead of "nameof(xxx)" ?
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/908#issuecomment-179681796
.

@dotnet-bot test this please

We're experimenting with establishing a lightweight RFC process for F#.
We've created F# RFC FS-1003 for this, with a discussion thread

It would be awesome to have this revived and shipped in next language update.
I imagine there is a way to do similar thing with code quotations but a proper operator (even if it doesn't cover all the cases) is a much better option.

@smoothdeveloper You can like this:
 match <@@  (Unchecked.Defaultof<MyType> : MyType.MyMethodWithIntParam<'a>(0) @@> with
 | Patterns.Call(_,mi,_) -> ...

@forki,
@dsyme,
This one I'm going to close, since it seems to have atrophied.
Please re-open if you get the time to do more work on it.
Thank you
Kevin

@KevinRansom Let's keep open any WIP PRs that correspond to an approved F# RFC. Inactivity is OK for RFCs - they can take months or years to get through the system.

I rebased this on master

@forki @dsyme Can I help with this?

Closing

Why are you closing this?

@isaacabraham I think it's reasonable to close PRs that haven't been touched for a very long time.
The PR can definitely be resubmitted as part of F# 4.2 work, it's not lost.
It's probably on my plate to review the RFC and implementation once again.  But the RFC tracks the issue for now.

I'd like to be able to write inline functions that takes the parameter that nameof receives and passes it in to the operator invocation. It seems like this should be possible. Is this possible with the current implementation? If not, can we add that as a feature? The ability to compose nameof using inline functions would help remove several thousand characters of boilerplate in my code bases. It would also slightly lessen my need for syntactic macros in F#.

Actually, as it turns out, my above comment in its initial form was not entirely accurate.
I looked at my code base, and as it turns out, I only have speculative uses for making the nameof parameter forwardable from inline functions. It wouldn't do anything for my current code base after all. Doesn't mean the capability isn't worth consideration tho, I suppose.
Apologies for the misdirection.


Training set list for few-shot task

Could I get the few-shot data list of each object?
(= data index for 4-shot, 8-shot, 14-shot(onside))
Thank you

based on our commit history, here is the 14 samples' indexes for ship & lego scene (we only experimented ship & lego scene for 14 samples):
        # LEGO SCENE: [2,5,10,40,52,53,69,78,83,85,90,94,96,97]
        # SHIP SCENE: [12, 28, 32, 33, 35, 36, 38, 44, 55, 66, 67, 88, 95, 96]

As for 4-shot and 8-shot, we didnt hard code the samples' indexes but implemented as follows for sampling:
        if few_shot > 0 and split == 'train':
            np.random.seed(self.shuffle_seed)
            np.random.shuffle(frames)
            frames = frames[:few_shot]


Add base URL option

Hi, thanks for the great work!
I would like to use Vizanti under some subfolder like this: http://<host_ip>/vizanti
Possible adding a base URL option or use relative URLs?

Hmm well if by host_ip you mean without the :port part that's a bit of a problem. ROS really doesn't want anything to do with root and linux won't let anything but root host on port 80, so it's a bit of a standoff. One option that I've used in the past is to set up a root hosted redirect server like nginx on port 80 separately, that then routes to 5000. Or to run server.py with sudo python3.
The server expects some specific paths which would need to be adjusted on both ends to support the extra /vizanti, but in theory it would be doable.

I am using Traefik as a reverse proxy to route subfolder /vizanti to port 5000 but found some errors due to some URLs is not relative URLs. I'll try to fix this and create a PR.

To add a base URL, I've tried blueprints but can not get it to work.
I've also tried to hardcode the base URL: https://github.com/MoffKalast/vizanti/compare/noetic-devel...tony2guo:vizanti:base-url
It works, but there should be some better ways. I'm not familiar with Flask, so I'm not sure how this should be done.

Hmm well the current setup uses jinja templating, which seems to have some kind of url_for function that would be added to returned templates. Not sure if it's the correct URL though.

Thanks for the info. I'll try url_for().


Create entity_to_kgroup converter  ( pom_som_mapper)

This would be symetrical with  kgroup_to_entity in pom_som_mapper.py

The difficulty is that there is no representation in the database of the meta data about the group from which the entity was imported other that the group name and the column to element mappings in POMClassAttributes.


grandorgue: 3.15.2-1 -> 3.15.3-1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/GrandOrgue/grandorgue/releases.
meta.description for grandorgue is: Virtual Pipe Organ Software
meta.homepage for grandorgue is: https://github.com/GrandOrgue/grandorgue
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.15.3-1 in filename of file in /nix/store/bgyd37p0s40ka3vv5nc49751xd55ibir-grandorgue-3.15.3-1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

grandorgue

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bgyd37p0s40ka3vv5nc49751xd55ibir-grandorgue-3.15.3-1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A grandorgue https://github.com/r-ryantm/nixpkgs/archive/2e030327a622c11372787d40c48b6c358d055429.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/2e030327a622c11372787d40c48b6c358d055429#grandorgue

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bgyd37p0s40ka3vv5nc49751xd55ibir-grandorgue-3.15.3-1
ls -la /nix/store/bgyd37p0s40ka3vv5nc49751xd55ibir-grandorgue-3.15.3-1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

grandorgue

Maintainer pings
cc @puzzlewolf for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362659

aarch64-darwin

:white_check_mark: 1 package built:

grandorgue


Mavlink act ctrl

Quick fix for actuator control messages which were chewing up far too much bandwidth.

Can you also rename the class name? (MavlinkStreamAttitudeControls)

@thomasgubler I renamed and cleaned up further a bit - please have a look again.

looks nicer now. I guess if this is bench tested it can go in.


Layout change - Data file view moved to left side

GUI layout change to move Data file view from right side to left side.
I think this change makes sense for the layout as it creates a divide between the temperature monitoring views on the right and the config/other sensors on the left.

However now that the Data file view is on the left, its width is shortened and the directory/filename fields are a bit stubby. I suggest changing the layout of the Data file view so that these two fields are vertically aligned. I've attempted to do this with the QGridLayout but I still need to play around to get the elements aligned properly.

Opening in draft to get feedback on this layout change before it's refined.
Closes #420

I think moving the Data File panel to the left makes sense now that the left-hand side is less busy. If you want to align the components, you can stick each of them in a QGridLayout - an example of this is in temperature_monitor_view.py, but actually, most of the panels contain QGridLayouts so there should be a few examples to refer to.

This is the best I've been able to align the elements with the QGridLayout. Still not perfect, not sure if anyone has some ideas on getting it a bit tidier.

Ah perfect, that's done it. Looks much cleaner now!


[stable/influxdb] Adds support for InfluxDB Enterprise

Follows the newly introduced pattern of StatefulSets for pods, extending InfluxDB's
sts and allowing parameter for it's ReplicaSet, and adds a new sts for InfluxDB
"meta nodes" which hold distribution state for InfluxDB's high availability and
horizontal scaling.
Documentation for these new features has been added, but I can not stress enough
how important it is to follow, as Enterprise not only uses different container
images for meta containers, but also expects new container images for InfluxDB
(appropriately labeled data in Docker Hub) which in my experience was easy to
miss if a user is following one set of instructions over another.
A caveat of trade off of deploying Enterprise is an expectation of human
interaction when creating or scaling the cluster (influxd-ctl usage). This
makes Enterprise decidedly not "cloud native", but is less obnoxious when using
a StatefulSet with PVCs, and no pod auto-scaling, of which I suppose is unlikely
given InfluxData's approach to licensing.
Still forcing k8s operator involvement feels "bad", and I left in provisions to
improve this expectation in a later version, namely a headless service and using
publishNotReadyAddresses, such that a pod can use the Service to find
available influxdb-meta and influxdb pods to create/join an InfluxDB
cluster.
Signed-off-by: Weston Johnson<EMAIL_ADDRESS>
Is this a new chart
No.
What this PR does / why we need it:
Adds support for InfluxDB Enterprise, specifically HA clustering.
Which issue this PR fixes

fixes #18685

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @aisuko

Thanks a lot, the new feature is so awesome. @wgj Please solute the conflict.  cc @naseemkullah Hello, sir. If you can be here help us to review or give some advice, it is our pleasure.

I believe for the Statefulset we need to add the headless type of service

Line 78 of README.md instructs users to use a headless service. It sucks I'm making them do it by hand, but the current .service.type: ClusterIP in values.yaml is open ended, and I wanted to be polite and not make assumptions for anyone.

I know what's your mean. I have a consider here, Should we need to make the service to multiple just like the service file in the stable/rabbitmq  ?https://github.com/helm/charts/tree/master/stable/rabbitmq/templates
This may be useful for new users who do not know statefulset. That could be let them know the statefulset need to create the service with no ClusterIP and it just for the pods belongs to the statfulset communicating with each other.  Another reason is out-of-box for the chart, If somebody
who is deploying that then they can use in one second instead of need to create the service type in ClsuterIP, NodePort, Loadbalance manually.

/ok-to-test

@Aisuko I'm not sure I fully understand your suggestion, but after thinking about what you said, I thought of this...
How about the .spec.type in templates/service.yaml use a conditional: if .Values.enterprise.enabled, then use a headless service, else continue normally? Is that better for users?

statefulsets generally require headless services regardless if enterprise is enabled or not, I would make it more generic. i.e. If you have statefulset you have a headless service.

Good point @naseemkullah. I believe this is a "bug" introduced from #17619, which I'm imagining it's been harmless thus far since non-Enterprise deployments are a statefulset of 1. Happy to "fix" this in my branch, or leave it to be part of another PR.
Let me know what you'd guys like me to do.

Yes #17619 was a little bit rushed for such an impactful change imho, i am still pinned to a version of the chart prior to it until everything is sorted out.
Although a separate "hotfix" PR would be the correct thing, its up to you guys :)

I mean there would have two headless services, one for the existing service another one for enterprise.  And you are right, this is the bug for the PR which changed the source type to statefulset.  But, these two different issues.  I would fix this in other PR. For this PR, please just set the headless service for enterprise statefulset.

The metadata pods are not exposed to end-users. After running kubectl exec influxdb-meta-0 influxd-ctl add-meta ... & kubectl exec influxdb-meta-0 influxd-ctl add-data ... from README.md, end-users use InfluxDB data pods only. Would you like me to rename the file meta-service.yaml to meta-service-headless.yaml?


Inflate with GitHub Action

Testing to see whether the new Action runs for this pull request and whether it works.

https://github.com/HebaruSan/xKAN-meta_testing/tree/github_action

ckan compat add 0.90 1.2

It ran, and the output was as expected!

Let's see if this occurrence of #overwrite_cache triggers our check for it


decorateHandler.js:80 Cannot read property 'object' of undefined

If I
import withDragAndDrop from "react-big-calendar/lib/addons/dragAndDrop"
it throws:

decorateHandler.js:80 Uncaught TypeError: Cannot read property 'object' of undefined
    at decorateHandler.js:80
    at Object.decorateHandler [as default] (decorateHandler.js:189)
    at decorateSource (DragSource.js:62)
    at compose.js:22
    at Object../node_modules/react-big-calendar/lib/addons/dragAndDrop/DraggableEventWrapper.js (DraggableEventWrapper.js:227)
    at __webpack_require__ (bootstrap 6328d40d0d9f7c1cbb2e:678)
    at fn (bootstrap 6328d40d0d9f7c1cbb2e:88)
    at Object../node_modules/react-big-calendar/lib/addons/dragAndDrop/withDragAndDrop.js (withDragAndDrop.js:25)
    at __webpack_require__ (bootstrap 6328d40d0d9f7c1cbb2e:678)
    at fn (bootstrap 6328d40d0d9f7c1cbb2e:88)

I don't think this is still an issue. Closing for now.


cl--assertion-failed on malformed build cache

I get a stack trace starting with:
Debugger entered--Lisp error: (cl-assertion-failed ((cl-every (function stringp) list) nil))
  cl--assertion-failed((cl-every (function stringp) list))
  straight--load-build-cache()

This should never happen, as the calls to cl-assert are wrapped in an ignore-errors. I don't know what's happening here.

What was happening was that cl-assert signals cl--assertion-failed on a failed assertion, instead of throwing an error like I had assumed. Fixed.


rerun failed analysis commands with gdb (debug feature)

With the new database cleanup the failed analysis action will not be stored into the database
only the reports.
The debug command will be removed which worked from the database information  #715.
The new analyze command filters out the failed #679 analysis actions.
A new debug option could be introduced for the analyze command to rerun the failed analysis actions with gdb without the database.

With the tu_collector collecting compilation information can be done easily. The problematic analysis command can be rerun with gdb.


Intermittent timeouts in FlushAsync functional tests time

Needs some investigation to figure out if something's amiss

Are folks still seeing this? I haven't see the tests timeout locally or on the CI.

I haven't seen this in a long long time. IMO we'd need a repro or a dump to really investigate this.

Closing this because apparently no one has seen it. If we see it again we can log a new bug.


navigate directly to loaded symbol for load aliases.

Hopefully this is the correct way to do it.

Ok so I am now working on struct member completion that works for local and imported symbols.
I can see when working on member completion that what I put in for resolving navigation elements for anonymous symbols in the last commit is not consistent with the general strategy for symbol resolution. I'll know more as I continue the work. Any comments welcome.
@brendandouglas I can't use rules-kotlin as my test workspace  as master doesn't have Kotlin changes. Also if I spend a lot of time on this and other isolated feature patches it would mean I won't get to benefit from any of them for months without managing my own releases. Releasing just Kotlin is fine but multiple patch streams would become crazy.


Clean up master branch

The master branch was recently brought up to date with gh-pages. There are a few things left to do to clean it up.

Remove some files that are included in .gitignore so they are not tracked. ./cumulative.json, ./doc-data-full.js, and ./doc-data.js
Remove the old video.js source code from ./videojs-src. The video.js source is now cloned to ./video.js and is ignored
Run the grunt task. Doing so updates ./docs/api/js/doc-data.js

Now that PRs are done against master, the grunt task won't be needed to run until we need to do a deploy. Probably makes sense to do now, though.

Sounds good. I figured master should be kept in a deployable state, but there's no reason it needs to be.

Deployable doesn't mean deployed. The grunt task feels like the deployment for me. Would also keep the master branch cleaner moving forward.

agreed -- good changes
Gary Katsevman wrote:

Deployable doesn't mean deployed. The grunt task feels like the
deployment for me. Would also keep the master branch cleaner moving
forward.
—
Reply to this email directly or view it on GitHub
https://github.com/videojs/docs/pull/10#issuecomment-154778261.


Question of calling DZI files

Hi,
I have a question about calling the dzi file in OpenSeaDragon. That's how I wrote in the html file.

I've changed the tileSources address from .xml to.dzi, or changed the dzc_ouput file (Generated from Windows deep zoom composer) from .xml to .dzi but it still doesn't work.
In addition, I download the github project of website to my desktop, but what it shows in the openseadragon is that: "Unable to open [object, object]: HTTP 404 attempting to load TileSource."
That's how it looks like in the browser.

I know it's a really simple question but I try all those methods above and still can't figure out.
Could anybody please help me to solve it?
Thanks,

Is your html file in the same directory as the UntitledProject4 directory?
(I'm no expert just starting with openSeadron myself)

Indeed...sounds like you've got your paths wrong somehow. Can you share your folder structure? Or can you put it on a server somewhere to share?


Review getting started examples

rerun tests

@radekosmulski is it ready for a 2nd review? I am not sure, if you added the changes to the TF notebook

We need to move the tests to Merlin/tests/unit/examples/

rerun tests

rerun tests


trailing-comma false positive

Bug Report

TSLint version: 4.5.0
TypeScript version: 2.2.1
Running TSLint via: (pick one) CLI, VSCode

TypeScript code being linted
interface A {
    b: {
        c: string // ok
        d: string // [tslint] Missing trailing comma (trailing-comma)
    }
    e: string // ok
}

with tslint.json configuration:
{
    "rules": {
        "trailing-comma": [
            true,
            {
                "multiline": "always",
                "singleline": "never"
            }
        ]
    }
}

Actual behavior
Requires comma at last nested type member
Expected behavior
Should not

That's the same problem we also have with interfaces #1810. Members can end with comma, semicolon or just a line break.
Instead of finding a smart solution to guess if a trailing comma is desired, we should add config options for those cases (and maybe even more options for other cases #2286)
I recently refactored this rule in #2236. The PR is not merged yet so I could add the new options while I'm already touching the code.

Why would you not want trailing commas on interfaces with "multiline": "always"? Can you provide a convincing argument that the rule needs to be customizable in that way? There's even an auto-fixer; this should be trivial to fix in your codebase.

This particular rule is about a trailing comma only, not about all separating commas. And I think it should not interfere with other style decisions, like omitting optional semicolons and commas whenever possible.

Ah, sorry I think I misread the first code snippet. This looks like a duplicate of #1810.

duplicate of #1810

In that case, closing.


Fix segmentation fault on KDE

Fixes https://github.com/ppy/osu-framework/issues/6238
Upstream fix: https://github.com/libsdl-org/SDL/pull/9523
I'll continue to work with upstream on a fix that meets their code quality standards/etc, but I'd like to get this in in quick order as it's a blocker for our existing developers.

Fixed upstream in https://github.com/libsdl-org/SDL/pull/9524


Replace linked list with array

This is a substantial change replacing the usage of a linked list as the
internal data structure for the choices with an array. Here are some
advantages in no particular order:

Arrays are simple and have a great cache locality

A large portion of the code in src/compat.h can be removed

Enables usage of the libc qsort(3) sort function

Reduces LOC

Still left TODO:

Rename choices members in order to adapt the convention of codebase

Remove all SLIST/QUEUE related code from src/compat.h and automake

On a side note: AFAIK setting the choice score to 1 is used in order to trigger sorting when the query is empty. Therefore 1 should be defined as a macro with a more descriptive name. While at it: the 0 score should also be defined as a macro.

Hey @mptre, this is awesome! Thanks for your great work! I left some comments above but other than that this looks great and I'd be happy to merge it. I also agree with your TODO notes.

Great, I will fix the remaining TODO:s in a separate pull request
On Mon, Jan 04, 2016 at 01:46:02AM -0800, Calle Erlandsson wrote:

Hey @mptre, this is awesome! Thanks for your great work! I left some comments above but other than that this looks great and I'd be happy to merge it. I also agree with your TODO notes.

Reply to this email directly or view it on GitHub:
https://github.com/thoughtbot/pick/pull/82#issuecomment-168625731

--
:wq

Merged as d7d6992. Thanks again, @mptre!


geemap.shp_to_ee generating error <urlopen error [Errno -2] Name or service not known>

Environment Information

Wed Nov 29 21:39:56 2023 UTC
OS | Linux | CPU(s) | 2 | Machine | x86_64
Architecture | 64bit | RAM | 12.7 GiB | Environment | IPython
Python 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0]
geemap | 0.29.3 | ee | 0.1.379 | ipyleaflet | 0.17.4
folium | 0.14.0 | jupyterlab | Module not found | notebook | 6.5.5
ipyevents | 2.0.2 | geopandas | 0.13.2
Description
Trying to use shapefile in EE using Jupyter Notebook on Google Colab. Command started producing error <urlopen error [Errno -2] Name or service not known> as of today. Was working fine previously.

It works fine on my end. Please share the dataset that can reproduce the error.


globus-automate-client doesn't install using pip on python 3.10 or 3.11

Tried to install globus-automate-client in a clean virtual environment using the following commands;
pip install --upgrade pip setuptools wheel
pip install globus-automate-client
This results in a failure. Any ideas?
└─(14:38:58)──> pip install globus-automate-client                                                                                ──(Wed,Jul19)─┘
Collecting globus-automate-client
  Using cached globus_automate_client-0.17.0-py3-none-any.whl (54 kB)
Collecting PyYAML<6.0.0,>=5.3.1 (from globus-automate-client)
  Using cached PyYAML-5.4.1.tar.gz (175 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [62 lines of output]
      /tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/config/setupcfg.py:293: _DeprecatedConfig: Deprecated config in `setup.cfg`
      !!

              ********************************************************************************
              The license_file parameter is deprecated, use license_files instead.

              By 2023-Oct-30, you need to update your project and remove deprecated calls
              or your builds will no longer be supported.

              See https://setuptools.pypa.io/en/latest/userguide/declarative_config.html for details.
              ********************************************************************************

      !!
        parsed = self.parsers.get(option_name, lambda x: x)(value)
      running egg_info
      writing lib3/PyYAML.egg-info/PKG-INFO
      writing dependency_links to lib3/PyYAML.egg-info/dependency_links.txt
      writing top-level names to lib3/PyYAML.egg-info/top_level.txt
      Traceback (most recent call last):
        File "/home/allsoppj/Repositories/temp/test_loading_globus_automate_client/env/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/home/allsoppj/Repositories/temp/test_loading_globus_automate_client/env/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "/home/allsoppj/Repositories/temp/test_loading_globus_automate_client/env/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 341, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 323, in _get_build_requires
          self.run_setup()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 338, in run_setup
          exec(code, locals())
        File "<string>", line 271, in <module>
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/__init__.py", line 107, in setup
          return distutils.core.setup(**attrs)
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/_distutils/core.py", line 185, in setup
          return run_commands(dist)
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
          dist.run_commands()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
          self.run_command(cmd)
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/dist.py", line 1234, in run_command
          super().run_command(command)
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
          cmd_obj.run()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/command/egg_info.py", line 314, in run
          self.find_sources()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/command/egg_info.py", line 322, in find_sources
          mm.run()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/command/egg_info.py", line 551, in run
          self.add_defaults()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/command/egg_info.py", line 589, in add_defaults
          sdist.add_defaults(self)
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/command/sdist.py", line 104, in add_defaults
          super().add_defaults()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/_distutils/command/sdist.py", line 251, in add_defaults
          self._add_defaults_ext()
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/_distutils/command/sdist.py", line 336, in _add_defaults_ext
          self.filelist.extend(build_ext.get_source_files())
        File "<string>", line 201, in get_source_files
        File "/tmp/pip-build-env-nvbf18tv/overlay/lib/python3.10/site-packages/setuptools/_distutils/cmd.py", line 107, in __getattr__
          raise AttributeError(attr)
      AttributeError: cython_sources
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

Hello James! This issue has been showing up across the Python ecosystem due to the latest release of Cython, version 3.0.0, which was released about 48 hours ago and has introduced an issue with PyYAML.
We're actively working to resolve this; PR #194 in this repository is being reviewed now.

Thanks for getting this merged so quickly, do you know when the change will be live on PyPi?

@JamesAllsopp This should be released as v0.17.1 now. :grinning:


fix message timestamp, body encoding issues

fix message timestamp (needed epoch offset and nanoseconds/100)
fix body encoding to respect StringEncoding (agree with other PR, should just use UTF8, we can discuss more if you like)
fix some compile errors
add some unit tests (needs work; added to validate changes without making major modifications)
upgrade to newtonsoft.json 6.0.8, xunit 1.9.2 (sort of by accident)

Hey, the PR looks good except for the encoding overloads I'd prefer the syntax I was using already (with the default values). Other than that it looks good.


Is the icon off-center on purpose?

It's kind of activating my OCD ;)

Hehe ... sorry about that :)
Which icon are you referring to ? The main app logo that says "Aural" ?

I have fixed the playing track indicator alignment in the playlist. You can verify it in the upcoming release if you want.
Since I haven't heard back from you, I'm going to assume ^ this is what you were referring to, and close the issue. You can reopen it if this was not it.


Cybersecurity: 1.7 Challenges in Physical Security?

Description
Mention challenges faced by organizations/companies in physical security?
Note:

Please avoid copy/paste, BE YOURSELF
Changes should be made inside the Cyber_Security/ directory & Cyber_Security branch.
Task will be assigned on first come first serve
Check it out Contribution Guidelines

I would like to be assigned this issue

I would like to be assigned this issue

Okay!
Go for it!👍


Add Analytics page

As a user I want to access a page, where I can see various analytics regarding various items.
For Mock-Up scroll down.
Dependencies

#42
#23

Acceptance Criteria

[x] button to access page should be on profile
[x] mobile and desktop version of this page is working
[x] style mostly matches the Mock-Up
[x] page should be available in German and English

Example of Analytics page:

Closed by #249 and #250


Second bug

Fixed Recent Activity story so that the owner of the item was correctly attributed. Also changed some of the language for stories.

Updated this per request per Dan's instructions.


llvmPackages.clang: Drop failing substituteInPlace for CompletionModel.cmake

Fix up #356120 for clang 12. The failing replacement target was removed
between llvm 12.0.0 and 12.0.1.
Ref: https://github.com/NixOS/nixpkgs/pull/348568#discussion_r1894533619
Ref: https://github.com/llvm/llvm-project/commit/3be5dbbc32dd348dddcec9e3baec96b7dcef8a35
Signed-off-by: Peter Waller<EMAIL_ADDRESS>
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Can't go to master, depends on https://nixpk.gs/pr-tracker.html?pr=356120 which is only in staging currently.


could you provide a pretrained model? thank you.I encountered the following problem when I loaded the downloaded model.

I encountered the following problem when I loaded the downloaded model.

I have checked the released model, and "tps.inverse_kernel", "tps.padding_matrix" and so on exist in it. So I am not sure what happened.

@keliangli Have you fixed that problem? I met the same problem but I haven't fixed it yet. Please tell me if you did it. Thank you so much!


Steps towards implementing bindings for the offline audio context

does not build.

See:  https://github.com/ircam-ismm/node-web-audio-api/issues/4

There is still a weird SegFault when the script exits (or maybe when the node is garbage collected which would be more annoying...), but I don't really how to fix that right now
Added a simple example in examples/offline.mjs

@bthj do you want to add/review something? or should I merge?

@b-ma it would be great if you'd merge.  Then I'll test this soon.  Thanks.

Ok, thanks for pushing this forward!


Fixes the references for the next and last icons

Resolves #1
Updated UI:

Hi Bryson, thanks for posting this. I am still seeing the backwards colored buttons in Pagination.class. I'm using this in my pom. How do I switch the buttons? Thank you
<dependency>
           <groupId>com.vaadin.addon</groupId>
           <artifactId>pagination</artifactId>
           <version>8.0.6</version>
       </dependency>

and in Pagination:
 public Pagination() {
        this.firstButton = new Button(VaadinIcons.ARROW_CIRCLE_LEFT);
        this.previousButton = new Button(VaadinIcons.ARROW_CIRCLE_LEFT_O);
        this.nextButton = new Button(VaadinIcons.ARROW_CIRCLE_RIGHT);
        this.lastButton = new Button(VaadinIcons.ARROW_CIRCLE_RIGHT_O);
    }


Promote some Azure account API typings

These have been there a long time and don't seem like something we'll be changing. This also makes it so the getSecurityToken being deprecated doesn't require the user to use the proposed APIs for the replacement.
I also clarified some of the doc comments and while I was there added comments for the token claims stuff (taken from the link provided in the doc comment for the class)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 42.005%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
28035

Relevant Lines:
62119

💛  - Coveralls


Add gflags to Mac build from source instructions

Note that gflags is already listed as something to install under Linux before build from source but was not included on Mac. I just got a reimaged Mac and needed to do this before I could build.

Interesting... I actually reimaged a mac Mac with latest (Apple vanilla) macOS last night and pretty sure did not need to install gflags (to make the default target at least). I believe we bundle it in third_party. Did you have all the dependencies the first time you invoked make? Since I have encountered difficulties trying to run make after installing dependencies with brew if it had once failed before and had to restart from scratch. Did you have submodules initialized before running make for the first time?

Yes, that was the surprising part for me since we definitely have this as a submodule. I had submodules updated properly before running make. My exact sequence of instructions very closely followed the INSTALL.md file:

git clone...
xcode-select --install
(get brew as instructed on brew.sh)
brew install autoconf automake libtool shtool
cd grpc
git submodule update --init
make buildtests
[ Lots of failures looking for gflags/gflags.h ]
brew install gflags
make buildtests
Profit!

Oh, I see, we're talking over each other. You're probably right about the default target. I was trying to build tests because the Linux instructions say "If you plan to build from source and run tests, install the following as well" . I guess we just need a separate category for that in Mac just as in Linux. I'll edit this PR accordingly.

Squashed commits. Merging since tests are irrelevant.


linux sdk fixes: qt6keychain, libkml, linking order

qt6keychain on linux was using libsecret which had the issue that it was using system libsecret instead of the one from vcpkg, but we do not really need it anyway - and we can then also skip linking to Glib which was causing frequent crashes upon app start for me.
app build was then failing when trying to link to libkml, which is not included in our build, so now it is explicitly disabled in gdal's portfile (should not be a problem because there's another KML driver for gdal/ogr that does not rely on libkml).
the linking order for the sdk test was slightly incorrect for geos and minizip, giving me link errors (spatialite needs geos, freexl needs minizip)

ios build fails because of something in homebrew, unrelated to this PR

LGTM


fonts & office photo

hey @Pineapple-1 or whoever, please add the office photo you had as a 0.5MB jpg in /+/office.jpg
and while you're at it, since I'm lazy, I guess there is /fonts and /+/fonts now, can you combine them into just /+/fonts delete /fonts and make sure there is no broken font references? Thanks.

update: I moved them, however... I noticed its messy... some fonts are loaded from google, some aren't, etc. etc. so my request is now: please update such that ALL fonts are loaded from us, NOT google, if possible.


Add: MTV Urheilu 1

Channel Name
MTV Urheilu 1
Alternative Names (optional)
MTV Urheilu 1 HD
Network (optional)
No response
Owners (optional)
No response
Country
FI
Subdivision (optional)
No response
City (optional)
No response
Broadcast Area
c/FI
Languages
fin
Categories (optional)
No response
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://fi.wikipedia.org/wiki/MTV_Urheilu_1
Logo
https://img.prod.telkku.com/channel-logos/MTV Urheilu 1 lightBG.png
Notes
No response

Please update your request:

link to the official website is incorrect
the size of the logo should not exceed 512x512px

https://github.com/iptv-org/database/blob/master/CONTRIBUTING.md#channels

Thank you, @freearhey. Updated the icon url and website url. There is not official channel url so I ended up cleaning that information (https://www.mtv.fi/urheilu might be the closest one for "official" url for it)


Validator default string parsed to number

Describe the bug
The default string value set in Validator is parsed to number, for example:
Validator("version", is_type_of=str, default="0.30", apply_default_on_none=True),
The setting version is expected to be a string "0.30", but will get a number 0.3.
Changing double quote in default="0.30" to single ('0.30') or triple ('''0.30''' or """0.30""") still output a number 0.3.
To Reproduce
Steps to reproduce the behavior:

Having the following folder structure

 Project structure
.
├── app.py
└── settings.toml

Having the following config files:

 Config files
The settings.toml is an empty file.

Having the following app code:

 Code
./app.py
from dynaconf import Dynaconf
from dynaconf import Validator

# Load config files
config = Dynaconf(settings_files=["settings.toml"])

validators = [
    Validator(
        "version", is_type_of=str, default="0.30", apply_default_on_none=True
    ),
]

config.validators.register(*validators)
config.validators.validate()

print(type(config.version))
print(config.version)

Executing under the following environment

 Execution
# other commands and details?
# virtualenv activation?

$ python app.py

After execution, you will get an ValidationError:
Traceback (most recent call last):
  File "app.py", line 16, in <module>
    config.validators.validate()
  File "/opt/homebrew/anaconda3/envs/npl3.10/lib/python3.10/site-packages/dynaconf/validator.py", line 492, in validate
    validator.validate(
  File "/opt/homebrew/anaconda3/envs/npl3.10/lib/python3.10/site-packages/dynaconf/validator.py", line 216, in validate
    self._validate_items(
  File "/opt/homebrew/anaconda3/envs/npl3.10/lib/python3.10/site-packages/dynaconf/validator.py", line 340, in _validate_items
    raise ValidationError(_message, details=[(self, _message)])
dynaconf.validator.ValidationError: version must is_type_of <class 'str'> but it is 0.3 in env main

Expected behavior
The ValidationError is not expected. The value of the setting version is expected to be a string "0.30" but not a number 0.30.
Environment (please complete the following information):

OS: MacOS (M1) 14.2.1 (23C71)
Dynaconf Version 3.2.4

Working on a fix, for now the workaround is to force the cast
validators = [
    Validator(
        "version",
        is_type_of=str,
        default="0.30",
        apply_default_on_none=True,
        cast=str,  # <-- THIS
    ),
]

Ideally it would assume the type passed on is_type_of as the casting type

The final fix is in draft https://github.com/dynaconf/dynaconf/pull/1066 and will be merged only on 3.3.0 because it is a bit of a breaking change if someone is relying on the opposite behavior.
I suggest you to use the provided workaround using cast=str for now.
thanks


ListBox SelectedIndices

Like in WinForms:
https://docs.microsoft.com/ru-ru/dotnet/api/system.windows.forms.listbox.selectedindices?view=netframework-4.8

It should be available inside of Selection, which is protected and can be accessed from custom listbox control
https://github.com/AvaloniaUI/Avalonia/blob/fbb35f530768cc08f9b408be440823e0d043ded8/src/Avalonia.Controls/Primitives/SelectingItemsControl.cs#L246

Selection is exposed in ListBox so should be available. As @maxkatz6 says - use listBox.Selection.SelectedIndexes.


Title Bar Icons Not Displaying Properly

After following all the recommendations prior to installing this version of Notion upon installation I face the following issue: minimize, maximize, close buttons are not displaying properly as well as the icon is missing in the task bar on the bottom of the screen.
Screenshot:

The icons on the top got fixed by themselves. The icon in the bottom got fixed by pinning and unpinning Notion from the taskbar.