demo app sample

A sample to test azure configuration?

Thank you Edward, a question: Is this a section of the repo to test azure configuration or what is it?

Yes, I was testing getting a very simple demo app deployed to make sure like networking is working as expected. This weekend I was able to get a k8s cluster finally authenticating and connecting properly with ingress the way I was expecting, which means if I get this demo app working we'll be able to attempt the same with some of our own components.

Ok great, keep us posted

I'm now troubleshooting this error
W0411 19:18:35.599448       7 controller.go:977] Service "human-connection/neo4j" does not have any active Endpoint.

This is a problem with the ingress service not discovering the app service.

Ok, this issue is solved now and we have an API service here http://redsol.eastus.cloudapp.azure.com/
It's good for development so we'll need to do some more testing to make sure it's working but they'll be more task to improve this for production. Those would be things like;

some test for the endpoints
backups
security, secrets in AKV
end point protection (ssl, creds ??)
ci/cd

There might be more as we progress but that's a good setup we have now. Currently this will cost 50-60 /month, so we can probably have a second env for production once we get there.


FFM-8260: adds java sdk v1.2.3 relnotes

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


vfox used as default, broken for Ruby

Describe the bug
Running mise install with a Ruby .tool-versions entry logs this error and prevents Ruby from working entirely (defaulting to the MacOS Ruby executable).
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk

This is solved by disabling the vfox backend (though it’s enabled by default now apparently?):
mise settings vfox=false

I wouldn’t mind using the vfox backend if it wasn’t broken, since it has precompiled binaries, which the asdf backend for Ruby does not. 😅
To Reproduce

Clone https://gitlab.com/gitlab-org/gitlab-development-kit.git
Run mise install
See the error above

Expected behavior
Ruby plugin works without error by default.
mise doctor output
activated: no
shims_on_path: yes

build_info:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Sat, 30 Nov 2024 00:54:51 +0000
  Rust Version: rustc 1.82.0 (f6e511eec 2024-10-15) (Homebrew)
  Profile: release

shell:
  /bin/zsh
  zsh 5.9 (arm64-apple-darwin24.0)

dirs:
  cache: ~/Library/Caches/mise
  config: ~/.config/mise
  data: ~/.local/share/mise
  shims: ~/.local/share/mise/shims
  state: ~/.local/state/mise

config_files:
  ~/.config/mise/config.toml
  ~/development/gitlab/gdk/.tool-versions

backends:
  aqua
  asdf
  cargo
  core
  gem
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:
  elixir             https://github.com/mise-plugins/mise-elixir.git#7b06439
  gitleaks           https://github.com/jmcvetta/asdf-gitleaks.git#0cc0d7e
  hugo               https://github.com/NeoHsu/asdf-hugo.git#125ce37
  markdownlint-cli2  https://github.com/paulo-ferraz-oliveira/asdf-markdownlint-cli2#0732edc
  minify             https://github.com/axilleas/asdf-minify.git#46954b9
  minio              https://github.com/aeons/asdf-minio.git#113b19d
  mockery            https://github.com/cabify/asdf-mockery.git#1cb81d6
  pnpm               https://github.com/jonathanmorley/asdf-pnpm.git#184f8cb
  postgres           https://github.com/smashedtoatoms/asdf-postgres.git#e970a77
  protoc             https://github.com/paxosglobal/asdf-protoc.git#a44c5cd
  redis              https://github.com/smashedtoatoms/asdf-redis.git#d5a67c5
  rust               https://github.com/code-lever/asdf-rust.git#95acf4f
  shellcheck         https://github.com/luizm/asdf-shellcheck.git#780d78d
  vale               https://github.com/pdemagny/asdf-vale#9926c81
  yarn               https://github.com/mise-plugins/asdf-yarn.git#74ea3b9
  yq                 https://github.com/sudermanjr/asdf-yq.git#772992f
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk
mise WARN  Error listing bin paths for<EMAIL_ADDRESS>runtime error: attempt to yield across metamethod/C-call boundary
stack traceback:
        [C]: in ?
        [C]: in function 'yield'
        [string "?"]:16: in function 'get'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:8: in function 'fetchManifest'
        ...sers/kev/.local/share/mise/plugins/ruby/lib/util.lua:23: in main chunk
        [C]: in function 'require'
        ...v/.local/share/mise/plugins/ruby/hooks/available.lua:1: in main chunk

toolset:
<EMAIL_ADDRESS>  asdf:minio@2022-07-15T03-44-22Z
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
env_vars:
  (none)

settings:

4 warnings found:

1. plugin bun overrides a core plugin

2. plugin python overrides a core plugin

4. plugin ruby overrides a core plugin

5. plugin rust overrides a core plugin

No problems found

Additional context
None.

https://github.com/jdx/mise/issues/3207 https://github.com/jdx/mise/issues/3087 https://github.com/jdx/mise/issues/3001#issuecomment-2489439805


Use do-while loop

Just a small improvement, this is the prime use case for do-while.

Thanks, Robin + I agree, but should this really be a separate PR?

I didn't notice until now, and there's no thematic overlap with #1119 so that's why there's a whole PR for this. The underlying code has been pretty much final since #970 so no further changes are planned there

I understand. And I agree that a PR shouldn't involve major unrelated changes in the code. But minor unrelated fixes or improvements are OK and we do that pretty frequently. I think it's nice to have a commit log with commits that correspond to "semantic" units of progress (of the project or of the code base), which are not too small and not too large.

In this case I agree with @hannahbast ,
there is only one reason for such small PRs and that is urgent bug fixes.
Such small improvements can and should be bundled with other things (e.g. other small fixes), you could for example fix some code smells etc. in the same go to get the critical mass.

Merged changes into #1119


support ShowError option in Pivot table

PR Details
This is a request to enable pivot table option. ShowError

Description
In Excel, PivotTable Options - Layout & Format - "For error values show"

Related Issue

