Compatibility Issue - pygrib/pyhdf/netCDF4

Running into an issue here that I can't imagine is that uncommon given the large number of users in the meteorological community. I work with datasets between each of the .nc, .grib and .hdf4 formats and it seems an oversight that there is non-conformity between this dependency for the three - this wasn't an issue in pygrib 1.9.9 as far as I'm aware (used to have all 3 playing nice):
In [1]: import pygrib
ImportError                               Traceback (most recent call last)
 in ()
----> 1 import pygrib
ImportError: libjpeg.so.9: cannot open shared object file: No such file or directory
Now - following a standard conda-forge install for pygrib you end up with jpeg9b which provides the above dependency issue. If you do this however, you get the same error for netCDF4 and pyhdf except for libjpeg.so.8. Not sure if there is a good resolution for this as I haven't been able to find out, but would prefer to be able to work between the formats as much as possible without relying on the dependency mess that is PyNIO.

Seems like a problem with the dependency tracking in the conda packages, not the python modules themselves.

Seems like a problem with the dependency tracking in the conda packages, not the python modules themselves.

Indeed.

Now - following a standard conda-forge install for pygrib you end up with jpeg9b which provides the above dependency issue. If you do this however, you get the same error for netCDF4 and pyhdf except for libjpeg.so.8. Not sure if there is a good resolution for this as I haven't been able to find out, but would prefer to be able to work between the formats as much as possible without relying on the dependency mess that is PyNIO.

conda-forge uses jpeg 9* while defaults uses jpeg 8*. To avoid conflicts we recommend you to read the docs (still un-merged) and follow the best practices there.
PS: please open issues like that in conda-forge feedstocks for the package in question.


[bug] Cannot install kube-prometheus-stack via https://*.kubernetes.omni.siderolabs.io

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Installing kube-prometheus-stack Helm chart fails with Error: create: failed to create: the server responded with the status code 413 but did not return more information (post secrets).
Here is the full error (when running with --debug):
history.go:56: [debug] getting history for release kube-prometheus-stack
Release "kube-prometheus-stack" does not exist. Installing it now.
install.go:200: [debug] Original chart version: ""
install.go:217: [debug] CHART PATH: /Users/gerhard/Library/Caches/helm/repository/kube-prometheus-stack-45.23.0.tgz

client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD alertmanagerconfigs.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD alertmanagers.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD podmonitors.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD probes.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD prometheuses.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD prometheusrules.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD servicemonitors.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
install.go:160: [debug] CRD thanosrulers.monitoring.coreos.com is already present. Skipping.
client.go:134: [debug] creating 1 resource(s)
Error: create: failed to create: the server responded with the status code 413 but did not return more information (post secrets)
helm.go:84: [debug] the server responded with the status code 413 but did not return more information (post secrets)
create: failed to create
helm.sh/helm/v3/pkg/storage/driver.(*Secrets).Create
	helm.sh/helm/v3/pkg/storage/driver/secrets.go:164
helm.sh/helm/v3/pkg/storage.(*Storage).Create
	helm.sh/helm/v3/pkg/storage/storage.go:69
helm.sh/helm/v3/pkg/action.(*Install).RunWithContext
	helm.sh/helm/v3/pkg/action/install.go:365
main.runInstall
	helm.sh/helm/v3/cmd/helm/install.go:286
main.newUpgradeCmd.func2
	helm.sh/helm/v3/cmd/helm/upgrade.go:130
github.com/spf13/cobra.(*Command).execute
	github.com/spf13/cobra@v1.6.1/command.go:916
github.com/spf13/cobra.(*Command).ExecuteC
	github.com/spf13/cobra@v1.6.1/command.go:1044
github.com/spf13/cobra.(*Command).Execute
	github.com/spf13/cobra@v1.6.1/command.go:968
main.main
	helm.sh/helm/v3/cmd/helm/helm.go:83
runtime.main
	runtime/proc.go:250
runtime.goexit
	runtime/asm_amd64.s:1598

I suspect that this is the proxy serving https://*.kubernetes.omni.siderolabs.io which is limiting uploads. FWIW https://github.com/prometheus-community/helm-charts/issues/3205
Expected Behavior
Installing kube-prometheus-stack Helm chart via https://*.kubernetes.omni.siderolabs.io should just work.
Steps To Reproduce
helm repo add prometheus-community https://prometheus-community.github.io/helm-charts
helm repo update

helm upgrade kube-prometheus-stack prometheus-community/kube-prometheus-stack \
  --install --debug \
  --namespace kube-prometheus-stack --create-namespace

What browsers are you seeing the problem on?
No response
Anything else?
No response

Thanks for reporting this, the fix is coming.

This problem should be resolved now.

OK! I have updated the node to Talos v1.4.1. Will report back when I have tested the new behaviour.

This worked as expected on both clusters, one running Talos v1.4.0 & the other one running v1.4.1.
Both clusters managed by Omni v0.8.1.
Closing - thank you! 💪


future-only tag is not working

<input ng-model="date1" datetime-picker date-only future-only />
In above example future-only is not working.

@agarwalankit95 future-only tag Fixed, have a look at forked project
https://github.com/mani-s17/angularjs-datetime-picker/commit/ffef86128d84dc113b9a7789fc07ce91894ad8c8
or pull request
https://github.com/kineticsocial/angularjs-datetime-picker/pull/36


[FLINK-2961] [table] Add support for basic type Date in Table API

Currently, the basic type Date is not implemented in the Table API. In order to have a mapping of the most important ANSI SQL types for FLINK-2099. It makes sense to add support for Date to represent date, time and timestamps of milliseconds precision. This PR implements FLINK-2961.

So all timestamps have to be in UTC timezone. Isn't this a bit limiting? Could it per default take the system timezone and allow to override the timezone in a cast statement?

I also also thought about that but I'm pretty sure that this introduces issues difficult to debug. I think the most important thing is that the conversion (from string to date and back) is always constant. Internally date is always UTC anyway. Introducing a .convert() function for that might be better. My goal was primarily to have this data type in the Table API.

It seems that nullCheck has never been true. The code generator had lots of bugs :D

I think the best solution (also for future features like the SQL API) is a TableConfig, where TimeZone, Locales, and also the nullCheck can be configured. Once #1237 is merged (hopefully soon), I will implement that into the TableEnvironment. What do you think?

I have updated my code again. Internally I already implemented a TableConfig that defines the TimeZone and NullCheck. Once #1237 is merged, I will make the config available through TableEnvironment as part of FLINK-3068. What do you think?

Looks good now.

I will merge this later today, if nobody has objections.

Changes Unknown when pulling<PHONE_NUMBER>017fc188882234fe38ebf6b3c58b84 on twalthr:TableApiDate into ** on apache:master**.


requestLayout() improperly called

requestLayout() improperly called by com.turingtechnologies.materialscrollbar.AlphabetIndicator{9626e1b I.E...... ......ID 1084,0-1346,262} during layout: running second layout pass
I am getting this issue when scrolling through the recyclerView.

Does it mention which lone is causing the problem?

No only this.

I'll need more context in order to debug this issue. (sorry for the delay)


Vulkan needs to correctly start even if someone doesn't have the validation layers locally

Describe the bug
Currently - if you try to run the project without validation layers, the engine will fail to create a vulkan instance. Investigate ways of disabling validation layers if they can't be found.

Resolved - Vulkan Version in vulkan instance creation was not correct


Scale images

Images such as the action button needs to be scaled properly

This has to do with the X symbol.  Also, this might have to do with the zoom level of the sqaures

This will be irrelevant when #34 is accomplished, which also relies on the architecture of #35


Unexpected c201C and c201D characters surrounding string literal  content

Issue Summary
When rendering string literal content MathJax is unexpectedly adding quotation marks (c201C ... c201D) around it. For example, when rendering this example from 3.2.6 String Literal  in MathML Core:
<math>
  <mi>s</mi>
  <mo>=</mo>
  <ms>"hello world"</ms>
</math>

MathJax version: 4.0.0-beta.6 renders it as follows (note the dueling quotation marks):

Expected Result
I'm guessing that the expected result is to render it as shown in the MathML Core spec (Where only one
set of quotes are shown - those supplied as part of the string literal):

Also, here is how the above mathML renders natively in Edge which behaves as the example in the MathML Core spec:

Steps to Reproduce:

Create MathML content with a string literal ....
Render it in a web page using MathJax 4.0.0-beta6
Note that MathJax adds double quotes to the string literal when it renders it.

Why do you consider this to be a bug?
MathJax's output looks sloppy if the expression author included beginning/ending quotation marks as part of the string literal and the result is a doubly quoted string. Also, MathJax appears to remove the author's ability to have a string literal displayed w/o quotation marks.
Technical details:

MathJax Version: 4.0.0-beta.6
OS Version:
-- OS Name: Microsoft Windows 11 Home
-- Version: 10.0.22631 Build 22631
-- Locale: United States
Browser: Microsoft Edge Version 126.0.2592.113 (Official build) (64-bit)

I am not manually configuring MathJax. I am loading MathJax via:
<script id="MathJax-script" async<EMAIL_ADDRESS>
Supporting information:
Here are two simple pages demonstrating the issue (one with MathJax, one without MathJax):

string_literal_ms_mathjax_4
string_literal_ms_no_mathjax

MathJax implements the MathML3 specification, whose description of the <ms> element states

By default, string literals are displayed surrounded by double quotes, with no extra spacing added around the string. ... For example, <ms>double quote is "</ms> might be rendered as "double quote is \"".

MathML Core is a newer, but more limited, standard, and it does not include a number of features from MathML3 that MathJax relies on.  It is unlikely that MathJax will implement MathML Core; instead, we are more likely to move to MathML4, whose description of <ms> is the same as that in MathML3.
The MathML Core section you link to above is followed by a note that indicates

In MathML3, it was possible to use the lquote and rquote attributes to respectively specify the strings to use as opening and closing quotes. These are no longer supported and the quotes must instead be specified as part of the text of the <ms> element.

But the MathML4 spec says that it should be a superset of MathML Core, so this seems to be an inconsistency with that statement.  I'm not sure which is to be considered authoritative for this.
You say that

MathJax appears to remove the author's ability to have a string literal displayed w/o quotation marks.

but you can use the lquote and rquote attributes of the <ms> element (available in MathML3 but not MathML Core) to specify empty strings for the quotes, as in
<math>
  <mi>s</mi>
  <mo>=</mo>
  <ms lquote="" rquote="">"hello world"</ms>
</math>

for example.  Using these attributes in MathML Core should not hurt anything, if you need to be compatible with both.
It is also possible to set the default values for the lquote and rquote attributes in MathJax to be empty strings with the following MathJax configuration:
MathJax = {
  startup: {
    pageReady() {
      const {MmlMs} = MathJax._.core.MmlTree.MmlNodes.ms;
      MmlMs.defaults.lquote = "";
      MmlMs.defaults.rquote = "";
      MathJax.startup.defaultPageReady();
    }
  }
};

