Code cleanup for future FtcRobot controller

We have a script that takes care of these issues, but thought the larger ftc community could benefit from this request.
1.)FIRST-Tech-Challenge/FtcRobotController/FtcRobotController/build.gradle
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_7
targetCompatibility JavaVersion.VERSION_1_7
}
should  these be JavaVersion.VERSION_1_8 to match build.common.gradle ?
2.) FtcRobotController/FtcRobotController/src/main/assets/
this folder should be removed, it adds undue size to the robot controller apk and does not add value.
These historic files can be added to a new repository along with the pdf target files; split into separate dependencies like the last 2 seasons within build.dependencies.gradle
dependencies {
implementation 'org.firstinspires.ftc:gameAssets-PowerPlay:1.0.0'
}
3.) FtcRobotController/FtcRobotController/src/main/java/org/firstinspires/ftc/robotcontroller/internal/FtcOpModeRegister.java
remove import org.firstinspires.ftc.robotcontroller.external.samples.ConceptNullOp;
this import is not used  and puts an undefined dependency on the external/samplesfolder on FtcOpModeRegister.java

Thank you for reporting this.  It is being tracked internally.

FtcRobotController/FtcRobotController/src/main/java/org/firstinspires/ftc/robotcontroller/internal/FtcOpModeRegister.java
remove import org.firstinspires.ftc.robotcontroller.external.samples.ConceptNullOp;
this import is not used and puts an undefined dependency on the external/samplesfolder on FtcOpModeRegister.java

ConceptNullOp is actually being used in the javadoc with the @link property, which means it is required for the javadoc build so the resultant documentation can reference the class.

I see the reference on line 52, but i am not sure i follow why the import
needs to reference that particular opmode, with the additions of on the
block java and blocky for novice teams to leverage to the robot controller.
On Mon, Jun 12, 2023, 12:27 PM Craig MacFarlane @.***>
wrote:

FtcRobotController/FtcRobotController/src/main/java/org/firstinspires/ftc/robotcontroller/internal/FtcOpModeRegister.java
remove import
org.firstinspires.ftc.robotcontroller.external.samples.ConceptNullOp;
this import is not used and puts an undefined dependency on the
external/samplesfolder on FtcOpModeRegister.java
ConceptNullOp is actually being used in the javadoc with the @link
https://github.com/link property, which means it is required for the
javadoc build so the resultant documentation can reference the class.
—
Reply to this email directly, view it on GitHub
https://github.com/FIRST-Tech-Challenge/FtcRobotController/issues/383#issuecomment-1587666689,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHRDLUMNR6COVLHAO36KCKTXK47OBANCNFSM6AAAAAAQW34JG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

1.)FIRST-Tech-Challenge/FtcRobotController/FtcRobotController/build.gradle compileOptions { sourceCompatibility JavaVersion.VERSION_1_7 targetCompatibility JavaVersion.VERSION_1_7 }
should these be JavaVersion.VERSION_1_8 to match build.common.gradle ?
This was done in version 8.2.

the assets folder was removed in 9.0

if line 52 was * See, for example the class annotations in {@link org.firstinspires.ftc.robotcontroller.external.samples.ConceptNullOp}. then the import on line 37 import org.firstinspires.ftc.robotcontroller.external.samples.ConceptNullOp; should be able to be removed without breaking javadoc

Created https://github.com/FIRST-Tech-Challenge/FtcRobotController/pull/802 but it was auto-closed.

@cmacfarl Just wanted to check in on the status of fixing the last part of this issue.  The fix already exists (https://github.com/FIRST-Tech-Challenge/FtcRobotController/pull/802) but due to the nature of SDK development, there's seemingly no way to properly submit these changes to upstream.  I also came across some more simple-but-potent optimizations in https://github.com/FIRST-Tech-Challenge/FtcRobotController/issues/797 but once again sadly PRs are not possible.


Enable nullable in repository

Description
Enable nullable feature so that we can identify and remediate potential null reference exceptions in our codebase. This will also help users extending our behaviors to better understand when we can/will pass null objects.

We are done outside of test projects where value is lower (especially for end users) so I'll mark as done.


Enhance README "Some other examples" section

Two modifications to make it easier to get to resources:

Modified so that "benchmark on a bunch of different tasks" sends the reader to a README.md file explaining the benchmarking process, instead of just dropping them into the directory on GitHub.
Added a link to ArXiV for "The Unreasonable Effectiveness..." article, so that now the interested reader can jump directly to ArXiV instead of having to go through the Register article. The link to the Register article is retained, though.

Thanks @rpgoldman !

@arnavsinghvi11 Thanks for the review and corrections (and merging)!


backBehavior: 'history' doesn't work

I've just upgrade react-navigation and material bottom tabs to latest version after few months. But i noticed that backbehavior history doesnt work. Order works well. With history,  back press closes app.

It works with 2.0.0, 1.1.1, 1.1.0 and 1.0.0 versions but only 1.0.0 version fully working. I didn't test other versions.

backBehaviour: 'history' still not working with createBottomTabNavigator and createMaterialTopTabNavigator (react-navigation-tabs ^2.5.6).

@robizzt install version 1.0.0 i think there's no big change. I'm using it without a problem. Better than nothing.

For anyone who has the same issue, please follow @TrueTiem 's advice. I downgrade react-navigation-tab from 2.6.2 to 1.0.0 and it's working properly now 💇♂ .
Wish I can get 2 hours of my time back on this 🤦♂


TransactionTooLargeException when go to the background

Hi there 🙂
I have found some problem, when we try send big data, in my example it is very long string. When user go to the background it will get TransactionTooLargeException
Fatal Exception: java.lang.RuntimeException: android.os.TransactionTooLargeException: data parcel size 2196356 bytes        at android.app.ActivityClient.activityStopped(ActivityClient.java:86)        at android.app.servertransaction.PendingTransactionActions$StopInfo.run(PendingTransactionActions.java:143)        at android.os.Handler.handleCallback(Handler.java:938)        at android.os.Handler.dispatchMessage(Handler.java:99)        at android.os.Looper.loopOnce(Looper.java:226)        at android.os.Looper.loop(Looper.java:313)        at android.app.ActivityThread.main(ActivityThread.java:8751)        at java.lang.reflect.Method.invoke(Method.java)        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:571)        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1135)
I have data class with Int, Booleands and Strings. In some cases, my data have very long string and in this case app Is crashes when go to the background.
navigator.navigate(                         LectureDetailsScreenDestination(                             book = it as Book                         )                     )
When I remove book argument inside LectureDetailsScreen my app is working well. Do you know guys how to get around this problem?
Problem exist only on older phones. (2/3+ years)

I found a certain workaround. To Destination now I'm send object converted to ByteArray, than in VM/Composable I decode it  back to object.
Here is my Zipper
fun Any.zipToByteArray(): ByteArray {     val bos = ByteArrayOutputStream()     GZIPOutputStream(bos).bufferedWriter(StandardCharsets.UTF_8).use { it.write(Gson().toJson(this)) }     return bos.toByteArray() }
inline fun <reified T> ByteArray.toObject() : T {     val unzipped = GZIPInputStream(this.inputStream()).bufferedReader(StandardCharsets.UTF_8).use { it.readText() }     val type = object : TypeToken<T>() {}.type     return Gson().fromJson(unzipped, type) }
This does not solve the problem 100%, but it definitely reduces the number of devices it occurs on.

@PawelSobaszek Hi!
So this is not a bug. You cannot send big data as navigation arguments, that is an anti pattern and can lead to these sorts of issues. Android Binder has limited space and so passing only ids or very small data structures is very much recommended.

So in your case, I would find another way to structure screens so that you don't need to pass this as nav argument. Consider saving books (maybe you already do?) in a database, and pass only the book id.


Fix bridge-cast error.

kSecAttrKeyTypeECSECPrimeRandom is a CFStringRef, not a number.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


getting wrong type after .connect()

Hi
I am unable to use .connect() as it changes the type to what it has already generated regardless of the type I determined.
import { ethers } from "hardhat";
import {Lock, Lock2} from "../typechain-types"

type Lock1AndLock2 = Lock2 & Lock

describe("Sample", function() {
    it("Should have the same type", async function() {
        const Lock = (await ethers.getContractAt(abi, address)) as unknown as Lock1AndLock2; // type: Lock1AndLock2
        const LockWithDifferentRunner = Lock.connect(undefined); // type: Lock2

        /*
          Currently:
            Lock: Lock1AndLock2
            LockWithDifferentRunner: Lock2
        */

        /*
          Expected:
            Lock: Lock1AndLock2
            LockWithDifferentRunner: Lock1AndLock2

            why the type of LockWithDifferentRunner changed after conencting to a different runenr?
        */
    })
})

This requires this type in return of https://github.com/dethcrypto/TypeChain/blob/ec6b8600042e78020af3e05647e5044bd44425b9/packages/target-ethers-v6/src/codegen/index.ts#L69


Suggestion : add checkv step beforehand to remove complete genomes

Hi,
I compared the results that I obtained using vibrant contigs before and after running phamb using checkv
Before

checkv_quality
n
mean
sum
max

Complete
557
46179.5
25721993
373392

High-quality
413
44008.8
18175622
275626

After

checkv_quality
n
mean
sum
max

Complete
351
54600
19164596
197996

High-quality
975
72413.1
70602775
622104

Where mean is the mean of contig length, sum is the total length of all contigs and max is the maximum size of the biggest “virus”.
This suggests to me that phamb wants to combine contigs even if they are considered "complete".
Maybe adding a checkv step beforehand to remove the complete ones from the phamb analysis could be useful?

Hi Greg
Yes, I have seen cases where VAMB clustering results in complete virus contigs (DTR presence) are binned with unrelated contigs (not very frequent though), i.e. bin contamination. Based on your results, it looks like it is a matter of 557-351=206 complete virus contigs that ends up in a HQ bin, is my interpretation correct?

I think you're absolutely right that an option would be to run a prior search on the individual contigs for the presence of complete virus contigs (i.e. a DTR search) then remove them from the set going into phamb. However, that would mean you end up running CheckV twice, which turns into a lot of computation time + money.
An alternative idea that I would appreciate as a user, would be to scan viral bins for the presence of DTR's (complete virus contigs) then extract the complete virus and discard the "contaminating" contigs. Howver, if the discarded contigs frequently represents virus sequences this solution is not ideal.

I would appreciate your thoughts on this, thanks!
Best,
Joachim

