RDKB-44710 RBUS Auto gen script context helper API wrong Implementation

RDKB-44710 RBUS Auto gen script context helper API wrong Implementation
Reason for change: The Helper APIs created from RBUS Auto Gen script is implemented wrongly. Its not returning pointer of context in the GetHandlerContext() API. This causes no update of UserData parameter on change of context and facing some crash(in Cellular Manager) Test Procedure: Tested Sample_app on VM Desktop with rbuscli; Run rtrouted on Term-1; Final/Sample_app on Term-2; rbuscli -i and perform Get, set, Add , Del operation for StaticTable; Writable table on Term-3 Risks: Med
Priority: P1
Signed-off-by: Deepak<EMAIL_ADDRESS>

RDKB-44710 Issue seems to be with Nested Writable tables Not with returning pointer to context change for this pull request #58 58


IQE exporter doesn't export animations

Filled out the "Animations" text field on export as the README.txt says to do, but no animation data shows up in the resulting file. I have not checked whether the binary format has the same issue.

@notnullnotvoid It turns out the animation names refer to Blender action (animation channels) names. So you have to create an action (Dope Sheet -> Action Editor) for each one, and it's basically "action:start:end" where start is 0 and end is the length of the animation; they can both apparently be left off.

@hemebond there was an issue with the Blender export not properly getting vertex information such as weights. I pushed a PR, it's been merged now. So, that would likely solve your issue too.

Hi @q009, I hoped so as well (someone alerted me yesterday to your PR) but unfortunately it's still mangled. Animated, but mangled.

@hemebond mind sharing some details about how you export and what you export? What sort of objects, do they have modifiers, what modifiers, etc.


[13pt] Stage-based CatFIM Action extents are very wide and often cover the other thresholds

See this vlab ticket for details. This appears to be limited to Stage-base CatFIM as Flow-based seems normal these locations. The LID in the vlab ticket is WSTM6, but the issue is very widespread.
The screenshot shows FIM v4 on the left and v3 on the right.

Is there any way to make the original vlab ticket available for viewing (I can't access it).. based on the screenshot with this ticket, it would -appear- that the services being compared a Flow-based CatFIM and not Stage-based, so it'd be helpful to see the original! I'm not seeing problems with the individual 1-foot CatFIM outputs at that site WSTM6, but I'll check the full run to see what got added there. Also curious if it's a visualization issue with the FIM 4 version (possibly including "other / non-categorized" data in with the Action stages, or a mis-ordering of the layers as can happen in QGIS).

I'll look into why you're able to see the ticket. It's good to know that our current version doesn't have this issue. Can you include a screenshot of it here? It would also be prudent to check out what our data looks like for this version (fim_4_0_13_1_stage_based) to get an idea of whether there's a visualization problem going on here or just a problem with an older version of FIM.

I'm pulling up the latest fim_4_0_13_1_stage_based catfim results (that ran EVERYTHING), but in the meantime I already ran and visualized the stage-based catfim code for the individual site WSTM6 (in the screenshots in the original ticket, it's the northwest-most blob). It's possible that there's an issue with the full catfim results run that doesn't affect a run on an individual site, but I think the chances of that are pretty low based on the code workflow.
Here's WSTM6 shown with its Action extent, which corresponds to stage 12 feet.

If I add the stages 13-15 feet, which are still below the Minor category, it looks like this:

The huge extent shown in the original ticket can correspond to either the 29 foot (maximum) extent or the no-label uncategorized option.

I'll update this ticket with definitive info about the full stage_based run.

Flow-based CatFIM for version <IP_ADDRESS> has a similar situation for the stage-based results for WSTM6 (screenshot). Action extent is reasonable, and the wide extents are for the Record category. Also worth noting is that an earlier version <IP_ADDRESS> is similar for the stage-based version (not shown).

I just checked this in the latest FIM <IP_ADDRESS> catfim runs, and it seems this site doesn't get included in the stage-based runs with that latest version. The flow-based data looks okay though.

Thanks for checking this one out! I talked with Corey and he confirmed that this was a HydroVIS issue. A quick re-publish of the service fixed this issue.


chore: more moderate readiness probes in hserver

I conducted a test using a 3-hserver cluster, and I discovered that almost all hservers experienced an average of 10 failed readiness checks. To address this issue, I updated the readiness probe with InitialDelaySeconds: 10 to reduce unnecessary checks because the server takes some time to bootstrap.
I have also increased the PeriodSeconds  appropriately, and reduced FailureThreshold by half relatively.

/cc @YangKian


Compiling content: "/xxx" issue

HI there, i got an issue about compiling property "content: " for an icon style.
So for example:
.fa-test:before {   content: "\f09a"; }
and then the result (min.csss) compiled file be like this
.fa-test:before{content:""}
Of course that is unidentified symbol.
Could you help me about this ?

Please refer to #39


Amigo not retrieving gene products

I see 17 gene product results for
http://amigo.geneontology.org/amigo/search/bioentity?q= MMACHC
but  I get
No results found.
No results given your input and search fields. Please refine and try again.

In fact search retrieval appears to be totally broken?

It was OK later...


Added code of conduct to the project

Looks good, but the markdown linter is raising two errors that will need to be addressed before we can merge this

[lint-md] CODE_OF_CONDUCT.md:71:14 MD034/no-bare-urls Bare URL used [Context: "https://www.contributor-covena..."]
[lint-md] CODE_OF_CONDUCT.md:76:1 MD034/no-bare-urls Bare URL used [Context: "https://www.contributor-covena..."]

Thanks for sending this. I've discussed this with the other maintainers and, for now, I am going to close this since the maintaining team needs to do some research on the different code-of-conduct options. Once we've completed that research, we can pick one that fits well with our governance model.


UI differential drive

The switch between swerve and differential is on the dimensions page of robot config, for now. In differential mode, the user needs to specify half of the trackwidth (temporary).

Been running this locally it works great


Create a single connection pool per host

When history service comes up it creates a gocql client for execution manager and then uses the same client for all shards for making persistence calls.  Also created config knobs to configure number of connections for both events and execution maanger clients.
fixes #307

Coverage decreased (-0.02%) to 66.44% when pulling 233f736938011ac912b691fd11d7812a7082fae7 on samarabbas:persistence_connections into 9169f8ffe30ed81e181337461f2ac1d4171fced0 on uber:master.

Coverage increased (+0.02%) to 66.49% when pulling 4913b8c953846cdffece8720add8903b7acea57a on samarabbas:persistence_connections into 9169f8ffe30ed81e181337461f2ac1d4171fced0 on uber:master.

Coverage decreased (-0.02%) to 66.469% when pulling 6024cf709c9f50b7f6255990cf9f94813d803ff7 on samarabbas:persistence_connections into affdf19891e1a2812b7f45395d4ae0520e77f1aa on uber:master.


ENH: adds API for comparing sourcetracker results

This partially addresses #57 and #31.
@wdwvt1, @johnchase, @lkursell this is an API to compare sourcetracker results. This could be used for the current benchmark and optimization projects as a way to determine how similar a pair of sourcetracker results are. I'm looking for input on the API before I spend too much time more time on it.
This defines two functions compare_sinks and compare_sink_metrics that become part of the public API as of the merge of this pull request (we'll commit to alpha-level stability of these in the next release - we're free to change these until then).
compare_sinks would take two pd.DataFrame objects containing observed and expected mixing proportions and a metric to use for comparing the mixing proportions. It would return a pd.DataFrame containing metric-specific data on the similarity/difference of the mixing proportions.
compare_sink_metrics is a simple helper function that returns a list of the available metrics. This would be necessary for a QIIME 2 plugin, so interfaces can determine what the available choices are for the metric parameter of compare_sinks.

Coverage increased (+1.9%) to 89.862% when pulling 7a3b19499b9a3d398fcd4585bd4152bd98eed217 on gregcaporaso:issue-57 into 08112ca469151e81b69cc9a77e5f6889608a3965 on biota:master.

@wdwvt1, @johnchase, @lkursell this is ready for review/merge whenever you guys are ready (I know you're working toward a deadline, so no rush).

Coverage increased (+2.2%) to 90.135% when pulling 1f91b8d3c475960d055d01d9533f1406416fa3a6 on gregcaporaso:issue-57 into 08112ca469151e81b69cc9a77e5f6889608a3965 on biota:master.

@gregcaporaso - this looks very useful!
Two real comments
I think the _absolute_difference function should sum the differences in each index. So rather than returning a per-source difference, it returns a total difference. In the test you have, I think it should be:
obs_m = [.5, .25, .25]
exp_m = [.1, .8, .1]
abs_diff = .4 + .65 + .15 = 1.2
There is no test for _validate_dataframe.
One style question
Do we need to document the private functions? Although users won't be accessing them directly, developers will (or we'll want to know in the future).

Coverage increased (+2.1%) to 90.351% when pulling 1a948ee18100daf04bb527d4f5f13835d0598677 on gregcaporaso:issue-57 into 33f5bffbfb8acbb2aaef8a115e2a4402b14b4f23 on biota:master.


News search fixed: removed unnecessary committing of files

Fixed the committing of all the unnecessary files in tmp/ folder

hi, @SirsikarAkshay Since we are refining the whole examples repo to get prepare for our hackathon, the examples will be mainly based on v0.5.5. Therefore, I'll hold this PR for the time being and merge it later.

Hello Nan.
Thank you for the update. I also wanted to remind you that I had applied
for a job at Jina for which the PR was necessary.
Kindly look into it and do the needful.
Thanks and regards
Akshay
On Tue, Sep 15, 2020 at 8:44 AM Nan Wang<EMAIL_ADDRESS>wrote:

hi, @SirsikarAkshay https://github.com/SirsikarAkshay Since we are
refining the whole examples repo to get prepare for our hackathon
https://jinaxhackathon.ai/, the examples will be mainly based on v0.5.5.
Therefore, I'll hold this PR for the time being and merge it later.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jina-ai/examples/pull/194#issuecomment-692436669, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXAAYOEV3XIVHWS3PCWLSLSF3LYBANCNFSM4RHF5AVA
.

Thank you for the information Nan. Hope I will be able to continue my
contributions to Jina.
Also wanted to inform you that I had applied for a job opportunity at Jina.
Kindly look into it.
Thanks and Regards
Akshay
On Tue, Oct 27, 2020 at 8:53 AM Nan Wang<EMAIL_ADDRESS>wrote:

Merged #194 https://github.com/jina-ai/examples/pull/194 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jina-ai/examples/pull/194#event-3923737218, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXAAYM4E7NNY6WA4BTSBRLSMY4MJANCNFSM4RHF5AVA
.


show fixes icon (light bulb) disappear when code folded

Issue Type: Bug
Wirite the following js code:
// @ts-check
const test = {
}
this is an error

the this is an error line causes an error, this shows a little light bulb (show fixes)

if the previous line is the end of a folded content, then the light bulb no longer appears.

the issue still appears after disabling all extensions
VS Code version: Code 1.22.1 (950b8b0d37a9b7061b6f0d291837ccc4015f5ecd, 2018-04-06T00:24:49.616Z)
OS version: Linux x64 4.4.0-116-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4690K CPU @ 3.50GHz (4 x 3805)

Load (avg)
1, 1, 0

Memory (System)
15.62GB (0.50GB free)

Process Argv
/usr/share/code/code

Screen Reader
no

VM
0%

Extensions (28)

Extension
Author (truncated)
Version

swagger-viewer
Arj
1.6.0

xml
Dot
1.9.2

latex-workshop
Jam
5.1.0

awesometypescriptproblemmatcher
Jua
0.3.0

vscode-docker
Pet
0.0.26

vscode-wakatime
Wak
1.2.0

latex-preview
ajs
0.5.1

vscode-svgviewer
css
1.4.3

vscode-eslint
dba
1.4.8

vue-snippets
hol
0.1.5

Go
luk
0.6.78

code-beautifier
mic
2.0.3

vscode-todo-parser
min
1.9.1

vscode-pugbeautify
mrm
1.0.2

vscode-kubernetes-tools
ms-
0.1.7

python
ms-
2018.3.1

vsliveshare
ms-
0.2.307

debugger-for-chrome
msj
4.3.0

vetur
oct
0.11.7

swaggitor
qns
0.1.1

vscode-yaml
red
0.0.11

sass-indented
rob
1.4.8

vscode-sql-beautify
sen
0.0.4

language-stylus
sys
1.9.1

tizentv
tiz
1.0.1

vscode-apielements
vnc
0.6.7

gitblame
wad
2.4.2

vscode-proto3
zxh
0.2.1

@jrieken A content widget painted on a folded line will not be rendered.
The next question is how was it possible to fold the } away like that. @aeschli is that a bug ?

@alexandrudima Yes, a known bug in the TypeScript folding provider.

The TS folding bug was fixed a few months back
@jrieken The lightbulb widget still doesn't render in this case however:
// @ts-check
const test = {

} this is an error

when test is folded the the cursor is over error.

A content widget painted on a folded line will not be rendered.

Seems to be an editor thing...

We now check if a line is folded and show the lightbulb above/below or iff possible inline. There can still be a case in which it doesn't show...

top line folded, use line below

use line above when possible

no line below, no line above, but inline space

no line below, no line above, no inline space -> no lightbulb


Stop using thyhjwb6 token for automated prerelease.

Package Automation.

New user and token created.  CircleCI variables updated. Awaiting test.

Confirmed up_next jobs are completing successfully using the new credentials.
New user credentials shared securely with team members.


find a way to refrain users from sending too much data.

We could use a body parser for the http request that would create a stream on the body and would look at the text section the http request. If we have read more than 20KB of this request and have not yet reached the end of the request we send a message back to the client telling him he should send no more than 20000KB =~ 20000 / 5 = 4000 (english) words.
Per minute we could limit the use to 40KB per user.
And to 100mB per all users.

well for the moment I use nginx as a reverse proxy to limit the body of httprequests from users to 1mbytes. Good enough..


hazard_to_survival() error

Hi Wilson,
I get the following error when calling hazard_to_survival():
Error in self$survival[i, ] <- cumprod(1 - hazard_here) :   number of items to replace is not a multiple of replacement length
This may be because I am setting t_max lower than max(T_tilde)?
I included the R code below that's generating the error.
library(MOSS)
vet_data <- read.csv("https://raw.githubusercontent.com/tlverse/deming2019-workshop/master/data/veteran.csv")
head(vet_data)
T_tilde <- vet_data$time
Delta <- vet_data$status
A <- vet_data$trt
W <- vet_data[, c(3, 6:9)]
t_max <- 200
SL.ranger.faster = function(...) {
SL.ranger(..., num.trees = 50)
}
sl_lib_decent <- c("SL.mean", "SL.glm", "SL.step.forward", "SL.bayesglm",
"SL.ranger.faster", "SL.gam")
initial_fit <- initial_sl_fit(T_tilde, Delta, A, W, t_max,
sl_censoring = sl_lib_decent,
sl_failure = sl_lib_decent)
initial_fit$density_failure_1$hazard_to_survival()
initial_fit$density_failure_0$hazard_to_survival()

Thanks for the catch! Let me take a look at this

I get the same error, but I did not even change the t_max value. Looking at the source code I can see why this error occurs.
hazard_to_survival = function() {
      # working
      self$survival <- matrix(NA, nrow = self$n(), ncol = max(self$t))
      for (i in 1:self$n()) {
        hazard_here <- c(0, self$hazard[i, ])
        hazard_here <- hazard_here[-length(hazard_here)]
        self$survival[i, ] <- cumprod(1 - hazard_here)
      }
      return(self)
    }

The max(self$t) call returns the maximum value of the time points, which is not what is needed here. For example in my case the values of self$t are equal to 0:99 + 0.2975822. This means that there are 100 values, but the maximum is 99.2975822, which results in a matrix with 99 columns while 100 are needed. I think this function can be fixed by replacing max(self$t) with length(self$t).
Please let me know when you fixed it.


Update Dockerfile

Add BUILD_DEPS with git

why do you need git in this image?

No answer, closing this.


Would You Port Bibliocraft Legacy To 1.20.1?

We are not planning to go back any amount of versions as we use modern features that did not exist prior to the forge/neoforge split


Add rule to check whether schema does not use anyOf or oneOf

What does this PR do?
This PR adds a new rule to check whether the schema does not use anyOf or oneOf as defined in R9 in this RFC.
What is the effect of this change to users?
Users that run the verify command with the --rule-set cluster-app flag will see additional recommendations.
How does it look like?
❯ go run ./main.go verify --rule-set cluster-app ./pkg/lint/rules/testdata/should_avoid_x_of/any_of_and_one_of.json

Recommendations (4)

- Schema at path '/age' should not use oneOf.
- Schema at path '/foo' should not use anyOf.
...

Any background context you can provide?
Towards https://github.com/giantswarm/roadmap/issues/1772
RFC
What is needed from the reviewers?
You can test the changes with:
go run ./main.go verify --rule-set cluster-app ./pkg/lint/rules/testdata/should_avoid_x_of/any_of_and_one_of.json

Do the docs/README need to be updated?
No
Should this change be mentioned in the release notes?

[x] CHANGELOG.md has been updated

R9 isn't up-to-date yet. As we discussed on Friday, we will allow anyOf and oneOf, but only for constraints.

R9 isn't up-to-date yet. As we discussed on Friday, we will allow anyOf and oneOf, but only for constraints.

Oops, thanks for noticing.

I just updated R9 in the RFC draft.

Unsubscribing for now. Please mark as ready for review and re-request review once this is ready.

@marians ping, because I am not sure whether you saw the re-request for review.


Repository is not marked as Template

Since this repository is not a template, the described quickstart can not be followed at the first bullet point already.

Click the Use this Template button.

Thanks @yannickberk ! I've made that change now

Link still not working. Am i missing something?

I see this as a template, @baumgars123 what are you seeing instead?

I get an error 404 page instead @alfredodeza

@baumgars123 not sure where you are seeing a 404. Are you saying that the button doesn't even appear? Or you click it and you get a 404?

@alfredodeza i didnt even notice the place of the button you are showing. All i was looking at is this. And clicking on the button shows me a 404 page. What is the difference? The same (404) goes for section "Getting started" and the ".NET Codespaces Template" link.


Zoom in/out to view larger card image

Codecov Report

Merging #647 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #647   +/-   ##
========================================
  Coverage    98.09%   98.09%
========================================
  Files          218      218
  Lines        11578    11578
  Branches        63       63
========================================
  Hits         11357    11357
  Misses         221      221

@thatbudakguy thanks for catching the cursor problem! I was applying the style too broadly.
I tried setting the overflow but it didn't have much impact — I guess it would need to be set on the body? Can you think of any way to do that in pure css?

ah, makes sense - I was afraid of that. no way to do it in css that I know of. not a big deal though.

@thatbudakguy the scroll doesn't so matter much since I'm linking back to # to zoom out, which puts you back at the top of the page (I wasn't sure if there was a better option).
I've adjusted the cursor styles if you want to review.

I'd say just go ahead and merge it then.


Reduce memory allocations in desktop implementation

We noticed a lot of allocations due to new ControllerQuerier instances and boxing from Float/Boolean to float/boolean.

You are right about that. Code changes look good to me, I will test the PR next week and merge it afterwards.
Thanks for your PR.

There is some more unnecessary boxing in CompositeControllerListener.

Thanks for your contribution. Feel free to change the CompositeControllerListener if you think it should be changed.
By the way, would be great to see you on the Discord server. Links is on the main readme.

Thanks. Made a PR for CompositeControllerListener :) Do you think a new release will be soon?
I'm already on the Discord, I don't talk much XD

Please PN me on discord, I didn't find your user handle.
Do you need a new version soon? I wanted to add a platform-independant way to retrieve the default button/axis constants to AdvancedController and am waiting for 1.9.10 to be released before releasing a new version. I aim to make a release at least every two months for all of my libs.


Going back after setting the location does not revert to the default location

To Reproduce
Steps to reproduce the behaviour:

Go to 'https://vaxx.nz/'
Pick a location, i.e. 'Epsom'
Press the back button, it'll leave Epsom showing in the location picker.

https://github.com/CovidEngine/vaxxnz/pull/63

Fixed by #63


Add option FLAGR_RECORDER_KAFKA_PARTITION_KEY_ENABLED

Description
Add option FLAGR_RECORDER_KAFKA_PARTITION_KEY_ENABLED
Motivation and Context
We have entity id that contains only user's segment. When google bot come to our site we have many messages in only one partition in kafka. This pull request disable key partition, so messages come to random partition but without order guarantee.
How Has This Been Tested?
Types of changes

[ ] 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 change)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@zhouzhuojie interesting PR!

Codecov Report

Merging #535 (6f042ac) into main (347a36d) will decrease coverage by 0.10%.
The diff coverage is 62.50%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #535      +/-   ##
==========================================
- Coverage   80.96%   80.87%   -0.10%
==========================================
  Files          28       28
  Lines        2222     2227       +5
==========================================
+ Hits         1799     1801       +2
- Misses        334      336       +2
- Partials       89       90       +1

Impacted Files
Coverage Δ

pkg/handler/data_recorder_kafka.go
72.41% <62.50%> (-1.47%)
:arrow_down:

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


Cannot specify sdk as a full path

If you use the full path to an sdk as an argument, xctool returns an error.  xcodebuild works properly when you do this.
ex:
xctool -sdk /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator8.1.sdk
ERROR: SDK '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator8.1.sdk' doesn't exist.  Possible SDKs include: macosx, iphoneos, iphoneos8.1, macosx10.10, iphonesimulator, iphonesimulator7.1, iphonesimulator8.1, macosx10.9

May I ask a reason why you specify full path? Doesn't combination of DEVELOPER_DIR= and iphonesimulator8.1 work for you?

My work's build tool createse full paths for every argument to increase reproducibility in builds.   The SDK is one of those items.
Also I want to be able to drop in replace xcodebuild with xctool as a build option without otherwise changing our build system.  Our build team wouldn't like supporting having two separate command line tools argument sets.  In order to convince them in the future to try it out, I have to make it easier to support for them.

Thanks for reporting. Fixed in b81200e0a20061dc3e7bef39dcd540d16fc2b086.


Grafana dashboards rework

All existing dashboards are re-worked to function properly and new many of the graphs are consolidated.
What this PR does / why we need it:
Following dashboards are now available:

Alertmanager Alerts with small collection of graphs for current and previous alerts.
Kubernetes Cluster Health presents general availability of some of the core components with links to Alertmanager Alerts dashboard.
Kubernetes Control Plane Status focuses on the control plane components - API Server, Scheduler and Controller manager with API Server latency and fail rate.

Kubernetes Deployments with all deployments in the Shoot's namespace in the Seed cluster and core components in the Shoot's kube-system

namespace. It's possible to drill down to individual Pods in the Kubernetes Pods dashboard.
Kubernetes Pods focuses on individual Pods in either Seed or Shoot cluster.

Prometheus dashboard with various Prometheus metrics.

Shoot Nodes for individual Node metrics (capacity, allocations), container and systemd services.

Which issue(s) this PR fixes:
Fixes #36
Special notes for your reviewer:
Release note:

All of the Grafana dashboards for every Shoot are revamped and consolidated.

Super nice, thank you!


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
d4b13032be169e4c0c923214deae3b8fd67bb06b
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Can the preference be stored？

Can the preference be stored？when I change a pc,it is convenient to set the preference

Preferences of Spyder are stored in the home folder under the .spyder-py3 folder. On windows that is: c:\Users\your_username\.spyder-py3\.
I have moved this folder across machines with the exact same Spyder versions without any (noticeable) issues.

Thanks @ba-tno for your help! That's right, you can copy that folder and drop it in the same location in another computer and you'll have the same settings.


[proposal] koord-scheduler should support debugging API

What is your proposal:
koord-scheduler should support debugging API.
The API should support the following services:

query the node info
query pod info
query the plugin's internal state
enable/disable log score details

Why is this needed:
It can efficiently troubleshoot, diagnose and analyze scheduling failures or abnormal problems.
Is there a suggested solution, if so, please add it:

cc @buptcozy


Allow Mac users to double click install.sh

Description
Based on support issue 1048, it seems that Mac users are unable to double click the the install.sh script, due to working in home directory when double clicking.
Steps to Reproduce
Run make, then double click install.sh (I have not verified that this fails), but seems to be the case in support issue 1048.
Expected Result
The app gets installed
Actual Result
Versions

Software Version 2.0.3
Ledger Version 2.1.0

It can be fixed by changing the relative paths in Makefile to paths relative to install.sh. For example, use $ROOT/bin/app.hex instead of bin/app.hex, where ROOT=$(dirname "$0").

Will not do according to the above reasoning.


stop_all only removes clipper/management_frontend

clipper.stop_all() only removes management_frontend container. It should remove all clipper containers.

Yeah this was a temporary regression. Fixed in #168.


Create DirtyFlipping

Target Label-Flipping Attack Using Dirty Label-Inversion : Speech Vulnerability !
A dirty label-flipping attack is used in the backdoor approach to produce poisoned data collection. Input consists of clean labels and clean data samples; output is a set of poisoned labels and data. The initial labels and data are kept if the target label is absent from the clean labels. The selected dirty label is applied to the labels of poisoned samples. With a given probability, the label is reversed once the trigger function is applied to the input data. The attack aims to introduce a backdoor for a potential model misclassification by carefully crafting a trigger and injecting it into clean data samples of a certain target class. This is a backdoor attack using "dirty label-on-label" techniques  that introduce a trigger into data samples specific to a target class
Testing
The full code
notebook Description
Hi guys @beat-buesser !, I just created the first dynamic backdoor attack by dirty label and label inversion, the attack is stealthy and undetectable, I test them on complex databases TIMIT and AudioMnist,
I also added speaker verification tests such as NeMo from Nividia, my attack was 100% deceptive, all HugginFace speaker verification link failed to detect the deception.
Additional work applying 'DirtyFlipping' to HugginFace models
notebook  HugginFace Backdoor link HugginFace Backdoor attack
Test Configuration:

OS
Python version
ART version or commit number
TensorFlow / Keras / PyTorch / MXNet version

Hi @OrsonTyphanel93 Thank you very much for your pull request! It will be reviewed as soon as possible targeting ART 1.18.

Codecov Report
Attention: 176 lines in your changes are missing coverage. Please review.

Comparison is base (0400813) 85.60% compared to head (2f9d216) 67.84%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@               Coverage Diff               @@
##           dev_1.18.0    #2376       +/-   ##
===============================================
- Coverage       85.60%   67.84%   -17.76%
===============================================
  Files             324      327        +3
  Lines           29326    30205      +879
  Branches         5407     5589      +182
===============================================
- Hits            25104    20494     -4610
- Misses           2840     8451     +5611
+ Partials         1382     1260      -122

Files
Coverage Δ

art/__init__.py
100.00% <100.00%> (ø)

art/attacks/evasion/__init__.py
98.24% <100.00%> (+0.03%)
:arrow_up:

...asion/adversarial_patch/adversarial_patch_numpy.py
9.58% <ø> (-64.68%)
:arrow_down:

art/attacks/evasion/dpatch.py
91.25% <ø> (ø)

...cks/evasion/imperceptible_asr/imperceptible_asr.py
90.33% <100.00%> (ø)

art/attacks/extraction/knockoff_nets.py
20.13% <ø> (-69.80%)
:arrow_down:

...ks/inference/membership_inference/shadow_models.py
44.82% <ø> (-49.14%)
:arrow_down:

...cks/poisoning/perturbations/audio_perturbations.py
88.09% <100.00%> (+0.29%)
:arrow_up:

art/defences/detector/poison/activation_defence.py
83.28% <100.00%> (+0.04%)
:arrow_up:

...nces/detector/poison/spectral_signature_defense.py
84.72% <100.00%> (+0.21%)
:arrow_up:

... and 32 more

... and 79 files with indirect coverage changes


[release-4.13] OCPBUGS-10532: update ovn-kubernetes to rhel9 based image

Signed-off-by: Zenghui Shi<EMAIL_ADDRESS>(cherry picked from commit f0585c67818cfe9fe2b68bfe9cb4581f520bb0a8)

/jira refresh

/jira refresh

/retest

/lgtm
/label backport-risk-assessed
/label bugzilla-valid-bug


Avro fails to maintain Comparer on Dictionary<string, T>

Title is fairly explanatory. Built the minimum repro to demonstrate the problem.
Root cause appears to be that the Comparer isn't stored as part of the type information so that when we serialize we lose that info.
Example of the bug
Repro.cs.txt
Going to see if I can track this down further and fix it.

Wrong repo.


Question

According to my understanding, the controller C at time t has two inputs. zt and ht, now ht is the prediction produced by M from zt-1, ht-1 and at-1.
So basically, the controller makes its predictions based on...

zt: the current state of the world,
M(zt-1, at-1, ht-1): i.e. what it thought the current state of the world would be given the previous state of the world + its previous chosen action + the previous hidden state of M.

It seems counter-intuitive to me that the current state of the world together with the expectation of the current state of the world should be a sufficient basis for a strong controller. It doesn't seem to make full use of the predictors capabilities.
Is this correct?

Hi @jpeg729,
That is a great question!
You are correct, technically, if we define t precisely to align z and h with h0 = hinit, then the controller's calculation of at is based on zt and ht-1. It doesn't use ht, since ht needs at to be calculated first, so kind of a chicken and egg problem.
In that sense, one can view ht-1 as a compressed representation of all of the zi and ai, i ∈ {0 ... t-1}. Thus in addition to the current observation zt, the controller's decision will be based on this compressed representation of the entire history up to the point in which it has to make a decision at.
This is something I have thought about when constructing the setup and the algorithm, since ideally we want to use the current h. For the experiments I tried this seemed to be good enough, though the fact that a more complicated controller (with an extra hidden layer) in the Car Racing setup improves the results quite a bit suggests that there is more we can do here.
One thing I have thought about trying, but haven't gotten to it, is to have the controller calculate a temporary ā = controller.action([z, h]), and by rolling forward using this temporary ā to arrive at a temporary h̅ = rnn.forward([ā, z, h]), and see if we can get a policy with this rolled-forward temporary hidden state. It doesn't look as elegant as the current approach though. Other methods of rolling forward to do planning might also help, at the expense of complexity and elegance.
If you come up with a more elegant way to calculate the forward state, feel free to share!
Best.

Hi @jpeg729,
That is a great question!
You are correct, technically, if we define t precisely to align z and h with h0 = hinit, then the controller's calculation of at is based on zt and ht-1. It doesn't use ht, since ht needs at to be calculated first, so kind of a chicken and egg problem.
In that sense, one can view ht-1 as a compressed representation of all of the zi and ai for i ∈ {0 ... t-1}. Thus in addition to the current observation zt, the controller's decision will be based on this compressed representation of the entire history up to the point in which it has to make a decision at.
This is something I have thought about when constructing the setup and the algorithm, since ideally we want to use the current h. For the experiments I tried this seemed to be good enough, though the fact that a more complicated controller (with an extra hidden layer) in the Car Racing setup improves the results quite a bit suggests that there is more we can do here.
One thing I have thought about trying, but haven't gotten to it, is to have the controller calculate a temporary ā = controller.action([z, h]), and by rolling forward using this temporary ā to arrive at a temporary h̅ = rnn.forward([ā, z, h]), and see if we can get a policy with this rolled-forward temporary hidden state. It doesn't look as elegant as the current approach though. Other methods of rolling forward to do planning might also help, at the expense of complexity and elegance.
Alternatively, we can modify the RNN's roll-forward operation to only depend on z, and not a, but have the MDN-layer's prediction based on a instead, so you can roll-forward h and use it in the same time step to make the prediction. This might be the best option I feel, if we really want to use the forward hidden state. While this will allow the use of a more current h, the MDN (currently just a linear layer) will need to be have more capacity to compensate for the extra processing needed there.
# modification to use the forward h:
def rollout(controller):
  ''' env, rnn, vae are '''
  ''' global variables  '''
  obs = env.reset()
  h = rnn.initial_state()
  done = False
  cumulative_reward = 0
  while not done:
    z = vae.encode(obs)
    h = rnn.forward([z, h])
    # (Note: MDN-layer modified to rely on a and h, not just h)
    a = controller.action([z, h])
    obs, reward, done = env.step(a)
    cumulative_reward += reward
  return cumulative_reward

If you come up with a more elegant way to calculate the forward state, feel free to share!
Best.

Hi @jpeg729,
That is a great question!
You are correct, the controller's calculation of at is based on zt and ht, and it doesn't use ht+1, since ht+1 needs at to be calculated first, so kind of a chicken and egg problem.
In that sense, one can view ht as a compressed representation of all of the zi and ai for i ∈ {0 ... t-1}. Thus in addition to the current observation zt, the controller's decision will be based on this compressed representation of the entire history up to the point in which it has to make a decision at.
This is something I have thought about when constructing the setup and the algorithm, since ideally we want to use the current h. For the experiments I tried this seemed to be good enough, though the fact that a more complicated controller (with an extra hidden layer) in the Car Racing setup improves the results quite a bit suggests that there is more we can do here.
One thing I have thought about trying, but haven't gotten to it, is to have the controller calculate a temporary ā = controller.action([z, h]), and by rolling forward using this temporary ā to arrive at a temporary h̅ = rnn.forward([ā, z, h]), and see if we can get a policy with this rolled-forward temporary hidden state. It doesn't look as elegant as the current approach though. Other methods of rolling forward to do planning might also help, at the expense of complexity and elegance.
Alternatively, we can modify the RNN's roll-forward operation to only depend on z, and not a, but have the MDN-layer's prediction based on a instead, so you can roll-forward h and use it in the same time step to make the prediction. This might be the best option I feel, if we really want to use the forward hidden state. While this will allow the use of a more current h, the MDN (currently just a linear layer) will need to be have more capacity to compensate for the extra processing needed there.
# modification to use the forward h:
def rollout(controller):
  ''' env, rnn, vae are '''
  ''' global variables  '''
  obs = env.reset()
  h = rnn.initial_state()
  done = False
  cumulative_reward = 0
  while not done:
    z = vae.encode(obs)
    h = rnn.forward([z, h])
    # (Note: MDN-layer modified to rely on a and h, not just h)
    a = controller.action([z, h])
    obs, reward, done = env.step(a)
    cumulative_reward += reward
  return cumulative_reward

If you come up with a more elegant way to calculate the forward state, feel free to share!
Best.

Hi @hardmaru
In you first response for the first question in this tread there is an statement which has confused me:

In that sense, one can view ht as a compressed representation of all of the zi and ai for i ∈ {0 ... t-1}. Thus in addition to the current observation zt, the controller's decision will be based on this compressed representation of the entire history up to the point in which it has to make a decision at.


Errors thrown in streams are not observable

Description:
Following is a simple database query scenario. Even though query() remote function is observable what it returns is a stream. Therefore even if some error occurs during the query it will not be observed since the error is actually thrown when iterating the stream in the forEach loop.
mysql:Client|error mysqlClient = new ("localhost", dbUser, dbPassword, "testDB", 3306);

if (mysqlClient is error) {
    io:println("connection error occurred");
} else {
    io:println("MySQL client created.");

    io:println("------ Start Simple Query -------");
    stream<record { }, error> resultStream = mysqlClient->query("Select * from customerrrr");

    error? e = resultStream.forEach(function(record { } result) {
                                        io:println("Customer full details: ", result);
                                        io:println("Customer first name: ", result["first_name"]);
                                        io:println("Customer last name: ", result["last_name"]);
                                    });
    if (e is error) {
        io:println("ForEach operation on the stream failed!", e);
    }
    io:println("------ End Simple Query -------");

    var closeErr = mysqlClient.close();
}

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Observability
Suggested Assignees (optional):

We need to add support to observe Ballerina Streams to get this working. Hence this is a new feature.


Ruby code not being shown on a certain task page?

http://rosettacode.org/wiki/Best_shuffle
When Ruby is selected as a language to be shown, it appears in the Table of Contents, but there is not code sample or section header for it.

Nevermind, it's working now.


##[error]Bash exited with code '1'.

Created bash task to run the command with arguments and it runs successfully, but pipeline fails with the message:
##[error]Bash exited with code '1'.
Task is following:
  - bash: anchore-cli --json --url $(anchorServer) --u $(anchorUser) --p $(anchorPassword) evaluate check $(acrRegistry)/$(imageRepository):$(tag) --detail > image-policy.json       displayName: Scan for the policy violations and save the report image-policy.json       continueOnError: true
Task runs and produces report. However, pipeline fails with error randomly. Sometimes, it runs successfully, sometimes comes back with ##[error]Bash exited with code '1'.
Agent is MS hosted agent Ubuntu.

That means that sometimes anchore-cli has a return code of 1.  the bash task simply reports the return code of the script that is run.

This error continues to show up in the Trustwallet
Check failure on line 160 in Build log

@azure-pipelines
azure-pipelines
/ trustwallet.assets

Build log #L160
Bash exited with code '1'.

`Test "blockchains" folder › Check Ethereum side-chain folders › Test chain ethereum folder
Custom message:
  Expect asset at path /home/vsts/work/1/s/blockchains/ethereum/assets/0x76ACE4E080C49f04A6C2756C036d3627f6b1E69 in checksum

expect(received).toBe(expected) // Object.is equality

Expected: true
Received: false

   98 |
   99 |                     const checksum = isChecksum(address)
> 100 |                     expect(checksum, `Expect asset at path ${assetPath} in checksum`).toBe(true)
      |                                                                                ^
  101 |
  102 |                     const lowercase = isLowerCase(address)
  103 |                     if (lowercase) {

  at src/test/index.test.ts:100:87
      at Array.forEach (<anonymous>)
  at Object.<anonymous> (src/test/index.test.ts:95:43)

`


Compatible with Python 3.5.2?

Looks like a really interesting project!
The Pipfile states a requirement on 3.6, but I can't see this stated anywhere else?  Is this a hard requirement or would 3.5.2 also work?
Thanks

Not its only compatible with 3.6

used `pathlib
think i also used fstrings but not sure.
used the secrets module earlier, but then i ditched in favor of uuid

What OS are you on?
You can use pyenv for installing 3.6

Ubuntu 16.04.  I'll take a look at pyenv

Actually I installed on 3.5.2 and it seemed to work. Test pass and examples ran.

Didn't know pathlib worked with 3.5, damn.
Will make sure to NOT use fstrings in future, I used a couple of them in my local branch.
Thanks for the report!

setup.py shall be fixed in next commit.

I will also switch to 3.4 in my local development, since pathlib also works on 3.4.
Pipfile will be updated in the next release

Looks like pathlib was incomplete in 3.4, 3.5 is the last supported version..

It's done.
Let me know if you build something cool with it, new features, API restructuring.. anything ;)


Prettify Ext. and Default Languages

Sorry to write again, but I'm trying to see if prettify can be used for anything useful.
Is it correct that the lang=matlab needs to be set as parameter to run_prettify.js since MATLAB is not part of the default supported languages in prettify ?
If using e.g. c++ I do not need to specify lang=cpp since this is a default language ?
If using pretttify.js instead of run_pretttify.js, I need to link each of the non-default language js files, e.g. lang-matlab.js ?
But if calling just pretttify.js I also need to call the function prettyPrint(); on my page to get it started ?
Why would you then ever call pretttify.js and not run_pretttify.js ?
Looking at the sample below, why is vb and c++ rendered differently ?
https://jsfiddle.net/cpede/1gkc7fjw/
Thanks for helping me.
-cpede

Is it correct that the lang=matlab needs to be set as parameter to run_prettify.js since MATLAB is not part of the default supported languages in prettify ?

yes, you need


Unable to download certain albums even if all music is available on YouTube.

System OS
Linux
Python Version
3.9 (CPython)
Install Source
GitHub
Install version / commit hash
3.9.0
Expected Behavior vs Actual Behavior
Well, I was trying to download the Made in Abyss soundtrack, and it didn't download. Said "Could not match any of the results found on YouTube. Skipping"
Thing is, all parts of the soundtrack are fully available on YouTube. Is there a fix for this bug? Is there a playlist downloader I can use until this is fixed? Thanks.
Steps to reproduce
Command: spotdl https://open.spotify.com/album/5oEzjkSYSfC0ITRiqK6m8l --output-format flac
Traceback
Fetching Album...
Could not match any of the results on YouTube. Skipping

Other details
No response

I had the same issue on windows 10. I restarted my pc & changed the target directory to C:/ (previously D:/music) and it worked fine. So I have to manually copy the downloaded file to my music folder.


Help

hi, I need some help
I run the rootkit through the shellcode like the c# example
when I try to make a registery key/value of my file to be added to startup hidden
I created the key using cmd:
REG ADD HKLM\Software$77config\startup /v $77myapp /t REG_EXPAND_SZ /d pathto/$77myapp
it gets added and I can see it using reg query, but when I restart the device the app doesn't open on startup
can you help me? am I doing something wrong?

Is r77 definitely running at startup? - you can check on that with the Test Console.
Is your $77myapp.exe a desktop application of some kind? Since the executable is started with SYSTEM privileges, it may not work, if it's a desktop app. Can you try something, like charmap.exe or some helloworld.exe to test whether startup doesn't work at all, or whether it just doesn't work with that executable?
Since I cannot debug on your computer, I can recommend ProcessMonitor to monitor the creation of processes, to see whether the process was crearted and crashed - or whether it was not created at all.
Of course, you don't need to restart your PC. You can simply run Install.exe over and over again to test the startup.

can I privately contact you? do you have telegram?

I don't have telegram, but you can shoot me a mail. My email address is on my website bytecode77.com


TextField crashing on kotlin compose desktop

Describe the bug
app is crashing with a simple textfield in an empty composable app when selecting text, typing again and dragging
Affected platforms
Desktop
Versions
1.8.0- Kotlin version*:
1.3.0- Compose Multiplatform version*:
windows 10 home - OS version(s):
x64- OS architecture :
<IP_ADDRESS> - JDK:
To Reproduce

Create a simple TextField
run the app
type anything that has more than 2 letters (I couldnt replicate with one letter, but I did with two).
select some part of the text (You have to select 2 or more letters, and prefer select going from the end to the left, otherwise its harder to replicate) without releasing the mouse button
type anything
move mouse button to anywhere
release mouse button

Expected behavior
Nothing
Additional Information
Bug doesn't happen on versions:
kotlin.version=1.6.10
compose.version=1.1.1

Reproduced on MacOs too: https://github.com/eymar/repr_compose_2891
Compose: 1.3.1
Kotlin: 1.8.10

I didnt expect anything at all actually..I did this by accident when trying to test masks on textfield. First I thought it was an issue with my mask, but then I saw it happened in a clean project.

@starglad Thanks! The issue is clear and reproducible.
What behaviour did you want to achieve my typing while keeping the mouse button pressed after selection + moving the mouse afterwards?
Did you expected anything useful from that sequence of actions? I'm trying to understand if there is some sort of UX pattern (common action) implemented with the steps you described.

I forgot to mention, but for some reason(may be related to the same issue) when typing fast, sometimes the last letters mix up. Like if I type "asdf" it comes "adsf". It happens on the same version as the issue above (1.8.0 kotlin and 1.3.0 compose). Additionally it also DOESN'T happen at kotlin 1.6.10 and compose 1.1.1

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


Use Clojurescript to transform html

The current method for transforming html is lame (regex, yuck!)
There exists a number of Clojure(script) libraries which could be used to transform html, as clojurescript can output to Node.js, it would be viable to build a handful of helpers utilizing these libraries to transform the html as required.

Used Hyperglue.

Used Hyperglue.


Add STL tests

Moves lest to its own nix file, allowing it to be used in integration tests
Adds a vmrunner parameter to shell.nix, for vmrunner development
Fixes coroutines test
Add a new test category to test.sh, STL, that tests aspects of the C and C++ runtimes

Result of ./test.sh:
🌈✨ Everything is awesome ✨

LGTM! Nice to have more tests


Permission denied on gradlew

Hi, im attempting to get Frogbot PR scanning working on a Java project in my github organisation - but whilst it detects it is gradlew that is needed, it is failing with permission issues. I have tried using chmod in a previous step to no effect.
Are there any parameters on the frogbot action that are related to this?
Any assistance would be appreciated.
10:36:16 [Info] Preforming scans on 1 targets:
  {
    "xray_version": "3.107.11",
    "xsc_version": "1.12.5",
    "jas_entitled": false,
    "command_type": "source_code",
    "start_time": "2024-11-28T10:36:15.248895194Z",
    "multi_scan_id": "9[82](https://github.com/******/actions/runs/12067188167/job/33649617256?pr=22#step:6:83)21207-ad74-11ef-99fe-62877ead494e",
    "targets": [
      {
        "target": "/tmp/jfrog.cli.temp.-1732790175-[86](https://github.com/*******/actions/runs/12067188167/job/33649617256?pr=22#step:6:87)0464753",
        "technology": "gradle",
        "sca_scans": {
          "descriptors": [
            "/tmp/jfrog.cli.temp.-1732790175-860464753/build.gradle"
          ]
        }
      }
    ]
  }

  Error: 7 [Error] target '/tmp/jfrog.cli.temp.-1732790175-860464753 [gradle]' errors:
  Failed to build dependency tree: failed while building 'gradle' dependency tree:
  error running gradle-dep-tree: fork/exec ./gradlew: permission denied

I am also experiencing this exact issue on my gradle project.

Got to the bottom of this - gradlew isnt marked as executable in git.
Resolved with:
git update-index --chmod=+x gradlew
git ls-files --stage   (lists all the files - gradlew should now be 00755 rather than 00754)
git add gradlew
git commit -m "Make gradlew 0755 (executable)"
Had to do this through the command line as GitHub desktop did not recognise the change.
This is now working for me.


it's a task of classification ? how to extract the relation and entity？

how to extract the relation and entity for a sentence？

I am having the same issue , how can i predict  the relations , if i give it to the model?

how to extract the relation and entity for a sentence？

it can just predict the relation type such as message-top and so on, it can't predict the entity in the relation

thanks


⚠ Bofrost.es has degraded performance

In 128633d, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1401 ms

Resolved: Bofrost.es performance has improved in f316215 after 7 minutes.


Bug 2117268: api-resource-collector: Don't attempt to parse empty Ignition

While filtering files out of MachineConfigs, we tried to also parse
ignition specification of MachineConfigs that didn't specify any,
leading to an error.
Let's just pass on MCs that don't have any Ignition spec as-is.
Resolves: rhbz#2117268

/hold
for QE review

/test e2e-aws

/test e2e-aws

@xiaojiey PTAL if this fixes the bug
@mrogers950 @rhmdnd @Vincent056 PTAL from code perspective

/lgtm

@sheriff-rh I added a CHANGELOG blob to the PR, is it sufficient for you to base a release note on? If yes, can you add the docs-approved label?

/bugzilla cc-qa

/label qe-approve

lgtm, thanks @jhrozek

Since this is a bug, I'm adding the px-approved label myself
/label px-approved

/hold cancel

@mkumku see this PR for px-approved

/lgtm


BackingImage does not download URL correctly in some situation

Remove the Referer header that is automatically added by the http client. Without this header it is possible to download files that are redirected several times. This mimics the behaviour of curl and wget which do not submit the Referer header by default.
Which issue(s) this PR fixes:

https://github.com/longhorn/longhorn/issues/7914
What this PR does / why we need it:
This PR removes the Referer header from the requests. This is necessary to allow downloads of files (e.g. ISO images) that are delivered from file hosters (e.g. SourceForge) via CDN which normally results in several redirects until the requested file is downloaded.
This PR mimics the default behaviour of tools like curl or wget.

Testplan

Create a file with the following content:

apiVersion: longhorn.io/v1beta2
kind: BackingImage
metadata:
  name: clonezilla
  namespace: longhorn-system
spec:
  sourceParameters:
    url: https://sourceforge.net/projects/clonezilla/files/clonezilla_live_alternative/20240116-mantic/clonezilla-live-20240116-mantic-amd64.iso
  sourceType: download

Apply it with k apply -f clonezilla.yaml.
Check the output:

$ k get backingimage -n longhorn-system
NAME                  UUID       SOURCETYPE   SIZE        AGE
clonezilla            42549254   download     102954      11s

This is the current state. The ISO is not downloaded properly.
4. Install the backing-image-manager image containing this PR.
5. Delete the clonezilla backing image via UI.
6. Apply it with k apply -f clonezilla.yaml.
7. Check the output:
$ k get backingimage -n longhorn-system
NAME                  UUID       SOURCETYPE   SIZE        AGE
clonezilla            9c339f0f   download     500170752   8s

Create another backing image.

apiVersion: longhorn.io/v1beta2
kind: BackingImage
metadata:
  name: systemrescue
  namespace: longhorn-system
spec:
  sourceParameters:
    url: https://fastly-cdn.system-rescue.org/releases/11.00/systemrescue-11.00-amd64.iso
  sourceType: download

Check the output:

$ k get backingimage -n longhorn-system
NAME                  UUID       SOURCETYPE   SIZE        AGE
systemrescue          28b76870   download     894435328   17s

@mergify backport v1.6.x v1.5.x


Update sk.json

Coverage remained the same at 92.162% when pulling 956b56472dba6d47b5e70926935d1f738156300f on ja-som:patch-16 into 3bfec8a89857d582b04c337576279623d5d6f116 on MarshallOfSound:master.


Validator should check for the existence of raw layer

In the schema v1.1.0 we require to have a "raw" matrix  and suggest to store it AnnData.raw.X.
The validator does not currently check for this requirement.
We should:

Include a check in the validator for the existence of a raw layer.
[Suggestion] check for int as the type in that layer.
[Suggestion] to make integration processes easily streamable and to make checks easier, we should require to have the raw layer in AnnData.raw, as opposed to just have this as a suggestion. And added plus to this is that AnnData has some extra checks/restrictions on AnnData.raw

Will create a new issue based on the schema 2.0.0 changes to raw requirements.


[GIE/IR] Support otherV() on Groot/Vineyard

What do these changes do?

As titled.
Related issue number

Fixes #2032

Codecov Report

Merging #2034 (e53b3e6) into main (958c49d) will decrease coverage by 31.59%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #2034       +/-   ##
===========================================
- Coverage   71.83%   40.24%   -31.60%
===========================================
  Files          89       89
  Lines       10290     9838      -452
===========================================
- Hits         7392     3959     -3433
- Misses       2898     5879     +2981

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_cython_ast.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_serailaize.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/analytical/udf/patch.py
3.47% <0.00%> (-96.53%)
:arrow_down:

python/graphscope/tests/unittest/test_lazy.py
0.00% <0.00%> (-96.30%)
:arrow_down:

...thon/graphscope/tests/unittest/test_scalability.py
0.00% <0.00%> (-96.16%)
:arrow_down:

python/graphscope/tests/unittest/test_app.py
0.00% <0.00%> (-95.66%)
:arrow_down:

...hon/graphscope/tests/unittest/test_create_graph.py
0.00% <0.00%> (-92.00%)
:arrow_down:

python/graphscope/tests/unittest/test_context.py
0.00% <0.00%> (-81.04%)
:arrow_down:

python/graphscope/analytical/udf/wrapper.py
27.27% <0.00%> (-72.73%)
:arrow_down:

... and 40 more

Continue to review full report at Codecov.

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


Add \xHH Unicode escape code to basic strings

Update tests and merge when https://github.com/toml-lang/toml/pull/796 makes it into a TOML release.

Codecov Report

Merging #202 (2b195e3) into master (7e563ee) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #202   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          507       509    +2
  Branches        99       100    +1
=========================================
+ Hits           507       509    +2

Impacted Files
Coverage Δ

src/tomli/_parser.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Ci/new dependencies mentioned in pr

A PR to debug new dependencies action inb the lint ci workflow.
Please do not merge this PR.

Codecov Report

Merging #428 (306d703) into main (a5897d5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #428   +/-   ##
=======================================
  Coverage   97.90%   97.90%
=======================================
  Files          61       61
  Lines         668      668
  Branches      106      106
=======================================
  Hits          654      654
  Misses         12       12
  Partials        2        2

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


how to use lottie components as spinner

@andfanilo assume i want to show lottie animation as a spinner. how can i delete the lottie component after the task is finished?
sample code:
import time
import streamlit as st
from streamlit_lottie import st_lottie
import requests
def load_lottieurl(url: str):
    r = requests.get(url)
    if r.status_code != 200:
        return None
    return r.json()

lottie_url = "https://assets9.lottiefiles.com/packages/lf20_wvwimamz.json"
lottie_json = load_lottieurl(lottie_url)

with st.spinner(
        text="Downloading.. "
    ):
        st_lottie(lottie_json, height=150, key="loading_gif")
        time.sleep(5)

Then the animation does not stop when spinner is finished.

Hello @fcakyon
You can wrap your lottie component inside an empty container that you clear at the end of the timer.
with st.spinner(text="Downloading.. "):
    lottie_container = st.empty()
    with lottie_container:
        st_lottie(lottie_json, height=150, key="loading_gif")
    time.sleep(5)
    lottie_container.empty() # clear container

Hope that does the trick!
Fanilo

@fcakyon
I've added a new st_lottie_spinner:
import time
import streamlit as st
from streamlit_lottie import st_lottie_spinner
import requests

def load_lottieurl(url: str):
    r = requests.get(url)
    if r.status_code != 200:
        return None
    return r.json()

lottie_url = "https://assets9.lottiefiles.com/packages/lf20_wvwimamz.json"
lottie_json = load_lottieurl(lottie_url)

with st.spinner(text="Downloading.. "):
    with st_lottie_spinner(lottie_json, height=150, key="loading_gif"):
        time.sleep(5)

Could you upgrade to streamlit-lottie 0.0.3 and tell me if it works on your side?

@fcakyon
I've added a new st_lottie_spinner:
import time
import streamlit as st
from streamlit_lottie import st_lottie_spinner
import requests

def load_lottieurl(url: str):
    r = requests.get(url)
    if r.status_code != 200:
        return None
    return r.json()

lottie_url = "https://assets9.lottiefiles.com/packages/lf20_wvwimamz.json"
lottie_json = load_lottieurl(lottie_url)

with st.spinner(text="Downloading.. "), st_lottie_spinner(lottie_json, height=150, key="loading_gif"):
    time.sleep(5)

Could you upgrade to streamlit-lottie 0.0.3 and tell me if it works on your side?

Welldone Fanilo.... great work


when SAS Token expires, please display a message saying 'Your SAS token expired, please generate a new one'

instead of this:
"Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature. RequestId:f6884515-501e-009f-41fe-505240000000 Time:2022-04-15T19:23:52.0482248Z"

@FaronFaulk it can be difficult trying to decide the various places where we should do deep investigations into error messages during runtime. Can you share with us what you were doing/where you saw this message appear?

Hi @FaronFaulk, if you'd like to use an image in your reply, you'll need to come to GitHub and reply there. Thanks.

Ok so during an explorer refresh, thank you!

We need to:

Have team discussion regarding error reporting in explorers (deficiencies with using the info bar)
Talk about checking SAS expiration in explorers
Investigate experimenting with more obvious displays of SAS expiration in the tree (e.g. changing node icons to make it more obvious there is an issue)

Is there a way to change the SAS token for an existing connection? It is nice to preserve open tabs from session to session.
Our teams allows for getting 7-day SAS tokens for a container, and not being able to change the token for and existing connection means that those saved tabs get lost. Ideally, we could just update the SAS token for an existing connection, and everything else would continue to work:

@jcookems That's a good suggestion. We can explore the possibility of renewing SAS-based connections with a new SAS token. I would consider this a separate issue, so I recommend opening a separate feature request.

Thanks @craxal, I opened https://github.com/microsoft/AzureStorageExplorer/issues/6107 to track updating SAS token. I'd expect that the solution to this issue would be some sort of popup or link to update the SAS token in-place, as described in my issue.

@FaronFaulk @jcookems We've settled on the idea of displaying a warning message in the explorer when the SAS is due to expire within 1 hour and when the SAS actually expires. Is this sufficient for your needs? Do you still want to see the authentication error message when the SAS is expired?


A limitation of metamodel should be updated

Hello,
I read: "Current limitations New relationships will always be association relationships."
Beside an Association relation, I can already create a relation of type Aggregation and Compostion as well. So I think the limitation is not valid anymore and can be removed or should state the three types (Association, Aggregation and Compostion) that are possible.
Best regards,
Henri Heijltjes
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 41ca0411-e2e8-2f7a-37c2-42039ba2d552
Version Independent ID: 2f8c8cd7-c893-55a1-2355-66eaa5c926e1
Content: Manage assets with metamodel - Microsoft Purview
Content Source: articles/purview/how-to-metamodel.md
Service: purview
Sub-service: purview-data-map
GitHub Login: @evangelinew
Microsoft Alias: evwhite

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

@HenriHeijltjes - Thank you! I'm currently looking into this.

@HenriHeijltjes - Quite right! You can also now create aggregation and composition relationships. I'm updating this. Since it's a preview feature, things are updating rapidly.
I am updating the doc to resolve this discrepancy now.

I've updated the document. Thank you for reaching out!
#please-close


GitHub ci jsonschema/v4

cc @jasonish

extends the schema
use the rust utility as it is faster than python

The schema should then be extended with turning "additionalProperties": true to false in only one remaining place (the base object)
Replaces #585

Should schema.json be part of Suricata rather than Suricata-verify in the end ?
When we modify Suricata to extend the logging, we will want to modify schema.json, right ?

But then when the suricata PR gets merged, S-V will fail until its PR gets merged.
If schema.json is in suricata, S-V should stay green

@jasonish do you know openssl name for Alma ? It is not libssl-dev

@jasonish do you know openssl name for Alma ? It is not libssl-dev

Its openssl-devel as it should be ;)

But then when the suricata PR gets merged, S-V will fail until its PR gets merged. If schema.json is in suricata, S-V should stay green

Good point.  I had never really thought of the schema being that strict.  I'm more interested in it failing when something that exists changes, rather than new stuff showing up.  Not that new stuff isn't as important, but something changing is usually a red flag.  It could probably go in qa/ or etc/, any thoughts @victorjulien ?

@satta doing this schema, I realize that for Mqtt, some common fields like qos are logged into message-specific (like publish) fields.
Could we make this generic ?

Its openssl-devel as it should be ;)

It looks to fail, or am I missing something @jasonish ?

Stil todo : MQTT and base to be completed

Stil todo : MQTT and base to be completed

jsonschema-rs-iter schema.json -i tests/*/output/eve.json | grep -v VALID | sort | uniq -c | sort -n
   2 1. Additional properties are not allowed ('dcerpc' was unexpected)
   4 1. Additional properties are not allowed ('krb5' was unexpected)
   6 1. Additional properties are not allowed ('rfb' was unexpected)
   8 1. Additional properties are not allowed ('rdp' was unexpected)
  23 1. Additional properties are not allowed ('email' was unexpected)
  25 1. Additional properties are not allowed ('dhcp' was unexpected)
  43 1. Additional properties are not allowed ('http2' was unexpected)
  60 1. Additional properties are not allowed ('modbus' was unexpected)
  86 1. Additional properties are not allowed ('dnp3' was unexpected)
  89 1. Additional properties are not allowed ('ike' was unexpected)

lua

Its openssl-devel as it should be ;)

It looks to fail, or am I missing something @jasonish ?

Odd, when I copy and paste those commands into an almalinux docker container it works fine.  When I add it as the only change it also works: https://github.com/jasonish/suricata-verify/runs/4305413036?check_suite_focus=true#step:3:132
And I don't see any specials chars in there either.

@satta doing this schema, I realize that for Mqtt, some common fields like qos are logged into message-specific (like publish) fields. Could we make this generic ?

In theory, yes, these could be moved outside the message-specific subobject, as these are indeed part of the fixed header that is common to all MQTT messages. However, in the current protocol versions they are only ever really used for PUBLISH (see https://docs.oasis-open.org/mqtt/mqtt/v5.0/os/mqtt-v5.0-os.html#_Toc3901023). I think my idea there was to not even log them for non-PUBLISH messages, but that must have been overlooked in the final code.

And I don't see any specials chars in there either.

But there was a special char for the space after the package's name, seen with my hex editor  ;-)
Copy/paste are bad for this :-/
Thanks for the tip

@satta thanks for the explanation.
I think one benefit of this schema.json is to make us think about what we want in the output logs ;-)

@jasonish this should pass now with all S-V tests.
How should we order this schema ?
I guess for each object, its properties should be order by type size : bool, integer, string, array, object, and then by name...
What is left to do here ?
Then, we can look into logs improvements such as the one with MQTT.
(for instance in DNP3, some fields are often logged in the root, but sometimes in request or response)

How should we order this schema ?

How about just alphabetically?  The only thing I'd do is group all top-level non-object types together at the top, alphabetically, then do the object types (dns, stats, http, etc) alphabetically after the top level types.

Replaced by https://github.com/OISF/suricata-verify/pull/588
Here is my dummy script to order the schema.json :
import json
from collections import OrderedDict

f = open("schema.json")
schema = json.load(f)

def reorder(s):
    if s["type"] != "object":
        return
    d1 = dict()
    d2 = dict()
    d3 = dict()
    for k in s["properties"]:
        if s["properties"][k]["type"] == "object":
            d1[k] = s["properties"][k]
        elif s["properties"][k]["type"] == "array":
            d2[k] = s["properties"][k]
        else:
            d3[k] = s["properties"][k]

    df = OrderedDict()
    for k, v in sorted(d3.items()):
        df[k] = v
    for k, v in sorted(d2.items()):
        df[k] = v
    for k, v in sorted(d1.items()):
        df[k] = v
    s["properties"] = df

def recurs(s):
    try:
        for k in s["properties"]:
            if s["properties"][k]["type"] == "array":
                recurs(s["properties"][k]["items"])
                reorder(s["properties"][k]["items"])
            elif s["properties"][k]["type"] == "object":
                recurs(s["properties"][k])
                reorder(s["properties"][k])
    except:
        pass

recurs(schema)
print(json.dumps(schema, indent=4))

Is this for https://redmine.openinfosecfoundation.org/issues/1369 ?

Is this for https://redmine.openinfosecfoundation.org/issues/1369 ?

Indeed


watch command

💡 Feature description
It seems that a "watch source" is missing for the build subcommand.
Workaround
First, install fswatch on your system; then, at the root of your project :
fswatch -o -r ./src|xargs -I {} wasm-pack build --debug

i think this sounds great! once i get 0.6.0 out the door we can schedule this for the next release

update! looking to do this soon :) cc @fitzgen https://github.com/rustwasm/rfcs/pull/10/commits/8aae141bb09eb91b50a53e74bc33bd95813b9da9

there's a draft implementation of this (and the proposed serve command from rustwasm/rfcs#10 as well) in #745, for what it's worth

@ashleygwilliams @boringcactus what happened to this? Is there still any work in progress?

I hope this comes through. I used onchange b/c I didn't want to force fellow developers to install cargo-watch, but having Watch Mode built in would be great.

Any news?


azurerm_virtual_machine_configuration_policy_assignment does not exist.

azurerm_virtual_machine_configuration_policy_assignment referenced under https://learn.microsoft.com/en-us/azure/governance/machine-configuration/how-to-create-assignment#assign-a-configuration-using-terraform does not exist.
It has been replaced with azurerm_policy_virtual_machine_configuration_assignment --> https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/policy_virtual_machine_configuration_assignment

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

ID: 6a222e8b-5335-9c8c-9bd7-74b04cdf4ba8
Version Independent ID: 1364c2ba-149c-2b0a-0b37-8abd34ae736c
Content: How to create a machine configuration assignment using templates
Content Source: articles/governance/machine-configuration/how-to-create-assignment.md
Service: machine-configuration
GitHub Login: @michaeltlombardi
Microsoft Alias: mlombardi

@JelleBroekhuijsen Thanks for your comment! We will review and provide an update as appropriate.

It looks like the change has already been made.
#please-close


[Feat] Custom MicroAgents.

Custom MicroAgents can now be added directly to the connected workspace.
End-user friendly description of the problem this fixes or functionality that this introduces
Existing Microagents are only possible when pre-declared in the OpenHands repo. We want users to be able to create their own triggers for their connected repos.

[ x] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions
In addition to existing microagents, we also setup custom microagents using the same .md format and logic. These can be declared in the ${WORKSPACE_BASE}/.openhands/microagents directory.

Link of any specific issues this addresses
#4924

@RajWorking just wanted to check if this is something still on your radar?

@RajWorking just wanted to check if this is something still on your radar?

Yes, I am working on this. I am still not exactly sure how to fix this, can you help me?

@RajWorking I made a couple more changes, but this is looking good to me!
We have a PR freeze for the next hour or so, but take a look at my changes and lmk if you have concerns. Otherwise I'll merge shortly!

The changes look good, we can merge this now!


tasks: remove windows args in system probe and security agent tasks

What does this PR do?
This PR removes all argument passing related to windows/is_windows platform and replaces everything with top level is_windows based on current running platform.
The end goal of this is to allow a future is_darwin to co-exist with this.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge


install python-dotenv before run python

Module: 02-03-04-05-06
Lab/Demo: 02-03-04-05-06
Fixes #51 .
Changes proposed in this pull request:

Update Lab2-6 python instructions as installing python-dotenv in advance

Discussion in the issue, but author indicated this is no longer a problem


Cleanup pointer casts in network and tarcap classes

Use templated getSpecificHandler method to get rid of those pointer casts in network and tarcap classes
Fixed https://github.com/Taraxa-project/taraxa-node/issues/1722

prnet cleaned-up successfully


Docs/add progressbar icons

Add the story Icon of the Progress bar component. And add svg icon for check, clock and cross

Codecov Report

Merging #403 (4537980) into main (6d4c52a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #403   +/-   ##
=======================================
  Coverage   97.90%   97.90%
=======================================
  Files          61       61
  Lines         668      668
  Branches      106      106
=======================================
  Hits          654      654
  Misses         12       12
  Partials        2        2

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Pull new versions of deis in advance?

When Deis is upgraded, most time is spent pulling new images. It would be great to have an ability to pull new images on all hosts in the cluster before issuing deisctl stop platform, so start platform would be much faster.

See http://docs.deis.io/en/latest/managing_deis/production_deployments/#preseeding-containers

Yes, like that, but this script downloads images for currently running version of Deis. Upgrade docs suggest to stop platform first and update /deis/platform/version afterwards. It means that during preseeding whole cluster will be down, like:

deisctl stop platform (apps are not responding now)
deisctl config platform set version
run preseed on each hosts (takes a while, apps are still not responding)
deisctl install && deisctl start.
What I propose is to force preseed for release to be installed before stopping old platform to minimize downtime while performing upgrade.

What I propose is to force preseed for release to be installed before stopping old platform to minimize downtime while performing upgrade.

Ah, okay. Well, you could set the platform version (deisctl config platform set version) to the new release version before running the preseed script. This config value is only checked when a component service first starts up, so as long as you don't have host failure during that time (which would cause components on the new host to use the new version), you would be fine.
You could always set the version back to the previous release until you're ready to actually perform the upgrade.

I see. So it's

set platform version to the new release
run preseed on every machine
perform platform uninstall && stop && install && start

What do you think about adding some command like deisctl preseed <version> to call preseed with version argument, in which case it could skip checks for /deis/platform/version? Then upgrade could be like (from 1.6.0 to 1.7.0)
curl -sSL http://deis.io/deisctl/install.sh | sh -s 1.7.0
export DEISCTL_TUNNEL=...
deisctl preseed 1.7.0 # fetches images on all nodes, takes a while, cluster is up
deisctl stop platform && deisctl uninstall platform
deisctl install platform && deisctl start platform # should be much faster as images are already downloaded

And, as images will already be preseeded, we won't need to do run it after installation.

I disagree that most of the time is spent pulling Docker images when upgrading (or provisioning from scratch). Unless your net connection is really slow, it's more likely to be deis-builder startup consuming most of that time, and unfortunately that won't benefit much from pre-seeding deis/builder:v1.7.0. Most of its startup time happens after that image is already resident. (And in v1.7.0 all images but deis-store-* are significantly smaller than before thanks to alpine:3.1.)
So I can see how a deisctl preseed command could help, I just don't think it would help nearly as much as you expect. And you can already accomplish the same by using the existing preseed script as @carmstrong noted.
@ineu if you were able to try the preseed script as suggested, did it make a dramatic difference? I'd be happy to guess wrong about this. :smile: If not, I'm not sure it would be worth adding something like deisctl preseed.

And in v1.7.0 all images but deis-store-* are significantly smaller than before thanks to alpine:3.1

Probably yes, I was mostly talking from my experience going through versions from 1.4, when there was ubuntu image. With tiny images it shouldn't be a problem anymore.
What bothers me is extra work happening after platform stop. I believe there was a ticket about stopping deis-routers as soon as possible, maybe it would also help a lot.

Another usecase for early preseed: trying to upgrade to 1.7.2 now. Docker pulls are extremely slow for me. So in case docker hub is down or slow, it is almost impossible to upgrade. It may also be not possible to run previous version of Deis if preseed wasn't run before, as some images may be absent.
So preseed is vital for reliable work, as unresponsive Docker Hub may break cluster any minute.

deisctl is going away in Deis v2 and I don't think we will be able to get to this for the LTS release so I'm going to go ahead and close this. Thank you for the report though!


error in path

After many attempts, I knew that the error was in the path. I mean that the methods used to obtain the extension path have a problem, and that if the path contains any space, an error occurs in using the Google extension
ex : C:\Users\unkown\Desktop\Newfolder => no problem
ex2 : C:\Users\unkown\Desktop\New  folder => problem
in ex2: new folder include a space this is make error in use Google extension

To keep the repo organized let's leave Github issues for Feature Requests, bugs and issues.
I kindly ask you to create a question on Stack Overflow. I follow the tag puppeteer-sharp. This will help us get more eyes on those questions and leave answers easily available to the community.
Feel free to paste the Stack Overflow link here.


Remove smi collector

Remove SMI collector
Signed-off-by: Johnson Shi<EMAIL_ADDRESS>

@SanyaKochhar // @shalier // @draychev - Removing SMI collector from the OSM collector PR so that they can be merged separately.


THX a lot

New Issue Checklist

[ ] I have read and understood the CONTRIBUTING guide
[ ] I have read the Documentation
[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. ios / tvos

Platform Version
e.g. 8.0

SDWebImage Version
e.g. 3.7.6

Integration Method
e.g. carthage / cocoapods / manually

Xcode Version
e.g. Xcode 7.3

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.

closing due to missing information regarding the issue you are having.


onLoopEnd is not called in iOS but is called on android

Description
For the same .riv file, onLoopEnd is called properly on android it is not called on iOS.
Use case is to store when that animation is shown to the user after animation is played and not animate again
Provide a Repro
            <Rive
              ref={animationRef}
              style={{ width: 48, height: 48 }}
              resourceName={'offers_tab'}
              onLoopEnd={onAnimationEnd}
            />

Source .riv/.rev file
Offer's tab (R) (1).rev.zip
Expected behaviour
Trigger onLoopEnd callback similar to android
Device & Versions (please complete the following information)

Device: iPhone 11
OS: 17.4.1
NPM Version: 10.2.4

Hi! Did you solve it?

@BrandonEscamilla Tried the examples from Rive website but the same callback is not getting triggered as in the android. Worked around it on iOS by using onPause and using onLoopEnd on android


Autostart didn't work for LinuxGSM tf2server on us1, us3, as1

Both servers restarted today and tf2server was not started on boot like it should. Investigate systemd.

as1 is now offline, possibly same issue

After investigating possible causes, the most likely cause was tf2server monitor
systemd indicates it started tf2server after system reboot as it should, but the log also displays tf2server shutting down about a minute later. I expect monitor bugged out and tried to restart the server while it was starting.
monitor is supposed to not do anything if the server is stopped or starting:
executing tf2server start yields:
tf2server@dane-as1:~$ ./tf2server monitor
[ ERROR ] Monitoring tf2server: Checking lockfile: No lockfile found: ERROR
* Start tf2server to run monitor.
tf2server@dane-as1:~$ ./tf2server monitor
[  OK  ] Monitoring tf2server: Checking session: OK
[  OK  ] Monitoring tf2server: Querying port: gsquery: <IP_ADDRESS>:27015 : 0/1: DELAY
tf2server@dane-as1:~$ ./tf2server monitor
[  OK  ] Monitoring tf2server: Checking session: OK
[  OK  ] Monitoring tf2server: Querying port: gsquery: <IP_ADDRESS>:27015 : 0/1: DELAY
tf2server@dane-as1:~$ ./tf2server monitor
[  OK  ] Monitoring tf2server: Checking session: OK
[  OK  ] Monitoring tf2server: Querying port: gsquery: <IP_ADDRESS>:27015 : 0/1: OK

ERROR indicates the server is stopped
DELAY indicates the server is still starting (don't query port)
OK indicates the port was queried and answered (no need to restart server)
I don't know why this happened, but if it continues to happen in the future my first idea will be to widen the cron delay on monitor (currently */1 minute). We used to have this set to */5 minute but this was changed a while ago. It's possible that changing this to */1 minute caused this to happen, although a better solution would be to prevent tf2server monitor from executing at all around restart time (maybe from 05:00 to 05:05 server time?). Closing for now but will look into a possible fix if it continues to happen.


1.11 till master may 28´th on PixRacer gyro 0 and acc 0 get wrong values

Pixracer gets strange raw values from gyro 0 and acc 0 so gyro cal always ends with “motion retrying” error.
(with 1.10 it the whole cal procedure could be finished with success)
Reducing imu_gyro_ratemax didn´t solve it.
gyro0 and acc0 disabled doesn´t help as well, the cal procedure still tries to use gyro0
Here are the sensors values from console:
sensors status
INFO [sensors] gyro status:
INFO [ecl/validation] validator: best: 1, prev best: 1, failsafe: YES (4 events)
INFO [ecl/validation] sensor #0, prio: 1, state: TOUT ECNT
INFO [ecl/validation] val: 0.4073, lp: 0.1746 mean dev: 0.0076 RMS: 1.0619 conf: 0.0000
INFO [ecl/validation] val: 1.2216, lp: -0.1350 mean dev: 0.0624 RMS: 1.2962 conf: 0.0000
INFO [ecl/validation] val: -0.2300, lp: -0.0765 mean dev: -0.0917 RMS: 1.1297 conf: 0.0000
INFO [ecl/validation] sensor #1, prio: 100, state: OK
INFO [ecl/validation] val: -0.0058, lp: 0.0028 mean dev: -0.0000 RMS: 0.0023 conf: 1.0000
INFO [ecl/validation] val: -0.0019, lp: -0.0011 mean dev: 0.0000 RMS: 0.0019 conf: 1.0000
INFO [ecl/validation] val: 0.0027, lp: 0.0015 mean dev: 0.0000 RMS: 0.0014 conf: 1.0000
INFO [sensors] accel status:
INFO [ecl/validation] validator: best: 1, prev best: 1, failsafe: YES (4 events)
INFO [ecl/validation] sensor #0, prio: 1, state: TOUT ECNT
INFO [ecl/validation] val: -0.1547, lp: 6.1306 mean dev: 0.6197 RMS: 10.1262 conf: 0.0000
INFO [ecl/validation] val: 24.8830, lp: 6.1518 mean dev: -0.2533 RMS: 9.6765 conf: 0.0000
INFO [ecl/validation] val: 13.0815, lp: 6.6939 mean dev: 0.1947 RMS: 9.7891 conf: 0.0000
INFO [ecl/validation] sensor #1, prio: 100, state: OK
INFO [ecl/validation] val: 0.2108, lp: 0.2164 mean dev: 0.0005 RMS: 0.0379 conf: 1.0000
INFO [ecl/validation] val: 0.1070, lp: 0.0857 mean dev: 0.0001 RMS: 0.0378 conf: 1.0000
INFO [ecl/validation] val: -9.5672, lp: -9.6160 mean dev: -0.0002 RMS: 0.0269 conf: 1.0000
INFO [sensors] mag status:
INFO [ecl/validation] validator: best: -1, prev best: -1, failsafe: NO (0 events)
INFO [sensors] Airspeed status:
INFO [ecl/validation] no data
INFO [vehicle_acceleration] selected sensor: 2359306 (1)
INFO [vehicle_acceleration] bias: [0.000 0.000 0.000]
INFO [vehicle_acceleration] sample rate: 799.041 Hz
INFO [vehicle_angular_velocity] selected sensor: 2359306 (1)
INFO [vehicle_angular_velocity] bias: [0.000 0.000 0.000]
INFO [vehicle_angular_velocity] sample rate: 799.041 Hz
INFO [vehicle_air_data] selected barometer: 3997714 (0)
vehicle_air_data: cycle: 12001 events, 822591us elapsed, 68.54us avg, min 20us max 622us 93.233us rms
INFO [ecl/validation] validator: best: 0, prev best: 0, failsafe: NO (0 events)
INFO [ecl/validation] sensor #0, prio: 75, state: OK
INFO [ecl/validation] val:<PHONE_NUMBER>, lp:<PHONE_NUMBER> mean dev: -0.0567 RMS: 4.3212 conf: 1.0000
INFO [ecl/validation] val: 30.7600, lp: 30.7019 mean dev: 0.0802 RMS: 0.0169 conf: 1.0000
INFO [ecl/validation] val: 0.0000, lp: 0.0000 mean dev: 0.0000 RMS: 0.0000 conf: 1.0000
nsh> listener sensor_gyro
TOPIC: sensor_gyro 2 instances
Instance 0:
sensor_gyro_s
timestamp: 399377360 (0.008741 seconds ago)
timestamp_sample: 399376921 (439 us before timestamp)
device_id: 3801098 (Type: 0x3A, SPI:1 (0x00))
x: -2.3780
y: 2.7612
z: 0.8742
temperature: 0.0000
Instance 1:
sensor_gyro_s
timestamp: 399392838 (0.004444 seconds ago)
timestamp_sample: 399392599 (239 us before timestamp)
device_id: 2359306 (Type: 0x24, SPI:1 (0x00))
x: 0.0080
y: 0.0018
z: 0.0081
temperature: 47.070

Thanks for reporting. Does that sensor (icm20608g) ever give you valid data?

Could you try this branch? https://github.com/PX4/Firmware/tree/pr-invensense_imu_reset
It monitors for consecutive failures and issues a full sensor reset if necessary.

icm20608g
in the versions before 1.11 the sensor has worked it was only often necessary to calibrate at different temperatures

It's a completely new driver using the sensor much differently. We now use the sensor's FIFO and SPI DMA to transfer raw 8 kHz data and do all the filtering PX4 side.
Can you try the branch?

currently I´m fixed to Mac OS 10.12 (Sierra) and 10.15 (Catalina) and I´m still trying to get the build procedure working above PX 4 V1.8
The gyro 0 x,y,z values are absolutely random and fluctuate very quickly between -5 and 5 accels 0 between -50 and 50

I'll open a PR and point you to a binary.

Got the build procedure in 10.12 working that branch behaves identical:
or can I manually do this reset?
temperature of accel 0 doesn´t read as well.
listener sensor_accel
TOPIC: sensor_accel 2 instances
Instance 0:
sensor_accel_s
timestamp: 145735613  (0.009395 seconds ago)
timestamp_sample: 145735160  (453 us before timestamp)
device_id: 3801098 (Type: 0x3A, SPI:1 (0x00))
x: -20.7062
y: -12.1798
z: -17.7917
temperature: 0.0000
Instance 1:
sensor_accel_s
timestamp: 145752130  (0.004013 seconds ago)
timestamp_sample: 145751594  (536 us before timestamp)
device_id: 2359306 (Type: 0x24, SPI:1 (0x00))
x: -0.2159
y: 0.1276
z: -9.7805
temperature: 40.6004

This board had two magnetometers till v 1.10.1 now only one of them is found.
(Tried an alternative GCC toolchain 9.2.1, then the random gyro 0 values get a bit lower.)
I understand all advantages of the faster driver structure but as long as some imu sensors give strange values they should be able to ignored like the mag otherwise the cal procedure keeps failing.

Now I got the chance to test another PixRacer unit R14 the previous was a very old R12
With master 1.11 it worked few minutes and afterwards the same strange values on gyro 0 accel 0 shown as on R12 from start.
With pr-invensense_imu_reset branch (build works only with GCC 7.2.1) it works but shows failsafe events in regular timeframes.
One of the mags is not found anymore same as on R14 board.
i would like to do a flight test, what can happen if such an imu reset is performed in flight?

@dagar Have you taken the sensor reset on that board away on boot? There was a required sensor reset (resetting the sensor power supply, NOT a register reset) being done on boot because some of these boards boot with browned out sensors.
Almost all of the "quirks" in software boot on PX4 are there for a reason to deal with broken hardware, so it's important to not to try to be clever with "cleanups".

This board had two magnetometers till v 1.10.1 now only one of them is found.

The pixracer has 2 internal mags, a dedicated mag (hmc5883 or lis3mdl) plus the ak8963 mag included in the mpu9250. The mpu9250 mag is now disabled by default because the internal i2c slave disrupts the scheduling (which we can see now), and it's also low quality.

i would like to do a flight test, what can happen if such an imu reset is performed in flight?

It will switch over to the secondary IMU.

Now I got the chance to test another PixRacer unit R14 the previous was a very old R12
With master 1.11 it worked few minutes but afterwards there are the same strange values on gyro 0 and accel 0 shown as on R12 from start.

@taileron anything else you can share about your usage? How are you powering these boards? Can you verify which have the icm20608g (older pixracers) and which have the icm20602? It you share a log from boot (SDLOG_MODE 1) I'll be able to tell.

@dagar Have you taken the sensor reset on that board away on boot? There was a required sensor reset (resetting the sensor power supply, NOT a register reset) being done on boot because some of these boards boot with browned out sensors.

@LorenzMeier no definitely not, and in fact I wanted to expand that mechanism so it can be more tightly integrated with actual sensor init. It's worth double checking it's working as intended.

Almost all of the "quirks" in software boot on PX4 are there for a reason to deal with broken hardware, so it's important to not to try to be clever with "cleanups".

These checks are evolving to catch more subtle failures that show up later in usage, not as a band-aid for bad hardware. For example this issue with the icm20689 in all CUAV fmu-v5 hardware (https://github.com/PX4/Firmware/issues/13083) that's plagued us long before the new drivers came into place. We want to try and recover if possible, but properly flagging and propagating errors from various sources (eg FIFO data integrity checks) is just as important.

@taileron I've tried about 5 random pixracers I have on hand, but so far haven't reproduced any failures or bogus data. Most are mRo pixracers < 1 year, but 1 is an old AUAV pixracer with the icm20608g. I'll let them run a bit longer.
Any thing else you can share about your setup? What else is connected? Using a power module, USB power, or both?

I've found an older board with an icm20608g that's able to reproduce the problem (after running continuously for quite a while). Debugging...

@taileron could you give https://github.com/PX4/Firmware/pull/15004 a try?
Binary: https://github.com/PX4/Firmware/suites/739248622/artifacts/7483749
https://github.com/PX4/Firmware/actions/runs/120856803

the old R12 has a 20608g as well. #15004 doesn't change the behavior of imu 0 but sometimes emu 1 fails too. the FC are connected via USB but via wifi and power module it´s identical.
Sometimes there are 70 failsafe events around the start procedure.
https://review.px4.io/plot_app?log=8fbbe75b-7ef2-44b4-8f0f-92b4838c93ee
Imu rate max = 400
https://review.px4.io/plot_app?log=324c18ec-09a6-400c-8ed2-64adb6e875e8
imu rate max = 0 (789.7Hz)
The R14 currently works with emu_reset branch

I can confirm the icm20608g data is still bad in your R12 logs and now looks good in the R14. What's noteworthy is that it's bad initially, and not the same as what I thought might be a similar problem with the ICM20689 going bad later.
Next for the R12 I'll look at implementing the icm20608g self test procedure.

is there a trick to edit "extras.txt" over mavlink, cat /fs/microsd/etc/extras.txt only shows it ?
Not that I'm aware of, but I haven't tried it. If this isn't working please open a separate issue and we'll take a look at what's missing in the Mavlink FTP code.

Glued Pixracer R14 board now flies great (mission still needs a bit of tuning to avoid oscillations in strong wind) missing entries could be solved within parameters:
https://review.px4.io/plot_app?log=fc50db22-3070-4083-9090-a0a0d62d257f
How can I proceed further to get V1.11 running on the R12 board, which is my only one that is not damaged by the makeshift repair.

Could you please try the current https://github.com/PX4/Firmware/pull/15004 again? I believe I found the problem with older ICM20608g units.

works now .... calibration finished without errors
listener sensor_gyro
TOPIC: sensor_gyro 2 instances
Instance 0:
sensor_gyro_s
timestamp: 489968009  (0.009620 seconds ago)
timestamp_sample: 489967769  (240 us before timestamp)
device_id: 3801098 (Type: 0x3A, SPI:1 (0x00))
x: -0.0038
y: 0.0057
z: -0.0024
temperature: 53.9841
error_count: 0
Instance 1:
sensor_gyro_s
timestamp: 489987386  (0.009146 seconds ago)
timestamp_sample: 489987152  (234 us before timestamp)
device_id: 2359306 (Type: 0x24, SPI:1 (0x00))
x: -0.0109
y: -0.0141
z: -0.0040
temperature: 50.2329
error_count: 0

#15004 works now on R12 .... calibration finished without errors - are there further test needed?

Nothing further here, but please continue testing master and reporting issues as we close in on a v1.11.0 release.
Thanks for the help debugging this.