Motivation and Context
This Pivot table option allows to remove Error(#DIV/0!)

How Has This Been Tested
New option added into test file

Types of changes

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

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

LGTM, thank you.


Click on Images

#Click on Images
##User Story
AS AN employer
I WANT to view a potential employee's deployed portfolio of work samples
SO THAT I can review samples of their work and assess whether they're a good candidate for an open position
##Acceptance Criteria
GIVEN I need to sample a potential employee's previous work
WHEN I click on the images of the applications
THEN I am taken to that deployed application

Unable to complete as there are no applications


With caluse crash array_concat crash

Steps to reproduce the behavior (Required)
 CREATE TABLE `t` (
  `in` varchar(65533) NULL COMMENT "",
  `out` varchar(65533) NULL COMMENT "",
  `linkattr` bigint(20) NULL COMMENT ""
) ENGINE=OLAP
DUPLICATE KEY(`in`)
DISTRIBUTED BY RANDOM
PROPERTIES (
"replication_num" = "1",
"in_memory" = "false",
"enable_persistent_index" = "false",
"replicated_storage" = "true",
"compression" = "LZ4");

insert into t values ("B","F",9),("C","B",3);

-- crash
WITH `tc` AS (
  SELECT
    [] AS `aa`,
    MD5SUM_NUMERIC(t.`IN`, t.`LINKATTR`, t.`OUT`) AS `bb`
  FROM
    t
)
SELECT
  ARRAY_CONCAT(
    `tc`.`aa`,
    [ (`tc`.`bb`) ]
  ) AS `res`
FROM
  `tc`

---ok

SELECT
  array_concat(
    [],
    [MD5SUM_NUMERIC(t.`IN`, t.`LINKATTR`, t.`OUT`) ]
  )
FROM
  t;

Expected behavior (Required)
Real behavior (Required)
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()
3.1

on main resulting into different resutls
mysql> SELECT    array_concat(     [],      [MD5SUM_NUMERIC(t.`IN`, t.`LINKATTR`, t.`OUT`) ]   )  FROM    t;
+-------------------------------------------------------------+
| array_concat([], [md5sum_numeric(t.IN, t.LINKATTR, t.OUT)]) |
+-------------------------------------------------------------+
| [166447080664827458533765763306809118404]                   |
| [98790225452812278081487393989635148350]                    |
+-------------------------------------------------------------+
2 rows in set (0.01 sec)

mysql>  WITH `tc` AS (   SELECT      [] AS `aa`,      MD5SUM_NUMERIC(t.`IN`, t.`LINKATTR`, t.`OUT`) AS `bb`    FROM      t )  SELECT    ARRAY_CONCAT(     `tc`.`aa`,      [ (`tc`.`bb`) ]   ) AS `res`  FROM    `tc`;
+-------+
| res   |
+-------+
| [196] |
| [206] |
+-------+
2 rows in set (0.01 sec)


HDoujin Metadata Support

Hey, love your software, has great potential!
Just found it the other day and already liking it a lot.
I have a rather large collection that I download with HDoujin Downloader,
I'm sure you're familiar with it, all my archives have the saved metadata as info.json in them,
it would be awesome if exhentai-manga-manager could read those files and import that metadata. Or at least the e/xhentai URLs inside them would be great.
kind regards,
ari

Thank you for your feedback. However, I currently have no plans to support metadata files from third-party download software. You can refer to step 5 in https://github.com/SchneeHertz/exhentai-manga-manager/wiki/English-Instruction#library to download a backup of the E-hentai gallery data before July 30, 2023, and import it into the software. This should address most of your metadata retrieval needs.


Clean up formatting and comments

Made changes so that the plot_80 tutorial and boxy.py now (hopefully) conform to pep8 standards.
For some of the longer, more complicated lines I was quite sure where to split everything. I wasn't getting any formatting errors so hopefully they are all still readable. Let me know if you have any suggestions or changes.

@kuziekj I merged so now we can focus


[Package Request]: CnCNet Red Alert 2 & Yuri's Revenge

Package Requested

[X] I would like someone else to build the manifest.
[ ] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[X] I have checked that the installer for this package is not contained in a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: CnCNet
Publisher Url: https://cncnet.org
Package Name: CnCNet Red Alert 2 & Yuri's Revenge
Description: CnCNet Red Alert 2 & Yuri's Revenge is a community-run multiplayer service which allows people to play the classic Command & Conquer games Red Alert 2 and it's expansion Yuri's Revenge online.
Package Version: 8.15
Installer URL: https://downloads.cncnet.org/CnCNet5_YR_Installer.exe

I managed to create a manifest for this installer, but I don't actually own Red Alert 2 so I get an error when I try to install it. I'm not sure if it would pass validation on Azure, any insight is welcome. I attached the manifest in case anyone wants it for themselves.
CnCNet.zip

@warshanks  I finally had some time to test this out these manifests work for me since I have Red Alert 2 and Yuri's Revenge installed.  Is there any other testing we need to do or should we attempt a PR and see if it passes the validation on Azure?

@jeffwlawson I'll create a PR for it and we'll see how it goes. Fingers crossed.

@warshanks I actually created a PR earlier today. #92549
Seems to be failing due the game dependency. Are able to test to see if the /SILENT switch skips the game file check?

No change with /SILENT. I'm going to poke around some more to see if there are any other switches.

I found these switches:
/VERYSILENT /SUPPRESSMSGBOXES /NORESTART /SP-
Still no change with any of them sadly.

[Policy] Interactive Only Installer


Update rate-limits.md

Removing this page as content is covered under - https://developer.harness.io/docs/platform/security/rate-limits/
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Crashplan waiting for connection issues

Hi,
Crashplan enterprise waiting for connection issues, looks like an SSL issue not sure how to resolve it?
Brand new container deployed as follows
docker run -d
--name=crashplan-enterprise
-e USER_ID=0 -e GROUP_ID=0 -e TZ=Etc/AU -e KEEP_APP_RUNNING=1
-p 5800:5800
-v /volume1/docker/appdata/crashplan-enterprise:/config:rw
-v /volume1/Fileshare:/storage:ro
jlesage/crashplan-enterprise
See Log. many thanks!
re-event-5-1 handler.AppProtocolStartListener] SABRE:: Channel failed to reach state for app protocol start. reason=javax.net.ssl.SSLHandshakeException: (SERVER) Not Trusted.  Reasons: {Sabre Trust Manager: CertificateException->java.security.cert.CertificateException: Not Trusted - Invalid CN of 'Unknown'. }, {Whitelist: CertificateException->server certificate with unknown fingerprint: CN=Unknown, OU=PROe Server, O=Code42, L=Minneapolis, ST=Minnesota, C=US}, {DefaultTrustStore: ValidatorException->PKIX path building failed: java.security.cert.CertPathBuilderException: Unable to find certificate chain.}, channel=[id: 0x46f6feab, L:/<IP_ADDRESS>:41926 - R:/<IP_ADDRESS>:443]
[03.30.21 06:21:36.173 INFO  re-event-5-1 .handler.ChannelExceptionHandler] SABRE:: Decoder issue in channel pipeline! msg=javax.net.ssl.SSLHandshakeException: (SERVER) Not Trusted.  Reasons: {Sabre Trust Manager: CertificateException->java.security.cert.CertificateException: Not Trusted - Invalid CN of 'Unknown'. }, {Whitelist: CertificateException->server certificate with unknown fingerprint: CN=Unknown, OU=PROe Server, O=Code42, L=Minneapolis, ST=Minnesota, C=US}, {DefaultTrustStore: ValidatorException->PKIX path building failed: java.security.cert.CertPathBuilderException: Unable to find certificate chain.}. cause=java.security.cert.CertificateException: (SERVER) Not Trusted.  Reasons: {Sabre Trust Manager: CertificateException->java.security.cert.CertificateException: Not Trusted - Invalid CN of 'Unknown'. }, {Whitelist: CertificateException->server certificate with unknown fingerprint: CN=Unknown, OU=PROe Server, O=Code42, L=Minneapolis, ST=Minnesota, C=US}, {DefaultTrustStore: ValidatorException->PKIX path building failed: java.security.cert.CertPathBuilderException: Unable to find certificate chain.}. Closing ctx=ChannelHandlerContext(EXCEPTION, [id: 0x46f6feab, L:/<IP_ADDRESS>:41926 ! R:/<IP_ADDRESS>:443])
[03.30.21 06:21:36.174 INFO  re-event-5-1 .handler.ChannelLifecycleHandler] SABRE:: Channel became inactive. closedBy=THIS_SIDE, channel=[id: 0x46f6feab, L:/<IP_ADDRESS>:41926 ! R:/<IP_ADDRESS>:443], sessionState=RemotePeer-[guid=63, state=CONNECTING]; Session-[localID=1000488633526847464, remoteID=0, closed=false, expiration=null, remoteIdentity=ENDPOINT, local=null, remote=null]

Does the server you are connected to have a valid certificate ?

I haven't ever had to deal with Certs with crashplan?
Are you referring to a cert installed into our crashplan console? the server is central.crashplan.com

Hi Yes using central.crashplan.com Ive never got it working, also created the container multiple times with no resolve.

all of out other instances that run on macs have no issues, we also have instances that run in Ubunto 20.04 no issues.

When you tried to re-create the container, did you also clear the /config folder ?

I did, I trash everything each time I create a new container.

Is there a way I can reach you privately so you can provide me with a temporary account?  It would be easier if I can reproduce myself.

Thanks Jocelyn,
I actually created a ticket with crashplan, got no reply and then the
following week all instances just started working.
Really weird i know!
On Fri, Apr 16, 2021 at 12:03 PM Jocelyn Le Sage @.***>
wrote:

Is there a way I can reach you privately so you can provide me with a
temporary account? It would be easier if I can reproduce myself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jlesage/docker-crashplan-enterprise/issues/5#issuecomment-820856607,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATOFPDQS2FWLVB7C35ZJIITTI6LH3ANCNFSM42BKSZLQ
.

--
Kind Regards
Sean Inkley
Level 2, 106 Parramatta Rd, Stanmore NSW 2048
https://maps.google.com/?q=106+Parramatta+Rd,+Stanmore+NSW+2048&entry=gmail&source=g
Phone: +61 2 9550 1110 Web: www.macumen.com.au
http://www.macumen.com.au/
This communication, including any attachments, may be confidential. If you
are not the intended
recipient, can you please contact me immediately, destroy the message, and
do not copy or use any
part of this communication or disclose anything about it.

Ok, maybe a glitch on their side.  Keep me updated if you get answers from your ticket.
I will close this issue, please reopen if needed.


How to parse usertype in C++ side from sol::table?

