incident plugin, backend and frontend

Author two new plugins designed to work together:

plugins/incident => to display associated incident.io incidents in the entity dashboard page, under an overview card.
plugins/incident-backend => syncing catalog data from Backstage into incident.io using the scheduled tasks system.

Users of this plugin are expected to sync Backstage data into incident.io in some way – either using the backend plugin in this commit, or the catalog-importer – and create incident custom fields such as "Affected components" which are powered by the synced catalog data.
Then the frontend plugin is configured to point at those fields so it can find any associated incidents.

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

As an FYI for upstream maintainers: I expect we won't merge this, and will create a separate plugin repo. It just made sense to PR this against the example app while we figure things out.


InlineCompletion PartialAccept additional info

TLDR; Would it be possible to extend the handlePartialAccept with metadata field, allowing to attach additional info, such as origin?
It would help understanding in the downstream completion metrics, where acceptances are comming from to better understand user behavior. The field could be later on extended with any additional information if neccessary, e.g. in the case of partial accept it might be helpful to report back also the length of the accepted token (not only the length of the full acceptance from start of insertion to cursor position ).

That sounds good to me, thanks!
Do you preffer I add the change in to new proposed API, or extend the current ?

Going through what I think would be interesting to have in the additional metadata I settled on this:

replacementRange
partialAcceptanceKind
documentVersionBeforeAcceptance
text [very optional as it can be directly extracted using the replacement range + added chars]

The reason for that is, to be able to get a better visibility on what happened in the core without having to hack in addtional listeners for typing/cursor changes, directly from the partialAccepts. The data mentioned above would allow to easily reconstruct all edits that happened in a document, and allow for better post processing of the data.
I decided to split this in to 2 PRs:

1st - proposing the structural changes going through the whole stack (proposed API - extHost - mainThread - Core) and adding the base types
2nd - adding the other properties & convertors (which should be a smaller PR)

The 1st is being sent out right now, while the 2nd will be sent out after discussion, as there may be some points that we might want to clarify further.


Connector-1.0.0-beta.24+1.20.1 still doesnt work with Cobbreeding cobblemon add on mod

Mod Name
Cobbreeding
Mod Homepage
https://modrinth.com/mod/cobbreeding/versions
Describe the bug
exit Code Zero crash on Startup
Steps to reproduce

cobblemon forge add on mods the mod and question load and crash kotlinforforge as well
latest.log

Logs
uploaded it
Additional context
No response

Both Cobblemon and Cobbreeding work fine for me on beta 24

Closing as stale / cannot reproduce.
If you can provide the necessary logs and information to reproduce the problem or identify the faulty mod, please open a new issue with all necessary details.


Use a pull model to withdraw deposit ETH balances

Push models have security and stability concerns, a pull model allows for better security, as well as more flexibility and freedom to third-party contract fallbacks given .call.value()'s gas-forwarding.

Accumulated Eth balance can only be withdrawn after the contract reaches an end-state

close in favor of #570


test browser extensions and browser modes

It will be interesting to look at popular browser extensions, such as uBlock Origin and Privacy Browser, and "extra privacy" modes, such as Firefox Strict Mode.

Another suggestion: pi hole

Hi, take a look : https://github.com/rztprog/Firefox-extentions

Hi, take a look : https://github.com/rztprog/Firefox-extentions

Thank you! Great list.


enh: Place executables in "tools" subdirectory, add "mirtk" Python package [CMake, Applications]

Improves the mirtk wrapper script implementation and enables users to import the mirtk Python module in their own Python (pipeline) scripts to call the MIRTK commands from Python using the mirtk.call function (instead of subprocess.call). The PATH and [DY]LD_LIBRARY_PATH environment variables can be modified by calling mirtk.putenvs to insert the paths of the external shared libraries that are required by the MIRTK commands. Otherwise the user has to manually set up the environment before calling mirtk.call.

As a side effect, this changes the build tree and installation location of the MIRTK command executables. They can now be found in the lib/tools (e.g., when <prefix>=/opt/mirtk) or lib/mirtk/tools (e.g., when <prefix>=/usr/local or <prefix>=/usr) subdirectory, respectively.


fix deploy with secrets docs

Yep! Added https://github.com/basetenlabs/docs.baseten.co/pull/89 to address that, now going to look at root cause.


User-specific keyboard layout with desktop manager

As far as I can tell the only way to set the keyboard layout is with environment variables when sway is started. This means that root must change the .desktop file in order to change the keyboard layout. As far as I can tell there isn't a way to change the keyboard layout after the session has started. (In x this could be done with setxkbmap, but that doesn't  work in wayland).

This has been discussed and there are not presently any plans to support this. Environment variables are the way to go for now.

But is there any way to set these environment variables for a single user?

Write a script?


Fix managed certs validation prompt

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

If you want to release a new version for this extension, please also upgrade the version defined in setup.py

We decided not to bump the version in this PR. Should I mark the notes of changes in History.rst as "Upcoming"?

Yes, sounds good!

We decided not to bump the version in this PR. Should I mark the notes of changes in History.rst as "Upcoming"?

Yes, sounds good!

Marked as Upcoming. Could you please take another look when you get a chance? Thanks!


Dots component does not account for large values in case of infinite carousel

When infinite option is turned on, dots will scroll to indexes 0-n (where n is index of last element). When current index is very large (in case of autoplay value is being incremented so it can be any large number) clicking on dot or thumbnail will make carousel animate through all those values instead of to the nearest one.
I see two solutions:

Just calculate nearest one in Dots component
Make mechanism to reset value in case of infinite (can be tricky as this is controlled component: triggering onChange, will in turn trigger state change and Carousel will have difficult time figuring out if the change was just it's resetting mechanism and animation is unnecessary or it's triggered from outside and we should animate)

Fixed in https://github.com/brainhubeu/react-carousel/pull/46


bump node 15 LTS

Applied mkpr on undefined
sed s/"node": ">=16.14.[0-9]"/"node": ">=15.15.0"/

:tada: This PR is included in version 1.16.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Factor out DescriptiveStats.percentileInPlace

The percentile function always copies and sorts the input data, which provides good safety for the general case. This is inefficient, however, in cases where data already happens to be sorted or where the user wants to calculate a series of percentiles on a single dataset.
We factor out the sorting step and the percentile calculation step here and make a public percentileInPlace function that takes in a pre-sorted array so that users can opt in to the responsibility of array creation and sorting on their own.
Note that I'm relatively new to scala breeze, so very open to feeback on naming, documentation conventions, etc. If this simply doesn't fit in with the overall API you want to present to users, that's a perfectly fine outcome as well. Thanks for your work on this library!

thank you for doing this. sorry for being slow

Thanks, @dlwh!


neomutt: support reverse sorting methods

Per the neomutt documentation:

You may optionally use the "reverse-" prefix to specify reverse sorting order (example: " set sort_browser=reverse-date").

I figure it's best to not hard-code the reverse sorting options, but maybe map over the existing enum and include reverse- options?
Or is this not best practice when it comes to defining type defaults and such?
cc @teto

you refer to the set sort = "${cfg.sort}" I suppose ?
Apparently we don't describe every possible solution so either we can fix it or leave it freeform.
I believe it's best to not hardcode things but pass is as some structured default that can be overriden. Feel free to send a patch.

Oops yes forgot to link to the option.
Sounds good, I'll take a look at some implementations.

also seems like it would make more sense as an account specific setting. Along with most settings in fact... my bad


k3s installation script failing

Environmental Info:
K3s Version: latest/stable

Node(s) CPU architecture, OS, and Version:
*
Cluster Configuration:
*
Describe the bug:
We cannot use the convenience script to install k3s
Steps To Reproduce:
❯ curl -sfL https://get.k3s.io | sh -
[INFO]  Finding release for channel stable
[INFO]  Using stable as release
[INFO]  Downloading hash https://github.com/k3s-io/k3s/releases/download/stable/sha256sum-amd64.txt
[ERROR]  Download failed

Expected behavior:
Installation Success
Actual behavior:
Installation Failure due to failure to download
Additional context / logs:
Tried removing sha verification still fails to download binary. Seems like Github has deprecated certain URLs.
https://update.k3s.io/v1-release/channels
The host is down for this website,

Hi !
There seems to be something wrong with your DNS server setup. In my case, it is normal

Our backend channel servers appear the be experiencing intermittent issues. I've reached out to our infrastructure team.

Hey Folks, It seems to be working as expected now.
@brandond thanks for forwarding the issue.
@Miaoxiang-philips the issue was happening across machine and even in our Github CI runners 😅 Our runners are now running as expected tho.

Reopening to pin

We could probably enhance the install script to be a little more resilient to channel server outages.

Just to share a little bit about what this looks like on our side - this seems to be triggered by an anomalous use pattern:

sudden influx from a usual 50k a min to a spike over 8mil a min and it stayed around 1-1.5 mil for several hours

Currently seeing this as well. It sounds like this is something expected to clear up soon? Or is this something you need help/logs to debug? Happy to provide assistance if I can.

I am also facing this issue.

i have done this by passing the version : curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=v1.29.6+k3s2 K3S_KUBECONFIG_MODE=777 sh -

I am also experiencing this issue.

Hello @brandond now when I want install cluster - I have download error hashsoum  - https://github.com/k3s-io/k3s/releases/download/v1.29/sha256sum-amd64.txt .Could you resolve that?

Hello @brandond now when I want install cluster - I have download error hashsoum - https://github.com/k3s-io/k3s/releases/download/v1.29/sha256sum-amd64.txt .Could you resolve that?

I am experiencing the same issue on v1.29.3+k3s1 install k3s script
sudo bash k3s_install.sh --write-kubeconfig-mode "0644" --docker"
[INFO]  Finding release for channel stable
[INFO]  Using stable as release
[INFO]  Downloading hash https://github.com/k3s-io/k3s/releases/download/stable/sha256sum-amd64.txt
[ERROR]  Download failed

@alexlianides — I updated my comment. I previously copied the text from @Aias00's comment but after seeing yours realized that was not exactly what is working for me.
This is working for me right now:
curl -sfL https://get.k3s.io/ | INSTALL_K3S_VERSION=v1.30.3+k3s1 K3S_KUBECONFIG_MODE=777 sh -

@alexlianides — I updated my comment. I previously copied the text from @Aias00's comment but after seeing yours realized that was not exactly what is working for me.
This is working for me right now:
curl -sfL https://get.k3s.io/ | INSTALL_K3S_VERSION=v1.30.3+k3s1 K3S_KUBECONFIG_MODE=777 sh -

@mikeschinkel while this fix does work, this fix introduces security issues due to the open permissions it sets to the k3s cluster. Thank you for the tip though.
@brandond is the current issue still due to a server availability issue? or is this a new error?

