wsl Windows Subsystem for Linux Preview shutdown on opening

Version
Microsoft Windows [version 10.0.22621.674]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
kali-linux (today Ver)
Other Software
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 348: getpwuid(1000) failed 2
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 352: getpwuid(0) failed 2
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 559: execvpe /bin/sh failed 2
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 568: Create process not expected to return
Repro Steps
hello
so i wanted to install kali-linux, so in the microsoft store i install
Windows Subsystem for Linux Preview
and i install right after from the microsoft store
kali linux
when i tried tu run Kali one windows open and desapear as soon
so i tried to ope the Windows Subsystem for Linux Preview from the store and very quick a window open and close with a message i could not read, so i had to film it and make a pause and write it
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 348: getpwuid(1000) failed 2
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 352: getpwuid(0) failed 2
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 559: execvpe /bin/sh failed 2
<3>WSL  (104)  ERROR: CreateProcessEntryCommon: 568: Create process not expected to return
finally i can't run anything because the windows alway shut down.
What i have done before ?
i was on windows 10, i install Windows Subsystem for Linux Preview and kali and run kali whiout problem. i uninstall them, i update windows 10 to 11 and i install Windows Subsystem for Linux Preview and kali, as the documentation said i change wsl1 to wsl2 and i tried to open kali and problem happen
Thanks for reading all of this
and english is not my language so sorry for the mystakes.
Expected Behavior

Actual Behavior
done up
Diagnostic Logs
No response

the picture is nt good because the windows shutdown quick, i had to film and to pause -- lecture -- pause --lecture -- pause to see the message

/logs

Hello
What is /log ?
Thanks
Le lun. 17 oct. 2022 06 h 36, Blue @.***> a écrit :

/logs
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/WSL/issues/9018#issuecomment-1280650303,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEI7RQDBRBFR4LJRKPMMPN3WDUT4NANCNFSM6AAAAAARFV7IQE
.
You are receiving this because you authored the thread.Message ID:
@.***>

hello
not sure how it works, i execute and here the zip log, but in the power shell it's been written that i have to reproduce the probleme, but it s difficult because it s when i try to open wsl from the microsoft store. maybe there is a command line to open and save a log file in the same time ?
thanks for your answer.
WslLogs-2022-10-17_09-51-39.zip

Make sure your distro is set as default. I had some issues where my default distro was assigned to my docker desktop.
open your terminal and type "wsl -l -v" to list your distros then something like this:
wsl --setdefault DISTRO-NAME
Not sure if this will fix your problem but its worth a try,

Looks like WSL is configured to launch a user that doesn't exist. I'd suggest running wsl.exe --user root and figuring out what user accounts are actually present in the distro.

Looks like WSL is configured to launch a user that doesn't exist. I'd suggest running wsl.exe --user root and figuring out what user accounts are actually present in the distro.

i running wsl.exe --user root and here the answer :
C:\WINDOWS\system32> wsl.exe --user root
Processing fstab with mount -a failed.
<3>WSL (8) ERROR: CreateProcessEntryCommon:327: getpwnam(root) failed 2
Utilisateur introuvable.
Error code: Wsl/WSL_E_USER_NOT_FOUND
PS C:\WINDOWS\system32>

utilisateur introuvable = user unknown

Make sure your distro is set as default. I had some issues where my default distro was assigned to my docker desktop. open your terminal and type "wsl -l -v" to list your distros then something like this:
wsl --setdefault DISTRO-NAME
Not sure if this will fix your problem but its worth a try,

here the answer
C:\WINDOWS\system32> wsl --setdefault kali-linux
L’opération a réussi.
l'operation a reussi = it worked
but still the same problem

C:\WINDOWS\system32> wsl --system -d kali-linux-u root cat /mnt/wslg/distro/etc/passwd
Il n’existe aucune distribution avec le nom fourni.
Error code: Wsl/Service/WSL_E_DISTRO_NOT_FOUND
Il n’existe aucune distribution avec le nom fourni. = no dristo related to the name given

@osteopathievilleraymontreal Sorry I made a typo.
Can you please share the output of wsl --system -d kali-linux -u root cat /mnt/wslg/distro/etc/passwd instead ?

Ok that makes sense. It looks like your distribution is corrupted and if /etc/passwd is missing a lot of things won't work.
The easiest way to fix this would be to uninstall and reinstall the distribution. To do this you can use wsl.exe --unregister kali-linux.
If you need to recover files from your distribution before deleting it, you can try \\wsl.localhost, or if that doesn't work, you can mount the distribution vhd in another distribution with wsl --mount

Hi please someone help. I tried installing wsl using the command wsl --install on cmd and i keep getting A system shutdown is in progress. My PC meets the requirement as stated on learn.microsoft. I dont know whats goin on

Windows é um lixo!

The problem is that the only WSL installed is the one provided by Docker.
You should instead install Ubuntu or Debian from the Marketplace and then follow there instructions: https://github.com/microsoft/WSL/issues/5923

Im facing this error, after following an online solution to run composer in bash with Windows PHP and composer settings (Changing Distribution WSL version and etc).
But after I change the registry Flags to decimal 15 for : Computer\HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows\CurrentVersion\Lxss(DistributionUID)\Flags
Problem solved, my Ubuntu running again via WSL
I dont know if you are facing the same problem, but hope it can lead to something with your problem.

Hi you can try command ' wsl -l ' see
Windows Subsystem for Linux Distributions defalut setting
if docker-desktop (Default), you can command ' wsl --setdefault DISTRO-NAME' change defalut setting to wish linux

Ok that makes sense. It looks like your distribution is corrupted and if /etc/passwd is missing a lot of things won't work.
The easiest way to fix this would be to uninstall and reinstall the distribution. To do this you can use wsl.exe --unregister kali-linux.
If you need to recover files from your distribution before deleting it, you can try \\wsl.localhost, or if that doesn't work, you can mount the distribution vhd in another distribution with wsl --mount

This helped solve my problem thank you so much 🙏🙏

This helped solve my problem thank you so much

uninstall and reinstall the distribution
wsl.exe --unregister Ubuntu
This helped solve my problem thank you so much

Im facing this error, after following an online solution to run composer in bash with Windows PHP and composer settings (Changing Distribution WSL version and etc).
But after I change the registry Flags to decimal 15 for : Computer\HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows\CurrentVersion\Lxss(DistributionUID)\Flags
Problem solved, my Ubuntu running again via WSL
I dont know if you are facing the same problem, but hope it can lead to something with your problem.

Thanks for the info. It did work for me but I already had a hypervisor installed (Oracle VM).

Make sure your distro is set as default. I had some issues where my default distro was assigned to my docker desktop. open your terminal and type "wsl -l -v" to list your distros then something like this:
wsl --setdefault DISTRO-NAME
Not sure if this will fix your problem but its worth a try,
You are the single greatest human in the world!!! Thank you so much for this

Make sure your distro is set as default. I had some issues where my default distro was assigned to my docker desktop. open your terminal and type "wsl -l -v" to list your distros then something like this:
wsl --setdefault DISTRO-NAME
Not sure if this will fix your problem but its worth a try,

You are the single greatest human in the world! Thank you so much for this!!!

For me, restarting the subsystem did the job

Run wsl -l -v to find the default distro:

  Ubuntu                 Running         2
* docker-desktop         Stopped         2
  docker-desktop-data    Stopped         2

First mistake is that docker-desktop was set as default. Run wsl --set-default {YOUR_DISTRO}
If you still get errors, that may be because YOUR_DISTRO is running like in my case.
Run wsl -t YOUR_DISTRO
Reconnect. Everything should work now.

Make sure your distro is set as default. I had some issues where my default distro was assigned to my docker desktop. open your terminal and type "wsl -l -v" to list your distros then something like this:
wsl --setdefault DISTRO-NAME
Not sure if this will fix your problem but its worth a try,

Isso funcionou para mim.
Muito obrigado!

我遇到了同样的问题，解决办法：关闭linux子系统，卸载Docker-Desktop，并且删除wsl的更新包，安装重新打开linux子系统，先安装ubuntu，然后再安装Docker-Desktop。具体原因应该是Docker-Desktop导致的。

Certifique-se de que sua distribuição esteja definida como padrão. Tive alguns problemas em que minha distribuição padrão foi atribuída à área de trabalho do Docker. abra seu terminal e digite "wsl -l -v" para listar suas distros e algo assim:
wsl --setdefault DISTRO-NAME
Não tenho certeza se isso resolverá seu problema, mas vale a pena tentar,

Genial, já estava sem opções para resolver o problema.
Obrigado!


Improve error message on manifest schema mismatch

Fixes #1322.
But I'm really unhappy with it.  Putting it out as a draft in the hope that someone has an insight on how to simplify it (and make it more test-friendly!).

https://github.com/lann/spin/commit/eb4e6dbeb0f744e1533b6782ebd0be6ef6a0ad9c


Clean up naming supervisord-image -> init-image

This image contains more than supervisord, this updates naming and paths to be more generic to reflect current use.

clean up some unnecessary lines from Dockerfile
use  /opt/odo-init/ instead of /opt/supervisord inside the initContainer image
use /opt/odo instead of /var/lib/supervisord insde the builder container

Ping @girishramnani @mohammedzee1000 @cdrage

Ping @girishramnani @mohammedzee1000 @cdrage

@kadel Make sure that travis CI should pass

@kadel Make sure that travis CI should pass

It won't pass as it requires changes in https://github.com/openshift/odo/pull/2027. And tests for https://github.com/openshift/odo/pull/2027 won't pass without this :-D Chicken and Egg problem.
The less evil is to merge this first, as it won't break any existing code on odo side.

/lgtm


Warnings in 2021.12 beta

HA Startup log:
2021-12-08 08:46:42 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that accessed discovery_info['hostname'] instead of discovery_info.hostname; this will fail in version 2022.6. Please report issue to the custom component author for mopidy using this method at custom_components/mopidy/config_flow.py, line 110: mdns_address = discovery_info["hostname"][:-1]

There is no rush, time until June 2022. :smile:

Thank you for reading the debug logs ;)
Will fix soon.

Any progress on this? I know that HA version 2022.6 is not coming anytime soon yet, but it bothers my OCD ( :smile ) to have this warning recorded in my log all the time.
And btw it's not only hostname, most likely the same issue will appear with "host" and "port" and probably something else also...

I'm really, really, really sorry about keeping your OCD anxious... We really don't want that ;)
I believe I have fixed the code, feel free to test and provide feedback!

Thanks for the fix.
Minor issue: the fix doesn't seem to be included in the 1.4.2 release (commit 1ad58ac, before the fix). I noticed it cause I upgraded via HACS and still getting the warning.

