[WFLY-16677][WFLY-16733] Fix spec-api module; move building 'ee 8' modules to a profile

This is the same as #15873 but without the final commit that sets up a profile. This isn't my preference but if we decide we don't want the profile, or at least not yet, this does the most important code cleanup parts.
https://issues.redhat.com/browse/WFLY-16677
Also https://issues.redhat.com/browse/WFLY-16733

I put the 'hold' label on this and removed 'Critical' as #15873 has the 'ready-for-merge' label. So no point tracking 2 PRs. This will auto-close when #15873 is merged.


Cross compile to linux 386

Hi,
I'm very new to go, and I have trouble to cross compile a project of mine that uses your library.
When I try to compile for linux 386 (on a linux amd64 host) I get this error
$ GOOS=linux GOARCH=386 go build
github.com/djimenez/iconv-go
../github.com/djimenez/iconv-go/reader.go:10:21: undefined: Converter
go version go1.12.6 linux/amd64
Am I missing a flag, package, library I should install, or I have to get a 32 bit Linux and compile in it. I guess is not really an go-iconv issue but I guess you have knowledge for this issue.
Thanks in advance for your help.
Hans

This library requires CGO
From CGO docs: https://golang.org/cmd/cgo/

The cgo tool is enabled by default for native builds on systems where it is expected to work. It is disabled by default when cross-compiling. You can control this by setting the CGO_ENABLED environment variable when running the go tool: set it to 1 to enable the use of cgo, and to 0 to disable it.

That's probably as much as I can help unfortunately, as I've never tried to cross compile it. As I tell many people who have trouble with cgo or don't realize its required. You should only use this library if you actually need to do conversion with libiconv because of some behavior it has you require. If you don't have such a requirement I suggest you use https://godoc.org/golang.org/x/text/encoding instead - it is pure go.

Thanks, moved my code to https://godoc.org/golang.org/x/text/encoding, as suggested.
Has some encodings missing, but is good for what I need.


upgrade aircompressor to 0.20

Fix #11789
Modification

upgrade aircompressor from 0.19 to 0.20

Move release/2.7.5 label to #16893


Different behaviour from conda/mamba when installing OpenCL

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
$ pixi add ocl-icd-system

Issue description
So the use of OpenCL requires linking the NVIDIA drivers to the Python virtual environment, see https://documen.tician.de/pyopencl/misc.html#using-vendor-supplied-opencl-drivers-mainly-on-linux
Using conda/mamba, this is done automatically when installing
https://github.com/conda-forge/ocl-icd-system-feedstock
However, doing pixi add ocl-icd-system doesn't do this and one is forced to
$ cp /etc/OpenCL/vendors/nvidia.icd .pixi/env/etc/OpenCL/vendors/
by hand.
Expected behavior
pixi should create etc/OpenCL/vendors/ocl-icd-system/nvidia.icd

Thanks for this issue, its good to get visible.
This is a related issue to this issue in rattler. Maybe you could leave a comment there on how you would like this feature to behave?

Closing in favor of: https://github.com/prefix-dev/pixi/issues/1573


How to deploy this app?

I would like to host it on my own server. But I don't figure out how to do it.

I'm sorry but this is so not smth to address here.
You start the the app via npm start and that starts an HTTP server on a port (printed in stdout, and configurable in config.coffee) and then you need to make that port available to the outside world (I'm not going to tell you how because there are a bunch of possible ways to do that, depending on how your server setup looks like)


receive: use async remote writing

Instead of spawning new goroutines for each peer that we want to remote write to, spawn a fixed number of worker goroutines and then schedule work on top of them.
This has reduced the number of goroutines in our case about 10x-20x and the 99p of forwarding dropped from ~30s to just a few hundred milliseconds.

@douglascamata thanks for your review, I have made adjustments and rebased it on top of your recent nice cleanup/refactoring :bow: please take a look when you have some time


DefinePlugin works

https://webpack.github.io/docs/list-of-plugins.html#defineplugin
'ProvidePlugin' didn't work.

Ouch, my mistake. Thanks. :+1:


deleteOrphaned seems to have no effect

🐞 Describe the bug
Despite the inclusion of deleteOrphaned: true in .github/sync.yml, files deleted in the source repo don't get deleted from the destination repo. In addition, folders renamed in the source repo appear twice in the destination repo, with old and new names.
📚 To reproduce
In source repo
# .github/sync.yml
zoylendt/zoylendt.github.io@v4:
  - source: public/
    dest: content/
    deleteOrphaned: true

# .github/workflows/sync.yml
name: Sync Files
on:
  push:
    branches:
      - main
  workflow_dispatch:
jobs:
  sync:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout Repository
        uses: actions/checkout@master
      - name: Run GitHub File Sync
        uses: BetaHuhn/repo-file-sync-action@latest
        with:
          GH_PAT: ${{ secrets.GH_PAT }}
          SKIP_PR: true

Folder structure:
 /
 ├── .github
 │   ├── sync.yml
 │   └── workflows
 │       └── sync.yml
 ├── private
 │   └── .gitkeep
 └── public
     ├── A.md
     └── folder2
         └── C.md

In destination repo
B.md has been deleted in the source repo, folder1 has been renamed to folder2 (both after they have been synced to the destination repo).  However, they don't get deleted from the destination repo.
Folder structure:
 /
 ├── .github
 ├── misc
 │   └── .gitkeep
 └── content
     ├── A.md
     ├── B.md
     └── folder1
     │   └── C.md
     └── folder2
         └── C.md

💡 Expected behavior
Deleted/renamed files in the source repo and folders should be deleted/renamed in the destination repo aswell, including files/folders in subdirectories.
🖼 Screenshots
No response
⚙ Environment
Tested with v1 and latest, as mentioned here: https://github.com/BetaHuhn/repo-file-sync-action?tab=readme-ov-file#versioning
📋 Additional context
No error messages appear in the actions log.
I authenticated using GH_PAT, with full repo scope, see #31
Maybe the issue is related to SKIP_PR: true?

+1 to this bug. I just added deleteOrphaned for a file sync and the sync action ignored it. Despite there being a folder that was removed in the source repository, it still said files were all up to date.
Locally syncing file(s) between source and target repository
Source is directory
Source is directory
File(s) already up to date
Cleanup complete


Fix Login Page View and Add Dashboard Dynamic Capability

Fixes #99 Make TextBoxes and the Submit button visible when software keyboard shows
Add change to dashboard to make more dynamic for different screen aspect ratios

Can you describe what you did to make it better looking/more dynamic?

I redid the xml to resize the space between tabs automatically for bigger screens


[Request]  Please Add EiffelStudio

Package Request
Criteria
For a package to be acceptable in the Extras bucket, it should be:

[x] a well-known and widely used tool (e.g. if it's a GitHub project, it should have at least 100 stars and 50 forks)
[x] the latest stable version of the program

Information
Name: [EiffelStudio]
Description: IDE for Eiffel which is an object-oriented programming language.
Homepage: https://www.eiffel.org/
Download link(s): This download page
Some indication of popularity/repute: /
Relevant WIKIs: This and This too

Thanks

No Problem.


TwitarrTeam cannot create announcements

Related to https://github.com/jocosocial/swiftarr/issues/151
TwitarrTeam cannot create announcements. This may be OK, in which case we should disable the form/link in the UI. Currently the permissions error is not shown until the submit button is pressed.

TT can create, edit, and delete announcements now.


Create a task with log message

Created a task using log message to display the greetings based on the input provided.

@Radha93M
Project Task -1
Code verified.


Console is logged off when connection is made  Win 10 Pro

Console is logged off when you login remotely. You can then log the console back in as a 2nd session.
Windows 10 Pro version 1709 OS Build 16299.248
RDP Wrapper version 10.0.16299.15

I have the exact same issue since upgrading a system to Windows 10 1709, definitely a change since 1703, if you use different user accounts all is fine but if the same account is used the above behaviour occurs.

Same here, #358

Same here too. At the moment, the only workaround to fix this is set the Authentication Mode to GUI/Default

Another weird workaround, I mentioned in #358
If 2 sessions of the same user are logged in, then concurrent users are open with no issues, regardless of method..

@josh900
thank you for the info. is there any command to open a new console session of the same user?

I find setting to GUI/Default does not work, but putting two different local host entries for the same computer and telling RDP these hostnames and saving the credentials in the Windows Credential store for each one rather than in the RDP file works ok.... really bizarre work around for a Windows client but this does not work for my iOS based RDP app (Remotix).

I'm having similar issues. 3 Greens, and [fully supported], but not allowing concurrent sessions. I've down the patch as per the other current issue, and not working.

workarounds, for opening multiple concurrent sessions for a single user - not working (by default?):

select GUI Authentication Only
If 2 sessions of the user are logged in
using default windows rdp (mstsc.exe), after saving credentials for the user

Any others?

This solved my issue: https://github.com/stascorp/rdpwrap/issues/456#issuecomment-388382757

I tried replacing the termserv file as mentioned,but it did not fix my issues of opening concurrent sessions of a single user.
I'm assuming that it fixes another issue, maybe 1803 update.
Does concurrent users for a single user work fine with this termserv patch on 1803?

Still on 1709, but updated rdp wrapper. Though concurrent sessions still logs off existing session.
I'm curious to see if updating to 1803, with an updated rdp wrapper will fix this.

I'm testing the last ini file under Win 10 Home 1803 and whenever i create a new concurrent session of the same user, if there already are some session running, it doesn't display the session's select window anymore for choose which session it must connect. It always starts a new session of the same user.

it doesn't display the session's select window anymore

This select window appears only if you have disconnected inactive sessions.

I have disconnected inactive sessions but it always starts a new one

@graphixillusion.. I want that problem... ;)  I am unable to open concurrent sessions to the same user, without workarounds (after updating to 1709)
fyi, on 1803, concurrent sessions to the same user still does not work by default without workarounds
Anyone know of a way/policy to control this behavior (forcing open a new session, or force open existing session)
or what changed from 1703 to 1709, that caused the this new behavior?

@graphixillusion a
re you saying that #456 solves the issue where the same user can open multiple sessions over rdp using default authentication?

i am having the same issue, when the users close the session its disconnect it and close their sessions.
when i update it using the version 1.6.2 it does works and keeps the sessions active but the printers are not working over the RDP and are not redirected.

Here is the new and old behavior i'm seeing,
When loging in to a user who is already logged in and has an open session.
with the check box for multiple concurrent sessions unchecked
The new behavior (post 1803)
Using gui authentication,
A new session will always open, regardless of whether the existing session is active (in use) or not active (disconnected)
Using default authentication,
The existing session is always opened, with no new session created, regardless of whether the existing session is active (in use) or not active (disconnected)
There seems to be no option to choose the behavior of opening a new session or not, other than the authentication choice which is an odd way to determine the behavior?
The old behavior (pre 1803)
If the current logged in session is active (in use),
A new session will always open, regardless of which authentication method is enforced
If the current logged in session is not active (disconnected),
The existing session is always opened, with no new session created, regardless of which authentication method is enforced
This is preferred, because the result of a disconnected open session, causes behavior of a connecting to the disconnected session.

Is this correct, or is it more complicated?
Does anyone like the new behavior?
Is this a new change to windows? or an issue? what are the chances of the old behavior returning? Is there any way to force connect to existing session or open new session?

I can confirm it:
[  ] Single session per user (Activate multiple concurrent sessions)
Since Windows 10, Version 1803 (Build 17134.48) there is no change to reconnect a disconnected (not active) session while login. A new session will always open.
There is no pop-up "select a session to reconnect..."

Haven't tried yet, but this might be worth a shot. Though I think it is referring to the non same user sessions that was fixed with the last rdpwrap update.
 , also mentioned in #501

I was able to recreate the problem on a second PC with Windows 10 Version 1709.
After Update to Windows 10 Version 1803 it's not possible to connected a disconnect session.
A new session will always open.
I found a workaround:
If you lock the screen before disconnecting RDP, then you can reconnect the session while login.
the steps with cmd.exe:
first find the session id from the disconnected session

query session
... connect the session (sample 3) to console
tscon 3 /dest:console
... lock the screen with sysinternal psexec ;-)
PsExec.exe -s -i 3 -d C:\Windows\System32\rundll32.exe user32.dll LockWorkStation
... now disconnect the locked session from console
tscon 1 /dest:console

