/model/predict returns unexpected result

No big deal, but
{
 "text": [""]
}

it's odd that nothing can be interpreted as something
     {
        "toxic": 0.0015665121609345078,
        "severe_toxic": 0.00011611604713834822,
        "obscene": 0.0004790136299561709,
        "threat": 0.00007217624806798995,
        "insult": 0.0003883792378474027,
        "identity_hate": 0.0003294776252005249
      }

and similarly a meaningless combination of characters
{
 "text": ["khdcuc4wcc5jlcnw"]
}

and numbers
{
 "text": ["123"]
}

Note sure if there's something that can be done to eliminate noise that's a result of garbage input.

For the first case, I think we can catch an empty string input and return either all zeros or an empty dict? Would probably need to be an empty dict since the empty string could be in position other than 0 in the list.
For the others there is not much to be done, but I am somewhat surprised it returns any prediction since those words won't be in the vocab (and I don't think this BERT model has char-level embeddings cc @splovyt?). I would expect it to return zeros for out of vocab words.

Interesting observation!
I was able to print out the generated tokens (used for classification) and this is what I can derive.

For the empty string:

tokens: [CLS] [SEP]

For khdcuc4wcc5jlcnw:

tokens: [CLS] k ##hd ##cu ##c ##4 ##wc ##c ##5 ##j ##lc ##n ##w [SEP]

For 123:

tokens: [CLS] 123 [SEP]

The [CLS] token is the sentence-level token that will be used for the actual prediction. I think it generally carries some weight, even when no sentence is present.. I definitely don't know for sure though.. Regarding the other meaningless combination of characters, it gets split up in word groups who might carry some level of meaning. The example often used to showcase the split is this one: puppeteer to puppet ##eer. I think this gives some insight, but I cannot draw a definite conclusion on why this happens.
Regarding filtering garbage input, this will 'cost' a couple of rule-based lines of code. I probably won't be able to cover all of the adverse results with it, but I can give it a try. Do you want me to try and cover this? @ptitzler @MLnick


selfupdate: Prints number of outdated ports after updating ports tree

Hi,
Here is a tiny suggestion about user interface.
People may want to know how many ports can be upgraded after the ports tree has been updated by port selfupdate.
This patch prints about numbers of outdated ports like this:
# port selfupdate
--->  Updating MacPorts base sources using rsync
MacPorts base version 2.8.1 installed,
MacPorts base version 2.8.1 downloaded.
--->  Updating the ports tree
--->  MacPorts base is already the latest version

The ports tree has been updated. To upgrade your installed ports, you should run
  port upgrade outdated

2 ports are outdated. Run 'port outdated' to see them.

and when no installed ports are outdated will look like this:
# port selfupdate
--->  Updating MacPorts base sources using rsync
MacPorts base version 2.8.1 installed,
MacPorts base version 2.8.1 downloaded.
--->  Updating the ports tree
--->  MacPorts base is already the latest version

The ports tree has been updated. To upgrade your installed ports, you should run
  port upgrade outdated

All installed ports are up to date.

Thanks for the contribution

Thanks for the update. I'm currently merging some other work involving selfupdate, so I'll come back to this once that's done.


Yoruba GV NER added

I just added Yoruba GV NER dataset from this paper https://www.aclweb.org/anthology/2020.lrec-1.335/

Thank you. Okay, I will add the dataset card.


Stage 1: Enable Application gitops on a single cluster

Define and manage Applications in the apps directory of a git repo, deployed into a k8s cluster

Closing as part of an issue cleanup effort discussed with @JamWils and @markramm


CreateResource api param externalLink not work

Describe the bug
CreateResource api param externalLink not work，see the postman api response, externalLink value is blank.
Steps to reproduce
See the screenshots
The version of Memos you're using
0.22.3
Screenshots or additional context

It's no longer supported in this version.
https://github.com/usememos/memos/blob/46ca5bb8e7f6ba103a8dfe342c774bcb3aa18338/server/router/api/v1/resource_service.go#L40-L84


GeneralTypeIssues: "mymod.MyClass" is incompatible with "mymod.MyClass" 🤷♀

Hello! I have multiple of these "problems" highlighted:
Argument of type "Self@A2Window" cannot be assigned to parameter "parent" of type "A2Window" in function "setup_ui"
  "a2ui.A2Window" is incompatible with "a2ui.A2Window"

The setup is with .pyi files in a /typings dir. I'll line out on of these quickly:

a2app.pyimport a2ui
import a2core
from singlesiding import QSingleApplication
# ...

class A2App(QSingleApplication):
    # ...
    def do_stuff(self):
        self._core = a2core.A2Obj()
        #                        v____v<--HERE----------------
        self._win = a2ui.A2Window(self)
        # Argument of type "Self@A2App" cannot be assigned to parameter "app"
        # of type "A2App" in function "__init__"
        # "a2app.A2App" is incompatible with "a2app.A2App"

        #         v___v<--HERE----------------
        self._core.app = self
        # Cannot assign member "app" for type "A2Obj"
        # Expression of type "Self@A2App" cannot be assigned to member "app" of class "A2Obj"
        # "a2app.A2App" is incompatible with "a2app.A2App"

a2ui.pyiimport a2app
from a2qt import QtWidgets
# ...

class A2Window(QtWidgets.QMainWindow):
    app: a2app.A2App
    # ...
    def __init__(self, app: a2app.A2App): ...

a2core.pyiimport a2app
# ...
class A2Obj:
    app: a2app.A2App = ...
    # ...

This is Qt for Python 5.12 stuff.
Environment data

• Language Server version: v2021.11.1-pre.1
• Python version: 3.9.7
• Version: 1.63.0-insider (user setup)
• Commit: a7104f13ff61bcd9f4c827d8fe3c0d27bb479fd8
• Date: 2021-11-08T05:14:49.440Z
• Electron: 13.5.1
• Chrome: 91.0.4472.164
• Node.js: 14.16.0
• V8: 9.1.269.39-electron.0
• OS: Windows_NT x64 10.0.19043

This "seems" like a bug. Even if its my fault (of course) this information is not helping ☹

There's not quite enough information here for me to reproduce the problem because the sample isn't complete.
One thing that jumps out at me is the cyclical dependency between the three modules in your example. The fact that you're importing a non-stub file from a stub is a red flag; normally that's not something you would do.
Pylance supports detection of module cylces, but it's disabled by default. If you want to enable it, the configuration option is called reportImportCycles.
I don't know if that's the real cause of the problem in this case because I'm not able to repro the actual error. If you're able to provide a complete example or point me to the repo where this code is checked in, I can investigate further.

The file names are links to my repository. But i'll also try to cook it down further...

Hello @erictraut ! I created a mini repo: https://github.com/ewerybody/_typing_test
Same issue in https://github.com/ewerybody/_typing_test/blob/main/app.py#L6
Argument of type "Self@MyApp" cannot be assigned to parameter "backend" of type "MyApp" in function "__init__"
  "app.MyApp" is incompatible with "app.MyApp"

Hope that helps 🙇♀

Thanks for the repro.
The problem here is that you're using type stubs in an inappropriate manner. A type stub is meant to describe the interface of a module, typically part of an external library that your program imports. Think of it as a stand-in for a source file when that source file isn't available (e.g. it is written in another language).
You have created type stubs that have the same names as modules within your own source base. When app.py imports from ui, the type checker resolves the import using ui.pyi (consistent with the rules documented in PEP 484 and PEP 561). The file ui.pyi imports the module app, and this is being resolved using app.pyi. Both app.py and app.pyi declare a class called MyApp, but these are different classes from the perspective of the type checker. That's why you're receiving this (somewhat confusing) error message.
The typings folder is normally where you place type stubs for external libraries that your code consumes. If your intent is to use static type checking on your own code, you should delete the type stubs and add the type annotations directly to your code (your ".py" files). There's no need for type stubs in this case, and the use of type stubs will cause lots of problems like the one you're seeing here.
When you add types to your source files directly, you're may run into another issue. You have a rather gnarly cyclical dependency between app.py and ui.py. Each is using a class declared in the other. As a general rule of good software development, that's something I always try to avoid. If these classes are so tightly related that they depend on each other, then the should probably be in the same module. There are other ways to refactor code to eliminate cyclical dependencies, but this is the most straightforward.

Yeah the idea was to try to keep the typing stuff out of the source files as it serves no runtime benefits.
Being able to offload all this to a stub file seemed SO enticing to me!!
And yes, I had the illusion that the stub files don't suffer from this cyclical imports problem and it DOES work for a lot of things! :(
Well, this code is years old, one of my hobby projects. I just started learning Python when writing this. And well, I try to keep learning. Unfortunately I dislike huge files and rather separate things into dedicated modules...
So the right thing to do here is probably "just" to resolve the downstream dependencies.

