DLT-1593: add quick replies icon to editor

fix(editor): add quick replies icon to editor
Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[x] Fix
[ ] Feature
[ ] Performance Improvement
[ ] Refactor

:book: Jira Ticket
DLT-1593
:book: Description
Received some feedback from design team regarding the new icon that I'd added for quick replies. They requested that we add a label this icon and move it to the front because it's 'new' and it will be clearer for the users to see it this way.
The idea is to leave it like this for now and in the future we can revert it back to just the icon (no label) once users have familiarized themselves with what the icon does.
Change was requested by Pablo Martel and Ted Goas from design team, in case we need to discuss this change further.
:bulb: Context
I added this quick replies icon in this previous PR to be used for the quick replies in email feature.
:pencil: Checklist
For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

For all Vue changes:

[x] I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script.
[x] I have validated components with a screen reader.
[x] I have validated components keyboard navigation.

:camera: Screenshots / GIFs

...it's rendering an empty label on each button...

Other than the "Quick reply" button, each of these should be d-btn--icon-only. The Link button currently has it.

Thanks team for careful review and for putting up with my PRs XD

Not sure if this is related to this ticket, but the "Ordered List" and "Bullet List" aren't applying any style or list to the content in the editor.

@yorbi-dp thanks for catching that. It looks like it's happening in staging because I can reproduce it here https://dialtone.dialpad.com/vue/index.html?path=/docs/recipes-conversation-view-editor--docs
But it's not happening in the version we have in the app in beta. I will try to figure out what's going on and fix it here.

Not sure if this is related to this ticket, but the "Ordered List" and "Bullet List" aren't applying any style or list to the content in the editor.

Found it. Looks like it was being rendered, just hidden. Adding a margin on the left seems to do the trick.


Need up-to-date releases

Self explanatory really, how can you say you're a developer when you dont have any runnable code.

https://github.com/freddycansic/OpenGL/releases/tag/v0.1.0-alpha


Passing string with spaces as skinName in HumanNPC throws an exception.

Throws exception!

I don't think fetching offline player skin even works, actually.

This was fixed in #148


docs: fix naming of external port and domain envVars

I was trying to run Zitadel on fly.io, but had some issues getting the backend to use the right domain. It turns out, the docs had some incorrect naming of the ZITADEL_EXTERNALPORT and ZITADEL_EXTERNALDOMAIN environment variables.
This pull request fixes the naming of these variables.
Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

Hi @denniseijpe, thank you for the PR. I think this is  being already covered by @eliobischof in this PR: https://github.com/zitadel/zitadel/pull/6336

You're right! I'll close the PR


After running isso, meet TypeError: must be string without null bytes, not str

The console output is :
isso -c /opt/isso/isso.conf run
2017-03-14 18:16:15,926 INFO: connected to https://localhost/
Traceback (most recent call last):
  File "/usr/local/bin/isso", line 9, in <module>
    load_entry_point('isso==0.10.6', 'console_scripts', 'isso')()
  File "/opt/isso/local/lib/python2.7/site-packages/isso/__init__.py", line 270, in main
    wsgi.SocketHTTPServer(sock, make_app(conf)).serve_forever()
  File "/opt/isso/local/lib/python2.7/site-packages/isso/wsgi.py", line 203, in __init__
    HTTPServer.__init__(self, sock, SocketWSGIRequestHandler)
  File "/usr/lib/python2.7/SocketServer.py", line 419, in __init__
    self.server_bind()
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 110, in server_bind
    self.server_name = socket.getfqdn(host)
  File "/usr/lib/python2.7/socket.py", line 137, in getfqdn
    hostname, aliases, ipaddrs = gethostbyaddr(name)
TypeError: must be string without null bytes, not str

what happened? thanks

Could you please supply more information?
Also, does this issue still occur on python 3?


User configured box/line thickness

Per-user setting lives in local storage.  Applies

I would think per folder level would be a better route as opposed to per user level

Reminder that the selected/edited box should be in addition to the thickness of the current box.  If we are storing this at a folder level I can utilize the same metadata used to store the colors and add this to it.


MacOS M2 - install issue

Hi 👋 Thanks for this excellent repository!
I wonder if you had any debugging advice for a nix-noob who is trying to set up nix on a fresh M2 MacBook Pro running Sonoma 14.1.1
I'm encountering an issue while running the build step of bin/build. To my eyes it looks like glibc isn't successfully building. It says the GNU C library is currently unavailable for this platform. I can see an entry for glibc on search.nixos.org (but, indeed, no darwin support). It gets through to the configuring phase
My question is: why do you think it is looking for glibc when doing a build on darwin? I've tried a few things (clean reinstall, adding gnugrep and glibc to list of packages, using NixOS/nixpkgs - main and unstable) but am a bit hampered by my Nix ignorance here. Do you have any suggestions for how to address this?
Starting build...
warning: Git tree '/Users/iggy/nixos-config' is dirty
error: builder for '/nix/store/d20vsmbkvs93qcdhlaliig2g4bz38lsw-glibc-nolibgcc-2.38-23.drv' failed with exit code 1;
       last 10 log lines:
[I've added in a few more log lines from nix log for extra context:
...
@nix { "action": "setPhase", "phase": "configurePhase" }
configuring
configure flags: --disable-static --prefix=/nix/store/csvckbkqpbcq61kriv05prznm2ln1y1c-glibc-nolibgcc-2.38-23 --bindir=/nix/store/s4i9zfnjadh2qxgb1qsa8ri4j1gavr4i-glibc-nolibgcc-2.38-23-bin/bin --sbindir=/nix/store/s4i9zfnjadh2qxgb1qsa>
configure: creating cache config.cache
checking build system type... aarch64-apple-darwin23.1.0
checking host system type... aarch64-apple-darwin23.1.0
checking for gcc... gcc
checking for suffix of object files... o
checking whether the compiler supports GNU C... yes
checking whether gcc accepts -g... yes
checking for gcc option to enable C11 features... none needed
checking whether the compiler supports GNU C++... yes
checking whether g++ accepts -g... yes
checking for g++ option to enable C++11 features... none needed
checking whether g++ can link programs... no
checking for sysdeps preconfigure fragments... aarch64 alpha arc arm csky hppa i386 loongarch m68k microblaze checking for grep that handles long lines and -e... /nix/store/kxbn7dv0i7p0ylsbr53j7z0gf45kim78-gnugrep-3.11/bin/grep]
       > checking for egrep... /nix/store/kxbn7dv0i7p0ylsbr53j7z0gf45kim78-gnugrep-3.11/bin/grep -E
       > mips nios2 or1k powerpc riscv s390 sh checking for grep that handles long lines and -e... (cached) /nix/store/kxbn7dv0i7p0ylsbr53j7z0gf45kim78-gnugrep-3.11/bin/grep
       > checking for egrep... (cached) /nix/store/kxbn7dv0i7p0ylsbr53j7z0gf45kim78-gnugrep-3.11/bin/grep -E
       > sparc x86_64
       > configure: error:
       > *** The GNU C library is currently unavailable for this platform.
       > *** If you are interested in seeing glibc on this platform visit
       > *** the "How to submit a new port" in the wiki:
       > ***   https://sourceware.org/glibc/wiki/#Development
       > *** and join the community!
       For full logs, run 'nix-store -l /nix/store/d20vsmbkvs93qcdhlaliig2g4bz38lsw-glibc-nolibgcc-2.38-23.drv'.
error: 1 dependencies of derivation '/nix/store/br1z6ln523bd2ngwv1q1il49wb864ril-home-manager-applications.drv' failed to build
error: 1 dependencies of derivation '/nix/store/728vr3w204fnsi0j7jhd70w5p429x5d3-home-manager-fonts.drv' failed to build
error: 1 dependencies of derivation '/nix/store/7538zh42mv1s7jh9rjpg13hhcj6vdal4-home-manager-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/g7k28kvkwmjxl5pmavryihhm0yd77x7a-system-applications.drv' failed to build
error: 1 dependencies of derivation '/nix/store/4n8k3dfx9h3g31lji33bma9jmmda2wbj-darwin-system-23.11.20231114.e5e3346+darwin4.e67f2bf.drv' failed to build

Can you export your current environment and share the result?
$ export

Sure (thanks for a speedy reply! 🚀 )
TERM_PROGRAM=Apple_Terminal
SHELL=/bin/zsh
TERM=xterm-256color
TMPDIR=/var/folders/dc/cnzrbwx50_1cpdmxz839vmg00000gn/T/
TERM_PROGRAM_VERSION=452
TERM_SESSION_ID=85D518F6-96F9-4D08-B771-54C1A1F1B752
USER=iggy
SSH_AUTH_SOCK=/private/tmp/com.apple.launchd.zytT31kHRe/Listeners
PATH=/Users/iggy/.nix-profile/bin:/nix/var/nix/profiles/default/bin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/local/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/appleinternal/bin
LaunchInstanceID=22FE9B71-CEF3-4A7E-9F56-5D06035F31F3
__CFBundleIdentifier=com.apple.Terminal
PWD=/Users/iggy/nixos-config
XPC_FLAGS=0x0
XPC_SERVICE_NAME=0
SHLVL=1
HOME=/Users/iggy
LOGNAME=iggy
SECURITYSESSIONID=186b4
OLDPWD=/Users/iggy/nixos-config
NIX_PROFILES=/nix/var/nix/profiles/default /Users/iggy/.nix-profile
NIX_SSL_CERT_FILE=/nix/var/nix/profiles/default/etc/ssl/certs/ca-bundle.crt
LANG=en_NZ.UTF-8
_=/usr/bin/env

@dustinlyons - I tried again with nixpkgs.url = "github:NixOS/nixpkgs/nixpkgs-23.05-darwin"; in the inputs block of flake.nix and it got past that error!! 🎉
Currently in the "Switching to new generation" and installing phases.
I see you use your own fork of nixpkgs. How does my somewhat hacky fix fit in with that pattern?
Again, thank you very much for all the hard work you've put into making your config easily sharable 🙇

@ignatiusm That's great to hear! My fork provides a few options:

It brings in this package that displays a font in polybar on NixOS
It adds a timeout setting for Emacs