Does anyone know if this change happened in windows server remote desktop services?
https://github.com/stascorp/rdpwrap/issues/422#issuecomment-397284365
#513


[tensorflow] Upgrade tensorflow to 2.7.0

Change-Id: I20ba464309c52564cb75698de72433b9749841bf
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1462 (b4ba395) into master (bb5073f) will decrease coverage by 1.42%.
The diff coverage is 48.19%.

@@             Coverage Diff              @@
##             master    #1462      +/-   ##
============================================
- Coverage     72.08%   70.66%   -1.43%
- Complexity     5126     5297     +171
============================================
  Files           473      493      +20
  Lines         21970    23222    +1252
  Branches       2351     2533     +182
============================================
+ Hits          15838    16410     +572
- Misses         4925     5534     +609
- Partials       1207     1278      +71

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <ø> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../modality/cv/translator/YoloTranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...i/djl/modality/cv/translator/YoloV5Translator.java
5.69% <0.00%> (ø)

...odality/cv/translator/YoloV5TranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...pi/src/main/java/ai/djl/ndarray/BytesSupplier.java
54.54% <0.00%> (-12.13%)
:arrow_down:

...i/src/main/java/ai/djl/ndarray/NDArrayAdapter.java
15.23% <0.00%> (-0.83%)
:arrow_down:

...l/training/loss/SigmoidBinaryCrossEntropyLoss.java
64.00% <0.00%> (ø)

... and 162 more

Continue to review full report at Codecov.

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


(juju) Fixes Update.ps1 to handle missing release notes

Description
This fixes the update script so it doesn't choke if it can't find release notes within the GitHub release body.
Motivation and Context
The build was failing due to passing an empty string to Update-Metadata. This should fix that issue.
Please note that though I added a fallback method ("if the GitHub release exists but there is no link found, link to the GitHub release") in theory if the release page doesn't exist the releasenotes may point to an older version. We might consider updating Update-Metadata to accept blank strings and handle itself appropriately in that situation.
The example of the failing release is 'https://github.com/juju/juju/releases/tag/juju-3.3-beta1'
How Has this Been Tested?

Ran upgrade_all.ps1 -Name juju before and after changes were made

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
~[ ] New feature (non-breaking change which adds functionality)~
~[ ] Breaking change (fix or feature that would cause existing functionality to change)~
~[ ] Migrated package (a package has been migrated from another repository)~

Checklist:

[x] My code follows the code style of this repository.
~[ ] My change requires a change to documentation (this usually means the notes in the description of a package).~
~[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).~
~[ ] I have updated the package description and it is less than 4000 characters.~
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review


CanvasKit crashing when calling Vertices.raw without colors

Calling Vertices.raw with null colors crashes.
Due to this line: https://github.com/flutter/engine/blob/master/lib/web_ui/lib/src/engine/compositor/vertices.dart#L8
Colors should be optional for Vertices.raw, we are calling .length on the optional color array.

Could it be the root of https://github.com/flutter/flutter/issues/56938?

Fixed in https://github.com/flutter/engine/pull/19172


Can not nativefier Gitbook due to Google OAuth

Description
After linking to "https://gitbook.com" in nativefier, the sign-in page does not work when using Google's oAuth sign-in feature. An error related to a missing "href" is disclosed.
Steps to reproduce issue

Nativefier https://gitbook.com
Attempt to "Sign in" via Google Account option

Node: v10.15.3
OS: MacOS

@a-marionette can you confirm if this issue still exists now that we support known login urls by default in the current version?

Closing as I have not heard a response back. We can reopen if anyone is still experiencing this.


Adding lambda version parameter to outputs.tf

Some projects need lambda version to associate it some resource

Hi @ivankatliarchuk ,
It would be great to use terraform v12 feature about exporting whole object in output.tf like:
# Configuration for Terraform 0.12

output "lambda" {
  value = aws_lambda_function.example
}

i changed it but some checks are failing


Build fails due to GLM/GLX experimental flags missing

Compilation failed due to using glm/gtx/transform without proper enabling of the glm experimental extensions.
[ 24%] Building CXX object chapter02/CMakeFiles/chapter02.dir/scenetwoside.cpp.o
In file included from /home/marius/Downloads/glslcookbook-master/chapter02/scenetwoside.cpp:14:0:
/usr/local/include/glm/gtx/transform.hpp:23:3: error: #error "GLM: GLM_GTX_transform is an experimental extension and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL before including it, if you really want to use it." # error "GLM: GLM_GTX_transform is an experimental extension and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL before including it, if you really want to use it."
To fix this I added this macro to the cookbookgl.h file (which seems to be included by all other main files)
#define GLM_ENABLE_EXPERIMENTAL
My system configuration:
OS: Ubuntu 14.04 64 bit
GLM: v<IP_ADDRESS>
GLFW3: v3.2.1

Thanks for the report!   It looks like this was just added in GLM 0.9.9:
https://github.com/g-truc/glm/blob/master/manual.md#-74-should-i-use-gtx-extensions
I think that we should remove all use of GTX extensions.  There's no need to use them.  I think I just found them more convenient.

As of 37903a2c93902639177a30c3e7e6984a691cdc06, I've removed all use of GTX extensions.


non-URI characters should be excluded from the link parser

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
Currently when you use a valid tld, the link parser doesn't check whether the domain characters are valid or not.
There are really 2 approaches you could go with this, you can either URI encode the characters within the white-space, imo that's unnatural and doesn't feel right for a chat program. The better thing to do is just exclude them like webchat does.
Screenshots
c2:

webchat:

OS and Chatterino Version
Chatterino Nightly 2.4.4 (commit 381d5c42) built on 2023-08-14 with Qt 5.15.2, MSVC 193532217 Running on Windows 10 Version 2009, kernel: 10.0.22621

Added a PR to help with this #4778

I would prefer if the approach to fix this tackled the most user-facing problems, compared to patching tiny situations where a non-existent link is shown. Though this would likely require more of a refactor as it would result in two message elements.
Similar to the above mentioned () problem, links that trail with . are a more common user-facing problem. When this happens it's often due to a user writing a bot command, and ending the sentence with a ., which browser chat will ignore but Chatterino does not. In this case unless the link host handles this, the link will fail to load.
examples that automatically handle this problem:
youtube
examples where the link fails to load
github
twitch
runelite's plugin-hub
dashduck's website | working version https://drscovy.com/raids

I think parsing a link like this should work:
prefix                   link                          suffix
╭┴╮╭───────────────────────┴───────────────────────────╮╭┴─╮
 (  https://  foo.bar.baz.app  /my/path?query=alien#abc  ).
   ╰───┬────╯╰──────┬────────╯╰───────────┬────────────╯
    protocol      host                  rest

This could be stored in a result like this[^1]:
struct Link {
    QStringView prefix;
    QStringView link;
    QStringView suffix;

    QStringView protocol;
    QStringView host;
    QStringView rest;

    QString source;
};

If prefix isn't empty, a text element is added for that. Then the link is added and finally the suffix. This way, the link would be properly highlighted.
The possible prefixes should be (. The possible suffixes should be ), ., ,, ). ),.
[^1]: 🤓 On Qt 5.12 the string-views would be refs, because string-view support there is less than minimal (or lock link parsing to Qt 5.15 and up, so people switch to these versions and support for 5.12 can be dropped).


Monte Carlo Mode for the velocity kernel

Has this been fixed to be a proper Monte Carlo method to propagate a distribution through a function? Please see the pdf I send you in slack for details on this.

Implemented via https://github.com/theislab/cellrank/pull/301, here's the code if you want to do sanity checking: https://github.com/theislab/cellrank/blob/master/cellrank/tl/kernels/_velocity_kernel.py#L479

Let's go though this in our call today. This is important.

We can easily test this mode by comparing with Jax. As I said before, Jax is an approximation, Monte Carlo converges to ground truth for n -> infinity. So we don't expect to get exactly the same transition probabilities, but they should be similar compared to the deterministic transition probabilities.


Can't select time zone in position config

Describe the bug
In the edit position-config node the TZ Offset selection is disabled (gray), and thus I can't change it from Operating system standard
Screenshots

System information (please complete the following information):

Node Version 1.1.3
Node-Red Version 1.0.6

Additional context
I think this got broken here:
https://github.com/rdmtc/node-red-contrib-sun-position/commit/b918421ed093ff9b45c0b0eaf4ab2067aa07d198#r39883569

The time zone selection is disabled, because it is not fully tested.


Update vendor deps

Towards https://github.com/giantswarm/giantswarm/issues/8436
Pinning apiextensions to dep-compatible versions.

Going with a ping.

What is the plan here? I think this breaks the operators now as 0.1.0 doesn't have the CRD management capabilities I merged into master and cluster-operator and aws-operator need that when they get released next as for preparation of HA Masters. This is for the Control Plane CR management. With this change here the whole thing gets rolled back when I understand it correctly.

Are you talking about https://github.com/giantswarm/k8sclient/pull/31? Or something in apiextensions? It's not clear to me which change is missing or rolled back with this PR.

Just checked it out. https://github.com/giantswarm/apiextensions/pull/349 got merged 4 days ago. 7 hours ago the tag got created, so we are good here. I wasn't sure how the timing was. Should have checked in the first place. Thanks for getting back.

Okay, cool. Thanks for checking on it--wouldn't want to break anything. I tried to wait to tag until a few of the important PRs made it into apiextensions master.


ImportError: No module named enum

eh... help? I'm new-ish to linux, so naturally I have no idea what to do or what this means.
can anyone help me out here?
When I type "sc-xbox.py enable" :
File "/usr/local/bin/sc-xbox.py", line 27, in
from steamcontroller import
File "/usr/local/lib/python2.7/dist-packages/steamcontroller/init.py", line 24, in
from enum import IntEnum
ImportError: No module named enum

