[BUG] Live view flickering on windows

When updating a live view (with a table inside a grid), the whole table flickers on windows. On linux everything seems very smooth.
Especially when printing output via logging (above the live view), the table flickers.
python -m rich.live

also shows this behaviour.
I am not sure if this happens only on update() and not on refresh(), as the example does not flicker when only new entries are displayed in the table, but always when there is a new log entry.
Diagnose
I am running rich 9.11.0 on Python 3.9.1 (the error was also on Python 3.7) and tried it in git bash in windows terminal, cmd in windows terminal and also directly in cmd.
│ │                                                                                │
│     color_system = 'truecolor'                                                                                │
│         encoding = 'utf-8'                                                                                │
│             file = <_io.TextIOWrapper name='' mode='w' encoding='utf-8'>                                                                                │
│           height = 50                                                                                │
│ is_dumb_terminal = False                                                                                │
│   is_interactive = True                                                                                │
│       is_jupyter = False                                                                                │
│      is_terminal = True                                                                                │
│   legacy_windows = False                                                                                │
│         no_color = False                                                                                │
│          options = ConsoleOptions(size=ConsoleDimensions(width=183, height=50), legacy_windows=False, min_width=1, max_width=183, is_terminal=True, encoding='utf-8', justify=None, │
│                    overflow=None, no_wrap=False, highlight=None, height=None)                                                                                │
│            quiet = False                                                                                │
│           record = False                                                                                │
│         safe_box = True                                                                                │
│             size = ConsoleDimensions(width=183, height=50)                                                                                │
│        soft_wrap = False                                                                                │
│           stderr = False                                                                                │
│            style = None                                                                                │
│         tab_size = 8                                                                                │
│            width = 183                                                                                │
platform="Windows"
WindowsConsoleFeatures(vt=True, truecolor=True)

It will flicker on Windows. cmd is a lost cause, it's not going to be updated. Windows Terminal is not too bad, but the way it repaints will cause a little flicker.  Full screen interfaces will suffer less from flicker, see fullscreen.py in examples.
Windows is not the best platform for animated things in the terminal, and Rich can't compensate for that. Although Windows Terminal is improving all the time.

Thanks for the feedback. What is then the recommended setup for windows?
I would use windows terminal with git bash but it flickers like cmd.
I think I have to try the new layout mechanism and if replacing different parts works better than updating the whole live output with a grid.
The strange thing is, that when only rows are added in the rich.live example, there is no flickering at all.
But anyway. Thanks for the great library!

Git bash may be using conhost, which essentially means it will work exactly like cmd. Although I'm not certain about that.
I'm afraid Windows have neglected terminal support for an age, and have only recently started to correct that. You may have to accept a little flicker.

I have the same issue. Python 3.8.6 and rich 9.11.0.
I used Powershell terminal, Teminus, Windows Terminal, and Command Prompt.
I used the fullscreen.py example.

In Command Prompt I see a flicker and when the jobs finish, the dashboard remains in the view.
In Powershell the flicker is about the same as CMD and the dashboard remains in the view.
In Windows Terminal this is the best option now as the screen doesn't flicker, but the dashboard disappears when complete.
In Terminus we have the same flicker but mostly at the rendering of the header bar. And the dashboard disappears when the script finishes.

I do see how this would be an issue with the terminal used. Would there be suggestions on settings to look for on how to limit some of the flicker?

The dashboard should disappear when alternate screen is used. I'll fix that in the next release.
Like I said, you will have to accept some flicker if its not on Windows Terminal.

@willmcgugan But I am using Windows Terminal and see flicker.
@cjerrington What shell are you using in Windows Terminal? I tried with git-bash in Windows Terminal and have some flicker.

There is a small amount of flicker, even on Windows Terminal. How about posting a video so we know if we're talking about the same thing.

Ok I made some more tests with a short test script:
When using PowerShell in Windows terminal I get no flicker.
https://user-images.githubusercontent.com/7723101/108501090-0b579880-72b1-11eb-9c43-a6451dee4757.mp4
When using git bash in Windows Terminal I get flicker.
https://user-images.githubusercontent.com/7723101/108501108-14486a00-72b1-11eb-80c3-e9901c05dcfa.mp4
So it seems I should use powershell.
But when I use the same output in my program (which samples sensor data, uses multiple threads, gevent, ...) the screen flickers even in powershell.
Please note also that the flicker is much more when a log entry is added (at the beginning of the video), and less when no log changes (at the end)
https://user-images.githubusercontent.com/7723101/108501544-bf592380-72b1-11eb-8e81-73d746b7d79a.mp4
In my program it must be something with the other threads/gevent, ... The CPU is at about 40% so this should not be the limiting factor. Do you have any hints what I should look for. Unfortunately I can't post the code.
This is the test output I used for the videos.
import random
import time
from rich.console import Console
from rich.live import Live
from rich.table import Table

class MyTable():

    def __rich__(self):
        grid = Table.grid(expand=True)
        grid.add_column()
        for t in self.rich_data_tables():
            grid.add_row(t)
        return grid

    def rich_data_tables(self):
        tables = []

        def rand(num):
            return [random.random() for _ in range(num)]

        def ran_table(name, num):
            table = Table(title=name)

            for val in [str(x) for x in range(num)]:
                table.add_column(val, justify="right", width=10)
            table.add_row(*[f'{a:.2f}' for a in rand(num)])
            return table

        tables.append(ran_table("tab1", 3))
        tables.append(ran_table("tab2", 3))
        tables.append(ran_table("tab3", 2))
        tables.append(ran_table("tab4", 16))
        tables.append(ran_table("tab5", 1))
        tables.append(ran_table("tab6", 3))

        return tables

if __name__ == "__main__":
    console = Console()

    myTable = MyTable()
    with Live(myTable, console=console, refresh_per_second=10) as live_table:
        while True:
            time.sleep(0.001)

You probably don't want to do such a small sleep, suggest sleep(1).
You can tweak the refresh_per_second. 10 per second is probably higher than it needs to be, try 2. If you can live without the logs, try setting screen=True.
Ultimately though, there isn't much you (or Rich) can do about it. Other platforms fair much better with essentially the same output.

@bobatsar on Windows I usually always use powershell in various terminal emulator programs. I do agree sleep of 0.01 is almost not calculated. 1 Second is enough of a visible notice and a good time frame to judge a process by.


"propagation check failed" from internal cluster

Describe the bug:
Not sure if this is a bug or expected.  Using DNS01 challenge provider from a cluster that only has Internet access via a web proxy (http & https only), we are getting error when enrolling a new cert:
E1108 08:27:38.647486       1 sync.go:184] cert-manager/controller/challenges "msg"="propagation check failed" "error"="dial tcp <IP_ADDRESS>:53: i/o timeout" "dnsName"="host.ourdomain.com" "resource_kind"="Challenge" "resource_name"="host.ourdomain.com-41473054464-4010201230-1730459797" "resource_namespace"="our-namespace" "type"="dns-01"
The IP address of the DNS server being queried (<IP_ADDRESS>:53 above) changes on each attempt (every 40 seconds).  Our pods cannot query any external DNS servers.  Is this a requirement for using Cert Manager?
We have attempted to force this propagation check to use our internal DNS server using the following helm values:
podDnsPolicy: "None"
podDnsConfig:
  nameservers:
    - "<IP_ADDRESS>"

where <IP_ADDRESS> is an internal DNS server our pods can query.  While this causes the message that preceeds the error to change from:
I1108 08:03:33.891579       1 dns.go:121] cert-manager/controller/challenges/Check "level"=0 "msg"="checking DNS propagation" "dnsName"="host.ourdomain" "domain"="host.ourdomain.com" "resource_kind"="Challenge" "resource_name"="host.ourdomain.com-41473054464-4010201230-1730459797" "resource_namespace"="our-namepace" "type"="dns-01" "nameservers"=["<IP_ADDRESS>:53"]
to
I1108 08:03:33.891579       1 dns.go:121] cert-manager/controller/challenges/Check "level"=0 "msg"="checking DNS propagation" "dnsName"="host.ourdomain.com" "domain"="host.ourdomain.com" "resource_kind"="Challenge" "resource_name"="host.ourdomain.com-41473054464-4010201230-1730459797" "resource_namespace"="our-namespace" "type"="dns-01" "nameservers"=["<IP_ADDRESS>:53"]
the same error occurs.  Can we use Cert Manager without opening up port 53 access to the world?
Expected behaviour:
Cert is enrolled
Environment details::

Kubernetes version: 1.15.3
Cloud-provider/provisioner: bare metal
cert-manager version: v0.11.0
Install method (e.g. helm or static manifests): helm

I was missing switches:
--dns01-recursive-nameservers-only
--dns01-recursive-nameservers="<IP_ADDRESS>:53"


Collect example code that must be importable in the first release

This is mostly concerned with what is needed for the first release of a Qiskit importer that uses this parser.  Qiskit is the only consumer at the moment and so is driving priorities for features.
There are two tasks

[ ] Decide minimum required features and write some kind of corresponding code that must be importable.
[ ] Extracting snippets or programs from the result of the previous item and organizing them in order to track progress. Eventually these will be used to make unit tests.

There are a few ways to organize this.

It's probably necessary to put several requirements that work together in one program.
To track progress it would be convenient to have code snippets with names like good_n.qasm and fail_n.qasm, for ordinal n. The idea is to develop and fix bugs to move all of the fails to goods.
It might be useful to make the previous item more granular. Instead of good and fail, just have prog_n.qasm. And have check boxes for some or all stages in the pipeline.

This ship has already sailed. This issue referred to the release of Qiskit 1.0


jindo: fix master and worker ports of jindo runtime

Ⅰ. Describe what this PR does
fix the issue: when jindocache/jindofsx runtime use container network, it can't use default DEFAULT_MASTER_RPC_PORT=8101 and DEFAULT_WORKER_RPC_PORT=6101.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
when use JindoRuntime with networkmode: ContainerNetwork like:
apiVersion: data.fluid.io/v1alpha1
kind: JindoRuntime
metadata:
  name: liuxiang-fluid-jindocache-demo1
  namespace: fluid-system
