x/build/cmd/cl: build broken

x/build/godash was removed recently, but x/build/cmd/cl depended on that package, and is now broken. The right answer is to rewrite it using godata or (maybe) remove it, if no one's using that tool anymore.
https://travis-ci.org/kevinburke/build/jobs/258917130
$ go install ./cmd/cl/...
cmd/cl/cl.go:95:2: cannot find package "golang.org/x/build/godash" in any of:
	/Users/kevin/src/golang.org/x/build/vendor/golang.org/x/build/godash (vendor tree)
	/Users/kevin/go/src/golang.org/x/build/godash (from $GOROOT)
	/Users/kevin/src/golang.org/x/build/godash (from $GOPATH)

@andybons, I think @rsc still uses this tool.
Probably what you should do is take the devapp logic that's converting *maintner.Corpus -> your data structure for the devapp HTML template and instead move that into a Go package that x/build/cmd/cl can use to render the same data in text format from a CLI.
Or, better: make cmd/cl just be an HTTP client that hits https://dev.golang.org/foo.txt and returns the same data in text format. Then cmd/cl won't have the maintner start-up latency.

I'd like to take a shot at this. Have it working partially for now. Will keep you guys updated.

@bradfitz I'm on the path of importing the code from goodish, just about what's required. Am I on the right path?

Sorry - by goodish do you mean godash?
On Sun, Aug 6, 2017 at 10:57 Jude Pereira<EMAIL_ADDRESS>wrote:

@bradfitz https://github.com/bradfitz I'm on the path of importing the
code from goodish, just about what's required. Am I on the right path?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/21216#issuecomment-320522071, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOSI0zbA0qvPYQekaJONRnkF2TCsMlrks5sVf6ZgaJpZM4Ona4Z
.

--
--
Kevin Burke
925.271.7005 | kev.inburke.com

Yes, godash.

We’ve removed godash — https://go-review.googlesource.com/c/50654

Yes, I am aware. I thought of importing the parsing methods from it.
A cleaner way would be to just print the ChangeInfo object from the gerrit package, without doing some extra stuff that cmd/cl wants to do.

On 07-Aug-2017, at 04:06, Andrew Bonventre<EMAIL_ADDRESS>wrote:
We’ve removed godash — https://go-review.googlesource.com/c/50654
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

My first comment above contains two possible designs. I think the latter one is better.
Neither of them involve making Gerrit or GitHub API calls.

What would the output be like then? Also, what's the endpoint for change lists? The gerrit code review endpoint? It wouldn't be console friendly.
Not quite sure what use it'd be of then.

On 07-Aug-2017, at 10:03, Brad Fitzpatrick<EMAIL_ADDRESS>wrote:
My first comment above contains two possible designs. I think the latter one is better.
Neither of them involve making Gerrit or GitHub API calls.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

The intent of this bug is to change the cl tool's backend only (to maintner, probably via dev.golang.org or gRPC calls to https://maintner.golang.org), and not change its CLI UI or output at all.

Got it. Let me take a look at maintner. Will let you know how it goes.
Thanks!
On 7 August 2017 at 11:14:28, Brad Fitzpatrick<EMAIL_ADDRESS>wrote:
The intent of this bug is to change the cl tool's backend only (to
maintner, probably via dev.golang.org or gRPC calls to
https://maintner.golang.org), and not change its CLI UI or output at all.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/21216#issuecomment-320573719, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAtmTwm4I5rkD65xYzqzigKkFQ21V4l5ks5sVqQ8gaJpZM4Ona4Z
.

@bradfitz I got around to using the gRPC calls to maintner. However, most of the relevant information is not sent back as part of the api spec. Should I expose the stuff that's required for build/cmd/cl in build/maintner/maintnerd?
I see that all the data that I required for cl is already present and available in maintnerd's api.go, but just not being sent down.

Any updates here? I'm really looking forward to fixing this issue.

hey Jude - really sorry about this. Instead of reverse engineering the GRPC calls, it's probably better to expose a new API on the Corpus with the data that you need, see the ForeachXX API's that already exist. The data probably already exists in the protobuf files on disk, we just need to expose it properly.

Hey Kevin,
Do you mean this here?
As of what I understand, using the Corpus would make cmd/cl take 500 MB worth of memory (described here)
I think the gRPC way is slightly more efficient here - it would get fully processed data (and not use a ton of memory).
Or am I missing something?

What do you guys think? Should I go the gRPC way or the Corpus way?

@judepereira sorry about this - for the moment we'd rather focus on downloading the Corpus and adding API's on it, I think. The good news is once you have it downloaded you don't need to do it again and you can run other tasks off of that.

Thanks @kevinburke. I'll start work on it immediately. Should have something working early next week.

@kevinburke What's the importance of the reviewer column? Is it still being used?

Reviewer column in Gerrit is similar to the "Assign X" field in Github - it designates the person responsible for giving a Code-Review +2 to the PR.
If there's any way to break this up into chunks - I'm happy to review smaller parts to avoid any confusion.

Thanks, got it. Going to implement something similar to what it was in godash earlier.

I've hit a roadblock, which I've been trying to find a way around for the past three days. The reviewer information in the mutation log is of the form "Brad Fitzpatrick 5065@62eb7196-b449-3ce5-99f1-c037f21e1705".
There seems to be no way to resolve a Gerrit UID (5065 in the example) to a person's real email address.
The best that I can do is to link it via a user's name (using a name to email map - where I get the actual email via the author line, and then use it to resolve a reviewer's email address).
Is there a better way to do this? Or am I missing something obvious here? None of the reviewer lines contain the reviewer's actual email address.
This is the only thing that's remaining. Everything else works well with the Corpus.

I can use this API to download and cache the Gerrit account IDs to email addresses, and call it whenever I find a new account ID which is not mapped.
What do you think?

Strangely, I'm not able to use that API - although I see that I'm a part of the Registered Users group on go-review.googlesource.com, when I hit the list groups API (via the gerrit.NewClient()), I don't see those groups.
If I hit the GetAccountInfo API with self, it shows me my account info (account ID, email address and name).
Is the Gerrit list groups API broken?

So it’s not broken, but limited. There is no programmatic way to get access to Registered Users via the /groups/ endpoint.
You can try using /accounts/ search (possibly with is:active to trim it down) to get all users.

Thanks a lot @andybons! That worked very well.

As a pre-requisite for this issue to be fixed, I've added QueryAccounts in the the gerrit client. Please review when feasible - https://go-review.googlesource.com/c/build/+/61970
As this is ready, I should be able to get cmd/cl to be working as it was before :)

Okay, I've managed to get this working. Here's how I've solved the resolving of Gerrit email addresses to actual email addresses:

On the very first run, fetch all the accounts information mapping and serialise it to a file - stored alongside golang-maintner (it's called golang-gerrit/accounts.bin)
Now, for subsequent runs, deserialise this map, and use it
Let's say that a new user comes in - this entry will not be present in the Gerrit map. Make a new call to fetch all the accounts info once again, and serialise the mapping back on disk
If for whatever reason a Gerrit email address could not be mapped to an actual email address, then a default entry will be set, which is the same as the Gerrit email address (so far, this has happened for 10 open CLs)

Does that sound good? If yes, I'll have a commit available for review this week.

On the very first run, fetch all the accounts information mapping and serialise it to a file - stored alongside golang-maintner (it's called golang-gerrit/accounts.bin)

I would call it golang-build-cmd-cl or something maybe less specific but enough that we know where the binary that's writing to it is located.
I have no other concerns but @kevinburke what do you think?

(I'm on vacation and don't have very strong feelings - I can look into this
more next Monday if you like)
On Mon, Sep 18, 2017 at 21:37 Andrew Bonventre<EMAIL_ADDRESS>wrote:

On the very first run, fetch all the accounts information mapping
and serialise it to a file - stored alongside golang-maintner (it's called
golang-gerrit/accounts.bin)

I would call it golang-build-cmd-cl or something maybe less specific but
enough that we know where the binary that's writing to it is located.
I have no other concerns but @kevinburke https://github.com/kevinburke
what do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/21216#issuecomment-330347874, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOSI5oFkJRuC40h-LAQsvi-4WlSf7Eiks5sjtRvgaJpZM4Ona4Z
.

--
--
Kevin Burke
925.271.7005 | kev.inburke.com

I've got it working, as it was before, except that the following is a todo for me:

Output as JSON
Support for the do-not-review flag
Support for postpone to next Go release

Please review when feasible. This is my first major commit, so I expect a lot of updates to be made. I'm open to all of them. Thanks.


Allow setting of graph name via graphviz.name attribute

graphp/graphviz#3

Thanks for filing this PR! :+1:
Does it make sense to add this to the documentation so people learn about this feature? Also, in the linked ticket you stated that you found some documentation why name may be preferred over id, can you elaborate on this? Again, thanks!

Will do for the divorce, when I get into work tomorrow.
With regards to name vs I'd, I saw this in a pdf man page for circo, but
each pdf for each engine tends to cover all the engines, and specifically
refers to that field as name. Unfortunately, the GraphViz site was having
issues today, but maybe I'll be able to reference doco on their GitHub
account.
On Thu., 6 Apr. 2017, 9:14 pm Christian Lück<EMAIL_ADDRESS>wrote:

Thanks for filing this PR! 👍
Does it make sense to add this to the documentation so people learn about
this feature? Also, in the linked ticket you stated that you found some
documentation why name may be preferred over id, can you elaborate on
this? Again, thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/graphp/graphviz/pull/20#issuecomment-292142472, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxzB2bmhCnscTgG2B_TWy7tV9LtKeDkks5rtMkLgaJpZM4M0-Gw
.

With regards to the usage of name vs id, the man pages use name and the abstract grammar uses ID.
If the Fhaculty\Graph\Graph accepted an id in the constructor or had it changeable as a property, then I would have used the constructor or methods to set the id and be consistent with Vertex.
But since those are not there, and the GraphViz class is something else, going with what is in the manual pages for the installed product seemed the next best thing, for consistency.
However, if you'd really like to see it as id, I'd be happy to make that change. I'm just glad there's any easy to use API out there, given that PHP 7 bindings have been removed from GraphViz for the moment.

@rhelms Again thank you for filing this PR and for your patience!
I agree that this feature makes perfect sense and have just merged a slightly modified version with #28 (with proper attribution of course) :shipit: :tada:
Thanks for your investigation! Interestingly, https://www.graphviz.org/doc/info/lang.html and also https://graphviz.readthedocs.io/en/stable/manual.html#basic-usage suggest that the root graph name is actually optional. Accordingly, I've changed this to omit the default "G" root graph name by default and otherwise added documentation for the new graphviz.name attribute as introduced in this PR.
This means that this now works as expected out of the box and there should be no need to change this for most common use cases. Again, thank you! :+1:


create(go): add -s -w ldflags to reduce the artifact size

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

It is consistent with the go formulae practices we are doing in the homebrew-core
This is the article reference for shrinking the binary (thanks @bayandin)

Review period will end on 2021-08-24 at 00:00:00 UTC.

Review period ended.


rancher/agent accepts anything given in CATTLE_AGENT_IP

Rancher versions:
rancher/agent: v1.2.2
Steps to Reproduce:

Set up rancher/server
Add Custom Host with a non-IP value in CATTLE_AGENT_IP:

sudo docker run -e CATTLE_AGENT_IP="asfsdfdsfdsfdsfdsfdsfsdf"  --rm --privileged -v /var/run/docker.sock:/var/run/docker.sock -v /var/lib/rancher:/var/lib/rancher rancher/agent:v1.2.2 http://x:8080/v1/scripts/x:x
Results:
Agent runs successfully. Being added to cluster with no IP showing in Host view and will never be Active.
This can be fixed by:

Checking valid IP in run.sh (rancher/agent)
Checking valid IP when host is being registered (rancher/cattle)

Available with rancher/agent:v1.6.8-rc1

Tested with rancher/agent:v1.2.8-rc1.
Able to validate that we allow only for valid ip format ( without http:// or https://) to be set for CATTLE_AGENT_IP when running registration command for Custom Hosts.


Create Secure WebView message passing API

User Story
As an extension developer, I want to be able to interact with my webviews via a message passing API so that I can communicate between them and my extension backend effectively without having to create commands and things on the frontend that don't belong on the frontend.
As a user, I want to be able to enter secure input like passwords and such into forms that won't be phished by other parties.
Description
We should implement a message passing API with webviews. Iframes have a postMessage api that we can layer over and inject into each iframe maybe somewhat similarly to how VSCode does it.
We should pick up those messages in web-view.component.tsx, I suppose, then route them through the web view provider service straight to the appropriate web view provider. Or something along those lines. That way, the web view provider that provided the web view definition is the only one who gets to see those messages. The web view provider should also be able to send messages to the iframe as well.
This way, people can turn off allowSameOrigin on their webviews to prevent other webviews from phishing their user input and then send messages to the web view provider.
We should also consider how to make sure we are properly safeguarding these webview messages to make sure only the web view provider can interact with the webview iframes. This may require creating a cryptographically secure token that is used to validate the communication.

I believe #1185 and #1305 would complete this functionality


chef 14 compatibility (4.1.1) not published to chef supermarket

Thanks for the work already done by https://github.com/chef-cookbooks/bluepill/issues/36
I would really appreciate releasing version 4.1.1 to make it available in chef supermarket - chef 14 compatibility is urgently needed.
Using git fetcher is not possible for production systems.
Thanks a lot

@dhohengassner Pushed to Supermarket

thanks - that was quick 😄


Migrate version to 15.1

It is important to think in the new versions of KC, lest start by the 15.1.1 Once we use a KC CLI instead the realm file we should go for the lastest version available

When migrating:
... (vert.x-eventloop-thread-9) Error while creating User: The arguments provided to the function are not compatible with the keycloak's standard
The user representation provided to Keycloak is incorrect

TODO: keep original exception


Add an onKeyDown prop to allow custom shortcuts

Added the ability to use 'k' and 't' to multiply value by 1000
and 'm' to multiply value by 1000000.
Limit the valid keys that can be used on number editor to prevent
entry of non-numeric characters.

I'm not sure if this is generally applicable but in my use case it is useful to be able to quickly enter large values using short cuts.
This change will also prevent backspace from going back through history when non in edit mode and the field was doing unexpected things when there were invalid characters in the field. To fix this I've limited the accepted keys in the field.

I like it, but what t stands for?

It stands for thousands (so you can use either 't' or 'k'). I have to admit the 't' comes from a legacy application I am replacing :-)

Instead of adding all this logic inside this component, maybe we can add a onKeyDown prop so it will allow developers the possibility to change the value based on the received key event?

Sure. That makes sense. I'll tweak the pull request along these lines.
I was thinking of splitting this in two. The k,t and m shortcuts would be removed and implemented using the new onkeydown prop and the key filter to prevent non-numeric characters would stay.
Does that make sense?

Actually . and , could be allowed as well. But otherwise it makes sense :)

Of course :-)
On Tue, 29 Sep 2015 at 23:13, Tommy<EMAIL_ADDRESS>wrote:

Actually . and , could be allowed as well. But otherwise it makes sense :)
—
Reply to this email directly or view it on GitHub
https://github.com/tleunen/react-number-editor/pull/16#issuecomment-144206239
.

--
Thanks,
Stuart
Tel: 07989 301439

I didn't add ',' to the list of allowed characters as the way strings being converted to numbers (using a Number cast) doesn't support commas either as thousand separators or decimal points. I think do that we should use something like numeral.js to format and parse the strings.

Thanks for the work @yoink00 !


Article: Migrate to latest version ( 2 ) of Uikit from prior versions

[x] I’ve read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I’ve read and agree to the Code of Conduct.
[x] I’ve searched for any related issues and avoided creating a duplicate issue.

What happened
Users of earlier versions of UI-Kit are wanting to adopt newer (or the latest) version of UI-Kit with a minimum of pain and effort. If no automated path exists, step-by-step guides would assist, either as community posts or docs in the GitHub repo.

Thanks @gordongrace definitely a missing piece in the puzzle!
I'm going to move this to the design system repository as that is where we currently have a backlog of article ideas.


Timeout too short

Thank you for this program.
I am very close to getting it working on my Raspberry Pi 4, but I believe the timeout is too short for my system. It takes longer than 18 seconds for nrsc5 to lock on to a signal and play audio. As a result, when I use yellowShoes, it generates an error log that looks like the beginning of an nrsc5 console output about to begin playing a stream. I think it needs just a few more seconds for audio to begin.
I believe the yellowShoes timeout is currently set at 18 seconds. When I play nrsc5 without yellowShoes on the same device, it takes 20+ seconds, but it does seem to work. I found someone else who identified this issue here, but I don't know how to increase the timeout so that I can use yellowShoes.
Thank you again.

Thank you for taking the time to file this report. I will look into making playerTimeout  a tunable setting. Until then, can you please try changing it by hand and see if it works better for you?
user@lego:~/git/yellowShoes$ sed -i 's/18000/30000/g' static/page.html

In the example above, I've changed it from 18000 to 30000 milliSeconds.
Please give this a try and report back!

Thank you! This worked wonderfully. It was just what I was looking for.
I wasn't savvy enough to find where in the program the timeout was coded.
It's up to you whether to make the timeout a tunable setting - I'm satisfied regardless.
Thanks again!

Glad to hear it worked for you!
I've gone ahead and made and published the changes in branch tunableTimeO:
https://github.com/evuraan/yellowShoes/tree/tunableTimeO
Changes:

playerTimeout raised to 30 seconds
set playerTimeout section in settings -

Please give it a try, and let me know how it goes. I am looking to merge to main in a few days of scream tests.

It worked for me - I replaced the html file as you described and I can now access the menu item where I can change the timeout. Thank you again!

Changes merged to main, published 1.09d-ui-1.5f release.


Added MCQ-Quiz-App as a template

I have added a new template(MCQ-Quiz-App) made with HTML, CSS, and JavaScript.

Add hacktoberfest-accepted label, If you like the template

Welcome Sir
Please some Screenshots of your App

Okay..

now I have added README.md file.

now I have added README.md file.

Hey, thanks for the contribution. But you haven't added screenshots of your template in the Readme.


Make all probability estimators have an explicit outcome space?

It's like 99% sure you'll agree with this @kahaaga , but shall we just go ahead and ensure all probabilities Estimators have a well defined outocme space? Which means, if input x is necessary, it must already be given at the point where the probability estimator is instantiated?
This has numerous benefits:

The estimators always have a well defined outcome_space which is by now an integral part of the API
Maximum entropy is simplified and doesn't need input x.

It's like 99% sure you'll agree with this @kahaaga , but shall we just go ahead and ensure all probabilities Estimators have a well defined outocme space? Which means, if input x is necessary, it must already be given at the point where the probability estimator is instantiated?

I think we agreed on this at some point, in some comment, in some PR, somewhere. I'm still totally on board with this.

Also outcome_space becomes a 1 argument function. ok. we do it then. first things first, #184 first.

Yes, let's do one thing at a time, so we don't end up with more conflicting branches.


Black screen on second scan

Tested on Android (multiple devices such as Samsung S10e)
Everything runs great when scanning. After I've processed other stuff and navigate back to the component that holds the scanner. Everything is initialized again (tested with console.log):
onLoaded(args) {
    this.camera = args.object;
    this.requestPermission();
  }

  requestPermission() {
    this.camera.requestCameraPermission();
  }

However the MLKitView is black and doesn't do anything.
When pressing back (phone button) and navigating again to the scanner, everything starts working fine again.
<ActionBar [title]="'QRSCANNERTITLE' | L">
    <NavigationButton [text]="'BACK'| L"></NavigationButton>
</ActionBar>

<GridLayout rows="*, auto" height="100%">
    <MLKitView height="100%" [torchOn]="torchOn" cameraPosition="back" detectionType="all" (detection)="onDetection($event)" (loaded)="onLoaded($event)">

    </MLKitView>
    <Button row="1" height="40" [text]="torchOn ? ('TURNOFFTORCH' | L) : ('TURNONTORCH' | L)" (tap)="toggleTorch($event)"></Button>
</GridLayout>

Setting an *ngIf="initialized" works when setting this to false after scanning and true onInit. So there is an easy workaround.
But this shouldn't be necessary.

@Noppey can you try with the latest version 1.0.7 and lmk if this is still an issue.

I have a similar issue in 2.0.0.
Using some ngIf may fix the problem, but it was not that easy to find the good timing: having this in the ngInit was not sufficient, so what I currently do is something like:
<GridLayout (loaded)="switch2 = true">
  <MLKitView *ngIf="switch2" [...] ></MLKitView>
</GridLayout>

After upgrading to 8.8.0-alpha the initialize ngIf workaround does not work anymore for me. What I did now is removed the *ngIf and just run this.myMLKitView.disposeNativeView();. This works well so far when testing.


minecraft-server: correct java options

Motivation for this change
It's incorrect to use a larger initial heap size (-Xms) than the max heap size (-Xmx). So copying the example options would give an error. These values are now set to the same size
Things done

Built on platform(s)

[ ] 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.

note that this doesn't change functionality, just an example that would go wrong if people copied it verbatim.

This is only a documentation change and thus can be merged easily


parse query (search params)

Any reason why the query part is not parsed as nicely and more natively available as JSON like the others, most notably the path part?  Why is it a separate project query-string-cli, at least not included into this one?

I followed the one tool for one job philosophy. But since whatwg-url exposes URLSearchParams anyways, we can expose query string parsing via a flag.

At least a flag: https://www.reddit.com/r/linuxmasterrace/comments/t4ucv1/comment/hz2hvze/

I found this utility because I did not have, JSON-ready, thus more efficient, access to all URL parts.

which one?

Let's add query/search params support via URLSearchParams.


Mini max sum

Changes Made In Pull Request

Added Solution for mini-max-sum hackerrank problem
Problem URL: https://www.hackerrank.com/challenges/mini-max-sum/problem

@lydamian please resolve conflict so that i can merge your PR.


Few Questions

Hello, great concept here, but I have a few questions....

If the install is done manually on a vps/cpanel account, can it be installed in a subfolder.... e.g. domain.com/hauk

How long is the map/route available for after it has expired? Can we set it for example to be deleted 24-48hrs after share link has expired?

If I take a 6hr flight, but share my location for 12hrs, will the map show the two countries once my location is back on when I land?

Am I right in saying, that without the android app, there is no way for my family to share their location in hauk?

If the install is done manually on a vps/cpanel account, can it be installed in a subfolder....

Yes

How long is the map/route available for after it has expired?

No longer available

If I take a 6hr flight, but share my location for 12hrs, will the map show the two countries once my location is back on when I land?

The browser shows the path from when you open it until you close it, there is no past.
Does GPS work ok in planes?

Am I right in saying, that without the android app, there is no way for my family to share their location in hauk?

Make apps for all platforms? :)

Thank you for that fast response!
About the planes, what I meant was the gps will be lost on a plane but if the share is still active, it should stay in the same position until i connect in another country right? and then it would be drawn on the map after connection.... all if the share is still valid and not expired.
About the expiry, it would be nice if the route was still available for a set period of time that could be changed per share.
One more thing, there should be a 'distance to me' showing if the follwer has their location services on.
I'm excited to see how this project develops but at the moment i think the use case is very small and too temporary for practicality.

If the install is done manually on a vps/cpanel account, can it be installed in a subfolder.... e.g. domain.com/hauk

Yes, this is supported. In this case, you set the public_url to the URL to the directory (e.g. https://domain.com/hauk/. The server URL you enter in the app is then also https://domain.com/hauk/.

How long is the map/route available for after it has expired? Can we set it for example to be deleted 24-48hrs after share link has expired?

The route is deleted immediately when the share expires or is stopped, and points are also pruned as you go to respect the max_cached_pts setting (though this can be changed to a higher value, to store more points before they're deleted).

If I take a 6hr flight, but share my location for 12hrs, will the map show the two countries once my location is back on when I land?

The server only keeps the max_cached_pts (default=3) number of points in memory at any time. The map itself will keep up to max_shown_pts (default=100) points visible as part of the trail behind the current position map marker, however. That said, as soon as you reload the page, the browser will only know the 3 latest points from the server, and as updates from the server arrives, it will keep adding mote points to the trail on the map until it reaches 100, at which point it starts trimming the tail.
If you don't post any location updates to Hauk while in the air, then it's possible that there could be a long leg between countries A and B visible on the map as soon as you post an update from B. If you wish to test this behavior, you can download a GPS emulator from the Play Store to trick your phone into moving large distances.

Am I right in saying, that without the android app, there is no way for my family to share their location in hauk?

Correct. Only an Android app is available for sharing locations, and without this, you can only view the map. An iOS app is not planned due to the cost of an Apple membership (#43) and other platforms are not supported because I don't have devices (Purism, Blackberry, Windows Phone etc.) to test with. A stable web client is also not possible currently because the technology required to maintain a sharing session in the background is still a working draft and no browsers support it yet (w3c/geolocation-sensor#30).

About the planes, what I meant was the gps will be lost on a plane but if the share is still active, it should stay in the same position until i connect in another country right? and then it would be drawn on the map after connection.... all if the share is still valid and not expired.

This is correct.

About the expiry, it would be nice if the route was still available for a set period of time that could be changed per share.

You could create a new issue for this, it should be possible to implement this with the current Memcached/Redis backend.

One more thing, there should be a 'distance to me' showing if the follwer has their location services on.

There's no space for this on user markers currently, but maybe this could be possible to add in the menu suggestion in #101.

I'm excited to see how this project develops but at the moment i think the use case is very small and too temporary for practicality.

It's intended to be a pretty narrow scope. The functionality mimics that of Glympse. For more permanent tracking, there are already many other solutions like PhoneTrack (Nextcloud addon), OwnTracks, etc.


404 page isn't working

Seems to work when you go to https://jpasholk.com/jekyll-uikit/404/ in your browser but it's not redirecting correctly.

have you done all the steps listed here? https://help.github.com/articles/creating-a-custom-404-page-for-your-github-pages-site/
Note particularly, Step 4....


[1.12.2] Quarry Builder dropping Chisel blocks in-world

Issue description:
When mining an area with the Builder, with a Quarry Card, all blocks from the Chisel mod are dropped in-world, instead of being put in the inventory on top of the builder.
Steps to reproduce:
Place Chisel Marble, I used a 7x7x7, but that shouldn't matter.
Place Builder.
Use Quarry Card on builder, and set the area to the Marble cube.
Supply the Builder with power, and put a chest above it.
Run the Builder
-> Bug

Versions:

Minecraft: 1.12.2
Forge: 1.12.2-<IP_ADDRESS>73
CompatLayer (only if on Minecraft 1.10 or 1.11):
McJtyLib: 1.12-2.6.7
RFTools:1.12-7.33

Possibly incompatible mods and versions:

Relevant logs, if any:
The latest.log has no information relating to RFTools.
First reported in https://github.com/NillerMedDild/Enigmatica2Expert/issues/43

Please report to chisel too. It may just as well be a bug on their side. I'm also pretty sure this didn't happen in the past and the Builder hasn't changed recently

Good point, will do.

Hm, it might be unique to my modpack then. Can I have your modlist for comparison?

Im afraid not, since this pack is private with mods form Authors who don tiwsh information out, im not able to.  But it has the most common things, Chisel Nucelearcraft, rftools, thermal*, DE, EnderIO, thaumcraft, tinkers, aroma*

Fair enough, thanks anyway.
Since the issue seems to be on my end, I'll close the issue.

For anyone who's looking at this report in the future, the actual problem was caused by Tool Progression.


Aiden Huynh, Period 3 - 2015 FRQ CollegeBoard and PBL

Question 1: Arrays/2D Arrays
While our project does not currently use any 2D arrays, there are several points in which we operate on arrays. We prefer to use ArrayLists most of the time due to their versatility, and an example of this can be seen in the following code block.
@PostMapping("/newCode")
    public ResponseEntity<QrCode> newCode(@RequestBody QrCodeRequest qrCodeRequest) {
        QrCode qrCode = new QrCode();

        List<String> links = qrCodeRequest.getLinks();
        List<Double> frequencys = qrCodeRequest.getFrequencies();

        for (int i = 0; i < links.size(); i ++){
            qrCode.addLink(new LinkFreq(links.get(i), frequencys.get(i)));
        }

        repository.save(qrCode);

        return new ResponseEntity<>(qrCode, HttpStatus.OK);
    }

In this example, we use lists to store data being used in our QR Code generator, which takes an input of different URLs and specified frequencies for how often to be redirected to a certain URL. These URLs and frequencies are stored in arrays in our backend and operated on with our key algorithm, newCode() which adds the links with their frequencies to the qrCode generating API.
This differs from the FRQ in that we are not checking for certain conditions while traversing the array, but instead just using them to more conveniently move data.
Question 2: Classes
I don't know how our project could possibly exist without classes, but one example of them being used is in our seating chart, which is not fully implemented yet, but will be used to store a map of where students sit within a classroom.
public class SeatingChart {
    private Map<Integer,Map<Integer, String>> chart;
    private long classId;

    public SeatingChart(Map<Integer,Map<Integer, String>> chart, long classId) {
        this.chart = chart;
        this.classId = classId;
    }

    public Map<Integer,Map<Integer, String>> getChart() {
        return this.chart;
    }

    public long getClassId() {
        return this.classId;
    }

    public void setChart(Map<Integer,Map<Integer, String>> chart) {
        this.chart = chart;
    }

    public void setClassId(long classId) {
        this.classId = classId;
    }
}

This code block shows the SeatingChart class complete with a constructor, setters, and getters, which are all basic components of a class, which can also be seen in the FRQ. The most important takeaways from stuff like this for me is to know when to use public and private, and I've noticed private is used for setting local variables stored only within the object. Also, I learned from this FRQ that constructors in java do not put a return type.
Question 3: Method and Control Structures & 2D Arrays
This question was the most difficult of the 4 for me because it used a very interesting data structure that I had never heard of before, a SparseArray. This is not used anywhere within our project, but the main components of the question, that being using classes and arrays, very much are used in the project. An example of this can be seen in our model for how the seating charts will work.
{
  "chart": {
    "1": {
      "1": "John",
      "2": "Alice"
    },
    "2": {
      "1": "Bob",
      "2": "Eve"
    }
  },
  "classId": 123
}

This code block shows how we intend to navigate and label certain table groups, which will require very similar algorithms such as the one in the FRQ to traverse.
Question 4: Method and Control Structures
This question involved using interfaces and implementing them and overriding their methods. This is used in multiple points in our project, mostly implementing templates from JPA.
public interface ClassPeriodJpaRepository extends JpaRepository<ClassPeriod, Long> {
    ClassPeriod findByName(String name);

    List<ClassPeriod> findAllByOrderByNameAsc();

    // JPA query, findBy does JPA magic with "Name", "Containing", "Or", "Email", "IgnoreCase"
    List<ClassPeriod> findByNameContainingIgnoreCase(String name);

    List<ClassPeriod> findByLeadersContaining(Person leader);
    List<ClassPeriod> findByStudentsContaining(Person student);
}

This code example for storing class periods extends the basic JPA repository, allowing the ClassPeriodJpaRepository template to inherit useful methods from JpaRepository, while also adding on its own methods.
This FRQ was very helpful to complete for me, as I hadn't really tried to make my own interface before and also I got to work with inheritance which is always fun.
Overall
Most of this will be restating certain points I made on each individual reflection on my FRQs, but here is a comprehensive reflection on all of the FRQs in one spot:
Highlights:

I learned that constructors in Java omit the return type to clarify that it is a constructor and not another method
Learned how to make and use interfaces
Learned the structure of a CSA FRQ, as I have never worked on them before

Future Plans:

I definitely need more practice with Java, as for the first two problems I had a couple syntax errors here and there, which I feel like are going to be my main problem for the actual AP Exam
I want to try doing another practice exam but actually timed, to make sure that I can do them within the allotted timeframe for the actual exam, as I just did these problems periodically

Code grading
FRQ 1
Part A: After pulling and running it, the output is correct. We solved it in essentially the same way. Since this is an easy part of an easy FRQ question, this is to be expected.
Part B: After pulling and running it, the output is correct. There are a few differences in the way we each solved this part, but both solutions are valid. Your use of another method to turn the array into a string shows how deeply you understand the concept of arrays in Java.
Part C: After pulling and running it, the output is correct. We have a few differences in our solutions. Your use of nested for loops shows a mastery of that concept.
Overall: As you said, this was a pretty easy question, and you did well on it. Therefore, this FRQ earns a 0.90/0.9.
FRQ 2
When pulling and running your code, the output is correct. I like how you went above and beyond and added a check to make sure that the user's guess is the same length of the secret word. Especially because of this, you will earn a 1.1/0.9 on this FRQ.
FRQ 3
Part A: When pulling and running your code, the output is correct. Your code shows that you have a good understanding of arrays.
Part B: When pulling and running your code, the output is correct. Your code shows that you have a good understanding of arrays and how to work with them.
Overall: Good job on this FRQ. I'd give you a 0.9/0.9 for this FRQ
FRQ 4
Part A: When pulling and running your code, the output is correct. You showed that you had an understanding of interfaces, and although it may not be on the AP exam this year, it is still valuable to know.
Part B: When pulling and running your code, the output is correct. You showed that you knew how to use inheritance, and how to override methods.
Part C: When pulling and running your code, the output is correct. You showed that you knew how to work with for loops and classes very well.
Overall: Good job. I'd give this FRQ a 0.9/0.9
Total: 0.95+1.1+0.9+0.9 = 3.80/3.6
Reflection grading
I like all your associations between the FRQs and your project. The associations, along with your explanations of your code, show that you're thinking deeply about how you can use the FRQ to further your knowledge in Java, and better your project in the future. I also liked how you noted your shortcomings while taking the FRQ test, and you you can improve in the future. I think that one thing you could do to improve is continue to work on this FRQ and others like it.
For the Reflection component, I'll give you a 3.6/3.6

Crossover Reviews Reflection
Grading

I feel that I graded as fairly as I could, especially dealing with clear cheating on the assignment, as I consulted Mr. Mortensen on how to handle that situation
Overall I think grading on 1.1 is a little crazy

Learning

Seeing different approaches to problems is useful for learning new methods

i.e. StringBuilder (did not know that existed)

This was useful to see the capabilities of other people in the class to prepare for 12 person teams (I see that Vinay has a decent work ethic and teamwork skills)

Future Plans

Suggested to do more FRQs for learning and solidifying syntax
Future project will be decided upon after finding new members for the team

Me and Drew are scouting for prospects...
I am very, very excited to work with larger teams!


Created ikalog/ui/options.py file.

No user visible behavior is changed by this PR.
Moved ikalog.ui.gui.OptionsGUI and ikalog.ui.panel.options.OptionsPanel to
the file.
Integrated OptionsPanel into OptionsGUI.

Thanks!


WIP: CNTRLPLANE-2: Update kubernetes to 1.32

I had to cherrypick most of Joel's changes from https://github.com/openshift/api/pull/1776 to get the api/tools updated as upstream has moved to gengo/v2.

/retest


Experimental: detect pattern variables in reactions

Current coverage is 98.55% (diff: 100%)

Merging #59 into master will increase coverage by <.01%

@@             master        #59   diff @@
==========================================
  Files            12         12
  Lines           552        555     +3
  Methods         491        502    +11
  Messages          0          0
  Branches         61         53     -8
==========================================
+ Hits            544        547     +3
  Misses            8          8
  Partials          0          0

Powered by Codecov. Last update 0547621...40bfda1


Update latest version to v0.7.1

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Update latest version to v0.7.1
Which issue(s) this PR fixes:

Part of #2542
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/cherry-pick website

/retest

There is not enough capacity in the test infra.
/rest

/restest

/retest


Build instructions

For someone who never did Qt dev work (like me), it's not clear how to build this code. I guess Qt Creator is involved?
I'll go figure that out now, but a couple pointers here and there in the Readme would be much appreciated!

Just running 'qmake' to generate a Makefile, and then 'make' to build the program is what works for me.

That worked for me too!


Improved Look "Tutorials Using Other Tools" section in gitkraken-tutorial.md

Names now appear under their respective programs and added name in Contributors.md

@ubatin This is the output of gitkraken-tutorial.md.

@vipulgupta2048, can we keep the table style we follow in other tutorials? It'll be great to have consistency.
||||
|---|---|---|
|[Command Line](README.md)|[Visual Studio 2017](github-windows-vs2017-tutorial.md)|[GitHub Desktop](github-desktop-tutorial.md)|

Command Line
Visual Studio 2017
GitHub Desktop

No issues, That actually looks better. And uniformity should be maintained. Will implement the same.

Implemented. Ready to merge.

@vipulgupta2048, Looks good to merge now. Thank you for your time and effort


Lambdatest upload 0.0.1

What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[ ] I will not move an already shared step version's tag to another commit
[ ] I read the Step Development Guideline
[ ] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[ ] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[ ] I read and accept the Abandoned Step policy

New Step
Thank you for the new Step share! The CI check might will fail due to our extended validation engine. Nothing to worry about yet, we will get back to you shortly.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Made the requested changes. Please verify now @mateherber @tothszabi

All the changes are done. Thankyou

Hey @rohithmone! The commit hash has changed to 150c91cc2a8daaf2741c912e2f70ab29ef4ec21d which has to be reflected on the twelwth line. (You'll need to edit the yml in the steplib fork, not in the original repo)

May I know how much time does it take to get added our step in integrations page? @mateherber

@rohithmone https://www.bitrise.io/integrations/steps/lambdatest-upload
usually takes a few minutes


Song change event double fires

Seems to only happen when a song is skipped

https://github.com/Sponges/DubTrack4J/commit/4ac4b2be80088a9fc4c4dd98ecc2f11944a02862 fixes this


Floodlight Cam Support

Is your feature request related to a problem? Please describe:
I wish my Eufy wired doorbell and floodlight cameras were supported in HomeKit. :)
Describe the solution you'd like:
Floodlight cameras working in HomeKit. Bonus points if light can be exposed as a switch to HomeKit as well.
Describe alternatives you've considered:
Buying different floodlights that work with HomeKit... Ugh.

If I can get video stream to work in general, I can support more video cameras from eufy.
I can also add the light as a button, sure thing 😀

@birkir yep, I saw your note in README.md that it's "for now" a doorbell only plugin, and I assumed that once video stream for one class of Eufy device was functional then the others may not be far behind. 👍

Good news, I got video to work (with audio).
I have the 2C cameras I can test with. I'll hopefully have something by tomorrow.

Excellent work man! I agree with @mariusstrom that possibility for other eufy products to Homekit would be amazing. I've got motion sensors and door/window sensors that would be ideal to be added to homekit. Any chance for those? Thank you.

It's very hard to program for something you don't have 😁
I would either need to get the products or have someone that has the products help me.

True. I could try, anyway I can of course.
Lead the way! 😃

I would need to get a dump of the logs ("found device X, and all the events "push message: ")
I need to add more logs for the parameters details.

How can I acquire the logs?
On Mon, Dec 7, 2020 at 10:27 AM Birkir Gudjonsson<EMAIL_ADDRESS>wrote:

I would need to get a dump of the logs ("found device X, and all the
events "push message: ")
I need to add more logs for the parameters details.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/birkir/homebridge-plugin-eufy-security/issues/2#issuecomment-739758999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFHYP6BVIK54L647DYOBGETSTSGW5ANCNFSM4UJWJNBQ
.

@birkir if you're in the USA (or a country with 110V power), I would strongly consider gifting you a Eufy Floodlight cam if that's the impediment to progress on this.

I think I've got enough logs in there for you guys to help me out through github (use version 0.1.3, homebridge -D)
If you can give me a dump of

homebridge startup (device discovery)
2.push messages when turning things on/off through the eufy app (light, etc.)

is there any way to get these logs from HOOBS? I've got the plugin working great but there's nothing in the logs for discovery that mentions by floodlight cameras.

Is there any update regarding this feature?


Add invalid character replacement for filenames

It's configurable.

wrong base branch used for fork. Will create a new MR


LB service scaled without updating the scale value when it volume mounted a service with a different scale.

Server Version - V0.21.3
Steps to reproduce the probelm:
Created environment - e1.
Created a service S1 with scale 2 in e1.
Created a LB service S2 with scale 1 and dataVolumesFromService pointing to S1.
Activate environment.
When the environment gets activated , there are 2 LB agents created in service S2. But the scale of S2 is still 1.
Expected Behavior:
Scale of S2 should be 2.
This issue is not reproducible if S2 is a regular service( not a LB service). In this case , we see the scale being updated to 2.

Retest when new Sidekick feature is implemented.


WFLY-6743 See if PassivationTestCase.testPassivationMaxSize is till f…

…ailing
https://issues.jboss.org/browse/WFLY-6743

@rhusar once done, you also need a downstream PR for this to be considered


Mapview not displayed

Trying to use the test app, no map is shown.
I tried adding this style url in all locations: mapbox://styles/mapbox/streets-v12 and also the access token, with no luck.
Steps to trigger behavior

Download the latest repo
Start the test app
No map is displayed on either of the 2 screens.

This repo as well as the other repos in the maplibre organization are developed indepndently fomr Mapbox and special Mapbox notation like mapbox://styles/mapbox/streets-v12 might not work.
I haven't tried it, but it might work using this endpoint: https://docs.mapbox.com/api/maps/styles/
You can try an older version < v2 or you can try different map providers like:
https://www.maptiler.com/
https://stadiamaps.com/
https://mapilion.com/
As this is not a navigation issue, I will close this here.


undefined behavior sanitizer complains about get_value_int()

Hi.
Caught this accidentally, while experimenting with sanitization of darktable-org/rawspeed with all libs built in-tree:
/home/lebedevri/rawspeed/build/pugixml/pugixml-src/src/pugixml.cpp:4531:51: runtime error: unsigned integer overflow: 0 -<PHONE_NUMBER> cannot be represented in type 'unsigned int'
    #0 0x6395f2 in pugi::impl::(anonymous namespace)::get_value_int(char const*) /home/lebedevri/rawspeed/build/pugixml/pugixml-src/src/pugixml.cpp:4531:51
    #1 0x6395cb in pugi::xml_attribute::as_int(int) const /home/lebedevri/rawspeed/build/pugixml/pugixml-src/src/pugixml.cpp:5134:36
    #2 0x5fa23d in RawSpeed::Camera::parseCFA(pugi::xml_node const&) /home/lebedevri/rawspeed/src/librawspeed/metadata/Camera.cpp:123:34
    #3 0x5f9278 in RawSpeed::Camera::parseCameraChild(pugi::xml_node const&) /home/lebedevri/rawspeed/src/librawspeed/metadata/Camera.cpp:289:5
    #4 0x5f8a4b in RawSpeed::Camera::Camera(pugi::xml_node&) /home/lebedevri/rawspeed/src/librawspeed/metadata/Camera.cpp:57:5
    #5 0x538bbd in RawSpeed::CameraMetaData::CameraMetaData(char const*) /home/lebedevri/rawspeed/src/librawspeed/metadata/CameraMetaData.cpp:49:21
    #6 0x511471 in main /home/lebedevri/rawspeed/src/utilities/rstest/rstest.cpp:331:24
    #7 0x7f1f8ff312b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #8 0x4dfe99 in _start (/home/lebedevri/rawspeed/build/src/utilities/rstest/rstest+0x4dfe99)

The code in question is: https://github.com/zeux/pugixml/blob/v1.8.1/src/pugixml.cpp#L4531
PUGI__FN int get_value_int(const char_t* value)
{
  return string_to_integer<unsigned int>(value, 0 - static_cast<unsigned int>(INT_MIN), INT_MAX);
}

It does not like that unsigned subtraction.

I'm not sure I understand the reason for the error. As far as I am aware unsigned overflow and underflow are perfectly defined (this is mostly why the code in string_to_integer is able to perform a fast out-of-range check without doing extra work for every processed charcter).

Ah, okay:

-fsanitize=unsigned-integer-overflow: Unsigned integer overflows. Note that unlike signed integer overflow, unsigned integer is not undefined behavior. However, while it has well-defined semantics, it is often unintentional, so UBSan offers to catch it.

pugixml intentionally uses the fact that unsigned integer overflow is defined here to get the negative INT_MIN in unsigned - that is, to get the absolute value of the minimum integer in the range that is representable. I'll take a look at possibly simplifying this code to avoid this but it may prove unnecessarily hard.

Looks like it's possible to disable this for the specific functions, although I'm not sure I like this idea in general (http://clang.llvm.org/docs/AttributeReference.html#no-sanitize-clang-no-sanitize).
My guess is that this would also trigger on hash functions (specifically for XPath variable set operations with long-enough variable names and also for PUGIXML_COMPACT). Note that -fsanitize=undefined does not include the unsigned integer overflow checks - you have to explicitly request additional integer sanitization to catch "suspicious" integer operations (that are well defined).

Thanks for looking into this.
I do understand that there may be cases where semi-ub is intentional.
However such cases may decrease SNR of the sanitizer, like i have expirienced in this case.
So if this can not be fixed properly, please consider marking this function with __attribute__((no_sanitize("unsigned-integer-overflow"))) to hide the issue.

This is not semi-UB. Unsigned integer overflow is perfectly defined; it's basically impossible to implement a fast hash function without it. The fact that ubsan offers an option to check for this is kinda separate; I think I had issues with floating-point division by zero with UBsan as well, which will be triggered by the XPath test suite and is important to following the XPath spec (that maintains the precise behavior that 1 div 0 results in a NaN).

This is not semi-UB.

Yes, sorry, i don't disagree.
I just felt like reporting that clang's -fsanitize=integer complains.
It is up to you to decide what to do with it :)
For me, it would be nice if it could be either fixed, or silenced, or documented.


feature: delete old packs after serialization

What kind of change does this PR introduce?
feature: delete old packs after serialization
closes #13291

Did you add tests for your changes?
not yet

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?
nothing

Any plan to merge this?

@hardikpatel043 near future

Hi Team, This would benefit the world greatly and avoid burning SSD faster...

The webpack cache is unusable on the CI right now if the cache keeps growing for every build.
Is the near future approaching?


CLOUD-703 update info about user secrets and random passwords

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[WIP] PhpdocSeparationFixer - prove bug

closes https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/2365
@GrahamCampbell happy debugging!

Ah, thank you. :)

Well, I mean, naively just reverting to the original algorithm if the new one doesn't match the correct number of items works:

:+1:  however it would be nice if we could avoid the second preg_match* though

Thank you @SpacePossum.

I don't like the way this fix is implemented. You made the line class silently fail. It should be the responsibility of the fixer to decide what to do imo.

we quickly fix the tool to not crash, which was the main concern.
if you want to provide a fix to change phpdoc in that scenario - go ahead ;)


modify configure prefix to configure sources with prefix as well

Background
currently conf/sources.json contains an example prefix.  Because of the way its described, silly people (like me) may make the assumption that the prefix will be replaced when a prefix is configured. (it is not)
Desired Change
enhance the configure prefix command to also change the prefixes in sources.json

Maybe something like this would be cool:
$ python manage.py configure sources --entity kinesis --name prod_stream_alert_kinesis --logs osquery,cloudwatch,ghe

this doesn't really make sense, since "sources" and their log definitions can be arbitrary and we shouldn't do what we "expect". the onus has to be on the user for this. the comment from jack would be a nice feature, but a completely different feature than the request.


Feature: Generate the resource listing

pyramid_swagger.ingest.build_schema_mapping() expects a file in a very specific format where the paths to api docs match the filename containing the docs (with the .json suffix removed). The rest of the listing file is pretty minimal (a swaggerVersion and an apiVersion).
In another project I took the approach of generating this file instead. From the list of files in the directory (plus the two versions) we can create the entire resource listing.
I propose we do the same in pyramid_swagger using pyramid_swagger.generate_resource_listing to enable the feature (default it to false for backwards compatibility).  The apiVersion and swaggerVersion could be specified in a file in the directory.

Hm. Is the argument here that the resource listing is completely uninteresting and should be done automatically? I guess if we can encode arbitrary resource names into filenames, this would be pretty easy. Or are you proposing pulling it from the resourcePath in the API Declaration?

Ya, I think that's the idea. pyramid_swagger requires a very specific mapping between resource path and filename.  /api-docs/resource must be provided by a file at <schema_dir>/resource.yaml.
So instead of creating a file with all of those mappings, we could instead just glob.glob('%s/*.yaml' % schema_dir) to get the list of resources, and then build the resource listing from the base file (which can contain api versions and authentication I guess), and the apis section would be generated from the file list.

I created a folder api_docs which contains 3 files : api_docs.json , pet.json and I added resource.yaml when I execute the command pserve development.ini , I got nothing , the file resource.yaml is empty , I got the error
"override this listing.".format(schema_dir, API_DOCS_FILENAME))
pyramid_swagger.ingest.ResourceListingGenerationError: C:/Users/sabrine/Desktop/
MyProjectSwagger/api_docs/api_docs.json has an apis listing. Generating a list
ing would override this listing.

Hi Sabrina,
If your api_docs.json already has an apis entry, you'll want to make sure the configuration switch for resource generation is set to False: pyramid_swagger.generate_resource_listing = false in your .ini file.

Hi,
I got a new error :'(
400 Bad RequestThe server could not comply with the request since it is
either malformed or otherwise incorrect.
Could not find the relevant path (/api_docs/pet) in the Swagger schema.
Perhaps you forgot to add it?
I verfied all the endpoints that exist under my folder /api_docs
2015-05-01 0:21 GMT+02:00 Scott Triglia<EMAIL_ADDRESS>
Hi Sabrina,
If your api_docs.json already has an apis entry, you'll want to make sure
the configuration switch for resource generation is set to False: pyramid_swagger.generate_resource_listing
= false in your .ini file.
—
Reply to this email directly or view it on GitHub
https://github.com/striglia/pyramid_swagger/issues/83#issuecomment-97988686
.

If it runs coorectly, it should generate a file swagger.json under the
api_docs folder?
2015-05-04 9:55 GMT+02:00 Sabrine Moumni<EMAIL_ADDRESS>
Hi,
I got a new error :'(
400 Bad RequestThe server could not comply with the request since it is
either malformed or otherwise incorrect.
Could not find the relevant path (/api_docs/pet) in the Swagger schema.
Perhaps you forgot to add it?
I verfied all the endpoints that exist under my folder /api_docs
2015-05-01 0:21 GMT+02:00 Scott Triglia<EMAIL_ADDRESS>
Hi Sabrina,
If your api_docs.json already has an apis entry, you'll want to make
sure the configuration switch for resource generation is set to False: pyramid_swagger.generate_resource_listing
= false in your .ini file.
—
Reply to this email directly or view it on GitHub
https://github.com/striglia/pyramid_swagger/issues/83#issuecomment-97988686
.

Let's have this discussion in a single issue (#104), so it's easier to keep track of.


Add community section

Create section for see and joint our community in Discord , Twitter , Github and etc.

This is already resolved.


Add an initial smaller sleep time before net dialing plugin socket.

Signed-off-by: Anusha Ragunathan<EMAIL_ADDRESS>

LGTM


Ability to star starboard posts

https://gyazo.com/1681ab06fd13726a6a4a45dbfee05193

Such a useful feature. I 100% agree. 🤣

Probably shouldve provided more details..... ignore the starboard channel from stars so that you cant star starboard messages.


Implement Recursive Nested Array Visualization (fixes #15)

Codecov Report

Merging #16 (41f5207) into master (88c84df) will increase coverage by 12.98%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master      #16       +/-   ##
===========================================
+ Coverage   76.81%   89.79%   +12.98%
===========================================
  Files           3        3
  Lines          69       98       +29
===========================================
+ Hits           53       88       +35
+ Misses         16       10        -6

Impacted Files
Coverage Δ

structoscope/lib.py
85.29% <100.00%> (+20.07%)
:arrow_up:

tests/test_lib.py
100.00% <100.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 88c84df...41f5207. Read the comment docs.


breaking changes changelog?

Hey,
I really like this module, great work!
When I tried to upgrade to 3.0.0 a few tests broke. I was just wondering if there is a changelog somewhere mentioning the breaking changes?
Why are there breaking changes, is it because Sinon or Chai released new major version? If so, maybe I can find a document describing the breaking changes there?
Cheers.

just a short update on the breaking change I experienced.
before 1.0 the sinon.fakeServer was present and just worked.
In 1.0 the fakeServer is all of a sudden undefined.
Is this the intended behavior?
Would appreciate a short comment on this issue

also fakeTimers is disappeared

Added Changelog to README, the breaking changes were due to broken sinon version


 Since there is currently work underway on the GitHub workflows, don't turn on enforcement of this. The changes for that should be small, so long as it is started soon.

Tag @ryanpeach  for info.

Closing since apparently @Harsha-Nori is close to pushing a working version of this.

I recommend ruff. It will format and lint your code, and is very fast. isort, black, flake8 all in one


Transpose ChatFeed with CSS

Make it default scrolled to the bottom

Long story short this won't work.
This implementation inverts scroll directions and you cannot switch this back around on react yet.


Error handling improvements

#51 and #20 are related.
Upd: more work needs to be done, e.g. $ref should be processed nicely.

Fixed now.


cache and proxy

Hey guys,
I have an issue. In real world of scrappers when using proxies, checking if You have response code 200 or 500 is simply not enough. Plenty of proxies or even website itself can throw an output with code 200 which is invalid from developer point of view and yet its getting cached. Is there any way to apply some kind of custom logic which would determine if response is what we want? For example:
colly.CacheDir("some/dir", func(resp) {
   if bytes.Contains(resp.Body, []byte("some magic marker")) {
      return false // don't cache
   }
})

I'm new to colly and i was digging thru source code for a bit, but didn't find such of feature. It's also useful without proxies.

Actually, i was playing around and hacking the code for a bit, this seems like much more reasonable then returning bool:
func CacheFilter(resp) error {
   if bytes.Contains(resp.Body, []byte("some magic marker")) {
      return errors.New("something went wrong")
   }
   if bytes.Contains(resp.Body, []byte("some magic marker 2")) {
      return errors.New("something went wrong 2")
   }
  return nil
}

colly.CacheDir("some/dir", CacheFilter)

It's working really well, but its implementation is kind of ugly atm., so i don't think its good enough for PR. Also i don't really know yet, how to bubble up the error, so it could be returned ie. in debug output.

I like this idea.  I've run into cases like this as well where I'll get a 200 response code, but the page may be a "you appear to be a bot", in which case the content is not meaningful.
As for bubbling up the error, could this be done in httpBackend.Cache() ? That way it will naturally end up in the OnError() callbacks.


Non-negative conjugate gradient?

We are using jaxopt to solve Ax = b s.t. x >= 0 using ProjectedGradient:
pg = jaxopt.ProjectedGradient(objective, projection_non_negative, ...

However our matrix A is positive definite, so I would love to use conjugate gradient.  In some testing, using jax.scipy.sparse.linalg.cg doubles our pipeline performance, which is great, but then we lose the non-negative constraint.  Googling around I found this:
https://github.com/david-cortes/nonneg_cg
I was going to try it out in JAX, but before I do: is this something that would be useful to jaxopt?  Or is there a way to do non-negative conjugate gradient in jaxopt already that I haven't noticed?  Thanks!

It seems like the paper "A conjugate gradient type method for the nonnegative constraints optimization problems" is a bit low on the citation number side (only 5 citations). IMO, it's better to focus on more established, standard algorithms. @fabianp may also have an opinion on this particular case.

For sure, defer to you guys as to whether this makes sense for jaxopt.
In case it's useful to know, non-negative conjugate constraints are useful in physics simulation, where you have very large constraint jacobian matrices that determine collision forces and other things.  MuJoCo has implemented non-negative cg:
https://github.com/deepmind/mujoco/blob/main/src/engine/engine_solver.c#L710
And we plan to implement this in JAX for our simulator, Brax.
So the approach may not have many research citations but it is useful, and in practice converges much faster than other approaches, for our domain at least.  But totally get if this isn't appropriate for jaxopt, feel free to close the issue if so.  Thanks!

If the code is not too large and it's confirmed that it works better for some use cases, then I would be open to it. We may want to add (vanilla) CG directly to JAXopt too. Indeed our API gives access to individual iterations, which is sometimes useful.

OK after some tinkering I was able to get ProjectedGradient to be just as fast as jax.scipy.sparse.linalg.cg.  Two things I had to do:

set stepsize > 0 to disable the backtracking line search
set implicit_diff= False, this also had a pretty significant impact on perf

I don't yet have a firm grasp as to why these two changes had such a dramatic impact, or why my use case is different from the average use case for which jaxopt has determined those default parameters - maybe in some future iteration of jaxopt documentation, you could include some guidance about when it is appropriate to change these default parameters?
But for now at least I'm happy with the performance of ProjectedGradient and probably won't pursue adapting non-negative cg.
Thanks for the great work in jaxopt!

Thanks a lot @erikfrey for the heads up. Happy to hear that you managed to reach a satisfactory level of performance. I agree we could do a better job at explaining what are the defaults and how to change them.


[Question] How do I use or configure swarm manage in multi-region?

According to the README.md, if I were right, swarm manage should be able to be used in other networks which is not the same as the network of swarm join.
# on each of your nodes, start the swarm agent
# <node_ip> doesn't have to be public (eg. 192.168.0.X),
#  as long as the other nodes can reach it, it is fine.
$ swarm join --addr=<node_ip:2375> token://<cluster_id>

# start the manager on any machine or your laptop
$ swarm manage -H tcp://<swarm_ip:swarm_port> token://<cluster_id>

But, after some testing, I realized that swarm manage(the manager) needs to be able to reach it. If I want to join via private networks(192.168.0.X) and also I want to multi region(us-east-1,us-wast-2 ...) too, how do I use and configure swarm manage(the manager)?
Thanks for reading. :-)

I have the same question related with the swarm manageand its requirements.

@anarcher @ssoto you are both right, we updated it in the userguide, but we forgot in the README.mds
You should read
diff --git a/README.md b/README.md
index a2ababb..5d8c900 100644
--- a/README.md
+++ b/README.md
@@ -44,7 +44,7 @@ $ swarm create

 # on each of your nodes, start the swarm agent
 #  <node_ip> doesn't have to be public (eg. 192.168.0.X),
-#  as long as the other nodes can reach it, it is fine.
+#  as long as the manager and the docker cli can reach it, it is fine.
 $ swarm join --addr=<node_ip:2375> --discovery=token://<cluster_id>

 # start the manager on any machine or your laptop

I'll make a PR asap.


[Doc][ALERT-12555] submit Slack docs

Purpose of the pull request
submit Slack docs
close https://github.com/apache/dolphinscheduler/issues/12555
Brief change log
submit Slack docs
Verify this pull request
Manually verified the change by testing locally.

Change LGTM, but Image check failed and it found not to use img with name, can you check whether we should use it in our docs, or remove it if not used? @Tianqi-Dotes

Change LGTM, but Image check failed and it found not to use img with name /img/new_ui/dev/alert/alert_slack_enable.png, can you check whether we should use it in our docs, or remove it if not used? @Tianqi-Dotes

Dear Jiajie,
Have added the 'enable' image to the docs. Please help review again.

Restart the failed test


indian version of comma instead of foreign

Can we have 1,00,000 instead of 100,000, 10,00,000 instead of 1,000,000.
Hint: for first three number from right, add a comma after third, after that add a comma after every two words.

https://github.com/Tele-Bots/CovidBot/commit/57cbf0d53e585cb90383c2e4814facc2b0f73da7


cannot read property 'replace' of undefined

Hello,
I am trying to do this my jquery node_module is v2.1.4 and jsdom is v6.5.1
//jsdom dependancy of jQuery
var jsdom = require('jsdom').jsdom, document = jsdom('paddio');
global.window = document.defaultView;

//test below line again new jQuery 3.0.0 module again
global.XMLHttpRequest = window.XMLHttpRequest;

//jQuery
var $ = require('jquery');
$.support.cors = true;

While initializing jsdom i am getting exception i have attached the image for this.
I am using Visual Studio Code to debug node program.

Please help me to resolve this issue.

Your code works fine for me - but also seems kinda incomplete...

What Node.js version are you using? I imagine VSCode probably comes with an old one.

So Visual Studio Code give option to break program for All Exceptions. When i choose that option it catches this exception of image. If i don't choose that option then it doesn't break there and it initializes jQuery also but then when i tries to initialize signalr using
var signalr = require('./util/jquery.signalR-2.2.0.js');
Signalr fails saying jQuery is not initialize. So due to this error my entire program stops.
So i wondered what could be the cause then i found out jsdom was failing initially and because of that jQuery and SignalR not working.

@domenic I installed latest version of Node v4.1.2 verified using node -v

OK. In that case I think the break on all exceptions thing is a red herring. signalr is probably confused for other reasons.
In general you should not be trying to use jsdom by attaching properties to the node.js global object. You should instead be running code inside the jsdom browser window, as all the examples in the readme show.

I was trying this. https://github.com/tmpvar/jsdom#for-the-hardcore-jsdomjsdom
It was working so far. After node upgrade from 0.12 to 4.1.2 and jquery, jsdom uprade this entire flow broke.

SignalR checks for the global window.jQuery, so that's where you need to append jQuery.
However it's probably better if you use jsdom.env(), you can load the jQuery and signalr script there and it'll do the right thing.

@livesankp your example in the original post bears no resemblance to https://github.com/tmpvar/jsdom#for-the-hardcore-jsdomjsdom. It is setting global properties, instead of e.g. doing window.document.appendChild(scriptElement).

@Sebmaster window.jQuery = $; this is working. No error in Signalr are now.
Looks like issue is resolved but can you suggest what will be best way to use here. Combining all 3.

The best way is to stop mucking with the global. Here is an example to get you started, but you'll have to finish it.
const jsdom = require("jsdom").jsdom;
const document = jsdom();
const window = document.defaultView;

const script1 = document.createElement("script");
script1.src = pathToJqueryJSFile(); // fill this in yourself
document.body.appendChild(script1);

// now do the same for angular and signalr

var jsdom = require("jsdom").jsdom;
var doc = jsdom(undefined, undefined);
global.window = doc.defaultView;
//test below line again new jQuery 3.0.0 module again
global.XMLHttpRequest = window.XMLHttpRequest;
//jQuery
var $ = require('jquery');
$.support.cors = true;
window.jQuery = $;
//SignalR added externally, Configuration, Common functions, Constant
var signalr = require('./util/jquery.signalR-2.2.0.js');
@Sebmaster @domenic So this is working now. Is this the correct way to go with ?
Once this is initialize i will be able to do other serialport connection and other stuff.

You're doing it wrong. That way it can still break depending on what else singalR accesses.
Use jsdom.env and load your jquery and signalR with the scripts option. They'll get a proper environment in there (which prevents other stuff from breaking) and you can access them by accessing window.prop (for example window.$ for jquery).


Schedule being recalculated instead of stopping

Version 2.0.1
Configuration
Easee charger connected to EV
Describe the bug
I have configured a manuel soc and target_soc entity to plan my charging schedule. At the same time I have configured automations to manage when the car is connected.
My issue is that when I set soc the schedule is being calculated as it should. The charging also starts at the planned time. Unfortunately the charging does not stop again as it should. The schedule is just being recalculated when the time is up (I do not fully charge).. This will continue to happen until the car reports that the battery is full in which case I set soc to 100.
Am I missing something here because schedule should not be recalculated when charging is in progress.
keep_charger_on is set to false btw.
Debug log

Add your logs here.

I finally made it work. I made two changes to my settings:

I use automations to start and stop charging instead of the configuration setting with the switch.
The automation to monitor if the EV is connected to my charger only sets the new state if it actually has changed (several states in Easee means “on” or “off”).


AndroidTV error after bump androidtv to 0.0.11

Home Assistant release with the issue:
python version: 3.7.2
HA version: 0.90b0
virtualenv
Last working Home Assistant release (if known):
Component from 0.90b0
Component/platform:
media_player/androidtv
Description of problem:
After bump androidtv to 0.0.11 PR2025 I have error:
Fri Mar 15 2019 09:34:01 GMT+0100 (czas środkowoeuropejski standardowy)
Update for media_player.mibox fails
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/helpers/entity.py", line 220, in async_update_ha_state
    await self.async_device_update()
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/helpers/entity.py", line 348, in async_device_update
    await self.hass.async_add_executor_job(self.update)
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/androidtv/media_player.py", line 186, in _adb_exception_catcher
    return func(self, *args, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/androidtv/media_player.py", line 342, in update
    self.aftv.update()
  File "/srv/homeassistant/lib/python3.7/site-packages/androidtv/androidtv.py", line 100, in update
    screen_on, awake, wake_lock_size, media_session_state, _current_app, audio_state, device, muted, volume_level = self.get_properties(lazy=True)
  File "/srv/homeassistant/lib/python3.7/site-packages/androidtv/androidtv.py", line 374, in get_properties
    if not self.max_volume_level:
AttributeError: 'AndroidTV' object has no attribute 'max_volume_level'

With new init.py file:
019-03-15 14:48:34 ERROR (MainThread) [homeassistant.loader] Error loading homeassistant.components.androidtv. Make sure all dependencies are installed
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/loader.py", line 166, in _load_file
    module = importlib.import_module(path)
  File "/usr/local/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/androidtv/__init__.py", line 6, in <module>
    from .androidtv import AndroidTV
ModuleNotFoundError: No module named 'homeassistant.components.androidtv.androidtv'
2019-03-15 14:48:34 ERROR (MainThread) [homeassistant.loader] Error loading homeassistant.components.androidtv.media_player. Make sure all dependencies are installed
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/loader.py", line 166, in _load_file
    module = importlib.import_module(path)
  File "/usr/local/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/androidtv/__init__.py", line 6, in <module>
    from .androidtv import AndroidTV
ModuleNotFoundError: No module named 'homeassistant.components.androidtv.androidtv'
2019-03-15 14:48:34 ERROR (MainThread) [homeassistant.loader] Unable to find platform androidtv.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- platform: androidtv
  host: !secret androidtv_hostname
  adb_server_ip: <IP_ADDRESS>
  name: MiBox

Looks like you did not copy the files properly. You can use the files in the new PR https://github.com/home-assistant/home-assistant/pull/22072 in custom components.

Yes, my mistake. Now I use androidtv==0.0.12 and media player entity isn't initiated.
2019-03-15 15:56:41 WARNING (SyncWorker_2) [homeassistant.components.androidtv.media_player] Could not connect to MiBox at androidtv.bieniu.lan:5555 using ADB server at <IP_ADDRESS>:5037

In syslog:
Mar 15 15:56:41 raspberry hass[30595]: 15-03-2019:15:56:41,64 DEBUG    [connection.py:24] Connect to adb server - <IP_ADDRESS>:5037
Mar 15 15:56:41 raspberry hass[30595]: 15-03-2019:15:56:41,71 DEBUG    [connection.py:24] Connect to adb server - <IP_ADDRESS>:5037

I recall seeing similar errors. Restart typically helps.


feat: Improved way of creating migs and networks.

What?

Improved way of creating migs and networks.
Updated readme.

Why?
The code for creating the mig and network resources was static, needing to create new variables and copy the code if it was necessary to create more of these resources. Changed to make it more dynamic.
Testing ?
Integration and Lint tests were performed.
Anything Else?
None.

/gcbrun


Added functions for deep searching

Makes it easy to filter through descendants (as opposed to just an element's immediate children)

Thanks @noahcgreen, this is now merged in version 4.2.0.


Error loading libclang-cpp.so

I followed the instructions here for Ubuntu 22.04: https://mull.readthedocs.io/en/0.21.0/Installation.html
When running mull-runner-14 --version, I get:
/usr/bin/mull-runner-14: error while loading shared libraries: libclang-cpp.so.14: cannot open shared object file: No such file or directory

I've installed libclang-cpp-dev and the file exists at /usr/lib/llvm-14/lib/libclang-cpp.so.14. I've tried exporting LD_LIBRARY_PATH and creating symlinks, but nothing works.
Am I missing steps, and if so, can these steps please be added to the installation instructions?

The problem in my case was trying to run on ARM64.
However, libclang-cpp# should be added as a dependency of mull-# to ensure that lib clang-cpp.so.# is installed.


cannot install on ubuntu 16.04.1

Cookbook version
8.0.1
Chef-client version
Chef Development Kit Version: 0.16.28
chef-client version: 12.12.15
delivery version: master (921828facad8a8bbbd767368bfc72f19bd30e7bd)
berks version: 4.3.5
kitchen version: 1.10.2
Platform Details
ubuntu 16.04.1
vagrant
vagrant@default-ubuntu-1604:~$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.1 LTS
Release:	16.04
Codename:	xenial
kitchen.yml reads
platforms:

name: ubuntu-16.04

Scenario:
install mysql-server on ubuntu
Steps to Reproduce:
mysql_service 'default' do
version '5.7'
action [:create, :start]
end
Expected Result:
install sys default currently at 5.7.13-0ubuntu<IP_ADDRESS>
Actual Result:
---- Begin output of apt-get -q -y install mysql-server-5.7=5.7.12-0ubuntu1.1 ----
STDOUT: Reading package lists...
Building dependency tree...
Reading state information...
STDERR: E: Version '5.7.12-0ubuntu1.1' for 'mysql-server-5.7' was not found
---- End output of apt-get -q -y install mysql-server-5.7=5.7.12-0ubuntu1.1 ----
Ran apt-get -q -y install mysql-server-5.7=5.7.12-0ubuntu1.1 returned 100

possibly dupe of
Incorrect default package versions #439

Fixed up in 8.0.2


Failed to run verify-system-status when executing chef-server-ctl reconfigure

Failed to run [verify-system-status] when executing "chef-server-ctl reconfigure", but when calling " http://<IP_ADDRESS>:8000/_status" by firefox, it returns "{"status":"pong","upstreams":{"chef_solr":"pong","chef_sql":"pong"}}"
error log:
Expected process to exit with [0], but received '22'
---- Begin output of curl -sf http://<IP_ADDRESS>:8000/_status ----
STDOUT:
STDERR:
---- End output of curl -sf http://<IP_ADDRESS>:8000/_status ----
Ran curl -sf http://<IP_ADDRESS>:8000/_status returned 22

Hi,
if you can still reproduce this issue, please open a new issue on chef-server at https://github.com/chef/chef-server/issues . Please can you include the version of chef-server that you're installing and what topology and so on.
Thanks!


Update FallCreatorsUpdate.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/release-notes/commercial/FallCreatorsUpdate.md
:white_check_mark:Succeeded

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.


[FEATURE]: Update validation for hackathon

Detailed description
I would like to update the hackathon validation schema.
Context
As the database structure for hackathon has changed we need to update the backend validation for hackathon according to the hackathon validation schema.
Possible implementation
No response
Additional information
I would like to work on it under gssoc23
Code of Conduct

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

@suzy-g38 assigned the issue. good luck!


[Bug]: Incomplete bot transcript issue

Brief Description
Hi Team,
I have recently upgraded to vocode latest version but I am facing a issue where bot transcriptions are incomplete, it only shows the first word of the whole sentence in the transcript. Human transcripts are seems to be fine at other end.
I am using deepgram as transcriber, gpt-4o for agent and elevenlabs websocket synthesizer for TTS.
for example:
BOT: Hello
HUMAN:  Hello sir I am xxx person from  yyy company
BOT: Hello xxx,  you are talking from yyy, Can I know  -
HUMAN:  Sir for the whole day hello hello
BOT: yes
HUMAN:  Hello. Sir is there any requirements for cargo and courier services ?
BOT: Right now
HUMAN:  Sorry?
BOT: Any
HUMAN:  No sir, thank you.
BOT: okay
And the actual conversation after transcribing the recording with deepgram using same model is given below, differences are highlighted in the bold. I have replaced the actual names with xxx, yyy and zzz.
BOT: Hello
HUMAN:  Hello sir I am xxx person from  (yyy company)
BOT: Hello (xxx person),  you are talking from (yyy company), Can I know  -
HUMAN:  Sir for the whole day hello hello
BOT: yes, tell me xxx, what you want to talk about
HUMAN:  Hello. Sir is there any requirements for cargo and courier services ?
BOT: Right now (zzz person) is busy, do you want to leave a message
HUMAN:  Sorry?
BOT: Any message you want to give to  (zzz person)
HUMAN:  No sir, thank you.
BOT: okay, thank you
LLM
ChatGPT
Transcription Services
Deepgram
I have used hi-IN language model from deepgram nova-2 general model
Synthesis Services
Eleven Labs
Telephony Services
None
Conversation Type and Platform
No response
Steps to Reproduce
The issue is coming whenever there is interruption in the conversation by the caller, the bot transcript is showing incomplete.
Expected Behavior
The bot transcript should be complete.
Screenshots
c4922829-9461-4ebd-bc77-a9a80d92622b_r.zip
Please find the attached conversation recording for your reference.

I have tried to debug this a bit further. I think its because of this code: https://github.com/vocodedev/vocode-core/blob/main/vocode/streaming/streaming_conversation.py#L494
Only in case of maybe_synthesis_result is None if its not first token, hence only the first token is getting added into transcript.

thanks for the report! is this on latest master or on vocode 0.1.113?

I am using vocode 0.1.113

Found the root cause. This issue seems to be fixed in following commit: https://github.com/vocodedev/vocode-core/commit/60d21879d76e21a85bd25f9e607f3322eb3d3c45#diff-b086df98bf635ee6a71718985b64311e895fb6a750e1b83f633b6c39baed34c9L958

Awesome! Yes - the ElevenLabs websocket synthesizer had some problems with interruptions / transcripts which recently got fixed in the output device refactor. We're looking to push this to 0.1.114 soon, but the interim we have it in prerelease: https://pypi.org/project/vocode/0.1.114a0/


update oauth2-lib to 1.4.0

Codecov Report

Merging #358 (02c3707) into main (57fe6cf) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #358      +/-   ##
==========================================
- Coverage   72.85%   72.75%   -0.10%
==========================================
  Files         183      183
  Lines        8782     8781       -1
  Branches     1439     1438       -1
==========================================
- Hits         6398     6389       -9
- Misses       2197     2207      +10
+ Partials      187      185       -2

Files Changed
Coverage Δ

orchestrator/security.py
85.71% <100.00%> (-0.96%)
:arrow_down:

... and 6 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix chan creation when first message is otr msg

Fixes #5

I'm pretty much OK with this PR, good job! One comment before I give my :+1:: it seems odd that startQuery is in the Client's prototype while getQuery is in the Network's prototype.
Shouldn't/Couldn't we make this nicer by bundling these similar functions together?
(One could DRY up a bit more - too much maybe - by adding createOrStartQuery after that, but that's not the main concern of my comment)

I'm pretty much OK with this PR, good job! One comment before I give my :+1:: it seems odd that startQuery is in the Client's prototype while getQuery is in the Network's prototype.
Shouldn't/Couldn't we make this nicer by bundling these similar functions together?

Yep, I do agree, my rationale was :

the logic would like that both of those methods are supposed to be encapsulated inside Network
…but startQuery() requires to emit an event from client, which you can't (or you don't want to) do from Network.

A complicated solution would be to add events to network, but it escapes the scope of the PR, as it would require to touch more of the code to remain consistant. Furthermore, that will be a lot of event handling for a small quantity of code (= harder to read & debug).
So I'm going to move both methods to Client, simply.

@astorije ok with that ?

A complicated solution would be to add events to network, but it escapes the scope of the PR, as it would require to touch more of the code to remain consistant. Furthermore, that will be a lot of event handling for a small quantity of code (= harder to read & debug).

 I also noticed that after a certain point, events are the trash bin when you don't know where to put something because you have to keep design and encapsulation in mind.
Anyway, yes, I :+1: :-)


For the HPM event selectors, clarify what is an "event type", ...

... and how one is supposed to determine which event types do and do not support filtering by ID.
Section 5.23, "Performance-monitoring event selector", says

When a event type that does not support filtering by IDs is programmed then the associated counter does not increment.

What exactly is an event type?  If they are listed in Table 16, "Standard Events list", then which of those don't support filtering by IDs?
(And if E is an eventID code that doesn't support filtering by IDs, then how would anyone ever count those events, if the counter "does not increment"?  It seems those eventID codes must all be the same as 0, Do not count.)

It should read "filtering by Filter ID type"

PR #126


Mismatched options.watchOptions in main webpack function

Do you want to request a feature or report a bug?
BUG

What is the current behavior?
When options is an array, the check for options.watchOptions looks at all elements in the array, but then the internal logic only references the first element in the array.
(see https://github.com/webpack/webpack/blob/master/lib/webpack.js#L42 as compared to line 43.
If the current behavior is a bug, please provide the steps to reproduce.
To reproduce bug: Call webpack with an options array where the second or later object contains watchOptions, while the first does not.

What is the expected behavior?
compiler.watch should be called with the correct watchOptions
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Found during code review.

(cc @zokioki)

Yep, this could be solved in a better way. Do you want to do a PR?
MultiCompiler.watch should also accept an array of watch options which are than passed to the Compilers.
bin/webpack should pass all watchOptions to the MultiCompiler.watch call.

Sure, either @zokioki or I will take a pass on this.


Treat WITH: parameters as text, otherwise breaks parser

The following yaml breaks the parser;
  - gather-service-artifacts:
      with:
        SERVICE: insurance
        SERVICE_REGEX: [i]nsurance

This does not;
  - gather-service-artifacts:
      with:
        SERVICE: insurance
        SERVICE_REGEX: "[i]nsurance"

Parameters in the WITH: clause of a script definition should be treated as a string literal, otherwise any contents are evaluated as part of the yaml document.
Maybe we need to validate and escape any literal string before parsing the yaml config?

This also works correctly;

gather-service-artifacts:
with:
SERVICE: insurance
SERVICE_REGEX: [i]nsurance

As discussed with @willr3  SERVICE_REGEX: [i]nsurance should result in a parsing error

Resolved


feat: provide convenience access to GitHub API

Checklist for contributing

[ ] refer to a valid ticket id in the PR title (if applicable)
[x] use proper Angular commit format for semantic release
[x] provide a brief summary for the proposed changes here,
to a failing service

Summary
Provide some convenience functionality for using Github, oAuth and handling tokens via environment

:tada: This PR is included in version 1.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[RNAdam-60] Remove non-ADAMContext *Context references.

Resolves #60. Upgrades to Spark 1.2.0 and Avro 1.7.4 in the process.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/RNAdam-prb/67/
Test PASSed.


Update the ingress RBAC for allowing leader election

Back porting the #2813 to 1.21 .
This is needed due to the 1.0.5 update.

LGTM, Users already affected would need to disable/enable the ingress addon for this to work, right? Is this acceptable?

Unfortunately, this is the case yes. As with the ingress version update the only way to get the update is to re-apply the manifest.


fix: @W-8612427@ load project, if available, to establish loginurl

What does this PR do?
Corrects the behavior to see if the command is being run from a sfdx project dir and then load the project json to acquire login URL if present
What issues does this PR fix or reference?
None

@amphro I had to change the logout tests. The spy on config.unset count had to be cut in half for ALL occurrences. Not sure why.


UglifyJS error on Production

When i run prod/production i get:
ERROR in js/app.js from UglifyJs
Unexpected token: name (Pilot) [js/app.js:111,8]
It looks like there are some solutions here, i havent tried any yet.
https://github.com/joeeames/WebpackFundamentalsCourse/issues/3
Any advice?

I have tried a common solution of adding this to the .babelrc file but it doesn't work

{
"presets": [
["es2015", {"modules": false}]

]
}

On further digging, it looks like babel is not transpiling ES6 -> ES5. My final public build files still have the "let" instead of "var". any ideas?

There is a small error in app.config.js file. We picking up a javascript directory instead of js
Please change in 18 line of your build/app.config.js to this one:
javascript: path.resolve(__dirname, '../resources/assets/js'),

I'm gonna address this issue immediately. Thanks for catching this up!

Did that solve the problem?

Yes! Thank you


Use one Python

Currently pkg-config is using python as a build dependency. As a result conda-smithy in conjunction with conda-build-all generates a matrix of python versions. However, pkg-config doesn't depend on python at run time. So this is a waste of resources. We should pick one python and re-render so as to fix this.

Fixed in PR ( https://github.com/conda-forge/pkg-config-feedstock/pull/20 ).