After all: I DO want to write good code.
On the one hand we're still in Python, a scripting language! Anything goes! Pylance won't even flinch overriding a built-in type. (That's where other linters as pylint come in handy.)
On the other: I DO want some help. Having no code completion is a no-go. Also obvious errors like misspelled members must be highlighted! So turning "basic" type checking OFF is out of the question.
Its just that it feels like there is not much middle ground. Investing in this ... Stepping into this feels like your code either needs to be good for rocket-surgery or you don't get help.
So. Thanks again for explaining. 🙏
I'm having a hard time not feeling put down again. But that's just me. Nothing personal. We're here because there are specifications to keep out opinions. :)
... this. Pardon me, probably needs to go to some blog post.


Add env_var parameter to dev appserver

Dev appserver now exposes --env_var for passing in environment variables

@loosebazooka I don't understand how these variables are going to be used for Java Standard apps since they configure their environment variables via the appengine-web.xml.

@patflynn I wrote up a short explanation of why Java Standard apps need access to environment variables here: https://github.com/GoogleCloudPlatform/app-maven-plugin/issues/150

This seems to have been done in https://github.com/GoogleCloudPlatform/appengine-plugins-core/commit/93567bd1932ae16e32d02a2a571824eb68aea653


Fuzzing examples database

Export Hypothesis' example database feature to our users.

I think this is too specific and niche. Not worth working on now.


Starving extension host can block file navigation

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

VS Code Version:
OS Version:

Steps to Reproduce:

Open VS Code repository
While TS server is still initializing, try to navigate files through quick pick
The file search will not work until TS server finishes initialization

It feels wrong that any extension can block the file navigation. Maybe we should run the search extension in its one extension host (through extension affinity).

@rebornix is this still happening? I was not able to repro
https://github.com/user-attachments/assets/3e548545-c2bb-4fa7-87b4-c43d2de5dcd1


Styling - Website

To style website based on wire frames designed by Sierra.

Top, side navigation
Documentation container
Home page

dependent on...

Add data team to issue

Mobile Styling:

nav bar
sidebar
home page

@s-duffey and @LemieuxGen to review when first draft complete.

Added one new comment to checklist (see above)

Add more padding to bottom of documentation, so that I can scroll a bit below the bottom of the article.


Using ResourceSerializer for integration tests

Hello,
I'm using trying to write some end-to-end tests for my jsonapi-resources application.  Currently I build simple hashes and POST/PUT json representations of them but thought it would be nice to start with a resource object and use the serializer to build some jsonapi compliant payload to send across in my tests.
Do you have any guidance on how to use the serializer to do this with a new object for POSTs?  Existing objects work fine based on the guidelines in the README.
Many thanks.

I haven't tried to do what you are asking. In my own testing I simply pass in a hand coded hash in the controller tests.
The Resource object can be initialized with any model. You could create a simple wrapper around a hash that mocks the expected methods on the models. I'm not sure how much that would help over simply maintaining a hash in the JSON-API format however.

Thanks @lgebhardt .  I'll stick to hashes myself then as well.  Great job with this.  I hope to contribute in the future.


Properly handle ranking of ties

The calculation of confusion matrices currently does not handle ties in the ranking. This needs to be fixed and could lead to incorrect results in the meantime. This only affects types of scores where ties are possible such as class labels. Floating point scores may be affected if there are repeated numbers due to some artifact of the scoring process (e.g. zero scores, default scores, bounded scores).

Fixed in commit 1c5c3d4.


Axon Development Updates 12/21/2022

Sometimes you go out for coffee not for the coffee (although they have the best Hambella in town), but for other species. 🐈
If you've read the last update #900, you should know that Axon has done most of the base module development and EVM compatibility debugging.
At this stage, our main focus is on interoperability and the abstraction of the economic model.
What we have accomplished

We have finalized the design of the cell visitor. #896 presents our current solution that grants Axon to access cells in CKB. It is a modular solution and can reuse the existing infrastructure in the CKB ecosystem.
#899 describes the design of gassless transactions, a highly requested feature which is mainly about sending transactions and interacting with dApps without paying gas fees by the user. We put forward a system contract to handle such transactions to reduce any difficulty of adaptation for developers.

What we are doing

To enable better development of system contracts, we have refactored the interface in executor module #901.
The implementation of the design described in #896 consists of three parts, where the following two have the highest priority:

The image cell system contract (ICSC) is the storage module of CKB cells and block headers. Although it is a system contract, the interface is generated by Solidity. Besides, other features of ICSC are also under development here #917. We are open to any suggestions and questions. Feel free to create a pull request!
Cell emitter is an indexer-like component, which needs to index a specific cell and interact with ICSC through a relayer. The code is here.

The staking mechanism design is still in progress, and the assignee is developing the POC in #898.
We are redesigning Axon’s logo and we’d like to know what you think #923.

Contact us to learn more:

Mail
GitHub
Docsite
Twitter
Medium

looking forward to it


[Feature] Enable/disable Proxy Groups on Servers and Groups

Verify Steps

[X] Tracker 我已经在 Issue Tracker 中找过我要提出的问题
[X] Need 当前 OpenClash 并不包含该功能特性或者还不完善
[X] Framework 这是 OpenClash 应包含的特性, 并非 Clash 特性
[X] Meaningful 我提交的不是无意义的 催促更新或修复 请求

Describe the Feature
1. Proxies section on Servers and Groups has too many column so edit and delete button not showing (need scroll to right).
2. Proxy Groups section doesn't have enable/disable feature.
Describe the Solution
1. Please optimize Proxies section column so all will fit without scroll to right whenever edit or delete.
2. It is possible to add feature enable/disable on Proxy Groups? or it was intended like that?
Thanks.
Describe Alternatives
No response
Additional Context
No response

thanks for implementation


"Valea Viei" is listed in the drop-down menu

Description:
"Valea Viei" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Valea Viei" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Viei, GORJ (TURCENI)".


aws_elasticache_cluster - configuration_endpoint not returned as JSON

The following query:
SELECT configuration_endpoint FROM aws_elasticache_cluster

Returns:
{Address:0xc0013b6690 Port:0xc00062bb5c noSmithyDocumentSerde:{}}

Instead of a JSON
{ "Address": "cluster1.zqck3t.cfg.euw1.cache.amazonaws.com", "Port": 11211 }

Steampipe v0.23.2
aws@latest, 0.139.0

@ParthaI,
It's not returned as JSON but a map, e.g. the query
SELECT configuration_endpoint->>'Address' FROM aws_elasticache_cluster

returns ERROR: operator does not exist: text ->> unknown (SQLSTATE 42883)
Steampipe v0.23.2
aws@latest 0.142.0


Support Facebook Graph API 2.2

To support Graph API 2.2 I created a subclass, and updated the version in the urls. Is this all that should be needed? Everything seems to work ok for me so far.
class Facebook22OAuth2(Facebook2OAuth2):
"""Facebook OAuth2 authentication backend using Facebook Open Graph 2.2"""
AUTHORIZATION_URL = 'https://www.facebook.com/v2.2/dialog/oauth'
ACCESS_TOKEN_URL = 'https://graph.facebook.com/v2.2/oauth/access_token'
REVOKE_TOKEN_URL = 'https://graph.facebook.com/v2.2/{uid}/permissions'
USER_DATA_URL = 'https://graph.facebook.com/v2.2/me'

Same here, FYI.   Worked like a charm.

Sorry about the delay, I've updated it to v2.3 (which changed the message response from querystring to json).


Harbor fails to install when setting tlsCertificateSecretName

Bug Report
The secretName for the ca-download volume in the harbor-core Deployment is hard-coded to harbor-tls.
If tlsCertificateSecretName in the harbor values.yaml file is set to any value other than harbor-tls, then the secret can't be found, the ca-download volume is not satisfied, and the harbor-core Pods fail to start.
The simplest fallback may be to set the secret as optional in the harbor-core overlay, but this will cause a silent failure when:

self-signed certificates are being used, and
tlsCertificateSecretName is not set to harbor-tls
In these cases, the volume would be empty, causing the CAs to not include the self-signed certificate.

Additionally, in my case, I was using a delegated certificate for contour, outside the harbor namespace (e.g. tlsCertificateSecretName: developer-certificates/wildcard). The solution to this problem needs to account for this use case and ensure that the secret name would pass Deployment API validation and potentially be separately configurable.
Finally, there should be a way to omit the ca-download volume & mount when using production TLS certificates (e.g. lets-encrypt). Including the production certificate in the CA bundle is cryptographically invalid (and dangerous). It would probably be better to disable this behavior by default, but existing users using self-signed certificates would have to enable the option, so it is a breaking change.
cc @stealthybox @vrabbi
Environment Details

Build version (tanzu version):  Harbor package 2.3.3

Additional Notes
I would be happy to be involved in fixing this once a decision is reached about the best approach :)

For anyone else setting a non-default tlsCertificateSecretName (delegated or not), this workaround works:
cat <<EOF | kubectl apply -f -
---
apiVersion: v1
kind: Namespace
metadata:
  name: harbor
---
apiVersion: v1
kind: Secret
metadata:
  name: harbor-tls
  namespace: harbor
EOF

Then use tanzu package install harbor as usual to install the package.

For those who prefer a quick fix that removes the ca-download volume and mount altogether, you can use the following workaround.

Note:

This installs the Harbor package declaratively so you don't need to use tanzu package install harbor.
Notice the PackageInstall has an annotation that references an overlay Secret below, which contains a ytt overlay to remove the volume and mount
The values Secret contains the modified harbor config values. Make sure to update them to your hostname, TLS secret name, and other passwords.
If you use a different package-management namespace, update namespace: default in all resources below
You can use tanzu package installed delete harbor --yes to remove the package. You will get an error saying it failed to delete a clusterrole, but you can safely ignore this error for the purposes of this workaround.

cat <<EOF | kubectl apply -f -
apiVersion: packaging.carvel.dev/v1alpha1
kind: PackageInstall
metadata:
  annotations:
    tkg.tanzu.vmware.com/tanzu-package-ClusterRole: harbor-default-cluster-role
    tkg.tanzu.vmware.com/tanzu-package-ClusterRoleBinding: harbor-default-cluster-rolebinding
    tkg.tanzu.vmware.com/tanzu-package-Secret: harbor-default-values
    tkg.tanzu.vmware.com/tanzu-package-ServiceAccount: harbor-default-sa
    ext.packaging.carvel.dev/ytt-paths-from-secret-name.0: harbor-patch-overlay
  name: harbor
  namespace: default
spec:
  packageRef:
    refName: harbor.community.tanzu.vmware.com
    versionSelection:
      constraints: 2.3.3
  serviceAccountName: harbor-default-sa
  values:
    - secretRef:
        name: harbor-default-values
---
apiVersion: v1
kind: ServiceAccount
metadata:
  name: harbor-default-sa
  namespace: default
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: harbor-default-cluster-rolebinding
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: cluster-admin
subjects:
  - kind: ServiceAccount
    name: harbor-default-sa
    namespace: default
---
apiVersion: v1
kind: Secret
metadata:
  name: harbor-default-values
  namespace: default
stringData:
  harbor-default-values.yaml: |
    hostname: harbor.example.com
    tlsCertificateSecretName: developer-certificates/wildcard
    harborAdminPassword: ABCDEFG890123456
    secretKey: ABCDEFG890123456
    database:
      password: ABCDEFG890123456
    core:
      secret: ABCDEFG890123456
      xsrfKey: ABCDEFG890ABCDEFG890ABCDEFG89012
    jobservice:
      secret: ABCDEFG890123456
    registry:
      secret: ABCDEFG890123456
---
apiVersion: v1
kind: Secret
metadata:
  name: harbor-patch-overlay
  namespace: default
stringData:
  harbor-patch-overlay.yaml: |
    #@ load("/kinds.lib.yaml", "kind_overlays")
    #@ load("@ytt:overlay", "overlay")

    #@ kind = kind_overlays
    #@ core_metadata = overlay.subset({"metadata": {"name": "harbor-core"}})

    #@overlay/match by=overlay.and_op(kind.deployment, core_metadata)
    ---
    spec:
      template:
        spec:
          containers:
            #@overlay/match by="name"
            - name: core
              volumeMounts:
                #@overlay/match by="name"
                #@overlay/remove
                - name: ca-download
          volumes:
            #@overlay/match by="name"
            #@overlay/remove
            - name: ca-download