So you'll be fine if you're sticking with Mac.
In re: to your error, I was able to reproduce it! So that's actually great. When I try to run bin/build I now get the same thing. I only got the error after updating my flake locally to the latest nixpkgs. So previous flake inputs worked, latest flake inputs don't.
This is a good introduction to Nix. This will happen from time to time; you'll update your dependencies and the build will break. But the amazing thing about nixpkgs is someone is likely already on it! A patch is probably likely already in the world, the community is fast; all you have to do is wait, run nix flake update, and build again.
nixpkgs is amazing because it's like this massive community all patching software together :)
So, here's my general process when I update the flake and it doesn't build:

Go to nixpkgs Issues and search for the error. If nothing is there, I may submit an Issue myself
If I don't need to update right away, I'll usually just stop and wait a few days
If I do need to update, I'll try to figure out what package is broken and drop a 6-7 line patch in my overlays/ dir that locks the bad package to a previous working commit hash.
Or if I can't be bothered to do that, I'll just remove whatever is broken (if I can) with a comment that I should add it back later.

This is great because think about the past, when you get strange errors building dependencies: you're usually on your own. Here, we lean on the community of nixpkgs.

I recently added CI to this repository; I need to do the same for nixpkgs so that my fork doesn't update unless the template build passes. And need to procur a Mac Mini to run aarch64-darwin. On the list :)

Looks like it's the docker package failing to build. I commented out the lines docker and docker-compose in shared/packages.nix and it builds.

Also, the reason nixpkgs.url = "github:NixOS/nixpkgs/nixpkgs-23.05-darwin" works for you is because whatever bug this is hasn't hit that input yet. My nixpkgs fork grabs the latest commit hash.

I went ahead and submitted an issue for this: https://github.com/NixOS/nixpkgs/issues/267685

Someone released a patch to fix the issue, there is now an open PR: https://github.com/NixOS/nixpkgs/pull/267751

Thank you for your super informative and welcoming response!! I'm very grateful for your help and encouragement.
One question: how did you narrow it down to docker that was broken? It wasn't listed in the log snippet above as one of the pkgs that had glibc as a dependency. Is there a command to see the tree of dependencies?

When I reproduced it locally, the log output showed 'docker' as the last derivation in the call stack. Usually it will show you the broken package at build time.
For tree of dependencies. you can play around with nix-tree.
$ nix shell nixpkgs#nix-tree

$ cd nixos-config && nix-tree


remove Dynamical Billiards version 3.5.2 since it is impossible to install due to empty compat range

The version spec 0.5.2 - 0.2 for MuladdMacro for DynamicalBilliards at version 3.5.2 is empty. Installing it is impossible. This fact also makes it a bit weird for tools that rewrite the registry so it is best to remove it.
julia> Pkg.Versions.VersionSpec("0.5.2 - 0.2")
VersionSpec("∅")

(jl_QrFp3d) pkg> add<EMAIL_ADDRESS>    Updating registry at `~/.julia/registries/General`
    Updating git-repo `https://github.com/JuliaRegistries/General`
   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package MuladdMacro [46d2c3a1]:
 MuladdMacro [46d2c3a1] log:
 ├─possible versions are: 0.1.0-0.2.2 or uninstalled
 └─found to have no compatible versions left with DynamicalBilliards [4986ee89]
   └─DynamicalBilliards [4986ee89] log:
     ├─possible versions are: 2.0.0-4.0.0 or uninstalled
     └─restricted to versions 3.5.2 by an explicit requirement, leaving only versions 3.5.2

This patch was created by deleting the version from Versions.toml and then reloading + resaving the registry data for DynamicalBilliards

For context, this is the Project.toml for DynamicBilliards 3.5.2:
name = "DynamicalBilliards"
uuid = "4986ee89-4ee5-5cef-b6b8-e49ba721d7a5"
repo = "https://github.com/JuliaDynamics/DynamicalBilliards.jl.git"
version = "3.5.2"

[deps]
Elliptic = "b305315f-e792-5b7a-8f41-49f472929428"
MuladdMacro = "46d2c3a1-f734-5fdb-9937-b9b9aeba4221"
Requires = "ae029012-a4dd-5104-9daa-d747884805df"
StaticArrays = "90137ffa-7385-5640-81b9-e52037218182"

[compat]
julia = "≥ 1.0.0"
MuladdMacro = "≥ 0.5.2"
StaticArrays = "≥ 0.8.2"
Elliptic = "≥ 0.5.0"

Here is what I suspect happened:

DynamicBilliards 3.5.2 was registered with a Project.toml file that contains the compat entry MuladdMacro = "≥ 0.5.2". In the registry, this is stored as MuladdMacro = "0.5.2 - *".
Sometime later, we did a registry-wide RetroCapping. At the time that we did the RetroCapping, the latest registered release of MuladdMacro was 0.2.x. [^1]. Therefore, in the registry, RetroCap changes MuladdMacro = "0.5.2 - *" to MuladdMacro = "0.5.2 - 0.2".

I think we have two options here:

Completely remove DynamicBilliards 3.5.2 from the registry.
Make both of the following changes:

In the registry, change the compat entry (for DynamicBilliards 3.5.2 only) from MuladdMacro = "0.5.2 - 0.2" to MuladdMacro = "0.2 - 0.2", which is likely a more accurate representation of what the package authors intended.
Yank DynamicBilliards 3.5.2

[^1: I know that this is true, because today, the latest registered release of MuladdMacro is still 0.2.x: https://github.com/JuliaRegistries/General/blob/master/M/MuladdMacro/Versions.toml

Since it is literally uninstallable I think it is fine to delete it like is done here. It won't break instantiate. Changing to 0.2-0.2 also works.
The yank option is not great because after normalization (https://github.com/JuliaRegistries/General/pull/65997) this will turn into an empty range which is printed as [] which I am not sure even parses back as a VersionRange.


Blog catageories do not appear

Hello,
I recently started using this theme and now I noticed that blog categories do not seem to appear. The page works, but there are no posts in it. Weirdly, the pagination does appear.

The blog page in general does however seem to work.

https://domain.tld/category/news/ does not work
https://domain.tld/blog/ does work

Am I missing something? Do I need to create a template file? There are no PHP errors.

Hi @grandeljay, thanks for reporting this!
Found the issue right away and will fix it asap. In the meantime, you can fix it by yourself by creating template-parts/content file that contains the things you want to show in archives for each post. Or replace the loop in archive.php with the same loop that is used for the blog page https://github.com/digitoimistodude/air-light/blob/master/index.php#L34-L46

That fixed it, thanks!

Fixed in d317b49fb175a80ad8b6001e7ae3fd97b1ae7905. Once again, thank you @grandeljay for noticing this!


Maximum call stack exceeded

App crashes with error Maximum call stack exceeded when using this svg: https://upload.wikimedia.org/wikipedia/commons/0/0a/Argentina_(orthographic_projection).svg
Environment info
System:
OS: macOS 12.6
CPU: (8) x64 Apple M2
Memory: 38.04 MB / 24.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 16.15.1 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 6.14.17 - /usr/local/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 16.0, macOS 12.3, tvOS 16.0, watchOS 9.0
Android SDK:
API Levels: 23, 28, 29, 30, 31, 33
Build Tools: 29.0.2, 30.0.0, 30.0.2, 30.0.3, 31.0.0, 33.0.0
System Images: android-33 | Google Play ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2021.2 AI-212.5712.43.2112.8815526
Xcode: 14.0/14A309 - /usr/bin/xcodebuild
Languages:
Java: 11.0.12 - /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.6 => 0.70.6
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Library version: 13.7.0
Steps To Reproduce
Add
<SvgCssUri
                width="100%"
                height="100%"
                uri="https://upload.wikimedia.org/wikipedia/commons/0/0a/Argentina_%28orthographic_projection%29.svg"
              />

App crashes
Describe what you expected to happen:
Image load

Hello @Pingou,
Do you still have that issue?
I tried to reproduce that, but everything worked well.

@bohdanprog Sorry I do not remember anything about this issue.


Memory usage of a wasm program

Greetings,
Is there a way of knowing how many pages a wasm program used during its execution? I did not find references to this on the documentation. Would it be possible to use the data method of the Store in order to do this?
Thanks!

For this you can use the ResourceLimiter trait combined with Store::limiter to sum up all memories grown within a Store. That will indicate memory that wasm had accessible but not necessarily the memory that wasm used (e.g. reserved address space but it it may not all be paged in by the OS). If you're curious about actual page usage you'd need to get the actual Memory object to use APIs like mincore to see if it's actually paged in.

I think this is solved with what I mentioned above so I'm going to close this, but if it doesn't work for you feel free to reopen and we can try to continue helping.

Would memory.data_size work in this use case?


Fix broken CodeMirror after webpack upgrade

Ticket: Closes #2584
Note that you still won't see it load due to an component bug -- that has been filed separatedly.
Cause
Webpack 5 changed how it handles ES6 modules by default, and here's a reminder:

Note that when using import() on ES6 modules you must reference the .default property as it's the actual module object that will be returned when the promise is resolved.

Change
We probably don't need to be backward compatible with older webpack, but look for both syntax nonetheless.

Thanks KP!


Add config to another branch

Hmm, why close it, we still need this pr I think.

I am doing this update also with v1.x:)
Do you mean we should update v1.3.3-f to databend otherwise v1.x?

Databend currently is using v1.3.3 version of bincode,  I can't modify databend to depend on  v1.x, there are many conflicts that need to be resolved.

Got it.


WIP figures

Description

Related Issue

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[ ] I have read the Code of Conduct
[ ] I have updated the documentation accordingly.
[ ] All commits are GPG signed

Hello @2lambda123! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file figures/make_figure.py:

Line 9:1: E303 too many blank lines (3)
Line 11:24: W291 trailing whitespace
Line 16:21: W291 trailing whitespace
Line 22:80: E501 line too long (83 > 79 characters)
Line 29:67: W291 trailing whitespace
Line 30:47: E225 missing whitespace around operator
Line 30:80: E501 line too long (93 > 79 characters)
Line 31:1: W293 blank line contains whitespace
Line 33:80: E501 line too long (89 > 79 characters)
Line 42:80: E501 line too long (85 > 79 characters)
Line 49:1: W293 blank line contains whitespace
Line 52:1: E302 expected 2 blank lines, found 1
Line 65:80: E501 line too long (92 > 79 characters)
Line 73:80: E501 line too long (95 > 79 characters)
Line 76:1: E305 expected 2 blank lines after class or function definition, found 1
Line 76:12: E225 missing whitespace around operator


Remove CL2_ENABLE_IMMUTABLE_SECRETS_AND_CONFIGMAPS usage when plausible

This basically means reverting https://github.com/kubernetes/perf-tests/pull/1692 when support for 1.17 is dropped.
/kind cleanup

/lifecycle frozen


feat: add combined uid feature

What does it do?
Add support for combined UID keys.
Why is it needed?
solves feature request: #39
How to test it?
Create a config with custom types and inside the type config use an array for uid and also a single string. For example:
...
            customTypes: [
                {
                    configName: 'product',
                    queryString: 'api::product.product',
                    uid: ['productId', 'locale']
                },
                {
                    configName: 'page',
                    queryString: 'api::page.page',
                    uid: 'slug'
                }
            ]
...

Related issue(s)/PR(s)
#39

Hey @tilman
Looks good.
I fixed a couple of bugs I found.
Could you tick the Allow edits from maintainers checkbox in the PR?
Then I can push my changes to your branch.


Add DA property window

As an engineer
I would like to see more details of an DA in the IED editor
So I can understand the IED configutation better
Background
When the user click on the (i) Icon, a widow should pop-up:

The pop-up window should contain (read only):

Field
Value

nsdoc description
example

description in SCL
Check switching authority

IED name
CoupField_QA1_QB1_QB2

Accespoint
P1

Sever
server

Logical device
CBSW

Logicalnode
LLN0

Data attribute name
Mod [Modus]

Common data class
SPS [single point status]

Functional contraint
ST [status]

Data attribute type
INT32

The related to requests on showing details to run with Fledge: https://github.com/openscd/open-scd/issues/554


fixed an error for 'GET_DECODED_TRANSACTION_INPUT' when the 'transaction.input' was '0x'

Ganache
Thank you for contributing! Take a moment to review our contributing guidelines
to make the process easy and effective for everyone involved.
You must open an issue before embarking on any significant pull request, especially those that
add a new library or change existing tests, otherwise you risk spending a lot of time working
on something that might not end up being merged into the project.
Before opening a pull request, please ensure:

[x] You have followed our contributing guidelines
[x] Pull request has tests
[x] Code is well-commented, linted and follows project conventions
[x] Documentation is updated (if necessary)
[x] Description explains the issue/use-case resolved and auto-closes related issues

Be kind to code reviewers, please try to keep pull requests as small and focused as possible :)
IMPORTANT: By submitting a patch via a Pull Request, you agree to allow the project
owners to license your work under the terms of the MIT License.