The 'projection' is my binding function, and its return type is 'cd' (complex number). It seems I can successfully get the array table as follows:
local fid = {}
for i = 1, 1024 do
fid[i] = projection(i)
end
print(#fid)
Now I'd like to deal with the 'fid' table with C++.
cx_vec fft_1d(const sol::table& t) {
for (auto& kv : t) {
sol::object val = kv.second;
cout << val.as<cd>()<<"\n";
 }
...
}
lua side:
local spec = fft_1d(fid)
But It failed at the data parsing (val.as<cd>()).  Thanks for your help.

Other similar problems for me also include class, struct parsing in sol::table.

What does "it failed at the data parsing (val.as<cd>()) mean? Is that a compile-time error? A runtime error? What is the runtime error? What does it say when you have #define SOL_CHECK_ARGUMENTS and other (safety features)[http://sol2.readthedocs.io/en/latest/safety.html] turned on?
I need more information to help you, as that looks like valid code that should work just fine.

Thanks for your quick response.
It is a runtime error. The code was supposed to crash at the val.as<cd>().
After I added the #define SOL_CHECK_ARGUMENTS ahead of #include <sol.hpp>, and it throwed out an exception as follows:

My script was

That indicated the error should locate at local spec = fft_1d(fid).
This is my overloaded functions:

I believe that it should have nothing to do with the overloaded functions, because the function cx_vec fft_1d(const sol::table& t) was called (with ## output).

Thanks.

Thanks for letting me know. I still have a questions or two to figure this out.
Did you provide a custom binding for "cd", or are you just using the plain binding? The error indicates that, somehow, the value for second is essentially a table: do you know if your projection function is returning a table that holds userdata, or a userdata itself? As I understand it, projection should be returning a single cd object, and that should be getting serialized to lua inside of that fid table and then taken out.
The ONLY thing I can think of as going wrong here is that the table t has more than just the projection information you put on it. So when you iterate over it, it gives you some key "whatever" that has a value of table. To further inspect what's going wrong, I would suggest printing out the key values to be sure there's no extra stuff being added to your table.

Actually, I defined two projection functions, but the binding was the other.

So when I change the binding, everthing goes well.  However, it indicates that the Sol is much more powerful. Thanks for your help!

Yeah, you don't have to create bindings that specifically create Lua objects. sol2 handles any C++ type in the world, and serializes it as an "opaque pointer" kind of thing called a userdata (with member functions and variables, if you define them with new_usertype, see the tutorial and examples).
If you need to customize how cd is taken from and given to the Lua system through sol, you can also use customization points.
Good luck!

ONE FINAL WORD OF WARNING:
The iterators you use to walk through a sol::table are NOT guaranteed to iterate in numeric order, or ANY kind of order. They may iterate backwards, forwards, in the style of cuckoo-hashing, by accumulating a visited list while calling rand() to find the next target...
It is NOT something you should rely on. If you want to figure out the length of a table, call the length operation (int count = mytable.size(); using the sol API) and then iterate from 1 to count. This will save you some headaches in the future when the implementation decides not to iterate in numeric order.

Thanks for your suggestions, I have relpaced iterators with numeric indexing.


Always create tags via CLI

Fixes #239. There is a bug with the GitHub API where you can only create tags for SHA's which are the HEAD commit on a branch (https://github.com/orgs/community/discussions/68932). Attempting to register tags for other SHAs results in failures like this: https://github.com/JuliaRegistries/TagBot/issues/239#issuecomment-2246021651

I need #318 to test locally so I'll rely on the CI for testing this change.

@christopher-dG can you review this?

Bump, I'd love to see this merged!


Change homepage message

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Sorry about that.


work with chrome canary now

follow the drafts

By the way, should open source projects unify code formatting tools?

I'm sorry that I don't have enough time to contribute to this project, I will close this PR first, thank you for your time.

Thanks for your contribution, Gaubee - I'm going to clean up and reland your patch.
On a side-note, our team has cleaned up the infrastucture to make work like this easier - your patch helped us better understand those infra-related shortcomings.  So thanks for that as well.


Adds ordered list user state support to fnapi accessor cache. (#31923)

Cherry-pick #31923 into release 2.58.0

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

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

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

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @jrmccluskey

Run Java PreCommit

The Java Examples Dataflow precommit is red but it looks like the issue is pulling licenses not the actual code, so I'm inclined to go ahead and get this in after trying one last run

Same license failure seen elsewhere, LGTM

Thanks a lot!


Cargo panics when building a proc-macro crate with build-std present in any parent .cargo/config.toml

Problem
If you have a .cargo/config.toml file with build-std = ['std'] (and valid target), and you try to build a crate with proc-macro = true, Cargo panics. This is only really a problem if that .cargo/config.toml exists in some parent directory of the crate.
Steps

Clone this minimal repo.
Run cargo build, or RUST_BACKTRACE=full cargo build if you so desire.

Possible Solution(s)

Display an error message saying that build-std is not allowed for proc-macro crates.
Fix build-std for proc-macro crates.
Ignore build-std for proc-macro crates.
Implement something like #8687.

Notes
Output of cargo version: cargo 1.50.0-nightly (d274fcf86 2020-12-07)
Backtrace: hastebin
My actual use case
I have a crate that targets wasm32-unknown-unknown, and I'd like to have a small child crate so I can use procedural macros. Since I'm using atomics in the wasm32 crate, I need build-std. In this case, I will still most likely need something like #8687, so that I can reset the rustflags array. I'll probably just end up figuring out how to pull the proc-macro crate up a level to get around this whole issue, but I figure a panic is still undesirable and at the very least can be turned into a helpful error message.

This has been fixed in a corresponding nightly version between stable 1.55.0 to 1.56.0. I didn't dive deeper but at least the reproducer doesn't panic on 1.56.0 and today's nightly (cargo 1.84.0-nightly (66221abde 2024-11-19)), and in https://github.com/rust-lang/cargo/pull/14850. Hence closing.
If you still encounter this, let us know about it!


RVC Connection Time Out

Hey Mate
First of all: Thx for all the work!
I'm relatively new to this, so I hope you can help me. Every time I try to use RVC I'll get a Connection Timeout after several seconds without an error message in the console.
It only says: Press any key...
If I press one however, nothing happens.
Is there a complete log to find anywhere?
I also get some minor errors when I use the installer, i will provide a log of this after a clean reinstall. Don't know if they are part of the Problem.
So long o/ have a great day

Status update - I confirmed that for some reason it is broken with the new installer while it still works with the old installation. (I did not check if doing a clean install with the old installer works).
After debugging several things I decided to upgrade RVC package because that is now an option.
However it requires changing torch from 2.0.0 to a newer version.
As a result I am upgrading the whole project to 2.3.1 which after many tests appears to not break everything.
Consequently I am also upgrading xformers to 0.0.27 and probably updating some projects along the way.
After torch 2.3.1 migration can be done, the new RVC module can be integrated.

Finished upgrade, please confirm if you were able to update.

Thx again! Less errrors while installing... and nice ui overhaul btw.
Just tested it but now I can't open any models in RVC. If i click the dropdown nothing happens, if i click "open location" it shows the right folder however.

Thanks. As for the models - did you put them in separate folders or all in
the base folder? The requirement has been to put them each in a separate
folder, like /checkpoints/singer/singer.index; however, maybe with the
newer RVC I can just let people put them wherever they want.
On Wed, Jul 17, 2024, 3:21 AM Sp4wnf3rk3l @.***> wrote:

Thx again! Less errrors while installing... and nice ui overhaul btw.
Just tested it but now I can't open any models in RVC. If i click the
dropdown nothing happens, if i click "open location" it shows the right
folder however.
—
Reply to this email directly, view it on GitHub
https://github.com/rsxdalv/tts-generation-webui/issues/339#issuecomment-2232054757,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTRXI3GF7RE6BITKOK57TLZMW2IJAVCNFSM6AAAAABKWFP7JWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSGA2TINZVG4
.
You are receiving this because you commented.Message ID:
@.***>

Sorry for my late reply, was on a short vacation... and another Thx! Putting the models in seperate folders did the trick. Everything I tested so far is working fine now including RVC.
 :D

In fact I am encountering another issue: Tortoise takes forever generating autoregressive samples. With 16 samples its about or two to four minutes or so and with 256 i would guess over an hour with 256, for a sentence with 5 to six words. Is this normal?
Did some research and the only thing i could find, was ppl mentioning to upgrade the torch drivers with cuda support, what you did anyway if I recall that right. Also Cuda is marked as working anyway.
4070 / 12 GB


Create a better error message for moving self

Thanks, it would be nice to improve this papercut!
For the sake of alternatives, can I throw out two ideas:

allow self receiver only for Copy types. I think we could use a custom compile-time assertion to hand out a nice error message in non-copy cases.
have a helper type which we could use as slf: CopyFromPy<Self>. (Or CloneFromPy even.) I think that all would be needed would be an implementation of TryFromPyCell for the helper type.

In particular, for small enums like #[repr(u8)] it seems a shame to take by reference & all accompanying borrow checking overhead.

Allowing self for copy types sounds nice, but may be a gotcha for Python oriented folks, where all methods take self. But a good guide explanation should counter that.

In particular, for small enums like #[repr(u8)] it seems a shame to take by reference & all accompanying borrow checking overhead.

This sounds rather like it would require something similar to #1979 ?

To avoid the borrow tracking, yes I think that would be needed too.

Yeah we could make it work, but I think it would be mistake. I think most of the time people do not actually want a by-value receiver and when they do they can, if it is Copy, just copy it out themselves. We would just be adding a footgun similar to "pyo3(get) clones my field". For example the Server struct in https://github.com/PyO3/pyo3/issues/2225 doesn't sound like it's Copy, so moving it out would never work.
That said, does this discussion block merging the PR? I'd prefer to move forward with this, and we can change/remove it if we decide self receivers is what we want.


Add golint check for source code

Now there is gofmt check by hack/update-gofmt.sh.
Need to add golint check for source code and refine code according to golint result. Such as, adding a new script as hack/update-golint.sh for the check.

We also need to get code pass golint check instead of ignoring failure :).

At least policy/plugin related directories :)


Setup Identity Server

Ih think it is possible and we will do it before merging the pull request. BTW today we started integrating the IS with Asp Net Identity -> https://github.com/IdentityServer/IdentityServer3.AspNetIdentity,
However we think that this is corssing BC boundries as its accessing directly the database. Users data is not part of the Identity server invariants, the identity server is reposnisble for clients, scopes, tokens etc.
He should acces the users only tought an API(the same way its accessing Facebook, and google Users). We will build this in the Idenetity Manager and expose this functionallity trough an HTTP api, protected with scope authorization. The IS will get a token from himslef via Client Flow and will access the Identity manager. What are your toughts on this?


HBASE-26036 DBB released too early in HRegion.get() and dirty data fo…

…r some operations
This is another UT to reproduce the problem without customized ByteBuffAllocator.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  3s
Docker mode activated.


CON-2528 Update to React v16

Coverage remained the same at 24.493% when pulling 61afa408a7b0d25ceacd185f2058650caea8e07d on CON-2528-Update-to-React-16 into 65d8e8597c853b74da7c5fc8c08f174110660262 on master.

Coverage remained the same at 24.493% when pulling 30079703a09b61e86033c41b30c72d7befc6b409 on CON-2528-Update-to-React-16 into 65d8e8597c853b74da7c5fc8c08f174110660262 on master.

Coverage decreased (-0.04%) to 24.456% when pulling 3cd0af4394f24e3a4ad4ae835d65da6e9e13cc05 on CON-2528-Update-to-React-16 into 65d8e8597c853b74da7c5fc8c08f174110660262 on master.


Updated the variables and set the default values

Updated the hard coded values.
Into the variables and also created the variables for the default values.

Created new one.


Question about gid consistency

Hello,
just a quick question about gid. Can I assume that these values stay the same in between updates and could therefore be used as foreign keys?
Cheers!

I believe this is an artifact of some GIS program that creates a unique ID for each row. There is only one provided attribute for each zone and that is the timezone identifier value (America/Los_Angeles for example). Therefore, I believe that gid value you are seeing cannot be relied upon to stay the same.


Reconsider release strategy of elemental-operator and elemental-channel

Elemental channels require some rethinking:

Define versioning and what to include in them
Define desirable automation (e.g. based on OS rebuilds)
Define the relation within Elemental-operator releases

Consider the relation of elemental-operator with the OS build stack:

How it relates the supportability?

Progress on this epic:

Channels automation is done.
https://github.com/rancher/elemental-operator/pull/750 (+ 1.6.x backport https://github.com/rancher/elemental-operator/pull/752)

Still to do:

https://github.com/rancherlabs/eio/issues/2388
https://github.com/rancher/elemental/issues/1462
https://github.com/rancher/elemental-ui/issues/183

Nice to have:

https://github.com/rancher/elemental-operator/issues/759
https://github.com/rancher/elemental-operator/issues/760

Since all subtasks are closed, I'm closing this epic issue as well.


Add missing Nouvelle-Aquitaine in France

It misses one of the biggest region in France, Nouvelle-Aquitaine.

Thanks @Calyhre!


GBQ: Handle deprecation of oauth2client.tools.run()

Right now, when you use GbqConnector, you get the following warning:

WARNING:root:This function, oauth2client.tools.run(), and the use of the gflags library are deprecated and will be removed in a future version of the library.

I'm not sure how quickly this deprecation will happen or if there are alternatives to keep things the way that they are; but at least worthwhile to have an issue here to address it (in the future).
As an aside, I've been glancing at the source for the bq client that Google provides as part of Google Cloud SDK and I'm wondering how public the methods are (i.e., if we could adapt some of the code or functions that bq.py uses to authenticate and be able to get credentials set up from gcloud auth login rather than requiring a separate oauth flow). Not sure though.
Again, cc @jacobschaer , I'd be interested in your opinion on whether this is worth investigating at all. I've been finding read_gbq and to_gbq quite useful - thanks for putting it together!

Any news on this one? I'm packager of pandas on Arch Linux and gflags is only dependency left that isn't compatible with python 3 - and seems won't be in any time soon.

@aginiewicz IIRC gbq is not tested / enabled for py3 at all. I am not sure that status in general.
cc @jacobschaer
cc @azbones

Hey @jtratner, @jacobschaer and I are going to try to look at the following in the next week:
-options for Google's deprecating oauth2client.tools.run() method
-that feature that I think someone was adding to allow server auth
-python 3 support (we had some concern when we first considered this, but we'll look into again)

@jacobschaer @sean-schaefer @jtratner  Also, I just saw https://github.com/pydata/pandas/issues/9713, so we can look at testing again as we should catch things like that.

@azbones Just came over to take a look at this and did a bit of digging. It's actually not a huge change, and I've already written a patch to fix it. The replacement for tools.run is tools.run_flow(). Change the import from
from oauth2client.tools import run

to
from oauth2client.tools import run_flow, argparser

and invoke with
credentials = run_flow(flow, storage, argparser.parse_args([]))

Not sure if I should push this to #9722 or wait for that to accept, and make a new pull for this issue.
@jreback @jacobschaer - Thoughts on how to handle this?

@cgrin Cool- that is the same conclusion we had come to! We'll leave it to you, but let us know if you need any help.
The other feature we were thinking about in gbq was https://github.com/pydata/pandas/issues/8489 which is generally focused on an improved auth flow for servers. We haven't researched that one yet, but I think someone had taken a cut at it.
cc: @jacobschaer @sean-schaefer

closed by #9722


Appstudio update test-component-pac-itth

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-itth-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
24 seconds
clone-repository

✅ Succeeded
2 minutes
build-container

✅ Succeeded
39 seconds
sanity-inspect-image

✅ Succeeded
39 seconds
deprecated-base-image-check

✅ Succeeded
4 minutes
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
42 seconds
sbom-json-check

✅ Succeeded
13 seconds
sanity-optional-label-check

✅ Succeeded
12 seconds
sanity-label-check

✅ Succeeded
16 seconds
show-summary

Pipelines as Code CI/test-component-pac-itth-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
33 seconds
build-container

✅ Succeeded
12 seconds
sanity-inspect-image

✅ Succeeded
12 seconds
deprecated-base-image-check

✅ Succeeded
12 seconds
clair-scan

✅ Succeeded
54 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
7 seconds
sanity-optional-label-check

✅ Succeeded
8 seconds
sanity-label-check

✅ Succeeded
10 seconds
show-summary


Clarified a few things in the Intro tutorial

I suggest these changes to the intro tutorial.
tutorial01.txt
The location of the root URLconf isn't magic but as a beginner, it can seem so. Added a note about how Django knows where it is and that familiarity with settings will be valuable throughout the tutorial.
tutorial02.txt
Removed the suggestion that timezone be changed to local in settings.py since Django recommends leaving it as UTC which is the default when a project is started using django-admin startproject. The tutorial shouldn't direct a new user to go against Django's recommendation. Also removed a later reference to the timezone change.
tutorial04.txt
In the section Amend URLconf, Django raises an error when attempting to restart the app if the change is saved. Added a note that this is ok and will be rectified once the change in Amend Views is saved.

I have no idea why this check failed ... "pull-requests-windows/database=sqlite3,label=windows,python=Python39check". I didn't touch anything in that area of the file tutorial02.txt.

Hi @swright573 -- thanks for the interest here and working on this. 👍
My view is that most of these changes are one of personal style (for example adding next to line 718). We generally avoid these kind of changes as any change causes additional work for the translators, and there are alot of supported languages. See https://www.transifex.com/django/django/
If there is anything truly "wrong" or new helpful clarifications then I'd be supportive of those changes. The last call out on the error could be worth thinking about, for example. I've not reviewed in detail, but it seems like a nice addition at first glance.
I think it would be helpful if you could focus this PR on the new additions and any true corrections. It will drastically increase the likelihood of your proposals being accepted.
Don't worry about the failing aws test one of the fellows is looking at fixing this.
Thanks again.

Hello smithdc1 ... Thanks for your feedback. Regarding the translations, I see your point.
I will leave the "error" addition in the PR to see how it goes.
I would like to argue a little bit with you about what is "personal style" versus additional clarity for someone new to Django. I'll hold off on removing/revising these until I get your feedback.

Re the timezone-related changes ... The current tutorial suggests timezone be changed to local yet elsewhere in the Django documentation it suggests it is good practice to use UTC. Why suggest something to someone new to Django that is not considered good practice? Rather than my explanation about the option of local/UTC, what do you think about just removing all references to timezone and let it default to UTC? If someone notices the time is "different" they will have to figure it out themselves (and learn something in the process :-).

Re the location of the root URLconf ... This probably seems obvious to you as an experienced Django developer but to a beginner, it is not. A hint can give someone a sense of how Django works (e.g. settings.py is VERY IMPORTANT) which will enable them to figure things out for themselves and therefore progress more quickly.

I'm not "married" to any of my other suggestions and will remove them.
Susan

Hi Susan,

experienced Django developer

That's very kind of you but I consider myself very much a beginner. 🙂
I think the changes you've presented here are nice additions.
For now can I ask you to wrap the lines at 79 chars. There's also one line which seems to have additional white space in it. Then I think we should put this on a mergers review list.

I think it is ready to go. I don't see any unintended white space.

@swright573 did you mean to chop all of the timezone docs in the latest commit?

I did. I had suggested to you it was a bad idea to direct people to set timezone to local when Django docs elsewhere recommend leaving at the default UTC and therefore those directions should be dropped from the tutorial.  I understood from your response that you agreed. If not, I can add them back in.
Susan
Sent from my iPhone

On May 12, 2021, at 12:55 PM, David Smith @.***> wrote:

@swright573 did you mean to chop all of the timezone docs in the latest commit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi Susan,
Appologies for taking some time to get back to this.
I agree it seems odd to recomend folk to change it when elsewhere we say the sane defauly is UTC. However, I quite liked the pieces you added to point folk at the timezone docs. (It's UTC, that's best practice, but if you want to change it, see here...)

David ... I've added back the bits that you described as It's UTC, that's best practice, but if you want to change it, see here.... Let me know if there is anything else.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/24407

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/26063

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27711

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27711

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/29313

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/30991

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/31008

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/31440

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/32229

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/33406

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/36158

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/36781

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/37609

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/42649

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


serve is failing

I was not able to run serve and it looks like old-man-yells-at.herokuapp.com/ is down too (mentioning in case it is related)

I'm happy to report that the service is back up at https://oncilla.github.io/old-man-yells-at/ thanks to the magic of github pages and webassembly

Closing for now. Feel free to re-open if you encounter the error again


Typing is not recognized in Vscode

Version: v1.1.1
Issue
Running ng add @wdio/schematics will add a test/tsconfig.e2e.json file.
But Vscode only knowns typescript configuration file named tsconfig.json.
This results in multiple typing errors :

Solution
Use tsconfig.json instead of tsconfig.e2e.json to name the typescript configuration file.

@Badisi what is your experience of having different tsconfig files for app and test files?

For one, angular is using a tsconfig.json with "module": "es2020" whereas for the tests we need a tsconfig.json with "module": "commonjs"

Ah I remember the issue. Yeah in this case we surely need a separate tsconfig.json.

The schematics already adds a second tsconfig file in /test, which is great !
But the issue here is that the file is named tsconfig.e2e.json which Vscode doesn't know about...
Making sure we get a test/tsconfig.json instead of test/tsconfig.e2e.json would allow Vscode to find it and properly resolves typing

Making sure we get a test/tsconfig.json instead of test/tsconfig.e2e.json would allow Vscode to find it and properly resolves typing

Sounds like a reasonable solution to me, mind sending in a PR?

Working on it as we speak but also working on other improvements too ;-)

Thanks so much! Keep them coming ☺

For this one to work completely we would also have to make sure that the wdio.conf.ts is located in the /test folder.
Otherwise Vscode would also complain about typing.


Introduce some kind of sanity to coercions

In order to manipulate some config files, I need to deserialize some JSON that represents an array of strings, apply a transformation to each string, and serialize the resulting array back to JSON.
Most of my time on this task has been spent discovering the many wonderful surprises in Powershell's system of coercions between arrays of various lengths, non-arrays, and nulls.
#!/usr/bin/env pwsh
# Let's try an array with two elements
$x = '["foo", "bar"]'
$result = ConvertTo-Json ((ConvertFrom-Json $x) | % { "Hello, $_" })
echo $result
# [
#  "Hello, foo",
#  "Hello, bar"
# ]

# So far so good

# But what about a 1-element array?
$x = '["foo"]'
$result = ConvertTo-Json ((ConvertFrom-Json $x) | % { "Hello, $_" })
echo $result
# "Hello, foo"

# So apparently a 1-element array of strings just turns into a string

# 0-element arrays?
$x = '[]'
$result = ConvertTo-Json ((ConvertFrom-Json $x) | % { "Hello, $_" })
echo $result
# <no output>

# A 0-element array turns into $null, which in turn can't be serialized at all

I appreciate that a scripting language has slightly different concerns than a full blown programming language, but this is basically bizarro-world. I don't know what the solution is exactly, but a sense of consistency and type-safety in the primitive data structures is sorely missing when trying to use PowerShell for non-trivial tasks.

To add to @vexx32's comments:
In a nutshell, the price of using PowerShell's pipeline - both on input and output processing - is that you lose the distinction between:

a single item and single-element array
nothing / $null and an empty array

It is a slight irony that ConvertFrom-Json is not a good pipeline citizen in that it actually sends arrays as a single object through the pipeline as opposed to enumerating its elements, the way cmdlets usually do:
PS>  '[]' | ConvertFrom-Json | ConvertTo-Json
[]  # !! empty array was preserved

Because % (ForEach-Object) was also involved in your case, however, you saw the usual enumeration behavior; also note that even enclosing a command in (...) by itself forces enumeration too:
PS>  ('[]' | ConvertFrom-Json) | ConvertTo-Json
 # !! No output - (...) forced enumeration, so no element was sent to ConvertTo-Json

While you could argue that  ConvertFrom-Json's current behavior is valuable for its ability to round-trip without loss of information, that ability is easy to disrupt, as demonstrated here, and overall it is more important for ConvertFrom-Json to exhibit standard behavior - though a change may not be made for the sake of backward compatibility - see https://github.com/PowerShell/PowerShell/issues/3424
@vexx32  has already mentioned @(....), the array-subexpression operator that forces interpretation of a result as an array, unless it already is one.
ConvertTo-Json has recently gained a similar ability with its -AsArray switch:
PS> 1 | ConvertTo-Json -AsArray -Compress
[1]

Applied to your command:
PS> ConvertFrom-Json $x | % { "Hello, $_" } | ConvertTo-Json -AsArray
[
  "Hello, foo"
]

While this is not automatic round-trip behavior, at least you can predictable output JSON arrays if you know that they're expected.

In all your examples from other languages, this is only doable on values that can be confirmed to be collections at every part of the sequence.
PowerShell fundamentally isn't like that with its pipeline. Collections are broken apart, and there is fundamentally no difference between one item and many.
Consider:
$Value = 1
$Result = $Value | ForEach-Object {$_}

$Value = 1, 2
$Result = $Value | ForEach-Object {$_}

What you're suggesting is for both of these to result in arrays, even when the original item is not an array.
C#, Haskell, none of the other languages do this.
PowerShell, frankly, isn't particularly comparable to them in this regard.

$Value = 1
$Result = $Value | ForEach-Object {$_}
...

What you're suggesting is for both of these to result in arrays, even when the original item is not an array.

Wrong. I am not suggesting that 1 | ForEach-Object {$_} should be an array. I am suggesting that @(1) | ForEach-Object {$_} should be an array. In other words, whenever it is the case that the output of $x.GetType() is:
IsPublic IsSerial Name                                     BaseType
-------- -------- ----                                     --------
True     True     Object[]                                 System.Array

then the output of $x | % {$_} should similarly have type Array. You can apply this check yourself to @(1) and 1 to see the difference.

In all your examples from other languages, this is only doable on values that can be confirmed to be collections at every part of the sequence.

This sounds like nonsense. There's no repeated confirmation of the input value being an array in any of the examples I posted above. [].map(x => x * 2).map(x => x * 3) works identically to [1].map(x => x * 2).map(x => x * 3) works identically to [1, 2, 3].map(x => x * 2).map(x => x * 3): all three examples produce an array.

Correct. All of those methods, however, are only valid on collections themselves. You cannot take a scalar value as you can in PowerShell and apply such a method.
PowerShell intentionally doesn't keep track of the fact that the original item is an array or not. I don't think there is a solution to your quandary, save keeping track of it yourself. 😄

To make this more precise, currently neither $Result in:
$Value = 1
$Result = $Value | ForEach-Object {$_}

$Value = @(1)
$Result = $Value | ForEach-Object {$_}

is an array. The second should be, the first should not. Since this isn't the case, it should at least be possible to construct your own map function to provide the desired behavior as @(1) ?? map(...) ?? map(...). If we try to use | we're foiled again, since PowerShell won't leave arrays alone.
So we should perhaps have some other operator that acts as simple function application to allow working around this behavior with userland libraries.

There is such a thing, sort of, and it's quite close to hand.
You're looking for .ForEarch{} and .Where{} -- PowerShell magic methods. They can be applied to literally any object in PowerShell, just like their pipeline cousins. However, they too are not perfect.
Counter to the pipeline, using these methods will always result in a collection. Seems to be a generic collection, from what I can see.
Example:
PS> @(1).ForEach{$_}.GetType().FullName
System.Collections.ObjectModel.Collection`1[[System.Management.Automation.PSObject, System.Management.Automation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35]]

PS> (1).ForEach{$_}.GetType().FullName
System.Collections.ObjectModel.Collection`1[[System.Management.Automation.PSObject, System.Management.Automation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35]]

Although both cases require parentheses due to how PS handles parsing on decimal numerals (yes, that also is weird and strangely not exhibited by hexadecimal literals, but that's a story for another time), only the first is actually an array.
And yes, both result in a generic Collection[PSObject]. Effectively the reverse of the pipeline methods.
In this case, perhaps you could argue that PS could remember if the value was scalar and return a scalar value, but it's also not possible to know whether or not in the .ForEach{} method you opt to return multiple values for each input... so, I suppose, it's the most elegant solution to simply return a collection.

You're looking for .ForEach{} and .Where{} -- PowerShell magic methods. They can be applied to literally any object in PowerShell

Small note here, they can also be applied to $null (e.g. $null.ForEach{}.GetType().FullName returns the same)
Also they can't be applied to objects that already have a Where or ForEach method (e.g. using ForEach on List<>  will call List<T>.ForEach(Action<T>) instead of the magic method)


[mesheryctl] Read the auth token from Meshery server rather than prompting user

See mesheryctl Command Tracker for current status of commands.
Current Behavior
A set of mesheryctl subcommands depend on user to download auth.json from Meshery UI and provide it as command line argument. This does not give good UX.
Commands dependent on auth token:
mesheryctl pattern
mesheryctl perf
mesheryctl system config
Desired Behavior
Modify the mesheryctl to query meshery server for auth token and use it, rather than prompting users to provide token with "--token" option.
Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker or Kubernetes

Implementation

Acceptance Tests

Mockups

Contributor Resources

mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
Layer5 Community Slack

@hexxdump what's the state of this effort?

@hexxdump what's the state of this effort?

I am yet to start on this @leecalcote . I am still working on #3758
I hope to carve out some time for these two issues this weekend.

@hexxdump thank you. I meant to inquire less in the spirit of piling on pressure, but more in the spirit of understanding what is left to complete considered the couple of merged PRs that are related to this issue. // @piyushsingariya you might have comment here.

we have already implemented this among various places after the introduction of OAuth in mesheryctl.


Use better default names for classes and files

Currently, the default file names for generated base code are always the same generic "filename_base". I believe it would be better if the file name was derived from the class name, e.g., "MyFrame_base" for class "MyFrameBase".
The name of the derived class is always "Derived class". I believe it would be better if the name was derived from the base class name, e.g., "MyFrame_derived" for class "MyFrameDerived". Perhaps the derived file name should also be prefilled, just like the base file name is?

I wondered if the use of filename_base as a flag that the user didn't specify anything was going to be confusing -- this issue confirms that indeed it is confusing.
The properties base_file, derived_class_name, and derived_file all change when the user changes the base class_name property. Prefilling them with placehoder names is what's causing the confusion. We do need to use something in the Source code display for the #include line, which is why filename_base was used, but we should be able to special-case an empty field for this property and hard-code a temporary name.
It makes sense to have these fields be empty to avoid confusion -- we should add a tooltip to indicate that the class name should be changed first.
While we're making changes to avoid confusion, the Derived panel should display something to indicate why it's blank (the user didn't specify a class name for it).

It makes sense to have these fields be empty to avoid confusion

I guess it is a matter of opinion. I prefer to have those prefilled with sensible defaults. IMO, when a class is renamed, the user should be notified about this and asked whether the file names should be changed as well.

PR #636 also improves automatic file naming, though the addition of the option to create a non-derived class means that the derived class name and file will be ignored if the user prefers to create a non-derived class. We still don't auto-create a base filename if the default class name is used. The assumption here is that by the time the user actually wants to generate the code rather than just looking at what would be generated, they will also switch to a more reasonable class name which will automatically fill in a suggested file name.


acts_as_list_no_update does not work with nested blocks.

I encountered this issue when trying to copy a parent - child structure while still preserving positioning.
The code would have been similar to:
::Parents.acts_as_list_no_update do
  Parents.all.each do |parent|
    copy = parent.dup
    copy.save!

    ::Children.acts_as_list_no_update do
      parent.children.each do |child|
        child_copy = child.dup
        copy.children << child_copy
        child_copy.save!
      end
    end
  end
end

The problem is that acts_as_list_no_update is automatically disabled for all classes at the end of each block.
So after all the children have been copied for a parent, and the end of the ::Children.acts_as_list_no_update block, the no_update for the parent is disabled, meaning for each subsequent parent save acts_as_list alters the positions.
(Note that the above snippet is only for illustration, in my situation the parents are in seperate scoped lists).
I believe this issue stems from
https://github.com/swanandp/acts_as_list/blob/248bf23161de8c0fbc1ca05b3b7f8ba59a12f79b/lib/acts_as_list/active_record/acts/no_update.rb#L95
This line clears every class that no_update is enabled for.
Instead of:
extracted_klasses.clear
this line should probably read
extracted_klasses.reject!{|klass| klasses.include?(klass)| }

Hi @conorbdaly, thanks for bring this up. There's a couple of approaches you could take.

I have similar copying functionality in my application and I just let acts_as_list assign new positions for the duplicated items as they're made. As long as you are fetching them in order (I assume you enhance your has_many relationships with an order on the position column? has_many :children, -> { order(:position) }) the duplicated items should be added in order also.
You can use the extra classes functionality of acts_as_list_no_update:
https://github.com/swanandp/acts_as_list/blob/248bf23161de8c0fbc1ca05b3b7f8ba59a12f79b/lib/acts_as_list/active_record/acts/no_update.rb#L73 You can specify also pausing updates for the Child class like: Parent.acts_as_list_no_update([Child])

Give those a go and let me know how you get on :)

Hi @brendon, thanks for your response.
Yes, we currently have a workaround, by only explicitly wrapping each .save! call instead of wrapping each full loop.
I think that my point is more that its unexpected behavior to close the outer no_update block when an inner block ends. By using the block syntax, I would expect all acts_as_list operations for that class to be disabled until the end of the block, regardless of what other operations happen within the block.
We actually got hit with this as a bug in production, and it can be pretty hard to spot. Most DBMS's will give the results back based on actual DB id, so if the list was never re-ordered, the results will be correct.
Now, I know that one was our fault for not testing properly, but the end result was intermittent and difficult to track down.
I do understand if you guys don't want to change the functionality, but would you consider maybe adding a line or two to the readme.md to explain that nested blocks wont work with no_update?

Thanks @conorbdaly, I think you're right here in that the documentation is kind of hidden away as to how to correctly use this feature: https://github.com/swanandp/acts_as_list/blob/248bf23161de8c0fbc1ca05b3b7f8ba59a12f79b/lib/acts_as_list/active_record/acts/no_update.rb
Perhaps we could forbid nested no_update blocks and warn people to use the additional class method instead, or we could change this so that one could nest no_update blocks.
Would you be keen to look into this?

Threw an hour into this and came up with #343. This should enable nesting of no_update blocks.

Thanks @conorbdaly, I really appreciate the time you put into this. I've merged this and will do a new release shortly.


[Create New Issue]: Checked out the website, there is some glitch in data retrieval on profile pictures

Description
Description:
When checking the site out for mentors, i could spot the same image for different profiles. [repeated images]
Have taken a screenshot and attached it below:

GSSOC 2022 contributor.
Domain
Backend Dev (Express)
Code of Conduct

[X] I follow Contributing Guidelines of this project.

Hello @whitewalker01 currently we use mock data, that's why we have duplicate, when we release in production we would use real data


[POLL] Support Flash audio extraction

Right now I haven't implemented audio extraction for maintainability && SRP reasons. If you're interested leave a comment here and I'll port it if enough people respond.

That would be a great @jamesqo

Could be a nice option. Keep it lightweight though.

To make this feature perfect yet i guess #8 need to be resolved.
Right now it only works with flv links having sig or signature.

I wouldn't invest too much time in this.
You can easily get just the audio by using one of the DASH types where the audio and video are already separated.

Maybe @jamesqo can add it in the next release.
protected byte[] ExtractAudio(Stream stream)
{
    var reader = new BinaryReader(stream);

    // Is stream a Flash Video stream
    if (reader.ReadChar() != 'F' || reader.ReadChar() != 'L' || reader.ReadChar() != 'V')
        throw new IOException("The file is not a FLV file.");

    // Is audio stream exists in the video stream
    var version = reader.ReadByte();
    var exists = reader.ReadByte();

    if ((exists != 5) && (exists != 4))
        throw new IOException("No Audio Stream");

    reader.ReadInt32(); // data offset of header. ignoring

    var output = new List<byte>();

    while (true)
    {
        try
        {
            reader.ReadInt32(); // PreviousTagSize0 skipping

            var tagType = reader.ReadByte();

            while (tagType != 8)
            {
                var skip = ReadNext3Bytes(reader) + 11;
                reader.BaseStream.Position += skip;

                tagType = reader.ReadByte();
            }

            var DataSize = ReadNext3Bytes(reader);

            reader.ReadInt32(); //skip timestamps
            ReadNext3Bytes(reader); // skip streamID
            reader.ReadByte(); // skip audio header

            for (int i = 0; i < DataSize - 1; i++)
                output.Add(reader.ReadByte());
        }
        catch
        {
            break;
        }
    }

    return output.ToArray();
}

private long ReadNext3Bytes(BinaryReader reader)
{
    try
    {
        return Math.Abs((reader.ReadByte() & 0xFF) * 256 * 256 + (reader.ReadByte() & 0xFF)
            * 256 + (reader.ReadByte() & 0xFF));
    }
    catch
    {
        return 0;
    }
}

Source: (Extract Audio from FLV stream in C#)[http://stackoverflow.com/questions/1693656/extract-audio-from-flv-stream-in-c-sharp#answer-1694235]

Alright, alright. I was planning to keep it a surprise, but I'm working on an upcoming library that will add support for this. Here is a snippet of its code: https://gist.github.com/jamesqo/6784f734025e083cb011

@jamesqo looks great!

Is the support working ? I'd like to, like YoutubeExtractor does, get the audio streaming from a youtube URL. Anyway, this is a really great project!

@bbougot I've got most of the code down, it's proving pretty hard to debug though so it may take another week or so. I'm glad you found this useful!

@jamesqo  Perfect! Thank you so much, this is awesome!

Would be mind-blowing!

I feel that you guys are missing the fact that you can already get just the audio straight out of YouTube using one of the DASH types.
I don't see any need in parsing *.flv files to do that.
I've already implemented that in YouCast if you're looking for an example:
https://github.com/i3arnon/YouCast/blob/master/Source/Service/YoutubeFeed.cs#L153

@i3arnon  Hey how would i implement something like that into my current application? so that I can download the audio?

@narqs You need to call GetAllVideos(Async) and filter the results looking for DASH audio.
You can look at how I implemented it in the link above.

Oh that sure helped Thanks mate, I realized that I did not take a good look at your code, it's working now thanks again :)  :+1:

@i3arnon Thanks for helping out; I hadn't realized that was possible myself. In the meantime I've already started a new project (had to tear down the old one because of maintainability reasons), so it's too late I guess haha.

BTW, it looks like the YoutubeExtractor author has ported FlvExtract to PCLs: https://github.com/flagbug/FlvExtract


Problem: CLI Arg in task only in JSON format

Issue
It would be great to not have to write in JSON the CLI args when running a task.
Currently to limit an Ansible playbooK I need to do :

But if I add the arg in the task template, it works without being in JSON format :

The goal would be to be able to limit the scope of an ansible playbook easily like in CLI without having to format in JSON every time.
Impact
Web-Frontend (what users interact with), Ansible (task execution), Configuration
Installation method
Package
Database
MySQL
Browser
Chrome
Semaphore Version
2.10.22-e44910d-1721658299
Ansible Version
ansible [core 2.14.14]

Logs & errors
No response
Manual installation - system information
    'c:.
    lkkkx, ..       ..   ,cc,           ----------------
    okkkk:ckkx'  .lxkkx.okkkkd          OS: AlmaLinux 9.4 x86_64
    .:llcokkx'  :kkkxkko:xkkd,          Host: VMware20,1
  .xkkkkdood:  ;kx,  .lkxlll;           Kernel: Linux 5.14.0-427.33.1.el9_44
   xkkx.       xk'     xkkkkk:          Uptime: 18 hours, 39 mins
   'xkx.       xd      .....,.          Packages: 1237 (rpm)
  .. :xkl'     :c      ..''..           Shell: bash 5.1.8
.dkx'  .:ldl:'. '  ':lollldkkxo;        Display (Virtual-1): 1280x800 @ 60 z

.''lkkko'                     ckkkx.      Cursor: Adwaita
'xkkkd:kkd.       ..  ;'        :kkxo.      Terminal: /dev/pts/0
,xkkkd;kk'      ,d;    ld.   ':dkd::cc,     CPU: Intel(R) Xeon(R) Silver 4216 (z
.,,.;xkko'.';lxo.      dx,  :kkk'xkkkkc    GPU: VMware SVGA II Adapter
'dkkkkkxo:.        ;kx  .kkk:;xkkd.    Memory: 932.52 MiB / 15.34 GiB (6%)
.....   .;dk:.   lkk.  :;,           Swap: 0 B / 32.00 GiB (0%)
:kkkkkkkdoxkkx                 Disk (/): 7.35 GiB / 85.95 GiB (9%)s
,c,,;;;:xkkd.                 Local IP (ens33): <IP_ADDRESS>/22
;kkkkl...                   Locale: en_CA.UTF-8
;kkkkl
,od;
Configuration
Inventory / task scope limitation.
Additional information
No response

Hi @SysAdminShed
You are right. Looks like new args UI works not as expected :)

Wanted to add the following :
When adding multiple limit of the scope (with proper JSON format), it gives the following error :
9:51:06 AM
ERROR! the playbook: Producer could not be found
JSON format used : ["-l", "benchtest","benchtest2"]

i tried limiting host by
["-l", "host"]
but getting error
ERROR! the playbook: ["-l", "host"] could not be found

Don't use the JSON format, just input the -l host.
Best,
On Fri, Nov 1, 2024, 5:44 a.m. kingp0dd @.***> wrote:

i tried limiting host by
["-l", "host"]
but getting error
ERROR! the playbook: ["-l", "host"] could not be found
—
Reply to this email directly, view it on GitHub
https://github.com/semaphoreui/semaphore/issues/2367#issuecomment-2451602217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7DTQ5SJZHUKRGEXIFYDNZDZ6NEPFAVCNFSM6AAAAABO7XLPY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGYYDEMRRG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

that worked thank you
On Fri, Nov 1, 2024 at 7:38 PM Tristan Theroux @.***>
wrote:

Don't use the JSON format, just input the -l host.   Best,
  On Fri, Nov 1, 2024, 5:44 a.m. kingp0dd @.> wrote:
  > i tried limiting host by  >  >
["-l", "host"]  >  > but getting error
 >  > ERROR! the playbook: ["-l", "host"]
could not be found  >  > —  > Reply to this email
directly, view it on GitHub  > <
https://github.com/semaphoreui/semaphore/issues/2367#issuecomment-2451602217>,
 > or unsubscribe  > <
https://github.com/notifications/unsubscribe-auth/A7DTQ5SJZHUKRGEXIFYDNZDZ6NEPFAVCNFSM6AAAAABO7XLPY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGYYDEMRRG4>
 > .  > You are receiving this because you were
mentioned.Message ID:  > @.>  >
—
Reply to this email directly, view it on GitHub
https://github.com/semaphoreui/semaphore/issues/2367#issuecomment-2451736813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGD6KWGON54WYQ5KOLVTM3Z6NR2TAVCNFSM6AAAAABO7XLPY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRG4ZTMOBRGM
.
You are receiving this because you commented.Message ID:
@.***>

Odd behavior discovered today:
This works: -l groupName
This fails: --limit groupName
The error message received is: ERROR! the playbook: --limit groupName could not be found
It seems that the template's CLI args field cannot handle --limit, though -limit and -l both work.

This also means that things like --skip-tags and --vault-password-file won't work currently when defined in CLI Args.


Add a '_protected' flag to pointers

This is useful for config purposes.
Turns out that we didn't do a safety check for __type__ on insert,
but would generate bogus SQL and ISE, so this fixes that.
Clean up some error messages in similar id related code, also.
I think we could just call it 'protected', though.

I don't see this being gated for use in stdlib mode only. IIRC we decided to gate it.

I don't see this being gated for use in stdlib mode only. IIRC we decided to gate it.

Schema fields are automatically gated for stdlib mode only unless we opt in with allow_ddl_set=True

Schema fields are automatically gated for stdlib mode only unless we opt in with allow_ddl_set=True

Got it, cool


helm chart rendering fails

Operating system
macOS Monterey 12.4
Kluctl Version
2.10.2
Kubernetes Version
v1.22.6
kubectl Version
v1.24.0
Bug description
I'm getting the following error while rendering the sig-storage-local-static-provisioner helm chart:

Steps to reproduce
Baseline:
helm-values.yaml
helmChart:
  repo: example.com
  chartName: provisioner
  chartVersion: 2.6.0-alpha.1
  releaseName: provisioner
  output: deploy.yaml

I can't see why this should cause any errors. I did have no issues while deploying it via helm.
Relevant log output
No response

This sounds like the Helm Chart is not properly excluded from Jinja2 template rendering. Is it possible that the helm-chart.yml' is missing? In your issue, you mentioned only the helm-values.yamlbut nohelm-chart.yaml`.

@codablock I did abstract it a bit and push it into a new repository (here). Maybe that helps you for a better understanding.

Ah ok, the reason for the issue is that you put the pulled chart into local-volume-provisioner/chart while it should have been in local-volume-provisioner/charts. I assume this is the one where you manually pre-pulled the image due to #31?
Kluctl internally adds a templating exclude for the charts directory, as it would otherwise try to render the helm templates through the kluctl templating engine, which of course messes up everything. The exclude pattern assumes that the charts are inside the charts directory.

Ah... that's now embarrassing 😅 Sometimes you forget to check the little things. But you are totally right with your assumption. Thank you nevertheless and sorry for causing some trouble.


Fix amip with diagnostic edmf longrun

The amip target with diagnostic edmf longrun is currently failing in the most recent build. We should fix it soon. Maybe try increasing hyperdiffusion and/or sponge.

@szy21 Here are some plots at the time of crash after three hours. The NaNs are coming from radiation (maybe related to the rrtmgp issue?) but there are other aspects we should address in this config to ensure the spin up is as smooth as possible. For example, the ocean albedo (atmos is now released for the more realistic version), air density is too low over topo (maybe this will get better with sim time though), initial land T is perhaps too warm, and roughness lengths over land and ocean should be adjusted to more realistic values.

w fluctuations are pretty high around Antarctica too:

@szy21 Here are some plots at the time of crash after three hours. The NaNs are coming from radiation (maybe related to the rrtmgp issue?) but there are other aspects we should address in this config to ensure the spin up is as smooth as possible. For example, the ocean albedo (atmos is now released for the more realistic version), air density is too low over topo (maybe this will get better with sim time though), initial land T is perhaps too warm, and roughness lengths over land and ocean should be adjusted to more realistic values.

Oh, great, thank you! If it's NaNs in radiation we know how to fix it. It should be already fixed in the latest RRTMGP version.

But this is still with Float64 radiation, I'm a bit surprised it could have issues too. But let's see if the latest RRTMGP version can fix it.

Reopen it to track if we can change back use_reference_state to true

This now runs to completion with the new update to ClimaAtmos and ClimaLand (see this build).

Note the albedo hooks need still need to be added (see #660 ), but getting close.

Thanks! I actually want to change use_reference_state back to true. I can start a build after the albedo hook is added.

It's still breaking after the update: https://buildkite.com/clima/climacoupler-longruns/builds/465#018e0aba-da1e-4b67-846b-1efe3742af3e. The latest climaatmos release should include all the radiation fixes, we should check if radiation still has NaNs.

Here is the plot for the breaking simulation at hour 3. Radiation seems fine. It might break due to the large vertical velocity near the topography. I will try increasing hyperdiffusion.

Yes, everything I have been talking about is with use_reference_state: true. (use_reference_state: false works fine in the main.) Here is the build that breaks after ~2 hours. This is using the main after #632 is merged, but I don't think any latest change should change the result?
I start to think that there might be something wrong with the combination of reference state, topography, and edmf. Given that use_reference_state: false works fine, maybe we don't need to worry too much about it for now, if there is nothing obviously wrong. I'll think more carefully about the reference state.

The error message shows that there is a negative number in sqrt in RRTMGP. I don't know if it's because the state is wrong or something inside RRTMGP. I'll look into it.

Sounds good, thanks! And in atmos this combination of reference state, topography, and edmf? Even if we use the slab ocean? 🤔

No, we test edmf and topography separately in atmos as I think it makes more sense to have land with topography. But if this problem persists I can go back to aqualanet for testing. I did test ref state + no topo + edmf in coupler and it seems to work. I may add that to longrun if it’s consistently more stable.

This seems to work now with a stronger sponge: https://buildkite.com/clima/climacoupler-longruns/builds/478#018e1c1b-f16e-47f7-b6b7-dbaa79d48de2. I just realized that we have been using a stronger sponge for topography runs in the atmos, but not for edmf runs. As the coupler jobs use the toml file in atmos I'm using the (weaker) one for edmf. So this might be it. I think it might be easier to just specify the coupler toml file.


refactor/gui_launcher

this PR decouples the GUI protocol from the enclosure service, the GUI is fully independent from the hardware integration and does not belong there imho
a new gui service is added that fully integrates the GUI protocol logic in a standalone fashion.
Moving forward this allows us to work in PHAL independently from the mycroft HAL spec while ensuring compatibility, any mycroft enclosure importing the base enclosure class will no longer conflict when used under ovos-core
This effectively makes the enclosure service optional, it should only be used in the mark_1 after this PR

I've been thinking about this in terms of NeonCore, should gui be pulled out into a separate module? It's the most optional module in core since some installations won't implement it at all.

in the context of neon-core microservice architecture i think it would make sense to basically have the __main__.py file added in this PR in a new module with ovos-core as a dependency
I assume the neon enclosure service will also be replaced with PHAL when ovos implements it

Love this change - just curious what's the PHAL?

Plugin based Hardware Abstraction Layer


API access broken on v280.1.11 on using hostname. Host header blocked.

After an update to bosh v280.1.11, login to the bosh-director is not possible on using the hostname.
With calling the director API /info endpoint on using the hostname url it shows the same error:
curl https://bosh-bosh.service.consul:25555/info
Host not permitted
On the bosh vm itself all monit jobs are running in a healthy state and a local curl to the API
is accepted and looks fine:
bosh/65ff9923-5195-4335-99a0-30ee807588b3:~# curl -k https://localhost:25555/info
{"name":"bosh-bosh","uuid":"2413ef39-947a-47f9-b16b-d61c4e5ed8c5","version":"280.1.11 (00000000)","user":null,"cpi":"vsphere_cpi","stemcell_os":"-","stemcell_version":"1.639","user_authentication":{"type":"uaa","options":{"url":"https://bosh-bosh.service.consul:8443/","urls":["https://bosh-bosh.service.consul:8443/"]}},"features":{"local_dns":{"status":true,"extras":{"domain_name":"bosh"}},"snapshots":{"status":false},"config_server":{"status":true,"extras":{"urls":["https://bosh-bosh.service.consul:8844/api/"]}}}}

Also calling the endpoint on using the bosh VM IP is succesful.
curl https://<IP_ADDRESS>:25555/info
Comparing to bosh v280.1.10, the behaviour with the same call on using the hostname is successful:
curl https://bosh-bosh.service.consul:25555/info
{"name":"bosh-bosh","uuid":"2413ef39-947a-47f9-b16b-d61c4e5ed8c5","version":"280.1.10 (00000000)","user":null,"cpi":"vsphere_cpi","stemcell_os":"-","stemcell_version":"1.639","user_authentication":{"type":"uaa","options":{"url":"https://bosh-bosh.service.consul:8443/","urls":["https://bosh-bosh.service.consul:8443/"]}},"features":{"local_dns":{"status":true,"extras":{"domain_name":"bosh"}},"snapshots":{"status":false},"config_server":{"status":true,"extras":{"urls":["https://bosh-bosh.service.consul:8844/api/"]}}}}

This issue was discussed on slack:
https://cloudfoundry.slack.com/archives/C02HPPYQ2/p1732707019069449
Initial investigations show the cause in the Sinatra version used, which was fixed for a CVE.

@renelehmann we believe this is fixed as of https://github.com/cloudfoundry/bosh/releases/tag/v280.1.12, can you verify?
Thanks,
Chris

@rajathagasthya, @selzoc Thanks for fixing, implementing and the notification.
Tested v280.1.12 and all just works fine again.


Include a more descriptive debugger for the collector and processor

Description of the PR
This PR incorporates a child logging system that records a unique documentHash. Since the documentHash is being added to the child logger, the entire logging path will contain the documentHash, greatly enhancing the comprehensibility of the log entries.
This PR also includes tests for the child logger.
Log Outputs for the Processor
{"level":"info","ts":1712930572.1896873,"caller":"parser/parser.go:80","msg":"parsing document tree with root type: CycloneDX","documentHash":"sha256:d49be087aece91236ed2e464f5c768028b4106f5e9cd572d9b7f8e57e0a11218"}
{"level":"info","ts":1712930572.1907766,"caller":"helpers/bulk.go:38","msg":"assembling Package: 73","documentHash":"sha256:d49be087aece91236ed2e464f5c768028b4106f5e9cd572d9b7f8e57e0a11218"}
{"level":"info","ts":1712930572.1945672,"caller":"helpers/bulk.go:54","msg":"assembling Source: 0","documentHash":"sha256:d49be087aece91236ed2e464f5c768028b4106f5e9cd572d9b7f8e57e0a11218”}
...

Log Outputs for the Collector
{"level":"info","ts":1712931336.685455,"caller":"collector/collector.go:135","msg":"Successfully marshaled document.","documentHash":"sha256:6b9873ca3443dc0080208d3bfa61274026bfd5191f147a943eae442ac4c07b38"}
{"level":"info","ts":1712931336.687148,"caller":"collector/collector.go:142","msg":"Successfully wrote document to blob store.","documentHash":"sha256:6b9873ca3443dc0080208d3bfa61274026bfd5191f147a943eae442ac4c07b38"}
{"level":"info","ts":1712931336.6875935,"caller":"collector/collector.go:148","msg":"Successfully created an event.","documentHash":"sha256:6b9873ca3443dc0080208d3bfa61274026bfd5191f147a943eae442ac4c07b38"}
{"level":"info","ts":1712931336.6876295,"caller":"collector/collector.go:154","msg":"Successfully marshaled document key.","documentHash":"sha256:6b9873ca3443dc0080208d3bfa61274026bfd5191f147a943eae442ac4c07b38"}
{"level":"info","ts":1712931336.6876893,"caller":"collector/collector.go:159","msg":"Successfully published event.","documentHash":"sha256:6b9873ca3443dc0080208d3bfa61274026bfd5191f147a943eae442ac4c07b38"}
{"level":"info","ts":1712931336.6877022,"caller":"collector/collector.go:161","msg":"doc published: file:///testdata/syft-cyclonedx-docker.io-library-docker.latest.json","documentHash":"sha256:6b9873ca3443dc0080208d3bfa61274026bfd5191f147a943eae442ac4c07b38”}
...

PR Checklist

[ ] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[ ] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If OpenAPI spec is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

@nathannaveen just need to fix up the e2e test! Thanks!

LGTM with what I think this is doing, but I just want to double check:

this is not adding a debugger per-se, just another logging key to the logging object
There is some re-factoring so as to remove duplicate code between the colector and the certifier ?

This latter part is not exactly necessary but somewhat of a drive-by consequence of working on this prt of the code.

That is correct, the logger already can emit "debug" logs. This is just using that. This adds a new "child" object to the logger so that the logs can be grouped together based on the "document" being collected/ingested. This allows for better tracking of which document failed where.

Yeah, the certifier is not calling the publish at the moment (it needs to) and proper logs were being collected via the "collector" implementation of publish. So I killed that to get ready for future PRs.


updated 'Gearboy' folder and file references

'Gearboy' subfolder and gearboy.bin have been moved/renamed in the gearboy git.

Thanks.


Request - great extension

You've done a nice job.  Would you consider the following request?  Perhaps it's something if it's not easy to do that I can assist.
On add field from constructor, a good practice is to use _fieldName instead of without the underscore.  Additionally, using "this." is not a coding practice everyone uses.  Perhaps this is as simple as a configuration setting on "use underscore" on fields, and "don't use this" when possible.  You have to when of course the field matches the parameter name exactly.
Thoughts?

I'm no longer maintaining the project but I believe the this/underscore is
an option in the plugin
On 10 August 2018 at 19:26, CoderJason123<EMAIL_ADDRESS>wrote:

You've done a nice job. Would you consider the following request? Perhaps
it's something if it's not easy to do that I can assist.
On add field from constructor, a good practice is to use _fieldName
instead of without the underscore. Additionally, using "this." is not a
coding practice everyone uses. Perhaps this is as simple as a configuration
setting on "use underscore" on fields, and "don't use this" when possible.
You have to when of course the field matches the parameter name exactly.
Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jchannon/csharpextensions/issues/41, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAGapugcGFq7AylqVj8TClxsMarM0Ly1ks5uPdBDgaJpZM4V4oBA
.


Missing board image for Waveshare esp32s3_touch_lcd_1_28 board

Describe the bug
Waveshare esp32s3_touch_lcd_1_28 board is missing board img
To Reproduce
The documentation page for the Waveshare esp32s3_touch_lcd_1.28 board lacks an image of the board itself.
Expected behavior
The documentation page should display a clear image of the Waveshare esp32s3_touch_lcd_1.28 board.
Impact
The documentation appears incomplete and less user-friendly.

Please submit a PR adding an image.


FIX run with Docker environment / problem with IP address of agent

Use local IP address for agent endpoints
Add output in terminal to print endpoint (usefull for scenario)
Add Makefile to simplify commands
Update README with simplified commands and more informations

@TelegramSam Hi :-) . I will be soon ready to do a new PR with "dockerization" for vcx/wrappers/python3/demo, but for this, I need the current PR. Is it possible to merge it ? Thanks

@xaviernopre Very nice! Possibly could you fork your own PR here in the meantime until this is approved, specifically just the makefile and your use of docker IP commands.  As @dbluhm mentioned regarding the use of a Makefile, it's possible to do what von-network has done, a series of scripts: https://github.com/bcgov/von-network/tree/master/scripts?
One option regarding the concerns about creating a private network, since this is a reference agent (maybe we should define if we want to learn from a reference agent and get it going quick, like @swcurran would suggest, or if your changes should be a suggested feature for learning quickly but not apart of the official documentation?).  Regardless, for projects like https://github.com/kdenhartog/indy-dev we found that a Makefile and docker commands like yours do were really helpful for your stated preference of

very useful to be able to checkout the repo, and quickly and simply be able to run the demo

@blhagadorn I'm not sure to good understand your comments, and don't really understand what you're waiting from me, but I will try to answer... ;-)

@xaviernopre Very nice! Possibly could you fork your own PR here in the meantime until this is approved, specifically just the makefile and your use of docker IP commands.

Sorry, I don't understand. Which PR are you talking about ? What do you want me to do ?

As @dbluhm mentioned regarding the use of a Makefile, it's possible to do what von-network has done, a series of scripts: https://github.com/bcgov/von-network/tree/master/scripts?

In my opinion, using a Makefile is very useful, simple and standard. And it is a (re)emerging standard. For example, https://github.com/bcgov/von-network/blob/master/manage do the same but without being a standard. If you want to know available commands, you have juste to take a look at the targets in the Makefile. Conversely, with manage(above), you have to read many code to extract available commands. And obviously, I have all documented in the README file.

One option regarding the concerns about creating a private network, since this is a reference agent (maybe we should define if we want to learn from a reference agent and get it going quick, like @swcurran would suggest, or if your changes should be a suggested feature for learning quickly but not apart of the official documentation?).

My approach is the following : I list what I'm missing trying to use a lib or trying to run a sample code. For example, following this way, I have contributed with this https://github.com/hyperledger/indy-sdk/tree/master/docs/how-tos/write-did-and-query-verkey/nodejs, https://github.com/hyperledger/indy-sdk/blob/master/samples/nodejs/src/anoncredsRevocationScenario.js, ... It is the same here, this repo is a "reference implementation", that can be run, and the only/better solution for me (and other future ;-) ) on MacOs, is Docker :-) . As I said, developers are able to understand that this is a reference implementation, with some adjustments for the run.

Regardless, for projects like the Indy-SDK learning and getting started guide https://github.com/kdenhartog/indy-dev we found that a Makefile and docker commands like yours do were really helpful for your stated preference of

Sorry, I don't understand what you mean...

I have resolved conflicts. Is it possible to merge this PR ? ;-)

@TelegramSam @dbluhm @swcurran @blhagadorn Why not merge this PR ? I have done this work and effort to help further developpers. I have answered your questions. Is there still a problem ? Thanks

@xaviernopre Sorry to take so long on this. I am testing out your changes now and will approve once everything checks out. Thanks for your patience!

@xaviernopre I submitted a PR against your branch with some minor clarifications and additions. Once you review that, I think this PR is just about ready to go.

@dbluhm Great ! 😀