It's the same thing. When it's resolved we'll close and unpin the issue. I don't have any additional information to provide.
Note that only the INSTALL_K3S_VERSION argument is necessary to work around the channel server outage. You can reference the docs for more information on passing args to the install script.


Adding Quota option for ZFS

Proposed changes
Try to explain how you can use quota on ZFS as it was implemented/discussed in #3804.
Related issues (optional)
#3804
Signed-off-by: Andreas Steinel<EMAIL_ADDRESS>

Hi John,
if you need additional changes, just apply them and we close this pull request. It feels like a lot of overhead just for a simple doc enhancement. I just wanted to raise the awareness that this should be in the docs.
Best,
Andreas

Thanks, @lnxbil :)


Docker container

Hi there. I found myself wanting to use this on a number of machines which all had docker installed. I quickly build your script into a distroless docker container. I've also setup a Github action to publish it as a package so it's available more widely.

Hey, thanks for the contribution but I'm not sure if it would be very useful, mind sharing your opinion?

As I mentioned, this is useful when you want to use this application on a system which may not have node/npm installed, but does have a docker runtime. You can easily use this in a single command without having to install any software. With the --rm flag, there is no extra cleanup required (uninstalling etc). The same command can be used on multiple systems with the exact same results, regardless of the environment.

Was hoping for something like this, tested it on my server and it works great!

Can Multiarch support be added?

This is perfect ffor speedtesting K8s on-prem nodes and my swarm of Flatcar linux vms

I see someone posted up a docker version of this at https://hub.docker.com/r/p3terx/speed-cloudflare


Add ServiceExport Object for Discovery Server

What would you like to be added?
Discovery server custom resource currently maintains three Kubernetes resources:

a pod (if .spec.type is Server)
a service (if .spec.type is Server)
a config map

Discovery server should maintain ServiceExport object to enable connection between two clusters over VPN.
Why is this needed?
It's needed to solve connectivity issues between two cluster's namespaces.

Information about ServiceExport can be found here.


Sandcastle terrain example cleaned up

#4575

#4568

Thanks for the contributions, @abhishekvp!
@erikmaarten do you want to review this?
All - we have a CLA from @abhishekvp


[core] The record level expired time field type is automatically recognized.

Purpose

The record level expired time field type is automatically recognized.
The user is not required to enter the field type record-level.time-field-type, and the input may be incorrect.

Tests

API and Format

Documentation

+1, nice job


Custom styling does not work for DropdownMenu - Item, Content, Title

I've been trying to use custom styles to change the background color or text color of each item on the menu, but the style prop seems to be ignored by each component

native menus don’t allow styles like that.
see this for what is possible on iOS: https://github.com/dominicstop/react-native-ios-context-menu

style is only used on Web.

The screenshot on the home page shows a menu for ios with gray, black, and red text. How can we implement that without style? Or is that feature not supported

Can you try destructive and disabled


improve support for overzooming

Some vector tile servers only supply tiles up to zoom level 14, however it's desirable to be able to display maps at higher zoom levels. Mapping libraries commonly support rendering tiles at a higher zoom level so that it's possible to zoom in further. This is called overzooming.
This library supports overzooming, however overzooming by several levels can cause excessive memory usage to the point of crashing the application as described in issue #24.
The investigation in issue #24 revealed that memory usage was driven by raster picture memory usage, increasing exponentially with overzoom level. In testing, I found that overzooming had to be limited to 2 levels to keep memory usage within reasonable limits imposed by mobile devices (~2GB)
Digging deeper, I discovered that line data in vector tiles can extend far beyond the tile boundary. In one example, with a tile having a size of 4096, I observed at least one line extending to an offset of 58,000. It's likely that tile servers are clipping lines based on line segment points, rather than trimming lines to the tile boundary.
In an experiment I was able to determine that applying a clip to lines and dropping polygons and points that do not intersect the clip bounds resulted in stable memory usage. Ad-hoc testing has shown raster picture memory usage < 80MB even when zooming up to level 22 on a level 14 tile.
Clipping is CPU-intensive, so could potentially contribute to lag or jank. Further, the approach taken in the experiment causes an intermediary tile model to be created which further adds to computational overhead, both by creating the model and in GC. Some of this overhead can be alleviated by performing model calculations on a non-UI isolate. More investigation is needed to determine whether CPU overhead can be reduced further.
Though the approach has proven promising, it's possible that it doesn't fully solve the problem. Since polygons are not reshaped as part of the clipping process, it's possible (for example) that a large polygon extending into the clip bounds only partially could cause high memory usage as previously observed with lines. I haven't encountered such an issue in testing, however it may be that I got lucky with the tile data that I used. Alternatively, it's possible that Canvas-native clipping works better with polygons than it does with lines. Further testing is needed to determine if the problem is fully resolved with the clipping approach in its current form.
Anyone interested in trying out this experiment will need the following branches:

flutter-vector-map-tiles/experiment-clip
dart-vector-tile-renderer/clipping

🚀


Addressed a number of issues flagged by Jenkins

Contains
Fixes for a number of warnings given by Jenkins during the build process.
Jenkins' output can be found at the end of each PR, under the Details link. The issues this PR has been addressing are just a small portion of the total, but have been found in this specific build output.
How to test

start Terasology
load a game
everything should work normally

Note

Given the breadth of the changes, even though many are small and probably inconsequential, we need to check if this PR gives any problem to known modules. I.e. a number of changes regard unused methods and fields. Perhaps modules were using them? This would need verifying in a workspace including all the modules. But who, who could have such a megaworkspace?
I've used @SuppressWarning annotations in some cases. But I don't know if Jenkins will recognize them as I've used only default strings, i.e. "unchecked" or "access-method".

Hooray Jenkins reported success with all tests good!

The second commit has the same message because I thought I was amending the first but didn't quite work. :confused:

Hooray Jenkins reported success with all tests good!

By the way, @indianajohn a good number of warnings are generated in
org.terasology.rendering.openvrprovider (link to the build output)
Can you address them now that the code is still "fresh"? :)

Strange. The build for the second commit found a bunch of new warnings. But it doesn't make sense because the second commit only modifies a couple of files. :confused:

@emanuele3d likely that's just from the PR job in Jenkins jumping around between different PRs. So the change in warnings between builds might be reporting a comparison to a completely different branch :-)
More of a safe thing to judge after we merge. Which I'll do momentarily!

Tested out briefly in my mega-workspace - no issues noted. The quirky "Access could be package-private" warnings (by IntelliJ, I think, more than Checkstyle) are all over the place. Not really doing much about them as they're a best practice possibility, but a bit of an edge case I don't think we've thought much about. Really low priority in any case and might even just look weird to some "Huh, shouldn't this explicitly be public or private?"
I ran over 4 classes in the OpenVR package real quick since one change was made to one of them anyway (other style fix PR). Most the warnings are gone but a few trickier ones may remain.


Crashing when theirs no storm

The engine will crash if you play in the game mode "FortniteCloneGameMode"
I am working to fix this btw

This Issue has now been fixed


Can it be ported to rails 4?

Hi, I would like to port this gem to rails 4, any pointers?

Hi, have you some news for Rails 4 ? :)

@alagunambi Does this work well with Rails 4.2?

It is not currently compatible with Rails 4.
I'm hoping to update the gem soon. I would welcome a PR for the update if you're interested.
Jeremy
Sent from my iPhone

On Jan 21, 2016, at 22:41, Al<EMAIL_ADDRESS>wrote:
@alagunambi Does this work well with Rails 4.2?
—
Reply to this email directly or view it on GitHub.

Guys version 2.0.1 is made compatible with rails 4. Try it..

https://github.com/jrmyward/honor/pull/3 fixes this issue.
Thanks again @rahul100885 for the PR!


AGEM sometimes blocks on Split CIFAR 100

This requires further investigation

The problem does not happen with GEM, nor with AGEM on Permuted MNIST.
On Split CIFAR-100, the first experiences are fine, but training becomes increasingly slower up until a point in which it is basically stuck. The problem seems to be somewhat related to dataloading of / backpropagation on the buffer sample at each iteration.

wild guess: there is a missing detach somewhere and it's backpropagating where it shouldn't?

may be, but it is strange that this happens only with CIFAR-100 then. It seems more of a dataloading issue (still strange). I plan to profile the steps and get some more insights.

The bug has been tracked and solved in the avalanche repo at this issue


Use opt_einsum for message passing in SVI

Adresses #915
Pair coded with @eb8680
This uses the opt_einsum package to implement a cheap batched sumproduct() operation which is then used in TraceEnum_ELBO to perform message passing. This allows us to marginalize out discrete latent variables. E.g. for HMMs, this allows linear-time inference (per SVI step).
Currently this implementation does not handle iarange and falls back to naive inference for models with iarange.
Tested

[x] added tests for pyro.ops._einsum module
[x] added new tests for TraceEnum_ELBO
[x] enabled larger HMM test cases

Codecov Report

Merging #1313 into dev will decrease coverage by 0.39%.
The diff coverage is 88.55%.

@@            Coverage Diff             @@
##              dev    #1313      +/-   ##
==========================================
- Coverage   95.78%   95.39%   -0.39%
==========================================
  Files         117      119       +2
  Lines        6063     6357     +294
==========================================
+ Hits         5807     6064     +257
- Misses        256      293      +37

Impacted Files
Coverage Δ

pyro/infer/traceenum_elbo.py
97.78% <100%> (ø)
:arrow_up:

pyro/ops/sumproduct.py
61.64% <61.64%> (ø)

pyro/ops/einsum/deferred.py
97.21% <97.21%> (ø)

pyro/infer/util.py
96.25% <97.62%> (-2.06%)
:arrow_down:

Continue to review full report at Codecov.

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


Add support for CR line endings

No thanks. Intentionally missing. Nobody uses those anymore.

Okay, I thought so. Thanks for your fast responses.


[WIP] Show onboarding before init

Summary
A description of the changes proposed in the pull request.
Fixes #number
Checklist

[ ] Change is small and easy to review (please split big changes into multiple PRs)
[ ] Change is consistent with architecture
[ ] Change is consistent with test architecture
[ ] Change is consistent with naming conventions
[ ] New code is covered with tests
[ ] Tests related to old code are updated
[ ] Documentation is up to date with changes

We cannot introduce async methdods in init without waiting for it


1.0.0 API docs

API has settled for 1.0.0.  Just needs docs!

Add usage too.

Closed by https://github.com/yarn-co/cors-light/commit/8829fa32c62dfa025fc1f7820090c9b8c5ea2093.


PeerDep Problem