Ah! Thanks for the in-depth and timely reply! I was told by someone (not a MathJax dev) to keep to MathML Core and I apparently grew lazy and forgot to go back and look at the MathML3 specification.
And, thanks for the info about the MathJax config and the lquote/rquote attributes.
Best regards,
Brian

Just FYI, I mistakenly used pageReady() rather than ready() and deafultPageReady() rather than defaultReady() in my configuration.  I've changed it above for anyone looking at this in the future.  The pageReady() would work, but one should use return MathJax.startup.defaultPageReady() in that case.


Added float->int in decompression

Added the option to turn some variables into integers during decompression. these are determined in the config

@fritjof-b I think this was a failed rebase paired with a pull. How come Github shows 48 files changed after a rebase to dev and with a PR to dev?


Still getting duplicated text with beta version

Hello,
So I've been using listr2 for 3 days now and I noticed that it has duplicated lines.
Searched in issues and found that it was fixed in the beta version, I installed the beta version and it is still happening
Version 4.0.0-beta.3 as seen here:

Sorry for closing and reopening, just realized the issue wasn't the timing but this always happen when I have subtasks inside another subtask instead

Hello @Palm2570Playz ,
The beta version is currently too much out of date, so I do not even know what could be the problem.
But my educated guess as always for this duplicated stuff is something is trying to write to stdout. As I always discuss in other issues if I can swap the library of the terminal renderer, this should be fixed. But the problem is beta version was not a hundred percent working.


Build the Scoreboard

Stop me if this is too clever. We had an idea to communicate all game state via the scoreboard
This thing.. that big black box at the top center of the game board.

We can use it a bunch of different ways, but primarily it will communicate the game score

If someone scores a goal it can change to reflect that
https://github.com/keep-starknet-strange/tsubasa/assets/3332646/b8548383-d5e9-4823-a6ad-75844b141906
A horizontal scroll effect would be really fun here

Scoreboard needs to be smaller on mobile

Resources
Components
Pages

YOOOO thé ANIMATION!? absolutely killing it

Fonts:
New airport
New Airport.zip

Looks cool! @LucasLvy what are the acceptance criteria for this one?

all the score boards are integrated and they are displayed one after the other ?

all the score boards are integrated and they are displayed one after the other ?

Ok, looks good ! I can take this one


[stable/nfs-client-provisioner] Fix typo in README

What this PR does / why we need it:
Parameter storageClass is misspelled in README, It got me confused for a moment.
Which issue this PR fixes

None

Special notes for your reviewer:
None
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@zerda Please sign your commits and bump the chart version like it says in the check-list in the issue-template.

@davidkarlsen Done. :)

/assign
/ok-to-test

/lgtm


[JS] test failures in carousel.spec.ts

Details in
https://microsoft.visualstudio.com/AdaptiveCards/_build/results?buildId=43488759&view=logs&j=12f1170f-54f2-53f3-20dd-22fc7dff55f9&t=35f53584-79fc-539f-463c-6471d0663a3a
FAIL src/__tests__/components/carousel.spec.ts (8.015 s)
  carousels
    when default constructed
      ✓ should have reasonable default values (4 ms)
    when parsed from a valid object
      ✓ shouldn't throw exceptions (2 ms)
      ✕ should have all of its expected properties and child elements (1 ms)
      ✕ shouldn't allow anything in pages but CarouselPages (1 ms)
    certain elements
      ✕ shouldn't be allowed within a Carousel (4 ms)
  carousel pages
    ✓ shouldn't be parseable outside of a Carousel (1 ms)

  ● carousels › when parsed from a valid object › should have all of its expected properties and child elements

    TypeError: Cannot read property 'timer' of undefined

should enable test on js-ci pipeline #6833


Rewrote architecture

Ryubook now uses Interface-based design
Options have been moved to new models namespace and directory
Split options into respective source files
Clean option is temporarily disabled due to change in architecture
AppConsts is now ProjectFiles

Closed because a more simplified design was introduced following the switch to .NET 6.


Removing first block of file

First of all I should thank you for the great work, saving me a lot of headache.
I use LittleFS in a project to store logs in LIFO style. The issue is that I need the log file to be bounded, that means when the log file size reaches a specific limit, first block of file should be removed before adding new entries. I understand that it's possible to implement a Circular Buffer over a pre-allocated file to do the job but with its own overhead. Also worth mentioning that entries are added to file in a block size chunk.
I wonder if it's possible to simply change meta data to remove the first block of file? Any hints expect for reading the whole spec? ;)

Ok, I have read the Spec, seams that due to backward nature of CTZ-Skip list, removing first block requires a large amount of update (COW) to structure. Is that right?

Hi @DrArmaggedon thanks for creating an issue, sorry for the late response.
This is actually a feature I'm personally quite interested in exploring once other pressing issues in the filesystem are sorted out. It would be a nice feature for a common use case of LittleFS : )
Unfortunately the CTZ-skip makes this a bit difficult. You could in theory keep an extra "offset" field that indicates low-numbered blocks are unused and return them to the block allocator, though there are some assumptions about how large the file can ever get and this would break that.
Unrelated to this, https://github.com/littlefs-project/littlefs/issues/27 indicates that the current data-structure is insufficient for the general case, so I'm planning to adopt a more general purpose B-tree structure in the future. With this I'm hoping to sneak in this feature of dropping the first n-bytes of a file cheaply.

For now the most efficient way to store logs in littlefs is to use a system of rotating files, common in Linux and others. That is to write a file to some fix size, and when it's full rename it to log.2 and start writing a new file. This works well with littlefs's append-friendly file data-structure.


short_term forecasting

您好，非常感谢您所做出的杰出的工作，相信它一定能推动学界相关领域的进步。
我在复现您工作中的M4_short_term_forecasting时遇到了一些问题，我发现您的short_term_forecasting的代码中并没有用到您论文中所提到的两大创新点—文本时间戳嵌入、上下文预测。请问在short_term_forecasting的工作中，您是否没有采用这两个方法呢，亦或是我对于您的代码理解有误？
真诚地期待您的回复，祝好。

The reason for not using timestamps is that the M3 and M4 datasets do not include timestamp information. Each sequence in these datasets only has a starting point in time, and most of these are default placeholder values rather than actual timestamps. Of course, if a way to construct timestamp sequences is devised, adding them would certainly be useful. As for the context forecasting code, it is located in exp_in_context_forecasting, and the experiments include the M3 and M4 datasets.

I get it, Thank you very much for your reply！

The reason for not using timestamps is that the M3 and M4 datasets do not include timestamp information. Each sequence in these datasets only has a starting point in time, and most of these are default placeholder values rather than actual timestamps. Of course, if a way to construct timestamp sequences is devised, adding them would certainly be useful. As for the context forecasting code, it is located in exp_in_context_forecasting, and the experiments include the M3 and M4 datasets.


feat(build): cleaner logs output

There are two improvements that goes together:

Don't display source map on a separated line. This create a lot of noise for something that is not downloaded by the user.
Sort chunks based on the size and align numbers to the right (and having lines for map would create a lot more noise here)

Before:

Note: IMO map size should not be in yellow.
After:

Note: This is clearly not a carefully optimised webapp, but this new report allow to quickly see images that could be compressed.
Example with chunk size limit & compress size:

Note: In Rollup 3 maps have their own assets entries, so they are currently displayed twice in v4

Open question: I would prefer to display size in kB instead of KiB. Is there any reason for using the later?

I like this. At first, about the order from smaller to bigger, my first reaction was to want the opposite but maybe it makes sense to see the biggest files near the prompt. So I think the order is fine.
Maybe we could group JS, CSS, and assets files though? I think it is easier to parse if we see them separately, and between each group, we could order them by size.
I'm unsure though if some people would prefer to see an alphabetic list, I think it is worth adding this PR to the team board so we can discuss it with the rest. Maybe we need an option here.

I like this.
I was debating the separator (e.g. maybe - instead of /), but don't have strong feelings about it.
The grouping idea is an interesting question. I can see arguments either way depending on the scenario. Maybe I just want to optimize load speed regardless of file type and there's chances to do that with both images and JS. Or maybe I have one page with lots of big images and so then maybe I'd want them displayed separately as they're just confined to a single page. After debating it with myself, I think I would be in favor of the grouping. A big reason in my case is that I use vite-imagetools and so I'm going to have a lot of jpg images that are larger, but I don't care so much because they're only going to be served in legacy cases and most users will get avif and webp, so including the images would just meaninglessly clutter the results.

Batch experiment with groups, - separator and kB:

I like it a lot, but I prefer the / separator

I like - as a separator, it feels less noisy. And I love the grouping ❤
@ArnaudBarre I fear that seeing so many numbers on the same line may be a bit overwhelming though. Maybe we should bold the main unzipped number? (so the number for gzip and map ends up feeling a bit dimmed, as some kind of extra information?)

Will try to bold the main size. Will also try a "table" layout to avoid repeating "gzip" and "map" now that we have a headers line

I like the - separator too. I'm not so sure about the group title since the colors are already grouping them (if we sort by groups). Do we also want to show the gzip size for sourcemaps too? Though that would make the line quite long

Update:

Two questions:

If we only use kB, should we add 1000-separators? Like 3,550.63 kB?
Is it possible to add a summary that sum up all kBs? I would like to know the full target/dist size before I deploy to production. (Not sure if in scope of this PR)

Why not for thousand separators 👍. Also, as asked by @patak-dev I will do a separate PR to discuss KiB -> kB
For full dist size it requires to know the size of copy pasted public folder, so it would require a separate sys call. Probably better in another PR

I like the idea of a summary, maybe more interesting even if it is per group. I think we should try first to merge this PR with groups only so we separate the discussions.

| might not be bad. It could make it look a little like a table, which could possibly be good. It'd be interesting to see what that looks like

In that case you could also try \u2502 for the "Box Drawings Light Vertical" character which should join together better between multiple lines in the terminal.

The issue with a blank line is that for very small projects it looks strange

Good point @ArnaudBarre, better to leave it without the extra blank line 👍🏼
@bluwy I think you can merge it if you approve it

@ydcjeff I think playing with the colors is an interesting idea! Let's merge this PR first and then we can create a PR that is only about the colors for each group so we better document the rationale there and don't block this one that is ready to be merged.


Requirements file missing

Hi,
using OpenSUSE 42.2 and installing from GitHub source. You mention (in README):
sudo python setup.py install --record installed.txt

But there is no such file. Also, maybe include required libs in setup.py itself?
I had to manually install:
pexpect
python-gconf
pyxdg
python-appindicator

to get fluxgui command to run without ImportErrors.

You said

But there's no such file.

What file?

Requirements file. Usually requirements.txt

OK, I think I understand now: the installed.txt file is created by the setup.py install --record installed.txt: the --record <filename> option tells setup.py to record the names of all of the files it installed in <filename>. This makes it easy to uninstall with xargs rm -vr < installed.txt.
I don't know anything about "requirements files". Can you point to some documentation or explain why this is better than just listing the requirements in the README?