Regarding the above issue, I have found a few hundred contigs representing complete viral genomes (based off CheckV - DTR presence and high confidence only). I am using both VAMB and PHAMB. My questions are:

When would be the best time to remove these contigs from the dataset?
When would be the best time to add them back in (and how)?


Try to avoid ssh-agent warning

Resolving "ssh-agent not found" is a FAQ and there's really no reason not to try to avoid it... the vast majority of the time its location can be resolved from the location of git.exe (specifically, "..\bin\ssh-agent.exe").

Implemented in #141, closing.


Bugfix

Facebook changed their API. Now including "address" field in call to "me" returns error from Facebook.

This PR couldn't be easily merged due to conflicts. Given that it's a simple change, I've applied it manually.


Adjust github workflow go.yml to not fail due to false positives on error test output

We have many unit tests that emit "error" output as a valid output from error tests; however, the default "go.yml" workflow provided by github does not know this.  As a result, it "fails" the pre-check builds.  For example:
=== RUN   TestValidateSyntaxErrorCdx13Test2
Error:  schema_formats.go(463) schema.DisplayJSONErrorDetails() JSON Syntax error: offset: 4452, line 123, character 28: invalid character '[' after object key
--- PASS: TestValidateSyntaxErrorCdx13Test2 (0.00s)

Although the test PASSED, the github action sees the word Error and fails the pre-check.
Adjust command to pass in quiet flag to unit test to not emit any output from tests at all...

@markstur darn Mark that was a fast review! Thanks!


Hl 129 delete drafts

Description :sparkles:
This PR refactors the previously added  delete_cancelled management command into delete_applications command that can be used to schedule the deletion of applications that have either cancelled or draft status.  A container running Mailhog is added  to the local development environment for easier email testing.
A new command check_drafts_to_delete is added for checking application drafts that are closing on a deletion date (180 days at the moment) and sending email notifications to the contact emails specified in the applications.
For manual testing, the seed command is also updated to create at least 5 drafts that will be deleted and 5 that will be deleted in 14 days.
Issues :bug:
The emails sent by the applications are plain text, they need to be styled a bit at some point.
Testing :alembic:
After spinning up the local environment with the updated docker-compose.benefit.yml, Mailhog can be accessed at http://localhost:8025/. Run python manage.py seed and then to test pre-delete notification, run python manage.py check_drafts_to_delete and check that emails are coming into the Mailhog inbox.
Run python manage.py delete_applications --days_to_keep=180 --status="cancelled" to test the deletion of applications that have been drafts for 180 days.
Runpython manage.py runjobs dailyto run all current daily jobs.
Run pytest applications/tests/test_application_tasks.py to test with Pytest.
Screenshots :camera_flash:

Additional notes :spiral_notepad:

Codecov Report

Merging #1835 (b9c0995) into develop (a8d126b) will increase coverage by 28.41%.
The diff coverage is 96.67%.

:exclamation: Current head b9c0995 differs from pull request most recent head 3a8eb69. Consider uploading reports for the commit 3a8eb69 to get more accurate results

@@             Coverage Diff              @@
##           develop    #1835       +/-   ##
============================================
+ Coverage    66.71%   95.13%   +28.41%
============================================
  Files          920      206      -714
  Lines        26585     8922    -17663
  Branches      2559        0     -2559
============================================
- Hits         17737     8488     -9249
+ Misses        8501      434     -8067
+ Partials       347        0      -347

Impacted Files
Coverage Δ

.../applications/jobs/daily/daily_application_jobs.py
62.50% <0.00%> (ø)

backend/benefit/users/api/v1/permissions.py
76.92% <76.92%> (ø)

backend/benefit/users/api/v1/authentications.py
83.33% <83.33%> (ø)

backend/benefit/users/api/v1/views.py
93.75% <92.59%> (-1.49%)
:arrow_down:

backend/benefit/users/tests/keys.py
93.75% <93.75%> (ø)

backend/benefit/applications/api/v1/serializers.py
94.09% <100.00%> (+0.12%)
:arrow_up:

...ions/management/commands/check_drafts_to_delete.py
100.00% <100.00%> (ø)

...cations/management/commands/delete_applications.py
100.00% <100.00%> (ø)

...d/benefit/applications/management/commands/seed.py
96.00% <100.00%> (+0.65%)
:arrow_up:

backend/benefit/applications/tests/conftest.py
99.31% <100.00%> (+2.31%)
:arrow_up:

... and 748 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Blocked a frame with origin "http://example.com" from accessing a cross-origin frame.

There is a bug that appears when the component gets rerendered.
Steps to reproduce:

Open a popup by clicking on a social button (of Facebook, for example) and leave it open.
Make the component to rerender via v-if or just reload the current page
Click on a social button of another social network (Twitter, for example)

An error will be thrown

And it appears on this line:

If this info is needed:
Chrome version - 69.0.3497.100
vue-social-sharing version - 2.3.3

has this been resolved? I'm having the same issue

Hello @MarineLB,
this is a security behavior enforced by the browser, it should work if you're testing the package with an https protected domain. See https://stackoverflow.com/questions/42825369/i-want-to-access-popup-functionswindow-open-with-different-domain

I could not test it but I think, If you change sharer to _blank then sharing scope for all social media get separated in the browser.
this.popup.window = $window.open(
        this.shareLink,
        '_blank',
        ...
      )

Hello @MarineLB,
this is a security behavior enforced by the browser, it should work if you're testing the package with an https protected domain. See https://stackoverflow.com/questions/42825369/i-want-to-access-popup-functionswindow-open-with-different-domain

@nicolasbeauvais Hy i have the same error on the server which has the https

any update?

Facing same problem on my production domain. But it's working my application last release.
working on node version: 14.15.1
now Update node version: 16.17.1 then facing this blocked error when open popup


Export file saved to tmp folder

I have defined export="${project.basedir}/bin/export for the cap task, but the export files are not generated there. I have already deleted the folder, but the problem persists under Ubuntu 22.04.
Take not of /tmp/jccpro15000839968602750102/com/example/foo/javacard/foo.exp

[INFO]    [verify] Verification passed
[INFO]       [cap] CAP saved to /home/foobar/Projekte/foo/my-lib-dev/bin/my-lib.cap
[INFO]       [exp] EXP saved to /tmp/jccpro15000839968602750102/com/example/foo/javacard/foo.exp
[INFO]       [jar] Building jar: /home/foobar/Projekte/foo/my-lib-dev/bin/export/foo.jar
[INFO]       [jar] JAR saved to /home/foobar/Projekte/foo/my-lib-dev/bin/export/foo.jar
[INFO] Executed tasks

That's has been broken indeed, thanks!

This should be fixed in https://github.com/martinpaljak/ant-javacard/releases/tag/v23.08.07


Purging mixin

Start to consolidate all the purging.rb implementations.

Purging realtime metrics no longer brings back all ids. (still necessary for the rollups)
Using more of the purging mixin boilerplate (e.g.: delete by date, configuration variables)
Using arel for field comparison.
metric/purging now looks very similar to the mixins/purging
removed some implementation specific specs around calling delete_all.

/cc @matthewd @dmetzger57 fyi

results

metric
before
after

avg time
191 ms
34 ms

queries
5
5

sql time
45 ms
17 ms

mem
15.5 mb
250kb

objects
173k
3k

raw
10000.times { DriftState.create(:timestamp => 1.second.ago) }
10000.times { DriftState.create(:timestamp => 5.years.ago) }
puts DriftState.count
# (2.3ms)  SELECT COUNT(*) FROM "drift_states"
# => 20000

thrices("drift_state_after") { DriftState.purge(:date, 6.months.ago) }

prep drift_state_before             544ms    7+9q[    47+6ms]   28_256_724b    309_056obj/356_937/1_598_668
drift_state_before 1               197ms    5+0q[    42+0ms]   15_463_614b    173_027obj/      0/       1
drift_state_before 2               188ms    5+0q[    45+0ms]   15_557_593b    173_024obj/      0/       1
drift_state_before 3               181ms    5+0q[    42+0ms]   15_463_574b    173_026obj/      0/       1
drift_state_before 4               201ms    5+0q[    43+0ms]  -34_123_115b    173_024obj/  5_466/       1

prep drift_state_after              233ms    6+5q[    32+3ms]   12_153_941b    130_159obj/     -4/   1_739
drift_state_after 1                 33ms    4+0q[    15+0ms]      237_956b      3_089obj/      0/       1
drift_state_after 2                 32ms    4+0q[    15+0ms]      237_916b      3_088obj/      0/       1
drift_state_after 3                 31ms    4+0q[    16+0ms]      237_916b      3_088obj/      0/       1
drift_state_after 4                 34ms    4+0q[    16+0ms]      237_916b      3_088obj/      0/       1

Huh. so the batching is set to 10k, So this did not show batching, but rather sub query difference. I ran with 50k records (4 batches of 10k to be removed) I notice that the count(*) is showing up in the query counts too.

metric
before
after

avg time
638 ms
34 ms

queries
11
7

sql time
173 ms
87 ms

mem
15.5 mb ?
300kb

objects
684k
4.2k

data over wire
40,000 rows
0 rows

raw
before 50k
prep drift_state_50k_before 979ms   13+9q[190+6ms]  30_811_682b  820_712obj/368_522/1_599_193
 drift_state_50k_before 1   635ms   11+0q[171+0ms]  12_494_455b  684_683obj/  1_548/       2
 drift_state_50k_before 2   642ms   11+0q[173+0ms]      70_327b  684_680obj/    136/  55_387
 drift_state_50k_before 3   643ms   11+0q[175+0ms]  -3_039_094b  684_682obj/-520_913/       1
 drift_state_50k_before 4   634ms   11+0q[179+0ms]     -45_087b  684_680obj/-520_731/       1

after 50k
prep drift_state_50k_after  388ms    9+8q[ 83+5ms]  12_906_791b  138_709obj/358_009/1_596_373
 drift_state_50k_after 1     34ms    7+0q[ 83+0ms]     304_706b    4_236obj/      0/       1
 drift_state_50k_after 2     34ms    7+0q[ 87+0ms]     304_477b    4_233obj/      0/       1
 drift_state_50k_after 3     37ms    7+0q[ 83+0ms]     304_666b    4_235obj/      0/       1
 drift_state_50k_after 4     35ms    7+0q[ 79+0ms]     304_477b    4_233obj/      0/       1

