Causal surrogate tests are not failing

Describe the bug
Since I updated the pandas version to be compatible with the lifelines library, some tests in tests/surrogate_tests/test_causal_surrogate_assisted.py are failing because the new version of pandas no longer supports the append method. @f-allian, please can you take a look at this when you get time.
To Reproduce
Steps to reproduce the behavior:

Go to the temporal-shenanigans branch
Run pytest
See error

Expected behavior
All the tests should pass

@jmafoster1 Can you let me know what versions of pandas and lifelines you've upgraded to please? A pip freeze output would be helpful.

It's the versions in the pyproject.toml file, so lifelines~=0.29.0 and pandas>=2.1


can't use Ctrl+. or Ctrl+; as a keyboard shortcut

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

Version: 1.81.1
Commit: 6c3e3dba23e8fadc360aed75ce363ba185c49794
Date: 2023-08-09T22:18:39.991Z
Electron: 22.3.18
ElectronBuildId: 22689846
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0

Steps to Reproduce:

Open Keyboard Shortcuts

Change the Keybinding for "Show All Commands" by double-clicking it

Press Ctrl + . to set the keybinding

Notice that the display shows "Ctrl + Unknown"

Press Enter to save it

Notice that the Keybindings.json shows "Ctrl + Unknown"
 {
         "key": "ctrl+unknown",
         "command": "workbench.action.showCommands"
 },

Change the Keybindings.json to read "Ctrl + ." and save it

Notice that Ctrl + . does not perform the bound action

Note: The same problem occurs for Ctrl+;.  I could not find any other keys that have this behavior.

Please use the advice at https://github.com/microsoft/vscode/wiki/Keybinding-Issues to investigate and add Information here.

Dup of #142583


Integrate RAGAS as part of the app

Apps need a clear way to think about how to evaluate various benchmarks on R, A, G etc. There is a good opportunity for us to include it directly here. We need to figure out how the database schema etc. for this is going to be. I will research the option by next week so that I can get an idea of how to do this.

Closing -- Library isn't as robust as we though


Notifications not working with Node v8.4.0

Laravel Mix Version: v1.4.2
Node Version: v8.4.0
NPM Version: v5.4.0
OS: macOS Sierra (10.12.6)

Description:
With notifications enabled I get following error when running Laravel Mix on Node v8.4.0:
internal/child_process.js:325
    throw errnoException(err, 'spawn');
    ^

Error: spawn EACCES
    at _errnoException (util.js:1041:11)
    at ChildProcess.spawn (internal/child_process.js:325:11)
    at exports.spawn (child_process.js:493:9)
    at Object.exports.execFile (child_process.js:208:15)
    at Object.module.exports.fileCommand (node_modules/node-notifier/lib/utils.js:53:13)
    at NotificationCenter.notify (node_modules/node-notifier/notifiers/notificationcenter.js:66:11)
    at module.exports.WebpackNotifierPlugin.compilationDone (node_modules/webpack-notifier/index.js:62:18)
    at Compiler.applyPlugins (node_modules/tapable/lib/Tapable.js:61:14)
    at emitRecords.err (node_modules/webpack/lib/Compiler.js:268:11)
    at Compiler.emitRecords (node_modules/webpack/lib/Compiler.js:375:38)
    at emitAssets.err (node_modules/webpack/lib/Compiler.js:262:10)
    at applyPluginsAsyncSeries1.err (node_modules/webpack/lib/Compiler.js:368:12)
    at next (node_modules/tapable/lib/Tapable.js:218:11)
    at Compiler.compiler.plugin (node_modules/webpack/lib/performance/SizeLimitsPlugin.js:99:4)
    at Compiler.applyPluginsAsyncSeries1 (node_modules/tapable/lib/Tapable.js:222:13)
    at Compiler.afterEmit (node_modules/webpack/lib/Compiler.js:365:9)
    at require.forEach.err (node_modules/webpack/lib/Compiler.js:354:15)
    at node_modules/async/dist/async.js:421:16
    at iteratorCallback (node_modules/async/dist/async.js:998:13)
    at node_modules/async/dist/async.js:906:16
    at node_modules/graceful-fs/graceful-fs.js:43:10
    at FSReqWrap.oncomplete (fs.js:135:15)
npm ERR! code ELIFECYCLE
npm ERR! errno 1

A temporary workaround is to set mix.disableNotifications() in webpack.mix.js.

Permissions error. Can you delete node_modules directory and run npm install or yarn again to see if it works?

@ruchern I have already done that and it had no affect.

Okay, just letting you know that I tried with:
laravel-mix v1.4.2
npm v5.4.0
node v8.4.0
and notifications are working fine.

Seems to be related to https://github.com/npm/npm/issues/18399.

Yes, it does seems like an npm issue. Strangely, I am on v5.4.0 and I do not have problems with executing binaries.

@ruchern What operating system are you using?

@jackmu95 macOS 10.13 Developer Beta 9.

@ruchern Hmm weird. I'am using npm v5.3.0 for now.

Related and duplicate of #1065

This error got fixed with npm v5.4.1.

@jackmu95 Yes. I have updated it in #1065


Export sourceBuffer. Create initSegment's

Hi. I use this project in ws-scrcpy to display smartphone screen in a web browser.

For a long session it is possible to use all available memory, so I need to be able to remove data from SourceBuffer.

const converter = new VideoConverter(tag, fps, fpf);

// later, when we know we have too much data buffered
if (!converter.sourceBuffer.updating) {
    converter.sourceBuffer.remove(start, end);
}

When device is rotated, I receive SPS frame with new dementions. To handle changes I need:
2.1. Parse each SPS & PPS
2.2. Create initSegment's for each IDR frame

Also in Safari video rendering will stop, if you remove last initSegment, i.e. we can't remove segments from SourceBuffer in Safari in the current version (because VideoConverter creates only one initSegment).
refs https://github.com/NetrisTV/ws-scrcpy/issues/111

@drauggres Thank you for your contribution!


docs: fix typo

What type of PR is this?

/kind documentation
What this PR does / why we need it:
fix typo, readerr => reader
Which issue(s) this PR fixes:

null

@justlorain  Thank you so much for the PR!
Don't forget to contact<EMAIL_ADDRESS>so we can send you some freebies 😀


Update Documentation for transcriptUploadInteractor (support MultiWOZ update)

See #83 : updated transcriptUploadInteractor, extending it to support not only single transcripts but multiple MultiWOZ transcripts in one file.
Aidan Thurs 1 Dec: Managed to enable proper updating of user's intents such that we can upload one dialogue with multiple transcripts in one go, and see the aggregate visualization of all the transcripts with the Visualize All function. Ensured this adheres to the Open-Closed Principle because we still support single transcripts like we did previously, but we also have the ability to upload multiple transcripts in one file. This also agrees with the Liskov Substitution principle because we can substitute a single transcript into a multiple transcript format.

Aidan 5 Dec: Have updated the headers of all the Use Case files with new features, and fixed some erroneous documentation in Get Use Case files. Need to check that these changes have been deployed to JSDoc with @BerkeAltiparmak .
Should think about updating the main README if necessary.

The changes on the structure of the code (MultiWOZ update + having Transcript as an entity) has been reflected on the documentation page using JSDoc. Next is to deploy it again on the documentation website.


Added Windows 10 Kit and UAP SDK capabilities detection

I see no proper capabilities detection since new Windows 10 versioning schema.
So I propose this implementation:

Fixed Windows Kit detection for version 10;
Added Windows UAP SDK capability with version of the last installed SDK.

I don't see the value of listing all the installed UAP SDK because only the last is referenced in the "Windows Kits\10\SDKManifest.xml" file. But I'm open to a proposition if someone see a value for this :)
Thanks!

Up! It would be nice if someone can review this Pull Request 🙂
Thank you.

Hello!
I think this PR is always a good proposal to detect and provisoning automatically Windows 10/11 Kit and UAP SDK capabilities.
What do you think?

/azp run


Tiller does not clean up old config maps

Hi,
I almost never check the kube-system namespace, but when doing yesterday I noticed Tiller keeps a lot of old state / history around in config maps in its namespace, and never does any cleanup as far as I can see?
This seems like problems waiting to happen as these will just keep growing, and I don't see any Helm commands to do a purge.
Is there a way we can limit this? (similar to Deployments RevisionHistoryLimit?)

hey @bcorijn -- Tiller keeps release history around in case you want to rollback to a particular version. You can remove the configmap for a release when you do helm delete it by adding the --purge flag.
Does that help?

Yes, but without a limit the ConfigMaps keep growing.

@michelleN: I understand they are used to rollback to a particular release and they can be removed by adding a --purge flag during delete. However this does nothing to stop the fact that in a busy cluster sooner or later there will be a huge number of old, useless resources, something Kubernetes often does not deal very well with in my experience.
In Kubernetes core the amount of old resources left behind by a deployment or cron job are limited, a concept I had assumed Helm followed as well. I don't think there is a need to be able to rollback 30 versions ago?

We are running into this issue frequently. I support an application that is building and deploying using a CI model that frequently releases several versions of the application a day. We have now run into an issue multiple times where the configmap has grown so large that helm times out trying to retrieve it. We end up manually trimming the releases, but that assumes that we can retrieve the configmap in the first place (i.e. when this happens even kubectl times out trying to retrieve the configmap).
We are probably going to have to build a step into our CI system that retrieves the releases and trims them, but having a flag in helm that does this automatically would save us quite a bit of work.

I am just about done with a fix for this.

@technosophos Nice! Could you explain what you mean exactly with To enable this feature, use helm init --history-max NNN. Note that because of the restrictions on Deployment objects, you will have to re-install Tiller to add a limit. Will that init automatically re-deploy Tiller in my cluster or will I have to take any manual precautions? What will happen with releases that already have more history than the limit you set?

@bcorijn if you have an existing tiller instance in your cluster, you'll also need to supply the --upgrade flag. Either that or helm reset && helm init --history-max NNN if --upgrade doesn't work.