spec:
  networkmode: ContainerNetwork
  replicas: 1
  tieredstore:
    levels:
      - mediumtype: SSD
        path: /mnt/liuxiang-fluid-jindocache-demo1
        quota: 10Gi
        high: "0.9"
        low: "0.7"

check the master/worker rpc port at configmap liuxiang-fluid-jindocache-demo1-jindocache-values is not default DEFAULT_MASTER_RPC_PORT=8101 and DEFAULT_WORKER_RPC_PORT=6101, otherwise is like:
master:
  ports:
    rpc: 18364

worker:
  ports:
    rpc: 19971

Ⅴ. Special notes for reviews

/assign @cheyang

/test fluid-e2e


Cannot sign into azure

Does this occur consistently? Yes
Repro steps:

Make sure Azure Functions and Azure CLI Tools are installed
Go to the Azure functions tab
Click Sign in to Azure ...

Expected Results: I will be able to sign into Azure
Actual Results: My browser opens up to localhost:[some varying port]/signin?nonce=[some random string token] and then hangs there forever.
Version: 1.3.0
OS: linux
OS Release: 5.4.0-66-generic
Product: Visual Studio Code
Product Version: 1.54.3
Language: en

Could be a duplicate of https://github.com/microsoft/vscode-azure-account/issues/179
In that issue, they recommend Azure: Sign In with Device Code as a workaround

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


go get -u gorgonia.org/gorgonia fail

macbookpro:Downloads fredlee$ go version
go version go1.12beta2 darwin/amd64
macbookpro:Downloads fredlee$ go get -u gorgonia.org/gorgonia
go: finding github.com/awalterschulze/gographviz latest
go: finding github.com/leesper/go_rng latest
go: finding gonum.org/v1/gonum/blas latest
go: finding gonum.org/v1/gonum/graph/topo latest
go: finding gonum.org/v1/gonum/graph latest
go: finding gonum.org/v1/gonum/blas/gonum latest
go: finding gonum.org/v1/gonum latest
go: downloading gonum.org/v1/gonum v0.0.0-20190110212618-fe2cf67cb467
go: extracting gonum.org/v1/gonum v0.0.0-20190110212618-fe2cf67cb467
gorgonia.org/gorgonia
../Documents/develop/go/workspace/pkg/mod/gorgonia.org/gorgonia@v0.8.4/walker.go:43:43: cannot use g (type *ExprGraph) as type graph.Directed in argument to topo.SortStabilized:
*ExprGraph does not implement graph.Directed (wrong type for From method)
have From(int64) []graph.Node
want From(int64) graph.Nodes
../Documents/develop/go/workspace/pkg/mod/gorgonia.org/gorgonia@v0.8.4/walker.go:53:33: cannot use g (type *ExprGraph) as type graph.Directed in argument to topo.Sort:
*ExprGraph does not implement graph.Directed (wrong type for From method)
have From(int64) []graph.Node
want From(int64) graph.Nodes

I'm facing the same issue, maybe I have to wait for this package dependencies to be updated.

@lynic see https://github.com/gorgonia/gorgonia/pull/241

same problem here
# gorgonia.org/gorgonia
..\..\go\pkg\mod\gorgonia.org\gorgonia@v0.9.1\graph.go:569:2: cannot use e (type edge) as type graph.Edge in return argument:
        edge does not implement graph.Edge (missing ReversedEdge method)
..\..\go\pkg\mod\gorgonia.org\gorgonia@v0.9.1\node.go:756:15: cannot use e (type edge) as type graph.Edge in argument to n.g.SetEdge:
        edge does not implement graph.Edge (missing ReversedEdge method)

This is an issue with Gonum's versions. Can you try go mod tidy && go mod vendor and report back>?

I already tried that. No change.
What works for me was adding to graph.go last line:
func (e edge) ReversedEdge() graph.Edge { e.from, e.to = e.to, e.from; return e }

which is on the master but not on the version v0.9.1. I am using go mod, with this definitions:
go 1.12

require (
	gorgonia.org/cu v0.9.0-beta
	gorgonia.org/gorgonia v0.9.1
	gorgonia.org/tensor v0.9.0-beta
)

Same issue here.  Master branch works fine.

Closed as we are now several versions ahead and this is no longer an issue.


Add 2D Probabilistic UNet Training Setup

PUNet Source Training Workflow

(TODO: evaluation script will follow)

The PUNet Source Workflow is completed now. Just updated the pipeline for source evaluations. Thanks!


  find a way to compile it to all platforms in order to ship it via the godot asset store

Im having a hard time trying to cross compile, if anyone knows how to do so with cargo, it would be helpful