Overall looks good...a very minor comments.  Can you remove the "not sure if we need", "not sure who uses" comments as well.
Also, I personally prefer using the it string, to say it's private, as opposed to a comment..what do you think?
    it "#purge_ids_for_remaining (private)" do

# as opposed to

    # private method - not sure if we need
    it "#purge_ids_for_remaining" do

@Fryguy thanks for the pointers - made all of those changes.
This should be ready to go

@Fryguy please review

@matthew @Fryguy please review

@matthewd the next step after this would be to just leave all the ids in the database (of course) - and THEN we should see the numbers go down for the use case when there are nested transactions.

@Fryguy made those changes. Anything else to do here?

@matthewd @Fryguy made those changes. I have a followup PR after this and getting some pressure to put this to rest. anything else you want before we merge?

Some of the comments are weird (like the "darn" one), but merging anyway.


Add missing 'v' prefix on release/tag

/test-force-approve
changes in workflow not covered by PR/branch test flow


fix(progress): properly set and use max value in progress component

Resolves #3082.
This PR resolves two issues with the existing <Progress /> component and provides extra documentation on how to use the max attribute.
a max value besides 100 did not look as expected
I created a reproduction of this issue which illustrates the four key scenarios of the <Progress /> component.

[!NOTE]
You can check out the live reproduction and code here:
https://shadcn-progress-max-repro.vercel.app/ | https://github.com/troypoulter/shadcn-progress-max-repro

It previously did not take into account the max value when updating the ProgressPrimitive.Indicator, which it now does and uses Math.min() to account for scenarios where the value is greater than the max.
the primitive attributes [data-value] and [data-max] were not set
The value and max attributes were not sent through, which we now do to be more compatible with the underlying primitive.

[!NOTE]
Though this PR is similar to #475 and #2088, I still raised it as this PR also: adds an example to the documentation, updates the three locations of the component, updates the registry and solves this in a slightly different way with evidence it works for the four key scenarios.

me and my team ran into this issue yesterday.  did some searching and this pr came up.  it aligns with the fix that we ended up doing on our end to get this to work how it should.  namely, adding the value prop to the wrapper.
this pr should be merged in

Ran into these exact issues with the Progress component today. Thanks for this PR. It fixed all the things I ran into.


Recursion impossible in freeCodeCamp console

Bonfire: Sum All Odd Fibonacci Numbers
I make a litle recursion to get the Fibonacci numbers and the console is blocked
In repl.it it''s run perfectly. What is the problem?
Here is the code.
function sumFibs(num) {
  if (num === 1 || num === 0) {return 1;}
  else {return sumFibs(num - 2) + sumFibs(num - 1);}
}
sumFibs(4);

I'll copy my response from the other thread:

@koouty I know recursion is a popular way to solve the fibonacci problem, but it's an incredible inefficient solution in Javascript. Function calls in javascript are VERY slow, so once you start doing test cases like FCC has (1000+) your recursive solution is orders of magnitude slower than using a while loop. Check out what happens when you toss your solution into a repl with some of the large number test cases and you'll see what I'm talking about.

You might be able to make the resursive solution work by implementing memoization, but you'd just be putting a bandage over the issue and it still probably won't be fast enough for the larger test cases.

Recursion is possible in the challenges, but if the solution is too slow it will look like an infinite loop.
Closing as not an issue.

@ltegman Thank's


Rialto 307 change owner and group of socket

Summary: Add settings which allow specifying owner and group of the rialto socket
Type: Feature
Test Plan: Ran unit tests. Loaded the image onto Sky0 box and changed the appsservice code to specify a different owner and group for the socket and the socket was created as expected. Also, a test was conducted with the standard appservice code (not specifying an owner and group... default empty strings) and the socket was created as expected (with the user and primary group running the process)
Dependencies and Impacts: Extra parameters for socket owner and group are added alongside the pre-existing socket permissions
Jira: RIALTO-307
To change the appsservice so that it specifies an owner/ group...
RDK/AppManager/RialtoServerManager/source/RialtoServerManager.cpp
@@ -59,6 +59,10 @@ RialtoServerManager::RialtoServerManager(const std::liststd::string &environme
firebolt::rialto::common::ServerManagerConfig config;
config.sessionServerEnvVars = addMandatoryEnvVars(environmentVars);
config.numOfPreloadedServers = numOfPreloadedServers;
+

config.sessionManagementSocketPermissions.owner = "app-500";
config.sessionManagementSocketPermissions.group = "apps";
mServerManagerService = rialto::servermanager::service::create(mStateObserver, config);
if (!mServerManagerService->registerLogHandler(mLogHandler))
{

https://jenkins.code.rdkcentral.com/job/build-component/80546/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/7955/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/41665/ started

https://jenkins.code.rdkcentral.com/job/build-component/80547/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/7956/ started

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29045/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29046/

https://jenkins.code.rdkcentral.com/job/github-triggered/41679/ started

https://jenkins.code.rdkcentral.com/job/build-component/80557/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/7967/ started

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29063/

'['RPI4 Hybrid']' Sanity test FAILURE, investigation pending.
Build Job/Num: build-rpi4-hybrid/10449
Flash rdkv_rpi4_hybrid [SUCCESS]
rdkv_rpi4_hybrid - Video Test [FAILURE]
Video Accelerator Home Screen Test [FAILURE]
Video Accelerator Home Screen Test [SUCCESS]
Video Accelerator VOD Video Detection Test [FAILURE]
HLS video streaming using gst-launch uri=aamp [SUCCESS]
HLS Video Streaming using aamp-cli [SUCCESS]
DASH Video Streaming using aamp-cli [SUCCESS]
Video local (H264) using gst-play-1.0 [FAILURE]
Video local (H264) using gst-play-1.0 [SUCCESS]
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-raspberrypi-rdkv-rpi4-flow/2392/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29081/

@skyproudmanp generally its going to be difficult to unittests these linux functions, maybe we should create a new wrapper in media/server/wrappers for these and then we could mock them properly. Can do that as a separate PR if you'd like, rather than lump it in here.

You don't have to create new wrapper - we already have one in common directory -> rialto/common/interface/ILinuxWrapper.h

https://jenkins.code.rdkcentral.com/job/github-triggered/41861/ started

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rialto/201/rdkcentral/rialto

Commit: e00fa37f1d7379f8d4aef8641cf1a3455bbdbd39

Report detail: gist

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/8095/ started

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29331/

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Not a problem - similar API usage but not a real code match

Commit: e00fa37f1d7379f8d4aef8641cf1a3455bbdbd39

https://jenkins.code.rdkcentral.com/job/github-triggered/41869/ started

https://jenkins.code.rdkcentral.com/job/build-component/80730/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/8100/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/42057/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/8253/ started

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29460/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/8286/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/42101/ started

https://jenkins.code.rdkcentral.com/job/build-component/80855/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/42106/ started

https://jenkins.code.rdkcentral.com/job/build-component/80870/ started.

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29527/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/8301/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/42120/ started

https://jenkins.code.rdkcentral.com/job/build-component/80900/ started.

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29582/

'['RPI4 Hybrid']' Sanity test FAILURE, investigation pending.
Build Job/Num: build-rpi4-hybrid/10669
Flash rdkv_rpi4_hybrid [SUCCESS]
rdkv_rpi4_hybrid - Video Test skipped [FAILURE]
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-raspberrypi-rdkv-rpi4-flow/2602/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/29793/


I cant use "Alt+Space" to open the Powertoys Run.

PowerToys version: v0.19.1
When I pressed "Alt+Space", the search box didn't open correctly, even if I restart it.
I tried to modify the Hotkeys, but, as always, it doesnt work.

Same problem as #4922

At some app, for example, Notepad++(in admin mode), Alt+Space opens a menu that appears if LeftClick at the app icon at the top-left corner.

hi @DonPangPang sorry about this. We have discussion regarding this issue here: #4798 and  #4826, Please follow their for any temporary solutions and updates on trying to get this fixed. If you're not running as admin: #4938 should cover it
closing as duplicate


feat(scoped-elements): add scoped elements feature

This feature allows to define custom elements, scoping them when necessary, allowing to use different versions of the same component without naming collisions.
import { css, LitElement } from 'lit-element';
import { createScopedHtml } from '@open-wc/scoped-elements';
import MyButton from './MyButton.js';
import MyPanel from './MyPanel.js';

const html = createScopedHtml({
  'my-button': MyButton,
  'my-panel': MyPanel,
});

export default class MyElement extends LitElement {
  static get styles() {
    return css`
      .my-panel {
        padding: 10px;
        background-color: grey;
      }
    `;
  }

  static get properties() {
    return {
      text: String,
    };
  }

  render() {
    return html`
      <my-panel class="my-panel">
        <my-button>${this.text}</my-button>
      </my-panel}>
    `;
  }
}

@thepassle thanks for your comments

Cool idea to support scoped elements in this way. 👍 😎
Some things that popped up when looking into the PR:

I don't think styles are matching when unique tags are generated. You could have a custom css function, though.
Do we want to support unsafeHTML directive within the scoped html function? For now, it will just fallback to the regular tag creation.
Is it clear (enough) for the user that that some regular functionality will be behaving unusually? eg. using querySelector. Most of this functionality is of course related to the unique tag name, but still.

And maybe a more general , open, question: is the pattern of using scoped elements to enable multiple versions of a component something that should be used in / promoted for live web sites? 🙈  I think it makes sense in controlled environments like test cases where one need to test multiple extensions of a component.
But in live web sites, it's often a slippery slope:

inconsistent design and behavior within the page / app
pages / apps become larger / slower as they need to load x amount of code
Also, the usage of multiple versions components can be tricky (eg. when using packages to release the components, only some package manager can handle multi versions (ie. npm, and not bower))

What do you guys think? 😊

Hello @tmsns,

I don't think styles are matching when unique tags are generated. You could have a custom css function, though.

Yes, you are right. It's possible to create a custom css function also. However we thought have out of scope the css, but we could add it later in another iteration. By the moment classnames could be used instead tagnames in order to style the child components in case you need it.

Do we want to support unsafeHTML directive within the scoped html function? For now, it will just fallback to the regular tag creation.

I'm not sure if I'm understanding your question. If your question is how to use a scoped tag inside the unsafeHTML you can use the getTags function.
import { getTags } from '@open-wc/scoped-elements';
import { unsafeHTML } from 'lit-html/directives/unsafe-html.js';
import { MyButton }

const [myButtonTag] = getTags([MyButton]);

const markup = `<${myButtonTag}>Some button to click.</${myButtonTag}>`;
const template = html`
  Look out, potentially unsafe HTML ahead:
  ${unsafeHTML(markup)}
`;

I'm not sure if I answer your question.

Is it clear (enough) for the user that that some regular functionality will be behaving unusually? eg. using querySelector. Most of this functionality is of course related to the unique tag name, but still.

If you want to use querySelector you must use the previous getTags function inside your component, but the doc must be clear about this and I forget to add it. Thanks!

is the pattern of using scoped elements to enable multiple versions of a component something that should be used in / promoted for live web sites? 🙈 I think it makes sense in controlled environments like test cases where one need to test multiple extensions of a component.
But in live web sites, it's often a slippery slope:

inconsistent design and behavior within the page / app
pages / apps become larger / slower as they need to load x amount of code
Also, the usage of multiple versions components can be tricky (eg. when using packages to release the components, only some package manager can handle multi versions (ie. npm, and not bower))

This feature is going to be used mostly in corporate applications in which several teams (that probably are also distributed across several locations) are involved in the development. In this scenario, when there are shared component libraries that must be used by the teams (for example to maintain a corporate brand), when those libraries are updated not all teams have the capability to update their code at the same velocity, blocking the shipping of new features until all teams update their code to the same components library version. This feature allows the shipping of new features while eventually different versions of the same component are used.
I hope I have clarified your doubts, and thank you for your comments :)

Yes, you are right. It's possible to create a custom css function also. However we thought to have out of scope the css part by the moment, but we could add it later in another iteration. By the moment classnames could be used instead tagnames in order to style the child components in case you need it.

True, and by reading your comment I rechecked the readme.md file. I did not catch the . in the current css example.  🙈

I'm not sure if I'm understanding your question. If your question is how to use a scoped tag inside the unsafeHTML you can use the getTags function.
import { getTags } from '@open-wc/scoped-elements';
import { unsafeHTML } from 'lit-html/directives/unsafe-html.js';
import { MyButton }

const [myButtonTag] = getTags([MyButton]);

const markup = `<${myButtonTag}>Some button to click.</${myButtonTag}>`;
const template = html`
  Look out, potentially unsafe HTML ahead:
  ${unsafeHTML(markup)}
`;

That will indeed work.
I was thinking more in the line of creating a drop-in for unsafeHTML as well. So basically:
import {unsafeHTML} from<EMAIL_ADDRESS>
In the end, it's the same remark as the css function.

If you want to use querySelector you must use the previous getTags function inside your component, but the doc must be clear about this and I forget to add it. Thanks!

In the end, as you are probably well aware, any usage of the regular tag should be done in a different way. This might be a deal-breaker for some users. But as long as they get a fair warning upfront, I think we should be good. 😃

This feature is going to be used mostly in corporate applications in which several teams (that probably are also distributed across several locations) are involved in the development. In this scenario, when there are shared component libraries that must be used by the teams (for example to maintain a corporate brand), when those libraries are updated not all teams have the capability to update their code at the same velocity, blocking the shipping of new features until all teams update their code to the same components library version. This feature allows the shipping of new features while eventually different versions of the same component are used.

I see what the use case is. But, they need to be very careful with this type of usage. I see a number of pitfalls:

sub components
When a component will use this technique, it will indeed allow multiple versions of the same component to be on the page. However, any sub component it uses should also use the same strategy. eg:
<my-component-v1>
#shadowRoot
  <my-subcomponent></my-subcomponent> // v3
</my-component-v1>
<my-component-v2>
#shadowRoot
  <my-subcomponent></my-subcomponent> //v4
</my-component-v2>

In this example, you can run into trouble if v2 has been created with another (breaking) version (v4) of my-subcomponent, it will be broken as v3 was loaded. This means to make the scoping work consistently, any used component within should use the same pattern.
Very often, this is a problem, as companies tend to reuse other libraries as well (ie. lion), that don't use this feature.

developer experience
This is related to the previous points. For regular developers, they will need to take great care to always use the scoped element export, and to always map the tag name before using it. Next to the fact that this is "some" burdon, sometimes it will be impossible to use when using external libraries.

inconsistent design and larger size of sites
This is what I mentioned before. In the company I work for we had the exact same problem. There was a UI library. But every team started to use a different version. They didn't have the time nor the budget to keep up to date, which resulted in a degraded experience for their customers.
Web components force their users to use the same component, and to stay in line. With a good CI/CD, it is possible to have a great velocity.  💪
Btw, if the size get so large of the bundle, there is actually a web feature that allows multiple multi versioning of shared components per page, and that is to leave the SPA and go for an actual page reload. 🙈 This is also what was used in the company. But the inconsistent design and larger size became a problem.

Now, don't get me wrong. I love the idea! ❤
I have been following a similar initiative by @justinfagnani: https://github.com/w3c/webcomponents/issues/716, where point 1 and point 2 will be tackled by. I think scoping can be very useful. I'm using a similar approach when unit testing mixins. I'm just not sure that it is a silver bullet in every case, because of point 3. So I think it's recommendation should be done very carefully.

I hope I have clarified your doubts, and thank you for your comments :)