This of course will not work until Helm 2.7 is released (or you're using the canary deploys.

What will happen with releases that already have more history than the limit you set?

You can expect them to be removed the next time you call helm upgrade, as far as I understand the code.

@bacongobbler Perfect, thanks! Will check it out once 2.7 drops.

This is great. We have this issue because we are using tiller in a resource-limited namespace where we are allowed to create only 10 configmaps and we frequently hit this limit.

I couldn't wait until 2.7 landed - my tiller was crashing with OOM because of way too many configmaps. I wrote this script to clean up old releases. Use with caution, but it unblocked me until 2.7 lands.
#!/usr/bin/env bash

TARGET_NUM_REVISIONS=10
TARGET_NUM_REVISIONS=$(($TARGET_NUM_REVISIONS+0))

RELEASES=$(kubectl --namespace=kube-system get cm -l OWNER=TILLER -o go-template --template='{{range .items}}{{ .metadata.labels.NAME }}{{"\n"}}{{ end }}' | sort -u)

# create the directory to store backups
mkdir configmaps

for RELEASE in $RELEASES
do
  # get the revisions of this release
  REVISIONS=$(kubectl --namespace=kube-system get cm -l OWNER=TILLER -l NAME=$RELEASE | awk '{if(NR>1)print $1}' | sed 's/.*\.v//' | sort -n)
  NUM_REVISIONS=$(echo $REVISIONS | tr " " "\n" | wc -l)
  NUM_REVISIONS=$(($NUM_REVISIONS+0))

  echo "Release $RELEASE has $NUM_REVISIONS revisions. Target is $TARGET_NUM_REVISIONS."
  if [[ $NUM_REVISIONS -gt $TARGET_NUM_REVISIONS ]]; then
    NUM_TO_DELETE=$(($NUM_REVISIONS-$TARGET_NUM_REVISIONS))
    echo "Will delete $NUM_TO_DELETE revisions"

    TO_DELETE=$(echo $REVISIONS | tr " " "\n" | head -n $NUM_TO_DELETE)

    for DELETE_REVISION in $TO_DELETE
    do
      CMNAME=$RELEASE.v$DELETE_REVISION
      echo "Deleting $CMNAME"
      # Take a backup
      kubectl --namespace=kube-system get cm $CMNAME -o yaml > configmaps/$CMNAME.yaml
      # Do the delete
      kubectl --namespace=kube-system delete cm $CMNAME
    done
  fi
done

Not sure if helpful to anyone, but I made a small script that will clean up all DELETED helm deployment config maps. Essentially just runs through them all and purges. Was helpful to me as I had lots of short-lived deployments during testing which only had 1 or 2 revisions.
#!/usr/bin/env bash
RELEASES=$(kubectl --namespace=kube-system get cm -l OWNER=TILLER -l STATUS=DELETED -o go-template --template='{{range .items}}{{ .metadata.labels.NAME }}{{"\n"}}{{ end }}' | sort -u)

for RELEASE in $RELEASES
do
  helm delete $RELEASE --purge
done

Run into the same issue, It caused that kube-apiserver use too much memory up to the limits. kube-apiserver was OOM killed. Following is kube-apiserver.log:
I1018 02:27:10.465142       1 trace.go:76] Trace[967179655]: "List /api/v1/namespaces/kube-system/configmaps" (started: 2018-10-18 02:27:07.589957638 +0000 UTC m=+50270.823456577) (total time: 2.875147723s):
Trace[967179655]: [1.308105695s] [1.30803837s] Listing from storage done
Trace[967179655]: [2.87514482s] [1.566432149s] Writing http response done (702 items)
I1018 02:29:08.443211       1 trace.go:76] Trace[1423692943]: "List /api/v1/namespaces/kube-system/configmaps" (started: 2018-10-18 02:29:07.346663261 +0000 UTC m=+50390.580162229) (total time: 1.096515156s):
Trace[1423692943]: [1.079106655s] [1.079002293s] Listing from storage done
E1018 02:29:20.446953       1 writers.go:149] apiserver was unable to write a JSON response: client disconnected
E1018 02:29:20.446978       1 status.go:64] apiserver received an error that is not an metav1.Status: client disconnected
I1018 02:29:20.448104       1 trace.go:76] Trace[1455099037]: "List /api/v1/configmaps" (started: 2018-10-18 02:29:08.626741462 +0000 UTC m=+50391.860240414) (total time: 11.821327899s):
Trace[1455099037]: [11.821325849s] [11.819570395s] Writing http response done (783 items)

https://github.com/helm/helm/pull/2636 could fix it.

We have now run into an issue multiple times where the configmap has grown so large that helm times out trying to retrieve it.

If the configmap size for a release is too large to retrieve, it may that the tgz package is being included in the configmap. If you do not add *.tgz in the .helmignore file of the chart directory, the configmap will grow in size along with the tgz file.
In other words, each package command will include the previous package file (if present) if you do not add *.tgz to .getignore. The size of the package file will grow exponentially if it's not excluded.

Unfortunately that wasn't the case, we made sure the packages were included long ago. This is just an application (that isn't our problem anymore, btw :) ) that released many times every day and created huge configmaps in the kube-system namespace for storing the helm release information.


Reproducible Benchmarks

TODOs:

[x] Use deterministic point cloud generators
[x] Use non-inplace versions of operators
[ ] Update Kaolin scripts

bump

I am also changing the format of PointCloud to 3xNxB, so this benchmark need to be updated. I will make necessary changes to the benchmark once I finish this change.

@avik-pal  I have added kaolin equivalent script for benchmark, can you please review it.

LGTM. Can you run the benchmarks and report the results here before it goes in?

(Kaolin)
DEVICE: CPU
Running benchmarks for npoints = 4096
ScalePointCloud: 0.061511993408203125 ms
RotatePointCloud: 0.049591064453125 ms
RealignPointCloud: 8.382320404052734 ms
NormalizePointCloud: 0.48279762268066406 ms
Compose: 0.6380081176757812 ms

Running benchmarks for npoints = 16384
ScalePointCloud: 7.851362228393555 ms
RotatePointCloud: 7.779836654663086 ms
RealignPointCloud: 34.340858459472656 ms
NormalizePointCloud: 23.39315414428711 ms
Compose: 70.2211856842041 ms

Running benchmarks for npoints = 65536
ScalePointCloud: 7.91621208190918 ms
RotatePointCloud: 7.879734039306641 ms
RealignPointCloud: 34.55471992492676 ms
NormalizePointCloud: 23.876667022705078 ms
Compose: 70.770263671875 ms

Running benchmarks for npoints = 262144
ScalePointCloud: 8.008003234863281 ms
RotatePointCloud: 8.074760437011719 ms
RealignPointCloud: 39.08681869506836 ms
NormalizePointCloud: 26.52740478515625 ms
Compose: 75.27351379394531 ms

Running benchmarks for npoints = 1048576
ScalePointCloud: 9.147882461547852 ms
RotatePointCloud: 9.363174438476562 ms
RealignPointCloud: 74.33509826660156 ms
NormalizePointCloud: 48.63238334655762 ms
Compose:<PHONE_NUMBER>6594238 ms

CUDA is on. Running GPU Benchmarks
DEVICE: GPU
Running benchmarks for npoints = 4096
ScalePointCloud: 0.19551999866962433 ms
RotatePointCloud: 0.10787200182676315 ms
RealignPointCloud: 0.9790719747543334 ms
NormalizePointCloud: 0.23318399488925934 ms
Compose: 1.2248640060424805 ms

Running benchmarks for npoints = 16384
ScalePointCloud: 0.1963520050048828 ms
RotatePointCloud: 0.1074879989027977 ms
RealignPointCloud: 3.4454400539398193 ms
NormalizePointCloud: 0.2341119945049286 ms
Compose: 3.7479360103607178 ms

Running benchmarks for npoints = 65536
ScalePointCloud: 0.19500799477100372 ms
RotatePointCloud: 0.12303999811410905 ms
RealignPointCloud: 13.480992317199707 ms
NormalizePointCloud: 0.25571200251579285 ms
Compose: 13.843168258666992 ms

Running benchmarks for npoints = 262144
ScalePointCloud: 0.19657599925994873 ms
RotatePointCloud: 0.30796799063682556 ms
RealignPointCloud: 80.64806365966797 ms
NormalizePointCloud: 0.25577598810195923 ms
Compose: 81.30508422851562 ms

Running benchmarks for npoints = 1048576
ScalePointCloud: 0.2795200049877167 ms
RotatePointCloud: 1.0577919483184814 ms
RealignPointCloud:<PHONE_NUMBER>980469 ms
NormalizePointCloud: 0.3883199989795685 ms
Compose:<PHONE_NUMBER>75 ms

How are you running the benchmarks?

How are you running the benchmarks?

on cyclops

Cyclops can be a bit noisy, but more than that, use timeit to get your times.

Also please add graphs for these numbers to the readme.

Final graphs, I think using horizontal would be better.

The horizontal plot is definitely the one we should go with

I am thinking to add TriMesh benchmarks and graph in this PR before merging. Will rebase this once TriMesh PR is merged.

Maybe let's merge this PR before the TriMesh. We might want to do performance regressions in the benchmark PR for Meshes.
Also, don't forget to add the plots in the README

Sure

@nirmal-suthar You missed adding the plot to README

I have updated readme on master, missed it here.

Pretty sure it's on the readme rn


Unable to select files through the "Create a new shortcut" dialogue

Description
When trying to create a new shortcut to a file using the "New" -> "Create a new shortcut" dialogue, I am unable to select a file through the explorer as the shortcut target, despite the description saying "Create shortcuts to local network programs, files, folders, computers or Internet addresses".
I categorized this issue as a Bug because the description makes it sound like selecting files should be something a possibility.
Steps To Reproduce

Create a new Shortcut using the top bar or the right click context menu.
Click on "Browse" and navigate to the file you want to create a shortcut for.
Users are only able to select a folder, and not a single file in the explorer.

Requirements
Allow users to select local network programs, files, folders, computers and Internet addresses as the dialogue textbox says.
Files Version
<IP_ADDRESS>
Windows Version
22H2 Build 19045.3208
Log File
debug.log

Thanks for the report, I've added this to the project board.

@LucasOe does entering the path manually work?

Manually entering the path does work as intended. I was thinking there might be a flag for changing the behavior of the file explorer when invoking it.

There isn't a flag that allows picking files & folders from the same picker, but perhaps we can have a combo box to let users choose between files and folders.

@LucasOe there isn't a flag that allows both, but I just checked File Explorer and they have the same behavior. My recommendation is to use picker to locate the folder, and to manually enter that item name at the end of the path.


[Backport v2.9 - Windows] System Agent and rke2 startup may conflict with one another

This is a backport issue of https://github.com/rancher/rancher/issues/46344. The primary change which addresses this issue was done in the system-agent-installer-rke2, which will be pulled by both 2.9 and 2.10.

Ready for testing as the rancher/rancher PR has been merged. the KDM data included in this PR should be used for testing https://github.com/rancher/kontainer-driver-metadata/pull/1543
URL: https://raw.githubusercontent.com/rafaelbreno/kontainer-driver-metadata/refs/heads/october_rke2_9_patch/data/data.json

Validated this against v2.9 https://github.com/rancher/rancher/commit/c1a3cd5fd6d60f2851739d26af862592497ccdbb. Details have been recorded in Qase test run /RM 2661.
ENVIRONMENT DETAILS

Rancher install: Helm
Rancher version: v2.9 https://github.com/rancher/rancher/commit/c1a3cd5fd6d60f2851739d26af862592497ccdbb

TEST RESULTS

#
Scenario
Result

1
Validate rancher-wins and sc.exe output shows no errors on highest K8s version
:white_check_mark:

2
Validate rancher-wins and sc.exe output shows no errors on upgraded K8s version
:white_check_mark:

3
Validate rancher-wins and sc.exe output shows no errors on upgraded Rancher
:white_check_mark:


Create initial routes for NodeJS backend

Describe the feature
Create the following initial routes.

User

Create, Read, Update, Delete

Interest

Create, Read, Update, Delete

Question

Create, Read, Update, Delete

UserInterest

Create, Read, Update, Delete

UserQuestion

Create, Read, Update, Delete

ChannelInvitation

Create, Read, Update, Delete

SendImageToChannel
SendAudioToChannel
LeaveChannel

Describe the solution you'd like
The routes should all exist, pending implementation. They should all respond with 501 Not Yet Implemented.
Describe alternatives you've considered
You can additionally create skeletons of the models, too.
Additional context

@all-contributors please add @petrutsi for infra and code


[Test] Avoid code scanning falsely identify

Fix #3195
Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

GitGuardian Security also check history commits? After I add new commit, security still failed at old commit id, why?

Since GitGuardian also scan the history commit, I will close this PR and submit a new clean one.


Match package.json node engines to the docs

It refuses to install against node 10.8.0 due to the spec in package.json. After adjusting node engines the tests are still passing.

@mcollina I'm using yarn. I've actually just notice that there are some dependencies (at least on-net-listen) of this project configured the same way.

Thanks!!!!


Release notes not looking good in cheat.sh panel

Lacks formatting or at least new lines

Now looks a little better


client.zig: Correct handshake key length

The WebSocket spec states that:

The request MUST include a header field with the name
|Sec-WebSocket-Key|.  The value of this header field MUST be a
nonce consisting of a randomly selected 16-byte value that has
been base64-encoded (see Section 4 of [RFC4648]).  The nonce
MUST be selected randomly for each connection.

I must have overlooked this when I was implementing handshaking. Thanks for the PR.


For consideration: Add explicit support for globalized routes

Hi,
First off, this is a great library.  Thanks so much for the investment of time and effort... very well put together.
  Any thoughts on adding some defaults into your library to help with this?
I'm a little crunched for time right now, but if you think it's a good idea, I wouldn't mind throwing a few hours towards a nice implementation.
Thanks again,
...brent

Any progress? Would like to include localization support but have been slammed recently with a job, of all things.

Sadly, I'm in the same boat.  It's still on my list to look at, but I can't say when I'll be able to get a chance.

I just added the first crack at localization support in v1.5, now available on nuget. FYI.


Backup API documentation is severely inaccurate

Hey, I am in the process of using the Backup API to periodically backup my data to S3.
The documentation says to issue a PUT to /v1/backup/start and expect a response with the body {"status":"started", message:""} (which is invalid JSON fyi).
In reality, the API returns a response with an empty body. This can be verified here.
Also, the documentation says I can check the status of a backup by issuing a GET to /v1/backup/status. This endpoint does not exist. Looking at the source code, I confirmed it here. The lack of a status endpoint is disappointing as I was planning on monitoring the backup process.
Additionally, that source file shows that there is an undocumented endpoint to terminate a backup that is in process. It would be nice to have that endpoint documented.

I just checked out the source for the cli. Unless I am misreading it, it looks like the cli hits the /v1/plan endpoint to get the status. Which returns a huge json blob. But I think it gives me the info I need.

Also, /v1/plan returns a 503 http status code even though it seems to be functioning and returning the json payload. I guess 503 is considered expected behavior? The cli even has explicit code to ignore the 503 error.

@bishtawi Thanks very much for your detailed issue. The /v1/status endpoint is missing and the response to v1/backup/start is empty.  I'm filing a ticket to have the status endpoint put back and am updating the docs re: the expected response from v1/backup/start.
You are correct that v1/plan contains the status information you need.


Tidying

Tidying completed