EOF

Credits: paired with @stealthybox  :)

Thanks all, we will fix this in harbor 2.4.2 and backport it to 2.3.3.

I'm not sure if this was fixed as intended.
Additionally, in my case, I was using a delegated certificate for contour, outside the harbor namespace (e.g. tlsCertificateSecretName: developer-certificates/wildcard). The solution to this problem needs to account for this use case and ensure that the secret name would pass Deployment API validation and potentially be separately configurable.
I too am using a wildcard cert in different namespace, and just specifying tlsCertificateSecretName such as "developer-certificates/wildcard" (and nothing in the tls fields) results in a proper Contour HTTPProxy, but harbor-core fails to launch because of failure to mount a volume.

In case you hadn't seen it, take a look at the README.md for this project. You may need to check out the free version of the commercial product to see if issues have been addressed there.


docs: update the documentation for new users

Hello @yokwejuste, I could help work on this

Go on @Davidthecode , feel free to open a PR


Add documentation for watch plugins

WIP: Do not merge
Contributes to #5478
This PR adds documentation for watch plugins.
It is still in a really rough spot, but feedback is more that welcome!!

This is a great!
The only think missing is an explination of how you register a plugin -- will that be through an option similar to runner and reporters like watchers?

Can we get this merged?

Almost, although I wonder if we need points 1 and 2 of https://github.com/facebook/jest/issues/5478 to be resolved first

Should we stick the docs into master, and do a backport to 22.4 if needed later?

Just pinging this again. Can we merge this PR this week?

what about just setting it to utf8 and see what happens? I suppose we can also set it manually to hex if any of our plugins needs it anyways?

Codecov Report

Merging #5895 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5895   +/-   ##
=======================================
  Coverage   64.22%   64.22%
=======================================
  Files         220      220
  Lines        8481     8481
  Branches        3        4    +1
=======================================
  Hits         5447     5447
  Misses       3033     3033
  Partials        1        1

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 77d2e8e...9cee1de. Read the comment docs.


config server using vault but Unable to display source node

I use vault as the configuration.
config-client can't show source node , but postman can show info.
client config :

client launch:

postman show :

I am debug the config server having info , but client can't show source info.

Why do you expect it to show the source in the logs?

How do I get the foo parameter? in config client ?
becase I can't get parameter for this code.

when I start the config client.

Have you set spring.application.name? This is what config server uses to look up an application, not spring cloud config name.

this is config.

but, I start the config client , I cant debug config server.

Unfortunately, there's not much we can do with the information and screenshot you've provided.
Can you provide a complete, minimal, verifiable sample that reproduces the problem?

Hmm, I don't know why. I restart it cant work and cant get paramter.

I haven't changed anything.

Unfortunately, there's not much we can do with the information and screenshot you've provided.
Can you provide a complete, minimal, verifiable sample that reproduces the problem?

this is the project address.
https://github.com/zhongzunfa/ch13-1.git

What about vault? Did you try setting spring.application.name=myapp?

Yes about vault , I don't try setting spring.application.name=myapp,  I am all setting in project
https://github.com/zhongzunfa/ch13-1.git

try setting spring.application.name=myapp. That's how config server works.

the config server setting spring.application.name=myapp, or other , It still works.
and I find changing spring.cloud.config.name=myapp or application, the config client can get different paramter.

I think it is right. Isn't my understanding wrong?

spring.application.name should be set in the client not the server

Is this just confusion about which endpoint the application should hit? Rather than / you should hit /-.json (or .yaml or .properties)?

@HarryEMartland no, the is the config client.


offsets incorrect when drawing on canvas

desktop is fine - just get offsetX, offsetY from mouse event
Touch doesn't have these, and insists on being dumb all the time.

use a 3rd party library that just works ™


MenuItem order wrong for .addExitMenuItem()

When building a Tray Icon using the Builder API and adding .addExitMenuItem() the exit menu item shows up on top of the list.
Check:
FXTrayIcon trayIcon =
            new FXTrayIcon.Builder(stage, resource).menuItem("Settings",
                    e-> myMethod()).separator().menuItem(item).addExitMenuItem().show().build();

@azplanlos - Yeah, this is been fixed in pull request #47 which has been merged, but not published yet...
Heres a screen shot from that PR:

Closing this as @EasyG0ing1's commit fixed this. I've just been too busy to publish a release. I will publish the next release with this fix later this evening. Thank you both for bringing this to my attention.


Uncaught Error: channel closed

[Enter steps to reproduce:]

...
...

Atom: 1.25.0 x64
Electron: 1.7.11
OS: Microsoft Windows 10 Pro
Thrown From: platformio-ide-terminal package 2.8.1
Stack Trace
Uncaught Error: channel closed
At events.js:163

Error: channel closed
    at ChildProcess.target.send (internal/child_process.js:562:16)
    at Task.module.exports.Task.send (~/AppData/Local/atom/app-1.25.0/resources/app/src/task.js:113:33)
    at PlatformIOTerminalView.module.exports.PlatformIOTerminalView.input (/packages/platformio-ide-terminal/lib/view.coffee:271:17)
    at /packages/platformio-ide-terminal/lib/view.coffee:149:8)
    at Terminal.EventEmitter.emit (/packages/platformio-ide-terminal/node_modules/term.js/src/term.js:109:12)
    at Terminal.handler (/packages/platformio-ide-terminal/node_modules/term.js/src/term.js:3493:8)
    at Terminal.keyDown (/packages/platformio-ide-terminal/node_modules/term.js/src/term.js:3196:8)
    at /packages/platformio-ide-terminal/node_modules/term.js/src/term.js:594:29)

Commands

Non-Core Packages
atom-django 0.3.2
autocomplete-python 1.10.5
platformio-ide-terminal 2.8.1

Hello there, I used to follow previous advice that says...
"Try to open a new terminal on package tab"
...but that didn't work all the time and now it just does not seem to be working anymore.  Here's more info on what I get:
Error: channel closed
at ChildProcess.target.send (internal/child_process.js:562:16)
at Task.module.exports.Task.send (C:\Users\Ricardo\AppData\Local\atom\app-1.27.2\resources\app\src\task.js:113:33)
at PlatformIOTerminalView.module.exports.PlatformIOTerminalView.input (file:///C:/Users/Ricardo/.atom/packages/platformio-ide-terminal/lib/view.coffee:271:17)
at Terminal. (file:///C:/Users/Ricardo/.atom/packages/platformio-ide-terminal/lib/view.coffee:149:8)
at Terminal.EventEmitter.emit (C:\Users\Ricardo.atom\packages\platformio-ide-terminal\node_modules\term.js\src\term.js:109:12)
at Terminal.handler (C:\Users\Ricardo.atom\packages\platformio-ide-terminal\node_modules\term.js\src\term.js:3493:8)
at C:\Users\Ricardo.atom\packages\platformio-ide-terminal\node_modules\term.js\src\term.js:3253:12

I love Atom!!!!!  But I am experiencing the same issue on Windows 10 Pro.  When you first startup Atom, and then you click on the + button on the bottom left hand corner to launch the terminal, I receive a random number of errors indicating uncaught exception error.  I have to close and reopen a new terminal anywhere up to 11 times in order for the command prompt to appear.  I used to only have to do this 4 times every time I tried to launch the terminal, but now it seems to be getting worse.  How can I help get this on a list to be fixed?

@dunkinman @githubrarp @juliocvbgomes  Hey Guys I was also facing same problem a lot of times. I installed one more package ("platformio-ide-debugger") in Atom. Now it's not giving that error. I'm using w10 pro.

@ppl monitoring this.
These issues also need to be reproduced with current builds of both products and if fails still
also trying what @finnabc123 sugested which

As luck would have it, my system crashed today and upon coming back up, I still had the problem even after installing the platformio-ide-debugger package.  I then rebooted again, and now the problem seems to have gone away as I am no longer getting the error when I launch the terminal.  Not enough data to support that it's fixed, but I'm wildly optimistic.  Installing platformio-ide-debugger seems to have done the trick.  Suggest leaving this open for now.  I will report back in a couple of weeks or sooner if the problem reoccurs.  @finnabc123 thanks for the suggestion.  Was a mild annoyance, usually accompanied by panic that I'd have to switch OS or editor...wasn't thrilled about either.  So far so good....

Unfortunately even after installing platformio-ide-debugger, this issue continues to plague me.  I had to restart Atom yesterday, and the same behavior continues.  What exactly do I need to provide to help with the resolution of this issue?  @githubrarp @juliocvbgomes @finnabc123 @the-j0k3r

In one comment you say its fixed, the other its not, you talk about system crashes, who knows if the issue is even with this package or Atom.
Im not convinced it is.

I used to run this same config with Windows 10, not the Pro edition.  Once I switched over to the Pro version via a new laptop, the issue started.  The crash was unrelated to this event.  Been running this config for about a year, and have always had this error message.  Maybe there's another package I'm running in atom that causing this?

Perhaps there's a problem with my setup or my settings?  It's strange to me that opening and closing the editor an undetermined amount of times resolves the issue on it's own.  I would think that it would work or wouldn't.  Here are my settings.  Maybe something is wrong with them?  I followed a tutorial a long time ago to install the terminal editor.  Maybe something needs to change with the new laptop now that I'm running a 64 bit machine and Window10 Pro OS?

I just spent the last hour getting more information about this issue.  The issue only seems to occur after a reboot.  Installing the platformio-ide-debugger has seemed to help, as I only seem to now get an error during my initial launch of Atom after the reboot.  It took several attempts to be able to successfully launch the platformio-ide-terminal until it actually started successfully.  I took a screenshot of the error that I receive upon my initial launch of the platformio-ide-terminal as shown below.  I no longer get the channel closed error message rather it is an Assertion failed!  message.

Clicking on retry and then opening and closing the terminal up to a dozen times eventually gives me a command prompt so that I can interface with the OS.

Sorry, cant help or duplicate your issue.

I've have the same error after updating Atom and can't get rid off it. Also disabling the "platformio-ide-terminal" doesn't help since it is mandatory for "Update PlatformIO Core" and Update platforms, packages and libraries" :(

Coincidentally...an update for Atom came out today 1.28.2 x64.  I updated Atom to 1.28.2 x64 and after a reboot, only received the Assertion Error 1 time when I tried to launch the terminal.  For fun, I rebooted again and when I launched the terminal, did not receive an error.  For more fun, because it is Friday night...I rebooted again...and did not receive the error after this reboot either.  Can't say for certain it's fixed but it appears to be trending in the right direction.


fix: add append to the allowed rule fields

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
#955
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: vjjmiras
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [vjjmiras]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 40c9b73c44ebce2abed68d14b25588ea20b47ac6

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, vjjmiras
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [Issif,vjjmiras]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add .Wait to Events/Functions.

What's the use case for something like this?

Example you create a new thread and way for the client to accept the trade request and then continue to create the trade, just a random quick idea


Migrate codebase to modern JS

Pull Request Test Coverage Report for Build 413

196 of 200 (98.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+6.5%) to 96.053%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/helpers.js
141
143
98.6%

src/index.js
55
57
96.49%

Totals

Change from base Build 410:
6.5%

Covered Lines:
198

Relevant Lines:
202

💛  - Coveralls


Implement a "basic" renderer for PropertyReportTask

Expected Behavior
As noted on #19493, it'd be great, when we're using a single --property, to get output such as:
./gradlew properties --console=plain -q -Dproperty=version
7.3.0-SNAPSHOT

Which allows use in scripts.
Current Behavior
Currently, when using the changes in #19493, we'd see:
./gradlew properties --console=plain -q
------------------------------------------------------------
Root project 'reporting-plugin' - Cucumber reporting plug-in
------------------------------------------------------------

version: 7.3.0-SNAPSHOT

Context
As per #19493, this allows i.e. listing a specific project's version property, but could also allow other automagic interactions.
Would depend on #19493 and #19917.

@jamietanna #19500 should be merged to the master branch soon. Are you interested in contributing this feature too?
Btw, similar  to #19500, I think the renderer should be controlled with another optional task option (--renderer):
Use-cases
1) default renderer has original output
./gradlew properties --renderer=default -q
------------------------------------------------------------
Root project 'reporting-plugin' - Cucumber reporting plug-in
------------------------------------------------------------

version: 7.3.0-SNAPSHOT
allprojects: [project ':reporting-plugin']
...

2) basic renderer omits header
./gradlew properties --renderer=basic -q
version: 7.3.0-SNAPSHOT
allprojects: [project ':reporting-plugin']
...