Yup, thanks for taking the time! 😃

@tmsns you are absolutely right.
IMHO I think the problem with custom elements is that it is difficult to follow a micro frontend strategy with them. Most JS frameworks give you more flexibility, although you must be careful with the size of the application and the inconsistent design. My point here is that these things should be handled by the product strategy, and tools like a good CI / CD help you a lot as you mentioned. However, the difference is that you have the flexibility of having possible inconsistencies in the design or size of the application favoring the delivery of value to the user.
I also believe that the auto definition is wrong. A CE shouldn't tell you which tag you have to use. You should be able to import a CE and decide how to use it in your component. Otherwise, you may even have problems if two different libraries decide to use the same prefix for their components.


add missing docs (external scheduler and plugin extender)

What type of PR is this?
/area simulator
/kind documentation

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #285
Special notes for your reviewer:
Some parts are using the implementation of https://github.com/kubernetes-sigs/kube-scheduler-simulator/pull/288. 🙏
/priority next-release
/label tide/merge-method-squash

Ignore the test failing, that's because of:

Some parts are using the implementation of https://github.com/kubernetes-sigs/kube-scheduler-simulator/pull/288. 🙏

/cc @196Ikuchil

/hold
https://github.com/kubernetes-sigs/kube-scheduler-simulator/pull/288 should be merged first.

@196Ikuchil fixed 🙏

rebased.
/unhold

@196Ikuchil done 🙏


Correction for raw affiliation CIRAD-BIOS, UMR AGAP, PO Box 946, Port-Vila, Vanuatu, South Pacific

Correction needed for raw affiliation CIRAD-BIOS, UMR AGAP, PO Box 946, Port-Vila, Vanuatu, South Pacific
raw_affiliation_name: CIRAD-BIOS, UMR AGAP, PO Box 946, Port-Vila, Vanuatu, South Pacific
new_rors: 057k8nt63;02w4exq36
previous_rors: 057k8nt63
works_examples: W2116527458
contact: 0f885ace455b9888377d14221ea4683b:746a838aa290e017c79e17f2d18be15ac32f @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


Hello C# - Migrating the codebase from C++/CX to C# (Phase 1)

Hello C# - Migrating the codebase from C++/CX to C# (Phase 1)
This PR is Related to issue #893
Description of the changes:

Migrates the View layer from C++/CX to C#
Changes CalcViewModel into a Windows Runtime Component library
Adjusts pipelines and project configuration for the C# codebase

How changes were validated:

Manual Tests Results for Branch feature/UICSharpCalculator

Test State: Completed
Test Health: Green
Platforms(s): Desktop VM & Desktop & Surface Pro 7+ & HP laptop
OS Build(s): RS4 (17134.1.180410-1804), RS5 (17763.1.180914-1434), 21H1(19043.1052), VB (19041.1.191206-1406), Co (22000.1)
App Build: Calculator -Release 0.2106.1402.0

Hi everyone,
It's so glad to proposing this PR to bring the UI part of Calculator codebase to C# world.
Though this PR is changing too many files, we've set up a feature branch(feature/UICSharpCalculator) to track and break down changes. So, for each important change, there should be a corresponding PR that helps you focus on key parts while reviewing the whole picture.
Feel free to leave your comments here.
Appreciate it.

DesignData has not been converted?

DesignData has not been converted to C#?

The DesignData files are not being included as translation units. Keeping it as what it is doesn't block the progress of C# migration.


Create .uid file when creating new Resource

Remaining part of
https://github.com/godotengine/godot/pull/97352#issuecomment-2471455519

Thanks!


How to limit YouTube Music search to tracks only?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.01.06 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Is there a way to return only tracks in a YTmusic search? Sometimes music videos have sound effects, while I'm only interested in the original song.
I'm using this command:
yt-dlp -f bestaudio --playlist-items 1 --default-search "https://music.youtube.com/search?q=" -a list-of-tracks.txt
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Give example query, explaining what you are getting, what you expect to get and how you'd do this in website

This query returns the top result, which is the music video of the song:
yt-dlp -f bestaudio --playlist-items 1 --default-search "https://music.youtube.com/search?q=" "Anna of the North - I Do You"
The video version (which is downloaded by the query above) starts with bird sounds and an alarm clock, while the single/album track doesn't have those effects.
On the YouTube Music search page it's possible to select only tracks. I'm wondering if this can be done by yt-dlp as well.

while the single/album track doesn't have those effects.

This link gives me "Video unavailable", but I think I understand what u want

You want the equivalent of clicking this, right?
If so, use https://music.youtube.com/search?q=Anna+of+the+North+-+I+Do+You#songs as the URL to yt-dlp. The known fragments (#) are https://github.com/yt-dlp/yt-dlp/blob/87ebab0615b1bf9b14b478b055e7059d630b4833/yt_dlp/extractor/youtube.py#L6676-L6683
Any other filter can also be used if you find it's params from the devtools. E.g. https://github.com/yt-dlp/yt-dlp/blob/87ebab0615b1bf9b14b478b055e7059d630b4833/yt_dlp/extractor/youtube.py#L6659

That's very helpful, thanks!
One more question: I usually use this with a bunch of tracks in a text file (as in my original question). Is there a way to add the #songs fragment to that query? If not, then I can just add that to every line in the text file.

No. You will need to add it to each query

That's no problem. Thanks again!