Done -Line 4 needs to be indented once
Done-Line 15 is half indented which is fine (its even preferred by coders i'm told) but if all is not half indented then VS Code does not create the line for at and that makes life much harder for you to see where and if it gets closed.
Done -Line 20 can be on the same as line 19 (so all can come back one also)
Done -line 33 open tag needs to come back to be inline with its closing tag.
Done -and you p tags within div line 31-47 need to be on livel with the close p tag on line 46.-
Done- line 49 div needs to be brought back one indent so it can see its closer.
Done-Looking great now up to, the p brackets line 68 - 81, these need to be brought back one indent so they are level with the h3 tag.
Done-Line 81 p needs to be spread out over multiple lines like the ones above them.
Done-line 85 para needs to be brought back to the same as h3 on line 84.
Done -Lines 102-104 can be brought back just one space so they are the same as line 101.
Done -Line 117 needs to just be the p open tag.
Done-Line 133 is correct to be inline with its opener on line 14, but will of course be brought back to just 1 indent in from nothing :D
Done- Line 134 needs to see its closer one line 206 as does
Done -Line 135 with line 205.
Done -Lines 138-167 need to be brought back one indent.
Done -Line 174 -184 can come back one indent and the
The contents of the
Done -tags need to be on there own lines. Line you have done one 140-168.
Done -Lines 188-203 need to be brought back one indent also.
The contents of the
Done -tags in lines 189-200 need to be on there own lines. Line you have done one 140-168 Done - the close li tag on line 202 needs to be on its own line.
Done - Line 205 div needs to see its opener as does line 206 aside tag.
Done -Line 208 body close tag can be brought back to no indent line you have done to its opener.
Thanks heaps!

All done!
line 13 body tag needs to come back one so it can see its close tag.
Lines 21 - 47 all need to come back one indent.
Line 49 div and all below it need to be brought back one indent.
Line 51 needs one "space" adding to it so it is level with the p tab below it.
Lines 102 - 130 need to come back one indent.
Line 109+ 116 +128 p tags need to be on there own line.
Line 141 a tag needs pulling back one indent.
Line 143 div needs to be brought forward one indent.
Line 144- 218 need to be brought back one indent Lines 145-216 need to be pushed forwards 1 indent
Cheers

All above revised revisions done,

line 99 + 100 still needs indenting to the right once.
line 146 needs to come left x1
line 149 needs to go over 3 "spaces"
lines 152 - 182 needs to be indented once to the right.
line 186 needs indenting x 1
lines 188-196 need indenting x1
Finished these.


Infrastructure: integration tests

We need to improve testing infrastructure, and in particular integration tests. For this task we only want to test stories editor independently and as such:

We want to look at the editor as a whole.
We want to run in a real browser environment.
WP dependencies can be mocked out. Specifically it means that media and fonts APIs can be mocked out, unless we find an easy and stable way to run it locally or on Travis.
Load/save story APIs can be simply mocked up by feeding JSON in and out.
Anything related to media uploads is mandatory to have tests for

The test itself could be executed this way:

As far testing is concerned, we have an option is to either construct and load the complete editor, or at least a bigger section. The preference is for the whole editor.
The driving script has to read naturally: "(a) add the element; (b) check element selected; (c) go to the design panel; (d) change font size; (e) check the font size is reflected and the height of the text block is expanded".
Using the true mouse/keyboard events could be beneficial. As such, we can settle on a single predefined screen size.

Note:
Consider Pasting to upload #761 as a first candidate
@spacedmonkey needs to review this before marking as done to make sure anything media related has proper test coverage

Related: #374. The storybook could be navigated on using Puppeteer

Since visual regression tests were mentioned:
In my experience BackstopJS is too heavy for that, even when tweaking thresholds etc. jest-image-snapshot could perhaps be a lighter alternative as it works on React components and not the whole browser. Bonus: via StoryShots it has a seamless integration with storybook

AMP uses http://percy.io/ for visual testing.

See #1317 for how we can run E2E tests on Firefox.
See #1318 for an example visual regression test for individual components.

Looking at #1316, which is related to media, could we test useCanvasSelectionCopyPaste by constructing our own clipboard data event?

Some initial notes from our call:

Need to mock API and media related parts to make editor work standalone (#374)
There's a current Puppeteer setup (which supports Chromium - and now also Firefox! #1317)
Alternative solutions are Selenium WebDriver and Cypress
Cypress does not support native events
Puppeteer does not support other browsers at the moment
Extremely good browser coverage is not too important for now. Even working tests in one browsers would be great
Switching from one tool to another would be difficult
We want to focus on testing the most crucial areas first. This includes things like:

Drop Targets
Text selection
Integration between design panel and canvas (e.g. change font size -> change reflected on canvas)
Copy / Paste
Media upload

Ideally E2E tests are fairly easy to write and read

Current Puppeteer setup uses Jest to write assertions
https://codecept.io/basics/ is an alternative way with BDD-style syntax (Gherkin)
https://github.com/testing-library/pptr-testing-library would allow using same test syntax as in unit tests

New PRs should include proper tests

New code coverage report helps encouraging this
Decide on a case-by-case basis whether this means more unit / integration tests / E2E tests
We could add labels to PRs to clearly mark them as "needs tests"
Reduce back-and-forth between development & QA by having proper tests documenting the intention and validating the feature/fix.
need to properly document the whole test process
writing these kinds tests should become more natural / a habit

visual regression tests
visual regression tests for single components are doable (see #1318 as PoC)
need to be done on the same platform, otherwise there can quickly be mismatches
on a case-by-case basis. don't always make sense
one benefit of Percy is less overhead for us, as the platform deals with most of the things
@merapi offered to tackle some critical test areas (as mentioned above) with Puppeteer

if we can test those with Puppeteer, it's an indicator that this tool suffices for us
if it doesn't work, we might need to switch tools

@cvolzke4 offered to kickstart a design doc for this

Please leave a comment if I forgot anything.

@swissspidy Thanks, this looks good! Also, someone (you?) mentioned that we could use the existing storybooks for a low-hanging fruit e2e tests?

I've created a draft design go/greendoc so that we can comment on the different approaches in a shared doc format:
go/storytime-e2e-ideas

I created a related ticket on areas of the media component that need some extra testing.  #1346

Closing this for now as the foundation for this has been built.


Add regex decoder

Hi @michaljanocko ! I really like schemawax 🚀
But I was missing a regex decoder. It can be created in userland but I figured it's a pretty common use case that could be offered by this library.
✌

Hi, thanks for your contribution! I love it. It doesn't add any dependencies—nice. I'm gonna have a better look at it when I come back home on Monday and possibly merge it and release it.

Thank you very much! It's been released in v1.0.8. 🎉

Ooof, yeah, you're completely right. I'm gonna fix that soon! That would also help to reduce the package size.


[SVLS] Mark serverless unit test TestLoadConfigShouldBeFast as flaky

What does this PR do?

Mark Serverless unit test TestLoadConfigShouldBeFast as flaky.
Motivation

Prevent internal pages on flaky failures.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Compilation failed in Android Java environment

May I ask if it supports Java project usage?

I never tried by myself. Probably prebuilt binaries from maven should work if interoperability with Kotlin is configured


Create release process

We need to determine the process to follow when creating a release. It should include at least:

Building and testing on all supported platforms.
Updating the OpenGEE website with the documentation for the latest release (we plan to make the documentation for several releases available, but the website should go to the docs for the latest release by default).

Also publish 5.2.0 documentation on opengee.org

The website for this is available at https://tst-eclamar.github.io/earthenterprise/

The release process is documented here: https://github.com/google/earthenterprise/wiki/Development:-Release-Process.


fix contract ID for blank transitions

This PR fixes the contract ID used for blank transitions.
Fixes https://github.com/RGB-WG/rgb/issues/116

Thank you, this is really helpful. I usually spent weeks on hunting that kind of errors caused by a sort of "typo"


Remove package.json files from non TS projects

The are projects outside of yarn-project that contain package.json files. These should be removed and any scripts that are relied upon should be converted to bash scripts.

fixed in #4867


JTAG programming on the simulator results in corrupted memory

When running hellow through the simulator, the output when using gdb/OpenOCD/JTAG is:
HellHellHellHellHellHellHellHellHellCPU requested halt. Exit code 123

When dumping the memory for the string from the simulated and via gdb shows that this is indeed the string value. OpenOCD shows some errors and warnings:
Error: [hazard3.cpu] Error when writing memory, abstractcs=0x2000301
Warn : [hazard3.cpu] Failed to write memory via program buffer.
Error: No working memory available. Specify -work-area-phys to target.
Warn : not enough working area available(requested 1100)

I'm assuming there''s some kind of memory allocation problem for gdb to push data into the target (so the first four bytes of the string are repeated across the memory). When asking gdb to verify the sections:
(gdb) compare-sections
Section .text, range 0x0 -- 0x17e2: MIS-MATCHED!
Section .rodata, range 0x17e4 -- 0x1808: MIS-MATCHED!
Section .eh_frame, range 0x1808 -- 0x180c: matched.
Section .init_array, range 0x180c -- 0x1814: MIS-MATCHED!
Section .fini_array, range 0x1814 -- 0x1818: matched.
Section .data, range 0x1818 -- 0x1d50: MIS-MATCHED!
Section .sdata, range 0x1d50 -- 0x1d60: MIS-MATCHED!
warning: One or more sections of the target image does not match the loaded file

The code does actually run correctly (the simulator terminates with a value of 123, as expected). I guess this could be down to 'luck' though.
The versions of the software are:
Open On-Chip Debugger 0.12.0+dev-03959-gc85d4e185 (2024-09-12-14:42)

GNU gdb (GDB) 15.1

Thanks, I'm travelling without a PC at the moment but will look at this when I get back. So that I can reproduce could you please let me know:

The Hazard3 version (tag or hash)
The OpenOCD config file you are using
The exact sequence of commands you run (bash + gdb)

The hash is
a4412c0b00fd67788e42bc61599bf7a8c106d7b1
(the current repository HEAD).
The OpenOCD config file is unchanged.
The gdb commands are exactly as shown in Readme.md (they were copied and pasted).
My colleague discovered this first and so I followed the exact same build sequence, and replicated their result.

I just tried to run in the simulator, get not same as above but very similar.
root@debian12:/home/hazard3/test/sim/tb_cxxrtl# riscv32-unknown-elf-gdb
GNU gdb (GDB) 15.1
Copyright (C) 2024 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "--host=x86_64-pc-linux-gnu --target=riscv32-unknown-elf".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
https://www.gnu.org/software/gdb/bugs/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word".
(gdb) target extended-remote localhost:3333
Remote debugging using localhost:3333
Info : accepting 'gdb' connection on tcp/3333
hazard3.cpu halted due to debug-request.
warning: No executable has been specified and target does not support
determining executable automatically.  Try using the "file" command.
0x00000000 in ?? ()
(gdb) file ../hellow/tmp/hellow.elf
A program is being debugged already.
Are you sure you want to change the file? (y or n) y
Reading symbols from ../hellow/tmp/hellow.elf...
(No debugging symbols found in ../hellow/tmp/hellow.elf)
(gdb) load
Loading section .text, size 0x17e0 lma 0x0
Loading section .rodata, size 0x24 lma 0x17e0
Error: [hazard3.cpu] Error when writing memory, abstractcs=0x2000301
Error: [hazard3.cpu] Failed to write memory (addr=0x0)
progbuf=failed, sysbus=disabled, abstract=disabled
Load failed

Thanks, I was able to replicate this. Looks like there was a regression with abstract commands in 78a5cb9, so between the RP2350 tapeout and 1.0.
This slipped through the debug tests being were run using SBA, so whatever combination of abstract command/auto features gets used for memory downloads was not being covered.
While I investigate, you can add this line to the end of the openocd config to force SBA use for memory access, assuming your Hazard3 deployment supports SBA:
riscv set_mem_access sysbus

Could you please try this patch on the current 1.0 and confirm whether it fixes the issue for you: cb700f30

Confirmed on my end: running the debug regression tests with SBA disabled passes with that patch, and failed without it. I'll release that patch as a point release on stable since it only affects the DM.

Yes it works now ;)


Use arrow_options with computing meta

On a call with @ncclementi today I noticed we're not using arrow_options when computing meta but we are when computing partitions. This could lead to meta not being correct. This PR makes sure we use arrow_options accordingly.

LGTM, @jrbourbeau can you fix the linting issue?

Thanks @ncclementi. That unrelated linting failure should be fixed by https://github.com/coiled/dask-snowflake/pull/32
Also, let me see if I can add a quick test for the changes here...

I wasn't able to get my local snowflake setup working...let's see if this test passes in CI 🤞

Okay, this is good to go -- thanks @ncclementi

It's nice to see the rapid merging of small but important changes like this :)


io.camunda.common.http.DefaultHttpClient does not reset expired tokens

Describe the bug
The io.camunda.common.http.DefaultHttpClient does not reset expired tokens. The bug is in the parseAndRetry(CloseableHttpResponse response, Class<T> responseType) method where the resetToken method of the Authentication class is invoked. The Product argument passed to this method is always null and the token does not get reset.
The line with the bug is authentication.resetToken(getProduct(responseType.getClass()));
We debugged this and found out that because responseType is already a Class, the getClass() method returns a generic Class, which does not have an entry in the productMap map. So there is no Product (null) provided to the Authentication.resetToken method and the token does not get cleared.
To Reproduce
We observed this issue when invoking the CamundaOperateClient.

The first try works because there is no token and a new token is fetched from the Camunda Platform.
After the token expiry time (5 mins. later), the second invocation will fail because expired token is sent to the Camunda Platform. In this case a 401 is returned and the Authentication.resetToken is invoked and then an SdkException is thrown back to the caller.
The caller catches the SdkException and retries. But because the token is not actually reset, the same expired token is sent again to the Camunda Platform and the 401 error continues.

Expected behavior
The token should be reset correctly so that when we catch the SdkException and retry, a new token must be fetched from the Camunda Platform.
We downloaded the code and made a fix to send just the responseType to the getProduct method. With this fix the token was reset.
Log/Stacktrace
We only see an SdkException with message Response code is 401.
Environment:

OS: Windows and Linux
Deployment: Self Managed 8.4
Spring-Zeebe Version: 8.4.0
SpringBoot Version: 3.2.5
Configuration: N/A

@rkodaman thank you for figuring this out! Would you please provide your fix as PR?


feat: support scanning node_modules generated by NPM in container images

The actual support was easy because this is really just the NPM extractor applied to a package-lock.json with a different name, but this also includes an initial approach at handling management of image fixtures without requiring the actual tars (which can be quite sizable) to be committed to the codebase, thus avoiding bloat.
It's actually turned out pretty well - the main downside is it's not easy to support automatically building if/when the image tests are run specifically but I don't think that's a higher priority vs than just having something to let us avoid committing tars.
~Hopefully it'll work in CI and on Windows + macOS 🤞~

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

Project coverage is 61.11%. Comparing base (b28c1c8) to head (899a038).
Report is 22 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #851      +/-   ##
==========================================
+ Coverage   59.78%   61.11%   +1.33%
==========================================
  Files         136      144       +8
  Lines       11268     9146    -2122
==========================================
- Hits         6737     5590    -1147
+ Misses       4102     3081    -1021
- Partials      429      475      +46

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

I just realised I think this might not work on macOS and Windows for the same reason they didn't in CI - those platforms will want OS-specific versions of the images.
It might "just work" on macOS (it didn't in CI because the Docker license actually means GitHub Actions can't include it by default) in which case it's probably good enough to land for now as I assume no one on the core team is using native Windows to develop (personally I'm using WSL), but this'll most likely require follow up - I'm not actually that familiar with docker beyond the basics but I suspect it might be possible to have it build e.g. "linux only" containers regardless of platform (like how you can build other os binaries for Go), which I'll look into next.
Alternatively maybe we should mark these as acceptance tests for now?

Hmm I'm not clear on what the issue with windows docker containers are. I don't have a windows machine to test this at the moment, but wouldn't it already be building Linux containers by default?

You can switch it to build Linux containers, but I don't know if that's the default - GHA seems to default to Windows containers (though that makes more sense); I think the Windows installer for humans actually asks you to pick - but yes I think you're right that for local development people should be able to use linux containers and get the same result.

Ah I see, I would have guessed that the docker file starting with a Linux base image would change docker to use Linux containers, but maybe that requires installing a completely different runtime, so that's why you choose on install.
Yeah let's mark these as acceptance tests for now.

maybe that requires installing a completely different runtime, so that's why you choose on install.

It might have changed, but when I first used Docker on my Windows machine the Windows and Linux containers were completely different - as in you had to active switch between the two, couldn't have both at the same time, and if it didn't wipe your alternative containers it at least cold storaged them somewhere on disk until you switched back...

Looking at this a bit more, it could be good to add a check to rebuild the docker images when the dockerfile changes. Something like checking the modified date of the dockerfile is after the .tar file might be enough.

Yup I've got that down as a follow up since there's a few things in that area we can explore but they should be less critical than unblocking general crafting of tests for images etc


Strict mode

Hello!
I have been studying your project, to understand more about using Three.js with Vuex. I know you have build it more than a year ago, and maybe don't remember it much. But anyway, I thought of discussing some aspects of it.
I have been poking it around, trying to make it work on strict mode. Moving everything that visibly changes state to a mutation and everything, but I could not make it work, it never finishes loading the page once I activate strict mode...
When reading about strict mode on Vuex, it is said:

Do not enable strict mode when deploying for production! Strict mode runs a synchronous deep watcher on the state tree for detecting inappropriate mutations, and it can be quite expensive when you make large amount of mutations to the state. Make sure to turn it off in production to avoid the performance cost.

Maybe it never stop loading because of the expense of those deep watchers? Is the code you have now already "ok" with strict mode and state mutability only on commits? Or is it something not doable with Vuex and Three.js maybe?
Do you have any thoughts on that?
thanks!

Hello,
I have not tried using this code in strict mode, so I can't speak from experience here.  However, I can say that even without strict mode I had a fair amount of trouble getting Three.js to work well with Vuex.  It was very easy to overwhelm vuex with a large number of state change commits and bring down the whole page.  My guess is that with the watcher in place this balance becomes even more tenuous.
I don't know for sure, but if I had to guess I would say that it is not possible to get Vue and Three.js working together with strict mode enabled.

This may or may not be a moot point in Vue 3, which has been implemented in PR #6.   I am going to close this for now but feel free  to reopen it if you would like.


GitPrePushHook on a new branch with no remote

Looking at the current prepush script
git diff --name-only HEAD origin/$(git rev-parse --abbrev-ref HEAD) | grep '\.kt[s"]\?$' | xargs ktlint --relative .
The script assumes there is a remote present for the current local branch.
But we are seeing unlinted code getting through the pre-push check when a dev is creating a fresh local branch, and trying to push it for the very first time with no remote present. In that case the above doesnt run and all the unlinted code leaks to the repo.
Expected Behavior
Ktlint prepush hook should trigger on a new local branch with no remote
Steps to Reproduce

Create a new local branch, git checkout -b <local branch>
Add some blank lines in the end to trigger the Needless blank line(s) error
Push it to a remote git push -u origin <local branch>
No Ktlint is triggered in this case.

Your Environment

Version of ktlint used: org.jlleitschuh.gradle:ktlint-gradle:9.1.1

@ahetawal-p I would suggest you to open same issue in ktlint-gradle repo. This Git hook is generated by the Gradle plugin and not by ktlint itself.

@Tapchicoma it seems that the script is comming from this repo :
https://github.com/pinterest/ktlint/blob/master/ktlint/src/main/resources/ktlint-git-pre-push-hook.sh
The bug is trivial to reproduce.


Istio Ansible playbooks aren't packaged within distro

Istio Ansible playbooks aren't packaged within Istio distro (0.6.0, 0.5.0, ...) under instio/ansible folder
wget -O istio-0.6.0-osx.tar.gz https://github.com/istio/istio/releases/download/0.6.0/istio-0.6.0-osx.tar.gz
--2018-03-19 12:24:02--  https://github.com/istio/istio/releases/download/0.6.0/istio-0.6.0-osx.tar.gz
Resolving github.com... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to github.com|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://github-production-release-asset-2e65be.s3.amazonaws.com/74175805/e20b7b6e-1ca4-11e8-8e25-5266d9f81962?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20180319%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20180319T112403Z&X-Amz-Expires=300&X-Amz-Signature=b0329959a4d4c2dce5b32c5bf3250bd2051bb13323e0c5eeca6179f3d4ac93a8&X-Amz-SignedHeaders=host&actor_id=0&response-content-disposition=attachment%3B%20filename%3Distio-0.6.0-osx.tar.gz&response-content-type=application%2Foctet-stream [following]
--2018-03-19 12:24:03--  https://github-production-release-asset-2e65be.s3.amazonaws.com/74175805/e20b7b6e-1ca4-11e8-8e25-5266d9f81962?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20180319%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20180319T112403Z&X-Amz-Expires=300&X-Amz-Signature=b0329959a4d4c2dce5b32c5bf3250bd2051bb13323e0c5eeca6179f3d4ac93a8&X-Amz-SignedHeaders=host&actor_id=0&response-content-disposition=attachment%3B%20filename%3Distio-0.6.0-osx.tar.gz&response-content-type=application%2Foctet-stream
Resolving github-production-release-asset-2e65be.s3.amazonaws.com... <IP_ADDRESS>
Connecting to github-production-release-asset-2e65be.s3.amazonaws.com|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 11504732 (11M) [application/octet-stream]
Saving to: ‘istio-0.6.0-osx.tar.gz’

istio-0.6.0-osx.tar.gz                                    100%[====================================================================================================================================>]  10.97M  2.90MB/s    in 4.6s

2018-03-19 12:24:08 (2.40 MB/s) - ‘istio-0.6.0-osx.tar.gz’ saved [11504732/11504732]

tar -vxf istio-0.6.0-osx.tar.gz
tar: Failed to set default locale
x istio-0.6.0/
x istio-0.6.0/LICENSE
x istio-0.6.0/bin/
x istio-0.6.0/bin/istioctl
x istio-0.6.0/tools/
x istio-0.6.0/tools/vagrant/
x istio-0.6.0/tools/vagrant/Vagrantfile
x istio-0.6.0/tools/vagrant/provision-vagrant.sh
...
x istio-0.6.0/install/ansible/
x istio-0.6.0/install/ansible/README.md
x istio-0.6.0/istio.VERSION

ll istio-0.6.0/install/ansible
total 16
-rw-r--r--  1 dabou  staff   5.8K Feb 28 20:51 README.md

@vbatts @christian-posta Can one of you help us to resolve this issue please as this is blocking ?

I had already opened https://github.com/istio/issues/issues/223 about a month ago since it seemed like the more appropriate location for the issue, but I guess it wasn't :)

Not yet packaged for release 0.5.0 unfortunately

@costinm is there any reason this is not going to be released with 1.0?

@sdake can we mark this issue resolved? This was fixed with the merge of #7194


Add Support for Oasis v23

This PR updates dependencies and types for Oasis v23, released earlier this week.

Hey @VitalyVolozhinov - so sorry for not getting back to you. I missed your comment with the holidays.
Should be good to go now.

Thank you @keefertaylor


feat: miscellaneous smaller changes and fixes

This PR is a collection of several smaller things I've been working on. I can split this into multiple PRs if needed.
Each commit is for an independent feature and the commit messages contain some more details, here's a quick overview:

add new animationMode option to allow for lazy rendering

add new addDefaultLighting option to set up basic lighting for the scene

use overlay.setAnchor() instead of public property overlay.anchor to be more in line with the rest of the API and as a first step towards implementing floating anchors

extended the readme with more information about the coordinate conversions

update all dependencies to their latest major version, including necessary changes and some DX improvements like sourcemaps and no more warnings when building

@amuramoto the latest changes should now ensure that the tests pass. With that, I think we can merge this into the beta-branch and I can prepare the new release from there

@usefulthink Can you take a look at the error being thrown when npm i is run?


[Yahoo Weather Platform] name parametere with Unicode value crashes weather platform

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.49.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
Yahoo Weather
Description of problem:
Yahoo Weather platform not working when i specify "name" parameter in Georgian Language. Seems like some problem with UTF. For example, if i write name: Weather it works, but when i try name: ამინდი, there is an error in log and there is no weather in dashboard.
Expected:
Working weather platform.
Problem-relevant configuration.yaml entries and steps to reproduce:

weather:
  - platform: yweather
    name: ამინდი

enable Yahoo weather
enable name parameter and write value in Georgian

Traceback (if applicable):
2017-07-16 20:32:31 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/lib/python3.4/asyncio/tasks.py", line 237, in _step
    result = next(coro)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 381, in async_process_entity
    new_entity, self, update_before_add=update_before_add
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 231, in async_add_entity
    'Invalid entity id: {}'.format(entity.entity_id))
