[regression] Ingress does no longer work

Describe the bug
kubecf-credhub-0 and kubecf-uaa-0 are in CrashLoopBackoff because container cf-cli-6-linux-*-internal-security-group-setup crashes when using ingress (with globally valid certificate).
RCA:

setup_internal_security_group.sh failes.
credhub.yaml and uaa,yaml set the CF_API_CA_CERT env to the router CA.
https://github.com/cloudfoundry-incubator/kubecf/blob/c60006d131c0339b947733c95f52f5fbc5496a11/deploy/helm/kubecf/assets/operations/instance_groups/credhub.yaml#L57-L65
https://github.com/cloudfoundry-incubator/kubecf/blob/c60006d131c0339b947733c95f52f5fbc5496a11/deploy/helm/kubecf/assets/operations/instance_groups/uaa.yaml#L57-L65

To Reproduce
Deploy kubecf with ingress enabled and valid certificate configured.
Expected behavior
kubecf should boot normally.
Environment
master @ https://github.com/cloudfoundry-incubator/kubecf/commit/c60006d131c0339b947733c95f52f5fbc5496a11
Workaround
I created a workaround in our fork. https://github.com/akhinos/kubecf/commit/c40ba44d9cef94782336701929961a886be34c97
The CF_API_CA_CERT env won't be set, if  ingress is enabled (.Values.features.ingress.enabled=true). I'm not sure if this is correct in all cases.
Should I open a PR for this or is there a better solution?

https://github.com/cloudfoundry-incubator/kubecf/issues/480 should also fix this.

#480 should also fix this.

I think so

@phil9909 I'm working on this.

Can you share your values.yaml file passed to helm to configure kubecf please ?
Do you also configure the externalIP when you use ingress ?
@phil9909


json-ice crashes if field not annotated

Looks like go-json-ice requires annotation of each field.  This is inconvenient, as other serializers have a sensible default.
To reproduce, remove the annotation from one field in the example in README, then do
go generate
This fails with
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x11f4576]

goroutine 1 [running]:
main.main()
	/Users/dkegel/go/pkg/mod/github.com/moznion/go-json-ice@v0.0.2/cmd/json-ice/main.go:182 +0xc76
main.go:6: running "json-ice": exit status 2

At the very least, it should not crash, and should print an error saying that annotation is mandatory.
Better would be to behave as other serializers do, and default to the same name as the field for the json key.

Hi, @dkegel-fastly!
Thank you for your reporting. This unexpected behavior has been fixed by #25 and I've released the patched version v0.0.3.


Comments are the wrong format

The comments for the root CSS use the wrong styling, this fixes it. I barely know any CSS so this messed me up when copying from the README

Ooh i never noticed that - good spot!
I'm obviously too used to SCSS lol. Thanks!


vc: Detach device when unable to store sandbox device

In Container#mountSharedDirMounts, if sandbox.storeSandboxDevices() returns error, we should detach the device.
Fixes #2301
Signed-off-by: Ted Yu<EMAIL_ADDRESS>

@jodh-intel @grahamwhaley
Mind triggering ubuntu test run ?
thanks

@amshinde
Mind triggering ubuntu test run ?
Thanks

@jodh-intel @grahamwhaley
Mind triggering ubuntu test run ?
thanks

/test-ubuntu

@jodh-intel @grahamwhaley @amshinde
All tests passed

@grahamwhaley @amshinde
Please take a look.

@egernst @amshinde
Can this be merged ?


Improve compatibility in token media component

Description
Improve compatibility in token media component.

Add compatibility with image that are video
Add compatibility with blob and data image
Add support for base64 images

How to test

Checklist

[x] Base branch of the PR is dev

Changed image: string | null | undefined with image: string as image has to be present all the time
Added getUnlockedContentUrls helper function

I don't see any improvement based on the info in the trello card with the following NFT that has an issue https://nft-test-uaan-liteflow.vercel.app/tokens/1-0x3b3ee1931dc30c1957379fac9aba94d1c48a5405-3

I reverted your fix, testing the extension is definitely not a proper way to do that, as file may not have extensions, also, what happened with other video files?...
The whole system had the entire fallback system that worked pretty well, the only issue we had was that the onError was not properly delegated to the next component and thus not triggering the fallbacks.


Created package and added CAN msg and srv

Description
Issue #6
Created moa_msgs package that contains all of the custom interfaces and have currently created the CAN msg and srv interface.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Builds

[x] Builds

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

The CAN msg being over 11 bits will need to be checked by the CAN hardware interface node bc there is no way to do that in the message unless I want to make a array of 11 1s or 0s so ya stuck with this :)


Can't handle wildcards

Using asterisk sign in cmd cause variety of invalid behaviour:

Sometimes it can't find proper candidates for completion
It inserts whole record after asterisk, instead of only missing part

Using double asterisk in path cause major lagging during typing e.g find ~/project/**/asdf.lua

Of course, wildcards are redundant with this plugin, but they shouldn't cause errors when people use them because of muscle memory.

You don't need wildcard, I think. You just input nvimvvar instead.


Regarding the concept datasets

