Kubernetes: can't create '/var/lib/clamav/bytecode.cvd': Read-only file system

Deployed to a kubernetes cluster based on https://github.com/filefrog/clamav/blob/master/deploy/k8s.yml
I'm seeing (the second time the DaemonSet starts), that:

The init container seems to work as expected.
$ kubectl logs -f clamd-krlld -c init
seeding main.cvd from image...
'/var/lib/clamav.master/main.cvd' -> '/var/lib/clamav/main.cvd'
seeding daily.cvd from image...
'/var/lib/clamav.master/daily.cvd' -> '/var/lib/clamav/daily.cvd'
seeding bytecode.cvd from image...
'/var/lib/clamav.master/bytecode.cvd' -> '/var/lib/clamav/bytecode.cvd'
done

The freshclam container seems to encounter some warnings, but remains in the Running state.

$ kubectl logs -f clamd-krlld -c freshclam

Wed Mar 24 21:02:32 2021 -> --------------------------------------
Wed Mar 24 21:02:32 2021 -> freshclam daemon 0.102.1 (OS: linux-musl, ARCH: x86_64, CPU: x86_64)
Wed Mar 24 21:02:32 2021 -> ClamAV update process started at Wed Mar 24 21:02:32 2021
Wed Mar 24 21:02:32 2021 -> ClamAV update process started at Wed Mar 24 21:02:32 2021
Pruning unwanted or deprecated database file bytecode.cvd.
Wed Mar 24 21:02:32 2021 -> daily database available for update (local version: 25749, remote version: 26119)
Wed Mar 24 21:02:32 2021 -> daily database available for update (local version: 25749, remote version: 26119)
Wed Mar 24 21:02:34 2021 -> WARNING: downloadFile: file not found: https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> ^downloadFile: file not found: https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> WARNING: getpatch: Can't download daily-25750.cdiff from https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> ^getpatch: Can't download daily-25750.cdiff from https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> WARNING: downloadFile: file not found: https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> ^downloadFile: file not found: https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> WARNING: getpatch: Can't download daily-25750.cdiff from https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> ^getpatch: Can't download daily-25750.cdiff from https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> WARNING: downloadFile: file not found: https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> ^downloadFile: file not found: https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> WARNING: getpatch: Can't download daily-25750.cdiff from https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> ^getpatch: Can't download daily-25750.cdiff from https://database.clamav.net/daily-25750.cdiff
Wed Mar 24 21:02:34 2021 -> WARNING: Incremental update failed, trying to download daily.cvd
Wed Mar 24 21:02:34 2021 -> ^Incremental update failed, trying to download daily.cvd
Wed Mar 24 21:02:37 2021 -> Testing database: '/var/lib/clamav/tmp/clamav-d9833945f9e5f017777108eb8f3654f5.tmp-daily.cvd' ...
Wed Mar 24 21:02:37 2021 -> Testing database: '/var/lib/clamav/tmp/clamav-d9833945f9e5f017777108eb8f3654f5.tmp-daily.cvd' ...
Wed Mar 24 21:02:46 2021 -> Database test passed.
Wed Mar 24 21:02:46 2021 -> Database test passed.
Wed Mar 24 21:02:46 2021 -> daily.cvd updated (version: 26119, sigs: 3965409, f-level: 63, builder: raynman)
Wed Mar 24 21:02:46 2021 -> daily.cvd updated (version: 26119, sigs: 3965409, f-level: 63, builder: raynman)
Wed Mar 24 21:02:46 2021 -> main.cvd database is up to date (version: 59, sigs: 4564902, f-level: 60, builder: sigmgr)
Wed Mar 24 21:02:46 2021 -> main.cvd database is up to date (version: 59, sigs: 4564902, f-level: 60, builder: sigmgr)
Wed Mar 24 21:02:46 2021 -> ERROR: NotifyClamd: Can't connect to clamd on <IP_ADDRESS>:3310: Connection refused
Wed Mar 24 21:02:46 2021 -> !NotifyClamd: Can't connect to clamd on <IP_ADDRESS>:3310: Connection refused
Wed Mar 24 21:02:46 2021 -> --------------------------------------

But the clamd container runs into an issue -- it appears to want to WRITE to the `` file but the file system is read-only.
$ kubectl logs -f clamd-krlld -c clamd
seeding bytecode.cvd from image...
cp: can't create '/var/lib/clamav/bytecode.cvd': Read-only file system

My thought is this line from the freshclam is concerning: Pruning unwanted or deprecated database file bytecode.cvd.
Perhaps the init container creates the file, freshclam deletes it, and clamd attempts to recreate it?
Have you encountered this or is there a clear path forward? I investigated the clamd configuration file options and don't see anything that would disable attempting to write to that location while still maintaining the Bytecode yes default which sounds like the suggested configuration.
I think I roughly understand the purpose of the 3 containers (init gets files ready, freshclam probably keeps them up to date over time, and clamd does the actual scanning) -- so the separation of readOnly file mounts makes sense. Just not sure how to work around this issue -- would it be to remove Bytecode no from the freshclam config?

Finally, is ERROR: NotifyClamd: Can't connect to clamd on <IP_ADDRESS>:3310: Connection refused anything to be concerned about?

After commenting out Bytecode no from the freshclam config, it does seem to start successfully. Any chance you can speak to why that config option was added as an explicit override on that particular config?


feat(macros): property generate declarations for nested modules

Currently we can't have lint rule declarations like
mod1::rule1
mod1::rule1

as it gets generated as
mod1 {
  pub rule1
}
mod1 {
  pub rule2
}

The fix is to properly nest module declarations:
mod1 {
  pub rule1;
  pub rule2;
}

Merging this one instead of #213 because this one has refactored things a bit, and the CI is passing.


Multiple Classes - " [CLASS NAME] does not conform to protocol 'Encodable'"

When installing the package, there are several issues with a few classes:

Type 'CreateContactOptions' does not conform to protocol 'Decodable'

Type 'CreateContactOptions' does not conform to protocol 'Encodable'

Type 'ContactDto' does not conform to protocol 'Encodable'

Type 'ContactDto' does not conform to protocol 'Decodable'

Within the 'ContactDto' you also have this issue:
2. Cannot automatically synthesize 'Decodable' because 'Any?' does not conform to 'Decodable'

Type 'CreateContactOptions' does not conform to protocol 'Decodable'

Type 'CreateContactOptions' does not conform to protocol 'Encodable'

Needed to add the master branch


Fix bug on URL in idea box - see comments

Idea Box line 75
link does not work if user copies full web site address containing https://
It only works if user inputs www.stuff.com