homeassistant.exceptions.HomeAssistantError: Invalid entity id: weather.

Additional info:
I think somewhere python code is not accepting UTF-8 value, but i dont know where to find this code.
Thank you!

I'm not 100% if there is a reason behind not using unicode for entity names, but you can resolve this by using customization to add a Georgian name to your Yahoo sensor.
Example config:
homeassistant:
  <Your config here>
  customize:
    weather.yahoo:
      friendly_name: ამინდი

...

weather:
  - platform: yweather
    name: yahoo

will produce the desired outcome. Hope that helps :)

The reporter of the issue is not participating in the discussion. Going to close it.


Improve cross-instance login and logout flow

Currently, whenever a user logs in, a new "ephemeral" Mastodon app is created. The app, in fact, does not appear in their Mastodon authorized applications, and is effectively nuked and recreated whenever a new login attempt is done.
Why recreating the Mastodon app on login?
Currently, if I login with a mastodon.social account, logout, and then try to login again with - say - a mastodon.cloud account, the login will fail because there will be a mismatch with the original app.
To prevent this from happening, I currently delete the client before creating a new one:
if "mastodon" in oauth._clients:
        del oauth._clients["mastodon"]

Possible approaches
Investigate whether multiple clients can coexist, perhaps giving them custom names. This might make it necessary to actually store user data (on a database?). I am not a big fan and it seems like it could be a rabbit hole. Any input is welcome.

Fixed in https://github.com/AlexKalopsia/mastodon-gender-distribution/commit/316cd2f34f6e19f31527a0d975499caab0b599a9


Process crashes with out-of-memory

A build using this plugin reported the following error during the onPostBuild of this plugin:
┌──────────────────────────────────────────────────────┐
│ 6. onPostBuild command from netlify-plugin-hashfiles │
└──────────────────────────────────────────────────────┘

 ✔ 0.001 secs: logEvents
 ⚠ WARN: inline JavaScript in public/index.html - Parse error in inline JavaScript in public/index.html
Unexpected token (1:31)
Including assets:
public/index.html
 ✔ 1.760 secs: loadAssets
 ⚠ WARN: ENOENT: no such file or directory, open 'public/sw.js'