Hi,
I have some problems with the concept datasets while trying to re-train the model.
The data from MS COCO (https://cocodataset.org) contains images that need to be pre-processed (e.g., cropped) before feeding into the model.
Can you please provide the concept datasets?
Thank you!

Hi,
We just upload the preprocessing code for the COCO dataset. Please check cropping_images_COCO.py.


Question: [Short summary]

[x] I have checked the documentation
[x] I have done a quick search in past issues or Stack Overflow for similar problems
[x] I have brought up a conversation in the Slack User Issues Channel

Question

Why doesn't sdkman include some java versions like Java 15, Java 16 etc?

System info

OS: openSUSE Tumbleweed x86_64 (Linux)
Shell and version: zsh 5.9
SDKMAN 5.16.0

@MetuMortis-code we remove all non-LTS Java versions once the next LTS version is released. In other words, 11 was the previous LTS release, so when 17 was released we ditched all the superseded non-LTS releases from 12 onwards. Hope it makes more sense now.


(GodMode) Could not pass event PlayerMoveEvent to WorldGuard because "user" is null

Hello,
I'm getting an error spammed in the console saying that it "could not pass event PlayerMoveEvent to WorldGuard because "user" is null".  It appears it's happening when plugin is checking for the GodMode flag. I've deleted all GodMode flags from all regions but it didn't help. I don't know how to reproduce this issue nor what's causing it. Here's the error:
https://pastebin.com/5fifWY9L
I'm using the latest Paper 1.19 with the latest available WorldGuard and WorldGuardExtraFlags.
Thank you in advance for your help.

Was fixed in dc6613f


Error loading angular-oauth2-oidc

Hi Manfred,
I just spend over a hour to get things work, but I keep getting this error. As I am out of options, any help would be appreciated.
I took an Angular2 seed and NPM installed angular-oauth2-oidc
At my web.module.ts I did the import and their it will go wrong. Keep getting this error:
Error loading http://localhost:5555/node_modules/angular-oauth2-oidc.js as "angular-oauth2-oidc" from http://localhost:5555/web.module.js

Hi yannyann,
thanks for provider this example.
Wishes,
Manfred

Hi origicom,
thanks for this info. As yannyann mentioned, things are more easy with webpack or the CLI, which uses wepback. The reason is that webpack aligns with the conventions of NodeJS and loads everything automatically after it has been installed into node_modules.
It will also work with SystemJS, but in this case you need to configure the library and all of its 3 dependencies with System.config. I've quickly generated a configuration with jspm (jspm.io) that works:
System.config({
  baseURL: "/",
  defaultJSExtensions: true,
  paths: {
    "github:*": "jspm_packages/github/*",
    "npm:*": "jspm_packages/npm/*"
  },

  map: {
    "angular-oauth2-oidc"<EMAIL_ADDRESS><EMAIL_ADDRESS>{
      "base64-js"<EMAIL_ADDRESS>      "js-base64"<EMAIL_ADDRESS>      "sha256"<EMAIL_ADDRESS>    }
    [...]
  }
});

If you want, you can share a working sample for this here.
Wishes,
Manfred

I started to try this with angular-quick-start and SystemJS and I kept having issues. I manually updated my map with your guide so it looks like,
     // Oauth
      "angular-oauth2-oidc": "npm:angular-oauth2-oidc/dist/index.js",
      "npm:angular-oauth2-oidc": {
        "base64-js": "npm:base64-js/index.js",
        "js-base64": "npm:js-base64/base64.js",
        "sha256": "npm:sha256/lib/sha256.js"
      }

I'm not sure if it's correct to link to the specific files, but that seemed to make progress.
Then each of the modules was trying to load modules like rxjs and couldn't find the right files. I started manually updating the libraries, but I'm going to try and use the CLI option instead since you've said you had success there. :(


docs: update labels and annotations requirement

remove the label section from service account
remove references of azure.workload.identity/use used in service account across docs

Codecov Report

Merging #860 (27a3d36) into main (098e99c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #860   +/-   ##
=======================================
  Coverage   51.76%   51.76%
=======================================
  Files          36       36
  Lines        2376     2376
=======================================
  Hits         1230     1230
  Misses       1100     1100
  Partials       46       46

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


plugins should not enforceArray for data parameter

this breaks the way that data params are handled in the core library

fixed on /dev


_prepare_connection not called on write connections

I was trying to initalize SpatiaLite in a write connection:
>>> from datasette.app import Datasette
>>> ds = Datasette(memory=True, files=[], sqlite_extensions=["spatialite"])
>>> db = ds.add_memory_database('geo')
>>> await db.execute_write("select InitSpatialMetadata(1)")
UUID('3f143baa-4e3d-5842-a36f-4fa2f683b72f')
no such function: InitSpatialMetadata

It looks like the code that loads additional modules only works on read-only connections, not on write connections:
https://github.com/simonw/datasette/blob/92a5280d2e75c39424a75ad6226fc74400ae984f/datasette/database.py#L146-L153
Compared to:
https://github.com/simonw/datasette/blob/92a5280d2e75c39424a75ad6226fc74400ae984f/datasette/database.py#L124-L132

Made this change while working on this issue:

#1567

I'm going to write a test for this that uses that sleep() function.


Explicitly disable precompilation

Add explicit __precompile__(false) per https://groups.google.com/d/msg/julia-users/RLlYPlsT-dU/mhdukrp-AgAJ to disable precompilation.

Yeah, I can reproduce the test failure when precompiled. Not sure if it'll be possible to fix, there are a lot of pointers in this package.
julia> Base.compilecache("LightXML")
"/home/tkelman/.julia/lib/v0.4/LightXML.ji"

julia> Pkg.test("LightXML")
INFO: Computing test dependencies for LightXML...
INFO: No packages to install, update or remove
INFO: Testing LightXML
running parse.jl ...
ERROR: LoadError: LoadError: UndefRefError: access to undefined reference
 in parse_string at /home/tkelman/.julia/v0.4/LightXML/src/document.jl:94
 in include at ./boot.jl:254
 in include_from_node1 at ./loading.jl:263
 in anonymous at no file:6
 in include at ./boot.jl:254
 in include_from_node1 at ./loading.jl:263
 in process_options at ./client.jl:308
 in _start at ./client.jl:411
while loading /home/tkelman/.julia/v0.4/LightXML/test/parse.jl, in expression starting on line 24
while loading /home/tkelman/.julia/v0.4/LightXML/test/runtests.jl, in expression starting on line 3
========================================[ ERROR: LightXML ]=========================================

failed process: Process(`/home/tkelman/Julia/julia/usr/bin/julia --check-bounds=yes --code-coverage=none --color=yes /home/tkelman/.julia/v0.4/LightXML/test/runtests.jl`, ProcessExited(1)) [1]

====================================================================================================
INFO: No packages to install, update or remove
ERROR: LightXML had test errors
 in error at ./error.jl:21
 in test at pkg/entry.jl:746
 in anonymous at pkg/dir.jl:31
 in cd at file.jl:22
 in cd at pkg/dir.jl:31
 in test at pkg.jl:71

Yeah, no worries. I'm not sure I like this idea that precompile follows all dependencies. Too easy to have something break the chain.

That's why it's still an opt-in feature. The risks and failure cases aren't entirely understood yet across the community, will need a learning and experimentation period (aka now).

Well, it's opt-in except when it isn't. That is, I opted in LightGraphs, but I didn't opt-in LightXML. I have to explicitly opt-OUT all dependencies that don't play well with precompilation if I want it in my own package. That seems... backwards.

I don't know why travis is taking so long here.

The JuliaLang org is pretty busy right now, lots of queued builds.
The fact that the opt-in automatically opts in any of your dependencies which haven't themselves explicitly opted out is interesting and does make it even more risky.

I don't think it's too likely that we'll need to do a bisect on package code that's had fewer than 100 commits in total, but whatever.

Note that #32 fixes the underlying problem and re-enables precompilation.


[FR] Autosuggest/Autocomplete like Metadata Menu does

Example: If I have a column named "Pillars", when I go to edit a cell in that column provide a dropdown with preloaded options for that field via Metadata Menu. I believe an API was just added for this?
Here's what the Metadata Menu autosuggest looks like:

Hi @arminta7 , could be nice to implement the suggester like kanban plugin does, sure

https://github.com/RafaelGB/obsidian-db-folder/issues/107
Will be implemented in that issue. Closing this one as duplicated


Update the error message in some access-control tests

The error message for access-control-pod-service-account is a bit confusing, because it makes you think that you are not using a service account, but it may happen that the pods are attached to the default service account.
As described in the docs, the service account must be a valid one. So, updating the message consequently to emphasise that
Same for access-control-pod-role-bindings; error message said namespace but it should say serviceAccountName

Let me add another message that is wrong to this PR

Modiifcation done


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?
Bicep
Module Classification?
Pattern Module
Module Name
avm/ptn/authorization/resource-role-assignment
Module Details
Create an Azure Role Assignment at the scope of a Resource E.g. on a Storage Container or Key Vault
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
peterbud
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

Related issue: https://github.com/Azure/bicep-registry-modules/issues/937

Hi @peterbud,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

Hi @peterbud,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
Before starting development, please ensure ALL the following requirements are met.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - for naming your module
TelemetryIdPrefix - for your module's telemetry

For your module's repository:

Repo name and folder path are defined in RepoURL
Create GitHub teams for module owners and contributors and grant them permissions as outlined here.
Grant permissions for the AVM core team and PG teams on your GitHub repo as described here.

Check if this module exists in the other IaC language. If so, collaborate with the other owner for consistency. 👍
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment and request a review from the AVM core team once your module is ready to be published! Please include a link pointing to your PR, once available. 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

Should we close this issue, given the new pattern module has been merged?

[x] Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.

Thank you for your work @peterbud! I'm closing this issue now.


support/render: move json functions to httpjson package

The keystore has been using the handler and the io functions in the hal
package for json rendering. However, the keystore doesn't respond with
hal content but just pure application/json. Responding application/hal+json in the header
without actually following the convention in the content will confuse keystore clients (and it's probably a bug).
In summary, this PR moves handler.go, handler_test.go, and io.go out of
the hal package to httpjson package and modifies the Render function so that it knows which
Content-Type to write.

@accordeiro that sounds good to me! Will make a change

@accordeiro feedback addressed here.
At this moment I don't see the advantage of turning the parameter of contentType type into a pointer type, so I left it out. Right now if a developer passes any value other than httpjson.HALJSON or 1, we return application/json; charset=utf-8 by default.

I’ve also made a wrapper for Handler and ExecFunc as well. Bear in mind that even though we don’t have to changes those callers of Render with the help of the wrapper in Hal (which I like a lot), most of those callers will be removed by the ongoing refactoring. In other words, those endpoints will just be wrapped with a handler.

OK, that makes sense – but I think we can cross that bridge when we come to it :)


Improve code coverage for Bucket operations.

We were no doing any integration tests for failures, we should at least
verify that the exceptions raised match our minimal expectations.

This change is

Rebased to resolve conflicts. PTAL.

PTAL.


Pr tiltrotor sim

This brings in support for a quad tiltrotor in gazebo.
The model works, it needs a review and more testing.

@PX4BuildBot rebase

Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently :warning:

Automated rebase with GitMate.io was successful! :tada:

CI was full, rebasing again to kick it off. @PX4BuildBot rebase

Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently :warning:

Automated rebase with GitMate.io was successful! :tada:

@RomanBapst CI style failure. http://ci.px4.io:8080/blue/organizations/jenkins/Firmware/detail/PR-8936/1/pipeline/720
Locall you can install astyle and run make check_format or make format.

@dagar Thanks! I fixed code style and added a pre-commit hook.


where can i download the benchmarking data ？ Please

where can i download the benchmarking data ？ Please

Please check out the data availability section of the paper, all the datasets of the benchmark should be listed here: https://www.nature.com/articles/s41592-021-01336-8#data-availability


Language-specific guides request

Problem description
Add guide for C#
Problem location

I saw a problem on the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL:

Other:

Project version(s) affected

Suggestions for a fix

Thanks for requesting; let me close this as duplicate of https://github.com/docker/docker.github.io/issues/12697


strconv/quote: refactor - moved quotes to constants

in strconv/quote.go moved to constants ''" and '"'
small refactoring

@googlebot I signed it!

Okey, so what you think If i change to private const? I made public by mistake


Support using custom CNI

Sometimes user needs to use its own preferred CNI solution instead of EKS-A Cilium. It would be ideal if EKS-A can provide the possibility to swap out the default CNI with custom one, and EKS-A shall not revert the CNI changes made by the user during CLI upgrade / controller reconciliation.
In terms of solution, there are 2 routes we can go:

add a CustomCNI field in ClusterNetwork where we apply the CNI manifests user input during create/upgrade workflow.
add a UserManaged field in ClusterNetwork indicating user is managing its own CNI. During cluster creation, we still install the default EKS-A Cilium CNI to guarantee the successful provision of the cluster. After the cluster is running, user can update the ClusterNetwork with UserManaged true, manually delete the default EKS-A Cilium CNI and install custom CNI. Whenever the UserManaged flag is presented, the EKS-A CLI upgrade and EKS-A controller will skip updating/applying Cilium components.

We shall go with option 2 first, as option 1 is hard to validate and test given all the open options that exposes to the user. Also different CNI needs to be installed in various way, some of which cannot be set up properly with single manifest apply. The workflow to support it can be less trivial.

Hi @jiayiwang7, @chrisdoherty4,
Thanks a lot for working on this feature. We were really happy when we saw this listed in the release notes.
We just want to cross check: did all the necessary commits made it to the v.0.15.0 release?
If we check the list the commits since the last release (v.0.15.1) there seems to be still some commits related to this change:

Add Cilium skipUpgrade flag to v1alpha1 API (https://github.com/aws/eks-anywhere/pull/5298[)](https://github.com/aws/eks-anywhere/commit/32dd16e6c0cc5219271d0ec95d8bf025015079a4)
Add Cilium skipUpgrade flag to v1alpha1 API (https://github.com/aws/eks-anywhere/pull/5298[)](https://github.com/aws/eks-anywhere/commit/32dd16e6c0cc5219271d0ec95d8bf025015079a4)
Implement unmanaged CNI for CLI based clusters (https://github.com/aws/eks-anywhere/pull/5305[)](https://github.com/aws/eks-anywhere/commit/ba3bf74b203087ea6d0fc17957b21279b996e0d8)

Also note: For all of these PR-s the milestone is still set to v0.15.0...

Hi @Cajga. I've been out the last few weeks so hadn't seen your message. I'll get back to you soon.

@Cajga, the commits were squashed into a single commit as seen in https://github.com/aws/eks-anywhere/pull/5366/commits, or cherry-picked so you won't see the commits from main.
We'll reflect on our process to make identifying commits in release branches easier in the future.

Hi @chrisdoherty4 ,
Thanks for confirming it.
I was trying to figure it out myself but was not 100% sure what was happening.


support for ap-east-1 region for serverless s3 bucket

added support for ap-east-1 for serverless s3 buckets so that clients using those regions can get benefit from it.
not sure if the bucket name "coralogix-serverless-repo-ap-east-1" is available or if there is anything else that needs to be done from the coralogix side

solves #90

This is fixed by another pr, closing this one


Update instantiate config command

There is currently no message/ command to modify the instantiate settings on code info except for gov (#796).
This functionality should be limited to the creator who uploaded the code initially.
This can be achieved by adding a new method to the AuthorizationPolicy and calling it in the keeper.setAccessConfig.
Also the change rules in #840 need to be taken into account

If we do this, it should only be the actor who can instantiate the code, not the creator.
Basically, allowing someone with that ability to transfer it or share it

Removing this from v0.29 milestone as not urgent and needing more discussion first

Adding to v0.31.0 as part of the UX around permissioning.


Return value does not match

The return value does not provide as stated.

Do you have an example where the return value is not the one that is documented?

I did not make any observations in the test environment. Just a warning I came across in the PHP Storm PHPDoc extension. @GrahamCampbell

This is a false positive from phpstorm.


Update main.py

Fixed Issue #87
This code will check if the list returned by findall is not empty, and only access the first element in the list if the list is not empty. This should prevent the IndexError from occurring.

Thanks a lot have been busy lately


Can not Delete Location stored

My Environment

Plugin version: 0.4.1
Platform: Android
OS version: Android 6.0.1
Device manufacturer and model: SAMSUNG GalaxyJ7
React Native version: 0.50.3
Plugin configuration options:

{
 desiredAccuracy: 10,
  stationaryRadius: 50,
  distanceFilter: 50,
  debug: false,
  startOnBoot: false,
  stopOnTerminate: false,
  locationProvider: BackgroundGeolocation.ACTIVITY_PROVIDER,
  notificationTitle: 'Driver Apps',
  notificationText: 'GPS is On',
  interval: 20000,
  fastestInterval: 5000,
  activitiesInterval: 10000,
  stopOnStillActivity: false,
  maxLocations: 5
}

Context
I am working on application that update user location about every 30 seconds
Expected Behavior
I am trying to limit and control size of locations store in the apps. I expect it to be control.
Actual Behavior
deleteLocationand deleteAllLocations actually do nothing and they always return success status. I know it because I console.log in location event nothing happen after mentioned method. When the stored location reach its limit. it stay maximum no less
Best

If you delete location it will not be automatically synced (if you use option syncUrl) and it will not be returned by getValidLocations method. However it still might appear in getLocations until it is overwritten by new location.


Error with packages installer.vm

What's the problem?
At the end of installation packages to the system there is an error with package.
Steps to Reproduce
clean iso of windows 11 from microsoft -> new vm on virtualbox -> default selected packages on install stage
Environment

VirtualBox
Windows 11 - 10.0.22631
5.1.22621.3880
2.3.0
Boxstarter|3.0.3

Additional Information
terminal_logs.txt
chocolatey.log
chocolatey.1.log
chocolatey.2.log
log.txt

With the provided information it is not clear to me which package fails and what the error is. Please provide the console output as required in the bug issue template so that we are able to help you.

Thanks for reporting this @monfora!
@Ana06 I think I see the issue here. It looks like the CustomStartLayout.xml is not being applied and this seems to be an issue that others on newer Windows 11 builds are also experiencing: https://www.reddit.com/r/SCCM/comments/1cus7g3/importstartlayout_for_taskbar_stopped_working/
It seems that Microsoft may have either changed the format spec (though I can't find an updated one on the internet), or they are disallowing this ability via PowerShell (some people say their old configs still work when applied via GPO).
Fortunately, it looks like there is a simple fix, which is to store the layout as LayoutModification.xml and place it in C:\Users\default\AppData\Local\Microsoft\Windows\Shell\.
I just tested the fix on a Windows 10 pre-built FlareVM and it worked fine. I'll need to get a proper new Windows 11 build set up to test it there. I also need to test it from a fresh ISO through the full build to see if there may be any other tweaking that is needed (my Win 11 test should help with that as well).
To test, once the file is named correctly and placed in that location (and assuming you've removed any other pinned icons), you can run Stop-Process -Name explorer -Force and it should work. But this will need to be adjusted in our repo and installer.vm if it is the route we decide to take.

fixed by adding terminal log

Thanks for reporting this @monfora!
@Ana06 I think I see the issue here. It looks like the CustomStartLayout.xml is not being applied (errors out while trying to import it) and this seems to be an issue that others on newer Windows 11 builds are also experiencing: https://www.reddit.com/r/SCCM/comments/1cus7g3/importstartlayout_for_taskbar_stopped_working/
It seems that Microsoft may have either changed the format spec (though I can't find an updated one on the internet), or they are disallowing this ability via PowerShell (some people say their old configs still work when applied via GPO).
Fortunately, it looks like there is a simple fix, which is to store the layout as LayoutModification.xml and place it in C:\Users\default\AppData\Local\Microsoft\Windows\Shell\.
I just tested the fix on a Windows 10 pre-built FlareVM and it worked fine. I'll need to get a proper new Windows 11 build set up to test it there. I also need to test it from a fresh ISO through the full build to see if there may be any other tweaking that is needed (my Win 11 test should help with that as well).
To test, once the file is named correctly and placed in that location (and assuming you've removed any other pinned icons), you can run Stop-Process -Name explorer -Force and it should work. This will need to be adjusted in our repo and installer.vm if it is the route we decide to take.

Thanks for fix.

my netcat.vm package is not installing, other tools are pinned now at taskbar by following the instructions, but the desktop wallpaper is still not changed! @emtuls
Still this error!

The content of CustomStartLayout.xml file is as follows, which is created by itself when installer is run!

<defaultlayout:StartLayout GroupCellWidth="6">
</defaultlayout:StartLayout>

defaultlayout:TaskbarLayout
taskbar:TaskbarPinList
<taskbar:DesktopApp DesktopApplicationLinkPath="%AppData%\Microsoft\Windows\Start Menu\Programs\System Tools\File Explorer.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Productivity Tools\Windows Terminal.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Utilities\CyberChef.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Disassemblers\ida.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Networking\fakenet.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\PE\CFF Explorer.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Utilities\procexp.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Utilities\procmon.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Productivity Tools\notepad++.lnk"/>
<taskbar:DesktopApp DesktopApplicationLinkPath="%TOOL_LIST_DIR%\Productivity Tools\VisualStudio.lnk"/>
</taskbar:TaskbarPinList>
</defaultlayout:TaskbarLayout>

Noted! Will try to take a look this week or very soon. Some other priorities came up, but this is on my radar. Thank you for the follow up. :)

fixed by adding terminal log in first message

What exactly did you do here? and post that did you restart the installation?

@luffy-cmd I believe they were responding to @Ana06's message in regards to adding additional logs to their message, not actually fixing of the issue at hand.
In regards to the issue, I have a PR in that should fix the problem once it is merged: https://github.com/mandiant/VM-Packages/pull/1137
For now, a workaround that should work would be to copy lines 45-189 from installer.vm (https://github.com/mandiant/VM-Packages/blob/6b95e46f12c9e6fd6247bd9fe271203b0008315d/packages/installer.vm/tools/chocolateyinstall.ps1#L45C5-L190C1) and run them in an admin powershell after the installation fails.
After this, you can then run the following commands to have the taskbar set as expected:
Copy-Item "${Env:VM_COMMON_DIR}\CustomStartLayout.xml" "C:\Users\default\AppData\Local\Microsoft\Windows\Shell\LayoutModification.xml"
Stop-Process -Name explorer -Force

This should now be fixed with https://github.com/mandiant/flare-vm/pull/617 and https://github.com/mandiant/VM-Packages/pull/1137


Video Analytics additions for ExperienceEvent, part 2

As a follow-up to #364, 4 properties have been broken out of PR #417 to be treated post-0.9.3 release:

airDate
digitalDate
mediaSegments
bitrateAverageBucket

See PR #433.
What are the schemas that are affected by the issue
/context/experienceevent.schema.json
What are examples of products that are impacted by the issue
Video Analytics
Video Ads
Media marketing in Target

Just noticed that PR #433 shows as merged while the actual code history of the master branch does not show this merge and the pieces intended to be added by this PR are not actually in master.
We'll have to further investigate this and maybe respin the PR with the intended contents.


johnny-reborn: move to xesf's fork

Description of changes

Move to the more active and CI tested fork of @xesf;
Add a proper FHS-like folder structure;
Add a meta.mainProgram.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 290661 run on x86_64-linux 1

2 packages built:

johnny-reborn
johnny-reborn-engine


SMS send error

System Health details
System Information

version
core-2024.1.2

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.11.6

os_name
Linux

os_version
6.1.63-haos-raspi

arch
aarch64

timezone
Pacific/Auckland

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
4972

Installed Version
1.33.0

Stage
running

Available Repositories
1451

Downloaded Repositories
10

HACS Data
ok

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 11.3

update_channel
stable

supervisor_version
supervisor-2023.12.0

agent_version
1.6.0

docker_version
24.0.7

disk_total
115.2 GB

disk_used
8.3 GB

healthy
true

supported
true

board
rpi5-64

supervisor_api
ok

version_api
ok

installed_addons
VLC (0.3.0), Advanced SSH & Web Terminal (17.0.2), File editor (5.7.0), Music Assistant BETA (2.0.0b82), go2rtc (1.8.5), Nginx Proxy Manager (1.0.1)

Dashboards

dashboards
2

resources
2

views
6

mode
storage

Recorder

oldest_recorder_run
2024年1月8日 23:15

current_recorder_run
2024年1月16日 15:50

estimated_db_size
1215.38 MiB

database_engine
sqlite

database_version
3.41.2

Spotify

api_endpoint_reachable
ok

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue

"stored_data" is none in line 545 at api.py
Reproduction steps

add the integration
2.SMS Verification (Recommended)
3.after input area and phone
...

Debug logs
2024-01-16 02:54:35.211 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_protocol.py", line 452, in _handle_request
resp = await request_handler(request)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_app.py", line 543, in _handle
resp = await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_middlewares.py", line 114, in impl
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 80, in ban_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 233, in auth_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/headers.py", line 31, in headers_middleware
response = await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 149, in handle
result = await handler(request, **request.match_info)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/decorators.py", line 63, in with_admin
return await func(self, request, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 177, in post
return await super().post(request, flow_id)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 72, in wrapper
result = await method(view, request, data, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
result = await self._flow_mgr.async_configure(flow_id, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 320, in async_configure
result = await self._async_handle_step(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 416, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/akuvox/config_flow.py", line 125, in async_step_sms_sign_in
request_sms_code = await self.akuvox_api_client.send_sms(country_code, phone_number, subdomain)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/akuvox/api.py", line 313, in send_sms
await self.async_init_api_data()
File "/config/custom_components/akuvox/api.py", line 274, in async_init_api_data
await self.start_polling_personal_door_log()
File "/config/custom_components/akuvox/api.py", line 546, in start_polling_personal_door_log
if "last_polling_time" in stored_data:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable

Diagnostics dump
No response

Hi @dxh000130.
I have just released a new version, v0.1.1, which hopefully addresses your issue. Can you please upgrade to it and let me know if it now resolved?

I'm also getting the same issue @nimroddolev , when I enter my Australian phone number for SMS verification it says the "SMS code request failed. Check your phone number."
Both the 0.1.1 and 0.20 produce the same issue.
Going back to 0.07 seems to continue to work.

Hi @SirNoble, i'm sorry you've been having issues.

Which version of the integration are you running, and is this the entire log?

Thank you, yes is the entire log.
Using V0.1.1

Thank you. Have you tried entering your mobile number both with & without the leading zero at the start? I.e:<PHONE_NUMBER> and<PHONE_NUMBER>

I have, has the same issue both methods.

Very odd...
I think perhaps the best way to understand the issue would be to use mitmproxy to get the actual API requests for:

Requesting the sms verification code, and
Logging in with the tokens

Are you able to connect your phone via mitmproxy and send the request & response data for these 2 requests?
I can guide you through the process.

Sure, happy to provide these if you could please guide me through what you require.

Ok great, thank you.

Install mitm from https://mitmproxy.org/
Sign out of Akuvox and quit the app
Connect your phone to mitm (see their documentation)
Launch Aluvox and sign in via SMS
In the results from mitm, click on the send_mobile_checkcode request
Select the `Request tab and please send me a screenshot of the request parameters

Thank you
See below:

Thank you for the screenshot, unfortunately everything looks ok from what I can tell - beside the fact that it's not working for you 😀
Could you please confirm for me which was the most recent version that worked for you?
I will review the changes since that version to hopefully pinpoint the issue.

The latest version that worked was 0.07. which is when you added support for Australia.

Can also confirm Aussie mobile only works with 0.07 :)

Thanks for the great work @nimroddolev! Unfortunately I'm running into the same issue on US region - v0.0.7 is the last known version to work. Using later versions (commit 7c33d71) fails at auth token based login (same tokens work on v0.0.7):

HA version - 2024.4.3
HA debug logs
2024-04-25 23:39:24.074 WARNING (SyncWorker_2) [homeassistant.loader] We found a custom integration akuvox which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-25 23:39:24.075 WARNING (SyncWorker_2) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-25 23:39:25.249 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'hacs' accesses hass.components.frontend. This is deprecated and will stop working in Home Assistant 2024.9, it should be updated to import functions used from frontend directly at custom_components/hacs/frontend.py, line 68: hass.components.frontend.async_register_built_in_panel(, please create a bug report at https://github.com/hacs/integration/issues
2024-04-25 23:39:50.882 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'akuvox' calls `async_run_job`, which is deprecated and will be removed in Home Assistant 2025.4; Please review https://developers.home-assistant.io/blog/2024/03/13/deprecate_add_run_job for replacement options at custom_components/akuvox/api.py, line 81: hass.async_run_job(self.async_set_stored_data_for_key, "wait_for_image_url", self.wait_for_image_url), please create a bug report at https://github.com/nimroddolev/akuvox/issues
2024-04-25 23:40:09.407 WARNING (MainThread) [py.warnings] /usr/local/lib/python3.12/asyncio/base_events.py:1989: RuntimeWarning: coroutine 'AkuvoxData.async_set_stored_data_for_key' was never awaited
handle = None # Needed to break cycles when an exception occurs.
2024-04-25 23:40:10.972 WARNING (MainThread) [custom_components.akuvox] 🤨 Response: {'message': 'request is invalid', 'result': -1}
2024-04-25 23:40:10.972 ERROR (MainThread) [custom_components.akuvox] ❌ Unable to retrieve server list.

Mitmproxy for servers_list call
mitmproxy screenshots of servers_list API call:

Response:
{
  "datas": {
    "access_server": "<IP_ADDRESS>:8501",
    "access_server_ipv6": "[2600:1f1c:19d:ef11:c9d9:6958:e854:3df6]:8501",
    "pbx_server": "pbx1.ucloud.akuvox.com:5070",
    "pbx_server_ipv6": "[2600:1f1c:19d:ef00:ad66:e397:7c8c:66ca]:5070",
    "platform_ver": "6700",
    "rest_server": "ucloud.akuvox.com:8080",
    "rest_server_https": "ucloud.akuvox.com:8443",
    "rest_server_https_ipv6": "[2600:1f1c:19d:ef00:55a2:5f88:f6ba:1e8]:8443",
    "rest_server_ipv6": "[2600:1f1c:19d:ef00:55a2:5f88:f6ba:1e8]:8080",
    "smarthome_site": "ucloud.akubela.com",
    "vrtsp_server": "rtspsrv1.ucloud.akuvox.com:554",
    "vrtsp_server_ipv6": "[2600:1f1c:19d:ef11:ad10:2b7b:8bc5:505b]:554",
    "web_server": "ucloud.akuvox.com",
    "web_server_ipv6": "[2600:1f1c:19d:ef00:55a2:5f88:f6ba:1e8]"
  },
  "message": "success",
  "result": 0
}

Quick update - I noticed that the requests were being sent to https://gate.subdomain.akuvox.com:8600/servers_list as defined in const.py:
https://github.com/nimroddolev/akuvox/blob/7c33d71fe86d89e8f60292e6540ed20c00c3799f/custom_components/akuvox/const.py#L20-L21
However, my mitmproxy intercepts showed the queries were being sent to https://gate.ucloud.akuvox.com:8600/servers_list.
I changed REST_SERVER_ADDR = "gate.ucloud.akuvox.com" in const.py and login seems to be working now. These URLs are region specific (?) and perhaps not being set correctly for US/AU regions?

Quick update - I noticed that the requests were being sent to https://gate.subdomain.akuvox.com:8600/servers_list as defined in const.py:
https://github.com/nimroddolev/akuvox/blob/7c33d71fe86d89e8f60292e6540ed20c00c3799f/custom_components/akuvox/const.py#L20-L21
However, my mitmproxy intercepts showed the queries were being sent to https://gate.ucloud.akuvox.com:8600/servers_list.
I changed REST_SERVER_ADDR = "gate.ucloud.akuvox.com" in const.py and login seems to be working now. These URLs are region specific (?) and perhaps not being set correctly for US/AU regions?

I got it working as well by changing the URL. The problem I have now is that the live camera feed shows up on the web version of HA but when I try to livestream the camera from the iOS HA app, I get an error. Anyone else have the same problem? I attached a picture below of what happens on iOS when I try to play the livestream. I’d really appreciate any help on this, thanks!

Hi everyone. I just released a new beta, v0.2.0-beta2, which should address the issue of the missing SMS verification code when adding your SmartPlus account.
I'd like to ask you to please upgrade to the new version and let me know if you are now able to sign in via SMS verification.
To install the new beta version:

Open the Akuvox SmartPlus repository in HACS
Click on the ⋮ button in the top right
Select the ↻ Redownload option
Enable the Show beta versions option (it may take a moment to update the list)
Select version v0.2.0-beta2 from the updated version drop-down list
Restart Home Assistant

The problem I have now is that the live camera feed shows up on the web version of HA but when I try to livestream the camera from the iOS HA app, I get an error.

Hi @namdgk. This issue is not related to the Akuvox SmartPlus integration but with the Home Assistant companion app for iOS.
There is another custom component, WebRTC Camera, which can fix this issue that you might want to check out.

Hi everyone. I just released a new beta, v0.2.0-beta2, which should address the issue of the missing SMS verification code when adding your SmartPlus account.
I'd like to ask you to please upgrade to the new version and let me know if you are now able to sign in via SMS verification.
To install the new beta version:

Open the Akuvox SmartPlus repository in HACS
Click on the ⋮ button in the top right
Select the ↻ Redownload option
Enable the Show beta versions option (it may take a moment to update the list)
Select version v0.2.0-beta2 from the updated version drop-down list
Restart Home Assistant

Just tried this and unfortunately still having the same issue with using it via the app token method, when entering the tokens and press okay it gives the error "sign in failed" please check the values entered and try again. Tested via the SMS method and that seems to be work fine.

Could you please try deleting the instance from

Could you please try deleting the integration and adding it again?

Sure, will try this and report back.

Could you please try deleting the integration and adding it again?

Sorry, same issue

Thanks for trying. I will take another look as to why it’s not working for you.
Just to confirm: you are sure your auth_token and token values are correct? They get reset each time you log in via SMS verification.

That's correct, there are new values each time.

The integration sets the regional subdomain used to communicate with Akuvox according to the list from this repo, so it's unclear to me why the SMS request fails.
I released a new beta, v0.2.0-beta3, which allows you to manually override the API regional subdomain used to communicate with Akuvox. You can try to use another subdomain/

hey - i am struggling to get seperate token and auth token using MITM Proxy.
I am unable to login with mobile number which is listed on smart plus account.
I am using MITM Proxy on an Ipad. I can see a token, but it is the same in all. but im not seeing the "token and auth-token" in the
"https://gate.aucloud.akuvox.com:8600/servers_list"
I also try to login using SMS method and it says number doesnt exist
Located in Australia
using version Beta V2.03
Thanks for your help on this integration, I will support you if i can get some assistance

hey - i am struggling to get seperate token and auth token using MITM Proxy.
I am unable to login with mobile number which is listed on smart plus account.
I am using MITM Proxy on an Ipad. I can see a token, but it is the same in all. but im not seeing the "token and auth-token" in the "https://gate.aucloud.akuvox.com:8600/servers_list"
I also try to login using SMS method and it says number doesnt exist
Located in Australia
using version Beta V2.03
Thanks for your help on this integration, I will support you if i can get some assistance

I just downgraded to Beta 2.02 and then itallowed me to login via SMS. Just for reference.
It also shows login token and auth tokens in the configuration, would I be able to logout and then use those tokens to log back in, or is the token value a one-off use

@robotsmarthomes have you ever figured out the token retrieval issue? I was trying to grab it from mitmproxy but there is no POST request (anymore?) for the /servers_list endpoint. I can only see a GET request with no body in the request:

I am stuck with getting the auth_token, I can just find the token value.
Thanks !


Implement Integration Test for APIM APK Agent

Description:
Implement Integration Test for APIM APK Agent

Part fix: https://github.com/wso2/apk/pull/2080


[widgets][paragraph]: Truncate long lines when wrap is false

Fixes #77

Thanks for the detailed bug report and for taking the time to look into it


Isomorphic rendering

Hi Zeek! Just kidding :)
I'm not seeing the isomorphic rendering in action - when I visit https://preact-pwa.appspot.com/articles for example the article list markup is not being served in the body. Does it have to be enabled somewhere?
cheers!

@ricardobeat Yo!  boo!
if you peek in the root route:  https://github.com/ezekielchentnik/preact-pwa/blob/master/src/server/routes/root.js#L54  you can adjust the withTimeout threshold to yield SSR - depending on the speed of your API.
The idea is to allow an 'escape hatch' for abnormal or slow API requests.  If any server side API calls are slow, they will be aborted (raced against the threshold) so that we do not make the client wait 'too long' and can allow the client to take over.  In the cases that the API response is fast, you get the advantage of SSR.
This logic can be improved.  Likely if the server request is slow, by the time the client makes the request, it too will be slow. maybe.
This can also be used to warm your API cache so by the time the client hits it, it gets a cached response.
Also, there is a case with service workers that needs to be fixed:  when SSR does work the page will be cached by the service worker - The client needs to be smarter and invalidate the cached data inside redux and refetch (based on whatever threshold make sense).  This would ensure a nice SSR but then still allow the client to update and  be fresh. Right now it works because that demo api call returns a static set of data.
Finally, the server logic could be smarter and cancel the original API requests if the 'escape hatch' is used and bypassed to client.  this will reduce redundant requests to your API.
Any ideas or help in this regard would be awesome!  Cheers.


Test App working fine but new installs not recorded on live app

Hi, We are integrated  AppVirality in our app .It is work fine in test mode.But in live mode It is not recorded number install count in referrer.
We Tested as per your procedure It gives no clicks and no installs properly.

Hi, can you please write to ram[at]appvirality.com along with your account details and query.


ERR: Request Rejected

This two errors appear almost all the time the bot goes on searching  --> ERR: Request Rejected || ERR: Too many request from this user

i know it sucks


MiniOxygen encountered an error while running your app's entry point

What is the location of your example repository?
No response
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
latest from npm
What version of Remix are you using?
No response
Steps to Reproduce
npm create @shopify/hydrogen@latest
Javascript
NPM yes
Tailwind
Then:
h2 upgrade
Then:
npm run dev
Expected Behavior
Should open page with front
Actual Behavior
error
│                                                                                │
│  MiniOxygen encountered an error while running your app's entry point:                 │
│                                                                                │
│  SyntaxError: Unexpected token '<', "<!DOCTYPE "... is not valid JSON
NPM version: v10.8.2
Node version: v20.16.0

The issue might be that port 3000 on your local machine is being used by another program, which is what happened in my case.

The issue might be that port 3000 on your local machine is being used by another program, such as Grafana. This is what happened in my case.

I don't have anything else running. One week ago I got this error only sometimes. But since that I always got this error. So impossible to run application in "Dev" mode

It was the PORT 3000 that was already used ! Thanks :)
Will be nice to add different port on the ENV / also say in the console if we would like to open using another port (like npm run dev on react project)

The dev server should already start in a different port if 3000 is in use. However, there might be a bug in the way MiniOxygen communicates with the Vite server when there are more than 1 Vite servers up (e.g. 3000 and 3001). Leaving this open for now while we try to reproduce it.

Make sure you port 3000 is available ;)


"for" keyword is no longer highlighted

See examples:
https://github.com/PHPMailer/PHPMailer/blob/master/src/PHPMailer.php#L1841
https://github.com/nawarian/raylib-ffi/blob/6e748edac33afb6a99260c911c727213a574d89b/examples/shapes/shapes_easings_rectangle_array.php#L40
Not really sure why this occurs (came here from Linguist, which listed this grammar as the used grammar for PHP on GitHub), but I figured I should report this.

Maybe you can just add it to highlights.scm if you needed it locally

This works for me locally, not sure why the github links aren't highlighting them (nothing really to do here).

Unless you want to add for to the list of keywords:
https://github.com/tree-sitter/tree-sitter-php/blob/594b8bad093abe739c3d2a2cae5abae33c5fb23d/queries/highlights.scm#L99


I have to open two files to make git_status runnable?

As title. When I open a git-tracking project with some files modified but not yet commit, I cannot trigger git_status(with my own key-binding) before I open a second file(can be any) of the same project. Any idea?

Does the git_status window open empty or does the not open at all before you have two files changed?
For me, git_status can always open but will open empty when no files are changed, there is nothing in code limiting git status from opening with results or without, if your window doesn’t open at all I suspect it might not be fzf-lua’s fault.

Does the git_status window open empty or does the not open at all before you have two files changed?
For me, git_status can always open but will open empty when no files are changed, there is nothing in code limiting git status from opening with results or without, if your window doesn’t open at all I suspect it might not be fzf-lua’s fault.

Thanks for your kindly reply.
I understood and expected to see the empty window shown when no file is changed. But I've experienced two situations:

It blinks. The empty window was shown and disappeared very quickly.
As my OP.

I'm using packer.nvim and these are the settings near to the config = function() ... end:
use {
  'ibhagwan/fzf-lua',
  requires = {
    'vijaymarupudi/nvim-fzf',
    'kyazdani42/nvim-web-devicons'
  },
  config = function()
    require'fzf-lua'.setup {
      winopts = { -- Only valid when using a float window
        border = { '╭', '─', '╮', '│', '╯', '─', '╰', '│' },
        height = 0.65,
        width = 0.7,
        row = 0.9,
        col = 200,
        preview = {
          default = 'bat',
          title = false,
          wrap = 'nowrap',
          vertical = 'up:80%',
          layout = 'vertical'
        }
      }
    }
  end
}

If the window flashes something tells me git status -s returns an unexpected result or an error.
While the problem persists, can you run (from inside neovim) :git status -s and paste the output here?

 I moved these key-mappings of the LSP's server:setup { ... } and now everything works. Sorry for the falsy bug-report. That's why I have to open a file to trigger the LSP so these key-mappings will be set.

No worries, happens to everyone :)


OneSignal not accepting player id 'invalid_player_ids' when sending notification via REST API

When I try to send a notification if there were invalid_player_ids in the ids, the process of sending notifications is stop, how it's possible to let the one signal ignore the invalid_player_ids?

Hey @ahmedsafadii ,
The issue you mentioned is related to OneSignal itself but not this library. You should check onesignal documentation for the answer: https://documentation.onesignal.com/reference/create-notification#results---create-notification
My suggestion would be not sending invalid player id s in the first place and filter them beforehand.
I am closing the the issue.


🤖 Control panel: Migrate for using task to handling installation of tooling helm chart

User Story
Background
We started the movement to migrate the control panel to use queue/task to handle all those background tasks,  we have done the parts for registering an app and managing the se bucket.
Task
This ticket is to migrate the Control panel to use  sns queue to handle the task of installing the tooling helm charts.
Value / Purpose

Make the task easier to be tracked on AWS
Make the task handling more robust.
Prepare the future to split the Control panel to light-weight app for UIs and several backend components for handling different tasks

Useful Contacts
No response
User Types
No response
Hypothesis
If we... [do a thing]
Then... [this will happen]
Proposal
No response
Additional Information
No response
Definition of Done

[ ] Documentation has been written / updated
[ ] The helm chart can be installed via sns queue and celery task
[ ] Tests are green

Please add some details on the implementation @ymao2

17/10/23 refinement: moving to backlog, this is a piece of work that is not essential functionality, but will align Helm with the other components.

Closed as we're focussing on AP UI work.


Flaky test_rolling_with_local_store_writes fails due to timing

Description
This test should have more reliable output or be disabled until we write better tests that cover this use case.
See: https://github.com/whylabs/whylogs/actions/runs/4506601154/jobs/7933562528?pr=1155

[x] I have reviewed the Guidelines for Contributing and the Code of Conduct.

Can you review tests that use actual time and see if we can have better tests?


Do not mention pre-commit hook in the contrib docs

...as it is disabled for now I think we should not mention it in the docs?

Coverage remained the same at 39.913% when pulling 2fb856854884a947f265d521448363e776f9ad3d on kmws:doc-hook into 6aa4a9ac011fb159a627dd418e24cb90b976ad6a on newsboat:master.


Discussion: Self-host emoji set and replace some important dark emojis in the dark mode

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

Some heavy emojis shown as reactions and in the emoji picker are currently hard to see on the dark background of the dark mode, especially the heavy plus icon that is used for up-voting.
Moreover, the emoji and emoji-picker components currently fetch the emoji sheet from unpkg, which is a privacy / GDPR problem.
Fixes #5118.
Description

Self-host the emoji sheet on the Artemis server to prevent requests to external CDNs.
Provide inverted versions of some heavy icons for the dark mode
Created new wrapper components for emojis and the emoji picker to hide the dark mode handling details, and to make them reusable elsewhere in the application in the future (outside of metis).

The sheet will be downloaded from the Artemis server as soon as the data is needed. The full sheet is currently a bit less than 4MB large, which is unproblematic. We do not use the native / system-provided sheet to ensure a consistent look of emojis across platforms.
For dark mode, some icons will be individually replaced. We do not replace the full emoji sheet, but only a few certain emojis using individual PNGs to minimize traffic / loading times. Each of the replaced images is just a few KB at most.
This will make upgrading the emoji set easy in the future, as the emoji sheet can simply be replaced with a new one without the need to edit anything - the replaced icons should still work after an upgrade if their position in the unicode didn't change, which is unlikely to ever happen.
Fixes caching of static resources; the previous config did not have any effect due to wrong paths being used.
Replaced it with a new, selective caching config for about us images and emoji assets.
Steps for Testing

Prerequisites:

1 Student

Log in to Artemis, open Network Tab in Dev Tools
Navigate to the discussion page of a course
Add / remove reactions; play around:

Make sure that all emojis still work
make sure that the emoji picker still works

Switch the theme. Verify that reactions with the replaced heavy icons do change their color accordingly. Verify that they change their color in the emoji picker as well
Using Chrome, please verify that the emojis do not visually reload if you change the category in the picker or add a new reaction to a post
Reload the page (no force reload!). Verify that emoji assets are served from the cache.
Verify that requests to fetch emoji assets (either the full sheet or single images) have a caching header with max-age=126230400, public
Verify that other requests still have a no-cache value in the same header field
Visit the About us page using the link in the footer. Verify that the images have a long duration in the caching header just like the emoji assets, but the "about-us.json" does have a no-cache header

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Screenshots

https://user-images.githubusercontent.com/23171488/171414432-daf17367-bbcb-4481-adaa-1f4757f244b1.mov

when adding an emoji all other existing emojis reload again as well
Could you explain that bit more in detail? I can't observe any similar behavior

when adding an emoji all other existing emojis reload again as well

Could you explain that bit more in detail? I can't observe any similar behavior @jbllhff

I encountered the same situation. Whenever a new emoji is added, all others disappear and need to reload (which takes a few seconds). When switching between emoji tabs, the pages start off blank (see screenshot) and need to load for a few seconds. Even switching back to a previously loaded tab requires waiting a few seconds for the emojis to reload.

The reloading issue reported above seems to only happen in Chrome. Need to investigate that

@jbllhff @TheTrueVester @ge65cer Could you please re-test this PR with my new fix on Chrome? Caching should now work properly and no flickering be visible anymore after the emoji assets were loaded initially

Some additional client code was introduced in this PR which seems to be untested. Could you please add some client tests and also mention the coverage in the description? Thanks

Added some client tests for the new components


Jupyter Notebook

Prework

[x] Read and agree to the code of conduct and contributing guidelines.
[x] If there is already a relevant issue, whether open or closed, comment on the existing thread instead of posting a new issue.

Question
Does great_tables work within jupyter notebooks?

It works fine for me with vscode notebooks (which are based of jupyter)

I just tried in Google Co-lab and it runs fine. There must be some issue with my local installation of Jupyter Notebook.

Since no further discussion has taken place, I will close this issue.


Word2vec benchmark crashes when saving dump file

Word2vec benchmark crashes when saving dump file after an epoch with the following info:

ValueError: The feeded_var_names[0]: 'analogy_a' doesn't exist in pruned inference program. Please check whether 'analogy_a' is a valid feed_var name, or remove it from feeded_var_names if 'analogy_a' is not involved in the target_vars calculation.

It looks like caused by #506
reproduction steps:
run on local machine, CPU only
cd models/recall/word2vec/benchmark
python -u ../../../../tools/static_ps_trainer.py -m benchmark.yaml

you shouldn't use static_ps_trainer.py,  static_ps_trainer.py is only used when training in distributed mode.

AFAIK the above crash is because of changed in #506 feed variables not whether it's the distributed mode. It was running OK before. Do you have more information why those variables where changed?


Frontend Product Design Ticket - Beckn-In-A-Box Project

Description
This issue aims to address the frontend product design requirements for the Beckn-In-A-Box (BIAB) project. The BIAB platform provides an integration framework for beckn-enabled applications. The frontend design needs to be intuitive, user-friendly, and aligned with the project's objectives.
Context
The Beckn-In-A-Box project aims to simplify the development and deployment of beckn-enabled applications. The project requires a frontend user interface that guides users through the configuration and build processes. The UI should be modular, accommodating various product configurations and modules. The design should align with the overall project vision and existing components.
Goals

Design a user-friendly and intuitive product development wizard UI.
Ensure the UI aligns with the existing BIAB platform design and branding.
Accommodate the various product configurations and modules in the UI flow.
Create a seamless user experience during the application build process.

Design Requirements

The UI should follow modern design principles and best practices.
Responsive design to ensure compatibility across tab and above devices and screen sizes.
Clear and descriptive UI components for each step of the product configuration process.
Integrate visual cues and feedback to guide users through the build process.
Consistent color scheme and typography in line with the BIAB branding.
Incorporate UI components for selecting pre-built modules and microservices.
Collaborate closely with the backend development team to ensure UI-Backend integration.

Resources

BIAB Product Overview and Definition: link

Milestones

Design Concept and Wireframes
UI Prototype Development
Finalize Frontend Design and UI/UX Review

How to Contribute

Review the Detailed Product Requirements Document for a thorough understanding of the project.
Study the existing BIAB UI to ensure design consistency.
Collaborate with the design team, frontend developers, and mentors to refine the design concept.
Create wireframes and prototypes for review and feedback.
Regularly update the issue with progress, and any design-related updates.
Upon completion, submit the design for UI/UX review and feedback.

I am working on this issue


feat: add local and remote ipv4 network cidr

Description
This PR adds  additonal inputs for the vpn-gateway module requested in issue  https://github.com/terraform-aws-modules/terraform-aws-vpn-gateway/issues/44 namely additonal inputs of  local_ipv4_network_cidr and remote_ipv4_network_cidr

In order to finish this PR, we need to update terraform code to use variables added by this PR and update the minimum version of AWS provider in versions.tf.
Please revert changes in README. pre-commit run -a will update the documentation according to those markers.


Bug: Optional props are typed as | undefined

Describe the bug
When using the .props() method on a VueWrapper for a prop which is not required but has a default value the returned type includes undefined.
To Reproduce
https://github.com/DesselBane/test-utils-reproduction/blob/master/src/components/foo.spec.ts
Expected behavior
I would argue that the type should not include undefined since it has a default value and this default value is returned at runtime.
Related information:
This happened on an update from v2.4.1 to v2.4.2, most likely somewhere in this PR: https://github.com/vuejs/test-utils/pull/2137 (but I'm not sure)

I'm not sure about that.
@pikax What's your take on this?

I'm actually writing this in the advanced typescript docs here :D
That's caused because the props inferred are the wrong ComponentPublicInstance props, it should be the internal ComponentPublicInstance instead of the rendered props.
Is a bug, at runtime:
    const a = mount(
      defineComponent({
        props: {
          a: {
            type: String,
            default: 'a'
          }
        },
        setup(props) {
          return () => props.a
        }
      })
    )

    expect(a.props('a')).toBe('a')

    a.setProps({ a: undefined })
    expect(a.props('a')).toBe('a')

That's passing.


Don't catch IOError

Evented mailbox should not know anything about IOError.
Some notes:

Method << is changed to catch any error that wakeup can raise. It can be not only IOError.
Method check is more interesting. It just converts IOError to MailboxShutdown and I think it's a cryptic behavior. It is also very related to https://github.com/celluloid/celluloid-io/issues/151. Celluloid::IO's reactor calls task.resume inside run_once. This can lead to the situation when IOError is caught by mistake (raised by user's code for example or by socket operation). So MailboxShutdown should be raised by reactor itself but not by generic Mailbox::Evented (if there's really any need in this exception — now it's indistinguishable from MailboxDead in its behavior) .

How to fix a build? It fails on initialization phase:

git://github.com/celluloid/timers.git (at master).

Source contains 'timers' at: 4.1.0```

@digitalextremist build is okay now. what do you think?

Rebased.

Simple explanation of this bug:
class Worker
  include Celluloid::IO

  def perform
    # if I raise IOError here it will be dispatched in Celluloid.
    # If Worker is supervised it normally restarts after this.
    @socket.read
    # ... task is resumed by Celluloid::IO::Reactor
    # if I raise IOError here it will be dispatched in Celluloid::Mailbox::Evented.
    # If Worker is supervised it doesn't restart.
  end
end

Can we go ahead and merge this? It looks good.

:+1: based on the description of the bug, I need this for my supervised workers I use with my celluloid-io-pg-listener project.


Address CI failures in test_reporter_from_westeros

The test message_ix/tests/test_reporting.py::test_reporter_from_westeros began to fail during PR #478 on all testet python versions (3.6, 3.8 and 3.9) on macOS and Ubuntu, and on python version 3.6 on Windows. At first glance the test run successfully with python versions 3.8 and 3.9 on Windows. However, opening the "Run test suite using pytest" for both versions, it seems the test fails also (link).
https://github.com/iiasa/message_ix/runs/3338664331?check_suite_focus=true#step:17:154
_________________________ test_reporter_from_westeros __________________________

self = MultiIndex([('Westeros Electrified', 'baseline', 'Westeros', ...),
            ('Westeros Electrified', 'baseline', 'W...ctrified', 'baseline', 'Westeros', ...)],
           names=['model', 'scenario', 'region', 'variable', 'unit', 'year'])
level = 'value'

    def _get_level_number(self, level) -> int:
        count = self.names.count(level)
        if (count > 1) and not is_integer(level):
            raise ValueError(
                f"The name {level} occurs multiple times, use a level number"
            )
        try:
>           level = self.names.index(level)
E           ValueError: 'value' is not in list

../../../../hostedtoolcache/Python/3.6.14/x64/lib/python3.6/site-packages/pandas/core/indexes/multi.py:1426: ValueError

Per #499, the cause was a change in the behaviour of pyam-iamc from version 1.0.0 to 1.1.0.
The test function test_reporter_from_westeros() (but not the actual reporting code) relied on this behaviour.
The previous behaviour was unadvertised and untested in pyam, so it's not really a bug that it changed; we simply adjust to the new behaviour.

Thanks @LauWien!


Add secureBoot EFI flag

What this PR does / why we need it:
This PR enables the use of secure boot by adding a boolean option under EFI.
If set, alternative (secboot) OVMF roms will be loaded and secure boot will be enabled.
If not set or set to false, the usual OVMF roms will be loaded and secure boot will not be enabled.
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:
The SMM domain feature must be enabled along with EFI secure boot. The dependency is enforced by libvirt.
The state of secure boot should be reflected in the VM's dmesg (dmesg | grep secureboot)
Release note:

Add "secureBoot" EFI option to enable secure boot in OVMF.

Force-pushed to add missing update of generated files.

/test pull-kubevirt-e2e-k8s-multus-1.13.3

Force-pushed to add a validator

/test pull-kubevirt-e2e-windows2016

@rmohr Is there something further missing from this PR that you'd like to see before we merge?

Rebased and added a commit to make the OVMF path configurable.

@rmohr the path is now configurable, a functional test is being worked on as a separate effort.
This should be good to go.

@jean-edouard this PR has merge conflicts. Once you resolve that I'll approve.

/lgtm
/approve


chore(main): release iso-passkeys 0.1.1

:robot: I have created a release beep boop
0.1.1 (2023-05-15)
Bug Fixes

use iso-did (eb0648f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugomrdias/iso-repo/releases/tag/iso-passkeys-v0.1.1 :sunflower:


add stella vaults

// UST-GLMR
// vaultAddr = "0xb0b90AD6588F3B072Bc4B3E7237189E6AC6808A7"
// stratAddr = "0x9D3b9d52D40778F542F69a2fa0EfC5C2D86Fb687"
// wantAddr =  "0xb6835036C9A2A96D3BfB8DBE1722971fEb456a83"
// DAI-USDC
// vaultAddr = "0x0b68f9585E24c8D271eFCf60ebd35c1a94A5F91f"
// stratAddr = "0x46B75692C835f93C98800624329E987C0e1BA595"
// wantAddr =  "0x5Ced2f8DD70dc25cbA10ad18c7543Ad9ad5AEeDD"
// BTC-GLMR ~~ PENDING FIRST HARVEST
// vaultAddr = "0x721Bf3d97e2E89e12B73D8b40C8B998AF8E0E9a4"
// stratAddr = "0x5b239E0E222200CB48D7fF7cF7D2Fd68Ce948227"
// wantAddr =  "0x4CDf77828b839506bF6B6e219017B65623042400"
// USDC-ETH
// vaultAddr = "0x44f04CA4B29c5F5471345dcaBAB6e40320374bf4"
// stratAddr = "0x79F26046D122d2F26B1D37237D5C056fB580447A"
// wantAddr =  "0x0Aa48bF937ee8F41f1a52D225EF5A6F6961e39FA"
// GLMR-ETH
// vaultAddr = "0x3871E20e08ae59bDa4b42A1a6C27403bEedDa042"
// stratAddr = "0xA27BF4d56870A37e37a687B8ab3740d8487Ee2C6"
// wantAddr =  "0x49a1cC58dCf28D0139dAEa9c18A3ca23108E78B3"

pending ownership transfers

Make sure in the future to setPendingRewardFunctionName. Also please run creationdate script with include accurate timestamps before I can merge.

Make sure in the future to setPendingRewardFunctionName. Also please run creationdate script with include accurate timestamps before I can merge.

where do I find setPendingRewardFunctionName ?

Make sure in the future to setPendingRewardFunctionName. Also please run creationdate script with include accurate timestamps before I can merge.

where do I find setPendingRewardFunctionName ?

On the strategy contract


Update for cmake compilation with ROOT6 (+ scripts to compile on sukap)

The current CMakeLists.txt is not compatible with ROOT6, I updated it so it is compatible with both ROO5.34 and ROOT6. In addition, I updated the old scripts available to compile on sukap and outside sukap.

Hi Tom, I followed your comments and moved the scripts to the setup/ directory. I also merge the sukap scripts and the make scripts.
One point however: I kept the make script in the root directory. If moved, we will have to consider how to be sure  the script is executed from the correct directory. In addition, I think users will then see it more clearly if it stay there,

I also included the modifications made by Nick on WCSim hybrid for the CMakeList, there was a bug happening in some cases.


Unable to find the autoloader

Hello Support Team,
I can not find the autoloader file on git

We do not include an explicit autoload script but you can use composer to load the PHP package, or easily setup rules for your frameworks autoloader, or include required sources on demand.


fix: keep state accross yield points when reading data

This is a cherrypick of 2de982823b93440f1d6527a94ac53c7f0d38bb3d, with some modifications to make it compile, which fixes dignifiedquire/async-tar#5

oops, I did not see #10


Sub-sub annotation commands

Please describe your suggestion
Being able to have commands like /command subcommand subsubcommand argument within the annotation-way of building commands.
Please supply examples of the desired inputs and outputs
Example:
Input: /permission group register <name>
Output: Creates a permission group with the provided name.
Input: /permission group list
Output: Lists all permission groups
Input: /permission rank register <name>
Output: Creates a permission rank with the provided name.

Work for this is currently taking place in the dev/annotations branch, and will be released whenever the annotations branch is ready

@JorelAli is this possible now?

@JorelAli is this possible now?

Work on the annotation system was effectively halted in favour of focusing on other aspects of the CommandAPI. We did have a member from the CommandAPI Discord server (I believe it was @XHawk87?) that was interested in continuing development on the annotation system, but I've not been actively keeping track of progress on this.

@JorelAli is this possible now?

Work on the annotation system was effectively halted in favour of focusing on other aspects of the CommandAPI. We did have a member from the CommandAPI Discord server (I believe it was @XHawk87?) that was interested in continuing development on the annotation system, but I've not been actively keeping track of progress on this.

Interesting, but you care that it always works on the latest version, it just doesn't get new features. right?

Yes, we still ensure that the annotation system is compatible with the latest versions that we release. (Of course, if that doesn't happen to be the case, that's a high priority issue to fix because that breaks backwards compatibility!)

Yes, we still ensure that the annotation system is compatible with the latest versions that we release. (Of course, if that doesn't happen to be the case, that's a high priority issue to fix because that breaks backwards compatibility!)

Ah, nice! Where can i find the code for the annotation processing?

Where can i find the code for the annotation processing?

The annotation processing code can be found in the commandapi-annotations module
We have two branches where annotation development was taking place: dev/annotations and dev/annotations2. It's been so long since I last touched these branches that I no longer know what work has been done, nor the current state of them.
We have a specification document for the new annotation system which can be found here: https://github.com/JorelAli/CommandAPI/blob/master/annotations_spec_doc.md
We also have a dedicated discussion channel for the new annotation system over on our Discord server here: https://discord.com/channels/745416925924032523/1121006613126459452

Where can i find the code for the annotation processing?

The annotation processing code can be found in the commandapi-annotations module
We have two branches where annotation development was taking place: dev/annotations and dev/annotations2. It's been so long since I last touched these branches that I no longer know what work has been done, nor the current state of them.
We have a specification document for the new annotation system which can be found here: https://github.com/JorelAli/CommandAPI/blob/master/annotations_spec_doc.md
We also have a dedicated discussion channel for the new annotation system over on our Discord server here: https://discord.com/channels/745416925924032523/1121006613126459452

Are you sure that commandapi-annotations module is the correct link? I only see the annotations and no processing there?

I'm very sure! In that module you can find Annotations.java with the bulk of the annotation processing logic :)

I'm very sure! In that module you can find Annotations.java with the bulk of the annotation processing logic :)

Ah, I just saw that too. Thank you!

Can i just pr my changes to the master branch or is there a specific (active) branch for that?

Given that either dev/annotations and dev/annotations2 are both in a fairly dead state, I'm happy to accept contributions to annotations to our main dev branch dev/dev please 🚀

Be aware that my local branch has significant structural changes aimed at dev/annotations2, so best go for quick and simple solutions. Don't bother spending a lot of time cleaning everything up.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jorel Ali @.>
Sent: Tuesday, August 6, 2024 9:43:04 PM
To: JorelAli/CommandAPI @.>
Cc: XHawk87 @.>; Mention @.>
Subject: Re: [JorelAli/CommandAPI] Sub-sub annotation commands (Issue #263)
Given that either dev/annotations and dev/annotations2 are both in a fairly dead state, I'm happy to accept contributions to annotations to our main dev branch dev/dev please 🚀
—
Reply to this email directly, view it on GitHubhttps://github.com/JorelAli/CommandAPI/issues/263#issuecomment-2272114412, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA3KFSNHGH3CJO4XLPQZWM3ZQEYNRAVCNFSM6AAAAABMC7GNYCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZSGEYTINBRGI.
You are receiving this because you were mentioned.Message ID: @.***>

Be aware that my local branch has significant structural changes aimed at dev/annotations2, so best go for quick and simple solutions. Don't bother spending a lot of time cleaning everything up. Sent from Outlook for Androidhttps://aka.ms/AAb9ysg
…
________________________________ From: Jorel Ali @.> Sent: Tuesday, August 6, 2024 9:43:04 PM To: JorelAli/CommandAPI @.> Cc: XHawk87 @.>; Mention @.> Subject: Re: [JorelAli/CommandAPI] Sub-sub annotation commands (Issue #263) Given that either dev/annotations and dev/annotations2 are both in a fairly dead state, I'm happy to accept contributions to annotations to our main dev branch dev/dev please 🚀 — Reply to this email directly, view it on GitHub<#263 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA3KFSNHGH3CJO4XLPQZWM3ZQEYNRAVCNFSM6AAAAABMC7GNYCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZSGEYTINBRGI. You are receiving this because you were mentioned.Message ID: @.***>

Would be nice if you push them before i touch anything. 😄

I don't have access to push to this repo, it'd be a Pull Request and it's not yet at that stage. Also, dev/annotations2 is a rewrite of the annotations system aimed at 10.0. You won't be seeing this code on dev/dev anytime soon. If you want to see more immediate changes, you're welcome to submit them to dev/dev as @JorelAli said, just be aware that this is all going to get rewritten for 10.0.
That said, I'll try to clean things up and submit a PR for dev/annotations2 soon, so I can get feedback on the new structure.


pip install devsim

How can we make devsim available in PyPi?
it would make the installation much easier in the instructions for the gdsfactory plugin
@proppy
@mithro

also see https://github.com/hdl/conda-eda/issues/240

I'd encourage you to take a look at https://github.com/pypa/cibuildwheel which could help to put together the automation to build multiple platform wheels.

this has truly been a dreadful exercise.  exploring using the wheel package.
# setup.cfg
[bdist_wheel]
py-limited-api = cp36

#setup.py
from setuptools import setup, Extension

module1 = Extension('devsim.foo',
                    py_limited_api = True,
                    sources = ['foo.c'])
setup(name='devsim',
      version = '1.0.0',
      packages=['devsim', 'devsim.python_packages'],
      package_dir = { '' : 'lib'},
      package_data={'devsim' : ['./*.so',]},
      ext_modules = [module1],
      )

and using the command line:
python setup.py bdist_wheel

I am able to create a wheel package from an existing devsim distribution.  I still need to find the example files.  The dummy foo module is unfortunate, and I may have to use trickery to remove it from the final distribution.
To summarize, python packaging is terrible.

I have create an automated build with binary wheels for the following intel 64 bit platforms:
macOS, Linux, Windows.
They should be able to be installed using this command:
pip install -i https://test.pypi.org/simple/ devsim

On a conda system, the example files can be tested here:
cd ${CONDA_PREFIX}/devsim_data/testing
python cap2.py

Please note that an MKL installation is required.
conda install mkl

If someone could try this, it would be greatly appreciated.
If the pypi test server has been purged, the wheel files are available here:
https://github.com/devsim/devsim/releases/tag/vbdist6

The examples ran for me on both Ubuntu and Windows after creating a new conda environment.
I also tried with pip install mkl which also worked, so hopefully this also works in regular non-conda venvs.
Is the https://test.pypi.org/simple/ URL just a placeholder?

@simbilod
Thank you for trying the examples out.  The test.pypi.org server is a meant for testing deployments.  The packages can be purged at any time.
I intentionally left out any dependencies since MKL is huge, and zlib and sqlite are part of the Python standard library.
Are there any major platforms missing, or is intel 64 bit on Linux, macOS, Windows sufficient?

Hi Juan,
let us know when you push to pypi so we can simplify the devsim installation for gdsfactory

Hi @joamatab
It should be done by this weekend.  I will  update this ticket accordingly.

Hi @joamatab,
I have created the 2.3.0-rc2 release, and it can be installed using:
pip install --pre

One remaining issue is that the MKL is not found in the Ubuntu 22 python 3 environment, unless I set the LD_LIBRARY_PATH to $VIRTUAL_ENV/lib.  Do you know if there is a way to set the environment in this situation?
I am still on track for a release this weekend, after updating the documentation.

Hi @joamatab , I have made a release.  Please let me know if you encounter any issues.

while this release is a good start, I still need to work at getting the directory structure to be properly respected  in devsim_data.  globbing files results in files all in the same directory

This appears to be fixed in 2.3.2, aa7e166e5c

Hi Juan,
pip install devsim works great on linux

Thanks @joamatab for testing it out.  The latest patch releases are just to fix the metadata and the relative links on the pipy project page.


[Bug]: HTML Coverage does not understand template strings

Version
27.1.1
Steps to reproduce

clone https://github.com/hasezoey/jest-template-string-repro
cd into the repository
run yarn install
run yarn run test
open coverage/lcov-report/some_source.js.html in a browser (in my case it was firefox)

Expected behavior
To handle and show Template strings properly
Actual behavior
Template Strings are not recognized and syntax highlighting can go haywire (see complex repository / original problem repository)
Additional context
The original Project where i noticed this problem, is https://github.com/typegoose/typegoose/, in particular it is the errors.ts file
Steps for this complex repository:

clone the mentioned repository
cd into the repository
run yarn install
run yarn run test:coverage
open coverage/lcov-report/src/internal/errors.ts.html in a browser (in my case firefox)
look at the syntax highlighting going haywire (and also showing wrong values on the wrong lines?)

Environment
$ yarn envinfo --preset jest
yarn run v1.22.11
$ /home/hasezoey/Downloads/jest-test/node_modules/.bin/envinfo --preset jest

  System:
    OS: Linux 5.14 Manjaro Linux
    CPU: (8) x64 Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz
  Binaries:
    Node: 16.8.0 - /tmp/yarn--1631453447308-0.8985475973800452/node
    Yarn: 1.22.11 - /tmp/yarn--1631453447308-0.8985475973800452/yarn
  npmPackages:
    jest: ^27.1.1 => 27.1.1

Done in 0.18s.

updated reproduction repository to jest 28, where it still happens (added more example cases)
to clarify, it seems like somehow the template string is not seen as a string and applies colors based on keywords (like undefined is a different color from and, even though it is not in ${})

updated reproduction repository to jest 29(.5.0), where the same result can still be observed
example screenshot, i would expect the template string to be the same color before and after the ${} and to not highlight undefined / null while not being in a ${} and also not change the color if a normal string is inside the template string:

still a issue with jest 29.7.0.
also updated the reproduction repository to that version.


Incorrect text wrapping position

describe:
When the text exceeds one line, the position of the text in the generated picture is out of order
Bug reports:
Rendered dom:

dom to image by html2canvas:

Specifications:

html2canvas version tested with: v1.3.2
Browser & version: safari
Operating system: ios 13.6.1

同样问题，请问你是如何解决的

https://github.com/niklasvh/html2canvas/issues/2648
If i change version to 1.0.0 everything works as expected
如上所述，使用 1.0.0 版本问题就解决了~
https://github.com/niklasvh/html2canvas/releases/tag/v1.0.0

用 1.0.0 版本还是有其他问题，脑壳疼(-｡-;。

我用1.0.0直接空白了，换了好多版本，多多少少都有问题

我用1.0.0直接空白了，换了好多版本，多多少少都有问题

我暂时换了 dom-to-image，不过也有其他问题

我用1.0.0直接空白了，换了好多版本，多多少少都有问题

我暂时换了 dom-to-image，不过也有其他问题

我在issue中看见了这样一个解决方案，每个字符都加上span标签，亲测有效

After debugging, I found that the bug is in this file https://github.com/niklasvh/html2canvas/blob/master/src/css/layout/bounds.ts In Safari, the first character after the first line behaves strangely.  When measure it, getClientRects() return two DOMRect and the first is empty and incorrect (It's on the end of last line).
But bounds.ts now just take the first DOMRect....

Thank you! You just saved my day!
I'll build a tweaked version for my own use, and waiting for the fix.
Here is how I tweak it:

Thanks, you saved my life


control-sidebar overflow issue

If you change the tab in the control-sidebar and the content exceeds the height of the page you are unable to scroll down to view the rest of the content in the sidebar and it simply remains hidden.
This is how you can fix it:
$('.control-sidebar a[data-toggle="tab"]').on('shown.bs.tab', function () {
    $.AdminLTE.layout.fix();
});

@SplicePHP Thank you very much!

Due to some neglect in my part, many issues weren't addressed for some time. Some of these issues are no longer relevant since a solution might have been found. Therefore, we decided to close all issues that are over a month old. However, if the issue still persists and you'd like help, then by all means repost your issue and we will consider it again. In the mean time, we will close this post.
Thanks for your understanding and we apologize for any inconvenience <3.


Fix when reloading station after selecting it again from the station …

…list caused the stream to reload.
Steps to reproduce:

select a station from the station list
arrive to the Now Playing screen
go back to the previous screen
select the previously selected station

Actual result: the station reloads causing a short pause in the stream
Expected result: there's no pause to the stream, transitioning back to the Now Playing screen works  like tapping the Now Playing button.

@hevi-public Thanks for the PR 👍


datefmt: init at 0.2.1

datefmt is a simple C program that formats unix timestamps in text streams
Signed-off-by: William Casarin<EMAIL_ADDRESS>

Result of nixpkgs-review pr 148822 run on x86_64-linux 1

1 package built:

datefmt

Result of nixpkgs-review pr 148822 run on x86_64-darwin 1

1 package built:

datefmt

@legendofmiracles @sikmir done

Result of nixpkgs-review pr 148822 run on x86_64-linux 1

1 package built:

datefmt

On Tue, Dec 07, 2021 at 04:58:43AM -0800, 0x4A6F wrote:
@.*** approved this pull request.

You could add testVersion (implemented in pkgs/build-support/trivial-builders.nix) with datefmt, testVersion in scope:
passthru.tests = testVersion { package = datefmt; };

what does this do?

From pkgs/build-support/trivial-builders.nix:

Checks the command output contains the specified version
Although simplistic, this test assures that the main program
can run. While there's no substitute for a real test case,
it does catch dynamic linking errors and such. It also provides
some protection against accidentally building the wrong version,
for example when using an 'old' hash in a fixed-output derivation.

This test can be run with nix-build -A pkgs.datefmt.tests.version
and is just a simple datefmt --version |& grep -Fw 0.2.1 in this case.
There is also a treewide pr which will add this in the future:
--- a/pkgs/tools/misc/datefmt/default.nix
+++ b/pkgs/tools/misc/datefmt/default.nix
@@ -1,4 +1,4 @@
-{ lib, stdenv, fetchurl }:
+{ lib, stdenv, fetchurl, datefmt, testVersion }:

 stdenv.mkDerivation rec {
   pname = "datefmt";
@@ -11,6 +11,8 @@ stdenv.mkDerivation rec {

   makeFlags = [ "PREFIX=$(out)" ];

+  passthru.tests.version = testVersion { package = datefmt; };
+
   meta = with lib; {
     homepage = "https://jb55.com/datefmt";
     description = "A tool that formats timestamps in text streams";

oh that sounds useful. I'll add this.

Thanks!