3) using nonexistent renderer results in build failure
./gradlew properties --renderer=basic -q

FAILURE: Build failed with an exception.

* What went wrong:
Problem configuring task :properties from command line.
> Unknown renderer: basic.

Hey, yes I think I'd like to get it contributed - I would probably want there to be a way of omitting the property name, so we see:
./gradlew properties --console=plain -q -Dproperty=version
7.3.0-SNAPSHOT

Or if we're listing all properties, we get:
./gradlew properties --renderer=basic -q
version: 7.3.0-SNAPSHOT
allprojects: [project ':reporting-plugin']

Not sure if we'd want that to be a separate rendered?

It makes sense to use the same basic renderer for this.

I'd also be very much interested in omitting the header and property name when querying a single property, in order to pipe the property's value (in my case the version) to somewhere else. @donat, any hints how to move this forward?


Yeelighter error for ceiling14

Hi,
I have yeelight ceiling14 and I installed the Yeelighter plugin to connect to it in homebridge but encountered error below:
[Yeelighter] ingoring device with corrupt DeviceInfo {
location: 'yeelight://undefined',
id: undefined,
model: undefined,
support: undefined,
power: false,
bright: 0,
color_mode: -1,
ct: 0,
rgb: 'string',
hue: 0,
sat: 0,
host: 'string',
port: 0,
debug: false,
trackedAttributes: [],
fw_ver: '0,0.0',
name: 'string'
}
Appreciate any suggestion how to solve this issue.
Thanks.

Looks like you have manually configured a device in the config. Removing that should solve this.

below is the default settings from the config, is it not correct? how should I change it?
{
"name": "Yeelighter",
"timeout": 5000,
"interval": 60000,
"ctforcolor": true,
"blocking": false,
"split": true,
"manual": [
{
"ignorePower": false
}
],
"platform": "Yeelighter"
},

and the error:

It looking at each object in the "manual" array as a device, since you've created a "device" with only "ignorePower" parameter, it says that the data of this device is corrupted (missing "id" and "address" that are required).
Just remove { "ignorePower": false } from the "manual" array.


Define Initial API Endpoints

Real Neighbourhood Data
GET /api/feed

Drawer Data
GET /api/feed/<neighbourhood_id>

Mock Neighbourhood Data
GET /api/feed/mock

Retrieve Form Data
POST /api/volunteer-form

Name, surname, email, phone, kvkkAccepted, neighbourhoodId, (ip etc?)
Validate fields
All fields are required

https://idratherbewriting.com/learnapidoc/docapis_resource_endpoints.html This documentation can be used for instructions.

I'm working on it

it's merged on #24


⚠ DDoS-Relay US-W-sfo3-01 has degraded performance

In cc2638b, DDoS-Relay US-W-sfo3-01 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 146 ms

Resolved: DDoS-Relay US-W-sfo3-01 performance has improved in 3e16630 after 12 minutes.


Remove TravisCI configuration

Many PASS repos still have a .travis.yml configuration file, which was used for automated build/test when the project was using TravisCI for this purpose. Now that we've transitioned to using GitHub Actions instead, these should be removed.
Note that the .travis.yml file can make reference to other files/scripts, so each should be reviewed to determine if there are other files that were used as part of the TravisCI build, as those additional files would likely also need to be removed.
The only potential lingering value of these files is a reference for building portions of the GitHub Actions workflow. Prior to removal, each file should be reviewed to determine if there is any lingering value as a reference.

Tech debt

Almost done, I only see it pass-authz.

From scrum: pass-authz is no longer used. Only look at the currently deployed repos and confirm that TravisCI is not in those repos.

Main: TravisCI does not exist (although there is mention of it in the pipeline.md - perhaps can be cleaned up during documentation tickets)
Pass-Core: TravisCI does not exist
Pass-Support: TravisCI does not exist
Pass-UI: TravisCI does not exist
Pass-Auth: TravisCI does not exist
Pass-Acceptance-Testing: TravisCI does not exist
Pass-Docker: TravisCI does not exist


azure-sql-edge docker images fail on kernel 6.7+

Hey there,
this is a follow up to #868 , which was about a bug that prevented the MSSQL server container to start on docker when running on kernel 6.7 and later.
The bug seems to also affect azure-sql-server images, whose last update was 13 months ago (so before the fix for mssql-server was released).
This has been reported by several users whose workflow was broken after updating to Docker Desktop 4.33, which ships kernel 6.10: issue
Additionally, users also report non-2022 mssql-server images being affected by the issue.
Questions:

Are there plans to update the azure-sql-edge images to include the fix for kernel 6.7+?
Are there plans to update non-2022 mssql-server images to include the fix?

Thanks!

This article indicates there won't be any further support releases for azure-sql-edge
https://techcommunity.microsoft.com/t5/sql-server-blog/azure-sql-edge-update-september-2023/ba-p/3930827

Hello, I'm using mcr.microsoft.com/azure-sql-edge:latest on my M1 Mac, is there a better image to run MSSQL in Docker?

@albertovelo
azure-sql-edge does no longer support Apple Silicon, see https://techcommunity.microsoft.com/t5/sql-server-blog/azure-sql-edge-update-september-2023/ba-p/3930827
Good news: Docker for Mac includes virtualization for AMD images (under Settings > General), so it's possible to use the image mcr.microsoft.com/mssql/server:2022-latest which includes the fix for the newer linux kernels.
See also https://www.justinjbird.me/blog/2024/running-sql-server-on-macos-using-docker/

@nuarhu thank you so much !  :-D

Also, the latest Cumulative Update for the SQL Server 2019 container includes the fixes, so I was able to get that working on my latest docker install (34)


Evaluate OpenSSF's allstar

We may want to evaluate https://github.com/ossf/allstar to ensure that we are following the good security practices.
Allstar will open issues like this (after 24 hours)

