Cluster member with CIS admin role is NOT able to list clusterscans

On 2.5-head - commit- 395b4fe77

As an admin deploy CIS v2, create a scan CR
Create a user - user-1 with cis-admin roles using

apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: clusterrolebinding-new
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: cis-admin
subjects:
- apiGroup: rbac.authorization.k8s.io
  kind: User
  name: u-pglzp

Login as user-1 --> Dashboard --> CIS benchmarks

API response lists 1 scan, but no scans seen on the UI

Response:

{"type":"collection","links":{"self":"https://rancher-server/k8s/clusters/c-hlsw7/v1/cis.cattle.io.clusterscans"},"createTypes":{"cis.cattle.io.clusterscan":"https://rancher-server/k8s/clusters/c-hlsw7/v1/cis.cattle.io.clusterscans"},"actions":{},"resourceType":"cis.cattle.io.clusterscan","revision":"99211","data":[{"id":"scan-cbjfv","type":"cis.cattle.io.clusterscan","links":{"remove":"https://rancher-server/k8s/clusters/c-hlsw7/v1/cis.cattle.io.clusterscans/scan-cbjfv","self":"https://rancher-server/k8s/clusters/c-hlsw7/v1/cis.cattle.io.clusterscans/scan-cbjfv","update":"https://rancher-server/k8s/clusters/c-hlsw7/v1/cis.cattle.io.clusterscans/scan-cbjfv","view":"https://rancher-server/k8s/clusters/c-hlsw7/apis/cis.cattle.io/v1/clusterscans/scan-cbjfv"},"_type":"cis.cattle.io.clusterscan","apiVersion":"cis.cattle.io/v1","kind":"ClusterScan","metadata":{"creationTimestamp":"2020-12-08T13:06:51Z","fields":["scan-cbjfv",null,null,null,null,null,null,null,"",null],"generateName":"scan-","generation":1,"managedFields":[{"apiVersion":"cis.cattle.io/v1","fieldsType":"FieldsV1","fieldsV1":{"f:metadata":{"f:generateName":{}},"f:spec":{".":{},"f:scanProfileName":{},"f:scheduledScanConfig":{".":{},"f:retentionCount":{},"f:scanAlertRule":{}},"f:scoreWarning":{}}},"manager":"agent","operation":"Update","time":"2020-12-08T13:06:51Z"},{"apiVersion":"cis.cattle.io/v1","fieldsType":"FieldsV1","fieldsV1":{"f:status":{".":{},"f:NextScanAt":{},"f:ScanAlertingRuleName":{},"f:conditions":{},"f:display":{".":{},"f:error":{},"f:message":{},"f:state":{},"f:transitioning":{}},"f:lastRunTimestamp":{},"f:observedGeneration":{}}},"manager":"cis-operator","operation":"Update","time":"2020-12-08T13:06:51Z"}],"name":"scan-cbjfv","relationships":null,"resourceVersion":"97750","selfLink":"/apis/cis.cattle.io/v1/clusterscans/scan-cbjfv","state":{"error":false,"message":"Scan is Pending, Waiting for another scan to finish","name":"pending","transitioning":true},"uid":"25407b44-9826-46a1-aea5-2e7ffaeebef3"},"metadata.annotations":{},"metadata.labels":{},"spec":{"scanProfileName":"rke-profile-permissive-1.6","scheduledScanConfig":{"retentionCount":3,"scanAlertRule":{}},"scoreWarning":"pass"},"status":{"NextScanAt":"","ScanAlertingRuleName":"","conditions":[{"error":false,"lastUpdateTime":"2020-12-08T13:06:51Z","message":"ClusterScan run pending","status":"True","transitioning":false,"type":"Pending"}],"display":{"error":false,"message":"Scan is Pending, Waiting for another scan to finish","state":"pending","transitioning":true},"lastRunTimestamp":"","observedGeneration":0}}]}

Updated this page so the listing should work, but determining whether or not to show the 'warn' column requires looking at the cis install config, which this role doesn't seem to have permission to do.

@prachidamle should a user with cis-admin have access to "cis install configs" ?

@mantis-toboggan-md what is the resource that you look for cis install config? @sowmyav27 yes we need to add that resource to access

@prachidamle I'm using the catalog.cattle.io.app rancher-cis-benchmark

Waiting for this issue to be fixed

@mantis-toboggan-md I do not understand why is access to install-config needed? On a 2.5-head with a user having cis-admin role I can see the "warn" column filled. For any scan that was run prior to upgrade to 2.5-head, there wont be any warn value in the clusterscan object and so it is empty.
@sowmyav27 @mantis-toboggan-md I do not think cis-admin need to have this access to install configs.