What version of Python do you have? I'm guessing you might be using an older version, which doesn't work because enum is a module that was added in Python v3.4.

As stated in the readme for python for python 2.7 you must install python enum backport:
pip install enum34
This dependency should be added in setup.py if python version < 3.4.

yeah thanks, after looking up and down the page I saw that it was a required thing. Thanks though!


How to build softgym quickly

According to #9, in ubuntu 18.04, I have to jump in and out of docker to compile and run, which is inefficient. Is there a one-line command or pycharm configuration method that can help us build quickly?

A simple workaround is to open two terminals and keep the docker oopen in one of them


Documentation fix: '~/.config/youtube-dl/config' -> '~/.config/youtube-dl.conf'

README.md lists the user-wide configuration path incorrectly. Manual testing reveals it to be ~/.config/youtube-dl.conf, and not ~/.config/youtube-dl/config. The PR fixes the documented file path.

youtube-dl supports both paths, but ~/.config/youtube-dl/config is preferred, and therefore documented.
Can you share your test? Works fine for me:
$ mkdir -p ~/.config/youtube-dl
$ echo '--verbose --age-limit 199' > ~/.config/youtube-dl/config
$ youtube-dl -v
[debug] System config: []
[debug] User config: [u'--verbose', u'--age-limit', u'199']
[debug] Command-line args: [u'-v']
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968, pref ANSI_X3.4-1968
[debug] youtube-dl version 2016.01.27
[debug] Python version 2.7.10+ - Linux-3.18.25-x86_64-with-debian-stretch-sid
[debug] exe versions: ffmpeg 2.8.2-1, ffprobe 2.8.2-1, rtmpdump 2.4
[debug] Proxy map: {}
Usage: youtube-dl [OPTIONS] URL [URL...]

youtube-dl: error: You must provide at least one URL.
Type youtube-dl --help to see a list of all options.

@phihag your demo does indeed work on my system, and I can't reproduce the problems I was having earlier when using the ~/.config/youtube-dl/config path. I must have made a typo somewhere.
Apologies for the inconvenience.


Receiving "no tables found in database" error while database contains tables

If you're having a generation problem please answer these questions before submitting your issue. Thanks!
What version of SQLBoiler are you using (sqlboiler --version)?
SQLBoiler v2.4.0
If this happened at generation time what was the full SQLBoiler command you used to generate your models? (if not applicable leave blank)
sqlboiler postgres
If this happened at runtime what code produced the issue? (if not applicable leave blank)
What is the output of the command above with the -d flag added to it? (Provided you are comfortable sharing this, it contains a blueprint of your schema)
Please provide a relevant database schema so we can replicate your issue (Provided you are comfortable sharing this)
CREATE TABLE IF NOT EXISTS users (
    id serial NOT NULL PRIMARY KEY,
    email varchar(256),
    password_hash text,
    username varchar(256)
);

Further information. What did you do, what did you expect?
My sqlboiler.toml file is in the current directory of where the command is being run and appears as such:
blacklist=["migrations"]
[postgres]
  dbname="lhdb"
  host="localhost"
  port=5432
  user="lhuser"
  pass="lhpass"
  sslmode="disable"

As shown in the picture below, I can connect as the user and see the table.

Perhaps I'm missing a configuration piece or there is something missing? I couldn't find anything related to the error other than the line of code that prints it out.
All access on the database has been granted to the lhuser user.

I made another database that it appears to work fine with. Though I'm not sure the actual cause of the issue (since I can login as the user and see the tables). It might be specific to databases created with https://github.com/rubenv/sql-migrate.

Figured it out.
Logging in with psql -d lhdb -U lhuser to test wasn't working. Had to use psql -d lhdb -U lhuser -W to force the password prompt, then it showed why the tables were restricted in psql. What a weird user experience.
The actual issue was that even though the user had all privileges on the database, the user needed priviliges to all tables in the schema.


Remove dss callback

This PR removes the dss callback, which is never used (the timestepper calls dss!)-- we now kind of depend on these post-explicit and post-implicit callback hooks, so there's probably a few more things that we can clean up. Also, I don't think it's likely that we can ever go back to using OrdinaryDiffEq with these requirements, so we can probably remove some of that code, too.

bors r+


EBS root volume resizing doesn't work on m5.* instances

Issue description
After deploying a NixOS machine with
      deployment.ec2.instanceType = "m5.large";
      deployment.ec2.ebsInitialRootDiskSize = 30;

the root filesystem is only 2 GB:
/dev/disk/by-label/nixos  2.0G  765M  1.1G  42% /

though the disk has the correct size:
# cat /proc/partitions
...
 259        0   31457280 nvme0n1
 259        1    2096128 nvme0n1p1

NixOS 17.09.2356.cb751f9b1c3, AMI ID ami-d40185ad.
The probable fix is that we need to point grow-partition.nix at the right device. From the boot log:
Jan 04 11:06:48 localhost stage-1-init: skipping unknown device type /dev/xvd[abcde]*

Has this gotten solved? I've run into the same issue after installing Nix 2.0 and the newest nixops 2 days ago, spinning up a m5large.

@edolstra a little confused at what's going on. If I look at the code that produces the message you pasted, it's definitely wrong and needs to be updated to support the NVME devices, but also isn't responsible for resizing partitions:
https://github.com/NixOS/nixpkgs/blob/eddf30cc93e1b778b6bc10d0eb1bb175385d034f/nixos/modules/virtualisation/amazon-image.nix#L94-L109
On the other hand, the partition resizing code seems fairly innocent:
https://github.com/NixOS/nixpkgs/blob/eddf30cc93e1b778b6bc10d0eb1bb175385d034f/nixos/modules/system/boot/grow-partition.nix#L31-L39
Meaning that it looks up the disk by label and resizes whatever that ends up being.
I did also spot this other place that needs updating for grub in particular, but that doesn't seem like it would prevent disk resizing:
https://github.com/NixOS/nixpkgs/blob/eddf30cc93e1b778b6bc10d0eb1bb175385d034f/nixos/modules/virtualisation/amazon-image.nix#L47

Ah, I think the issue is described in detail in https://github.com/NixOS/nixpkgs/issues/33092


[FLINK-12562][table] Improve code in ExpandColumnFunctionsRule

What is the purpose of the change
This pull requests resolve IntelliJ warnings in ExpandColumnFunctionsRule. Also, change TableException to ValidationException for the validation as ValidationException is used to indicate something wrong from users while TableException means unsupported stuff or some weird state for Flink.
Brief change log

Change TableException to ValidationException for validation.
Fix code warnings.
Adjust tests.

Verifying this change
This change is already covered by existing tests, such as ColumnFunctionsValidationTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot attention @twalthr @sunjincheng121

@twalthr Hi, thanks a lot for your suggestions. I have updated the PR according to your comments. Sorry for the late response.
Best, Hequn


Short-term deliverables for dev summit

Accel

[ ] user can add a dependency to sky settings from a foreign ship and poke said settings into their own sky settings, refresh the page and see the settings applied
[ ] adding a dependency that does not resolve doesn't introduce any show-stopping behavior

Docs

[x] add everything that is not "introduction" and "feather" to a folder called "tutorials" within imp/home init sequence; remove "chapter 0" from introduction title
[x] add to dev setup common troubleshooting steps, such as |nuke %neo and |start %neo, take note of other common troubleshooting patterns that occur during today's workshop and include those as well.
[x] comb through docs, remove any language that is self-deprecating and replace it with descriptions of problematic phenomena
[x] remove the .md docs files from /fil. the .txt ones are the up-to-date ones
[ ] stretch goal: show how to inspect state of counter with debug output sample
[ ] stretch goal: Sky usage guide by migrev
[ ] stretch goal: sur/neo types, working with piths, pishes, ports, welding and snoccing piths

Messenger

[x] automatic refreshing of received messages
[x] scrolling through long chat histories
[x] clean up timestamps
[x] auto-scrolling within long chat histories
[x] starting a chat with existing counterparty doesn't overwrite anything

Sky

[ ] create new sky pill at end of workshop, include link to sky pill in environment setup doc
[ ] stretch goal: merge in c1-demos branch, create new pill with this included

[ ] Need a warning about infinite loops somewhere

We should really get the terminal output quieted down before release.

We should really get the terminal output quieted down before release.

This is allegedly just a single flag that can be flipped during the desk shrub release

Current status on the incompleted sub-issues here:

When we try to add a remote dependency in accel, we get some gargantuan error output, similar to the errors we were getting in Messenger yesterday prior to hastuc's fix, but continuous and longer. @liam-fitzgerald some help with this and the point below would be great.
Related, there seems to be a lot of extraneous error output related to dependencies in general; our hypothesis is that %tomb-ing a hawk-eyre-handler doesn't tell its dependencies to stop poking it with relies, leading to compoundingly long load times the longer you use Sky.
Adding items to a folder within a folder in this specific way would require changing imp/folder a bit, didn't seem like the best use of time today.
It probably makes more sense to wait to make a new Sky pill until after everything is ready for launch, otherwise we'll have to re-do it anyway.

We also need to make sure to flag the possibility of infinite loops before releasing this to people's real ships.

Dev summit has passed, if we are going to do any of these they are going to be new issues


Support embedding FlutterView in an Android Fragment

Android fragments have a different lifecycle than activities. It would be common for existing apps to add FlutterView to their fragments and expect all the fragment transactions to work properly. Just like we have an FlutterActivityDelegate, we likely need a FlutterFragmentDelegate that wires in all the lifecycle events to FlutterView.
In a recent hackathon with an internal team, we discovered that the app hangs while transitioning out of a fragment containing a full screen flutter view to a fragment that contains native views. While this could not be reproduced in a simple app, the problem was resolved by calling view.destroy() and view.detach() before the fragment was removed via FragmentManager. Developers should not need to discover this on their own.

I want 'FlutterFragment' also...

This has landed :-)

See this: https://github.com/flutter/flutter/wiki/Add-Flutter-to-existing-apps


Update Composer dependencies (2021-06-20-00-07)

Visual regression test passed!
View the visual regression test report


Add a job to automatically update nix flake

I'm not familiar with Github's workflows but it seems straightforward. Hyprland has it: https://github.com/hyprwm/Hyprland/blob/main/.github/workflows/nix-update-inputs.yml.