https://github.com/GoogleContainerTools/distroless/issues/861
https://github.com/GoogleContainerTools/distroless/issues/771
(config: https://github.com/GoogleContainerTools/.allstar)

cc @jandubois : I'm going to install this

Good idea! I just saw that one of the CNCF graduation requirements is

Have achieved and maintained a Core Infrastructure Initiative Best Practices Badge.

Can't start too early to work on that! 😄

Installed the app and more than 24 hours passed, but nothing seemed to happen 😞

copy

We can just create single-line SECURITY.md with "See https://github.com/lima-vm/lima/blob/master/SECURITY.md" so that we do not need to sync the document across all the repos


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 575

Questions ?
Please refer to the Backport tool documentation


deploy:writable not working on CentOS

I had issues where deploy:writable is not actually changing the configured directories writable. Here is the full output with -vvv. The release is CentOS 7.1.1503. Something wrong with the chmod detection here, since it returns "chmod: missing operand"?
Executing task deploy:writable
ÔñÀ on [puuhax]
Run: ps aux | grep -E '[a]pache|[h]ttpd|[_]www|[w]ww-data|[n]ginx' | grep -v root | head -1 | cut -d\  -f1
> nginx
>
Run: cd /usr/share/nginx/html/project/releases/20150522010612 && chmod 2>&1; true
> chmod: missing operand
> Try 'chmod --help' for more information.
>
Run: cd /usr/share/nginx/html/project/releases/20150522010612 && if hash setfacl 2>/dev/null; then echo 'true'; fi
> true
>
Run: cd /usr/share/nginx/html/project/releases/20150522010612 &&  setfacl -R -m u:"nginx":rwX -m u:`whoami`:rwX storage vendor
Run: cd /usr/share/nginx/html/project/releases/20150522010612 &&  setfacl -dR -m u:"nginx":rwX -m u:`whoami`:rwX storage vendor
done on [puuhax]
Ok

I have circumvented this problem by running this manually for now:
task('storage:writable', function () {
    run('cd {{release_path}} && chmod -R 777 storage');
    run('cd {{release_path}} && chmod -R 777 vendor');
});

Hi! You can override tasks:
task('deploy:writable', function () {
    run('cd {{release_path}} && chmod -R 777 storage');
    run('cd {{release_path}} && chmod -R 777 vendor');
});

Can you run which setfacl and show output?
And this: if hash setfacl 2>/dev/null; then echo 'true'; fi

[root@puuhax ~]# which setfacl
/usr/bin/setfacl
[root@puuhax ~]# if hash setfacl 2>/dev/null; then echo 'true'; fi
true

So setfacl is exists.
Try run: setfacl -R -m u:"nginx":rwX -m u:whoami:rwX storage vendor
And this sudo too.

Hi,
That is the exact same command by that is run by deploy:writable? Well I run it manually anyway, the end result is the same. One of packages requires a subfolder to be writable. If run chmod -R 777 vendor it runs fine.
[24-May-2015 13:07:12 UTC] PHP Fatal error:  Uncaught exception 'ReflectionException' with message 'Class bugsnag does not exist' in /usr/share/nginx/html/project/releases/20150524160306/vendor/laravel/framework/src/Illuminate/Container/Container.php:776

Hmm, here the getfacl before and after running the recursive chmod.
[root@puuhax current]# getfacl vendor/
# file: vendor/
# owner: root
# group: root
user::rwx
user:root:rwx
user:nginx:rwx
group::r-x
mask::rwx
other::r-x
default:user::rwx
default:user:root:rwx
default:user:nginx:rwx
default:group::r-x
default:mask::rwx
default:other::r-x

[root@puuhax current]# chmod -R 777 vendor/
[root@puuhax current]# getfacl vendor/
# file: vendor/
# owner: root
# group: root
user::rwx
user:root:rwx
user:nginx:rwx
group::r-x
mask::rwx
other::rwx
default:user::rwx
default:user:root:rwx
default:user:nginx:rwx
default:group::r-x
default:mask::rwx
default:other::r-x

Notice how the other is now changed to 'rwx'. Then I realized that php-fpm is running on the username 'apache' instead of 'nginx'. Changing that and restarting php-fpm did the trick.


Stop Ghouls that are in the process of harvesting from being teleported by Plague of Undeath

Feedback tells us that having all Ghouls being teleported is annoying because they tend not to be in Control Groups, so they block the player's real army. We should be able to exclude Ghouls that are currently harvesting lumber using the GetUnitCurrentOrder native function.

I’ll grab this one.


[EOEPCA/IAM] Create overall Helm Chart of the IAM BB

Currently the IAM BB is deployed via standard Helm charts of Keycloak, OPAL and APISIX, which are glued together using an ArgoCD app of apps. As an alternative or replacement for the latter, a custom Helm chart should be provided in order to reduce the deployment effort. The recent addition of the Identity API to the IAM BB also increases the importance of an overall Helm chart.

Optimization of the helmchart structure to simplify the usage.
An entire Helm chart is no longer preferred. The use of two Helmcharts is more user-friendly. One chart for the PEP component (APISIX). One chart for the PDP/PIP components.

There is a single Helm chart that allows installing the complete IAM at once or single components of it. So APISIX can be installed separately if desired.
Note, however, that there is a chicken-and-egg problem regarding client secrets: Clients are referenced by routes, which are installed by the Helm chart. However, they can only be when Keycloak is available. A way to handle this is to store the client secrets as K8s secrets that initially contain a dummy client secret, which is changed to the actual value when it is available.


Internal anchor links/links with fragment identifiers. Fixes #1049

Fixes #1049
Tested on Chrome, Safari and Firefox.

User is able to input some know fragment identifier.
Anchor field currently lacks any validation.

merged in via https://github.com/wagtail/wagtail/commit/794d40b86b3d9c652c37acbc00163890077a78b5 thanks @imansyed000

Awesome! Thanks a lot @imansyed000 this PR will help me understand how the all thing works and be used in my project as well! 👍 🎉

https://github.com/jacobtoppm/wagtail_draftail_anchors/ provides the ability to add/edit anchors so should go well with this feature

Wonder if we should have this switched off by default? It's not a commonly requested feature and requires some frontend development to make it work. I worry it could actually be confusing for editors of sites that don't actually support it.

Wonder if we should have this switched off by default? It's not a commonly required feature and needs some frontend development to make it work. I worry it could actually be confusing for editors of sites that don't actually support it.

There also seems to be a bug in draft vs publish. I coupled this with the package zerolab linked above your reply, which is really a requirement to use this because there's otherwise no way to insert anchor ids, and when saving a draft all of the links got converted to 'default' draftail links.  After editing them again, and not previewing them or saving a draft but rather publishing the changes directly, they stuck properly.


Set telemetry user tracking default to be disabled

Description

Closes #285
How Has This Been Tested?

Try to get rid of the disableTracking field in the dashboard config to see if the telemetry tracking is disabled by default.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/assign

Do we want to squash this down to one commit? Otherwise looks good.
/lgtm


Additional column headings?

Hello,
Thanks for your great tool (and the detailed explanations in the Readme, which made me understand Explorer's behavior a lot better)!
Is it possible to somehow provide additional column headings? You wrote:

There are many more heading choices in Explorer, but these are the ones that are most likely to be applicable globally.

But I'm missing at least the "Owner" column which I have included in all my views. And others users may also have some more columns they regularly require. I understand that offering the full list of more than hundred columns which are available in Windows is neither feasible nor useful, but a way to add a few more than the current sixteen would be fine…
This could be accomplished in two ways:

Fixed/for all users: statically adding new columns to your tool from time to time which got requested by users (like me in this case)

or

Flexible/for each user: offering functionality where users themselves could enter the columns they need (e. g. via some technical ID referencing the column heading as used by Explorer)

Thanks for considering such a feature!

Thank you for the feedback and excellent suggestions. Related to this, I have another request to also allow setting global column widths. I'll try to work both into a future upgrade. It will require a significant UI redesign and probably will include a more advanced method to save user's choices, such as an INI file, XML file, or registry entries. I'll also consider a quick fix, to just add a few more column headings, depending on what other requests come in.

I'm thinking of adding four more column choices for now and then later do a
major UI redesign to accommodate user column choices. In addition to
"Owner", what other three column headings would you suggest?
On Mon, Mar 29, 2021 at 2:39 AM engelhro @.***> wrote:

Hello,
Thanks for your great tool (and the detailed explanations in the Readme,
which made me understand Explorer's behavior a lot better)!
Is it possible to somehow provide additional column headings? You wrote:
There are many more heading choices in Explorer, but these are the ones
that are most likely to be applicable globally.
But I'm missing at least the "Owner" column which I have included in all
my views. And others users may also have some more columns they
regularly require. I understand that offering the full list of more than
hundred columns which are available in Windows is neither feasible nor
useful, but a way to add a few more than the current sixteen would be fine…
This could be accomplished in two ways:

Fixed/for all users: statically adding new columns to your tool
from time to time which got requested by users (like me in this case)

or

Flexible/for each user: offering functionality where users
themselves could enter the columns they need (e. g. via some technical ID
referencing the column heading as used by Explorer)

Thanks for considering such a feature!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/brake4bots/WinSetView/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS25QO5WN6N2OE6ZNTNR7Q3TGAVEFANCNFSM4Z662OCA
.

[…] Related to this, I have another request to also allow setting global column widths.

Ah, that would be awesome! Especially as some widths cannot be set manually in the Explorer (the input field seems to be only three digits wide, so a width of more than 1000 cannot be entered – you can only achieve such widths by dragging the column heading).

[…] probably will include a more advanced method to save user's choices, such as an INI file, XML file, or registry entries.

That would be pretty neat also. A file based storage would make the settings "portable" (so they can get backed up and re-used later).

[…] In addition to "Owner", what other three column headings would you suggest?

Right away I have no proposal, maybe others can chime in here and propose a few columns in the next days.
One information I regularly need is the link type (if an entry is not a native file or directory), i. e., what you see in a command shell in a directory listing usually as 3rd column. For example:

Symlink (file)
SymlinkD (directory)
Junction (directory)

I'm not sure whether that classification is available as column (it probably is, but I haven't found it yet – I use a German Windows, and maybe the correct title of that column is just difficult to identify due to the translation).
If you know that column, that would useful to add as well.
Just two columns left… ☺

1000 pixels on my old 1600x900 screen would be crazy wide, but on an 8K (or even 4K) monitor it's not unreasonable. That's one reason why Explorer should be updated to prompt for the column width in ems instead of pixels. Another reason is that the column width defaults are already stored in the registry as ems. Ems are relative and are roughly equivalent to 1 character. On my little 96 dpi screen at 100%, 1 em is 8 pixels, at 125% it's 10 pixels, and so on. I mention all this because I'm implementing the column width option to work in ems (with a Help button to explain). I was thinking two characters (i.e. 99 ems) would be plenty (the Windows default for the Name column is 34 ems). I could allow up to 999 ems, but my concern is that people won't read and will enter large values thinking it's in pixels. Well, I guess I can't be blamed for that, but guidance is good. What do you think? 99 ems is more than 100 characters wide. Would you really want to go wider as a global default?
AFAIK, there's no link type column option built into Explorer. Tools already exist to provide overlay icons to indicate the link type. Take a look at: https://elsdoerfer.name/ntfslink and https://schinagl.priv.at/nt/hardlinkshellext/hardlinkshellext.html. There may be more out there. Hopefully one of those will work for you.
I'll add a new checkbox row with the "Owner" column and leave room for adding three more column headings as requests come in. In the future I would like to do a completely new UI design with flexible column choices, but that will take a lot of work, so hard-patching will provide a bridge for now.

A new version is now available. It adds an "Owner" column heading and column width settings for the Name column (first column) and a separate width setting for any Path type column headings. Selecting, and setting widths for, any of Explorer's column headings is on the To Do list.

WinSetView v2 now provides an interface to choose from any column headings (a list of more than 300 on Windows 10).

Wow, great! Thanks a lot, much appreciated! 👏
Unfortunately after testing I discovered some grievances with the current WinSetView implementation (width of columns only in "em" and not in pixel as Explorer shows it by default, weird error messages with meaningless message boxes due to missing error description and empty header, etc.), so I'll guess I need to wait a little bit for a more advanced and bug-free / polished version…
Limited time currently on my side, sorry 😓, otherwise I'd create proper issues for my problems. Maybe I can do so in a few days.

Hi. Thanks for commenting.
Yes, Explorer shows pixels for column widths, but it only uses ems in the
registry, for good reason. Since WinSetView is essentially a FolderTypes
registry key editor, using the registry values makes sense. Also, pixels
don't make sense for the interface because pixel density can vary
dramatically. The programmers knew what they were doing by using ems, but
the designers thought pixels are more "familiar". Anyhow, ems are explained
in the readme and I'm not going to translate ems to pixels (you have to
account for pixel density and screen scaling). It's easy enough to get used
to ems and you won't have to change the values when you upgrade to a higher
pixel density screen (or I won't have to figure out in code what the new
values should be).
The messages are intentionally as brief as possible to simplify translation
to other languages. An important goal of this project is to make it
multilingual. I've provided translations for 31 languages. Any dialogs that
have help or explanations greatly increases the translation work, so the
dialogs will remain brief. I will add information to the readme file as
questions come up. Yes, the readme is in English only, but non-English
speakers can use a combination of online translation and/or asking an
English speaking friend to interpret. Not perfect, but in time, maybe
others will translate the readme file.
The only dialog I can think of that matches what you describe is this one:
[image: image.png]
In order to avoid additional stuff to translate, it's very much like an
Ikea instruction page, but the readme file does say that Explorer is
limited to 21 column headings. Most users will never try to select more
than 21 column headings and, if they do, I figured they would pretty
quickly figure out the error means they reached the limit. I'll update the
readme to show a picture of this dialog. I intend to keep it Ikea-like. ;-)
Please let me know if there was any other unclear dialog. I will fix any
bugs promptly, but I don't see any to fix yet. More specific feedback will
be very much appreciated.
Thanks,
Les
On Mon, Jul 5, 2021 at 10:01 AM engelhro @.***> wrote:

Wow, great! Thanks a lot, much appreciated! 👏
Unfortunately after testing I discovered some grievances with the current
WinSetView implementation (width of columns only in "em" and not in pixel
as Explorer shows it by default, weird error messages with meaningless
message boxes due to missing error description and empty header, etc.), so
I'll guess I need to wait a little bit for a more advanced and bug-free /
polished version…
Limited time currently on my side, sorry 😓, otherwise I'd create proper
issues for my problems. Maybe I can do so in a few days.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/LesFerch/WinSetView/issues/2#issuecomment-874136756,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS25QO4RJFDKNFZ7BXAHOITTWG3LPANCNFSM4Z662OCA
.

Hi. Thanks for commenting. Yes, Explorer shows pixels for column widths, but it only uses ems in the registry, for good reason.

I understand the internal format used for storing the columns widths and that it makes sense there. I just partially disagree with your conclusions.

Since WinSetView is essentially a FolderTypes registry key editor, using the registry values makes sense.

Well… You consider WinSetView to be kind of a registry key editor, while me (and other users probably) consider it to be be tool that should make their life (with Windows and its explorer) easier :smiley:
And that's what it does not currently:

I would have to find out my screen density.
I would need to determine the current scaling factor.
I would need to calculate.
I would even need to find out the correct formula to apply! The readme only lists some values for the concrete screen density of 96 dpi with three different scaling factors, but does not explain the general formula I would need to use for other values of both screen density and scaling factor.

I consider that as not optimal from an end user's point of view.

[…] It's easy enough to get used to ems and you won't have to change the values when you upgrade to a higher pixel density screen (or I won't have to figure out in code what the new values should be).

I have at least two issues with that:

The width in ems can be specified in WinSetView only as integer, resulting in multiples of a value in pixels. At the same time, I can set the width in the explorer on a scale of pixels (and that gets remembered correctly by Windows!). So I assume the width (in ems) is stored internally as float?
Can at least that degree of detail get exposed via WinSetView, so I can set the desired widths with the precision I want?

If you really insist on ems, would it be possible to read the columns and width values (for a specified folder) on startup and use those as a starting point?
Showing a fixed list of proposed columns with a default width of 34 ems would destroy the user's view once he hits the Submit button, overriding his/her current settings (while the intention might have just have been to adapt the current view a little bit or even apply it to other folders from a given "master" directory).
Looking at ems as unusual unit intead of seeing pixels obscures this potentially destructive behavior and makes the user less aware of the unintended effect.

[…] Unclear error message
The only dialog I can think of that matches what you describe is this one:

In order to avoid additional stuff to translate, it's very much like an Ikea instruction page, but the readme file does say that Explorer is limited to 21 column headings.

Yes,that is indeed the error message I meant.
The problem is: it appears even with less than 21 columns selected!
Actually, I start WinSetView, click on the Columns button in the "General" section, and then every click on any checkmark box triggers the message. Both lists (columns to show and columns for right click menu) are left on default, i.e., with way less than 21 entries preselected in each of them.

Most users will never try to select more than 21 column headings and, if they do, I figured they would pretty quickly figure out the error means they reached the limit.

Well, not having reached that limit as all, the error message was quite perplexing to me, nothing to "figure out". WinSetView, out of the box, was just not working for me. So while it might be clear and "easy" for you as developer, such a message is nothing but confusing for the user.
Developer's problem (I've seen that too often): what you think users would do or figure out, is almost never what users actually do or figure out :wink:

More specific feedback will be very much appreciated.

You're welcome. I hope I have described my view in a helpful manner.

Well… You consider WinSetView to be kind of a registry key editor, while me (and other users probably) consider it to be be tool that should make their life (with Windows and its explorer) easier 😃

The two views are not mutually exclusive. My goal is to make the task of setting File Explorer default views as easy as possible (or perhaps I should say "practical"). The default views are stored in the FolderTypes registry key, so that drives some of the interface choices.
WinSetView v1 had a simpler interface, but also many limitations. I could not address all of those limitations without adding a bit of interface complexity, but I certainly did my best to keep it simple (somewhere around  two hundred hours of work actually).
In regards to the >21 columns error message...
When I posted v2, I should have included a Win10.ini file that only had the default Windows 10 columns (and right-click column choices) selected, but I actually included an INI file where I had already selected 21 column headings for the right-click menu:

The limit of 21 applies to the column heading right-click menu as well. So, with those selections already made, you need to uncheck one or more items from that last column before you can select a different column heading. I have since replaced the INI file in the Main branch on GitHub with one that has only a few pre-selected items. You can also just delete the INI file to start up with nothing more than the "Name" column selected.
I hadn't at first realized I had posted a file with 21 items pre-selected. When I saw that, I realized why you encountered the 21 limit error message right away. Apologies for that.
In regards to "ems"...
The default column width for any given column in Details view is saved as an integer value in ems.
When you set a column width by pixels in File Explorer's More menu, it's saved in binary format in the Bags registry key, along with any other view settings you may have set for that folder using File Explorer. The Bags registry key is the volatile registry storage key for folder views that overrides any of Windows default views. WinSetView sets default views using the FolderTypes key. It does not mess with the Bags key (which is a sequence of numbered binary entries) with the lone exception of the "This PC" and "Network" views option (which breaks easily because it's not a proper default - see readme).
So, I can't set default widths with any more precision than ems allow and it's a major can of worms to have the interface in pixels. I have put a lot of thought into it. It's not nearly as simple as it looks, but I don't agree that you have to do any calculations. Just think of ems as characters, enter a number and try it out. I set my default "Name" width to 40 and my "File Extensions" column to 6. A bit of trial and error, but nothing more difficult then entering a number, clicking Submit, and seeing if it looks okay. Once you've got it set you're done and you won't have to rethink it when you upgrade to a higher pixel density display.
Having said that, I'll take one more look at it. Perhaps an easy update will be some way to show the estimated pixel equivalent when you right-click or hover over the value.

You can now enter the column widths in pixels. Hold the *Alt key and click a column width input field to get a dialog to edit the value in pixels:


drive.py runs training?

After saving my model using model.save('model.h5') I run 'python drive.py model.h5'. I expected it just to start wsgi and then run the car, but instead it runs the training over again. I must be doing something incorrectly?

@codepete Can you paste and output you get from running python drive.py model.h5. There's nothing in that file that would cause it to train so that's quite strange.

(carnd-term1) C:\Users\ungpe\Udacity\CarND-Behavioral-Cloning>python drive.py model.h5
Using TensorFlow backend.
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\stream_executor\dso_loader.cc:128] successfully opened CUDA library cublas64_80.dll locally
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\stream_executor\dso_loader.cc:128] successfully opened CUDA library cudnn64_5.dll locally
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\stream_executor\dso_loader.cc:128] successfully opened CUDA library cufft64_80.dll locally
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\stream_executor\dso_loader.cc:128] successfully opened CUDA library nvcuda.dll locally
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\stream_executor\dso_loader.cc:128] successfully opened CUDA library curand64_80.dll locally
Epoch 1/10
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\core\common_runtime\gpu\gpu_device.cc:885] Found device 0 with properties:
name: GeForce GTX 970
major: 5 minor: 2 memoryClockRate (GHz) 1.1775
pciBusID 0000:01:00.0
Total memory: 4.00GiB
Free memory: 3.31GiB
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\core\common_runtime\gpu\gpu_device.cc:906] DMA: 0
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\core\common_runtime\gpu\gpu_device.cc:916] 0:   Y
I c:\tf_jenkins\home\workspace\release-win\device\gpu\os\windows\tensorflow\core\common_runtime\gpu\gpu_device.cc:975] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX 970, pci bus id: 0000:01:00.0)
 1792/19286 [=>............................] - ETA: 63s - loss: 0.1862forrtl: error (200): program aborting due to control-C event
Image              PC                Routine            Line        Source
libifcoremd.dll    00007FFB66A643E4  Unknown               Unknown  Unknown
KERNELBASE.dll     00007FFB84ED7E23  Unknown               Unknown  Unknown
KERNEL32.DLL       00007FFB871D8364  Unknown               Unknown  Unknown
ntdll.dll          00007FFB885970D1  Unknown               Unknown  Unknown

😕  Can I see the drive.py file you're using.

Let share a gist of my code... I must be something stupid...
https://gist.github.com/codepete/4eb1ccf1dbe5f153d8adb0d69f846c4b
https://gist.github.com/codepete/b4521b1f4ada124445405139732b6242

Ahhh, when you import model.py you're running the whole file so you also rerun https://gist.github.com/codepete/4eb1ccf1dbe5f153d8adb0d69f846c4b#file-mode-py-L97-L109. Hence why it retrains.
You probably only want to import the preprocessing functions so you would write
from model import resize_images, normalize

Or you could write if __name__ == '__main__': before those lines I linked to. That way that part will only run if it's the main file, python model.py and not when it's imported.

Thanks you for helping me. Hmmm I just tried doing:
from model import resize_images, normalize
And it still runs the training.

Try this https://gist.github.com/domluna/de3d134bda4c52345663dae59712d1dc, it shouldn't run when you import it this way

Ah you rock! Thank you so much!


Use github's discussion tab.

Could you please add the github's discussion tab?

Discussions are now open.


Enable specific language for user interface

Description
Could you please enable Sinhala (si or si_LK) language for ungoogled chromium interface?
Who's implementing?

[ ] I'm willing to implement this feature myself

The problem
Google doesn't enable this language for UI saying "binary size constraints" but other chormium based browsers already have this language enabled.
https://issues.chromium.org/issues/40073213
What is "binary size constraints"?
I got these details from another party and this response from google. I guess this is not a problem for you and i hope to use the next ungoogled version in Sinhala.