@prachidamle I was using the install config to check if the version of CIS installed supported 'warn' state tests and add the column to the table accordingly; I updated (https://github.com/rancher/dashboard/pull/2065) the table to check the scan schema instead so access to install configs isn't necessary for this to work correctly now.

On 2.5-head - commit id: 182e3de71

CIS v 1.0.3, security scan - v0.2.2-rc4
As an admin deploy CIS v2, create a scan CR
Create a user - user-1 with cis-admin roles using (cluster owner and cluster memebr)
Login as user-1 --> Dashboard --> CIS benchmarks
User is able to list/create scans


Allow for sharing single relay

Allow for using a single multi-channel relay, with one channel for power and one for other things such as controlling a boot switch/jumper on the DUT.
Configurable with env vars:
POWER_RELAY_NUM
DIGITAL_RELAY_NUM
If not specified it defaults to 0 which toggles all channels
Change-type: patch

@balena-ci I self-certify!


3452_bugfix_set_search_input_on_lastserver_refresh

https://github.com/boxture/oms/issues/3452

URL to fetch dropdown list returns filter collection instead of single item, from the list matching value is selected and set as search input value. page size and page limits the number items (instead of the collection it returns first item) so commenting this .


HDR-supported Print Screen

Description of the new feature / enhancement
It would be great if I was able to continue using Print Screen as usual with a HDR display, meaning the HDR image gets properly tone mapped to SDR for easy sharing.
Scenario when this would be used?
This would be useful on a daily basis and make it much easier to share screenshots with my team or create and share previews for the public.
With this feature I wouldn't need to 1. run any additional software like Xbox Game Bar, or 2. disable HDR every time I want to take a screenshot.
Supporting information
No response

@MicaLovesKPOP best spot for this would be Windows Feedback Hub as this would be the best spot to get this behavior corrected


readdir throws an error when withFileTypes is true

    await zenfs.configure({
      mounts: {
        "/user": {
          backend: WebAccess,
          handle: await navigator.storage.getDirectory(),
          disableAsyncCache: true,
        },
      },
    });
    console.log(await zenfs.promises.readdir("/user/Path of Building/Builds", { withFileTypes: true }));

This throws error.
Error: File is not a directory
    at _ErrnoError.With (chunk-LLIBHIR3.js?v=15a94a03:8481:12)
    at WebAccessFS.getHandle (@zenfs_dom.js?v=15a94a03:227:26)
    at async WebAccessFS.stat (@zenfs_dom.js?v=15a94a03:154:20)
    at async stat (chunk-LLIBHIR3.js?v=15a94a03:13198:19)
    at async Object.readdir (chunk-LLIBHIR3.js?v=15a94a03:13340:118)
    at async DriverWorker.start (worker.ts?worker_file&type=module:94:21)

Since entry is already a relative path, join seems unnecessary.
-        values.push(typeof options == 'object' && options?.withFileTypes ? new Dirent(entry, await stat(join(path, entry))) : entry);
+        values.push(typeof options == 'object' && options?.withFileTypes ? new Dirent(entry, await stat(entry)) : entry);

entries should be the list of file names, not full paths. Looking at WebAccess.readdir, it should return the file names. dom:08c3592 fixes this. See @zenfs/dom v0.2.11

@atty303 I feel bad having such buggy code... Please tell me if the DOM release fixes the issue. Everything looks to be working correctly in my testing.

Everything works well now. Thanks!


feat: Adapt skore to be compliant on python >=3.9, <3.14

How hard is it to support 3.13?

How hard is it to support 3.13?

Vega is not compatible https://github.com/vega/altair/issues/3620 😢 .


[MBO-51] Clear catalog cache when user authenticate or logout. Inject services for controllers

Questions
Answers

Branch?
3.x

Description?
When a user logs in or logs out, the modules catalog cache is cleaned because results are dependent to the connected user. We also inject services to controllers instead of using the container inside the controllers themselves

Type?
improvement

BC breaks?
yes

Deprecations?
no

Fixes
#MBO-51

How to test?
-

Congrats & thanks a lot for the rework with injected dependances !


Add transform function to TXT

To make it easy to edit or reject a transaction based on the current state of TXT a transform function has been added where the second parameter is a function that will return data from the TXT.
At the moment this PR is only for the technical aspect. When you are OK with how it works I will update the documentation

This change is

Sorry for a long turnaround here. Will get back to this at the end of the week.


Clerk components are not rendering on Remix Run + Vite

Preliminary Checks

[X] I have reviewed the documentation: https://clerk.com/docs

[X] I have searched for existing issues: https://github.com/clerk/javascript/issues

[X] I have not already reached out to Clerk support via email or Discord (if you have, no need to open an issue here)

[X] This issue is not a question, general help request, or anything other than a bug report directly related to Clerk. Please ask questions in our Discord community: https://clerk.com/discord.

Reproduction / Replay Link
https://github.com/rodolphoasb/clerk-error
Publishable key
pk_test_b2JsaWdpbmctZ3VsbC00Ny5jbGVyay5hY2NvdW50cy5kZXYk
Description
I think the title says it all. I'm importing and using the component but they just are not rendering.

Environment
System:
    OS: macOS 13.3.1
    CPU: (8) arm64 Apple M1
    Memory: 165.69 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
    Yarn: 1.22.15 - /opt/homebrew/bin/yarn
    npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
    bun: 1.0.3 - ~/.bun/bin/bun
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 120.0.6099.234
    Safari: 16.4
  npmPackages:
    @clerk/remix: ^3.1.16 => 3.1.16
    @flydotio/dockerfile: ^0.5.0 => 0.5.0
    @playwright/test: ^1.41.0 => 1.41.0
    @remix-run/css-bundle: 2.5.1 => 2.5.1
    @remix-run/dev: 2.5.1 => 2.5.1
    @remix-run/eslint-config: 2.5.1 => 2.5.1
    @remix-run/express: 2.5.1 => 2.5.1
    @remix-run/node: 2.5.1 => 2.5.1
    @remix-run/react: 2.5.1 => 2.5.1
    @remix-run/testing: 2.5.1 => 2.5.1
    @testing-library/jest-dom: ^6.2.0 => 6.2.0
    @testing-library/react: ^14.1.2 => 14.1.2
    @testing-library/user-event: ^14.5.2 => 14.5.2
    @types/eslint: ^8.56.2 => 8.56.2
    @types/express: ^4.17.21 => 4.17.21
    @types/morgan: ^1.9.9 => 1.9.9
    @types/react: ^18.2.48 => 18.2.48
    @types/react-dom: ^18.2.18 => 18.2.18
    @vitejs/plugin-react: ^4.2.1 => 4.2.1
    @vitest/coverage-v8: ^1.2.1 => 1.2.1
    all-contributors-cli: ^6.26.1 => 6.26.1
    autoprefixer: ^10.4.17 => 10.4.17
    chalk: ^5.3.0 => 5.3.0
    cross-env: ^7.0.3 => 7.0.3
    dotenv: ^16.3.1 => 16.3.1
    eslint: ^8.56.0 => 8.56.0
    eslint-config-prettier: ^9.1.0 => 9.1.0
    eslint-plugin-simple-import-sort: ^10.0.0 => 10.0.0
    express: ^4.18.2 => 4.18.2
    inngest: ^3.11.0 => 3.11.0
    isbot: ^4.4.0 => 4.4.0
    jsdom: ^23.2.0 => 23.2.0
    morgan: ^1.10.0 => 1.10.0
    npm-run-all: ^4.1.5 => 4.1.5
    postcss: ^8.4.33 => 8.4.33
    prettier: ^3.2.4 => 3.2.4
    prettier-plugin-tailwindcss: ^0.5.11 => 0.5.11
    react: 18.3.0-canary-b30030471-20240117 => 18.3.0-canary-b30030471-20240117
    react-dom: 18.3.0-canary-b30030471-20240117 => 18.3.0-canary-b30030471-20240117
    remix-development-tools: ^3.7.2 => 3.7.2
    remix-flat-routes: ^0.6.4 => 0.6.4
    remix-typedjson: ^0.4.1 => 0.4.1
    rimraf: ^5.0.5 => 5.0.5
    tailwindcss: ^3.4.1 => 3.4.1
    typescript: 5.1.6 => 5.1.6
    vite: ^5.0.11 => 5.0.11
    vite-tsconfig-paths: ^4.3.1 => 4.3.1
    vitest: ^1.2.1 => 1.2.1

I'm facing the same issue.

Hi @rodolphoasb. I tested your repo and when I just cloned it, added my own keys (as a secret key is also needed) and ran the app I could see components not rendering. I also noticed that window.Clerk returned null in the dev console, indicating Clerk hadn't loaded.
The app looked fine. I deleted node_modules and the lock file, then ran pnpm up to install and upgrade packages and then pnpm dev to run the app. This resulted in the components rendering and Clerk loading.
Can you test this and see if it works for you? Can you also just update all packages in this repo and see if that helps with using npm.

@nsb reported that changes to their application resolved this issue, and there is no bug. With no reply from the OP at the this time, I'm going to close this Issue.

Hello, I have the same issue. By looking at the terminal, I see these errors if it may help.

@Kfnig24 it seems like this is related to the optimized-deps plugin. Could you check if this resolves your issue: https://github.com/vitejs/vite/discussions/8749#discussioncomment-4187108

It still doesn't work. It seems to have a problem with react, have an idea why ?

@Kfnig24 Could you share a repo that reproduces the issue to investigate it?

Here a repo: https://github.com/Kfnig24/remix-vite-error.
I let the routes:

/auth/sign-in
/auth/sign-up
/dashboard

Sorry, I can't publish the rest of the code.


Use vgo instead of dep

This PR moves from the vendor system dep to go modules (vgo) https://github.com/golang/go/wiki/Modules#quick-start

We probably want to stick to what kubectl does (not vgo).


Case insensitive searching and indexing in UI and API

Addresses #2180

@pkdash Test was added.

+1


Major issue: not closing temporary file

I finally noticed that you missed close() after an uptime of one week:
ERROR:uwsgi_file__***:Exception on / [POST]
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/var/www/cgi-bin/***/webhooks/github.py", line 52, in index
    with open(join(path, 'github/config.json'), 'r') as cfg:

Ouch.
This was determined by this:
# sh -c 'readlink /proc/30756/fd/*'
/tmp/tmpjCxknv (deleted)
/tmp/tmp5p0JBV (deleted)
/tmp/tmp7W680K (deleted)
/tmp/tmpLp_FMI (deleted)

Wait, seems this is caused by PR #3. I will review that.

Hmm... Hello Python beginner, mkstemp() returns opened file descriptor. Thanks.

Tip: I have already resolved this by fdopen().

@ishitatsuyuki
Thanks for the report. I'll would be better if you had added a better description, relevant traceback or explained the situation at all. I'm pushing the correct way:
# Save payload to temporal file
osfd, tmpfile = mkstemp()
with fdopen(osfd, 'w') as pf:
    pf.write(dumps(payload))

If you have any other comment you're welcome to submit them.


Fixing npm error when building a contributor-site docker image

Hello.
Trying to follow instructions and build a docker image to run contributor website locally.
Getting below error from npm:

make container-image
docker build . -t k8s-contrib-site-hugo --build-arg HUGO_VERSION=0.69.2
Sending build context to Docker daemon  25.65MB
Step 1/8 : FROM alpine:latest
---> 14119a10abf4
Step 2/8 : ARG HUGO_VERSION=0.69.2
---> Using cache
---> 9f0b486ae2a6
Step 3/8 : RUN apk add --no-cache     bash     build-base     curl     git     grep     libc6-compat     rsync     sed     npm
---> Using cache
---> 0d785ca3bfea
Step 4/8 : RUN npm install -G     autoprefixer     postcss-cli
---> Running in 46980f105b5d
npm ERR! Tracker "idealTree" already exists
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2021-09-29T18_08_36_087Z-debug.log
The command '/bin/sh -c npm install -G     autoprefixer     postcss-cli' returned a non-zero code: 1
make: *** [Makefile:56: container-image] Error 1

According to StackOverflow: https://stackoverflow.com/questions/57534295/npm-err-tracker-idealtree-already-exists-while-creating-the-docker-image-for all we have to do is specify WORKDIR before running npm install:
This issue is happening due to changes in nodejs starting version 15. When no WORKDIR is specified, npm install is executed in the root directory of the container, which is resulting in this error. Executing the npm install in a project directory of the container specified by WORKDIR resolves the issue.
I believe it's a trivial fix, please, let me know if you'd like me to open an issue first.

/assign @parispittman


[AzureMonitorExporter] replace ConcurrencyBag with List in all tests

Background
When we started this project, we were suffering some concurrency bugs in another project so we started here with ConcurrencyBag. This project doesn't have the same issues, so this is no longer needed.
Changes

replace ConcurrencyBag in all tests with List

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


multiple once.('destroy' events from scene plugins

plugins 'destroy' on 'scene'.events are being multiplied every time a scene 'start'.
such event should only be needed to be added once in the the initialise function or in the re-added boot event maybe?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Yeah, we could change it to happen in either the boot method or the constructor. I actually think this is a bug in the EventEmitter library though. The function is identical in each call, the event name is the same and it's flagged as once, it should never get added again.

yep, adding something to eventemitter3 will create a event however it is, the library doesn't check if such call is already on the emitter or not it simple pushes it into it. that also causes issues when using scene.events in personal code if you are not careful in removing each event on scene shutdown, since it will keep multiplying every time create() is called (if one uses the same scene multiple times). this only happens to scene.events tho, since that is the only one that removeAllListeners() can't be called once shutdown is triggered.

I’m quite tempted to just take eventemitter3 and tweak it slightly, it’s small enough and not often updated so would be easy to maintain.
I’m also thinking about making System extend EE and have that for system and plugin events only, then leave scene.events for just user events, no internal ones, making it safe to clear entirely on shutdown.
It’s good practise for devs to tidy up events anyway, it’s not like the dom does it for you, so I don’t mind forcing them to do it, but mixing system with user events is potentially dangerous, so keeping them apart might be wise. Will think it over.

In my opinion, eventemitter3's behavior is correct. If I call .once(... multiple times, same event/callback or not, it should fire it multiple times.
Tweaking this behavior to only ever call the same listener once per event would not only add additional overhead to the core, but also possibly cause unexpected situations for developers.
Cleaning up after yourself is always to be expected.

I agree with antriel, EE design choice is alright. i don't think tweaking it would be worth it. one should be careful anyway. I only mentioned it because the behaviour used on the event plugin class is differently from the other classes extended from EE, those all call removeAll on shutdown, but events can't because it is being used by the plugins, due to seeing that behaviour before i knew why the 'destroy' event was being multiplied the moment i saw it.
Personally after understanding this i had the same idea Rich had, to separate internal and user events, so i did that on my end. I think it was more logical having systems being/having a separated instance of EE (so i left scene.events alone) and that is for now emitting (update, wake, sleep, etc) events from another instance of EE (i made scene extend form EE, so it is like the same as all the others plugins). With that I can be lazy and add one line to remove all the events that i added in the respective scene instance.
Since i have it already tweaked it like that in my end, I like Rich proposal a lot.
I do wonder tho why ScenePlugin itself isn't extended from EE like all the other gameobjs and plugins are (at least the ones that have events emitting capability) and rather have it as a separated plugin that must be used anyway or things simple wouldn't work. Is't it better having scene.on('update', since that follows the same pattern was input.on('keydown', sprite.on('drag', etc? and then have systems have or be another instance of EE to deal with the internals one that including emitting the respective events on the scene? I don't know if it is only me, but having scene.events.on felt like an eyesore since it didn't follow the 'recipe' like the rest, so i ended changing it on my end, but since i had been careful the change wasn't worth of a PR, it is too core for me to dare to change deeper, but since Rich is considering on separating it I finally have a change to point this out. I hope this makes sense.

I'm not interested in making the Scene itself extend EE.  The whole point of the Scene in v3 was that it didn't have anything required in there other than sys, and making it extend EE would go completely against that. If a user wants a Scene to extend EE it's trivial for them to do it themselves.
Which just leaves the Scene Systems instance of EE. My suggestion was simply that Systems itself extends EE, and that is what all the plugins use for internal chatter, and that Systems.events (which is the one that gets mapped into the Scene as a property) is just empty and not touched internally, other than to clear it when a Scene shuts down.
However, this does mean there are cases where we'd have to dispatch the same event twice. transitionstart for example is needed internally and also useful externally, but if we're isolating the two it would need sending down both channels. The same goes for most events the ScenePlugin emits actually.
Which makes me think separating them doesn't actually solve anything much, it just doubles the workload of every Scene. I will, however, resolve the annoying double-once issue.


make Go tests forward-compatible

When mocking servers, we generate all methods to satisfy the server
interface. On the other hand, adding more methods to services is
considered non-breaking change in protobuf.
This breaks the existing tests though, since the newly added method
won't be implemented.
This change fixes the problem by embedding a nil server interface
in the mock. This way the mock always implements the interface,
even though some methods might not actually be implemented.
The methods on both the client and the mock will just be added
the next time the generator is run.

Current coverage is 82.27% (diff: 100%)

Merging #862 into master will not change coverage

@@             master       #862   diff @@
==========================================
  Files           327        327
  Lines         12559      12559
  Methods           0          0
  Messages          0          0
  Branches       1649       1649
==========================================
  Hits          10333      10333
  Misses         1797       1797
  Partials        429        429

Powered by Codecov. Last update 157e074...b4e01c1


Choose between network type (4G hotspot, WI-FI ...)

Hi,
I read on the Shift Project report that there are not the same data footprint if you are using mobile network or WI-FI network (model 1byte).
Sometimes people use their computer with a mobile network by the hostpot feature. it would be great to have the possibility to change the type of network, even manually, since the difference between WIFI and mobile is about *3 according to the 1byte model :)
thanks for your work !

In some cases we can perfectly do it automatically.
Except firefox edge safari opera IE desktop


Some trader assorts unlocked via quests have placeholder prices

There's a number of items that have a price of 6969 roubles, these are inserted by spt to ensure the assorts exist when we don't have any data on them.
If you spot any and can provide evidence via videos/screenshots of their cost/ buy limits on live. We can add them in.

Originally written by chomp

Missing values
Prapor
Level 3
62e7e7bbe6da9612f743f1e0 (GP-25 "Kostyor" 40mm underbarrel grenade launcher)
Unlocked via: Best Job in the World
Prapor
Level 3
5656eb674bdc2d35148b457c (40mm VOG-25 grenade)
Unlocked via: Best Job in the World
Jaeger
Level 4
5fc22d7c187fea44d52eda44 (SWORD International Mk-18 .338 LM marksman rifle)
Unlocked via: Hunter
Jaeger
Level 4
63920105a83e15700a00f168 (SOG Voodoo Hawk tactical tomahawk)
Unlocked via: Slaughterhouse

Originally written by chomp

Therapist
Propitol
Level 2
Quest: Ambulances Again

Originally written by chomp

Mechanic
6389c7f115805221fb410466 (Far-forward GPS Signal Amplifier Unit)
Level 4
Quest:<PHONE_NUMBER>13f06a7c3b2377 (Following the Bread Crumbs)

Originally written by chomp

Jaeger
63fc44e2429a8a166c7f61e6 Armasight Zeus-Pro 640 2-8x50 30Hz thermal scope
Level 4
Quest: 64e7b971f9d6fa49d6769b44 Huntsman Path Big Game

Originally written by chomp


Feature: Option to hide Xorg's STDOUT and STDERR / redirect it to /dev/null

see Title. This bother my otherwise silent boot and shutdown

I solved it for me by changing config.ini to have the line:
# xorg server command
x_cmd = /usr/bin/X > /dev/null 2>&1

I solved it for me by changing config.ini to have the line:
# xorg server command
x_cmd = /usr/bin/X > /dev/null 2>&1

i forked it and made a change to the source code to achieve this in the meantime.
i think your solution is better cus i would need to recompile it to revert it

i forked it and made a change to the source code to achieve this in the meantime.
i think your solution is better cus i would need to recompile it to revert it

I have this in config and it still shows errors at shutdown. Please help.

I was having similar issues with my setup.  Redirecting to /dev/null in .xinitrc worked for startup but I was still getting Xsession messages during shutdown. Found out that xsetup.sh has an echo statement before sourcing Xsession scripts. Do we really need to print this info ?

I solved it for me by changing config.ini to have the line:
# xorg server command
x_cmd = /usr/bin/X > /dev/null 2>&1

even after doing this I'm still getting the default Xorg logs on login.

I was having similar issues with my setup. Redirecting to /dev/null in .xinitrc worked for startup but I was still getting Xsession messages during shutdown. Found out that xsetup.sh has an echo statement before sourcing Xsession scripts. Do we really need to print this info ?

would you mind elaborating on how you did that? haven't even managed to suppress the startup logs, let alone the shutdown ones


HacktoberFest_entry++

another entry for hacktoberfest

Thanks for the entry!

Thank you for considering!
Ajitesh Gupta
IEC2016502
On 26-Oct-2017 19:08, "Allen Computer Science Club" <
<EMAIL_ADDRESS>wrote:

Merged #298 https://github.com/AllenCompSci/Hacktoberfest/pull/298.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AllenCompSci/Hacktoberfest/pull/298#event-1311972825,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT78KQVQgBOuoxu47bFoMqK1ItvyQYHRks5swIs3gaJpZM4QHlP1
.


[kmac] ERR_CODE not cleared

@ballifatih found that the KMAC IP's ERR_CODE is read-only in #14637 . So that the error code cannot be cleared.
The suggestion is either SW writable or err_processed may clear the register.

ah that sounds like a good idea.
@cindychip

I was thinking RO feature of ERR_CODE maybe is not the problem here.
An alternative suggestion might be to correctly update the event_error so that entropy_err.valid changes do trigger an event (in the top-level module kmac.sv). If I am not mistaken, the datapath and the FSM parts are correctly implemented for clearing ERR_CODE, what is missing is the correct implementation of the enable signal for the register.

I was thinking RO feature of ERR_CODE maybe is not the problem here.
An alternative suggestion might be to correctly update the event_error so that entropy_err.valid changes do trigger an event (in the top-level module kmac.sv). If I am not mistaken, the datapath and the FSM parts are correctly implemented for clearing ERR_CODE, what is missing is the correct implementation of the enable signal for the register.

Yep. Clearing the ERR_CODE is missing in the design. So, either SW is allowed to clear the logic or err_processed to clear the code can be a solution. I think latter is simpler.

How about the other sources of error? For example, if sha3 core generates an error (ErrSha3SwControl), is it possible to clear this error by using err_processed?. In the case of sha3 core, it looks as if err_processed is not even used to reset its internal FSM (I'm not sure about this though).

How about the other sources of error? For example, if sha3 core generates an error (ErrSha3SwControl), is it possible to clear this error by using err_processed?. In the case of sha3 core, it looks as if err_processed is not even used to reset its internal FSM (I'm not sure about this though).

I was saying how to clear the ERR_CODE not the actual error handling. I agree KMAC does not have concrete error scenarios yet. But eventual goal is err_processed being indicator of the error handling completion to HW.

Thanks for filing this issue! Sorry I am late to the discussion.
Just so I can catch up - I guess the current design will do the following if an error happened:
1). Trigger an error interrupt.
2). Update err_code.
But the issue here is that when next operation does not have an error, the err_code still shows the previous error code.
If that is the case, I have two questions:
1). Do you think it is okay to leave the err_code as it is an enhance the documentation instead?
If the following operation is successful, can SW just check no kmac_err interrupt and make sure kmac_done interrupt is triggered? Then SW do not need to check err_code because it means the current operation finished without an error?
2). If we decide to enhance the error_code logic, I personally prefer to change the err_code register from "RO" to "RW1C" (or something with similar function)? I think like Fatih mentioned above, err_processed gives me an impression that we can still finish the kmac operation despite the current error condition.