If so, use music.youtube.com/search?q=Anna+of+the+North+-+I+Do+You#songs as the URL to yt-dlp (Note the #songs)

This also fixed an issue I ran into where the simple search returned a weird version of the track I actually tried to download, because it seemingly skipped the "Top result" section.
So for the example query https://music.youtube.com/search?q=Weekend+-+Schatz%2C+du+Arschloch it chose a track called "Schnaps, du Arschloch! (MFDDNSODUWDDNU EP)" instead of "Schatz, du Arschloch!"

Appending #songs chose the expect one.


Suddenly pigz is no longer available in Ubuntu-20.04

Description
My worksflows suddently started failing 2 days ago because pigz could not be found anymore. This tool has been available in the ubuntu-20.04 image, but seems to have been removed without notice. Is the removal expected? If so, why? Is it possible to re-add it?
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[X] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
It's not a public repo
Is it regression?
It's not a public repo
Expected behavior
pigz should be installed in the image
Actual behavior
pigz is not installed in the image
Repro steps
Try to execute pigz in Ubuntu 20.04, it should be available.

Hello @Luc45, we will take a look, however temporarily you can switch to gzip.

The "pigz" was never in our software list, however, it was installed as a recommended package along with docker. Recently we have changed the docker installation source and stopped installing "pigz".
If you would like to use it you can install it in runtime, using a snippet like this
- name: check issue
        id: test_out
        shell: bash
        run: |
          apt-get -yq update
          apt-get install -y pigz

If you have any questions, feel free to reach us.

Thanks, that makes sense now. I'll adapt my workflows.

I opened:

https://github.com/actions/runner-images/issues/8205

to try to get this installed by default as in our workflows the step to install pigz adds 2 mins which seems excessively long given it's only a 259kb install...


Patch 6.5 - ConfigOption enum off by one

Hi, you're pobably still working on updating FFXIVClientStructs but just wanted to let you know this enum seems to be off by one now.
https://github.com/aers/FFXIVClientStructs/blob/8d9cbaea5db885e13a7a0f9ff43761c2a492b231/FFXIVClientStructs/FFXIV/Client/UI/Misc/ConfigOption.cs#L197
My plugin code which is now hitting BGM instead of master.
        /// <summary>
        /// Mutes/Unmutes the sound
        /// </summary>
        /// <param name="enabled"></param>
        public static unsafe void SetMasterSoundEnable(bool enabled)
        {
            Framework.Instance()->SystemConfig.CommonSystemConfig.ConfigBase.ConfigEntry[(int)ConfigOption.IsSndMaster].SetValueUInt((uint)(enabled ? 0 : 1));
        }

        public static unsafe bool GetMasterSoundEnable()
        {
            if (Framework.Instance()->SystemConfig.CommonSystemConfig.ConfigBase.ConfigEntry[(int)ConfigOption.IsSndMaster].Value.UInt == 0)
                PluginLog.Debug("Enabled");
            return (Framework.Instance()->SystemConfig.CommonSystemConfig.ConfigBase.ConfigEntry[(int)ConfigOption.IsSndMaster].Value.UInt == 0);
        }

Heya,
I pinged Cara, which has a script to update this (if they can find it 😁).
Assuming you're using Dalamud, I also wanted to tell you that there is a GameConfig service, which is safer to use.
[PluginService] public static IGameConfig GameConfig { get; private set; }

With that, you can use it like this:
public static unsafe void SetMasterSoundEnable(bool enabled)
{
    Service.GameConfig.Set(SystemConfigOption.IsSndBgm, enabled);
}

public static unsafe bool GetMasterSoundEnable()
{
    return Service.GameConfig.TryGet(SystemConfigOption.IsSndBgm, out bool isSndBgm) && isSndBgm;
}

These enums are safer to use, since GameConfig looks up values by name instead of index (as seen by their attribute).
Alternatively, you can use their names directly:
public static unsafe void SetMasterSoundEnable(bool enabled)
{
    Service.GameConfig.System.Set("IsSndBgm", enabled);
}

public static unsafe bool GetMasterSoundEnable()
{
    return Service.GameConfig.System.TryGet("IsSndBgm", out bool isSndBgm) && isSndBgm;
}

Hope this helps! 🙂

@Haselnussbomber That is actually super helpful, thanks so much! I imported the GameConfig service and migrated over to it and everything seems to work exactly how I want now toggling the master volume on and off instead of setting values 0 & 1.
        public static void SetMasterSoundEnable(bool enabled)
        {
            Api.GameConfig?.Set(SystemConfigOption.IsSndMaster, !enabled);
        }

        public static bool GetMasterSoundEnable()
        {
            return Api.GameConfig != null && Api.GameConfig.TryGet(SystemConfigOption.IsSndMaster, out bool isSndMaster) && isSndMaster;
        }

I'll leave this ticket open so the enum could be fixed up (or decided by maintained) but super happy with GameConfig existing!


Sql test record2022.05

Completed Sql Test re-record

API change check
API changes are not detected in this pull request.

@fengzhou-msft I made some changes to this part of the Sql test, on the one hand, because the sql test takes too much time to run, so I implemented playback passing by modifying the json file in SessionRecords, and on the other hand, in the SqlManagementClientBase .cs，The original code

caused an error to be reported，because

So I changed SourceType="Microsoft.Sql" to SourceType="/Microsoft.Sql" to ensure that the test case passed, but the type in the json file also changed

I wasn't sure if there was a more appropriate way to modify it, so I left this comment


Project monitoring still scraping the metrics of a pod which has been moved to another project

rancher/rancher:v2.2.4-rc15
Steps:

Enable project monitoring in Project A
Add a workload with custom metrics exposed
Check the project monitoring targets to make sure it is configured correctly
Move the workload to another  Project B
Check the targets in the  Project A

Result:
The pod target still shows up for around 10+ mins sometimes
Expected:
It should not take that long time to update the prometheus config.
Reason:
There will be a delay between the new target is added into the prometheus configuration and the prometheus begins to scrape the metrics from the metrics. It is because the prometheus configuration configmap is mounted as file in prometheus pod. There will be a delay that the mounted file updates after the configmap is updated. From Kubernetes document:
https://kubernetes.io/docs/tasks/configure-pod-container/configure-pod-configmap/#mounted-configmaps-are-updated-automatically

This is solved in project monitoring v2.


chore(tools): use npm files directive for smaller publish size

npm publish --dry-run before this change
npm notice === Tarball Details ===
npm notice name:          snabbdom
npm notice version:       3.5.1
npm notice filename:      snabbdom-3.5.1.tgz
npm notice package size:  89.8 kB
npm notice unpacked size: 387.6 kB
npm notice shasum:        e1e33224b1397bc6633a3cff1c8af718da612c01
npm notice integrity:     sha512-FDUepV7ay0KHD[...]W2N0zF8lSXffg==
npm notice total files:   116

npm publish --dry-run after this change
npm notice === Tarball Details ===
npm notice name:          snabbdom
npm notice version:       3.5.1
npm notice filename:      snabbdom-3.5.1.tgz
npm notice package size:  32.8 kB
npm notice unpacked size: 125.3 kB
npm notice shasum:        bf8c7416b440f50635dcd435eb8befccfae4344f
npm notice integrity:     sha512-wxR/BsS+6FphQ[...]+ve0bRMPqh5oQ==
npm notice total files:   57

Two more changes that would be great for reducing the final package size,

use npm's script.prepublishOnly directive to remove un-needed parts of package.json. For example, "prepublishOnly": "npm pkg delete scripts devDependencies commithelper release-it" would reduce package.json from ~2.7kB to ~1.3kB,
reduce the size of the main README the current snabbdom README is ~31.9kB. For comparison, the react-dom README size is ~1.1kB

Thanks, this is a great change! There's no need to include all the other files in the package.
Using a prepublishOnly script to delete parts of the package.json seems fine, but also like a small gain.
I'm skeptical about changing the readme file, I think that file should be optimized for its use as documentation.

If you merge this... the npm-published package will no longer have this path referenced in the README <img alt="Snabbdom" src="readme-title.svg" width="356px">
oh you merged

To address the missing image you could: add it to the files directive, you could change the image reference to point specifically to the github location or (recommended) you could upload the image to an image hosting service and use the url from that.

I'll submit a PR to reference the image from an image host and if you'd lke to include the image in some other way instead, that's fine :)


Fix broken test case test_resource_evolution

Description
The test_resource_evolution test case broke due to the update to the std module that sets the purge_on_delete flag to false by default. The branches merge-tool/2959/iso4 and issue/2958-set-purge-on-delete-to-false-iso3 which should merge #2959 into the ISO4 and ISO3 respectively are also stuck on this issue. I will apply this change on those branches as well.
Self Check:

[ ] Attached issue to pull request
[ ] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches master in 10a56e630182361ad31e838e900d88c18a609aec


Timezone issue in batch edit date picker

The batch edit datepicker seems confused when in the Europe/Paris timezone. Selecting a Thursday in the calendar auto-selects a Wednesday in the select field and that's the one that's applied when saving changes.

Reassigning to @Coenego for review.

Also added a fix for #448 to this PR as both are related. Over to @Coenego for review.

Back to @bp323 for follow-up

Coverage remained the same at 100.0% when pulling 11acf2e4bca7ad4829f8d66e3539c1c55fd5e262 on bp323:issue-447 into 4b0643c6bbe06cc634273c0c4f04a62dec52b390 on CUL-DigitalServices:master.

Coverage remained the same at 100.0% when pulling 579e4916dd2f92c755284cc91f6ef47a6c246d19 on bp323:issue-447 into 4b0643c6bbe06cc634273c0c4f04a62dec52b390 on CUL-DigitalServices:master.

Back to @Coenego


Feature: FuzzySearch

It would be great if there would be an option to support fuzzy matching. Maybe this library by @bevacqua will help: https://github.com/bevacqua/fuzzysearch

@marcobiedermann you can implement this in awesomplete's current form using the filter option

I think the question was about providing support out of the box, which we shouldn't probably do despite https://github.com/bevacqua/fuzzysearch being excellent.
As @Secretmapper mentioned we can use filter. Sadly the arguments order in filter and fuzzysearch is reversed, so:
filter: function (text, input) {
	return fuzzysearch(input, text);
}

Would it be possible to post a demo link with the filter code and fuzzysearch?

What is the case against providing it out of the box? Just a matter of avoiding a dependency?