Seems good, except for the commit message (we're using the gitmoji standard) and the PAT secret (it seems unnecessary for cloning, i think we can just remove it)
Apart from that we might be able to just copy the file


Interval on computed property macros fromNow and toNow

There appears to be a way to update the from now and to now helpers at a given interval but I don't see a way to do the same for the computed properties.

There isn't an API for this for now, but I would live to see this feature as well.

There appears to be a way to update the from now and to now helpers at a given interval

How can I do that?

Ok, this undocumented approach worked for me:
{{moment-from-now (moment timeMs) (now interval=1000)}}

This also works, kinda documented:
{{moment-duration (moment-diff (moment timeMs) (now interval=1000))}}

@lolmaus feel free to send a PR to update the docs where you feel it's lacking.

computed property macros have been removed in v9


Reference DigestValue and  SignedInfo SignatureValue

Hi guys, first of all I would like to thank you for the work you have done to build this tool, I am trying to sign some documents, however I have problems when the summary of the reference URL is generated, unfortunately the entity where I need to send the documents does not have documentation in this regard.
XML BASE
<?xml version="1.0" encoding="utf-8"?>
  <SemillaModel xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
    <valor>KNthlAufp9LwXC3FmU1p2NcMOMKXhut0qq+Tjhfn1FL2abSxvr3m98tpkp+A6wzciqiRgzNA8m6HwbydbAePupf2itlvo+0OlLDVFGacmFLnP+9694AENSERjM48TcUIqVW7oXvVpqgjjKo9xGvYxw==</valor>
    <fecha>2022-10-04T13:51:12.0942715-04:00</fecha>
  </SemillaModel>

XML RESULT
<?xml version="1.0" encoding="utf-8"?>
<SemillaModel xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
  <valor>KNthlAufp9LwXC3FmU1p2NcMOMKXhut0qq+Tjhfn1FL2abSxvr3m98tpkp+A6wzciqiRgzNA8m6HwbydbAePupf2itlvo+0OlLDVFGacmFLnP+9694AENSERjM48TcUIqVW7oXvVpqgjjKo9xGvYxw==</valor>
  <fecha>2022-10-04T13:51:12.0942715-04:00</fecha>
  <Signature xmlns="http://www.w3.org/2000/09/xmldsig#">
    <SignedInfo>
      <CanonicalizationMethod Algorithm="http://www.w3.org/TR/2001/REC-xml-c14n-20010315" />
      <SignatureMethod Algorithm="http://www.w3.org/2001/04/xmldsig-more#rsa-sha256" />
      <Reference URI="">
        <Transforms>
          <Transform Algorithm="http://www.w3.org/2000/09/xmldsig#enveloped-signature" />
        </Transforms>
        <DigestMethod Algorithm="http://www.w3.org/2001/04/xmlenc#sha256" />
        <DigestValue>jrXIlDuc4Bta8qTeBmxSTEJqRL4IRkf6eIGNFbxmE+Y=</DigestValue>
      </Reference>
    </SignedInfo>
    <SignatureValue>ofQ7kCSXLrxiQZ1DzDPKTs4ZI2+YeyTZGNIyUHlBRfw4jGLeKYy8YBw1qOKS/BM9NyMHB8hH2Y8L0AVUBpVl1Jbbea1bZfeRqvsmUKEEeIJQSDEZ3zvChhWb5J+xNJF3G1Zm7XyDCBklXCamV4djwf5d/ZwtwZ+dDecQJokc4GsLyfp3Qz+gXhmRBAs64Bai+vMfP50fwrtX8+S+UqhYCOfDVBam6YgNsGbEsCragvkYroYyW3Qc6qbpqm0GFVyjB3iAir46vd9n0quoq1KzkeHq4DIWpiPnwNj11dpdsNPq/7JY2+iNumak9oaIG8yn/MGVQj2v0jEsVAjVnJb67w==</SignatureValue>
    <KeyInfo>
      <X509Data>
        <X509Certificate>......</X509Certificate>
      </X509Data>
    </KeyInfo>
  </Signature>
</SemillaModel>

The entity has a software where it gives the result in this way, I understand that by doing the same operations and applying the same algorithms, it should give me this result as long as the document has not been modified, however when signing with xml-crypto, it gives me a different result for hashes and digestvalue
    console.log('Creando certificado Firmado');
    var sig = new SignedXml()
    sig.digestAlgorithm ="http://www.w3.org/2001/04/xmlenc#sha256";
    sig.signatureAlgorithm = "http://www.w3.org/2001/04/xmldsig-more#rsa-sha256"
    sig.keyInfoProvider = new keyProvider(public_key)
    sig.canonicalizationAlgorithm = "http://www.w3.org/TR/2001/REC-xml-c14n-20010315"

    sig.addReference(`/*`, ["http://www.w3.org/2000/09/xmldsig#enveloped-signature"], "http://www.w3.org/2001/04/xmlenc#sha256", "", "", "", true)

    sig.signingKey =signingKey;
    sig.computeSignature(xml)
    fs.writeFileSync(xml_sign, sig.getSignedXml())

    console.log('Certificado firmado guardado en',xml_sign);

RESULT WITH xml-crypto
<?xml version="1.0" encoding="utf-8"?>
  <SemillaModel xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
    <valor>KNthlAufp9LwXC3FmU1p2NcMOMKXhut0qq+Tjhfn1FL2abSxvr3m98tpkp+A6wzciqiRgzNA8m6HwbydbAePupf2itlvo+0OlLDVFGacmFLnP+9694AENSERjM48TcUIqVW7oXvVpqgjjKo9xGvYxw==</valor>
    <fecha>2022-10-04T13:51:12.0942715-04:00</fecha>
  <Signature xmlns="http://www.w3.org/2000/09/xmldsig#">
  <SignedInfo>
  <CanonicalizationMethod Algorithm="http://www.w3.org/TR/2001/REC-xml-c14n-20010315"/>
  <SignatureMethod Algorithm="http://www.w3.org/2001/04/xmldsig-more#rsa-sha256"/>
  <Reference URI=""><Transforms><Transform Algorithm="http://www.w3.org/2000/09/xmldsig#enveloped-signature"/>
  </Transforms><DigestMethod Algorithm="http://www.w3.org/2001/04/xmlenc#sha256"/>
  <DigestValue>BMBZVPKXYm5pHI06/1AEr5zodueiXoOQgLbHjlEwqLc=</DigestValue>
  </Reference>
  </SignedInfo><SignatureValue>OT0HezpWgEZQIfzVcsS77sxjA7KnUYLflTJqrKOSDiRdQxTaSqxcyeOZVPLhU1iyFWdUpsaZx+vYkkCbHtG1woGfurym3BNsLdafGpzhzY3uGIqehhx1v34EzrVkSh//mobg6Hpl1VBNXLEW/xW1SgXqh6cfuDzoBzGB0wIPQbB3jNQZMmsDg5jdpupTtR97N1Cx/yP9TPFaC5+a80Or0r+/vP3SQH3sd2IyHcHRc1HNZ7ifFDs5uwx/gwy3b6boYMVA72rfT0/w0T8Cts0iFdt1LtkRAtA4Xg22MVnH+HNmvQ9GIjXMcMbHLeAVJ3tdcdcC7wMRaZDgbzS1mJlAPw==</SignatureValue>
  <KeyInfo><X509Data><X509Certificate>....</X509Certificate>
  </X509Data></KeyInfo>
  </Signature></SemillaModel>

As you can see the digestvalue is BMBZVPKXYm5pHI06/1AEr5zodueiXoOQgLbHjlEwqLc= , but the result should be jrXIlDuc4Bta8qTeBmxSTEJqRL4IRkf6eIGNFbxmE+Y=
and since the SignatureValue depends on the latter, I understand that this will also be different
Try to sign it with other tools in another language like .net and the digest value ends up being this bE1UYd1MsS1yd5vsg/a+eErotsoUfhIOQ3ZHRnW3lj0=
Another clarification when I sign using the tool, when validating I have noticed that if I only have one transform when validating it adds another one by default and they end up being 2 , this breaks the validation using signed-xml
  signed-xml.js  // line 685
  if (transforms.length === 0 || transforms[transforms.length - 1] === "http://www.w3.org/2000/09/xmldsig#enveloped-signature") {
      transforms.push("http://www.w3.org/TR/2001/REC-xml-c14n-20010315")
  }

My question is, why does the digest value vary, when I change between tools, if in the end they end up being the same algorithms
Update1
I have noticed that everything depends on the canonXml, since this variable is used to make the signature of the reference, when I validate, for some reason due to the default behavior, it ends up changing the order of my headers, which breaks the validation
Original
  <SemillaModel xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
validation when adding a transformation by default    signed-xml.js  // line 685 change  order
 <SemillaModel xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" >

Hello @dbrab ,
I stumbled upon the same and decided to spend some time investigating the issue as it is critical for node-saml and passport-saml projects.
Here are my findings, indeed the final canonXML that is hashed to compute the digest is wrong because it contains the signature it self!
So I dug a bit more into exactly what happened, and the canonXML that is based on the fullXML from the currentNode goes through the listed transformers (max 2 transformers).
The important transformer as you may guess is the http://www.w3.org/2000/09/xmldsig#enveloped-signature which basically tells that the signature is included in the signed node.
After logging some fields in the transformer here is what i found >>>

Two strings that are strictly identical when printed but return falsy.
The answer is indeed hidden out there, as always, and the fix is the following :
.replace(/\r?\n/g, '')

Now if we compare the 2 strings without carriage return, boom it works like a charm!
I'll link the PR bugfix and hope it is merged soon :+1:

hello friend check your change add or not that patch does not change the final result of the signature that I have however I notice something strange it is supposed that if the algorithms are the same and use the same standard everything should be fine, so change the tools (start session in another language or another library as long as it follows the standard), but it seems to give me a different summary here than if I used the native .net cryptographic tools or the Java libraries
.
It doesn't validate me either, because it adds a transform by default when validating and this changes the structure of my initial signed xml, the cannonxml.
So I'm between a rock and a hard place, I think I'm missing something, or does the entity I'm trying to send the certificates to have their own transformation implementation that doesn't follow the standard or does each lib use a different standard? , what do you think ?

@Mikescops Currently the only way I have to sign xml is using my entity tool, but this does not allow me to automate the process, I hope that someone with more understanding in this can give me some clue, if it is my problem, that of the entity, each tool has its own implementation, etc.

How does your signature compare with one generated over at SAML Tool? They have been my go-to reference for a long time when dealing with SAML and have never led me astray.

Ah sorry I misunderstood your initial thread, I thought you were validating the signature not building it 😅

I have not used the SAML Tool, but I will take a look at it, I am building and validating the signature, only the problem I have is that I do not know if the algorithm they use is wrong, I am wrong or something strange is happening.

Closed due to inactivity. Feel free to reopen if the issue persists.


energy_model: Fix read-multiple-files helper

This helper executes a "cat" command to read a large number of files
in one go. In order to avoid this command becomind too long for ADB
to cope with, it was written to use glob patterns to cat the files,
instead of the explicit paths (which it knows).
However a user has reported an issue with this approach, which I
suspect is because the order of the paths returned by find differs
from the order of the shell's expansion.
Fix the cat command to explicitly list the paths to be read. In order
to avoid the command-too-long issue, write the potentially-too-long
command to a file, push it, and execute it via sh.

LGTM

Thanks for the review. Before I merge I'm gona come back and test/review this again tomorrow with fresh eyes because I wrote this patch all in one go then found that it worked the first time I ran it... too good to be true.

Is not something like this robust enough?
adb shell 'find /sys/devices/system/cpu/cpu**/topology/*_siblings | while read FILE; do echo -n "$FILE: "; cat $FILE; done'

Which returns:
/sys/devices/system/cpu/cpu0/topology/core_siblings: 0f
/sys/devices/system/cpu/cpu0/topology/thread_siblings: 01
/sys/devices/system/cpu/cpu1/topology/core_siblings: 0f
/sys/devices/system/cpu/cpu1/topology/thread_siblings: 02
/sys/devices/system/cpu/cpu2/topology/core_siblings: 0f
/sys/devices/system/cpu/cpu2/topology/thread_siblings: 04
/sys/devices/system/cpu/cpu3/topology/core_siblings: 0f
/sys/devices/system/cpu/cpu3/topology/thread_siblings: 08
/sys/devices/system/cpu/cpu4/topology/core_siblings: f0
/sys/devices/system/cpu/cpu4/topology/thread_siblings: 10
/sys/devices/system/cpu/cpu5/topology/core_siblings: f0
/sys/devices/system/cpu/cpu5/topology/thread_siblings: 20
/sys/devices/system/cpu/cpu6/topology/core_siblings: f0
/sys/devices/system/cpu/cpu6/topology/thread_siblings: 40
/sys/devices/system/cpu/cpu7/topology/core_siblings: f0
/sys/devices/system/cpu/cpu7/topology/thread_siblings: 80

From there you can easily use a regexp to exactly parse IDs and corresponding values?
As an aside comment,it seems that the read_multiple_oneline_files() function is used only a couple of time and always to parse em data... if we really wanna got for a script based solution, should not be better to have a new EM related set of functions directly integrated into devlib::shutils?
There are already other  examples of devlib::shutils which parse multiple files are return a single output which is also easy to parse from the python side... for example, mayber we can code a shutil which returns (in the previous example) something like:
cpu0 core_siblings 0f
cpu0 thread_siblings 01
cpu1 core_siblings 0f
cpu1 thread_siblings 02
cpu2 core_siblings 0f
cpu2 thread_siblings 04
cpu3 core_siblings 0f
cpu3 thread_siblings 08
cpu4 core_siblings f0
cpu4 thread_siblings 10
cpu5 core_siblings f0
cpu5 thread_siblings 20
cpu6 core_siblings f0
cpu6 thread_siblings 40
cpu7 core_siblings f0
cpu7 thread_siblings 80

Hmm, yeah I think a generic "shutil" that reads loads of files in one go is a good idea.. I will try to cook something up.

Aside from the issue of where the script should live, I think its faster if you cat a bunch of files at once (in one command), than using find to find the files, and then cat them individually so I like that in @bjackman 's initial patch. (I could missing context so in that case I am sorry about making a silly comment)

Updated with a version somewhere in the middle - a bit like @derkling solution but using xargs instead of while read to reflect @joelagnel's point (which is very true - I tested it and a single cat command is indeed massively faster). However I've kept it as two separate commands in an attempt to detect the particular failure mode where the files are not present (rather than a blanket catch of TargetError).


Using with CoffeeScript

I have a file containing CS called test/test.coffee
When passing this to mocha-webpack with mocha-webpack --webpack-config config.js test/test.coffee the file seems to compile fine using webpack and ends up in .tmp/mocha-webpack but it keeps the .coffee file extension, even though it contains js.
The whole thing crashes with Error: Use CoffeeScript.register() or require the coffee-script/register module to require .coffee files., I assume because it is trying to require a file with a .coffee extension.

mocha-webpack --require coffee-script/register --webpack-config config.js test/test.coffee

The entry handling was completely refactored for 1.0.0 and this issue does no longer occur. You can give it a with v1.0.0-beta.1


Short solution needed: "HTTP protocol usage example" (manticoresearch)

Please help us write most modern and shortest code solution for this issue:
HTTP protocol usage example (technology: manticoresearch)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

HTTP protocol usage example


Echelon availability vs Quantity

Good afternoon,
From what I can tell the availability of amplifier B (Echelon) is not enforced correctly in JMSML as per table VII of MIL-STD-2525D. Table VII shows that amplifier B is valid for Units and Activities. In contrast, amplifier C (Quantity) is valid for Equipment. As an example, consider the following two SIDCs:
10031001111800001212 (Land Unit)
10031501112200000500 (Land Equipment)
The first should display amplifier B, whereas the second shouldn't. Both however do. Interestingly the Echelon graphic for the second touches the symbol frame which is not consistent with Figure D-3 of the spec, although ultimately it shouldn't be displayed at all.
Are you able to verify this? I've had a thorough look through existing issues and change requests, so hope I'm not reporting a known problem.

I came across this while looking at amplifier C and where it should be rendered. Table VII makes it clear that Echelon (B) and Quantity (C) are mutually exclusive across Units and Equipment. I wanted to ask a quick related question that you're probably best positioned to answer given your expertise with 2525 symbology.
Have you ever seen Quantity (numerical amplifier text) displayed at the bottom-right of the symbol frame? Perhaps in older releases of 2525, or other symbology systems? I've been shown a number of symbols where it is displayed there.
I have checked the 2525C and 2525D specs and can find no evidence of support for this. I therefore assume the symbols I saw have deviated from the spec. Finally, figure 13 of MIL-STD-2525C shows amplifier C in action whereas MIL-STD-2525D has done away with this useful graphic.
Kind Regards,
Jan

This appears to be related to, but not a duplicate of #113

@jrweakland - I cleared my assignment - this seems to be a standard question for @abouffard (in this case: "why/was that attribute removed in D")

@csmoore - I assigned this to you because of the reorg on the team. Please reassign as you see fit.

@jrweakland - it is a bit confusing but I think the repo owners are supposed to manage these issues for their repo, but maybe we need to clarify that role/responsibility, and I don't mind trying to answer this question.
@meterlongcat - from helping to put together table/schemas for each symbol set I know it is somewhat confusing which text attributes go with which symbol set (the results of our exercise in doing this is here: https://github.com/Esri/joint-military-symbology-xml/tree/master/samples/military_feature_schemas ) but the overall placement seems to be clear  and we only found a few inconsistencies in going through these tables ( example ) - here is the graphic/table from the 2525D standard (in Appendix D) showing this placement (I can't speak for the earlier versions) :

I'll close this if I don't hear back (or assign to Andy if we do)

The placement hasn't changed much over the years / through the different versions.  See below, version is included in the filenames - MIL-STD-2525B Ch1, Ch2, and MIL-STD-2525C.

From top to bottom:
MIL-STD-2525B Ch1 Figure 3
MIL-STD-2525B Ch2 Figure 3
MIL-STD-2525C Figure 2

@csmoore @ottenw Thank you for your responses. Based on your images it's clear that Quantity has always been positioned above the symbol frame and never on the bottom-right. I therefore conclude that the images I was shown (which displayed a numerical quantity on the bottom-right corner of the frame) were deviations from the 2525 specs.
The question that still remains unanswered is whether amplifier B should be displayed in the two example SIDCs I listed. My understanding is that it should for the first, but shouldn't for the second. Both SIDCs however show the amplifier.

Yes, Table VII does suggest, by its wording, that Quantity and Echelon are mutually exclusive between Equipment and Units respectively.  The relevant descriptions in table D-II would seem to reinforce that interpretation as well.  I'm sure @ottenw can confirm this as Army doctrine.
JMSML contains a definition of the 2525D codes and their meanings, it doesn't, in general, enforce doctrine, like guidance as to when a user should use an echelon code or when they should not.  It does, however, include the description text for each entry in Table VII, so it does capture the doctrinal information therein.
JMSML's C# sample code parses and draws what it sees with any given SIDC, it doesn't, currently, apply doctrine either, or test to see if a given SIDC violates said doctrine.
If, for example, your system considers an equipment SIDC with an echelon code a "mistake" and you choose to model the doctrine that says one shouldn't apply an echelon code to an equipment symbol, then you, as an implementer, must decide how to draw that symbol, with or without the echelon amplifier.  Our implementation, for development and test of JMSML, goes ahead and draws it.

I concur with @abouffard - this is doctrine, and I believe it is not limited to the Army; rather, it's more about training and knowledge of the individual creating the report.  For example, an individual could see 20 armored personnel carriers (APCs) lined up on a road and report that they see a quantity of 20 pieces of equipment - Vehicle, Armored, Armored Personnel Carrier (symbol set 15, 120103).  If the individual is close enough to read the vehicle bumper numbers and observes "A 1-16IN", that individual understands that all 20 APCs are assigned to "A" Company, 1st Battalion 16th Infantry Regiment and knowing that a US mechanized infantry company is usually authorized 20 APCs, then they would report a company sized unit - Movement and Maneuver, Infantry, Armored/Mechanized/Tracked (symbol set 10, 121105) , echelon = company.

Sounds like the question has been answered (and then some) - thanks for everyone's input - closing


Instrumentation for MySQLConnector

I am using Elastic APM for the first time and I was surprised that it did not instrument the MySQL Connector database calls. After some investigation I figured out that there is no actual instrumentation for MySQL Connector. By studying the existing implementations of other drivers I quickly figured out how to implement one for MySQL Connector and it works.
But you put the bar quite high for some simple code contributions and I am not sure if it is worth the effort.
This is my mysqlconnector.py version:
from elasticapm.instrumentation.packages.dbapi2 import (
    ConnectionProxy,
    CursorProxy,
    DbApi2Instrumentation,
    extract_signature,
)

class MySQLConnectorCursorProxy(CursorProxy):
    provider_name = "mysql.connector"

    def extract_signature(self, sql):
        return extract_signature(sql)

class MySQLConnectorConnectionProxy(ConnectionProxy):
    cursor_proxy = MySQLConnectorCursorProxy

class MySQLConnectorInstrumentation(DbApi2Instrumentation):
    name = "mysql.connector"

    instrument_list = [("mysql.connector", "connect")]

    def call(self, module, method, wrapped, instance, args, kwargs):
        return MySQLConnectorConnectionProxy(wrapped(*args, **kwargs))

This works as far as I could see. Maybe some of the existing authors/contributors can pick this up and implement it properly with all the necessary test cases.

Hi @trancee! This looks great, thank you! You're quite right that for instrumentations, writing and running tests could be easier and better documented. I hope to work on that a bit in the future.
As for now, it would be great if you could open a pull request with this code and sign our CLA (if this is your first contribution to an Elastic project). This would allow us to build a complete instrumentation including tests based on your code.
I can't promise when I'd get around to whip up the tests, but it'll probably be at least a few weeks.

fixed & supported


Not possible to include/restore .git directory via .s2iignore

It would seem the default ignore expression cannot be overriden by a .s2iignore file.
Specifically it is not possible to ensure the source .git directory is tarred over to the new image if specifically desired. At least no pattern I've included has been able to accomplish this. The addition of the .s2iignore file doesn't automatically override the default pattern, and using a not expression doesn't work either.
The use-case in particular would be that some deployed applications benefit from information on the specific build (like it's commit id). I'm going to open a suggested feature improvement with regards to that example, but generally speaking it should be possible to have the .git directory deployed as a workaround when necessary. The nature of a default exclusion pattern would imply that it can be overridden.

Interesting idea.  A couple in internal implementation points that have some relevance:
a) the current tar support does have a public method to override the exclusion pattern (https://github.com/openshift/source-to-image/blob/master/pkg/tar/tar.go#L64-L66), but we currently do not bubble that up to a s2i command line option that would allow you to set it
b) at the moment, the exclusion pattern leverages golang regular expressions, where as the .s2iignore file leverages golang Glob's a la the docker ignore files ... so different algorithms here, and not as clean an overlap of use case as one might think.
Given these facts, I'm inclined to simply expose a new s2i cmd line option to allow users to set the exclusion regular expression string for s2i's tar, and not muddy the waters by bringing .s2iignore into this space.
@binarybabel - would that suit your needs?
@bparees - please chime in here or let's discuss offline when you get the chance.

@gabemontero That would fit our needs, at the moment, but here are a couple of observations:

Even though the pattern mechanics are different, the presence of .s2iignore could be seen as replacing the default pattern, eliminating the overlap. Users still wishing to exclude .git directories could then add them to their ignore file. Admittedly though, that's a user-design question as to what is the preferred behavior. However the amount of time I spent trying to figure out when/where/why all .git directories were missing makes me think the existing system isn't perfectly intuitive.
Something to keep in mind is that the default exclusion pattern can also interfere with save-artifacts restoration. Specifically bundler installed gems from github sources. The next bundle will fail because their .git directories (just like the root .git directory) are missing. Currently the only way around that is to remove those gems and let them install fresh, I.E. rm -Rf bundle/ruby/bundler bundle/ruby/cache/bundler Granted, that's ruby specific, but I feel that issues like these will keep popping up.

My point being that excluding .git directories isn't bad default behavior, but can cause real issues. So long term I think an SCM/repo level way of configuring this is necessary (as opposed to external dc/pod level options). My vote would be to just have the presence of .s2iignore bypass the default pattern.

My vote would be to just have the presence of .s2iignore bypass the default pattern.

honestly we probably want to do both.

flag that lets you override the default exclusion pattern from the CLI
if there's a .s2iignore file, don't apply the default exclusion pattern and use the .s2iignore instead.

@soltysh what do you think?

Frankly said I'd ho with just .s2iignkre file without introducing yet
another cli option. It's simpler and cleaner solution than doing the mix.
The other thing that also seems to make sense, is to completly drop
excluding something when tarring, unless there's a good reason to have
those defaults,  I don't remember any atm.
On Oct 6, 2015 11:32 PM, "Ben Parees"<EMAIL_ADDRESS>wrote:

My vote would be to just have the presence of .s2iignore bypass the
default pattern.
honestly we probably want to do both.

flag that lets you override the default exclusion pattern from the CLI
if there's a .s2iignore file, don't apply the default exclusion pattern
and use the .s2iignore instead.

@soltysh https://github.com/soltysh what do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/openshift/source-to-image/issues/299#issuecomment-146008187
.

yeah i was curious where that default exclusion came from too.  i guess we figured there's no likely need to have .git history in your final application image (and probably some good reasons not to).
i can live with the "just .s2iignore" path (and it sounds like @binarybabel can too), just thinking it kinda sucks to have to create that file in your source repo to do a simple override.  but sure we can start there.

Forgot about this one and finally dug it up today.
Turns our, since last we talked, the exclusion pattern was surfaced as a s2i arg here as part of my fix for https://github.com/openshift/source-to-image/issues/457 (and I obviously forgot to cross reference with this one)
@bparees since option 1 from the old discussion above has been added, do we still want to do option 2 at this point ?

Updating .s2iignore was in fact dismissed in the discussion in https://github.com/openshift/source-to-image/issues/457

seems good enough to me.


Fix collapse status not persisting (#24)

This ensures that collapsed headers stay collapsed after a page refresh.

Sorry for the extreme delay here, somehow escaped my attention that I hadn't actually merged this.


[backport] Warn NaN in FP16 mode in dcgan example

Backport of #7344

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 1c486470629597898e6aeef1976c17e51c2d8e34, target branch v6) failed with status FAILURE.

LGTM! The CI failure is not relevant to this change.


HDRJPGs with resolutions higher than 4K crash on mobile (iOS iPhone 15)

Using a HDRJPG with higher resolutions than 4096 in width seem to crash mobile webpage. Not sure if this is due to iOS and some Safari artificial VRAM limitations or something in the codebase but my guess is iOS. Above 4K HDRJPGs seem to work fine on iPad Pro M2 fine, and desktops as well. Gotten up to 6K on iPad Pro and 8K on desktop.
Curious to hear if anyone else has encountered this problem.

Here is the code and a link for mobile to test. You can see that when you switch from 4K to 8K it renders black on mobile
Video
https://github.com/MONOGRID/gainmap-js/assets/23407511/9903f92b-b017-4640-a0f5-18ecdc7a0b5b
Code
https://stackblitz.com/~/github.com/ianpetrarca/gainmap
Test Link for Mobile
https://gainmap-ef100.web.app/

at first glance the problem can be reproduced on android and your page can be inspected with chrome://inspect, this is the result:

the source of the log
THREE.WebGLRenderer: Texture has been resized from (8192x4096) to (4096x2048).

seems to be this line in threejs
https://github.com/mrdoob/three.js/blob/34cfc4d11cf0a0fd83dc66a75acfc181df9bc972/src/renderers/webgl/WebGLTextures.js#L85
so apparently there's a limit of some kind (which I ignore the motivation behind) and three.js tries to resize the texture and (apparently) fails to display the resized texture.
I'm going to close this for now as this doesn't appear to be a gainmap.js related problem


November 2023 main Spack merge + updates

Summary
I'll keep the notes for this update, which includes https://github.com/JCSDA/spack/pull/371, in this PR.
This PR pulls in Spack release 0.21.0. Additionally, I've reconciled NCEPLIBS recipes with those in the library repos themselves (very few changes), as well as added in changes to support NCO's stack. Ran style checks/fixes on all packages. This PR also includes a proposed solution for the MySQL/Boost problem (#705).
Misc. notes:

wgrib2 now creates limitation for gmake version; this may be fixable with patching (modeled on<EMAIL_ADDRESS>which is appropriately corrected)
MySQL now has an option to use internal boost libraries (this approach could probably be improved by creating resources)
I added a fix for a random gmake build issue
I tweaked a few default version settings, including removing a few; obviously these can be re-added as needed but so far so good based on my testing.
I tested out the new spack deconcretize command. Pretty nifty :)
Still need external flex to avoid duplicates.

Waiting on:

https://github.com/spack/spack/pull/41207

Testing

Tested concretizing and some building on PC.
Tested building NCO's stack (needed some changes, incl. supporting deprecated packages).
Tested building UFS WM & GW deps on Hera; no duplicates

Tests to do:

[ ] Do another round of diffing Spack core (non-recipe) changes to inventory our changes
[ ] Hercules
[ ] Orion
[ ] Discover
[ ] Casper
[ ] Derecho
[ ] Acorn
[ ] Gaea C4
[ ] Gaea C5
[ ] Hera
[ ] Jet
[ ] Narwhal
[ ] Nautilus
[ ] S4
[ ] AMI RH
[ ] Parallelcluster JCSDA
[ ] RDHPCS Cloud (ParallelWorks)

Applications affected
All.
Systems affected
All.
Dependencies

[ ] https://github.com/JCSDA/spack/pull/371

Issue(s) addressed
Addresses #705 (w.r.t. MySQL)
Checklist

[ ] This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
[ ] These changes have been tested on the affected systems and applications.
[ ] All dependency PRs/issues have been resolved and this PR can be merged.

@climbfuji have you run into Undefined symbols for architecture arm64: / ld: symbol(s) not found for architecture arm64 (see https://github.com/JCSDA/spack-stack/actions/runs/6951315212/job/18913085682?pr=884)? It's failing on gsl, but it's the same package/version/spec as the current one.

Details

That's new to me. If I had to guess I'd say that the binary cache needs to be cleared. Let me do that and rerun the failing tests.

Same error when building from scratch.

@AlexanderRichert-NOAA I am trying to fix the macOS and Ubuntu CI errors. Given the problems we are seeing I would like to have a good testing coverage across the various HPCs that we support, ideally we build this branch on all of the tier-1 platforms before merging?

Yep agreed. I'll fill out the test list in a bit.

After fixing gsl on macOS (see https://github.com/JCSDA/spack/pull/371#issuecomment-1823554867), I am now hitting an error in py-scipy.

and errors in prod-util, and many other packages are following ...

Looks like the prod-util one is because of a patch where it's been fixed in the code, so that should be an easy one.

Looks like the prod-util one is because of a patch where it's been fixed in the code, so that should be an easy one.

Yep, will push later. Problems with py-scipy, I somewhat suspect that it's the reversal for the MACOSX_DEPLOYMENT_SYSTEM variable that I just cherry-picked from spack mainline. Sigh.

I just pushed two commits, one for prod-util and one for scipy (a bug introduced by the merge it seems):
commit c1b6bd5415f6cad49b61768417b122883031b3bd (HEAD -> nov2023_spackmerge, alex/nov2023_spackmerge)
Author: Dom Heinzeller<EMAIL_ADDRESS>Date:   Wed Nov 22 16:10:42 2023 -0700

    Fix merge bug fix in var/spack/repos/builtin/packages/py-scipy/package.py

commit cd2ec2855da52081050f8bf66e185dde6c351dbf
Author: Dom Heinzeller<EMAIL_ADDRESS>Date:   Wed Nov 22 15:09:49 2023 -0700

    Remove invalid/unnecessary patch apple-clang-13.0.0-stdlib.patch for prod-util 2.1.0

Please take a look. I don't think this solves the scipy 1.11.3 problems, but it's definitely needed (I compared to spack dev)

Thanks, both changes look good.

@AlexanderRichert-NOAA I was able to build unified-env on my macOS with the various bug fixes. I cleared the build cache on all three CI systems (necessary due to the macOS MACOSX_DEPLOYMENT_TARGET bug fix) and pushed an empty commit to trigger CI.
I am going to stop here and take off for the long weekend. Let's pick this up on Monday. Happy Thanksgiving!

Sounds great. Have a good Thanksgiving weekend!

I'm working on rust, it looks like the issue is that the trick of using an older py-cryptography doesn't work because all of openssl@1 is deprecated now.

@climbfuji would you be open to us modifying awscli-v2 to add support for using the precompiled version with vendored dependencies? Rust seems to be a pretty massive dependency just for this one little thing, but I also don't want to use a deprecated openssl indefinitely if we can help it. If yes, then I can work on that and keep going with the ubuntu CI if you can tackle the macOS CI (though right now it looks like rust is the issue for both).

@climbfuji would you be open to us modifying awscli-v2 to add support for using the precompiled version with vendored dependencies? Rust seems to be a pretty massive dependency just for this one little thing, but I also don't want to use a deprecated openssl indefinitely if we can help it. If yes, then I can work on that and keep going with the ubuntu CI if you can tackle the macOS CI (though right now it looks like rust is the issue for both).

I am fine with that approach. Not having to build rust sounds like a good idea, and awscli is just a simple package from which we use binaries, not link against it. FWIW, rust built on my mac, but let's try to avoid it if we can.

@climbfuji FYI the macos-ci-x86_64-build timed out previously after 6 hours. Looking at the current run, it seems that some of the stage times are particularly long, so I wonder if the runner is having internet issues? In any case do you mind keeping an eye on that?

@climbfuji FYI the macos-ci-x86_64-build timed out previously after 6 hours. Looking at the current run, it seems that some of the stage times are particularly long, so I wonder if the runner is having internet issues? In any case do you mind keeping an eye on that?

I'll need to hop onto the machine and do one build out of CI to create a new binary cache. I'll check if there are issues with getting data, or the disk running full.

As for #880, this still needs a fix for Derecho (yep, once again) and a test on Nautilus (is under maintenance until today).

@AlexanderRichert-NOAA I updated your spack branch from jcsda_emc_spack_stack, updated your spack-stack branch from develop, and committed the submodule pointer update. I think we can start testing on the HPCs?

:cowboy_hat_face: Yep sounds good

@climbfuji it occurs to me that base-env is still specifying zlib as a dependency, but every zlib-using package is using zlib-api where zlib-ng is always being selected as the provider. This could lead to inconsistencies in downstream applications, and we probably don't need both. Would it make sense to just go all in with zlib or zlib-ng (either way, we should add it to the list of providers in common/packages.yaml)? Maybe it makes the most sense to use zlib for now, since that's what we currently use, then test upgrading to zlib-ng later. If you agree I'll update the config.

@climbfuji it occurs to me that base-env is still specifying zlib as a dependency, but every zlib-using package is using zlib-api where zlib-ng is always being selected as the provider. This could lead to inconsistencies in downstream applications, and we probably don't need both. Would it make sense to just go all in with zlib or zlib-ng (either way, we should add it to the list of providers in common/packages.yaml)? Maybe it makes the most sense to use zlib for now, since that's what we currently use, then test upgrading to zlib-ng later. If you agree I'll update the config.

That sounds like a good plan.
I ran into problems with the container bulds because it says that ecmwf-atlas doesn't have a variant ectrans anymore (it's called trans in spack develop):
> git diff jcsda/jcsda_emc_spack_stack var/spack/repos/builtin/packages/ecmwf-atlas
diff --git a/var/spack/repos/builtin/packages/ecmwf-atlas/package.py b/var/spack/repos/builtin/packages/ecmwf-atlas/package.py
index d720ed1ff0..44cdd7adc2 100644
--- a/var/spack/repos/builtin/packages/ecmwf-atlas/package.py
+++ b/var/spack/repos/builtin/packages/ecmwf-atlas/package.py
@@ -52,9 +52,8 @@ class EcmwfAtlas(CMakePackage):
     variant("openmp", default=True, description="Use OpenMP")
     depends_on("llvm-openmp", when="+openmp %apple-clang", type=("build", "run"))
     variant("shared", default=True, description="Build shared libraries")
-
-    variant("ectrans", default=False, description="Enable ectrans", when="@0.31.0:")
-    depends_on("ectrans@1.1.0:", when="+ectrans")
+    variant("trans", default=False, description="Enable trans")
+    depends_on("ectrans@1.1.0:"<EMAIL_ADDRESS>+trans")
     variant("eigen", default=True, description="Enable eigen")
     depends_on("eigen", when="+eigen")
     variant("fftw", default=True, description="Enable fftw")
@@ -73,9 +72,9 @@ def cmake_args(self):
             "-DPYTHON_EXECUTABLE:FILEPATH=" + self.spec["python"].command.path,
         ]
         if self.spec.satisfies("@0.31:0.34"):
-            args.append(self.define_from_variant("ENABLE_TRANS", "ectrans"))
+            args.append(self.define_from_variant("ENABLE_TRANS", "trans"))
         if self.spec.satisfies("@0.35:"):
-            args.append(self.define_from_variant("ENABLE_ECTRANS", "ectrans"))
+            args.append(self.define_from_variant("ENABLE_ECTRANS", "trans"))
             args.append(self.define_from_variant("ENABLE_TESSELATION", "tesselation"))
         if "~shared" in self.spec:
             args.append("-DBUILD_SHARED_LIBS=OFF")

We'll need to update the following two files with the variants (no need to re-test building on the systems we already tested, ectrans built there fine every time). Do you want me to push these updates?
> git diff configs/
diff --git a/configs/common/packages.yaml b/configs/common/packages.yaml
index df4d820..a98ba2b 100644
--- a/configs/common/packages.yaml
+++ b/configs/common/packages.yaml
@@ -52,7 +52,7 @@
       variants: linalg=eigen,lapack compression=lz4,bzip2
     ecmwf-atlas:
       version: ['0.35.0']
-      variants: +fckit +ectrans +tesselation +fftw
+      variants: +fckit +trans +tesselation +fftw
     ectrans:
       version: ['1.2.0']
       variants: ~mkl +fftw
diff --git a/configs/containers/specs/jedi-ci.yaml b/configs/containers/specs/jedi-ci.yaml
index 3cc655d..eb53ea3 100644
--- a/configs/containers/specs/jedi-ci.yaml
+++ b/configs/containers/specs/jedi-ci.yaml
@@ -1,7 +1,7 @@
   ### spack-stack-1.5.1 / skylab-6.1.0 containers for fv3-jedi and mpas-jedi (but not for ufs-jedi)
   specs<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>ecflow@5,
-<EMAIL_ADDRESS><EMAIL_ADDRESS>+fckit +ectrans +tesselation +fftw<EMAIL_ADDRESS><EMAIL_ADDRESS>+fftw,
+<EMAIL_ADDRESS><EMAIL_ADDRESS>+fckit +trans +tesselation +fftw<EMAIL_ADDRESS><EMAIL_ADDRESS>+fftw,
<EMAIL_ADDRESS><EMAIL_ADDRESS>fms@release-jcsda, <EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Yes please go ahead with that, sorry I missed those. I'll add the zlib update momentarily (worked fine on Acorn just now).

Yes please go ahead with that, sorry I missed those. I'll add the zlib update momentarily (worked fine on Acorn just now).

Done, thanks for the quick reply. I am surprised that the configs/common/packages.yaml variant ectrans didn't lead to an error, but probably because these are just "soft requirements". The explicit request for the variant in the container spec did trigger the error.

The container build also doesn't like the precision variant for sp:
   echo '    sp:' &&   echo '      version: [2.3.3]' &&   echo '      variants: precision=4,d,8'

-->
1. Cannot set variant 'precision' for package 'sp' because the variant condition cannot be satisfied for the given spec. You could consider setting `concretizer:unify` to `when_possible` or `false` to allow multiple versions of some packages.

That's strange ... do we need some extra quotes?

I think we need to update the version of sp in configs/common/packages.yaml for the precision variant to be valid. 2.5.0 ?

Yep that sounds good.

Yep that sounds good.

Done

Error on Narwhal on the home stretch, for mapl with GNU (worked with Intel):
work1/heinzell/spack-stack-nov2023_spackmerge/cache/build_stage/spack-stage-mapl-2.40.3-bpfups3oqez3cqbg5wqu4zn6ofoxalty/spack-src'

1 error found in build log:
     22    -- Cray Programming Environment 2.7.19 C
     23    -- Detecting C compiler ABI info
     24    -- Detecting C compiler ABI info - done
     25    -- Check for working C compiler: /p/work1/heinzell/spack-stack-nov2023_spackmerge/spack/lib/spack/env/gcc/gcc - skipped
     26    -- Detecting C compile features
     27    -- Detecting C compile features - done
  >> 28    CMake Error at ESMA_cmake/esma.cmake:5 (esma_check_install_prefix):
     29      Unknown CMake command "esma_check_install_prefix".
     30    Call Stack (most recent call first):
     31      CMakeLists.txt:29 (include)
     32
     33
     34    -- Configuring incomplete, errors occurred!

See build log for details:
  /p/work1/heinzell/spack-stack-nov2023_spackmerge/cache/build_stage/spack-stage-mapl-2.40.3-bpfups3oqez3cqbg5wqu4zn6ofoxalty/spack-build-out.txt

I can look into it later but if @mathomp4 has an idea right away (hopefully!) ...

That macro is fairly benign:
https://github.com/GEOS-ESM/ESMA_cmake/blob/main/esma_support/esma_check_install_prefix.cmake
it just checks to make sure you aren't installing to /usr/local, you don't have a comma in the directory path[^1], and where you are installing, you have write permissions.
[^1]: That one was a fun bug to figure out.

Noted. That looks suspiciously like an error I ran into ages ago involving the esma_cmake resource and the caching thereof.

Hmm, but given that it installed fine a few hours beforehand with Intel that seems rather unlikely. And there's no difference between jcsda_emc_spack_stack mapl and this PR, and it previously built just fine. Grrrrrr...

Try relocating the cached version so it redownloads? But don't delete it, in case I'm wrong :)

Solution, a lot easier: run spack clean -a once, try, it fails. Wait a few hours, run spack clean -a again, try, it installs just fine.

spack PR was merged, new hash is https://github.com/JCSDA/spack/commit/8683cc4f1a42329cab38d2fd607476e202ce50d1

@AlexanderRichert-NOAA @climbfuji -
let me update or include README.md files for noaa- platforms

@AlexanderRichert-NOAA @climbfuji - let me update or include README.md files for noaa- platforms

@natalie-perlin How quickly can you do that? If you need a bit longer, we can make it a small follow-up PR (since no testing needed for adding/updating README.md files).


Npm version not working

Check Npm package!

Hi @doArcanjo!
Thanks! I'll get this fixed ASAP!

Ok, I've verified that it's working now...
quick online test here: https://tonicdev.com/npm/number-format.js
require("number-format.js")( "#,##0.####", 1234567.890 );
// result: "1,234,567.89"

No problem, keep up the good work!


"Deploy to Heroku" button in github doesn't seem to work.

I tried across two browsers to deploy to heroku on a newly-created account via clicking the button at https://github.com/juice-shop/juice-shop#deploy-on-heroku-free-0month-dyno and this seemed to load indefinitely.
Visiting https://elements.heroku.com/buttons/bkimminich/juice-shop and clicking the "Deploy to Heroku" button works.

Please try with a different browser and make sure that there is no interference from plugins etc. with the referer header that the button is relying on. So far all Heroku button issues we ever encountered were solvable by using a "vanilla" browser instance. Just tested on Chrome and it works just fine for me.

Thanks for the quick response. My initial attempt was with Firefox that contained several addons, it worked without the addons. My second attempt was with Safari, which still doesn't seem to work, but I can live without Safari support! That would likely be a Heroku issue regardless. I'll close my issue, thanks again.


Update README.md

Hi, I am doing a technical writing project. It is good practice to keep the writing concise. For example, I removed ...supposed to... Though it may lead to loss in meaning, readers will benefit from this concision. Hope it helps

This is only failing because next is currently broken.


Can't install on Windows

Hi, I tried installing KCC but I don't know what to do. The only file that looked like it might work for installation was 7z.exe but when I tried opening it, I just see black window and that's all. Please help

Sounds like you are trying to run the python code directly. Download the exe from the releases tab


Binding element implicitly has an 'any' type.

VSCode Version: 1.29.1
OS Version: MacOS Sierra

Steps to Reproduce:

Use currently latest vscode (1.29.1)
Use currently latest typescript (3.2.1)

If I use VS code typescript 3.1.4

It doesn't show that.

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

Please share some complete, example code (as text)


File->Open 'keep file open' has side-effects

CMainFrame::OnFileOpen Load / LoadAsync have completely different behaviour
Other then 'keep file open' it should behave the same; which means loading very large files should not block the ui, but just read using LoadAsync but not keep tailing it.

Trying to open larger log file  (about 60000 rows) loads only small part of lines if "Keep file open" is not checked. Maybe related to this?

I think I found the problem, thanks for reporting again!


Adding Distributed GPU Computing, Distributed Training/Inference, GPU Profiling and more

This is to add the following knowledge to HPC Handbook including code examples with easy-setup experiment.

GPU Computing
Distributed GPU Computing
Distributed Training
Distributed Inference
Advance Topics

@amazloumi , this is done, right?

It is partially done. The distributed inference is not there yet.


How can I listen to consuming messages and decide to consume or cancel them based on message content?

I receive messages from different sources on the same queue and I want to discard the consuming of similar messages when they arrive nearly at the same time.
The problem is that I need to access the content of the message to make the comparison and at the same time I must be able to cancel the consuming.
What is the best way to achieve this?

By cancel, do you mean discard the message, removing it from the queue?

Yes, I want to remove the message from the queue.
I was trying to use a middleware listening on consuming messages and, after analyzing the message content, establish when the consuming is needed or the message should be skipped.

Cool, thanks!


State switch UI double activation

Using the latest version I have problem in Yeelight Sunflower component (the only one I have installed now together with LG WebOS TV).
I have two bulbs, they appear as two switches. But randomly when I  deactivate one switch in UI, it will deactivate also the other one in UI. No message nor data is being sent for second switch, so it looks like purely UI problem.
I have also problem how state is being reported (unavailable), but I hacked my way through and this is covered in last comment here: https://github.com/home-assistant/home-assistant/issues/10113
UPDATE: now I have daemon shutdown and I use only UI (connection is lost reported correctly). Now it does not happen. So I think it must be some bad state behavior getting from backend to frontend. Any idea how to debug?

I have 2 bulbs Yeelight, latest version of HA. Same problem: Unavailable :(

Found solution! :)
"Enabling the LAN Control option" in Yeelight app, rebooted HA and all is work!

Can you please provide more details?
What Yeelight software and type you have?
I have:
Yeelight Sunflower (app & hardware)
Software version: 2.0.3
Firmware: 2.1.2
It only works on local network already as Sunflower contains 3 bulbs plus their custom router from RJ-45 (Lan) to Zigbee network.
There is no "Enabling the LAN Control option". There is only Authorize Remote Control and I already provide that. So I had to provide my own implementation of Yeelight HA component.

In the past I was distributor of several Chinese hardware startups, also was making my own hardware. The problem with Chinese hardware is they make new iteration of products every 6 or 12 months with different architecture and new software and they never update software or fix bugs. Chinese don't write quality software and don't have enough resources to support any product for longer time.


/me is not working on current android app

When someone uses /me in example the desktop environment it shows a "status" message.
When opening the message on teh current android app (v13.5.89) it shows "/me" and the "status" message.
The messages should be equaliy displayed.

This repository is for the web app. Can you reopen this issue at https://github.com/zulip/zulip-mobile?


How pass RenderFragment template=null to a Component

The source of the issue can be found at https://stackoverflow.com/questions/56118436/check-if-renderfragment-is-empty where I would like to check if HeaderTitle is empty or null. I thought it could use more discussion here.
    <MyComp Open="false">
        <HeaderTitle>
             @if (!String.IsNullOrEmpty(hdr)) {
                 ...

 @hdr

            }
        </HeaderTitle>
        <Detail>
            ...
        </Detail>
   </MyComp>

It seems that the root issue is that it is not simple to set HeaderTitle=null. The only way to do this is to not include it in MyComp. So you have to write something like:
@if (!String.IsNullOrEmpty(hdr)) {
    <MyComp Open="false">
        <HeaderTitle>... </HeaderTitle>
        <Detail>... </Detail>
   </MyComp>
} else {
    <MyComp Open="false">


Please provide binary releases

this is a duplicate of #177 but as it was closed by the oringinal author i created a new issue instead of commenting on the old.
please add binary builds to the assets in the realease section here on gitlab.

Check this https://github.com/Telmate/terraform-provider-proxmox/issues/218#issuecomment-696013579

@c33s  releases to the terraform registry were added today!   Check out #228   and the official listing here:  https://registry.terraform.io/providers/Telmate/proxmox/latest
If you'd prefer to still pull from GitHub releases though, they are now available there as well.  https://github.com/Telmate/terraform-provider-proxmox/releases/tag/v2.6.1

I'm very excited to see the provider published in the official registry!  I'm new to terraform and am having trouble getting the provider though.
Here are the relevant parts of my .tf file:
terraform {
  required_providers {
    proxmox = {
      source = "Telmate/proxmox"
    }
  }
}

provider "proxmox" {
    pm_api_url = "${var.PROXMOX_URL}"
    pm_user = "${var.PROXMOX_USERNAME}"
    pm_password = "${var.PROXMOX_PASSWORD}"
    pm_tls_insecure = "true"
}

Here's the command and the output:
$ terraform init

Initializing provider plugins...
- Checking for available provider plugins on https://releases.hashicorp.com...

Provider "proxmox" not available for installation.

A provider named "proxmox" could not be found in the official repository.

This may result from mistyping the provider name, or the given provider may
be a third-party provider that cannot be installed automatically.

In the latter case, the plugin must be installed manually by locating and
downloading a suitable distribution package and placing the plugin's executable
file in the following directory:
    terraform.d/plugins/linux_amd64

Terraform detects necessary plugins by inspecting the configuration and state.
To view the provider versions requested by each module, run
"terraform providers".

Is there a way to tell terraform to automatically get the required provider from the registry, or should I manually download a release, unzip it and put it in terraform.d/plugins/linux_amd64?

This plugin requires Terraform v0.12 or later (which is when the plugin API 5 was introduced).  The latest version from snap is 0.11.11.  Upgrading Terraform to v0.13.5 using the instructions from Hashicorp's docs solved the problem of not being able to find this plugin in the repo.
This issue can be closed as far as I am concerned, but it would be nice to update the documentation to explain these dependencies.  Note: I'm not sure if the plugin would actually be found with Terraform v0.12, I can only attest that it is found with Terraform v0.13.5.


Login button didn't work after verification of email and phone

I create new account using my iPhone browser and accidentally filled in a wrong phone number. After verifying my email, I went back to the INFO step, corrected my phone number. Went back to the VALIDATION step. Click "RESEND..." and validated my phone number. And then clicking the LOGIN button did nothing.

Was this a "normal" registration or part of an oauth flow. If so, what scopes were requested?

It seems that the server resends an email validation request even if the email address is already validated. The login handler then checks if the new validation request has been completed before allowing login

This is acutally caused by multiple issues:

When info on the first registration page changes, a new user is created in the db, instead of an update of the previous user.
When the resend validation is triggered, the old key is simply invalidated an a new validation with a new key stored.

When the registration is completed, the validationkey for the last request is checked to see if it is confirmed.

Moved to bugfix release branch


Important note on prerequisite to pull images from ACR into ACA

Hi team,
When creating a new Container in an Azure Container App, a prerequisite is to enable an admin user account on the particular Azure Container Registry that the Azure Container App will be attempting to pull images from.  It is, therefore, worthwhile mentioning that regardless of how the Azure Container App is authenticating against the Azure Container Registry, the latter will have to be set up with an admin user account enabled, even though the usage of managed identities won't require the actual credentials' values.

@yuvalherziger : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yuvalherziger : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks, @yuvalherziger!
#sign-off


add function_export_name_changed lint

Second bullet point of #502.  Should I put all the lints in one (draft, I haven't finished them yet) PR or do separate PRs?

FWIW here's the line that breaks nightly tests - it doesn't seem like an issue with this lint.

Separate PRs for each lint is fine. That's usually faster and easier to review, and shows up more nicely in the changelog as well.
That test has been flaky for a while and I'm not sure why. I've never managed to reproduce the flakiness locally, so my gut feeling is that it might possibly be a GitHub Actions bug of some sort? Not sure, haven't had time to dig into it... Don't worry about it, I don't think you broke any of it — I just restarted it and I expect it'll probably pass.

Yeah, we can just the missing one for both cases, I think that's fine.
On Sat, Mar 30, 2024, 6:59 PM Max Carr @.***> wrote:

@.**** commented on this pull request.
In src/lints/function_export_name_changed.ron
https://github.com/obi1kenobi/cargo-semver-checks/pull/728#discussion_r1545508250
:

           item @fold @transform(op: "count") @filter(op: "=", value: ["$zero"]) {

               ... on Function {

                   export_name @filter(op: "=", value: ["%export_name"])

               }

           }

Yeah, I was thinking about that when I was writing it - it was pretty much
the same query except checking if the current export_name is null. If
we don't need granular feedback between missing vs changed, they can be
easily combined. Do you think it's necessary?
—
Reply to this email directly, view it on GitHub
https://github.com/obi1kenobi/cargo-semver-checks/pull/728#discussion_r1545508250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR5MSRHMSANZ27FFUX257TY247WRAVCNFSM6AAAAABFPX6VJOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNZQGA3DMOJVGU
.
You are receiving this because you commented.Message ID:
@.***>

All good. For these categories from the original issue:

function with the same export name takes different number of arguments
function with the same export name has a parameter whose type changes to one with a different ABI
it doesn't matter what the type is called, just whether it has the same ABI or not
function with the same export name changes what it returns (starts / stops returning data, or changes to return type with different ABI)
this can be multiple lints; e.g. separate lints for starts / stops / changes return type ABI

Would these (just) apply to exported non-public functions? Because changing the API of a public function would already be a breaking change, so do we want to flag this twice?

That's a great question.
I think it's reasonable to start by flagging twice here, because we're flagging two separate problems: the API is broken, and the ABI is also broken.
Down the line, if this proves to be problematic for any reason, we can build something more sophisticated. For example, we could have three lints instead of two: broken API but not ABI, broken ABI but not API, and both broken at once. But going straight for this seems like major overkill right now, so let's go with the simpler option of flagging twice for now.

OK, I consolidated the two cases

Thanks for the feedback!