If that is the case, I have two questions: 1). Do you think it is okay to leave the err_code as it is an enhance the documentation instead? If the following operation is successful, can SW just check no kmac_err interrupt and make sure kmac_done interrupt is triggered? Then SW do not need to check err_code because it means the current operation finished without an error?

I guess it is also possible to use the interrupts to determine whether the operation completed without an error. And if we choose to leave the implementation intact and update the documentation, then ERR_CODE refers to the error code of the last error event. So, it will be updated from one error to another, instead of being updated for every KMAC call.
On the other hand, from the driver implementation/software perspective, checking ERR_CODE is easier than handling interrupts.

2). If we decide to enhance the error_code logic, I personally prefer to change the err_code register from "RO" to "RW1C" (or something with similar function)? I think like Fatih mentioned above, err_processed gives me an impression that we can still finish the kmac operation despite the current error condition.

"RO" to "RW1C" sounds like a neat solution. Should we be concerned if it has any security implications, e.g. like an attacker suppressing the time-out error by clearing the ERR_CODE register?

Maybe another alternative is to clear ERR_CODE at the beginning of the next KMAC call, but this is probably not the best idea.

2). If we decide to enhance the error_code logic, I personally prefer to change the err_code register from "RO" to "RW1C" (or something with similar function)? I think like Fatih mentioned above, err_processed gives me an impression that we can still finish the kmac operation despite the current error condition.

"RO" to "RW1C" sounds like a neat solution. Should we be concerned if it has any security implications, e.g. like an attacker suppressing the time-out error by clearing the ERR_CODE register?

I suggest to change valid to RW1C and other fields remained RO.

2). If we decide to enhance the error_code logic, I personally prefer to change the err_code register from "RO" to "RW1C" (or something with similar function)? I think like Fatih mentioned above, err_processed gives me an impression that we can still finish the kmac operation despite the current error condition.

"RO" to "RW1C" sounds like a neat solution. Should we be concerned if it has any security implications, e.g. like an attacker suppressing the time-out error by clearing the ERR_CODE register?

I suggest to change valid to RW1C and other fields remained RO.

I am happy with this proposal.

#14818 has been created to address this issue.

The DIF should be updated, right? Can we keep the issue opened until we update the kmac DIF?

Based on #14818 the DIF should be updated, right? Can we keep the issue opened until we update the kmac DIF?

Once #14818 is merged, I can also update the DIF accordingly. In #14123, I also need to implement some KMAC DIF changes anyway.

Based on #14818 the DIF should be updated, right? Can we keep the issue opened until we update the kmac DIF?

Once #14818 is merged, I can also update the DIF accordingly. In #14123, I also need to implement some KMAC DIF changes anyway.

Sounds good!

As dicussed in #14818 This is no longer an issue. INT_STATE.error clearing may indicate the error has been addressed to SW. Let me close.


Changed the term existing SSO to linked SSO

Changed the term existing SSO to linked SSO - Arvind

@DeploymentPlan : Thanks for your contribution! The author, @barbkess, has been notified to review your proposed change.

@DeploymentPlan : Thanks for doing this.
#sign-off


Add dashboard templates for secondary-analysis.

This PR creates the dashboard templates for secondary-analysis following the instructions. The example dashboard is already on the dev Grafana instance named "analysis-staging".
I'm not sure about the correctness of the parameterization of the data sources in Terraform, so please point out if there is anything weird.

@rexwangcc should the x-axis of this graph be the "Falcon-igniter-effective-release" metric? Right now it says "Falcon-Igniter-heartbeat":

👍 Looks great to me!

@mweiden I revised the analysis dashboard by merging your changes and taking suggestions I received from the DCP Demo audience, could you take another look at the current dashboard when you have a chance? Thanks!

@rexwangcc I've rebased and tested this for you by uploading your dashboards. The staging dashboard works, but they dev, integration, and prod dashboards won't until you've given the service account Monitoring Viewer permissions for those accounts.
I have not deployed prod. Please use the steps outlined in https://github.com/HumanCellAtlas/dcp-monitoring/blob/master/README.md#deploying-a-module-into-an-environment and https://github.com/HumanCellAtlas/dcp-monitoring/blob/master/README.md#upload-to-grafana to complete the deployment to prod.

Thanks for deploying staging, dev, integration dashboards for us! I've verified that they work as expected by giving the right permission to the Grafana data source service account. Will try to deploy the dashboard for prod now.


Transfer link not working with singapore M1 and australia vodafone?

Via @zhangzhongnan928

Now, Cannot even load the import page

Australia Vodafone network neither.
This should be the issue from the payment server @colourful-land

I tested with a Ropsten transfer link and it worked on M1. Is it reproducible every time on your side for M1 @zhangzhongnan928?

Sorry not M1, Circles
Victor Zhang
CEO, AlphaWallet
RT @mwill_crypto: 3/ 🅰 @AlexSaundersAU sits down with @Victor928(CEO & Co-Founder, @Alpha_Wallet) to discuss how Alpha Wallet differentia…	Read More

On 23 Apr 2019, at 11:58 pm, Hwee-Boon Yar<EMAIL_ADDRESS>wrote:
I tested with a Ropsten transfer link and it worked on M1. Is it reproducible every time on your side for M1 @zhangzhongnan928?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Works for me for Circles.Life too.

yes, I tested just now, it works on Circles now.
Can you test on Vodafone @James-Sangalli ?

Circles.Life is an MVNO leasing from M1. So they probably fails/failed at the same time.

I tested last night on circles. It keeps loading for the free transfer link.
On wifi, it shows up import page instantly.
Victor Zhang
CEO, AlphaWallet
Not only display information, @Alpha_Wallet let you preform the functions: DEPOSIT, WITHDRAW, PAYBACK, GENERATE, MO… https://t.co/6dJfdvnM3i

On 24 Apr 2019, at 12:26, Hwee-Boon Yar<EMAIL_ADDRESS>wrote:
Circles.Life is an MVNO leasing from M1. So they probably fails/failed at the same time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Just keeping this issue for a bit to remind us ask Victor to confirm that it works for M1/Circles.Life. Not sure it's the same problem as Vodafone's because M1/Circles.Life doesn't consistently fail.

Works fine today.
Let me try again tomorrow, then we can close it.

@hboon @zhangzhongnan928 you cannot know this until you get a new build, we changed the routing using elastic load balancer on AWS. This stops the need for use of port 80 as the ELB will reroute all the calls via port 443. This problem should absolutely not exist at all when using the new build, no matter what network you are on.

This problem should absolutely not exist at all when using the new build, no matter what network you are on.

@James-Sangalli yes, hence the need to verify after the build specially for M1/Circles.Life because I'm not sure it's the same problem which is solved by the port change.

works fine on Circles now


Implementation of CONFIG_MAX_PTHREAD_COUNT

Describe the bug
CONFIG_MAX_PTHREAD_COUNT appears to have been implemented as the maximum number of POSIX threads that can ever be created, rather than what might be expected i.e the maximum number of active POSIX threads. If the former is intended IMHO this is a "bad idea"
in that it breaks any dynamic creation of threads and should be documented, if the latter then is clearly a bug.
To Reproduce
In a loop call pthread_create on a function that immediately returns i.e. the thread starts then
terminates, after a few iterations the function returns EAGAIN as the maximum thread count
has been exceeded.
Expected behavior
I would expect to be able to create POSIX threads and only get an  EAGAIN error if I exceed
a configured maximum of running threads. The POSIX use of EAGAIN implies this i.e. try
again later and you may be able to create a thread.
Impact
Breaks standard TCP handling program, where listener creates a new thread to handle a connection.
Environment (please complete the following information):
Am using the v1.14-branch GA release branch and testing on a frdm-k64f, although looking at the source is not platform specific.
Additional context
In lib/posix/pthread.c take a look at use of pthread_num (as a minimum should be an atomic variable or accessed via a mutex, even for existing semantics).
Fix is straight forward:

Create a mutex to protect pthread_num and associated stack array (posix_thread_pool)
Add an array to mark usage of posix_thread_pool (e.g. boolean array)
On thread creation, iterate usage array to find free pool element and increment pthread_num
On thread exit decrement pthread_num and mark pool element as free.

Probably about 20 lines of code :)

not related to your reported issue, did you enable CONFIG_USERSPACE when using POSIX threads?

CONFIG_USERSPACE was not configured.

https://github.com/wentongwu/zephyr/commit/8b6edc1c36fadcd9d9ac5dec042f860cda28ba8e
@SteveOss  would you like try this patch? Thanks!

@wentongwu Patch looks good. We have a bank holiday here today and Monday, so won't be able
till test till Tuesday. Will let you know how I get on.

@SteveOss  I test this patch on my side, it works well, how about your test?

@wentongwu Have pulled your commit and is working for me. Guess the thread pool index could have been stored in the posix_thread struct to avoid the lookup on thread exit, but is no big deal. Thanks for sorting this, is this going to be back ported to the 1.14 release branch?

Is this fix going back onto the 1.14 TLS branch ?


SSL error: decryption failed or bad record mac

not sure if anyone else has faced this error?
I'm using python3-psycopg2 version 2.8.6-2
root        : INFO     Part 3 of 6 : Start raw admin boundary load : 2021-11-14 03:35:11.941076
root        : INFO      - Step 1 of 3 : raw admin boundaries loaded : 0:06:19.379195
root        : INFO              - 15 duplicates removed from raw_admin_bdys_202111.aus_mb_category_class_aut
root        : INFO              - 7 duplicates removed from raw_admin_bdys_202111.aus_remoteness_category_aut
root        : INFO              - authority tables deduplicated
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/gnaf-loader/geoscape.py", line 45, in run_sql_multiprocessing
    pg_cur.execute("SET search_path = {0}, public, pg_catalog".format(settings.raw_gnaf_schema,))
psycopg2.OperationalError: SSL error: decryption failed or bad record mac

"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/gnaf-loader/load-gnaf.py", line 1011, in <module>
    if main():
  File "/usr/local/gnaf-loader/load-gnaf.py", line 120, in main
    prep_admin_bdys(pg_cur)
  File "/usr/local/gnaf-loader/load-gnaf.py", line 549, in prep_admin_bdys
    geoscape.multiprocess_list("sql", sql_list, logger)
  File "/usr/local/gnaf-loader/geoscape.py", line 27, in multiprocess_list
    result_list = list(results)
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 870, in next
    raise value
psycopg2.OperationalError: SSL error: decryption failed or bad record mac

It's pointing towards being a multiprocessing issue related to the use of a Postgres connection pool. More investigation required as all connections should be running independantly in each process.

Hey Hugh - have the exact same issue as Andrew.
Also encountering it when running with --max_processes=1
Am happy to help testing / root causing it, cheers!

I've reverted the Postgres connect pool used in multiprocessing in favour of "standard" PG connections per process. There's evidence that Psycopg connection pools aren't multiprocess-safe. There's only a small impact to performance anyway.
Code changes are currenly in the 202111 branch if you want to test (will work with 202108 data, but not 202111 data yet). WARNING: I've haven't done a full run with the code changes yet!

I tried running the 202111 branch against 202108 data but this after a lot of constraint violation messages. I'll just wait Nov 21 to be ready.
root        : INFO      - Step  7 of 14 : addresses populated : 0:01:29.125589
root        : INFO      - Step  8 of 14 : principal alias lookup populated : 0:00:04.159270
root        : INFO      - Step  9 of 14 : primary secondary lookup populated : 0:00:13.173880
Traceback (most recent call last):
  File "/usr/local/gnaf-loader/load-gnaf.py", line 1010, in <module>
    if main():
  File "/usr/local/gnaf-loader/load-gnaf.py", line 128, in main
    create_reference_tables(pg_cur)
  File "/usr/local/gnaf-loader/load-gnaf.py", line 642, in create_reference_tables
    pg_cur.execute(geoscape.open_sql_file("03-10-reference-split-melbourne.sql"))
psycopg2.errors.UndefinedTable: relation "admin_bdys_202111.locality_bdys" does not exist
LINE 14:   FROM admin_bdys_202111.locality_bdys AS bdy

The new release is done and in master. Please test.

I'll close this as I ran the last release, Feb 24 without issue.


An exception has occurred, use %tb to see the full traceback.

System information

I copied/pasted the classify_image.py
window 10 64 bits
anaconda 64 bits / anaconda navigator 1.8.7 / spyder 3.3.1
TensorFlow version 1.10 installed form anaconda/python prompt
Python version: 3.5 spyder (installed 3.7)

Hello guys,
I opened classify_image.py in spyder from the master of tensorflow (cloned on my PC before), When I executed the script I have this  message:

"""""""""""""""""""
runfile('E:/developpements/tensorflow_projects/models-master/tutorials/image/imagenet/classify_image.py', wdir='E:/developpements/tensorflow_projects/models-master/tutorials/image/imagenet')
giant panda, panda, panda bear, coon bear, Ailuropoda melanoleuca (score = 0.89632)
indri, indris, Indri indri, Indri brevicaudatus (score = 0.00766)
lesser panda, red panda, panda, bear cat, cat bear, Ailurus fulgens (score = 0.00266)
custard apple (score = 0.00138)
earthstar (score = 0.00104)
An exception has occurred, use %tb to see the full traceback.
SystemExit
E:\developpements\anaconda\envs\tensorflow\lib\site-packages\IPython\core\interactiveshell.py:2969: UserWarning: To exit: use 'exit', 'quit', or Ctrl-D.
warn("To exit: use 'exit', 'quit', or Ctrl-D.", stacklevel=1)
"""""""""""""""""""""
(tensorflow) C:\Users\david>python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
b'v1.10.0-rc1-19-g656e7a2b34' 1.10.0
thanks,
David.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

When I executed the script I have this message:

master\p1.PNG
[C:\Users\Tahira Shehzadi\Desktop\Master\semester1\MLCS\Image-Super-Resolution-master\p1.PNG ...]
main.py: error: the following arguments are required: C:\Users\Tahira Shehzadi\Desktop\Master\semester1\MLCS\Image-Super-Resolution-master\p1.PNG
An exception has occurred, use %tb to see the full traceback.
SystemExit: 2
C:\Users\Tahira Shehzadi\Anaconda3\lib\site-packages\IPython\core\interactiveshell.py:2969: UserWarning: To exit: use 'exit', 'quit', or Ctrl-D.
warn("To exit: use 'exit', 'quit', or Ctrl-D.", stacklevel=1)

can any one help me  how  to remove  this error An exception has occurred, use %tb to see the full traceback.
SystemExit: 2

"parser = argparse.ArgumentParser(description='Code for Changing the contrast and brightness of an image! tutorial.')
parser.add_argument('--input', help='Path to input image.', default='id1.jpg')
args = parser.parse_args()"    remove this part and replace this code (image = cv.imread(cv.samples.findFile(args.input))) to image = cv.imread("your_image_path")

@72ohit this does not solve the error. Did you figure out why the error is occurring due to parse_args()?

can any one help me how to remove this error An exception has occurred, use %tb to see the full traceback.
SystemExit: 2

python my_programme.py --arg1=5 --arg2=7

just put square brackets in [-arg], [--arg2]
ap = argparse.ArgumentParser()
ap.add_argument(["-i"],["--image"],required = True,
help = "Path to the image to be scanned")
args = vars(ap.parse_args())

can anyone help me ?
my error:
the following arguments are required: -i/--images/example_test.png
An exception has occurred, use %tb to see the full traceback.
SystemExit: 2

the function takes -i and image_name as argument. So to run the app you need to type on the console:
python color_detection.py -i colorpic.jpg

use
app.run(debug=True, use_reloader=False)

https://community.plotly.com/t/dash-pythonanywhere-deployment-issue/5062

use
app.run(debug=True, use_reloader=False)

It worked for me, tnx a lot.

I have the same problem: `usage: get_frozen_graph.py [-h] --checkpoint CHECKPOINT
[--out_graph_name OUT_GRAPH_NAME]
get_frozen_graph.py: error: the following arguments are required: --checkpoint
An exception has occurred, use %tb to see the full traceback.
SystemExit: 2could you please tell me how can I useapp.run(debug=True, use_reloader=False)` in my code

use
app.run(debug=True, use_reloader=False)

It worked for me, tnx a lot.

it showing error like app no define . i tried other things but not solve error ..plzz suggest .

I have the same problem: usage: untitled1.py [-h] video
untitled1.py: error: the following arguments are required: video
An exception has occurred, use %tb to see the full traceback.
could you please tell me how can I use app.run(debug=True, use_reloader=False) in my code

use
app.run_server(debug=True, use_reloader=False)


add vcname to metadata

Nowadays, syncer will add tenancy.x-k8s.io/cluster to object annotations to distinguish which vc this object belongs to. But the annotation is not convenient in real product because the cluster id is generated by vc-manager. The cluster id's format is ${namespace}-${uuid}-${vc-name}, where user can't get uuid without access k8s super master.
In real product system, there are always many modules and not all modules need to access k8s. For example, module A deals with users order and pass a vcname to the k8s; module B collects pods information and module C get order information from module A and pods information from module B. Without vcname in pod metadata, module C can't match module A's information with module B unless module C get the vcname and cluster id mapping from super k8s.
In this issue, I changes the build metadata method to add tenancy.x-k8s.io/vcname to object annotation and label. In this way, module C can match module A and B's information by vc name. I aims to let all module's to use vcname as unified id in the whole system and do not need to convert vcname to cluster id.

/ok-to-test

Thanks for working on this.
Mostly LG.
+1 to zhuangqh's comment, then we are good to go.

@Fei-Guo @zhuangqh
Thanks for the comments.
I've fixed the typo in the newest commit. Please help to review it again.

please squash your commits

/lgtm

/approve


When I debug flutter, dar use a lot of memory, how to find out which part cause this problem.

Dart SDK Version (dart --version)

Dart VM version: 2.7.0 (Mon Dec 2 20:10:59 2019 +0100) on "macos_x64"

Whether you are using Windows, MacOSX, or Linux (if applicable): MacOSX
Whether you are using Chrome, Safari, Firefox, Edge (if applicable): Flutter

When you launch your Flutter application, the Flutter tool will print a message "Observatory ... is available at http://<IP_ADDRESS>:xyz/abcdef/". This URL will take you to a web page with many debugging, inspecting and profiling tools. For understanding memory, look at the "allocation profile" and "heap snapshot" tools.

@rmacnak-google it's a little tricky, it's not  happening when I debugging, it's happening when after when I debugging. I have stopped the debug, but not close the vscode, still typing code. After sometime(maybe hours, and I have left computer open when I left), I found memory spend much more, so I checked the Activity Monitor.


The code is inconsistent with the paper

Thank you for your excellent work. However, I have a doubt. According to the statement of your paper, the depth assumption of SRC image is used to obtain the similarity score by projection transformation to ref image. However, in Code, the depth assumption is set in the Ref Image coordinate system to obtain 3D points, which are transformed to the SRC image coordinate system and sampled from the SRC image. Finally, the similarity score is obtained by dot product with the REF image. This is actually helpful to get the depth of the ref image, instead of SRC image. I don't understand why you make the depth assumption in the Ref image coordinate system and then project it.

Hi @cjd24-coder, sorry for the confusion. I realize the variable naming(ref,src) in the code is flipped from the paper. We indeed make the depth assumption in the coordinate system of the image that we want to predict depth.

Hi @cjd24-coder, sorry for the confusion. I realize the variable naming(ref,src) in the code is flipped from the paper. We indeed make the depth assumption in the coordinate system of the image that we want to predict depth.

Hi @cjd24-coder, sorry for the confusion. I realize the variable naming(ref,src) in the code is flipped from the paper. We indeed make the depth assumption in the coordinate system of the image that we want to predict depth.

Hi @cjd24-coder, sorry for the confusion. I realize the variable naming(ref,src) in the code is flipped from the paper. We indeed make the depth assumption in the coordinate system of the image that we want to predict depth.

Hi @cjd24-coder, sorry for the confusion. I realize the variable naming(ref,src) in the code is flipped from the paper. We indeed make the depth assumption in the coordinate system of the image that we want to predict depth.

Hi @cjd24-coder, sorry for the confusion. I realize the variable naming(ref,src) in the code is flipped from the paper. We indeed make the depth assumption in the coordinate system of the image that we want to predict depth.

but the proj matric in homo_warping should be: proj = torch.matmul(ref_proj, torch.inverse(src_proj)) if you flipped from the paper. If I understand the formula for homo warping correctly, the formula for proj should correspond to the transformation.

Hello author, I want to know how you get this accuracy, in the case of projection transformation error. Secondly, after I fixed your projection bug, the ABS_rel error in another Dense Depth for Autonomous Driving (DDAD) dataset was reduced from 20% to 12%. However, monocular single-view networks can also achieve 13% accuracy on this data set. Do you have any tuning skills for MVS2D? Can you tell me? Thank you

I think mVS2D performance should be more than one percent more accurate than single-view networks

Hi @cjd24-coder , I don't think the proj matric calculation in homo_warping is wrong in homo_warping. It should be just proj = torch.matmul(src_proj, torch.inverse(ref_proj)), given that ref_proj is the world2cam projection matrix of ref frame(the frame we want to predict depth)

sorry, i misunderstand.


latest php 8.2

Good morning!
Any chance you will build the latest version of php 8.2?

Re-ran the PHP 8.2 build. It should be the latest from the master branch of php-src.
https://github.com/shivammathur/php-builder-windows/releases/tag/php8.2

Awesome thanks!


Fixed geodetic perf test

 Flatten API (get rid of convert/calculate)

Coverage remained the same at 100.0% when pulling 52f642565b59a26e89fe5a5429b118b2358c54b9 on bugfix/perf-tests into bee81b4641384675579ad36344b259f49a7ac66d on master.


⚠ ESPN has degraded performance

In 9bd75d3, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 1666 ms

Resolved: ESPN performance has improved in 3d67e8d.


Instructions for setting up on CakePHP 3

Hi Wan,
I am trying to get this set up on Cake3, is there any chance you can update the README with any differences (for example I cant pass in a custom config when loading the plugin any more).
With the latest version I can get stats to show me the workers, I can get a worker to start but as soon as I try to process a job I get:
2015-09-17 06:03:12 Warning: Warning (2): Attempt to assign property of non-object in [/var/www/vhosts/forumapi/vendor/kamisama/php-resque-ex/lib/Resque/Job.php, line 178]
Trace:
Cake\Error\BaseErrorHandler::handleError() - CORE/src/Error/BaseErrorHandler.php, line 139
Resque_Job::getInstance() - ROOT/vendor/kamisama/php-resque-ex/lib/Resque/Job.php, line 178
Resque_Job::perform() - ROOT/vendor/kamisama/php-resque-ex/lib/Resque/Job.php, line 191
Resque_Worker::perform() - ROOT/vendor/kamisama/php-resque-ex/lib/Resque/Worker.php, line 278
Resque_Worker::work() - ROOT/vendor/kamisama/php-resque-ex/lib/Resque/Worker.php, line 241
startWorker - ROOT/vendor/kamisama/php-resque-ex/bin/resque, line 109
[main] - ROOT/vendor/kamisama/php-resque-ex/bin/resque, line 100

I have tried the following:
Created AppShell with:
<?php
namespace App\Shell;

use Cake\Console\Shell;

class AppShell extends Shell
{
    /**
     * Perform function needed for CakeResque
     * @return mixed
     */
    public function perform()
    {
        $this->initialize();
        $this->loadTasks();
        return $this->runCommand($this->args[0], $this->args);
    }
}

As per the 2.x docs and then had my shell extend this, I have tried it without and same issue.
Also tried with:
 public function perform()
    {
        $this->initialize();
        $this->{array_shift($this->args)}();
    }

Which is what I have working fine in my 2.x app.
Some pointers would be appreciated, I have a fair bit of code for custom configuration that I have been playing around with on my local machine so if this is a task that needs to be done, once i have it working I should be able to do a PR.
Thanks

Cake 3 compatibility is work in progress, and is not ready yet

OK - anything that needs doing? I can probably update some of the tests to PHPUnit 4, @RTLer @wa0x6e are either of you working on these at the mo - if so where shall I start?

This is the roadmap: https://github.com/wa0x6e/Cake-Resque/issues/90


CAMEL-4725: Batch support. Single Producer for both SQL-components.

Added batch support for StoredProcedures. It uses Spring API as much as possible.
StoredProcedure can be read from message body and parameters are read from header.
Only Map is supported as parameter in this case.
Refactored SqlProducer to support both CallableStatement and PreparedStatement. New name is DefaultSqlProducer(as it is now more general class). Adapters  for PreparedStatement, CallableStatement and CallableStatement batch mode are in "wrapper"-package.
Moved parameters only sensible for SQL component into SqlComponent and
left common properties into DefaultSqlComponent.
There are a lot of changes to avoid duplicate logic in code between CallableStament and PreparedStatement implementations.

I started wondering, should it be single component, because now with these wrappers, Producer could easily select at runtime which one to use(Prepared or Callable wrapper). Callable statement syntax could start with something that differentiates it from SQL clause.

The camel-elsql component extends the sql component so all this code changes may break it. Wonder if you could take a look and compile and test that component too with all these changes.

I checked, but it does not compile. Should I just make sql-stored endpoint and component, without extending sql-component ? Then there would be much less code changes. Maybe that refactoring can be done later, if wanted.

Yes we favor separating concerns - much easier to reason and understand.
So try to separate the refactoring and the added code for the batch support.
+1 to do refactoring later.

Thanks @snurmine I got your latest code merged into the master branch. You are welcome to close this PR. And keep on the good work on the component.

Thank you. There probably will some at least one new PR.


Cannot store reports with relative include paths: If the clang-tidy report refers to relative paths the resulting plist contains invalid absolute filepath

This error prevents storing the results.
The problem can be reproduced like this:
In the original compiler call the header was included like this:
clang .. -include ../../../Sources/some_header.hpp
the clang-tidy textual output:
gtest_pred_impl.h:40:10: warning: circular header file dependency detected while including 'gte
st.h', please check the include path [misc-header-include-cycle]
   40 | #include "gtest/gtest.h"
      |          ^
/Users/dkrupp.ext/CCMain/Bin.MacARM/Modules_LLVM_4_0_64.dev/GoogleTest/gtest/gtest.h:388:10: note: 'gtest_pred_impl.h' included from here
  388 | #include "gtest/gtest_pred_impl.h"
      |          ^
**./../../../Sources/some_header.hpp:66:10: note: 'gtest.h' included from here**
   66 | #include "gtest/gtest.h"

./../../../Sources/some_header.hpp:66:10: note: 'gtest.h' included from here

In the plist file the file section some_header.hpp is referred to as:
 </array>
    <key>files</key>
    <array>
            <string>/Sources/some_header.hpp</string>

I couldn't reproduce this issue. All clang-tidy versions I tried are emitting absolute source patsh. Could you, please, check this reproducer again and attach the source files and exact commands you run? Thank you!


Limit AR Image Recognition Area

There is a way to limit AR Image Recognition to a specific area? I want to place an overlay crosshair and make the app recognize target images only if they enter that crosshair. It's possible? thanks

There is a way to limit AR Image Recognition to a specific area?
Not at the moment, Image Recognition is performed on the visual data that is sent to the camera / whatever the camera is looking at, which at this point would be the scene.

make the app recognize target images only if they enter that crosshair

By the above, do you mean have your app respond differently if it recognizes a certain image in your scene that your user is looking at? If so, you could use our image marker detection api that would recognize target images in your scene, with which you can place 3D items with onHover callbacks (so that they are triggered when the user "focuses" on them).
Here are some docs to add ViroARImageMarker components to your scene:
https://docs.viromedia.com/docs/viroarimagemarker
Then add your model components as children of the ViroARImageMarker components. When the system "finds" the target for that ViroARImageMarker, you'll get an onAnchorFound callback on that particular ViroARImageMarker and the children components will be unhidden and rendered relative to the image. Your children components can then have onHover callbacks.
Here's a sample code with 1 ViroARImageMarker component, but you can add many more: https://github.com/viromedia/viro/blob/master/js/ARCarDemo/ARCarDemo.js

no i was talking about triggering the onAnchorFound callback only if the target enter a specific camera area, for example a rectangle in the middle of the screen

if the target enter a specific camera area, for example a rectangle in the middle of the screen.

Again, you would first have to detect and recognize targeted markers in the scene, before knowing where the targets are, and currently image recognition is performed on the entire camera visual data that is sent to the marker. There isn't really a way currently to filter the recognition area to an even smaller rectangular in the middle of the screen, from the given input camera data (probably not the right approach).
You can potentially achieve the same results by managing a list of detected markers, and then determining if they are in the center of the screen or not (if the user is focused on them via onHover). Or doing a positional check based on where the camera is, and the user is looking at.

HI Tamiyadd ! can you help me,how to check target in rectangle in the middle of the screen because every times the app recognize target func on Hover always have different position


[openmvg] Update version to v2.0

Fix #21364
Update [openmvg] to the latest version 2.0
Note: No feature needs to test.

build_fixes.patch is still in the port folder even if not used anymore? why the patch rename, that was hiding the problem?

@cenit  Thanks for your reminder, I will corrcet it right away. :)

now the situation is opposite. There is the new fix-build.patch that you introduced totally unused but still pushed in this PR. Please delete that file

now that graphviz has been merge, openmvg[tool] feature can depend on that, and you can remove the message info in the portfile (patch also the python scripts to use neato from vcpkg folder!)
https://github.com/microsoft/vcpkg/pull/20564

This PR has been resolved by another PR #22020


Dynamic sizing of the modal

Does anyone know how to dynamically size the modal?  For example, if I hard-coded the width of some content to be 100px wide, and the app element that the modal is bound to is 200px wide, I want the modal to be centered in the window at 100px wide.  Any ideas?

Here is a hack I used to do a similar thing, vertically rather than horizontally.

Add a ref name (ie: ref='content') to your content container div.
Grab the height of the content container element once it is in the DOM and put it in state:

  componentDidUpdate() {
    if (window.requestAnimationFrame) {
      window.requestAnimationFrame(this.sizeDialog);
    }
    else {
      // IE <10 CYA - Note: I haven't actually tested this in IE - YMMV
      window.setTimeout(this.sizeDialog, 50);
    }
  }

  sizeDialog = () => {
    if (!this.refs.content) return;
    let contentHeight = this.refs.content.getBoundingClientRect().height;
    this.setState({
      contentHeight: contentHeight,
    });
  }

Note that use requestAnimationFrame here - the ref was not yet defined otherwise. Maybe this has to do with the DOM magic react-modal does... At any rate this is a hack and I'm sure there is a better way to do this.

In render(), use this.state.contentHeight to style the dialog:

    const padding = 90; // adjust this to your needs
    let height = (this.state.contentHeight + padding);
    let heightPx = height + 'px';
    let heightOffset = height / 2;
    let offsetPx = heightOffset + 'px';

    const style = {
      content: {
        border: '0',
        borderRadius: '4px',
        bottom: 'auto',
        height: heightPx,  // set height
        left: '50%',
        padding: '2rem',
        position: 'fixed',
        right: 'auto',
        top: '50%', // start from center
        transform: 'translate(-50%,-' + offsetPx + ')', // adjust top "up" based on height
        width: '40%',
        maxWidth: '40rem'
      }
    };

    return (
      <Modal isOpen={this.props.show} style={style} className='dialog'>
        <div className='dialog__content' ref='content'>
        ... your content goes here ...
        </div>
      </Modal>
    );

Hope this helps.

Thanks @davidmfoley.
It works quite well.

Nice @davidmfoley

No problem.
We are now using pure CSS to size the modal, which works as long as you style the components in the modal so that they result in proper width/height, relieving the need for any requestAnimationFrame/manual coordinate math:
...
      content: {
        border: '0',
        borderRadius: '4px',
        bottom: 'auto',
        minHeight: '10rem',
        left: '50%',
        padding: '2rem',
        position: 'fixed',
        right: 'auto',
        top: '50%',
        transform: 'translate(-50%,-50%)',
        minWidth: '20rem',
        width: '80%',
        maxWidth: '60rem'
      }
...

+1 Thanks for the pure CSS solution!

It's been a while since there was any discussion on this issue.  I'm assuming that it isn't a problem anymore.  If it is, feel free to re-open the issue.

Hello guys,
did you find a solution to let the modal resize to fit its content's width, without disabling vertical scrolling of the modal ? The modal should move, the content inside the modal should not scroll.

@amangeot As David pointed out, the layout will break if the children inside the modal exceed the dimensions of the viewport, but defining those maximums in viewport units should take care of that. Building on his code I ended up doing this:
.modal {
  position: fixed;
  top: 50%;
  right: auto;
  bottom: auto;
  left: 50%;
  transform: translate(-50%,-50%);
  outline: 0;
}

.children {
  max-height: 90vh;
  max-width: 90vw;
}

At least for the image previews I am using it for, it seems to work really well, even on mobile.

I get this fixed by writing HOC  where i am using refs to get content/modal height.

It would seem there are enough people asking for this feature. It's not sufficient to declare static CSS classes as this doesn't meet the needs of the OP - i.e. dynamic modal sizes.
It would be relatively trivial to implement additional props on the Modal component that affect the size.

Late to the game but not following how to dynamically size the modal.  What I see is that the dialog width is set to 50%, (.popup-content width:50%). That ends up being 50% of the viewport.  If I constrain the width of the contents that makes the content smaller but there is still a bounding rect that is 50% of the viewport.

What about dynamic sizing of the modal in typescript?

my solution is
  ...    right: 'auto',   bottom: 'auto',   ...

I'm still having issues getting modal to auto-expand when react-datepicker is clicked on within the modal.  Posted a question here:  https://stackoverflow.com/questions/72890966/react-modal-how-to-get-modal-to-auto-expand-when-react-datepicker-is-clicked

thanks - I've tried but did note"

adding a:  height: 'auto' - doesn't change anything
additing the "height auto" and also removing the "bottom auto" - the size of the dialog is bigger, however it is big for the initial state before the date field has been selected, as well as after the date field is selected and the month calendar appears?  so not dynamic.  That is like this:

then

.ReactModal__Content--after-open {
width: 1000px !important;
top: 50% !important;
left: 50% !important;
transform: translate(-50%, -50%) !important;
min-height: max-content !important;
overflow: hidden !important;
}

.ReactModal__Content--after-open {
width: 1000px !important;
top: 50% !important;
left: 50% !important;
transform: translate(-50%, -50%) !important;
min-height: max-content !important;
overflow: hidden !important;
}

@polixisNG Actually, you do!! You can use the style attributes do manage the size of the content and overlay.
http://reactcommunity.org/react-modal/styles/


Documentation question

I wish to confirm this section if this is correct because the way i understood is the opposite.
Note -> Using QUERY_ALL_PACKAGES permission may cause your app to be rejected on playstore if your app doesn't have any core functional which is
required broad package visibility (eg antivirus app that needs limit visibility to scan all apps in your device). If your app does not require QUERY_ALL_PACKAGES permission as a core functionality in your app, consider using the
<uses-permission android:name="android.permission.QUERY_ALL_PACKAGES"/>

OR

<queries>
 <package android:name="com.whatsapp.businessapp"/>
 <package android:name="in.techbyvishesh.myapp"/>
</queries>

I am confused with this part "If your app does not require QUERY_ALL_PACKAGES permission as a core functionality in your app, consider using the"
If the app does not require QUERY_ALL_PACKAGES, how the documentation says it is to add the user permission query all packages. Wouldn't that get rejected instead? The app should maybe have something that removes that permission?
Please advise.

Hi @chitgoks ,
Sorry for any confusion this documentation has caused you; I have not been active in updating this document in the last few months.
The Play Store's policy was updated last year, noting that if your app does not require all package visibility, do not use the QUERY_ALL_PACKAGES permission. Instead, use the  meta tag to specify which packages you are querying. If you still require QUERY_ALL_PACKAGES permission, you have to provide a valid use case to the the Google Play store team, or your app will be rejected.
I will update this package with the latest features and updated documentation (which I promised in the document) at the end of this month or next month.
If you still have any other query related to this package then  please let me know.
Thank You

Hi @vishesh005  no worries. thank you very much for the clarification.


Added custom tooltips for XYPlots

#545
This adds support for custom tooltips for XYCharts. I only had time to re-implement this feature for XYCharts, other charts are still missing functionality I believe.

closing due to inactivity