Simply having a +page.server.ts page crashes static builds

Describe the bug
500
Internal Error
Reproduction

Create a src/routes/+page.server.ts file with this content

import type { PageServerLoad } from './$types';

export const load = (async () => {
	return;
}) satisfies PageServerLoad;

Make a build with adapter-static
Open it with http-server

Repro: https://github.com/NatoBoram/bug-report-sveltekit-ssr-csr/tree/main
Logs
GET
http://<IP_ADDRESS>:8081/
[HTTP/1.1 404 Not Found 1ms]

TypeError: can't access property "then", a.default.detectStore(...) is undefined
h1-check.js:1:1301
XHRGET
http://<IP_ADDRESS>:8081/__data.json?x-sveltekit-invalidated=01
[HTTP/1.1 404 Not Found 1ms]

SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data app.306aa39f.js:1:5729

System Info
System:
    OS: Linux 6.2 Pop!_OS 22.04 LTS
    CPU: (12) x64 AMD Ryzen 5 5600X 6-Core Processor
    Memory: 12.92 GB / 31.26 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 20.4.0 - /usr/bin/node
    npm: 9.8.0 - ~/.local/share/pnpm/npm
    pnpm: 8.6.6 - ~/.local/share/pnpm/pnpm
  Browsers:
    Chrome: 114.0.5735.198
  npmPackages:
    @sveltejs/adapter-auto: ^2.1.0 => 2.1.0
    @sveltejs/adapter-node: ^1.3.1 => 1.3.1
    @sveltejs/adapter-static: ^2.0.2 => 2.0.2
    @sveltejs/kit: ^1.20.4 => 1.22.1
    svelte: ^4.0.0 => 4.0.5
    vite: ^4.3.6 => 4.4.1

Severity

Blocks all usage of SvelteKit for apps that need both adapter-node and adapter-static

Additional Information
The workflow I was aiming for is:

Catch a locale cookie in +layout.server.ts
In +layout.ts, return either the server's cookie or the browser's cookie
In +layout.svelte, setup a store with the cookie, put it in a context, use that context to translate the app

Because of that, it seems impossible to make SSR + CSR apps that have authentication via cookies; you either have to completely disable SSR or completely disable static builds, which is unacceptable.
Related

#7637
#7967
#10022
#10215
#10344
#10346

You can't use non-prerenderes .server files with adapter static. Adapter static assumes you have completely static output, no node server running.
We should make that more obvious through a clear error message.

Not being able to use them is fine by me, the issue is that it crashes the web page.
When I read Adapter static assumes you have completely static output, no node server running, I understand that the rendered web page will assume you have completely static output, no node server running. But in reality, it doesn't actually assume that because a request to __data.json?x-sveltekit-invalidated=01 was made.
Not only that, but there's a whole section about when not to pre-render

Note
Not all pages are suitable for prerendering. Any content that is prerendered will be seen by all users. You can of course fetch personalized data in onMount in a prerendered page, but this may result in a poorer user experience since it will involve blank initial content or loading indicators.

Accessing url.searchParams during prerendering is forbidden. If you need to use it, ensure you are only doing so in the browser (for example in onMount).

So it seems like pre-rendering is not suitable to applications with user authentification.
Essentially, what I'm saying is that adapter-static should behave exactly as if you were running this:
find src/routes -name '*.server.*' -delete
pnpm build
http-server ./build

In my project, layout.server.ts has only below options with adapter-static:
export const prerender = true;
export const csr = true;
export const ssr = false;
export const trailingSlash = 'always';

It works on svelte v3 and I created this for some reason at that time.
But I faced __data.json loading after update v4 so searched this issue.

In my project, layout.server.ts has only below options with adapter-static:
export const prerender = true;
export const csr = true;
export const ssr = false;
export const trailingSlash = 'always';

It works on svelte v3 and I created this for some reason at that time. But I faced __data.json loading after update v4 so searched this issue.

I was able to fix this issue by exporting those options in my layout.js instead of layout.server.js. So it indeed seems that .server files are causing the calls to __data.json with adapter-static.

Something like this works (on Linux):
FOUND=$(find src/routes -name '+*.server.*')
echo $FOUND | xargs rm

BUILD_ADAPTER='static' pnpm run build
echo $FOUND | xargs git checkout --

svelte-kit sync

You basically delete the +*.server.* files, make the static build, then restore them. This makes SvelteKit usable in both server-side deployments and client-side deployments from a single code base.

But the .server. files are important for the static build, too. They are used during SSR and collect data from the filesystem.

There's no such thing as SSR in a CSR build. If you want to make a SSR build, then make a SSR build.

If I use adapter-static, SSR generates the HTML pages that are hydrated into an interactive client-side-rendered (CSR) page in the browser. During the build, the .server. files are used.

Essentially, what I'm saying is that adapter-static should behave exactly as if you were running this:
find src/routes -name '+*.server.*' -delete
pnpm build
http-server ./build
git checkout -- 'src/routes/+*.server.*'

And even if you were using data in +page.ts, it would just come out as null and you would be able to account for that and have an application that you can deploy on both a server with SSR and on a static file host like GitHub Pages.
But really, the elephant in the room is that simply having a +page.server.ts file shouldn't crash the page. Requests to __data.json?x-sveltekit-invalidated=01 should be completely disabled in static builds.

These requests are required. The output of the server load function is saved during the prerender process as a __data.json file. This data file, along with the javascript to render the page, is requested during client-side rendering.
I'll close this because I can't reproduce the original issue. It throws an error during build if there is a non-prerendered server load function, rather than during the build preview. Added in https://github.com/sveltejs/kit/pull/7264
> Using @sveltejs/adapter-static
  @sveltejs/adapter-static: all routes must be fully prerenderable, but found the following routes that are dynamic:
    - src/routes/

  You have the following options:
    - set the `fallback` option — see https://kit.svelte.dev/docs/single-page-apps#usage for more info.
    - add `export const prerender = true` to your root `+layout.js/.ts` or `+layout.server.js/.ts` file. This will try to prerender all pages.
    - add `export const prerender = true` to any `+server.js/ts` files that are not fetched by page `load` functions.

    - pass `strict: false` to `adapter-static` to ignore this error. Only do this if you are sure you don't need the routes in question in your final app, as they will be unavailable. See https://github.com/sveltejs/kit/tree/master/packages/adapter-static#strict for more info.

  If this doesn't help, you may need to use a different adapter. @sveltejs/adapter-static can only be used for sites that don't need a server for dynamic rendering, and can run on just a static file server.
  See https://kit.svelte.dev/docs/page-options#prerender for more details

You can test this by cloning git@github.com:NatoBoram/bug-report-sveltekit-ssr-csr.git. I have updated dependencies to their latest and the issue is still present.
There are further instructions in the README.md to make it run with the current issue or with a workaround.
// +page.server.ts
import type { PageServerLoad } from './$types';

export const load = (async ({ cookies }) => {
	const locale = cookies.get('locale');
	return { locale };
}) satisfies PageServerLoad;

// +page.ts
import { browser } from '$app/environment';
import cookie from 'cookie';
import type { PageLoad } from './$types';

export const load = (({ data }) => {
	if (browser)
		return {
			locale: cookie.parse(document.cookie)['locale'],
			from: 'browser'
		};

	return {
		locale: data.locale,
		from: 'server'
	};
}) satisfies PageLoad;

locale cookie: {data.locale}

locale obtained from: {data.from}

<button on:click={setLocaleCookie}>Set locale cookie</button>

> Using @sveltejs/adapter-static
  Wrote site to "build"
  ✔ done
✓ built in 2.38s

GET
http://<IP_ADDRESS>:8080/
[HTTP/1.1 404 Not Found 3ms]

XHRGET
http://<IP_ADDRESS>:8080/__data.json?x-sveltekit-trailing-slash=1&x-sveltekit-invalidated=01
[HTTP/1.1 404 Not Found 1ms]

SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data [app.6b60deac.js:1:5717](http://<IP_ADDRESS>:8080/_app/immutable/entry/app.6b60deac.js)

Ah you’re right. Sorry I missed that. We should probably throw an error when +page.server.js is exported without being prerendered.

Rather than throwing an error, I'd appreciate if it simply didn't make network requests and if data was null so one wouldn't have to rip out files from the project to build it for different environments.

I'm not sure if a silent error is good here. Someone might want the data but forgot to add a export const prerendered = true to the file.

I like that solution.
This way, if I set export const prerender = false but still use adapter-static, then I'm clearly saying that the data from that +page.server.ts or +layout.server.ts shouldn't exist at runtime and to not fetch it. In my +layout.ts or +page.ts, data should be undefined or null and I should be able to continue as normal.
Someone could still enable pre-rendering on these files by simply adding export const prerender = true, in which case data should return whatever was done during build time.


What is the problem?

I have encountered an error when I run the command "mvn  clean package appassembler:assemble" after I clone the repo.

I guess this is because of the dependency versions
From REAME.md:
Anserini was recently upgraded to Java 11 at commit 17b702d (7/11/2019) from Java 8. Maven 3.3+ is also required.

ok,thanks,I will try old version.

Hi, the problems is Apache Maven Compiler Plugin's configuration in pom.xml .

@demeiyan did you roll back to commit 17b702d?

@Peilin-Yang @lintool  ok,thanks ! I have solved the above problems. I need Anserini Retriever , mainly retrieve top-k passage from passage collection given a question (I have passage collection) , like [https://arxiv.org/pdf/1902.01718.pdf](End-to-End Open-Domain Question Answering with BERTserini) . I try [https://github.com/castorini/anserini/blob/master/docs/experiments-msmarco-passage.md](BM25 Baselines on MS MARCO (Passage)) , but always failed when download  msmarco-passage. Is there any better guide I can try? Thanks!

What do you mean? This seems to work fine for me...
wget https://msmarco.blob.core.windows.net/msmarcoranking/collectionandqueries.tar.gz

GWF issue?

yes! maybe GWF, I need better IR experiments I can try to use anserini and then I can apply to my dataset.

@demeiyan I don't understand what you are trying to ask... Anserini should work for any of the collections listed in the README.

Hearing no further follow-up, closing issue.


[3.0] Known result tests for EC compression compatibility validation + #16624 changes

This is mostly identical to #18320 , with the following differences:

target openssl-3.0 rather than master, as 3.0 is the primary target of the fix and validation
a temporary drop! skip CLI round-trip tests in 15-test_ec_comp is applied, as these tests are currently failing
cherry-pick from the rebased and squashed #16624, on top of the new validation tests, as it makes it easier to see what failed before and how the fix corrects the issue. Also, by cherry-picking we preserve the commit message of the original fix which has some extra documentation.

@hlandau surprisingly, checking out 9e437b20c3db2d42a92ffaeb8cb717dae5cf01c8 (the commit just before the patch from #16624 , which does intentionally disable the CLI round-trip tests) I see test/ec_comp_test.c passing!
I was expecting it to fail before the #16624 changes: do you have an idea why it does not?


RuntimeError: cuDNN error: CUDNN_STATUS_NOT_INITIALIZED

Hey there, I was trying to run test_pose_estimation.py on RTX 3090Ti, but I encountered a RuntimeError: cuDNN error: CUDNN_STATUS_NOT_INITIALIZED. Do you happen to know what could be causing this issue?

Hi @shirleyanan,
This error may result from various causes, such as an incorrect environment or GPU memory issues. Please refer to the following link for potential solutions:
https://stackoverflow.com/questions/66588715/runtimeerror-cudnn-error-cudnn-status-not-initialized-using-pytorch
I guess that the RTX3090 may not have sufficient GPU memory to run the back-propagation of the diffusion model (inversion process). The code was tested on a Tesla V100, and it consumed approximately 28GB of memory.

Thank you for your response, I tried lowering the torch version and it seems to have fixed the issue. But yeah, like you said, the RTX3090 can't handle the full test, so I'll switch to using the V100 for testing.


Added error reporting section to help menu

Automates error reporting process to include Topsoil Version, Java version, and Operating system of the user.

Very cool! Definitely a good start! I'm excited about this feature.
There are some things we need to clean up though. Off hand:

[ ] Button should read "Report Issue". "Issues" button should probably be removed.
[ ] Formatting of the new issue could use a little love.
[ ] Functionality should be incapsulated in a separate class.
[ ] See URLEncoder.

I see you're already making changes, though, so I'm not too concerned. I just wanted to give you quick feedback. See you Friday.

Replaced by #242.


IIGS Mouse Pointer Tracking

This implements mouse desktop pointer tracking on the IIGS (GS/OS).
Apple II mouse (slot 4 screen hole data injection) and host mouse pointer hiding still TODOs.

TODO:

Hide debug display
Hide mouse pointer if being tracked within view


Minor fix in PowerBI template

Description

Checklist

[ ] This pull request was tested on both of:

[x] PowerShell 5.1 (Windows)
[ ] PowerShell 6.x
[ ] PowerShell 7.x (Latest PowerShell)

[ ] Scripts do not contain static passwords or other secret tokens.
[ ] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform and PowerShell version: [pscustomobject]$PSVersionTable | Select-Object OS, BuildVersion, PSVersion
Az version: (Get-InstalledModule -Name Az).Version

#sign-off

@mikefrobbins - Kindly review so the changes can be pushed.


TypeError: init() missing 2 required positional arguments: 'doc' and 'pos'

Help!!
how to solve this proplem!

same error

Problem solved by checking and modifying annotation files.


Feature freeze day for documentation for POPL?

I think we decided that it would be too time-consuming and fragile to have attendees trying to write code along with us, so it's not so high priority that we have perfect documentation when POPL rolls around. Of course, it would be nice if people could try it for themselves afterwards.

Fair enough.


Fix design and functions for home page and taken images page

Fix design and functions for home page and taken images page


responseTemplate and responseTopic in creation of webhooks

Hey,
Is there any reason why responseTemplate and responseTopic are not arguments on the webhook creation call? https://github.com/spark/particle-api-js/blob/master/src/Particle.js#L331
Happy to open up a pull request to add them there - but I wanted to follow up with you guys on this before doing so.
Thanks.

No reason, a PR would be helpful!

Thanks @brycekahle! I hope this one helps https://github.com/spark/particle-api-js/pull/22

PR merged.


[opt](inverted index) add performance profiling for remote io access in inverted index

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:

Implement the accounting of remote I/O operations for the inverted index in profiling statistics.

Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run p0

run buildall

run buildall


Reset original color in resetProgress()

To be able to reuse an existing progress indicator after it was cancelled, it should reset the original color in resetProgress().

👍


adding instructions for angular install

since there isn't a simple set of steps for integrating sigma.js with
angualr, I wrote one.

Thanks for the PR @woolfel. Could you add your documentation to the wiki, rather?

I will take a look

I edited the wiki with the instructions.


Create a favicon

The favicon should match the logo of the application.

Placeholder favicon created. Will be updated once we have a proper logo.


On El Capitan isn't working

I just configured my SSH keys, the top bar menu shows correct and the examples also, but when I click on them nothing happens.
Where can I go to see logs or errors to share with you?
Thank you.

Hi @pabloleone When you say nothing happens what do you mean exactly? No terminal windows open or a terminal window opens but your not getting connected ie your command is not running?
What terminal are you using?

Just nothing. No terminal, nothing. I downloaded the code and it doesn't work from xcode either and sadly I don't know objective c. I ran the app using xcode and there's no issues.
The examples don't work.

If you have a really old version of shuttle you will have to update your shuttle.json to have the nightly directive "iTerm_version":"nightly"  Also check out this issue. #176 and this pull #175
Also make sure that your global settings reflect the default profile in iTerm or make sure that your iTerm has a default profile and that profile is called "Default" also make sure that your iTerm is not set Don't Open Any Windows in Preferences/General/Startup
I'm using 10.11.5 and I have iTerm 3 nightly everything is working so it seems that its a configuration problem.
Please check these things and report back.

Thank you, the issue was the iTerm_version value, I just updated with nightly value and everything is working.

@pabloleone You're welcome, Awesome! Thanks for the update.


low quality pen

Please make a high pen option

High quality pen is not an option in vanilla Scratch, so the HTMLifier doesn't offer it either


Link to privacy policy

Bug: https://bugs.gentoo.org/613938

Merged to fork at https://gitweb.gentoo.org/proj/tyrian-sphinx-theme.git/.


Feature request: ability to disable redux logs in the console

@sonaye This seems to be a duplicate of https://github.com/react-community/react-navigation/issues/753
Fire a pull request and I think this will get merged in :+1:

Glad someone already pointed that out, I am not familiar with the redux setup unfortunately, learned it from scratch for the sake of learning and having a better understanding of state management in general, ended up switching  to another solution though. Thanks for the heads up!

@sonaye Out of interest, what alternative did you go with? :)

MobX! of course, it just feels right, from the first line of code, been using it in production for a while now.


Add optimized file access that also works for non-seekable files

There are some use cases that want to create the index while downloading, so something like wget | tee downloaded-file | pragzip --export-index downlaoded-file.gzindex.
I think this should be solvable with a new non-seekable FileReader derived class with these two main ideas:

A working implementation could simply cache the whole file on-demand in memory. This way seeking will always work.
This obviously would use up too much memory for very large files. To remedy that an interface would be required to mark everything before an offset as not needed anymore.

This way seeking will always work as long as the caller does not try to access parts that he marked himself as not needed anymore!
When processing a block everything before the end of its compressed offset can then be marked as to be dropped!

Assuming that https://github.com/mxmlnkn/ratarmount/issues/106 is caused by the non-sequential file access, then this addition might also fix that.. Assuming I could use the non-seekable file reader during index creation. From the outside I know that I only have to go over the file sequentially but I would also require a Python interface reflecting that. It might be easier to do the gzip index creation as a separate step / pass. This way I could also implement a version that does not actually decompress anything but only gathers index seek points. That way memory usage would be limited even without implementing #2! This idea would be implementable with the refactoring done for #11. It just needs another specialized ChunkData subclass.

See https://github.com/mxmlnkn/indexed_bzip2/tree/single-pass-reader for an attempted implementation. I think I stopped working on that branch because the performance didn't look very good but still much better than single-threaded gzip. Also has lots of todos in the code comments in the last commit.

Partly fixed with https://github.com/mxmlnkn/indexed_bzip2/commit/9502781a9d2fa8b265fbe0a3b42177a1d5406836, but automatic detection is not yet implemented. Instead, I'm currently shooting for an --sequential or similar option. Maybe even make the pread the non-default case because sequential access probably is faster even on SSDs. It probably only is necessary to reach peak speeds for files on /dev/shm. Note that this does not affect usage via Python because the SinglePassReader is only to be used when seeking is not required. This means that https://github.com/mxmlnkn/ratarmount/issues/106 requires a different fix altogether. Maybe using mmap with sequential prefetching.


FEAT-#3042: support concat via join

What do these changes do?
Add support for concat operation requiring a join operation in the OmniSci engine.

[x] commit message follows format outlined here
[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #3042
[x] tests added and passing
[x] module layout described at docs/developer/architecture.rst is up-to-date

Left a minor suggestion, otherwise LGTM
@YarShev any other comments?


"Diamond" like inheriting structure with Processes and Recipes

I'm fiduring out while tweaking the merge_sections function that we have some problematic classes:

EtchingRecipe
AnnealingRecipe
and so on

Those are never really used because what we do in the code is:
MyEtchingRecipe(MyEtching, Recipe)
not
MyEtchingRecipe(EtchingRecipe)
So i propose to get rid of those and explain in the Recipe class  how to use it,
is it a topi for datamodeling @ka-sarthak @hampusnasstrom ?

That sounds very interesting! Yes, would be great to hear about it on Wednesday :)

ask for inheritance of annotation so the get the hide inherited


Resolve svg sprite path

Thanks for the awesome package, Really love the the work you're doing
This is just a PR fixing the sprite path import in plugin.js
Unix systems lint different cases within a import path fine
but when we ran on a Linux based system (travis), the linter failed due to the path error

Cool, thanks!


Help to compile raylib.a under windows

Hi Anthony Walter !!!
could you tell me how you compiled libraylib-win64.a.
I've already spent all my brains. I have errors under Windows when linking. And with your libraylib-win64.a everything links.
But I need the latest version of ray gui. on linux there are no problems with compilation and linking.
And windows 10 is not how it does not work.

Just for your information, I use mingw64 to build libraylib-win64.a.


NoMeta error getting packages with v>0.2.2

Hello,
I seem to be getting NoMeta errors whenever running bingo get (for any package) with bingo v0.2.2 or greater.
See below for steps to reproduce:
# docker run -it golang:1.14.2-buster /bin/bash
# go version
go version go1.14.2 linux/amd64
# export GO111MODULE=on && export GOBIN="$HOME/.bin" && export PATH="$GOBIN:$PATH"
# go get -u<EMAIL_ADDRESS># bingo get github.com/golangci/golangci-lint/cmd/golangci-lint
Bingo not used before here, creating directory for pinned modules for you at .bingo
go: downloading github.com/golangci/golangci-lint v1.27.0
go: found github.com/golangci/golangci-lint/cmd/golangci-lint in github.com/golangci/golangci-lint v1.27.0
go: downloading github.com/spf13/cobra v0.0.5
go: downloading github.com/fatih/color v1.7.0
go: downloading github.com/mitchellh/go-homedir v1.1.0
go: downloading github.com/go-critic/go-critic v0.4.1
go: downloading github.com/go-lintpack/lintpack v0.5.2
go: downloading github.com/spf13/pflag v1.0.5
...
 Reproducing consistently with:
docker run -it golang:1.15.5-buster /bin/bash
export GO111MODULE=on && export GOBIN="$HOME/.bin" && export PATH="$GOBIN:$PATH"
go get -u<EMAIL_ADDRESS>bingo get github.com/golangci/golangci-lint/cmd/golangci-lint

Somehow the following works, but I can't really explain why:
docker run -it golang:1.15.5-buster /bin/bash
export GO111MODULE=on && export GOBIN="$HOME/.bin" && export PATH="$GOBIN:$PATH"
git clone https://github.com/bwplotka/bingo && cd bingo && go install
bingo get github.com/golangci/golangci-lint/cmd/golangci-lint

@bwplotka, do you have any idea of why this could be happening?

Interesting (: Will take a look, looks like installation path matters? :thinking:

Let's close this for now. What we could try next time this happens is -u or  -upatch bingo options.  Let's try to repro with 0.3.0 (:
https://github.com/bwplotka/bingo/releases/tag/v0.3.0

Let's close this for now. What we could try next time this happens is -u or  -upatch bingo options.  Let's try to repro with 0.3.0 (:
https://github.com/bwplotka/bingo/releases/tag/v0.3.0

Let's reopen if seen again.

Let's reopen if seen again.


MongoDB 3.2

From pairing w/ @krallin
This adds support for MongoDB 3.2, with  { sslMode: 'requireSSL' }. Two new integration tests have been added, to ensure that connections are accepted over SSL, and rejected without SSL.
We'll plan to backport SSL support to 2.6 as well. Because of limited client support for SSL on 2.6, we'll opt to use { sslMode: 'allowSSL' } there.

@fancyremarker
As mentioned in #11, I was adding tests for backup / restore. Turns out SSL is somewhat problematic here:

We need to connect over SSL from mongodump and mongorestore on 3.2 where SSL is required.
We use a database URL parse by parse_url, but that doesn't understand the ?uri.ssl option in the mongo:// connection string.
Note: I also realized we have the same problem with --client. We added --ssl in the command line, but:

We disregard whatever is passed in the URL
We don't let suers configure SSL trust options

Cheers,

@krallin: Good point. I'm closing this PR, and will look for a new one to replace this and #11, with the SSL fixes you cited above.


Change ivy home when running scripted tests

This commit changes the build so that when running scripted:

sbt is published to a local repository in a different ivy home. We
publish using the ivy home /path/to/sbt/sbt/target/scripted-ivy
(instead of ~/.ivy2).
this custom ivy home is explicitly specified at when sbt is launcher.

Fixes sbt/sbt#2539

Superseded by https://github.com/sbt/zinc/pull/215.


vm.synced_folder 'type:' is handled by Vagrant automatically

Problem/Motivation
My Ubuntu 18.04.1 desktop throws the following error on vagrant up
It appears your machine doesn't support NFS, or there is not an
adapter to enable NFS on this machine for Vagrant. Please verify
that `nfsd` is installed on your machine, and try again. If you're
on Windows, NFS isn't supported. If the problem persists, please
contact Vagrant support.

Vagrant should already detects the synced_folder type so this logic may not be needed in the Vagrantfile. https://www.vagrantup.com/docs/synced-folders/basic_usage.html#type
Proposed changes
You could probably remove the type: all together https://github.com/hassio-addons/hassio-vagrant/blob/master/Vagrantfile#L136
Or if Windows is an issue, leaving the else empty seems to work for me
RbConfig::CONFIG['host_os'] =~ /mswin|mingw|cygwin/ ? 'smb' : ''
 GitHubhassio-addons/hassio-vagrantVagrant box running Hass.io / Home Assistant. Contribute to hassio-addons/hassio-vagrant development by creating an account on GitHub.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Apparently the terminal message I posted above isn't accurate or very helpful for Ubuntu users.
nfsd isn't available in the Ubuntu repositories.
According to this https://github.com/hashicorp/vagrant/issues/1534
An extra step of sudo apt-get install nfs-common nfs-kernel-server is needed to get NFS going on Ubuntu.
I'll try to test this out soon.


Adding back the new CLI link in the README.md

Was quite confused when I don't see the new docs link in the repo or in the flutter pub website. I see that it is replaced with the ads for Flutter web course, but maybe erasing the new docs link was not intentional?

Thanks. I'll merge this in now.

Closing this, the new docs is in the reamde. It's big and bold so can't be missed. Right under the header. we don't need the CLI specific doc in here.


Fix LICENSE

Could you please fix the license?
https://github.com/deshanadesai/acorns/blob/59b208e002a5ef020910f705ec18f9faeb461113/LICENSE.txt#L3
It is missing the year and authors.
After that, could you please add the license to be package along with the source distribution?

Added. Thank you!


GC: Use a page allocator to avoid extra copy

Currently our GC works like this:
We begin with
[stack][static data][from-space]← End of Wasm memory

then GC copies to to-space:
[stack][static data][dead-space][to-space]← End of Wasm memory

and then it copied to-space into dead-space (with a simple memcopy, i.e. the updates pointers already take that into account):
[stack][static data][to-space]\/[dead space]← End of Wasm memory

where \/ marks the “end of heap” where we begin allocation.
Better would be the following:
After
[stack][static data][dead-space][to-space]← End of Wasm memory

we go to
[stack][static data]\/[dead-space][to-space]← End of Wasm memory

i.e. don’t move to-space again, but instead begin allocating at \/.
After a while we have:
[stack][static data][live]\/[dead][live]← End of Wasm memory

and if the next thing we want to allocate is bigger than [dead], we probably have to this:
[stack][static data][live][dead(slack)][live]\/ ← End of Wasm memory

i.e. move the end-of-heap until the end of memory and maybe request more pages from the system, and the small [dead] becomes wasted slack.
I can’t quite wrap my head around what happens after many GCs. Will the number of regions to be tracked (as dead or alive) grow, or are they bounded? Will the slack regions clean up again?
So maybe takes this as a starting point to think this through :-)

Superseded by incremental GC


Unsupported Windows version

I'm currently unable to install AppGet, as it says I run an unsupported version of Windows : Windows 7 Enterprise SP1 (64 bits)
Tried on 0.6.51, 0.6.50, 0.6.47
I couldn't find the list of supported OSes, which OS are supported ?

Nevermind, it was a leftover from using the XP SP3 compatibility tool, looks like it requires Win7 to be installed

Yes, we don't support Windows XP/Vista


Ctrl-L fails at the bottom of the buffer

Thanks so much for this package- it helps me feel right at home in sublime.
An observation: When pressing Ctrl-L at the end of the buffer, the current line remains at the bottom of the screen, versus emacs which still centers it, even though the space below point has no content.
Is this a limitation of sublime in that it cannot show space below the end of the buffer? or is it a design decision? or is it an issue that can be corrected?
My version is sublime text 2.0.2 on mac os 10.10.5

Here’s the sublime setting to fix that.
"scroll_past_end": true,

Wow, speedy and immediately useful! thanks so much.


Spec and implementation disagree on clu_nd_range

The spec says all the members are cl_uint, but in reality the array members are size_t.

The implementation matches the API spec for clEnqueueNDRangeKernel.
The spec probably will never be kept in synch with the implementation.


Authentication didn't work in Modules

Hi,
in the controllers and views of my modules the Auth didn't work:
In Module: Modules/<module_name>/Http/Controllers/HomeController:
public function show()     {         if (Auth::check()) {             echo "test";         }     }
Didn't work!
In vanilla controller: App/Http/Controllers/HomeController:
public function show()     {         if (Auth::check()) {             echo "test";         }     }
It works!
It's the same with the @auth @endauth or displaying data of user.
Is there anything I need to consider?
I'm using laravel 5.6 and laravel-modules 3.0
Thanks for help

Solved it with following code in Modules/<module_name>/Http/routes.php:
Route::group(['middleware' => ['web'], 'namespace' => 'Modules\<module_name>\Http\Controllers'], function() {     Route::get('/', '<controller_name>@show'); });


Add support to create VM from image gallery

REF: aztfmod/terraform-azurerm-caf#4
As part of level 3 solution deployment, we have the ability to create a shared image gallery. I am unable to find any documentation or reference on how to deploy a virtual machine from a shared image gallery. We need to support deploying a virtual machine from a shared image gallery published with the following configurations:

[ ] Shared Image gallery in the same subscription as the virtual machine
[ ] Shared Image gallery in a different subscription but same tenant as the virtual machine
[ ] Shared Image gallery in a different tenant and subscription

hi @hriaz have you looked there?
https://github.com/aztfmod/terraform-azurerm-caf/blob/32593cd85c33c988be595edd48a87fffcd5ea6e6/modules/compute/virtual_machine/vm_linux.tf#L101
So I look at this code, when you specify custom_image_id you should be able to create it.
As I'm looking at the shared image gallery module, we dont have the right output there: custom_image_ids, so it will require a bit of updating.

OK will have a deeper look @hriaz


Quincy's UI recommendations

https://www.freecodecamp.com/design-style-guide/

Some quick feedback:


Music Doubles and does'nt show's that it's playing

Everything worked perfectly but after adding to the mods folder Sounds mod and Footsteps mod the music started to double and didn't even shown that anything IS playing. Is it possible to keep the Footsteps mod and the Sounds mod too? Reaaaaaly want to keep those mods all together if it's possible.
I tried to record how it sounds but for some reason, both window's and NVIDIA players don't want to record properly.

Hi, thanks for the report! Could you pleases give me the actual names of the mods you tried to play with? Just recently, someone told me an incompatibility with the "The Immersive Music Mod". If that's your Music mod, this is the issue, and you can likely keep your footstep mod (never had any issue with footstep mods). This sound mod is very intrusive and not clean at all the way it is made, so the issue doesn't come from my mod. I will try to investigate though

Alright, I will check those. Thanks!


Is there a more generic way to load struct parameters from local file?

Code like this can be used to load address list:
address[] memory output = cheats.envAddress(key, delimiter);
But is there a more generic way to load data from local file(preferably in json format), into structs or structs array ?

close since https://github.com/foundry-rs/foundry/pull/2293 is merged.


Table of contents

Would be possible to add headings/table of contents to articles on the right?

Thank you for your valuable suggestion!
We understand that a Table of Contents (TOC) is a helpful tool for quickly scanning and jumping to specific sections of an article. However, our design philosophy leans towards maintaining a clean page layout to encourage deep reading. We believe this offers a less distracted and more focused reading experience.
We will continue to collect feedback from all users and consider balancing simplicity with functionality in future updates. Please stay tuned for our updates, and we look forward to providing you with an even better reading experience.

Thank you for your valuable suggestion!
We understand that a Table of Contents (TOC) is a helpful tool for quickly scanning and jumping to specific sections of an article. However, our design philosophy leans towards maintaining a clean page layout to encourage deep reading. We believe this offers a less distracted and more focused reading experience.
We will continue to collect feedback from all users and consider balancing simplicity with functionality in future updates. Please stay tuned for our updates, and we look forward to providing you with an even better reading experience.


remove explicit style ignore

This pr fixes a broken style link in the static site.
Before this change the drop down list items would not lay on top of each other.

After this change the drop down will properly lay on top of each other.

Also the footer of the page was incorrectly laid out because of this.
Before this PR..

After this PR...


Replace set-output command

Warning: The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

New usage:
- name: Set output
  run: echo "{name}={value}" >> $GITHUB_OUTPUT


DHARMA_INSCIK00121 - Zotero Short Title Issue

Dear @chhomkunthea,
In your edition of K 121, you have used the ZST Finot1904_02 but it matches 2 entries in Zotero: Nécrologie: prosper Odend'hal & Notes d'épigraphie: VIII. Inscription de Prah That Kvan Pir. Can you check this please?
Thanks

I will take care of this.
however, @chloe and @kunthea: I fear you have no yet finished your zotero work, and/or have not being paying attention to possible duplication of short titles with items stored outside of the E-Cambodia folder. if the latter is true, it means you will both need to double check all zotero short titles in E-Cambodia to make sure the sort titles are unique in the entire DHARMA group library.
Le 13 avr. 2021 à 10:54, ajaniak<EMAIL_ADDRESS>a écrit :
Dear @chhomkuntheahttps://github.com/chhomkunthea,
In your edition of K 121, you have used the ZST Finot1904_02 but it matches 2 entries in Zotero: Nécrologie: prosper Odend'hal & Notes d'épigraphie: VIII. Inscription de Prah That Kvan Pir. Can you check this please?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/erc-dharma/tfc-khmer-epigraphy/issues/29, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGMAE66C6JI3DD5TQIDBTLTIQBC3ANCNFSM42244MTA.

Dear Axelle and Arlo,
I see that there are indeed two short titles Finot1904_02 in the Zotero. The one we need is VIII. Inscription de Prah That Kan Pir. I think that we should correct its short title to Finot1904_06. I have tried to change it but the short title Finot1904_06 was accepted in the Oxygen. So I should wait for Arlo to set it right.
I plan to check all the items in the folder E-Cambodia by the end of this month. While doing so, I will also check the possible duplication of short titles with items stored outside of the E-Cambodia folder.
Thank you!

I think the problem was the same as in the other case: your XML file did not validate in Oxygen because you hadn't adjusted the tag in zotero. Anyhow, I have now made the changes to distinguish Finot1904_06 from Finot1904_02, so that the file now validates.
I have simultaneously inserted some comments into the file for K. 121, marked with "argr2kuch". Please make the necessary changes in this file (and check all your other files on the same points) and then close this issue.

Yes, I've checked your comments in K.121 and made corrections. I'll check other files too. Thank you!


New Item Creation

Is it possible to add new item created by maya software instead of blender.
please help to how to add new item in http://furnishup.github.io/blueprint3d/example/ existing project.
i have gone through all the issues and not found any feasible solution for the same please help.

Is it possible to add new item created by maya software instead of blender.
please help to how to add new item in http://furnishup.github.io/blueprint3d/example/ existing project.
i have gone through all the issues and not found any feasible solution for the same please help.

https://blender.stackexchange.com/a/22956

I have been trying to figure out how to add an option myself.


[SourceKit/CodeFormat] Indent lines in multi-line strings

The first commit fixes a couple bugs in the Lexer::getLocFor*Line methods in a couple edge cases.
The second updates the formatter to indent multi-line strings where it can. When it's an un-closed string it will indent new lines to the first previous line that isn't whitespace-only, unless that line is the same as the start quotes. In that case it will either indent to the start quotes if that line is only quotes or add an extra indent level otherwise. For closed strings we have to be a little more careful since we don't want to add significant (and possibly unintended) whitespace, so those lines are only indented when they would otherwise be invalid (ie. less than the end quotes).
This could add trailing whitespace to existing lines, but I think that's better behaviour than what's there now. This could be fixed if the whole range was passed in instead of a line at a time, as the indentation could be different when it is actually a single line in that case.
Resolves rdar://32181422

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 655b008455e74b8c96776d57e6c7cd32a3c1e4c1

Build failed
Swift Test Linux Platform
Git Sha - 655b008455e74b8c96776d57e6c7cd32a3c1e4c1

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 655b008455e74b8c96776d57e6c7cd32a3c1e4c1

Build failed
Swift Test OS X Platform
Git Sha - 655b008455e74b8c96776d57e6c7cd32a3c1e4c1

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 8f906393268b8edd918f5ba6f64c3ef649015b73

Build failed
Swift Test OS X Platform
Git Sha - 8f906393268b8edd918f5ba6f64c3ef649015b73

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 006472cf5041d919a762383900f51dd39dff30fc

Build failed
Swift Test Linux Platform
Git Sha - 006472cf5041d919a762383900f51dd39dff30fc

@swift-ci please test

@swift-ci please test and merge

@swift-ci please test and merge


关于正确率

你好，这个代码得到了与论文中一致的结果吗？

Best result is 2% lower than that in paper. Some trick is not published now.

Yes, but it was a long time ago.


docker-compose file

Here is a quick docker-compose file which will pull the latest stable pigallery2 and mariadb to allow you to quickly transistion from sqlite to mysql.
this will create the pigallery table in mariadb, which you can then connect to.
Note i had to put in the docker IP address into the MySQL configuration
just make sure you update your volume paths and passwords.
pigallery-db:
   container_name: pigallery-db
   image: mariadb
   # links:
   #    - pigallery:pigallery-db
   volumes:
      - /storage/pigallery/data/db:/var/lib/mysql
   environment:
     - MYSQL_ROOT_PASSWORD=putsomethinginherethatyouwouldlikeasthepassword
     - MYSQL_USER=pigallery
     - MYSQL_PASSWORD=pigallery
     - MYSQL_DATABASE=pigallery

pigallery:
   container_name: pigallery
   image: bpatrik/pigallery2:nightly-stretch
   volumes:
      - /storage/pigallery/data/config.json:/pigallery2-release/config.json
      - /storage/pigallery/data/sqlite.db:/pigallery2-release/sqlite.db
      - /storage/pigallery/photos:/pigallery2-release/demo/images
      - /storage/pigallery/data/temp:/pigallery2-release/demo/TEMP
   environment:
      - NODE_ENV=production
   ports:
      - "1112:80"

@bpatrik, perhaps you could add this to the installation instructions.
cheers,

I have added this to: https://github.com/bpatrik/pigallery2/tree/master/docker/docker-compose/with-mysql


Metadata: WowUp.CFbeta version 2.9.1-beta.8

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Change name to indicate beta version as #135378 adds the stable version.
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Another round of fixes for mapping of DataType to DType

Getting closer, but there is still a lot more to do.
One thing to note is that for part of this I changed Gpu{Unary,Binary,Trinary}Expression to return a ColumnVector instead of a GpuColumnVector this let me put the conversion in a single location and also fixed some issues with some UnaryExpressions that couldn't know the right spark type to return because they were going to be cast to the final type shortly afterwards.
Actually with the last push this should completely fix the issues with not doing the conversion correctly in all cases.

build

build

build


Add a network policy to allow backstage-sonata traffic

Add a network policy to allow backstage-sonata traffic

When installing Sonataflow operator, also set those network policies based on
namespaces labels:
1. A policy on the workflow namespace[1] that allow ingress from sonata
   namespace and RHDH namespace
2. A policy on the Sonata namespace where the cluster services are
   running that allow ingress from workflows namespaces[2] and RHDH
   namespace

[1] workflow namespace is set as part of of hack/setup.sh script
[2] workflows namespaces are any namespaces with label key rhdh.redhat.com/workflow-namespace

Signed-off-by: Roy Golan<EMAIL_ADDRESS>

/hold last checks before I'm merging

I'm holding this work to make use of a more efficient single label selector -https://github.com/apache/incubator-kie-kogito-serverless-operator/pull/482

I think we may want to make this work a document instead of something that chart will set, here's why:

we don't know ahead of time on which namespace workflows will be deployed.
we don't know in which namespace the sonataflow services (data-index, jobs, database) will be installed. It depends if there will be a cluster platform or a namespaced platform is used.
If a namespaced platform is used then the platform services are collocated with the workflows they use, hence we just need to make sure there is a network policy to isolate this namespace.  By default Openshift doesn't place a default policy so the admin that installs workflow need to add that.
When a cluster platform is deployed then the sonata services are deployed on a known namespace, 'sonataflow-infra'
In this case a network policy can should allow traffic from all pods labled with app.kubernetes.io/component=serverless-workflow AND from backstage backend.
If there is a case where a mix of cluster platform and namespaced platform is supported(is it?) , then during the namespaced platform installation the network policy to isolate the ns should will prevent ingress but will the sonataflow-infra namespace will remain open to calls from the platform namespaces.

After revisiting this we can rely on the installation using hack/setup.sh to create a workflow namespace with a label.
So this means that the helm chart knows what is the name of all three participating namespaces - workflows, sonata, and rhdh and that is a good start.
If other workflows are deployed in a different namespace then this namespace must be labelled with rhdh.redhat.com/workflow-namespace

should be a disallow all traffic, and after that allow that one, no? If not is not blocking at all.

@eloycoto according to the docs and according to my tests this works. pods outside of the specified namespaces can not reach the target namespaces
https://docs.openshift.com/container-platform/4.16/networking/network_security/network_policy/about-network-policy.html

@masayag I can't reproduce the helm lint error locally. any clue?


Support of custom user model

Ref #2347
Some usages of "id" were replaced with "pk", but probably it's not enough to support custom user model.
I checked following:

[x] Files with get_user_model
[x] Files with request.user.id
[x] Files with user.id

Any suggestions?

Merged in 1172349ccab5ca9edbe0ffd7021844436d4f4486 - I also updated the CustomUser test model to use a non-standard primary key (4f50e576aa3112200710162bed59484ffa8cc5c4) and fixed the resulting failures in 3b09ffaee59ea97bfd18b5916f7f140db107e720, so we can be reasonably confident that this fixes #2347 now.


feat: Adds print styles for Starlight docs pages

What kind of changes does this PR include?

Changes to Starlight code
Something else!

Description

Adds print-specific styles for pages built with Starlight
Hides navigation elements like the sidebars, search, and mobile menu
Forces the light theme color palette when printing

size-limit report 📦

Path
Size

/index.html
14.14 KB (+8.31% 🔺)

/_astro/*.js
16.51 KB (0%)

/_astro/*.css
9.06 KB (+0.52% 🔺)

I think my preference is likely for pulling this out to a separate global file to avoid extra downloads. (We’d need a ?url import I’d guess to pass that to the appropriately configured style tag.)
I am a bit torn because as you highlighted, that does come with extra maintenance complexity. Especially as this is exactly the kind of thing we’re likely to forget to check as we add stuff 😅 That said, most stuff is done pretty safely via print:hidden, so we should be OK with that approach?
(There’s a chance even doing that will fail the CI check here — it’s a bit of a rough tool at the moment, just globbing dist/*.css and measuring that gzipped, so even CSS that doesn’t actually get loaded is liable to get taken into account. Tried another approach and found it unreliable, but this is definitely on a list of tooling to improve.)

@delucis Yep that's no problem, I'll refactor that this morning and pull the styles out into a global sheet 👍

@delucis done!  Everything is moved out to the separate print.css file
Looks like the HTML size-limit is failing by ~1kb, that should just be from the extra print: class names + the stylesheet <link>

Best laid plans!  It turns out there isn't currently a great way to get around the CSS being inlined there, I'm going to close this out for now as blocked.
@delucis, @bluwy, and I were catching up on Discord and it sounds like this may be worthy of a feature in Vite to be able to ?url import and make sure it's never inlined.  My vote is still on ?no-really-i-want-a-url, but I'll leave that up to the Vite team to decide 😄

Will need to wait on https://github.com/withastro/astro/pull/12758 to use the ?url&no-inline import query without type errors.


StatefulRedisConnection returns RedisCommands instead of RedisPubSubCommands

Context: lettuce 4.2.2.Final, Java api.
This makes it quite difficult at present to do pubsub with Sentinel enabled. Was this intentional?
This question could also be phrased slightly differently. Instead, it could be: why is it that RedisPubSubCommands inherits from RedisCommands, instead of vice versa?

Not sure I follow about the Sentinel part. Using a redis-sentinel://sentinel1,sentinel2,sentinelN?sentinelMasterId=… URI with any RedisClient.connect/RedisClient.connectPubSub method will use Redis Sentinel look up the according master. Works for regular and Pub/Sub connections.
The reason for the distinction between RedisCommands and RedisPubSubCommands is that once a Redis connection enters a Pub/Sub state (SUBSCRIBE, PSUBSCRIBE) it's no longer usable for other Redis commands. Therefore, Pub/Sub with subscription methods is an add-on to normal Redis connections.

Perhaps I'm misunderstanding what's possible. Consider the following situation: we have a set of sentinel instances running, which are monitoring a master and two slaves. We connect a client using a sentinel URI, and subscribe.
While we are subscribed, the master becomes impaired. Sentinel detects this, and promotes one of the slaves to be the new master, automatically slaving the old slave to the new master.
I would expect in such a situation that the PubSub connection would break (because of the impaired master), and would automatically be established once the new master is elected, without my having to do anything. Is that what will happen? Based on my understanding, I'd have to manually implement the reconnect.
Regarding the distinction: I'm not sure I see how that makes sense. A RedisPubSubCommands object is also a RedisCommands object -- because of inheritance, the relationship described isn't obvious from the class hierarchy. Given that, I don't see why it makes sense to separate them out. Wouldn't it make more sense to have RedisPubSubCommands be treated like every other command set, and just have the underlying command dispatcher throw an exception (i.e. fail fast) if a client ever tried to do a regular command while subscribed via PubSub?

I would expect in such a situation that the PubSub connection would break (because of the impaired master), and would automatically be established once the new master is elected, without my having to do anything. Is that what will happen?

Yes, lettuce will ask the configured and available Sentinels to obtain the new master IP address.

Based on my understanding, I'd have to manually implement the reconnect.
How so?

Ah, I see - it was a misunderstanding of how reconnects are done. Never mind -- thanks for your time. :)


Strange $injector error on module load

var myApp = angular.module("myApp", ['tmh.dynamicLocale']);

As soon as the module is included into my app's module this error gets thrown:
Uncaught Error: [$injector:unpr] Unknown provider: $rootScopeProvider <- $rootScope <- tmhDynamicLocale

Libraries:

Angular 1.2.16
angular-dynamic-locale 0.1.9

edit
I removed the .run(['tmhDynamicLocale', angular.noop]); from the module and now it seems to be working.

I just ran into this issue too. Turns out it was user error.
When injecting, I used:
localeReader.$inject = ['tmh.DynamicLocale'];

instead of
localeReader.$inject = ['tmhDynamicLocale'];
(There is an extra dot in the 1st version).
It's a bit confusing, because the name of the module is 'tmh.DynamicLocale' (with a dot), but the name of the exposed service is 'tmhDynamicLocale'


Stalled Publication/Subscription

Version 1.0.1
I'm debugging a case where sub.poll suddenly doesn't receive any more messages, and it stays that way. On the sending side pub.offer is successful (positive return value and connected=true closed=false).
Only two nodes.

node1: 172-31-10-77
node2: 172-31-8-204

node1 tries to send to node2, but node2 is not started yet. ~30 seconds later node2 is started and they successfully exchange a few messages (I have application level logs for that), and then node2 stops receiving more messages in sub.poll. onFragment in the FragmentAssembler is not invoked even though I repeatedly call poll. Those messages are below mtu size.
The systems are rather loaded in this scenario but not overloaded, and the load is stopped after a while.
AeronStat from node1 172-31-10-77:
 23:                    1 - recv-channel: aeron:udp?endpoint=ip-172-31-10-77:25520
 24:        1,627,656,896 - snd-pos: 8<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 25:                    1 - send-channel: aeron:udp?endpoint=ip-172-31-8-204:25520
 26:        1,636,045,504 - pub-lmt: 8<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 27:              170,432 - sub-pos: 1 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520 @0
 28:              170,432 - rcv-hwm: 7 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520
 29:           19,750,176 - sub-pos: 2 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520 @0
 30:           19,750,176 - rcv-hwm: 9 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 31:           19,750,176 - rcv-pos: 9 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 32:           28,953,216 - pub-lmt: 10<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520
 33:           20,564,608 - snd-pos: 10<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520

AeronStat from node2 172-31-8-204:
 23:                    1 - recv-channel: aeron:udp?endpoint=ip-172-31-8-204:25520
 24:                    1 - send-channel: aeron:udp?endpoint=ip-172-31-10-77:25520
 25:            8,559,040 - pub-lmt: 3 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520
 26:              170,432 - snd-pos: 3 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520
 27:           28,138,784 - pub-lmt: 4 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 28:           19,750,176 - snd-pos: 4 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 29:                2,176 - sub-pos: 1<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520 @0
 30:        1,627,656,896 - rcv-hwm: 5<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 31:                2,176 - rcv-pos: 5<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 32:           20,500,992 - sub-pos: 2<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520 @0
 33:           20,500,992 - rcv-hwm: 6<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520
 34:           20,500,992 - rcv-pos: 6<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520

The problematic session is<PHONE_NUMBER>. The other streams seem to progress. I kept it running for minutes after the stall.
Stream 1 is our control stream and it's low traffic, a few messages per second on this stream.
I have all files, if you need more information.

The sender and rcv-hwm are setting at position 1,627,656,896. However, the subscription has only consumed (and the rcv-pos) is 2,176.

Any other settings in use from the defaults?
Also, you mention this is low traffic. But did you really send something like 1.5GB of data on that channel?

On stream 1 it's just a few heartbeat messages of a few hundred bytes each per second. It is stuck almost immediately (only few messages get through). Reproducing is not deterministic (about 1 in 10 it happens).
On stream 2 it's a lot more messages.
1.5 GB on the sender side. How can that be possible? I use offer with UnsafeBuffer wrapping an ByteBuffer.allocateDirect of capacity 256 KiB (from our pool). Byte order LITTLE_ENDIAN.
Term sizes not changed, using default. All settings default except increased a few timeouts:

client-liveness-timeout = 20 seconds
image-liveness-timeout = 10 seconds
driver-timeout = 20 seconds

The positions don't make a lot of sense, honestly.
Are you sure that all drivers and clients are using the same 1.0.1 version? it might be worth a shot to use the latest snapshot/head also just to see.

@mjpt777 anything come to mind for you?

I think it would be worth looking at the publication and image buffers if you still have them. Also what at the first 30 counters so we can see the aggregate picture?
@tmontgomery I think it might be worth putting the Aeron version AeronStat output.

I'm sure about the version number. Running this on fresh ec2 instances from our sbt build and we have been using 1.0.1 since long back.
Here are all files: https://dl.dropboxusercontent.com/u/14579199/test35.zip

If the positions are correct, then looking at the images is going to be mostly fruitless. The hwm and sub-pos, for example, will be many term buffers in difference. Surely violating the cleaning assumptions.

The stats show only 20MB sent/received, so the high position value is definitely off for some reason. The only reason I can think for this occurring would be corruption of the header in some way that infers a different position. This seems to be happening as the message is put into the term as the snd-pos and pub-lmt are also corrupted. A possibility might be setting the reservedValue with an incorrect offset.
node-1
  0:           20,745,216 - Bytes sent
  1:           20,370,624 - Bytes received
  2:                    0 - Failed offers to ReceiverProxy
  3:                    0 - Failed offers to SenderProxy
  4:                    0 - Failed offers to DriverConductorProxy
  5:                   16 - NAKs sent
  6:                4,917 - NAKs received
  7:                3,340 - Status Messages sent
  8:                3,282 - Status Messages received
  9:                5,697 - Heartbeats sent
 10:                5,311 - Heartbeats received
 11:                    3 - Retransmits sent
 12:                    0 - Flow control under runs
 13:                    0 - Flow control over runs
 14:                    0 - Invalid packets
 15:                    0 - Errors
 16:                    0 - Short sends
 17:                  696 - Client keep-alives
 18:                    7 - Sender flow control limits applied
 19:                    0 - Unblocked Publications
 20:                    0 - Unblocked Control Commands
 21:                    0 - Possible TTL Asymmetry
 22:                    0 - ControllableIdleStrategy status
 23:                    1 - recv-channel: aeron:udp?endpoint=ip-172-31-10-77:25520
 24:        1,627,656,896 - snd-pos: 8<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 25:                    1 - send-channel: aeron:udp?endpoint=ip-172-31-8-204:25520
 26:        1,636,045,504 - pub-lmt: 8<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 27:              170,432 - sub-pos: 1 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520 @0
 28:              170,432 - rcv-hwm: 7 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520
 29:           19,750,176 - sub-pos: 2 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520 @0
 30:           19,750,176 - rcv-hwm: 9 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 31:           19,750,176 - rcv-pos: 9 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 32:           28,953,216 - pub-lmt: 10<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520
 33:           20,564,608 - snd-pos: 10<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520

node-2
  0:           20,086,208 - Bytes sent
  1:           20,681,152 - Bytes received
  2:                    0 - Failed offers to ReceiverProxy
  3:                    0 - Failed offers to SenderProxy
  4:                    0 - Failed offers to DriverConductorProxy
  5:                4,917 - NAKs sent
  6:                   16 - NAKs received
  7:                3,282 - Status Messages sent
  8:                3,320 - Status Messages received
  9:                5,312 - Heartbeats sent
 10:                5,657 - Heartbeats received
 11:                   16 - Retransmits sent
 12:                    0 - Flow control under runs
 13:                    0 - Flow control over runs
 14:                    0 - Invalid packets
 15:                    0 - Errors
 16:                    0 - Short sends
 17:                  599 - Client keep-alives
 18:                    2 - Sender flow control limits applied
 19:                    0 - Unblocked Publications
 20:                    0 - Unblocked Control Commands
 21:                    0 - Possible TTL Asymmetry
 22:                    0 - ControllableIdleStrategy status
 23:                    1 - recv-channel: aeron:udp?endpoint=ip-172-31-8-204:25520
 24:                    1 - send-channel: aeron:udp?endpoint=ip-172-31-10-77:25520
 25:            8,559,040 - pub-lmt: 3 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520
 26:              170,432 - snd-pos: 3 434696915 1 aeron:udp?endpoint=ip-172-31-10-77:25520
 27:           28,138,784 - pub-lmt: 4 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 28:           19,750,176 - snd-pos: 4 434696916 2 aeron:udp?endpoint=ip-172-31-10-77:25520
 29:                2,176 - sub-pos: 1<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520 @0
 30:        1,627,656,896 - rcv-hwm: 5<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 31:                2,176 - rcv-pos: 5<PHONE_NUMBER> 1 aeron:udp?endpoint=ip-172-31-8-204:25520
 32:           20,500,992 - sub-pos: 2<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520 @0
 33:           20,500,992 - rcv-hwm: 6<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520
 34:           20,500,992 - rcv-pos: 6<PHONE_NUMBER> 2 aeron:udp?endpoint=ip-172-31-8-204:25520

I don't see how the rcv-pos or sub-pos can get so far behind the rcv-hwm.  No flow control over or under runs so this would have been rejected. It is like something corrupted the counters. The image should have the latest contents.

@mjpt777 agreed. The only way snd-pos increments is via sending data. And as there is no way I can see sending that much data because of the byte counts, the snd-pos, hwm, and pub-lmt must be invalid somehow.

For the Publication logbuffer. Data is 0 until offset 2176 (position 2176). Which happens to be the sub-pos and rcv-pos. For a publication, this seems very very unusual normally.
For the Image logbuffer, there is data up to position 2176, then it is 0.
One possibility is that snd-pos was corrupted. If it got corrupted, then it would update pub-lmt with a bad value. Also, sending would use that position for scanning and might send junk (depending where it was). Wouldn't quite explain why rcv-hwm is at that value, though. But whatever hit snd-pos could hit rcv-hwm and corrupt it separately.... but it is a separate machine and file.
The 0 of the data in the publication might be explained by the snd-pos. Cleaning might do that if snd-pos was corrupted.

@patriknw is the stat values being read (or written to) by the app? Anything that could touch the cnc file?

@mjpt777 BTW, the rcv-hwm is incremented on a heartbeat irregardless of overrun. So, it would be possible to have a heartbeat like message come in that sets the rcv-hwm up to some value.

We should probably filter that out as an overrun.

Probably. But it's an orthogonal issue. The corruption of snd-pos is problematic.

Looking at the code I think we should validate status messages for flow control to be in a valid range.

Sure. Can validate that the position is one that was sent. I assume you mean on the Publication side processing an SM. Although, not seeing how a bad SM could effect the snd-pos like what is observed. The Publication logbuffer has data from 2,176 to 317,632 and is 0 afterward. A bad SM could allow more to be sent, but not seeing how it could change snd-pos directly.

There is no counter for NAKs that are filtered by being outside of a termLength. Most of the NAKs received in the stats are filtered because of snd-pos being off by so much.

something else that is "odd" is that there is no rcv-pos on node-2 for the image with sessionId 434696915. The image logbuffer is there, though. And it has data up to the hwm, which is 170,432.
Now, it could be that the snapshot was taken just at the moment that the image was being closed and the rcv-pos was removed. But it is odd that it is not there.

I don't use reservedValue or tryClaim. Only offer and poll.
I read the stats from a separate process with aeron-samples AeronStats. We are not touching the cnc file, apart from that we poll it periodically for log errors using (a copy) of AeronErrorLog.
The media driver is started embedded, if that matters.
Could it be something with that I continue to offer (in busy-wait loop) even though I get NOT_CONNECTED?
The returned positions from offer on node1 looks good (to me):
[09/29/2016 03:38:37.837]  AeronSink offer result [-1] connected [false] closed [false]
[09/29/2016 03:39:25.376]  AeronSink offer result [-1] connected [false] closed [false]
[09/29/2016 03:39:25.444]  AeronSink offer result [320] connected [true] closed [false]
[09/29/2016 03:39:25.452]  AeronSink offer result [448] connected [true] closed [false]
[09/29/2016 03:39:26.584]  AeronSink offer result [576] connected [true] closed [false]
[09/29/2016 03:39:26.643]  AeronSink offer result [768] connected [true] closed [false]
[09/29/2016 03:39:26.643]  AeronSink offer result [1152] connected [true] closed [false]
[09/29/2016 03:39:26.647]  AeronSink offer result [1280] connected [true] closed [false]
[09/29/2016 03:39:27.053]  AeronSink offer result [1664] connected [true] closed [false]
[09/29/2016 03:39:27.643]  AeronSink offer result [2048] connected [true] closed [false]
[09/29/2016 03:39:27.738]  AeronSink offer result [2176] connected [true] closed [false]

...

[09/29/2016 03:44:26.642]  AeronSink offer result [315776] connected [true] closed [false]
[09/29/2016 03:44:27.054]  AeronSink offer result [316160] connected [true] closed [false]
[09/29/2016 03:44:27.067]  AeronSink offer result [316288] connected [true] closed [false]
[09/29/2016 03:44:27.642]  AeronSink offer result [316672] connected [true] closed [false]
[09/29/2016 03:44:28.642]  AeronSink offer result [316864] connected [true] closed [false]
[09/29/2016 03:44:28.642]  AeronSink offer result [317248] connected [true] closed [false]
[09/29/2016 03:44:29.642]  AeronSink offer result [317632] connected [true] closed [false]

@patriknw in the test, does the application create and release/close any other Publications or Subscriptions not shown in the stats?

I didn't use any filters apart from type=[1-9] with AeronStat.
However, node1 is trying to send to node2 before node2 has been started and after a while we give up that and close the Publication. I see in the logs that this happened in this test run. At 03:38:57. node2 is started 03:39:21 (clocks are in sync within 1 sec).
So there was one Publication before the things you see in the stats here.


Basic PMU profiling by PID

Given a list of PIDs and a list of PMU events, profile their values in the past time interval.

Put this here before we forgot, the libpfm does not work well initializing the file descriptors for zombie processes.

I tested the code on the server, and it seems the function always fails after 3 iterations. Here is the dump from gdb.
Program received signal SIGSEGV, Segmentation fault.
__GI___libc_free (mem=0x15) at malloc.c:2970
2970    malloc.c: No such file or directory.
(gdb) bt
#0  __GI___libc_free (mem=0x15) at malloc.c:2970
#1  0x00007ffff7ad88aa in pfm_perf_terminate (this=0x7ffff7dd07e0) at pfmlib_perf_event_pmu.c:793
#2  0x00007ffff7ad46c6 in pfm_terminate () at pfmlib_common.c:795
#3  0x00000000004037b8 in get_pmu_sample ()
#4  0x0000000000401395 in main ()

There is an issue with multi-threaded / multi-processed workloads, that the inherit flag can only work when the new threads or processes are invoked during profiling. For example, if a fork() happens at time t_0, we are not able to get the PMU profiling of the child process at a later time t_1 due to the fact that we are creating new fds every sample interval.
However, it seems perf stat works fine in this case, so maybe we can look into their implementation.


Permit all users to make analytics requests

A previous change (from card) removed the ability to access this feature from all users and instead only permitted those with 'single point of access' and 'user managers' permissions. This commit reinstates the ability of all users to make analytics requests following feedback from the support team.

:+1:


Implement XEP-0084 (User Avatar)

This PR allows go-xmpp to request a user's avatar data or metadata and also react to avatar notifications via PubSub.
This PR was tested with a modified example that basically just requests a subscription to the metadata node, requests the metadata and upon receiving it, request the avatar data.
It should be noted (Perhaps I can write a comment somehwhere in another commit?) that notifications via PubSub can only be received if a presence subscription exists. Though it is stated in XEP-0060, it is - at least for me - non-obvious if you havent read the specification.
This PR closes #118

Thank you

Thank you


Clear environment does not call listeners when rmtree fails

environment_clear does not call listeners when rmtree fails. This commonly fails when someone manually as the root user did a git pull on a module.
When the environment is deleted in core, it should first call all listeners before it tries to delete the code tree.

Hey team! Please add your planning poker estimate with ZenHub @sanderr @arnaudsjs @FloLey @bartv


Add New DataFrame

Refactor Array as list-based
Refactor DataFrame as a multi-array data container
Add DFIterator for DataFrame iteration

Codecov Report

Merging #259 (5294447) into main (ca3685f) will increase coverage by 0.27%.
The diff coverage is 82.12%.

@@            Coverage Diff             @@
##             main     #259      +/-   ##
==========================================
+ Coverage   70.75%   71.03%   +0.27%
==========================================
  Files         171      174       +3
  Lines        5834     6018     +184
  Branches      600      629      +29
==========================================
+ Hits         4128     4275     +147
- Misses       1500     1523      +23
- Partials      206      220      +14

Impacted Files
Coverage Δ

towhee/dataframe/dataframe_v2.py
71.42% <71.42%> (ø)

towhee/array/_array_ref.py
75.00% <75.00%> (ø)

towhee/array/array.py
81.81% <79.54%> (+4.04%)
:arrow_up:

towhee/array/utils.py
100.00% <100.00%> (ø)

towhee/tests/array/test_array.py
100.00% <100.00%> (ø)

towhee/tests/dataframe/test_dataframe_v2.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Need an ability to generate style bundle separatelly under a directory

Which @angular/* package(s) are relevant/related to the feature request?
compiler
Description
Hello Friends,
After some research and reading the official documentation, I couldn't find any helpful information and I think it is not possible right now, to build the style files as a separate bundle under the subdirectory via the configuration in angular.json.
So let's say we have 2-3 theme CSS files

default-theme.scss
dark-theme.scss
system-theme.scss

I want to build these scss files as a 3 separate bundles, fortunately it is possible via the angular.json configuration. I can add several lines and it will be ready:
angular.json: projects.{PROJECT_NAME}.architect.build.options.styles
...............................
"styles": [
         {
                "inject": false,
                "input": "src/themes/default-theme.scss",
          },
          {
                "inject": false,
                "input": "src/themes/dark-theme.scss",
          },
          {
                "inject": false,
                "input": "src/themes/system-theme.scss",
          },
]
...............................

and this will generate 3 separate files in the root directory in the dist/{PROJECT_NAME}/.

default-theme.css
dark-theme.css
system-theme.css

I also noticed that the Style Object has one more additional property: bundleName which will help us to change the generated bundle name(which is good).
But unfortunately, there is no way to generate these bundles under the sub-directory.
Proposed solution
So it will be helpfull to modify the style object, by adding output property, that will get the path(string) as a value:
angular.json: projects.{PROJECT_NAME}.architect.build.options.styles
...............................
"styles": [
         {
                "inject": false,
                "input": "src/themes/default-theme.scss",
                "output": "themes",
                "bundleName": "default-theme"
          },
          {
                "inject": false,
                "input": "src/themes/dark-theme.scss",
                "output": "themes",
                "bundleName": "dark-theme"
          },
          {
                "inject": false,
                "input": "src/themes/system-theme.scss",
                "output": "themes",
                "bundleName": "system-theme"
          },
]
...............................

and by these configuration I expect to have, my generated bundles under the themes directory

themes/default-theme.css
themes/dark-theme.css
themes/system-theme.css

Alternatives considered
The second option is to have some standard objects for example:
$styles, $assets by which we can have access to the generated styles and assets to manipulate with them in angular.json
For example, by using the above example:
angular.json: projects.{PROJECT_NAME}.architect.build.options.styles
...............................
"styles": [
         {
                "inject": false,
                "input": "src/themes/default-theme.scss",
          },
          {
                "inject": false,
                "input": "src/themes/dark-theme.scss",
          },
          {
                "inject": false,
                "input": "src/themes/system-theme.scss",
          },
]
...............................

We can use assets object to change the directories:
angular.json: projects.{PROJECT_NAME}.architect.build.options.assets
...............................
"assets": [
         {
                "input": "$styles['default-theme.css']",
                "output": "themes/default-theme.css",
                "glob": "compiled"                 <------------ this is also can be modified a little bit to indicate that the file is not static
          },
         {
                "input": "$styles['dark-theme.css']",
                "output": "themes/dark-theme.css",
                "glob": "compiled"                 <------------ this is also can be modified a little bit to indicate that the file is not static
          },
                   {
                "input": "$styles['system-theme.css']",
                "output": "themes/system-theme.css",
                "glob": "compiled"                 <------------ this is also can be modified a little bit to indicate that the file is not static
          },
]
...............................

What is the use-case the these stylesheet must be under a different directly?

@alan-agius4 Thanks for the reply.
Actually, this is just an example for better understanding. the problem is a little bit different and connected with the server configurations.
In the development mode I am changing the assets directories, to make it similar to the server environment, as in the server angular is configured under the folder with the nginx redirectons. It will be really hard to explain the server configurations, that's why I have explained by using this simple example.
Best,
Ashot

Why are themes under themes/ on the server? I would think that if styles are bundled as part Angular CLI, it would make more sense to use the paths Angular is putting them at and aligning the production server to match. You could always move the themes post-build, but that would likely be annoying to orchestrate and break ng serve.
Since the challenge is a disagreement about file location between Angular (which wants to put them in the root) and your production server (which wants them under themes/), why is aligning on the themes/ directory better? Is there a reason it is difficult or impractical to avoid a themes/ directory on the server?

Hello @dgp1130,
As I said previously, the issue is a bit complicated and is not connected only with themes. The whole angular project is under the folder and the angular project is accessible by the nginx redirections, and this is the main problem.
So why one style bundle is not working in our case:
In the project, we are using angular material, and the initial loading is important for us. Each theme bundle is ~80kb, so if we build 1 style bundle in the case of 3 themes the bundle will be ~240kb which is not good, in the case of separate bundles each theme bundle will be 80kb, so that's why we decide to proceed with this solution.
Thanks

I get not wanting multiple style bundles, using different bundles for each theme and only downloading the one you want is pretty reasonable.
I'm less clear on what a separate themes/ folder is desirable in the first place? Do you just want to do this because you feel it is better organized or is there a deeper configuration challenge with the server? I would assume it is possible for the server to serve or redirect at the paths Angular is generating, is that not feasible for some technical reason?

@dgp1130 no no, I don't want this change because of better organization. It is connected with the server configuration and the local development/testing: Let me clarify a bit the server configuration, it is a little bit messy but I hope it will be helpfull.
On the server the whole angular project is under folder let's say under folder /angular and by nginx configurations the requests are navigation to take the angular resources from the /angular folder, so for all of the images and all of the assets resources I am using the /angular as a prefix to be able to take the static resources from the assets. So everything is working as expected, but I have some additional logic for all of the static assets to generate an additional folder(/angular) for development/testing and  I am generation that by using the configuration for assets:
 {
                  "input": "projects/alison-core/src/lib/styles/fonts",
                  "output": "angular/assets/web-components/fonts",
                  "glob": "**/*"
                },
                {
                  "input": "src/assets",
                  "output": "angular/assets",
                  "glob": "**/*"
                }

this is generating:
other-bundles .......
/angular
    /assets/..................
    /assets/web-components/fonts

Thanks

This case is quite particular use-case, and as of now, there isn't a plan to support the configuration of were styles are outputted.
Let's reassess this in the future if there's increased demand for this feature from users.


Some pages say "the repository’s default branch (usually master)" despite it being "main" more often now.

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/rest/repos/contents#download-a-repository-archive-zip
What part(s) of the article would you like to see updated?
The docs say "The branch name. Default: the repository’s default branch (usually master)" despite a new repo having instructions that would lead to a "main" branch being created instead.
Additional information
Apparently this is from the OpenAPI definition so it can't be changed in this repository.

@HimbeersaftLP Thanks for opening this issue and submitting a PR. Any changes in the rest description have to be done internally. Thanks for bringing this to our attention! ✨


which vsphere ,vcentre machine is created??

{
"type": "vmware-iso",
"iso_url": "http://old-releases.ubuntu.com/releases/precise/ubuntu-12.04.2-server-amd64.iso",
"iso_checksum": "af5f788aee1b32c4b2634734309cc9e9",
"iso_checksum_type": "md5",
"ssh_username": "packer",
"ssh_password": "packer",
"shutdown_command": "shutdown -P now"
}

We have downloaded the iso image from website, found the iso checksum....using md5sum file.iso

I dont understand wer is the ubuntu vm created on which silo or which datacentre.,,Can you help me these details using the code. need to built an image using packer on vsphere.

use this script
{
"variables": {
"myaccess": "xxxxxxxxxxxxxxxxxxxxx",
"mysecrete": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
},
"builders": [
{
"type": "amazon-ebs",
"region": "us-west-2",
"instance_type": "t2.micro",
"ami_name": "packer-rehl",
"source_ami": "ami-079596bf7a949ddf8",
"access_key": "{{user myaccess}}",
"secret_key": "{{user mysecrete}}",
"ssh_username": "ec2-user"
},
{
"type": "hyperv-iso",
"iso_url": "http://releases.ubuntu.com/12.04/ubuntu-12.04.5-server-amd64.iso",
"iso_checksum": "769474248a3897f4865817446f9a4a53",
"iso_checksum_type": "md5",
"ssh_username": "packer",
"ssh_password": "packer",
"shutdown_command": "echo 'packer' | sudo -S shutdown -P now"
}
],
"provisioners": [
{
"type": "shell",
"inline": [
"sudo yum update -y",
"sudo yum install git -y",
"sudo yum install tree -y"
]
}
]
}

I dont understand wer is the ubuntu vm created on which silo or which datacentre.,,Can you help me these details using the code. need to built an image using packer on vsphere.

The builder builds a virtual machine by creating a new virtual machine from scratch, booting it, installing an OS, provisioning software within the OS, then shutting it down. The result of the VMware builder is a directory containing all the files necessary to run the virtual machine

But here why did u include amazon ami and zones of ami , which is not related to vmware

"type": "vmware-iso",
"iso_url": "http://old-releases.ubuntu.com/releases/precise/ubuntu-12.04.2-server-amd64.iso",
"iso_checksum": "af5f788aee1b32c4b2634734309cc9e9",
"iso_checksum_type": "md5",
"ssh_username": "packer",
"ssh_password": "packer",
"shutdown_command": "shutdown -P now"
for this script , i need to know wer is the vm getting deployed on which silo, which datacentre.

Can you add more attributes to my code, so that can demonstrate as bright idea in office..I want to explain about packer that it s trying to create an image into some x datacentre , y silo, z host,,Thats my moto.

"type": "vmware-iso",
"iso_url": "http://old-releases.ubuntu.com/releases/precise/ubuntu-12.04.2-server-amd64.iso",
"iso_checksum": "af5f788aee1b32c4b2634734309cc9e9",
"iso_checksum_type": "md5",
"ssh_username": "packer",
"ssh_password": "packer",
"shutdown_command": "shutdown -P now"
for this script , i need to know wer is the vm getting deployed on which silo, which datacentre.

you can find it in vm datasource

Hey Anoop if you are using vm ware workstation,hyper-v, virtual box the image is created locally and if it VMware exi it will store it in your exi server where you can log in


getFlagList RPC creates null service objects

Calling getFlagList() returns a FlagList object. flagList.flags returns a list of nulls instead of a list of Flags.
[null, null, ... null, null]

Looks like the Flag object doesn't have a type property so it isn't being parsed properly. We'll need to update the vm_service package to not try to create a service object for each element in FlagList but just create the Flag objects directly using its constructor. I'll make that change once the vm_service package lands in the SDK.

Fixed here.


Update to TypeScript 5.5

Hi. Unless I'm mistaken, using TypeScript 5.5 should allow importing the library properly. As described here: Easier API Consumption from ECMAScript Modules.
Because I have the same issue as described in this PR: microsoft/TypeScript#57133 when using this library.
Some other dependencies, including RBush, have updates as well.
None of the updates should require any significant changes, if any.

will look into this ASAP thx @Ewok167

@Ewok167 done please test

check since v9.9.0

@Ewok167 done please test

Thanks. But it doesn't seem to have fixed my issue, hmm. Were you already aware of some problems with importing this lib?
I use esbuild to build for browser, I tried esm and cjs formats but the same thing happened. TS says the imports are good and there is no default import, however after build it is the vice-versa. I can do import abc from "detect-collisions" and abc is an object containing all the exports. I can't import anything specifically. Only this library has had issues with my esbuild setup, I will try to get a simple example to test.

@Ewok167 how do you import the library?
import { System } from 'detect-collisions';
should normally work
what error you get?
can you create a minimal example repository I can look at?
I'm reopening the issue since you say it's not done yet
mind you, the library is in typescript in src and the dist folder is compiled tsc compiled 5.5 into
https://github.com/Prozi/detect-collisions/blob/master/tsconfig.json#L12
target: es2015
it should work totally in any environment node webpack rollup browser as module import whatever

@Ewok167  I have no clue why in your project
import { System } from 'detect-collisions'

doesn't work, maybe the problem is elsewhere in your code

why?
https://github.com/Prozi/detect-collisions/blob/master/package.json#L5
https://github.com/Prozi/detect-collisions/blob/master/dist/index.js#L40
https://github.com/Prozi/detect-collisions/blob/master/dist/system.js#L181
that's why

closing as I cannot reproduce so I wont fix
and you @Ewok167 didnt provide any example repository


Set Process Refresh

Hi Demian,
This is an enhancement request for the SetProcess tool.
There is an out of the box SetProcess tool available that works in a similar way to your one. They both have an issue as opposed to say setting the active process via JavaScript.
When the process is set, the user has to refresh their screen in order to see that the business process flow has changed. I am using this in a real time workflow that triggers when a field on the record changes.
Would it be possible to include an optional switch to perform a form refresh after the process has been set?
I know this is not ideal but it is preferable to me than needing the user to remember to refresh the screen. This feature would be a real differentiator from the out of the box Action.
(It would be better if the new business process flow showed up immediately in a similar way to when it is changed manually from the ribbon at Process-->Switch Process, but I'm not sure if that would be possible...)
Thanks,
Sean
CRM Version 8.2 - onpremise - WorkflowTools version <IP_ADDRESS>

Hi Sean,
is a good idea, but I cannot trigger a refresh (in Javascript) from the workflow.
kind regards,

Hi Demian, yes I thought as much, ended up triggering the refresh myself in JavaScript.


FR: Could we get sensors for high and low temps and a description of what the weather will look like?

Hi,
First off, thanks for a great integration, it's serving me super well, and I find the forecasts to be a lot more accurate than any of the other weather integrations (and I've tried them all).  As I am slowly preparing for the end of the dark sky integration, and this works better anyway, I thought I'd hit you up for a few more sensors if possible. I use these to control my heating system, so the increased accuracy from this integraiton would be super useful. A few of the ones that dark sky provides that would help me a lot are these ones:

a sensor for the high temperature in today's forecast
a sensor for the low temperature in today's forecast
a description of the general weather conditions for the day (in the dark sky integration, this is shown as the icon, but I find it very useful as a single variable to control automations too). The possible options are
'partly-cloudy-day', 'rain', 'snow', 'sleet', 'wind', 'fog', 'cloudy','clear-day'

Thanks for the consideration.

While you wait for me to do this, you can actually really easy make these three sensors as template sensors yourself. I have added and example below:
- name: Temperature Day High
   unique_id: some_unique_id
   icon: mdi:thermometer
   state: >
      {{ "%0.1f" | format(state_attr('weather.weatherflow_day_based_forecast', 'forecast')[0].temperature | float(default=0) ) }}
- name: Temperature Day Low
   unique_id: some_unique_id
   icon: mdi:thermometer
   state: >
      {{ "%0.1f" | format(state_attr('weather.weatherflow_day_based_forecast', 'forecast')[0].templow | float(default=0) ) }}
- name: Current Condition
   unique_id: some_unique_id
   icon: mdi:weather-partly-cloudy
   state: >
      {{ state_attr('weather.weatherflow_day_based_forecast', 'forecast')[0].condition }}

I have decided not to implement this. I suggest you use the above method, to get the sensors.
Sorry.

I did that already before submitting the FR, but I still think it should be a standard sensor as it's something everyone needs on their weather forecast card. Not all users are as good with templates as I am, and many weather forecast cards require this sensor.


samples: subsys: display: lvgl: rename and move to samples/modules directory

Currently LVGL samples exists in samples/subsys/display/lvgl and samples/modules/lvgl
In this PR I rename the sample samples/subsys/display/lvgl to samples/subsys/display/lvgl_input then move it to the samples/modules/lvgl directory.

Hi @faxe1008,
I guess a better name would be input or input_device? What is your preference? I will clean up the other mentions and also put it all into one commit.
Thanks

I'd go with "input_devices" (plural) as the folder name, and "lvgl-input-devices" for the code-sample identifier

Hi @faxe1008,
I guess a better name would be input or input_device? What is your preference? I will clean up the other mentions and also put it all into one commit.
Thanks

I'd go with "input_devices" (plural) as the folder name, and "lvgl-input-devices" for the code-sample identifier

Yup 👍 for that.

The question is why other samples started to appear in the modules directory.

The reason why the new demo sample was situated there is that it showcases the demos of the lvgl module. I think that its sensible to have the demos for a module inside of samples/modules/<module>. Other modules do this as well, eg. the canopennode sample does not live inside the samples/subsys/canbus folder.
Also, I think the samples/subsys/display is a unfortunate place for it since the display is not really a subsystem.

I don't agree with the fact that LVGL doesn't belong to samples/modules -- it happens that samples that are there today are only for optional modules, true, but they became optional pretty recently, and samples/modules was deemed a good home for them when they were not optional.
The "problem" here I guess is more that maybe neither samples/subsys/display nor samples/modules are perfect homes for something like LVGL, hence why we are having this "former location is not ideal, but it's been like this for a long time so let's keep it", vs. "ya but LVGL is effectively a module :)" conversation.


feat: add dockerfile

Containerizing intermodal makes it easier to use without installing it.
The docker image can be hosted on ghcr.io and its build automated via github action:
https://docs.github.com/en/actions/publishing-packages/publishing-docker-images
The container can then be used like this:
docker run --rm --init --interactive --tty --volume "$(pwd)":/mnt ghcr.io/casey/intermodal torrent show /mnt/example.torrent

@casey I've simplified the dockerfile to use the official docker library rust image as the build stage. The drawback with this approach is that any change in the repository invalidates the docker cache and results in a full rebuild. In the end it shouldn't matter if the image is only built once per release.

Gotcha, in that case distroless sounds good. I probably won't wind up publishing the docker image myself, I avoid managing package repo packages myself because it's extra work to remember then releasing, so for now let's remove the reference to docker pull ghcr.io/casey/intermodal (since I suppose that would mean I would have to publish it)

@casey I've removed the reference to ghcr in https://github.com/casey/intermodal/pull/533/commits/e4cf429083a8e0f3c637dabcfd4e61b6ce0c7871
In case you'd like to publish it automatically in the future it can be done easily with a github action as described in https://docs.github.com/en/actions/publishing-packages/publishing-docker-images#publishing-images-to-github-packages

Looks good to me! I changed /app to /usr/local/src, which is a well-known location for source files. Also, I removed the AS runtime, since as far as I can tell, it's not necessary to name a layer if it's not referred to in the future. I also added a just recipe to build and run the image using podman. I'm on macos, and podman is nice because it doesn't require root.

The build failure is unrelated to this PR. I'll fix it in another PR and then merge this.

Blocked on #537.

Merged, thanks for the PR!