Also, what does fuzzysearch do that isn't already being done in awesomplete?

At this point the last commit in fuzzysearch was in 2016, so even if this becomes baked in, perhaps another library would be more suitable? Though I do think it's better suited as a(n official) plugin.


-help description of few s_client and s_server options (master)

add -help description of sigalgs, client_sigalgs, curves
and named_curve

closed, thanks!


Look into adding an option to run the gdal_contour tool when uploading raster data

The gdal_contour tool can convert raster data into vectors (lines or polygons) this makes them more compatible with GeoSPARQL amongst other things.

If contours are only required for visualisation purposes it looks like GeoServer supports generating contours from rasters through styling as described here https://docs.geoserver.org/main/en/user/styling/sld/extensions/rendering-transform.html


Computercraft APIs for new Multiblocks

Since the new induction matrix can store more then 2^31 RF it causes trouble with the 32 bit behavior of ComputerCraft. OpenPeripherals easily overflows if getEnergyStored() / getMaxEnergyStored() is called in this case. .
Can we have some methods which can provide the energy stored / max energy stored (maybe, returning a table with a value + factor) ?

Computronics adds special handlers for both mekanism V8 as well as Draconic Evolution's large power storage.  I'd recommend adding that, as it works for both OpenComputers and ComputerCraft.

As you stated, getEnergyStored() and getMaxEnergyStored() are part of Open Peripheral, which has nothing to do with Mekanism nor ComputerCraft. Since those methods are for the RF api, I doubt they will ever return a double but it may be a good idea to ask the author of Open Peripherals.
I implemented those 2 functions on my induction matrix (as well as an average output) and it works perfectly fine (ComputerCraft handle doubles natively). I can send a pull request if the Meka team wants it.

I'd rather not do any kind of additional integration along this path.  I'd be happy to accept a feature PR though :)


'--package-root' doesn't work on 2.1.0-dev.6.0