Transaction input was '0x' when transfer ETH to a contract address.
At this time, the absence of decodeddata will cause the system to report an error

#4803 https://github.com/trufflesuite/ganache-ui/issues/4803

#4803

@Pemburu88 This PR has been completed and can be merged

Thanks for the PR, and sorry for taking so long to review it. I think this is more properly handled by https://github.com/trufflesuite/ganache-ui/pull/5341


[Error] Error with Exodus database

name: Error with Exodus database
about: File an issue
title: '[Error] < Error with Exodus database >'
labels: Error
assignees: ''

Current Behavior:

Example fails at writing the output to the exodus database ⭕
Expected Behavior:

Example should produced the expected exodus database for post-processing in paraview.
Steps To Reproduce:

julia> using Pkg
julia> Pkg.update("PeriLab")
Updating registry at ~/.julia/registries/General.toml
No Changes to ~/.julia/environments/v1.10/Project.toml
No Changes to ~/.julia/environments/v1.10/Manifest.toml
julia> Pkg.add("PeriLab")
Resolving package versions...
No Changes to ~/.julia/environments/v1.10/Project.toml
No Changes to ~/.julia/environments/v1.10/Manifest.toml
julia> Using PeriLab
julia> PeriLab.get_examples()
"~bin/examples"
julia> PeriLab.main("examples/DCB/DCBmodel.yaml")
Environment:

OS: Ubuntu 20.04

Anything else:

PeriLab.main("examples/DCB/DCBmodel.yaml")
PeriLab.                   d8b 888               888       |  Version: 1.0.6
888   Y88b                 Y8P 888               888       |
888    888                     888               888       |  Copyright: Dr.-Ing. Christian Willberg, M.Sc. Jan-Timo Hesse
888   d88P .d88b.  888d888 888 888       8888b.  88888b.   |  Contact<EMAIL_ADDRESS><EMAIL_ADDRESS>8888888P" d8P  Y8b 888P"   888 888          "88b 888 "88b  |
888       88888888 888     888 888      .d888888 888  888  |  License: BSD-3-Clause
888       Y8b.     888     888 888      888  888 888 d88P  |
888        "Y8888<PHONE_NUMBER>8888 "Y888888 88888P"   |  Gitlab: https://gitlab.com/dlr-perihub/PeriLab.jl
┌ Warning: No current git info.
└ @ PeriLab ~/.julia/packages/PeriLab/pZpHW/src/PeriLab.jl:210
[ Info: Read input file examples/DCB/DCBmodel.yaml
[ Info: Read mesh file examples/DCB/DCBmodel.txt
[ Info: Read node sets
[ Info: 2d problem assumed, please define plane stress or plane strain if needed in Physics
[ Info: Init Neighborhoodlist
[ Info: Finished init Neighborhoodlist
[ Info: Apply bond filters
[ Info: Finished applying bond filters
[ Info: Start distribution
[ Info: Finished distribution
[ Info: Create Overlap
[ Info: Finished Overlap
[ Info: Mesh input overview
[ Info: -------------------
[ Info: Number of nodes: 1710
[ Info: Geometrical degrees of freedoms: 2
[ Info: -------------------
[ Info: Synchronize data over cores
[ Info: Get node sets
[ Info: FEM option is set to false
[ Info: Finish init data
[ Info: Init solver
[ Info: Read properties
[ Info: Init models
[ Info: Init damage models
[ Info: Init material models
[ Info: Init pre calculation models
[ Info: Init Boundary Conditions
[ Info: Init Verlet
[ Info: =======================
[ Info: ==== Verlet Solver ====
[ Info: =======================
[ Info: Initial time: 0.0 [s]
[ Info: Final time: 0.0004 [s]
[ Info: Minimal time increment: 1.7958019959575423e-6 [s]
[ Info: Safety Factor: 0.95
[ Info: Time increment: 1.7021276595744682e-6 [s]
[ Info: Number of steps: 235
[ Info: Numerical Damping 5.0e-6
[ Info: Finished Init Solver
[ Info: Init write results
┌ Warning: "Temperature" is not defined as variable
└ @ PeriLab.IO.Parameter_Handling ~/.julia/packages/PeriLab/pZpHW/src/Support/Parameters/parameter_handling_output.jl:154
[ Info: Create output examples/DCB/DCBmodel_Output1_red.e
ERROR: MethodError: no method matching Exodus.ExodusDatabase{Int32, Int32, Int32, Float64}(::String, ::String, ::Exodus.Initialization{Int32})
Closest candidates are:
Exodus.ExodusDatabase{M, I, B, F}(::Any, ::Any, ::Any, ::Any, ::Any, ::Any, ::Any, ::Any, ::Any, ::Any, ::Any, ::Any) where {M, I, B, F}
@ Exodus ~/.julia/packages/Parameters/MK0O4/src/Parameters.jl:503
Stacktrace:
[1] create_result_file(filename::String, num_nodes::Int64, num_dim::Int64, num_elem_blks::Int64, num_node_sets::Int64)
@ PeriLab.IO ~/.julia/packages/PeriLab/pZpHW/src/IO/exodus_export.jl:41
[2] init_write_results(params::Dict{Any, Any}, output_dir::String, path::String, datamanager::Module, nsteps::Int64, PERILAB_VERSION::String)
@ PeriLab.IO ~/.julia/packages/PeriLab/pZpHW/src/IO/IO.jl:373
[3] macro expansion
@ ~/.julia/packages/TimerOutputs/RsWnF/src/TimerOutput.jl:237 [inlined]
[4] macro expansion
@ ~/.julia/packages/PeriLab/pZpHW/src/PeriLab.jl:239 [inlined]
[5] macro expansion
@ ~/.julia/packages/TimerOutputs/RsWnF/src/TimerOutput.jl:237 [inlined]
[6] main(filename::String, output_dir::String, dry_run::Bool, verbose::Bool, debug::Bool, silent::Bool)
@ PeriLab ~/.julia/packages/PeriLab/pZpHW/src/PeriLab.jl:188
[7] top-level scope
@ REPL[7]:1

Hi @oldninja. It looks like there is an issue with the Exodus Package.
Can you tell me the Version of the Exodus Package.
Pkg.status("Exodus")

julia> Pkg.status("Exodus")
No Matches in ~/.julia/environments/v1.10/Project.toml

Is there a way to force it to update all the packages?

Thank you @JTHesse ! I updated the packages, and now it is working fine.
Do we need to update all the packages as we upgrade the versions with PeriLab? Or what do you think is the best way to maintain compatibility with the packages you are using as development continues?
For reference, these were all the updates that it applied:
julia> ~/.julia/environments/v1.10/Manifest.toml
~/.julia/environments/v1.10/Manifest.toml
   Installed MPItrampoline_jll ── v5.3.2+0
   Installed DataStructures ───── v0.18.17
   Installed LogExpFunctions ──── v0.3.27
   Installed Quaternions ──────── v0.7.6
   Installed Rotations ────────── v1.7.0
   Installed NetCDF_jll ───────── v400.902.209+0
   Installed CSV ──────────────── v0.10.12
   Installed Compat ───────────── v4.14.0
   Installed MacroTools ───────── v0.5.13
   Installed CodecZlib ────────── v0.7.4
   Installed TupleTools ───────── v1.5.0
   Installed ArgParse ─────────── v1.1.5
   Installed TensorOperations ─── v4.1.0
   Installed Nettle_jll ───────── v3.7.2+0
  Downloaded artifact: Blosc
  Downloaded artifact: MPICH
  Downloaded artifact: libzip
  Downloaded artifact: OpenSSL
  Downloaded artifact: Hwloc
  Downloaded artifact: Bzip2
  Downloaded artifact: XZ
  Downloaded artifact: GnuTLS
  Downloaded artifact: PMIx
  Downloaded artifact: prrte
  Downloaded artifact: P11Kit
  Downloaded artifact: libevent
  Downloaded artifact: Lz4
  Downloaded artifact: NetCDF
  Downloaded artifact: Nettle
  No Changes to `~/.julia/environments/v1.10/Project.toml`
    Updating `~/.julia/environments/v1.10/Manifest.toml`
  [c7e460c6] ↑ ArgParse v1.1.4 ⇒ v1.1.5
  [336ed68f] ↑ CSV v0.10.11 ⇒ v0.10.12
  [944b1d66] ↑ CodecZlib v0.7.3 ⇒ v0.7.4
  [34da2185] ↑ Compat v4.10.1 ⇒ v4.14.0
  [9a962f9c] ↑ DataAPI v1.15.0 ⇒ v1.16.0
  [864edb3b] ↑ DataStructures v0.18.15 ⇒ v0.18.17
⌅ [f57ae99e] ↑ Exodus v0.11.0 ⇒ v0.11.1
  [8ac3fa9e] ↑ LRUCache v1.6.0 ⇒ v1.6.1
  [2ab3a3ac] ↑ LogExpFunctions v0.3.26 ⇒ v0.3.27
⌃ [da04e1cc] ↓ MPI v0.20.19 ⇒ v0.20.16
  [1914dd2f] ↑ MacroTools v0.5.12 ⇒ v0.5.13
  [d96e819e] - Parameters v0.12.3
  [94ee1d12] ↑ Quaternions v0.7.5 ⇒ v0.7.6
  [6038ab10] ↑ Rotations v1.6.1 ⇒ v1.7.0
  [90137ffa] ↑ StaticArrays v1.8.1 ⇒ v1.9.3
  [fd094767] - Suppressor v0.2.6
  [6aa20fa7] ↑ TensorOperations v4.0.8 ⇒ v4.1.0
  [3bb67fe8] ↑ TranscodingStreams v0.10.2 ⇒ v0.10.3
  [9d95972d] ↑ TupleTools v1.4.3 ⇒ v1.5.0
  [3a884ed6] - UnPack v1.0.2
  [409d34a3] ↑ VectorInterface v0.4.2 ⇒ v0.4.4
  [0b7ba130] + Blosc_jll v1.21.5+0
  [6e34b625] ↑ Bzip2_jll v1.0.8+0 ⇒ v1.0.8+1
  [0951126a] + GnuTLS_jll v3.7.8+1
  [e33a78d0] + Hwloc_jll v2.10.0+0
  [5ced341a] + Lz4_jll v1.9.4+0
  [7cb0a576] ↑ MPICH_jll v4.1.2+1 ⇒ v4.2.0+0
  [f1f71cc9] ↑ MPItrampoline_jll v5.3.1+1 ⇒ v5.3.2+0
  [9237b28f] ↑ MicrosoftMPI_jll v10.1.4+1 ⇒ v10.1.4+2
  [7243133f] ↑ NetCDF_jll v400.902.208+0 ⇒ v400.902.209+0
  [4c82536e] + Nettle_jll v3.7.2+0
  [fe0851c0] ↑ OpenMPI_jll v4.1.6+0 ⇒ v5.0.2+0
  [458c3c95] ↑ OpenSSL_jll v3.0.12+0 ⇒ v3.0.13+0
  [c2071276] + P11Kit_jll v0.24.1+0
  [32165bc3] + PMIx_jll v4.2.7+0
  [ffd25f8a] + XZ_jll v5.4.6+0
  [1080aeaf] + libevent_jll v2.1.13+1
  [337d8026] + libzip_jll v1.10.1+0
  [eb928a42] + prrte_jll v3.0.2+0
  [781609d7] + GMP_jll v6.2.1+6

In the future you just need to update PeriLab, normally you don't need to worry about the dependencies.


fixes for react 14, add react-dom and keys to list items

Fix some warnings for React v14 upgrade:
Added dependency 'react-dom'
fixed warnings regarding React.findDOMNode and React.render
fixed warnings regarding missing key attribute in list items

Thanks @robreiss!


Can not import the preview module from file ./wxglade_out.py

Hi,
I have the following error:
Can not import the preview module from file ./wxglade_out.py
The details are written to the log file
when I click on the preview button.
In the log file  /home/myusername/.wxglade/wxglade.log
there isn't written anything about the error.
I also tried to change the output path but nothing is changed.

This means that importing the preview code fails completely.
Is this with all .wxg files? Can you isolate the problem into a small sample file?
What is the version and the platform?

It's not with all the wxg files. It started with the same project when I added a sizer and some other object like one statictext, a line, and one textctrl. Then with diff I tried to see the difference.. after that I tried to remove the object I have added. To the end I have to file that are pretty the same but one work an the other not work.
Here is the diff:
$ diff xxxxxx-0.3.wxg NGxxxxxxx.wxg
2c2
<
---
>
I also tried to remove pycache directory.. but nothing was changed.
It's very weird.

Could you please try with a recent version? At least 0.9.6 or better from the github master branch?

Yes, I downloaded the 0.9.6 from sourceforge, and it's the same.
I also tried to rename the three file under /home/myuser/.wxglade, but nothing.

Then please upload a small sample .wxg.
From the diff above I can't conclude anything.

Ah, hold on. I see at least one possible source of the problem.

Well, probably it's not related. There's a bug that enables you to add a menu separator as toplevel menu. That will fail.
Could you please check whether you have a menu separator at level 0?

I think I have found the problem but it's really strange. Depend of the . (dot) in the name of the file. All the file that have a dot in the name (excluded the .wxg extension) don't work. Indeed the problem is appeared when I renamed my app with myapp-0.3.wxg

OK, I have updated the master branch to replace "." with "_" for the preview filename.
All in all there's some room for improvement on this. Maybe I will find the time...
If you don't want to update, to the git version, you may fix the issue yourself.
Go to application.py. In Application._get_preview_filename add a line with .replace:
                dirname, basename = os.path.split(self.filename)
                basename, extension = os.path.splitext(basename)
                basename = basename.replace(".", "_")
                if not os.path.exists(dirname):
                    error = "Directory '%s' not found"%dirname
                elif not os.path.isdir(dirname):
                    error = "'%s' is not a directory"%dirname

Thanks for reporting.

it works!
wow, so fast!! thank you very much!!


Remove support for system models

Support for system models has complicated the codebase and makes it hard to reuse models, as they now have an implicit dependency on a system model.
On balance I think we should make a breaking change and remove the support for system models, forcing people to explicitly declare dependencies on their own models using an import with a from.

A few comments:

We need a definition for what the structure of a transaction event participant etc is. Those need to be built in in some sense
One of the motivations for the current structure is to be able to switch those systems model (I think two main instances are for hyperledger composer and for Cicero). There are a set of defaults for hyperledger. This could be simplified if this gets removed (not sure if it's acceptable).
If we revise the top of the type hierarchy, it might be nice to decide on a super type for all of classes (e.g., concept could be the super class for everything or it could be object or class)

Why having a top type would be useful: https://github.com/accordproject/ergo/issues/420

Agreed
Yes, I think this is acceptable. We can preserve the hyperledger system model as a "standard" model on models.accordproject.org so that people can import it into their models and refactor if required.
Agreed. concept seems like a natural candidate, but that may have some API implications.

Small fact finding mission. I can't seem to find a clear dependency from Concerto to HL:
bash-3.2$ grep -r concerto composer
Binary file composer/packages/composer-admin/test/data/businessnetwork.zip matches
composer/packages/composer-common/test/data/zip/test-npm-archive/node_modules/animaltracking-model/.npmignore:!chaincode/src/concerto
composer/packages/composer-common/test/data/zip/test-npm-archive/node_modules/animaltracking-model/models/mozart.cto:namespace com.ibm.concerto.mozart
Binary file composer/packages/composer-common/test/data/zip/test-archive.zip matches
Binary file composer/.git/objects/pack/pack-19a8cd93ba0365a189b8f632a9e5f3440428f9ae.pack matches
Binary file composer/.git/index matches

With that project being deprecated, can we assume supporting the hyper ledger composer system model is not a requirement?

Small fact finding mission. I can't seem to find a clear dependency from Concerto to HL Composer:
bash-3.2$ grep -r concerto composer
Binary file composer/packages/composer-admin/test/data/businessnetwork.zip matches
composer/packages/composer-common/test/data/zip/test-npm-archive/node_modules/animaltracking-model/.npmignore:!chaincode/src/concerto
composer/packages/composer-common/test/data/zip/test-npm-archive/node_modules/animaltracking-model/models/mozart.cto:namespace com.ibm.concerto.mozart
Binary file composer/packages/composer-common/test/data/zip/test-archive.zip matches
Binary file composer/.git/objects/pack/pack-19a8cd93ba0365a189b8f632a9e5f3440428f9ae.pack matches
Binary file composer/.git/index matches

With that project being deprecated, can we assume supporting the hyper ledger composer system model is not a requirement?

Yes. Composer still has an old/private copy of Concerto packaged with it - so those people will be fine.

The current Accord Project system model is as follows:
namespace org.accordproject.base

/**
 * System model for Accord Project models. If you use the CiceroModelManager
 * from the @accordproject/cicero-core npm module you do not need to import this file.
 */
abstract asset Asset {  }

abstract participant Participant {   }

abstract transaction Transaction identified by transactionId {
  o String transactionId
}

abstract event Event identified by eventId {
  o String eventId
}

Defined in: https://github.com/accordproject/models/blob/master/src/cicero/base.cto

Small fact finding mission. I can't seem to find a clear dependency from Concerto to HL Composer:
bash-3.2$ grep -r concerto composer
Binary file composer/packages/composer-admin/test/data/businessnetwork.zip matches
composer/packages/composer-common/test/data/zip/test-npm-archive/node_modules/animaltracking-model/.npmignore:!chaincode/src/concerto
composer/packages/composer-common/test/data/zip/test-npm-archive/node_modules/animaltracking-model/models/mozart.cto:namespace com.ibm.concerto.mozart
Binary file composer/packages/composer-common/test/data/zip/test-archive.zip matches
Binary file composer/.git/objects/pack/pack-19a8cd93ba0365a189b8f632a9e5f3440428f9ae.pack matches
Binary file composer/.git/index matches

With that project being deprecated, can we assume supporting the hyper ledger composer system model is not a requirement?

Yes. Composer still has an old/private copy of Concerto packaged with it - so those people will be fine.

Ah, I see. Thanks for clarifying!

The fundamental role of the systems model is to define the type names Asset Participant Transaction and Event and the build-in attributes/fields in them (e.g., in the Accord Project system model Asset has no built-in field and Transaction has a single builtin transactionId field.
With that in mind I could see a couple of options:

Fold the current Accord Project system model and make it built-in, so all users model assume the same definitions. A couple of decisions to make:

What namespace should it be in. Are we happy with org.accordproject.base.
Should we have Concept be a super type of the other which would provide us with a well formed hierarchy with a root Concept type. This would be the equivalent of having extends Concept in those definitions.

Remove the system entirely and define those names the same way we define Concept:
This might make more sense if the implied semantic is as if we had the following systems model:

namespace org.accordproject.base

abstract asset Asset {  }
abstract participant Participant {   }
abstract transaction Transaction {   }
abstract event Event {   }

Still similar decisions on namespace and whether to add an implicit extends Concept for all of those.
A drawback of 2. is that identifiers for some of those has to become part of the user models.
A benefit of 2. is that if anyone has used a system model with different assumptions on their system models (e.g., in which transactions do not have built-in identifier), those can more easily be ported.

Agreed
Yes, I think this is acceptable. We can preserve the hyperledger system model as a "standard" model on models.accordproject.org so that people can import it into their models and refactor if required.
Agreed. concept seems like a natural candidate, but that may have some API implications.

One more question on this. Where can I find a definition of the hyperledger system model?

The fundamental role of the systems model is to define the type names Asset Participant Transaction and Event and the build-in attributes/fields in them (e.g., in the Accord Project system model Asset has no built-in field and Transaction has a single builtin transactionId field).
With that in mind I could see a couple of options:

Fold the current Accord Project system model and make it built-in, so all user models assume the same definitions. A couple of decisions to make:

What namespace should it be in. Are we happy with org.accordproject.base.
Should we have Concept be a super type for the others, which would provide us with a well formed hierarchy with a root Concept type. This would be the equivalent of having extends Concept in those definitions.

Remove the system model entirely and define those names the same way we define Concept:
This might make more sense if the implied semantic is as if we had the following systems model:

namespace org.accordproject.base

abstract asset Asset {  }
abstract participant Participant {   }
abstract transaction Transaction {   }
abstract event Event {   }

Still similar decisions on namespace choice and whether to add an implicit extends Concept for all of those has to be made.

A drawback of 2. is that identifiers for some of those has to become part of the user models.
A benefit of 2. is that if anyone has used a system model with different assumptions from the AP ones (e.g., in which transactions do not have built-in identifier), those can more easily be ported.
@dselman @mttrbrts @sstone1 I would love your feedback on this.

[For what it's worth, I think my preferred options would be 2. + extends Concept essentially equivalent to a built-in system model as follows:
namespace org.accordproject.base

abstract asset Asset extends Concept {  }
abstract participant Participant extends Concept {   }
abstract transaction Transaction extends Transaction {   }
abstract event Event extends Event {   }

Just to make sure we are clear option 2. will break every Accord Project templates! There is certainly a very big benefit in 1. for backward compatibility in other Accord Project projects.

Thanks for the useful analysis @jeromesimeon. I suggest we take this up at the next WG meeting.

Update on this. The release-1.0 branch includes full removal of application setting systems model. This uses a base system model internally which is as follows:
namespace system
abstract asset Asset {  }
abstract transaction Transaction { }
abstract event Event { }
abstract participant Participant {  }

Some remaining open issues are listed in the corresponding PR: https://github.com/accordproject/concerto/tree/release-1.0
Those relate to the question of whether we should have a root class for the type hierarchy (https://github.com/accordproject/concerto/issues/180) and revisions we should discuss for identifiers and relationships (https://github.com/accordproject/concerto/issues/181)

@dselman Can we consider this closed based on the latest review and support in release-1.0. Remaining related questions (notably on relationships) have their own issues now.


Add CI configuration

i think here are some fixes are missing @tuxmea is doing at the puppetserver atm with https://github.com/voxpupuli/puppetserver-docker/pull/2
or at least something similar.


Create board from a template

Actual state
If a template directory is configured, all template files present in it are automatically loaded and boards created upon user creation (actually, at first login).
When a user creates a new board, a hard-coded template is used: three lists Todo, Doing, Done.
New expected behavior
Don't automatically create boards from templates upon first login. Instead, let the user create boards from the availble templates, as detailed below. No more hard-coded templates.
On their homepage, users can create new boards. In order to do so, a drop-down menu of all available templates let them chose the template they wish to start from. No more text input, the title of the board can be modified later, on the board itself.
In the dropdown menu, templates are identified by their titles, not their filenames.
When the user clicks the Create button near the dropdown, a new board is created from the selected template, and the user is redirected to the new board.
Provide some default templates into the distribution (at least, the Todo, Doing, Done template). The shipped templates are always prepended to the list of templates found in the configured template directory.
Technical notes
Use a kind of delegation pattern: each level (board, column, card) is responsible for configuring itself for its own JSON fragment.

Board configures itself and create columns, passing to each column its JSON fragment ;
Column configures itself and create cards, passing to each column its JSON fragment ;
…

See #97


Validation Exception While Running actions-sync from K8s CronJob

Running the actions-sync tool from a CronJob in K8s always results in an exception, while running the exact same command on local machine (Macbook Pro) always succeeds.
Command I tried:
/bin/actions-sync sync --cache-dir=/tmp/ --destination-token <token> --source-url=https://github.com --destination-url=https://github-<enterprise-url> --repo-name-list "Azure/setup-kubectl:actions/setup-kubectl"

Results (inside CronJob):
fetching * refs for Azure/setup-kubectl ...
syncing `actions/setup-kubectl`
error creating github repository `actions/setup-kubectl`: error creating organization actions POST https://github-<enterprise-url>/api/v3/admin/organizations: 422 Validation Failed [{Resource:Organization Field:login Code:custom Message:login is not available} {Resource:Organization Field:admins Code:missing_field Message:}]

On Macbook machine with exact same command:
fetching * refs for Azure/setup-kubectl ...
syncing `actions/setup-kubectl`
successfully synced `actions/setup-kubectl`

One thing I noticed is that it succeeds on the CronJob only when the organisation and repo are not existing before, so it creates them and works, but if org/repo are existing it fails.
Used version: https://github.com/actions/actions-sync/releases/tag/v202009231612
Any idea what causes this discrepancy?

@wazery Which release bundle are you using.
We have a known issue with 32 bundles.
https://github.com/actions/actions-sync/issues/18

That was this version https://github.com/actions/actions-sync/releases/tag/v202009231612
@ajaykn thanks for checking 🙇

@wazery  Closing this, since this issue is from 32 bundles (gh_202009231612_linux_386.tar.gz, gh_202009231612_windows_386.tar.gz)
And as a workaround, we have to use 64 bundles.
Please feel free to reopen if u see any concerns.


Typing a dot in a string literal selects first item in auto-suggest

VSCode Version: 1.6.0-insider
OS Version: Windows 7

Steps to Reproduce:
interface Options {
    webpackConfig: string;
}
let opts: Options = {
    webpackConfig: "./webpack.config.js"
    //                       ^ typing dot here autocompletes with "webpackConfig"
}

Expected behavior:
Auto-suggest should trigger when pressing ENTER (or maybe TAB?), but not on dot
Actual behavior:
When typing a dot (.), auto-suggest selects the first item
Workaround:
Pressing escape before the dot. Very annoying!!

@use-strict I used your code and couldn't get any suggestions on ..
I get suggestion for "webpackConfig" when typing "webpack" but not after the .
Can you share your user settings so that we can try to repro this?

I'm not sure if we're on the same page: first you type webpack and get a suggestion dropdown with webpackConfig. Then you type the dot and it should select the suggestion, preventing you from typing webpack. and replacing it with webpackConfig.

first you type webpack and get a suggestion dropdown with webpackConfig

Yes, I got the suggestion dropdown.

Then you type the dot and it should select the suggestion

No, it did not select the suggestion when I typed the dot
Therefore, I am asking for your user settings to see if something there is causing this for you

User settings:
{
    "window.zoomLevel": 2,
    "typescript.tsdk": "D:\\Settings\\npm\\node_modules\\typescript\\lib",
    "diffEditor.renderSideBySide": false,
    "editor.rulers": [80, 120],
    "editor.referenceInfos": false,
    "editor.folding": false,
    "terminal.external.windowsExec": "C:\\Program Files\\Git\\git-bash.exe"
}

Workspace settings:
{
    "files.exclude": {
        "**/.git": true,
        "**/.svn": true,
        "**/.hg": true,
        "**/.DS_Store": true,
        "**/node_modules": true
    },
    "typescript.tsdk": "./node_modules/typescript/lib"
}

I'm using tsc 2.0.2 in this project, if anything that would be the primary suspect.

And the real-life project where I encountered the issue: https://github.com/use-strict/webpack-runner/blob/master/test/test.ts#L78

@use-strict What does your keybindings read like (both sides)? Note that usually . is bound to accept a selection if auto-acceptable. TypeScript suggestions are, textual aren't. Also what in what language do you see this?

My keybindings are:
[
    {
        "key": "ctrl+d",
        "command": "editor.action.deleteLines",
        "when": "editorTextFocus"
    },
    {
        "key": "ctrl+e",
        "command": "workbench.action.quickOpen"
    }
]

The link in my previous comment shows the exact line where I get this behavior.

Note that usually . is bound to accept a selection if auto-acceptable

I'm not sure what you mean by this. Even if it weren't a string literal, accepting a suggestion by typing dot would be problematic (unless maybe there is a single suggestion). Still it would be a weird behavior.

In the default keybinding configuration you will find this
{ "key": ".",                     "command": "^acceptSelectedSuggestion",
                                     "when": "editorTextFocus && suggestWidgetVisible && suggestionSupportsAcceptOnKey && editorLangId == 'typescript'" },

which means insert selected suggestion and insert . when . is pressed, the file is TypeScript and when the suggestion is auto-acceptable. This is useful for instance when completing and triggering member completion at once (type cons, press . such that it completes to console. and triggers suggest against, hit tab such that it completes to console.log). Other language may have other accept characters defined, users can un-configure this.
Still, from your steps it's not clear what you are actually doing. To get a better understand of your words:

Auto-suggest should trigger when pressing ENTER (or maybe TAB?), but not on dot

I assume you mean by 'trigger' insert from an active suggest window and not show suggestions, correct?
I have tried both variation and I cannot reproduce. Please update the your step and attach an animated gif.

I assume you mean by 'trigger' insert from an active suggest window and not show suggestions, correct?

Correct
While trying to reproduce this again with a minimal project, I noticed that it does not reproduce with ts 2.0.0 and no workspace settings. It only reproduces with 2.0.2, installed in the project folder with npm install and "typescript.tsdk": "./node_modules/typescript/lib"
Judging from your gif, I realize I've also mislead you without realizing. This gif will hopefully clarify that:

ok - it seems like TypeScript is giving you completions in string literals. Unsure if this is a bug or not - @dbaeumer or @waderyan will know. For the auto-acceptability we have issue #7326 and I'll close this issue as duplicate

@waderyan can you look into this please.

I looked into this and was able to reproduce. This seems to be a bug.

This issue was moved to Microsoft/TypeScript#11044


Splitting doesn't cange the search url, which appears before the file list - does it matter?

Asked by @lshendrik
@lshendrik  Can you please explain a bit more. Not exactly sure what you mean.

Checked it, doesn't matter


Replace custom Deque implementation with swift-collection Deque

Motivation
For AsyncStream we created a custom internal Deque implementation to use for the buffering. This implementation was relatively bare-bones compared to the one in swift-collections. Furthermore, it lacked some methods that we need to implement the new AsyncStream APIs that support producer backpressure.
Modification
This PR copies over the Deque implementation of swift-collections and makes it internal and non-inlinable.
Result
We now have a fully functional Deque.

@swift-ci please smoke test

LGTM about getting the swift-collection's deque.
This keeps it internal so that's fine, but maybe prepare a pitch as well to make it part of stdlib? It is used all over the place being copied around 🤔 WDYT @lorentey ?
If we intend to keep this copy around like this though, would it be good to automate copying the files a bit so it is easier to keep the two impls in sync? Or working towards removing the one in swift-collections in favor or a public pitched one from the stdlib.

Yeah it would be great if we could get Deque officially moved into the stdlib. It is the one type that I consistently reach for in every AsyncSequence implementation.

Up to @lorentey to give the thumbs up or down here, but yeah this type would definitely be useful in concurrency library even for our internals

Yes, this is clearly an interim solution. Manually vendoring these implementations into every single ABI stable OS component is not going to be a good approach in the long term.
The big blocker for adding Deque as a public stdlib type is that there is a really good chance that Swift's incoming ownership features will render the implementation obsolete in a few months/years -- so we'd probably regret if we baked its current layout into the ABI right now.

@swift-ci please smoke test

@swift-ci please test

@swift-ci please smoke test

@swift-ci please test

@swift-ci please test macos

@swift-ci please smoke test

@swift-ci please test

@swift-ci please smoke test

@swift-ci please test macos


Python lib connectors

anyone can help me how to use elasticsearch connector in python code?

@bloodyjuice I think there are no python connectors now, all connectors are based on Java Function. But Pulsar has python functions, I think it's easy to implement a connector based on the python functions. If you are interested in implement it, you are welcome to push a PR.

Closed as stale and inactive.


Discrete locally euclidean wrapup

Finalizing contribuiton at #507

That choice of "locally $n$-Euclidean" is most sensible, I think.  Will review in more detail later today.

I added clarifications.  Please see how it looks.

Looks good. (I think you've implicitly approved this modulo your changes, so merging.)


🐛 FlyoutMenu doesn't work well

Describe the bug
The FlyoutMenu and Flyout context menu doesn't work as expected in the demo application.

I though that maybe come from #768 but that's not the case !

https://github.com/bdlukaa/fluent_ui/blob/master/lib/src/controls/flyouts/menu.dart#L449-L453

This is due to the multiple navigators


Better error handling

Changes:

[x] better error handling in case of missing labels

Bugs:
Part of https://github.com/PeterNgTr/codeceptjs-testrail/issues/40

The above change would result in an output similar to this one:
15:23:26  tests-runner | Cannot add result for case due to Field :results contains one or more invalid results (case C362669 unknown or not part of the test run)

@PeterNgTr Any chance you can have a look over this one? It would clearly help others in terms of more verbose logs on error cases 🙃

That's great! Thanks for deep dive in it.

So @vladbalanescu @Georgegriff I've published new version with your improvement. Thanks.

absolutely brilliant, thanks a lot!


[PATTERN] Add Alphabet pattern in C, Pattern42

Description
Task:
for the given input n the following pattern should print (n=6 for the below pattern):
Pattern Image

Filename
Pattern42.c

Hi there, I would like to contribute to this task. Could you please assign it to me?

ok, @GiuliaBortone you can work on this issue.


cancelling ollama progress is "hard"

im not sure what is going on but when ollama is fetching model it updates very rapidly and cancelling with Ctrl-c takes a while / requires multiple attempts.
I think its because the progress is updated/refreshed so often that it result in delayed queuing of the ctrl-c over the many % updates.
Reason I think that is because when my ctrl-c is actually registered I get this output:
Ollama model pull: 2024-11-04 15:11:28,735 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.01%
Tests paused
Press [e] to edit command line args (currently ''), [r] to resume testing, [o] Toggle test output, [:] for the terminal, [h] for more options>
2024-11-04 15:11:28,796 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.05%

2024-11-04 15:11:28,858 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.09%

2024-11-04 15:11:28,916 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.13%

2024-11-04 15:11:28,975 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.16%

2024-11-04 15:11:29,035 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.21%

2024-11-04 15:11:29,095 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.24%

2024-11-04 15:11:29,155 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.28%

2024-11-04 15:11:29,215 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.33%

2024-11-04 15:11:29,275 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.38%

2024-11-04 15:11:29,335 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.42%

2024-11-04 15:11:29,395 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.46%

2024-11-04 15:11:29,455 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.50%

2024-11-04 15:11:29,518 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.55%

2024-11-04 15:11:29,575 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.60%

2024-11-04 15:11:29,635 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.64%

2024-11-04 15:11:29,699 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.69%

2024-11-04 15:11:29,764 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.73%

2024-11-04 15:11:29,815 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.78%

2024-11-04 15:11:29,876 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.83%

2024-11-04 15:11:29,935 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.85%

2024-11-04 15:11:29,995 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.87%

2024-11-04 15:11:30,056 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.89%

2024-11-04 15:11:30,117 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.91%

2024-11-04 15:11:30,175 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 27.95%

2024-11-04 15:11:30,235 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 28.00%

2024-11-04 15:11:30,300 INFO  [io.qua.lan.dep.dev.DevServicesOllamaProcessor] (HttpClient-2-Worker-2) Progress: 28.05%

Suggestion is to not care about updating the progress if its not changing with at least 0.2% or even 1%.

We can probably only update every 500ms or so


[Doc enhancement][test mstp harsh]

Please review.

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/test-mstp-harsh.md
93
1
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 749b25d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/messaging-extensions/test-mstp-harsh.md
:warning:Warning
View
Details

msteams-platform/messaging-extensions/test-mstp-harsh.md

Line 12, Column 1: [Warning-multiple-h1s] Multiple H1s(H1 'Who is your audience?') are not allowed. You can only have one top-level heading.
Line 16, Column 1: [Warning-multiple-h1s] Multiple H1s(H1 'What do they want to accomplish?') are not allowed. You can only have one top-level heading.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/test-mstp-harsh.md
93
1
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit dd3e17e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/messaging-extensions/test-mstp-harsh.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Added Header and Footer

@AnkitMajee can i see the screensort o the header and footer her in comment sectn

Yes Sure


Folder paths do not work for get_variable/set_variable

Problem:
Folder paths are not working for the get/set variable functions.
With the following test variable layout, with each variable value set to the folder it is in:

The tests show that the folder paths are being completely ignored:

Also, as the same case with my addendum on #739 testing found with the timelines, it's also simply going by the order in the JSON, which means in this example screenshot it's actually pulling the value of Test2/TestValue, as that one was created first, rather than taking the root TestValue since that was created after it, which also can lead to unexpected behavior.

From additional testing, I have found two additional problems:
First, it also doesn't get them parsing the text in a text node, either:

Second, something weird is happening when calling set_variable to the way Dialogic does logic calls.
Take the following test timeline:

While the If statement doesn't reflect the subfolders (which is not ideal, but fine since in the background it uses the event_id instead), each one of these If statements have been set separately: /TestValue has been set to 1, /Test2/TestValue has been set to 2, /Test3/sub/TestValue has been set to 3.  when running normally, since all three of them are set, they all execute correctly and show the text event assoicated to them. However, calling Dialogic.set_variable('TestValue', "updated value") in code before creating the dialogue isn't just updating the first found TestValue, as we were expecting from the previous test, but it seems to be updating all of them because then none of them are running.
And if we print Dialogic.export() from before and after doing anything, we can indeed see that all three of them were changed, which isn't the behavior we want either:

Thoughts/Info

Not necessarily a bug, more a design problem/decision
Yes text events do not consider the path of the definitions only the name and will pick the first one with a matching name (same with set and get functions)
thus definitions where never supposed to have duplicate names. It would be good to change this probably, but that's a 2.0 thing imo.
The definition picker does show the folders in 1.4 WIP

on the third point, I would say it absolutely is something that should be changed going forward, because it is important for larger projects. for example, an RPG where you want to keep interaction flags for say an NPC remembering that they talked to you once already, you would need variables for every one of them
on the last point, what i mean is looking at it from the timeline. you can use the picker for the folders just fine in 1.3 too, but neither 1.3 nor 1.4 WIP actually show the label as being in subfolders after that

but since the actaul timeline file is testing them by their IDs,<PHONE_NUMBER>-787,<PHONE_NUMBER>-648, and<PHONE_NUMBER>-146 in this case, the visual appearance isn't so important otehr than maybe looking back later to try and cross reference it

so im trying to step through the code myself to try and figure it out to submit a PR. get_variable(), set_variable(), and the DialogNode all are calling the internal function _get_definitions(), which is just a dictionary of all the variables with four values: id, name, type, and value. it would seem to me that just adding one more item in that dictionary, the path from the folders definitions, would allow being able to extend the function to be able to both preserve the way it works currently, as well as be able to check it by a full path, i.e. makng get_variable('Variable') work as it does now, and get_variable('/Variable') return one from the root of the folder structure.
since I dont think there's a function to create new definitions at runtime, just set them, all it would need would be going through and adding the path in once at runtime, that's where I'm at right now.

This would have to be recalculated everytime you run the game, right? But yeah. That's certainly one way to achieve this.

This was fixed with PR #748


Introduce anonymous checkout

Allow creating anonymous checkouts - without email
Add email validation to:

addPromoCode
checkoutComplete
checkoutPaymentCreate

Wipes old checkouts

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

@fowczarek Yes, it works correctly.
@eCzerniak, the first problem is a global issue. The problem with the avalara doesn't exist anymore.


WIP - sshd_config and smart_auth test cases

Fixed test_sshd_config test case.

FTR can be rebased on top of #238


Document cloning in the standalone-cinder provisioner

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

/ok-to-test

You are proposing a PR with 3 commits that makes a bit of a mess in the git tree.
Could you make your branch aglitke/doc-cloning just a cherry-pick of commit 67e2bfdad8a06e5ac1e51c2931ce6a4ab3df75fb on top of the current origin/master ?
If you push force this change to your local branch the PR will look much better.
The way you submit the PR at the moment you push into the git tree also commits like 6da022780d17d2c1759cdcaada786290777035c6 but the actual changes from this commit seem to be already merged in master long ago.
thank you

/approve

Alright, cleaned up the history @zioproto .

/lgtm

/approve
/lgtm
Why is the gate failing on this patch?

/approve

/lgtm

/test cloud-provider-openstack-acceptance-test-flexvolume-cinder
/test cloud-provider-openstack-acceptance-test-keystone-authentication-authorization
/test cloud-provider-openstack-acceptance-test-standalone-cinder
/approve
/lgtm

/approved
/lgtm


protocol: switch to binary MessageCommand

closes #888

Looks good, have you tested it with private net?

Looks good, have you tested it with private net?

Yes, consensus works with both #969 and #970


Fresh install, cannot access https default site

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug

I am attempting to migrate from the HomeAssistant plugin version (it's broken and there's no way to upgrade without losing all info) of this over to this standalone version. After following the basic example in the readme, I can access the config UI, the site at http://<ip>:80, but I cannot access the site at https://<ip>:443.
reenignearcher@docker:~$ curl http://localhost:80
<!DOCTYPE html>

        <meta charset="utf-8">
        <meta http-equiv="X-UA-Compatible" content="IE=edge">
        <meta name="viewport" content="width=device-width, initial-scale=1">
        <title>Default Site</title>
        <link href="https://maxcdn.bootstrapcdn.com/bootstrap/3.4.1/css/bootstra     p.min.css" rel="stylesheet">

Congratulations!

You've successfully started the Nginx Proxy Manager.

If you're seeing this site then you're trying to access a hos     t that isn't set up yet.

Log in to the Admin panel to get started.

Powered by Nginx Proxy Manager

reenignearcher@docker:~$ curl https://localhost:443
curl: (35) OpenSSL/3.0.13: error:0A000458:SSL routines::tlsv1 unrecognized name
reenignearcher@docker:~$

I am using portainer, and Ubuntu server 24.04, with bridge networking. I can't use host networking because the container tries to use port 3000, but that's already in use on the host (probably by portainer itself). Port 443 is not used by the host at all, and the issue persists if I remap the container's 443 to another port, such as 4443.
As far as I understand the 443 port should serve the default site? Since it cannot be accessed I believe this is causing other issues, such as obtaining ssl certs from letsencrypt, and in the off chance the ssl certificate generation does succeed, I get 400 errors when trying to access the host.
There is probably something simple that needs to change on my host, but I have no idea what that could be and have been searching the documentation, discussions, and issues for hours without success.
If it's helpful, here are the full logs.
❯ Dynamic resolvers ...

❯ IPv6 ...

Enabling IPV6 in hosts in: /etc/nginx/conf.d

- /etc/nginx/conf.d/default.conf

- /etc/nginx/conf.d/production.conf

- /etc/nginx/conf.d/include/assets.conf

- /etc/nginx/conf.d/include/ip_ranges.conf

- /etc/nginx/conf.d/include/force-ssl.conf

- /etc/nginx/conf.d/include/letsencrypt-acme-challenge.conf

- /etc/nginx/conf.d/include/proxy.conf

- /etc/nginx/conf.d/include/block-exploits.conf

- /etc/nginx/conf.d/include/log.conf

- /etc/nginx/conf.d/include/ssl-ciphers.conf

- /etc/nginx/conf.d/include/resolvers.conf

Enabling IPV6 in hosts in: /data/nginx

❯ Docker secrets ...

-------------------------------------

 _   _ ____  __  __

| \ | |  _ \|  \/  |

|  \| | |_) | |\/| |

| |\  |  __/| |  | |

|_| \_|_|   |_|  |_|

-------------------------------------

User:  npm PUID:0 ID:0 GROUP:0

Group: npm PGID:0 ID:0

-------------------------------------

❯ Starting nginx ...

 Since it cannot be accessed I believe this is causing other issues, such as obtaining ssl certs from letsencrypt, and in the off chance the ssl certificate generation does succeed, I get 400 errors when trying to access the host.

That's correct. But the "default site" doesn't provide a SSL certificate, that's why curl throws this error. So you'll need to setup a proxy host to HA, and bind a SSL certificate to that hostproxy.

@kerstenremco Thank you for the response.

So you'll need to setup a proxy host to HA

What do you mean by "HA"? If you mean HomeAssistant, I think you might be misunderstanding. I am currently using the nginx-proxy-manager plugin for HomeAssistant, but am trying to migrate away from using the plugin to this standalone version. In the HomeAssistant version, everything works perfectly fine, but the standalone version doesn't work at all.
Currently I cannot even get the SSL certificates to pass validation, which I have to assume is because of this error on port 443.

Yes I understand. I try to explain that it's correct behaviour you can't just make a curl to "localhost" because that indeed would show the default page that doesn't provide a ssl cert.
Anyway, can you share how you setup your proxy host to homeassistent? Which hostname did you use?

I believe I've kind of solved this.

Obtaining the SSL certificates fails randomly. Continually retrying will eventually get a working certificate... but it can fail 9 out of 10 times. Maybe this is a bug in the version of certbot that's included with this container. Hopefully this can be fixed sooner rather than later, as I'm foreseeing this to cause issues with certificate renewals.
I was getting 400 errors in HomeAssistant after getting it setup in the new nginx proxy manager. That led me to believe there was some issue with NPM, but instead I just had to add the new NPM proxy to the trusted hosts list in HA: https://community.home-assistant.io/t/home-assistant-400-bad-request-docker-proxy-solution/322163


implement imgui

imgui was ignored, because the strucutre of the gameloop doesn't allow it.
but it is too tempting. it seems too useful.
use a statemachine, with two states: input and drawing

input frame does not run, if it is not in input state. if it is in input state, mouse events are captured and changed to drawing state afterwards.

output frame does not run, if it is not in drawing state. if it is in drawing state, the vertex buffer is drawn and changed to input state after wards.

this way, the global singleton strucutre still works. this may not be performant, but we can strip it easily in relase builds

close https://github.com/Rismosch/ris_engine/issues/43 when imgui works

statemachine wasn't necessary, because backend doesn't depend on sdl2 now. instead it depends on my input system.


ec2_securitygroup vpc parameter invalid

Hy,
I am trying to add a vpc parameter to ec2_securitygroup and get following error:
Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Invalid parameter vpc on Ec2_securitygroup[test-sg] at /etc/puppet/modules/profile/manifests/puppetmaster_tasks.pp:23 on node puppet.3amlabs.net
The declarations is following:
ec2_securitygroup { 'test-sg':
ensure      => present,
region      => 'us-east-1',
vpc         => 'x-test',
description => 'Security group for VPC',
ingress     => [{
security_group => 'test-sg',
},{
protocol => 'tcp',
port     => 22,
cidr     => '<IP_ADDRESS>/0'
}]
}
It's a basic one taken from https://github.com/puppetlabs/puppetlabs-aws/blob/master/examples/vpc-example/init.pp#L10
Has VPC option been removed?
The vpc is a valid one:
puppet resource ec2_vpc | grep test
ec2_vpc { 'x-test':
thanks
Krisztian

Hi @s4mur4i. Could I check how you installed the module? The version from the Forge (installed with puppet module tool) currently doesn't have the VPC security group support, although it's available in master from GitHub. We'll be releasing a new version soon so I'll update here when we do so. If that's not the problem let me know and I'll help debug.

Actually I was testing this directly from github master and was getting this:
Notice: /Stage[main]/Main/Ec2_securitygroup[sample-sg]: Dependency Ec2_vpc[sample-vpc] has failures: true
Warning: /Stage[main]/Main/Ec2_securitygroup[sample-sg]: Skipping because of failed dependencies
Error: Could not find a suitable provider for ec2_vpc
Error: Could not find a suitable provider for ec2_vpc_internet_gateway
Error: Could not find a suitable provider for ec2_vpc_routetable
Error: Could not find a suitable provider for ec2_vpc_subnet
Notice: Finished catalog run in 13.88 seconds
This is directly from the examples with the exception that I'm doing us-west-1

Hy @garethr .
Okay got it working from cloneing the github version.
On a side note: the runtime for one sg creation is a little high, but I will investigate it further:
Notice: Finished catalog run in 108.97 seconds
Thanks for the info.

@s4mur4i thanks. I'll close this issue. We'll be releasing a new version to the Forge soon.
Regarding performance, particular for security groups, it's a factor of the number of security groups you have in all regions at the moment.
We're currently testing the following PR https://github.com/puppetlabs/puppetlabs-aws/pull/102 which I've seen reduce this by 80% in similar runs. And we'll be looking to more performance improvements at a later date.
You can also cut this down by if you're only managing resources in a single region by using the AWS_REGION environment variable.

@daveseff I'm guessing you might have an older version of the module installed on the module path. You can override that with the --modulepath like so:
puppet apply tests/create.pp --modulepath ../ --test

If that's not that case please do open an new issue with the steps to try and recreate it. Thanks.

@garethr I'm using Puppet Enterprise and there's only one module installed.
Using This still fails with same errors:
puppet apply vpc_test.pp --modulepath /etc/puppetlabs/puppet/environments/production/modules --test

@garethr This was my fault. I didn't have the retries gem installed. this must have been a new requirement?


general: Remove redundant usage of cat

In general, the substitution ‘$(cat foo)’ may be replaced by the
equivalent but faster ‘$(<foo)’.

@belak any view?

I held off on this one before because I wanted to test it myself - it might be a little while until I can do that

This is an interesting PR, to be sure - I didn't know this was possible. It mostly makes sense, but it's not the direction my brain would go for these.
I like the change in modules/python/init.zsh - that's a fairly common convention when reading from files which I've seen in multiple places before.
I have no strong opinions on the heredocs - they work, as far as I can tell, they're solid, and will probably perform marginally better. However, if it was purely my decision, I probably wouldn't push for that replacement - it's a bit less readable and doesn't follow with common conventions, especially as seen in other shells. That hasn't stopped us before, at least when there's a zsh convenience, but for me in this specific case it wouldn't be worth it.
I'm a fan of the replacement in modules/python/init.zsh, and if you feel strongly about the heredoc change or think it adds enough value, you're definitely welcome to merge it as-is.

Yes, readability does take a hit, I admit. But Zsh "hieroglyphs" (will all esoteric single-character flags and switches) probably don't focus on readability anyway I guess :)
One probable reason we don't see this convention around is probably because of POSIX compliance. Besides, pure Zsh optimized interactive shell is a niche space.
I'll hold off merging this just yet. Let's see if this PR can evolve a bit.


Disable ssh-agent if it is disabled in user's SSH configuration

Can you give me some details on the motivation for this change? Do you have these options set in your own config? I read the SSH_CONFIG docs you pasted, but I'm not quite sure if the idea is that the user setting these options means they don't want to use ssh-agent or if when these options are set, the ssh-agent won't work anyway so we might as well not try. If it's the former, I'm not sure that actually matters? We're not starting a long-running ssh-agent; once the bb task finishes, the agent gets killed. There is already the :biff.tasks/skip-ssh-agent option if someone really doesn't want the agent to be used at all for some reason (mainly I put it there in case there were bugs, so there'd be a workaround instead of having bb tasks totally broken).

On further thought I've decided this is not a good change to make so i'm closing this pr.
tl;dr:

running bb tasks was printing an ugly ssh-agent warning
i had ssh-agent disabled in my ssh config (probably the cause of the warning)
setting :biff.tasks/skip-ssh-agent fixed it, but it felt wrong having a personal setting in the :dev config
might as well infer ssh settings from user's ssh config
on further thought, :biff.tasks/skip-ssh-agent is fine for a solo project and if there's several devs I could add a :personal or : config map to config.edn.


[Feature Request] - Manage subtask

just started getting into RTM. Love that there's a cli tool. Is there any plans on adding support for subtasks?

so thats exactly what I was thinking as well.
01 (1) Main Task Description [4] <- brackets would describe the number of tasks

then we can use a subcommand like
sls [task] #list subtasks
sadd [task] #add subtasks
s... etc

Hi @dwaring87, I have been playing with the RTM API this weekend and it seems that the filter isSubstack:tru does not work.  Returns a {rev: 'hasCode'} object. Any news about this?
I have opened a ticket for support. II think t would be an easy way to get subtasks.
Congratulations on the project, very open and dynamic 😄

I was poking around with the rtm-api and rtm-cli today and it does not look like the RTM API returns a task's subtasks.
I dumped the rtm-api user.get('rtm.tasks.getList') response to the console and got the following for a task that has 4 sub-tasks. This may be a blocker to this very useful feature as RTM isn't returning that data. 😢
{
    "id": "6xxxxx6",
    "taskseries": [
        {
            "id": "42xxxxx33",
            "created": "2020-07-23T02:23:24Z",
            "modified": "2022-08-28T07:00:31Z",
            "name": "Take care of the pets",
            "source": "android",
            "url": "",
            "location_id": "",
            "rrule": {
                "every": "1",
                "$t": "FREQ=DAILY;INTERVAL=2;WKST=SU"
            },
            "tags": {
                "tag": [
                    "pets"
                ]
            },
            "participants": [],
            "notes": [],
            "task": [
                {
                    "id": "8xxxxxx86",
                    "due": "2022-08-28T07:00:00Z",
                    "has_due_time": "0",
                    "added": "2022-08-26T07:00:25Z",
                    "completed": "",
                    "deleted": "",
                    "priority": "1",
                    "postponed": "0",
                    "estimate": ""
                }
            ]
        }
    ]
}

Unfortunately, until the RTM API adds subtasks into the API response, there's not much that can be done...


UI does not get updated when upgrading the local RKE2 cluster w/ AWS external tree provider config

Setup

Rancher version: v2.7.11-798b3da89d6c71f8a30994c0bd7303a755ae55a8-head

Browser type & version: Chrome 120.0.6099.199 (Official Build) (arm64)

Describe the bug
Rancher UI is not updating when upgrading local RKE2 clusters to Kubernetes version 1.27+ (version 1.27.8 precisely).
To Reproduce

Install Rancher on a 3-node v.1.24.17+rke2r1 cluster (Amazon EC2 nodes) via helm
From the UI, go to Cluster Management > Select the local cluster from the 3-dot button, select Edit Config
Upgrade the local cluster from version 1.24.17 to 1.27.8 and observe the UI behavior.

Result
The UI does NOT display any warning messages and does not update when the Kubernetes version is upgraded to 1.27+
Expected Result
It is expected that the UI displays a warning message instructing the user to configure a cloud-controller manager for the out-of-tree architecture, as the in-tree architecture is no longer supported for Kubernetes versions higher than 1.27, with a link to the docs that explain how to configure the cloud-controller manager.
Docs: https://ranchermanager.docs.rancher.com/how-to-guides/new-user-guides/kubernetes-clusters-in-rancher-setup/migrate-to-an-out-of-tree-cloud-provider/migrate-to-out-of-tree-amazon

Found while testing https://github.com/rancher/dashboard/issues/9204

@mantis-toboggan-md let me know if we have to backport this to 2.7.patch1 and/or 2.8.next1

Possible duplicate to https://github.com/rancher/dashboard/issues/10290 though rereading

/backport v2.7.patch1

/backport v2.8.next1

For the record, we've deployed a Rancher instance to AWS on RKE2 with the in-tree provider configured, targeting v1.24.17+rke2r1. I would expect to see something to indicate that the in-tree cloud provider has been configured in /v3/clusters/local, but nothing is present here. We will need to sync with somebody on the backend team to see if it's possible to retrieve the data we need to satisfy this issue. I'm moving this to backend blocked for the time being, until we've had this opportunity to discuss further.
Alternatively, we can consider displaying a generic message based solely on the selected kubernetes version. This might not be the ideal solution, but it could allow us to communicate that there could be issues associated with upgrading to 1.27+ if an in-tree cloud provider had been configured in the past.
@IsaSih @izaac @gaktive

Following up on this issue with conversations that we had out of bound.
We have confirmed that the local cluster will have no way to display the information that we need to satisfy this request. Since the local cluster is an imported cluster, Rancher will not be able to retrieve details that will allow us to determine if it is using an in-tree cloud provider.
We will move forward with the alternative approach described in my previous comment. The goal is to check the current version and compare with the target version. If the current version is less than 1.27 and the target version is greater than or equal to 1.27, we will display a informational message to help users determine if they need to take action. If the current version is 1.27 or above, this message will not display.


ITVR - DL# correction: database changes

Describe the task
A clear and concise description of what the task is.
Purpose
The reason why this task is needed and/or what value it adds.
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

[https://app.zenhub.com/files/223241604/7db3e7f3-eb62-4ac0-bfa0-e92e07cfa3bd/download]


Update get_started.md

replace deprecated function

Can one of the admins verify this patch?

I signed it!

initialize_all_variables() is the deprecated API, the new one is global_variables_initializer().  Thanks for the PR though!


New PyCharm version soon to be released on pi-apps

You may be interested to know that Pycharm on Pi-Apps will be updated soon.
Below is the beta app, for testing: https://github.com/Botspot/pi-apps/issues/926#issuecomment-903023438

Great job keeping it going will certainly help a lot of people enjoying Pycharm.  I have been keeping it updated on https://github.com/gam3t3chelectronicshobbyhouse/PyCharmInstaller   which I will continue to maintain so if you need to pull from there to help out feel free to update through it.  I update and double check all updates every time its updated with new and upgrade installs.

Great job keeping it going will certainly help a lot of people enjoying Pycharm.

As there were no bug reports for the beta testing zip I posted, I have gone ahead and pushed the update to all Pi-Apps installations.
This appears to have made at least one github user concerned over there as they say the 2018 version of Pycharm is the last "good" version for 32-bit. Would you mind sharing your thoughts on the subject there?
Link: https://github.com/Botspot/pi-apps/issues/924#issuecomment-903323526

Why not update to 2021.2.1 version?
I tested by changing the downloadlink variable in the current Pycharm CE install script to https://download-cf.jetbrains.com/python/pycharm-community-2021.2.1.tar.gz and removing sudo mkdir /opt/$name/lib/pty4j-native/linux/arm || error "Failed to make directory /opt/$name/lib/pty4j-native/linux/arm" from the script as I met this error when installing: Failed to make directory /opt/pycharm-community/lib/pty4j-native/linux/arm.
Pycharm installed successfully and runs well for a week.

I did update the 2021.2.1 installer script but no errors like you were having so if continues will investigate further.    So if anyone has this issue please post about it.

Pi-Apps has been updated to use Pycharm version 2021.2.1.


Fix: Update inference config map with torchserve 0.3

Update inferenceservice config map with torchserve 0.3 version
Update qucik_install.sh with v0.5.0-rc1 version

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

/hold

/unhold


Error 500 A composable that requires access to the nuxt instance....

Hello,
Since recently none working due error message;
"[nuxt] [request error] [unhandled] [500] [nuxt] A composable that requires access to the Nuxt instance was called outside of a plugin, Nuxt hook, Nuxt middleware, or Vue setup function." Any idea? I did this on a fresh installation.
Regards,
Kohlzz

Hi, I am unable to reproduce this. Can you show me the output of this command npx nuxi info?

This is the output of that command;

Operating System: Windows_NT
Node Version:     v18.17.1
Nuxt Version:     3.8.2
CLI Version:      3.10.0
Nitro Version:    2.8.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, runtimeConfig, imports, modules, app
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Have you removed this from nuxt.config.ts file?
{
  experimental: {
    asyncContext: true,
  }
}

That was indeed the case, I removed it from the config somehow. Silly i didn't notice it but thank you, this worked :)

Glad it worked out for you.

Hey, sorry i was too fast. It seems to work ony under ssr:false
Verzonden vanuit Mail voor Windows
Van: Amr Noman
Verzonden: zaterdag 25 november 2023 18:52
Aan: amrnn90/breeze-nuxt
CC: kohlzz; Author
Onderwerp: Re: [amrnn90/breeze-nuxt] Error 500 A composable that requires accessto the nuxt instance.... (Issue #20)
Glad it worked out for you.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


In demo,photo width are different at edit and render HTML

This is handled by css. You can override the style with your custom css.