fix(threading): Improved threading for the AWS Service

Context
Improved the threading implementation for the AWSService base class to allow for an optional iterator parameter to be passed to it. It will then create a thread per item. This can be used in cases when we are not just listing resources in a region - such as collecting user data for each EC2 instance or collecting details for each EBS snapshot.
Also changed the implementation to use a thread pool.
Description
Added an optional iterator for when threading needs to be implemented against a list of items, not just the regional clients
Also changed from using the threading lib to the concurrent.futures.ThreadPoolExecutor thread pool. It manages a pool of reusable threads and efficiently handles a large number of tasks. This is to reduce the overhead of creating a thread once-per-iterator-item.
The behavior is nearly identical to the previous implementation when an iterator parameter isnt passed to the threading_call function (except for the max_workers set to 10, which may be less than the number of concurrent threads before when 1 was created per region. This could be increased to ~20 (this is approximately the max regions I think?) to replicate the concurrency achieved before). The back-off strategy implemented by boto for retries should avoid the API rate-limiting being an issue. We could also increase the max retries to 5 by default for added safety.
But now an additional iterator parameter can be passed. I have implemented this for the functions that collect information for the snapshots and ec2 instance user data.
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The threaded functions were logging a message, which became a lot, especially when threading per item (when collecting user data/snapshot data).
I modified the __threading_call__ function to print a message based on the call that was being threaded - so that this message is only logged once per call
Also added the thread pool

Can also use this to replace the __threading_call__ implementation in the s3 service

Can also use this to replace the __threading_call__ implementation in the s3 service

Can you include a # TODO comment there?


Trying to install gem results in YAML error

Specifically, trying to install session_off 0.3 with rubygems 1.8.10 results in: "Psych::SyntaxError: couldn't parse YAML at line 46 column 13"

Reproduced on ruby 1.9.2 - caused by YAML line:
description: `session :off, :only => :foo, :if => Proc.new { |req| req.params[:bar] }`

Will release a 0.3.1 version with a fix.
Thank you for reporting this !


refind: 0.13.2 -> <IP_ADDRESS>

Among other fixes contains support for newer gnu-efi.
Changes: https://sourceforge.net/p/refind/code/ci/master/tree/NEWS.txt
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

LGTM. Can you maybe add the boot.uefiCdrom test to passthru.tests?

Attempted to add it as passthru.tests = { uefiCdrom = nixosTests.boot.uefiCdrom; };. Not sure if I need to do anything special to make sure this test is not attempted on unsupported platforms.


[HOTFIX] fixing missing create marquee between 900 and 1200

description
Fix issue where the create marquee disappears between 900 and 1200px width.
Resolves: MWPW-NUMBER-PENDING
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/create/flyer?lighthouse=on
After: https://create-marquee-fix--express--adobecom.hlx.page/express/create/flyer?lighthouse=on

Codecov Report

Merging #206 (4f95c00) into stage (3b293fc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            stage     #206   +/-   ##
=======================================
  Coverage   35.49%   35.49%
=======================================
  Files          27       27
  Lines        9465     9465
=======================================
  Hits         3360     3360
  Misses       6105     6105

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


The new editor doesn't showup from swagger node

When i do swagger project node ,the new editor (v3) is not shown. old one is shown..

Running into this as well, is there a way to configure the js/html used for the editor?

It looks like you can specify an environment variable swagger_swagger_editorDir to override it, however v3 of the the swagger-editor project doesn't ship with an index.html file and I end up getting Cannot GET / when I try to use v3.
swagger_swagger_editorDir=node_modules/swagger-editor  swagger project edit

So it looks like v3 doesn't have save support yet, https://github.com/swagger-api/swagger-editor/issues/1241.
I was able to get the swagger project edit command to serve v3 by dowloading the tar.gz and pointing the command to the extracted directory. However since it doesn't support saving files yet it seems like sticking with v2 for the time being is the best bet.

our swagger file is huge and v2 is very choppy.. takes 10 secs to load.. V3 takes 2 sec..


Let the possibility to removeObservers

To give full control to the client, we need to let him delete all the registered observers.
Originally posted by @adrien-nf in https://github.com/tylp/memo-draw-engine/pull/10#discussion_r716566200

After a bit of thinking, shouldn't it be the opposite?
Start with no observer, but give a possibility to add the default observers, and a possibility to add custom observers.


Remove fab toolbar on projects page

This may have already been solved in another branch. Please check.


Fix/already closed stream

Description
Handle case where the stream observer was already closed by ignoring problems during the onError call.

Codecov Report

Merging #58 (1bf62e0) into main (5eafd9a) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main      #58      +/-   ##
============================================
- Coverage     11.91%   11.90%   -0.01%
  Complexity       35       35
============================================
  Files            65       65
  Lines          3324     3326       +2
  Branches        356      356
============================================
  Hits            396      396
- Misses         2872     2874       +2
  Partials         56       56

Impacted Files
Coverage Δ

...nce/engine/grpc/observers/EnvelopeEventObserver.kt
0.00% <0.00%> (ø)

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 5eafd9a...1bf62e0. Read the comment docs.


is this compatible with protractor 4

@bennyhat: do you know if by the chance this plugin is compatible with protractor6?
\cc @cnishina, @juliemr:
Related to: https://github.com/angular/protractor/issues/370


tt-smi indexing doesn't align with what's in /dev/tenstorrent/ and causing slight confusion on what needs to be passed where

Until such time as we have some other means of having a unique identifier for a card we should probably have the number that shows up in /dev/tenstorrent/ match the ID that tt-smi shows.
I.E.
# ls -lah /dev/tenstorrent/
total 0
drwxr-xr-x 2 root root     60 Dec  9 14:34 .
drwxr-xr-x 8 root root    400 Dec  9 14:34 ..
crw-rw-rw- 1 root root 236, 6 Dec  9 14:34 6
#

vs.

This will tend to lead folks to using 0 instead of 6 for things like reset.

Related to #56


Fix typo

Fix MicrosoftDocs/azure-docs#109168

@reachnijel
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off Looks good


scripts: generate certs

This automates the commands documented here:
https://docs.pingcap.com/tidb/stable/enable-tls-between-components
What problem does this PR solve?
Testing TiKV locally with TLS
What is changed and how it works?
./scripts/certs-generate.sh can generate all component certs and configuration files needed for local TLS.
We can add more TiDB components to this script.
Related changes
Eventually I plan to update our docs to point to this utility, but want to give it a chance to improve a bit more first.
Check List
Tests

Manual test

I ran tikv, pd, pd-ctl, and tikv-ctl with tls enabled using the output from this script (certs and config).
Release note
Add a utility to generate TLS certs for local development

Can something like https://github.com/cloudflare/cfssl be used to simplify the setup?

Can something like https://github.com/cloudflare/cfssl be used to simplify the setup?

That looks like it does the same thing as openssl in thia case but also has the downside of requiring some users to compile it from source.

It changes scripts into configurations (json), the later is easier to view and edit obviously.

It changes scripts into configurations (json), the later is easier to view and edit obviously

Certainly. I would be happy if we had any tool for generating TLS certs and I would welcome other approaches. It is good to have this current approach available that only requires installing the openssl cli which is widely available.

If we want to have a really good implementation of this I would suggest doing it in tiup as part of playground rather than re-using CLI tools (I actually asked about this, currently tiup has support for enabling TLS generally but not for playground). This PR was the quickest approach since I got to use our existing docs.

@BusyJay I am hoping to get this PR committed now to help those that need to test TLS.

I don't feel comfortable for including the generating scripts in this repository. On the one hand, existing tools like cfssl I mentioned above should be sufficient for the job. On the other hand, people may think it's the standard way to setup certificates chain for TiKV, which is not, it's just for local testing.

I don't feel comfortable for including the generating scripts in this repository. On the one hand, existing tools like cfssl I mentioned above should be sufficient for the job. On the other hand, people may think it's the standard way to setup certificates chain for TiKV, which is not, it's just for local testing.

that's my concern as well. we should leave sufficient hint for the user (like change the script name to include "test") that this script is for test use only.

existing tools like cfssl I mentioned above should be sufficient for the job

Again I am happy to have a tool that uses cfssl or any other approach. Unfortunately this doesn't actually exist right now. That's the same as saying openssl is sufficient which is true but whatever tool is used the process still needs end-to-end automation and some default configuration.

Thanks for the feedback! I changed the name of the script to certs-self-signed.sh which should make its purpose self-explanatory. I also updated the  comments at the top of the script to say it is for testing and not production.
I would like to put in context the scenario we seem to be concerned about: someone decides to checkout the repo, go looking for things in the scripts directory and assume that a script is good for production usage instead of for development (all scripts are for development), not inspect the script, not read any of our deployment documentation, not get advice from anyone else on how to deploy to production, and then use the self-signed certs in production. I don't know what I could possibly do to help such a user have a secure deployment. We would probably need to delete our docs page on self-signed certs for testing as well in case they would use that in production.

...I don't know what I could possibly do to help such a user have a secure deployment...

I wish everyone just knows how things work like you do and always do correct things, so that the scripts won't be miss used.

I wish everyone just knows how things work like you do and always do correct things, so that the scripts won't be miss used.

Will you file an issue to remove the documentation page on self-signed certificates? There are no warnings there about misuse and the page is much more visible.

Now the script will just exit and ask the user to copy an openssl.cnf themselves. I can test out generating a minimal openssl.cnf a little later. There are so many options and I have no idea if some of them are needed.

Actually I'm OK with a doc to explain what's the goal, what each step is doing and what's the best practice

I think the docs have the same problem. Removing automation does not fix the issue and changing what is written doesn't fix it either. The careless users we are worried about that put self-signed certs into production from a test script can also skip through the docs without properly reading them.

Tested on my Linux environment, found this:
# lxt @ lxt-dev in ~/repo/tikv/scripts on git:xt/handle-encryption-logic-correctly x [12:01:16]
$ ./certs-self.sh
generate root.crt
unable to load Private Key
140060178023744:error:0909006C:PEM routines:get_name:no start line:../crypto/pem/pem_lib.c:745:Expecting: ANY PRIVATE KEY

Tested on my Linux environment, found this:
# lxt @ lxt-dev in ~/repo/tikv/scripts on git:xt/handle-encryption-logic-correctly x [12:01:16]
$ ./certs-self.sh
generate root.crt
unable to load Private Key
140060178023744:error:0909006C:PEM routines:get_name:no start line:../crypto/pem/pem_lib.c:745:Expecting: ANY PRIVATE KEY

