Proposal: Add --plugin flag instead of manually bind-mounting socket

Basically, a docker plugin is essentially this:
docker run -d -v /var/run/docker.sock:/var/run/docker.sock cpuguy83/something_crazy
And if you want to share that with people, in the instructions we are telling them to do the same thing, but not everyone necessarily uses the unix socket, and even if they do it may not be in this location.
In addition, for such a common pattern this is pretty silly.
Instead I propose that we can automatically bind mount a socket if requested using --plugin from the CLI.
docker run -d --plugin cpuguy83/something_crazy
This would have a default path for placing the socket in the container, but could potentially be customizable.

I don't like this at all right now.  We have a flag for manually bind mounting files.  Explicit is better than implicit and we don't want to define a flag right now that sets the tone for future features.  What if two months down the road plugins are not containers that bind mount in the socket?  We are screwed because we have a flag like this.
Lets not make any changes like this until we have requirements for what a plugin is.  If you want the socket mounted just use -v.  There is no reason to abstract this away.


Apps with map, map views don't work 7.1

So I have "Nominatim", "Mozilla Location Service", and "WiFi Location Service" along with the microG stuff all installed from F-Droid. I also flashed the Legacy v1 Maps zip but nothing seems to work.  I thought apps that used Google Maps as a backend instead got OpenStreetMaps instead.
One app I'm noticing this on is the Starbucks app.
System:
Moto X Pure (Clark)
LineageOS 14.1 with "NLP" and "Signature Spoof" patches compiled in.

@ShapeShifter499: I suggest to post a logcat.


Encrypting raft logs

I am trying to decrypt docker swarm raft logs, edit them and finally encrypt them.
I found out that I can decrypt raft logs using swarm-rafttool decrypt, now I am wondering how to encrypt them back.
Any ideas?

@hichamuntitled We do not provide a tool to encrypt them back - swarm-rafttool decrypt is more of a debugging tool so you can see raft history or downgrade.  We do not want to encourage editing raft logs manually - however, you can probably write a tool to [get the DEK from the TLS key](based on the code in https://github.com/docker/swarmkit/blob/master/cmd/swarm-rafttool/dump.go#L32-L39), create an EncryptedRaftLogger, and write the modified entries and snapshots to a new location.

@cyli The idea behind this is that I want to be able to restore all swarm managers in new VMs, so different IPs than the ones stored in raft logs.

@hichamuntitled Would something like https://github.com/docker/swarmkit/pull/2199 fix your issue instead?

@hichamuntitled Also, I'm dumb.  Before raft encryption was added, the raft logs were stored in wal-v3 and snap-v3 directories.  After encryption was added, they were stored in wal-v3-encrypted and snap-v3-encrypted.  Possibly, if you wanted them re-encrypted, you could just move the decrypted directories to wal-v3 and snap-v3, and restart swarm - I believe it should be able to read the older format for logs and migrate them to the new, encrypted format.


:seedling: Change finding Values to map[string]string

What kind of change does this PR introduce?
follow-up of #3558

[X] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
The finding.Finding.Values maps strings to ints. This leads to problems with probes which want to store non-int values.
What is the new behavior (if this is a feature change)?**
The finding.Finding.Values maps strings to strings. More things can be represented as a string, including ints (with an extra conversion step).

[X] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
Related to #3824 (see "Why aren’t things like this already?") and #2928

Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

Leaving as a draft while I figure out coordination with #3829

/scdiff generate CII-Best-Practices,Maintained,Token-Permissions,Pinned-Dependencies,SAST,Signed-Releases,Branch-Protection

For some of the keys that are used across multiple probes, like  ReleaseNameKey or BranchNameKey, it would be nice to define them once, but I'm not sure where.

My goal was to have everything needed to interpret the probe, defined in the same package. Which leads to a little duplication.


Playing Videos In Succession

Hello! I am making a video chat startup that uses this library. The app downloads the video files from the server, get a URI in the system for it, and finally playing those videos in a local system of sorts. The last part is the issue, as react-native-video will only take one video or an ONLINE stream or video. How could I make these videos the app saves play like a stream?

It is on the todo list but I needed it now.  This is what I am doing currently - I have two video players in my view, each with an opacity setting such that only one will show at a time and each with a paused setting so one is always paused and the other running. I load the videos into the players such that the next video is always paused and has opacity 0.  My render looks like this:
        const videoStyle = StyleSheet.flatten(styles.video);
	return (
                <View style={styles.videoContainer}>
                    {this.state.uri0 ? <Video
                        key={'video1'}
                        source={{uri: this.state.uri0}}
                        style={{...videoStyle, opacity: this.state.opacity0}}
                        paused={this.state.paused0}
                        resizeMode='contain'
                        onEnd={() => this.videoEnd()}
                    /> : null}
                    {this.state.uri1 ? <Video
                        key={'video2'}
                        source={{uri: this.state.uri1}}
                        style={{...videoStyle, opacity: this.state.opacity1}}
                        paused={this.state.paused1 || this.props.paused}
                        resizeMode='contain'
                        onEnd={() => this.videoEnd()}
                    /> : null}
                </View>
      )

The setup of the clips is like this
    setupClips() {

        // Current video drives clip selection and opacity setting (e.g. show/hide)
        const currentVideoPlayer = this.state.currentClip % 2;

        // Determine which clips go in slot 0 and 1
        const lastClip = this.state.currentClip === this.props.clips.length - 1;
        const currentClip = this.props.clips[this.state.currentClip].uri;
        const nextClip = lastClip ? '' : this.props.clips[this.state.currentClip + 1].uri;
        if (currentVideoPlayer === 0) {
            this.setState({uri0: currentClip, paused0: false})
            this.setState({uri1: nextClip, paused1: true})
            this.setState({opacity0: 1, opacity1: 0})
        } else {
            this.setState({uri1: currentClip, paused1: false})
            this.setState({uri0: nextClip, paused0: true});
            this.setState({opacity1: 1, opacity0: 0})
        }
    }

In the onEnd handling I move the clip index along and call the setup again
        if (this.state.currentClip < this.props.clips.length - 1)
            this.setState({currentClip: this.state.currentClip + 1}, () => this.setupClips());

Maybe there are easier ways but this is pretty flexible and should in theory (and from what I see in practice) avoid lags on just loading up the videos into the same player.  Curious if others have found different ways to do this.

Seems like it would work.

TODO: Add support for playing multiple videos in a sequence (will interfere with current repeat implementation)

Is there any idea of when this would be? @selsamman 's workaround seems good enough, but built-in support would be awesome.


Release!!!1!!1 D: D: D:

Lots of progress has been made, but a mostly feature complete version of the updated kanye needs to be pushed to production ASAP. Once again: Congrats! But seriously get this done right now. Go! Go! Go!

3d61aad is a fantastic example of why not to test in production, you genius. It's literally the only bit of sensitive information that the bot holds.


[icon] add ShareAlt icon

Motivation and Context
We have request for the new alternative icon for social sharing features.

How has this been tested?
Tested it locally with our playground.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Nice improve.

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly or it's not required.
[ ] Unit tests are not broken.
[x] I have added changelog note to corresponding CHANGELOG.md file with planned publish date.
[ ] I have added new unit tests on added of fixed functionality.

Add task number in title please [TASK_ID][icon] add Share...


i18n(fr): Add reference/errors/astro-response-headers-reassigned

Description (required)
Add reference/errors/astro-response-headers-reassigned in French language

Related issues & labels (optional)

Suggested label: i18n

Closed because https://github.com/withastro/docs/pull/7444


add http status check for download_file

This pr add http status check for download_file and reports error if it fails to download file.
I met a issue that curl does not report error when it failed to pull model from huggingface through proxy and get http error 503 .
But the output still is that
Downloaded gpt2_124M_bf16.bin to /home/bily/llm.c-fork/dev/../gpt2_124M_bf16.bin
despite that it does not downloads the file successfully.

Hi @miro5lav , thanks for review. Do you how this pr can be merged? It looks like it has to go through workflow that can only be triggered by maintainer.


Workshops with tag "circuits" don't trigger the 7 days post-workshop email

example: https://amy.carpentries.org/workshops/event/2020-07-21-nmcsa-online/
cc: @Talishask

The reason why Circuits events are omitted is that "LC" /  "DC" / "SWC" is expected. I can extend the list to cover Circuits as well, however please confirm what should show in the email subject (the template is Completed {DC, SWC, LC} workshop at {site} on {dates}).

@sheraaronhurt - Piotr is working on an update so that the post workshop email also goes out for Circuits/Mix-n-Match workshops. For the Circuits workshops specifically - would you like the subject of the message to :

include the word circuits like: Completed Circuits workshop at {site} on {dates}).
or
show DC/LC/SWC but nothing if the workshop is circuits like: Completed workshop at {site} on {dates}).

Thanks @Talishask and @pbanaszkiewicz
I would prefer the 2nd option: show DC/LC/SWC but nothing if the workshop is circuits like: Completed workshop at {site} on {dates}).
Could it say Completed "Carpentries" workshop? If that require more work then, I would say to keep it as is.

@Talishask @sheraaronhurt currently the subject for 7days-post-workshop email template shows:
Completed {% if workshop_main_type %}{{ workshop_main_type }}{% endif %} workshop at {{ workshop.venue }} on {{ dates }}

The part {% if workshop_main_type %}{{ workshop_main_type }}{% endif %} conditionally displays DC/LC/SWC. In case of Circuits this part will not display anything. @fmichonneau can add {% else %}"Carpentries" in the middle to support Completed "Carpentries" workshop scenario.


⚠ ClamAV has degraded performance

In 988ec50, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 79 ms

Resolved: ClamAV performance has improved in 7b2e364.


New version: NexusMods.Vortex version 1.9.10

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:


odottaa has no default export