pico: Tweak ST7789 gamma and content-adaptive brightness

Switches to manufacturer supplied gamma settings, which are clearly better with more brightness/contrast.
Turns on content-adaptive brightness, which seems to give a significant boost in display brightness. May need testing with various games to see if there are any artifacts or drawbacks - other than potential additional power usage - of using this feature.

Hmm, it's definitely... bright. Looking at a random ST7789 datasheet, looks like that CAB (or WRCACE ) setting is for "medium colour enhancement" and adaptive brightness is off?

Oof, yeah I think I turned off content-adaptive brightness because it appeared to do nothing while "Colour Enhancement" appears to make it... B R I G H T. Possibly too bright. Pure gamma settings on their own still feel a little dim, however, despite being much nicer than the previous setup.
It doesn't help that this is a little subjective.

Hmm, hardware-test/serial-debug (HSV rainbow background) look a bit bad with "enhancement" on. (too bright at the top)


slide-box rendered event

Hi,
is there an event in 1.1.1 version which is fired when the slide-box directive is rendered?
I've seen solutions that use ng-init but this event is fired when the angular app is loaded, right?
Thanks a lot!

Greetings @satore11!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Thanks for opening an issue with us! Since this is more of a question you will be able to get a better answer over at our forum (https://forum.ionicframework.com/) or our slack channel (http://ionicworldwide.herokuapp.com/). Thanks again!


NU5129 is not so clever to detect a file renamed by the nuspec

This warning confused me because it is straight up gaslighting me.
WARNING: NU5129: - At least one .props file was found in 'build/native/include/', but 'build/native/BCGControlBarPro-Win32.props' was not.
Except, it was:

    <file src="build\BCGControlBarPro.props" target="build\BCGControlBarPro-Win32.props"/>
    <file src="build\BCGControlBarPro.targets" target="build\BCGControlBarPro-Win32.targets"/>

Could you please either get nuget.exe to acknowledge my nuspec has renamed the file appropriately, or make build\BCGControlBarPro-Win32.props a folder, not a file, so that the warning is accurate?
The docs say I'm doing the right thing
Ta.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 81f342a2-c4fa-9704-bf6d-c739d974eb6e
Version Independent ID: f5f90f1e-9d35-c109-aff3-2f657eaa0c57
Content: NuGet Warning NU5129
Content Source: docs/reference/errors-and-warnings/NU5129.md
Product: nuget
GitHub Login: @zivkan
Microsoft Alias: karann

This is a product issue, not a docs issue. Moving to NuGet/Home.


txn: pessimistic lock returns multiple values

Signed-off-by: youjiali1995<EMAIL_ADDRESS>
What have you changed?
Return values when acquiring pessimistic locks successfully. In some cases, it can reduce get RPCs and it brings about 10% improvement in TPC-C benchmark.
What is the type of the changes?

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?

Unit test
Integration test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?

No
Does this PR affect tidb-ansible?

No
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/issues/6680

/run-all-tests

@coocood

LGTM

LGTM

LGTM but a concern that using this interface we cannot distinguish a non-existing key from a pair with empty value. This does not affect the use case of TiDB but may cause some trouble if we will extend pessimistic transactions to pure kv services.
(We don't need to solve it at the moment IMO)

Yes. I can file an issue now.

/merge


[WIP] cli output formatting

Very good API design 👍
Don't you like mysql-like formatting style? It's like this:
[frsyuki@xcore embulk]$ embulk preview zz-gussed.yml
2016-06-23 16:16:32.391 -0700: Embulk v0.8.8
2016-06-23 16:16:33.823 -0700 [INFO] (0001:preview): Listing local files at directory '/Users/frsyuki/project/embulk/try1/csv' filtering filename by prefix 'sample_01.csv.gz'
2016-06-23 16:16:33.830 -0700 [INFO] (0001:preview): Loading files [/Users/frsyuki/project/embulk/try1/csv/sample_01.csv.gz]
+---------+--------------+-------------------------+-------------------------+----------------------------+
| id:long | account:long |          time:timestamp |      purchase:timestamp |             comment:string |
+---------+--------------+-------------------------+-------------------------+----------------------------+
|       1 |       32,864 | 2015-01-27 19:23:49 UTC | 2015-01-27 00:00:00 UTC |                     embulk |
|       2 |       14,824 | 2015-01-27 19:01:23 UTC | 2015-01-27 00:00:00 UTC |               embulk jruby |
|       3 |       27,559 | 2015-01-28 02:20:02 UTC | 2015-01-28 00:00:00 UTC | Embulk "csv" parser plugin |
|       4 |       11,270 | 2015-01-29 11:54:36 UTC | 2015-01-29 00:00:00 UTC |                       NULL |
+---------+--------------+-------------------------+-------------------------+----------------------------+

Vertical style:
[frsyuki@xcore embulk]$ embulk preview zz-gussed.yml -G
2016-06-23 16:18:29.138 -0700: Embulk v0.8.8
2016-06-23 16:18:30.050 -0700 [INFO] (0001:preview): Listing local files at directory '/Users/frsyuki/project/embulk/try1/csv' filtering filename by prefix 'sample_01.csv.gz'
2016-06-23 16:18:30.055 -0700 [INFO] (0001:preview): Loading files [/Users/frsyuki/project/embulk/try1/csv/sample_01.csv.gz]
*************************** 1 ***************************
      id (     long) : 1
 account (     long) : 32,864
    time (timestamp) : 2015-01-27 19:23:49 UTC
purchase (timestamp) : 2015-01-27 00:00:00 UTC
 comment (   string) : embulk
*************************** 2 ***************************
      id (     long) : 2
 account (     long) : 14,824
    time (timestamp) : 2015-01-27 19:01:23 UTC
purchase (timestamp) : 2015-01-27 00:00:00 UTC
 comment (   string) : embulk jruby
*************************** 3 ***************************
      id (     long) : 3
 account (     long) : 27,559
    time (timestamp) : 2015-01-28 02:20:02 UTC
purchase (timestamp) : 2015-01-28 00:00:00 UTC
 comment (   string) : Embulk "csv" parser plugin
*************************** 4 ***************************
      id (     long) : 4
 account (     long) : 11,270
    time (timestamp) : 2015-01-29 11:54:36 UTC
purchase (timestamp) : 2015-01-29 00:00:00 UTC
 comment (   string) : NULL

example code used in embulk: https://github.com/embulk/embulk/tree/master/embulk-core/src/main/java/org/embulk/command

I think mysql style might be preferrable for data. I think I prefer docker style for a cli like digdag displaying information about entities like projects, workflows etc.

This may be good PR, but unfortunately many files are conflicted.
If no one take over, I will close in near future.

Too old to merge. So close.


GPL-438 Record additional detail on asset_ausites

See: https://github.com/sanger/asset_audits/issues/141
User story
As Adam F, when I complete a bed verification in Asset Audits, I would like the audit record in Sequencescape to include which plates were scanned on which beds.
Who the primary contacts are for this story
Adam F
Acceptance criteria
To be considered successful the solution must allow:

'Asset Audit' record in SS includes information about which plates were scanned on which beds
For all verifications or just new Heron ones?

Covers the Sequencescape side of things for persisting data

[ ] Migrate features to rspec
[ ] Write migrations to add column
[ ] Update model to serialize metadata
[ ] Update tests for setting metadata
[ ] Allow setting of metadata
[ ] Update tests for viewing of metadata
[ ] Add viewing of metadata
[ ] Review need to add select scope which does not include TEXT column (performance)

Hmm, might actually be able to use the MySQL JSON data type

Yup. Works nice and smoothly

Before:

After tweaks


feat(collapsible): disabled

add disabled state
add new props to set a custom icon
Proposed Changes
Collapsible

add disabled state
add props to override the default collapsible button

Potential Breaking Changes
none
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Code is nice, :wave:  me once you have your build.

Same - looks good but there's conflicts preventing the demo. Can check and approve once fixed 😄


This is a complete implementation of the Scheme specification it?

No. It is heavily inspired and influenced by Scheme (MIT/GNU scheme in particular). GoLisp 1.0 is much more in line with Scheme, but it is at it's heart a language for our purposes internally. It's growth and evolution reflect that.
GoLisp evolves along 4 axis:

becoming more compatible with MIT/GNU Scheme (e.g. normal/applicative order, control structures)
supporting more capabilities of the underlying Go implementation language (e.g. recent work supporting concurrency and channels)
our particular needs (e.g. byte array support)
useful additions (e.g. frames)

If GoLisp has a form or function that is also in MIT/GNU scheme, we strive to make it work the same way.
As much as possible the internals echo those in MIT/GNU Scheme. For example, the lexical environment structure/handling.
So, while it is intentionally very much like MIT/GNU Scheme, it is not constrained to follow the specification.  That said, unless our extensions are used, code will usually be indistinguishable.
Dave


Add more details to 'resource (.rsrc) section' rule

@Ana06

Add more details to the resource (.rsrc) section rule about the content found in this section. This could be done for example by adding subrules for .exe, .dll, .png, etc.
It could also be a good idea to exclude known non dangerous file from the rule such as the following manifest found in PMA Lab21-01:
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
  <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
    <security>
      <requestedPrivileges>
        <requestedExecutionLevel level="asInvoker" uiAccess="false"></requestedExecutionLevel>
      </requestedPrivileges>
    </security>
  </trustInfo>
</assembly>

ideally, "empty" or standard resources should be ignored
that requires updates in the extractors and potential additional expressivity around resource types and IDs


Group resource doesn't handle delayed evaluators that evalutate to strings

The group resource can't handle delayed evaluators that result in a string rather than an array:
chef:recipe > r = group "foo" do
chef:recipe > members lazy { "foo" }
chef:recipe ?> end
 => <group[foo] @name: "foo" @noop: nil @before: nil @params: {} @provider: nil @allowed_actions: [:nothing, :create, :remove, :modify, :manage] @action: :create @updated: false @updated_by_last_action: false @supports: {} @ignore_failure: false @retries: 0 @retry_delay: 2 @source_line: "(irb#1):4:in `irb_binding'" @guard_interpreter: :default @elapsed_time: 0 @resource_name: :group @group_name: "foo" @gid: nil @members: #<Chef::DelayedEvaluator:0x007fe82f8dfbe8@(irb#1):5> @excluded_members: [] @append: false @non_unique: false @cookbook_name: nil @recipe_name: nil>
chef:recipe > r.members
Chef::Exceptions::ValidationFailed: Option members must be a kind of [Array]!  You passed "foo".

Most proximately, this is because the validation in set_or_return is set to only allow arrays. Strings are handled by coercing them into arrays when the attribute is set:
https://github.com/opscode/chef/blob/master/lib/chef/resource/group.rb#L58

This is similar to the root cause of https://github.com/chef/chef/issues/1640 where scalar values aren't cast into an array.

think this got fixed due to converting to properties at some point along the way.


Support creating immutable bytes from mutable one

To be able to mutate the inner array and then make it safe(r) for direct altering.

Fixed in f0e5cacf357aa854a38b7085093ecac89048f7e3


Unable to get clientWidth (always equals to 0) after upgrading to 2.3.0

Hi @cheton ,
After upgrading to 2.3.0, child components in <Modal.Body> are unable to get correct clientWidth on componentDidMount(). It works good on version 2.2.2.
componentDidMount() {
    const element = ReactDOM.findDOMNode(this.node);
    const clientWidth = element.clientWidth; // => 0
}

Thanks @cheton I'll try flexbox later.
I do know the render of parentNode comes after ExtractingPreview itself.
I'm just wondering the difference between react-modal 2.2.2 and 2.3.0, because in 2.2.2, I can still get parentNode's width & height even ExtractBgImg's componentDidMount is not fired. And I didn't find the answer in your recent commits.

You can get the size of the element with clientWidth or getBoundingClientRect().width. In some cases, I'm not quite sure why, clientWidth didn't work, but getBoundingClientRect did. So I get the width of the element in that order.
I added a setTimeout because during rendering, the element may be adjusting to the layout, so it's important to be aware that its size may change depending on its CSS (or display of the scroll bar).
React 18.2.0
import { useEffect, useRef } from "react"
import "./Component.scss"

function Component() {
    const elementRef = useRef(null)

    useEffect(() => {
        const element = elementRef.current
        let width = element.clientWidth || element.getBoundingClientRect().width
        console.log(width)

        setTimeout(() => {
            width = element.clientWidth || element.getBoundingClientRect().width
            console.log(width)
        }, 1000)
    }, [])

    return (

element

    )
}


stripping logs are respected as stripped_*_log block place

Description of issue:
When stripping logs by right clicking on it with an axe, Jobs respects it as if it has been placed which results in unwanted payments.
CONFIG SECTION:
*Extracted from jobConfig.yml*
Jobs:
  Gaertner:
    Place:
      STRIPPED_ACACIA_LOG:
        income: -2.49
        experience: -2.49

-2.49 will be charged if a player right clicks with an axe on acacia log to strip it. (should only happen when placing acacia log)

Server Type (Spigot/Paperspigot/etc):
paperspigot
Server Version (using /ver):
git-Paper-204 (MC: 1.16.3)
Relevant plugins (Delete if this isn't needed):
Jobs version 4.16.2

Somehow the Place action is being called here and somehow the PlayerInteractEvent recognizes the player hand item as placed block.
https://github.com/Zrips/Jobs/blob/master/src/main/java/com/gamingmesh/jobs/listeners/JobsPaymentListener.java#L1709-L1713


fix drag jitter

this fixes some jitter when dragging views. because the view could have moved between the time that it is painted, the computed offset can be outdated. This causes lots of jitter in the draggable example.
I feel like there should be a simpler way of doing this but I couldn't figure out another way

Do you mean the pointer could have moved between the time that it is painted? But if we don't receive the pointermove event, there's no information the pointer has been moved.
Do I understand this issue correctly?

Thanks for the clarification.
Because the dragged view should be always follows the pointer, maybe we can simply it by storing the offset of the view relative to the pointer at drag start, and then at paint time, we know where it should be according to the pointer position. E.g. we don't use the relative value with the original view which could have moved as well, but use the relative value to the pointer and it should always be right.

https://github.com/user-attachments/assets/77b7fd25-190e-4cd5-9bc3-c6360da6ed56


Conditions for finding Team Meeting recordings don't seem to work.

When testing the conditions for finding Teams Meeting recordings:
ProgID:Media AND ProgID:Meeting
I get "Unknown property name: ProgID" when building the query.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 8a4716f6-31db-306c-2804-56591e3af7e6
Version Independent ID: 8dfc13ff-3b6a-ba25-2911-db7a3dcd2e8a
Content: Automatically apply a retention label - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/apply-retention-labels-automatically.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

@DPdlund Sorry to hear this isn't working for you.  Please open a support ticket for investigation.


Another round of NYU morpheus changes

This PR delivers the following JIRAs:

https://jira.sakaiproject.org/browse/SAK-32443
https://jira.sakaiproject.org/browse/SAK-32414
https://jira.sakaiproject.org/browse/SAK-32430
https://jira.sakaiproject.org/browse/SAK-32431
https://jira.sakaiproject.org/browse/SAK-32420
https://jira.sakaiproject.org/browse/SAK-32433
https://jira.sakaiproject.org/browse/SAK-32418

@payten any chance you can resolve these conflicts in syllabus real quickly so we can merge?

Yep, sure thing @ottenhoff.  I just had a quick go but was unable to test the merge locally as viewing existing and adding new items seems to be broken in Syllabus? This problem seems to be happening on nightly too: https://trunk-mysql.nightly.sakaiproject.org/portal/site/ec659311-5d34-410a-acc0-8aedd3871dd4/tool/64e72248-847f-4e9a-a3d5-c72e93362570.  I'll be able to look into in more depth on Tuesday (long weekend here in Australia 🏖 ).

Yeah, there were 3 changes in the last few days against syllabus. I'd guess one of those is causing this problem. I can look this bug at least on Monday so you can fix this up for Tuesday. Regular weekend here.

Hi @jonespm.  Thanks to @marktriggs for doing some long weekend sleuthing, syllabus is back on track!  I've rebased the branch and bits seem to be happy.


Typing issue: 面白い姉

Category of the typing issue
3. "面白いねえ" is not converted as expected.
Issues
Write issues to the following table. (It's in the markdown format)

input [e.g.ゆうひ]
expected [e.g. 夕日]
actual [e.g. ユウヒ]

ねえ
ねえ
姉

Version or commit-id
Mozc-2.28.5065.102+24.11.oss
Additional context
かわいいねえ => かわいい姉
楽しいねえ => 楽しい姉
そうねえ => 総姉

Hi utuhiro78,
d23536b updated test cases based on this issue.
ca75f3d updated the collocation data based on this issue.
Thank you for your feedback.


Resume after stops replays

Hi there,
I could not see a way to detect whether the player was actually playing or not and need a way to pause and resume sounds. In Ocarina however if you play a sound and the sound ends if you pause and resume it the resume will replay the sound when it should not as it was not playing.
So it would be nice if either resume worked as it should and not restart the sound or a way to detect if the sound is actually still playing is made available.
Thanks
Alan

We have the isPlaying attribute now, as well as some listeners for it on the latest version 0.1.2


Dodge is explicitly reverted back to default button 'B'

Dodge is explicitly reverted back to default button B.
A snippet of code in Action.cpp shows:
…
CharacterAction::ActionType::Dodge,
R"(<mapping name="Dodge_Button" type="Button" >
		<button id="IK_Pad_B_CIRCLE" />
…

It doesn't contain a format token for fmt::format to process.

Issue is now fixed.


adding peer to write list to the :dbname data store is failing

I'm trying to append peer ids to give write access in my system but its failing to append.
curl -X POST https://localhost:3000/db/docstore/access/write -d 'id=045757bffcc7a4...
and the code returns {} empty object
{                 method: ['POST', 'PUT'],                 path: '/db/{dbname}/access/write',                 handler: dbMiddleware( async (db, request, _h) => {                     await db.access.grant('write', request.payload.id)                     return {};                 })             }

/assign @phillmac

@pontiyaraja , is this the same issue we are facing with Node 2 not able to append new entries to Node 1 ?
If yes, I think this is a bug that must be fixed ASAP.

Some of the endpoints will return {} on success. The simplest check is to verify if the response is HTTP code 200.
I think this is  failing on my part in not documenting success / failure responses very well.
Most failures will result in a HTTP code 500 response with a generic error.
If {} isn't as sufficient response on success, what would better suit your needs? e.g. {"statusCode":200} ?

@phillmac it's not a question it's a bug.
I executed command
curl -X POST https://localhost:3000/db/docstore/access/write -d 'id=045757bffcc7a4...
I got success response with empty object  {}
When I get information about my DB, I'm not getting updated write data
{     "address": {         "root": "zddfgfhgjkjkjftettsghfhfgh",         "path": "Peer"     },     "dbname": "Peer",     "id": "/orbitdb/zdgfdgfhjhgjfgsdrtretrefdhgfgjgfgfdg/Peer",     "options": {         "create": "true",         "indexBy": "_id",         "localOnly": false,         "maxHistory": -1,         "overwrite": true,         "replicate": true     },     "canAppend": false,     "write": [         "033e01213143245tftftre5234rgrg7b9080a22f3609af44bf37f"     ],     "type": "docstore",     "capabilities": [         "get",         "put",         "query",         "remove"     ] }

Note: curl -X POST https://localhost:3000/db/docstore/access/write -d 'id=045757bffcc7a4... is returning success response with empty object even I pass wrong data. And I'm using id ant public key or private key

Should we close the issue?

Theres's a change in the default access controller that needs to go thru the release pipeline, then I'll close this issue

Should we close the issue?

@0zAND1z there is no fix PR for this issue, @phillmac will close it once orbit team fix the issue


Cleanup streaming app sample

What this PR does / why we need it:
This PR streamlines the streaming app sample so that both main camera feed and small camera feeds can be dynamically refreshed based on detected cameras
Further work could be done to integrate with Helm and patch up the RBAC to target any namespace

What do you propose for the Helm integration? Or leave that for another PR? @kate-goldenring @bfjelds

What do you propose for the Helm integration? Or leave that for another PR? @kate-goldenring @bfjelds

helm integration to me is moving the samples yaml into the deployment/helm/templates folder and adding some parameterization.  i think it is fine to be a separate PR.  i also think we should update the ocpua monitoring app with this same type of logic if we can (and then add it to helm)

What do you propose for the Helm integration? Or leave that for another PR? @kate-goldenring @bfjelds

helm integration to me is moving the samples yaml into the deployment/helm/templates folder and adding some parameterization. i think it is fine to be a separate PR. i also think we should update the ocpua monitoring app with this same type of logic if we can (and then add it to helm)

A separate PR to do these two steps makes sense to me. the parameters could be deploy-streaming-sample and deploy-anomaly-sample or something along those lines

@jiria, can you bump the version twice with version.sh -u -p. You want it to be 0.0.1 greater than https://github.com/deislabs/akri/blob/main/version.txt.

Version updated, sorry for the delay.

it looks like this needs another version bump


[Bug] [admin] Dlink start Faile 该任务实例不存在

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
After the multi-tenant version ， when start dlink , report this error ,It does not affect the run, but it invalidates the task update or fails to update the task status
com.dlink.exception.RunTimeException: 该任务实例不存在
	at com.dlink.assertion.Asserts.checkNull(Asserts.java:115)
	at com.dlink.service.impl.TaskServiceImpl.refreshJobInstance(TaskServiceImpl.java:829)
	at com.dlink.service.impl.TaskServiceImpl$$FastClassBySpringCGLIB$$168bea78.invoke(<generated>)
	at org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218)
	at org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:689)
	at com.dlink.service.impl.TaskServiceImpl$$EnhancerBySpringCGLIB$$888004a6.refreshJobInstance(<generated>)
	at com.dlink.job.FlinkJobTask.dealTask(FlinkJobTask.java:75)
	at com.dlink.daemon.entity.TaskWorker.run(TaskWorker.java:45)
	at java.lang.Thread.run(Thread.java:750)

What you expected to happen
The reason is that MybatisPlusInterceptor intercepted table scan at startup，And after delete it, the program is normal
 @Bean
    public MybatisPlusInterceptor mybatisPlusInterceptor() {
        log.info("mybatis plus interceptor execute");
        MybatisPlusInterceptor interceptor = new MybatisPlusInterceptor();
        interceptor.addInnerInterceptor(new TenantLineInnerInterceptor(new TenantLineHandler() {
            @Override
            public Expression getTenantId() {
                Integer tenantId = (Integer) RequestContext.get();
                if (tenantId == null) {
                    log.warn("request context tenant id is null");
                    return new NullValue();
                }
                return new LongValue(tenantId);
            }

            @Override
            public boolean ignoreTable(String tableName) {
                return !IGNORE_TABLE_NAMES.contains(tableName);
            }
        }));

At the start up ， the tenant ID always be null，intercepted all the TaskWorker
and ... this function similar to solve it ,but it  didn't work , it intercepted  all IGNORE_TABLE_NAMES content, if i delete IGNORE_TABLE_NAMES content, its work again.
Maybe there's a problem with the logic here? I guess
  @Override
            public boolean ignoreTable(String tableName) {
                return !IGNORE_TABLE_NAMES.contains(tableName);
            }

How to reproduce
Maybe？？？？
 @Override
            public boolean ignoreTable(String tableName) {
                return IGNORE_TABLE_NAMES.contains(tableName);
            }

Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

+1
I debugged the bug and it looks like the thread of Http is different from the thread of execution and can't get the tenantId inside the threadLocal.

At present, I have solved the problem locally.
I am willing to submit a Fix!


Question 31 and 78 are duplicated

What is the difference between createElement and cloneElement?

JSX elements will be transpiled to React.createElement() functions to create React elements which are going to be used for the object representation of UI. Whereas cloneElement is used to clone an element and pass it new props.

What is the difference between createElement() and cloneElement() methods?
In JSX the React element is transpiled to React.createElement() which represents an UI element. Whereas React.cloneElement() is used in order to clone an element and pass it new props.

Thank you @MujunZ for the finding. I will fix it with a new question.

Fixed


How to handle invalid input!

Given a MMS custom service that expects JSON as input,
If the input is invalid JSON such as { "image": "I missed a quote}
MMS will return
{ "code": 500, "type": "InternalServerException", "message": "Worker died." } and reload model.
Is something can do to verify the accuracy of input to avoid such direct returns and model reload when it cannot be modified input?

@maaquib


CI: Replace calls to save-state

The save-state command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
This command is used in annotate, simplecov, notify slack success and notify slack failure actions.

These deprecation warnings have been resolved.


Integrating the saved state with the extension

Hello,
Is it possible to import the saved state file (OTDState.json) into the extension's zip so it loads automatically when a new session starts?

not possible


Merge Query from multiple files when splitting up schema.

It would be nice to design schemas by Functionality. In my current app,  I  have calendars, users, blog schemas.   I'd like to put each in its own GraphQL file along with any related Mutations or Queries.  Currently, that causes the error
There can be only one type named "Query".
There was an error pushing the API resource
Error: There can be only one type named "Query".

It would be nice if the cli merged all my queries and mutations and warned me or errored if there were duplicates so that all the graphql relative to a single domain is in one file for easy editing.
This seems to work for mutations.

Thank you for your suggestion. Given that this enhancement hasn't gotten more traction, we're going to be closing this issue for now. If more people are running into this in the future, I'm more than happy to re-open this issue.


Make folder icons work as twisties

This PR fixes #95948

Sorry for not taking this, there were a few more changes necessary.


Dev

I've fixed some CSS styles that were not done properly.

It's perfect! You can merge the branch :)


Are you trying to mount a directory onto a file (or vice-versa)?

Got an error while executing command:
docker run -p 6969:8080 -v $(pwd)/externall-file-on-host.csv:/app/computer.csv dabondi/go-rest-wol
Error:
docker: Error response from daemon: OCI runtime create failed: container_linux.go:349: starting container process caused "process_linux.go:449: container init caused "rootfs_linux.go:58: mounting \"/etc/docker/wol/go-rest-wol/externall-file-on-host.csv\" to rootfs \"/var/lib/docker/overlay2/362ba73c41a6dbeec788a61feaf4eea29a9f34391d21808d3d0415bd6f88d1b2/merged\" at \"/var/lib/docker/overlay2/362ba73c41a6dbeec788a61feaf4eea29a9f34391d21808d3d0415bd6f88d1b2/merged/app/computer.csv\" caused \"not a directory\""": unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type.

Sorry for my late Answer and my inaccurate Issue.
I was just installing and running everything and this issue happened.
Do you know what i can do about that?
I tried to change the path but it didn't work for me.
Kind Regards,
Roman Lysser

Von: David Baumann<EMAIL_ADDRESS>Gesendet: Samstag, 28. März 2020 22:16
An: daBONDi/go-rest-wol<EMAIL_ADDRESS>Cc: Roman Lysser<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [daBONDi/go-rest-wol] Are you trying to mount a directory onto a file (or vice-versa)? (#10)
Can you describe the problem what you mean a little bit better?
The Message you describe is from Docker itself, you need to mount the File to an existing file from Docker Commandline or whatever you use to spin the Container
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/daBONDi/go-rest-wol/issues/10#issuecomment-605520570, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJSIRIRTUF2FQJTITMXSV3LRJZSJTANCNFSM4LLKM6JA.


OpenCV Installation improvements & fixes

One of the fixes from the dicussion in https://github.com/rsnk96/Ubuntu-Setup-Scripts/issues/52

Travis is passing the opencv builds, but I think it'll be good to do a double check in a Docker container

@rsnk96 By any chance, were you able to test these changes in a docker container?
I think both these changes should be good to go, but just confirming

Tested once, didn't work out of the box, primarily because run_and_echo() reloads the environment within the bash script effectively, but doesn't update the environment which started the script in the first place (i.e. the shell process). Easy way around this is to just start a new sub-process with the updated SHELLRC file like here just before the script exits (and this has to be done within an if [[ ! -n $CIINSTALL ]]; then....fi so that Travis doesn't fail with this addition)

@rsnk96 Ohk, thanks for the detailed explanation and testing, otherwise wouldn't have caught the problem. Would have tested it myself but don't have a very good internet connection right now.
I've Pushed a fix as you described

Things are definitely wrong right now -
./opencvDirectInstall.sh: line 30: export: `--variable': not a valid identifier
./opencvDirectInstall.sh: line 30: export: `pkg-config)': not a valid identifier

https://travis-ci.org/rsnk96/Ubuntu-Setup-Scripts/jobs/626129056#L1195
Need to look into this and fix it

Error's not there now, changed run_and_echo () {  $1 ... } to run_and_echo () {  eval $1 ... } to force it to run the embedded commands

Lapacke change fixes the Atlas not found problem, the OpenBLAS ones generally need code change
Will try to have a go at some of the other errors in https://github.com/rsnk96/Ubuntu-Setup-Scripts/issues/20

Updates looks good, and haha didn't know that there was a version of lapack with an e that has more features. Whenever you feel you've hit a wall wrt fixing the issues in https://github.com/rsnk96/Ubuntu-Setup-Scripts/issues/20, feel free to merge this PR

@rsnk96 Fixing the other errors will need more time, will try to do that in another PR, this one has enough things in it already
It'll be good to have another test in a docker container soon, but will merge this for now. Also will update the issue with the current errors


Error when passing --release-it

Currently running:
ember addon ember-new-v2-addon-testr -b @embroider/addon-blueprint --pnpm --skip-npm --typescript --release-it

and getting:
Error creating new application. Removing generated directory `./ember-new-v2-addon-test`
Command failed with exit code 1: create-rwjblue-release-it-setup --no-install
[HttpError: Not Found] {
  statusCode: 404,
  headers: {
    server: 'GitHub.com',
    date: 'Wed, 22 Mar 2023 02:37:06 GMT',
    'content-type': 'application/json; charset=utf-8',
    'content-length': '106',
    'x-github-media-type': 'github.v3; param=symmetra-preview; format=json',
    'x-github-api-version-selected': '2022-11-28',
    'x-ratelimit-limit': '60',
    'x-ratelimit-remaining': '48',
    'x-ratelimit-reset': '1679456172',
    'x-ratelimit-used': '12',
    'x-ratelimit-resource': 'core',
    'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset',
    'access-control-allow-origin': '*',
    'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
    'x-frame-options': 'deny',
    'x-content-type-options': 'nosniff',
    'x-xss-protection': '0',
    'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
    'content-security-policy': "default-src 'none'",
    vary: 'Accept-Encoding, Accept, X-Requested-With',
    'x-github-request-id': 'E2D8:2E7B:5B89B1:BC9C65:641A69D2',
    connection: 'close'
  },
  body: {
    message: 'Not Found',
    documentation_url: 'https://docs.github.com/rest/reference/issues#create-a-label'
  },
  method: 'POST',
  endpoint: '/repos/esbanarango/ember-new-v2-addon-test/labels'
}
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "Error: Not Found".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

Node.js v18.14.2

Stack Trace and Error Report: /var/folders/jq/k7tk0_dx5yvc0cxl4q3ll7680000gp/T/error.dump.581f5380f88e7166d5a81988b66fa7e3.log

Seems like / similar to https://github.com/rwjblue/create-rwjblue-release-it-setup/issues/81.

I tried this out, and it was working fine for me!

Seems like / similar to https://github.com/rwjblue/create-rwjblue-release-it-setup/issues/81.

That issue has been resolved, and seems to refer to a bug when the git repository is empty. And in fact, when running the blueprint, it should be empty, as we cannot know this upfront. However, in the stack trace above, the create-rwjblue-release-it-setup seems to call the Github API for creating the labels, but actually with a known repository! See:
endpoint: '/repos/esbanarango/ember-new-v2-addon-test/labels'

So if esbanarango/ember-new-v2-addon-test does not exist (it does not), that would explain the error. But I wonder where that comes from even? @esbanarango any idea?


Too many files in output directory

Using the build tasks adds a lot of (unnecessary) files to the output directory:

dSPACE.Runtime.InteropServices.dll
Microsoft.Build.Tasks.Core.dll and all its dependencies

All this files have nothing to do with the COM-Server I am building and they are not needed to run it. Is there a way preventing these files to be copied to the output directory?
I assume they are put in the output directory because they are referenced by the dSPACE.Runtime.InteropServices.BuildTasks-package.

Maybe you should add <PrivateAssets>all</PrivateAssets> to the <PackageReference Include="microsoft.build.tasks.core" Version="17.0.0" /> in dscom.build.csproj to prevent this behaviour?

I managed to get the expected behaviour by deleting runtime; from the IncludeAssets.
Thus the reference in the  .csproj file shall look like
    <PackageReference Include="dSPACE.Runtime.InteropServices.BuildTasks" Version="0.17.0" NoWarn="NU1701">
      <IncludeAssets>build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
      <PrivateAssets>all</PrivateAssets>
    </PackageReference>

instead of
    <PackageReference Include="dSPACE.Runtime.InteropServices.BuildTasks" Version="0.17.0" NoWarn="NU1701">
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
      <PrivateAssets>all</PrivateAssets>
    </PackageReference>

Hi @EO3-Kopf, if you have a fix for the problem, feel free to submit a pull-request.
We would be happy about it :-)

Hi @marklechtermann, I would if knew where / what to change. I think it has something to do with the publishing of the nuget package. Since I have not yet published on nuget.org I don't know how to influence the PackageReference. (The one shown at https://www.nuget.org/packages/dSPACE.Runtime.InteropServices.BuildTasks#package-reference contains runtime; And it is obviuosly the same reference that is added to the project file when installing this package. I don't know how to change this behaviour.) However I could do a pull request and change readme.md at the respective point.

🤔 @marklechtermann, as it seems I am not allowed to create a new branch. At least I don't find the respective button here. How to create a pull request then?

Hello @EO3-Kopf,
contributing to open source projects in GitHub is actually quite easy.
In most projects, such as dscom, you can not directly create a branch.
This is not desired for security reasons.
Instead you have to create a fork.
On the top right corner of the dscom github site you will find the button "Fork".

This fork is then part of your account.
After the code change you can make a pull request, so that your fork is taken over into the main branch of dscom.
Here you can find more information:
https://www.freecodecamp.org/news/how-to-fork-a-github-repository
If you have any questions, please feel free to contact me.
You are also welcome to send me a direct message on twitter.
https://twitter.com/marklechtermann

Maybe you should add <PrivateAssets>all</PrivateAssets> to the <PackageReference Include="microsoft.build.tasks.core" Version="17.0.0" /> in dscom.build.csproj to prevent this behaviour?

In contrast to #145 and #146 this seems to be the correct solution for this. Following the NuGet Specification and the Microsoft Learning Center, it would be more correct and actually drop the dependencies.
I'll try myself to get more or less handy with this. In the end, removing the assets should not be the way.

Yes, I think that might do the trick. Eliminating the cause is always better than mending the symptoms.

If I understand you correctly the problem here was how the Build Task was included in the built project and not with the build task itself.
In that case this Issue can be closed and the Pull Request #145  can be closed, right?
Or did I misunderstand your discussion?

@SOsterbrink I think we can abandon the PR, but the issue still remains open. I'm currently updating the package references,

#150 was merged. The dependencies are no longer reflected to the nuspec file.


MrGeo WMS Issues in Qgis

Currently the tobler layers, slope-gradient-30m and osm-raods-kph-30m do not show correctly in Qgis, they render as solid black boxes.

These 3 layers have some outlier values that cause the default black->white color scale to cluster the colors toward the lower end (black).  We've created other scales to display them better.
If they also don't already have one, we can set a default color scale tag on the pyramids themselves to force the correct scale instead of the black->white one


equals method is missing

I would suggest the following equals method
equals: function equals(otherSet){
    //if (!otherSet instanceof Set){return false;} //comment out for typecheck
    var otherKeys = Object.keys(otherSet._values).sort();
    var myKeys = Object.keys(this._values).sort();
    return JSON.stringify(otherKeys) === JSON.stringify(myKeys);
}

If you would like to open a pull request with this, I'd be open to reviewing/merging it.
The only difference I would suggest is, instead of using JSON.stringify, use this._hashFunction, which defaults to JSON.stringify.


Z versions of the datatypes

Relies on pull request geo#111, and provides support for the Z versions of the datatypes implemented there.

Thanks! I'll try to get it published as soon as possible

Published! https://hex.pm/packages/geo_postgis/3.2.0


Vertical carousel scrolls page sometimes

Bug is related to

[x] embla-carousel (core package)

Embla Carousel version

v8.0.0-rc0x

Describe the bug

Scrolling a vertical carousel on a touch device/screen don’t prevent vertical scrolling of the page sometimes.

Expected behavior

Scrolling a vertical carousel on a touch device/screen should prevent vertical scrolling of the page.

Additional context
This is a blocker and has to be fixed before the release of stable v8 if the problem is caused by the Embla code.

Seems to be a webkit bug. Maybe this is the bug.

Solved here and here.


sw-6566 adding colours for due date #minor

Codecov Report
Patch coverage: 73.68% and project coverage change: +0.32 :tada:

Comparison is base (1f8bc63) 62.83% compared to head (2441e5a) 63.16%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #331      +/-   ##
==========================================
+ Coverage   62.83%   63.16%   +0.32%
==========================================
  Files          13       13
  Lines        1036     1086      +50
==========================================
+ Hits          651      686      +35
- Misses        341      356      +15
  Partials       44       44

Impacted Files
Coverage Δ

internal/sirius/get_task_list.go
82.87% <73.68%> (-4.92%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Miscolored terminal output

https://github.com/kennethreitz/pipenv/blob/dba68e00f4734d07881d51173c03c27089dc41af/pipenv/cli.py#L985
The above line results in output like this (due to a small oversight in the crayons package):

The issue is also mentioned here: kennethreitz/crayons#9
And I've written the fix for it here (it just needs to be reviewed and merged and the pypi package updated): kennethreitz/crayons/pull/13
Thanks everyone for all your hard work on this great software :)

Thanks for the report @kathawala!  We'll get the crayons package updated and the bug will be fixed!


Feature/veldrid window 2

Going to use https://github.com/mellinoe/veldrid-docs/blob/master/updates/4.6.0.md#better-sdl2-input-handling instead


Cleanup gz-sim-yarp-plugins-check-model tests

Fix ENVIRONMENT and ENVIRONMENT_MODIFICATION arguments to work on Windows
Uncomment tests that were failing
Fix URI of laser.ini in laser tutorial (that was not an URI before)

Great, now all the test pass!

Rebased on main.


Remove 9092 lb port

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

Codecov Report
Patch coverage has no change and project coverage change: +54.63% :tada:

Comparison is base (b71013f) 25.47% compared to head (2545df8) 80.10%.
Report is 8 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1736       +/-   ##
===========================================
+ Coverage   25.47%   80.10%   +54.63%
===========================================
  Files         262        9      -253
  Lines       30324      548    -29776
  Branches     5126      246     -4880
===========================================
- Hits         7726      439     -7287
+ Misses      22336      109    -22227
+ Partials      262        0      -262

Flag
Coverage Δ

adapter
?

common-controller
?

config-deployer-service
?

idp-domain-service
80.10% <ø> (ø)

router
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 253 files with indirect coverage changes

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


When video is playing, save position intermittently

What is the current behavior?
Currently, the position of a video is saved when pausing, seeking, or navigating to a url that is handled by the router. However, if you close the tab or navigate away from the site with a hard link, the play position is lost.
What is the new behavior?
This patch will intermittently save while the video is playing to better prevent losing video progress.  An arbitrary interval of 15 seconds was chose, but this can be adjusted if there is some more appropriate interval.
Other information

PR Checklist

Toggle...
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

Thanks David! Not sure how I feel about 1500 seconds - I'm sure the performance implications are low, but we can probably get away with 5-10 seconds. What do you think?

Ah, my mistake.  I'd upped it to 15 seconds while getting the PR set, but forgot to push that commit.  Just pushed it now.  Does 15 seconds work, or would you like me to lower it 5 or 10?

15 seconds should be perfect, thanks again!

Would be nice if the interval isn't running when paused or ended

Yeah, I was sorta torn on that one.  To do that, the code is just a pinch more cryptic.  Weighing the complexity against the ever-so-slight increase in code clarity with the overhead of an extra function call every 15 seconds when idle, I tipped to the latter.
I went ahead and implemented the changes to only run when playing in the most recent commit here: https://github.com/OdyseeTeam/odysee-frontend/pull/1013/commits/5ce6ab9689c1610f24869c7347b823cc2023b767
It's not a huge difference either way, but let me know what you think.  I can keep or revert that commit. And, of course, I can implement any other suggestion on that front.

Yeah, I thought the previous one was a good tradeoff for clarity, less complex and one less state.  Sorry, should have made it clear in my previous sentence that it was ok, knowing the tradeoffs.  But at least we have a reference on how to do that if we really need it.
Let's revert to the simpler ref version?  Thanks!

No worries. Just rolled that one back.


Docker compose

Use docker-compose to make docker usage easier. Now, instead of rebuilding the entire image every time you tweak a theme, you can just stop and restart it. You still benefit from the self-contained nature of the docker image.
This is a small PR, but I'm now using docker-compose for development and it is a dream. Change the command from 'node' to 'nodemon' for dead simple development without installing any dependencies on your local system.
It's so much easier to use, that in my opinion, docker should be the default installation method!

Never mind, mounting node_modules sometimes breaks dependencies :(


No releases on the github repo

Hello,
I am working on packaging gpuctypes for the nixpkgs repository.
For this, we rely on git releases/tags to fetch the right sources from github.
Despite having clear releases on pypi for this library, the github repo doesn't have any tag for version.
Would it be possible to create a tag corresponding to the latest release of the library ?
Also, thank you very much for developing tinygrad and its ecosystem !

Yes please!


Nullable check with class fields lost after first expression

When doing a nullable check on a class field from within the class like if(this.field) {}, the field is only considered to be not null in the first expression within the checked scope (the if). All subsequent usages of the field are considered to be potentially null again.
if(this.obj) {
  this.obj.call(); // < works
  this.obj.apply(); // < fails "this.obj might be null/undefined"
}

At first I thought it might have to do that within call() I could set the field to null, but then the same thing should also not work for simple variables, but it does:
var x:?{ call: Function, apply: Function } = null;

if (x) {
  x.call();
  x.apply();
}

Because here I could also potentially set x to null within x.call().
So I'm not sure if this is a bug, but to me it seems it should work as it is pretty common to do this kind of checks.
Added a failing test case.

This behavior is expected and is documented here: https://flowtype.org/docs/dynamic-type-tests.html#caveats
Flow is scared that the function call might change the value of this.obj back to null. Local variables like x don't have this problem since the functions don't have access to them.


Decouple and upgrade dependencies to latest version

Description

Upgrade evm-connector to 0.6.2
Decouple redstone-protocol and upgrade to use latest version of @redstone-finance/protocol instead

Acceptance Criteria
The dependent logic should be adjusted and tested for correct functionality

https://github.com/redstone-finance/redstone-evm-examples could be used for reference to update the evm-connector  dependency.

Fixed by #220.


Why can I load the same file multiple times into the variation service?

Fixed in https://github.com/PATRIC3/p3_web/pull/301


Return an explicit 500 if there's an exception generating metrics

The current behaviour observed in #84 is pretty bad. This implements the simplest possible fix: if there's any exception of any kind generating metrics, log it as we are now, but change the response code to a 500.
send_error is used, which dumps a random bit of html into the response, which doesn't seem like a problem. send_response(500) could be used instead, to get a 500 with a blank body.

Obsoleted by #119


captureStoreUpdates — return the full path to the property that changed

Describe The Problem To Be Solved
I'm migrating from on-change to using captureStoreUpdates().
When using captureStoreUpdates(), setting a new value mutable.settings.model = model will produce the path: ["settings"]. However, if I replace the entire settings object this will trigger more updates than just updating the settings.model property.
Suggest A Solution
I propose for an option or change in behavior where setting mutable.settings.model produces path: ["settings", "model"].

captureStoreUpdates is returning the path to the object, not property, by design. It allows to simplify the implementation a lot—there is no easy way of checking which property was changed without cloning each object and diffing it.
It's not properly communicated in the readme, which need to be corrected.
I'm also not against adding another helper for diffing the objects that would wrap captureStoreUpdates. But captureStoreUpdates should remain as is, because getting the changed object is often what you need.

I would choose the simpler implementation as well.
It would be great then to have another helper as well. However, if this isn't something you are interested in at the moment you can close this issue.
Thanks for the detailed explanation!


[DT-1077] typescriptify datasetstatistics and fix

Addresses
I noticed the issue appeared to be based on properties missing, so I added a test with no additional properties that still renders.
I opted not to put in the additional effort to typescriptify the test, because test files are much less impact as js vs ts, and if we decide to move unit tests to jest eventually, I wanted to prioritize effort.
Summary

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

Here's a screenshot, but I don't think we can get much clarity on whether it worked without the specific prod data.