@hunterlxt you can add set -x to see exactly where the failure occurred. Does this happen every time you run it?

@hunterlxt you can add set -x to see exactly where the failure occurred. Does this happen every time you run it?

@hunterlxt Stack Overflow has some suggestions about what you encountered. Maybe there is an encoding issue, etc?

@hunterlxt Stack Overflow has some suggestions about what you encountered. Maybe there is an encoding issue, etc?

@gregwebs Maybe something like https://github.com/pingcap/tidb/pull/24141 could work for TiKV as well?

@dveeden I am happy with any solution. Not sure if auto-tls will be useable for mTLS for example, but seems good for many testing use cases.

@AstroProfundis do you have plans on supporting TLS for playground?

@AstroProfundis do you have plans on supporting TLS for playground?

No at present, but I'd be happy if anyone would send us a PR to do so.

@AstroProfundis do you have plans on supporting TLS for playground?

No at present, but I'd be happy if anyone would send us a PR to do so.

How would that look like?
Could tiup use something like https://github.com/pingcap/tidb/blob/16d947d7cb9cdb1e0a8ed9276cd4198a4ab52d28/server/tidb_test.go#L753 to create certificates and then start the various components with the right options/config?
Could we make that so that this can be re-used for tiup cluster?

Could tiup use something like https://github.com/pingcap/tidb/blob/16d947d7cb9cdb1e0a8ed9276cd4198a4ab52d28/server/tidb_test.go#L753 to create certificates and then start the various components with the right options/config?
Could we make that so that this can be re-used for tiup cluster?

tiup, tiup-playground and tiup-cluster are 3 different binaries.
tiup-cluster supports deploying clusters with enable_tls: true set in the topology file, and it automatically generate CA and component certs. It does not support custom certs now. The code used to generate certs can be found here and here.
tiup-playground does not support TLS at present, I'm not sure how many code from tiup-cluster could be re-used, while I believe most of them should be useful.


Portfolio Info Architecture - About Me

About Me - Dive deeper into your journey, skills, and passion.

For the sake of time purpose, my About Me portion will be featured on each page


Update ReleaseNotes.md.

Reference Issue

Ref. #690
Ref. #692
Ref. #702
Ref. #706
Ref. #713

What does this implement/fix? Explain your changes.

Any other comments?

Sorry...
I completely forgot to update ReleaseNotes😅


pull request for e2e test at 01/25/2017 13:56:06

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Assignment in scope reverts to value out of scope with goto

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version go1.6.2 linux/amd64 + play.golang.org.

What did you do?

https://play.golang.org/p/-T0o_8p5_T

What did you expect to see?

Err: Hello world (inner scope)
Err: Hello world (outer scope)

I expected this because the err variable which is modified in the inner scope is declared in the outer one. If inner scope uses = for assignment (instead of :=), then err holds it's value in the outer scope. An example of a working modification of err: https://play.golang.org/p/Fh0hwMvb53

What did you see instead?

Err: Hello world
Err: Should not exist

If i declare err in the outer scope, and use isCondition, err := conditional(); in the inner scope, then the value of err is reset to whatever it was when I hit goto Finish (jump to label).

Investigating further, it seems it's not related to goto, but a general capability of the language. For example, this gives the same result:
func testLabelScope() {
	var err error
	err = errors.New("Should not exist")
	{
		err := errors.New("What is this?");
		fmt.Printf("Err: %s\n", err)
	}
	fmt.Printf("Err: %s\n", err)
}

In this case I'd expect a build error. I'm not sure that allowing variable redecleration in inner scopes is... fine. You can see and modify outer scope variables, but as soon as you declare a variable in the inner scope, you can't modify the one in the outer one. It's... confusing.

That is how the language works.  The := operator used in a inner block declares a new variable whose scope is only within that block.  See https://golang.org/ref/spec#Short_variable_declarations .
For much more discussion about := see issue #377.  Closing this one.

It's also allowed with 'var' @ianlancetaylor
var x int
x = 1
{
    var x int
    x = 2
}

Results in same behaviour, no short declaration there.

@titpetric Yes.  Go is a block scoped language.

Okay, thank you. I'll file this under TIL :)

@titpetric Just FYI, this is not in any way unusual or specific to Go. This is absolutely normal behavior for block-structured languages, and established practice since the 1960s.

@griesemer From what I remember, variable declarations in C, C++ and Java are block scoped, but don't allow redeclaration within a nested block. I never noticed redecleration in any of these languages. You couldn't do something like this in any of them:
for (int x=0; x < 10; x++) {
    offset = x;
    for (int x=0; x < 10; x++) {
         pos = offset + (x*10);
    }
}

There might be other block-scoped languages that apply a similar behaviour as golang, I just didn't come across them in the last 20 years or so. And the whole redecleration pattern is anti- even in php, javascript, etc. - since they are "anything goes" languages, it takes discipline to avoid problems.
I'd appreciate a lint check or some kind of static analysis step on build time tell me that I'm doing something that might be unexpected/edge case/fringe. I realize execution behavior here is not likely to change, so I'm wondering if the tools around it can notify the developer of this at some point (a la gcc)

Java and C# do not allow shadowing of local variables. A local variable can
shadow a field though. If I remember correctly C and C++ have the same
rules as Go wrt variable shadowing.
You can use
go vet -shadow [-shadowstrict]
to see shadowed variables in Go files.
I must admit that shadowed variables have been the source of a few hard to
find bugs for me initially. After some practice this becomes less of an
issue.

@griesemer @kostya-sh I stand corrected, c has same result as golang in this case. I threw up a small program just to test it, and oh my... :)
int main() {
    int pos, offset;
    int x;
    for (x=0; x < 3; x++) {
        offset = x;
        printf("x %d ", x);
        int x;
        for (x=0; x < 3; x++) {
            pos = offset + (x*3);
            printf("y %d (%d) ", x, pos);
        }
        printf("ex %d\n", x);
    }
}

@kostya-sh thank you for info about go vet, i'll look into it - seems like a good pre-build step if you're trying to discourage variable shadowing. Global-vs-function scope is easier to control just by keeping global scope clean, but inside function blocks I expect I'll be having a few more hard to find bugs in my future, as you had ;)

@titpetric To be very clear: "Shadowing" is not the same as "redeclaration". A redeclaration is a Go-specific concept which permits a variable, which must have been previously declared in the same block, to appear again on the lhs of a := short variable declaration.
Shadowing is the notion of a new, different, but equally named object (variable, constant, type, etc.) making another object declared in an outer block (scope) inaccessible as that object is now "in the shadow" of the object declared in the inner block.
Java is one of the first languages which doesn't permit shadowing, and it was then followed by C#. The idea in Java was to prevent programming errors.
Preventing shadowing is a pretty severe violation of the encapsulation notion of a block: If shadowing is not permitted, the validity of identifier declarations inside a block depend on the surroundings of that block. The choice of new identifiers is all of sudden restricted by the enclosing environment.
The solution here is not to restrict the language unnaturally, but to provide tools (go vet) to find common errors.


Implement main index page design

Problem statement
As part of the STLT Dashboard, we need to implement the design of the main index page.  This implementation is strictly visual, and we'll have a separate ticket for hooking it up to the API afterward.
What you need to know

Acceptance criteria

[ ] The design of the main index page is implemented

Note that this page will not be interactive; it's strictly the visual design.

https://www.figma.com/file/6mwI5ac6rprACKDzDo4Ady/ReportStream-Workspace-|-2023?node-id=2369-26965&t=crOeR0RfLyczPtzy-0


Utilize map for newMounts to speed up replaceOCIMountSource

Which feature do you think can be improved?
Currently replaceOCIMountSource uses nested loop to match destinations of newMounts and ociMounts.
How can it be improved?
mountSharedDirMounts can return map for new mounts keyed by destination.
This way the inner loop of replaceOCIMountSource can become (one single) map lookup.
Additional Information

@jodh-intel
What do you think of this proposal ?

@tedyu - sounds reasonable to me.
You might also want to enable runtime tracing in configuration.toml to see where the current code hotspots are as that sort of technique may help in other areas.

@jodh-intel
I am working on the PR.
Can you drop the needs-review label ?
Thanks

Done.


Add danger-junit to our Dangerfile

danger-junit lets you report your test suite problems back to the PR elegantly, so instead of having to open Travis every time to see why the test failed, it's already there on the PR.

Oh holy CI God, can you help us again? @dkhamsing

Unfortunately we do not generate the supported xml test output so this is not yet possible

I was reading and apparently Travis doesn't support JUnit XML 😢

Orta on Twitter:

it’s not really the job of travis to do it per-project, for example xcpretty outputs in JUnit ( we use it in artsy/Eigen )

Might be able to achieve something similar by parsing results, will keep you posted


consider a database switch or a multi option for databases

mongodb and couchdb could be two options

for now the litedb database is ok....maybe later we can consider a mongodb or couchdb alternative, as it is really simple to integrate from litedb.


NavigationController interactivePopGestureRecognizer does not work ?

How to Fix it ?
I try to use this
 if let reg = self.navigationController?.interactivePopGestureRecognizer {
            tableView.panGestureRecognizer.require(toFail: reg)
        }

It does not work ?
Can you help me ?

I added another navigation controller on the example Mail app and it works. Are you customizing the back button in your app?

yes, I mean the cell swipe gesture may be conflict with navigationController interactivePopGestureRecognizer. the interactivePopGestureRecognizer can work in tableView another dispaly area (such as tableview footer or header).

In the example Mail App, it does not conflict. So make this could be something else.


Add RDM queue