This is weird.. I thought I added the tag after merging the PR...

Bumped the version to 1.4.3 and all looks good now
thanks for raising this, @chatziko

The update worked fine, thanks.
On the other hand, I'm getting this now in the log:

Logger: homeassistant
Source: custom_components/mopidy/config_flow.py:136
Integration: Mopidy (documentation)
First occurred: 11:22:47 AM (1 occurrences)
Last logged: 11:22:47 AM
Error doing job: Task exception was never retrieved

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/discovery_flow.py", line 74, in _async_process_pending_flows
    await gather_with_concurrency(
  File "/usr/src/homeassistant/homeassistant/util/async_.py", line 170, in gather_with_concurrency
    return await gather(
  File "/usr/src/homeassistant/homeassistant/util/async_.py", line 168, in sem_task
    return await task
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 203, in async_init
    flow, result = await task
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 230, in _async_init
    result = await self._async_handle_step(flow, flow.init_step, data, init_done)
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 325, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/config/custom_components/mopidy/config_flow.py", line 136, in async_step_zeroconf
    getattr(discovery_info, CONF_NAME)[: len(getattr(discovery_info, CONF_TYPE) +1) * -1]
TypeError: can only concatenate str (not "int") to str

Thanks for raising this.
I fixed it, but haven't updated tags or release.

Thanks! I'll try it when I find some time.


add docs for config env

I think it's quite a neat way to change the meta.env, so I've also added a couple of mentions.


[RFE] ability to accept a reference to an operator catalog by a file based catalog directory/file in addition to actual image

In looking at the imageset-config-ref.yaml file today it looks like the operators section contains an array of operator catalogs and requires the catalog to be a container image.  It would be a great addition if the catalog could also point to a file based catalog directory or json yaml file without needed to actually be "baked" into a container image
Today
mirror:
    - catalog: registry.redhat.io/redhat/redhat-operator-index:v4.8 # References entire catalog
      headsOnly: true # References latest version of each operator in catalog (true is the default value and can be omitted)

New feature
mirror:
    - catalog: file://<directory> or actual fbc.json/yaml file # References entire catalog
      headsOnly: true # References latest version of each operator in catalog (true is the default value and can be omitted)

I like the flexibility that this RFE suggests. To overcome the challenge that @jpower432 highlights, I suggest the following approach:
Create a new nested struct called something like raw_fbc, that has all of the same elements as the existing catalog struct, but with elements that allow a user to specify a raw fbc and a base image to build it with: https://github.com/openshift/oc-mirror/blob/41752468247a9337091b5979faee915154633cc4/pkg/config/v1alpha2/config_types.go#L65
I consider this to be a very advanced use case, so creating a new struct is appealing to not disrupt the selection mechanisms within the current catalog struct.
Thoughts @jpower432 @joehuizenga ?

@afflom @jpower432 sorry for the delay,  I am still wondering if it makes it cleaner to keep the idea of the catalog input to be  either an image or file and then add some sort of new struct called something like build_info and this struct could contain things like the opm image to use and maybe other elements like the catalogSourceName, or maybe even a dockerfile to use to build the new catalog?

Hi @afflom @jpower432 just circling back to this topic as I think 4.11 feature freeze is sometime this month.  We were hoping not to have to build "intermediate" catalog images just to drive the oc-mirror cmd if possible. I could take a pass at providing more details if that would help

Hi @joehuizenga, my apologies for missing your previous message. I do not believe that we will be able to fit this feature into our 4.11 development without an external contribution. If possible, could you provide more information about the specific use case and proposed solution?
cc @dinhxuanvu

Hi @jpower432 @afflom @dinhxuanvu  here is a mocked up sample of an ISC with "buildInfo" added just to kinda get the idea.
We are internally discussing the option of making an external code contribution and wondering what the timeline for this code contribution would need to be to get it into 4.11. I think feature freeze is sometime towards the end of this month but when would the code need to be delivered?
  operators:
    - catalog: registry.redhat.io/redhat/redhat-operator-index:v4.8 # References entire catalog
      full: true # AllPackages can be set to pull a full catalog and must be set to filter packages
      packages:
        - name: rhacs-operator
          startingVersion: '3.67.0'
          channels:
            - name: 'latest'
    - catalog: file://<path> # References entire catalog as a directory vs an image
      full: true # AllPackages can be set to pull a full catalog and must be set to filter packages
      buildInfo: # Additional data used to build the catalog source image and manifest
        baseImage: registry.redhat.io/redhat/ose-opm.....:v4.8
        targetImageName: <your name>
        catalogSourceName: my-custom-name
      packages:
        - name: ibm-operator-a
          startingVersion: '3.67.0'
          channels:
            - name: 'v3.67'

We have a requirement where we need to be able to manipulate a FBC prior to using it as a catalog specified in a imageSetConfig. We wanted to avoid having to add "container image building" as part of our process of manipulating an FBC just so we could then use this intermediate image as input defined in an ISC to drive oc-mirror.
We are adding unique metadata to our initial FBC that we want to "pre-process" to help us reduce the number of relatedImages that need to be mirrored to the absolute minimum.  In our case, we have the same operator that can manage a set of different relatedImages based on what a customer has purchased,  so we are looking for a way to pre-process the comprehensive list of relatedImaged down to just what a customer is "entitled" to, not the full list.  We can do the pre-processing with a client based app by just manipulating the json. This json is still a smaller valid fbc and this is what we would like to use as the input to oc-mirror vs having to build a container image for it.
From prior discussion in this issue, it sounds like you expect a catalog image so you can inspect it to identify the opm base image you need to build up a catalog image for disconnected. I am proposing that if we include something like "buildInfo" as an alternate way to provide this information.
I am also proposing that we provide a way to also allow the ISC to define both the catalogSourceName and target imageName in somthing like a "buildInfo" section.  We need this type of information to make in easier for customers that have already installed 1 to N catalogSources in their cluster and have active subscriptions created with very specific catalogSource names that we would like to control. If we have the ability to declare this catalogSourceName, we could preserve it and would not have to add some sort of subscription migration tooling/process to help a customer migrate.

@joehuizenga We were hoping to wrap up feature development by the end of this week so we can focus on polishing/testing next week. In my opinion, I would prefer not to expose the image-building functionality in the imageset config because I think it is out of scope for oc-mirror, but I would be happy to discuss a middle ground. Would it be possible to just vendor in the oc-mirror libraries to accomplish the image building in your solution? As for changes in oc-mirror, perhaps just setting the target image name and catalogSource?
@afflom @dinhxuanvu Any thoughts?

@jpower432 Thanks for the update, trying to digest,  what does this mean in regards to being able to specify a catalog elements as a file vs image?  Do you consider that "image-building functionality"?
  operators:
    - catalog: registry.redhat.io/redhat/redhat-operator-index:v4.8 # References entire catalog
    - catalog: <catalog.json>

@joehuizenga I would consider specifying the base image and content as image building, but not the addition of the file. The only problem there is there if you can't really use the file without the base image. I have submitted PR #432 to address some parts of this request, specifically customizing the target catalog name and tag and the catalog source name.

@joehuizenga This makes sense to me. The only concern I still have is that this opens up the possibility that the input does not produce a valid catalog due to an invalid catalog and override combination. It would be difficult to determine the validity before the catalog was being consumed or mirrored. If this is being used in CI, would a suitable workflow be to build the catalog image locally and push it to a local registry and have that as an input to oc-mirror?

@jpower432 Here is a simple diagram we can use at the oc-mirror working group tomorrow

Example of the additional steps necessary if we don't have a way to specify a file based catalog within a local directory:

View of our progression to oc-mirror

/remove-lifecycle stale


Update first new map notification message

Changing the action button on the notification to read "View your AppMaps"

:tada: This PR is included in version 0.50.0 :tada:
The release is available on:

v0.50.0
GitHub release

Your semantic-release bot :package::rocket:


Auto Update to latest version sometimes uninstalls the existing version, but it wont install the latest version. It keeps on showing the error somthing like this "some vscode instance is running please close and retry "

VSCode Version:
OS Version:

Steps to Reproduce:
Auto-updates from vscode will uninstall the current version and it keeps on showing the error something like this "some vscode instance is running please close and retry "
I have to download the latest version every time and install after it is uninstalled
At least user should be given the path of the updater to install after it is uninstalled.

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

/duplicate #47841
As a workaround, please disable Windows background updates by setting update.enableWindowsBackgroundUpdates to false.


WIP - adds some tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@762838c). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage        ?   83.22%
=======================================
  Files           ?        5
  Lines           ?      155
  Branches        ?        0
=======================================
  Hits            ?      129
  Misses          ?       26
  Partials        ?        0

Flag
Coverage Δ

unittests
83.22% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Cervus nippon Silhouette - Broken Link

There appears to be one available silhouette for Cervus nippon, but the path linked to this silhouette (https://www.phylopic.org/images/dbf886d8-42f3-4d7b-8a93-45d4b3e72f9c/cervus-nippon) leads to an error page (photo attached). Is this silhouette no longer available? Thanks in advance!

Works now, but I have noticed something like this one other time, when I was unable to access the page for Anura . I think there might be a very infrequent issue with API errors (timeouts) getting cached in the website layer. Hard to troubleshoot, given how rare it is, but I'll keep an eye out for it.

This is the same as this issue, I'm pretty certain: https://github.com/keesey/phylopic/issues/23


disable turbo checking for large amount of data

Signed-off-by: Ke Xu<EMAIL_ADDRESS>
Description

Areas changed

npm packages changed:

[ ] responsibleai/causality
[ ] responsibleai/core-ui
[ ] responsibleai/counterfactuals
[ ] responsibleai/dataset-explorer
[ ] responsibleai/fairness
[ ] responsibleai/interpret
[ ] responsibleai/localization
[ ] responsibleai/mlchartlib
[ ] responsibleai/model-assessment

Python packages changed:

[ ] erroranalysis
[ ] raiutils
[ ] raiwidgets
[ ] rai_core_flask
[ ] responsibleai

Tests

[ ] No new tests required.
[ ] New tests for the added feature are part of this PR.
[ ] I validated the changes manually.

Screenshots (if appropriate):
Documentation:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Codecov Report

Merging #1351 (182f192) into main (3b58c74) will decrease coverage by 5.70%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1351      +/-   ##
==========================================
- Coverage   86.13%   80.43%   -5.71%
==========================================
  Files          99       18      -81
  Lines        4509      322    -4187
==========================================
- Hits         3884      259    -3625
+ Misses        625       63     -562

Flag
Coverage Δ

unittests
80.43% <ø> (-5.71%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

responsibleai/responsibleai/_config/__init__.py

raiwidgets/raiwidgets/responsibleai_dashboard.py

...nalysis/erroranalysis/_internal/version_checker.py

...bleai/responsibleai/_tools/causal/causal_result.py

...s/erroranalysis/_internal/error_report/__init__.py

...tools/error_analysis/dashboard_schemas/__init__.py

...ponsibleai/responsibleai/rai_insights/constants.py

...ranalysis/erroranalysis/_internal/matrix_filter.py

...ibleai/responsibleai/managers/explainer_manager.py

erroranalysis/erroranalysis/__init__.py

... and 71 more

Continue to review full report at Codecov.

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


GEODIFF Stuck at make

I'm stuck on the make part of step number 4 as explained in the installation instructions. The error log is attached.

Hi @maharshirajbhandari , which OS do you use? You need to use sqlite3 with extensions allowed.


Iframe rendering under websurface

If the Aframe scene is embedded and there's HTML code over the Aframe scene, the iframe will render under the web surface

Also having this issue, looking for a way to correct his in the application

You can fix this by adjusting your z-index and/or your div positioning.  In my case, I made my divs fixed (absolute) where possible.  Like nav bars as an example.


Accelerating Taiko Transactions: A Performance Optimization Proposal

Describe the feature request
Project Name: Taiko Speed Boost
Abstract:
Taiko, a popular transaction library for Go, is renowned for its ease of use and expressiveness. However, optimizing transaction performance can be crucial for high-throughput applications. This proposal outlines several strategies to significantly accelerate Taiko transactions, enhancing the overall user experience for concurrent and demanding scenarios.
Specifications:

Profiling and Bottleneck Identification:

Conduct a thorough performance profiling using tools like pprof, go tool vet, and Taiko's built-in profiling features to pinpoint performance bottlenecks.
Analyze database queries, network calls, and code execution to identify areas for optimization.

Database Optimization:

Optimize SQL queries:
Utilize prepared statements to avoid query parsing overhead.
Employ indexes judiciously to minimize database scans and joins.
Consider alternative database architectures (e.g., NoSQL) for suitable use cases.
Connection pooling:
Implement connection pooling to reduce the cost of establishing new database connections.
Leverage connection management libraries like github.com/gocql/gocql for efficient database interactions.
Batching:
Combine multiple database operations into batches to minimize round trips and network overhead.
Explore Taiko's built-in batching capabilities or consider external libraries like github.com/jmoiron/sqlx.

Network Optimization:

Minimize network calls:
Fetch only the necessary data from the database, avoiding unnecessary data transfers.
Consider caching frequently accessed data to reduce database load and network trips.
Choose efficient protocols:
If applicable, employ protocols like gRPC or Protobuf for more compact and efficient data serialization.
Evaluate the use of Content Delivery Networks (CDNs) for geographically distributed applications.

Code Optimization:

Reduce unnecessary computations:
Avoid redundant calculations or data processing within transactions.
Leverage concurrency effectively using techniques like goroutines and channels when appropriate.
Optimize data structures:
Employ suitable data structures (e.g., maps instead of slices for frequent lookups) to enhance performance.
Consider memory allocation strategies to minimize overhead.
Review transaction logic:
Ensure transactions are atomic, consistent, isolated, and durable (ACID) while minimizing unnecessary complexity.
Break down large transactions into smaller, more manageable units if possible.

Leverage Taiko Features:

Parallelism:
Explore Taiko's built-in parallelism features for concurrent execution of database operations where appropriate.
Be mindful of potential data consistency issues and implement necessary synchronization mechanisms.
Caching:
Utilize Taiko's caching capabilities to store frequently accessed data within transactions.
Define appropriate cache invalidation strategies to ensure data freshness.
Other optimizations:
Keep Taiko updated to benefit from the latest performance improvements and bug fixes.
Consider using alternative transaction libraries like github.com/go-sql-driver/mysql or github.com/jackc/pgx if their performance characteristics are more suitable for your specific use case.
Evaluation Criteria:
Measure performance improvements using benchmarks and real-world scenarios.
Consider factors like transaction size, concurrency, and database load.
Prioritize optimizations based on their impact and feasibility.
Conclusion:
By implementing these optimization strategies, we can significantly enhance Taiko transaction performance, making it an even more compelling choice for building high-performance Go applications. We welcome feedback and collaboration from the community to make Taiko even faster and more efficient.
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

Reported for spam


node_exporter "Create textfile collector dir" task - Consistent "change" state

Description
The "Create textfile collector dir" task in the node_exporter role for the Prometheus Ansible collection is consistently reporting a "changed" state upon every run. The task in question
Other system services generate their own metrics and place the corresponding metric files in the "{{ node_exporter_textfile_dir }}" directory. These services belong to the "{{ node_exporter_system_group }}" group, enabling them to change the group ownership of created metric files to the desired "{{ node_exporter_system_group }}". The node-exporter can read from these metric files, but the services are unable to change the ownership to the "{{ node_exporter_system_user }}". This results in a "changed" state for the "Create textfile collector dir" task after every run, as long as there are new or updated metric files within the "{{ node_exporter_textfile_dir }}" directory.
Problem
This issue is problematic because I monitor Ansible changes for security reasons and to detect any unexpected changes. Constant changes are undesirable and should not occur under normal circumstances.
Expected Outcome:
When all files and directories within the node_exporter_textfile_dir directory have the correct group set, no Ansible changes should be reported. The role should neither alter the "user" for files nor report any changes regarding this.
Solution Proposal
I have tried to come up with a suitable solution but have been unsuccessful. Unfortunately, there is no associated PR. Possible solutions could include:

Setting recurse: false - This would set the desired ownership for the directory itself, allowing the user to manage and verify that text metric files have the correct group, which node-exporter can read from.
Using changed: false - This option is also not ideal.

I am unsure how to address this issue and welcome any suggestions or proposals for a resolution.

imo the solution with setting recurse: false should work, since, according to the source, collector only scrapes files in this directory, excluding subdirectories and g+rwX,o=rX guarantees, that all files created in the directory will have the desired group

I suspect that the recursive option was set as a safety precaution to ensure that node_exporter can read the files in the directory.
But as you are experiencing there are different scenarios where node_exporter can very well read the directory contents, without enforcing the user/group/mode that the task sets.
A better solution would be to confirm that the user can read the contents, with the ansible.builtin.stat module and become and then only change the permissions if the node exporter user can't access the files.
But to avoid over complicating the role we can just simply drop the recurse option.
The role isn't installing anything in that directory and if it were, then we would just make sure that those files were installed with the correct permissions.


Fix strategies bug in pandas==1.4.0

Describe the bug
The strategies module errors out for pandas==1.4.0 for the pandas-native string dtype, see here for an example.

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest version of pandera.
[ ] (optional) I have confirmed this bug exists on the master branch of pandera.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Error
self = <StringArray>
['', '', '', '', nan]
Length: 5, dtype: string

    def _validate(self):
        """Validate that we only store NA or strings."""
        if len(self._ndarray) and not lib.is_string_array(self._ndarray, skipna=True):
>           raise ValueError("StringArray requires a sequence of strings or pandas.NA")
E           ValueError: StringArray requires a sequence of strings or pandas.NA

Expected behavior
There shouldn't be an error when synthesizing dataframes with a schema that uses the pandas string dtype.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

fixed by #748


TomEE MP Support

I changed the description to be Apache TomEE 8.0.0-M2. This will be in line with TomEE download page: http://tomee.apache.org/download-ng.html
Hope it is ok like this.

Hi Roberto,
The link from the index.html page to the 'system' URLS (metrics, Health, openAPI, ...) are not correct.  Please add the following patch to your PR.
Thx.
Tomee.patch.txt

Done! Thank you!


Fix bug: deleting wrong task after tlist

Fixes #252

Codecov Report

Merging #254 (da15bca) into master (f08a114) will decrease coverage by 0.08%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #254      +/-   ##
============================================
- Coverage     49.47%   49.39%   -0.09%
+ Complexity      559      557       -2
============================================
  Files           124      124
  Lines          2565     2555      -10
  Branches        355      355
============================================
- Hits           1269     1262       -7
+ Misses         1215     1212       -3
  Partials         81       81

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/Model.java
57.14% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
58.33% <ø> (+3.07%)
:arrow_up:

...seedu/address/logic/commands/task/TdelCommand.java
100.00% <100.00%> (ø)

src/main/java/seedu/address/model/module/Name.java
90.90% <0.00%> (-9.10%)
:arrow_down:

...java/seedu/address/model/module/task/TaskList.java
59.67% <0.00%> (-3.23%)
:arrow_down:

Continue to review full report at Codecov.

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


⚠ Bofrost.lu has degraded performance

In ee82eb1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1362 ms

Resolved: Bofrost.lu performance has improved in 070fd05 after 13 minutes.


Reason Frame Format is not implemented

I am trying to understand why Frame Format is not included in the implementation. Would a file compressed by this implementation be decompressed by someone without knowing any details about how it was compressed as long as it was LZ4 compression?
Here is my understanding:

This implementation is cross compatible with the CLI of LZ4. This means, the person decompressing a file compressed with this implementation will need to know details that otherwise would have been in the Frame and block Header?
Other implementations that don't get this info from the user and try to find it in the compressed file won't be able to decompress it?

Frame format is necessary to let anyone decompress a file without knowing anything about it. Is that a fair understanding?

Nevermind, I found the function makeHeader which seems to be creating the Frame Header.


Exception on /api/v1/orchestrations/pipeline-schedules AttributeError: 'NoneType' object has no attribute 'get'

I get this error sometimes when strating meltano ui on a windows machine.  The error also happens when I navigate between tabs in the UI.  The error is associtated with the following API call in the log
Exception on /api/v1/orchestrations/pipeline-schedules [GET]

Here is the ending bit of the error
  File "C:\development\meltano\src\meltano\core\project_plugins_service.py", line 83, in current_plugins
    self._current_plugins = self.config_service.current_meltano_yml.plugins
  File "C:\development\meltano\src\meltano\core\config_service.py", line 35, in current_meltano_yml
    self._current_meltano_yml = self.project.meltano
  File "C:\development\meltano\src\meltano\core\project.py", line 230, in meltano
    return MeltanoFile.parse(self.project_files.load())
  File "C:\development\meltano\src\meltano\core\project_files.py", line 66, in load
    included_file_contents = self._load_included_files()
  File "C:\development\meltano\src\meltano\core\project_files.py", line 150, in _load_included_files
    for path in self.include_paths:
  File "C:\development\meltano\src\meltano\core\project_files.py", line 59, in include_paths
    include_path_patterns = self.meltano.get("include_paths", [])
AttributeError: 'NoneType' object has no attribute 'get'

It can't find the attribute get which I think is suppose to be a function of the class ?  I also have not been able to determine what the function get is suppose to do.  It seems like it is reading the meltano.yaml for the include_paths entries and return all the paths in a list.  In the project_files.py there are these back to back function you can see there is a meltano function.
    @property
    def meltano(self):
        """Return the contents of this projects `meltano.yml`."""
        if self._meltano is None:
            with open(self._meltano_file_path) as melt_f:
                self._meltano = yaml.safe_load(melt_f)
        return self._meltano

   @property
    def include_paths(self) -> List[Path]:
        """Return list of paths derived from glob patterns defined in the meltanofile."""
        include_path_patterns = self.meltano.get("include_paths", [])
        return self._resolve_include_paths(include_path_patterns)

In project.py there is a meltano function that returns a MeltanoFile class
    @property
    def meltano(self) -> MeltanoFile:
        """Return a copy of the current meltano config.

        Returns:
            the current meltano config
        """
        with self._meltano_rw_lock.read_lock():
            return MeltanoFile.parse(self.project_files.load())

The MeltanoFile class that is in meltano_file.py has a get_plugins_for_mappings.

@kgpayne @alexmarple looping you two in here since this seems to be a possible bug with the UI and/or multi-file YAML.
@BuzzCutNorman can you share a bit about your Meltano setup too? Version of meltano and python version along with your meltano.yml file?
Also, cc @visch since this is Windows 😄

@tayloramurphy sure thing.  I have python 3.9.12 installed.  Meltano was installed using pipx and is meltano, version 2.1.0 .  I get the error message even when I start meltano ui in a freshly initialized test project with the default meltano.yml file.
cc @visch @kgpayne @alexmarple

@aaronsteers adding to Engineering Assignments so we can get some eyes on it then we can figure out when to prioritize the fix.

File "C:\development\meltano\src\meltano\core\project_files.py", line 59, in include_paths
include_path_patterns = self.meltano.get("include_paths", [])
AttributeError: 'NoneType' object has no attribute 'get'

This points to self.meltano being None which I can't immediately explain per se.
@cjohnhanson - Can you take a look?

@BuzzCutNorman (cc: @aaronsteers) this may be a bit hard to fully debug without a windows machine to test on. But I was able to replicate the error and stacktrace by replacing my meltano.yml with an empty file. If the meltano.yml file isn't found in the project root, then meltano gives the informative "meltano ui must be run inside a Meltano project" error message, but if the meltano.yml is an empty file, there's no specific informative error that's thrown, it just parses the empty yaml into a None.
Do you get this same error when you try to run other commands in the same project where you get the error for the meltano ui command? E.g., if you get this error after running meltano ui if you then run meltano config in the same project, do you get the same error? If so, that might indicate that Meltano is finding an empty meltano.yml file in the directory that it considers to be the project root. Which could mean:

The project is somehow getting initialized with an empty meltano.yml
The meltano.yml file is somehow getting overwritten with a None object somewhere in the call stack
Meltano considers the project root to be a directory which either has an empty meltano.yml file in it OR the behavior of with open(<meltano file path>) as melt_f: and then yaml.safe_load(melt_f) behaves differently on Windows than on Unix machines, such that an empty file is created in whatever directory Meltano considers to be the project root.

@cjohnhanson (cc: @aaronsteers ) I can verify that after running 'meltano init --no_usage_stats test-meltano the project did get a populated meltano.yml .  This is the content of the test-meltano project's meltano.yml file after initialization:
version: 1
default_environment: dev
project_id: fcda620d-db88-4a3a-8c4f-aaf6b404e5a7
send_anonymous_usage_stats: false
environments:
- name: dev
- name: staging
- name: prod

I added a extractor tap-postgres to accomplish the the second requested test.  The new meltano.yml looks like this.
version: 1
default_environment: dev
project_id: fcda620d-db88-4a3a-8c4f-aaf6b404e5a7
send_anonymous_usage_stats: false
plugins:
  extractors:
  - name: tap-postgres
    variant: transferwise
    pip_url: pipelinewise-tap-postgres
    config:
      dbname: datawarehouse
      default_replication_method: FULL_TABLE
      filter_schemas: raw
      user: username
environments:
- name: dev
- name: staging
- name: prod

I started the meltano ui and once it opened in my browser I navigated from extractor tab to loader tab to pipeline tab the back until I got the error.  Once I got the error in another command prompt I ran the following command meltano config tap-postgres list which returned this output:
PS C:\development\test-meltano> meltano config tap-postgres list
2022-06-23T23:24:50.065452Z [info     ] Environment 'dev' is active
host [env: TAP_POSTGRES_HOST] current value: 'localhost' (default)
        Host: PostgreSQL host
port [env: TAP_POSTGRES_PORT] current value: 5432 (default)
        Port: PostgreSQL port
user [env: TAP_POSTGRES_USER] current value: 'username' (from `meltano.yml`)
        User: PostgreSQL user
password [env: TAP_POSTGRES_PASSWORD] current value: 'password' (from `.env`)
        Password: PostgreSQL password
dbname [env: TAP_POSTGRES_DBNAME] current value: 'datawarehouse' (from `meltano.yml`)
        Database Name: PostgreSQL database name
ssl [env: TAP_POSTGRES_SSL] current value: False (default)
        SSL: Using SSL via postgres `sslmode='require'` option. If the server does not accept SSL connections or the client certificate is not recognized the connection will fail
filter_schemas [env: TAP_POSTGRES_FILTER_SCHEMAS] current value: 'raw' (from `meltano.yml`)
        Filter Schemas: Scan only the specified comma-separated schemas to improve the performance of data extraction
default_replication_method [env: TAP_POSTGRES_DEFAULT_REPLICATION_METHOD] current value: 'FULL_TABLE' (from `meltano.yml`)
max_run_seconds [env: TAP_POSTGRES_MAX_RUN_SECONDS] current value: 43200 (default)
        Max Run Seconds: Stop running the tap after certain number of seconds
logical_poll_total_seconds [env: TAP_POSTGRES_LOGICAL_POLL_TOTAL_SECONDS] current value: 10800 (default)
        Logical Poll Total Seconds: Stop running the tap when no data received from wal after certain number of seconds
break_at_end_lsn [env: TAP_POSTGRES_BREAK_AT_END_LSN] current value: True (default)
        Break At End LSN: Stop running the tap if the newly received lsn is after the max lsn that was detected when the tap started

To learn more about extractor 'tap-postgres' and its settings, visit https://hub.meltano.com/extractors/tap-postgres--transferwise

I set --log-level debug when I started the UI here is the output from the console
buzzcutnorman-meltano-ui-console-none-type-20220623.log
and the meltano-ui.log file from the project.

@cjohnhanson (cc: @aaronsteers ) Yesterday I created a local branch of Meltano and added the following logger to line 51 of project_files.py to assist in trouble shooting the third scenario you mentioned.
logger.debug("project_files meltano property used")
With the logger added I started a poetry shell from the Meltano directory and navigated to the test-meltano project folder and ran meltano --log-level debug ui .  No matter what I do I can not get the error to happen.  If I do a ctrl + c and stop the ui then comment out the logger from project_files.py save, then start meltano --log-level debug ui I can get the error to occur by navigating down the tabs then back up them.

@cjohnhanson (cc: @aaronsteers ) Working more with this I found that the logger only helps if you have debug logging level in enabled.  Once that is taken away you are right back to the error occurring.  So the logger is just giving a slight amount of time for something to complete.  Which lead me to this line of thinking.  The error might occur when the ProjectFiles class is loaded and since during __init__ it sets
self._meltano = None
and the load() function which is being called does a reset_cache() which is really setting
self._meltano = None
happens really close together.  The self._meltano variable is getting cleared right when it is needed.  I tested this out by removing the logger and instead commented out line 65 in the project_files.py
#self.reset_cache()
and ran meltano ui.  I am not able to make the error occur just like when the logger was in place.  I think this might be an explanation as to why the logger was fixing the issue.

@BuzzCutNorman that's a great find, thanks for diving in deeper here.
Just to 100% confirm -- you're only seeing this behavior with the meltano ui command, correct?
If so, I think we can safely assume that this is a race condition between the two worker threads that refer to the same underlying Project which are run by meltano ui.
As far as I can tell, we wrap references to theProject.project_files property in an appropriate lock such that it should be thread safe. But it looks like we don't require a lock within the body of the project_files property itself. I think a quick solution to this would be to just wrap the body of the Project.project_files property method within a context manager which acquires the relevant read lock. CC: @aaronsteers and @pandemicsyn in case you have more insight here.
I'm both surprised we've never had reports of this behavior before and that it happens so consistently on your system. I'll get a PR with the change put together and @BuzzCutNorman once I do it'd be great if you could double check that it resolves the issue on your Windows machine, since we're having trouble replicating the issue elsewhere.

@BuzzCutNorman Glad you were able to test locally, I was working out how best to test on my end and was struggling to find a Windows machine that I could easily test on.
Go ahead and share results in the PR and we can continue the discussion there.

I can verify that I get this on Meltano with the Uvicorn feature flag enabled. I can't replicate this on Linux.

@visch I have concluded I am not the best at testing upstream PRs locally.  When you get a chance please test the PR for this issue so Cody can get a trustworthy Windows machine result.  Thank you in advance.

fyi - Took some grinding but was actually able to reproduce this race on my Mac as well, so not limited to Windows anything:
2022-06-30T18:15:10.409270Z [error    ] Exception on /api/v1/plugins/all [GET]
<!--->
    self._current_meltano_yml = self.project.meltano
  File "/Users/syn/projects/meltano/src/meltano/core/project.py", line 236, in meltano
    return MeltanoFile.parse(self.project_files.load())
  File "/Users/syn/projects/meltano/src/meltano/core/project_files.py", line 66, in load
    included_file_contents = self._load_included_files()
  File "/Users/syn/projects/meltano/src/meltano/core/project_files.py", line 150, in _load_included_files
    for path in self.include_paths:
  File "/Users/syn/projects/meltano/src/meltano/core/project_files.py", line 59, in include_paths
    include_path_patterns = self.meltano.get("include_paths", [])
AttributeError: 'NoneType' object has no attribute 'get'
2022-06-30T18:15:10.454269Z [info     ] Error: 500 Internal Server Error: The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

The most consistent way for me was just using one of the http bench mark tools and just hitting a few of the API endpoints as fast as possible.


Testing Mines: Mixes different types of AT or AP mines

Not sure if on purpose but could it be possible to create mine specific minefields (e.g. AT minefield with M4A1 SLAM mines only)?
P.S. Tested with ACE in the fix_Minefields branch in MP

At the moment they are either AT or AP mines. Why using mine types ("M4A1 SLAM") would make more sense than mines functionality ("AT mine")?


FXML LCD gauge

I've been porting a Java 7 JavaFX app using JFXtras-labs to Java 8 with Medusa 8.2.
When initializing the LCD gauge from FXML like this:
                    <Gauge fx:id="xLcd" skinType="LCD"                    title="Title" unit="mm" decimals="3" subTitle="Subtitle"                    minWidth="200" prefWidth="200" maxWidth="200"                    minHeight="75" prefHeight="75" maxHeight="75"                    oldValueVisible="false" minMeasuredValueVisible="false" maxMeasuredValueVisible="false"                    onMousePressed="#handleDroMouseClick">
I get this: https://drive.google.com/open?id=1T-GM9sCIIDnrwcGPD16dW98wlJr3WxMV
It seems to be ignoring the visibility settings and the subtitle for some reason.
I walked back versions and found that something changed between 5.9 and 6.0
In 5.9, this is the way it looks:
https://drive.google.com/open?id=1WtouIOdVH13fynH2K9QzcfW9w2NIXyJp
Obviously it's not quite right in 5.9 either,
It looks correct when being defined in java-land:
                    Gauge root = new Gauge(Gauge.SkinType.LCD);           root.setMaxMeasuredValueVisible(false);           root.setMinMeasuredValueVisible(false);           root.setOldValueVisible(false);           root.setTitle("A Axis Position");           root.setSubTitle("mm/min");           stage.setScene(new Scene(root,200,75));           stage.show();
https://drive.google.com/open?id=1x6T7D-QVzmWV937o3Ji6bbsaC9LgvZko

I've done some digging and I think it's down to Gauge#setSkinType inside the switch where it sets the visibility of those fields.
               setMinMeasuredValueVisible(true);
                setMaxMeasuredValueVisible(true);
                setOldValueVisible(true);
Stepping through the code visibility is set twice. Once on initialization using my own values, and then again when it sets the skinType, which overrides my values. Commenting out these lines works for me, but I didn't want to presume to make a PR not knowing why they are there.
As a workaround, I'm able to use the properties to set them after the fact, but this needs to be done outside of my controller's init and I haven't thought of a good way to do that.
Thanks

Sorry for the late reply but I was too busy. So the problem was related to a missing constructor that supports all or nearly all properties. This is needed to initialize a control properly via FXML. I've added one for the Gauge control and it looks like it fixes your issue. Could you please try the latest version from the repo?

No need to apologize. I appreciate you taking a look.  Your fix looks to have solved my problem.
Thanks so much.

That's good news :)


add logging

right now only http requests are being logged, we need to have more information about what's happening during the job execution

PR incoming tonight too.

#77


"The ASP.NET Core SignalR .NET client can be used by Xamarin" is wrong until VS2017 15.8 Update gets out

The Statement "The ASP.NET Core SignalR .NET client can be used by Xamarin" is wrong until VS2017 15.8 Update gets out. For current 15.7 the support is broken.
https://github.com/aspnet/Announcements/issues/305

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c9bc1072-be2c-94f6-49bf-daa9e97dff2d
Version Independent ID: 36052173-7061-6d01-8aed-915685cd7cf6
Content: ASP.NET Core SignalR .NET Client
Content Source: aspnetcore/signalr/dotnet-client.md
Product: aspnet-core
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@tdykstra thanks, update is now live and can help others. I took me some days to figure this out.


chore: add tbtc dockerfile to project

This adds the tbtc-node to the project and docker registry for consumption. Also updated the docker-compose file for running it locally.

This increases the build check times from ~3 minutes to 25 minutes. imho that is way too long for devs to get feedback on a push.

@macterra : Agreed. So should we just build this separately in its own repo?

@macterra : Agreed. So should we just build this separately in its own repo?

Yes, we should add github workflow to https://github.com/selfidhq/self-bitcoin-docker


Change of folder structure to make hugo behave

Moved GCP guide to separate folder, change name from dess-gcp.md to _index.md move all images to /dess-gcp/images/

@TylerTrott you should check this out, as the other guides need the same treatment so that they can look OK in GitHub markdown and once rendered by Hugo


Configure server group of the openstack VM

References:

https://github.com/kubermatic/machine-controller/pull/1169

Not required since this is not part of cloud-config


Bundle development

resolution_fixed   |    by<EMAIL_ADDRESS>
Ben called to tell me that Chris was having issues with his 'bundle' development, where changes we're making are conflicting with changes he's making.
In the first instance I think this will be helped by more communication about what we're doing, I'll add Chris to the trac mailing list so he has an idea of where our development is going

Issue migrated from trac:65 at 2024-01-31 17:58:19 +1100

<EMAIL_ADDRESS>changed _comment1 which not transferred by tractive

<EMAIL_ADDRESS>commented

Thanks Scott, I've done that now

<EMAIL_ADDRESS>commented

My point was simply that "accessdev" is not currently using our baseline Puppet framework which means we have to spend time backporting and testing any fixes that are requested.  I guess at some point we'll have to migrate "accessdev" onto that framework since it will become infeasible (if not already) to continue backporting stuff, but there will definitely be conflicts where you've implemented OS level configurations that the framework already provides; eg. firewall configuration just to name a simple example.

<EMAIL_ADDRESS>commented

Accessdev forked from the NCI puppet over a year ago, the problems in moving modules between them are undoubtedly unavoidable at this stage.
I see two main issues if we were to combining the forks - there's no clear boundary between accessdev and NCI development, and there's a lack of communication on what both sides are doing.
This would probably be better off left to a meeting, a couple questions I'd like us to have (documented!) answers to are:

How do the NCI and accessdev repos relate?

Is NCI an upstream of accessdev?
Is accessdev a module that gets plugged in to the NCI repo, only setting up rose, umui, etc?
Is NCI a module that gets plugged in to the accessdev repo to set up LDAP, mail relay, etc?

How do changes from NCI get into accessdev?

Are there documented releases from NCI that we can pull from?

How do we request & keep track of changes in the NCI repo?

This trac site is becoming very useful for our development, should there be a similar setup for NCI's side?

<EMAIL_ADDRESS>changed priority from major to blocker

<EMAIL_ADDRESS>commented

Putting what we discussed on 4/2 in writing (at least what I recall from it, feel free to correct) -
The NCI repository will be an upstream of accessdev - NCI will tag stable versions in their repository and ask us to pull from those.
NCI intend to put up some form of developer site to host documentation for the cloud systems
ETA for a stable v1.0 of the NCI bundles is 3-4 weeks (i.e. early march)
The NCI repository will handle low-level infrastructure common to anyone running at the NCI cloud site such as:

LDAP
Firewall
Mail
/g/data1 mounts
Logging to shared infrastructure
etc.

I'd assume that our expectations of the NCI repository are (again I'd welcome input from others):

NCI releases are stable & tested
Releases will be documented (change logs, user guides etc.)
NCI are responsible for any issues related to their modules & will provide an issue tracker (e.g. RT, trac.nci, etc) where we can submit bugs & feature requests
Documented ways will be provided for us to hook into the NCI module's functionality so we can create our services (e.g. functions to open a firewall port, Puppet variables containing LDAP details for hooking up services etc.)

I understand that Chris will be doing the initial integration of accessdev with the NCI repository, I'd expect that after that the integration work will be minimal & can be left to the accessdev group. Updates would hopefully be backwards compatible & follow semantic versioning (i.e. updating from NCI 1.0 to 1.1 shouldn't require changes on our end unless we want to make use of new functionality)

<EMAIL_ADDRESS>changed status from new to assigned

<EMAIL_ADDRESS>changed owner from `` to cma900

<EMAIL_ADDRESS>changed milestone from Future to accessdev v1.0

<EMAIL_ADDRESS>changed component from accessdev server to NCI Puppet

<EMAIL_ADDRESS>changed status from assigned to closed

<EMAIL_ADDRESS>commented

Yep, that pretty much sums it up I think.
We will be testing the core framework in itself, however while we'll obviously take as much care as we can to keep things stable (at least during minor releases), it probably won't be practical for us to test integration into every downstream project especially since we aren't experts in each project's specialised applications.  Also, due to the nature of Puppet's design, there's always the risk that namespace conflicts may inadvertently occur as well.  For these reasons and as general best practice, we'll be advising that new versions should be pulled into a test branch first and thoroughly tested in the project before deploying into production.  We will also be staging changes that haven't been published yet in a separate branch which you will of course be more than welcome to pull from for pre-release testing and feedback.

<EMAIL_ADDRESS>removed milestone (was accessdev v1.0)

<EMAIL_ADDRESS>commented

Milestone accessdev v1.0 deleted

<EMAIL_ADDRESS>changed resolution from `` to fixed


feat: partial CA trust anchor

Closes #135
This proposed feature adds to TLS client:

allow_partial which toggles OpenSSL X509VerifyFlags::PARTIAL_CHAIN
This replicates the behaviour of Curl and other client implementations https://github.com/curl/curl/pull/4655

It's better if we create issue for this here.

need cargo fmt

So for tests I generated intermediary mediated chain in certs/ -
1. Subject CN: fluvio.io > Issuer CN: fluvio.io - test-certs/ca.crt
2. Subject CN: intermediate.fluvio.io > Issuer CN: fluvio.io - test-certs/intermediate-ca.crt
3. Subject CN: test.fluvio.io > Issuer CN: intermediate.fluvio.io  - test-certs/intermediate-server.crt

I loaded this into nginx and added start-intermediate-nginx
   ssl_certificate         test-certs/intermediate-server.crt;
   ssl_certificate_key     test-certs/intermediate-server.key;

Without any CA supplied for s_client and invalid host - pretty much everything wrong:
$ openssl s_client localhost:8443
CONNECTED(00000003)
Can't use SSL_get_servername
depth=0 C = US, ST = CA, L = Sunnyvale, O = End Point, OU = Testing Domain, emailAddress =<EMAIL_ADDRESS>CN = test.fluvio.io
verify error:num=20:unable to get local issuer certificate
verify return:1
depth=0 C = US, ST = CA, L = Sunnyvale, O = End Point, OU = Testing Domain, emailAddress =<EMAIL_ADDRESS>CN = test.fluvio.io
verify error:num=21:unable to verify the first certificate
verify return:1
---
Certificate chain
 0 s:C = US, ST = CA, L = Sunnyvale, O = End Point, OU = Testing Domain, emailAddress =<EMAIL_ADDRESS>CN = test.fluvio.io
   i:C = US, ST = CA, L = Sunnyvale, O = Fluvio, OU = Eng, CN = intermediate.fluvio.io
---

With intermediate CA as CAfile - complains about the root CA (intermdiate's issuer) now
$ openssl s_client -CAfile test-certs/intermediate-ca.crt test.fluvio.io:8443
CONNECTED(00000003)
depth=1 C = US, ST = CA, L = Sunnyvale, O = Fluvio, OU = Eng, CN = intermediate.fluvio.io
verify error:num=2:unable to get issuer certificate
issuer= C = US, ST = CA, L = Sunnyvale, O = Fluvio, OU = Eng, CN = fluvio.io
verify return:1
depth=0 C = US, ST = CA, L = Sunnyvale, O = End Point, OU = Testing Domain, emailAddress =<EMAIL_ADDRESS>CN = test.fluvio.io
issuer= C = US, ST = CA, L = Sunnyvale, O = Fluvio, OU = Eng, CN = intermediate.fluvio.io
verify return:1
---
Certificate chain
 0 s:C = US, ST = CA, L = Sunnyvale, O = End Point, OU = Testing Domain, emailAddress =<EMAIL_ADDRESS>CN = test.fluvio.io
   i:C = US, ST = CA, L = Sunnyvale, O = Fluvio, OU = Eng, CN = intermediate.fluvio.io
---

WIth -partial_chain that is essentially the PARTIAL_CHAIN X509VerifyOpt:
 openssl s_client -CAfile test-certs/intermediate-ca.crt -partial_chain test.fluvio.io:8443
CONNECTED(00000003)
depth=1 C = US, ST = CA, L = Sunnyvale, O = Fluvio, OU = Eng, CN = intermediate.fluvio.io
verify return:1
depth=0 C = US, ST = CA, L = Sunnyvale, O = End Point, OU = Testing Domain, emailAddress =<EMAIL_ADDRESS>CN = test.fluvio.io
verify return:1
---
Certificate chain
 0 s:C = US, ST = CA, L = Sunnyvale, O = End Point, OU = Testing Domain, emailAddress =<EMAIL_ADDRESS>CN = test.fluvio.io
   i:C = US, ST = CA, L = Sunnyvale, O = Fluvio, OU = Eng, CN = intermediate.fluvio.io
---
Server certificate

bors r+


ci: fix merge action to keep linear history

via: https://github.com/aeraki-mesh/aeraki/pull/156/checks?check_run_id=4787204867
What this PR does / Why we need it:

Pre-Submission Checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added related test cases?
[ ] Have you modified the related document?
[ ] Is this PR backward compatible?

This PR needs to be merged manually for changes in  configuration see: https://github.com/aeraki-mesh/aeraki/pull/157/checks?check_run_id=4787942003
Thanks


Issue resolved : Prefer "critical project" to "2fa required" badge on /manage/projects/

I changed the priority to display badge for this issue https://github.com/pypi/warehouse/issues/11806

@di

@di ?

@alex

@SamirPS The actual failing test suite tells us what to do: https://github.com/pypi/warehouse/runs/7585359799?check_suite_focus=true#step:9:532

@SamirPS The actual failing test suite tells us what to do: https://github.com/pypi/warehouse/runs/7585359799?check_suite_focus=true#step:9:532

Thanks i just commit the output of make translations and now i wait for the pipeline

@SamirPS Thanks! From the GitHub status, there's now conflicts that need to be resolved - would you mind trying to rebase or merge your branch from the current main to identify and resolve?

@SamirPS Thanks! From the GitHub status, there's now conflicts that need to be resolved - would you mind trying to rebase or merge your branch from the current main to identify and resolve?

@miketheman can you tell me the command to do ?

@miketheman good now ? because i see other conflits


[Python] Reuse the action-publish-pypi in release-pypi.yaml

🤔 What's changed?
Reuse the<EMAIL_ADDRESS>in release-pypi.yaml
⚡ What's your motivation?
To reduce duplication
🏷 What kind of change is this?

:bank: Refactoring/debt/DX (improvement to code design, tooling, documentation etc. without changing behaviour)

♻ Anything particular you want feedback on?

📋 Checklist:

[x] I agree to respect and uphold the Cucumber Community Code of Conduct
[x] Users should know about my change

[x] I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

This text was originally generated from a template, then edited by hand. You can modify the template here.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (408c063) 91.17% compared to head (48b8b9a) 91.17%.
Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #246   +/-   ##
=======================================
  Coverage   91.17%   91.17%
=======================================
  Files           2        2
  Lines          34       34
=======================================
  Hits           31       31
  Misses          3        3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Unable to open fontello site

I'm always getting this error Asset load error: "N.Wire" unavailable after asset load.

ctrl + f5 ?

ctrl + f5 not working either, i tried that.

Nope, still not working after browser cache clean up the error pop up message are coming in delay like 10 seconds later, but no icons are showing still. Tried with internet explorer, same problem.

That's strange, server works. Can your provider force transparent proxy cache?
Try in Chrome for shure:

go to fontello
open debugger (F12)
Right click on "reload" button and select "Cache cleanup and hard reload"

hi, just did that, i'm pasting what the 1 error i'm getting from the debugger
Uncaught TypeError: Cannot read property 'load' of undefined
    at Object.<anonymous> (/assets/bundle-fontello.en-US-e0f4e12618de6dd75302135cc51f749f.js:2)
    at Object.o.execute (fontello.com/:2)
    at /assets/bundle-fontello.en-US-e0f4e12618de6dd75302135cc51f749f.js:1
    at Object.application/javascript (fontello.com/:2)
    at m (fontello.com/:2)
    at fontello.com/:2

and the link address of the error is :
 /assets/bundle-fontello.en-US-e0f4e12618de6dd75302135cc51f749f.js

btw i'm working on localhost.

Asset load error: "N.Wire" unavailable after asset load. google chrome.
cache cleaning is not helping!

Yep. This is issue is still there

is there any solve to this error, I got a similar one in FlutterIcon.com

Let's close as outdated. Create new one if needed


Inform users when jupyter-server can't start properly

Use case 1
Pod is in CrashLoopBackOff because of too low disk space
Currently users that ask for a disk size smaller than what they need (especially if they pull data from LFS) are stuck.
In these cased, the jupyter-server keeps trying to start unsuccessfully.
The user needs to be notified about this somehow, ideally with a clear message: your session can't start because xyz please request more disk space or disable auto-fetch LFS.
Secondly, the user should be able to stop their sessions in this cases, related to https://github.com/SwissDataScienceCenter/renku-ui/issues/787
Use case 2
Incompatible images might also trigger this without the user being able to understand what is happening or stop the Pending session. ( cc @vigsterkr )

I think this belongs more in the notebook service. In fact in my opinion this will be addressed by this issue in the notebook service. https://github.com/swissdatasciencecenter/renku-notebooks/issues/868


Dataset sanity checks

FROM @gcampanella  FEEDBACK REVIEW
What is affected by this bug?
Sometimes, we received datasets full of ‘unrealistic’ values: just as an example, users who (supposedly) glance at their lock screen once a second!
Where on the platform does it happen?

Azure Data Science Virtual Machine.
Azure Databricks.

Expected behavior (i.e. solution)
I don’t think there’s any way to automate this entirely, but perhaps you could provide some sanity checks since the data format you consider seems somewhat fixed (e.g. flag any users who’ve rated more than X times the mean/median number of items, or that are rating items every other second).

very old issue, closing, feel free to reopen if needed


Fix missing Python parameter type defaults

🤔 What's changed?
The project documentation outlines for Customer Parameter Types that use_for_snippets "Defaults to true"  and that prefer_for_regexp_match "Defaults to false".
Although this aligns with the documented Python example, its implementation is missing default values for those keyword arguments:
class ParameterType:
    def __init__(
        self,
        name,
        regexp,
        type,
        transformer,
        use_for_snippets,
        prefer_for_regexp_match,
    ):

With this PR:

The appropriate defaults have been added to the ParameterType class.
Two versions of Python type hints in the module defining that class have been reduced to one e.g. typing.List and list (through__future__.annotations.
A Python Behave example has been documented - see python-bdd-playground for an example of it
Moves a 'Note' relating to Parameter Types to before language-specific examples - as it was beneath the Python example and was not clear whether it only related to that example.
Moves contributing documentation from the Python README to its own contributing file - as is unrelated to usage of the package (see PyPI listing for reference)

⚡ What's your motivation?

Fix a bug where default keyword arguments were not set for customer parameter types in Python
Document an example for using Cucumber Expressions with Python (behave)

🏷 What kind of change is this?

:bug: Bug fix (non-breaking change which fixes a defect)

♻ Anything particular you want feedback on?
Whether README changes look good.
📋 Checklist:

[x] I agree to respect and uphold the Cucumber Community Code of Conduct
[x] I've changed the behaviour of the code

[x] I have added/updated tests to cover my changes.

[x] My change requires a change to the documentation.

[x] I have updated the documentation accordingly.

[x] Users should know about my change

[x] I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

Closes #142, providing a valid example of Cucumber Expressions with a Python framework, Behave.

Appreciate the considered feedback @mpkorstanje - refined PR scope accordingly


Add an enabled os-vmguest service to the iso

The idea is that on first boot (need to figure out what that means for pre-made vm images) this small container detects if its running in a VM, and if so, enables (and pulls) the right guest service. os-vmguest then disables itself, so it adds no extra burden.
see https://github.com/RoastLink/roastlink.github.io/blob/master/v/vm-guest-tools.yml and https://github.com/RoastLink/roastlink.github.io/tree/master/images/10-vmguesttools for a debian based POC

uses https://people.redhat.com/~rjones/virt-what/ - as that seems to have support..
the choice is to use os-base, in which case i need to add dmidecode to it, or alpine, which should mean building an alpine package for virt-what
for extra points, run this at the end of the installer, and enable the right service...

memory usage is the biggest issue - as this is a very ephemeral service.

Luckily memory usage wouldn't be an issue with Docker Machine


citation-key based on file-name

Thanks for brilliant add-on! citation-key based on file-name feature would be a day saver for users who rely on Emacs, where a citation-key can be used to quickly open citation-key.pdf... Is it possible? If not, would it be very hard to add?

Can you try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.97-br673-3755.xpi with [attachment] for the pattern? Note that if multiple references across your library, not just across any collection in it, have the same attachment file name, they'll be getting attachment-name, attachment-namea, attachment-nameb etc as citation keys. This is not changeable (and won't be), unless you manually pin the citation key.
It's also possible to go the other way around BTW; if you have ZotFile and BBT installed, you can use [%b] in the file rename pattern; [%b] will expand to whatever citation key BBT generated. This already works in the current release versions of ZotFile and BBT, no need to install this pre-release.

Works like a charm. However I didn't know about the "other way around" that you have pointed out. It might be handier to use it as filenames contain whitespace characters replaced by "" and that char can cause problems in bibtex/biber and LaTeX. (I will try if I can force zotfile to use "-" instead of "" not only for the authors separation, but as any whitespace characters replacement, also I haven't tested the [%b] yet)
Anyway, the feature that I have kindly asked for works perfectly well. Thanks a lot!

This isn't yet merged to main; I've reopened this as a reminder to do so Tuesday when I have access to my dev system.

alright.


docs: add decision on responsive component spacing properties

FX-4226
Description
This pull request adds an RFC for responsive spacing properties of the Container component.
Additional information is mentioned in https://toptal-core.atlassian.net/wiki/spaces/FE/pages/3388998280/Responsive+component+spacing+properties.
How to test

Please review the https://github.com/toptal/picasso/blob/FX-4426-document-responsive-api/docs/decisions/19-responsive-component-spacing-properties.md

Development checks

[x] Self reviewed

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers


Removed deprecated 'gcm_user_visible_only'

According to ChromeStatus this key is already deprecated (https://www.chromestatus.com/feature/5778950739460096). Proper code is already put in https://github.com/GoogleChrome/samples/blob/gh-pages/push-messaging-and-notifications/main.js : 119

Thanks, @karolklp!


Typescript

As I mentioned in #5, I really like this library's goals. It is almost exactly what I'm looking for. However, I have one important requirement this does not meet: type safety using Typescript. Are you open to including .d.ts files in the repository?

I would contribute them at least for the functions I use. However, I don't have a good idea yet how to integrate them into your test setup.

Sure, I'm absolutely open to add the d.ts. However since I'm not planning to change the core language of the lib to ts, we'll have to add the definitions manually. I'll look into it, it shouldnt be too difficult considering the size of the lib.
With recent ts versions once we have the d.ts, I think the only thing we have to do is include it in the root of the repo, and you should be able to import from the package properly. As for testing, we definitely should add at some tests to check if the functions accept all the right type of arguments, and don't cause compile errors, functionality is tested ok with the standard js tests.

I'm currently thinking the nicest path forward may be for us to borrow from each-others' libraries. I plan to shamelessly steal tests from yours, and probably some implementations too, maybe over time as I have need for them in my own projects.
You should feel free to steal implementations from mine, too. As for the .d.ts files, they are generated automatically for all my functions by the build process, so for any functions where we both have the same signatures - done!  Judging by my dist/ directory, I'm guessing you would want an index.d.ts next to each index.js, including the one in the root directory which simply re-exports all the ones in subdirectories.  However, that still does not address testing them, which I should get for "free" as I port tests to typescript (a mistake in the type definitions would cause a compilation error).
Perhaps my next step will be finding a function we have in common, stealing your tests for it, then submitting a PR to get the .d.ts file in your repo.

Et voila: https://github.com/simontonsoftware/mini-dash/commit/c1945b10eaa4222e31a1973c3e052b42b02448e8
I think this is how it should be set up. Let me know if you think this is worth continuing, and I can add more like it.

Types needs some refinement, but looks good! I assume there would be an index.d.ts that simply imports and re-exports everything (I know definitions don't really import/export stuff, but that's how it would look like in the code)?
Also, don't be ashamed "steal" whatever you need 😄 but as we discussed some methods are nowhere near feature equivalent with lodash, so changes might be required.

Yes, there would be an index.d.ts that imports & re-exports everything ... and there already is!  When you looked at this it simply had the line export * from './clone';.  I just committed another one, so it should have 2 lines now.


Cannot inference certain type.

Repro repo:
https://github.com/crlf0710/ra_bug_repo
rustc is able to inference the type of variable _struct_data, while ra only shows "{unknown}".

The syn::DataStruct type is generated by a macro; I assume there's something in there we can't handle. I haven't checked in detail though.

My short analysis is that we can't resolve into_data_struct() because it is generated by a procedural macro easy_ext::ext. We don't currently support proc macros.

I don't think that's the problem, since we ignore attribute macros completely, so this just looks like a normal impl.

The impl syn::Data doesn't work here, because the crate isn't the one defining syn::Data. That means this is just #6029 – it would work if we expanded the attribute macro.


[patch] Removed repeated version comment for 0.11.4

Removed repeated version comment for 0.11.4

ok, fixed!

👍 Thanks @luislobo!  Looks like the changelog needs some updating in general... :-/


Tutorial Help!

Hi everyone, really need help. I used FME to convert IFC to citygml. I imported a 3D citygml into 3Dcitymodel, and when I try to export it in KML or also in citygml again, it seems to be 2D!   Why?

The exported KML can be 2D if you select the "Footprint" display form. But interestingly, the exported CityGML should never be 2D, since the CityGML-Exporter does not do any 3D -> 2D projection.
Could you share your 3D CityGML file converted from IFC? It would help us to reproduce and solve the issue.
PS: The tutorial here (https://github.com/3dcitydb/tutorials) is a good leaning material for getting startet with 3DCityDB.

Hi,
really thank you for the help. I practiced with the tutorial, but once i used my case study, I ran into problems.
At the wetransfer link --> https://we.tl/t-PvKZH6POO1     you can download all my files:
-IFC
-FME WORKBENCH
-Citygml as FME OUTPUT
-Citygml as 3DcityDB Output.
-KML after FME
-KML after 3dcitydb
Ps:  my case study is only a single building because I'm also having difficulty understanding how to enrich the database, not being present in the tutorial.
Thanks

Many thanks for sharing the datasets! I've also reproduced the problem. The triangulated surface geometries in the CityGML data from FME are not completely imported into the database. Consequenlty, the exported KML and CityGML data are not correct.
We'll look into the problem and try to solve it as soon as possible.

THANK YOU,
I AM WAITING FOR YOUR USEFUL HELP.

@yaozhihang
I found another issue. I can't publish my SPSHG on Google Fusion Tables reported in exercise 5 because Fusion Tables API have been discontinued. they suggest finding alternatives such as BigQuery, Cloud SQL, Maps Platform, and Data Studio. Do you have any suggestions for me? I don't know which one can I use and how.Thank you

There are already two alternatives (Google Sheets and PostgreSQL database with a RESTful API). You may get more details from this website: https://github.com/3dcitydb/3dcitydb-web-map/releases

Ciao @matteomandrile,  ti ringrazio.
Avevo già tentato di seguire le indicazioni riportate da yaozhihang ma alla fine non sono riuscita ad installare la parte di Google Sheet. Riproverò anche con l'aiuto delle tue indicazioni.
In ogni caso "una conversione da IFC a CityGML con dati riguardanti le performance energetiche" è esattamente quello che sto investigando anche io. Per farlo ho usato FME ed ho creato un workbench di conversione. Come ho descritto a yaozhihang trasferendo il citygml apparentemente corretto dopo la conversione su 3dcitydb, una volta riesportato perde le sue informazioni 3d.
@yaozhihang  hypothesis is "The triangulated surface geometries in the CityGML data from FME are not completely imported into the database. Consequently, the exported KML and CityGML data are not correct.
We'll look into the problem and try to solve it as soon as possible."
So now, I'm waiting for his suggestion, in the meantime, if you are interested I can send you the FME workbench, and maybe we can continue to discuss it, wanting to achieve the same goal
For now, I'll try again to use Google Sheet.
Thanks

@matteomandrile
sono riuscita a seguire tutti i passaggi. ho solo un dubbio sulla parte finale... Nell'immagine riportata ho messo dei numeretti. Se al primo passaggio sono loggout, allora quando clicco sulla chiavetta, riesco a scegliere il mio account in cui ho registrato API Google sheet, ma a seguito dell'accesso a me resta visualizzata la chiavetta e connesso Giulia al primo step, esattamente come vedi nell'immagine. Anche riprovando a cliccare sulla chiave, appare (step 3), attendi un istante... e poi si chiude. Ho provato molte volte, non riesco a raggiungere la situazione in cui al posto della chiavettaa solo loggata anche dentro webclient (come te con MM). Ho sbagliato qualcosa?

@GiuliaSpi, thanks for reporting the bug with importing your dataset. The current CityGML import assumes a gml:CompositeSurface as shell of a solid but your dataset uses a gml:TriangulatedSurface. Though being very uncommon, using a gml:TriangulatedSurface is technically allowed. A fix is proposed here https://github.com/3dcitydb/importer-exporter/pull/130. You can test the fix by building a new version of the Importer/Exporter following this guide.
Some more general guidelines:

Pick the appropriate issue tracker. For instance, your bug of the Importer/Exporter tool should be reported in the Importer/Exporter issue tracker rather than here.
Create one issue per bug or feature request instead of discussing multiple topics in one issue. You can link issues if you think they are related to each other.
Stick to English language so that others can follow and benefit from the discussions.

I've really appreciated your suggestions. Thank you.

Hello @clausnagel,
Thank you for your fix and suggestion. I will post my previous question in the 3dcitydb-web-map client issue tracker!

Thanks @GiuliaSpi and @matteomandrile, much appreciated. We will look into your further issues.

Fixed missing import of TriangulatedSurface with https://github.com/3dcitydb/importer-exporter/pull/130. The fix will be part of the next release of the Importer/Exporter.

Hello @clausnagel ,
I built a new version of the Importer/Exporter and it is still throwing errors:

I am trying to import a building created by converting an IFC to CityGML, following the same tutorial as @GiuliaSpi, therefore I am also generating triangulated Surfaces:

However if I search in the file the complete XLink reference that is generating the error, it appears that it refers to
<app:target>#fme-gen-9e016599-d8c6-4e4c-b097-13892c78010a_16301</app:target>
In the DB, the building appears to be correctly imported:

But the Export file (17 MB) does not show any geometry in FZK

I attach the "imported" file generated from the IFC->CityGML conversion (FME) and the "exported" file from the imp/exp Tool.
test.zip
Do you have an idea why this is happening?

The errors messages are correct because, as the log messages say, your file contains a lot of references that cannot be resolved.
An <app:target> must reference an existing surface geometry object. Take your example <app:target>#fme-gen-9e016599-d8c6-4e4c-b097-13892c78010a_16301</app:target>. There is no surface geometry with gml:id fme-gen-9e016599-d8c6-4e4c-b097-13892c78010a_16301 in your file. So the <app:target> property points to nowhere and cannot be resolved. Hence the error message. This needs to be fixed in your import dataset.
I also have no issues with visualizing your Bellevue_LOD2City_export.gml export dataset with the FZKViewer. I tested with both version 5.3.1 and 6.0 of the software.

Thank you for your explanation, I have resolved the issue by changing the CityGML Geometry Setter parameter City GML Lod Name from  lod2Solidto lod2MultySurface.


Set a default value if the database value is empty

Connected to issue number : #1145
I have the datepicker component like this
<DatePicker id="deadline" selected={moment(data.meta.deadline)} onChange={this.handleDeadlineChange} dateFormat="LL" scrollableYearDropdown yearDropdownItemNumber={15} showYearDropdown />
If the data.meta.deadline  is empty cant we have something like below
selected={moment(data.meta.deadline) || moment()}
if there is no data then show the current date.
Thanks in advance.

Any help on this

I might be wrong, but its working for me, as I provide moment() on this.state.selected and if database return any value initialize with value in componentDidMount() with other initialization.  Or you can try following
this.state = {
selected: ((data.meta.deadline)
? moment(data.meta.deadline)
: moment()
}


[Bug] Chat card spam when a token is deleted in combat

When a token on the combat tracker is deleted, the chat card will spam (looks to be about the same as the number of combatants).
I have a way to fix this, which isn't super pretty, but will work (change Hooks.on -> Hooks.once, plus set a timer to re-establish the hook after a short timeout).  Need to investigate if core bug/module conflict/my own damn fault.  Should be able to poke at the weekend.

... And now I can't reporduce this.  If anyone spots this happening and can give me solid reproduction steps, please add a comment on this.


SYNC messages

Dear Michael,
sorry to bother again but I have another question. According to https://canopen-stack.org/v4.2/usage/tpdo/#sync-producer, I only need to add objects 0x1005 and 0x1006 to the OD for generating the SYNC messages. I tried it here (I am using your code in a ESP32 node), but without success. Is there any other detail that I am missing?
Thanks a lot and kind regards,
Tomas

Hello, I tried a few things and the error seems to be that the COSync module initialization doesn't get the correct value from OD entry 0x1005, even if I put the 0x1005 and 0x1006 in the OD:
uint32_t obj1005 = CO_SYNC_COBID_ON | 0x80;    /* enable sync producer       */
uint32_t obj1006 = 1000000;                  /* sync cycle-time 1s in ms */

/* define the static object dictionary */
static struct CO_OBJ_T CnhCoNodeOD[APP_OBJ_N] = {
    {CO_KEY(0x1000, 0, CO_UNSIGNED32|CO_OBJ_D__R_), 0, (uintptr_t)0},
    {CO_KEY(0x1001, 0, CO_UNSIGNED8 |CO_OBJ____R_), 0, (uintptr_t)&Obj1001_00_08},
    {CO_KEY(0x1005, 0, CO_UNSIGNED32|CO_OBJ_D__R_), 0, (uintptr_t)0x80},
    {CO_KEY(0x1005, 0, CO_UNSIGNED32|CO_OBJ____RW), CO_TSYNCID, (uintptr_t)(&obj1005)},
    {CO_KEY(0x1006, 0, CO_UNSIGNED32|CO_OBJ____RW), CO_TSYNCCYCLE, (uintptr_t)(&obj1006)},
    {CO_KEY(0x1017, 0, CO_UNSIGNED16|CO_OBJ_D__R_), 0, (uintptr_t)0},

I only made it work by hardcoding the COB-ID inside the COSyncInit():
   sync->CobId =  CO_SYNC_COBID_ON | 0x80;
Any idea why is this happening? Could it be endianness?
BR,
Tomás

Hi, not clear to me. Can you try the following configuration, please?
Maybe the sync producer is working with direct values only (by mistake):
    {CO_KEY(0x1005, 0, CO_UNSIGNED32|CO_OBJ_D__RW), CO_TSYNCID, (uintptr_t)(CO_SYNC_COBID_ON | 0x80)},
    {CO_KEY(0x1006, 0, CO_UNSIGNED32|CO_OBJ_D__RW), CO_TSYNCCYCLE, (uintptr_t)(1000000)},

Ok, then I guess we need to change the SYNC type functions slightly. In the SYNC type functions, the functions COObjRdDirectand COObjWrDirect are used, instead of COObjRdValue and COObjWrValue.
I will add a specific test to the test-suite and check the behavior.

When you use normal process variables for OD entries, is this working?
uint32_t obj2000 = 0x12345678;

const struct CO_OBJ_T OD[] = {
    :
  {CO_KEY(0x2000, 0, CO_UNSIGNED32|CO_OBJ____R_), 0, (uintptr_t)&obj2000},
  {CO_KEY(0x2001, 0, CO_UNSIGNED32|CO_OBJ_D__R_), 0, (uintptr_t)0x90abcdef},
    :
}

Can you read the content of these variables via SDO correctly?

I didn't try to read the OD entries via SDO, but if a trigger a TPDO to transmit the hard-coded and variable referenced signals, it fails.
// Manufacturer Specific Objects
    {CO_KEY(0x2025, 0, CO_UNSIGNED8 |CO_OBJ___PR_), 0, (uintptr_t)&motor_temp},
    {CO_KEY(0x2026, 0, CO_UNSIGNED8 |CO_OBJ_D__R_), 0, (uintptr_t)2},
    {CO_KEY(0x2026, 1, CO_UNSIGNED8 |CO_OBJ___PR_), 0, (uintptr_t)0x02},
    {CO_KEY(0x2026, 2, CO_UNSIGNED8 |CO_OBJ___PR_), 0, (uintptr_t)&capacitor_temp},

Using only variable references works fine.
// Manufacturer Specific Objects
    {CO_KEY(0x2025, 0, CO_UNSIGNED8 |CO_OBJ___PR_), 0, (uintptr_t)&motor_temp},
    {CO_KEY(0x2026, 0, CO_UNSIGNED8 |CO_OBJ_D__R_), 0, (uintptr_t)2},
    {CO_KEY(0x2026, 1, CO_UNSIGNED8 |CO_OBJ___PR_), 0, (uintptr_t)&controller_temp},
    {CO_KEY(0x2026, 2, CO_UNSIGNED8 |CO_OBJ___PR_), 0, (uintptr_t)&capacitor_temp},

Please keep an eye on the flags:
´´´c
{CO_KEY(0x2026, 1, CO_UNSIGNED8 |CO_OBJ___PR_), 0, (uintptr_t)0x02},
´´´
with your config, you will get the content of address 0x02. I guess your intension was to return the value 0x02.
Key for controlling this, is the Flag ´D´ in ´CO_OBJ___PR_´ vs. ´CO_OBJ_D_PR_´. See Configuration/Object Entry.

Alles klar, now it works fine, sorry for the mistake.

You are welcome. Good to hear, that it now works.


[Task]: Make integration tests work without GCP file

What needs to happen?
When you do not provide the GCP_KEY_FILE argument the local tests fail
Issue Priority
Priority: 2
Issue Component
Component: Integration Testing

.take-issue


108.0.5359.124: libwayland: wl_surface@54: error 2: Buffer size ... is not divisible by scale

OS/Platform
Gentoo
Installed
Compiled from source
Version
108.0.5359.124
Have you tested that this is not an upstream issue or an issue with your configuration?

[ ] I have tried reproducing this issue in Chrome and it could not be reproduced there
[ ] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[ ] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile and it could not be reproduced there

Description
Browser exits when moving a picture
How to Reproduce?

Go to https://gab.com/WholeHearted4God/posts/109771967867525776
Click on the image to open it
Try moving the pic (left mouse press and hold and move)
Rejoice!

Actual behaviour
Browser exits with error.
Expected behaviour
Browser continues working and picture is movable.
Relevant log output
ERROR:wayland_event_watcher.cc(36)] libwayland: wl_surface@59: error 2: Buffer size (399x331) is not divisible by scale (2)

### Additional context

wayland-1.21.0

CHROMIUM_FLAGS="--enable-features=UseOzonePlatform --ozone-platform=wayland"
CHROMIUM_FLAGS="--enable-features=VaapiVideoDecoder --disable-features=UseChromeOSDirectVideoDecoder ${CHROMIUM_FLAGS}"
CHROMIUM_FLAGS="--disable-sharing-hub --hide-sidepanel-button ${CHROMIUM_FLAGS}"
CHROMIUM_FLAGS="--remove-tabsearch-button --scroll-tabs=never --tab-hover-cards=none ${CHROMIUM_FLAGS}"
CHROMIUM_FLAGS="--show-avatar-button=never ${CHROMIUM_FLAGS}"

Its highly unlikely that our patchset caused this as a regression. Please test with vanilla chromium (also this is an outdated version which won't be supported further)

Fixed in later versions.


False positive unused import when import is referencing extension function and is used only in kdoc

Steps to Reproduce
Run ktlint on the following two files:
package some.pkg

fun String.returnSelf(): String {
    return this
}

import some.pkg.returnSelf

/**
 * Do not forget that you can also return string via [String.returnSelf]
 */
fun test() {}

Expected Behavior
Ktlint should pass
Observed Behavior
Ktlint fails with FileB.kt:1:1 Unused import

Your Environment

Version of ktlint used: 0.43.0
Name and version (or code for custom task) of integration used (Gradle plugin, Maven plugin, command line, custom Gradle task): JLLeitschuh/ktlint-gradle 10.2.0
Version of Gradle used (if applicable): 7.2
Operating System and version: 5.15.1-1-MANJARO #1 SMP PREEMPT Sat Nov 6 14:11:10 UTC 2021 x86_64 GNU/Linux

Can you provide some background of the use case for which you want to keep the import when it is only used for a reference in a KDoc? IntelliJ IDEA Ultimate has no problem to resolve the reference without the imports.

IntelliJ IDEA Ultimate has no problem to resolve the reference without the imports.

Hm, it does not resolve it for me without the import:

It happens with the latest one 212-1.6.0-release-799-IJ547.46.
Are you sure your IntelliJ can resolve this kdoc if you remove an import? Can you take a screenshot? Because I don't even see how would that work (IntelliJ needs imports to resolve conflicts etc.)

I think I do understand the miscommunication between us.
Is it correct that your files are in different directories? If files are both in the same directory then IntelliJ indeed has no problems to display the link inside the kdoc as files belong to same package. However, if the files are in different directories (e.g. not in the same package) then IntelliJ needs the import for displaying the link inside kdoc correctly.

Sorry, you are correct. I thought that files having the different package implied that they are in the different directories (I know kotlin allows different packages in the same directory, but this is generally not the practice).
Updated the bug report to make that clear.

See commit above. It should be fixed with that.

Thanks!


feat: Update gitignore when using jaspr create

Description
The following should be added to .gitignore when using jaspr create.
# Files and directories created by flutter.
.flutter-plugins
.flutter-plugins-dependencies

Additional Context
When adding flutter packages that are going to be used through interop in a jaspr app, these files can get created and they should get ignored  - as they would for a conventional flutter project.
For reference, here is the flutter .gitignore template. It might be worthwhile to expand the jaspr template .gitignore, it is pretty sparse right now?
https://github.com/flutter/flutter/blob/master/packages/flutter_tools/templates/app_shared/.gitignore.tmpl

This is part of #168