Hi, I encounter the following problem when installing:
georgejor$ npm install --save react-skylight
npm ERR! Darwin 14.3.0
npm ERR! argv "/Users/georgejor/.nvm/versions/node/v0.12.5/bin/node" "/Users/georgejor/.nvm/versions/node/v0.12.5/bin/npm" "install" "--save" "react-skylight"
npm ERR! node v0.12.5
npm ERR! npm  v2.11.2
npm ERR! code EPEERINVALID
npm ERR! peerinvalid The package object-assign does not satisfy its siblings' peerDependencies requirements!
Please help! Thanks!

Can you understand where that object-assign dependency comes from? It doesn't look like react-skylight depends on it directly. Do you have some packages installed in your project already?


Make util scripts more graceful

The util scripts should:

Check that required CLI projects exist on the PATH (om, pivnet, bosh, etc)
Automatically set ENV_DIR if it is unset

do you think there would be any benefit to rolling these into the omg-cli as sub-commands?


Support .catchall for dynamic field names

See #289
I did not add types to the constraints, tainted and errors as I did not really understand the specifics of the SuperStruct type :sweat_smile:

This should work in v2, so if you're testing the latest alpha and it doesn't work with your use case, open an issue.


improve selection output serialization

column delimiter of output serialization is consistent with input
del superfluous char like "," in output row end

Signed-off-by: Dai Zhiwei<EMAIL_ADDRESS>

thanks for contributing.
indeed the csv_defintion should contain the input and output serialization.
the getMatchRow has access to csv_defintions, and also producing the results.
the star-operation is currently not enabling a generic approach (i.e. different input types) for such an operation.

https://github.com/ceph/s3select/pull/86/
the PR implements output-serialization by the user requests (this PR depends on other PR which add abstraction for projection values)


Add a command to check the database configuration

Codecov Report

Merging #377 (470269d) into master (f178427) will decrease coverage by 0.57%.
The diff coverage is 90.47%.

@@             Coverage Diff              @@
##             master     #377      +/-   ##
============================================
- Coverage     98.38%   97.81%   -0.58%
- Complexity      320      335      +15
============================================
  Files            40       41       +1
  Lines          1052     1096      +44
============================================
+ Hits           1035     1072      +37
- Misses           17       24       +7

Impacted Files
Coverage Δ

src/Service/ReleaseChecker.php
100.00% <ø> (ø)

src/Service/Setup/EnvironmentBuilder.php
93.22% <ø> (-0.12%)
:arrow_down:

src/Service/ApplicationContext.php
91.42% <72.72%> (-8.58%)
:arrow_down:

src/Command/Database/RestoreCommand.php
90.47% <80.00%> (-9.53%)
:arrow_down:

src/Command/Database/DumpCommand.php
92.50% <83.33%> (-7.50%)
:arrow_down:

src/Service/Setup/ConfigurationFiles.php
97.50% <90.90%> (-2.50%)
:arrow_down:

src/Service/Middleware/Database.php
97.14% <97.05%> (+5.61%)
:arrow_up:

src/Command/Database/DetailsCommand.php
100.00% <100.00%> (ø)

src/Command/UpdateCommand.php
100.00% <100.00%> (ø)

src/Service/Middleware/Binary/Docker.php
100.00% <100.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


[DO NOT MERGE] soc/intel_adsp: IPC4 mtrace logging support

[DO NOT MERGE] soc/intel_adsp: implement IPC4 mtrace layout

This is a proof-of-concept that modifies the existing winstream
backend to output logs according to alternative SRAM layout
used by IPC4 SOF drivers.

This patch is a partial implementation. Notably missing:
  - a IPC4 notification needs to be sent when new log data
    is added to the buffer (with some configurable thresholds
    that can be set by host)
  - the mtrace layout divides the SRAM window into multiple slots,
    this PR only uses a single slot
  - the slot configuration is hardcoded to expose a single
    debug slot, with no support for multicore debug

Current consensus how to proceed with this documented in https://github.com/thesofproject/sof/pull/6065#issuecomment-1209212002

Replaced with https://github.com/zephyrproject-rtos/zephyr/pull/49147


Move dataflow to root package

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

src/gravitorch/dataflow/base.py
ø

src/gravitorch/dataflow/__init__.py
100.00%

src/gravitorch/dataflow/dataloader.py
100.00%

src/gravitorch/dataflow/iterable.py
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


[Question] Multipath DirectLUNs and StorageLiveMigration

Dear Harvester developers,
is it planned to implement the mentioned features? We tried harvester in a small setup and are pleased from the simpicity. It seems perfect for transitioning from RHV/ovirt to it. However we are heavily using DirectLUNs and StorageLiveMigration features from RHV/ovirt and would love to see this in Harvester too.
I saw that it should be possible to attach multipath disks into Longhorn but that isn't what we want. We would like prevent migrating lots and lots of TiB data just to migrate to a new virtualization plattform.
Thanks and have a nice day
Timo

Harvester v1.2.0 introduced third-party storage support for non-root disks of VMs, via CSI. For background, see https://github.com/harvester/harvester/issues/2405 and https://github.com/harvester/harvester/issues/1199.
For documentation of this, see https://docs.harvesterhci.io/v1.2/advanced/csidriver. TL;DR: if your external storage system has a CSI driver, you should be able to install that driver in your Harvester cluster, and use it to access your external storage.

Dear tserong,
thank you for your reply. I already saw that third party storage support was introduced in 1.2.0. Unfortunately our storage systems don't support csi, only iscsi with multipath for redundancy. We have quite a lot of data on them. So a migration to harvester would force use to 1) buy new hardware which supports csi and 2) copy huge amounts of data. So it would cost a lot time and money.

I wonder if it's possible to do something with this: https://github.com/kubernetes-csi/csi-driver-iscsi, which appears to be a generic CSI driver for iSCSI targets. Note that I haven't tried it yet myself, the project status is listed as "alpha", and documentation is sparse, so we'd need to do a bit of experimentation.


Adding Schemes to the PostProcess removes baseUrl from the generated constructor

Hi,
I've added Schemes to the AddSwaggerDocuments PostProcess due to a projects requirement to use http instead of https, but when generating a new API Client using MsBuild's on build event the constructor no longer has the baseUrl parameter. I've tried looking around for documentation to explain this behavior but have yet to find anything. Any input on this will be welcome.
Thanks
Startup.cs
services.AddSwaggerDocument(settings =>
{
	settings.PostProcess = document =>
	{
		document.Info.Title = "Test Service";
		document.Info.Version = "v1";
		document.Info.Description = "REST API for Test Service.";
		document.Schemes = new[] { NSwag.OpenApiSchema.Http };
	};
});

TestService.generated.cs
public partial class TestControllerApiClient : ITestControllerApiClient
{
	private string _baseUrl = "http://";
	private Wrappers.IHttpWrapper _httpClient;
	private System.Lazy<Newtonsoft.Json.JsonSerializerSettings> _settings;

	public TestControllerApiClient(Wrappers.IHttpWrapper httpClient)
	{
		_httpClient = httpClient;
		_settings = new System.Lazy<Newtonsoft.Json.JsonSerializerSettings>(CreateSerializerSettings);
	}

What is the expected output?

That is the solution I found too. I just wanted to check if adding Schemes triggers removing the baseURL as a constructor parameter was an intended outcome.
Thanks for the swift response


Crash storybook when uses Sass files

Sorry, is first time that use dts-cli, but when I create a component with its associated scss file it show me this:

DeprecationWarning: Default PostCSS plugins are deprecated. When switching to '@storybook/addon-postcss',
you will need to add your own plugins, such as 'postcss-flexbugs-fixes' and 'autoprefixer'.

See https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#deprecated-default-postcss-plugins for details.

You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
> .xu-input {
|     display: flex;
| }
    at handleParseError (C:\Users\Javi\Documents\react-projects\reactlibrary\node_modules\webpack\lib\NormalModule.js:469:19)
....

you need to check out storybook docs to see how to setup sass config.


Release 1.24.0 - Missing Windows artifacts

Hello,
There are no windows artifacts in new release 1.24.0.
Could you check your build pipeline, please?
You can see the artifact in Release 1.23.2.
Same Issue:
Improve Windows CI Build #3570
Thank you.

Noted. Currently we are relying on this tool which hasn't received an update in a while, so I believe this is where the issue comes from. In the mean time we are going to manually build and upload the assets for 1.24.0.
In the future, we will need to switch to a self hosted build agent to build the required images


Using tmux and terminal-notifier causes failure