There are currently no plans to add it due to binary size constraints. That may be revisited by the Desktop team in the future.
Binary size increases are what's blocking Desktop Chrome from adding more languages. Each language is approximately 250kb, and any binary size increase causes a drop in updates.

(windows)
Alternatives
No response
Additional context
No response

Sorry, but your request is outside of the project goals. If the localisation in question is available in the source tree, it could be doable as it would be a packaging issue. It would be unfortunately impossible otherwise.
I don't know whether this localisation is shipped with Windows (@Nifury and @teeminus may know better), but in Linux it seems to be absent.

Could you not just copy the pak file from that specific browser you mention (curious which one it is) into the ungoogled chromium Locales folder?
By the way, the size restraint is bullshit of course, Google could easily implement downloading the language file once you select the interface language, just like they do with spelling languages.

Could you not just copy the pak file from that specific browser you mention (curious which one it is) into the ungoogled chromium Locales folder?

I can ask you the same question. Have you tried taking localisation file from other browser and using it in ungoogled-chromium?

implement downloading the language

This one would be actually against the goals of the project.

Could you not just copy the pak file from that specific browser you mention (curious which one it is) into the ungoogled chromium Locales folder?

I can ask you the same question. Have you tried taking localisation file from other browser and using it in ungoogled-chromium?

Sorry, it was meant as a response to d1Bihkte.

implement downloading the language

This one would be actually against the goals of the project.

I  didn't mean ungoogled-chromium should do this, I was talking about Google could do this. Not including all languages in the binary but instead downloading the interface language file once you select it would make the binary a lot smaller.
Of course I don't expect this project to implement that functionality!

I don't know whether this localisation is shipped with Windows (@Nifury and @teeminus may know better), but in Linux it seems to be absent.

@PF4Public They have the problem mentioned above, so they don't ship si.pak file with Chrome releases.

Google could easily implement downloading the language file once you select the interface language, just like they do with spelling languages.

@joopbraak i think this is not a good solution and will be a problem for some deployment environments that don't allow their devices to connect to Google servers. They can include this language file in the setup as others have already done.
"Each language is approximately 250kb, and any binary size increase causes a drop in updates."
They have said this. Maybe a problem with their update mechanism. 😃

Could you not just copy the pak file from that specific browser you mention (curious which one it is) into the ungoogled chromium Locales folder?

I can ask you the same question. Have you tried taking localisation file from other browser and using it in ungoogled-chromium?

I haven't tried this because popular chrome forks strings are in different order and i guess this method doesn't work because I think I have to modify the code and build it from source. (+ i don't know more about software development)

implement downloading the language

This one would be actually against the goals of the project.

@PF4Public I'm not asking to implement a whole new "download languages" feature. I've noticed that other Chromium-based browsers get translations from the Chromium project for Chromium strings and only translate their own strings.
If i am right, Ungoogle-chromium does not have new strings or UI additions except chrome://ungoogled-first-run/
Ungoogled chromium is my main browser. That's why I wanted to ask. :)

Chromium-based browsers get sinhala translations from the Chromium project

Do you know how do they do it?

Additional locales could be enabled by https://github.com/ungoogled-software/ungoogled-chromium/pull/3072


Firewall Rules For EKG.md & Prometheus.md

We need generic firewall rules that can be used with iptables to allow for external access to EKG and Prometheus including accessing browsers and exporting displays. We have good AWS guidlelines but it needs to be more generic

EKG is not meant for remote access by design. The tutorial explains a workaround using port forwarding, but professional users running a node on a remote machine should probably reach to Prometheus instead.
Yes, we need more generic instructions eventually - agreed.

@gmanroney @brunjlar
The follow on action here is that we need to document Prometheus use, right?

right


inconsistent space handling in inline plugin templates when switching in and out of editing mode

Versions
Django CMS: 3.1.2, 3.1.0
Django: 1.8, 1.7
Steps to reproduce

Create a project in which you have this plugin:
from cms.models.pluginmodel import CMSPlugin
from cms.plugin_base import CMSPluginBase
from django.utils.translation import ugettext as _
from cms.plugin_pool import plugin_pool

class Plugin(CMSPluginBase):
    text_enabled = True
    model = CMSPlugin
    name = _("Something")
    render_template = "something/something.html"
    cache = False

    def render(self, context, instance, placeholder):
        return context

plugin_pool.register_plugin(Plugin)

The something/something.html file contains:

  Foo Bar

Edit the above code as needed to make it work in your project.

Instantiate a Text plugin somewhere and instantiate a Something plugin in it. Save the Text plugin and switch back to "Content" view.

Click "Save as draft" and then click "Publish".

Expected behavior
The text of the Something instance should always be "Foo Bar" with a space between the two words.
Actual behavior
When in editing mode (step 2 above), the text appears as "FooBar". When out of editing mode, it appears as "Foo Bar" (step 3 above).
Observations
Adding any character other than a space between the spans that contain "Foo" and "Bar" prevents the problematic behavior from happening. For instance if I put a period just after the  that appears after "Foo", the rendered plugin is consistently: "Foo. Bar" whether in edit mode or not. It looks like something similar to the {% spaceless %} tag is stripping spaces when in edit mode.
I've searched the documentation but found nothing about this behavior.
Ultimately, once published the rendering is correct, but the incorrect rendering in editing mode could lead staff members to file spurious bug reports to the developers of a site that uses Django CMS.

This is related to https://github.com/divio/django-cms/pull/4458

Fixed in #4467


The dimensional storage actuator crashed my base’s chunk.

Describe the bug
I placed down the dimensional storage actuator with a storage stabilizer tier 1 having already being placed on the ground at the distance of three blocks to the left of where I put the actuator. That being two empty spaces and the stabilizer is on the third block. I also had a ritual being preformed in the background to Craft a second storage stabilizer of the same tier.
To Reproduce
Steps to reproduce the behavior:

Place block Storage stabilizer tier 1
Begin ritual somewhere within the same chunk like underground
Place dimensional storage actuator
See error

Expected behavior
A crash to saving world then to desktop and anyone who comes close experiences the same and a black skeleton of the item is seen when trying to log in.
Screenshots
I wish I could give them to you but the server owner removed your mod before I could capture any.
System (please complete the following information):

Occultism Version: 1.19.2

OS: Windows 11/Windows 10
Minecraft Version: 1.19.2
Modpack Link and Version, or list of mods: All the mods 8

Additional context
I don’t expect it to be repeatable but if it is or is a known bug all the better.

Please provide the crash log and any log files, then we can investigate. It sounds a bit like an issue we had with outdated geckolib but hard to tell without the log

I had the same issue as well in the All the Mods 8 pack. I just placed down the dimensional storage actuator and no stabilizers. Here is my crash log
crash-2022-11-12_15.57.44-client.txt

Ah yes that's a geckolib issue, it's outdated.
Can you try to update to latest geckolib? @ebrinley2001

Closing this for now - if the issue remains after updating to latest ATM or latest geckolib (3.1.36) please ping me to reopen


Allow to get the number of bytes queued in PendingWriteQueue

Motivation:
For some use-cases it would be useful to know the number of bytes queued in the PendingWriteQueue without the need to dequeue them.
Modifications:
Add PendingWriteQueue.bytes().
Result:
Be able to get the number of bytes queued.

@nmittler @Scottmitch PTAL ... this comes in handy if you want to allocate a ByteBuf that you can merge all writes in.

Cherry-picked into 4.0 (ff51dfce032e55b7bd0f46a69a2578771075257e) and 4.1 (b37a41a535e41b573c09a00226865561327fb01a)


add plugin erc721

Relates to:

Background
What does this PR do?
this PR add a plugin to support ERC721
What kind of change is this?
add ERC721 Plugin
Documentation changes needed?

Testing
Detailed testing steps
    const tools = await getOnChainTools({
        wallet: viem(walletClient),
        plugins: [sendETH(), erc721({ tokens: [memeNFT] })],
    });

    const result = await generateText({
        model: openai("gpt-4o-mini"),
        tools: tools,
        maxSteps: 5,
        prompt: "check my meme nft balance",
    });

For plugins

[x] I have tested this change locally with key pair wallets
[ ] I have tested this change locally with hosted wallets (e.g. Crossmint Smart Wallets, etc.)

Discord username
aithez

@aeither we upgraded how plugins work so this might not build if you rebase. Would you be open to updating it? Would love to merge it. It should be pretty similar to the ERC20 plugin :)

@aeither we upgraded how plugins work so this might not build if you rebase. Would you be open to updating it? Would love to merge it. It should be pretty similar to the ERC20 plugin :)

updated it


Question about best fit r_eff

Hi,
I've been starting to play around with pysersic to measure the sizes of compact galaxies, and I have two questions about the parameter, r_eff.

Is the best fit value of r_eff the effective radius of the base sersic profile before it has been convolved with the PSF?
How does one interpret best fit r_eff values that are smaller than the FWHM of the PSF?

Thank you so much!
Best,
Kelly

Hi @ke27whal!
Yes, pysersic forward models the galaxies, so it generates an intrinsic galaxy model using a set of properties (including reff) and then convolves this with the input psf before comparing to the data. When the posterior estimation / sampling results return an, e.g., 50th percentile r_eff smaller than the PSF FWHM, this is possible because the values are pre-convolution.
To answer your second question, it definitely depends a bit. Reff is a parameter of a model being fit to all light pixels, so if the galaxy is "well resolved" out to beyond the PSF FWHM, then it is probably fine even if Reff itself is smaller than the PSF. But the smaller Reff gets with respect to the psf, the better you need to know the psf to say anything about whether the source is resolved or not. It's not cut and dry because different sersic indices will put different fractions of the light inside, and outside, of the PSF FWHM.
I think the simplest approach is probably to fit such sources with the PSF itself as well as with a galaxy model. If the light distribution seen can be roughly as well modeled by just a PSF, then the galaxy is likely unresolved and its not well-motivated to attempt an extraction of sersic parameters. Conversely if the galaxy fit is considerably better than a PSF-only fit, that's an indication that the information is present to distinguish the galaxy as resolved.
Cheers,
Imad

Great, thank you so much!


hidden popover

Hi Mike, I have started to use your plugin and has helped me enough, just that I have a detail, I want to implement some popover's within a tab but it appears hidden within nav-tabs. I want to implement something similar as you did with the dropdown menu but I do not know if it is a good idea, what can I do?
Thanks a lot