<--- Last few GCs --->
[1833:0x2c00030] 709775 ms: Scavenge 1348.7 (1424.1) -> 1348.3 (1424.6) MB, 5.7 / 0.0 ms (average mu = 0.153, current mu = 0.035) allocation failure
[1833:0x2c00030] 711657 ms: Mark-sweep 1349.0 (1424.6) -> 1348.6 (1425.1) MB, 1880.9 / 0.0 ms (average mu = 0.082, current mu = 0.005) allocation failure scavenge might not succeed
<--- JS stacktrace --->
==== JS stack trace =========================================
0: ExitFrame [pc: 0x10e468bdbe1d]
1: StubFrame [pc: 0x10e468bdd3a6]
Security context: 0x16dd6e71e6c1 <JSObject>
2: _createSourceMapForInlineScriptOrStylesheet [0x3c3123ca2cb9] [/opt/build/repo/node_modules/assetgraph/lib/assets/Html.js:~276] [pc=0x10e46a11ed95](this=0x34b18c4a3341 <EventEmitter map = 0x2eae284a2c29>,element=0x09cece5691c9 <Object map = 0x125d72795e41>)
3: findOutgoingRelationsInParseTree [0x3c3123ca2cf1] [...
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
1: 0x8fb090 node::Abort() [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
2: 0x8fb0dc [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
3: 0xb031ce v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
4: 0xb03404 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
5: 0xef7462 [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
6: 0xef7568 v8::internal::Heap::CheckIneffectiveMarkCompact(unsigned long, double) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
7: 0xf03642 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
8: 0xf03f74 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
9: 0xf06be1 v8::internal::Heap::AllocateRawWithRetryOrFail(int, v8::internal::AllocationSpace, v8::internal::AllocationAlignment) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
10: 0xed0064 v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
11: 0x11701ee v8::internal::Runtime_AllocateInNewSpace(int, v8::internal::Object**, v8::internal::Isolate*) [/opt/buildhome/.nvm/versions/node/v10.20.1/bin/node]
12: 0x10e468bdbe1d

I unfortunately don't have more information. Do you know what might be happening?

Yeah, hashfiles is pretty naive in the way it loads in files: https://github.com/Munter/netlify-plugin-hashfiles/blob/master/lib/index.js#L25-L30
This basically loads everything you linked to, except js fetches. It all goes into memory. Sourcemaps, images, videos etc. This runs up the memory bill pretty quickly. In the link checker we do very specific work to unload assets as soon as possible to keep memory consumption low, which we can do because the check is pretty linear and we don't need much information to keep around to check existence of assets. This workload is different though.
In this type of work I need to explore the entire internal dependency graph, find leaf nodes, hash them, then traverse up the dependency graph to modify each asset referencing the hashed file, modify the href pointing to the hashed file, then repeat that work all the way up the tree. The naive approach is to just keep everything in memory. There are obvious limitations to this.
@papandreou do you think it's feasible to create a more custom graph traversal that will explore the graph in a different way to keep memory consumption low? I'm assuming we need to actually populate everything in order to find all relations, and we'll need to opt out of pretty much every convenience Assetgraph gives us for updating file paths if we unload parent assets as part of a custom traversal

Hmm, yeah, for one thing I guess it isn't really necessary to keep videos, images, fonts etc. in memory, as they can't have outgoing relations. Should be enough to compute their hash (by accessing their md5Hex getter), then unload them
To use less memory, you could even keep them unloaded, compute their hash via streaming, then set asset._md5Hex yourself:
const readStream = require('fs').createReadStream(
  require('urltools').fileUrlToFsPath(asset.url)
);
const hash = require('crypto').createHash('md5');
for await (const chunk of readStream) {
  hash.update(chunk);
}
asset._md5Hex = hash.digest('hex');

I was also considering the concept of assets that can't have any outgoing relations because none are plugged in. It's an interesting information to have for custom population and traversal. Might be worth adding that as a convenience getter to core, so it's queryable.
I've been considering how I could create a custom population for this plugin, and I think I might do that when I can find the time. It would mean discarding the current assetgraph transform, but I guess that's fine

I think the current assetgraph transform would work fine with the trick I suggested above (maybe with a bit of tweaking wrt. unloaded assets), but if you want to optimize further, I think you're right. Custom population isn't that terrible nowadays 🌥


Fix plot scale in AC UG

Set the scale of the "sitting" example in the activity classifier user guide to the same scale as the "walking" example.
Fixes https://github.com/apple/turicreate/issues/52.

Seems great. The difference between the two activities is much more apparent this way.


Request for Comment: Switch some courses from OCW/edX to OLL

Background:
MIT relatively recently started a third platform for their materials called Open Learning Library (OLL). It looks like edX, but it's completely free and always available like ocw. The courses in OSSU that are available in OLL are:

18.01 Single Variable Calculus
18.06 Linear Algebra
6.042J Mathematics for Computer Science

I took a cursory look and here were my findings:
Single Variable Calculus: has the unit 0 on limits from edX, but has lots of references to edX and skips homework a, homework b, and the final exam. I did not look at Calculus 1B and 1C because I am lazy.
Linear Algebra: makes you click more without providing more value
Math for CS: interleaves the materials so you don't have to keep looking at the resource index and bouncing between different parts of the site
Proposal:

Make the ocw link to Math for CS an alt link with the OLL link as the primary link.
Make OLL alt links for Single Variable Calculus. Maybe even make OLL the primary since edX does not follow the run regularly rule of OSSU only running once or twice a year?

dunno if this is worthy of a RFC rather than a Pull Request, but hoping to get some eyeballs and opinions on this

Good find! My spot check is in line with yours, this looks like a better wrapper around high quality material that MIT already offers.
Given that it's largely the same underlying content, and that it looks like MIT is trying increase the ease of use for students, I say let's make this the primary link for Calc and Math for CS.

I'm currently in the middle of MIT's Calculus 1B course on edX.  I posted a bit about my experience on the Discord a week or two ago.  I know you're asking about OLL, but it looks like the video content is the same on OLL as the edX content, as mentioned by Alaharon123, so I'll give my opinions about edX.
Basically, I haven't found the edX course to be particularly helpful, as far as instruction goes.  I've been doing OCW alongside edX and, for me, OCW has been much, much more helpful in learning the material.  The EdX course copies some problems and some videos from OCW, but it cuts some of the most useful content only to replace it with content that I found to be less useful.

EdX cuts many of the recitation videos, which were among the most helpful bits of the OCW course.  The two (student?) instructors take time to walk the viewer through each step of a problem, and they help to fill in any gaps you may have had after watching just the lectures.

EdX cuts much of the lecture content of OCW and replaces it with their own shorter lectures.  I guess personal preference, but I found the OCW lectures more helpful to a student like me who had not studied calculus before. The edX lectures move quicker and explain less.  Additionally, the OCW lectures are videos of actual classes, so they are occasionally punctuated by students asking for clarifications about things that I myself wished I could ask about.

OCW has a massive problem bank with answers to all of the hundreds of problems offered there.  OCW assigns selective problems for you to work on, but if you need more practice, the extra problems are there for you. No such extra practice exists in the edX class.

OCW provides free exams with solutions; edX's exams are behind a paywall

What edX does add to the learning experience is at the loss of what it cuts from OCW.  The videos they add are definitely provided to have fun and heighten student engagement (Professor Jerison gets pulled over and talks math with a cop, two of the instructors pretend they're math operations), but they are presented alongside a lack of clarity regarding the actual mathematics. The original classroom videos presented by OCW are more 'boring' but also clearer and more thorough.
Finally, the interactivity of the edX course, an advantage it has over OCW, often led me to be more discouraged about math, rather than feeling like I understood the topic better.  There are occasionally 4 or 5 part problems for which I could not even do the first part and failed to understand the problem even after staring at the provided answer for an hour. Yes, this speaks to my inability to do some more advanced problems, but I'm taking the course to learn how to think better and learn such things, and I'm just left feeling stupid and frustrated.  For some context, I am able to arrive at the correct answer for a good 90% of the OCW problems I've worked through, so it's not as though I can't do calculus at all.  In edX's course 1A, there were strings of comments about how difficult a particular unit was, and they were being posted by commenters who clearly already knew calculus (and much more) quite well.  If even these commenters were finding trouble with a whole unit's worth of problems, I was left wondering who the course is designed for?  It seems not to have been designed for someone learning the material for the first time.

Anyway, that's been my experience with these courses.  As I posted in the Discord, I'm curious what experiences other learners are having. My own approach I've landed on: I learn the material from the instruction, problem sets, and tests provided by OCW, and then I look to edX for some harder problems that I may or may not be able to solve.
I'd finally like to say that, regarding the recommendation here to switch to OLL, yes it's true that OLL won't have limitations as to when you can and can't sign up, but that will be at the expense of the instructor-moderated comments section in edX, which I have found useful.  In addition to that OLL will have all the problems, as I see them, that edX has.
My own recommendation, as someone who is not a mathematician or a computer scientist (yet!), would be to recommend to learners the OCW course.

An update on my last post:
Here are some posts from the current cohort of edX's Calculus 1B (found in the forums attached to Unit 3, Problem set 3A-9:
I'm quoting them without their screen names here.  Not sure what the right move is.  I figured privacy is better than citing in this case.  I can come back and attribute as necessary.
"Original MITOCW lectures have saved me. Edx materials are well constructed, but they are like patchwork, and I lost coherency. Edx way must be good for learners having knowledge, they can get information efficiently, but for me, it did not work. Then I found original lectures, and wow! Original lectures are theoretical and consistent, telling me where I am. Important thing is, original lectures have contexts! Contexts are very important for me to construct things. Only essence like edx is not enough."
"I'm doing fine at the regular class exercises, but when it comes to the Problem Sets, I've been consistently whacked. I think I read somewhere that people usually study from 5 to 10 hours a week for MITx Calculus programs, but for this last Problem Set I've been doing at least 3 hours a day. I almost gave up the course while doing this Problem Set 3A!"
"Try watching the lectures only in the OpenCourseware series, then memorize the basic concepts so they can be recalled immediately. At the end of the day I listen to these lectures only, they are more concise and accessible in OpenCourseware."
"I agree with watching the OCW videos that are very sequential."
"the gap between the exercises (which are already harder than what you would see in a typical calculus course) and the problem sets is so large as to be demoralizing even for a student with good math skills and a strong work ethic like myself. Maybe if I were at university taking physics and other technical courses in conjunction with this it would be a different story. I don't know."
Now, admittedly, I'm picking and choosing the comments that support my argument above, but I find it interesting that even students who had already taken calculus courses were saying that the problem sets here were beyond them.  There's nothing wrong with problems being difficult, or even too difficult; I just wonder what the purpose of these classes are in the OSSU curriculum.  Does OSSU require taking calculus so that learners learn calculus?  Then I don't think edX is the right class.  Does OSSU require calculus as a bank of mental puzzles to be worked through? Then maybe edX is a good fit.

@dougc85 Your proposal is different from the one highlighted above AND your proposal is well supported by evidence. Can you create a new RFC that is laser focused on switching OSSU from using MIT's edx course to using MIT OCW?
At the end of your most recent post you asked about OSSU's reason for including calculus in the curriculum. This comment in an earlier issue gives a good accounting for how the math section came to its current form:
https://github.com/ossu/computer-science/issues/768

I'd been pushing off making the pull request for this and closing the issue. Thanks Waciuma for the comment to Doug and great job on that RFC Doug.
Conclusion: #839 switches Math for Computer Science to OLL and keeps Linear Algebra with OCW. dougc85 has made a separate RFC on Calculus, #838, to potentially switch that to OCW rather than OLL or edX.
Now closing this RFC, thanks everyone

Very nice!
It lets OCW has better UI and interactive. It is an update of OCW.

Very nice!
It lets OCW has better UI and interactive. It is an update of OCW.

@dougc85
I agree what you said.
In the case of 6.00.1:
EDX version cuts in pieces and saves your time, but it lacks some good recitation videos and exercise explanations.
That means it seems like edx saves your time, but in fact you will spend much more time thinking about how to solve the problems because of lack something. And the experience will be gapped.
But anyway, edx keeps the core knowledges, I don't think it is a bad idea to go through it. But now I prefer to use OCW. It's full and the same as what students in MIT took.
That's why I really like CS50 because what we did is the same as Harvard students did.

@dougc85
I agree what you said.
In the case of 6.00.1:
EDX version cuts in pieces and saves your time, but it lacks some good recitation videos and exercise explanations.
That means it seems like edx saves your time, but in fact you will spend much more time thinking about how to solve the problems because of lack something. And the experience will be gapped.
But anyway, edx keeps the core knowledges, I don't think it is a bad idea to go through it. But now I prefer to use OCW. It's full and the same as what students in MIT took.
That's why I really like CS50 because what we did is the same as Harvard students did.

@timeneverdie
Check out this request for comment I made regarding EdX and OCW calculus.  It's still open for discussion.
https://github.com/ossu/computer-science/issues/838
Thanks for chiming in!

@timeneverdie
Check out this request for comment I made regarding EdX and OCW calculus.  It's still open for discussion.
https://github.com/ossu/computer-science/issues/838
Thanks for chiming in!


[Bug]: It is not clear for non UI experts how to use "dark mode"

Tremor Version
latest
Link to minimal reproduction
N/A
Steps to reproduce
Reading the docs says that i have to use prefers-color-scheme  from tailwind which I have no idea what it is.
What is expected?
something more straightforward or  simply a set of steps to get darkmode activated
What is actually happening?
docs are lacking this part making it not possible without previous frontend knowledge!
What browsers are you seeing the problem on?
No response
Any additional comments?
No response

usually this is in your browser appearance settings.
I am on edge and it looks like this:

I agree the whole prefers-scheme is annoying, and prefer a system like MUI but thats just the tradeoff you make when choosing SSR tailwind

If you are using Nextjs then pacocoursey/next-themes handles the switching to dark mode very nicely https://github.com/pacocoursey/next-themes#with-tailwind

Hey there! I added some notes to our documentation and linkednext-themes for further reading, I hope this helps!


Entrypoint Script Bug

There is a bug in the entrypoint script that prevents the ARTEMIS_USERNAME environment variable from being applied to the artemis-roles.properties file.
Line 9 in docker-entrypoint.sh:
sed -i "s/apollo=amq/$ARTEMIS_USERNAME=amq/g" ../etc/artemis-roles.properties
should be:
sed -i "s/amq=apollo/amq=$ARTEMIS_USERNAME/g" ../etc/artemis-roles.properties
Also, the documentation says to use the environment variables ACTIVEMQ_MIN_MEMORY and ACTIVEMQ_MAX_MEMORY, but the script actually uses ARTEMIS_MIN_MEMORY and ARTEMIS_MAX_MEMORY.

Thanks for the feedback @Borfswitch, I'll fix it as soon as possible.


How do we get coefficients for the interactions of 2 labeling functions?

Is your feature request related to a problem? Please describe.
Not a bug problem. I would like to know: where are the coefficients for the interactions of 2 labeling functions? For example, suppose we have 2 labeling functions L1 and L2. In the exponential term of the graphical model (from original paper here http://www.vldb.org/pvldb/vol11/p269-ratner.pdf), we fit w1 lambda(L1=y) + w2 lambda(L2=y) + w3 lambda(L1=L2), where w3 is the weight for the agreement of L1 and L2. How can I get this w3?
I can only see a way to get w1 and w2 here
Describe the solution you'd like
I have no solution since I'm not an expert at Snorkel.
Describe alternatives you've considered
I have no solution since I'm not an expert at Snorkel.
Additional context
None

Hi datduong,
Those parameters (sometimes these are called the "canonical parameters") aren't computed explicitly because they're not necessary for computing P(Y|Lambda). That is we don't need them to obtain the probabilistic labels. Instead what are used are called the "mean" parameters, mu in the codebase. Both kinds of parameters are generally equivalent. For more on the types of parameters and their role in exponential models this book http://people.eecs.berkeley.edu/~wainwrig/Papers/WaiJor08_FTML.pdf may be useful.
Hope this helps!

Thanks.


[BUG] - Network.Socket.bind: invalid argument (Invalid argument)

External
Other
Summary
when running cardano-node w/ --socket-path "/storage/testnet/sockets/nodes.socket" fails with Network.Socket.bind: invalid argument (Invalid argument)
changing it to  --socket-path "/storage/testnet/nodes.socket" resolves the issue.
NOTE: in both cases the directories were created
mkdir -p /storage/testnet/sockets
Steps to reproduce
Expected behavior
Node doesn't crash with the error listed
System info (please complete the following information):

Linux 9499afb59aec 4.19.121-linuxkit #1 SMP Tue Dec 1 17:50:32 UTC 2020 x86_64 GNU/Linux
Debian GNU/Linux 10 (buster)
Node version: 1.25.1

Additional context
root@9499afb59aec:/# cardano-node  run --config "/config/testnet-config.json"     --topology "/config/testnet-topology.json"     --database-path "/storage/testnet/db"     --host-addr "<IP_ADDRESS>"     --port 3001     --socket-path "/storage/testnet/sockets/nodes0.socket"
Listening on http://<IP_ADDRESS>:12798
[9499afb5:cardano.node.networkMagic:Notice:5] [2021-02-14 21:45:52.28 UTC] NetworkMagic<PHONE_NUMBER>
[9499afb5:cardano.node.basicInfo.protocol:Notice:5] [2021-02-14 21:45:52.28 UTC] Byron; Shelley
[9499afb5:cardano.node.basicInfo.version:Notice:5] [2021-02-14 21:45:52.28 UTC] 1.25.1
[9499afb5:cardano.node.basicInfo.commit:Notice:5] [2021-02-14 21:45:52.28 UTC] 9a7331cce5e8bc0ea9c6bfa1c28773f4c5a7000f
[9499afb5:cardano.node.basicInfo.nodeStartTime:Notice:5] [2021-02-14 21:45:52.28 UTC] 2021-02-14 21:45:52.2863632 UTC
[9499afb5:cardano.node.basicInfo.systemStartTime:Notice:5] [2021-02-14 21:45:52.28 UTC] 2019-07-24 20:20:16 UTC
[9499afb5:cardano.node.basicInfo.slotLengthByron:Notice:5] [2021-02-14 21:45:52.28 UTC] 20s
[9499afb5:cardano.node.basicInfo.epochLengthByron:Notice:5] [2021-02-14 21:45:52.28 UTC] 21600
[9499afb5:cardano.node.basicInfo.slotLengthShelley:Notice:5] [2021-02-14 21:45:52.28 UTC] 1s
[9499afb5:cardano.node.basicInfo.epochLengthShelley:Notice:5] [2021-02-14 21:45:52.28 UTC] 432000
[9499afb5:cardano.node.basicInfo.slotsPerKESPeriodShelley:Notice:5] [2021-02-14 21:45:52.28 UTC] 129600
[9499afb5:cardano.node.basicInfo.slotLengthAllegra:Notice:5] [2021-02-14 21:45:52.28 UTC] 1s
[9499afb5:cardano.node.basicInfo.epochLengthAllegra:Notice:5] [2021-02-14 21:45:52.28 UTC] 432000
[9499afb5:cardano.node.basicInfo.slotsPerKESPeriodAllegra:Notice:5] [2021-02-14 21:45:52.28 UTC] 129600
[9499afb5:cardano.node.basicInfo.slotLengthMary:Notice:5] [2021-02-14 21:45:52.28 UTC] 1s
[9499afb5:cardano.node.basicInfo.epochLengthMary:Notice:5] [2021-02-14 21:45:52.28 UTC] 432000
[9499afb5:cardano.node.basicInfo.slotsPerKESPeriodMary:Notice:5] [2021-02-14 21:45:52.28 UTC] 129600
[9499afb5:cardano.node.addresses:Notice:5] [2021-02-14 21:45:52.27 UTC] [SocketInfo <IP_ADDRESS>:3001]
[9499afb5:cardano.node.diffusion-mode:Notice:5] [2021-02-14 21:45:52.27 UTC] InitiatorAndResponderDiffusionMode
[9499afb5:cardano.node.dns-producers:Notice:5] [2021-02-14 21:45:52.27 UTC] [DnsSubscriptionTarget {dstDomain = "relays-new.cardano-testnet.iohkdev.io", dstPort = 3001, dstValency = 2}]
[9499afb5:cardano.node.ip-producers:Notice:5] [2021-02-14 21:45:52.27 UTC] IPSubscriptionTarget {ispIps = [], ispValency = 0}
[9499afb5:cardano.node.ChainDB:Info:5] [2021-02-14 21:45:52.31 UTC] Opened imm db with immutable tip at genesis (origin) and chunk 0
[9499afb5:cardano.node.ChainDB:Info:5] [2021-02-14 21:45:52.32 UTC] Opened vol db
[9499afb5:cardano.node.ChainDB:Info:5] [2021-02-14 21:45:52.32 UTC] Replaying ledger from genesis
[9499afb5:cardano.node.ChainDB:Info:5] [2021-02-14 21:45:52.33 UTC] Opened lgr db
[9499afb5:cardano.node.ChainDB:Info:5] [2021-02-14 21:45:52.34 UTC] Opened db with immutable tip at genesis (origin) and tip 1e64e74bd7ac76d6806480a28017deb0aedd356fb61844ec95c429ae2f30c7c3 at slot 0
[9499afb5:cardano.node.ChainDB:Info:5] [2021-02-14 21:45:52.35 UTC] Closed db with immutable tip at genesis (origin) and tip 1e64e74bd7ac76d6806480a28017deb0aedd356fb61844ec95c429ae2f30c7c3 at slot 0
Shutting down..
cardano-node: Network.Socket.bind: invalid argument (Invalid argument)

fixed here in my repo
https://github.com/floydcraft/cardano-node-k8s/commit/e878ed258313d9930d02ca7dc2d54a76ec142df1

NOTE: it's kind of annoying that invalid argument (Invalid argument) exception doesn't print the argument. Will look at the code tonight to see if it can be a simple cleanup

DiffusionErrored CreateNamedPipe ('C:\cardano-node\testnet\db\node.socket'): invalid argument (The filename, directory name, or volume label syntax is incorrect.)
←[34m[tobbT:cardano.node.ChainDB:Info:3]←[0m [2022-08-09 04:09:41.37 UTC] Closed Immutable DB.
←[34m[tobbT:cardano.node.ChainDB:Info:3]←[0m [2022-08-09 04:09:41.37 UTC] Closed db with immutable tip at genesis (origin) and tip 1e64e74bd7ac76d6806480a28017deb0aedd356fb61844ec95c429ae2f30c7c3 at slot 0

for running cardano-node in windows you can use this syntax  --socket-path \.\pipe\cardano-node    it will solve your problem

Closing this. If this is still relevant please reopen.

Anyone else running into Network.Socket.bind: invalid argument (Invalid argument) after following the developer docs, may need to check they use --host-addr <IP_ADDRESS> (not <IP_ADDRESS>), see this: https://stackoverflow.com/a/73792522 . That solved this problem for me.

The loopback interface (which is using <IP_ADDRESS>/8 addresses) only allows to connect to itself and not any other IPv4 addresses, hence the error.


Request for new Release on pypi.org

Could you please publish a new release to pypi.org?
There were some major improvements since last release 0.3.0 in May 2017.
Especially proxy-support and improvements to connection-handling are very needed.

+1

+1

Yeah, it's time to release an update. 0.4.0 is live on PyPi.

Thank you! :-)


Custom Settings Screen

I have changed quite a few parts of the bitmovin styles with the CSS but we are wondering if it is possible to add new menu buttons to the overlay provided by the options icon (cog wheel), and if it's possible to add a callback function to these buttons. We're trying to add multiple buttons for selecting different bitrates such as "720p" and "1080p".

This can be done by using a customized version of https://github.com/bitmovin/bitmovin-player-ui and load it as described in https://github.com/bitmovin/bitmovin-player-react-native#customize-html-ui-android-and-ios-only


add nodePort support

to deploy kafka-ui with nodePort add those commands when you install/update the chart
--set service.type=NodePort --set service.nodePort=31000

@RustamGimadiev a little improvement

who can approve this ?


Added command: spo list webhook remove

Added the command 'spo list webhook remove' which removes the specified webhook from the list.  Thank you very much! 👏

Merged manually. Thanks!


Fix: 1637 Always display a Tutorial in Desktop mode

Currently, if the preview is set to Mobile view and you click to view the tutorial, it remains in the mobile view. This fix always show the Tutorial file in desktop view. This means that if we're previewing the index.html page in the mobile view, we return the preview to mobile after showing the tutorial.
Fixes :  1637

@flukeout can you please try this out?  There's one small code change I want, but I think it works well.  See if you're happy with it.

Testing now

There's a brief flash where the content of the page is shown in desktop mode before the view switches to the Tutorial. Not sure if this is preventable in any way, seems like there's just a timing issue with the two events we fire (change view type, reload doc).

Good to go as long as the code check out to you @humphd

Testing with new changes.

Hey @Rajat-dhyani - with the latest change, the project no longer shows the Tutorial when it's first opened...

@flukeout  sir worked on this PR again and that brief flash is unavoidable due to reload function without which i am having a hard time previewing the tutorial in desktop mode. can you provide any other method for the same?

Re-testing now.

Hey great, tested this and the Tutorial opens in Desktop mode even if the preview is set to Mobile mode. @Pomax could you re-review the code again? I dismissed your original review because the tutorial was showing again.

@flukeout thanks sir for reviewing this. 👍 😃
@Pomax sir please review this PR and check if it is good to merge. Thank You 😅

@flukeout can you do a follow-up review here?

Thanks for your work on this @Rajat-dhyani - works as expected!


Add my info for documentation

Description: open source mentorship program exercise - Add my info for documentation

@sujono91 Nice work! In future, commit messages have to be written in third person perspective so that anyone reading it will understand. For other people, reading "Add my info for documentation" is not clear as they don't know who "my" is referring to without checking the author.

Nice info! Thanks @yangshun 😃


Default to using software gamma control when DDC information missing

Before opening the issue, have you...?

[X] Searched for existing issues

Is your feature request related to a problem? Please describe
I use a screen without proper EDID/DDC information/control, the second display here:

Sometimes MonitorControl seems to forget which monitor this is, probably because it lacks proper identifiying information.
Describe the solution you'd like
It'd be good if MonitorControl were to just default to software control for monitors with invalid EDID/DDC information, or if MonitorControl were able to store the fact this monitor requires software control consistently.
Describe alternatives you've considered
I have no tried alteratives to MonitorControl, I haven't done any detailed testing either. If there are logs I can grab or futher steps to run through, please let me know!
Anything else?
Thank you for all the work bringing third-party monitors up to native-feeling support for macOS!

The app falls back to gamma control if it thinks DDC is not available. For more nuanced display identification methods you might want to try BetterDisplay where you can configure various identification methods for the display (under Settings/Displays/display/Advanced). Choose one that works reliably with the display.


Web - Search - The DM of user does appear on the search result

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.52-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
**Email or phone of affected tester (no<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Open the website https://staging.new.expensify.com/
Create new group with user example<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Send a message to the user in group
Goto other conversation
Start searching
Try entering the user email<EMAIL_ADDRESS>or the name 'alice'
Goto the group
Try entering 'bob alice' to searching

Expected Result:

When the user searches for keywork is<EMAIL_ADDRESS>or the name 'alice' the result should appear the message with alice
When the user searches for keywork is 'bob alice' the result is none

Actual Result:

When the user searches for keywork is<EMAIL_ADDRESS>or the name 'alice' the result just appear the last message with bob
When the user searches for keywork is 'bob alice' the result should appear the group

Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/0317864c-946a-47bb-89b2-419eebf3e64e
View all open jobs on GitHub

@abekkala  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

this type of search would be expected

'bob alice' the result is none

But comma separating provides results - when search for 2 people the comma is needed.


C++20 fmt added to std. Replace fmt with std

Describe your changes
Issue ticket number and link
Checklist before requesting a review
For each PR

[ ] Add Copyright if it missed:
- "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

[ ] I have performed a self-review of my code.
Optional:

[ ] I have checked readme and add important infos to this PR.
[ ] I have added/adapted some tests too.

For core/new feature PR

[ ] I have checked readme and add important infos to this PR.
[ ] I have added thorough tests.

AFAIK it's not available on iOS.


Fix #720 undefined variable in foreach when passed by reference

Type: bugfix
Issue: Fix #720
Breaking change: no
The root cause of this bug is more likely in PDepend, but the following proposition of @char101 fix it properly (as unary operator can only be & in foreach) until PDepend would provide better tooling for read/write variables.

I am fine in fixing or better said working around this in PHPMD.
Do we have a related issue in PDpend, yet?
If not, we should have, so we can keep track of it.
There, we should note that we can remove this workaround here, once it is fixed in PDepend.

I planned to provide tools to getAllReadVariables and getAllWrittenVariable of a node (like an ASTMethod) in https://github.com/pdepend/pdepend/issues/444


Add TS declaration file

I wanted to use your code somewhere and the linter threw errors. Since I already wrote the definition file I though I might as well open a PR.

@msawastian ty 🙂


Aurora Download Not Working

What are you experiencing an issue with?

[x] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[ ] Aurora Profile
[ ] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[ ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[ ] Performance problems
[ ] Visual bug
[ ] Localization issue
[ ] Enhancement suggestion
[ ] Device issue
[x] Other, (Fill in another type of issue here): aurora-web-setup.exe fails saying "Download failed: The server name or address could not be resolved."  My internet connection is stable and working.

Expected behavior:
It should download properly.
Actual behavior:
It won't download.
How often does this problem occur?

Every time I try to download aurora.
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

go to project-aurora.com
click download.
try to run the downloaded file.

What is the version of Aurora you are experiencing this issue with?

My version is whatever is live on the website.
Does the previous version of Aurora have this issue?
First time ever trying to download.
Please include any logs, screenshots, or files that are related to this issue:

For logs, you can use: http://pastebin.com/
For screenshots, you can use: http://imgur.com/
For files, you can use: http://www.tinyupload.com/ , https://www.dropbox.com/ , or attach them to the issue.
https://imgur.com/a/6ffXD

+1 to this request. I am also experiencing this issue.

It is a temporary issue as the server that hosts the download files is migrating currently.

For now, you can download Aurora manually here on github, or here on the website. I'm going to update the web installer so that it can use the correct domain, but it may go out of action again in the near future temporarily.


Adding more details in agent hello and structured value

Describe the problem you'd like to have solved
We have an Agent function in version.go that returns some details about the peer (for example pactus/0.17.0). We need a structured value and additional details added:
func Agent() string {
	return fmt.Sprintf("pactus/%s", Version())
}

Describe the ideal solution
We want agent details similar:
func Agent() string {
	return fmt.Sprintf("node=pactus/version=%s/type={GUI or DAEMON}/os=%s/architecture=%s", Version(), runtime.GOOS, runtime.GOARCH)
}

In fmt.Sprintf() we have many key=value and split-ed by /

Hey @Ja7ad I would like to work on this issue. Can you Please assign me this issue?

@Ja7ad Create a simple PIP from this issue.

@b00f @Ja7ad
https://github.com/ipfs/kubo/blob/master/version.go
This is another good example for handling agents too. they also included Golang version, and added a comment that explains a bit about agent usage. we can consider these two things.

pip updated: https://pips.pactus.org/PIPs/pip-14


Feature/replace flask with sanic 3

Proposed changes:

Updated the web server from Flask to Sanic
Moved custom_channel code within channels
Updated the code to handle synchronous function call in async function
Added logging statements

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

Do we want to remove debug for sanic server as it increases the processing time?


Make library header-only.

Library should work on a header-only basis with no dependencies other than the standard libary.  Unit test application will have dependencies and depend on boost, abseil[cxx17], date, (double check for additional).

Resolved commit# 93ecc95.


[WIP] Pub/Sub graph generation

This script looks through each file within the scope passed to it to generate a list of the publishers and subscribers for each topic. The current output prints a JSON of each key and within that a list of the publishers and subscribers. Additionally there is the ability to use graphviz to generate a plot, but that is currently commented out.
Usage: ./px_generate_pubsub_graph.py <cmake-file-scoping>
See #6189

Nice! What is the value for <cmake-file-scoping>?

I used the files in the cmake/configs folder

Great idea! I've not tested it yet, but looking at the script I think not everything is captured. For example in the mc pos controller: https://github.com/PX4/Firmware/blob/master/src/modules/mc_pos_control/mc_pos_control_main.cpp#L2328
In this case, it actually depends on whether or not the vehicle is VTOL which topic is used. Therefore I'm also not sure what would be the appropriate solution.

@stephanbro what's the current status?

Continued in https://github.com/PX4/Firmware/pull/8393


How do I play a specified interval of audio?

For example, The audio is 10 seconds long, but I want to play 2nd to 5th seconds of audio. How do I resolve it ?

Hi @Blues9527 I don't there is a way to do it directly with this plugin. Rather, you can have a callback when listening to onDurationChanged to achieve the endgoal. This returns the duration/or how far you've played the song. Also play with the seek position to skip to a particular position. That should help you jump to the 2nd second. For example
_audioPlayer.seek(Duration(second: 2));

and you can stop the audio player at the fifth
_audioPlayer.onDurationChanged.listen((duration) {
     if (duration.inseconds == 5) {
         // Stop the player here....
   }
}

Hope this helps.

Hi @Blues9527 I don't there is a way to do it directly with this plugin. Rather, you can have a callback when listening to onDurationChanged to achieve the endgoal. This returns the duration/or how far you've played the song. Also play with the seek position to skip to a particular position. That should help you jump to the 2nd second. For example
_audioPlayer.seek(Duration(second: 2));

and you can stop the audio player at the fifth
_audioPlayer.onDurationChanged.listen((duration) {
     if (duration.inseconds == 5) {
         // Stop the player here....
   }
}

Hope this helps.

thanks bro!