 https://github.com/julienXX/terminal-notifier does not like it when message start with open brackets.
terminal-notifier -message '[]'
...
Note that in some circumstances the first character of a message has to be escaped in order to be recognized.
An example of this is when using an open bracket, which has to be escaped like so: ‘\[’.

Combine that with being in tmux
if test -n "$TMUX_PANE"
    set message (tmux lsw  -F"$__done_tmux_pane_format" -f '#{==:#{pane_id},'$TMUX_PANE'}')" $message"
end

with
    set -q __done_tmux_pane_format; or set -g __done_tmux_pane_format '[#{window_index}]'

is kicking in,
so that every message is prepended by the current tmux pane in brackets.
Resulting in every done execution failing when using default config:

@oschrenk thank you for reporting this. It seems terminal-notifier is a bit unpredictable when it comes to this issue and to which characters need to be escaped. I found that piping the message instead of using the option fixes this issue. Fixed in https://github.com/franciscolourenco/done/releases/tag/1.17.2


PROPOSAL: Partial Parameters Definition / Override parameters values

The PARAMETERS DEFINITIONS OBJECT should accept partial definition for parameters or properties overriding:
Currently supported:
...
  /recipe:
    post:
      operationId: recipe_new

      parameters:
        - name: id
          in: query
          description: Recipe ID
          required: true
          type: string
        - name: ingredients
          in: query
          description: list of ingredients
          type: array
          items:
            type: string
          collectionFormat: csv
        - $ref: "#/parameters/FoodType"

      responses:
        default:
          description: Error
          schema:
            $ref: "#/definitions/ErrorResponse"
        200:
          description: Success
          schema:
            $ref: "#/definitions/SuccessResponse"

# ---------------------------------------------------
definitions:

  SuccessResponse:
    properties:
      message:
        type: string

  ErrorResponse:
    required:
      - message
    properties:
      message:
        type: string
# ---------------------------------------------------
parameters:

  FoodType:
    name: foodType
    description: Type of food on the recipe
    required: true
    in: formData
    type: string
    default: cake
    enum:
      - cake
      - cookies
      - bread

As you can see on the endpoint definition we add FoodType as reference, but we are forced tu use all the defined properties.
It would be good if we could define a partial definition or override something like this:
...
# ---------------------------------------------------
parameters:

  FoodType:
    name: foodType
    description: Type of food on the recipe
    required: true
    type: string
    default: cake
    enum:
      - cake
      - cookies
      - bread

And use like this:
  /recipe:
    post:
      operationId: recipe_new

      parameters:
        - name: id
          in: query
          description: Recipe ID
          required: true
          type: string
        - name: ingredients
          in: query
          description: list of ingredients
          type: array
          items:
            type: string
          collectionFormat: csv
        - $ref: "#/parameters/FoodType"
          in: query
          required: false
...

I defined required as true but later I override this value to false, and in is defined on the endpoint only.

Generally $ref doesn't work together with other properties in a JSON object. If this were to be changed, this should be done for all the places where $ref is used.
Some related issues: #926, #648, #674, #556.

yes, supporting that would break all support for JSON pointers, which we really don't want to do.


Publish UX improvements

[ ] Retain state in publish area, so that navigating away does not clear the entered values
[ ] Upload all file types
[ ] For edits, better way to indicate url is readonly (and then eliminate text saying this, or show it only when the user tries to click it)

Another idea: Allow manual thumbnail upload (setting to override in future?)


send simple image

#198

Coverage increased (+0.02%) to 93.972% when pulling 3921827d0d34688e1cf24ecfde4ae2b8a4f345da on feature/attach-image into 1367e8907d6bc56015ab3f0f33fedf2014388852 on develop.


Update aws/aws-sdk-php to >=2.7.17

Needed for https://github.com/cargomedia/s3export_backup/issues/11 (method implemented there)

@tomaszdurka please review

Maybe rather use "~2.7" semantically correct versioning.

Otherwise looks good.

Looks good


Reports tree - two line record

In Cloud Intelligence - Reports tree, two line record change into one line record when is selected.
Not selected:

Selected:

@miq-bot assign @epwinchell


[Angel Gem] - Flight Speed Modification

Requesting to add a configuration option to nerf the speed cap that is granted from using the gem.
That or just disable the speed boost completely. Creative flight is extremely overpowered...
Let alone flying faster than chunks can load.
Only other option I have is to Disabled the item completely via SF's own configuration.

There is already a max speed configuration found in Slimefun's Items.yml

There is already a max speed configuration found in Slimefun's Items.yml

Thank you. Will change it there.


After update, a second Settings window is opened after click on button

Microsoft PowerToys version
0.56.1
Area(s) with issue?
Settings (OOBE -> runner)
Steps to reproduce
Just updated to .56.1 and it opened the SCOOBE page with updates (as expected). PowerToys Settings is automatically opened too. Then navigate to a page and click on the Open settings button.
✔ Expected Behavior
Currently open window accepts the "go to" command and shows page.
❌ Actual Behavior
Second window is opened with the page visible.

Tried replicating, without success.

PowerToys Settings is (automatically?) opened too.

This is the weird part here. Not sure if this means two instances running. What seems to have happened is that the OOBE page and the settings that (automatically?) opened too were two different settings processes.
Could you please send a bug report, perhaps something there might have a clue. /bugreport

I might have manually opened the settings immediately after installation. Don't know if that is the main thing. I'll upload a report when I turn that pc back on.

Just updated on my Surface and can't repro there.

Confirming this for my destop after 0.57 is installed.

@Jay-o-Way , How would I be able to repro this? Can you please try to walk me through some steps that might be followed to try and repro this?

@Jay-o-Way , How would I be able to repro this? Can you please try to walk me through some steps that might be followed to try and repro this?

AFAIK it's basic:

install an update
PT Settings and What's New windows are both opened automatically
click on a button Open Settings on an arbitrary page
new Settings window opens

Related to #8277 I guess

"PT Settings" and "What's New" windows are both opened automatically."
Wonder why PT Settings screen is opening in your case 🤔
I'd expect only "What's New" to open. I think this must be related to that. I'm guessing Settings always opened automatically for you?

Tried reproducing it many times, but without success. Still need someone from the developer team to be able to repro this in order to fix it.

Just tried some repro'ing:

uninstalled PowerToys
installed v0.56.2
download & install update
v0.57 did not open Settings, only What's New.

Must be something old that lingured?

Just tried some repro'ing:

uninstalled PowerToys
installed v0.56.2
download & install update
v0.57 did not open Settings, only What's New.

Must be something old that lingured?

But you had it happen on two different PCs, Anything in your config that might indicate what might be happening? Something like two different users or a different admin account than the user?

Nope, they're just personal computers. Did a total clean re-install of Windows (and PowerToys) on them in December.

I haven't seen this for a while, myself.


cups-2.3.3: auth.c:458:71: error: no member named 'cr_version' in 'struct xucred'

auth.c:458:71: error: no member named 'cr_version' in 'struct xucred'
      cupsdLogClient(con, CUPSD_LOG_DEBUG2, "cr_version=%d", peercred.cr_version);
                                                             ~~~~~~~~ ^
2 warnings and 1 error generated.
gmake[1]: *** [../Makedefs:270: auth.o] Error 1

what os/version?

The error refers to a missing declaration in cups's own code.

https://github.com/openbsd/src/blob/b66614995ab119f75167daaa7755b34001836821/sys/sys/ucred.h

Please report this to the OpenPrinting CUPS project.


[Typo]: Separating Events from Effects

Summary
Event handlers only re-run when you perform the same interaction again. Unlike event handlers,
Page
https://react.dev/learn/separating-events-from-effects
Details
In the first paragraph The comparison was between event handlers and event handlers instead of effects

Unlike event handlers, Effects re-synchronize if some value they read, like a prop or a state variable, is different from what it was during the last render.

I don't see how the comparison was not made between Effects and event handlers

Event handlers only re-run when you perform the same interaction again. Unlike event handlers,


Struggling with setup, devices being skipped

So I think I'm close to getting this working, but I seem to be having the same problem of several users of the Bridge skipping over devices because they're not properly recognized in the config. The Bridge can find my account, identify my online BH1-0001 hub and online HT25-0000 valve, however the HT25-0000 valve that is identified as an Irrigation System gets skipped "based on config". In the Home app, I see what looks like the BH1-0001 hub showing up as a device, but understandably no method to manipulate it, since it's just the hub.
I have all booleans set to true because I wanted to see what shows up in Homekit, are some of these conflicting or is there a reason the valve is being identified as online but then skipped?

Most of these cases are white space issues from an extra space at the end, typically on the orbit account side. You can double check this in the log as it should state what it is comparing from your config to what is found on your account, if they look the same then there is most likely a space in the mix. You could try to reentering the street address on both to be sure. maybe a copy paste situation. All that said if you only have the one location you can also leave that field blank in which case it will use everything it finds on the account.

I am not sure if you solved this but someone else reported the same issue and attached their config and it appears to be missing the following line after the initial install.
"showIrrigation": true,


[BUG][connector-file-base][HIVE]If hive serves as sink, an error message is reported when not all hive fields are mapped to the source #5995

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Please add some comment


implement updateTree & getTreesByVisibilityStatus

updateTree:

looks for tree in database and updates it with new information given

getTreesByVisibilityStatus:

returns trees from database based on whether or not they are published or unpublished

Looks good! Thanks for adding the 'published' attribute to the type and schema. The only thing I am concerned about here is if a tree is not able to be found in the DB, it might be a good idea to check the return value of updateTree() and throw an error, in case someone deletes just before someone updates it. @trevormangrum tell me what you think, but searching by ID might also be a good idea, just a tiny bit faster, since each will have a unique id regardless, and mongo
will only have to match that one field

I think you're right, we may want to consider implementing some sort of error handling for the case you mentioned. I didn't consider that initially, but I think it's best to have just in case. We modeled updateTree after Kemal's updateChapter from the MindVersity project. In updateChapter, Kemal uses a queryChapter object, which only contains the _id for the chapter to look for. We use a queryTree with the same premise here. I'm not exactly sure why Kemal implemented it this way, but it works well. We can change if it you like, but I think it should be fine.

Looks good! Thanks for adding the 'published' attribute to the type and schema. The only thing I am concerned about here is if a tree is not able to be found in the DB, it might be a good idea to check the return value of updateTree() and throw an error, in case someone deletes just before someone updates it. @trevormangrum tell me what you think, but searching by ID might also be a good idea, just a tiny bit faster, since each will have a unique id regardless, and mongo
will only have to match that one field

I think you're right, we may want to consider implementing some sort of error handling for the case you mentioned. I didn't consider that initially, but I think it's best to have just in case. We modeled updateTree after Kemal's updateChapter from the MindVersity project. In updateChapter, Kemal uses a queryChapter object, which only contains the _id for the chapter to look for. We use a queryTree with the same premise here. I'm not exactly sure why Kemal implemented it this way, but it works well. We can change if it you like, but I think it should be fine.

Ah, that makes sense. So whenever we call it, we will have to make a new tree?

Looks good! Thanks for adding the 'published' attribute to the type and schema. The only thing I am concerned about here is if a tree is not able to be found in the DB, it might be a good idea to check the return value of updateTree() and throw an error, in case someone deletes just before someone updates it. @trevormangrum tell me what you think, but searching by ID might also be a good idea, just a tiny bit faster, since each will have a unique id regardless, and mongo
will only have to match that one field

I think you're right, we may want to consider implementing some sort of error handling for the case you mentioned. I didn't consider that initially, but I think it's best to have just in case. We modeled updateTree after Kemal's updateChapter from the MindVersity project. In updateChapter, Kemal uses a queryChapter object, which only contains the _id for the chapter to look for. We use a queryTree with the same premise here. I'm not exactly sure why Kemal implemented it this way, but it works well. We can change if it you like, but I think it should be fine.

Ah, that makes sense. So whenever we call it, we will have to make a new tree?

I think we would just create a new object with the Tree type, then set that tree's _id to the id of the tree we're looking to edit


Move stress tester and swiftevolve to new swift_build_support build infrastructure

I’m planning to move SwiftSyntax to the new build infrastructure in swift_build_support and started off by migrating the SourceKit stress tester over.

@swift-ci Please test
@swift-ci Please Build Toolchain macOS Platform
@swift-ci Please Build Toolchain Linux Platform

Apparently, I’m not able to start multiple swift-ci commands in one comment. So here we go again.
@swift-ci Please test

@swift-ci Please Build Toolchain macOS Platform

Linux Toolchain (Ubuntu 16.04)
Download Toolchain
Git Sha - 25c842ba4b19d18294122e7cc47a891bf31e1833
Install command
tar zxf  swift-PR-27580-294-ubuntu16.04.tar.gz
More info

Build failed
Swift Test Linux Platform
Git Sha - 25c842ba4b19d18294122e7cc47a891bf31e1833

Build failed
Swift Test OS X Platform
Git Sha - 25c842ba4b19d18294122e7cc47a891bf31e1833

@swift-ci Please tests

Build failed
Swift Test Linux Platform
Git Sha - 25c842ba4b19d18294122e7cc47a891bf31e1833

Build failed
Swift Test OS X Platform
Git Sha - 25c842ba4b19d18294122e7cc47a891bf31e1833

@swift-ci Please Build Toolchain macOS Platform

@swift-ci Please test

preset=buildbot_skstresstester_macos
@swift-ci Please test with preset macOS Platform

Build failed
Swift Test Linux Platform
Git Sha - 0c8e1c7a738309815494ba6b856b4ac99e3027a5

Build failed
Swift Test OS X Platform
Git Sha - 0c8e1c7a738309815494ba6b856b4ac99e3027a5

https://github.com/apple/swift-stress-tester/pull/88
preset=buildbot_skstresstester_macos
@swift-ci Please test with preset macOS Platform

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 64c3435cb1ac0ec4f3cada698f08bb88f2a12ae1

Build failed
Swift Test OS X Platform
Git Sha - 64c3435cb1ac0ec4f3cada698f08bb88f2a12ae1

https://github.com/apple/swift-stress-tester/pull/88
preset=buildbot_skstresstester_macos
@swift-ci Please test with preset macOS Platform

https://github.com/apple/swift-stress-tester/pull/88
preset=buildbot_swiftevolve_macos
@swift-ci Please test with preset macOS Platform

@swift-ci Please build toolchain macOS

https://github.com/apple/swift-stress-tester/pull/88
@swift-ci Please build toolchain macOS

macOS Toolchain
Download Toolchain
Git Sha - 92a0e20fd2149083c8ea0042a84a3869c09daaae
Install command
tar -zxf  swift-PR-27580-402-osx.tar.gz --directory ~/

Manually verified that the files in the toolchain match the ones in the open source toolchains.

Yes, the PR only tackles macOS support. I’ll look into Linux support later.

It looks like this broke the macOS bot.  Could you run python-lint and commit the result?

Fixing python-lint issue https://github.com/apple/swift/pull/27807


[security] Change security level classification for local connections

Set security level to NONE for LOCAL_TCP type, Privacy&Integrity for UDS.

@erm-g Does this count as a breaking change? if so, it will need to be added to the release notes.

@sreenithi It's protected by an experiment but I'd say yes. I'm not sure which release we're targeting for it, so I think it's to early to put it in the notes.


Code crashes three times at startup

Type: Bug
Start Code, with or without extensions. Wait approximately 20 seconds; screen will flicker, each pane separately. Wait approximately 20 seconds more; repeat of last crash. Wait approximately 20 seconds more; repeat of last crash. At this point, I can continue using Code for hours with no issues. Curiously, Microsoft Edge browser for Linux exhibits the exact same behavior - three crashes approximately 20 seconds apart, then everything is fine for hours.
Here are pastebins of the crash reports.
https://termbin.com/774w
https://termbin.com/ryg7d
https://termbin.com/3qfc
VS Code version: Code 1.79.1 (4cb974a7aed77a74c7813bdccd99ee0d04901215, 2023-06-12T16:17:13.990Z)
OS version: Linux x64 5.15.0-73-generic
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 1700X Eight-Core Processor (16 x 2200)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
0, 0, 0

Memory (System)
31.26GB (27.99GB free)

Process Argv
--disable-extensions --crash-reporter-id 47cd3433-4bce-411d-b540-9b27ade1c499

Screen Reader
no

VM
0%

DESKTOP_SESSION
mate

XDG_CURRENT_DESKTOP
MATE

XDG_SESSION_DESKTOP
mate

XDG_SESSION_TYPE
x11

Extensions disabled
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30610678
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
2i9eh265:30646982
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixhidewlkth:30730051
pythongtdpath:30739704
i26e3531:30763805
dh2dc718:30763024
pythonnosmt12:30765603

Hi, thanks for the crash reports!
Unfortunately, we aren't able to symbolicate them. Could you send us a zip of some dmp files following the creating a crash report instructions?
Also, could you run the "Help: About" command and paste the output into this issue? That way, we can quickly identify which Electron symbols to use to symbolicate the files.

@rzhao271 Attaching the dumps here. And this is the output from Help | About.
Version: 1.79.2 Commit: 695af097c7bd098fbf017ce3ac85e09bbc5dda06 Date: 2023-06-14T08:59:55.818Z Electron: 22.5.7 Chromium: 108.0.5359.215 Node.js: 16.17.1 V8: <IP_ADDRESS>-electron.0 OS: Linux x64 5.15.0-73-generic
code_dumps.zip
@deepak1556 Yes, the GPU and/or drivers was my thought also. This entire system is a new install of Linux, so yes, the drivers are also new. I noticed in the Crash report this command line:
Command Line: $'/usr/share/code/code --type=gpu-process --crashpad-handler-pid=28951 --enable-crash-reporter=92be6071-f5a7-4852-87a0-b70ac18ac357,no_channel --user-data-dir=/home/ron/.config/Code --gpu-preferences=WAAAAAAAAAAgAAAIAAAAAAAAAAAAAAAAAABgAAAAAAA4AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAACAAAAAAAAAAIAAAAAAAAAABAAAAAAAAAAgAAAAAAAAACAAAAAAAAAAIAAAAAAAAAA== --shared-files --field-trial-handle=0,i,8301173418809640930,2175803506988919068,131072 --disable-features=CalculateNativeWinOcclusion,SpareRendererForSitePerProcess'
The --gpu-preferences option, what is that Base64 encoded value? Is it for some generic GPU? I tried to find help on that option, but had no luck.

Thanks for the info! I've symbolicated the files. The call stack is different in each case.
symbolicated-47.log
symbolicated-a3.log
symbolicated-f3.log

@fatman45 do you ESET anti-virus installed ? Based on the stack the gpu sandbox is killing the process due to seccomp violations and seem like it is a known issue in upstream https://bugs.chromium.org/p/chromium/issues/detail?id=1040117#c8. I suggest to either ignore VSCode executable from the AV or run with sandbox disabled using --no-sandbox --disable-gpu-sandbox

@deepak1556 yes, I do have ESET antivirus installed. I will try disabling it and see if the issue persists. BTW, if I use --disable-gpu then I do not experience the issue.

Thank you all, uninstalling ESET cured the issue, with Edge, also. As ESET is no longer maintaining their Linux code base, I will find a different antivirus. I'm going to close this issue now.


Move to xenial (Ubuntu 16.04) and test against Python 3.7.

In other projects, I've started testing against Python 3.7 using this technique.

Annoyingly, the tests are failing. I see this in the logs:
Build language: python
Build group: stable
Build dist: xenial
Build id: 429263907
Job id: 429263911
Runtime kernel version: 4.14.12-041412-generic
travis-build version: dc1da7641
Build image provisioning date and time
Tue Dec  5 20:11:19 UTC 2017
Operating System Details
Distributor ID:	Ubuntu
Description:	Ubuntu 14.04.5 LTS
Release:	14.04
Codename:	trusty

Wait, wtf?
On other projects, the same travis config results in Ubuntu 16.04.

Perhaps implicated: travis-ci/travis-ci#10110

Was that with sudo: false?

https://github.com/travis-ci/travis-ci/issues/7260#issuecomment-354350596

Was that with sudo: false?

Yes

So it looks like it's not officially supported, and should not be relied upon.

FWIW, here's how we do it in dateutil, with xenial and sudo only used for the Python 3.7 test.
Hopefully Travis will improve support for xenial so we don't need these workarounds, though.

Most of my projects are running fine with sudo: false. Here's a recent example.

So it looks like it's not officially supported, and should not be relied upon.

Yes, perhaps. And if Travis can't get support for Xenial over two years after its release, there's something broken with their model. It's probably time to migrate to something more nimble, maybe Circle-CI or Microsoft Team Services or GitLab.


WIP: slot provided by a hosted particle can be provided as a set slot by the multiplexer

Thanks for explaining. Having multiple flavors of list seems like a bad idea.
A possible solution would be to have 'wrapper' particles, something like:
<foo-item-wrapper>
  <list-annotation>
  <foo-item>
    <item-annotation>


Argument checks in internal functions (CFE_SB_TransmitMsgValidate, etc)

Is your feature request related to a problem? Please describe.
As discussed in #1197 we need to have some consensus on the proper level of argument checking for internal helper functions.
Sometimes internal functions have tests to validate their inputs (range check etc) on behalf of the caller, in the case where several public APIs need to repeat the same tests -- putting these in a helper can reduce repeated code and make all APIs consistent in their validation (a good thing).
But in other cases the helper is invoked from contexts where the inputs are never out of range or pointers can never be NULL.  Testing for such inputs can be redundant.
Describe the solution you'd like
Need to confirm/reach consensus on whether functions like CFE_SB_TransmitMsgValidate() in CFE SB need to validate all their arguments.  Probably should better document which args are tested and why - and if there are limitations on other args (e.g. certain args are assumed to be non-null).
Additional context
This just causes some confusion during review and probably some additional comments/documentation could help.
See thread here: https://github.com/nasa/cFE/pull/1197#discussion_r585070230
Requester Info
Joseph Hickey, Vantage Systems, Inc.

To summarize - the original subject that brought up this issue was about CFE_SB_TransmitMsgValidate():

This is an internal SB function that consolidates some common logic between CFE_SB_TransmitMsg and CFE_SB_TransmitBuffer
It accepts four pointer arguments.  First (MsgPtr) is an input, the other 3 are output buffers.
It validates the input MsgPtr on behalf of the caller.  This is because the pointer may have originated outside SB (i.e. it is an unknown/uncontrolled value) and as such coding standards require that pointer inputs from outside sources are checked for NULL before dereferencing.
It does NOT validate the other three pointers - the outputs.  These all point to local stack variables in the caller, and so can never be NULL in practice.

The questions are:

Given this functions role does it need to accept NULL on any input pointer, like a public API would?
Does it need to ensure all outputs are written/set even if it returns non-success?

There is also a similar issue described in #1194 where a common helper routine is invoked from a context with known-good inputs, and so the error checking on the output is skipped.  There are other places where the same function is invoked where the input is not known to be good - so the range check inside the function is intended for those cases.  But it is called from multiple contexts - and the question there is whether the caller needs to check for error returns even though it is not possible for it to fail in the current context where the input is known to be good.

To be honest I don't think we had different opinions on this (after discussion).  As an internal function it can do validation as a "helper" to the APIs, but doesn't require validation on the parameters that are guaranteed valid by the calling function.  I thought the open issue was more for setting outputs on errors - bf8f481e2d87c13e59e2a32827f24dbd13c27394.


release: highlights of v0.5.2

/changelog

/release

/shipit

🎇 🎉 🎆 New version has been shipped! 🚀🚀🚀


Fix routes on sidebar navigation

Fixes #4. Now you can refresh the url and keep the app in the same route/state.

LGTM!


Discord Server

I think a discord server for this project would be pretty cool, people can ask for help, report bugs, or just chat

Sure, but I don't know how to use discord. If you want, I'll give you a perm and you can organize the server, is that okay ?

I'll manage your server give me permission

Sure, but I don't know how to use discord. If you want, I'll give you a perm and you can organize the server, is that okay ?

Sure! my username is mallusrgreat, send the invite here and I'll join

https://discord.gg/yd2BQ6Pz


Helm Install Wizard: Create namespace notice's namespace is not aligned

Setup

Rancher version: 2.7-head

Describe the bug

The name of the namespace is not aligned with the rest of the test

To Reproduce

Start the wizard to install a chart
Choose to create a ns and enter some text
the notice at the bottom contains a missaligned code block

Expected Result

the ns name should be roughly in-line with the rest of the text

Screenshots

Note that this is a problem with how badges are shown inline with text and probably effects all badges

Pass verified in rancher v2.7.2-rc6

installed alerting-drivers 102.0.0
created new namespace to install alerting-drivers
new namespace was in line with verbiage at bottom of screen


Implicit Pipelining with Dynamic Events

In Filament, components are implicitly pipelined. For example, in the following component can execute in a pipelined fashion:
component MultAdd<G>(
    @interface[G, G+1] go_G: 1
    @[G, G+1] left: 32,
    @[G, G+1] right: 32,
    @[G+1, G+2] acc: 32
) -> (
    @[G+1, G+2] out: 32
) {
    M := new Mult;
    A := new Add;
    m0 := M<G>(left, right);
    a0 := A<G+1>(m0.out, acc);
    out = a0.out;
}

Since the interface for G specifies that it may trigger every cycle, the Filament type checker must check that repeated re-execution is safe.
Safety is checked is assuming that G will trigger any time after its delay, in this case 1, and showing that no re-execution of the pipeline after G's delay would conflict with the current iteration.
Pipelining with Dynamic Events
Consider the following component signature:
component Dyn<G, L>(
    @interface[G, G+3] go_G: 1,
    @interface[L, L+1] go_L: 1, ...
) -> ( ... ) where L >= G+3 { ... }

The interface specifies two events G and L and provides an ordering constraint between them, L >= G+3. The ordering constraint requires that any called of the component ensures that L arrives at least 3 cycles after G does.
In order to ensure safe pipelining, we have to reason about the retriggering behavior of both G and L and ensure that their delay do not violate the constraint on the interface. As provided, the interface is incorrect; here is an trace of events that is accepted by the interface but violate the constraint:

Cycle 0: $G_0$ triggers
Cycle 3: $G_1$ triggers, $L_0$ triggers
Cycle 4: $L_1$ triggers

Note that this trace is accepted by the delay specifications for $G$ and $L$ since $G_1 - G_0 = 3$ and $L_1 - L_0 = 1$. However, it violates the constraint on the interface since $L_1 \ngeq G_1 + 3$.
Restricting Interfaces
The key problem is that the delay specification for G and L do not constraint the event in such a way that subsequent retriggers obey the constraints of the interface. Ideally, we could change the delay specification such that the constraint holds. We would like trace history to look something like:
G0 --> G0+3 --> L0 --> G1

This implies that the delay for G must be dependent on the event L occurring; G may not retrigger unless L has triggered:
@interface[G, L] go_G: 1

Next, we need to ensure that L1 obeys the interface constraint:
... --> L0 --> G1 --> G1+3 --> L1

which is the same as (where dG is the delay for G):
... --> L0 --> G0+dG --> G0+dG+3 --> L1

This means that the constraint on the delay for L looks something like:
L + dL >= G + dG + 3

This shows that the delay for L depends on the delay of G; however, the delay of G is itself dependent on L.
TLDR: The history of L and G are interdependent; in order to figure out when the next L should occur, we need to know when the last G occurred. In other words, the @interface for both events need to be able to talk about the trace of the other event directly so that the constraint can be obeyed.
The Problem
The problem here is the constraint between G and L. By specifying it, we relate the traces of events in some manner, and we need the delay specifications to encode this information. Since interfaces only talk about delays, there is no way to relate histories of multiple events.
At this point we have two solutions:

Extend the delay specifications to handle more general histories
Disallow constraints over events in the language

I do not think (1) is worth doing. This is because even if we can make it work, this way of relating dynamic events together does not correspond well with dynamic pipelines. I think the right way of doing this probably requires some mechanism of specifying type-level "back pressure"; the retrigger of G and L is not dependent on the two events but rather the availability of the resources. The resources should be able to apply back pressure to the rest of the circuit to stall events from arriving. Doing all of this is definitely out of scope for the first paper.
This means that the proposed solution is disallowing ordering constraints in the language.
The Fix
The proposed fix is to disallow constraints on user-level components but to allow them on primitives. For example, the register primitive is still allowed to specify the constraint:
component Latch<G, L>(
    @interface[G, L] go_G: 1,
    @[G, G+1] in: 32,
) -> (
    @[S, L] out: 32
) where L > G+1;

User-level components do not have ordering constraints, there is no way to construct a dynamic use of the Latch:
component Main<G, L>(@[G, G+1] in: 32) -> (...) {
    L := new Latch;
    l0 := L<G, L>(in); // <- no way to prove L > G+1
}

This program will not type-check since the component cannot specify L > G+1.
However, uses of such Latch that use the same event are still valid:
l0 := L<G, G+3>(in) // G+3 > G+1

Furthermore, it might still be possible to use max expression to express a limited form of dynamism in the language.

The diagnosis of the problem here makes sense, I'm pretty sure! That is, you need to do something "globally" to all the interfaces used together in a component in order to make implicit pipelining safe. And that certainly is weird; you could imagine making things super complicated to derive those constraints.
Not 100% sure I understand the solution yet… removing this level of expressiveness is certainly a good shortcut!! But to confirm I understand: does this mean it's impossible to express a "safe" version of the dynamic MultAdd component listed above? Or does it mean that there's some other way to do it that does not involve constraints on user-authored component definitions?
typos, in case you end up using this text elsewhere

The ordering constraint requires that any called of the component ensures that L arrives at least 3 cycles after G does.

"called" -> caller, I think?

The key problem is that the delay specification for G and L do not constraint

"constraint" -> constrain

There will no longer be any way to relate two events together which means the dynamic version of that component cannot be written down anymore.
You can still have components with multiple events and maybe we’ll keep max around if I can convince myself that we can do so safely but there will be no way to order different events with each other. The only ordering we’ll support is the natural order of G+n events

Just attempting to clarify the limitations of delay definitions a little more, here is the problem visualized:

The delay definitions can only relate G and L events to themselves (orange and green). What we would need to do, however, is to relate the traces of G and L to each other (purple).

Next, attempting to explain the fix in more details and its consequences.
Implementing the Fix
Operationally, the compiler will ensure that user-level components are not allowed to specify any ordering constraints:
component Dyn<G, L, S>(...) -> (...) where S = G+1, L > G

Here, the S=G+1 constraint is still allowed but the L > G constraint is not. The compiler will simply reject any such constraints syntactically.
The compiler will allow primitive definitions to specify ordering constraints to allow definitions of things like the Latch primitive above.
component Latch<G, L>(
    @interface[G, L] write_en: 1,
    @[G, G+1] in: 32
) -> (@[G+1, L] out: 32) where L > G+1;

However, since primitives can only be used from inside user-level components, which have no way of specifying ordering between arbitrary events, they must rely on the natural order specified by $G+n$. In other words, this means that concrete invocations for Latch must use something like Latch.G = T and Latch.L = T+n.
Rejected Programs
The following programs cannot be type checked anymore.
Dynamic Intervals
An interval definition like the following cannot be specified for a user-level component:
@[G, L] in: 32

For this interval to be valid, the user-level component would have to specify that L > G which is no longer allowed.
Dynamic Resource Reuse
The following component cannot be type checked:
component DynReuse<G, L>(...) -> (...) {
    M := new Mult;
    mg := M<G>(...);
    ml := M<L>(...)
}

There is no way to specify the delay distance between the invocations mg and ml since we are not allowed to order G and L in any way. The only way this component would work is if L = G+k.
The max Question
As we've already seen, the only way to get ordering is using the natural order over $G+n$. The $max$ operation also induces an ordering; for example we have $G+n \leq max(G, L)+n$. This is because if $max(G,L)=L$, we have $L>G$ which means $L+n>G+n$.
This means that the $max$ operation can also create a natural order over events like $G$. The open question is whether presence of $max$ expressions still lets us do the reasoning needed to do safe implicit pipelining.


keybase fails to upgrade via apt

The package remains as partially installed, and keybase services are restarted with each apt run.

dpkg: error processing keybase package (--configure):
subprocess installed keybase package post-installation script returned error exit code 1.

However, the application seems to update and launches correctly.
The /keybase mount folder becomes corrupted?
$ ls -alh / | grep keybase
ls: can't access 'keybase': The other end of the connection is not connected
d?????????   ? ?           ?              ?            ? keybase

Keybase version: 5.9.0-20220120174718+95a3939b3a
OS: Linux Mint 19.3

Had the same issue. Used fusermount -u /keybase to unmount the fuse "device". Killed keybase and then ran apt autoremove just to test it out. apt came back with the following:
[2022-01-30 11:47:01]» apt autoremove
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages will be REMOVED:
  libstd-rust-1.51
0 upgraded, 0 newly installed, 1 to remove and 7 not upgraded.
1 not fully installed or removed.
After this operation, 149 MB disk space will be freed.
Do you want to continue? [Y/n] Y
(Reading database ... 654962 files and directories currently installed.)
Removing libstd-rust-1.51:amd64 (1.51.0+dfsg1+llvm-1~exp3ubuntu1~18.04.1) ...
Setting up keybase (5.9.0-20220120174718.95a3939b3a) ...
Autorestarting Keybase via systemd for chb.
Restarted existing root redirector via systemd.
Processing triggers for libc-bin (2.27-3ubuntu1.4) ...
W: APT had planned for dpkg to do more than it reported back (3 vs 7).
   Affected packages: keybase:amd64

Why is keybase apparently hooked into apt?

Potentially related: https://github.com/keybase/client/issues/24810

I guess I'm having the same issue but with keybase 5.9.3:
marcelo@marcelo-thinkpad:~$ LANG=C sudo apt upgrade
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
1 not fully installed or removed.
After this operation, 0 B of additional disk space will be used.
Do you want to continue? [Y/n]
Setting up keybase (5.9.3-20220216215910.c82d65a685) ...
Autorestarting Keybase via systemd for marcelo.
mkdir: cannot create directory '/keybase': File exists
chown: cannot access '/keybase': Transport endpoint is not connected
chmod: cannot access '/keybase': Transport endpoint is not connected
dpkg: error processing package keybase (--configure):
 installed keybase package post-installation script subprocess returned error exit status 1
Errors were encountered while processing:
 keybase
E: Sub-process /usr/bin/dpkg returned an error code (1)

The app works fine, tough. I'm running Ubuntu 20.04.3 fully updated. What can I do to add more info?


Toolbar dropdown opens under other editors toolbar

When two editors are close, and when I open the toolbars dropdown, it opens under other editors' toolbar.

Solve it by giving each editor a style.
ex)
#suneditor_editor1 { position: relative; z-index:1 }
#suneditor_editor2 { position: relative; z-index:2 }

or
suneditor.create("id", {
  defaultStyle: "position: relative; z-index:1"
})

Hi, that only solves the issue when the dropdown goes one way. If you open the other one and it has to open up instead of down it will still go under the previous toolbar.

Try setting "z-index" to 6 or higher.
#suneditor_editor1 { position: relative; z-index:6 }
#suneditor_editor2 { position: relative; z-index:7 }


Increase Usability of Rapid Test Display

I would like to propose a small usability improvement in the display of rapid tests. If the outlined current designs have specific reasons or if there is data against my proposal I would like to hear from that ✌🏼
Current Implementation

When the results of a rapid test is shown in the CWA there are two views where it is shown. First on the Start Screen-View (Screenshot A) and on the Detail-View (Screenshot B). The Primary Action in the Interaction is always shown via the button filled with the main color blue.
Even though I have no thorough data backed by usability tests, I find it always very tempting to continue the flow of tapping on the big blue buttons and deleting my Rapid Test entirely. I have heard the same feedback from friends using the app. This should not be intended by the design since a deletion of a test result ist not mandatory if e.g. a new rapid test is registered.
Edit: Furthermore it should be noted that "Delete Test" is an obstructive action and is not visually highlighted as such.
Screenshots are taken from corona-warn-app/cwa-app-ios v2.4.0-RC9

A
B

Suggested Enhancement

I propose either to change the Primary Button ("Delete Test" on Screenshot B) to a Secondary Button or to hide it via a control button. To change the button to a secondary button would match the presentation when the test result is not yet ready.
Edit: To highlight the obstructiveness of the action the button could be filled with a red color. A combination of these suggestions could also be taken into account: Red Signal Color + Secondary button.
Example of a Primary button and secondary button (Screenshot taken from corona-warn-app/cwa-app-ios v2.4.0-RC9)

Expected Benefits

The benefit would be to stop the temptation of deleting a test result, since the interaction flow of viewing a test result is stopped by the lack of primary button.

Internal Tracking-ID: EXPOSUREAPP-7983

@finnge
Thank you for your proposal. i have informed the dev team which will now take a look at your ideas. We'll notify your here if we get updates on the issue.

Corona-Warn-App Open Source Team

@heinezen Thank you 👍🏼

I can confirm the observation of finnge. I just got my PCR test result. Of course the first thing I do is to press the button to see the test. And the only big, visible button to go back will delete the test.
I would even suggest to extend the explanatory text here, especially if the test is negative, like:
"Infos zum Ablauf

Test wurde hinzugefügt ...
Befund ...
[new] Test behalten / keep test
Zeigen Sie Ihren negativen Test beim Eintritt in Veranstaltungen oder beim Einkaufen vor
Test entfernen..."

Rationale: at the moment where I receive the (negative) test result, I am very happy and my mind is preoccupied and I don't think much further how I could use the test result.
The app should guide me here and make suggestions.
Question to the dev team: actually, my suggestion extends the original proposal. Do you prefer that I open a separate wishlist item for that?

@ndegendogo I like your extension of the idea. To add an item to the list would be a could way of supporting my intention even further 👍🏼


Consider swapping permutohedron::control::Control & friends with std::ops::ControlFlow

This (in fact the whole file) https://github.com/bluss/permutohedron/blob/afdb9433abb21ffa1dbd43f7aba58b2c62d3fee6/src/control.rs#L13-L20
could be swapped with std::ops::ControlFlow, available since Rust 1.55.0.
But signatures like https://github.com/bluss/permutohedron/blob/afdb9433abb21ffa1dbd43f7aba58b2c62d3fee6/src/lib.rs#L44-L46
would become (something like)
pub fn heap_recursive<T, F, B, C>(xs: &mut [T], mut f: F) -> std::ops::ControlFlow<B, C>
    where F: FnMut(&mut [T]) -> std::ops::ControlFlow<B, C>,

or maybe even
pub fn heap_recursive<T, F, B, C>(xs: &mut [T], mut f: F) -> Result<C, B>
    where F: FnMut(&mut [T]) -> std::ops::ControlFlow<B, C>,

since std::ops::Try is not yet stable (rust-lang/rust#84277); that would be a breaking change.
What do you think @bluss?

I think some of the finesse is removed when C=() is no longer allowed?

I think some of the finesse is removed when C=() is no longer allowed?

This is very true. In any case, we can get rid of control::Control by making use of std::ops::ControlFlow<B, ()>. I'll make a PR to see how it looks like.

I think some of the finesse is removed when C=() is no longer allowed?

On a second thought, even forcing the usage of std::ops::ControlFlow as the output of every closure would not remove any finesse (one could return std::ops::Continue(()) forever), but would severely hurt ergonomics (e.g., functions returning Result<(), E> would have to be wrapped, etc.).
Even using std::ops::Try (when it stabilizes) would disallow C = () (as () is currently not std::ops::Try). So #11 might be a nice middle ground, swapping control::Control for std::ops::ControlFlow, but still as an implementation of control::Control trait.


Internal server error on /team

Internal Server Error
undefined method `strip' for nil:NilClass

@tallica I have checked this locally and works without any problems

Works on my machine too.

@jandudulski closing. Let me know if the problem still exists.


Ability to skip Updating ... output when refreshing plugins through URL Scheme

It would be nice to be able to skip the
Updating ...
output when refreshing the plugins through the URL Scheme.
In my case it creates a lot more noise than what the output would be when the update would be finished.
https://github.com/matryer/bitbar/blob/e140612a1c93dc9cc9611875178451d7ec6e8b44/App/BitBar/ExecutablePlugin.m#L63

In my case I'd like to use BitBar to show the current mission control desktop number.
But having an "Updating..." before having "2" makes the tray area "flicker" (because of the expansion/contraction of the widget width.


Question: epub support

Hi,
This is probably not the place to ask questions but i didn't know where to ask.
do xslTNG or the XSLT 2.0 version of the stylesheets support epub ?
i see an epub folder in the XSLT 1.0 version but i don't see the equivalent in those.
Thanks

No, I haven't had time to work on that yet. It's on my list, though.

If would kindly support this request  ! XSLT1 and 2 were providing this docbook -> epub3 transform that was very usefull to build crossmedia using XML DocBook as a pivot language. epub3.1+ accessibility 1.0 format  provides documents that can be more easily accessed with reading tools used by disabled people. It should be a great progress if we can get back a straight forward transformation. I am not able to contribute to clean XSLTs but I can contribute for testing if needed

DocBook XSLT1 provides support for epub3 transform in the epub3
directory, although it is probably in need of updating to meet the
latest standards.   I have successfully produced epub3 books in the past
with it.
Bob Stayton
On 3/2/2022 8:03 AM, Gérard Vidal wrote:

If would kindly support this request ! XSLT1 and 2 were providing this
docbook -> epub3 transform that was very usefull to build crossmedia
using XML DocBook as a pivot language. /epub3.1+ accessibility 1.0
format/ provides documents that can be more easily accessed with
reading tools used by disabled people. It should be a great progress
if we can get back a straight forward transformation. I am not able to
contribute to clean XSLTs but I can contribute for testing if needed
—
Reply to this email directly, view it on GitHub
https://github.com/docbook/xslTNG/issues/131#issuecomment-1057093997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC6W3QOGFYJPPYLNF5R34PDU56GLPANCNFSM5KKZ6Z5A.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>

Thanks for the answer ! I confirm that XSLT1 worked in the past we produced many epub3 too.  I'll check and try to point out the amendments that are needed (I found some problems with ACE validator).
My question targets the integration of epub3 transform in xslTNG, is it scheduled ?

It's absolutely something I'd like to do, but there are a lot of things going on right now, so I'm not prepared to commit to a timeline. I think epub3 and some sort of solution for print output are the two big "missing pieces" in the xslTNG stylesheets and I'd like to fix both of them.

Implemented in 1.11.+


Receive no email notification from installed app or manual copy into script editor

Expected Behavior: receive email notification as form owner and recipient when form is submitted
Sample URL: https://docs.google.com/forms/d/e/1FAIpQLSdCxNeZ4vKHHugqxMatmxlSBFcqIhzux-MDueE-zGW84bp86Q/viewform?usp=sf_link
Actual Behavior: no email notification received as either, but form records answers
Steps to Reproduce the Problem

Gsuite just installed, email in general works without problems
Installed Form Notifier from App Store, set notification configuration to "1" and for both, owner and responder to receive mail.
Filling out the form as linked & hit submit, nothing
Manually installed as per https://developers.google.com/gsuite/add-ons/editors/forms/quickstart/forms-notifications, hit submit, nothing

I just received from the https://developers.google.com/gsuite/add-ons/editors/forms/quickstart/forms-notifications manual install this notification: "No HTML file named CreatorNotification was found. (line 241, file "Code")" so it looks there is a name discrepancy between the add-on explanation and the code? I changed the template name to CreatorNotification.html instead of creatorNotification.html as described for the manual but still no email when I test it.

I looked into GSuite Developer Hub under executions and it says "Feb 20, 2019, 11:05:26 AM
ERROR
The form currently has no response destination. at sendCreatorNotification(notification:259) at respondToFormSubmit(notification:196)" over and over.

I've changed line 259:
//          DriveApp.getFileById(form.getDestinationId()).getUrl();
            DriveApp.getFileById("1FAIpQLSdCxNeZ4vKHHugqxMatmxlSBFcqIhzux-MDueE-zGW84bp86Q").getUrl();

And now the error is:
Feb 20, 2019, 11:44:30 AM
ERROR
No item with the given ID could be found, or you do not have permission to access it. at sendCreatorNotification(notification:259) at respondToFormSubmit(notification:196)

It looks like the code assumes that the Form is recording responses to a Google Sheet. Please ensure you are doing that and try again.

It is entering the information in a form, I can see the form and the entries made.

I'm not able to reproduce the issue. Let me know if you can find steps to reproduce this on a new form.


dateAdapter not have effect

parsing does not work and formatting result is not displayed
Input Data: "15.11.2008"
After editing - a different format is obtained (2022-11-15)
        {
          prop: "sale_from",
          name: "Sales From",
          columnType: "date",
          dateAdapter: {
            parse(value = "", createDate) {
              const matches = value.match(/^(\d{1,2})\.(\d{1,2})\.(\d{4})$/);
              if (matches) {
                return createDate(matches[3], matches[2], matches[1]);
              }
            },
            format(date) {
              return `${date.getDate()}.${date.getMonth() + 1}.${date.getFullYear()}`;
            },
          },
        },

Try v2.0.20+ and v4 of the grid. Show work now.


Add how to load untrusted code

Hi!
For a lot of the drivers listed, most of the commands for these drivers are:
sc.exe create ATSZIO64.sys binPath=C:\windows\temp\ATSZIO64.sys type=kernel && sc.exe start ATSZIO64.sys

This doesn't really explain why the driver is vulnerable and what is the issue with it.
For instance, one thing that would be super helpful for red teamers is a link to a what the actual issue with the driver is and how to get it to run attacker/untrusted code in Kernel (if that's the actual reason it's listed as vulnerable)

Yes, we agree on this. Many require exploit code. Unfortunately, many of it is not available. We will include a CVE or reference as we come across them. Thank you for the comment!


bug(toolbar): the CSS regarding the height of the toolbar is not included anymore

I think there is an undocumented breaking change in the toolbar component.
I’ve updated an Angular Material application from v9 to v10 and saw that the height of the toolbar is broken.
I’ve gone through the CHANGELOG, but I didn’t find anything related to that. I’ve done a diff between v9 and v10 prebuilt themes CSS files and saw a chunk of CSS removed.
Reproduction
For reproduction, compare the prebuilt themes for v9 and v10:

indigo-pink v9
indigo-pink 1v0

Expected Behavior
The CSS generated for the toolbar in Angular Material v9 included the following code:
.mat-toolbar-multiple-rows {
    min-height: 64px
}

.mat-toolbar-row, .mat-toolbar-single-row {
    height: 64px
}

@media (max-width: 599px) {
    .mat-toolbar-multiple-rows {
        min-height: 56px
    }

    .mat-toolbar-row, .mat-toolbar-single-row {
        height: 56px
    }
}

Actual Behavior
In v10, this chunk of CSS disappeared.
Environment

Angular: 10
CDK/Material: 10
Browser(s): All browsers
Operating System (e.g. Windows, macOS, Ubuntu): All operating systems

It's the other way around: the v10 theme styles have the height and the v9 ones don't. I don't think there's a breaking change since the styles are still there, but they were moved from the component styles into the theme so that we can support the new density system.

Yes, I mixed the two. Mea culpa. And, I agree that this is not a breaking change. My issue was not related to this change.


Add font exceptions for Kaiti SC and Kaiti TC

Fixes #4532. Tested on macOS:
#text("测试", font: "Kaiti SC", weight: "regular")
#text("测试", font: "Kaiti SC", weight: "bold")
#text("测试", font: "Kaiti SC", weight: "black")

Output PDF

Before

After

Thanks!


gen_maven_deps.sh no longer working on OSX

I just updated this repo after a few months.  It no longer works as expected on OSX:
It seems to generate a full path from the cwd to the workspace directory now, e.g. ROOT/Users/seanmcl/foo/....  See below.

$ cd ~/tmp/foo
$ git clone https://github.com/johnynek/bazel-deps.git
Cloning into 'bazel-deps'...
...
$ ls
bazel-deps  dependencies.yaml
$ cd bazel-deps
$ git log -1
d886f0566f867329b0f5c559edf41781c67eebca
...
$ bazel build src/scala/com/github/johnynek/bazel_deps:parseproject_deploy.jar
...........
INFO: Found 1 target...
...
$ cd ..
$ ls
bazel-deps  dependencies.yaml
$ ./bazel-deps/gen_maven_deps.sh generate -r `pwd` -s 3rdparty/workspace.bzl -d dependencies.yaml
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
wrote 90 targets in 39 BUILD files
$ ls
3rdparty  Users  bazel-deps  dependencies.yaml  target
$ ls 3rdparty
jvm
$ find Users
Users
Users/seanmcl
Users/seanmcl/tmp
Users/seanmcl/tmp/foo
Users/seanmcl/tmp/foo/3rdparty
Users/seanmcl/tmp/foo/3rdparty/workspace.bzl

this is strange.  we use this tool almost daily at stripe and don't see the issue you are describing...
Maybe I somehow recently introduced the bug, let me try to repro locally.

repro'd....
ugh... Sorry. I introduced this recently and we haven't bumped to the latest. Thanks.


update(doc): operations section under guides tab

Description
As a developer I want to update the operations section with the new doc formatting
e.g. - https://hyperledger-bevel.readthedocs.io/en/latest/guides/fabric/add-cli/

[ ] Besu
[ ] Quorum
[ ] Fabric
[ ] R3 Corda
[ ] R3 Corda Ent

Acceptance Criteria

All sample configuration are presents
The operation guides uses new format of referring directly the sample file lines

I'll take fabric

I'll take Indy


False and True should print as False and True, mean for bool should be %

Mean as % might be confusing, rest is covered  by this pr.


Rename live providers to avoid inevitable 'webrtc' confusion

See https://github.com/dermotduffy/frigate-hass-card/issues/335#issuecomment-1032223985 for why this is advisable.

Proposed new names:

"ha" (formerly: "frigate"): Home Assistant streams (includes anything "native" HA, e.g. HLS, LL-HLS, WebRTC)
"frigate-jsmpeg": JSMPEG from the Frigate server.
"webrtc-card" (formerly: "webrtc"): https://github.com/AlexxIT/WebRTC

Proposed new configuration stanzas:

"ha": No configuration stanza currently supported.
"frigate-jsmpeg": "jsmpeg" (unchanged)
"webrtc-card" (formerly: "webrtc")

great! I would just like to test if it works .. Do you think it is possible to push a beta?


Built-in functions SUBSTRING and REPLACE don't throw CX_SY_RANGE_OUT_OF_BOUNDS

While working on the Minesweeper exercise  i noticed, that instead of throwing the expected exception, replace just adds to the string, when the offset is out of bounds. Substring seems to ignore it.
Below is the code i was running:

CLASS zcl_minesweeper DEFINITION PUBLIC FINAL CREATE PUBLIC.

  PUBLIC SECTION.

    METHODS annotate
        IMPORTING
          !input        TYPE string_table
        RETURNING
          VALUE(result) TYPE string_table.
   PRIVATE SECTION.

ENDCLASS.

CLASS zcl_minesweeper IMPLEMENTATION.

  METHOD annotate.
    FIELD-SYMBOLS <other_line> TYPE string.
    FIELD-SYMBOLS <current_line> TYPE string.
    result = input.
*   Working from each mine outwards - adding 1 to each adjacent field that is not a mine
    LOOP AT result ASSIGNING <current_line>.
      CHECK <current_line> CA '*'.
      DATA(current_row) = sy-tabix.
*     start one row up
      DO strlen( <current_line> ) TIMES.
        DATA(row) = current_row - 1.
        DATA(current_off) = sy-index - 1.
        CHECK substring( val = <current_line> off = current_off len = 1 ) EQ '*'.
        DO 3 TIMES.
*         for each row, start one character to the left from current
          DATA(off) = current_off - 1.
*         "one row up, same row and one row down - if row exists
          READ TABLE result INDEX row ASSIGNING <other_line>.
          IF sy-subrc EQ 0.
            DO 3 TIMES.
*             except current character
              IF row NE current_row OR off NE current_off.
*               check bounds by catching exception
                try.
*                 Expecting to throw exception, when offset out of bounds.
                    DATA(current_field) = substring( val = <other_line> off = off len = 1 ).                       "<<<<<
                    IF current_field NE '*'.
                      DATA(cn_mines) = CONV i( substring( val = <other_line> off = off len = 1 ) ).                "<<<<<
                      cn_mines = cn_mines + 1.
                      <other_line> = replace( val = <other_line> off = off len = 1 with = |{ cn_mines }| ).        "<<<<<
                    ENDIF.
                catch CX_SY_RANGE_OUT_OF_BOUNDS.
*                  out of bounds - nothing to do
                endtry.
              ENDIF.
              off = off + 1.
            ENDDO.
          ENDIF.
*         row does not exist - try the next
          row = row + 1.
        ENDDO.
      ENDDO.
    ENDLOOP.
  ENDMETHOD.

ENDCLASS.

thanks, also opened https://github.com/abaplint/transpiler/issues/689

status:


Update controllergen commands to work with 0.9.0

Which issue this PR addresses:
fixes a local build failure
What this PR does / why we need it:
Right now we rely on controller-gen 0.5.0, which is old and gives the wrong version when we run with go run tool@version. So, we should update it to 0.9.0 (which is current).
Test plan for issue:
CI/E2E
Is there any documentation that needs to be updated for this PR?
N/A
How do you know this will function as expected in production?
E2E

/azp run ci, e2e

/azp run ci, e2e

/azp run e2e

fixes a local build failure

What error did you get?

/azp run e2e

@bitoku Running it with the real v0.5.0 outside of vendoring causes it to change the version number to the correct v0.5.0, and using the v0.9.0 that it reports causes another diff of newlines. This is because the v0.5.0 we're using misreports itself as v0.9.0 as per the go.mod before overrides.


keras pipe mode - Invalid header crc error

Hi,
when I run the pipe mode part of keras_script_mode_pipe_mode_horovod example, I find this error in the Cloudwatch console:
tensorflow.python.framework.errors_impl.InternalError: Invalid header crc

It seems related with tensorflow extensions: https://github.com/aws/sagemaker-tensorflow-extensions/blob/master/src/pipemode_op/RecordReader/TFRecordReader.cpp
It does not happen in other parts of the notebook, e.g. local mode, cloud, or horovod.
I could replicate this problem with several SageMaker notebooks in multiple regions.
What will How can I work around this?

I found what caused the issue.
https://github.com/awslabs/amazon-sagemaker-examples/issues/1149
I am resolving this issue.