The container option for popovers allows you to specify an element the popover should be appended to. (By default, it's appended to the parent of the element triggering the popover, so it ends up a sibling of the triggering element.)
So in your case, you could, for example, specify data-container="body" and the popover will be appended to the body and so will not be within the nav-tabs.
Here's a plunk with an example of how to do that. In Tab 2 I added an element that triggers a popover on hover.
Does this help? Is this what you're trying to accomplish?

Yes, that is precisely what I was looking for, thank you very much for the information.

Stupendous! Glad to have helped.


cktgen global routing updates

Do not merge global routes
Revise global route search box
Fix absolute path in runit

Does the global route merge disabling help?

Does the global route merge disabling help?

It does help with OpAmp.


"process with an id of _ is not running"

Describe the bug
When I try to add NFL Head Couch 09, I get the message "process with an id of (changing four digit numbers like 3048) is not running." The game works perfectly as usual, but I get the message after closing Xenia Canary, and it doesn't appear in Xenia-Manager.
To Reproduce
Attempt to add the game as usual and exit.
Expected behavior
Be added to the library.
Screenshots/Video

Log File

Xenia Manager Log: https://gist.github.com/soundsofsilence/45c67870b7df9863f0a3581885d67b97
Xenia Log: https://gist.github.com/soundsofsilence/fd53574d9fa25022c3215b65143ff1df

Does it happen with other games or just this one?

I found the reason: Xenia Manager must be run as an administrator.

In what directory do you have Xenia Manger? I would recommend putting it somewhere that doesn't need admin rights like Desktop or AppData etc.

In what directory do you have Xenia Manger? I would recommend putting it somewhere that doesn't need admin rights like Desktop or AppData etc.

It's not in a protected folder. To avoid such situations, I place all my games in C:\Games. I also tried disabling my antivirus to ensure there were no false positives, but it made no difference.


Rename to C7/C8 in deployment overlay

Related to https://github.com/bpmn-io/internal-docs/issues/481
Artifacts:
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/rename-C7-C8-deployment/camunda-modeler-rename-C7-C8-deployment-linux-x64.tar.gz
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/rename-C7-C8-deployment/camunda-modeler-rename-C7-C8-deployment-mac.dmg
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/rename-C7-C8-deployment/camunda-modeler-rename-C7-C8-deployment-mac.zip
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/rename-C7-C8-deployment/camunda-modeler-rename-C7-C8-deployment-win-ia32.zip
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/rename-C7-C8-deployment/camunda-modeler-rename-C7-C8-deployment-win-x64.zip
Screenshot:

@andreasgeier I changed the title and also the field labels since it was a quick change


Issue opening the offline sqlite store

I'm having an issue opening the OfflineSQLiteStore. I used the ToDo sample and don't see the problem. I'm running on Windows using the following connection string and the creation of the OfflineSQLiteStore is now failing. It was working in the past, but now fails. I'm using 5.1.3 of the Microsoft.Datasync.Client
Here is the connectionString:
var connectionString = new UriBuilder { Scheme = "file", Path = Path.Combine(FileSystem.CacheDirectory, localDbFileName), Query = "?mode=rwc" }.Uri.ToString();
var store = new OfflineSQLiteStore(connectionString);
Here is the exception:
Unable to open database connection to 'file://localhost/C:/Users/A_ANDERSON/AppData/Local/Packages/89572ed5-19ad-422e-9657-21583e4a02ce_hbve9c01gpnke/LocalCache/builderdb.db?mode=rwc': 14 unable to open database file
Should I use the FileSystem.Current.CacheDirectory instead? I tried it and it still didn't work. Maybe a permission issue? I also tried removing the database file entirely and it still doesn't work. It has worked in the past because my database in that directory was previously populated.
Ideas?
Thanks
Alan

Underlying this call (and the cause of the error) is that the system tries to open the file and fails.  This is normally permissions but could also be one of the directories doesn't exist.  From the information presented, I could not tell you which.
There is nothing I can do about this error.  It is something about your system.  I'd suggest (since you have control of your system) that you simplify the problem.  First off, try to use the regular file system calls and ignore OfflineSQLiteStore - directly before this line, insert code to open the file, read the first 10 bytes and then close the file.  Something like:
try {
  var path = Path.Combine(FileSystem.CacheDirectory, localDbFileName);
  byte[] test = new byte[10];
  using (BinaryReader reader = new(new FileStream(path, FileMode.Open)))
  {
    reader. Read(test, 0, 10);
  }
}
catch (Exception ex) {
  Console.WriteLine($"exception - {ex.Message}"); // Put breakpoint here
}

You can then review the exception which is likely to have more detailed information about the failure that the underlying SQLite driver is not passing back to C# (and hence the OfflineSQLiteStore and your application).
Please let me know if this helps and/or close the bug.

Quick sample - write Hello World to the same file path using File.WriteAllText (it works)
    try
    {
        var path = Path.Combine(FileSystem.CacheDirectory, localDbFileName);
        File.WriteAllText(path, $"Hello World.");
        File.Delete(path);
    }
    catch (Exception ex)
    {
        Debug.WriteLine(ex.Message);
    }

I remove the file, then I let it try to create the database (which would be empty since the file doesn't exist), same error as above - into the same exact path, but using the UriBuilder.
?

Underneath the covers, we use sqlite-net-pcl, which is a wrapper around the C version of sqlite.  This is actually done in SqliteConnection.cs with this code:
int rc = raw.sqlite3_open(connectionString, out connection);
            if (rc != raw.SQLITE_OK)
            {
                var errmsg = raw.sqlite3_errstr(rc).utf8_to_string();
                throw new SQLiteException($"Unable to open database connection to '{connectionString}': {rc} {errmsg}", rc, connection);
            }

The sqlite3_open method is defined here: https://www.sqlite.org/capi3ref.html#sqlite3_open and you can find the meaning of the "rc" here: https://www.sqlite.org/rescode.html - 14 is SQLITE_CANTOPEN which is very very generic.  You can find out about URI filenames here: https://www.sqlite.org/uri.html - there are some examples of what works.
Your error message is coming from the heart of SQLite.  There is nothing I can do about it.  It's a lower level error than what we operate at.  We're asking SQLite to open the file you request (with ?mode=rwc means open read-write and create if doesn't exist).  SQLite is responding with "Nope - can't do that" with no other reason.
Suggestions:

Try a different directory.
Try a different filename.
Dump the URI and make sure it is escaped properly (it should be, but weird things sometimes happen).

I used a different filename, offline.db - still didn't work. I'll try a different directory. It is odd because it worked at some point, but now doesn't. I'll explore more tomorrow and let you know.
Thanks for your help, Adrian.

Likely true.  We had an issue with another library as well.  Basically, the Batteries.init() call doesn't properly initialize after the sqlite3_config() call on the first db is used.  I'm thinking this is a bug in the sqlite database, so I'm looking at switching the database driver to a higher level one in the 6.0.0 release.

Great, I was thinking over night about what had changed in my app, then realized I added the service that uses the local SQLite db. Commented that out this morning and magic happened. It definitely worked in the past, so it is something with SQLite I suspect. I'll write it to a JSON temporarily and keep migrating to MAUI.
Thanks again.
ALan

I was running into a similar issue and it is because I was using the AppCenter SDK which also makes use of Local Storage. I removed the call to AppCenter.Start() and stopped receiving the error.


Get distance to each waypoints on route

Hello,
Is there a way to get distances from the start of the route or from the user location to each waypoint located on the route?
Thanks for you help!

You can create CLLocation objects from the coordinates and use CoreLocation’s CLLocation.distance(from:) but that would measure the distance as the crow flies.
To calculate the distance along the route line, you would have to use something like turf-swift’s LineString.distance(from:to:).
import Turf

let lineString = LineString(route.coordinates)
let distance = lineString.distance(from: wp1.coordinate, to: wp2.coordinate)


Tmc util tests

Coverage increased (+0.4%) to 69.258% when pulling f818dd9240975067820331f74f31fe47fe943852 on tmc-util-aleksi into 2f2d14a780882c6dae738d0f3c87eff6b48b4c2e on master.

Coverage increased (+0.4%) to 69.258% when pulling f818dd9240975067820331f74f31fe47fe943852 on tmc-util-aleksi into 2f2d14a780882c6dae738d0f3c87eff6b48b4c2e on master.

LGTM

Coverage increased (+0.4%) to 69.258% when pulling 6a80a0b6c8280b738de3d38b6b25beab609a5d16 on tmc-util-aleksi into 36638254a08573e18dbcf0f55ac1404a9feade10 on master.

Coverage increased (+0.4%) to 68.905% when pulling dcc58bb3e1f63afa54abfd6ff05161ee6c76468b on tmc-util-aleksi into 21ba0683b279467d32b622f05188be39c49d4128 on master.


Wrong elliptic filter ripple configuration

The elliptic filter ripple configuration in FiltersForm.AnalyzeEllipticFilter is converted to magnitude Db. But in PrototypeElliptic.Poles and PrototypeElliptic.Zeros it is converted as power Db. So my suggestion is not to use Db here. Or I am wrong?

Hi! The code is based on this wonderful pdf: http://eceweb1.rutgers.edu/~orfanidi/ece521/notes.pdf. Take a look at derivations at page 2. For instance, -20log(Gp) = 10log(1 + eps^2). The right side is programmed, but it was derived from magnitude dB. Also, IIRC, I tested it - the method worked as expected. Maybe I'm missing something, though, but the code seems to be OK.
Also, in FiltersForm the code is:
var ripplePassDb = Utils.Scale.ToDecibel(1 / deltaPass);

because 20log(1/d) = -20log(d). So the code is similar to MATLAB code from the pdf.

Ah OK. So rippleStop means the stopband ripple, in decibels (stopbandRipple) and ripplePass means the  stopband attenuation, in decibels (stopbandAttenuation)?

Not sure if I understood you correctly. Fig.1 at page 2 from the pdf should help (I'm using the same notation):
Passband gain: Gp
Passband ripple (Gp in dB): Ap = -20log(Gp) = 20log(1/Gp)
Stopband (attenuation) gain: Gs
Stopband (attenuation) ripple (Gs in dB): As = -20log(Gs) = 20log(1/Gs)
Ap and As are passed as parameters to EllipticFilter constructor (i.e. values in dB).
This is what the notation is in the pdf. I like the term 'attenuation' better because this value defines the amplitude loss (not the max amplitude of fluctuations in stopband) according to Fig.1. And the passband gain (ripple) here means also the max amplitude loss compared to level=1.0. Often it has slightly different meaning - illustrated here (when the center of oscillations is at level 1.0 : { 1-d, 1+d }). This one is used in Remez class.

Thanks a lot! Yes, the different terms make it difficult to learn...