Yes, but running pip freeze also lists f.lux-indicator-applet==1.1.10rc0 as installed and you could simply run pip uninstall f.lux-indicator-applet to get rid of it, no file tricks needed, I think. Most Python apps now use pip.
"requirements file" is more pythonic/easier way to manage dependencies: e.g. one should run pip install -r requirements.txt and that's it.
Official docs:
https://pip.pypa.io/en/stable/user_guide/#requirements-files
Or just modyfy setup.py a bit: https://packaging.python.org/requirements/

Thanks for explaining. The pip uninstall sounds useful. Right now I use the Ubuntu package manager to install the dependencies. If using the requirements.txt for the install, I guess that would install separate copies of the dependencies using pip? If "yes", then I prefer manually installing the dependencies. But having a requirements.txt for people that want to install things that way might not hurt.

requirements.txt may help for the guys who like to install through that, and adding those in setup.py helps too.
However, some of the required dependencies are not available through pip (appindicator, glade, libxxf86vm1) and pygtk is just a wrapper to gtk2, which you still need to manually install with sudo apt (or rpm in case there is working package with right dependencies name).

Based on @momozor 's comment I think this issue can be closed as it doesn't sound like there's much benefit to the requirements.txt file in our case.


FFMPEG issue with Videotoolbox when transcoding 10bit video track

Hi.
I post this here even though I think this could a FFMPEG issue.
Applying the other transcode command:
other-transcode --hevc --10-bit --crop auto --add-audio all --surround-bitrate 384 /Volumes/MyMovies/HDR10bit_movie.mkv
generate the command:
ffmpeg -loglevel error -stats -i /Volumes/MyMovies/HDR10bit_movie.mkv -map 0:0 -c:v hevc_videotoolbox -pix_fmt:v yuv420p10le -b:v 8000k -color_primaries:v bt2020 -color_trc:v smpte2084 -colorspace:v bt2020nc -metadata:s:v title= -disposition:v default -map 0:1 -c:a:0 ac3 -b:a:0 384k -metadata:s:a:0 title= -disposition:a:0 default -map 0:2 -c:a:1 aac_at -aac_at_mode:a:1 cvbr -b:a:1 256k -ac:a:1 2 -metadata:s:a:1 title= -disposition:a:1 0 -map 0:3 -c:a:2 aac_at -aac_at_mode:a:2 cvbr -b:a:2 256k -ac:a:2 2 -metadata:s:a:2 title= -disposition:a:2 0 -sn -metadata:g title= -default_mode passthrough HDR10bit_movie.mkv
it fails with the following:
Transcoding...
[hevc_videotoolbox @ 0x7fe04d04ee00] Transfer function smpte2084 is not supported.
Error initializing output stream 0:0 -- Error while opening encoder for output stream #0:0 - maybe incorrect parameters such as bit_rate, rate, width or height
/usr/local/bin/other-transcode: transcoding failed:
Also trying with Handbrake (nightly build with H.265 10bit Videotoolbox) it fails.
Thanks for your project!

@eugeniosz You are very welcome for the software. Thanks for using it.
Normally FFmpeg and VideoToolbox can fail when trying to convert a 8-bit video to 10-bit, but a 10-bit-to-10bit conversion should work.
Let me ask the Video Transcoding Hivemind™ if they've seen this before. Stay tuned...

@eugeniosz @donmelton
Hi all,
I've just tested the command above on my 2018 i7 Mac mini (which can output 10-bit from VideoToolbox as opposed to many other older Intel chips which cannot).
My source was Soul (2020) - 4K version with a 10-bit video track.
The test command run was:  other-transcode --hevc --10-bit --add-audio all --surround-bitrate 384 --duration 00:01:00 /Volumes/4K/Soul\ \(2020\).mkv. I'm running ffmpeg 4.3.2 from Homebrew.
other-transcode --hevc --10-bit --add-audio all --surround-bitrate 384 --duration 00:01:00 /Volumes/4K/Soul\ \(2020\).mkv
Verifying "ffprobe" availability...
Verifying "ffmpeg" availability...
Verifying "mkvpropedit" availability...
Finding encoders...
Trying "hevc_videotoolbox" video encoder...
Getting muxer capabilities...
Scanning media...
duration = 00:01:00
Stream mapping:
 0 = hevc_videotoolbox / 8000 Kbps
 1 = ac3 / 384 Kbps
 2 = aac_at / 256 Kbps / stereo
 3 = aac_at / 256 Kbps / stereo
 4 = aac_at / 256 Kbps / stereo
 5 = aac_at / 256 Kbps / stereo
 6 = aac_at / 256 Kbps
 7 = aac_at / 256 Kbps
Command line:
ffmpeg -loglevel error -stats -t 60 -i /Volumes/4K/Soul\ \(2020\).mkv -map 0:0 -c:v hevc_videotoolbox -pix_fmt:v yuv420p10le -b:v 8000k -color_primaries:v bt2020 -color_trc:v smpte2084 -colorspace:v bt2020nc -metadata:s:v title\= -disposition:v default -map 0:1 -c:a:0 ac3 -b:a:0 384k -metadata:s:a:0 title\= -disposition:a:0 default -map 0:2 -c:a:1 aac_at -aac_at_mode:a:1 cvbr -b:a:1 256k -ac:a:1 2 -metadata:s:a:1 title\= -disposition:a:1 0 -map 0:3 -c:a:2 aac_at -aac_at_mode:a:2 cvbr -b:a:2 256k -ac:a:2 2 -metadata:s:a:2 title\= -disposition:a:2 0 -map 0:4 -c:a:3 aac_at -aac_at_mode:a:3 cvbr -b:a:3 256k -ac:a:3 2 -metadata:s:a:3 title\= -disposition:a:3 0 -map 0:5 -c:a:4 aac_at -aac_at_mode:a:4 cvbr -b:a:4 256k -ac:a:4 2 -metadata:s:a:4 title\= -disposition:a:4 0 -map 0:6 -c:a:5 aac_at -aac_at_mode:a:5 cvbr -b:a:5 256k -metadata:s:a:5 title\= -disposition:a:5 0 -map 0:7 -c:a:6 aac_at -aac_at_mode:a:6 cvbr -b:a:6 256k -metadata:s:a:6 title\= -disposition:a:6 0 -sn -metadata:g title\= -default_mode passthrough Soul\ \(2020\).mkv
Transcoding...
frame= 1439 fps= 26 q=-0.0 Lsize=   73831kB time=00:01:00.03 bitrate=10075.2kbits/s speed=1.07x
Assembling `.log` file...
Adding track statistics...
The file is being analyzed.
The file is read in order to create track statistics.
Progress: 100%
The changes are written to the file.
Done.

Elapsed time: 00:00:56

And the output definitely created a 10-bit transcode as you can see below from MediaInfo:
Video
ID                                       : 1
Format                                   : HEVC
Format/Info                              : High Efficiency Video Coding
Format profile                           : Main@L5.1@Main
Codec ID                                 : V_MPEGH/ISO/HEVC
Duration                                 : 1 min 0 s
Bit rate                                 : 8 004 kb/s
Width                                    : 3 840 pixels
Height                                   : 2 160 pixels
Display aspect ratio                     : 16:9
Frame rate mode                          : Constant
Frame rate                               : 23.976 FPS
Color space                              : YUV
Chroma subsampling                       : 4:2:0
Bit depth                                : 10 bits
Bits/(Pixel*Frame)                       : 0.040
Stream size                              : 57.3 MiB (79%)
Writing library                          : Lavc58.91.100 hevc_videotoolbox
Language                                 : English
Default                                  : Yes
Forced                                   : No
Color range                              : Limited
Color primaries                          : BT.2020
Transfer characteristics                 : PQ
Matrix coefficients                      : BT.2020 non-constant

Without knowing your Mac, we need to check the following:

Can your Intel Mac actually output 10-bit even from source? From my testing, the Mac mini (2018) and later Macs can but M1 Macs do not support 10-bit output. I'm not sure when the Macs started being 10-bit output capable so worth checking.
Is your input truly 10-bit? If you drag-drop the source onto MediaInfo, you should see a "Bit Depth:  10-bit" (as above but from the source). For example:  my iPhone 11 Pro is set to 4K/60fps but it's only an 8-bit output. I don't know if the iPhone 12 models upgrade the bit-depth. But the real test would be a 4K movie rip like Soul 4K above.

My guess at this stage is the first one above ... Intel Mac version & (possible lack of) 10-bit VT support
Thanks,
Joe

@rhapsodians Thanks for the testing and feedback, sir! 👍

@eugeniosz
Without knowing your Mac, we need to check the following:

Can your Intel Mac actually output 10-bit even from source? From my testing, the Mac mini (2018) and later Macs can but M1 Macs do not support 10-bit output. I'm not sure when the Macs started being 10-bit output capable so worth checking.
Is your input truly 10-bit? If you drag-drop the source onto MediaInfo, you should see a "Bit Depth:  10-bit" (as above but from the source). For example:  my iPhone 11 Pro is set to 4K/60fps but it's only an 8-bit output. I don't know if the iPhone 12 models upgrade the bit-depth. But the real test would be a 4K movie rip like Soul 4K above.

My guess at this stage is the first one above ... Intel Mac version & (possible lack of) 10-bit VT support
Thanks,
Joe

Hi Joe,
Thanks for your answer.
Below more info,
I use an iMac 2019 with i9-9900K and AMD Radeon Vega 48 64GB RAM
Video track media info:
{
"@type": "Video",
"StreamOrder": "0",
"ID": "1",
"UniqueID": "13470081048869518320",
"Format": "HEVC",
"Format_Profile": "Main 10",
"Format_Level": "5",
"Format_Tier": "High",
"HDR_Format": "SMPTE ST 2086",
"HDR_Format_Compatibility": "HDR10",
"CodecID": "V_MPEGH/ISO/HEVC",
"Duration": "1572.655000000",
"BitRate": "24574412",
"Width": "3840",
"Height": "1920",
"Sampled_Width": "3840",
"Sampled_Height": "1920",
"PixelAspectRatio": "1.000",
"DisplayAspectRatio": "2.000",
"FrameRate_Mode": "CFR",
"FrameRate": "23.976",
"FrameCount": "37706",
"ColorSpace": "YUV",
"ChromaSubsampling": "4:2:0",
"ChromaSubsampling_Position": "Type 2",
"BitDepth": "10",
"Delay": "0.000",
"StreamSize": "4830884120",
"Language": "en",
"Default": "Yes",
"Forced": "No",
"colour_description_present": "Yes",
"colour_description_present_Source": "Stream",
"colour_range": "Limited",
"colour_range_Source": "Stream",
"colour_primaries": "BT.2020",
"colour_primaries_Source": "Stream",
"transfer_characteristics": "PQ",
"transfer_characteristics_Source": "Stream",
"matrix_coefficients": "BT.2020 non-constant",
"matrix_coefficients_Source": "Stream",
"MasteringDisplay_ColorPrimaries": "Display P3",
"MasteringDisplay_ColorPrimaries_Source": "Stream",
"MasteringDisplay_Luminance": "min: 0.0050 cd/m2, max: 1000 cd/m2",
"MasteringDisplay_Luminance_Source": "Stream",
"MaxCLL": "186 cd/m2",
"MaxCLL_Source": "Stream",
"MaxFALL": "26 cd/m2",
"MaxFALL_Source": "Stream"
},