This PR adds the RDM queue feature of RDM. It allows the RDM driver to save queued messages for retrieval by RDM controllers.
Support of queued messages are important to allow for better communication between the RDM responder and the RDM controller. If, for example, a technician manually changes the DMX address of an RDM responder without sending a SET RDM_PID_DMX_START_ADDRESS request (i.e., the technician uses the RDM responder's DIP switches), the RDM controller will have no way to know that the responder's DMX address was updated.
With RDM queued messages, when the RDM responder's DMX start address is manually updated a RDM message is placed in the responder's message queue. The RDM controller can see that the responder queued a message. The controller will read the queued message and see that the DMX start address was updated.
This PR is created as a draft to allow for discussion.

I believe the implementation for this feature can be somewhat simple. An uint16_t array is needed to store PIDs which have been updated. When a RDM_PID_QUEUED_MESSAGE request is received, the driver can fetch the current value of the PID (using rdm_pd_find() perhaps?) and copy that value into a packet to be sent to the RDM controller.
The uint16_t array may need to be a deque to allow random insertion and deletion of elements in a FIFO manner.
An additional uint16_t will be used to store the last sent queued message. I am not positive what the behavior of the "last sent queued message" feature is supposed to be, exactly. Using the implementation described above, there is potential for the last sent queued message PID to have been updated by the user between RDM controller requests for the last sent queued message. Therefore, the last queued sent message may differ between subsequent requests. I am not sure if the last sent queued message should always be the same.
There may be additional edge-cases that I have not considered yet.

This PR also updates the behavior of some RDM utilities, including the behavior for how parameters are flagged to be saved in NVS. When calling rdm_register_parameter() a flag (bool nvs) has been added to the signature which is stored within the DMX driver. This flag indicates to the driver that the parameter is non-volatile and must persist after power cycle. This will allow users to add custom RDM parameters which persist after reboot and should improve code readability in other places.
An add_to_queue flag has been added to rdm_set_parameter(). When set to true, this will add the parameter which is being set to the RDM queue. This value should be true whenever a parameter is being set as the result of physically interfacing with the ESP32, such as setting dip switches. The flag should be set to false when a parameter is being set as a result of responding to an RDM message.

I've roughed in the RDM_PID_QUEUED_MESSAGE response. In order to make testing more efficient, I'm switching gears to implement a rdm_send_get_queued_messages() function. This function is going to be unique amongst the other rdm_send_get_ functions. Its uniqueness is due to the fact that a queued message request can receive a response from almost any PID. In other words, the RDM controller can send a RDM_PID_QUEUED_MESSAGE request and receive a RDM_PID_DMX_START_ADDRESS response. Because of this I am adding a .pid member to rdm_ack_t so that users are able to read back the PID of the response packet.
I am considering what the signature of the rdm_send_get_queued_messages() function should look like. Because it can receive any PID as a response, handling response data may be difficult. If a known PID response is received it should be trivial to lookup the PID param_str to emplace the data into a buffer, but it is possible that a unknown PID is received which means it would be impossible to emplace data. In other words, it would be up to the user to emplace their own data from queued message responses. Example code may look like the below:
rdm_header_t header = {
  .dest_uid = 0xabcd12345678
};
rdm_status_type_t status_type = RDM_STATUS_TYPE_ERROR;
uint8_t pd[231];
rdm_send_get_queued_messages(dmx_num, &header, status_type, pd, &ack);

if (!ack.err && ack.type == RDM_RESPONSE_TYPE_ACK) {
  if (ack.pid == RDM_PID_DMX_START_ADDRESS) {
    uint16_t dmx_start_address;
    rdm_pd_emplace(&dmx_start_address, "w$", pd, 2, true);
    printf("New DMX start address is %i\n", dmx_start_address);
  }
}

I have been toying with this idea of adding a function which takes a PID as an input and returns a param_str as an output. This would likely reduce code size somewhat. This function would be helpful in this case for cleaning up the handling of these queued messages responses:
if (!ack.err && ack.type == RDM_RESPONSE_TYPE_ACK) {
  const char *param_str = rdm_get_param_str(ack.pid);
  rdm_pd_emplace(pd, param_str, pd, ack.size, true);
}
// The pd array can now be reinterpret casted to the appropriate type and be used.

This probably won't be exactly how I wind up implementing this feature, but I think this is likely the best way forward unless anyone reading this has thoughts!


Add Travis CI for tests

It would be great if travis was set up and badge added to README.

done #2


ordering of the slice doesnt matter for TrimmedParamName, and test wa…

…s failing intermittently due to ordering

Coverage remained the same at 99.621% when pulling 07995cb587687607666a59aa7bc7dfb3d8ce56c5 on trimmedparamname-bad-test into ac1af9b9444bb372e67b28926447fa02b8a8ca58 on master.


Adding more learning rate schedulers

The callback framework in skorch allows for more flexible learning rate schedulers. I would like to adapt my PR on Cyclical Learning Rates: https://github.com/pytorch/pytorch/pull/2016 to skorch. Do you think this would be a good fit for this project?

Yes, it would definitely fit. We already have a WarmRestartLR callback which is based on https://arxiv.org/pdf/1608.03983.pdf. While similar, it does not have the ramp-up phase in form of a saw-tooth LR schedule. Wouldn't it make sense to add this as a feature to the warm restart scheduler instead of providing a separate callback?

It seems like the _LRScheduler  framework restricts learning rate updates to each epoch. (_lr_scheduler.step is called at on_epoch_begin ). Cyclical Learning Rates is updating the learning rate every batch, meaning it would have to update at on_batch_begin. Technically, it is possible to add  a on_batch_begin to WarmRestartLR, but I think that would cause confusion for a user.
For reference here is the Cyclical Learning Rates paper: https://arxiv.org/abs/1506.01186 and its Keras implementation: https://github.com/bckenstler/CLR.

Yeah you are right, after comparing both approaches I agree that it makes little sense to combine both approaches in one implementation (implementation differences aside there is also the issue of semantic differences of restarts and a varying LR schedule).
It would be great if you could contribute this as a separate callback to skorch!

Yeah you are right, after comparing both approaches I agree that it makes little sense to combine both approaches in one implementation (implementation differences aside there is also the issue of semantic differences of restarts and a varying LR schedule).
It would be great if you could contribute this as a separate callback to skorch!


Add .pylidcrc configuration file

Overview
Currently, when running docker-compose -f local.yml up compile_docs, the following warning appears:
compile_docs_1   | reading sources... [ 85%] apidoc_prediction/prediction.src.tests
compile_docs_1   | reading sources... [ 92%] apidoc_prediction/src
compile_docs_1   | Using TensorFlow backend.
compile_docs_1   | reading sources... [100%] apidoc_prediction/src.tests
compile_docs_1   | /usr/local/lib/python3.6/site-packages/pylidc/Scan.py:52: UserWarning:
compile_docs_1   |
compile_docs_1   | `.pylidcrc` configuration file does not exist or path is not set. CT images will not be viewable.
compile_docs_1   | The file, `.pylidcrc`, should exist in /root. This file should have format:
compile_docs_1   |
compile_docs_1   | [dicom]
compile_docs_1   | path = /path/to/dicom/data/LIDC-IDRI
compile_docs_1   | warn = True
compile_docs_1   |
compile_docs_1   | Set `warn` to `False` to suppress this message.
compile_docs_1   |
compile_docs_1   |   warnings.warn(dpath_msg)

Expected Behavior
A .pylidcrc with sensible defaults should be added so that the warning does not occur
Technical details

configuration info.

Acceptance criteria

[ ] a .pylidcrc should be added to the repo with sensible defaults
[ ] running docker-compose -f local.yml up compile_docs should not produce the above error

NOTE: All PRs must follow the standard PR checklist.

Fixed by #199


New Feature Request - Keychain Management

Is it possible for Profile Creator to have a section on Keychain Management?
Our Macs are connected to Active Directory, whenever we reset the AD password users get that "Create New or Update Keychain prompt."
Is there a way for Profile Creator to auto create new keychains after an Active Directory password rest?

@jekastner the Keychain issues you're experiencing cannot be resolved by ProfileCreator.
You would want to look at something like NoMAD to help with this.  I know that a related project - NoMADLogin - can also help deal with this by checking if the supplied password at the loginwindow unlocks the keychain, and if not can optionally create a new keychain automatically.

@jekastner Yes, as @apizz said, ProfileCreator only creates profiles, so if Apple doesn't have a profile plugin och a setting for that that we could put in a profile, then ProfileCreator can't help with that.
I would also suggest looking at NoMAD. It's free for the AD version and it's a very good tool.


Improved printing for floats

All floats I've tried now print correctly.
See https://github.com/zig-lang/zig/issues/375#issuecomment-338805713 for the first commit. The further changes are

-0.0 now prints correctly, thanks @tiehuis
you actually need 9 digits for f32s (I think 8 would be enough if you used the next digit to round, but if there's a lot of 9s at the end you'd need to carry and it'd be a pain); for f64s, I just use the whole buffer Errol outputs
fixed some more small typos in our Errol code
Errol has a bug, possibly? See https://github.com/marcandrysco/Errol/issues/8. I had to make the change I mentioned in that issue to get the tests to succeed.

For the last bullet, maybe we want to wait for an upstream confirmation and patch.
Tests
The tests I made are here (see the usage file at the bottom). All the tests I've run have passed. In particular the f32 tests are intended to show by exhaustion that we're correct for all f32s.

f32 I iterate over all positive u32s, bitCasting them to f32s, skipping the NaNs and Infs, and printing the rest to stdout in zig. This is piped into a C program where I scanf with %f and check if the nth one I get bitcasts back to n. This should exhaustively test all positive f32s.
If you want to check all f32s, just change the 0x80000000s to 0xffffffffs in both files. I didn't just because it takes about 2 hours to run already.

rand This generates some random numbers with xoroshiro128+, bitcasts to f64, prints them, pipes them into C, scanfs them with %lf and checks if their the same. The more times you run it, the greater your confidence should be that we're correct :)

Excellent work, @scurest.
I'm going to wait for the tests to pass, and then merge.

Ah, I forgot to ask: I couldn't get the zig files for the tests to compile in release mode. How do you do it?

Are you asking how to run a test in release mode?
zig test foo.zig --release-fast
or
zig test foo.zig --release-safe

Well, I meant compile say f32-out.zig in release mode.
If I use zig build-exe --release-safe f32-out.zig I get linker errors from lld for __fixunsdfti, __umodti3, __udivti3, etc. The u128 stuff.
If I add --library c I get a binary but that binary only prints three floats before printing "index out of bounds" and segfaulting.

If I use zig build-exe --release-safe f32-out.zig I get linker errors from lld for __fixunsdfti, __umodti3, __udivti3, etc. The u128 stuff.

Hmm. This is really surprising, I'm looking into this right now. At first glance it seems like LLVM optimization is deleting functions that are marked with linkonce linkage, which according to http://llvm.org/docs/LangRef.html#linkage-types seems like shouldn't happen.

I fixed the compiler-rt problem in master branch. The other problems you ran into I see the same problem, and it's a bug in zig. I'm looking into it. I'll merge this PR though, because this is better than status quo.


TypeError: list_sessions() takes 1 positional argument but 2 were given

Description

Hi, thanks for implementing this fantastic client side resilient features, this is very helpful to recovery state from a client side restart, I tried it locally with the example list on readme (1 local kernelgateway, 1 local jupyterlab client), but saw below error when trying to run a regular jupyter lab with synchronizer enabled
.venv/lib/python3.8/site-packages/jupyter_server_synchronizer/manager.py", line 234, in list_sessions
        out = await super().list_sessions(self)
    TypeError: list_sessions() takes 1 positional argument but 2 were given

Reproduce

Following the readme

on one terminal run the kernelgateway server up

$ jupyter kernelgateway \
    --port 9999 \
    --JupyterWebsocketPersonality.list_kernels=True

on another terminal run the jupyter lab up

$ jupyter lab \
    --gateway-url=http://<IP_ADDRESS>:9999 \
    --ServerApp.session_manager_class=jupyter_server_synchronizer.SynchronizerSessionManager \
    --SynchronizerSessionManager.database_filepath=jupyter-database.db \
    --SynchronizerSessionManager.autosync=True \
    --SynchronizerSessionManager.log_level=DEBUG

Then observed the error msg been pop out on client side

.venv/lib/python3.8/site-packages/jupyter_server_synchronizer/manager.py", line 234, in list_sessions
        out = await super().list_sessions(self)
    TypeError: list_sessions() takes 1 positional argument but 2 were given

Expected behavior
The expected behavior is to see it run without exception, not sure if it's some deps incompatible issue, but according to https://github.com/jupyter-server/synchronizer/blob/main/pyproject.toml, only seen to required jupyter_server>=1.16.0 and python >=3.7
Context

Operating System and version: Linux ubuntu 20.04
Browser and version: Chrome
Python: 3.8
Jupyter Server version: 1.18.1
Jupyter Lab version: 3.4.5
Jupyter Server Synchronizer version: 0.0.8
Jupyter Kernel Gateway version: 2.5.1

Command Line Output

$ jupyter lab \
    --gateway-url=http://<IP_ADDRESS>:9999 \
    --ServerApp.session_manager_class=jupyter_server_synchronizer.SynchronizerSessionManager \
    --SynchronizerSessionManager.database_filepath=jupyter-database.db \
    --SynchronizerSessionManager.autosync=True \
    --SynchronizerSessionManager.log_level=DEBUG \
    --ip <IP_ADDRESS> \
    port 8080
[I 2022-08-15 22:20:32.616 ServerApp] jupyter_server_synchronizer | extension was successfully linked.
[I 2022-08-15 22:20:32.626 ServerApp] jupyterlab | extension was successfully linked.
[I 2022-08-15 22:20:32.634 ServerApp] nbclassic | extension was successfully linked.
[I 2022-08-15 22:20:32.784 ServerApp] notebook_shim | extension was successfully linked.
[W 2022-08-15 22:20:32.788 ServerApp] Config option `log_level` not recognized by `SynchronizerSessionManager`.
[I 2022-08-15 22:20:32.813 ServerApp] notebook_shim | extension was successfully loaded.
[W 2022-08-15 22:20:32.814 ServerApp] jupyter_server_synchronizer | extension failed loading with message: 'NoneType' object is not callable
[I 2022-08-15 22:20:32.815 LabApp] JupyterLab extension loaded from .venv/lib/python3.8/site-packages/jupyterlab
[I 2022-08-15 22:20:32.815 LabApp] JupyterLab application directory is .venv/share/jupyter/lab
[I 2022-08-15 22:20:32.818 ServerApp] jupyterlab | extension was successfully loaded.
[I 2022-08-15 22:20:32.821 ServerApp] nbclassic | extension was successfully loaded.
[I 2022-08-15 22:20:32.821 ServerApp] Serving notebooks from local directory:
[I 2022-08-15 22:20:32.821 ServerApp] Jupyter Server 1.18.1 is running at:
[I 2022-08-15 22:20:32.821 ServerApp] Kernels will be managed by the Gateway server running at:
[I 2022-08-15 22:20:32.821 ServerApp] http://<IP_ADDRESS>:9999
[I 2022-08-15 22:20:32.821 ServerApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[W 2022-08-15 22:20:32.825 ServerApp] No web browser found: could not locate runnable browser.
[C 2022-08-15 22:20:32.825 ServerApp]
[E 2022-08-15 22:20:38.509 ServerApp] Uncaught exception GET /api/sessions?1660602038126 (<IP_ADDRESS>)
HTTPServerRequest(protocol='http', host='localhost:8080', method='GET', uri='/api/sessions?1660602038126', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
Traceback (most recent call last):
File "venv/lib/python3.8/site-packages/tornado/web.py", line 1713, in _execute
result = await result
File ".venv/lib/python3.8/site-packages/jupyter_server/services/sessions/handlers.py", line 36, in get
sessions = await ensure_async(sm.list_sessions())
File ".venv/lib/python3.8/site-packages/jupyter_server/utils.py", line 182, in ensure_async
result = await obj
File ".venv/lib/python3.8/site-packages/jupyter_server_synchronizer/manager.py", line 234, in list_sessions
out = await super().list_sessions(self)
TypeError: list_sessions() takes 1 positional argument but 2 were given

Hi @chiawchen - thanks for opening this issue.  Looks like the list_sessions() call inadvertently included self over the course of the changes for #22 and #26.  Is this something you'd like to make a quick PR for?

@Zsailer - FYI - I directed @chiawchen to this repo because they're interested in a form of "client-side HA" where they may want to update the client packages (bring down lab, refresh, restart lab, and reconnect to the persisted sessions that will be serviced by a gateway server).  I couldn't recall if this particular scenario can be detected and recovered from with the synchronizer - but I suspect it can.

sure thing! just add a small fix on it

Addressed via #42

@chiawchen and @kevin-bates, great stuff here! Thank you for reporting and fixing this bug so quickly!
I'll make a new release now.


Add CreateStaticMember and CreateOverrideMember to SynMemberDefn

Adds two convenience functions for adding static members or override members when creating types

Released in 0.9.0

Thanks a lot @Krzysztof-Cieslak ❤


Android build fails when adding the plugin

Hi,
Build on Android fails on my side after I add the plugin. Here are steps to reproduce:

Checkout sampleGroceries app: git clone git@github.com:NativeScript/sample-Groceries.git
Checkout "start" branch: git checkout start
Add android platform: tns platform add android
Add the plugin: tns plugin add nativescript-google-maps-sdk
Build: tns build android

Here is the error:
WARNING: The file: /Users/ickata/Desktop/Tables/tns/sample-Groceries/node_modules/nativescript-google-maps-sdk/platforms/android/AndroidManifest.xml is depricated, you can read more about what will be the expected plugin structure here: https://www.nativescript.org/blog/migrating-n-android-plugins-from-version-1.7-to-2.0
WARNING: The file: /Users/ickata/Desktop/Tables/tns/sample-Groceries/node_modules/nativescript-google-maps-sdk/platforms/android/res/values/nativescript_google_maps_api.xml is depricated, you can read more about what will be the expected plugin structure here: https://www.nativescript.org/blog/migrating-n-android-plugins-from-version-1.7-to-2.0

:config phase:  createDefaultIncludeFiles
    +found plugins: nativescript-google-maps-sdk
Renaming plugin directory to flavor name: /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/src/nativescript-google-maps-sdk -> /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/src/F0
    +found plugins: tns-core-modules-widgets
Renaming plugin directory to flavor name: /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/src/tns-core-modules-widgets -> /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/src/F1

:config phase:  createPluginsConfigFile
     Creating product flavors include.gradle file in /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/configurations folder...

:config phase:  pluginExtend
    +applying configuration from: /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/configurations/include.gradle
    +applying configuration from: /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/configurations/nativescript-google-maps-sdk/include.gradle
    +applying configuration from: /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/configurations/tns-core-modules-widgets/include.gradle

:config phase:  copyAarDependencies

:config phase:  addAarDependencies
    +adding dependency: /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/libs/aar/widgets-release.aar

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'sampleGroceries'.
> Could not resolve all dependencies for configuration ':_F0F1DebugCompile'.
   > Could not find any matches for com.google.android.gms:play-services-maps:+ as no versions of com.google.android.gms:play-services-maps are available.
     Searched in the following locations:
         https://jcenter.bintray.com/com/google/android/gms/play-services-maps/maven-metadata.xml
         https://jcenter.bintray.com/com/google/android/gms/play-services-maps/
         file:/Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/libs/aar/
         file:/usr/local/opt/android-sdk/extras/android/m2repository/com/google/android/gms/play-services-maps/maven-metadata.xml
         file:/usr/local/opt/android-sdk/extras/android/m2repository/com/google/android/gms/play-services-maps/
     Required by:
         :sampleGroceries:unspecified

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 8.026 secs
Command /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/gradlew failed with exit code 1
# build android
┌─────────┬──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│ Usage   │ Synopsis                                                                                │
│ General │ $ tns build android [--compileSdk <API Level>] [--key-store-path <File Path> --key-store-password <Password> --key-store-alias <Name> --key-store-alias-password <Password>] [--release] │
│         │ [--static-bindings] [--copy-to <File Path>]                                                                                │
└─────────┴──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘

Builds the project for Android and produces an APK that you can manually deploy on device or in the native emulator.

### Options

    * --compileSdk - Sets the Android SDK that will be used to build the project.
    * --release - If set, produces a release build. Otherwise, produces a debug build. When set, you must also specify the --key-store-* options.
    * --key-store-path - Specifies the file path to the keystore file (P12) which you want to use to code sign your APK. You can use the --key-store-* options along with --release to produce a signed release build. You need to specify all --key-store-* options.
    * --key-store-password - Provides the password for the keystore file specified with --key-store-path. You can use the --key-store-* options along with --release to produce a signed release build. You need to specify all --key-store-* options.
    * --key-store-alias - Provides the alias for the keystore file specified with --key-store-path. You can use the --key-store-* options along with --release to produce a signed release build. You need to specify all --key-store-* options.
    * --key-store-alias-password - Provides the password for the alias specified with --key-store-alias-password. You can use the --key-store-* options along with --release to produce a signed release build. You need to specify all --key-store-* options.
    * --copy-to - Specifies the file path where the built .apk will be copied. If it points to a non-existent directory, it will be created. If the specified value is directory, the original file name will be used.

### Attributes
<API Level> is a valid Android API level. For example: 22, 23.

Before adding the plugin I was able to build for android and to run the demo app in the emulator. So my setup should be fine.
Let me know if you need additional information.

Please follow the directions on the readme.  This means you don't have the Android google play services SDK installed on your computer:
* What went wrong:
A problem occurred configuring root project 'sampleGroceries'.
> Could not resolve all dependencies for configuration ':_F0F1DebugCompile'.
   > Could not find any matches for com.google.android.gms:play-services-maps:+ as no versions of com.google.android.gms:play-services-maps are available.
     Searched in the following locations:
         https://jcenter.bintray.com/com/google/android/gms/play-services-maps/maven-metadata.xml
         https://jcenter.bintray.com/com/google/android/gms/play-services-maps/
         file:/Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/libs/aar/
         file:/usr/local/opt/android-sdk/extras/android/m2repository/com/google/android/gms/play-services-maps/maven-metadata.xml
         file:/usr/local/opt/android-sdk/extras/android/m2repository/com/google/android/gms/play-services-maps/
     Required by:
         :sampleGroceries:unspecified

I installed the package, but I have the same issue:
http://imgur.com/P1KVfiO

Probably have the wrong ANDROID_HOME set on your computer.  It looks like it's looking in /usr/local/opt/android-sdk, but the image you send has the sdk path at /usr/local/Cellar/android-sdk

That's not the problem. /usr/local/opt/android-sdk is a symlink to /usr/local/Cellar/android-sdk/24.4.1_1:
$ ls -la /usr/local/opt/android-sdk
lrwxr-xr-x  1 ickata  admin  30  4 Май 18:58 /usr/local/opt/android-sdk -> ../Cellar/android-sdk/24.4.1_1

Maybe try installing Google Respository in your SDK manager.

OK, I installed "Google Repository". Now I have the following error (I hope this will be the last one):
:mergeF0F1DebugResources FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':mergeF0F1DebugResources'.
> /Users/ickata/Desktop/Tables/tns/sample-Groceries/platforms/android/src/main/res/values/nativescript_google_maps_api.xml: Error: In DataSet 'main', no data file for changedFile. This is an internal error in the incremental builds code; to work around it, try doing a full clean build.

Can you please give me some directions?

@ickata did you solve this?

Yes, long time ago, but don't remember how :)
It is very important to have latest version of {N} and plugins. Also take a look at the App_Resources/Android/app.gradle file which is the version of Google Play Services (googlePlayServicesVersion). A lot of plugins are using it, some of them may require larger version than yours.
Good luck!

C:\Users\user\AndroidStudioProjects\FunFacts\app\src\main\res\values\dimens.xml:
Error: In DataSet 'main', no data file for changedFile. This is an internal error in the incremental builds code; to work around it, try doing a full clean build.

Scott Junner
on May 12, 2017
If you open the Build menu at the top of AndroidStudio, you will see two options there of interest to you. "Clean Project" and "Rebuild Project".
Try those one after the other and see what results you get. Other than that I don't really know.

It works for me :)


Data viewer: property changes reflection

We need to handle presentation preference changes in sync mode.
Currently user have to refresh data to see changes of most of preference options.

Done

verified


Any disk addition

Dear There，
I just got two questions for this page,

Do you have any disk addition procedures? can we disable the auto claim for disk addition
Is this disk replacement procedure just published in year 2022. just curious this because seems it is not there before.

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

ID: 5205c067-de4e-c9ff-ece6-db1ab176c3e8
Version Independent ID: 956a1232-e27a-902b-f38e-7670ceb53fd7
Content: Replace failed drives on Azure Stack HCI - Azure Stack HCI
Content Source: azure-stack/hci/manage/replace-drives.md
Service: azure-stack
Sub-service: azure-stack-hci
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

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

Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are currently closing this issue, however if you feel the problem persists, please re-open the issue no later than May 20, 2024 with additional information.
We appreciate your contribution and encourage you to propose changes using public PRs, or submit feedback through our updated control at the top of the page, as we phase out Git Issues for this repo. For more information on our feedback systems, see https://aka.ms/ContentUserFeedback.

@YeJianXin As we are moving to a new feedback system, we have opened corresponding work items and are tracking all open issues in our internal database.  We are closing all public issues, but as mentioned above, there are other ways to provide feedback. Thank you again for reaching out! #please-close


Fix alignment of server version on home screen

There is a horizontal alignment issue with the server version on the home screen. We should center it like all the other items on the page, and make sure that there is a reasonable margin around it.

As a debugging clue: I think this issue appears specifically when the version line is longer than the width of the screen, and therefore wraps. (Which is typical if you're on chat.zulip.org or Zulip Cloud, rather than a numbered release, and if you have a phone with default display settings.) When it fits as a single line, it gets centered as intended.


suffering with the problem : tensorflow.python.framework.errors_impl.NotFoundError: NewRandomAccessFile failed to Create/Open: HelmetDetection/frozen_inference_graph.pb : The system cannot find the path specified. ; No such process

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:

TensorFlow installed from (source or binary):

TensorFlow version (use command below):

Python version:

Bazel version (if compiling from source):

GCC/Compiler version (if compiling from source):

CUDA/cuDNN version:

GPU model and memory:

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

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
Describe the expected behavior
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@1advait,
In order to reproduce the issue reported here, could you please provide the TensorFlow version, the complete code and the dataset you are using. Thanks!

@1advait,
In order to reproduce the issue reported here, could you please provide the TensorFlow version, the complete code and the dataset you are using. Thanks!


Update README.md

Improve the readme to display the overview of this project. You can display the code of conduct, License, or Sourceror profiles.

I will work on the Code of Conduct, I will make that in a clean way, so I can take this, plz assign

@AvinashMahanthi Code of Conduct is already added, you can modify the README by embedding the Code of Conduct in it. You can refer to README of the Blog App.

Ok I will embed that, please assign this to me

@himanshusharma89 Please check the pl #28, I had done according to your request


add: nerd-fonts-jetbrains-mono

Same layout as nerd-fonts-hack, with different font.

Commit was given the wrong name - missing an "s" in the "fonts", will fix

Just going to make a new PR because I messed up like 3 times


Testing Task: All API endpoints should be tested for invalid and malformed JSON

Task:
Test all endpoints (see README for list) with incomplete, out of order and invalid JSON request data.
Example:
/report
expects:
{
"beer_id": 123,
"establishment_id": 345,
"device_guid": "6b981317-1c2d-4219-ad79-7235013ad597"
}
Testing should include:
incomplete - do not send all parameters, test leaving out each parameter in turn, and in pairs (eg. leaving out beer and establishments)
out of order - put establishment_id before beer_id and beer_id after device_guid
invalid - put invalid data such as very long strings, strings for integers (so beer_id = "Ricky likes to brew beer")
Also test by sending the wrong headers (eg. instead of application/json send text/html)
see the README file for curl syntax for testing.
For each test case you complete, put it in the comments:
(a) the curl command you ran
(b) the result you received
(c) any output from the brew.js (if you ran it locally for test)

haha.. just noticed this was issue #13 :)

Have at it Tim. Create issues as you see them. Sooner the better but we'll be working at hack night before if not sooner.

@TGrannen it would be awesome if you used a library like Chakram to write a test for each endpoint based on the docs. That way, any errors can be easily reproduced, and added to in the future.

@TGrannen think you could do this work tonight?

Working on this now. Is there a running version of this somewhere or do I need to run this locally? My request using the README example URL is timing out.

You can use http://<IP_ADDRESS>:3000

Thanks. Now I'm just receiving connection refused for everything I try.

curl -H "Content-Type: application/json" -X GET http://<IP_ADDRESS>:3000/beers
curl: (7) Failed to connect to <IP_ADDRESS> port 3000: Connection refused

curl -H "Content-Type: application/json" -X GET http://<IP_ADDRESS>:3000/establishments
curl: (7) Failed to connect to <IP_ADDRESS> port 3000: Connection refused

curl -H "Content-Type: application/json" -X DELETE -d '{"establishment_id": "1", "beer_id": "1"}' http://<IP_ADDRESS>:3000/admin/statuses
curl: (7) Failed to connect to <IP_ADDRESS> port 3000: Connection refused

@pagan8 do we need to whitelist @TGrannen's IP?

Port 3000 isn't blocked.. I think he was hitting the /establishment/:estid/beerstatuses URL issue...

I've created two chakram scripts to test the API. I was able to get a successful response from all of the public API requests, but I couldn't verify anything on the Admin side without a token. Also, I wasn't sure if the Get Establishments call was supposed to vary based on location, because it seemed like I would get the same results no matter what coordinates I put in.
Feel free to run these tests on your own machine and add to them as needed. There are a some control variables at the top of each file if you only want to run certain tests or test a different ip address.
To run the tests locally:
Install Chakram
run the TestAPI.sh
or
run each file individually with the following to output to the screen
mocha PublicAPI.js
mocha AdminAPI.js

Great work @TGrannen I made a few small changes in ba3c26b7e1df5d3b4a5d848f690f214231fc5ff2 that cleans up the TestAPI.sh script just a bit. But overall, this is great.  @pagan8 feel free to get that last test in PublicAPI.js passing

@rickychilcott You are getting a 403 error because the device_guid already reported the establishment_id + beer_id being out.
The logic for the /report works like this:

check if the device_guid already reported it out
if yes, the return 403 (Forbidden) - can't report it out twice
Next it checks for a beer_id + establishment_id in statuses
Gives 400 if there is an error
Gives 404 if not found
Returns 200 OK after updating and inserts entry into reportstate table


Public IP of VPN Clients with custom routes via VPN Gateway

While adding a custom route to a public IP as described on this page, which client's source IP will be visible on the remote resource i.e. storage account in this case? Will it be the public IP of the VPN Gateway? Does VPN Gateway do NATing of clients in this case? I'm trying to set up this scenario to pass traffic to AKS API server public IP via VPN Gateway and to whitelist source IP on AKS API server, but can't find related information.

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

ID: 4f9877ab-87ad-9743-e992-216dfa16c344
Version Independent ID: 03649d3f-3712-8228-0af3-4bd842be5c7a
Content: Azure VPN Gateway: Advertise custom routes for P2S VPN clients
Content Source: articles/vpn-gateway/vpn-gateway-p2s-advertise-custom-routes.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@antonmatsiuk The client's source IP will be an address within the range of the client address pool that you configured when you created the VPN Gateway.

@TravisCragg-MSFT client address pool is a private IP address pool, I mean which public IP address will get clients while accessing public IP <IP_ADDRESS> (storage account) in this guide? Does VPN Gateway perform NAT or should a NAT device be added to GatewaySubnet?  Do we need to add default route to Internet into GatewaySubnet? The documentation in this guide is definitely incomplete.

@antonmatsiuk There is a lot to clarify with this doc and your question, and a lot of dependent concepts. Let me outline the scenarios for you, and give you some guidance on a good direction to go in if you would like to secure your resources.
In all of these scenarios, you have a P2S Client trying to access an Azure Storage Account.
Scenario 1: Client connected to a VPN Gateway with no custom routes:
The P2S Client will connect via its public internet connection to the public endpoint of the storage account. Traffic will not go over the VPN gateway, and the Storage Account will see the normal public facing IP for the machine.
Scenario 2: Client connected to a VPN Gateway with custom routes:
(This scenario assumes you have set up a Private Endpoint for your Storage Account)
The P2S Client will direct traffic across the P2S VPN Gateway, where it will then be sent across the private endpoint to the Storage Account. The Storage Account will see the IP as the Local Network Gateway level IP.
If Private Endpoints have not been set up, then traffic destined for the storage account will go across the VPN Gateway, but never reach the storage account, as there is no path for it.
If you add a default route for all internet to the VPN Gateway, all Internet traffic will be directed there. If you do not have a way to consume the traffic & route it to the internet (via a firewall or other device) the traffic will go into the VNET and have nowhere to go.

@TravisCragg-MSFT thank you for the comprehensive explanation. Then, on the documentation side, the page is missing that this setup works only with services supporting Private Service Endpoints.
Coming back to the problem I described in the beginning. I'm trying to replicate this setup to reach  AKS API server via the VPN gateway. AKS API does not support private service endpoints at the moment. Is there a way to tweak routing in the GatewaySubnet by e.g. adding custom Route Table and NAT Gateway to achieve routing to any public IP address via VPN gateway?  In generic networking concepts this would require setting up a NAT gateway and a couple of custom routes within the GatewaySubnet, but as lots of things get abstracted and not allowed in Azure networking, I'm not sure whether this is possible at all.
Having no possibility to route traffic to AKS API server's public IP via the VPN gateway is a crucial feature missing in Azure at the moment, as there is no option to secure access for remote clients to AKS API server via P2S VPN connections.

Hello everyone. I have found this discussion while looking for a solution to setup a secure connection to a Linux webserver in Azure. So I will join @antonmatsiuk in his quest :) , hoping you guys have a solution/suggestion.
This webserver has specific subdomain/pages which are filtered by IP.
I have successfully setup Azure P2S VPN so the users are able to access those areas by using the private IP of the Webserver. I have then set the VPN gateway to advertise the Public IP of the webserver as custom route into Azure P2S VPN, but the website will not return any traffic so it does not effectively work.  looking at @TravisCragg-MSFT 's reply, I understand that I will need to create a Private Service Endpoint, but they seem only preconfigured services to Azure SQL, Storage etc...
What I would like to achieve is:

the user connects to Azure VPN
the Azure VPN routes the traffic to the public IP of the server through the VPN, plus its private IP.
The user types www.domain.com, which will resolve to the public IP of the server, now routed through the VPN.
at this point a service similar to Azure Private Endpoint / Link should understand that that public IP should be translated in a private IP, so the webpage will come up!

Is there any way I can achieve this?
Thank you!

I have a similar problem. I need to route public IP through VPN, sadly I had no luck with it.
It really should be supported. I hope they introduce it in near future.

Hello
Did anyone try Azure Application Gateway? I have been suggested so but I did not test it yet.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Gaweł Kazimierczuk<EMAIL_ADDRESS>Sent: Tuesday, November 24, 2020 7:06:17 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: mpiugl83<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Public IP of VPN Clients with custom routes via VPN Gateway (#55473)
I have a similar problem. I need to route public IP through VPN, sadly I had no luck with it.
It really should be supported. I hope they introduce it in near future.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/55473#issuecomment-733175426, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AG4B4WLVOY4RMV6QIZ4NIO3SRP72TANCNFSM4NH2CQMQ.

@antonmatsiuk : NAT gateways could not be deployed onGateway Subnet.
see limitations here: https://docs.microsoft.com/en-us/azure/virtual-network/nat-gateway/nat-overview#limitations


replace author comment

Good job.

On Feb 4, 2015, at 17:16, Excerts<EMAIL_ADDRESS>wrote:
You can view, comment on, or merge this pull request online at:
https://github.com/cst1201-d644/lab1/pull/12 https://github.com/cst1201-d644/lab1/pull/12
Commit Summary
replace author comment
File Changes
M Lab1/src/lab1/Pay.java https://github.com/cst1201-d644/lab1/pull/12/files#diff-0 (3)
M Lab1/src/lab1/SalesTax.java https://github.com/cst1201-d644/lab1/pull/12/files#diff-1 (3)
Patch Links:
https://github.com/cst1201-d644/lab1/pull/12.patch https://github.com/cst1201-d644/lab1/pull/12.patch
https://github.com/cst1201-d644/lab1/pull/12.diff https://github.com/cst1201-d644/lab1/pull/12.diff
—
Reply to this email directly or view it on GitHub https://github.com/cst1201-d644/lab1/pull/12.


BOUNTY: Documentation Wiki specialist

I'm looking for an enterprising individual to step up and take ownership of documentation. Please comment here and jump in optimizing the wiki.

Hi Dave, I'd be happy to help on the wiki. I'm by no means a specialist but I'd love to be involved somehow.
Also, I drafted a very vague Roadmap which I'd like your thoughts on:
HAAS Roadmap
Milestone 1: Multi-Agent Communication Protocol Establishment

Create protocols for agents to communicate effectively in a shared environment.

1.1 Build an internal Twitter (global communication space) for agents to surface trends and topics.
1.2 Build an internal Reddit (knowledge accumulation, topic management, etc).
1.3 Build an internal Discord (realtime chat channels, teams, groups, etc).

Deliverable: Demonstrate two or more agents exchanging information and making decisions based on shared data.

Milestone 2: First Collaborative Task Completion

Have multiple agents work together to complete a task that would be impossible for a single agent.

2.1 Identify potential tasks that would be impossible for a single agent.
2.2 Test all tasks with single agents.
2.3 Create list of tasks to act as benchmarks.

Deliverable: Successful collaborative task completion, demonstrating the synergy of the swarm.

Milestone 3: Dynamic Task Adaptation

Have agents be able to adapt to new tasks or changes in their environment without manual reprogramming.
Deliverable: Show an agent successfully adjusting to a new, unforeseen task or a significant change in its operational environment.

Milestone 4: Self-Optimization

Advance the development of algorithms that allow agents to optimize their performance based on past experiences.
Deliverable: Show an agent improving its efficiency or task completion rate over time through self-optimization.

Milestone 5: Inter-Agent Learning

Create mechanisms for agents to learn from each other’s experiences or solutions.
Deliverable: Demonstrate agents sharing knowledge or strategies to improve collective task performance.

Some Ideas:

Submit ideas here ...

Rather than post here you should update it and link it. @Kevin-Rush

I'll assist on managing this too.

Thanks guys. Abide by Wiki and documentation best practices, avoid sprawl, and keep it clean. Cheers.


[web]: pixelated images & icons on Firefox on Windows

On Firefox on Windows 10, I see that images (NetworkImage, AssetImage, even Icon) appear pixelated. For example, an avatar displayed via:
      CircleAvatar(
        backgroundColor: Colors.white,
        backgroundImage: NetworkImage(url),
        radius: radius,
      )

looks pixelated on Firefox on Windows 10:

But it looks fine on Chrome on Windows 10, or any browser (Chrome / Firefox) on MacOS:

For reference, the original image here is:

Maybe it's related to https://github.com/flutter/flutter/issues/41328 but that ticket seems to discuss device dependent properties, while here it depends on browser & OS.
flutter doctor -v
[√] Flutter (Channel dev, v1.10.12, on Microsoft Windows [Version 10.0.18362.388], locale en-GB)
    • Flutter version 1.10.12 at C:\flutter
    • Framework revision 0e605cc4dd (35 hours ago), 2019-10-04 07:52:15 -0700
    • Engine revision 759d3dba25
    • Dart version 2.6.0 (build 2.6.0-dev.5.0 7a7dcd1ed9)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at C:\Users\tibo\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 40.0.2
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] Connected device (2 available)
    • Chrome          • chrome          • web-javascript • Google Chrome 77.0.3865.90
    • Headless Server • headless-server • web-javascript • Flutter Tools

• No issues found!

@deckerst , closing this issue since we now use  tags to render and don't rely on canvas. Please reopen if you still see issue.


remove match-multiple directly when press backspace

remove match-multiple directly when press backspace on searchInput

This needs test fixed.

Closing due to age, lack of tests and because I think requiring double backspace helps guard against accidental deletion.


MGMT-9826: AgentMachine controller watches Agents

Configure the AgentMachine controller to watch Agents (this required
updating controller-runtime, which required moving to Go 1.17, and to
make it easier changed the dependency on assisted-service to be /api
only).
Remove unnecessary requeues when waiting for Agent updates.
Remove unnecessary requeues for error cases (unrelated to watching
Agents).

/retest

/lgtm

/test e2e-capi-provider-hypershift

/hold
Seems like e2e job doesn't pass

/lgtm

/unhold


Improve transition when removing login sessions

Resolves https://github.com/silverstripe/silverstripe-session-manager/issues/60

Yeah, I think it does cheers


ERROR: Cannot install opencv-python==<IP_ADDRESS> and opencv_python==<IP_ADDRESS> because these package versions have conflicting dependencies.

There is two versions of opencv-python, is this right?

Thanks, we have revised the opencv-python and opencv-python-headless versions to <IP_ADDRESS>.


QPPA-4750: Updated node packages, gitignore file

Updated react-scripts, eslint, babel, and jest node packages to latest versions as a security upgrade.
Also added .eslintcache to gitignore until the file is removed in a later react-scripts version.

NOTE: the /image folder references were also removed since the folder does not exist and was causing compile errors with newer node packages. The change to tsconfig is added automatically by<EMAIL_ADDRESS>which fixes an issue caused in v4.0.0

NOTE: the /image folder references were also removed since the folder does not exist and was causing compile errors with newer node packages. The change to tsconfig is added automatically by<EMAIL_ADDRESS>which fixes an issue caused in v4.0.0

The .snyk file needs to be updated to remove the 'ignore' for the react-scripts. Thanks.

Done, thanks for the reminder

The .snyk file needs to be updated to remove the 'ignore' for the react-scripts. Thanks.

Done, thanks for the reminder


fix NPE on streaming GeneratedReference

Spot this while working on delta based add column - on the first iteration I used Reference which could be a GeneratedReference and since ctor arg is Nullable (and in this case Symbol was redundant to send, raw String is enough) I passed null and faced NPE.
Not sure about 4.8 - we don't pass expression Symbol to ctor in this branch but looks like it can be null anyway if not assigned via setter - also worth backporting? I think so as we have null check on expression getter but it's a valid scenario to send only raw string and then receiving side can resolve Symbol from a raw string when needed.

Only if there's a way for a user to trigger this error - in which case it would also need a changes entry.

I don't think we streamed GeneratedReference before - otherwise would have failed with NPE which I faced when streamed locally in another PR. So no bp-s and no change then.

@Mergifyio rebase


Typescript type definition for globalDragModeEnabled()

Hi,
this should be modified to globalDragModeEnabled():
https://github.com/geoman-io/leaflet-geoman/blob/dbf6c6cd5bdff79cf312fb324da8d979c38deb88/leaflet-geoman.d.ts#L781
Thank you for your work,
Jumpy

Thanks for reporting, will be fixed with #966

Fixed with Release 2.11.3


reorderBundle w/ conditional references

I'm noticing an unexpected behavior of reorderBundle where it omits all entries not containing fullUrl. This behavior is clear from the implementation but the API doesn't make it obvious that this is what will happen. The documentation also doesn't make this super obvious, despite the alert being co-located with the fullUrl reference https://arc.net/l/quote/tcogmrdc. I'm leveraging conditional references and admittedly thought reorderBundle would resolve references in a similar way to fullUrl.
https://www.medplum.com/docs/migration/convert-to-fhir#linking-data-using-conditional-references
const bundle = reorderBundle({
  resourceType: 'Bundle',
  type: 'transaction',
  entry: [
    {
      resource: {
        resourceType: 'Task',
        code: {
          coding: [{ system: 'example.com/task-type', code: 'thing1' }],
        },
        status: 'ready',
        intent: 'order',
        for: {
          reference: 'Patient?identifier=example.com/patient|abc123',
        },
      },
      request: {
        method: 'POST',
        url: 'Patient',
        ifNoneExist: 'for:Patient.identifier=example.com/patient|abc123&code=example.com/task-type|thing1',
      },
    },
    {
      resource: {
        resourceType: 'Patient',
        identifier: [{ system: 'example.com/patient', value: 'abc123' }],
      },
      request: {
        method: 'POST',
        url: 'Patient',
        ifNoneExist: 'identifier?example.com/patient|abc123',
      },
    },
  ],
});
console.log(bundle); // { resourceType: 'Bundle', type: 'transaction', entry: [] }

Seems like it would be possible to support this behavior in reorderBundle w/ a more complex (cognitively & computationally) implementation. Curious - is there a desire or plans to support this?

@mattwiller  - can you confirm that this is fixed with the new batch system?

@dillonstreator Actually, that alert is no longer accurate after some recent changes to the Batch handling code; I went ahead and updated the docs to reflect this.  Additionally, there's an important difference: the urn:uuid: syntax is used to refer only to resources within the Bundle, and conditional references can only refer to already-created resources outside the Bundle.  In any case, it should no longer be necessary to call reorderBundle() before sending the JSON to the Batch API, since the new implementation handles reordering internally.


Check for Plugins

Would be nice if this SBT plugin would not only check dependencies, but also plugins itself. The VersionEye Maven Plugin is handling plugins as regular dependencies and is using for them a different scope with the name "plugin". /cc @mp911de

see plugin sbt-assembly.


README Documentation

Update Readmes in top level directory, KukaApp, and web directory.

Some prerequisites are required for this project that aren't pointed out yet:

NPM

We also need to run yarn global add firebase/npm install -g firebase to get the later commands to work


Feature: display time remaining

I want to have something like this:
an indicator that shows how much time is left in the current slide

is there a simple solution where I dont have to reinvent the wheel with my own hover triggers, then fix mobile  etc?
the remaining time would somehow have to be exposed
those indicators look really neat, really shiny and "fancy"

Have a look at this on Codepen as with a bit of tinkering with the CSS you can easily get it to look as per your example above: [https://codepen.io/rotemyagel/pen/oNXbEEz]
However, I personally wouldn't invest too much time in 'Slick' as it seems to have now been abounded for the last couple of years and as such, the issues are really stacking-up.
I now use and can fully recommend both 'Flickity' and 'Swiper'.
Mike

Have a look at this on Codepen as with a bit of tinkering with the CSS you can easily get it to look as per your example above: https://codepen.io/rotemyagel/pen/oNXbEEz
However, I personally wouldn't invest too much time in 'Slick' as it seems to have now been abandoned for the last couple of years and as such, the issues are really stacking-up.
I now use and can fully recommend both 'Flickity' and 'Swiper'.
Mike

thanks @MikeyJN I highly appreciate, will look into flickity and swiper.
The codepen: problem is it strips other important functionality. E.g. on hover, it doesnt stop


Backup-dbadatabase - SQL Server canot write to the location

Before submitting a bug report:

[x] Ensure you are able to reproduce it on the latest released version (we release often)
[x] Verified this bug is not already reported in an issue
[x] Verified errors are not related to permissions
[x] Can reproduce in a clean PowerShell session (clean = powershell -NoProfile)

Steps to Reproduce
<#
    $sqlcred = Get-Credential ipc_sysacc
    Backup-DbaDatabase -verbose -SqlInstance 'localhost,6201' -SqlCredential $sqlcred -Databases SPICE_4_STG -BackupFileName "test" -BackupDirectory D:\applications\prg\Backup\sql -Type Full -CompressBackup

VERBOSE: [Backup-DbaDatabase][14:42:15] Connecting to localhost,6201
VERBOSE: [Backup-DbaDatabase][14:42:16] 1 database to backup
VERBOSE: [Backup-DbaDatabase][14:42:16] Backup database [SPICE_4_STG]
VERBOSE: [Backup-DbaDatabase][14:42:16] Compression enabled
VERBOSE: [Backup-DbaDatabase][14:42:16] Creating full backup
VERBOSE: [Backup-DbaDatabase][14:42:16] Building file name
VERBOSE: [Backup-DbaDatabase][14:42:16] Building backup path
VERBOSE: [Test-DbaSqlPath][14:42:16] Connecting to XXXX,6201.
WARNING: [Backup-DbaDatabase][14:42:16] SQL Server cannot write to the location D:\applications\prg\Backup\sql
#>

Expected Behavior
In earlier versions of dbtools it works without this message. I have also verified that the user under the SQL Server runs have full access to the directory and also the user which starts the powershell command.
I get not output from $error[0] after running the command. So it is not clear what really does not work.
Environmental data

PowerShell:

PSVersion                      5.0.10586.117
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.10586.117
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

dbtools:

0.9.335    dbatools

SQL Server:
Microsoft SQL Server 2012 (SP4-GDR) (KB4057116) - 11.0.7462.6 (X64)
Jan  5 2018 22:11:56
Copyright (c) Microsoft Corporation
Standard Edition (64-bit) on Windows NT 6.1  (Build 7601: Service Pack 1) (Hypervisor)

I'm reviewing the backup function to see if there is something that needs fixing but in the meantime can you do
test-dbasqlpath -sqlinstance 'localhost,6201' -path D:\applications\prg\Backup\sql
?
if you don't pass -IgnoreFileChecks to Backup-DbaDatabase, that's expected to stop if the former command doesn't return "true".

I have run the test-dbasqlpath command: It returns True.

Hi Waffel, can you tru to test fix @niphlod just published https://github.com/niphlod/dbatools/tree/fix/Backup-DbaDatabase
Thank you!

I have tested it. The same result.
WARNING: [Backup-DbaDatabase][10:07:40] SQL Server cannot write to the location D:\applications\prg\Backup\sql

Can you attach a verbose run of it?
Il gio 24 mag 2018, 10:09 Waffel<EMAIL_ADDRESS>ha scritto:

I have tested it. The same result.
WARNING: [Backup-DbaDatabase][10:07:40] SQL Server cannot write to the
location D:\applications\prg\Backup\sql
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/3629#issuecomment-391627480,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHdB_laN1tTCVyjcgnsIEdTSOyKXhzQks5t1mq2gaJpZM4UKY5n
.

@waffel You can actually take a backup to that same location via a query in SSMS on this particular instance?

If you are able to create a support package will give us more details on what is going on in the module for your environment. You can use: New-DbatoolsSupportPackage to create a zip file and attach.

I can create the backup via SQL Management Studio in the same location without problems (same user and so on).
Here are the verbose output (I have used another DB on the same instance):
VERBOSE: [Backup-DbaDatabase][13:05:49] Connecting to localhost,6201
VERBOSE: [Backup-DbaDatabase][13:05:50] 1 database to backup
VERBOSE: [Backup-DbaDatabase][13:05:50] Backup database [SPICE_4_001]
VERBOSE: [Backup-DbaDatabase][13:05:50] Compression enabled
VERBOSE: [Backup-DbaDatabase][13:05:50] Creating full backup
VERBOSE: [Backup-DbaDatabase][13:05:50] Building file name
VERBOSE: [Backup-DbaDatabase][13:05:50] Building backup path
VERBOSE: [Test-DbaSqlPath][13:05:50] Connecting to DEFTHW99LPYSRV,6201.
WARNING: [Backup-DbaDatabase][13:05:51] SQL Server cannot write to the location D:\applications\prg\Backup\sql

and attached the support zip (be aware that the dbatools package is already the patched version from @niphlod )
dbatools_support_pack_2018_05_29-13_07_16.zip

What do you get if you add -IgnoreFileChecks @waffel ? If it succeeds, that'd suggest our logic is bad for the check.

hehe cool @potatoqualitee  ... you have found it. With this switch it works fine:
$sqlcred = Get-Credential ipc_sysacc
Backup-DbaDatabase -verbose -IgnoreFileChecks -SqlInstance 'localhost,6201' -SqlCredential $sqlcred -Databases SPICE_4_001 -BackupFileName "test" -BackupDirectory D:\applications\prg\Backup\sql -Type Full -CompressBackup

VERBOSE: [Backup-DbaDatabase][13:36:29] Connecting to localhost,6201
VERBOSE: [Backup-DbaDatabase][13:36:29] 1 database to backup
VERBOSE: [Backup-DbaDatabase][13:36:29] Backup database [SPICE_4_001]
VERBOSE: [Backup-DbaDatabase][13:36:29] Compression enabled
VERBOSE: [Backup-DbaDatabase][13:36:29] Creating full backup
VERBOSE: [Backup-DbaDatabase][13:36:29] Building file name
VERBOSE: [Backup-DbaDatabase][13:36:29] Building backup path
VERBOSE: [Backup-DbaDatabase][13:36:29] Devices added
VERBOSE: Performing the operation "Backing up SPICE_4_001 to D:\applications\prg\Backup\sql\test" on target "localhost,6201".
VERBOSE: [Get-DbaBackupHistory][13:36:31] Connecting to DEFTHW99LPYSRV,6201.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Connecting to DEFTHW99LPYSRV,6201.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Processing SPICE_4_001
VERBOSE: [Get-DbaBackupHistory][13:36:31] Executing sql query.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Processing as grouped output.
VERBOSE: [Get-DbaBackupHistory][13:36:31] 1 result-groups found.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Connecting to DEFTHW99LPYSRV,6201.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Processing SPICE_4_001
VERBOSE: [Get-DbaBackupHistory][13:36:31] Executing sql query.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Processing as grouped output.
VERBOSE: [Get-DbaBackupHistory][13:36:31] 1 result-groups found.
VERBOSE: [Get-DbaBackupHistory][13:36:31] No Diff found
VERBOSE: [Get-DbaBackupHistory][13:36:31] Connecting to DEFTHW99LPYSRV,6201.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Executing sql query.
VERBOSE: [Get-DbaBackupHistory][13:36:31] Processing as grouped output.
VERBOSE: [Get-DbaBackupHistory][13:36:31] 1 result-groups found.

SqlInstance    Database    Type TotalSize DeviceType Start                   Duration End
-----------    --------    ---- --------- ---------- -----                   -------- ---
DEFTHW99LPYSRV SPICE_4_001 Full 71.22 MB  Disk       2018-05-29 13:36:30.000 00:00:01 2018-05-29 13:36:31.000

it's soo impossible looking at the code to see why test-dbasqlpath would return True and get backup-dbadatabase to fail right where it it returns False it stops.

to "close" this, I'd just go on and modify the "SQL Server cannot write to the location"  with a more detailed message, such as "SQL Server cannot check if location is available, you can try with -IgnoreFileChecks"

This sounds perfect.


Some todos

Turns out the class won't be over just yet, so yay! This means I can write some more stuff.
So, this library's going pretty fine, but, there's some extra stuff I've identified that would be very useful to have:

[x] Lexer needs additional delimiters, e.g. if we wanna pass {}().,:; to it
[ ] Vec2 and Vec4

I've considered using GLM for this stuff, and it makes sense; I've used it before and it saves a lot of time. But it's not stylistically consistent with the rest of my projects. Casing, the way things are done etc. But it's okay, I can write my own stuff based on GLM and other open-source code I can refer to.

[ ] Quat like in GLM
[ ] Mat4 like in GLM
[ ] Spline interpolation utilities (Lerp.hpp is the most basic of the basics)
[ ] Keyframe animation utilities (refer to the point above)
[ ] Maybe use fmtlib for adm::format
[ ] Maybe a String class with easy string manipulation functions that inherits from std::string, or utility functions that work on std::string
[ ] File class similar to the one in .NET
[ ] DateTime utilities
[x] Plane like in Sledge Editor
[x] Polygon like in Sledge Editor, to be used in my TrenchBroom map parser
[ ] Frustum class, to be used in culling and other things
[x] Rename some things to sound less stupid
[x] Additional usings for stylistic consistency
[ ] Multi-threading utilities
[ ] BitBuffer to write bits and bytes into
[x] DLL loading (Library class), could be a separate library

Once these are done, then I pretty much don't need anything else, as far as basic utilities are concerned. Eventually, maybe, down the road, there could be a quadtree & octree in here, and maybe some sorting algorithms? But in general, this is pretty much it.
For things like model loading, texture loading, I can generally just use other libraries.

We got Plane now.

Polygon is underway.

Think I'll heavily reference JoltPhysics's maths library for Vec2, Vec4, Quat and Mat44.

More stuff for future reference, could probably carry over into a new issue:
Triangular barycentric coordinates:
https://codeplea.com/triangular-interpolation
Tetrahedral barycentric coordinates:
https://stackoverflow.com/questions/38545520/barycentric-coordinates-of-a-tetrahedron
Can derive tetrahedral interpolation from this, which might be useful for things like these:
https://www.gdcvault.com/play/1015312/Light-Probe-Interpolation-Using-Tetrahedral
Splines and trilinear interpolation:
https://codeplea.com/introduction-to-splines
http://paulbourke.net/miscellaneous/interpolation/
Investigate doing mesh tetrahedralisation (from now on: tetrameshing) with V-HACD and Qhull.
https://github.com/kmammou/v-hacd
https://github.com/qhull/qhull
V-HACD will approximately generate a collection of convex hulls and Qhull can then generate tetrahedral meshes out of those. Should work, right?

V-HACD supports non-watertight meshes BTW. This is pretty nice.
Also, still gotta cover Delaunay triangulation, tetrahedralisation and Voronoi fracturing.

Could refer to this for Delaunay triangulation in 3D:
https://github.com/obviousjim/ofxDelaunay
This one's constrained and in 2D:
https://github.com/artem-ogre/CDT
Voronoi in 3D:
https://github.com/chr1shr/voro
This one actually uses Voro++:
https://github.com/patriciogonzalezvivo/ofxVoro
Voronoi in 2D:
https://github.com/madc/ofxVoronoi
https://github.com/JCash/voronoi
I think that'll cover it all.

Geometry processing (convex decomposition, convex hull generation, triangulation, tetrameshing, Voronoi cells) will be a separate issue in a couple of months, but it's satisfied for now in terms of prerequisites.

We got the Library class now.

Added AABB and Octree (with a template N-dimensional tree of course), and a couple more things to the to-do list.

Added Vec2, Vec4 and DateTime.

Added a Mat4 draft, can be ticked off at this point. Mat34 will probably not be a thing, and frankly I don't know about Mat3 either, it looks like we could just use Mat4 for everything and enjoy the recent SIMD goodies.


No current tariff changes event

I tried to build an automation on the current tariff (put plug on below x cents and turn off above x cents).
But there is only current-low-tariff change, normal and high.
Possible to add the current one?😊
I know there is a sensor with the current tariff, but I cannot use that in an automation sadly.

Not sure if I understand the issue.
You have the Zonneplan current electricity tariff but it doesn't trigger change events?
Are you sure you use the correct entity and state in the automation? Can you post a screenshot of the history of the sensor?

Leave it...
I used the Connect device instead of the usage :/ ....

instead of:


Tainted Canvas

I'm getting the following error: 'Failed to execute 'toDataURL' on 'HTMLCanvasElement': Tainted canvases may not be exported at Image.r.onload' when trying to download a chart as a PNG. As a SVG it works.
It happens both on local and on hosting.
I'm using Vue 2.5.17 and the Vue version of apex charts.
- What is the behavior you expect?
Download the chart as a PNG image
- What is happening instead?
The download isn't working and I'm getting an error message
- What error message are you getting?
Failed to execute 'toDataURL' on 'HTMLCanvasElement': Tainted canvases may not be exported at Image.r.onload

Thanks for reporting, fixed it.

Hello,
For me it's not fixed, I'm getting the same message with fresh 3.5.1 download and Chromium 72


[dnm] Feature/message animation

TODO:

[ ] remove animation on existing messages

@Kiryous aside from the conflict is this gtg?

friendly ping on this @Kiryous

update this PR finally, could you please have a look? @fsher


Comparison operators: needed under actants for Values such as "more than 6"?

In the original table for modality and logic (https://docs.google.com/spreadsheets/d/1a6Wli7bLQ-UkGzhiNWM4Lo_6gC6CWDdMVldhCZfh2rk/edit#gid=0), we had comparison operators under logical operators, and thought that they are not needed for actants.
However, they might be something else than logical operators, and be needed under actants. E.g., it seems that to code "more than 6", we need V ">6", i.e. V "6" with comparison operator ">" (now, in gsheets, we save it as V ">6").
Or else, should in this case the operator be part of the value itself, i.e. ">6" as in GSheets? TBD before implementation!
After discussion, revise this thread: https://dissinet.slack.com/archives/C01K0EA45GC/p1632485654000100?thread_ts=1632208832.002300&cid=C01K0EA45GC.
EDIT: I now tend to think that if "more than six" is the value itself, it should remain in the value and the V should be literally ">6", same as in the data model, i.e. no change to the data model needed. But I need you @adammertel to think about this.
If we decided to implement the change, we would add a new column "comparison operators" in the table (link above) and do the following:

separate comparison operators in the data model from logical operators per se (and, or, xor);
separate them also in the modal windows into a separate select box;
implement them where they belong according to the table;
remove them where they do not belong according to the table.

I understand the problem. I think it should be decided as soon as possible as this involves the data model.
If you think that this will be needed in the future, lets do it now.

I now think it should be part of the value, i.e. ">6" is actually the value, the ">" is part of the V, not the relation of the V to something else. I.e., introducing a convention of having operators within values where they are part of the value itself. It is arguably something different than a comparison operator. We will confirm when we meet.

Closing this - confirmed in a call with Adam that we will store "more than six" in the value itself, ">6", because there is no point of comparison, the "more" is rather part of the value itself. We will then need to think about it for numerical values, and work with this type of free-text ">6" in data projections and probably also in the more semanticized DDB2 version, i.e. decompose (semanticize) this convention.