Cross compilation have recently been achieved ! :tada:  (see PR #85)

I've updated the download commit in the Godot AssetLib, pending approval by moderators. Once it goes live, this issue may be closed. Agreed?

Asset approved! I'm closing this issue.


Different signature for Android apk

Latest release of app-fdroid.apk (v1.2.17+366) has a signature that is different for all previous releases. It makes impossible to update the previous version.
Is it possible to use the same signature for all releases?

The key was updated so that I can upload the aab built by github to Google Play. This is a one time thing.
If you want to upgrade without having to reimport/resync your data, I suggest you use the batch import/export
https://ywallet.netlify.app/save/03.batch/index.html


[DOCS] bug in explicit createContext type example

I ran into a syntax error with the current docs' suggestion of:
const Theme = React.createContext<"light" | "dark">("light");

I was able to fix it by instead doing:
const Theme: React.Context<"light" | "dark"> = React.createContext("light");

Also, it seems to recommend here to use import * as React instead of import React (especially if you're going to use some types) so I changed that here as well.

Generics in function calls is a fairly recent syntactic addition, so you probably have to update tooling (like babel/its plugins or flow itself) for the documented syntax to work. It certainly does if you try it in flow.org/try.

Ah, ok I just saw that typing context should be supported as of flow 0.70, and I'm on 0.94, but yeah babel and eslint are a bit out of date, so that explains it.


Content: update library tags and languages

Type of change

[x] Content addition
[ ] Bug fix
[ ] Behavior change

Summary of change
Update all libraries with appropriate languages and tags.
Checklist

[x] The changes follow the contributing guidelines
[x] I have searched all existing content and verified my additions are novel
and unique
[x] All required content fields are accurately populated
[x] All items are tagged with all relevant tags

Closes: #556

Noted the libraries are not showing up when a tag is selected from the sidebar.

@iansamz are the filters curated, or is it just something that happened over time? I've tried to unify all tags, which made have been a wrong move if we want specific filters like these. Two options moving forward are, we filter by the new tags we have (I can auto cherry pick the tags used per framework). Or I can "clutter" the unified tags with the very specific tags.
CC @kenslachtajr

Sorry @mbicknese. Hadn't seen your comment.  We can create another PR to auto cherry-pick the tags per framework @kenslachtajr

The change was easy enough to implement quickly. Curious to know whether this is a step in the right direction. We can reconsider the tags we have and how libs are tagged.


typeaheadOptionField option useless with async queries

I use the typehead directive to search a backend of people. There a data structure like the following will be returned:
[{
  "id": 1,
   "name": "Hennry"
},
{
   "id":2,
   "name": "John"
}]

When I set the attribute typeaheadOptionField to "name" there's still [Object object] shown up.

I tried with a Observable Datasource and a Promise one. Non of them worked.
Any Ideas what I doing wrong ?

Check how you're setting the value, to be sure it's a string. For example: [typeaheadOptionField]="'name'". If you forget the '' surrounding the name you'll get the behaviour you're describing.

Hi,
I just had "name"....
Thank you


Make selfcontained

Hi Erik,
Can you create a selfcontained executable with this PR?

Is there some kind of continuous deployment in this project that I can integrate with?
Or do we create the executable manually and upload it as a git release?
The exec is zipped up so is an acceptable 69,3 Mb

I will do this in the build pipeline, thanks for doing the validation

@buckleyGI I did this instead: https://github.com/ErikEJ/SqlQueryStress/commit/e1254616b7fdcd292256aeea6a20301643513ad8

@buckleyGI I have just updated the GitHub release with a self contained .exe, would be grateful if you could try it out.

Sure, happy to help out
I was expecting the release on the following location or by clicking through.
Were can I find the test release? I waited a bit thinking it is a caching issue but it is not showing up.

Just open the release and download the Net6 zip file

Ha, so I had to ignore the outdated timestamp.
Download the Net6 zip file and it works as expected. Looks good.


Fix of "Built applications force discrete GPU usage"

https://github.com/federkasten/appbundle-maven-plugin/issues/25

Thanks. I will check and merge soon.


Support id_token in refresh_token

What version of UAA are you running?
{
  "app": {
    "version": "4.10.0"
  },
  "links": {
    "uaa": "https://uaa.<...>",
    "login": "https://login.<...>"
  },
  "zone_name": "uaa",
  "entityID": "login.<...>",
  "commit_id": "a4132cb",
  "idpDefinitions": {},
  "prompts": {
    "username": [
      "text",
      "Email"
    ],
    "password": [
      "password",
      "Password"
    ]
  },
  "timestamp": "2018-02-06T10:24:36-0800"
}

How are you deploying the UAA?
using cf-deployment 1.15.0 on bosh-lite
What did you do?
I'm trying to integrate UAA and Kubernetes with CFCR 0.14.0 using https://github.com/cloudfoundry-incubator/kubo-release/pull/101 .
It worked to access k8s with id_token returned by UAA.
After the token expired, kubctl used refresh_token to refresh the token but failed with the following error
Unable to connect to the server: token response did not contain an id_token, either the scope "openid" wasn't requested upon login, or the provider doesn't support id_tokens as part of the refresh response.

From the doc,
refresh token does not seem to support id token.
I've also confirmed the response of bellow didn't contain id_token field
curl -k -v https://uaa.<...>/oauth/token -d client_id=xxxxx -d client_secret=xxxxx -d grant_type=refresh_token -d refresh_token=xxxxx -d response_type=id_token

What did you expect to see? What goal are you trying to achieve with the UAA?
I expected UAA to support id_token in refresh_token in order to refresh token with kubectl.

@tnwang
Can you please prioritise this? I believe this is important for k8s integration.
https://kubernetes.io/docs/admin/authentication/#option-1-oidc-authenticator

Once your id_token expires, kubectl will attempt to refresh your id_token using your refresh_token and client_secret storing the new values for the refresh_token and id_token in your .kube/config.

Thanks for raising this issue. We are aware of this and will be working on it soon.

is this issue fixed in 4.20.0?

No, it's not fixed in 4.20.0. I have work in progress for this that I plan to PR for the OD-PKS team.

Closing due to inactivity.


SwiftUI demo is not working

the demo is not working, its not downloading the Swift pkg dependences .

Thank you, I’ll check! Is it in the master branch? Meanwhile, can you update packages from the main menu?

The master branch demo works for me. I also just updated it to use the latest version. Please grab the latest commit and try again.

thanks alot ,I will check it out. Sorry for the late response.


SHACL - support simple rdfs reasoner

Since SHACL is pushed as a replacement for spin it has a bunch of quirky features, such as an embedded reasoner that only supports rdfs:subClassOf and only works during validation and not on SPARQL queries that run on the validated data.

Copy code from the schema chaching reasoner

Forward chain the added and removed memory stores. Backwards chain the base sail. Set a connection wide Boolean field “validating” to enable the backwardschaining.

@hmottestad I see a PR was merged, is this issue done?

Yes


[TEVA 2187] update terms and conditions copy

https://dfedigital.atlassian.net/browse/TEVA-2187

Review app deployed to https://teaching-vacancies-review-pr-3404.london.cloudapps.digital


feat(nip-64): Display Chess (PGN) messages

In order for NIP-64 to be approved, it needs to be implemented in two clients.
Would coracle be interested in displaying kind := 64 as chess board in the web interface?
The spec is quite simple and straightforward–so it should be rather easy.
It would need to include a dependency on a library capable of drawing chess boards.
If you are interested, please let me know :pray:

I think I will give this one a pass, but it is a good reason to prioritize adding nip 89 support in #352

Completely understandable. Thank you for even thinking about it and considering it! :pray:


remove the src/delphi/static

configs will be moved to configs/ in the incoming PR for usability, and we don't need it for anything else
any utils related to this should be removed as well

This means that the configs won't be included in the python package.


Resolve vertical branch conflicts

Conflicts against Horizontal manifest baseline of WW01 release
Tracked-On: OAM-105613
Signed-off-by: Chenthati, Pradeep<EMAIL_ADDRESS>

@basilchew @bnkoppad kindly review the PR

It looks like a lot of patches are removed. Is it because the vertical patches have been ported back to the horizontal branch?
This PR is raised using OAM-105613, however, there is no further information in it. Previously, for every vertical patch added, there is a horizontal Jira submitted to port back to horizontal branch and a revert ticket to remove the patch from vertical.
For traceability, can we duplicate all such revert Jira tickets to OAM-105613? They should then appear as duplicates in the OAM-105613 ticket. With this info, we will then be able to check if the correct patches are removed.

Yes Basil, these patches are present in Horizontal branch. I have linked relevant revert Jira tickets as duplicate to OAM-105613.
Please review

@basilchew @bnkoppad please review


Correction for raw affiliation Department of Psychology, Western University.

Correction needed for raw affiliation Department of Psychology, Western University.
raw_affiliation_name: Department of Psychology, Western University.
new_rors: 02grkyz14
previous_rors: 02agqkc58
works_examples: W4361279043;W4321376816;W4388193897;W4388734565;W4317358871;W4386027195;W4323075331;W4377014268;W4387579313;W4387820405;W4389743227;W4390029901
contact: eb037a8b2b58584acfb96bd752453da4:02144c66e6138d4e @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


Affix scrolled alignment

Guys,
I dont know if someone have some had troubble already with this, I'm trying to center text/images when is scrolled down
leave link to help you.
http://jsfiddle.net/52VtD/11414/

Hi @andresclua!
You appear to have posted a live example (http://fiddle.jshell.net/brainbolt/52VtD/4/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 35, column 7: E003: Found one or more .rows that were not children of a grid column or descendants of a .container or .container-fluid
line 15, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 2.1.0

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Please strech window to see it clearly.
leave link to help you.
http://jsfiddle.net/52VtD/11414/

Hi @andresclua!
You appear to have posted a live example (http://fiddle.jshell.net/52VtD/11414/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 15, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 2.1.0

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

@cvrebert bootlint false positive on version number.

Forked. Should fix the version error it was showing.
http://jsfiddle.net/opo8ydvo/

@andresclua Not sure I understand the goal. Is your goal to tap into the location of affix to dynamically center an image? If that is the case I believe that per CONTRIBUTING.md this would fall under a personal support requests. Stack Overflow (twitter-bootstrap-3 tag), IRC, or Slack are better places to get help with these subjects.

@kkirsche Looks like a true positive to me.

@cvrebert Sorry, it was the mix of both the left and the right. Seems there were two copies of bootstrap being imported. I assumed the @import on the right not the external resources on the left. Sorry about that. My fault.

Closing since this it looks more like a personal support request than an allegation of a specific bug in Bootstrap itself. Try taking your question to one of the forums mentioned in https://github.com/twbs/bootstrap/issues/16505#issuecomment-103282141 instead.


add TLS setting in readme

There is not details about fill out which val into TLS setting field.
According to "How to configure cluster #35", add related info.
Signed-off-by: Xiang Dai<EMAIL_ADDRESS>

any input?

any update?

@jtlisi @daniellee any update?

any update?

any update on this please?

@jtlisi Hi, any update please?

@jtlisi Hi, any update please?

any update?

@daixiang0 sorry, but I'm not a maintainer of this project, so can't help you further.

@dennisameling do you know who is?

any update?

any update?

@daniellee hi, any input?

@jtlisi thanks!


Import/Export, more RDBMS

:white_check_mark: Build TheIdServer 1.0.0-ci.1+109.build.455 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/00f69f4248 by @aguacongas)

:white_check_mark: Build TheIdServer 1.0.0-ci.1+114.build.460 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/49d6c0e279 by @aguacongas)


Questions on SR Implementation and Intended Behavior

Hello,
Thank you for providing this excellent work. I have been experimenting with the code, but I encountered some ambiguities that I wanted to clarify to ensure I understand the intended behavior correctly. Below are my questions:
Input Data Format
File: turtle_super_t1_arch.py
Function: forward
In the current implementation, what is the exact expected format for the input tensor? Should it be (B, n_sequence, C, H, W) as suggested by the forward function, or is it (B, C, H, W) or another format? Clarifying this would help avoid potential shape mismatches.
For example, I encountered the following error, which led me to modify the code as follows:

def forward(self, inp_img_, k_cached=None, v_cached=None):
        B, _, C, H, W = inp_img_.shape
        # print("inforward")
        # print(inp_img_.shape)
        inp_img_ = self.check_image_size(inp_img_)

        # print(inp_img_.shape)
        H, W = H*4, W*4 # scale factor
        if k_cached == None:
            k_cached = [None] * 8
            v_cached = [None] * 8

        k_to_cache = []
        v_to_cache = []
        # print(self.use_both_input)

        if self.use_both_input:
            previous, current = inp_img_[:, 0, :, :, :], inp_img_[:, 1, :, :, :]
            inp_img = torch.cat([previous,
                                current], dim=1)

            inp_img = self.upsample_4x(inp_img)
            current = self.upsample_4x(current)
        else:
            inp_img = inp_img_[:, 1, :, :, :]

            inp_img = self.upsample_4x(inp_img)
            current = inp_img

I also tried modifying the check_image_size part forward and in a different way.
def check_image_size(self, x):
        _, _, _, h, w = x.size()
        # _, _, h, w = x.size()
        mod_pad_h = (self.padder_size - h % self.padder_size) % self.padder_size
        mod_pad_w = (self.padder_size - w % self.padder_size) % self.padder_size
        x = F.pad(x, (0, mod_pad_w, 0, mod_pad_h))
        return x

Here, I adjusted the line _, _, _, h, w = x.size() to _, _, h, w = x.size(). While this resolved the error, I am unsure if the changes align with the intended behavior (result) of the code.
check_image_size Function Input
File: turtle_super_t1_arch.py
Function: check_image_size
The check_image_size function seems to process a 4D tensor (B, C, H, W). Is this assumption correct, and is it guaranteed that all inputs to this function adhere to this shape throughout the pipeline?
Super-resolution Upscaling Process
File: turtle_super_t1_arch.py
Function: forward
In the current code, SR tasks appear to rely on self.upsample_4x. Is this the only intended upscaling step for SR tasks? How does this fit into the overall processing pipeline, and are there additional considerations to ensure SR quality?
Tile-based Inference and Boundary Handling
File: inference_no_ground_truth.py
Function: run_inference_patched
The run_inference_patched function performs tile-based inference, but it appears there is no explicit handling for blending or smoothing tile boundaries. Is the current E.div_(W) operation sufficient to ensure smooth boundary transitions, or is there a reason why blending logic is not included?
Dataset-Specific Logic for MVSR
File: inference_no_ground_truth.py
Function: run_inference_patched
The code includes a section for dataset_name == "MVSR", which applies downscaling and upscaling operations. Should similar logic be applied to other datasets, or is this process unique to MVSR? How does this impact SR performance for non-MVSR datasets?
use_both_input Parameter
File: turtle_super_t1_arch.py
Function: forward
The YML configuration uses use_both_input: False, suggesting single-frame inputs. However, for SR tasks, multi-frame inputs could be beneficial. Is single-frame input the intended behavior, or should use_both_input: True be used for better performance?
torch.clamp and Tile Boundary Blending
File: inference_no_ground_truth.py
Function: run_inference_patched
The torch.clamp operation after E.div_(W) seems to restrict output values to the range [0, 1]. Is this meant to improve tile boundary consistency, or is it purely for restricting output values? Is additional boundary smoothing unnecessary in this implementation?
Quality of Results
File: General observation during experiments
In my experiments, the output SR images appear to have significantly lower quality than expected. Are there specific configurations, settings, or additional steps required to achieve the optimal results?
Thank you for your time and support. I hope these questions help clarify my understanding and ensure I am following the intended pipeline correctly.
Looking forward to your guidance!
Best regards,
neng2

Hi,
Thank you for your interest in our work and for going through it in such detail. I appreciate the effort you’ve put into experimenting with the code and raising these important questions. Let me address them one by one:
Input Data Format
Although we process the video frame by frame, the model accepts two frames as input. However, in all our experiments, we used use_both_input: False, effectively relying only on the second (current) frame. This option exists because, during initial experiments, we tested the effect of passing two frames to the model versus passing one frame and relying on the CHM (Causal History Module) to borrow information from past frames. You are welcome to enable use_both_input: True to concatenate the two frames and compare the results for your use case.
To clarify the input tensor format:

By default, the input to the model is (B, 2, C, H, W), where 2 represents the two frames (previous and current).
If use_both_input: True, the frames are concatenated along the channel dimension, resulting in an input tensor of shape (B, 2*C, H, W).
If use_both_input: False, only the second frame (current) is used, resulting in a tensor of shape (B, C, H, W).

Regarding the check_image_size function: Since it operates on the tensor after this dimensional change, the modification you made is correct and aligns with the intended behavior.

Super-Resolution Upscaling
We applied a simple upscaling scheme (self.upsample_4x) without extensive testing of configurations or exploring advanced upsampling methods. The primary purpose was to demonstrate the generalizability of our model to tasks like SR.
Feel free to explore alternative upsampling schemes or configurations for improved results, and we'd be happy to hear about your findings.

Tile-Based Inference and Boundary Handling
For tile-based inference, we rely on overlapping tiles, with the overlap amount determined by the tile_size and stride. In overlapping areas, we take the mean to ensure smooth transitions, a method commonly used in similar works like VRT and RVRT.
That said, more sophisticated methods for blending or smoothing tile boundaries could further enhance the output quality. You are welcome to explore such techniques, and inference on full-sized images or increasing the overlap can also improve results.

Dataset-Specific Logic for MVSR
The logic for dataset_name == "MVSR" was implemented because MVSR was the only SR dataset we experimented with. The same logic should generally apply to any other SR dataset. Feel free to modify the code for additional datasets and share your findings.

torch.clamp and Boundary Smoothing
The torch.clamp operation ensures that output values are restricted to a valid range [0, 1] and is unrelated to boundary smoothing. If you have ideas or methods for smoothing tile boundaries, we encourage you to implement and test them. Let us know about your results, and we’d be happy to incorporate any improvements into the code.
Let me know if you have any other questions.

Thanks for going through our code in detail. As you mentioned, there were several issues with SR inference code, and we fixed all of them in the SR model and Inference codes. please clone the code again and run the latest version.
1. Scaling the Input
The MVSR4x dataset, used for real-world video super-resolution, is sourced from the EAVSR paper (link to paper). The dataset is collected using two side-by-side cameras on a mobile phone: an ultra-wide camera and a telephoto camera. The low-quality frame is captured by the ultra-wide camera, while the high-quality frame comes from the telephoto camera. The focal length of the ultra-wide camera is 13mm, while the telephoto camera has a focal length of 90mm.
By computing the ratio of these focal lengths (90/13), we find that the telephoto lens provides a zoom factor of 6.92 times more than the ultra-wide lens. However, for simplicity, the authors of the paper limit the zoom factor to 4x.
When the dataset is downloaded, the frame pairs (one from each camera) have the same spatial resolution, but the ultra-wide frame is of noticeably lower quality than the telephoto frame. In the super-resolution task, the authors suggest manually downscaling the ultra-wide frame by a 4x factor to simulate the degraded input. This downscaling ensures that the ultra-wide frame mimics being 4x worse than the ground-truth telephoto frame, aligning with the real-world degradation.
This approach to super-resolution differs from typical SR tasks, where the degradation is often modeled synthetically using bicubic interpolation (BI) or bilinear downsampling (BD), as seen in tasks like REDS SR.
2. SR Workflow
During inference, the low-quality input frame (img_lq) is processed to match the resolution of the ground-truth frame (img_gt), since both frames share the same spatial resolution in the MVSR4x dataset. To simulate the degradation, the ultra-wide frame (lq) is manually downscaled using bicubic interpolation, as shown below:
torch.nn.functional.interpolate(prev_frame, scale_factor=1/4, mode="bicubic")
torch.nn.functional.interpolate(current_frame, scale_factor=1/4, mode="bicubic")

3. Output Quality on REDS Dataset
The SR model was trained on the MVSR4x dataset, which, as explained earlier, is based on real-world degradation and differs from the synthetic SR tasks typically seen in datasets like REDS. To achieve optimal results for your specific task, you will need to train the SR model on the new dataset yourself.
We will be adding a Contributions section to the README. If you retrain the model on a synthetic SR task and open a pull request with your code and trained model, we will be happy to merge your model into the codebase. Additionally, we will acknowledge your contribution in the README.
Thanks again for your interest in our work.


Swift 5 Updates

💚 Summary

Update to Swift 4.2, then update to Swift 5
Update MainVC layout to support X+ phones (X, Xs, Xr, etc.)
Update build target to iOS 11.0+
Remove redundant extension (now part of Swift Core)

👍


Support Kafka 0.10 client

Kafka 0.10 exposes a new max.poll.records setting that can help with excessive rebalancing behaviors in Kafka.
https://cwiki.apache.org/confluence/display/KAFKA/KIP-41%3A+KafkaConsumer+Max+Records
This is an enhancement request for the Kafka plugin to support Kafka 0.10 client not just to get the latest fixes but also to expose this new option via the Kafka input plugin.

tracking PR: https://github.com/logstash-plugins/logstash-input-kafka/pull/94

@talevy Would this PR add SASL features to the kafka client as well ?

0.10.1.x is supported in version 6.x of this plugin.


unable to install addons

Hi,
I'm unable to install any kind of addon.
I tried:

WebMatic
Watchdog
HQ-WebUI

but after upload nothing happend.
My System:
Raspberry PI 3
Software-Version:	<IP_ADDRESS>70114
RF: HM-MOD-RPI-PCB
Kind Regards,
Sören

Please specify what exactly happens after the installation and what should actually happen. Have you checked that the addons are definitely not there and have you tried to see if the same happens with a complete new RaspberryMatic installation?

I'm  using RaspberryMatic since Sonday (it's the first time), so it is fresh, only one Actor installed.
I see the uploading in Browser (different browser, always the same), after it is finished no refresh of the site and no reboot.
Press back and again addon, no addon are listed.
For example WebMatic:
I try to get the URL:
http://IP-of-RaspberryMartic/addons/webmatic
404 - Not Found.
Where can I check if it is uploaded?
Can I do it manually?

@smindorf First of all, this is no support forum here but a bug tracker. However, your issue seems to be that the upload of the addon didn't finish correctly and therefore RaspberryMatic doesn't reboot automatically (which it should). Also please make sure that the Addon package you downloaded is valid (make sure that your browser didn't automatically unarchive the .tar.gz as an Addon package always have to be a .tar.gz file compressed with gzip).

@jens-maus yes I know.
Where will the package be uploaded to (so I can check the uploaded file)?
I've extract the package before I write this issue, and it seems to be valid.
Yes the file is a tarball with tar.gz extension. No automatically extraction while downloading.

Please tell use which addon versions your are using and try to install one of the rpi optimized addons eg https://github.com/jens-maus/cuxd/releases/latest or https://github.com/jens-maus/hm_email/releases/latest after correct upload the install button has to appears

I've tried to install webmatic-2.2.3.tar.gz, hm-watchdog-0.5.tar.gz and hq_webui_2.5.4.tar.gz without success.
Now I've tried hm_email-1.6.6.tar.gz and it is installed successfully. After installing hm_email-1.6.6 successfully I tried webmatic-2.2.3 with success. I tied the other plugins with success.
Thank you very much!


[RESOLVED] High ICMP ping response time

Problem started at 11:50:16 on 2023.07.26
Problem name: High ICMP ping response time
Host: ST-FG
Severity: Warning
Operational data: Value: 170ms
Original problem ID: 33116
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23135&eventid=33116

Problem has been resolved in 3m 0s at 11:53:16 on 2023.07.26
Problem name: High ICMP ping response time
Host: ST-FG
Severity: Warning
Original problem ID: 33116


Could you please create samples to showcase how to use the interface classes?

like ROBOT RAPID example. If we do not have rapid mod code , we can not use this lib ?

I'm going to close it's an old question.
But to give a quick answer: RWS does not require any RAPID code on the controller per se.
There are parts of it which let you manage RAPID (call routines, etc) which would of course require some code, but abb_librws itself does not require it.


Import and use individual angular_components libraries and components

cc @kwalrath

@kwalrath, @kevmoo, @nshahan  - while I can import individual libraries and components:

In templates/web-angular/lib/src/todo_list/todo_list_component.dart I replaced materialDirectives with individually listed ACX directives, but then when I attempt to run tests I get the following issue:
Running tests...
00:00 +0 -1: test/app_test.dart: heading [E]
  Assertion failed: "No value accessor for () or you may be missing formDirectives in your directives list."
  dart:sdk_internal                                                               assertFailed
  package:angular_forms/src/directives/shared.dart 25:28                          setUpControl
  package:angular_forms/src/directives/ng_model.dart 127:5                        ngOnInit
  package:__projectName__/src/todo_list/todo_list_component.template.dart 162:19  detectChangesInternal
  package:angular/src/core/linker/app_view.dart 407:7                             detectChanges
  package:__projectName__/app_component.template.dart 97:16                       detectChangesInternal
  package:angular/src/core/linker/app_view.dart 407:7                             detectChanges
  package:__projectName__/app_component.template.dart 331:16                      detectChangesInternal
  package:angular/src/core/linker/app_view.dart 407:7                             detectChanges
  package:angular/src/core/linker/view_ref.dart 106:12                            detectChanges
...

But I am also including formDirectives, so I'm unsure what I am missing. Also note that the angular_analyzer plugin isn't reporting any issues.

In templates/web-angular/lib/app_component.dart I seem to require materialProviders so the app ends up importing all of ACX. (I haven't found a list of individual providers that have worked yet; regardless, I end up having imports like package:angular_components/utils/browser/dom_service/dom_service.dart. Is this what we want to encourage users to do?)

With the new compiler, I think we're okay importing everything – it's smart
enough to drop what's not needed...
On Thu, Apr 26, 2018 at 8:55 AM Patrice Chalin<EMAIL_ADDRESS>wrote:

@kwalrath https://github.com/kwalrath, @kevmoo
https://github.com/kevmoo, @nshahan https://github.com/nshahan -
while I can import individual libraries and components:

In templates/web-angular/lib/src/todo_list/todo_list_component.dart I
replaced materialDirectives with individually listed ACX directives,
but then when I attempt to run tests I get the following issue:
Running tests...
00:00 +0 -1: test/app_test.dart: heading [E]
Assertion failed: "No value accessor for () or you may be missing formDirectives in your directives list."
dart:sdk_internal                                                               assertFailed
package:angular_forms/src/directives/shared.dart 25:28                          setUpControl
package:angular_forms/src/directives/ng_model.dart 127:5                        ngOnInit
package:projectName/src/todo_list/todo_list_component.template.dart 162:19  detectChangesInternal
package:angular/src/core/linker/app_view.dart 407:7                             detectChanges
package:projectName/app_component.template.dart 97:16                       detectChangesInternal
package:angular/src/core/linker/app_view.dart 407:7                             detectChanges
package:projectName/app_component.template.dart 331:16                      detectChangesInternal
package:angular/src/core/linker/app_view.dart 407:7                             detectChanges
package:angular/src/core/linker/view_ref.dart 106:12                            detectChanges
...
But I am also including formDirectives, so I'm unsure what I am
missing. Also note that the angular_analyzer plugin isn't reporting any
issues.
In templates/web-angular/lib/app_component.dart I seem to require
materialProviders so the app ends up importing all of ACX. (I haven't
found a list of individual providers that have worked yet; regardless, I
end up having imports like
package:angular_components/utils/browser/dom_service/dom_service.dart.
Is this what we want to encourage users to do?)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/stagehand/issues/496#issuecomment-384694051,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABCiiM0RLm-OfCJrOegonnb_GLuydueks5tse4NgaJpZM4TlUq5
.

@nshahan can you confirm that it's OK to import the big library rather than individual component libraries?

It depends on what you are trying to optimize. If you want a simple experience then you can import the single file that brings in every component into your app.
If you want the best build times and smallest possible output from dart2js then you should only import what you are using. Even the best tree shaking can't prove that are are not going to use some code when we are dealing with hash tables with dynamic types etc.
Based on @jakemac53 comments in https://github.com/dart-lang/angular_components/issues/242 I believe the development experience with a modular compiler like DDC is better when you are importing only the files you are using.
We can try running some benchmarks to see the difference, but I don't have any numbers at the moment.

To clarify, from an AngularDart perspective:

If you are using the new runApp (and not runAppLegacy) it doesn't effect code-size.
However, it might effect compilation time (iterating through 100s of items per component)

I'd recommend not using these giant lists of components, anywhere.
Its a little less terrible to use small lists of tightly connected things, for example formDirectives.

It looks like the error is related to having the wrong providers. Unfortunately we can't validate providers/DI, I saw somewhere a proof that it would be an NP-hard problem to try :( I think it also would be a whole-app analysis rather than a file-level analysis, which we don't have any infra for. That said, NP-hard for a small N isn't actually always that hard. So some day I may want to sit down with people to better understandwhat type of validation I'd have to do.
In the meantime, looks like there are a few questions in this thread:

Should you import giant directives lists?

What subset of directives and providers will work and how do users know that for any given directive?
Can we get a better error message in the compiler for this case?

Wish my plugin could catch this, but if I follow correctly so far, I don't think it can :(

@nshahan - I was just about to ask for a nice short list of material input directives, but it's there already! :) https://github.com/dart-lang/angular_components/blob/master/lib/material_input/material_input.dart#L24-L32

Hmm, well if it was a missing directive, I may be able to provide support.
Its probably not really feasible with angular as written, since directives are cross-linked via the DI system.
However, maybe if there were a new annotation like @OnElement then youd be able to do:
@Component(...)
class A {
  A(@OnElement ValueAccessor);
  ...
}
@Directive(...)
class MyValueAccessor extends ValueAccessor {
  ...
}

template:
<a-component with-no-value-accessor></a-component>

then I'd be able to flag something like, You're using component "A" which requires a directive on the element that implements "ValueAccessor", but no such directive exists. Try checking for missing directives in your component definition, misspelled attributes, or check API docs for which directives implement "ValueAccessor".
That said, its not exactly a great error message. But this I believe is the type of work I'd need to do to solve https://github.com/dart-lang/angular_analyzer_plugin/issues/233
I don't think I could do this with the way that angular is written, since for any set of injected dependencies, I have no real way of knowing whether they are base classes that should be implemented by directives or not.
Worse, in this case, it looks like it injects an @Optional List<ControlValueAccessor> which is two more reasons why I wouldn't be able to catch this (I will likely assume empty lists are OK, and I'll certainly assume missing @Optionals is ok, in whatever type of implementation I would throw together).
If everyone on this thread thinks I'm making sense and it would provide value, I can open an issue in the angular repo.

I can see how this would be a tricky situation to detect. I'll @nshahan and others comment.


Exception Handling Error

When you clone the repository and you try to run it and you have an exception handling error, you just need to recursively change the permission of the whole project folder using the chmod command in linux since that is the OS we are using for development.

Development on the project has been stopped. Thank you all.


Check .md/.geom parsing

elastic_constants has a script for parsing .md/.geom format. May be able to use some features from this for better reporting.

No useful extra features.


removed the fill value setting at load point and no more realising data

The latest iris 2.2.0 documentation says this: https://scitools.org.uk/iris/docs/latest/whitepapers/missing_data_handling.html so this means that if we load a file with a set fill_value = 1e+20 (like is the case of all normal CMOR-compliant netCDF data files) iris will not go about and change that on its own at load point, it will preserve it; a possible alteration of fill_value can only occur at save point (when not specifying the fill_value=1e20 **arg) but that is handled correctly at line 180 in _io.py (I have tested with iris 2.1 and 2.2 and saved intermediary cubes to file, all good, and the final preproc'd file has 1e20 for fill_value). The only problem would occuer when we use some crazy file that has a fill_value = some banana shit, but isn't that part of the CMOR check @jvegasbsc ?

have peeps seen this & tested? This PR feels so lonely :grin:

Testing now.

I do not remember why we introduced this piece of code, but I remember that we discussed this FillValue issue at some point, maybe when we tried to move to Iris 2?

@mattiarighi exactly - iris 2.x ignores an already set FillValue at save stage unless spoonfed to it; I'll merge, can always reopen if need be


[WIP] Type relations

resolves #448, this is a work in progress to start bringing the theory of subtype relation to reality and see if there is any issues with the current design
Initial implementation of type relation. Adds a isTypeRelatedTo which check if the source type can be assigned to the target type.
Logic for this:

Respect type hierarchy we have defined:

- any
  - object
    - Record<T>
      - [model with properties]
  - number
    - integer
      - int8
      - int16
      - int32
      - safeint
      - int64
      - uint8
      - uint16
      - uint32
      - uint64
    - real
      - float32
      - float64
  - bytes
  - plainDate
  - plainTime
  - zoneDateTime
  - duration

model with properties follow a pattern matching logic

must have all the properties of the target

Todo:

[ ] more types do not have relation implemented

[ ] union
[ ] enum

[ ] enable model property override
[ ] improve error messages and target location

It's really exciting to see progress on this! Thanks! I have only skimmed the code so far, but I will take a deeper look soon.


Daemon not respecting local CA store

I have a local certificate authority on my network and the CA cert is installed on the machine just fine (i can curl the panel) but when i try to start the daemon on the same machine this happens:
https://hastebin.com/secoyokina.sql

This is due to nodejs not using the system's CAs.
The optimal solution would be to add an option to provide additional CAs, as we are on feature freeze on the daemon this is not going to be implemented though. We'll have to check if the same occurs for golang's HTTP server as well and find a solution for the new daemon.
In the meantime, here are people with the same issue and some workarounds:
https://stackoverflow.com/a/39113115/4430124
https://github.com/nodejs/node/issues/4175

Until this is fixed one can also set the following system var for the daemon: NODE_TLS_REJECT_UNAUTHORIZED.
Just don't use this for actually productive environments as it completely disables certificate verification!

closed in https://github.com/pterodactyl/daemon/commit/3cb74bd49547de4b2f71948704ed0ea2eec67b69


make-test-python: disallow aliases

When importing Nixpkgs within Nixpkgs, we should not consider aliases
to ensure we don't rely on them internally.
There are probably more places that need to be converted.
Triggered by https://github.com/NixOS/nixpkgs/pull/101369#issuecomment-722585160.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I expect evaluation errors.

I expect evaluation errors.

Ofborg thinks differently.

Need to add it to the machine configuration instead.

@GrahamcOfBorg test plasma5

@GrahamcOfBorg test installer

just wanted to run some tests to make sure it wasn't fundamentally broken


intake plugin for butler

Create a intake plugin that can load a dataset using butler along with any metric tolerance metadata.

this doesn't make sense, closing.


irmin: fix support for random-order unforced tree traversal

Fix https://github.com/mirage/irmin/issues/1730.

Will merge once the CI is happy.


Typo "Azure Cosmos explorer"→"Azure Cosmos DB Explorer"

https://docs.microsoft.com/en-us/azure/cosmos-db/optimize-cost-throughput

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Allow dispatching messages without the need of a Telegraph

Right now, sending a message using the MessageDispatcher#dispatchMessage() method requires specifying a sender, which is an instance of Telegraph.
A very common case scenario is implementing the interface on the very same class you are dispatching the message from. So, typically, you end up with something like:
MessageManager.getInstance().dispatchMessage(this, 1);

Equally common is having classes that only generate events and do not handle any type of message. Currently, those classes are forced to implement Telegraph in order to broadcast events (or to have access to some other unnecessary Telegraph instance). So, you find yourself writing this
	@Override
	public boolean handleMessage(final Telegram msg) {
		return false;
	}

very frequently, just to dispatch a message. I tend to think the API could benefit from removing the need of having a mandatory sender parameter. If you do not specify one, just dispatch the message to every possible recipient from an anonymous sender.
What do you think?

Currently, those classes are forced to implement Telegraph in order to broadcast events (or to have access to some other unnecessary Telegraph instance).

Can't check right now., does it work if you pass a null sender? Not sure, (Telegraph)null might be needed.
If it works then you don't have to implement Telegraph, but yes an overloaded dispatchMessage without the sender argument might be useful.

does it work if you pass a null sender? Not sure, (Telegraph)null might be needed.

It does work, indeed! But then you might want to fix some debug logs in MessageDispatcher like
Gdx.app.log(LOG_TAG, "Instant telegram dispatched at time: " + currentTime + " by " + sender + " for " + receiver + ". Msg is " + msg);

To avoid logging a null string (if it is bothersome). That's the only drawback I could find.

Done!
The sender will be logged as a null string just like it happens with the receiver. :wink:

Great! Thanks for the fast response and work.


Unable to run script with arguments

I am currently running a command to produce documentation and I observed that is not possible to pass arguments.
  - gen-files:
      scripts:
        - python3 -m ansiblelint -L -f docs

mkdocs-gen-files does not run arbitrary shell commands at all.
If you want to use its ability to generate files, you need to write that as a Python script. (which can then call a subprocess or whatever is desired).
To just run custom code, you can more directly utilize mkdocs' hooks feature.

Regarding your specific command: this is just a lint feature, I wouldn't recommend integrating it with mkdocs because it just adds time to builds (even live rebuilds) - just run it separately or in a wrapper.
mkdocs_gen_files is not applicable even more so.

If you do however, still want to run a lint check, you could write a custom hook inspired by mkdocs-code-validator plugin.
https://github.com/oprypin/mkdocs-code-validator/blob/586842aa7e72d08d4cc61c1bf3644a3fe109b344/mkdocs_code_validator/plugin.py#L100-L113
Make a hook file that starts a subprocess in on_pre_build and gets results from that subprocess only in on_post_build.  That will run in the background and can avoid slowing down your build.

Hello @oprypin, sorry for reliving the dead, but before opening a new issue I just wanted to check if there is any intention to support this given my use case.
To generate API reference pages, we are using a script based on the recipe provided in https://mkdocstrings.github.io/recipes/#automatic-code-reference-pages:

https://github.com/frequenz-floss/frequenz-sdk-python/blob/v0.x.x/docs/mkdocstrings_autoapi.py

We have many projects and we would like to provide one script for all of them for maintainability reasons, so it would be nice to be able to ship this script in another python package and then in the mkdocs.yml do something like:
plugins:
  - gen-files:
      scripts:
        - python -m frequenz.repo.config.mkdocs

(and maybe pass some arguments too, so we can allow projects to eventually customize how the script is run (for example, if they need to discover source files in a directory different to the default src).
Without this, the alternative is to add to every project a wrapper script that will just import frequenz.repo.config.mkdocs, which is of course doable but a bit annoying.
Would you recommend to use the plain mkdocs hook for generating the API pages too? If so, what is the reason for proving a way to run scripts in mkdocs-gen-files at all if it can be done with mkdocs hooks?
Thanks!


Update gdcm 2020 02 12

PR Checklist

[X] :no_entry_sign: Makes breaking changes
[X] :no_entry_sign: Makes design changes
[X] :no_entry_sign: Adds the License notice to new files.
[X] :no_entry_sign: Adds Python wrapping.
[X] :no_entry_sign: Adds tests and baseline comparison (quantitative).
[X] :no_entry_sign: Adds test data.
[X] :no_entry_sign: Adds Examples to ITKExamples
[X] :no_entry_sign: Adds Documentation.

Refer to the ITK Software Guide for
further development details if necessary.

@thewtex @bradking kwrobot-v1  is repeatedly posting the same same error message.

@blowekamp that likely means that kwrobot has constructed a huge error message about problems with the PR.  It is probably whitespace complaints about the snapshot commit.  Try updating the Modules/ThirdParty/GDCM/UpdateFromUpstream.sh script to add a .gitattributes file to the GDCM tree's top level containing * -whitespace.  That should prevent the bot from caring about whitespace in the third-party code.
In the script, update the extract_source function to be:
extract_source () {
    git_archive
    pushd "${extractdir}/${name}-reduced"
    echo "* -whitespace" > .gitattributes
    popd
}

Make that the first commit on your branch.  Then run the script to create the updated content and merge again.

@bradking Done. Thank you very much for that help!

Maybe also incorporate changes proposed in #1608?

I'm inclined to just merge this as it's all green and ready to go now.

Sure, and add the other improvement in a new PR.


:memo: support Coordinate Type

Change Summary
Supporting the new type added to pydantic extra https://github.com/pydantic/pydantic-extra-types/pull/76
We can merge this once the PR in pydantic-extra-types is merged
Related issue number

Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Fix scale initials in lrn driver (SWDEV-274378)

To issue http://ontrack-internal.amd.com/browse/SWDEV-274378

@ce1adon Please change status of SWDEV-274378 to ANALYZED, thanks.


Suggestion: Warn when bindActionCreators encounters non-function property

I've found myself debugging 'Dude, where's my prop?' situations when I've done something like
import { doSomething } from './actions' // for whatever reason, `doSomething` is `undefined`

...

const MyComponent = props => {
   ...
   const onClick = () => {
     props.doSomething() // error at runtime --> doSomething is undefined
   }
}

const mapDispatchToProps = {
  ...
  doSomething
  ...
}

It would speed up debugging in these situations if bindActionCreators logged a warning when it's passed an object with non-function properties.

Seems reasonable to me. Anyone else have any input?

Seems reasonable.

OK, let's merge it in.

I think the automatic filtering was quite convenient for projects where you define both action creators and action types in the same file and then do import * as actions from 'actions' in your container and use that as second argument for connect().
This usage pattern is also used in the official universal example that now throws warnings.
container.js:
import * as actions from 'actions'

[...]

export default connect(mapStateToProps, actions)(Counter)

actions.js:
export const SET_COUNTER = 'SET_COUNTER'
[...]

export const set = (value) => ({
  type: SET_COUNTER,
  payload: value
})
[...]

Not sure if that approach is considered outdated, but at least I use it in my project a lot and it's still described in the official docs.
The not so pretty workaround I use now in my project:
export default connect(mapStateToProps, { ..._.pickBy(actions, a => _.isFunction(a)) })(Counter)

I updated the redux to 3.7.0 and got the error:
bindActionCreators expected a function actionCreator for key '__esModule',
instead received type 'boolean'.

Tell me how to fix it? Have any ideas?

I found the source of problem in my applications and replied some questions related.
And thought that, if the application code continues working as expected (in my case for example), could be helpful if the the warning is a console.warn() instead of an console.error. Maybe with another util?

I have to agree with @sebcode.
Importing in actions with import * as action from 'actions' is definitely a common pattern, and one that saves a lot of headache when you've got larger actions files. Using the * import not only saves me from a giant import block, but also stops issues of variables being declared in upper scope when you use destructuring assignment.
import { action1, action2, ... } from 'actions'
...
class App extends Component {
...
componentWillMount() {
  const { action1 } = this.props; // <-- action1 is now already defined in upper scope
  action1();
}
...
export default connect(
  mapStateToProps,
  mapDispatchToProps,
)(App));

Now I get that you could export out of your actions files differently or add a little util function (like @sebcode is using) to pull out any non-function values pulled in with import * to solve this - but in my opinion these changes are more disruptive and time consuming than the original issue here of "dude where's my props".

Agreed. Please revert this and consider backwards compatibility next time.

Any possibility of reverting this? The error is super annoying.

this merge is really a bad idea.

@markerikson @gaearon What do you think? This seems to be getting a lot of people in trouble with how Babel puts extra properties on modules (default and _esModule, by default). Outside of special casing those properties, which seems like the wrong solution, I think we should just remove this for now.

Well, on the one hand, it is just a warning.  On the other hand, it does seem to clash with how Babel currently works, and our own samples show the usage pattern that leads to that warning.
Given that bindActionCreators appears to skip over non-functions already, I'm okay with dropping the warning.
(I'm also still seeing pros and cons for having filtering for only functions in stuff like compose.)

I think that at least a warn instead of error message on the console would better suit this situation. When I saw the error msg I expected that my code was broke and things related to the error wasn't working, turns out it's just a warning, actually.

Agree that downgrading to a warning will be better but this should really just be reverted :-/

Wait, our warning isn't a warning? Hmmm... 🤔
Anyways, I'll look at getting this change reverted shortly. I need to switch laptops, but I'll do that tonight.

Running tests with Jest will see it as a console.error:
console.error node_modules\react-native\Libraries\Core\ExceptionsManager.js:71
      bindActionCreators expected a function actionCreator for key 'Actions', instead received type 'object'.

OP here... I'll get the revert PR together now. Sorry for all the trouble.

Reverted and pushed out in 3.7.1. Thanks for the feedback everyone!

Wait, our warning isn't a warning? Hmmm... 🤔

@timdorr I tried to say that here 👇

could be helpful if the the warning is a console.warn() instead of an console.error

Sorry I didn't chime in on this. That's exactly the reason we didn't do this in the first place 😛

Even filtering __esModule would not work well because people often export constants as named exports from the same file.


How can I optimize parameter to get book's imge for those attached typed files ?

you may need to retrain the network with your own datasets.

you may need to retrain the network with your own datasets.


TST: sparse.linalg: Loosen tolerance for the lobpcg test 'test_tolerance_float32'

This test has been failing occasionally in CI, with errors slightly greater than
the requested tolerances.
For example, in both https://github.com/scipy/scipy/pull/16754 and https://github.com/scipy/scipy/pull/16680, the failure report is
____________________________ test_tolerance_float32 ____________________________
[gw0] linux -- Python 3.9.13 /opt/hostedtoolcache/Python/3.9.13/x64/bin/python
/opt/hostedtoolcache/Python/3.9.13/x64/lib/python3.9/site-packages/scipy/sparse/linalg/_eigen/lobpcg/tests/test_lobpcg.py:355: in test_tolerance_float32
    assert_allclose(eigvals, -np.arange(1, 1 + m), atol=1.5e-5)
E   AssertionError:
E   Not equal to tolerance rtol=1e-07, atol=1.5e-05
E
E   Mismatched elements: 1 / 3 (33.3%)
E   Max absolute difference: 1.62124634e-05
E   Max relative difference: 5.40415446e-06
E    x: array([-1.      , -1.999997, -3.000016], dtype=float32)
E    y: array([-1, -2, -3])

The tolerances have been bumped before, in https://github.com/scipy/scipy/pull/15271.  I think we're still working out just how accurate we can expect the function to be when the inputs are float32.

The refguide check failure is unrelated, and it was just fixed in https://github.com/scipy/scipy/pull/16754.


[ENH] - Add a way for users to check if Nebari Workflow Controller is deployed

Users may not know if NWC is deployed or not.  It can be frustrating if it's not deployed b/c the expected result will not happen.
To fix, I could add a https:///argo/nwc/info subpath which returns {"version":"<nwc version>"} if it's deployed.

I should probably put all the nwc subpaths behind a common nwc subpath.  E.g. https://<nebari-domain>/argo/nwc/info, https://<nebari-domain>/argo/nwc/validate, https://<nebari-domain>/argo/nwc/mutate


Graph passes not showing as registered in libnnvm.a

If I link against libnnvm.a in my application, no passes are shown as registered. If I explicitly compile in (fir instance) $(NNVM_ROOT)/build/pass/saveload_json.o I have access to the LoadJSON and SaveJSON passes.
Is this expected behavior?

Will need to link the library via --whole-archive,  c.f. https://github.com/dmlc/nnvm/blob/master/Makefile#L41

Perfect, thanks!

have fun

@tqchen I have other questions about using NNVM - would you prefer I open issues each time to keep everything public, or if you're willing we could email as well (I sent one last night to your .edu address)

@slayton58 Interesting, I checked my inbox and didn't find the email. Maybe you could try send  another one.

@tqchen I just sent another one from my nvidia address, subject "NNVM C API"

Still not get it. Make sure it is tqchen at cs dot washington

I used tqchen at cs dot washington dot edu (just tried with my personal email too)


Configuration script for Cumulocity

Proposed changes
I added a new script that covers the this topic.
https://github.com/thin-edge/thin-edge.io/blob/main/docs/src/tutorials/connect-c8y.md
you can execute the script and it will walk you through all the stops
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist

[ x] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

@mstoffel-sag Thanks for your PR btw 😄 , much appreciated

Hmm shellcheck is not liking read -p but I check this syntax with dash which claims to be POSIX compliant...

Hmm shellcheck is not liking read -p but I check this syntax with dash which claims to be POSIX compliant...

No it is not the dash but rather the -p option which is not POSIX compliant. It is doing POSIX check because you are using the #!/bin/sh shebang, and not the bash one #!/bin/bash.
The former shebang tries to be more generic and compatible to different shells, whereas #!/bin/bash is only suitable for bash (so you need bash installed to run it).
Below is a link showing that the posix variant of the read command does not support -p.
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/read.html

dash

Ah right, sure. Though I just checked and technically the read command is not in-built function but rather a binary, so the version of read really depends what is provided by the operating system. POSIX just tries to normalize this by saying what functionality is supported by all implementations of read which are called POSIX compliant.
But should be easy enough to rewrite to be more compatible:
printf "Do you really want to do something? [Yn]"
read -r VALUE

# do something with the value
echo "Response: $VALUE"

Makes sense, thanks! Will convert it.

Closing as the motivation for this script was to prepare a demo thin-edge device quickly. A new demo project has been created to fulfil this requirement:

tedge-demo-container


Multi-install mode

Would be great to either default to multi-install mode, or have a setting for it.  Currently when I have two devices hooked up I get this:
[-] PARSING BUCK FILES...FINISHED 0.4s
BUILD FAILED: 2 device(s) matches specified device filter (1 expected).
Either disconnect other devices or enable multi-install mode (-x).

There's already a setting for "-x" option:
Preference > Tools > Buck
If not see it please update to the latest version:
https://plugins.jetbrains.com/plugin/7826
Thanks.

Awesome hadn't seen that!  Maybe add a little wrench icon to the buck tool
window as a shortcut to that?
On Tue, Aug 25, 2015 at 3:22 PM, Yanxing Wang<EMAIL_ADDRESS>wrote:

There's already a setting for "-x" option:
Preference > Tools > Buck
If not see it please update to the latest version:
https://plugins.jetbrains.com/plugin/7826
Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/wangyanxing/buck_idea_plugin/issues/2#issuecomment-134758977
.

Sounds a good idea!


Docker image contains unnecessary layouts and cached files

Hi!
After reading your blog at habr.com and experimenting with EventNative I've found that docker image contains unnecessary layers which makes it so huge.
Please see my pull request #82

Hi @iaroslavscript! Thank you! I've merged it.

Thanks!
By the way I highly recommend to move app/res/ and logs/events/ away from user home directory.
Because currently these volume paths depend on user name.


Doc update: Connecting a project

Required but missing from the quickstart doc:

@Crowbrammer do you mean running firebase init?  If so I believe you fixed this with #85, right?

@samtstern I believe so, too!


csp_crc32_verify: Exclude CRC size from packet length calculation

When verifying the CRC in csp_crc32_verify(), it is incorrect to calculate the CRC for the entire packet, including the CRC itself.
In csp_crc32_append(), we generate a CRC value for the entire packet and then append this value at the end.
This commit corrects the issue by subtracting the size of the CRC from the frame_length, thus fixing the end address calculation.
This fix aligns with the proposal made by clythersHackers in https://github.com/libcsp/libcsp/issues/514.
Fixes #514

I have tested my modifications using the server/client example.
sudo ip link add dev vcan0 type vcan
sudo ip link set up vcan0
ip link show vcan0
./examples/csp_server -c vcan0 -a 2 &
./examples/csp_client -c vcan0 -a 3 -C 2

The issue was with the CRC header part, so I defined CSP_21 to append the CRC on the header and the data.
Then, I used some debug prints to check if the code was working properly with and without the header.


Add Unitus to Base Ecosystem

Add Unitus to Base Ecosystem.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/2

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
1

Sum
2


displayPreference = DisplayMode.Flatscreen provides no interaction?

When developing on .Net desktop and running with displayPreference = DisplayMode.Flatscreen I do get the simulator window as expected, but it seems no interaction is possible. Plus I see the waiting indicator (blue rotating circle). The virtual hands do respond to mouse interaction, but nothing else seems to be possible. Any idea what could be causing this?
Below what little log output there is:
[SK diagnostic] Initializing StereoKit v0.3.5...
[SK diagnostic] Initialized Assets
[SK diagnostic] sk_gpu: Using Direct3D 11: NVIDIA GeForce RTX 3090
[SK diagnostic] Starting flatscreen mode
[SK diagnostic] Created swapchain: 1280x720 color:rgba32_linear depth:depth32
[SK diagnostic] Initialized Platform
[SK diagnostic] ISAC audio backend not available, falling back to miniaudio! It's likely the device doesn't have Windows Sonic enabled, which can be found under Settings->Sound->Device Properties->Spatial Sound.
[SK info] Using audio backend: WASAPI
[SK diagnostic] Initialized Audio
[SK diagnostic] Initialized Defaults
[SK diagnostic] Initialized World
[SK diagnostic] Initialized Sprites
[SK diagnostic] Initialized Lines
[SK diagnostic] Initialized UI
[SK diagnostic] Initialized Renderer
[SK diagnostic] Initialized Physics
[SK diagnostic] Initialized Input
[SK info] Initialization successful
[SK info] Fetching model index
[SK info] Waiting for model list
[SK info] Model: IridescentDishWithOlives.glb
[SK info] Model: plane-1m.glb
[SK info] Model: photogrammetry-sand-lime-brick-factory.glb
[SK info] Model: suzanne.glb
[SK info] Have 4 models

The regular OpenXR session begin seems to be the point at which output stops? Removing the displayPreference override makes the application work in an XR-3 headset.

Your log doesn't seem to show any red flags here, this is a successful initialization that has properly advanced into your application code!

I do get the simulator window as expected, but it seems no interaction is possible.

Could you provide a bit more detail about this? Which interactions are you trying to do, is this like, simulator inputs like WASD and pinch/grasp aren't working, or the UI isn't reacting to hand input?

Plus I see the waiting indicator (blue rotating circle).

I could imagine something like this if the app got caught in an infinite loop, but I'm not actually sure what would cause this if the simulator is still responsive! Is this in place of the mouse, or hovering over the simulator window?

Could you provide a bit more detail about this? Which interactions are you trying to do, is this like, simulator inputs like WASD and pinch/grasp aren't working, or the UI isn't reacting to hand input?

So the only interaction that appears to do anything is moving the mouse, for which the hand model moves. But I can find no keyboard or mouse action that does anything. This is with testing on my normal dev laptop (which doesn't have the Varjo headset connected).

I could imagine something like this if the app got caught in an infinite loop, but I'm not actually sure what would cause this if the simulator is still responsive! Is this in place of the mouse, or hovering over the simulator window?

Yes, the regular mouse cursor is replaced by the waiting indicator. But I'm trying to understand where that indicator comes from anyway. If I create a new SK .Net Core project and simply run the templated code (Debug, Any CPU mode) then I also see the busy indicator, even though all the interactions do seem to work ok. The indicator for some runs goes away after a few seconds, and for some stays around. If I do the same for the UWP template (Debug, x64 mode) the loading indicator stays around for 1-2 seconds, but then disappears and simulator interaction seems to be fine. Would you have any clue where the indicator comes from?

I'm starting to think the displayMode value was a red herring and my code doesn't run well (i.e. no interaction) in the simulator anyway. I have no idea what exactly makes it works when running on true OpenXR on the XR-3, but I suspect it might have something to do with async code. As I'm running a HttpClient request at startup, which I await somewhere further down the code (but before entering the main render loop) my Run() and Main()methods are markedasyncand have return valueTask`. This whole async programming is a bit new to me (I'm more at home with regular multi-threading), so I might be doing something wrong that is causing the lockup in simulator mode (but no idea why it doesn't also happen then in OpenXR mode).
On a high level my code is structured like this:
class Application
{
    HttpClient http_client;

    async Task Run()
    {
        http_client = new HttpClient();
        Task<HttpResponseMessage> fetch_models_task = http_client.GetAsync(FileURL("index.txt"));

        // scene initialization stuff

         HttpResponseMessage models_response = await fetch_models_task;
         if (models_response.StatusCode == System.Net.HttpStatusCode.OK) { /* process response */ }

         while (SK.Step(() =>
         {
              // Draw models and UI
         }
    }

    static async Task Main(string[] args)
    {
            SKSettings settings = new SKSettings
            {
                appName = "ModelViewer",
                assetsFolder = "Assets",
            };

            if (!SK.Initialize(settings))
                Environment.Exit(1);

            Application app = new Application();
            await app.Run();

            SK.Shutdown();
     }
}

Should this work as expected?

Okay, I do see the same issue as well when running this code! The window is becoming "unresponsive" while the application is still continuing, which is absolutely wild! I'm not an expert at async code either, but this definitely looks a little odd to me :D I've never seen an async Main before, that's for sure!
I trimmed the async code to be a bit less contagious, and this seems to remove the issue. Removing async from Main wasn't enough on its own, I also needed to remove async from Run. Still not certain what exactly was happening there though! Async code seems to have a tendency to be a little contagious? When it does that, it's often a good idea to stop the contagion using some of the Task methods/properties. I kinda use Task.Run, task.Result and task.ContinueWith a good bit, and usually use await to avoid nesting callbacks?
using StereoKit;
using System;
using System.Net.Http;

class Application
{
	HttpClient http_client;

	void Run()
	{
		http_client = new HttpClient();
		http_client.GetAsync("https://playdeck.net/").ContinueWith(t=>{
			HttpResponseMessage models_response = t.Result;
			if (models_response.StatusCode == System.Net.HttpStatusCode.OK) { /* process response */ }
		});

		// scene initialization stuff
		while (SK.Step(() =>
		{
			// Draw models and UI
		}));
	}

	static void Main(string[] args)
	{
		SKSettings settings = new SKSettings
		{
			appName = "ModelViewer",
			assetsFolder = "Assets",
		};

		if (!SK.Initialize(settings))
			Environment.Exit(1);

		Application app = new Application();
		app.Run();

		SK.Shutdown();
	}
}

Hmm, that's an interesting way of handling the async stuff. However, within the lambda function you added to process the response I need to do response.Content.ReadAsStringAsync() (as I don't see another way to get the string). Which then again leads to having to use await on that call, which then contaminates the lambda itself, and the function it's called within, etc. What would be a good way to handle that? Maybe I'm better off using WebClient.DownloadString()

You should be able to just say response.Content.ReadAsStringAsync().Result? You may also be able to turn the lambda into a named async function of its own, or perhaps there's some way to make an async lambda :D

You should be able to just say response.Content.ReadAsStringAsync().Result?

Thanks, that did the trick.
I guess we can close the original issue here, although I'll check next week to see if the lockup is indeed gone now on the system the XR-3 is connected to.

Sounds great! I expect that should do the trick, but yeah, feel free to pop this one back open if something persists :)


pycdc: init at unstable-2024-10-13

Pycdc is currently the state-of-the-art Python decompiler. I occasionally use it in my dayjob (I'm a reverse engineer), since it has the best support for modern python versions, and a reasonable support for malicious (obfuscated) python bytecode.
Unfortunately, even though it's mature and quite popular, it looks like it has no official release version, so I have to version it as unstable-2024-10-13.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

generated using nixpkgs-review
--------- Report for 'x86_64-linux' ---------
1 package built:
pycdc


Apps Draft

fixes #245
fixes #249
fixes #416
fixes #417
fixes #239
fixes #247
fixes #250
fixes #248
fixes #241
fixes #242
fixes #244
fixes #240

Interesting. Linter passes locally but not in the pipeline 🤔. #Investigating.

!deploy


cmd/compile: function argument hiding built-in function results in a compiler crash

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
Tested with GO 1.11
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Compiling on Windows, reproduced targeting both Windows and Linux
What did you do?
NOTE: This does not reproduce on play.golang.org, but I have reproduced it on 3 different machines
If a function argument hides a built-in function and there is an attempt to call the built-in function the compiler will crash with no indication of where it was in the compilation process, which makes it hard to find in a large codebase.
Here is a simple repo of the issue
package main
var a = []byte{1,2,3}

func bug(len int) bool {
    return len < len(a)
}

func main() {
    println(bug(5))
}

Here the argument len hides the built-in function len, and in that same function a call is made to the function len.
What did you expect to see?
Compiler Error : cannot call non-function len (type int)

What did you see instead?
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x70 pc=0xb94675]
goroutine 1 [running]:
cmd/compile/internal/gc.typecheck1(0xc000352580, 0x2, 0xc00033fa40)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:1270 +0x3ed5
cmd/compile/internal/gc.typecheck(0xc000352580, 0x2, 0xc00033fa40)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:238 +0x6b4
cmd/compile/internal/gc.typecheck1(0xc000352600, 0x12, 0xc000084998)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:585 +0x2cce
cmd/compile/internal/gc.typecheck(0xc000352600, 0x12, 0xc0000849d8)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:238 +0x6b4
cmd/compile/internal/gc.typecheckslice(0xc0000885d0, 0x1, 0x1, 0x12)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:68 +0x57
cmd/compile/internal/gc.typecheck1(0xc000352680, 0x1, 0xcbf373)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:2065 +0xadd4
cmd/compile/internal/gc.typecheck(0xc000352680, 0x1, 0x3)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:238 +0x6b4
cmd/compile/internal/gc.typecheckslice(0xc0000885d8, 0x1, 0x1, 0x1)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:68 +0x57
cmd/compile/internal/gc.Main(0xcdea30)
        c:/go/src/cmd/compile/internal/gc/main.go:518 +0x20d0
main.main()
        c:/go/src/cmd/compile/main.go:51 +0x9d

It reports the error correctly with Go 1.10. Looks like a regression.

cc @griesemer @mdempsky


Resources not visible after selecting subscription.

Hello,
I am trying to deploy changes and a new function to my function app however I am unable to get the resource group to show in order to select the app. I can find my subscription, but after selecting that. There are no resource groups returned at all.

Could you share a screenshot?
Also, try using the grouping options to get it to show up.

Closing in favor of https://github.com/microsoft/vscode-azureresourcegroups/issues/780


Adds and completes a number of methods on the Tokenizer

Adds decode_batch method
Adds skip_special_tokens handling to both decode and decode_batch
Adds vocab and vocab_size
Adds no_padding and padding methods
Fills out keywords in enable_padding
Adds to_str method
Adds optional pretty argument to save
Adds enable_truncation, truncation, and no_truncation methods

Thanks for another great PR @petergoldstein! Changed to_str to to_s to avoid implicit coercion (294278044e6c11973a498d0fc6c8e7ceaaeff86c).

@ankane Good change.
Do you see any large significant outstanding items that need to be added to this gem at this point?  If so let me know and I'll take a whack at them.

I think the main ones are around completing the training API (adding more decoders, models, normalizers, pre-tokenizers, post-processors, and trainers), but I imagine they'll be used significantly less than the pre-trained models.
I'd typically do a release right now, but want to wait a little bit to see if we can use a stable version of Magnus.


Add paragraph on how to install dep on Windows

Currently, the dep documentation does not provide information on how to install dep on Windows.
/ref: dep/docs/installation.md

Installing from source works well, adjusting for powershell (quote the directory and add a $ before DEP_LATEST:
go get -d -u github.com/golang/dep
cd "$(go env GOPATH)/src/github.com/golang/dep"
$DEP_LATEST=$(git describe --abbrev=0 --tags)
git checkout $DEP_LATEST
go install -ldflags="-X main.version=$DEP_LATEST" ./cmd/dep
git checkout master

Per 2086, this also works:


Use HTTPS in external resources

Insecure content warning occurred by robohash and company logo's url in HTTPS web site.
I think that's better to use https in any external resources.

I think robohash just turned on force-ssl, which now fucks with open-uri. @stympy any chance this could be merged? It shouldn't break anything since HTTPS content on an HTTP page doesn't qualify as "mixed content"
redirection forbidden: http://robohash.org/etsintdolorem.png?size=300x300&set=set1 -> https://robohash.org/etsintdolorem.png?size=300x300&set=set1

from al my heart, thank you :heart:


documize-community: 3.5.0 -> 3.5.1

Motivation for this change
https://github.com/documize/community/releases/tag/v3.5.1
This patch also switches to a go.mod-based build as upstream already
switched to it a while ago[1].
[1] https://github.com/documize/community/commit/b8268521372cdf8862af4f6924ec476b3e431690
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg test documize

Backported as 51f55f323cf6db8815f8bca5a2fc1645a88ff500.


feat: add translatable package and update Resource model to use it

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Nihon Kohden file (.LOG) annotations read incorrectly

Description of the problem
Hi there. I have some EEG data recorded using the Nihon Kohden (NK) system, this data includes .EEG files which store the raw EEG recording, but also the extension files: .21E, .PNT, .LOG ( https://mne.tools/0.21/auto_tutorials/io/plot_20_reading_eeg_data.html#nihon-kohden-eeg-data-eeg-21e-pnt-log)
The .LOG extension file store the annotated event time stamps. I need precise annotation times but I'm only getting timestamps rounded to the nearest full second. Has anybody dealt with this file format or have suggestions on how to add more precise annotation support
Steps to reproduce
import mne

f = mne.io.read_raw_nihon(file_path)
f.annotations

Link to data
No response
Expected results
Here is the NK file loaded into EDF browser using the Nihon Kohden to EDF+ tool. You can see the true annotations times on the right. Will also try to validate this on BESA

Actual results
mne.io.read_raw_nihon.annotations returns the 'onset' timestamp rounded to the nearest second

Additional information
I've played around with the _read_nihon_annotations method in
https://github.com/mne-tools/mne-python/blob/main/mne/io/nihon/nihon.py without any luck. Unable to decode to LOG files into a format other than the datetime '%H%M%S' format provided

Link to data

No response

Can you share data that we can use to test? Then someone can take a look and try to fix it