Recently working with Playwright and saw that odottaa adds some helpful assertions. So I decided I'd give it a try and see if I liked it.
After installing and following the instructions to add it to my playwright.config.ts.
import { defineConfig, devices, expect } from '@playwright/test';
import playwrightApiMatchers from 'odottaa';

My IDE Is warning me about this. TS1192: Module '"/node_modules/odottaa/lib/src/index"' has no default export.
Doesn't seem to cause an issue, but would be nice if this didn't show as an issue on that file.

Hey @abodnar
Thank you for using the library. That's weird. Can you please provide more details(os, typescript, playwright, odotta versions) on it? It would be fantastic if you could add a working example.

Appreciate you looking into it. The more I've played with Playwright, the more I'm not sure it is the right tool for API testing.


Unit tests for the Google.Cloud.Diagnostics.AspNetCore Error Reporting Exception Logger.

Oh I see I meant the Headers["User-Agent"] was a StringValue.  I think we were talking about two different things :)


build: pin versions of NPM global deps in Makefile

Motivation

Make sure that there are no version inconsistencies due to NPM global deps installed in the Makefile (which is used locally as well as in CI)

This has been a bit concerning to me ever since I noticed the installs are not pinned 😅
This also makes the "# update this in Nix" comments make more sense as there's a version specified now

Modifications

add @<version> for each npm i -g

use the versions from dev/nix/node-packages.nix and dev/nix/package.json#dependencies

Verification

Ran in my devcontainer successfully:
npm i -g<EMAIL_ADDRESS>npm i -g<EMAIL_ADDRESS>

Thanks!


Added AnkiWeb

Just to clarify, does the link provided bring you to the dialog prompt where you enter your e-mail address to confirm account deletion, or does it send you to your account page where you have to find the "Delete account" button? Seems either the link or the notes should change slightly to reflect that. (if it's the former, remove the first sentence of the notes; if the latter, the link should probably change)

The link takes you to the removal form (if you were logged in). Alternatively, you log in first, then click on "Account", then click on "Delete Account".

If it's not too much trouble, could you remove the first line of the notes then? If the link brings you straight to the removal form (which is consistent with entries for other sites), then the notes should only explain what need be done from there.

Done as requested and added a new entry.

Sorry, I had meant to get back to this after your latest commit but lost track of it. Similar question for AnkiApp—is there any direct "delete" link that can be provided, or a link to the account page, or does it require you to log in to the main page of the site through to the account page and the delete link?

There‘s no direct delete link - the URL takes you to your account first from where you can delete the whole shebang.

Well, that works for me. Thank you for your contribution!


Task submission enhancements

Add "Submit and End Session" button so that workers can submit their results and go back to Feed or Dashboard
Improve task text to be more readable, fine print, black text.
Other improvements to be specified

two default buttons the rest should go under More (horizontal or vertical more icon with drop down menu would be an idea)

add timer, in case timer has expired let the worker know that this task may be revoked from him in case someone else wants to attempt this task

#498 can be closed as it captured the same problem


vscode and atom require -w flag

When hooking up Git with vscode or atom (and others?), the editor needs to be set with -w flag to wait for the editor to close, otherwise Git aborts the commit with an empty commit message.

@essihe, can you give me a hint on how vscode should be configured...?
The installation instructions are here, I'm not sure if this should go there or in this lesson... https://coderefinery.github.io/installation/

What I meant here is that in ~/.gitconfig it may not be enough to say editor = vscode but rather editor = "vscode -w".

@essihe, can you give me a hint on how vscode should be configured...?
The installation instructions are here, I'm not sure if this should go there or in this lesson... https://coderefinery.github.io/installation/

I'm not sure I understand what is wrong. To use Git with vscode I didn't have to configure anything, comes automatically when making changes in a repository.

This ticket here is about the situation where somebody is working in the command line (during workshop) but prefers to edit commit messages with vscode. I have seen in 1-2 cases that without the -w the command line opens vscode for few milliseconds but then closes it again before one has the chance to write the commit message. If one works within vscode, not the command line, then this problem is not relevant.

I think this is dealt with now in our configuration videos.


Override OpenAIWrapper | hook streams and other client mechanisms

Why are these changes needed?
Give end-developers a way to hook the openai wrapper without having to duplicate huge blocks of code. This can be used to easily hook streamed responses with a generator. Now it is relatively painless to override client.chat.completions.create for instance.
I do realize Custom Client support #831 exists. This PR is something I implemented to accomplish a goal ahead of such merges, and so I share.
Related issues
Closes #1199
There are many related issues and ongoing works. I'll just reference #217 for brevity.
Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8f065e0) 30.59% compared to head (d1fd79b) 18.86%.
Report is 16 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1217       +/-   ##
===========================================
- Coverage   30.59%   18.86%   -11.73%
===========================================
  Files          30       32        +2
  Lines        4256     4368      +112
  Branches      983     1070       +87
===========================================
- Hits         1302      824      -478
- Misses       2861     3449      +588
- Partials       93       95        +2

Flag
Coverage Δ

unittests
18.86% <100.00%> (-11.69%)
:arrow_down:

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

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

Thanks for the PR! I think this is very much inline with the custom client effort @olgavrou is doing also with the discussion @davorrunje and I had in #1227. I am wondering if you are interested in continuing the work in #831? We can have a new PR as the old one seems quite outdated.

Thanks for the PR! I think this is very much inline with the custom client effort @olgavrou is doing also with the discussion @davorrunje and I had in #1227. I am wondering if you are interested in continuing the work in #831? We can have a new PR as the old one seems quite outdated.

np. Sure, I'll do it. I have continue in this vein anyway for my work.
I'll make a new issue and/or draft outlining what should be done, and continue from there. We need to be able to pass arbitrary data from any initiating message through to its response(s).

I presume somebody is going to ask me to write a test here. I have been using the PR with success for a few days, and will add a test shortly.

For anyone coming here from other PRs/issues looking for various forms of overrides and hooks, it looks like the major effort for that will be: https://github.com/microsoft/autogen/pull/1240

I presume somebody is going to ask me to write a test here. I have been using the PR with success for a few days, and will add a test shortly.

Let's focus on working on this #1240 first? Regarding wrapper client we have many hands already.

@bitnom There have been many recent changes e.g., the custom client and recent work on streming #1551. Do you still want to pursue this PR?

@bitnom There have been many recent changes e.g., the custom client and recent work on streming #1551. Do you still want to pursue this PR?

I'll be getting back to autogen PRs this weekend, and will re-evaluate my PRs and issues at that point. Other work had taken me out of it for a bit. I still think it's important to be able to override but I need to get caught-up and see where everything is at.
IOStream / https://github.com/microsoft/autogen/pull/1551 seems a good idea in its goal. I would just caution that it should probably be protocol agnostic (Without websockets). We have (In our programming arsenal) notions of queues, backpressure, channels, etc, and thread-safety for them. I think the protocol used at the edge of the module for IO should be an implementation detail left up to the end-user-programmer. Queues are dead-simple. Note that I haven't actually reviewed the iostream code yet (Will have time in a few days) so maybe it already addresses this.
@davorrunje

Closing because of inactivity. Please reopen if you need these changes.


[Feature Request]: Redis adapter

Package
lucia
Description
The migration guide of v3 specifies that the redis and unstorage adapters are deprecated. There doesn't seem to be a proper replacement for those. Given that Lucia now exclusively handles sessions, and those are most often stored in fast access db systems (e.g. Redis, other key-value dbs), it seems logical to support these. In fact, on may even argue that they are more relevant than the classical dbs that are currently implemented for the storage of session data.

@pilcrowOnPaper would it be suitable to use redis/KV store for session and other database table for users data in may be a hybrid adapter?

@yalamber It's relatively simple to implement it yourself. You can take https://github.com/JabRef/JabRefOnline/blob/64660d8a626ca092a39a4c123d830249ce25044e/server/user/UnstorageSessionAdapter.ts as a reference and modify it to use redis.
I still feel it would very much make sense to provide this out-of-the-box by lucia. If someone else needs the redis/unstorage adapter, please ping me and I'll consider publishing it as a new package.

@tobiasdiez thanks, I am trying to use cloudflare Workers KV, this should be helpful for us to follow.

This is really disappointing.
Can someone please explain the decision on deprecating v2 redis adapter and not building a proper replacement?

It may just be maintenance burden alongside the perception of low demand. Though from my experience, if there is enough demand, creating a separate package for the Redis adapter, colocated in the repo, is sustainable for contributors to add on top of it. Personally, I have many OSS libraries where I don't contribute any code to anymore, but one-time contributors continue keeping the library up to date which I'm super happy with.
No new features, just upkeep and bug resolutions from outside collaborators. I think the same could be done here with the expectation that it is not something officially supported in Lucia, but others who do use it should contribute code and let the maintainer here solely review the code and cut releases.


Implement personas based filter for listing page

Current GraphQL capabilities allow to filter just by one element value (in this case author). It would be great to showcase multiplier filtering conditions.
Current commented solution is here:

https://github.com/Kentico/kontent-sample-app-graphql-react/blob/main/src/ListingPage.js

Internal linked: blocked by https://kentico.atlassian.net/browse/DEL-3086
Bonus:
It would worth a try to add a query for loading all posts but just load persona definition - to list in dropdown only personas that are really used.

@winklertomas - this might be also fairly simple to put pack before release.

https://github.com/Kentico/kontent-sample-app-graphql-react/pull/22 merged -> closing


Fixes pfToolbar testing

Allows the test toolbar to be displayed correctly

vdb-bench.dataservice

Even though the angular-patternfly.js is included, the module still
needs to be imported for the pf-toolbar tag to be interpreted as the
directive.

pftoolbar.js

pfViewUtils is a constant within angular-patternfly so it must be
imported in the same way as REST_URI

pftoolbar.html

The pf-list-view is a directive and as such provides an internal scope
where each row is stored under item. Thus, vm should not be prepended
to the vars.

awesome - thanks for looking into this.


Better edge case output on catcher worker

Now indicate if we're trying an ultra if we have more than min.
State no balls if we're completely out.

Merging (just an output change)


Zoid usage in vuejs

Hello @bluepnume ,
I am having an issue with rendering zoid in vue.
No matter what I tried , I keep getting
Uncaught TypeError: Cannot read property 'component' of undefined     at Object.register (checkout.js:6817)     at Component._proto.driver (checkout.js:7139)     at Function.init.driver (checkout.js:7297)     at Object.<anonymous> (watulink.js:79274)     at __webpack_require__ (watulink.js:20)     at Object.<anonymous> (watulink.js:79078)     at __webpack_require__ (watulink.js:20)     at Object.<anonymous> (watulink.js:79055)     at __webpack_require__ (watulink.js:20)     at Object.<anonymous>

Are you using dist/zoid.frameworks.js?


Force reinitializing cameras

I apologize for the category error, but I wasn't sure what else to tag this with.
I have a complicated program where sometimes things go wrong and stuff doesn't close cleanly. My issue is that the cameras sometimes get stuck in a state where they simply refuse to be opened on a program restart, and the only thing that seems to work is either restart the computer or its even sometimes necessary to unplug the cameras from power and reconnect them. On program start, I open devices serially and get these errors (note program tries to open the camera 3 times with 2s intervals when exceptions are thrown before giving up, though all the "t=2442" are identical so not sure):
[2021-01-13 14:04:46.247] [critical] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/usbcommand/usbcommand.c (366): find_libusb_device(). libusb device(s) are all unavalable. Is the device being used by another application?
[2021-01-13 14:04:46.247] [error] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/depth_mcu/depth_mcu.c (68): usb_cmd_create(USB_DEVICE_DEPTH_PROCESSOR, device_index, NULL, &depthmcu->usb_cmd) returned failure in depthmcu_create()
[2021-01-13 14:04:46.247] [error] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/sdk/k4a.c (133): depthmcu_create(index, &device->depthmcu) returned failure in k4a_device_open()
[2021-01-13 14:04:48.260] [critical] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/usbcommand/usbcommand.c (366): find_libusb_device(). libusb device(s) are all unavalable. Is the device being used by another application?
[2021-01-13 14:04:48.260] [error] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/depth_mcu/depth_mcu.c (68): usb_cmd_create(USB_DEVICE_DEPTH_PROCESSOR, device_index, NULL, &depthmcu->usb_cmd) returned failure in depthmcu_create()
[2021-01-13 14:04:48.260] [error] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/sdk/k4a.c (133): depthmcu_create(index, &device->depthmcu) returned failure in k4a_device_open()
[2021-01-13 14:04:50.267] [critical] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/usbcommand/usbcommand.c (366): find_libusb_device(). libusb device(s) are all unavalable. Is the device being used by another application?
[2021-01-13 14:04:50.267] [error] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/depth_mcu/depth_mcu.c (68): usb_cmd_create(USB_DEVICE_DEPTH_PROCESSOR, device_index, NULL, &depthmcu->usb_cmd) returned failure in depthmcu_create()
[2021-01-13 14:04:50.267] [error] [t=2442] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/sdk/k4a.c (133): depthmcu_create(index, &device->depthmcu) returned failure in k4a_device_open()

Is there a way to forcefully free whatever context/state the k4a is in and start fresh?

You can reset all connected Azure Kinect devices from the command line
AzureKinectFirmwareTool -Reset
You can also reset a single Azure Kinect by specifying the serial number.

You can reset all connected Azure Kinect devices from the command line
AzureKinectFirmwareTool -Reset
You can also reset a single Azure Kinect by specifying the serial number.

Can this be done programmatically in C++? Is there a source file for this tool in the repo somewhere that I can use as an example?

Can this be done programmatically in C++? Is there a source file for this tool in the repo somewhere that I can use as an example?

Can this be done programmatically in C++? Is there a source file for this tool in the repo somewhere that I can use as an example?

Can you not use system() to call the argument? https://stackoverflow.com/questions/8832326/how-can-i-execute-a-command-line-command-from-a-c-program

Can this be done programmatically in C++? Is there a source file for this tool in the repo somewhere that I can use as an example?

Can you not use system() to call the argument? https://stackoverflow.com/questions/8832326/how-can-i-execute-a-command-line-command-from-a-c-program

Yeah we can use system(), but it's hard to get return/exception feedback on commands like that. I'd prefer a more direct c-api if it exists, but I can make the system() way work in the mean time I think.

Yeah we can use system(), but it's hard to get return/exception feedback on commands like that. I'd prefer a more direct c-api if it exists, but I can make the system() way work in the mean time I think.

Tested this solution on a faulty system, worked perfect. Will use system() for now to invoke the fix, but would like an api or some kind of instruction on how to do it myself. Closing, thanks for the help.

Tested this solution on a faulty system, worked perfect. Will use system() for now to invoke the fix, but would like an api or some kind of instruction on how to do it myself. Closing, thanks for the help.


Get the return value of Switch Task

Get the return value of Switch Task ;How do I get the return value of a Task through the API, because we use the separation of front and back ends

You cannot get the value of a task from the API. This feature is not implemented yet.

How to get it if I don’t use API
------------------ 原始邮件 ------------------
发件人:                                                                                "aelassas/Wexflow"<EMAIL_ADDRESS>发送时间: 2020年9月22日(星期二) 晚上10:52
收件人<EMAIL_ADDRESS>抄送: "火影.海诺"<1540334571<EMAIL_ADDRESS>主题: Re: [aelassas/Wexflow] Get the return value of Switch Task (#288)
You cannot get the value of a task from the API. This feature is not implemented yet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Add N150 and T3000 CI Pipelines for full model demos

We currently do not run the full model demos in CI, which is a gap in our coverage.
You can see the list of the demos that are functional on our github page.
https://github.com/tenstorrent/tt-metal?tab=readme-ov-file#wormhole-wh-models
For a simple batch=32 input prompt, these demos would probably run for 10-15 minutes. So, maybe we want to create a once-a-day pipeline.
We need verify both accuracy and performance for these. The demos already print tokens/second/user metric. We should just grep that from output and verify. We do not want to add any CI related logging/testing since the demos are meant to be external and clean. We will need to add to demos some kinda check to verify correctness (perplexity per token?).

fyi @tapspatel

@tapspatel I know you are busy but it would be good to take a look at this when you have the chance. If it is just a matter of adding a once-a-day pipeline config, we can fill the tests in.

There does not seem to be a big difference between multi-device-build-and-unit-tests-frequent.yaml and multi-device-build-and-unit-tests.yaml
One runs every 3 hours, the other every 4 hours.
Maybe we move all tests to the latter, and the former runs once a day (i.e. nightly as the name implies) so we can add demos to it?

taking a look now

https://github.com/tenstorrent/tt-metal/pull/8262
I added the new end of end demos pipeline. I also changed the frequency of the unit tests and frequent tests
unit: every 3 hours
frequent: every 8 hours
performance: every day at 6am
end-end-demos: every day at 12am

Merged into main: 6984d928162553e6e6b0717e1df4945a603f0ab6
Closing issue


Support bastion/jump SSH host

It's needed in order to initiate SSH connections to the instances inside "private" (without public facing IPs) subnets of a VPC.
depends on: #352

move to 0.7, to work with new API, instead of backporting changes from old to new

@kron4eg Can we add a note to the aws-private Terraform scripts that such setups are WIP?

aws-private terraform config itself is done and is not WIP, I don't want to confuse anyone with not related WIP tags on it (and remember to remove it later).

@kron4eg Terraform config is done but it's not possible to use KubeOne with such config out of box. This should be stated.

Feel free to do it as I’m not really sure what do you mean

/unassign kron4eg
/assign
/lifecycle active

Currently working on E2E tests
/unassign


Cannot Launch lrn kernel in Layer 0

Hello @doonny
I am obtaining the following error when I run this on Arria10 :
Executing Layer 1:
Launching single work-item kernel winbuffer
Launching single work-item kernel Conv
Launching single work-item kernel Pooling
Launching kernel MemWr with local size: 1, 1, 64  (global size: 27, 27, 128)
Launching kernel lrn with local size: 1, 1, 6  (global size: 27, 27, 6)
ERROR: CL_INVALID_EVENT
Location: host/main.cpp:870
Failed to finish lrn event
My VEC_SIZE is 16, LANE_NUM is 64, CONV_GP_SIZE_X is 7. My board model is
https://www.altera.com/products/boards_and_kits/dev-kits/altera/kit-a10-gx-fpga.html
Do you think I have used many resources?

I think this issue occurs multiple devices. With single device I face no issue. :)

Hi manojrohit doonny @manojrohit  @doonny
I met a CL_INVALID_WORK_GROUP_SIZE error when Launching kernel MemWr.
Can you give me some advice ?
Thank you all the time !
Executing Layer 1:
Launching single work-item kernel winbuffer
Launching single work-item kernel Conv
Launching single work-item kernel Pooling
Launching kernel MemWr with local size: 1, 1, 2  (global size: 27, 27, 96)
ERROR: CL_INVALID_WORK_GROUP_SIZE
Location: host/main.cpp:846
Failed to launch kernel memWr

Hi @doonny @manojrohit, I get a 'CL_INVALID_WORK_GROUP_SIZE' error when clEnqueueNDRangeKernel.
my log is below,
Launching kernel MemWr with local size: 1, 1, 2  (global size: 27, 27, 96)
ERROR: CL_INVALID_WORK_GROUP_SIZE
Location: host/main.cpp:846
Failed to launch kernel memWr
I`m running with opencv.
Hope for you advices. Thank you very much!


Signup changes

Issue #, if available:
Description of changes:
Removed the phone number field from signup form. After the changes, signup form contains only username, email and password. The verification code is sent via email.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

There is documentation that needs to be updated to reflect the new signup flow (instructions and screenshots). Here are two places I know for sure (may not be exhaustive--possible there are references in the workshops too).
https://github.com/aws-samples/retail-demo-store/blob/master/documentation/1-Creating-account.md
https://github.com/aws-samples/retail-demo-store/blob/master/workshop/Welcome.ipynb

Made the changes in the above specified files and some other areas where I found the mention of signup and SMS.


added problem:Allocation solution:Java

Please follow the guidelines, you've to follow them otherwise I can't merge your PR.
Close this PR and go to repository, check out the Guideline.md file & PR again.
@evandrix

alright, will do


[WIP] Improvement in messageActionSheet

#651

Completed 👍
Ready for review.

Updated branch 👍
also posted a question here

@borisyankov rebased 👍
this is also ready to go :runner:

Looks good now!


Update shown code to match source

I got very confused checking the shown code and the actual output, then realized the shown code was not synced with the actual source.

Fab! Thank you <3


loaderUtils is missing

I think you should add the module loaderUtils as a dependence

Right thanks, the dependencies are a mess, fixing it right now


Allow transitions when reordering pages in Navigator.pages

Use case
When I reorder pages in Navigator.pages, I need a transition for the existing page in the stack becoming the top one. Currently Navigator is only animating entirely new pages on the stack and pages being removed from the stack.
Here is the code reordering two pages and the resulting animation, we see the transition only when the page is added for the first time but not on subsequent changes:

Code
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

final _notifier = ChangeNotifier();
final _pages = <Page>[Page1()];

class Page1 extends MaterialPage {
  Page1()
      : super(
          key: const ValueKey('Page1'),
          child: Scaffold(
            appBar: AppBar(title: const Text('Page1')),
            body: const Center(
              child: ElevatedButton(
                onPressed: _showPage2,
                child: Text('Show Page2'),
              ),
            ),
          ),
        );
}

class Page2 extends MaterialPage {
  Page2()
      : super(
          key: const ValueKey('Page2'),
          child: Scaffold(
            appBar: AppBar(title: const Text('Page2')),
            body: const Center(
              child: ElevatedButton(
                onPressed: _showPage1,
                child: Text('Show Page1'),
              ),
            ),
          ),
        );
}

void _showPage1() {
  for (final page in _pages) {
    if (page.key == const ValueKey('Page1')) {
      _pages.remove(page);
      break;
    }
  }
  _pages.add(Page1());
  _notifier.notifyListeners();
}

void _showPage2() {
  for (final page in _pages) {
    if (page.key == const ValueKey('Page2')) {
      _pages.remove(page);
      break;
    }
  }
  _pages.add(Page2());
  _notifier.notifyListeners();
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: HomeScreen(),
    );
  }
}

class HomeScreen extends StatelessWidget {
  const HomeScreen({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return AnimatedBuilder(
      animation: _notifier,
      builder: (_, __) {
        return Navigator(
          pages: [..._pages],
          transitionDelegate: const MyTransitionDelegate(),
          onPopPage: (route, result) {
            if (!route.didPop(result)) return false;

            return true;
          },
        );
      },
    );
  }
}

Alternative solutions so far are:

To change the key of the new page. But I need keys to track some things in the app.

To add a non-zero duration between removing and re-adding a page. But this is noticeable.

Use 3rd party solutions for animations. In my opinion, this is inconsistent. Navigator.pages encapsulates the diffing of the stack, so it is the first one to know the changes. If we need to track the reordering for 3rd party solutions, then we effectively re-write the Navigator.

Proposal
Navigator.transitionDelegate seems like the right property to handle this. TransitionDelegate has resolve method to override transitions, but it is only called when a new route is pushed.
I propose to either call this method also when the order changes, or to add another method for that case (depending on how bad is the breaking change of additional calls to the existing method).

Hi @alexeyinkin, Thanks for filing the issue. This does sound like a reasonable proposal,
FYI: There's actually a work in progress to remove transitions from web and Desktop completely https://github.com/flutter/flutter/issues/99052.
cc: @darrenaustin


Feature/cleanup config

What? and Why?

What does this pull request change/fix? Why was it necessary?

Checklist

[x] CHANGELOG.md updated? (if required)

Codecov Report

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

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   83.57%   83.57%
=======================================
  Files           9        9
  Lines         548      548
=======================================
  Hits          458      458
  Misses         90       90

Impacted Files
Coverage Δ

app/settings.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 c3a4dc6...743a078. Read the comment docs.

Don't merge until log location in prod has been changed!


Resources for people who are new to DC

What are some things that people need to know when they move to DC? Where can they find information about those things?
Post in this thread with suggestions for inclusion in a DC orientation guide. This is primarily focused on essentials—how to get around, how to set up services—but should also include some fun, as well.

BYO gas mask and fire extinguisher when riding the metro ;)

Anyone new to DC needs to have access to handy tools regarding safety, well being, transportation, food , hospital, hotels, restaurants, bars, lounges etc.
When someone is new to DC, the first thing they need is a phone. Where are the closest phone companies att, verizon, t mobile etc. Depending on their budget, they may want to go with the cheapest options available like go-phones, or metro pcs, boost etc. Then the next thing they may want to know is the closest restaurant around wherever they are. How to call a cab in case they don't have an uber/lyft etc account.
Also they may want to have access to hotels around them, ranging from cheapest to most expensive. Same for restaurants.
Once they can have access to basic necessity, they can be proposed attraction sites like museums, tourist sites in downtown etc

So I will say the first milestone will be to have in a database the following :

Phone companies that can be ordered by prices, or affordability
Hotels
Restaurant
bars
clubs
Taxi cabs companies
The next milestone will be to have users be able to signup for app/website/retrieve password/login though phone etc.
The next milestone will be to make a dashboard that shows the different available menus.
Menus will need to be completely plug-able. Someone not interested in clubs/bars may unplug or deactivate the corresponding modules.

One of the best examples I've seen of what this could be was shared at this year's IA Summit. Honolulu Answers had topics, research, and answers sourced from the community during a meetup after the structure was built. Could we build the shell and host a similar community brainstorm/write day to work with other DCers to flesh it out?

Here are some ideas:

Including a social services section/database/webpage (á la https://www.auntbertha.com/ or United Way)
A repository linking to each Chamber of Commerce located in D.C.
Include an interactive map of neighborhoods/sections of D.C. and adding pins for landmarks, restaurants, etc...
Guides for Hospitals/Public Transportation/Other Transportation options (summaries if companies like Uber are not in use, for example, or other popular ride sharing apps)
Calendar of festivals, other important events


Create a list of any coded patterns that we plan to deprecate

Review telemetry data on stable and experimental patterns -> https://airtable.com/appY30x3q9FcdIcL1/tbl81O5yRLA4gGh1T/viwMUaHdUBS4rR9Jy?blocks=hide
Flag patterns to deprecate by adding to tracking issue
As we deprecate code from the package, need to create examples for the patterns where possible/applicable.

@matthewgallo and I opened #6488 as a follow up since we would ideally want to create the pattern code before deprecation.
Still evaluating some other potential patterns that may need to be deprecated in favor of example code.


clarifying "Azure Storage Explorer" workflow

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

#sign-off

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


iOS support

Do you have any iOS experience @villela ?

@theobouwman not really but if I had access to iOS development I would do the iOS part for sure


Dropdown component: Fixed cursor style(pointer) for caret icon

Fixes #243

Thanks for the interest and for the fix, @arorayash ! :)


[NetAppFiles]Anf 6195 swagger restapi to2020 06 01 bugfix

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

This is a bugfix for the NetAppFiles API to address an issue in SDK generation where snapshotPolicy schedule objects,  hourly- daily- weeklySchedlue were presented as objects instead of actual classes in generated SDK such as .net.
This would make it more cumbersome for SDK users to consume the objects than it needs to be.
Note the feature is in preview. the spec is new-ly created and generated SDK's have not been released yet, may be considered breaking but addresses a bug that we would like to fix.
Before
        /// <summary>
        /// Gets or sets hourlySchedule
        /// </summary>
        /// <remarks>
        /// Schedule for hourly snapshots
        /// </remarks>
        [JsonProperty(PropertyName = "properties.hourlySchedule")]
        public object HourlySchedule { get; set; }

After
        /// <summary>
        /// Gets or sets hourlySchedule
        /// </summary>
        /// <remarks>
        /// Schedule for hourly snapshots
        /// </remarks>
        [JsonProperty(PropertyName = "properties.hourlySchedule")]
        public HourlySchedule HourlySchedule { get; set; }

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[x] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

@audunn  If no customer impact, I will label it with approvedBreakingChange.

@jianyexi Could we add a new linter rule to check this issue? Swagger items could be set only in array data type.

@ruowan there should not be customer impact.


Color picker for markdown

Would it be possible to introduce some sort of color-picker for text in markdown, e.g.:

text

I see. It seems something similar is already implemented in VSCode Settings (see screenshot). Would it be possible to use it in markdown as well?

Well, after some search I find there are already some good solutions

Hi @neilsustc, yes, I am aware of these. However, these pick only the color code, not the whole snippet of

text

What I am looking for is if Markdown All in One could add this snippet of code whenever I pick a color from these color pickers. This would help to streamline the workflow without the need to write extra code that is all the time the same. Is this possible?

It is possible. I just don't have enough time if it is not a really popular feature.

Well, you would be very popular here :-) 👍


Database busy error

@kevinlang I am going to track the database busy issue here.
Previous discussions:

https://github.com/warmwaffles/exqlite/pull/38
https://github.com/warmwaffles/exqlite/pull/39#issuecomment-792447244

I need to dig into Ecto.Adapters.SQL.Sandbox more to ascertain if it can work with sqlite's restriction of 1 writer at a time. We are definitely hitting some issues where another test has opened a transaction but has not committed it yet and we are trying to read back those changes.
Yay for concurrency issues.

It is worth noting that the tests are running sequentially, and that the sandbox makes it so one checks out only a single connection at a time. So the issue we are hitting in our tests is more sequential than a real world scenario.
I think you are right that there is some issue where a sandboxed connection is not closing out correctly, affecting the other tests.
One other approach I thought of was to run tests against an in-memory database. At "checkout" time we would clone the on-disk database into a :memory: one. There we don't have to worry about cleanup or anything, assuming check-in drops the "connection" and thus the in-memory database.
However, I kind of like that the current way the sandbox does it, with transaction, raises these trickier transaction issues in our test suite, which perhaps reflect the real world a bit more than the memory approach.

However, I kind of like that the current way the sandbox does it, with transaction, raises these trickier transaction issues in our test suite, which perhaps reflect the real world a bit more than the memory approach.

This is precisely why I strayed away from using :memory: for integration tests.
Perhaps, instead of just returning {:error, "Database busy"} we can retry the statement again after a random wait?
https://github.com/warmwaffles/exqlite/blob/98dade3a5996d6930e3928eba6501a60e112db1e/lib/exqlite/connection.ex#L291-L292

The current test error originates from this line
https://github.com/warmwaffles/exqlite/blob/98dade3a5996d6930e3928eba6501a60e112db1e/lib/exqlite/sqlite3.ex#L105
I changed it to
:busy -> fetch_all(conn, statement)

And it just hung there
23:06:33.422 [error] Exqlite.Connection (#PID<0.434.0>) disconnected: ** (DBConnection.ConnectionError) client #PID<0.703.0> timed out because it queued and checked out the connection for longer than 15000ms

For future reference, https://sqlite.org/c3ref/busy_handler.html we could provide some tooling for handling busy callbacks. Though I don't know how just yet.

One approach that may be worthwhile is moving some of the multistep logic into the NIF. :thinking:

This was an interesting read. I'm still not sure why we are getting the lock, but it looks like we can improve reliability generally by responding to a busy response by doing appropriate rollbacks/resets.
https://sqlite.org/forum/forumpost/d2d81b46bc?t=c&unf&hist

One approach that may be worthwhile is moving some of the multistep logic into the NIF.

So the rationale that I used for doing the individual stepping in elixir rather than the NIF is that if you have multiple database connections open where the stepping takes some time to complete, it could block the dirty nif scheduler. I wanted to allow make it return as fast as possible and to play nice with other dirty nifs.


Trailing slash in path omits rule.

type path /x/y/ { read() = true; }

Results in
{
  "rules": {
    ".read": "true"
  }
}

Expect:
{
  "rules": {
    "x": {
      "y": {
        ".read": "true"
      }
    }
  }
}

This was parsing as:
type path /x/y
/ { read() = true; }

(two path statements)

Fixed in v0.4.1


Editor 6.2.1 OS X Bug — Copy always appears to copy a single additional line between lines.

Bug Report
Copy always appears to copy a single additional line between lines.
What is the current behavior?
This is regardless of how many line breaks are between the lines. For example copying this
one

two
three

four

will result in:
one

two

three

four

in the clipboard.
What is the expected behavior?
The line breaks should be copied verbatim.
For GitBook Editor
I believe the bug has been around in version 6 for a while, but not necessarily in previous versions.

Will be fixed in version 6.3.0-alpha.1


Discrepancy of generation from tsp and swagger for resource name property

Current tsp, we define a resource name property in the final resource model, not inherit it from the base resource model. This will cause the name property in the generated code for JS and Python to be breaking.
For example, a FooResource defined as:
https://github.com/Azure/typespec-azure/blob/a15638510b0f8059b69b78e05cdc08a3ec0ef7a2/packages/typespec-azure-resource-manager/test/resource.test.ts#L49-L56
It is sourced from TrackedResource template:
https://github.com/Azure/typespec-azure/blob/a15638510b0f8059b69b78e05cdc08a3ec0ef7a2/packages/typespec-azure-resource-manager/lib/models.tsp#L17-L25
Which is derived from TrackedResourceBase model:
https://github.com/Azure/typespec-azure/blob/a15638510b0f8059b69b78e05cdc08a3ec0ef7a2/packages/typespec-azure-resource-manager/lib/arm.foundations.tsp#L49-L55
Which is derived from ArmResource model:
https://github.com/Azure/typespec-azure/blob/a15638510b0f8059b69b78e05cdc08a3ec0ef7a2/packages/typespec-azure-resource-manager/lib/arm.foundations.tsp#L22-L37
You could see name property is defined in FooResource, not in ArmResource.
But with swagger common-types, the name is defined in the root Resource model:
https://github.com/Azure/azure-rest-api-specs/blob/84849e5293de3a8e01cbdde13b2d7086d3fb34d6/specification/common-types/resource-management/v5/types.json#L9-L37

One real issue from service team: https://github.com/Azure/azure-rest-api-specs/pull/27565#discussion_r1518977192

@tadelesh I don't think the referenced spec is actually blocked by this, because it cannot use the resource templates in any case, due to their inheriance form TrackedResource or ProxyResource.  The only way to represent this API without breaks is to use a custom-defined resource type, as shown in this playground
Note that this will be a problem for new specs that were originally introduced in TypeSpec, or any specs that use resource types that directly inherit from ProxyResource of TrackedResource (a small list).  This problem should be addressed by this issue: https://github.com/Azure/typespec-azure-pr/issues/3873

@tadelesh I don't think the referenced spec is actually blocked by this, because it cannot use the resource templates in any case, due to the resource type inheriance form TrackedResource or ProxyResource. The only way to represent this API without breaks is to use a custom-defined resource type, as shown in this playground
Note that this will be a problem for new specs that were originally introduced in TypeSpec, or any specs that use resource types that directly inherit from ProxyResource of TrackedResource (a small list). This problem should be addressed by this issue: Azure/typespec-azure-pr#3873

Then how should emitter deal with the ArmResource model in different namespace? In swagger, we will ask service team to choose either use common types or use self-defined base resource models to prevent duplicate schema. I'm afraid it will happen again in TypeSpec?

Pending design issue:
https://github.com/Azure/typespec-azure-pr/issues/3873

https://github.com/Azure/typespec-azure/pull/661 Merged


Generate test code programmatically

DIT source

[x] Generate type stability tests with @eval
[ ] Generate correctness tests with @eval
[ ] Generate benchmarks with @eval

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 15.07%. Comparing base (ea46769) to head (2c0a3f2).

:exclamation:  There is a different number of reports uploaded between BASE (ea46769) and HEAD (2c0a3f2). Click for more details.
HEAD has 49 uploads less than BASE

Flag
BASE (ea46769)
HEAD (2c0a3f2)

DIT
10
0

DI
41
2

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #438       +/-   ##
===========================================
- Coverage   96.84%   15.07%   -81.78%
===========================================
  Files         103       68       -35
  Lines        4884     2634     -2250
===========================================
- Hits         4730      397     -4333
- Misses        154     2237     +2083

Flag
Coverage Δ

DI
15.07% <ø> (-84.33%)
:arrow_down:

DIT
?

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

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


WIP router: implement path rewrite with regex

Co-Authored-By: Gabriel Taylor Russ<EMAIL_ADDRESS>Signed-off-by: Christopher M. Luciano<EMAIL_ADDRESS>Description: Support regex path rewrites
Risk Level: Medium, modifies an existing rewrite action
Testing: bazel test //test/common/router:config_impl_test
Docs Changes: TODO
Release Notes: TODO
Fixes #2092
TODO:

[ ] Add a few base tests for regex path
[ ] Add a test specifically showing rewrite happening after the RBAC filter

@htuch I believe the only remaining error failure is
[ RUN      ] RouteConfigurationV2.RegexPrefixWithNoRewriteWorksWhenPathChanged
[2019-10-18 14:32:29.254][1060923][critical][assert] [source/common/router/config_impl.cc:790] assert failure: regex_->match(pathOnly(headers)).
[2019-10-18 14:32:29.254][1060923][critical][backtrace] [bazel-out/darwin-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:83] Caught Abort trap: 6, suspect faulting address 0x7fff6eaa72c6
[2019-10-18 14:32:29.254][1060923][critical][backtrace] [bazel-out/darwin-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:70] Backtrace (use tools/stack_decode.py to get line numbers):
[2019-10-18 14:32:29.254][1060923][critical][backtrace] [bazel-out/darwin-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:71] Envoy version: 0/1.12.0-dev/redacted/DEBUG/BoringSSL
external/bazel_tools/tools/test/test-setup.sh: line 310: 70995 Abort trap: 6           "${TEST_PATH}" "$@" 2>&1

This ASSERT has a comment that I'm wondering about here.
Do you think this is a signal that it should be revisited as to whether this ASSERT is necessary or does it indicate something wrong within the current PR?

@cmluciano are you clearing the route cache as the comment indicates? You need to do this if you modify path (or any header).

Not stale-  I had some temporary compilation problems and now I'm working through basing this off of the RE2 library

Can we re-open this please?

@vikpck reopened, but not would be good to hear from @cmluciano on the plan. I've heard at least 2 folks in the past week in various conversations ask for this :)

Thanks for re-opening. I'll be getting back to working on this today.

We would really like this functionality as well! Is there an ETA?

We hope to see some TODOs be implemented soon. Meanwhile, seems like you need to merge master? And this: https://github.com/envoyproxy/envoy/pull/8462#pullrequestreview-307610270. Thanks!

No stale according to my exchange with @cmluciano.

Not stale


JSON support (unexpected quotes)

Trying to get data from JSON field in PostgreSQL.
my code:
db.Query("messages")                 .Select("message ->> 'deviceid' AS deviceid", "message ->> 'status' AS status")                 .Where("CAST ( message->> 'temperature' AS DECIMAL)", ">", 28).Get();
output query :
SELECT "message ->> 'deviceid'" AS "deviceid", "message ->> 'status'" AS "status" FROM "messages" WHERE "CAST ( message->> 'temperature'" AS "DECIMAL)" > 28
expected output :
SELECT message ->> 'deviceid' AS deviceid, message ->> 'status' AS status FROM "messages" WHERE CAST ( message->> 'temperature' AS DECIMAL) > 28
Not sure if it's an issue since JSON(NQSQL) is not supported. But it would be great if I could find a way to get rid of extra quotes.
Any suggestions are appreciated. thanks.

Currently, you can use the SelectRaw and WhereRaw methods instead, and use the ? as a placeholder for your parameters.
db.Query("messages")
  .SelectRaw("message ->> 'deviceid' AS deviceid", "message ->> 'status' AS status")
  .WhereRaw("CAST ( message->> 'temperature' AS DECIMAL) > ?", 28)
 .Get()

Thanks. 👍
Your code caused an exception
SELECT message ->> 'deviceid' AS deviceid FROM "messages" WHERE CAST ( message->> 'temperature' AS DECIMAL) > 'message ->> 'status' AS status' Npgsql.PostgresException (0x80004005): 42883: operator does not exist: integer > text
I needed to make a few changes.
db.Query("messages")
  .SelectRaw("message ->> 'deviceid' AS deviceid, message ->> 'status' AS status")
  .WhereRaw("CAST ( message->> 'temperature' AS DECIMAL) > ?", 28)
 .Get()


Group Memory Field Names with Common Prefix

Description
Comparison of the memory fields in the summary benchmarks is unintuitive.
Pandas sorts the fields in alphabetical order and because the memory tracking fields are not named with a common prefix in the current benchmarking code it ends up printing the report with memory values in columns far apart from each other in the table (see below image).

Proposed Solution
Renaming all memory values with a common mem_ prefix like the following

mem_nvidia_peak_reserved
mem_torch_peak_allocated
mem_torch_allocated

This way when the results are gathered and exported to csv, the columns are grouped together intuitively for comparison.

@achew010 pls also note that the index column is introduced into the benchmark csv. Can we have this issue to also remove the index column?


Load infobubble close button over https

Allows infobubble usage on https websites without mixed content warnings
Closes #184 and https://code.google.com/p/google-maps-utility-library-v3/issues/detail?id=184

I signed it!

Thanks Steve


Search  in a file that is not part of the open folder does not work

Issue Type: Bug
I have VS code open to a specific folder sets. If I use VS code to open a file that is not in the current open editor vide and try to search in that file the search will not find antying even if that file contrains the search parameters.
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:13:20.174Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-1620 v4 @ 3.50GHz (8 x 3492)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.90GB (15.68GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

badgerdevtools
Dan
0.0.8

color-manager
roy
0.5.7

perforce
sle
3.2.0

Please see these tips and let me know whether they help https://github.com/microsoft/vscode/wiki/Search-Issues#missing-search-results

I have "search.useIgnoreFiles": false in the settings. I am searching JSON files. I did not have this issue until the latest update.
https://imgur.com/a/b1dHW2t

You have the "whole word" option selected, so it will only match "tex" as a complete word


Move the hlsl shader tasks into their own folder

Fixes #1156
General Checklist:
Please ensure the following points are checked:

[ ] My code follows the coding style
[ ] I have reviewed file licenses
[ ] I have commented any added functions (in line with Doxygen)
[ ] I have commented any code that could be hard to understand
[ ] My changes do not add any new compiler warnings
[ ] My changes do not add any new validation layer errors or warnings
[ ] I have used existing framework/helper functions where possible
[ ] My changes do not add any regressions
[ ] I have tested every sample to ensure everything runs correctly
[ ] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[ ] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[ ] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

If this PR contains framework changes:

[ ] I did a full batch run using the batch command line argument to make sure all samples still work properly

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[ ] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the readme of the folder that the sample belongs to e.g. api samples readme
[ ] For new samples, I have added a tutorial README.md file to guide users through what they need to know to implement code using this feature. For example, see conditional_rendering
[ ] For new samples, I have added a link to the Antora navigation so that the sample will be listed at the Vulkan documentation site

@gpx1000 While I can't formally approve, I have tested this using both Xcode (macOS/iOS) and Visual Studio (Win 10) and it works well.  Just thought you might want to know about the Xcode improvement as well.

Awesome thanks @SRSaunders.  I think this should be good to go when we do our call on Monday.  I typically test XCode every other week or so and Windows around the same cadence so it's easy for me to miss bugs like this.  My day to day I'm in Linux.


[GSoC][Feature] add SkyWalking Terraform provider

Search before asking

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

Description
Now the deployment methods for SkyWalking are limited, we only have Helm Chart for users to deploy in Kubernetes, other users that are not using Kubernetes have to do all the house keeping stuffs to set up SkyWalking on, for example, VM.
This issue aims to add a Terraform provider, so that users can conveniently  spin up a cluster for demonstration or testing, we should evolve the provider and allow users to customize as their needs and finally users can use this in their production environment.
In this task, we will mainly focus on the support for AWS. In the Terraform provider, users need to provide their access key / secret key, and the provider does the rest stuffs: create VMs, create database/OpenSearch or RDS, download SkyWalking tars, configure the SkyWalking, and start the SkyWalking components (OAP/UI), create public IPs/domain name, etc.
NOTES
This is a GSoC task, and because it heavily relies on cloud vendor's resources, we are open to receive sponsorship from the cloud vendor during the GSoC timeline, we will need an AWS account during the development process for testing and debugging.
Use case
Users can use the Terraform provider to conveniently set up a SkyWalking cluster in AWS, with only one or two commands.
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I could sponsor this personally as an AWS container hero. We need to evaluate the PoC cost.
We should do this step by step to ensure no surprise in budget cost.

@kezhenxu94 Hello! I am very interested in this project and would like to contribute to SkyWalking. I have some understanding and practical experience with cloud-native, and I believe that multi-cloud is a trend. Looking forward to your guidance! Thank you!

How far do we have for now? Is this close to the first release?

How far do we have for now? Is this close to the first release?

We at least need to wait for GSoC 2023's end time, the project in GSoC 2023 is a MVP of this so only after that can we release

FYI, Terraform is now NOT open-source anymore.
A new fork is created on the latest MPL-2 license, https://opentf.org/announcement

Would you plan to release a version?


Bad Link on Webpage

The link to 'Visual Studio support for C++/WinRT' is invalid. The invalid link is: https://docs.microsoft.com/en-us/windows/uwp/intro-to-using-cpp-with-winrt.md#visual-studio-support-for-cwinrt-xaml-the-vsix-extension-and-the-nuget-package
This webpage does not exist.

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

ID: 0aa02bcc-0841-3108-8f7e-47abbe6850a1
Version Independent ID: 91c62675-45a3-5cc6-7d4a-55fbf511ad8e
Content: Set up the game project - UWP applications
Content Source: windows-apps-src/gaming/tutorial--setting-up-the-games-infrastructure.md
Product: uwp
Technology: games
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @Robert-Tortorelli, and thanks for filing this issue. I have a fix in the works, that should be live within 24 hours.
Thanks again!
Steve


 component - Lesson

The story:
As a learner that is timed against a clock, I want to feel a sense of urgency as the time runs out for me to complete a memory game

What we should do:
We should build a clock component that counts down so that user knows how much time is left and rushes to complete the game.

To be successful we need to:
Check these off as they are completed:

[ ] A <MemoryClock /> component
[ ] Takes a time prop as seconds as a number
[ ] Counts down to zero, and notifies the onDone prop
[ ] Takes a onDone prop as a function callback
[ ] Flashes red 1/second when < 10s left
[ ] Flashes red 2/second when < 5s left
[ ] Flashes red 4/second when < 3s left
[ ] Flashes red 8/second when < 2s left
[ ] Flashes red 16/second when < 1s left

This will create an increasing sense of urgency that gets more and more dramatic. It can be done simply with a CSS class that just changes the "animation-duration".

[ ] Plays a "tick" every 2 seconds
[ ] Plays a "tok" every other 2 seconds

Design
Normal

Flash

Done

LINK TO FIGMA DESIGN

I would like to get involved in this task.


Create NavyaHegde_ML.md

I have the knowledge onb the following fields:
1.t statistic test,P statistic test.
2.Hypothesis creations.
3.Regressions like Lasso,ridge,simpleLinear regression,multiple linear regression,multivariate linear regression.
4.Corelation ,partial co relation and auto corelation.
5.Time series forecasting models like AR,MA,ARMA,ARIMA,SARIMA.
6.Supervised Machine learning techniques like decision trees,classification problems.
7.Unsupervised machine learning models like Apriori algorithms,clusteriung.
8.Bayesian learning.
9.Heurestic functions.
10.Artificial neural networks.
11.Swarm intelligence.
12.Genetic algorithms.
13.Support Vector Machines
14.Boosting ALgorithms
15.Bias-Variance
16.Evalutation Metrics (ROC Curve)
17.Dimensionality Redcution
18.Ensmeble Leaning Methods
19.Backpropagation of errors in ANN

I have the knowledge onb the following fields:
1.t statistic test,P statistic test.
2.Hypothesis creations.
3.Regressions like Lasso,ridge,simpleLinear regression,multiple linear regression,multivariate linear regression.
4.Corelation ,partial co relation and auto corelation.
5.Time series forecasting models like AR,MA,ARMA,ARIMA,SARIMA.
6.Supervised Machine learning techniques like decision trees,classification problems.
7.Unsupervised machine learning models like Apriori algorithms,clusteriung.
8.Bayesian learning.
9.Heurestic functions.
10.Artificial neural networks.
11.Swarm intelligence.
12.Genetic algorithms.
13.Support Vector Machines
14.Boosting ALgorithms
15.Bias-Variance
16.Evalutation Metrics (ROC Curve)
17.Dimensionality Redcution
18.Ensmeble Leaning Methods
19.Backpropagation of errors in ANN


Support to TypeScript

[ ] Create a suite of tests to validate the TypeScript Support;
[ ] Add a Type Definition of the module;
[ ] Add some docs of how to use with TypeScript;

Hi, thanks for provide very useful module.
I using this with typescript.
my di-decorators.d.ts is this.
declare module "di-decorators" {

  export function inject(...args: any[]): <C>(Target: new (...args: any[]) => C) => new (...args: any[]) => C;

  export function instance<T>(x: new (...args: any[]) => T): T;

  export function provide<T>(x: new (...args: any[]) => T): {as: (factory: () => T) => void};

  export function singleton<T>(x: new (...args: any[]) => T): new (...args: any[]) => T;

  export function immutable<T>(x: new (...args: any[]) => T): new (...args: any[]) => Readonly<T>;

}

It has limitation of constructor arguments typing.
If typescript will support variadic kinds, can solve problem.
If you need its PR, I will create PR.
I hope this would be some of help...

Hi @hrsh7th
That looks great, thank you!
Would be great If you could open a PR and also add some docs on how to setup a type script project.
It has been a while since I worked with TypeScript and I'm not sure if it's need any configuration to use decorators.
But it already helped a lot.


Bug: 0001_migrate_metadata fails when building from empty database

What I'm trying to achieve
I am trying to build a local Saleor instance locally, from an empty database.
Steps to reproduce the problem
The migration core.0001_migrate_metadata fails with the following output:
File "/usr/local/lib/python3.9/site-packages/django/apps/config.py", line 268, in get_model
    return self.models[model_name.lower()]
KeyError: 'attribute'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 419, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 413, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 354, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 398, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 89, in wrapped
    res = handle_func(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/commands/migrate.py", line 244, in handle
    post_migrate_state = executor.migrate(
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 227, in apply_migration
    state = migration.apply(state, schema_editor)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/migration.py", line 126, in apply
    operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/operations/special.py", line 190, in database_forwards
    self.code(from_state.apps, schema_editor)
  File "/app/saleor/core/migrations/0001_migrate_metadata.py", line 33, in flatten_attributes_metadata
    Attribute = apps.get_model("product", "Attribute")
  File "/usr/local/lib/python3.9/site-packages/django/apps/registry.py", line 211, in get_model
    return app_config.get_model(model_name, require_ready=require_ready)
  File "/usr/local/lib/python3.9/site-packages/django/apps/config.py", line 270, in get_model
    raise LookupError(
LookupError: App 'product' doesn't have a 'Attribute' model.

Commenting out line 114 of the affected file "fixes" the problem.
System information

Saleor version:

[ ] dev (current main)
[x] 3.0
[ ] 2.11
[ ] 2.10

@onecrayon Thanks for reporting. I've passed the issue to the team. We'll keep you updated.

@onecrayon Hey, did you try deleting that part of the migration? I have the same issue right now, also I'm not sure how the models connect the attributes of a product, did you study that?


confused: error when push to repo own by another?

I decided to try LFS on a GitHub repo owned by a user (@mpinsky) who does not yet have access to the trial of LFS. I installed on my machine, did some work (adding some kinda big .csv's), and then tried to push.
I then get the message:
Git LFS support is coming! Sign up for early access: https://github.com/early_access/git-lfs
Docs: https://github.com/contact

a whole bunch of times. Is this because the owner of the GitHub repo doesn't have the LFS trial yet, or is it likely that I'm doing something else wrong?
Regardless, what's the best way to abandon my attempted trial of LFS so that I can push my work? I tried using untrack, and I tried deleting the .gitattributes file. Neither worked. I'm still a bit of a git novice, so it's possible I'm missing something obvious. Thanks.

It might be enough to delete .git/hooks/pre-push.  But only if your commit weren't made with files tracked by git-lfs.  If so, you need to rewrite the regarding commits.
@technoweenie We need an uninit command.

@technoweenie We need an uninit command.

+1 for an uninit command. This just bit me.

@michael-k thanks for the detailed solution to try. How do I get git lfs to stop tracking a file?

By deleting .gitattributes (one per directory) or by removing the respecting line from it.  There's also .git/info/attributes.
There's also the untrack command, but I wouldn't rely on that.

I got everything to work here. Thank you so much.
I deviated slightly from your instructions. Basically I took all the large files I had, and .zip them up into something small enough for github. Then I used rebase to go back and move a .gitignore *.csv as one of the first commits, and deleted the history of ever commit involving the initialization of the lfs. Then I used git rebase (could have done it at the same time if this wasn't so exploratory for me) to go back and edit the commits involving the .csv's (many of these commits also made changes to code that I wanted to save), and each of those steps just did a git rm for the problematic file. I can now push. Thank you. @mpinsky

Glad I could help.
csv files shouldn't be a problem for git. The delta compression is pretty good. When the csv files change over time, it's definitely a good idea to commit the raw files instead of zip files. With zip files, git must save every seperate file. (Consider reading man git-gc.)
Git's problem with large files has to do more with binary files than with file size.

The .csv's were too big to upload to github. Otherwise I would be simply
tracking them with git :-/
Good points though, thanks for sharing the wisdom.
On Monday, June 22, 2015, Michael<EMAIL_ADDRESS>wrote:

Glad I could help.
csv files shouldn't be a problem for git. The delta compression is pretty
good. When the csv files change over time, it's definitely a good idea to
commit the raw files instead of zip files. With zip files, git must save
every seperate file. (Consider reading man git-gc.)
Git's problem with large files has to do more with binary files than with
file size.
—
Reply to this email directly or view it on GitHub
https://github.com/github/git-lfs/issues/369#issuecomment-114302125.


enable TLS for metrics remote writer

pass an optional CA to the device
This is the CA that signs the metrics receiver's certificate
https://issues.redhat.com/browse/ECOPROJECT-579
enable TLS in device remote write
Signed-off-by: Yaron Dayagi<EMAIL_ADDRESS>

@ydayagi - please rebase; that should resolve build failure


Some solution for 06a

I renamed the original file, so I'll need to do some conflict resolution before merging this.
And I'll review the code.


Define GOT-Relative data relocation

This introduces a new relocation R_RISCV_GOT32_PCREL that follows the existing wording to “R_RISCV_32_PCREL”, but instead evaluates to the 32-bit offset between a GOT entry for a given symbol and the current location where the relocation is applied, so its equation would be “G + GOT - P + A”.

The format of the table is broken. Check it yourself and please fix the table format.

I think the formatting should be fixed now

s/PC relative/PC-relative/ for the sake of consistency.
Other than that, LGTM

LGTM, but just require a PoC implementation either LD.BFD or LLD as convention :)

I think this extension is fine.
We need a code example for my question on https://github.com/riscv-non-isa/riscv-elf-psabi-doc/issues/399 . You can edit the description of the first comment to link to #399

I think this extension is fine.
We need a code example for my question on #399 . You can edit the description of the first comment to link to #399

Done

We got LGTM from two linker experts, also PoC for lld https://github.com/llvm/llvm-project/pull/72587
c.c. @Nelson1225 just remind you there are new relocation added.


tf.estimator.Estimator breaks when using python 3.5 type annotations

Minimal example:
import tensorflow as tf

def model_fn(features: dict, labels: tf.Tensor, mode: str):
    pass

estimator = tf.estimator.Estimator(model_fn)

results in
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/estimator/estimator.py", line 173, in __init__
    _verify_model_fn_args(model_fn, params)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/estimator/estimator.py", line 742, in _verify_model_fn_args
    args = set(_model_fn_args(model_fn))
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/estimator/estimator.py", line 737, in _model_fn_args
    return tuple(tf_inspect.getargspec(fn).args)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/tf_inspect.py", line 45, in getargspec
    if d.decorator_argspec is not None), _inspect.getargspec(target))
  File "/usr/lib/python3.5/inspect.py", line 1045, in getargspec
    raise ValueError("Function has keyword-only arguments or annotations"
ValueError: Function has keyword-only arguments or annotations, use getfullargspec() API which can support them

System information

OS Platform and Distribution: Linux Mint 17.2
TensorFlow version: v1.2.0-0-g12f033d 1.2.0
Python version: 3.5

Hi, @ngc92 . I guess that your model_fn is invalid. This is a detailed tutor which might be useful for you.
def model_fn(features, targets, mode, params):
   # Logic to do the following:
   # 1. Configure the model via TensorFlow operations
   # 2. Define the loss function for training/evaluation
   # 3. Define the training operation/optimizer
   # 4. Generate predictions
   # 5. Return predictions/loss/train_op/eval_metric_ops in ModelFnOps object
   return ModelFnOps(mode, predictions, loss, train_op, eval_metric_ops)

@martinwicke can you comment or redirect? Thanks.

Yes, we're using getargspec, which doesn't support type annotations. This is worthwhile to fix. If someone wants to take it on, the code should be relatively isolated in a single function which is checking arguments.

Added a PR #12276 for this issue.


Fix local font urls

WHY are these changes introduced?
Follow up to #2616
@karreiro I found a bug in the regex that matches custom font URLs, which impacts the functionality of the CdnFonts middleware, and fonts from #{shop}/cdn/fonts) don't serve properly.
https://github.com/Shopify/cli/blob/37b90672fa259d52f606fe864889e6854f1148b4/packages/cli-kit/assets/cli-ruby/lib/shopify_cli/theme/dev_server/proxy.rb#L107
https://github.com/Shopify/cli/blob/37b90672fa259d52f606fe864889e6854f1148b4/packages/cli-kit/assets/cli-ruby/lib/shopify_cli/theme/dev_server/cdn_fonts.rb#L72

WHAT is this pull request doing?
I adjusted the regex that matches custom font URLs to exclude those containing /cdn/fonts so that the fonts from the Shopify font library and custom fonts from the theme's assets directory serve correctly.
I also added tests for that.

How to test your changes?
Serve a theme that uses fonts from the Shopify font library and custom fonts from the theme's assets directory. The fonts should load as expected.

Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

issue remains, i'm using:
   "@shopify/cli": "^3.49.1",
   "@shopify/theme": "^3.49.1",

👋 Hi @matedemorphy, the fix for this issue will be included in Shopify CLI's next patch, 3.49.3.

I'm still seeing this issue with 3.68.0
Using shopify theme dev --legacy works as expected.


Deploying grabdish fails with ERROR: provisioning-apply failed for code /home/normanaber/grabdish/microservices-datadriven/workshops/dcms-oci/config/threads/grabdish in state folder /home/normanaber/grabdish/dcms-oci-run/state/threads/grabdish.  No output file.

Deploying grab dish fails with the following error:
ERROR: provisioning-apply failed for code /home/normanaber/grabdish/microservices-datadriven/workshops/dcms-oci/config/threads/grabdish in state folder /home/normanaber/grabdish/dcms-oci-run/state/threads/grabdish.  No output file.

Potential issue, from grabdish/dcms-oci-run/log/terraform.log :
╷
│ Warning: Attribute Deprecated
│
│   with random_password.database_admin_password,
│   on database.tf line 14, in resource "random_password" "database_admin_password":
│   14:   number  = true
│
│ **NOTE**: This is deprecated, use `numeric` instead.
│
│ (and one more similar warning elsewhere)
╵
╷
│ Error: execution halted
│
│
╵


Federated login

I will investigate whether it is easier for now to just create a simple user registration system (whereby we have to approve the creation of the user) and at a later stadium integrate the federated login system.


[API] Rename #dtype to #backend or something like this

Reason: in many cases (like NMatrix), as well as intuitively, container.dtype reads like "type of data stored in this container" (like numbers or string or what). But Daru's #dtype means "storage backend", which is confusing.

Agreed. How about container, or data_store? We'll also need to make some non-trivial changes in notebooks.


This fixes the issue that the underline wasn’t correct

when tabView is initially setup

Thanks, @capostman. Nice PR 🙏 ❤


error: Error: Cannot find module 'lodash/Lang/isObject'

I get following error after sane up
server   | error: Error: Cannot find module 'lodash/Lang/isObject'
server   |     at Function.Module._resolveFilename (module.js:336:15)
server   |     at Function.Module._load (module.js:278:25)
server   |     at Module.require (module.js:365:17)
server   |     at require (module.js:384:17)
server   |     at Object.<anonymous> (/home/hackday/projects/hackday-sane/server/node_modules/sails-hook-dev/index.js:7:16)
server   |     at Module._compile (module.js:460:26)
server   |     at Object.Module._extensions..js (module.js:478:10)
server   |     at Module.load (module.js:355:32)
server   |     at Function.Module._load (module.js:310:12)
server   |     at Module.require (module.js:365:17)
server   |     at require (module.js:384:17)
server   |     at /usr/local/lib/node_modules/sails/lib/hooks/moduleloader/index.js:432:26
server   |     at /usr/local/lib/node_modules/sails/node_modules/lodash/dist/lodash.js:3740:15
server   |     at forOwn (/usr/local/lib/node_modules/sails/node_modules/lodash/dist/lodash.js:2105:15)
server   |     at Function.reduce (/usr/local/lib/node_modules/sails/node_modules/lodash/dist/lodash.js:3737:9)
server   |     at /usr/local/lib/node_modules/sails/lib/hooks/moduleloader/index.js:394:29

@tarlepp this is a known issue with a fix pending in the sails-hook-dev module that has not yet been released in npm.  You can get around it by installing that module from the source with this command
npm i balderdashy/sails-hook-dev --save in the server folder.

Yeah, that fixed my problem - Thanks!


Nested Scrolling

The library is amazing.
I have more than 10 columns and because of that I had to use the Horizontal Scroll View with nestedScrollingEnabled attribute enabled for the TableView. This combination works great, but nested scrolling is only available for API 21 and above, so is it possible that it can be enabled for API below 21?

Hi，How to achieve horizontal scrolling？I tried a lot of ways, did not solve this problem

I need this horizontal scrolling too!

Hi,

set a TableColumnDpWidthModel or TableColumnPxWidthModel so you are able to have a table that is wider than your screen by setting the width for each column.
put your TableView inside a HorizontalScrollView

Best regards,
Ingo


Additional quote is breaking password authentications

According to the Grafana documentation you need to set a password in tripple quotes, if it contains certain characters. Unfortunately this role has an additional closing quote, which causes the quotes to be recognized as part of the password...
https://github.com/cloudalchemy/ansible-grafana/blob/424d400e7786e3c69d98f9d60338b5d30dd23f7a/templates/grafana.ini.j2#L32

If you can, make PR to this role with a fix from mentioned repository (https://github.com/D3N14L/ansible-grafana/pull/1).

Sorry - did not notice that I created the PR in my own Repo.


Does Option work with Entity Framework?

Not sure if issues is the right place to ask a question. Please do tell me if that's not the case.
I just wanted to know if "Option" works with LINQ for Entity Framework.

Hi @zyx3nzh7
I haven't tried.  Have you tried and are experiencing problems?

No, but I can try and report back on Thursday.

I tried and it doesn't work since LINQ to Entity only supports some (primitive?) types like (Int32, String, int? and Guid) and Nullable. Maybe a few more but for Option an Exception is thrown at runtime. Unfortunately this makes this great library less useful for me.

Where do you wish to use the Option value?  In the query expression or the select projection?  (or both?)

Actually, if you can post an example query that'd help.  I'm wondering how you're using the value?  Perhaps there's some cunning that can be done by projecting to IEnumerable and doing an 'in'.

//relevant part in the model
[Column("age")]
public Option age { get; set; }
//the code
var db = new testContext();
var temp = db.TestTabelle.Where(x => x.AlterMonad == null).
Entity Framework tries to translates the lamda function to SQL at runtime and fails. Unfortunately i have no clue how the translation works internally and how one could extend it.


deposit block / render hasResource

Resource Collection

Solves issue Resource Collection #52

Robots now "carry" blue blob if they carry a resource
Robots can collect resource and deposit it if on base

What to test:

[x] Move to resource and collect it (blue blob should appear once "collect resource" is executed)
[x] Use the "move to" block to return to base (check if blob is rendere correctly)
[x] deposit resource on base (blue blob should disappear once "deposit resource" is executed)

Nice, graphic looks quite decent as well


Authorization not found when running influxdb 2.x in compose

Everything seems to start fine, I can log in, but when trying to create API tokens, I am never presented with the token, but it says it was created successfully.
Looking at the logs I see:
ts=2022-07-27T16:30:01.250710Z lvl=info msg=Unauthorized log_id=0bx7jsPW000 error="session not found"
ts=2022-07-27T16:30:05.854053Z lvl=info msg=Unauthorized log_id=0bx7jsPW000 error="authorization not found"
ts=2022-07-27T16:30:08.060165Z lvl=info msg=Unauthorized log_id=0bx7jsPW000 error="authorization not found"
ts=2022-07-27T16:30:09.173151Z lvl=info msg=Unauthorized log_id=0bx7jsPW000 error="authorization not found"
ts=2022-07-27T16:30:11.393559Z lvl=info msg=Unauthorized log_id=0bx7jsPW000 error="authorization not found"
ts=2022-07-27T16:31:06.948966Z lvl=info msg=Unauthorized log_id=0bx7jsPW000 error="authorization not found"
ts=2022-07-27T16:31:09.170845Z lvl=info msg=Unauthorized log_id=0bx7jsPW000 error="authorization not found"
Here is my docker-compose.yml
influxdb:
image: influxdb:latest
container_name: influxdb
restart: unless-stopped
privileged: true
ports:
- 8086:8086
environment:
- INFLUXDB_HTTP_AUTH_ENABLED
- DOCKER_INFLUXDB_INIT_MODE=setup
- DOCKER_INFLUXDB_INIT_BUCKET=hlab-influxdb
- DOCKER_INFLUXDB_INIT_USERNAME=labuser
- DOCKER_INFLUXDB_INIT_PASSWORD=pw
- DOCKER_INFLUXDB_INIT_ORG=home-lab
- DOCKER_INFLUXDB_INIT_RETENTION=365d
volumes:
- ./influxdb/data:/var/lib/influxdb2
- ./influxdb/config:/etc/influxdb2
networks:
- monitor-net
labels:
org.label-schema.group: "monitoring"

Are you creating the token through the UI? If so, once it is created you need to click on the name of the token to actually be able to see it.

Are you creating the token through the UI? If so, once it is created you need to click on the name of the token to actually be able to see it.

 I was looking at examples online and it looked like it displayed it right away and you captured it then and then it was never retrievable again. I figured that was the case with the banner text warning you could only access them once. This resolved my issue, thank you very much @jeffreyssmith2nd

Tokens are found under the Load Data menu item under API tokens.

ah, thanks

shouldn't this be "forbidden" 403 then?

I would say not, because in your case you are currently missing authentication entirely.
This status is similar to [401](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401), but for the 403 Forbidden status code, re-authenticating makes no difference. The access is tied to the application logic, such as insufficient rights to a resource.

I am facing the same issue. deployed a helm chart and also created a user with a password inside the pod. but i'm getting "authorization not found", "session not found"


Supported RL framework?

Hi there,
Thanks for your nice work! I tried to train an RL agent using stable baselines for your gym. But I've got several bugs since your gym is slightly different than the standard OpenAI gym. I was wondering which RL framework did you use in your RL experiments or which RL framework is compatible with your gym? Here are a few differences I encountered for stable baselines (these are rather easy to solve but I was just wondering which RL framework would work out of box with your env)

no observation_space and action_space defined
reset method returns more than just obs
to be discovered...

Hi,
Thanks for checking out our work.
This was a product of our research work and at the time we didn't use an RL framework. The gym API has also changed quite a bit since we've made this.
Currently, I'm the only maintainer of this and I'm aware of updates that I'll need to make to fully support different RL frameworks.
The plan for the upcoming summer is to first update some of the functions to the current version of Gym API. And update control input types to support more than just speed and velocity, and also fully define the observation and action spaces correctly.
Stay tuned for updates! Again, thanks for checking this out!