My case is to deploy VM application, including all its dependencies, to the server. To run it I've set '--package-root' and point to the packages directory that is placed in the same folder as my app.
The result is:
Error: Could not resolve the package 'di' in 'package:di/di.dart'.
(and more lines below)......
File .packages is removed.
The workaround is to generate .packages and specify FULL paths (relative paths don't work too).

We removed support for package directories entirely. The workaround is the solution – although it sounds like full paths are an issue when you deploy?

Hmm... Relative paths work locally and don't work on the server. So, the problem on our side! We'll continue investigation...
Thank you guys!


Implement /oauth/revoke

It should work the same as it does on Client API.

I don't want to seem rude but is there an ETA on this being merged?
I am trying to use a LogoutLink from stormpath-react but it's calling /oauth/revoke which is currently 404ing.
Happy to use a workaround if there is one?
Thanks!

Hi guys, any news about this? Workaround?
Thanks


Bad sublibrary gets seen as "missing codegen plugin"

https://github.com/dotansimha/graphql-code-generator/blob/e8bc671a0f4145e9a24436c86ab0d24f94e91725/packages/graphql-codegen-cli/src/plugins.ts#L21-L54
The above code isn't a sufficient try-catch for what I just experienced. In my case, having freshly installed and inited everything I was encountering a message saying libraries that I did have installed weren't installed, and I should try the list.
After digging with a debugger into the above lines I found that while pluginLoader() did return an error while looking for a plugin, it didn't match the condition err.code !== 'MODULE_NOT_FOUND', which resulted in the above mentioned warning message logic.
However in reality, the error that was occurring was because an underlying library wasn't the right version and had shipped with a bug! I never got to see this message because the catch is too strict.

I know I deleted the template and that's a dick thing to do (I maintain a few large libraries myself), but this situation is so unique I'm willing to come back later with more information. It's just that it's 2am and the barrier to entry should be low for a issue.

I also believe this is why #3275 happens.

@krainboltgreene I recently ran into a similar issue using @graphql/codegen-typescript. The plugin loader was throwing an error with the code MODULE_NOT_FOUND, but when you do a little logging you can see that the real error is Error: Cannot find module '@babel/runtime/helpers/objectSpread2'. My temporary solution was to install @babel/runtime as a dev dependency, however if I get some time this week I'll put together a PR.

That is the exact thing I'm describing, thanks for bringing receipts!

Sent a PR to add @babel/runtime @patrickocoffeyo  #6716

Awesome @carlosbaraza! Thanks for putting that together :raised_hands:


Fixed : Spelling  memeory to memory

Sorry I send pull request from main branch, I am closing this pull request


[Music] HUD message missing icons

Extension
https://www.raycast.com/fedevitaledev/music
Description
SFSymbols still missing definition, may be just remove it?

Steps To Reproduce

Execute commands such as play, pause, etc.
See HUD information

Current Behaviour
No response
Expected Behaviour
No response

Do you have SFSymbols Installed?

Do you have SFSymbols Installed?

Thanks，I thought the system was installed by default😅


adds additional reason for reporting an account profile

After spelunking in the code a bit, this seems like an existing solution to the app UI that allows people to submit offensive accounts ( handles, display names, etc ) for moderation.
This does not handle the separate issue of when users initially create a seemingly inoffensive account and then change it later, etc. This just enshrines offensive account-specific language as a reason for reporting an account.

Thank you! Had to check in with backend to make sure this works properly, and it does. I also tweaked the copy a bit.


Improve signalling for JSON payloads

For the combination JSON logging and JSON request/response payloads, make it so that processing details relevant for logging are propagated, so that duplicate work can be avoided.
Detailed Description
Currently JSON payloads are append as raw content within JSON request/response-logging statements. This means that payloads with invalid JSON syntax can / will result in log statements with invalid JSON syntax too. So log accumulation tools will not be able to parse the statement as JSON, and might discard the message, or fall back to handle the log statement as pure text.
However existing body-filters and/or databinding performed in the service may be leveraged to guarantee that the payloads are in fact valid JSON. The logging subsystem can safely append the raw data if it knows that the body

does not contain newlines
is well-formed JSON

If those parameters are unknown, the logging subsystem can attempt to validate the JSON syntax itself and remove newlines. Then continuing to append the payload as raw content, or escaping the payload as JSON text if invalid JSON syntax.
Context
A JSON message might be parsed up to three times,

in a body filter
in the service databinding
when writing the log

The last might be avoided.
Possible Implementation
Add a wrapper for body. This would be a breaking change.

Still relevant, might also be interesting to add some metadata indicating if/how a payload was filtered, i.e. "4500 bytes was filtered" etc.
See also https://github.com/zalando/logbook/issues/1330


Add a refresh indicator on the product page

Hi everyone!
In order to be less frustrating for users, I like to show a loading indicator on the product page when changes are being sent.
@monsieurtanuki Is there a simple way to know if a product has background tasks? and also to be notified when they're done ? (to remove that indicator)

Fun fact: I somehow asked the same questions exactly 2 years ago, in #3263.

In order to be less frustrating for users

Just to understand: what exactly is frustrating?

Is there a simple way to know if a product has background tasks?

I would say something like bool UpToDateChanges.hasNotTerminatedOperations(String barcode) ("Returns true if some actions have not been terminated."). Limited to actions on a single product: OK for details changes, not OK for app language refresh.

also to be notified when they're done ? (to remove that indicator)

As long as you use UpToDateMixin you're notified.

Fun fact: I somehow asked the same questions exactly 2 years ago, in #3263.

In order to be less frustrating for users

Just to understand: what exactly is frustrating?

We have many complaints (app reviews and emails) from users wondering: "What is going on?"
"I don't understand why my changes are not visible"
"Are my changes lost?"
Displaying the info on the product page is just a small step.
The real one would be to add a loader to each piece of info that is being uploaded (similar to what's done with images).

Is there a simple way to know if a product has background tasks?

I would say something like bool UpToDateChanges.hasNotTerminatedOperations(String barcode) ("Returns true if some actions have not been terminated."). Limited to actions on a single product: OK for details changes, not OK for app language refresh.

Ok perfect 👌, tkx

also to be notified when they're done ? (to remove that indicator)

As long as you use UpToDateMixin you're notified.

For example on the nutrition screen, if you save your edits and then reopen the screen, it still has the old values

For example on the nutrition screen, if you save your edits and then reopen the screen, it still has the old values

It's not normal. I'll try that one.

It's weird because I'm pretty sure that on the prod app, I've had this problem recently.
But on both the prod and dev apps, I can't reproduce it.
Have you changed anything lately?

It's weird because I'm pretty sure that on the prod app, I've had this problem recently. But on both the prod and dev apps, I can't reproduce it.

Generally speaking, don't hesitate to create an issue if you see a bug.

Have you changed anything lately?

In the last 2 weeks I added the "locked" symbol on owner produced nutrition values, which may have:

lowered your expectations if the field was explicitly marked as "probably not editable"
fixed the issue as a side-effect when I slightly refactored the related code


unhandled promise

Description
PromiseNotCaughtGeneral: No catch method for promise , This may result in an unhandled promise rejection.
Steps to reproduce

Screenshots
No response
Desktop (please complete the following information)
OS - fedora 38
Browser - firefox,chromium
Version - latest
Smartphone (please complete the following information)
No response
Additional information
I can fix the error can i go with it?
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Description
PromiseNotCaughtGeneral: No catch method for promise , This may result in an unhandled promise rejection.
Steps to reproduce

Screenshots
No response
Desktop (please complete the following information)
OS - fedora 38 Browser - firefox,chromium Version - latest
Smartphone (please complete the following information)
No response
Additional information
I can fix the error can i go with it?
Record

[x]  I agree to follow this project's Code of Conduct
[x]  I have checked the existing issues
[x]  I'm a GSSoC'23 contributor
[x]  I want to work on this issue

right, you can fix this!


Type declaration issue in TypeScript

It seems that generated TS types embeds contract address into type name, which is invalid in terms of syntax. Like this:
/** Struct representing the balance of each address. */
export type 0xcafe::BasicCoin::BalanceData = {
  coin: {
    value: p.U64;
  };
};

export type 0xcafe::BasicCoin::CoinData = {
  value: p.U64;
};

Generated from a small example here: https://github.com/move-language/move/tree/main/language/documentation/tutorial/step_4_sol/BasicCoin. It is a mere compilable coin module.

fixed


e.observable.object is undefined when using 0.57.0 on expo

I created a repo with the simplest way to reproduce the error: https://github.com/jeloagnasin/mobx-keystone-issue.
When I was using<EMAIL_ADDRESS>on expo, I receive the following error:
TypeError: undefined is not an object (evaluating 'e.observable.object')

This error is located at:
    in App (created by ExpoRoot)
    in ExpoRoot (at renderApplication.js:45)
    in RCTView (at View.js:34)
    in View (at AppContainer.js:106)
    in DevAppContainer (at AppContainer.js:121)
    in RCTView (at View.js:34)
    in View (at AppContainer.js:132)
    in AppContainer (at renderApplication.js:39)
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:104:6 in reportException
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:171:19 in handleException
at node_modules/react-native/Libraries/Core/ReactFiberErrorDialog.js:43:2 in showErrorDialog
at [native code]:null in commitRootImpl
at [native code]:null in performSyncWorkOnRoot
at node_modules/react-native/Libraries/ReactNative/renderApplication.js:54:4 in renderApplication
at node_modules/react-native/Libraries/ReactNative/AppRegistry.js:117:25 in runnables.appKey.run
at node_modules/react-native/Libraries/ReactNative/AppRegistry.js:213:4 in runApplication
at [native code]:null in callFunctionReturnFlushedQueue

If I downgrade to<EMAIL_ADDRESS>it works fine.
expo install<EMAIL_ADDRESS>

It seems to work for me with node 16 and the web version (at least I can see the ·Open up App.tsx to start working on your app!" in chrome without any errors in the console).
Where are you getting the error? An android device, ios...?

Found that it actually happened on android.
Please try 0.57.1


[IT-1046] Add required metadata

SAM requires name, description, and author metadata to exist,
and will refuse to publish the template if they don't.

you might want to just remove sam publish in .travis.yml instead

Ah, I was confused about when the function code was uploaded to S3, but that's done in sam package not in sam publish. I'll remove the latter since it isn't needed in our use case.


Support analytics (metadata and events) in the Tradeoff Analytics proxy

OK, @germanattanasio, I've added tests, worked in dev, aligned with the holy coding guidelines, the whole nine yards...
Please accept the pull request so we can get this over with...

Thanks @dolevdotan! the code looks pretty solid. Sorry for being too picky.
I will update the nodejs later today along with the demo


refactor: use JestMatcherUtils instead of jest.MatcherUtils

Describe the feature you'd like:
Use JestMatcherUtils instead of jest.MatcherUtils

:tada: This issue has been resolved in version 1.10.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


test some system commands

Description
Added tests to system commands view and add. Other commands need such as history and split need testing, will do so as those near completion.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[X] My code follows the style of this project
[X] I have performed a self-review of my own code
[X] My code is self documenting
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new major crashes/bugs
[X] Any new features/functionality have corresponding test cases added

Ok not sure what's going on with testing. Just gonna close this PR and figure this out later down the road.


Instance drawing

Instance Drawing
Once again, we need to ensure that the graph is scalable with data size. To reduce rendering time, we are using Performance Optimization so that the graph can be loaded faster.
Goals:

Reduce rendering time for graph
Objects outside the viewport will not be rendered

This helps reduce the number of drawing commands issued as it copies shapes which have already been rendered instead of drawing new shapes.

I don't think this is a controversial thing to implement instanced rendering and such so removed the RFC tag :^).

I believe this isn't very relevant atm (its already enabled by default now with g6?)


Updated git clone to correct repo url.

Description
Talk about the great work you've done!
What's included?

One
Two
Three

Test Steps

[ ] do this
[ ] then this
[ ] finally this

Screenshots or link to CodePen/Plunker/JSfiddle
I ran into an issue when I tried to clone with" git clone git@github.com:teradata/covalent-quickstart.git ". updated it to correct repo url.

Thx @alagiboy!


Contributing update README.md

Closing this as it's too much effort. We have decided for now to go with the simpler option of just manually creating and maintaining contributing files.


Need redirect url scheme of google fit

I want to redirect to google fir permission section as i want the user to redirect to google fit app from my app to enable or disable the permission to share the data.
Can you please share me the url scheme of this ?

does openFit() do what you're looking for? https://github.com/StasDoskalenko/react-native-google-fit?tab=readme-ov-file#other-methods


message typo

nextjs-sendgrid/pages/api/send-email.ts
messgae should be message

I have cloned the project but keep getting 404 error. Does anybody else run into this?


Revert "Added functionality for dynamic jar loading."

Reverts Ericsson/eiffel-remrem-publish#40 since accidentally merged without review

Changes Unknown when pulling e95011b70e1106481f420c216489ba4709a52593 on revert-40-dynamicJar into ** on master**.


508 compliance - fix the empty link in all the apps

(fix)the empty link in all the apps on the front page of center

Passed on Center Homepage.

Tested on FF and Chrome and did not hear any empty links


HWM problem

I have two applications:

a simple sender application which just sends messages (long strings) in a loop
a simple receiver which has a loop where it sleeps and then receives a message
I use PULL-PUSH and set the HWM to 1000.
In this case the memory of the sender application is constantly growing.

I use PyZMQ 14.7.0 and ZeroMQ 4.1.2. With PyZMQ 14.6.0, the error is not present.

Can you provide actual code? And more information about the test environment (Python version, OS, etc.)? What are zmq.__version__ and zmq.zmq_version() in both cases?

Sorry for not responding, I will provide more details soon.


Allow broker list to be passed to JDBC

Currently, JDBC driver fetches the brokers from the controller. However, the broker list returned by controller contains hostname and ports of brokers which might be unresolvable from the environment in which the driver is running.
To solve this, we can allow users to pass broker urls explicitly. This allow users to pass public urls or urls accessible from their VPCs.
Users need to take care though to specify the correct brokers for the tenant they are using to query. The driver doesn't implement any checks to take care of this.

Codecov Report

Merging #9037 (77804a5) into master (bb0505f) will decrease coverage by 3.21%.
The diff coverage is n/a.

:exclamation: Current head 77804a5 differs from pull request most recent head 8b7c59f. Consider uploading reports for the commit 8b7c59f to get more accurate results

@@             Coverage Diff              @@
##             master    #9037      +/-   ##
============================================
- Coverage     70.05%   66.83%   -3.22%
- Complexity     4733     4765      +32
============================================
  Files          1827     1368     -459
  Lines         96097    69817   -26280
  Branches      14360    10906    -3454
============================================
- Hits          67317    46661   -20656
+ Misses        24129    19871    -4258
+ Partials       4651     3285    -1366

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
66.83% <ø> (-0.01%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


ja: Make docs/concepts/services-networking/ingress.md follow v1.17 of the original text

This is a Feature Request

What would you like to be added
Update docs/concepts/services-networking/ingress.md to follow v1.17 of the original (Englist) text.
Why is this needed
content/ja/docs/concepts/services-networking/ingress.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.17-ja.2/content/ja/docs/concepts/services-networking/ingress.md
Original
https://github.com/kubernetes/website/blob/release-1.17/content/en/docs/concepts/services-networking/ingress.md
diff between translated and v1.17
https://gist.github.com/4771b16eebcdc3ecf7ebf7efce241d2b
Note
Currently, we use dev-1.17-ja.2 branch. Please open a PR targeting the branch.
ref
How To Contribute
翻訳スタイルガイド
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/assign


Add missing migration

Migrations are missing for:

bitbag_mollie_email_template (& translation)
abandoned_email on Order

Hi,
Is it possible to merge this PR ?


Temperature Decimal Places in Exported Data

The telemetry value for temperature is a floating point number, but it is rounded (based on the number of decimal places indicated) to an integer.  This is what gets displayed and also it is an integer value in the Continuously Exported Data file.  This loses precision, and fluctuation within a degree on a minute-by-minute basis is lost.  Is this correct, and if temperature is retained as a rounded, integer value, could this be changed to record a decimal value of 2 or more digits.  Example from decode of AVRO file; for some reason, the simulated value has plenty of decimal places but the real device temp is integer value:
Device ID: buq5ns   << device info; this is the SIMULATED device
Humidity: 68.99866421257508
Temperature 87.44558129760945
Device ID: f246e272-4bb7-4857-a5c3-eb8f492a62a1  << this is the ESP8266
Humidity: 36   << integer?
Temperature 21  << why no decimal shown??

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

ID: 8f98cd40-ed60-7ccb-f769-50834dbdb4ed
Version Independent ID: ce0b839b-f9b6-3ddf-e7f8-3c152b76da80
Content: Set up a device template in an Azure IoT Central application
Content Source: articles/iot-central/howto-set-up-template.md
Service: iot-central
GitHub Login: @viv-liu
Microsoft Alias: viviali

@smithGit
Thanks for your feedback! We will investigate and update as appropriate.

I am 95 % sure this is an error in IoT computation:  The specification of number of decimal places should be the number of decimal places in the RESULT, not the number of digits used for rounding to an integral number of degrees C.  Not preserving the fractional part of a degree C loses almost a full degree F of resolution, and any analysis of temperature trends needs the additional precision.  Could someone look into this (soon) and see if you would not agree?  I do not know about others, but a system that did not resolve values to fractional degrees would not be deemed adequate for analysis.  (I am sure the DHT22 sensor I'm using is working; 10 days ago I uploaded values to the Adafruit IO cloud with 20 second readings and the curves clearly showed small changes in temperature as the heat went on and off.)

Thank you for coming back and explain the steps through the resolution of the initial issue :). We will now close this issue (it will continue visible as it may help others with similar issues).
If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Thanks, António, for your response.  Sorry for the false alarm, although my ESP8266 must have something wrong with it.
I will be looking again at the Adafruit ESP8266 operation with IoT Central to see why values are being stored as integer degrees C, but it may be a few days before I tackle this.  I will let you know what I find if it looks informative for other users.  Have a Happy New Year!
Sam in Michigan (northern Michigan – we are at our vacation home here, which is what I will be monitoring with the Raspberry Pi and ESP8266)
From: António Sérgio Azevedo<EMAIL_ADDRESS>Sent: Monday, December 31, 2018 5:06 AM
To: MicrosoftDocs/azure-docs
Cc: Sam Smith; State change
Subject: Re: [MicrosoftDocs/azure-docs] Temperature Decimal Places in Exported Data (#21376)
Thank you for coming back and explain the steps through the resolution of the initial issue :). We will now close this issue (it will continue visible as it may help others with similar issues).
If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/21376#issuecomment-450627843 , or mute the thread https://github.com/notifications/unsubscribe-auth/AGGnqAMBO0Zb6eEMzHs2-AE5P5MEgXyBks5u-eF9gaJpZM4ZkHi1 .Image removed by sender.

Ahah:  Here is where the integer to rounding comes from:  It is in the Starterkit code for Remote Monitoring example, for the Adafruit ESP8266.  The Microsoft-copyrighted code segment is below - could someone look into this and revise to permit floating point values for temperature and (less important) humidity?  Otherwise, the example makes the Adafruit ESP8266 appear to have lower resolution than it does (it is a great microcontroller, and hooked to IoT Central directly, a very fine solution).  Thanks - Sam
#include "sample.h"
#include "AzureIoTHub.h"
#include "sdk/schemaserializer.h"
#include "sensor.h"
/* CODEFIRST_OK is the new name for IOT_AGENT_OK. The follow #ifndef helps during the name migration. Remove it when the migration ends. */

Temperature is defined as integer here:  (beginning line 31)
DECLARE_MODEL(Thermostat,
/* Event data (temperature, external temperature and humidity) */
WITH_DATA(int, Temperature),
WITH_DATA(.....
WITH_ACTION(SetHumidity, int, humidity)
);
temperature is rounded here, line 297
thermostat->Temperature = (Temp>600)?600:(int)round(Temp);

Thanks @smithGit for clarifying it. I believe this was the repo where you cloned your sample from?:
https://github.com/Azure-Samples/iot-hub-c-huzzah-getstartedkit
And this is the piece of code with the round(Temp): https://github.com/Azure-Samples/iot-hub-c-huzzah-getstartedkit/blob/master/remote_monitoring/remote_monitoring.c
The code is all open sourced and you can easily customize it, like removing the round(Temp) line. If you are having troubles doing it my suggestion is that you open a new issue on that repository where we will have the people who developed the sample helping with it.
Thank you!

Thanks again and sorry not to post to the original URL (by the time I felt there was a problem I must have changed to the inappropriate URL).
From: António Sérgio Azevedo<EMAIL_ADDRESS>Sent: Wednesday, January 2, 2019 9:59 AM
To: MicrosoftDocs/azure-docs
Cc: Sam Smith; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Temperature Decimal Places in Exported Data (#21376)
Thanks @smithGit https://github.com/smithGit  for clarifying it. I believe this was the repo where you cloned your sample from?:
https://github.com/Azure-Samples/iot-hub-c-huzzah-getstartedkit
And this is the piece of code with the round(Temp): https://github.com/Azure-Samples/iot-hub-c-huzzah-getstartedkit/blob/master/remote_monitoring/remote_monitoring.c
The code is all open sourced and you can easily customize it, like removing the round(Temp) line. If you are having troubles doing it my suggestion is that you open a new issue on that repository https://github.com/Azure-Samples/iot-hub-c-huzzah-getstartedkit/issues  where we will have the people who developed the sample helping with it.
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/21376#issuecomment-450884752 , or mute the thread https://github.com/notifications/unsubscribe-auth/AGGnqF5aE5yyojq8fkZ_zpYkUNhcX2l5ks5u_MkxgaJpZM4ZkHi1 .Image removed by sender.


Issues with census layer transparency setting

Transparency slider does not reflect change when layer (Reading Proficiency by District) is clicked on then off.  Resets census layer to original transparency setting?

It was hard coded to reset opacity to 0.8.  Now it is setting the opacity to the slider value.


Patterns: Right-size icons in browser; they're massive!

Fixed in https://github.com/rslifka/reckoner-app/commit/30a2f41c6a2fa61746b4c6cd9262a72f1fbcba92


BuildX error when using Ubuntu Focal Fossa 20.04 base image

#8 14.76 Created symlink /etc/systemd/system/multi-user.target.wants/systemd-resolved.service → /lib/systemd/system/systemd-resolved.service.
#8 14.77 ln: failed to create symbolic link '/etc/resolv.conf': Device or resource busy
#8 14.77 Created symlink /etc/systemd/system/multi-user.target.wants/ondemand.service → /lib/systemd/system/ondemand.service.
#8 14.77 Created symlink /etc/systemd/system/sysinit.target.wants/systemd-pstore.service → /lib/systemd/system/systemd-pstore.service.
#8 14.78 Initializing machine ID from random generator.
#8 15.02 Setting up networkoptix-metavms-mediaserver (4.3.0) ...
#8 15.09 Detected runtime type: docker.
#8 15.09 System has not been booted with systemd as init system (PID 1). Can't operate.
#8 15.09 Failed to connect to bus: Host is down
#8 15.09 System has not been booted with systemd as init system (PID 1). Can't operate.
#8 15.09 Failed to connect to bus: Host is down
#8 15.16 sysctl: setting key "kernel.core_pattern": Read-only file system
#8 15.17 dpkg: error processing package networkoptix-metavms-mediaserver (--configure):
#8 15.17  installed networkoptix-metavms-mediaserver package post-installation script subprocess returned error exit status 255
#8 15.17 Processing triggers for libc-bin (2.31-0ubuntu9.2) ...
#8 15.19 Errors were encountered while processing:
#8 15.19  networkoptix-metavms-mediaserver
#8 15.20 E: Sub-process /usr/bin/dpkg returned an error code (1)
#8 ERROR: executor failed running [/bin/sh -c apt-get update     && apt-get install --no-install-recommends --yes         ./vms_server.deb     && apt-get clean     && apt-get autoremove --purge     && rm -rf /var/lib/apt/lists/*     && rm -rf ./vms_server.deb]: exit code: 100
------
 > [5/7] RUN apt-get update     && apt-get install --no-install-recommends --yes         ./vms_server.deb     && apt-get clean     && apt-get autoremove --purge     && rm -rf /var/lib/apt/lists/*     && rm -rf ./vms_server.deb:
------
Dockerfile:62
--------------------
  61 |     # Install the mediaserver
  62 | >>> RUN apt-get update \
  63 | >>>     && apt-get install --no-install-recommends --yes \
  64 | >>>         ./vms_server.deb \
  65 | >>>     && apt-get clean \
  66 | >>>     && apt-get autoremove --purge \
  67 | >>>     && rm -rf /var/lib/apt/lists/* \
  68 | >>>     && rm -rf ./vms_server.deb
  69 |
--------------------
error: failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/sh -c apt-get update     && apt-get install --no-install-recommends --yes         ./vms_server.deb     && apt-get clean     && apt-get autoremove --purge     && rm -rf /var/lib/apt/lists/*     && rm -rf ./vms_server.deb]: exit code: 100
Error: buildx call failed with: error: failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/sh -c apt-get update     && apt-get install --no-install-recommends --yes         ./vms_server.deb     && apt-get clean     && apt-get autoremove --purge     && rm -rf /var/lib/apt/lists/*     && rm -rf ./vms_server.deb]: exit code: 100

Error appears to be in NxMeta 4.3 Beta.
Reverting to Bionic / 18.04 resolves the error.
Switch to Focal / 20.04 when upstream error is resolved.

https://support.networkoptix.com/hc/en-us/community/posts/1500000851801-4-3-R1-fails-to-install-on-Ubuntu-Focal-20-04-LTS-docker-environment
Error:
sysctl: setting key "kernel.core_pattern": Read-only file system
Preparing to unpack ./vms_server.deb ...
D020000: trigproc_activate_packageprocessing pkg=networkoptix-metavms-mediaserver:amd64

mediaserver-eula license could not be presented
try 'dpkg-reconfigure debconf' to select a frontend other than noninteractive

D020000: post_script_tasks - ensure_diversions
D020000: post_script_tasks - trig_incorporate
Unpacking networkoptix-metavms-mediaserver (4.3.0) ...
D002000: infodb foreach file this pkg
Setting up networkoptix-metavms-mediaserver (4.3.0) ...
D020000: trigproc_activate_packageprocessing pkg=networkoptix-metavms-mediaserver:amd64
Detected runtime type: docker.
System has not been booted with systemd as init system (PID 1). Can't operate.
Failed to connect to bus: Host is down
System has not been booted with systemd as init system (PID 1). Can't operate.
Failed to connect to bus: Host is down
sysctl: setting key "kernel.core_pattern": Read-only file system
dpkg: error processing package networkoptix-metavms-mediaserver (--install):
 installed networkoptix-metavms-mediaserver package post-installation script subprocess returned error exit status 255
D020000: post_script_tasks - ensure_diversions
D020000: post_script_tasks - trig_incorporate
D010000: trigproc_run_deferred
Errors were encountered while processing:
 networkoptix-metavms-mediaserver

Fixed by Nx in 4.3 (5.0) R3:
https://support.networkoptix.com/hc/en-us/community/posts/1500000851801-4-3-R1-fails-to-install-on-Ubuntu-Focal-20-04-LTS-docker-environment/comments/4409828373655#community_comment_4409828373655
Reverting to Focal base images in 6ef6f11adf8251924ee03b4700c39f5a37a497a5


MatMulInteger + Add for quantized attention blocks

example block:

The example graph looks great. The correction term computation looks good to me as well, although we should make sure it's well-documented, especially since these terms to handle the zero points are relatively poorly understood.
What is your testing plan for this?

@tlrmchlsmth updated with the bias correction taken out and a Mul op for the cast outputs added.
immediate steps for testing will be to evaluate converted models for correctness against a standard dataset - integration testing is on the backlog for this entire script and is blocked by a few other issues.


Ignore pyxform_validator_update files that are user-specific

Unlike installed.json which is repo-specific, these files are user-specific and so should be probably be ignored. One alternative might be that we move these to a /tmp, but .gitgnore seemed like the narrowest solution.
Rebased on https://github.com/XLSForm/pyxform/pull/346 to get a green build.

@yanokwa FWIW from way back in #171 I figured the /bin dir could be gitignored too, so that pyxform didn't need to be updated for every ODK validate release.

@lindsay-stevens We should do that eventually, but feels easier for users to have it bundled for now since they are fairly coupled.
Relatedly, we might want to deprecated Validate and put that verification into JR. https://github.com/opendatakit/javarosa/issues/463#issuecomment-512997521


Google_Client Does Not Exist?

Im abit confused with this package. The example code does not match the classes in the repository. I cant fined Google_Client instead i have been using the Client Class.

Have a look at the latest version of the package. It is hosted in this repo: https://github.com/google/google-api-php-client

Ah yes i see now thanks! Is there a packagist version of this repo? Im using laravel.

Sure. Look in the composer.json file