[Not Ready] PCM-847 Added guest screen share functionality.

Still need to write tests and update the docs. Just hoping for some feedback so I can make any changes.
EDIT: Here was my thinking behind these changes. I tried to setup the SDK to be easily extendable later when we start adding additional rtc functionality.
EDIT 2: Here are my remaining questions/comments
Comments
Constructor() changes

If an accessToken is passed into the constructor options, the user is assumed to be an authenticated user; else, they are a guest.
A sdkType option to say what kind of sdk it is ('softphone' (default) | 'screenshare').
Added validation to make sure the sdkType and isGuest are compatible

Initialize() changes

If a guest user, an object with accessCode is required to complete guest authentication to receive a jwt.
The functionality will be based on the sdkType and the user type (authenticated or guest).

Questions

How to handle window.navigator.mediaDevices.getDisplayMedia in Chromium versions and other browsers. Used here

How/should we use getDefaultChromeConstraints()?
Should we pull in the webrtc-adapter?

Should we attach the guest's screenshare stream to their DOM?

Function here. Not currently being used.

Should we make maxFrameRate configurable?
Add better error handling?

client-private.ts on line 80 -> const stream = await startDisplayMedia.call(this);

Should parseJwt() become a shared package? It is duplicated in streaming-client and valve.

TODO

New ticket to convert Streams to Tracks
Build out test-pages
Update documentation

This call for guests: https://api.inindca.com/api/v2/diagnostics/trace  returns a 401 because they are not authenticated. Is there a public endpoint we could hit with that information?

@djhouseknecht we'll probably have to disable the server side logging for guest users. There's no equivalent.

@djhouseknecht could you get the valve branch testing this synced with master, please? After that is good I'll run it and take a closer look at both.

I think I see where you're going with attachScreenShareMedia. Keep in mind a few things:

we don't want to attach local media for any conversation type
screen share doesn't have any remote streams (in guest context)
If you want to delve into using the SDK for the screen share on the agent side, then that's the place to consider how to attach the incoming screen share. We'll do that very similarly to acceptPendingSession where they can provide an HTMLVideoElement, but in the case of screen share, if none is provided, then the screen share stream probably wouldn't get attached at all, since there's no good "default" way of doing that. They'd have to listen to the events and attach it themselves, or provide the element (or maybe container?) to use

I just realized that I need to pull in the changes made in 1.2.x

Added the changes from https://github.com/MyPureCloud/purecloud-webrtc-sdk/pull/35/


Add WebVR API to lib.d.ts

Docs at https://developer.mozilla.org/en-US/docs/Web/API/WebVR_API

reported originally in https://github.com/Microsoft/TSJS-lib-generator/issues/294

PRs welcomed. You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

A Working Draft should go on DefinitelyTyped. A feature that is a Candidate Recommendation (or has passed that stage) should be added here.

Have the standard to add types on TSJS-lib-generator changed? WebVR didn't even reach WD and it has a Deprecated API warning.

Thanks for the note. we should not include it in the default lib untill it makes WD stage.


.env file`s icon.

Hello can you create iconf for .env files?

Since .env files are basically configuration files for the environment, I'm thinking that the same icon as .ini/.conf files could be used. This is also what the Seti icon theme does

Yes, nice idea :)


KeyError in full_preprocessing

Hi, thanks for your reply. I have fixed that problem.
However, I met another error when executing till line 210 of 'full_preprocessing.py'. It seems that the key of variable 'df' is wrongly defined, as below.

it seems that your line 125 reads that you already have a records_w_diag.pkl, my believe is that your final part of step 2 saved a records_w_diag.pkl file and after it broke (last issue) and now, since it is available, line 127 loads it and is missing several columns that are not being computed in the else statement in line 128. I would suggest you delete your records_w_diag.pkl and start the processing once again from the start, hope it helps
Juan


How to train with custom dataset?

I have custom data with coco format, how can I train with my own data?

Thanks for your interest in our work.
We bulid our detector based on Detectron2, please refer to https://detectron2.readthedocs.io/en/latest/tutorials/datasets.html for training / fine-tuning on your custom datasets.

I believe the issue at hand was addressed, as such I'm closing this. Feel free to ask if you have further questions.


How do i run this and does this program work outside China

pls help

Not exactly work outside of China, but I can give the idea how I support it inside China.

http://ws.webxml.com.cn/WebServices/MobileCodeWS.asmx is the the web service to get city name by cellphone number. Due to the service only support to search inside China, thus this is point where you'd like to replace
https://groups.google.com/d/topic/google-maps-js-api-v3/Ly4EHKYMFSc once you find the city or location name, call Google GeoCoder API would return you the corresponding geolocation

Not support outside China due to the webservice only support to find location name inside China. If you replace this part, the next step is also compatible to find geolocation by location name by Google Geocoder service.


Separate usage metrics from quality metrics

Separate usage metrics from quality metrics
The metrics tab shows usage metrics (download statistics). Ideally, this will also show some cited usages (https://github.com/gbif/portal16/issues/307) and alt metrics (https://github.com/gbif/portal16/issues/245).
I propose renaming the tab 'metrics' to 'usage metrics' and making a new tab 'quality metrics' displaying metrics related to the data itself. Note requirements for what to show in such a tab are being gathered in issue https://github.com/gbif/portal16/issues/138

fbitem-dataset38f06820-08c5-42b2-94f6-47cc3e83a54a
Reported by: @kbraak
System: Firefox 51.0.0 / Mac OS X 10.10.0
Referer: https://demo.gbif.org/dataset/38f06820-08c5-42b2-94f6-47cc3e83a54a/metrics
Window size: width 1276 - height 1041
API log
Site log

Makes sense. it is now back to what it was before. Stats and activity. names are kept from the current site. activity could be renamed usage metrics (download statistics) and show more than just a list of downloads. I will leave the naming to comms. For now, they keep their names from the current site. Thanks


Add more sshcmd tests

What is the purpose of this change? What does it change?
Provide more test data to hopefully catch coding errors.

Was the change discussed in an issue or in the forum before?
Not that I am aware of.

Checklist

[x] I have read the Contribution Guidelines
[x] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[ ] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Interesting corner cases, thanks! Are you also interested in solving these, do you plan to add this to the PR?

Replaced with https://github.com/restic/restic/pull/1952


Python3 not working

I cant run python3 and i dont know how to install it either? Help please?

What do you mean? This is a shell that is written in JavaScript. Everything is in the browser. To make Python3 work you will need to import Brython or a different Python interpreter (there was something recently in WASM) and add that to the code. You won't be able to install anything here.


Added meta tags for security purposes, console.log to see when it is updated

The lack of these tags resulted in the site not showing the video, which is anticlimactic.

LGTM


An option to target a particular ref in a GitHub repo

It would be handy to have an option to specify the ref that you are targeting in a repo, because the dependencies or code scanning results could be different on different branches etc... Not to mention that you need to specify the branch to retrieve the code scanning alerts if you have scanned anything other than the main branch. Right now the program seems to default to /refs/heads/main.
Is this repo still being maintained? I'm working on implementing the functionality mentioned above based on a more up-to-date fork of this project (here). I'd be happy to send a PR here as well when I'm done, but it would require some other open PRs to be merged first :)

I have a number of changes coming in to the vnext branch that should accommodate alternate branches.


kodi: support av1 video decoding

Describe the bug
On other distributions (LibreELEC, Arch Linux) I can play media files with an AV1 encoded video. On NixOS (kodi-gbm) the GUI shows a frozen loading screen instead of the playing video. The audio is playing fine.
Steps To Reproduce

Play an AV1 encoded video file in Kodi.

Expected behavior
The video plays alongside the audio.
Screenshots
(I didn't make one)
Additional context
That happens with all AV1 encoded videos in my library.
Kodi's default skin fails to show video metadata (video codec, audio codec, aspect ratio, audio channels) on AV1 encoded videos in the video library.
 Enabling any of those options doesn't help.
I already unsuccessfully tried

Adding dav1d as buildInput to kodi. CMake reports that Dav1d is found, but the resulting kodi still cannot play AV1 videos.
Adding dav1d and/or libaom as build inputs to kodi's ffmpeg, as well as setting --enable-libaom with configureFlags.
Setting kodi.ffmpeg to pkgs.ffmpeg_4-full (as the full versions should contain AV1 decoding support).

Notify maintainers

@aanderse @cpages @edwtjo @minijackson @peterhoeg @sephalon
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

system: "x86_64-linux"
host os: Linux 6.1.41, NixOS, 23.05 (Stoat), 23.05.20230729.d57e8c5
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.13.3
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Kodi Log when playing AV1 videos
[...]
2023-08-01 23:10:57.587 T:9863     info <general>: VideoPlayer::OpenFile: REDACTED
2023-08-01 23:10:57.587 T:10005    info <general>: Creating InputStream
2023-08-01 23:10:57.613 T:10005    info <general>: Creating Demuxer
2023-08-01 23:10:57.920 T:10005    info <general>: Opening stream: 0 source: 256
2023-08-01 23:10:57.920 T:10005    info <general>: [WHITELIST] Searching the whitelist for: width: 3840, height: 1600, fps: 25.000, 3D: false
2023-08-01 23:10:57.921 T:10005    info <general>: Display resolution ADJUST : 1920x1080 @ 50.000000 Hz (21) (weight: 0.000)
2023-08-01 23:10:57.929 T:9863     info <general>: VideoPlayer: OnLostDisplay received
2023-08-01 23:10:57.929 T:9863  warning <general>: CDVDMessageQueue(audio)::Put MSGQ_NOT_INITIALIZED
2023-08-01 23:10:57.929 T:9863  warning <general>: CDVDMessageQueue(video)::Put MSGQ_NOT_INITIALIZED
2023-08-01 23:10:57.929 T:9863     info <general>: GLES: Maximum texture width: 16384
2023-08-01 23:10:57.930 T:10005    info <general>: Creating video codec with codec id: 32797
2023-08-01 23:10:57.930 T:10005    info <general>: CDVDVideoCodecFFmpeg::Open() Using codec: Alliance for Open Media AV1
2023-08-01 23:10:57.930 T:10005    info <general>: Creating video thread
2023-08-01 23:10:57.930 T:10008    info <general>: running thread: video_thread
2023-08-01 23:10:57.933 T:10005   error <general>: Unable to lookup host: 'REDACTED'
2023-08-01 23:10:57.933 T:10005    info <general>: Opening stream: 1 source: 256
2023-08-01 23:10:57.933 T:10005    info <general>: Finding audio codec for: 86076
2023-08-01 23:10:57.933 T:10005    info <general>: CDVDAudioCodecFFmpeg::Open() Successful opened audio decoder opus
2023-08-01 23:10:57.933 T:10005    info <general>: OpenStream: Allowing max Out-Of-Sync Value of 10 ms
2023-08-01 23:10:57.933 T:10005    info <general>: Creating audio thread
2023-08-01 23:10:57.933 T:10009    info <general>: running thread: CVideoPlayerAudio::Process()
2023-08-01 23:10:58.444 T:9914     info <general>: PVR Manager: Starting cleanup of cached images.
2023-08-01 23:11:00.433 T:9863     info <general>: VideoPlayer: OnResetDisplay received
2023-08-01 23:11:00.440 T:10008    info <general>: VAAPI::Close
2023-08-01 23:11:00.440 T:10008    info <general>: CDVDVideoCodecFFmpeg::Open() Using codec: Alliance for Open Media AV1
2023-08-01 23:11:00.444 T:10009    info <general>: CVideoPlayerAudio: display reset occurred, checking for passthrough
2023-08-01 23:11:00.446 T:10009    info <general>: CDVDAudioCodecFFmpeg::Open() Successful opened audio decoder opus
2023-08-01 23:11:00.446 T:10009    info <general>: Creating audio stream (codec id: 86076, channels: 2, sample rate: 48000, no pass-through)
2023-08-01 23:11:00.447 T:9879     info <general>: CActiveAESink::OpenSink - initialize sink
[...]

awesome!


[Rendering] Set allowsEdgeAntialiasing on rotated views by default

So many apps don't antialias away the jaggies and it looks crappy. In iOS 7+ you can set allowsEdgeAntialiasing on individual CALayers instead of needing to set it globally or resorting to hacks with transparent borders so I believe it's a good idea to automatically set it on rotated views for a good-looking default.

BTW, it may be not so trivial for borderWidth > 0.

Enabling layer rasterization for non-iOS rendering solved this.
  layer.shouldRasterize = YES;
  layer.rasterizationScale = [UIScreen mainScreen].scale;

This was already enabled in @ide's pull request I think. Unless that didn't make it in for some reason.

@nicklockwood - good point, #1999 covered this and related commit b362072c83b5fe883e913e5752735fc6e561219b

Still have some issues with some jaggies with views inside a rotated view.

Source: https://gist.github.com/l-urence/17fe144ac6a9003cde02


Support passing in a script/command to get the keys rather than reading from a file

This is, for example, to be able to retrieve the age keys from a password manager or other secret storage engine. (In my usecase, from KDE's KWallet. But it could also be replaced with MacOS's Keychain, etc.)
Another option, rather than supporting arbitrary code execution, is to support using a platform's native secret storage. Something like go-keychain.

That is an awesome idea!
I'm only wondering if it makes sense to support this directly in git-age or if it wouldn't be better to implement an age plugin that supports this for all age use cases and implement the plugin API in git-age.
I had that already in mind as next step and that would also allow the usage of a Yubikey - besides others.
What do you think?

Hm, that's a good point. I forgot about the plugin interface! I'll have to see if this exists. If not, maybe I create it. :smile:

I haven't found something like it so far 😄 also I did some more research regarding keyring implementations and 99designs/keyring might be even easier to integrate than the one from keybase.
I'll try to add the plugin support soon ™  and if you didn't implement the plugin till then I will have a look if I can do it 😄


Design of Page: Switching Elements from Right to Left?

In the US market, because of the dominance of Shopify, the user is inclined to expect the checkout to have the "Order Summary" on the right side of the page and the Checkout flow on the Left side. In this repo, the situation is exactly reversed, with Order Summary on Left and Checkout on Right. What is the easiest way to switch this without modifying a ton of code? Also, you might consider offering this an option in an ENV variable. Something like "flow" string - with values of "right" or "left" depending on the design required. BTW, on a mobile view the order summary should always appear at the top of the page and the checkout at bottom, but I think that's how it is now anyway.

Hi @osseonews, I'm sorry but we are not thinking to include a variable to switch Order Summary with Checkout flow. If you want you can always fork our checkout and adjust it at your needs. It shouldn't be too difficult to exchange columns. There are no dependency in terms of logic and it should be a matter of styles.
Thanks anyway for your point.


New game candidates

https://icga.org/?page_id=3818

Another suggestion: Domineering.

Tak environment is requested
https://github.com/sotetsuk/pgx/issues/1039

Royal Game of Ur is requested
#1115

@sotetsuk Minor suggestion: I suggest putting the content of the image in the original post in text form (e.g. as a quote block) so that this issue shows up in the results when those game names are searched.

@sotetsuk Minor suggestion: I suggest putting the content of the image in the original post in text form (e.g. as a quote block) so that this issue shows up in the results when those game names are searched.

@sotetsuk Minor suggestion: I suggest putting the content of the image in the original post in text form (e.g. as a quote block) so that this issue shows up in the results when those game names are searched.

Thanks! Great suggestion👍 Done ✅

@sotetsuk Minor suggestion: I suggest putting the content of the image in the original post in text form (e.g. as a quote block) so that this issue shows up in the results when those game names are searched.

Thanks! Great suggestion👍 Done ✅

Ludii–the ludemic general game system, ECAI 2020 also provides a list of some interesting board games.


packagegroup-develop: drop subversion from package list

subversion is not widely used anymore, so let's drop it from the default development list.
If needed, it still can be included in custom builds using EXTRA_PACKAGES in local.conf, IMAGE_INSTALL:append or similar mechanisms.

My only concern here is that a customer still uses subversion, but i think this is most reliable way to figure this out.

This even (only) affects the target image, right? Building recipes on the build host will automatically DEPEND on subversion-native anyway:
https://git.yoctoproject.org/poky/tree/meta/classes/base.bbclass?h=kirkstone&id=e938b18b5342bd28eadb44ad39dbf1f5cf5be09b#n643

This even (only) affects the target image, right? Building recipes on the build host will automatically DEPEND on subversion-native anyway:
https://git.yoctoproject.org/poky/tree/meta/classes/base.bbclass?h=kirkstone&id=e938b18b5342bd28eadb44ad39dbf1f5cf5be09b#n643

Yes, of course. This only drops it from the target image installed set of packages - and maybe some libraries are dropped then too which have no other consumers then anymore.


Provide jsdoc linting

Add jsdoc linting to qx lint  via  https://eslint.org/docs/rules/valid-jsdoc - It should only output warnings, not errors so that it won't break automatic testing.

https://eslint.org/docs/rules/valid-jsdoc is deprecated - we should use eslint-plugin-jsdoc:
see https://eslint.org/blog/2018/11/jsdoc-end-of-life

Here are the settings that I found useful for qooxdoo code at this point:
  "extends": [
    "@qooxdoo/qx"
  ],
  "plugins": ["jsdoc"],
  "rules": {
    "no-console": "off",
    "no-eval": "off",
    "new-cap": "warn",
    "jsdoc/check-param-names": 1,
    "jsdoc/check-types": "off", // for example, "Boolean -> boolean"
    "jsdoc/require-jsdoc": 1,
    "jsdoc/require-param": 1,
    "jsdoc/require-param-description": "off",
    "jsdoc/require-param-name": 1,
    "jsdoc/require-param-type": "off",
    "jsdoc/require-returns-type": 1,
    "jsdoc/valid-types": 1
  },

This won't complain about @param varname {Type} which is not valid JSDoc, but required by the generator and the current APIViewer.


ABHIJEET SRIVASTAVA

Issue Type:
[ ] Bug Report
[ ] Feature Request
[ ] Documentaion
Describe the bug

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

If applicable, add screenshots to help explain your problem.

@abhijeet1312 make a new PR there is some problem in previous one
dont forget to fetch and merger and then  run git pull origin master


Cannot run on IIS

It was working well on Apache but when migrating to IIS I just couldn't log in.
Am I missing something?
Sorry I'm a rookie

I don't run it on IIS, but a user did get it going and provided a nice write-up.
Take a look at the wiki page: https://github.com/kiswa/TaskBoard/wiki/TaskBoard-on-IIS


bug: Accordion. Content is jumping down and up

What version of daisyUI are you using?
"daisyui": "3.2.1"
Describe your issue
From official docs:
https://daisyui.com/components/accordion/
If the content is large, it's more visible that content jumps on opening.
Is it possible to fix it? Thank you.
https://github.com/saadeghi/daisyui/assets/17801144/042e07eb-4389-4c74-8649-89e39f665965
What browsers are you seeing the problem on?
Chrome, Safari, Firefox, Safari iOS, Chrome Android
Reproduction URL
https://daisyui.com/components/accordion/

Is it ok, when I add here another issue that I think is also related to that?
When we use the default arrows on the right side in 3.5.1 the arrow is jumping too.
record-daisyui.webm
When the content get longer the arrow goes also more down.


Added Hadoop HDFS FSImage exporter to list of miscellaneous exporters

The exporter exposes Hadoop HDFS usage statistics by parsing the Hadoop HDFS NameNode FSImage file.
Announced on Prometheus Users.

Thanks!

@brian-brazil, looking forward to discuss your exporter feedback on the PromConf in Munich this week


feat: container resources

Thnx

@ebcFlagman I'll merge PR and will apply my adjustment by myself.

@ebcFlagman released 1.0.0


List of next examples to be added

Given the first example about FTP, what would be good candidate to backport here for having a strong subset of examples?
@davsclaus

Yeah we can look at the existing and find candidates.
One would be mqtt
https://github.com/apache/camel-kamelets-examples/tree/main/jbang/mqtt

And we can continue to add examples to the old repo when its a small "special" example, maybe for a new feature and you just want to create something to try yourself and see how it works with low-code camel - and that example is fine to not be here.

We would like to add some examples using the Camel JBang Kubernetes plugin

One would be mqtt
https://github.com/apache/camel-kamelets-examples/tree/main/jbang/mqtt

Already started to work on this one, will open A PR soon

We can consider donating this set of AI examples to here.
https://github.com/megacamelus/camel-ai-examples


Error adding node to cluster

I ran ./vmq-admin cluster join discovery-node=<node_ip> to add the current node to a cluster. It retrieves RPC to node <node_ip> failed for reason: non_empty_node. Can someone help me?

Release 2.0.0+ has a new setting: vmq_swc.prevent_nonempty_join which is on by default.
You can set it to off for backward compatibility.
But note: the point of  this is to prevent nodes with pre-existing history from joining a cluster. The setting triggers a self-check on the node that wants to join the cluster, and then prevents it from joining if it finds existing history. With existing history, metadata synchronization is not guaranteed to work (but might work, partially or fully).
existing history = all events on the broker before it gets clustered. An event could be an MQTT client registering on the broker.

👉 Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
👉 Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

I am using this git repository to deploy VerneMQ. Is this new setting active here?

Yes

👉 Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
👉 Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

If this is a bit confusing, here's more info.
In your release, you'll have a data directory with the following directories:

broker
generated.configs
msgstore
swc_meta

The history (= state) that is relevant for the self-check is metadata (mainly subscriptions). This is stored on disk in swc_meta. If an MQTT client connects and subscribes to a topic (as an example), swc_meta will be populated and prevent itself from joining a cluster.
To reset a node, it needs to be stopped and swc_meta needs to be renamed or deleted.
Then you can reboot the node, and the boot will create a new swc_meta directory.
Since you can only cluster a node when it is started, you will have an independent (=non-clustered) VerneMQ node running before you issue the cluster command. Within that time window (boot to clustering), you need to keep the node "protected" from MQTT clients.

👉 Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
👉 Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.


Extends model without fork

We are develop our own Kind (yes we know the danger of doing that). But we don't want to branch everything so we created our own model that extends the actual model.
Expected Behavior
export  ajvCompiledJsonSchemaValidator so we don't need to branch model
Current Behavior
ajvCompiledJsonSchemaValidator is not exported
Possible Solution
export ajvCompiledJsonSchemaValidator
Steps to Reproduce

Create a new plugin
Try to extends the model

Context
When we analyzed extends the model the Kinds used the schema validator that was exported on the catalog-model package so we were able to extends the schema without any issues. But today the schema validator is deprecated and the ajvCompiledJsonSchemaValidator is not exported so we can't validate the schema without branching.

I believe the exported validator above will serve this purpose.


Job or CronJob for spark-dependencies

Create a job or cronJob for https://github.com/jaegertracing/spark-dependencies.

What is the difference? Pros/Cons?

I don't know. But I have two requirements:

be able to run the job immediately with just kubectl -f
be able to schedule that job like a cron

You either run it immediately (job), or you run it scheduled (cron job). You can't have both, AFAIK :)
Reference:
https://kubernetes.io/docs/concepts/workloads/controllers/cron-jobs/
https://kubernetes.io/docs/concepts/workloads/controllers/jobs-run-to-completion

I am working on this.
I think we should go with cron job. For demo purposes scheduled time could be changed.

For demos the following command can be used:
kubectl run jaeger-spark-dependencies --env="STORAGE=cassandra" --env="CASSANDRA_CONTACT_POINTS=cassandra:9042"  --restart=Never --image=jaegertracing/spark-dependencies

It will create links for the current day.
--schedule="55 23 * * *"  can be used to schedule it. It actually creates CronJob object.
I am wondering whether we want to create a template for CronJob or just add commands to the readme.

If the templates are going to become storage specific (for production) then might be worth just including it in the main template rather than having it on its own?
It then provides complete functionality in a single template - but depends if some users don't want it.

It then provides complete functionality in a single template - but depends if some users don't want it.

They could delete it after creating all objects, although a little bit inconvenient.
CronJob template is storage specific, although storage can be configured as env var. (I don't want to copy it for all backends.)
So we have 3 options:

readme
separate resource file
include it directly in prod template

I think my preference would be readme or directly in prod template (if we will have a separate template for all backends)


[tf.data.experimental.cardinality] not working on FlatMapDataset ? [2.2.0-rc1]

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):  Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS 10.14.6
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below): v2.2.0-rc0-43-gacf4951a2f 2.2.0-rc1
Python version: 3.7.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version: - GPU model and memory: NA

Describe the current behavior
I am making some test with 2.2.0-rc1. When using data from tensorflow dataset tf.data.experimental.cardinality is returning the number of even
print(data['train'])
print(tf.data.experimental.cardinality(data['train']))

<DatasetV1Adapter shapes: {idx: (), label: (), sentence: ()}, types: {idx: tf.int32, label: tf.int64, sentence: tf.string}>
tf.Tensor(67349, shape=(), dtype=int64)

In such case I have a DatasetV1Adapter
Now when I am using Huggingface transformer that modify  the structure of the data:
train_dataset = glue_convert_examples_to_features(data['train'],
                                                  tokenizer,
                                                  max_length=128,
                                                  task='sst-2')

print(tf.data.experimental.cardinality(train_dataset))

<FlatMapDataset shapes: ({input_ids: (None,), attention_mask: (None,), token_type_ids: (None,)}, ()), types: ({input_ids: tf.int32, attention_mask: tf.int32, token_type_ids: tf.int32}, tf.int64)>
tf.Tensor(-2, shape=(), dtype=int64)

In this case this is a FlatMapDataset  and tf.data.experimental.cardinality is not able tor eturn the number of event ? Is this expected ? In which case is tf.data.experimental.cardinality working ?
The documentation for TF 2.1.0 just said:
dataset: A tf.data.Dataset for which to determine cardinality.
https://www.tensorflow.org/api_docs/python/tf/data/experimental/cardinality
For me (as a naive user) FlatMapDataset/DatasetV1Adapter are tf.data.Dataset
The way the transformer is modifying the data is by using tf.data.Dataset.from_generator
Describe the expected behavior
Be able to return the total number of entry after the data being transformer using tf.data.Dataset.from_generator
Standalone code to reproduce the issue
data, info = tensorflow_datasets.load(name='glue/sst2',
                                      data_dir=data_dir,
                                      with_info=True)
# recap input dataset
print(data['train'])
print(tf.data.experimental.cardinality(data['train']))
print(len(list(data['train']))

# Prepare data for BERT
train_dataset = glue_convert_examples_to_features(data['train'],
                                                  tokenizer,
                                                  max_length=128,
                                                  task='sst-2')

print(train_dataset)
print(tf.data.experimental.cardinality(train_dataset))
print(len(list(train_dataset)))

A full notebook can be found here:
https://github.com/tarrade/proj_multilingual_text_classification/blob/master/notebook/00-Test/08_SST2_Huggingface_model.ipynb
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

The answer is most likely that the glue_convert_examples_to_features method (for which I do not see source code) is doing something that prevents tf.data.experimental.cardinality from statically inferring the size of the output dataset.
Like shape inference, tf.data.experimental.cardinality will only produce a known answer if the answer can be determined by static analysis (i.e. without executing the input pipeline). When the input pipeline contains a flat_map, it is generally not possible to statically determine what will be the cardinality of the output from the cardinality from the input; for instance, the inputs to flat_map could be filenames that flat_map uses to create TFRecordDataset (that do not have statically known cardinality).
If you know the relationship between the input and output cardinality of your flat_map, you can provide it to the runtime using tf.data.experimental.assert_cardinality.

Hi @jsimsa,
Thanks for the follow up. I will check with the Huggingface's team. The code is here:
https://github.com/huggingface/transformers/blob/master/src/transformers/data/processors/glue.py
Thanks


Add GTM tags to Post

What does this pull request aim to achieve? 💯
Add GTM tags to Post
closes #1031

Hey @jyclam,
I'm looking at the inspect element of the build URL and I can't seem to see the "View More" id:

Am I looking at the wrong place? All the other id's look good though!!

Ooh also, for "PO_WC", there is a "_" missing between WC and the post number:

@doheekim93 thanks for catching those. Careless of me 🤦 . Fixed.


A user can edit details of a bedspace

Changes in this PR

A user can edit the details of a bedspace

This PR begins with some minor tidying to existing integration/E2E tests
Screenshots of UI changes

 A user can edit details of a bedspace


build issue on 20.04 jetpack 5.1.1 negotiated package can't be found

System information :
NVIDIA NVIDIA Jetson Xavier NX Developer Kit
L4T 35.3.1 [ JetPack UNKNOWN ]
Ubuntu 20.04.6 LTS
Kernel Version: 5.10.104-tegra
CUDA 11.4.315
CUDA Architecture: 7.2
OpenCV version: 4.5.5
OpenCV Cuda: YES
CUDNN: <IP_ADDRESS>
TensorRT: <IP_ADDRESS>
Vision Works: NOT_INSTALLED
VPI: 2.2.7
Vulcan: 1.3.204
ROS version : foxy (have tried using docker provided in isaac_common. Same issue)
Error :
Starting >>> isaac_ros_nitros
--- stderr: isaac_ros_nitros
In file included from /home/fx/isaac_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/types/nitros_empty.hpp:21,
from /home/fx/isaac_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/types/nitros_empty.cpp:20:
/home/fx/isaac_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/types/nitros_format_agent.hpp:23:10: fatal error: negotiated/negotiated_publisher.hpp: No such file or directory
23 | #include "negotiated/negotiated_publisher.hpp"
|          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
Thank you in advanced.

The negotiated package is prepared as part of our Isaac ROS Docker environment (see the Dockerfile here). If you're running inside the container we provide in the Isaac ROS Common repository and you've appropriately run source /opt/ros/humble/setup.bash, you shouldn't run into this issue.
Can you check if the negotiated package exists in your /opt/ros/humble directory inside the container?
Note that due to specific new features introduced in ROS 2 Humble, the Isaac ROS suite of packages is not compatible with ROS 2 Foxy or older versions.

Thank you for you kind response. I manage to run on docker with 20.04. (i was tested on 18.04 but failed)
So this issue can be closed.

--- stderr: isaac_ros_nitros
In file included from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/utils/vpi_utilities.cpp:11:
/home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/utils/vpi_utilities.hpp:84:2: error: #endif without #if
84 | #endif  // ISAAC_ROS_NITROS__UTILS__VPI_UTILITIES_HPP_
|  ^~~~~
In file included from /usr/local/cuda-11.4/include/cuda/std/cmath:19,
from /usr/local/cuda-11.4/include/cuda/std/complex:12,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/complex.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/tensor.hpp:30,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/multimedia/video.hpp:30,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/utils/vpi_utilities.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/utils/vpi_utilities.cpp:11:
/usr/local/cuda-11.4/include/cuda/std/detail/libcxx/include/cmath:569:101: error: ‘float cuda::std::__3::hypot(float, float, float)’ conflicts with a previous declaration
569 | inline _LIBCUDACXX_INLINE_VISIBILITY float       hypot(       float x,       float y,       float z ) { return sqrt(xx + yy + zz); }
|                                                                                ^
In file included from /home/badal/vslam_ws/build/isaac_ros_nitros/_deps/yaml-cpp-src/include/yaml-cpp/emitter.h:10,
from /home/badal/vslam_ws/build/isaac_ros_nitros/_deps/yaml-cpp-src/include/yaml-cpp/yaml.h:11,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/parameter_parser.hpp:31,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/core/parameter.hpp:29,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/core/component.hpp:25,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/allocator.hpp:21,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/multimedia/video.hpp:28,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/utils/vpi_utilities.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/utils/vpi_utilities.cpp:11:
/usr/include/c++/9/cmath:1868:3: note: previous declaration ‘float std::hypot(float, float, float)’
1868 |   hypot(float __x, float __y, float __z)
|   ^~~~~
In file included from /usr/local/cuda-11.4/include/cuda/std/cmath:19,
from /usr/local/cuda-11.4/include/cuda/std/complex:12,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/complex.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/tensor.hpp:30,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/multimedia/video.hpp:30,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/utils/vpi_utilities.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/utils/vpi_utilities.cpp:11:
/usr/local/cuda-11.4/include/cuda/std/detail/libcxx/include/cmath:570:101: error: ‘double cuda::std::__3::hypot(double, double, double)’ conflicts with a previous declaration
570 | inline _LIBCUDACXX_INLINE_VISIBILITY double      hypot(      double x,      double y,      double z ) { return sqrt(xx + yy + zz); }
|                                                                                ^
In file included from /home/badal/vslam_ws/build/isaac_ros_nitros/_deps/yaml-cpp-src/include/yaml-cpp/emitter.h:10,
from /home/badal/vslam_ws/build/isaac_ros_nitros/_deps/yaml-cpp-src/include/yaml-cpp/yaml.h:11,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/parameter_parser.hpp:31,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/core/parameter.hpp:29,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/core/component.hpp:25,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/allocator.hpp:21,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/multimedia/video.hpp:28,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/utils/vpi_utilities.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/utils/vpi_utilities.cpp:11:
/usr/include/c++/9/cmath:1872:3: note: previous declaration ‘double std::hypot(double, double, double)’
1872 |   hypot(double __x, double __y, double __z)
|   ^~~~~
In file included from /usr/local/cuda-11.4/include/cuda/std/cmath:19,
from /usr/local/cuda-11.4/include/cuda/std/complex:12,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/complex.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/tensor.hpp:30,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/multimedia/video.hpp:30,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/utils/vpi_utilities.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/utils/vpi_utilities.cpp:11:
/usr/local/cuda-11.4/include/cuda/std/detail/libcxx/include/cmath:572:101: error: ‘long double cuda::std::__3::hypot(long double, long double, long double)’ conflicts with a previous declaration
572 | inline _LIBCUDACXX_INLINE_VISIBILITY long double hypot( long double x, long double y, long double z ) { return sqrt(xx + yy + z*z); }
|                                                                                ^
In file included from /home/badal/vslam_ws/build/isaac_ros_nitros/_deps/yaml-cpp-src/include/yaml-cpp/emitter.h:10,
from /home/badal/vslam_ws/build/isaac_ros_nitros/_deps/yaml-cpp-src/include/yaml-cpp/yaml.h:11,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/parameter_parser.hpp:31,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/core/parameter.hpp:29,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/core/component.hpp:25,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/std/allocator.hpp:21,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_gxf/gxf/core/include/gxf/multimedia/video.hpp:28,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/include/isaac_ros_nitros/utils/vpi_utilities.hpp:23,
from /home/badal/vslam_ws/src/isaac_ros_nitros/isaac_ros_nitros/src/utils/vpi_utilities.cpp:11:
/usr/include/c++/9/cmath:1876:3: note: previous declaration ‘long double std::hypot(long double, long double, long double)’
1876 |   hypot(long double __x, long double __y, long double __z)
|   ^~~~~
make[2]: *** [CMakeFiles/isaac_ros_nitros.dir/build.make:188: CMakeFiles/isaac_ros_nitros.dir/src/utils/vpi_utilities.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:159: CMakeFiles/isaac_ros_nitros.dir/all] Error 2
make: *** [Makefile:146: all] Error 2
Failed   <<< isaac_ros_nitros [4min 14s, exited with code 2]
Summary: 9 packages finished [5min 20s]
1 package failed: isaac_ros_nitros
1 package had stderr output: isaac_ros_nitros
15 packages not processed


The problem of installing spyder(tensorflow) with Anaconda

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
File "C:\Users\DQWang\Anaconda3\envs\tensorflow\lib\site-packages\spyder\plugins\ipythonconsole.py", line 890, in run_script
self.execute_code(line, current_client, clear_variables)
File "C:\Users\DQWang\Anaconda3\envs\tensorflow\lib\site-packages\spyder\plugins\ipythonconsole.py", line 936, in execute_code
sw.execute(to_text_string(to_text_string(lines)))
File "C:\Users\DQWang\Anaconda3\envs\tensorflow\lib\site-packages\qtconsole\history_console_widget.py", line 47, in execute
source, hidden, interactive)
File "C:\Users\DQWang\Anaconda3\envs\tensorflow\lib\site-packages\qtconsole\console_widget.py", line 632, in execute
self._execute(source, hidden)
File "C:\Users\DQWang\Anaconda3\envs\tensorflow\lib\site-packages\qtconsole\frontend_widget.py", line 294, in _execute
msg_id = self.kernel_client.execute(source, hidden)
AttributeError: 'NoneType' object has no attribute 'execute'
Version and main components

Spyder Version: 3.2.3
Python Version: 3.6.2
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  None (NOK)
nu

This error is related to #5053 the console is failing to start.
Try to follow some of the advices of #3915 to make tensorflow work with spyder

Basically, you need to install the ipykernel module in your tensorflow environment.


test: tests/kernel/usage/thread_runtime_stats/ test fail

Describe the bug
The testcase "tests/kernel/usage/thread_runtime_stats/" gets assertion fail on both MEC15 and MEC17
Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using? ARM platform mec15xxevb_assy6853 and mec172xevb_assy6906
What have you tried to diagnose or workaround this issue?
do git bisect and found the test case is newly add by the commit of e83dde1628918e4751290d3fad8fa2519f5f1402

To Reproduce
Steps to reproduce the behavior:

west build -p always -b mec172xevb_assy6906 tests/kernel/usage/thread_runtime_stats/
check the serial log

Expected behavior
The test can pass
Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)
failed in daily tests
Logs and console output
Booting Zephyr OS build v3.1.0-rc2-15-ga8c7f825104a  ***
Running TESTSUITE usage_api
===================================================================
START - test_all_stats_usage
 PASS - test_all_stats_usage in 0.1 seconds
===================================================================
START - test_thread_stats_enable_disable
 PASS - test_thread_stats_enable_disable in 0.1 seconds
===================================================================
START - test_sys_stats_enable_disable
 PASS - test_sys_stats_enable_disable in 0.1 seconds
===================================================================
START - test_thread_stats_usage

    Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/usage/thread_runtime_stats/src/test_thread_runtime_stats.c:529: test_thread_stats_usage: stats5.average_cycles > stats3.average_cycles is false

 FAIL - test_thread_stats_usage in 0.20 seconds
===================================================================
TESTSUITE usage_api failed.
===================================================================
PROJECT EXECUTION FAILED

Environment (please complete the following information):

OS: ubuntu 20.04
Toolchain (e.g Zephyr SDK, ...): 0.14.1
Commit SHA or Version used: a8c7f825104aaead6cff624fb5e188b192664255

Additional context
None

@peter-mitsis: Is this issue related to the change of [RTOS-6829][RTOS-6830][RTOS-6833]?

@hunterhu3000: The change sets to which you refer is where this (failing) test case was first introduced. My expectation is that the cause of the bug is in one of three areas ...

The gathering of the thread runtime statistics
2A) A flaw in the test case
2B) The platform is behaving in a way for which the test was not designed

Upon my own initial investigation into the bug, the failure at line 529 (as per the details in the reported issue), the expected ordering of average times (An) for stats samplings 3, 4 and 5 not as expected. Instead of A4 > A5 > A3, it is being reported as A3 >= A5.
The expected sequence of events is ...

Helper thread has three windows of execution of 10 ticks each. A3 should then be about 30/3 (or 10) ticks long.
Helper thread then has 1 window of execution of 20 ticks. A4 should then be about 50/4 (or 12.5) ticks long.
Helper thread then has 1 window of execution of 1 tick. A5 should then be about 51/5 (or 10.2) ticks long.

The expected values for A3 and A5 have the potential to be relatively close together in terms of ticks. If there is something off about the timing, that could conceivably give the observed report of A3 >= A5.

I do not have a board with which to test the following idea. If it is indeed timing related, being off by a single tick with the numbers above has the potential to give the reported failure. To ensure that there is adequate buffering, the failing test could be modified such that the first three execution windows are of 5 ticks instead 10, and the fourth execution window could be set to 40 ticks instead of 20. This would result in expected average times for {A3, A4, A5} = {5, 13.75, 11.2}.
As I mentioned previously, I do not have the hardware with which to test whether this would resolve the issue or not.

@peter-mitsis
I have add some log before the assert(line 529)
*** Booting Zephyr OS build v3.1.0-rc3-32-g801fe1fa06bd  ***
Running TESTSUITE usage_api
===================================================================
START - test_all_stats_usage
 PASS - test_all_stats_usage in 0.1 seconds
===================================================================
START - test_thread_stats_enable_disable
 PASS - test_thread_stats_enable_disable in 0.1 seconds
===================================================================
START - test_sys_stats_enable_disable
 PASS - test_sys_stats_enable_disable in 0.1 seconds
===================================================================
START - test_thread_stats_usage
 stats1: execution_cycles, total_cycles, current_cycles, peak_cycles, average_cycles, idle_cycles: 0 0 0 0 0 0
 stats2: execution_cycles, total_cycles, current_cycles, peak_cycles, average_cycles, idle_cycles: 18 18 9 9 9 0
 stats3: execution_cycles, total_cycles, current_cycles, peak_cycles, average_cycles, idle_cycles: 27 27 9 9 9 0
 stats4: execution_cycles, total_cycles, current_cycles, peak_cycles, average_cycles, idle_cycles: 48 48 21 21 12 0
 stats5: execution_cycles, total_cycles, current_cycles, peak_cycles, average_cycles, idle_cycles: 49 49 1 21 9 0

    Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/usage/thread_runtime_stats/src/test_thread_runtime_stats.c:550: test_thread_stats_usage: stats5.average_cycles > stats3.average_cycles is false

 FAIL - test_thread_stats_usage in 0.69 seconds
===================================================================
TESTSUITE usage_api failed.
===================================================================

Theologically, it is true that "stats5.average_cycles > stats3.average_cycles". But actually the two values are very close. In the code, average_cycles is a integer value, so they can be the same value. So it seems we should change the assert to "stats5.average_cycles >= stats3.average_cycles". Or we can use a float value to record the average_cycles.

@hunterhu3000 : Apologies for the late reply ... I was out of town for 5 days.
Thanks for gathering the numbers/data. As I look at the data you provided, it looks to me that rate at which the cycle count is increasing matches the system clock tick rate.  Assuming that interpretation is correct, then it looks like the base of the problem is a timer resolution issue.
I can think of two ways to resolve this. One way is (as you suggested) to change the assert to "stats5.average_cycles >= stats3.average_cycles". The second is (as I previously suggested) to change the length of each thread's execution window.
I think your proposal is both sufficient and simpler--and thus the better option. Can you make that change (as you seem to have access to the hardware to test it), and I will be happy to review the change.

@peter-mitsis  who should be owning this bug, @hunterhu3000 ?

@rljordan I'll push a PR today

Close fixed


Vickel: Watch crackpigeon.com

Vickel requests the watch of the watch_keyword crackpigeon\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Makyen in Charcoal HQ

crackpigeon\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.


Experimental autoClosingPairs feature cursor offset

When using "experimental.autoClosingPairs": true automatically closing pairs works just fine. However when deleting the first element of the pair (e.g. an opening brace) the cursor gets offset by a character to the left.
Since I can't seem to explain it any better here's an example using <|> as the cursor position:
foo<|>
foo(<|>)  // insert an opening parenthesis; auto-closes the parenthesis
fo<|>o  // backspace the opening parenthesis; closing parenthesis is auto-removed

Since this only happens at the end of a line it might actually be related to another issue mentioned in discord earlier:

also, i was wondering why the cursor trails off instead of staying at the last character of the line?

Nice find, @necabo ! I believe this should be fixed now with https://github.com/onivim/oni2/pull/965 - made some fixes for auto-closing pairs in reason-libvim and pulled in. So I'll close this out, but let me know if you're still seeing any issues.
Going to work on having this on-by-default soon...


Updated Article Details widget to retrieve the correct document by ID

The Article Details widget is updated with the new useSearchResults syntax and it now correctly pulls the requested document. Before this change it was showing the first document regardless of the passed document ID.

Thanks @geann !


CM-923: added test

Added basic pytest test for main function from CM-923.

@cyplas @rodriguesk I've updated the test and added some recommendations ( + refactor a little). does it make sense?

The effect topics are personalized, but the solution topics aren’t at the
moment.
Once we have enough data on which topics are most engaged with by which
types of people given their personal values profiles, then we can start
doing personalized recommendations. the data we collect with this could
will enable us to personalize in the future because there’s a bit of
randomness to what we are recommending, while we still are using the most
popular topics from version 1 (so it’s not totally random).
On Thu, May 5, 2022 at 11:51 AM danmash @.***> wrote:

External Email
@.**** commented on this pull request.
In app/feed/process_alignment_feed.py
https://github.com/ClimateMind/climatemind-backend/pull/318#discussion_r866215877
:

@@ -55,12 +55,7 @@ def create_alignment_feed(
assign_alignment_iris(
alignment_feed,
"solution",

       find_alignment_solution_iris(

           CONVERSATION_SOLUTION_NAME,

           POPULAR_SOLUTION_NAMES,

           POPULAR_SOLUTION_COUNT,

           UNPOPULAR_SOLUTION_COUNT,

       ),

       find_alignment_solution_iris(),

well, since it's not personalized at all and will be improved in the
future (I hope so, otherwise it doesn't make sense) I would keep it as is
for now.
also, I suggest to rename it to avoid confusion.
I thought this part of the project is crucial to have to be able to
provide meaningful conversations. when we are planning to adjust this?
—
Reply to this email directly, view it on GitHub
https://github.com/ClimateMind/climatemind-backend/pull/318#discussion_r866215877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXL62VKOAHYPRHXIUY2RYLVIQKCXANCNFSM5UZ4JMGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@rodriguesk

Generally, I think it’s bad style to use scoping to keep track of access to constants.

sure, I think if we will use these constants somewhere else we need to move it to config. But it's ok at the moment since there's only one usage. I going to merge it as is right now.

Yeah, it’s minor, nothing to waste a lot of time over. The name changes
Daniil made helped reduce my worry about hidden constants and hidden usage
of them too. Which is good.
On Thu, May 5, 2022 at 2:48 PM Cyprian Laskowski @.***>
wrote:

External Email
@.**** commented on this pull request.
In app/feed/process_alignment_feed.py
https://github.com/ClimateMind/climatemind-backend/pull/318#discussion_r866339928
:

@@ -55,12 +55,7 @@ def create_alignment_feed(
assign_alignment_iris(
alignment_feed,
"solution",

       find_alignment_solution_iris(

           CONVERSATION_SOLUTION_NAME,

           POPULAR_SOLUTION_NAMES,

           POPULAR_SOLUTION_COUNT,

           UNPOPULAR_SOLUTION_COUNT,

       ),

       find_alignment_solution_iris(),

Generally, I think it’s bad style to use scoping to keep track of access
to constants. It’s better to have constants defined close to where they get
used or to have them be app-wide constants.
I think what happened here is that I assumed you weren't initially talking
about app-wide constants, as I didn't find similar precedents in the
codebase, and I assumed you weren't talking about constants declared inside
functions, as then I don't think of them as "constants", which left me with
this awkward middle ground. :) But indeed, we can improve this at another
time.
—
Reply to this email directly, view it on GitHub
https://github.com/ClimateMind/climatemind-backend/pull/318#discussion_r866339928,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXL62V46J2MDQZGJTPOLJ3VIQ63VANCNFSM5UZ4JMGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

The effect topics are personalized, but the solution topics aren’t at the
moment.
Once we have enough data on which topics are most engaged with by which
types of people given their personal values profiles, then we can start
doing personalized recommendations.

@rodriguesk maybe it's a good idea to synchronize proposed actions with https://www.un.org/actnow ?

We have the actions pulled  currently from Project Drawdown (I talked to them and got their permission to use their text from their website). It’s a good idea you have though, so I added it to a resource file I have for all the apps and websites that help offer more actions.

On May 8, 2022, at 12:52 AM, danmash @.***> wrote:

External Email
The effect topics are personalized, but the solution topics aren’t at the
moment.
Once we have enough data on which topics are most engaged with by which
types of people given their personal values profiles, then we can start
doing personalized recommendations.
@rodriguesk maybe it's a good idea to synchronize proposed actions with https://www.un.org/actnow ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

I'm sure we have all these actions already. I'm talking about  the  set of
actions to show to the user instead of randomized like how
On Sun, 8 May 2022 at 18:28, Kameron Rodrigues @.***>
wrote:

We have the actions pulled currently from Project Drawdown (I talked to
them and got their permission to use their text from their website). It’s a
good idea you have though, so I added it to a resource file I have for all
the apps and websites that help offer more actions.

On May 8, 2022, at 12:52 AM, danmash @.***> wrote:

External Email
The effect topics are personalized, but the solution topics aren’t at the
moment.
Once we have enough data on which topics are most engaged with by which
types of people given their personal values profiles, then we can start
doing personalized recommendations.
@rodriguesk maybe it's a good idea to synchronize proposed actions with
https://www.un.org/actnow ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

—
Reply to this email directly, view it on GitHub
https://github.com/ClimateMind/climatemind-backend/pull/318#issuecomment-1120448291,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA27BUATLP2T4KOOVJJSVSDVI7TUTANCNFSM5UZ4JMGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Are you thinking you’d like to see the same 10 things recommended as individual actions to every user? Like this graphic? https://www.un.org/sites/un2.un.org/files/actnow_action_guide_2021.pdf
There’s climate solutions that are individual actions and there’s ones that are system/societal changes/policy. We aim to show both types to users, as there’s different arguments for both individual action and societal change. Currently we show 7 actions (one or which is always to have climate conversations) and by having solutions change for each conversation with user Bs we are helping incentivize user A more conversations with people to get exposed to more solutions. In addition to showing everyone talking about climate change, we also show 3 random solutions from individual actions such as :

composting
eating lower down the food chain
talking about climate change
etc (these could pull from a pull of 10 individual actions like you said if that’s what you’re suggesting?)

Then we also show 3 random system/societal changes/policy that have been calculated by Project Drawdown to be the most impactful. These are things people can’t do individually as one person. So it requires exposure to lots of people to build support (system change).
What are you suggesting exactly? To shrink the pools of individual actions and societal actions? If we do that it will make the solutions offered for every conversation to be the same, which would get pretty boring I think for alarmed people talking to others. It would also reduce learning opportunities for alarmed people when they talk to user B.
That said, we could definitely expand options and weight things if you’re thinking of that.
The solutions shown are completely random though. They are prioritized in a data-driven way.

On May 8, 2022, at 1:44 PM, danmash @.***> wrote:

External Email
I'm sure we have all these actions already. I'm talking about the set of
actions to show to the user instead of randomized like how
On Sun, 8 May 2022 at 18:28, Kameron Rodrigues @.***>
wrote:

We have the actions pulled currently from Project Drawdown (I talked to
them and got their permission to use their text from their website). It’s a
good idea you have though, so I added it to a resource file I have for all
the apps and websites that help offer more actions.

On May 8, 2022, at 12:52 AM, danmash @.***> wrote:

External Email
The effect topics are personalized, but the solution topics aren’t at the
moment.
Once we have enough data on which topics are most engaged with by which
types of people given their personal values profiles, then we can start
doing personalized recommendations.
@rodriguesk maybe it's a good idea to synchronize proposed actions with
https://www.un.org/actnow ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

—
Reply to this email directly, view it on GitHub
https://github.com/ClimateMind/climatemind-backend/pull/318#issuecomment-1120448291,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA27BUATLP2T4KOOVJJSVSDVI7TUTANCNFSM5UZ4JMGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


dns/ddclient does not allow upper case characters in username field.

For Google Domains it is necessary for the username field to support upper and lowercase characters. However, if you enter a username with upper case characters in the GUI, validation fails stating there are invalid characters in the field.

https://github.com/opnsense/plugins/issues/2773


Add Scale method to QuantileDigest

What’s the use case for this?

What’s the use case for this?

Adding a scale method to Trino. We can do it in Trino, but I thought this is the proper place

Understood, but what's the use case for having a scale method?

The purpose of the scale method is to scale up or down QDigest summary for a given set of data. Say if we have a summary of an order delivered from point A to point B stored in QDigest, the scale function allows us to extrapolate if the point A or point B diverges or converges.

@martint Do we need any additional information here ?


[controller] Cleanup Helix disabled partition map

Cleanup Helix disabled partition map during dropping resources

During ingestion error in leader, we add  disabled partition map config in helix. But even if those resources are dropped later, the disabled map not cleared, which later might cause znode to blow up. This PR will cleanup disabled partition map (by calling enable, which will clear the node) during dropping of resources.
How was this PR tested?

unit test
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

@anaberezhnov brought up a valid point regarding whether helix resource drop API should take care of the disabled partition znode cleanup or not.
@majisourav99 Can you follow up with the Helix team on this?
We can adopt this temp solution for now and once Helix agrees and delivers such logic, we can get rid of the Venice logic.

@anaberezhnov brought up a valid point regarding whether helix resource drop API should take care of the disabled partition znode cleanup or not. @majisourav99 Can you follow up with the Helix team on this? We can adopt this temp solution for now and once Helix agrees and delivers such logic, we can get rid of the Venice logic.

yes, let me raise a ticket with Helix team on this.


TEST-#2702: add loc/iloc benchmark

Signed-off-by: Anatoly Myachev<EMAIL_ADDRESS>
What do these changes do?

[ ] commit message follows format outlined here
[ ] passes flake8 modin
[ ] passes black --check modin
[ ] signed commit with git commit -s
[x] Resolves #2702
[ ] tests added and passing

I suggest that we add some of the complex variants of selection when both rows and columns are selected from a multiindex dataframe. Here is a work in progress on adding such tests https://github.com/modin-project/modin/compare/master...gshimansky:issue-2576
I also attach a notebook with even weirder experiments on loc syntax.
multiindex_loc.ipynb.gz

I suggest that we add some of the complex variants of selection when both rows and columns are selected from a multiindex dataframe. Here is a work in progress on adding such tests master...gshimansky:issue-2576
I also attach a notebook with even weirder experiments on loc syntax.
multiindex_loc.ipynb.gz

@gshimansky added multiindex loc test

@gshimansky ready for review.


Experimental settings

Hello, thank you for your work.
I am reproducing PureACL and I have some questions.

GPU memory
I experimented on the RTX A5000 (24G) with the default setting: batch size 3, Adam optimizer with a learning rate of 10^(-4)
But, the GPU memory is exceeded even with a batch size of 2. The GPU memory fluctuates from 14G to over 24G.
I am curious about how the batch size was set. (For example, whether Distributed Data-Parallel (DDP) was used or not, 3 GPUs were used with a batch size of 1 per GPU..)

In the paper, The feature extractor weights of PureACL were initialized with the pre-trained weights from SIBCL. Could you provide the experimental results of PureACL without pre-trained weights of SIBCL?

Thank you!

During training, I used a single 3090 GPU without DDP. For handling 4 surrounding image inputs, you may need to reduce the batch size.
I haven't conducted experiments without pre-trained SIBCL weights, but I have tested across datasets. For example, training PureACL on Ford data using SIBCL Kitti weights resulted in the same level of accuracy.

I tried using the SIBCL Kitti weights from SIBCL's repository, copied encoder, decoder, and adaptation weight, but didn't end up with a good result with first epoch.
I checked out the provided weight contents, and it seems like it was trained for 1170365 steps, does it take as many steps to converge?

Yes, the training process can be quite slow, and sometimes there is an unexpected performance drop at a particular step. In such cases, I stop the training and restart from the last saved good weights, which helps to avoid this problem.


Solver Handler Transform Synchonization

Overview
Example of solver handler implementation using IMixedRealityControllerPoseSynchronizer
See https://github.com/Microsoft/MixedRealityToolkit-Unity/issues/2785
(I only added a solver handler to the parabolic pointer as trying 2018.3 may have temporarily broke prefab editing for me)

Is this solution is to put a ControllerPoseSynchronizer on every solver?

Is this solution is to put a ControllerPoseSynchronizer on every solver?

No, this doesn't make that a requirement

But to track controllers, it is?

But to track controllers, it is?

Not the Concrete type, but the IMixedRealityControllerPoseSynchronizer, yes. or else how would you get any data about any controller?

Using a controller pose synchronizer for every solver that needs to track a controller does not fulfill the acceptance criteria for #2785 of

Doesn't add another global listener to the scene

Doesn't add another global listener to the scene

It doesn't. It's using the ones we're already using.

It's using the ones we're already using.

If I create a new prefab that adds a cube to the front of my right controller via solver, I will not need to add a new controller pose synchronizer to it?

Nope, not as long as you get your right controller's IMixedRealityControllerPoseSynchronizer reference

Where can I get that reference?

It gets it for you, if the handler is already attached to a game object that has a IMixedRealityControllerPoseSynchronizer

Please feel free to take this and change it up to better fit what your needs are, open a PR on my branch, or open a new PR with your own take that tries to synchronize the pose in this way.

To clarify, back to my scenario a few comments up, if I have a brand new cube prefab and a add a solver to it, where do I get an existing reference to a pose synchronizer?

Added a DNM as this is just an example.


Encode entities in outgoing hubot messages to prevent Slack from trying to parse them.

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've been mindful about doing atomic commits, adding documentation to my changes, not refactoring too much.
[x] I've a descriptive title and added any useful information for the reviewer. Where appropriate, I've attached a screenshot and/or screencast (gif preferrably).
[x] I've written tests to cover the new code and functionality included in this PR.
[x] I've read, agree to, and signed the Contributor License Agreement (CLA).

PR Summary

Encode entities in outgoing messages to prevent Slack errors when handling the message.

Related Issues

Fixes #344

Test strategy

Added tests to verify encoding. Manually tested with Hubot
Signed-off-by: Kyle Reed<EMAIL_ADDRESS>

Changes Unknown when pulling 10e182bd1e46c94eb76851db8252e2e31d006bee on kallanreed:master into ** on slackhq:master**.

I just found that this breaks @ mentions and channel mentions which are encoded like <@U123413> and <#C12345>. I'll special case the those and submit another request.


Sanity Check: Only sequencers need to restake in s_agent_restake

Correct, current spec only has Sequencers with stake


update changelog; bump version

Codecov Report

Merging #861 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #861   +/-   ##
=======================================
  Coverage   63.01%   63.01%
=======================================
  Files          73       73
  Lines       12581    12581
  Branches     1987     1987
=======================================
  Hits         7928     7928
  Misses       3995     3995
  Partials      658      658

Impacted Files
Coverage Δ

autobahn/_version.py
100% <100%> (ø)
:arrow_up:

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 4884a73...bc2a1b3. Read the comment docs.


Removing support for certmanager.k8s.io annotations on Ingresses

We have supported certmanager.k8s.io/issuer-name as an annotation for a while now, to help users as they migrated from pre-0.11 versions of cert-manager.
We need a deprecation plan for removing these, and to communicate to the community when they are being removed.
I'd suspect at the very latest, we want to remove this in 1.0 😄
ref https://github.com/jetstack/cert-manager/pull/3038#discussion_r447645653
/kind cleanup
/area api
/priority important-longterm

Fixed in #3177
/close


• 'comp-lzo' was deprecated in OpenVPN 2.4 and has been or will be removed in a later version

Hi i get this pop up when i try to connect to the vpn..
Warning: This VPN may not connect in the future.
The OpenVPN configuration file for 'mac' contains these OpenVPN options:
• 'comp-lzo' was deprecated in OpenVPN 2.4 and has been or will be removed in a later version
You should update the configuration so it can be used with modern versions of OpenVPN.
Tunnelblick will use OpenVPN 2.4.7 - OpenSSL v1.0.2t to connect this configuration.
However, you will not be able to connect to this VPN with future versions of Tunnelblick that do not include a version of OpenVPN that accepts the options.
THIS pops up when i connect with the VPN client "TunnelBlick"
I've been using this client and the VPN script for a long time now, this just happened.

@Nyr Please see: https://community.openvpn.net/openvpn/wiki/DeprecatedOptions#Option:--comp-lzo
Looks like Tunnelblick is jumping the gun a touch.

I've been using this client and the VPN script for a long time now, this just happened.

I'm sure, because the script hasn't enabled compression for a long time already :)
Tunnelblick is trying to be helpful but indeed, they jumped the gun on this. This is a non-issue and has already been reported multiple times here. I'll open an issue with Tunnelblick.

From the original issue:

you can avoid that warning switching from comp-lzo to compress lzo in your client.


Simplify license deployer, support nested folders

This PR simplifies the license depolyer, while also adding support for nested license folders
It fixes a bug where the folder structure would be left in the output folder, but which didn't contain the licenses themselves. We now directly zip the licenseses
Closes #161
cc @lia-viam

Thanks for the super quick fix/turnaround time! This looks a lot cleaner and I copied the file locally and verified it now correctly handles OpenSSL and nlohmann JSON :)


Bug - Ionic 3 (ion-textarea) for iOS "Select | Select All | Paste" are not working

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[ X ] 3.x
I'm submitting a ...  (check one with "x")
[ X ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
On an <ion-textarea> and under iOS, the user cannot perform Select, Select All, Paste actions.
Expected behavior:
The user should be able to do all the stated action on an <ion-textarea> as on the native one.
Steps to reproduce:
Simply put <ion-textarea> and try to run it under iOS, try to any dummy text and then try to edit it, select it or even copy/paste content (you won't be able to do it, that's the bug !)
Note
The issue may appear also in Android, I haven't test it yet!
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.3.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.0
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: macOS Sierra
Node Version: v4.4.7
Xcode version: Xcode 8.2.1 Build version 8C1002

Did you put the <ion-textarea> in an  <ion-item>? I had this problem a while ago and this fixed it for me.

@judithhartmann Works like a charm using the way you suggested. I believe they need to talk about it in the official docs !
Thank you!


How to make parameters to lower case?

Hi, we have an existing REST API with many clients using it and i was trying generate swagger.json file using annotations and was successfully able to create and host it. But the problem is with query parameters.
The existing URL looks something like this: https://swagger/employee?id=value&project=value
When i use annotations, the parameters structs fields must be exported, so that the swagger:parameters annotation will recognize the params. This is fine in case where there are no query params in the URL.
But in query params case, after generating swagger.json file with annotations  the URL will look something like this: https://swagger/employee?Id=value&Project=value , where the first letter of the param key is always upper case and failing to fetch the data. This is because of exporting the fields in parameters struct.
Is there a way to generate these in lower case other than changing the API code and making the clients not to use new URLs ?
Environment
swagger version: 2.0
go version: 1.8
OS:  windows

you can reuse the json struct tag to provide a different name for those properties

Great ! Thanks for the reply :)


Error on flushing Lucene index

OrientDB Version, operating system, or hardware.

v2.2.14

Operating System

[x] Linux

Expected behavior and actual behavior
After index rebuild, the message appears (was told to be resolved in 2.2.14):
Error on flushing Lucene index
java.lang.NullPointerException
        at com.orientechnologies.lucene.engine.OLuceneIndexEngineAbstract.flush(OLuceneIndexEngineAbstract.java:282)
        at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.synch(OAbstractPaginatedStorage.java:2261)
        at com.orientechnologies.orient.core.index.OIndexManagerShared$RecreateIndexesTask.run(OIndexManagerShared.java:527)
        at java.lang.Thread.run(Thread.java:745)

hi @rdelangh
seems a different exception in terms of stacktrace
let me check
Where did you get this exception? On which server?
It should be

@maggiolo00
I got the exception in the logs of the single server that is running. The server has been started with the stand-alone mode (i.e. not distributed).

@rdelangh
when do you get this exception?
by manually running
rebuild index
?

@maggiolo00
I have not launched the index rebuild manually, this was done automatically by the server startup (after a sudden death).

So what happen was:

server running smoothly for a while, with lucene indexes
server crashes due to: reason unknown
server restarted: it rebuilds indexes
after OR during rebuild of Lucene's indexes, the server logs the excetion

Please confirm my doubts:

reason of crash
when and where the server logs the stacktrace

Thanks a lot

@robfrank

reason of crash : the O/S crashed (paniced), not the OrientDB server process only
no stacktrace from the process, since O/S was gone suddenly

After the reboot, the ODB server process was started, which started the index rebuilds. At the end of these rebuilds (when all of these indexes were rebuilt) the log messages were spawned .

@robfrank
it happens also still when I manually create a new index:
...
2017-01-26 22:51:57:966 INFO  {db=cdrarch} --> 99.94% progress, 29,089,070 indexed so far (58,281 items/sec) [OIndexRebuildOutputListener]
2017-01-26 22:51:58:711 INFO  {db=cdrarch} --> OK, indexed 27,012,134 items in 994,128 ms [OIndexRebuildOutputListener]$ANSI{green {db=cdrarch}} Error on flushing Lucene index
java.lang.NullPointerException
$ANSI{green {db=cdrarch}} Error on flushing Lucene index
java.lang.NullPointerException

Hi @rdelangh
do you still need help on this or it has been resolved?
Thanks

Hi @rdelangh
any news about this issue?
Thanks

hi @maggiolo00
That issue seems not to occur anymore with the versions which I run now (2.2.18, and recently 2.2.23)
-> can be closed.

Cool, closing it.


XD7Database not populating DependsOn

Hi Iain,
Just started playing DSC and these (beautifully written) resources. So far i have used the following resources to test this out:
File - to copy a zip over of the XD7 media top the test server
Archive - to then unzip the media
XD7Feature -  to install the controller roll
XD7Database - to set up the XD7database
I have noticed a oddity or two with the XD7Database resource.

When adding depends on, i am not able to enumerate any other resources but can manually type the resource i am wanting to depend on.

If i run the following resources in depends on order File, Archive then XD7Feature -  this works perfectly fine. However if i then run File, Archive, XD7Feature then XD7Database then dsc starts to error out complaining (The PowerShell DSC resource MSFT_ArchiveResource from module <PSDesiredStateConfiguration,1.0> does not exist at the PowerShell module path nor is it registered as a WMI DSC resource.) IF i remove XD7Database  then it works fine again. As a work around i can edit the MOF and set the archive resource to 1.1 then it works fine.

Please let me know if you need any more info.
Thanks
Alex

Hi Alex,
Glad you find them useful!
I think you may be suffering from the “DSC resource dependencies cannot contain a number” bug/feature. What versions of OS/WMF are you a) generating the MOF files on and what version(s) are on the target nodes?
Thanks, Iain

Hi Iain,
Thanks for the quick reply. I am running WMF 5 RTM on Server 2012R2 for both the authoring server and the XDController. This also happened with WMF5 prod preview.
Thanks
Alex

Hi Ian,
Apologies, I have found out why the depends on was not populating. I have a node statement for the controllers to install that roll then I have a second node statement for the primary Controller which also is the database creator. The population of the depends on is restricted to that node statement block.
Ill take up the module version on powershell.org and see if anyone else is having that problem too.
Thanks for your time and have a great Christmas
Alex

Hi Alex,
No problem.
The issue is on the target where the MOF is applied? Do you somehow have 2 versions of the Archive resource on the target that's been upgraded?
Have a great Christmas yourself :D
Iain


Feature request: Presign Google Urls

I noticed that the GCS backend is missing pre signing URLs. I think it should be relatively trivial to add with reqsign already being used there. I'll get around to making a PR at some point, but I thought Id already leave an issue here for future reference.

Sounds great!
But presign google URLs is a bit more complex than expected. The current signing logic uses google's JSON API along with OAuth2 Token which can't do presign. We need to add V4 API support which silimiar to AWS Sigv4: https://cloud.google.com/storage/docs/access-control/signed-urls

I think the algorithm is pretty much exactly like AWS V4. The official docs have an example that uses RSA-SHA256 for signing, but HMAC-SHA256 seems also available (https://cloud.google.com/storage/docs/authentication/signatures) which means that no additional dependencies should be needed.
I've started to copy over the aws v4 file and tried to make the adjustments for google::v4 but haven't tackled the config loader part yet :)

I have assign this issue to @wolfv & @baszalmstra. Any problems you encounter can be communicated here and I am willing to help :heart:

Hi, @wolfv & @baszalmstra, presign support for gcs has been added now! Are you still willing to use this feature?


When Click CheckBox error occure

When click the check box we getting error. I am not include any thing. I have this data only<CheckBox rightText={"Save Password"}>

#79 will fix this


Scoop git-with-openssh integration

Scoop installation of "neofetch" doesn't admit "git-with-openssh" as a "git" package alternative

[X] Yes, this (issue / lack of feature)  does still occur in the master branch !

This improvement is a easy task, only need for a dual scoop prefix [ git | git-with-openssh ] check...
Neofetch 7.1.0

I don't know what any of this means.


Provide totals filters for a wide variety of choices

Let's bring some granularity to totals filters.
Add filters for weapons, abilities, armors, rings, keys, dyes, skins, etc, etc, and then take it further and offer filters for swords, bows, spells, seals, etc.
A lot of miscellaneous items fall under the same SlotType of 10. They can be distinguished via other settings in constants (e.g. bagtype, soulbound, fp).

Dependent on #176

Published in v9.3.0.


Enforce schema by version

Update validateSchema to take a list of version to match again (for the case that an application can support multiple versions), and also to use the schemaVersion specifed in the protocol itself for the validation.

If protocol doesn't match any provided versions, it returns that as error.
If the declared schema does not exist it returns that as an error.
Finally it returns any errors created by running the schema itself against the protocol.

This also migrates to use plain numerical version format. Because previous protocols declare their schemaVersion as '1.0.0', this has been added as an additional schema, but is simply a clone of '1'.

One thing I haven't tacked here is the command line tool for protocol validation, perhaps it might be worth updating the syntax in order to support specifying the version?


Making a drill session custom layout

Hey! I tried to make a drill layout to exercise but failed to find the right place to do it.
Could you please give a hint on how to do it most simply?

You'll have to be more specific.

 E.g. six balls near each corner.
And the level up would be to spawn a ball after each pot

Sorry, that's not easy right now.
You could make a specific layout with the infrastructure provided in pooltool/layouts.py and then make a specific game with the infrastructure provided in pooltool/game/. Finally, change line 543 of pooltool/ani/animate.py to select your new practice mode.


HvPoI

Implementation of the HvPoI multi-objective optimization algorithm. Before this can be merged some steps are required.
I'm quite ok with the acquisition implementation, however I can't say the same about the Pareto class. This code is the result of two refactors based on port of the original MATLAB code. I think it already looks better but as several multi-objective strategies are based on hypervolumes, this makes that class quite significant in our framework. Its not up to standards yet. It also needs documentation. To improve this, I'm counting on you @icouckuy . Currently, our test coverage also misses 2 lines in pareto.py, but I understood the levels of improvement can be removed?
In addition, a notebook might be interesting also.

Codecov Report

Merging #36 into master will decrease coverage by 0.17%.
The diff coverage is 98.87%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
- Coverage   99.69%   99.52%   -0.18%
==========================================
  Files           8        9       +1
  Lines         660      838     +178
==========================================
+ Hits          658      834     +176
- Misses          2        4       +2

Impacted Files
Coverage Δ

GPflowOpt/__init__.py
100% <100%> (ø)
:arrow_up:

GPflowOpt/acquisition.py
100% <100%> (ø)
:arrow_up:

GPflowOpt/pareto.py
98.5% <98.5%> (ø)

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 7bba551...e3b057a. Read the comment docs.

Added notebook
BOOptimizer now returns OptimizeResult with the Pareto front

Next up: removing obsolute code in the cell bound calculation, and making it faster if possible.

While originally for GPflowOpt 0.2, we are thinking of merging this multiobjective acquisition function for 0.1:

the code is very independent and non-invasive
the code has been validated multiple times against the original matlab implementation so should be correct

If noone has objections I will merge it in a couple of days


Updates for CI and Nuget release

Rename to Eto.OpenTK
Migrate packages.config to PackageReference
Add to Azure Pipelines CI
Convert projects to sdk-style (where possible currently)
Support building nupkg
Added platform-specific build configuration platforms so we can build the .sln from CI without getting errors.

I made note of my own fork of this project in issue #24, but to point out one particular change that might be pertinent: I was able to make an SDK-style version of the Gtk2 project. As I recall the key to get it working was
<AssemblySearchPaths>$(AssemblySearchPaths);{GAC}</AssemblySearchPaths>

to allow the newer projects to search the Global Assembly Cache for GtkSharp2.

@philstopford thanks for the typo fixes!  I wonder why CI didn't catch those.. I'll need to investigate.

@ItEndsWithTens awesome, I'll give that a try!  I really would like to just move everything over to sdk-style csproj's.. the day is nigh when that will happen methinks thanks to your tip.
I'm also seeing some commits/PR's go into Xamarin.Mac regarding nuget packages, so hopefully that'll work much better too.


Keras: removing layers by creating new model not working

Please make sure that this is an issue related to performance of TensorFlow.
As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:performance_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): Colab
TensorFlow version (use command below):2.4.2
Python version:3.8
Bazel version (if compiling from source): Colab
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: Google Colab
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Problem facing
Unable to remove batch normalization layers from Keras DenseNet121 model
Describe the expected behavior
I am trying to remove batch normalization layer from DenseNet121 model by creating a new model and transferring layers, except Batch normalization layer, from DenseNet121 model to the new one.
[code provided below]
Describe the current behavior
Error was "ValueError: Input 0 of layer conv1/bn is incompatible with the layer: expected axis 3 of input shape to have value 64 but received input with shape [None, 512, 512, 3]"

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
https://colab.research.google.com/drive/1Efi-rVRZRzVWCJuPy06rm4doK_C2J2J_?usp=sharing
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
//code
model1=Sequential()
for layer in model.layers:
if(layer.name.find('bn')!=-1):
model1.add(layer)
input_shape = (None,512,512,3)
model1.build(input_shape)

@aditya-sarkar441,
On running the Colab notebook, I am facing an error stating FileNotFoundError: [Errno 2] No such file or directory: 'drive/My Drive/sscheat/train/normal'.
Could you please share all the supporting files necessary to run the notebook? Thanks!

Yeah, the link of the file is written below:
https://drive.google.com/drive/folders/1_mCr8xjjDXM7Myk95Fa1nxGxt7AWDIry?usp=sharing
Data will not be used in the model as I have not fitted the model on the data. I'm just trying to compile model with removed batchnormalisation layers from original keras Densenet121 model.
You can try running from second block after importing all the python modules.

@aditya-sarkar441,
On running the code I am facing an error stating OSError: Unable to open file (unable to open file: name = 'drive/My Drive/densenet.h5', errno = 2, error message = 'No such file or directory', flags = 0, o_flags = 0)
Could you please share the densenet.h5 file and the contents of filepath = 'dn_1.{accuracy:.4f}-{val_accuracy:.4f}.h5' as well? Thanks!

https://drive.google.com/file/d/1CoUDfv62Wbr3F64x2b3cEPOG5hVAjSDZ/view?usp=sharing
Here are the weights.
You will not be requiring dn_1.{accuracy:.4f}-{val_accuracy:.4f}.h5. I've removed that section from the code. I've issue only in building model. You can download the weights, and then use it for running my code.

Was able to reproduce the issue with TF v2.3 and TF-nightly. Please find the gist of it here. Thanks!

@amahendrakar It's still showing some error.

@aditya-sarkar441 Looks like your model1 has BatchNorm layers. When I tried to print(model1.layers[3].name) --> it prints conv2_block2_0_bn.
AFAIK The root-cause of the issue is Sequential model cannot be used for concatenation. Sequential model is for single-input and single-output. Thanks!

Ok, so @jvishnuvardhan how can I remove batch normalization layers from Densenet121 model ? Is there a way ?

@jvishnuvardhan have you got any solution?

Poor solutions from Google Tensorflow. Thanks !!

Poor solutions from Google Tensorflow. Thanks !!


explorer address must end with a slash

When trying to start dbreboot, explorer address must end with a slash
/target/release/dbreboot --explorer https://explorer.grid.tf/explorer/ --seed-file ghanem.seed -l <IP_ADDRESS>:50051

If it's passed like  --explorer https://explorer.grid.tf/explorer (without a trailing slash), join method won't work as expected and get_user request will be sent to https://explorer.grid.tf/users instead of https://explorer.grid.tf/explorer/users, which will fail with 404:
panic: rpc error: code = Unauthenticated desc = could not get user key: failed to get user: invalid type: integer `404`, expected struct User at line 1 column 3

Fixed


fatal error: ATen/cuda/CUDAGuard.h: No such file or directory

🐛 Bug
I'm facing the following error when trying to run python setup.py build develop:
/usr/local/cuda/bin/nvcc -DWITH_CUDA -I/content/maskrcnn-benchmark/maskrcnn_benchmark/csrc -I/root/miniconda/lib/python3.7/site-packages/torch/include -I/root/miniconda/lib/python3.7/site-packages/torch/include/torch/csrc/api/include -I/root/miniconda/lib/python3.7/site-packages/torch/include/TH -I/root/miniconda/lib/python3.7/site-packages/torch/include/THC -I/usr/local/cuda/include -I/root/miniconda/include/python3.7m -c /content/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cuda/SigmoidFocalLoss_cuda.cu -o build/temp.linux-x86_64-3.7/content/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cuda/SigmoidFocalLoss_cuda.o -D__CUDA_NO_HALF_OPERATORS__ -D__CUDA_NO_HALF_CONVERSIONS__ -D__CUDA_NO_HALF2_OPERATORS__ --compiler-options '-fPIC' -DCUDA_HAS_FP16=1 -D__CUDA_NO_HALF_OPERATORS__ -D__CUDA_NO_HALF_CONVERSIONS__ -D__CUDA_NO_HALF2_OPERATORS__ -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=_C -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++11
/content/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cuda/SigmoidFocalLoss_cuda.cu:7:10: fatal error: ATen/cuda/CUDAGuard.h: No such file or directory
 #include <ATen/cuda/CUDAGuard.h>
          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
error: command '/usr/local/cuda/bin/nvcc' failed with exit status 1

To Reproduce
Steps to reproduce the behavior:

clone this repository (master, bd39d2c1019200a84a0799976e96cf96fdb0d573)
install requirements
build cocoapi with python setup.py build_ext install
run python setup.py build develop

Expected behavior
No error.
Environment
Please copy and paste the output from the
environment collection script from PyTorch
(or fill out the checklist below manually).
You can get the script and run it with:
wget https://raw.githubusercontent.com/pytorch/pytorch/master/torch/utils/collect_env.py
# For security purposes, please check the contents of collect_env.py before running it.
python collect_env.py

PyTorch Version (e.g., 1.0): 1.0.0.dev20190325
OS (e.g., Linux): Ubuntu 18.04.1 LTS
How you installed PyTorch (conda, pip, source): conda
Build command you used (if compiling from source): python setup.py build develop
Python version: 3.7
CUDA/cuDNN version:

CUDA runtime version: 10.0.130
Nvidia driver version: 410.79
cuDNN version: /usr/lib/x86_64-linux-gnu/libcudnn.so.7.5.0

GPU models and configuration: GPU 0: Tesla K80
Any other relevant information:

Additional context
I can build that with no error when I checked out 9063850dc3069dce9d6a8ce9f65f8449b1cd3be7 in the same environment.

I've reverted the offending commit, this should fix the error you were having

Thank you!