_ffmpeg_90001_93456.mkv.log


SILGen: Don't allow referencing the 'var' box before it's fully initialized

In the following code, SILGen emits a mark_uninitialized instruction
on the projected box value, diagnosing the reference to 'x' before
it has been fully initialized:
var x: Int
x = { _ = x; 123 }()

However, when the binding has an initial value, we don't emit the
mark_uninitialized instruction, and the closure ends up capturing the
uninitialized box:
var x: Int = { _ = x; 123 }() // undefined behavior

Unfortunately, we can't unconditionally emit a mark_uninitialized here,
because DI does not recognize all the code patterns that SILGen can emit
when initializing an address-only value in-place.
Instead, what we want to do is only enter the box into SILGenFunction::VarLocs
after the initial value expression has been emitted. This ensures that the
forward-referenced-capture diagnostics catch this.
Note that this only addresses the case of a local binding; we still
generate invalid code here if you forward-reference a global 'var' in
this manner. Fixing that requires some additional changes.
Fixes https://bugs.swift.org/browse/SR-14747, rdar://problem/79051431, rdar://problem/77933460.

@jckarter Does this look reasonable?

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility


scopeUri must not be null or empty

I'm not able to get rid of this error coming from google signin module. This works fine with iOS. Any help appreciated.

Fixed it by adding scope, wonder why it works with iOS without defining the scopes..


Problem with target data file (2023-10-27)

The value column of target-data/archive/2023-10-27_rsvnet_hospitalization.csv is filled with NAs. The problem starts on row 70955 and lasts until the end of the file.
.
.
.
"49",2023-10-21,"50-64","inc hosp",0,479817
"49",2023-10-07,"65-130","rate hosp",0,405200
"49",2023-10-07,"65-130","inc hosp",0,405200
"49",2023-10-14,"65-130","rate hosp",0,405200
"49",2023-10-14,"65-130","inc hosp",0,405200
"49",2023-10-21,"65-130","rate hosp",0,405200
"49",2023-10-21,"65-130","inc hosp",0,405200
"06",2014-10-11,"0-0.49","rate hosp",NA,249617
"06",2014-10-11,"0-0.49","inc hosp",NA,249617
"06",2014-10-18,"0-0.49","rate hosp",NA,249617
"06",2014-10-18,"0-0.49","inc hosp",NA,249617
"06",2014-10-25,"0-0.49","rate hosp",NA,249617
"06",2014-10-25,"0-0.49","inc hosp",NA,249617
"06",2014-11-01,"0-0.49","rate hosp",NA,249617
.
.
.

@paulocv Thanks for your reporting. These target-data files refer to the same data as the ones in rsv-scenario-modeling-hub/ and there are NAs in the original data either. Please let us leave them as they are without modifying it.

This is gonna be closed


Upgrading to react 18

Upgrading to react 18 and its associated packages

Currently failing to build due to tests, unable to run tests because of this. I expect many changes needed in regards to this error

FYI: In your app, you will probably want to call somewhere
import { createRoot } from "react-dom/client";
UiFramework.childWindows.useCreateRoot(createRoot);

So the popouts works correctly with React 18 (otherwise it will render with React 17 level features, which may or may not be correct)


Chat doesn't display Messages, only Notifications

Encountered this problem on the latest beamng and latesr kissmp, the chat doesn't display any messages, there are only notifications visible. Tested on multiple pcs, highend, lowend... hardware doesn't have any effect on that.
Great Mod nontheless!
-C#

Most likely fixed in commit 8a38caccd4e612f9a7d273c5ebdeb14a49f3a19d


Wrong resource type output when generating single resource name

Describe the bug
When generating a name for a single resource where the category has multiple resource types causes the incorrect resource type abbreviation to be added to the name.
To Reproduce
Steps to reproduce the behavior:

Go to Generate
Ensure 'Generate Single Resource Type Name' is selected
Enter 'web' in resource type category filter
Select 'Web/sites - Web App (app)' in the resource type dropdown
Enter all required fields for units/projects/locations/etc
Select generate
The generated name contains the wrong resource type abbreviation

Expected behavior
I expect the name to contain the abbreviation of 'app' instead of 'stapp', since the Web App was selected in the list.
Installation Method
Installed using the docker container steps as outlined on the instructions page
Additional context
It appears the first item in the resource type dropdown that starts with 'Web/sites' is being selected to generate the name instead of the selected value.

Thank you @eddieh-digitalcontrol for the submission!
This is related to the auto-complete functionality on the Generate page for the Resource Categories/Types. This will be fixe dint he next version (3.0.1).

Bryan

You can test out the changes in the pre-lease here:
https://github.com/mspnp/AzureNamingTool/releases/tag/v3.0.1
Please let me know if you have any issues!
-Bryan

Thank you for the update, name generation is working correctly!

Awesome! We will release the final v3.0.1 later this/next week with the included changes.


Tasks should have an option to clear the terminal before executing

This clear option can be added into the exiting "presentation" task property and would be called "clear":
	{
		"label": "ExampleTask",
		"type": "shell",
		"command": "echo hello",
		"presentation": {
			"echo": true,
			"reveal": "always",
			"focus": false,
			"panel": "shared",
			"showReuseMessage": true,
			"clear": false
		}
	}

Added to TaskPresentationOptions:
/**
 * Controls how the task is presented in the UI.
 */
export interface TaskPresentationOptions {
        ….
	/**
	 * Controls whether the terminal is cleared before executing the task.
	 */
	clear?: boolean;
}

This doesn't work for me:
{
    "label": "My Task",
    "type": "shell",
    "command": "echo Hello",
    "problemMatcher": [],
    "presentation": {
        "clear": true
    }
}

I run it multiple times and see all the output, the terminal is never cleared.

Turns out that if the only presentation option set is clear, this doesn't work. Thanks for finding!


Validate the inputs to a job before submitting

e.g. make sure that the user has permission to read the inputs and write to the output/destination location.

not sure if this is part of a workflow? Does the 'stage-in' and 'stage-out' module handle this?


Add basic lua implementation

We have a Lua thread! We can ask it to do things! We can register Rust and Lua things for it to do!

Dependency on hlua
Lua module with thread spawned on compositor-ready
Basic LuaQuery and LuaResponse enums to send messages to the thread
lua::send to send messages
Lua thread which receives requests and runs Lua code
lib/lua folder with a utils library and init.lua to prepare the thread
lua::funcs module containing Rust functions to be called from Lua (such as in funcs::input). This module has #![deny(dead_code)] so functions written here will cause a compile error if they are not registered.
lua::funcs::init which adds all the Rust functions into the proper Lua APIs.
Key binding C-p which prints the pointer position using Lua
Key binding C-l which sleeps the Lua thread (demonstrating the main thread continues, Lua requests queue up)

This has the skeleton of the infrastructure we need for way-cooler. This is a huge part of the capability we need with the ability to have keybindings and Lua functions. After expanding on this API (add UUIDs on Lua messages, get back results, add connections to registry) we can start working on the config system, and adding more Rust and Lua functions to our API.

Added 05100c5 in response to your comments, removed the macro for now and the comment you commented on.
Given the way we have to manually access the tables in order to register the functions, I don't think a lua macro would be as automatic, unless it took in a reference to which table we register the function with.


API improvement for remove, permanent remove and restore features

Added new API endpoints
Added new model fields
Improved settings.py
Added Celery support

For running Celery you can use the next command:
celery -A mcserver worker -l INFO -B

@olehkorkh-planeks thanks!
Two questions:

See my comment above, but it looks like I should have a 'REDIS_URL' variable in my .env, which I don't have. What should I use?
Naive question but I am not sure what you mean with 'For running Celery you can use the next command: celery -A mcserver worker -l INFO -B'. Do we need to "manually" run celery? Is that for testing only? Thanks for elaborating on that.

Hi @antoinefalisse
You're right. You need to add the REDIS_URL to the .env file.
As you can see, I added the Celery as a dependency to the backend. For the current moment, we use it for running background tasks for permanent cleaning of trashed objects (session and trials). We also use Celery Beat for running those tasks by schedule. So, what we need:

The Redis instance for managing the tasks queue (it is the simplest way for starting with Celery)
I also added the "celery-redbeat" for managing scheduling metadata in the Redis as well. By default celery beat use a file on the disk for such kind of task, but it is not suitable for cloud hosting.

I wrote about the command for running the celery worker, but it depends on the environment. We need to run a worker and the beat service. You can use the command celery -A mcserver worker -l INFO -B for running celery and beat locally. In the production environment, it depends on your infrastructure. You need to check what is suitable in your case.
This is an article about running Celery worker on ECS:
https://python.plainenglish.io/deploying-django-on-aws-setting-up-celery-and-sqs-95632a6e79cb
In my case, I prefer to use Redis instead of SQS, because we can use Redis also for scheduling metadata, and it is possible to use it for caching in the future. SQS can be better for systems where background tasks contain some sort of critical operations, but our case is simpler.
Keep in mind that workers can be scalable. It is possible to run any workers in parallel we need (if you need it). But, in that case, you need to avoid use the -B (beat) parameter. In that case you need to run additional service specifically for beat: celery -A mcserver beat.

Thanks for the explanation @olehkorkh-planeks, we will try to deploy to AWS.


add a section that describes all the environment variables that can be used to configure Aegis, in detail.

Done.


Add Query StateStore API implementation for Redis StateStore

Describe the feature
Per https://github.com/dapr/dapr/issues/3662, we have added Query StateStore API and implemented it for ModgoDB and CosmosDB (https://github.com/dapr/components-contrib/pull/1153)
We need to implement this API for Redis StateStore
Release Note

RELEASE NOTE:

Reopened.

@dmitsh are you planning to work on this for 1.6?

@dmitsh are you planning to work on this for 1.6?

wasn't planning. @manojrupireddy, are you still interested in pursuing this task?

Natively, Redis only offers primary key access. However, with the help of two modules, it is possible to store and query JSON documents:

RedisJSON allows to store and manipulate values as JSON objects (https://oss.redis.com/redisjson/)
RediSearch enables secondary indexing and searching capability (https://oss.redis.com/redisearch/).

The current Redis component implementation does not require any additional modules, and stores the values as raw data.
The use of the aforementioned modules allows querying the state, but it comes with a price:

RediSearch requires setting the JSON object schema prior to querying
RedisJSON stores data in a special fashion.

This will be incompatible with the current implementation. One way to address this issue is to develop an alternative Redis component that supports JSON structures. The new component will require presence of RedisJSON and RediSearch, and will allow setting the object schemas.
To make the new Redis component to support other formats, we would need to introduce optional content type to the State Store API. If the content type equals to application/json, the component will store the value as JSON object. Otherwise the value will be stored as raw data.
In the future, the component could support additional formats.
This is a very high-level description of the proposed design.
Please comment if you have any comments and suggestions.
After gathering the feedback, I'm intended to propose more detailed design.

This starts to feel like a different implementation of the redis state store component.

This starts to feel like a different implementation of the redis state store component.

it can be the same component, but there should be an agreement that if JSON content type is declared, the underlying Redis service should include RedisJSON and RediSearch modules.

One thing to consider is that if the users are getting their Redis from a managed cloud offering (e.g. AWS ElastiCache for Redis), does that offering have the required modules @dmitsh mentioned installed. Given this features relies heavily on the modules being active/installed, it could limit the value that could come from this feature. Would users not shift to a Document DB style database when they have this usecase?


V0.1- Floating Task support (Add)

Refer to V0.1 of the handbook
Add support of floating tasks

done along with delete and list with @aljorhythm (10 Mar 5am)


Remove User friendly

Because it ended, see https://en.wikipedia.org/wiki/User_Friendly

Merged as https://github.com/sailfishos-applications/daily-comics/pull/9


Parser throwing exception

PackageName:spdx-tools
SPDXID: SPDXRef-spdx-tools-0.6.1
PackageSupplier: Person:Ahmed H. Ismail
PackageVersion: 0.6.1
Is throwing an exception on this spdx SBOM data: https://raw.githubusercontent.com/rjb4standards/REA-Products/master/SAG-DBOMPOC-SBOM.spdx
Here is the exception I'm seeing:
<class 'TypeError'> 'builtin_function_or_method' object is not subscriptable <traceback object at 0x000001B8B16DC0C8>

I found the culprit for this error too: PackageChecksum must be a single line of text, line: 15
LOOKS LIKE SHA256 not supported, only SHA1 allowed:
15: PackageChecksum: SHA256: aaa5ac584f40fe778013df0aa6544bf157799bd3f608364b451840ed2c8688de

def checksum_from_sha1(value):
"""
Return an spdx.checksum.Algorithm instance representing the SHA1
checksum or None if does not match CHECKSUM_RE.
"""
# More constrained regex at lexer level
CHECKSUM_RE = re.compile("SHA1:\s([\S]+)", re.UNICODE)*
match = CHECKSUM_RE.match(value)
if match:
return checksum.Algorithm(identifier="SHA1", value=match.group(1))
else:
return None

related to #185

Closing this issue as it should be solved with #311. Please reopen, if this is not the case.


IsURL checks bytes length instead of characters

IsURL contains code: len(str) >= 2083
Number 2083 comes from IE URL limitation, but there mentioned characters, not bytes.
Moreover, in validator.js url.length >= 2083 relates to characters count, not bytes.
I think this check should be replaced by utf8.RuneCountInString(str) >= 2083

bytes count always greater than runes so there is no problem :)


Incorrect parameters for OrderedDataStore:GetSortedAsync()

As seen in the attached image, you can see how the lsp expects the user to put 5 arguments for :GetSortedAsync() which is incorrect due to minvalue and maxvalue being optional parameters

I can fix this now. But in future, contributions are welcome to fix!
https://github.com/JohnnyMorganz/luau-lsp/tree/main/scripts#contributing


Just windows has the \ file path flag set

Please ensure you adhere to every item in this list.

The PR title is formatted as follows: os/gtime: fixed time zone issues

The package name goes before the colon
The part after the colon uses the verb tense + phrase that completes the blank in
Lowercase verb after the colon
No trailing period
Keep the title as short as possible. ideally under 76 characters or shorter
Title not Markdown

If there is a corresponding issue, add either Fixes #1234 or Updates #1234
(the latter if this is not a complete fix) to this comment
Delete these instructions once you have read and applied them

提交前请遵守每个事项，感谢！

PR 标题格式如下：os/gtime: fixed time zone issues

冒号前是包名
冒号后使用动词时态 + 短语
冒号后的动词小写
不要有结尾句号
标题尽量保持简短，最好在 76 个字符或更短
标题不要使用 Markdown

如果有对应的 issue，请在此评论中添加 Fixes #1234，如果不是完全修复则添加 Updates #1234
应用这些规则后删除所有的说明

@tekintian Hello, the ci failed, please have a check: https://github.com/gogf/gf/actions/runs/9048670552/job/24861951671?pr=3576

@tekintian Hello, it does the replacement no matter what the GOOS is, as it takes no effect if the system separator is not \.


View codes update

View code flow

View codes created manually when users create shows.
When shared, the view code is passed as a URL param.
If user who receives view code is marked as an editor of the show, they have edit access, otherwise they can only view.

I need to

Add 'generate one for me' button for generating UUID viewcode when user is editing show information
Create url builder with viewcode

'Share' button
BASEURL/shows/show-id?share=viewcode

Handle view code urls with an endpoint

'GetShowById' endpoint

Handle checking if users are editors

This requires an Editor table & 'view view mode'

Database updates needed

make view codes not nullable
create editor table

Today I got through most of steps 1 & 2. The 'Generate one for me" button will require that we manage the state of the ShowForm a little differently because directly changing the showForm.view_code forces a form submission. So that has been put on the backburner until we can discuss how we should handle that. However, on a form submission, the client now checks if the view_code is empty, provided that the modal is in new-form mode it will generate a UUID for the submission. The only reason for a 'generate one for me button' is if a user wants to change the view code of a show without having to think about what it should be changed to.
I also added a share button which renders on the showViewHeader if the show is marked as visible. As of right now it just constructs the share code url and copies it to the system clipboard.
I began working on the server side get_show_by_id endpoint and got so far as adding logic to get the Query parameters from any request, but that is all.
Before i finished today I encountered this strange bug where the SlideViewShow header would suddenly not render. Not sure how or why this happened as I made no changes to the client side, so I'm wondering if my small change to the endpoint has something to do with it.

Next time we meet I think we should put together a DB initialization transaction and fully reset the DB with that one schema so we don't have dozens of migrations in our migration folder. When we do that we need to create an editor table and ensure that view codes are no longer nullable


Question about facenet classifier training

Hi,
I was reading about the classifier training code.
Just have some question about the code, and how it works.
in the classifier python file facenet_train_classifier.py
there is a block of code as follow (line 111- 137):
        input_queue = data_flow_ops.FIFOQueue(capacity=100000,
                                    dtypes=[tf.string, tf.int64],
                                    shapes=[(1,), (1,)],
                                    shared_name=None, name=None)
                              enqueue_op=input_queue.enqueue_many([image_paths_placeholder,labels_placeholder],name='enqueue_op')

        nrof_preprocess_threads = 4
        images_and_labels = []
        for _ in range(nrof_preprocess_threads):
            filenames, label = input_queue.dequeue()
            images = []
            for filename in tf.unstack(filenames):
                file_contents = tf.read_file(filename)
                image = tf.image.decode_png(file_contents)
                if args.random_rotate:
                    image = tf.py_func(facenet.random_rotate_image, [image], tf.uint8)
                if args.random_crop:
                    image = tf.random_crop(image, [args.image_size, args.image_size, 3])
                else:
                    image = tf.image.resize_image_with_crop_or_pad(image, args.image_size, args.image_size)
                if args.random_flip:
                    image = tf.image.random_flip_left_right(image)

                #pylint: disable=no-member
                image.set_shape((args.image_size, args.image_size, 3))
                images.append(tf.image.per_image_standardization(image))
            images_and_labels.append([images, label])

As my understand, first we have created a FIFO queue which have shape of 1, 1:
        input_queue = data_flow_ops.FIFOQueue(capacity=100000,
                                    dtypes=[tf.string, tf.int64],
                                    shapes=[(1,), (1,)],
                                    shared_name=None, name=None)

Then on line 120, we called the following code:
        filenames, label = input_queue.dequeue()

As my understanding, the dequeue operation will only return the single object so
filenames =  tensor of [image_file_path of one image]
label = class_identification
Then we called
            images = []
            for filename in tf.unstack(filenames):
                file_contents = tf.read_file(filename)
                image = tf.image.decode_png(file_contents)
                if args.random_rotate:
                    image = tf.py_func(facenet.random_rotate_image, [image], tf.uint8)
                if args.random_crop:
                    image = tf.random_crop(image, [args.image_size, args.image_size, 3])
                else:
                    image = tf.image.resize_image_with_crop_or_pad(image, args.image_size, args.image_size)
                if args.random_flip:
                    image = tf.image.random_flip_left_right(image)

                #pylint: disable=no-member
                image.set_shape((args.image_size, args.image_size, 3))
                images.append(tf.image.per_image_standardization(image))
            images_and_labels.append([images, label])

First question, if tf.unstack returns a list of one item, the images will looks like [[rgb_of_the_image]], then
images_and_labels will contains data like [[[rgb_of_the_image], label_1],[[rgb_of_the_image], label_2]] ([rgb_of_the_image] = array of rgb_of_the_image)
how would this work, when we pass it into the training session, is that the ideal training data should be as follow:
[[ rgb_of_the_image, label_1],[rgb_of_the_image, label_2]]
If I didn't explain my question clear, please let me know, Appreciate on any feedback given.
Thanks

Yes, essentially you are right. A batch of examples is created by the call to tf.train.batch_join and that is then fed to the model.


Refactor TurbiniaClient.get_task_data() into state_manager

Right now the bulk of the get_task_data() function for the GCP implementation exists within client.py, and the same function for Redis is in the RedisStateManager object.   The two implementations have also not been kept in parity (the later doesn't currently support looking up tasks by user).  We should move the get_task_data() from client.py and add it to the BaseStateManager to make sure they are using the same interface and are both kept up to date.
@ericzinnikas FYI:  See above about the get_task_data() not currently being in parity between the two implementations.  If I don't get to this refactoring soon, we can probably just add a quick fix to support looking up tasks by user to start with in the Redis implementation.

FYI, I found other problems with the user attribute, so I filed https://github.com/google/turbinia/issues/359 (which also covers supporting this attribute in the Redis setup).

This is no longer applicable, we only use Redis as a state manager backend and it has its own get_task_data implementation.


[EOSF-549] Add query param for selected file on content page

Ticket
https://openscience.atlassian.net/browse/EOSF-549
Purpose
Allows passing in query param on content page that pre-selects a file to display if the id passed matches that of a file in the preprint

I didn't think to test this, but when editing a preprint, it fails. I think the issue the query parameters from the controller are somehow interfering with the route.


Add scaling to the skill icons in XP Orbs

Currently, when scaling an XP orb below the default size of 35, the edges of the skill can clip out of the edges of the orb, which looks very bad. Conversely, making the orb larger than 35 results in a lot of empty space around the icon, which looks bad.
To fix this, I propose that the icon scale. Scaling smaller than default will have to cap somewhere, however. The user should be given an option as to whether they want the icon to scale or not. It might also be a good idea to add a lower cap to the size of the orbs, now that we can easily do this.

This was added in e7f50433a352ab996b7b5b6114b504194312000d


Include new task metadata in manifest

The most-recent manifest schema is at https://github.com/fractal-analytics-platform/fractal-server/blob/main/fractal_server/app/schemas/v2/manifest.py.
Notable changes:

Manifest has a new top-level attribute authors: Optional[str] = None (note: it's a single string, not an array - this was done to simplify queries like "authors contains xyz)
Each task has three new attributes:

category: Optional[str] = None
modality: Optional[str] = None
tags: list[str] = Field(default_factory=list)

There is a new constraint that no two tasks of the same package can have the same name

To do:

[ ] Update what is needed in https://github.com/fractal-analytics-platform/fractal-tasks-core/tree/main/fractal_tasks_core/dev
[ ] Actually fill the values for the top-level authors and for all individual tasks
[ ] Document the "typical" conventions somewhere (fractal-tasks-core docs? or somewhere in https://fractal-analytics-platform.github.io/?)

Update what is needed in https://github.com/fractal-analytics-platform/fractal-tasks-core/tree/main/fractal_tasks_core/dev

This is mostly covered in #855, up to a decision of where the top-level information about authors should be stored.
I've started by placing it into the create_manifest.py script (which can also be automated, within fractal-tasks-template, based on the copier answers), but we can also consider other options (e.g. expanding the task_list.py module, to also host some top-level information).
The other two points (actually filling the metadata of fractal-tasks-core, and documenting the conventions) remain open.


Implement Shift+MultiClick Selection Expansion

Summary of the Pull Request
Adds implementation for user to shift double/triple click. Proper behavior (as described in #4557) is to only expand one selection point, not both.
PR Checklist

[X] Closes #4557
[X] Tests added/passed

Detailed Description of the Pull Request / Additional comments
Adding the bool targetStart was a bit weird. I decided on this being the cleanest approach though because I still want PivotSelection to be its own helper function. Otherwise, the concept of "pivoting" gets kinda messy.
Validation Steps Performed
Manual testing as described on attached issue.
Tests were added for Shift+Click and pivoting the selection too.

@DHowett Bumping this PR since you're blocking.

@msftbot merge this in 7 minutes


IAM - kc-postgres secret created after deployment

I understand that the section about IAM is currently work in progress. One little fix that could be applied is that the secret kc-postgres currently is created after the component is deployed with helm which leads to pods initially failing due to lack of said secret. The secret should be created before the actual deployment. I guess in the final version it will be nicely wrapped up in check-prerequisites and configure-iam scripts and there will be no need to create the secret with a separate command, I'm just sharing my observation from the current state.

This is now part of an apply-secrets.sh script in the IAM component here


OutOfMemoryError when inflating the view

Hey I have the exception I mention in the title.
It happen when I'm trying to inflate the layout.
the size of the image is 296Kb
what can I do?

use smaller images for small density, is even better if use drawable from XML


Command + Z or Undo action will freeze Editor

I copied and pasted some contents from the same page.  And then press Command + Z or Undo action, this will cause Editor become uneditable.
Sometime, it works. After this action,  when I press delete button, it will delete all content I just pasted at once.
GitBook Editor version 6.0.3
OS X EI Capitan

Let me know if this is still an issue with the latest versions (6.4.0)


fix: dco checks

Fixes DCO checks due to dcoapp issue - https://github.com/dcoapp/app/issues/211
Uses recommend approach from Hyperledger - https://discord.com/channels/905194001349627914/1156961689971998782/1247613586638110894

thanks @JamesKEbert. it looks like the issue was fixed upstream right after you made this fix 😅. is it still necessary?

I somehow managed to make this PR 7 minutes before the service was restored :man_facepalming:


[9.18] Add some methods related to merge requests approvals

This PR add methods for these API endpoints:

https://docs.gitlab.com/ee/api/merge_request_approvals.html#get-configuration
https://docs.gitlab.com/ee/api/merge_request_approvals.html#get-project-level-rules
https://docs.gitlab.com/ee/api/merge_request_approvals.html#get-the-approval-state-of-merge-requests
https://docs.gitlab.com/ee/api/merge_request_approvals.html#get-merge-request-level-rules
https://docs.gitlab.com/ee/api/merge_request_approvals.html#create-merge-request-level-rule
https://docs.gitlab.com/ee/api/merge_request_approvals.html#update-merge-request-level-rule
https://docs.gitlab.com/ee/api/merge_request_approvals.html#delete-merge-request-level-rule

Great job, thanks! 🚀


Improve AQE extraneous shuffle handling

Is your feature request related to a problem? Please describe.
#1655 deals with an issue with Adaptive Query Execution where Apache Spark is adding extra CPU shuffles that are then later removed.  It would be better to avoid having Spark add these undesired shuffles.  It adds them because it's unable to properly evaluate the semantic equality of a GPU hash partitioning expression relative to the equivalent CPU hash partitioning expression.
Apache Spark needs the ability to detect that the plugin GPU expressions are semantically equal to corresponding CPU expressions when checking whether shuffles need to be inserted to fix the required distribution of later nodes in the plan.
Describe the solution you'd like
Ideally we find a way for the plugin to advertise these expressions to Spark without changes to Spark, but this may not be possible.  One potential idea is to canonicalize the GPU plan as if it were a CPU plan, since canonicalization is mostly used for comparisons.  However there may be times where we do NOT want a GPU plan and a CPU plan to appear equivalent since one executes in a columnar fashion while the other is row-by-row.
This may require a change to Apache Spark to deal with different contexts of semanticEquals or canonicalize.

Fixed by #1655.


Preserve Navigation bar (Hide/Show) when navigated to Other module

How to preserve the navigation bar (minimized position) when navigated to other module.

In the Views\Shared\_Layout.cshtml file
put sidebar-collapse css class in the body tag

I don't want it as default. It should be like Theme Preference.

Closed because it's does not meet the issue requirements.
If you still need help, you can ask this on stackoverflow.
https://stackoverflow.com/tags/serenity-platform
Hint: run a simple js code on cshtml file which you want to collapse navigation.

I answered this issue on stackoverflow but in case someone want to search issue on github here is code and link
https://gist.github.com/minhhungit/54367f416d589e707e42a04772a2ef63
https://stackoverflow.com/questions/64334820/preserve-navigation-bar-hide-show-when-navigated-to-other-module


Add logo in title bar

ill work on this, what logo should I add?

If you want you can design a new logo this issue is also open (https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/issues/4) you can work on both issues
If you don't want to design a new logo you can add the existing logo or any good movie symbol.
But don't start working on any issue, make sure you are assigned to that issue.
I'll assign you this issue when the contribution period starts from 10th June, then you can start working on this issue.

Can I work on this issue? I'm a participant of LGM Soc21.

@Jagannath8 one participant has already asked to work on this issue.
Since this is a very basic and simple issue I cannot allow more than one person to work on it.
You can work on issue number https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/issues/4

Sir if this issue is still unresolved, Can I work on this issue?

Can you please assign this issue to me..

@JAsaxon are you working on this issue or should I unassign you?

sir if this issue is still unresolved will u please assign this  issue to me?

@Jagannath8 use this pic -> https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/blob/master/Images/Logo/Title.jpeg.

sir if this issue is still unresolved will u please assign this issue to me?

@prads46 others have asked prior to you they are first in line.
Look for any other issue.

@Jagannath8 are you working on this?

yes

yes

Fine


add how-to guide for flashing firmware with esptools

a how-to guide containing:

[x] - steps for cloning or downloading & extracting from release page
[x] - (link to) instructions for installing esptools.py
[x] - steps for flashing firmware onto nugget

Done!  We just pushed that documentation yesterday here - https://usbnugget.com/docs/guides/update-with-esptool/
We're working on making the documentation more coherent, but if you have any suggestions / requests, you can leave them on our repo.


Translation improvement ideas

As recommended in https://github.com/TerraformersMC/ModMenu/issues/450#issuecomment-1152964684 do I now open this issue in the hopes to start a discussion on possible improvements towards the current source strings for translations.
Right now, I see 2 minor problems that I think should be improved.
1. Usage of past tense in translations
This may depend on the actual translations done but the usage of past tense such as Shown or Hidden feels... Odd.
Sure Minecraft itself uses Shown in some of its settings, but it still doesn't mean it sounds or feels right.
I personally believe that the usage of Show and Hide would also work too.

2. Condense option values
There are a lot of duplicate values in the source String. I already mentioned two: Shown and Hidden.
I feel like there is no need to have separate entries for each option that has the exact same values. Sure, you can come up with the argument that a language may use a different word/character/... for this particular combination, but how likely is that? Can anyone that is translating any of the currently offered languages list such a situation for this mod?
It would benefit both sides for sure to have less duplicate text. Devs have less lines to maintain and translators don't have to repeat the same sentence/word x times.

There are a lot of duplicate values in the source String. I already mentioned two: Shown and Hidden.
I feel like there is no need to have separate entries for each option that has the exact same values. Sure, you can come up with the argument that a language may use a different word/character/... for this particular combination, but how likely is that? Can anyone that is translating any of the currently offered languages list such a situation for this mod?

In Russian:
hidden=скрыт, shown=показан, BUT:
Библиотеки: показаны/скрыты
Лицензия мода: показана/скрыта
And these are not all existing forms.

1. Usage of past tense in translations

Since there is precedent of using "Shown" and "Hidden" and Mod Menu has used them for a long time now the English source strings won't be changed.
This doesn't mean that the German translation has to follow that, though. What does Minecraft use for the chat settings in German?
Also please note that cloud based localization tools such as Weblate and Crowdin are sometimes a bit too eager with automatic translations. So it is possible that the translations that you dislike were automatically added. If you ever spot a literal translation that doesn't fit please feel free to change it.

2. Condense option values

It seems that at least the Russian translations need the distinction. But to make this easier for translators they get automatically translated. The current setting for duplicate strings in Crowdin is "Show, but auto-translate them", so if the translations are the same for all occurrences no action is required, but if there are different translations they can be added by translators.
The duplicate string setting used to be "Show within a version branch (regular detection) - duplicates will be hidden only between versions branches" with the intention to automatically sync matching translations to other version branches. I don't know if that still happens, if you end up changing the German translations please let me know if the translations get automatically synced to other version branches.

If we are talking about improvements, is it possible make a translation text in "modmenu.mods" depend on the additional conditions for the number of mods?
For example, there are 4 plural forms in Russian:

one: 1, 21, 31, 101, 1001 etc (excluding 11)
few: 2-4, 22-24, 32-34 etc (excluding 12-20)
many: 0, 5-20, 25-30, 35-40 etc.
other: 1.5, 0.8, 11.3 and other fractional numbers. In most cases, the spelling of the word form coincides with few.

Mathematically, this is determined by the following rules:

one: |n| × 10 + 1 |n| mod 10 ≠ 1
few: |n| × 10 + [2, 3, 4] |n| mod 10 ≠ 1
many: |n| × 10 + [5, 6, 7, 8, 9, 0] ∪ |n| × 10 + [1, 2, 3, 4] |n| mod 10 = 1
other: all numbers with a fractional part, even if this part is equal to 0 (12 → many, 12.0 → other)

Example:
Mod=мод

one мод
few мода
many модов
other мода

As far as I know, similar rules are in German.

Interesting! It is possible but comes with additional complexity. How does Minecraft handle this?

Minecraft has no solution to this problem. For example, the output of the /fill command always uses only the most popular form — many (the same solution is used in ModMenu now).
To be honest, I have not yet met anyone implementing such a system in Minecraft without using predefined sets of translation strings (although I created this for one of my datapacks, but this is a different area).

If we are talking about improvements, is it possible make a translation text in "modmenu.mods" (and other strings using numbers) depend on the additional conditions for the number of mods?
For example, there are 4 plural forms in Russian:

one: 1, 21, 31, 101, 1001 etc (excluding 11)
few: 2-4, 22-24, 32-34 etc (excluding 12-20)
many: 0, 5-20, 25-30, 35-40 etc.
other: 1.5, 0.8, 11.3 and other fractional numbers. In most cases, the spelling of the word form coincides with few.

Mathematically, this is determined by the following rules:

one: |n| × 10 + 1 |n| mod 10 ≠ 1
few: |n| × 10 + [2, 3, 4] |n| mod 10 ≠ 1
many: |n| × 10 + [5, 6, 7, 8, 9, 0] ∪ |n| × 10 + [1, 2, 3, 4] |n| mod 10 = 1
other: all numbers with a fractional part, even if this part is equal to 0 (12 → many, 12.0 → other)

Example:
Mod=мод

one мод
few мода
many модов
other мода

As far as I know, similar rules are in German.

German only has separate words for a single and multiple instances:

1 Mod
2 Mods
3 Mods
...

Oh, ok.

This doesn't mean that the German translation has to follow that, though. What does Minecraft use for the chat settings in German?

From looking at it, do they use the present tense for the option, but also a IMO completely different word.
Like, right now did I translate Shown to Anzeigen. In Minecraft are they using Sichtbar however, which usually translates to Visible in English.
Either way, they use present tense, so I'll update the german translation too (Currently working on swiss translation as that's my most native language... Tho I make a proper translation of it and not the joke-translation like MC did since the grammar is a bit different at certain points.)

One other thing I noticed and that I personally find relatively weird is, that english has all-caps for stuff like "on" and "off" while german uses normal capitalization...
It's not a big deal for me, but I certainly feel confused why there is such a noticeable difference.

@Felix14-v2 would it be enough to add the few form to strings dealing with a number of objects? AFAIK one and many are already supported because English uses them, too.
@Andre601 translations aren't necessarily literal, they are supposed to communicate the same information as the source text but in a different language. When I learned English in school this was called "mediation".
Minecraft is translated by volunteers so the opinion of these volunteers on how to translate things end up in the official translations. Maybe the German volunteers just didn't like the capitalization so they decided not to capitalize in the translation.
My opinion on translations for Mod Menu is that they should follow what Minecraft does as best as possible. Because if Minecraft uses specific language to communicate a concept and this gets translated in multiple different ways in the same language people might get confused.
So since there is a precedent of using Sichtbar as the German translation for Shown that should be used. I believe in this case people wouldn't get confused because it is rather obvious, so using Angezeigt is fine. But if another translator later changes this to Sichtbar it would trump your translation because it is closer to what Minecraft does.
Sadly the same goes for your proper translation to Swiss. I think your passion for your language is great and I am always amazed by the efforts and dedication of translators. But since the official Minecraft Swiss translations are humorous if somebody comes along and changes the Mod Menu Swiss translations to be more like that that would be allowed.
You might be able to update the Minecraft Swiss translation to be proper. Minecraft's Crowdin project can be found here: https://crowdin.com/project/minecraft. If Minecraft itself uses proper Swiss then Mod Menu will aim to use proper Swiss, too.

would it be enough to add the few form to strings dealing with a number of objects? AFAIK one and many are already supported because English uses them
Exactly. We won't need the other form, since we don't use fractional values in ModMenu.

Exactly. We won't need the other form, since we don't use fractional values in ModMenu.

Cool, that's doable. Please open an issue for that. If you know Java a PR is also welcome.

@Andre601 thank you for your suggestions! Since they won't be implemented I'm closing this issue. Please feel free to open another issue if you have any more suggestions.


FileDialog on Gnome 43.2 does not apply setFilenameFilter

Describe the bug
Setting setFilenameFilter for FileDialog is never applied.
Affected platforms
Select one of the platforms below:

Desktop (Gnome 43.2 on debian testing)

Versions

Kotlin version*: 1.8.0
Compose Multiplatform version*: 1.3.0
OS version(s)* (required for Desktop and iOS issues): debian testing
OS architecture (x86 or arm64): x86
JDK (for desktop issues): temurin 17.0.6

To Reproduce
 val dialog = FileDialog(ComposeWindow()).apply {
                        isVisible = true
                        setFilenameFilter { _, name ->
                            name.endsWith(".png") // is never called
                        }
                    }

Expected behavior
setFileNameFilter should filter files in the file dialog.

I think this is a Swing issue that filename filter was never implemented on some platforms: https://bugs.openjdk.org/browse/JDK-4031440
I don't think this is an issue with Compose.

FileDialog is not a part of Compose Multiplatform. You should create an issue in the OpenJDK bug tracker.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Fix: Fixed focus flickering between columns or panes

Resolved / Related Issues

[X] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #12280
Closes #12285

Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[X] Are there any other steps that were used to validate these changes?
#12285 is 100% reproducible with the following steps. This should be fixed once this PR is applied.

Start dual pane mode.
Open any folder that is not the home page in each pane.
Start renaming an item.
Click on the other pane while renaming.
The focus will keep switching between panes.

Start dual pane mode.
Open any folder that is not the home page in each pane.
Start renaming an item.
Click on the other pane while renaming.
The focus will keep switching between panes.

It resolves the flickering, but there's another bug:
if you start renaming and then click on an item on the other pane, it will be selected and then immediately deselected.

if you start renaming and then click on an item on the other pane, it will be selected and then immediately deselected.

I know, but this is another and inconsequential (I belive) issue that already exists.
Similar behavior happens if you click on the address bar while renaming even without applying this PR.


feat: improve instance not found error

Adds the ability for ZITADEL developers to template error messages.
Templates instance not found errors like this:

Errors:
  IAM:
    NotFound: Instance not found{{ if .Host }} by origin {{ .Origin }} and host {{ .Host }}. {{ if ( not ( hasPrefix .Host .ExternalDomain ) ) }}You might try the domain {{ .ExternalDomain }} or check out https://zitadel.com/docs/self-hosting/manage/custom-domain {{ end }}{{ end }}

Results in:
ID=QUERY-1kIjX Message=Instance not found by origin http://<IP_ADDRESS>.sslip.io:8080 and host <IP_ADDRESS>.sslip.io:8080. You might try the domain localhost or check out https://zitadel.com/docs/self-hosting/manage/custom-domain

The {{ if .Host }} condition ensures that instance by ID errors don't include the hints about the domain.
:warning: Use with care, as maintaining the templates for all languages is painful
For example in this PR, languages, that were too hard to translate, have the variables just appended.
Japanese, makedonian and Chinese are printed like the following:

ID=QUERY-1kIjX Message=Инстанцата не е пронајдена. Проверете https://zitadel.com/docs/self-hosting/manage/custom-domain. Origin=http://<IP_ADDRESS>.sslip.io:8080, Host:8080=<IP_ADDRESS>.sslip.io, ExternalDomain=localhost
ID=QUERY-1kIjX Message=インスタンスが見つかりません https://zitadel.com/docs/self-hosting/manage/custom-domain. Origin=http://<IP_ADDRESS>.sslip.io:8080, Host:8080=<IP_ADDRESS>.sslip.io, ExternalDomain=localhost
ID=QUERY-1kIjX Message=实例未找到。查看 https://zitadel.com/docs/self-hosting/manage/custom-domain. Origin=http://<IP_ADDRESS>.sslip.io:8080, Host:8080=<IP_ADDRESS>.sslip.io, ExternalDomain=localhost

Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

I would like to have a discussion if templating is really the best approach here. As you mentioned yourself, managing translations will be difficult and hard to maintain. Why don't we just attach key/value context to the error message, just like loggers do. For example (eg err = zerrors.WithValues(err, map[string]any)  and print that like:
ID=QUERY-1kIjX Message=Instance not found. Check out https://zitadel.com/docs/self-hosting/manage/custom-domain exernal_domain=zitadel.cloud origin=https://mysite.com host=https://wrong.zitadel.cloud

As the user story mentions we want more guidance on the error. Doesn't simply changing the message to Instance not found. Most probably because of a typo in the requested host. Checkout <URL> solve that entirely?
Perhaps moving the URL to a documentation page that is not only applicable to self-hosters? Because the error also affects cloud users. What about https://zitadel.com/docs/apis/introduction#domains? And there be more elaborate on how zitadel uses the domain to determine the instance.


_eyetrack.json wrongly define StartTime and StopTime

For the first session we have:
{
    "StartTime": 5549944.0,
    "StopTime": 7350028.0
}

And the data:
In [1]: import pandas as pd

In [2]: pd.read_csv("/data/datasets/hcph/sub-001/ses-001/dwi/sub-001_ses-001_acq-highres_dir-LR_eyetrack.tsv.gz", sep="\t", index_col=None)
Out[2]:
         5549944.0         289.5         566.0  5881.0  0.0  0.0.1  0.0.2
0        5549945.0  2.892000e+02  5.664000e+02  5877.0  0.0    0.0    0.0
1        5549946.0  2.889000e+02  5.669000e+02  5872.0  0.0    0.0    0.0
2        5549947.0  2.886000e+02  5.675000e+02  5867.0  0.0    0.0    0.0
3        5549948.0  2.885000e+02  5.673000e+02  5866.0  0.0    0.0    0.0
4        5549949.0  2.883000e+02  5.668000e+02  5865.0  0.0    0.0    0.0
...            ...           ...           ...     ...  ...    ...    ...
1800079  7350024.0  1.000000e+08  1.000000e+08     0.0  0.0    1.0    1.0
1800080  7350025.0  1.000000e+08  1.000000e+08     0.0  0.0    1.0    1.0
1800081  7350026.0  1.000000e+08  1.000000e+08     0.0  0.0    1.0    1.0
1800082  7350027.0  1.000000e+08  1.000000e+08     0.0  0.0    1.0    0.0
1800083  7350028.0  1.000000e+08  1.000000e+08     0.0  0.0    0.0    0.0

[1800084 rows x 7 columns]

This is suspicious as the recording starts before the task kicks in (more so in the case of the DWI where the first trigger happens with calibration). For these files to be useful, StartTime and StopTime cannot be trivially set to the first and last timestamps of the array.

Can you clarify what you're expecting in StartTime? Currently, it marks when the eye-tracker recording begins. The value of the timestamp for DWI seems fine to me—I checked it from the EDF file messages. The task started at 5549930 ( These values are relative to when the eyetracker was started) , and eye-tracking kicked in at 5549944. You can double-check in the bidsphysio Docker image output, specifically in 'sub-001_ses-001_acq-highres_dir-LR_eventlist_raw.tsv.' The timestamp 5549944 matches the message '!MODE RECORD CR 1000 2 0 R,' which I believe signals the start of recording.

Currently, it marks when the eye-tracker recording begins.

Which is moot, because by this definition it's going to be the first and last rows in the timestamp column (the first). The problem is that the BIDS specification (or the BEP in this case) is very clear that StartTime and StopTime indicate the timestamp when the run started and ended. This is clearly NOT when the ET starts recording and stops recording, but rather when the first trigger kicks in and when the last volume of the dwi/fmri is acquired.
Without appropriate StartTime and StopTime we cannot synchronize the MRI experiment with the eyetracking recording.

Ok, the concept of 'run' in the context of the eye-tracker was a bit unclear for me. Regarding StartTime, we can use the timestamp linked to the onset of the psychopy task. However, pinpointing the exact timestamp of the last volume in the eye-tracking (ET) data will be challenging. We can, however, retrieve the end of the psychopy task, even though it might not be perfectly synchronized with the final volume in the ET data.


Add a guide for a basic auth set up using Lucia

The docs right now doesn't provide an easy and approachable way to start using Lucia. It shows you how to set up Lucia and from there it's all up to you. I think it can be a bit more hand hold-y. I think it'd be easier for users to understand how Lucia works and how it's used if there's a super basic tutorial on getting username/password auth setup with SvelteKit. The tutorial should cover most of the topic in the basic section of the docs.
Something like this from the old docs: https://lucia-sveltekit-rd8onucrw-pilcrowonpaper.vercel.app/guides/email-and-password

Done with #147


Acceleration parameters missing in TSV output

The three parameters are present in the object, but not found in the to_tsv method. Please update the method to also write out the X, Y, and Z axis acceleration parameters.
Thanks goes to Gryphon962 for reporting this!

The commit adds these, appears to be working correctly.


Order all imports lexicographically

Motivation

Part of #2136

Changes

Adds OrderedImports rule to .swift-format
Removes a trailing space in format.py
Formats the codebase to comply with OrderedImports

@swift-ci please test

@kimdv, thank you for reviewing this! ❤


New package: tschoonj.GTKForWindows version 3.24.31

[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.

Microsoft Reviewers: Open in CodeFlow

Hi @licon4812,
This package appears to be a Batch file. Is this the case?

the gtk is a UI library for dev. It isn't ran it is just something that is added to path for Frameworks such as Uno to utilize. The crux of the tool should be in bin


install command-line error

Following the doc to install template, a error that "no matchs found" occured.
doc location:
en-US
zh-CN
It seems it works without the suffix "-*"
environment:
Ubuntu 20.04LTS
shell : zsh
dotnet version 6.0.100
 λ >  dotnet new --install AntDesign.Templates::0.1.0-*
zsh: no matches found: AntDesign.Templates::0.1.0-*

 λ >  dotnet new --install AntDesign.Templates::0.1.0
The following template packages will be installed:
   AntDesign.Templates::0.1.0

Success: AntDesign.Templates::0.1.0 installed the following templates:
Template Name          Short Name  Language  Tags
---------------------  ----------  --------  --------------
Ant Design Pro Bla...  antdesign   [C#]      Ant Design Pro

So, what cause this error and if there's any diff between "AntDesign.Templates::0.1.0-*" and "AntDesign.Templates::0.1.0" ?

Thanks for contacting us @HUGHNew
You can install the latest version now and no need to specific the preview version.
dotnet new -i AntDesign.Templates

Got, thx


Feature: Admins can delete connectors and resources

As a Broker administrator I need a simple way to delete a resource or a connector, because there are connectors and resources that cannot be unregistered by the connector admins themselves.
E.g. IDs of resources are created by the connector automatically. This is intended. If the database of a connector is deleted for whatever reason then a resource with the same ID cannot be recreated. Since only resources that exist on a connector can be deleted by a connector there is no way to delete a previously existing resource.

@sebplorenz  As discussed earlier the resources were not being removed from indexing but in the newer versions, they are deleted from Indexes also just like Connector.

@sebplorenz  This issue has been fixed and we have provided you with the newer version. Hence I am closing this ticket.

Do we have a delete button for connectors and/or resources in the Broker now?

Again, do we have a delete button now. I have a request for deleting a connector from a customer. Since the customer has updated his connector to a new version (probably with another name), he is not able to delete the old entry in the broker. Thus having two entries of the connector in the broker.


ott: 0.27 -> 0.28

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/ott/versions.
These checks were done:

built on NixOS
ran ‘/nix/store/2lbl8zpp2lrrh9pgh2gnyhimq6i86rl1-ott-0.28/bin/ott --help’ got 0 exit code
ran ‘/nix/store/2lbl8zpp2lrrh9pgh2gnyhimq6i86rl1-ott-0.28/bin/ott.opt --help’ got 0 exit code
found 0.28 with grep in /nix/store/2lbl8zpp2lrrh9pgh2gnyhimq6i86rl1-ott-0.28
directory tree listing: https://gist.github.com/177f63b8c23bae6301ced29fb0e617c4

cc @jwiegley for review

Success on x86_64-darwin (full log)
Attempted: ott
Partial log (click to expand)
ln -s -f ott.opt ott
make[2]: Leaving directory '/private/tmp/nix-build-ott-0.28.drv-0/source/src'
cp ./ott ../bin/ott
make[1]: Leaving directory '/private/tmp/nix-build-ott-0.28.drv-0/source/src'
installing
post-installation fixup
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/k6b01d3ph1n1jgsh1f83n0a6xdjh4f7z-ott-0.28/bin
patching script interpreter paths in /nix/store/k6b01d3ph1n1jgsh1f83n0a6xdjh4f7z-ott-0.28
/nix/store/k6b01d3ph1n1jgsh1f83n0a6xdjh4f7z-ott-0.28

Failure on aarch64-linux (full log)
Attempted: ott
Partial log (click to expand)
make[3]: *** [../Makefile:74: unix.cmxs] Error 2
make[3]: Leaving directory '/build/ocaml-4.05.0/otherlibs/unix'
make[2]: *** [Makefile:1055: otherlibrariesopt] Error 2
make[2]: Leaving directory '/build/ocaml-4.05.0'
make[1]: *** [Makefile:465: opt.opt] Error 2
make[1]: Leaving directory '/build/ocaml-4.05.0'
make: *** [Makefile:531: world.opt] Error 2
builder for '/nix/store/ajqgdzi3mmwacpk8mzfja0ivpxhdxwsh-ocaml-4.05.0.drv' failed with exit code 2
cannot build derivation '/nix/store/5l5n8jxri9bl8b1z3xykvw8vczna6j97-ott-0.28.drv': 1 dependencies couldn't be built
error: build of '/nix/store/5l5n8jxri9bl8b1z3xykvw8vczna6j97-ott-0.28.drv' failed

Success on x86_64-linux (full log)
Attempted: ott
Partial log (click to expand)
these paths will be fetched (0.87 MiB download, 4.29 MiB unpacked):
  /nix/store/2lbl8zpp2lrrh9pgh2gnyhimq6i86rl1-ott-0.28
copying path '/nix/store/2lbl8zpp2lrrh9pgh2gnyhimq6i86rl1-ott-0.28' from 'https://cache.nixos.org'...
/nix/store/2lbl8zpp2lrrh9pgh2gnyhimq6i86rl1-ott-0.28


Bump AK version to 5.5.1

This is a manual version Bump to 5.5.1
Upon approval, I will git push this change directly to confluent/kafka.

Job failed on the very thing I'm trying to fix here:
[INFO:2020-04-23 01:42:41,888]: [ERROR] Failed to execute goal on project confluent-connect-secret-registry-plugin: Could not resolve dependencies for project io.confluent:confluent-connect-secret-registry-plugin:jar:5.5.1-SNAPSHOT: The following artifacts could not be resolved: io.confluent:rest-authorizer:jar:5.5.1-ce-SNAPSHOT, io.confluent:authorizer:jar:5.5.1-ce-SNAPSHOT: Could not find artifact io.confluent:rest-authorizer:jar:5.5.1-ce-SNAPSHOT in confluent-artifactory-jenkins (https://confluent.jfrog.io/confluent/maven-public) -> [Help 1]

(The 5.5.1 bump).


Datepicker doesn't display the selected days

I have add a button to set some of selected days when it's clicked, but it doesn't render the selected days correctly until I select another days. For example when I click the button I will set 11, 13, 15 May 2017 as the selected days, as shown by this code:
html
<button ng-click='$ctrl.setDate()'>Set Dates</button>
js
this.setDate = function() {
    _this.selectedDates = [
      new Date('2017-05-11').setHours(0, 0, 0, 0),
      new Date('2017-05-13').setHours(0, 0, 0, 0),
      new Date('2017-05-15').setHours(0, 0, 0, 0)];
  }

The datepicker doesn't set the date of 11, 13, and 15 in blue highlighted background color, but if you click another days for example, 20 May 2017 then the datepicker will set the 11, 13, 15, and 20 in blue highlighted.
Furthermore, if you unselect the 20 May 2017 by clicking it once again and click the Set Dates's button you cannot select another days.
Here is the plunker to show the problem: http://plnkr.co/edit/yUyYcANwSxtmpTsgCmi1?p=preview

I found a workaround for the 'setting dates not updating view' issue in #57. Please see the following plunk for a fix:
http://plnkr.co/edit/jhJ8KWqAb7aX1pqqZ7xy?p=preview
The second issue is a puzzler. I'd have to dig further to find out what's going on.


Methods for time series classification

try out sktime

[ ] RocketClassifier
[ ] HIVECOTEV2
[ ] KNeighborsTimeSeriesClassifier

reference

Time Series Classification with sktime

Dataset :x: -> :o:

check load_all_data for non-shuffle dataset

[x] split_data :o:
[x] getting the segment :o:

segment_with_threshold
get_segment_data

[x] downsample :x:

didn't seed np.resample

Results:
After seeding the np.resample in  downsample , the load_all_data 's output stays the same :o:

check load_all_data for shuffle dataset
First Try: output shape is not consistent

[x] _shuffle :x:

Results:
After change the if not self.randome_state to  if self.randome_state :o:

ROCKET transform :o: