In author edit view if page is refreshed, 'Add new component' wizard appears without xblock buttons

In author edit view if page is refreshed, 'Add new component' wizard appears without xblock buttons to add child xblocks.

This is for Assessment XBlock

Hi @vikrantuk , thanks for the report.  First, note that we don't use these xblocks in studio; we only use the apis, so it's unsurprising that the studio view may be buggy.
To help debug this, could you share any interesting messages/errors from the console?  Also, does it happen only on certain xblock pages?  This may be an issue with studio itself too.


Account management (work in progress)

Account management flows.  Things are quite ugly and not finalized both UI-wise and code-wise, but just getting this out there.
  Before I do a deep code review, I loaded this up on my device to play around.  A few things:

When I change my name, the name saved in SmartLock does not change.  Should we update the credential?
When I go to change my name (purple theme) the text color in the EditText is all white which makes it impossible to read, have to type blind.
Tried changing email but keep getting "this email address isn't correct".  Not sure about that error message, what does it mean by "not correct"?  Does it not belong to me?  Is it invalid format?

@amandle as discussed yesterday you won't have time to come back to this in the near-term so how about this plan:

Address the few comments that I left in first pass
Get the build to pass
Merge it into a feature branch, we can pull it back out when you're ready to come back to it.

What do you think?

Resolved review feedback, and got the build passing. The branch is now pushed to https://github.com/amandle/FirebaseUI-Android/tree/2.0.0-account_management. Closing this PR. Development will continue on the branch.

@amandle thank you for handling that


Help with OpenAPI account

Hi,
could someone tell me how to create an account for the OpenAPI?
Does only one installer have permission to create it?
Are these the correct steps?
https://www.youtube.com/watch?v=L7BOo96cLaw
If I have to ask the installer, I have to tell him exactly what to do.
Thanks

Thx to @aretokas the documentation has been updated just a few moments ago.
You can find all steps required at https://github.com/tijsverkoyen/HomeAssistant-FusionSolar#northbound-api--openapi

Thanks

Hi again,
I need to change this url?

But, by default, is it the correct url?
Because the login fails and I don't know the reason.

Your url depends on your installation.
You should use the url that you use to login on the web interface.


geospatial: document gr2

There are many different geospatial data types. We won't support all of them natively by default. However, there are tools which can convert popular data types to those we do support. We should focus on adding documentation for these tools early on in out documentation journey.
ogr2ogr - allows conversion of Geopackage and Geodatabase data.
osm2pgsql - allows import of OSM data

@rmloveland i added this issue for early prioritization once @otan finishes the work to add support here

note: for anything involving PGDUMP that results in

root@:26257/defaultdb> import pgdump 'nodelocal://0/census.sql'
;
ERROR: postgres parse error: line too long

run with increased size: https://www.cockroachlabs.com/docs/v20.1/migrate-from-postgres.html#max-row-size


Replace distutils.version.StrictVersion with packaging.version.Version

Closes #32
This replaces distutils.version.StrictVersion with packaging.version.Version, since distutils is going to be removed in Python 3.12.
Signed-off-by: Christophe Bedard<EMAIL_ADDRESS>

Testing --packages-above tracetools_trace:

Linux
Linux-aarch64
Windows


X.509 HTTP client example

Describe the solution you'd like
An example shown how to send a HTTP request with X.509 certificate. Like this curl command:
curl --cert < certificate> --key < device certificate key pair> -H <HTTP header> --cacert <CA> <URI>

That depends on the TLS library you may use with hyper (it can be any). But I see you also opened an issue in the reqwest repo, so I'll close this one.


Implement "New Papers" list on the homepage

Implement the list that displays the newest created 10 papers, with rating and title. Base on this mockup: https://drive.google.com/open?id=0B6Ost2dc4IjFWExZbXNyYVJWRTA

working on it

Implemented. There may be some color issues, it may not look that great on smaller device sizes, but this is not probably going to be the final design for how papers are summarized so I'm not going to spend a bunch of time polishing it.


Image: Add static is_format_compressed function.

Adds a new static function to Image for detecting whether a format is compressed, as well as replaces instances of format >= Image::FORMAT_DXT1 and format > Image::FORMAT_RGBE9995 with it across the codebase.
This is done mainly to simplify code, but also for futureproofing in case the Format enum ever gets changed.

This is somewhat related to https://github.com/godotengine/godot/pull/88669/files , since that PR heavily expands upon is_compressed and "asks a question" about the format.
Makes you wonder if it would be better to have those as a static methods, or if exposing is_format_compressed to the public API is even desirable. @Calinou

Thanks!


Views + Reactions for blog articles

Hi,
This great repo here
https://github.com/braydoncoyer/braydoncoyer.dev
tracks views using plausible.io and reactions using react-share in Supabase,
how can we implement something similar?

This feature request is outside of the scope of the boilerplate. I'm planning on scaling things back quite a bit, so adding more features like this would be moving in the wrong direction for the project.
You're more than welcome to fork it and implement it yourself, though 😃


Add A Settings Page

Description of the new feature/enhancement
There doesn't appear to be any way to change the settings from the GUI.
The settings button opens a generated json file and not the actual settings file.
Provide a UI to change the settings.
Proposed technical implementation details (optional)
Its fine to store the settings in a json file and allow people to edit it as an advanced option.
However, provide a UI for people to click on and change common settings easily.
e.g.

Tick a button to set my default shell
Select from a drop down the font I want to use
etc etc

Don't just pop up the json file to edit though.

This week's Windows Terminal Preview v1.6.10272.0 already has a settings UI. Users have been finding bugs in it today.

This week's Windows Terminal Preview v1.6.10272.0 already has a settings UI. Users have been finding bugs in it today.

Yea, please search before filing new issues.
/dup #1564

Yea, please search before filing new issues.
/dup #1564


Add host[:port] property in appengine-web.xml to tell build server app host url when deploy it on other GAE implementation such as AppScale.

Add host[:port] property in appengine-web.xml to tell build server app host url when deploy it on other GAE implementation such as AppScale.
Signed-off-by: wanddy<EMAIL_ADDRESS>

Thanks!
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sat, Jan 17, 2015 at 8:15 PM, wanddy<EMAIL_ADDRESS>wrote:

...host url when deploy it on other GAE implementation such as AppScale
Signed-off-by: wanddy<EMAIL_ADDRESS>You can view, comment on, or merge this pull request online at:
https://github.com/mit-cml/appinventor-sources/pull/225
Commit Summary

Add host[:port] property in appengine-web.xml to tell build server
app host url when deploy it on other GAE implementation such as AppScale

File Changes

M
appinventor/appengine/src/com/google/appinventor/server/project/youngandroid/YoungAndroidProjectService.java
https://github.com/mit-cml/appinventor-sources/pull/225/files#diff-0
(13)
M appinventor/appengine/war/WEB-INF/appengine-web.xml
https://github.com/mit-cml/appinventor-sources/pull/225/files#diff-1
(2)

Patch Links:

https://github.com/mit-cml/appinventor-sources/pull/225.patch
https://github.com/mit-cml/appinventor-sources/pull/225.diff

—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/225.

The logic here looks wrong. If you set the default value of appengine.host to "localhost:8888" then it will use it, even in production (because you test with appengineHost.get()==""). Perhaps the default should be the empty string both in the code and in the appengine-web.xml file. A comment in appengine-web.xml can explain how to use it.
-Jeff

I have reviewed my code and configuration,and test on Appscale and GAE,it works fine by setting appengine.host or remain blank.

@wanddy Sorry for the delay, LGTM. Currently squashed and submitted to Gerrit. Should be merged shortly.

@wanddy Note: I fixed some whitespace issues

MERGED


Upgrades sbt-org-policies to fix the microsite

Please @juanpedromoreno could you review? Thanks

Outdated pull request, closing


Possible violation of YouTube Terms of Service

Hello. Are you sure your player can use custom controls? I found this document
https://developers.google.com/youtube/terms/required-minimum-functionality#overlays-and-frames
where you can find this paragraph: “You must not display overlays, frames, or other visual elements in front of any part of a YouTube embedded player, including player controls. Similarly, you must not use overlays, frames or other visual elements to obscure any part of an embedded player, including player controls.”

Hi, that shouldn't be an issue because the player controls are not always visible, they automatically disappear after a while.
The player has been used by published apps for years now, it's quite safe.

Yeah, I understand your concerns.
But considering the state of the official API this is pretty much the only way to play YouTube videos on Android. Also there's tons of apps on the store using a webview-based player with custom controls, even some very popular apps.
Unfortunately that's the best we can have for now 😅


[CI] Disable Codecov annotations on Github PRs

What does this PR do?
This PR disables Codecov annotations on Github PRs.

Motivation
It makes it hard to read for developers.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge

/merge


update common-jvm version

removes the requirement that root certs must have AKIDs to be trusted.

This change is


Error loading elements for freedesktop-sdk

See original issue on GitLab
In GitLab by [Gitlab user @traveltissues] on Sep 9, 2019, 15:33
Summary
Buildstream fails to load elements in the process of setting up the local cache quota. Affects buildstream master.
Steps to reproduce

clone freedesktop: git clone https://gitlab.com/freedesktop-sdk/freedesktop-sdk and checkout 973d7bb340e605a2a143f2b1d3f2fdbf286dfd77
apply https://gitlab.com/snippets/1893382
Setup a bst-artifact-server
bst build bootstrap/linux-headers.bst
fails to load elements: https://gitlab.com/snippets/1893394

In GitLab by [Gitlab user @jennis] on Sep 9, 2019, 15:46
Which version of BuildStream are you using?
I've been able to load "desktop.bst" and all of its dependencies with BuildStream master on the freedesktop branch "freedesktop-sdk-<IP_ADDRESS>-bst2"

In GitLab by [Gitlab user @jennis] on Sep 9, 2019, 15:46
Which version of BuildStream are you using?
I've been able to load "desktop.bst" and all of its dependencies with BuildStream master on the freedesktop branch "freedesktop-sdk-<IP_ADDRESS>-bst2"

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 10:30
What distro are you running and is this on x86-64 or a 32-bit architecture? multiprocessing.Value('q') is supposed to work on all systems with long long, which should include anything remotely modern.

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 10:30
What distro are you running and is this on x86-64 or a 32-bit architecture? multiprocessing.Value('q') is supposed to work on all systems with long long, which should include anything remotely modern.

In GitLab by [Gitlab user @jennis] on Sep 10, 2019, 10:46
I've just been able to reproduce this on Debian Stretch and on x86-64 architecture

In GitLab by [Gitlab user @jennis] on Sep 10, 2019, 10:46
I've just been able to reproduce this on Debian Stretch and on x86-64 architecture

In GitLab by [Gitlab user @BenjaminSchubert] on Sep 10, 2019, 10:51
Would that be due to a specific python version then? Stretch is 3.5 correct?

In GitLab by [Gitlab user @BenjaminSchubert] on Sep 10, 2019, 10:51
Would that be due to a specific python version then? Stretch is 3.5 correct?

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 10:56
Possibly, yes.
As this code is only used by the interactive frontend, it's not currently covered by our test suite. I'll write a test to exercise this code without the interactive frontend and then we can check what distros it fails on in CI.

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 10:56
Possibly, yes.
As this code is only used by the interactive frontend, it's not currently covered by our test suite. I'll write a test to exercise this code without the interactive frontend and then we can check what distros it fails on in CI.

In GitLab by [Gitlab user @jennis] on Sep 10, 2019, 10:57
Correct. I produced this with Python 3.5.3

In GitLab by [Gitlab user @jennis] on Sep 10, 2019, 10:57
Correct. I produced this with Python 3.5.3

In GitLab by [Gitlab user @traveltissues] on Sep 10, 2019, 11:03
yes to all the above

In GitLab by [Gitlab user @traveltissues] on Sep 10, 2019, 11:03
yes to all the above

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 16:14
mentioned in commit 71a96ab998b90b1675ac7df14872e44f662d9d24

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 16:14
mentioned in commit 71a96ab998b90b1675ac7df14872e44f662d9d24

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 16:16
mentioned in merge request !1597

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 16:16
mentioned in merge request !1597

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 17:09
mentioned in commit 505316c6c27478711a0cc68edb5407dce0a81e8e

In GitLab by [Gitlab user @juergbi] on Sep 10, 2019, 17:09
mentioned in commit 505316c6c27478711a0cc68edb5407dce0a81e8e

In GitLab by [Gitlab user @marge-bot123] on Sep 10, 2019, 18:08
closed via merge request !1597

In GitLab by [Gitlab user @marge-bot123] on Sep 10, 2019, 18:08
closed via merge request !1597

In GitLab by [Gitlab user @marge-bot123] on Sep 10, 2019, 18:08
mentioned in commit 3eedd7a7a5db75650085275c63596a54f412feab

In GitLab by [Gitlab user @marge-bot123] on Sep 10, 2019, 18:08
mentioned in commit 3eedd7a7a5db75650085275c63596a54f412feab

In GitLab by [Gitlab user @juergbi] on Sep 18, 2019, 14:51
mentioned in commit c3413a71b0ed9404c4e620d1ad715135b1cf8451

In GitLab by [Gitlab user @juergbi] on Sep 18, 2019, 14:51
mentioned in commit c3413a71b0ed9404c4e620d1ad715135b1cf8451


Separate webhook and API definition

Feature Request
More of a debate and potential feature request than an outright feature request
Describe the problem you need a feature to resolve.
We have multiple operators that import APIs between each other. And we import the APIs of other open operators or operators written by separate internal teams. This causes a dependency version nightmare, because a long with importing the go API we have to import the webhook code.
The problem with this is whilst the go API defn is very structured and at most will import other APIs, such as the meta API etc. An operators webhook code may import a wide range of other go modules. For example older operators imported
"sigs.k8s.io/controller-runtime/pkg/runtime/log" in controller-runtime 0.6.3 which is removed in favour of "sigs.k8s.io/controller-runtime/pkg/log" in controller-runtime 0.8.3. This means when I import the API of an operator that has not upgraded their controller-runtime dependency. I am also stuck on 0.6.3 of controller-runtime. Even though that import has nothing to do with the API I want and is only there for their webhook code, which I did not need.
Now arguably this problem exists regardless of whether the webhook code is in the API package or not as you could cause the same problem with a broken change in an API import. Though to me at least that feels a lot less likely.
If we consider the kube APIs they are completely separated (thinking about, core, meta, apps etc.)
Additionally, though it's not something I have done yet. Having the webhook defined where it is feels like it makes setting up a webhook that manages multiple API versions more complicated. Though this could be because I haven't done it yet, so am just missing something
Describe the solution you'd like.
A debate around what best practise is here?
Whether we should consider separating webhook code by default?
/language go

I recommend defining your shared APIs in separate repo(s), similar to how operator-framework has the api repo https://github.com/operator-framework/api.

Ah, that's an interesting thought. You're right that is probably the best solution. I guess the problem is you never really know when someone is going to want to import and use your api as part of some larger solution. But operator-sdk also clearly can't default to generating two separate projects. So maybe there is not a lot we can do here. Thanks for the thought though. I'm happy to close this one out if you are

Happy to help! As a rule of thumb, when dependencies become an issue it's time to move shared code into a separate repo.
I will close this issue.


operator/ingress: Only TLS 1.1 or 1.2 can be used

Add a note to the IngressController documentation that the minimum and maximum TLS versions for ingress controllers are TLS 1.1 and TLS 1.2, respectively, and that the Modern TLS profile type therefore cannot be used.
This commit is related to bug 1763639 and bug 1763605.

operator/v1/types_ingress.go (IngressControllerSpec): Document restrictions on the TLS protocol version and profile type.
operator/v1/0000_50_ingress-operator_00-custom-resource-definition.yaml:
operator/v1/zz_generated.swagger_doc_generated.go: Regenerate.

/hold
Also need to note that the maximum is 1.3, and possibly state that the Modern profile type is not allowed.

I thought TLS 1.2 is the maximum until haproxy is built with OpenSSL 1.1.1?

I thought TLS 1.2 is the maximum until haproxy is built with OpenSSL 1.1.1?

Yes, sorry, my earlier comment had a typo.  I've update the PR.

/hold cancel

/lgtm


Update command should display date/time of latest patch

The command ned update should have a timestamp of the latest content update.

Added in: 5d3783f43d2a93d1beac9518c748a9673a104809.
From now on, I will update the date for this command whenever I push new content updates.


Grunt init results in unterminated string constant error

Issue Summary
Running grunt init in windows 8 with node version 0.12.4 results in an error and the command aborts.
Error:
Build failed.
File: ghost/templates/setup/three
Unterminated string constant (140:105)
SyntaxError: Unterminated string constant (140:105)
    at Parser.pp.raise (C:\Users\john\Code\Ghost\core\client\node_modules\ember-cli\node_modules\broccoli-es6modules\node_modules\esperanto\node_modules\acorn\dist\acorn.js:1745:13)
    ...
Warning: Command failed: C:\Windows\system32\cmd.exe /s /c "C:\Users\john\Code\Ghost\core\client\node_modules\.bin\ember build"
 Use --force to continue.

Aborted due to warnings.

Steps to Reproduce

npm install (should pass successfully)
grunt init
See failure in all its glory

Technical details:

Ghost Version: master
Client OS: Windows 8.1
Node Version: 0.12.4

regarding the repro, this seems to be specific to windows builds. I have seen a few other people bring it up in slack as well

@novaugust I figured as much. If this was happening on OSX and *nix builds I think there would've been more commotion.
Changing it to just one user no longer demos the functionality but it does remain written that you can add multiple users so we ok with that fix?


Tetra10 node ordering

Going over several gmsh files I noticed that all (simplex) cell types maintain the gmsh node ordering except for tetra10, which has the last two nodes flipped compared to Gmsh's Tetrahedron10. Is this intentional, and if so, is there a reference for node orderings of the several element types as employed by meshio?
Steps to reproduce:
Create tetrahedron.geo
// single element tetrahedron
Point(0) = {0,0,0,10};
Point(1) = {1,0,0,10};
Point(2) = {0,1,0,10};
Point(3) = {0,0,1,10};
Line(4) = {0,1};
Line(5) = {0,2};
Line(6) = {0,3};
Line(7) = {1,2};
Line(8) = {1,3};
Line(9) = {2,3};
Line Loop(10) = {4,7,-5}; // 0-1-2
Line Loop(11) = {4,8,-6}; // 0-1-3
Line Loop(12) = {5,9,-6}; // 0-2-3
Line Loop(13) = {7,9,-8}; // 1-2-3
Plane Surface(14) = {10};
Plane Surface(15) = {11};
Plane Surface(16) = {12};
Plane Surface(17) = {13};
Surface Loop(18) = {14,15,16,17};
Volume(19) = {18};

Generate tetrahedron.msh
$ gmsh -3 -degree 2 tetrahedron.geo
$ tail -2 tetrahedron.msh
15 1 3 4 2 6 10 7 5 9 8
$EndElements

Load in meshio
>> m = meshio.read('tetrahedron.msh')
>> m.cells[-1].data+1
array([[ 1,  3,  4,  2,  6, 10,  7,  5,  8,  9]])

As far as I know, meshio is following VTK's convention. For tetra10, it looks like this: http://web.mit.edu/calculix_v2.7/CalculiX/ccx_2.7/doc/ccx/node33.html
As you can see, nodes 8 and 9 are inverted compared to Gmsh's convention.

That explains, thank you very much for that reference! I'll close the issue then.

meshio uses a node ordering as seen in the Wiki.


update README

Codecov Report
Base: 33.24% // Head: 33.24% // No change to project coverage :thumbsup:

Coverage data is based on head (87cee48) compared to base (fc32b75).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   33.24%   33.24%
=======================================
  Files          63       63
  Lines         379      379
  Branches       37       37
=======================================
  Hits          126      126
  Misses        220      220
  Partials       33       33

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Avoid type ambivalence in Experiment.screen and Experiment.eyetracker

Current Behavior
At least Experiment.eyetracker can be None (if sampling_rate is passed in Experiment.__init__()). Experiment.screen should always be of type Screen, unless it is tampered with after initialization.
Details: https://github.com/aeye-lab/pymovements/pull/884#discussion_r1833448210
Expected Behavior
Experiment.screen should always be of type Screen, Experiment.eyetracker should alwys be of type EyeTracker.
Minimum acceptance criteria

Implement Experiment.screen and Experiment.eyetracker as read-only properties.

Remove unnecessary None check in from_asc() (see https://github.com/aeye-lab/pymovements/pull/884#discussion_r1833448210)

[x] I am running the latest version

[x] I checked the documentation and found no answer

[x] I checked to make sure that this issue has not already been filed

[x] I have provided sufficient information for the team

Hello, I tried to fix the issue
This is what I did:
Implemented read-only properties for Experiment.screen and Experiment.eyetracker to ensure they are always of the correct type. Removed unnecessary None check in the init method.
I also created pull request: https://github.com/aeye-lab/pymovements/pull/892

[!CAUTION]
Disclaimer: This fix was created by Latta AI and you should never merge before you check the correctness of generated code!
The fix provided by Latta AI might not be complete and it can serve as an inspiration.

This bug was fixed for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt fixing issues on your repository, you can block this account.

I labeled it nonessential as it's only protecting the user against themselves. feel free to raise the priority in case


Update ADOPTERS.md

Added Grovo to the list of adopters.

LGTM

Thanks, I'll merge this internally!

👍 thanks

Merged in 6e85b5f1be0ac4256d2135eeade070eb7228d419


⚠ Bofrost.de has degraded performance

In 51a7509, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 980 ms

Resolved: Bofrost.de performance has improved in 94c439f after 6 minutes.


Every call to a setter results in a call to refreshUrl

Every call to a setter results in a call to refreshUrl, which fetches the items again.
https://github.com/FlowingCode/RssItemsAddon/blob/685c5abf8cfb801386f0c7f06931b56426cd0ca2/src/main/java/com/flowingcode/vaadin/addons/rssitems/RssItems.java#L115

When a property changes, do not refreshUrl immediately. Instead, schedule a beforeClientResponse execution and track what is the currently scheduled execution, so that it can be cancelled when another property is modified during the same request.


Error generating Chargeback reports when storage fields are added to the report

I'm unable to generate Chargeback reports when the following fields are added to the report.
I can successfully generate Chargeback reports when other fields are added to the report.
I have seen this issue with SCVMM. I'll test with other providers and update this issue with the results.
Storage Allocated , Storage Allocated Cost
Here's a snippet from evm.log:
[----] E, [2017-01-19T14:57:46.543893 #2945:73f140] ERROR -- : [FloatDomainError]: Infinity  Method:[rescue in _async_generate_table]
[----] E, [2017-01-19T14:57:46.544229 #2945:73f140] ERROR -- : /var/www/miq/vmdb/app/models/chargeback_rate_detail.rb:62:in to_r' /var/www/miq/vmdb/app/models/chargeback_rate_detail.rb:62:in <='
/var/www/miq/vmdb/app/models/chargeback_rate_detail.rb:62:in block in find_rate' /opt/rubies/ruby-2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.1/lib/active_record/associations/collection_association.rb:92:in block in select'
/opt/rubies/ruby-2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.1/lib/active_record/associations/collection_association.rb:92:in select' /opt/rubies/ruby-2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.1/lib/active_record/associations/collection_association.rb:92:in each'
/opt/rubies/ruby-2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.1/lib/active_record/associations/collection_association.rb:92:in select' /opt/rubies/ruby-2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.1/lib/active_record/associations/collection_proxy.rb:110:in select'
/var/www/miq/vmdb/app/models/chargeback_rate_detail.rb:60:in find_rate' /var/www/miq/vmdb/app/models/chargeback_rate_detail.rb:84:in hourly_cost'
/var/www/miq/vmdb/app/models/chargeback_rate_detail.rb:236:in metric_and_cost_by' /var/www/miq/vmdb/app/models/chargeback_rate_detail.rb:26:in charge'
/var/www/miq/vmdb/app/models/chargeback.rb:71:in block (2 levels) in calculate_costs' /var/www/miq/vmdb/app/models/chargeback.rb:70:in each'
/var/www/miq/vmdb/app/models/chargeback.rb:70:in block in calculate_costs' /var/www/miq/vmdb/app/models/chargeback.rb:69:in each'
/var/www/miq/vmdb/app/models/chargeback.rb:69:in calculate_costs' /var/www/miq/vmdb/app/models/chargeback.rb:29:in block in build_results_for_report_chargeback'
/var/www/miq/vmdb/app/models/chargeback/consumption_history.rb:15:in block (2 levels) in for_report' /opt/rubies/ruby-2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.1/lib/active_record/relation/delegation.rb:38:in each'
/opt/rubies/ruby-2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.1/lib/active_record/relation/delegation.rb:38:in each' /var/www/miq/vmdb/app/models/chargeback/consumption_history.rb:14:in block in for_report'
/var/www/miq/vmdb/app/models/chargeback/consumption_history.rb:13:in each' /var/www/miq/vmdb/app/models/chargeback/consumption_history.rb:13:in each_cons'
/var/www/miq/vmdb/app/models/chargeback/consumption_history.rb:13:in for_report' /var/www/miq/vmdb/app/models/chargeback.rb:19:in build_results_for_report_chargeback'
/var/www/miq/vmdb/app/models/chargeback_vm.rb:45:in build_results_for_report_ChargebackVm' /var/www/miq/vmdb/app/models/miq_report/generator.rb:193:in _generate_table'
/var/www/miq/vmdb/app/models/miq_report/generator.rb:167:in block in generate_table' /var/www/miq/vmdb/app/models/user.rb:241:in with_user'
/var/www/miq/vmdb/app/models/miq_report/generator.rb:167:in generate_table' /var/www/miq/vmdb/app/models/miq_report/generator/async.rb:96:in _async_generate_table'
/var/www/miq/vmdb/app/models/miq_queue.rb:347:in block in deliver' /opt/rubies/ruby-2.3.1/lib/ruby/2.3.0/timeout.rb:91:in block in timeout'
/opt/rubies/ruby-2.3.1/lib/ruby/2.3.0/timeout.rb:33:in block in catch' /opt/rubies/ruby-2.3.1/lib/ruby/2.3.0/timeout.rb:33:in catch'
/opt/rubies/ruby-2.3.1/lib/ruby/2.3.0/timeout.rb:33:in catch' /opt/rubies/ruby-2.3.1/lib/ruby/2.3.0/timeout.rb:106:in timeout'
/var/www/miq/vmdb/app/models/miq_queue.rb:343:in deliver' /var/www/miq/vmdb/app/models/miq_queue_worker_base/runner.rb:106:in deliver_queue_message'

@miq-bot assign @isimluk
@miq-bot add_label bug, chargeback


Package breaks Laravel Notifications because of overwrites

Laravels database notifications use $table->uuid('id')->primary(); by default so using this package breaks them which results in SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'id' at row 1.
$table->char('id', 36)->primary(); solves the issue but having to change the default migrations is kinda off-putting.
<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreateNotificationsTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('notifications', function (Blueprint $table) {
            $table->uuid('id')->primary();
            $table->string('type');
            $table->morphs('notifiable');
            $table->text('data');
            $table->timestamp('read_at')->nullable();
            $table->timestamps();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('notifications');
    }
}

Any thoughts besides what already discussed in #14 how we could fix this problem?
@freekmurze do you have any input?

The easiest solution would be to add a little note in the readme with instructions on how to edit the default migration.
Let's do that first, maybe that is good enough for most.

I've added a little more explanation in the README: https://github.com/spatie/laravel-binary-uuid#a-note-on-the-uuid-blueprint-method
Sadly I don't see a better workaround for this, unless some changes are made in the core. These changes are in discussion over here: https://github.com/laravel/internals/issues/645


Feat: Delete build file after successful installation

Is your feature request related to a problem? Please describe.
I usually build pretty often so it's annoying when I have my builds lying around when there is a new one available. So it would be great if the old ones were just auto-deleted.
Describe the solution you'd like
After a successful build installation, just delete that build. This can be a setting in the settings that is toggled OFF by default because I'd assume most people would get confused if this was ON by default
Additional context
No response

Hi @ansh, thanks for your suggestion, we already store builds in a temporary path, so your computer automatically deletes them once in a while. One thing that we could do is add a button to the settings screens where users could manually clean the builds folder and delete all cached apps

Thanks for the quick response @gabrieldonadel
What do you mean you already store builds in a temporary path? Do you mean builds downloaded from EAS?
I was referring to local .ipa files that were used.


Double shears in the hotbar

LOL? Im going to close this

sorry, I misspelled it, no need to laugh xD

NP, im cloes the issue xD

I think it is not fixed, i ll confirm this


Canvas arrow labels visibility issue

When adding canvas items, connecting them with arrows, and adding a label on top of the arrow, the arrow doesn't have a gap and/or z-ordering is off and/or something else that makes it less readable.
Screenshot with Origami enabled:

Screenshot with default theme:

Otherwise the canvas looks pretty neat in multiple ways, so cheers for the tweaks!

sorry i never commented, that is an unfortunate side effect for a workaround to get arrow heads rendering over the card backgrounds. i had to pull this design for now, because i can't get it up to a level of stability i feel comfortable sharing :) hope to have a new version at some point when i can fully review canvas features (for now this also fixes #16, with special features as an opt in :))


Add debug attribute to NeuronPartialState

As per title.
Fixes: #235

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


"/data/data/com.termux/files/usr/bin/spotdl" line 5

How to fix this type of bug
spotdl -p https://open.spotify.com/playlist/37i9dQZF1DWYztMONFqfvX?si=SFcx5fKPS0OLBPgIgV4nEA
Traceback (most recent call last):
File "/data/data/com.termux/files/usr/bin/spotdl", line 5, in
from spotdl import cli
ImportError: cannot import name 'cli' from 'spotdl' (/data/data/com.termux/files/usr/lib/python3.8/site-packages/spotdl.py)

Are you sure you are using the right spotdl package? Because
$ grep -rnw "from spotdl import cli"

returns nothing.

hey @karim4648, please try to keep your issue titles descriptive because it really helps handling of the many issues we get -"/data/data/com.termux/files/usr/bin/spotdl" line 5 doesn't tell us what the problem really is.
Something like - ImportError: cannot import name 'cli' from 'spotdl', Terminux is much more clear as to what the problem is.

same error

Well try v3. We aren't going to do anything here if you can't do something as small as rename an issue.


Distribute entire git repository artifacts

Hi,
Unless I'm mistaken the source controller exclude the .git dir from the artifacts it generates.
This way of doing this will limit the ability to make kustomize use local paths using git references when go-getter will have merged its support.
So I'm proposing an optional property for GitRepository which will allows to create artifacts containing all the git data.
Regards.

You can include any dir using spec.ignore check the docs https://toolkit.fluxcd.io/components/source/gitrepositories/#excluding-files

Brilliant! Sorry for the noise.


Tests for source install?

I've been trying to use the source installation, but i'm encountering a lot of issues.
Is this expected? Is the source install tested?

Hi. Support for source installs has been remove as of 3.0.0.
Closing this.
-s


Video streaming issue on  traffic server

Hi ,
We are using traffic server as video streaming CDN. We have 2 server each 1,4TB SSD and 10G network interface. During peak hour video stream hard to watch and buffering in client side. Server's load average and Memory are higher than normal state. But streaming bandwidth is around 3Gbps on all 3 server total. We wondering that we have potential only 3Gbps bandwidth on streaming but it has high load on traffic servers. Does those server can handle this load?
load average 106.94, 93.30, 88.32 as 32 core CPU
memory usage total 62GB used 62GB
established tcp connection is around 1200 on each server
traffic server's error log is increasing as following
20201115.16h42m02s CONNECT:[0] could not connect [CONNECTION_CLOSED] to <IP_ADDRESS> for 'http://localhost/vod/encrypt/prod/8a01918b72167aad01722a8007db243e/8a01918b72167aad01722a8007db243e_1500_2/media-88320000.mp4'
20201115.16h42m02s CONNECT:[0] could not connect [CONNECTION_CLOSED] to <IP_ADDRESS> for 'http://localhost/vod/encrypt/prod/8a01918b752cbbb7017577a9a1fc05ba/8a01918b752cbbb7017577a9a1fc05ba_1500/media-46260040.mp4'
20201115.16h42m02s CONNECT:[0] could not connect [CONNECTION_CLOSED] to <IP_ADDRESS> for 'http://localhost/vod/encrypt/prod/8a01918b752cbbb7017549e7cc523b37/8a01918b752cbbb7017549e7cc523b37_1000_2/media-220416000.mp4'
20201115.16h42m02s CONNECT:[0] could not connect [CONNECTION_CLOSED] to <IP_ADDRESS> for 'http://localhost/vod/encrypt/prod/8a01918b69817ebf016a951962ab7ccc/8a01918b69817ebf016a951962ab7ccc_1500/media-4513608.mp4'
Best Regards,

how about disk util,and how much MB every disk,if each sever around 1200 on each server,it is abort 1.2G,the disk may be over used

loadavg is a very broad metric, which includes CPU, memory wait, disk wait, disk usage, and potentially other things.
Can you look at specific metrics on your system, and see what specific things have high load? Is it just CPU? Memory? Disk io_wait? All of the above?
ATS will use as much memory as you tell it to. You can allocate ramdisks and give them to ATS as block devices. Each disk given to ATS also has a memory cache in front of it, the size of which is configurable.
See:
https://docs.trafficserver.apache.org/en/8.0.x/admin-guide/files/records.config.en.html#ram-cache
https://docs.trafficserver.apache.org/en/8.0.x/admin-guide/files/storage.config.en.html
https://docs.trafficserver.apache.org/en/8.0.x/admin-guide/files/volume.config.en.html
ATS does have some known memory leaks, but they're generally pretty small. It shouldn't use much more memory than what you allocated for storage and ram_cache, and the memory shouldn't grow much over time.
Many people run ATS in production with bandwidth much higher than 3Gbps. My company has caches doing in excess of 20Gbps. If you're having trouble achieving those speeds, another possibility is Linux Kernel Parameters. It's common to have to do a lot of tuning of Linux Kernel Parameters to achieve high performance. Though I wouldn't expect a great deal of tuning to be necessary under 10Gbps.
I assume this is somewhat recent hardware, with decent CPUs? We do have some Prod servers that struggle to exceed 10Gbps, from underpowered CPUs with few PCI lanes. Platforms with too few PCI lanes can also cause network bottlenecks like that.
 Some SSD brands have poor performance. It may be worth testing their sequential and random speeds, just to be sure that isn't the problem.

traffic server's error log is increasing as following
20201115.16h42m02s CONNECT:[0] could not connect [CONNECTION_CLOSED] to <IP_ADDRESS> for 'http://localhost/vod/encrypt/prod/8a01918b72167aad01722a8007db243e/8a01918b72167aad01722a8007db243e_1500_2/media-88320000.mp4'

I'm not sure I understand. Your initial question is about bandwidth bottlenecks and high loadavg, but this looks like an error? This looks like an origin (on localhost?) is misconfigured, or unable to handle the requests or load?
Are you saying you see a lot of these errors as you approach 3Gbps? That sounds like the Origin server isn't able to handle the load, that the problem is with the Origin, not ATS. Can you verify your Origin itself is capable of the request load?
Are these requests mostly Cache Hits or Misses? For a CDN, ATS should be caching I assume. Is the full traffic going to the Origin? Could that be causing the problem? Maybe the Origin can't handle the full 3Gbps, because everything is a Cache Miss, and you need to set Cache-Control to make ATS cache the content so the Origin can handle it.
Certain SSL Certificates can also cause high CPU usage, especially RSA. Are you using HTTPS?
In short, there are a huge number of factors that can cause bottlenecks like you're seeing. You'll have to narrow it down further, and inspect your hardware usage to figure out what the bottleneck is, and how to fix it. But ATS can definitely do +20Gbps, potentially even 100Gbps, and many large corporations are doing so in Production.

Please try to reproduce this in ATS 9.1, and reopen this issue if you still see it.


Cannot type into the SimpleList input in the simple mode, while it works in the advanced mode

Description
This PR introduced new UI for Subscriptions tab. There is a SimpleList defined as follows:
   - path: spec.types
      simple: true
      widget: SimpleList
      defaultExpanded: true
      children:
        - path: '[]'

Currently, in the prod Dashboard one cannot type anything in this input in the simple mode. Typing in the advanced mode works as expected.

Hi @VladislavPaskar
In this case SimpleList should be defined like this:
   - path: spec.types
      simple: true
      widget: SimpleList
      defaultExpanded: true
      children:
        - path: '[]'
           simple: true

Please check it and if problem appears please reopen the issue.


Non-identifier exports not detected?

I came across some unexpected behavior:
const cml = require('cjs-module-lexer')
cml.init().then(() => {
  const result = cml.parse(`module.exports = {fi, if: 0}`)
  console.log(result)
})

Expected:
{ exports: [ 'fi', 'if' ], reexports: [] }

Observed:
{ exports: [ 'fi' ], reexports: [] }

I expected this to work because ESM lets you do this:
import { if as foo } from './example.js'

Wait, maybe it doesn't have anything to do with keywords. It looks like mapping to null works but mapping to 0 doesn't? Here are some tests with various expressions:
module.exports = {fi, if: 0}
{ exports: [ 'fi' ], reexports: [] }

module.exports = {fi, if: !0}
{ exports: [ 'fi' ], reexports: [] }

module.exports = {fi, if: -0}
{ exports: [ 'fi' ], reexports: [] }

module.exports = {fi, if: (true)}
{ exports: [ 'fi' ], reexports: [] }

module.exports = {fi, if: [true]}
{ exports: [ 'fi' ], reexports: [] }

module.exports = {fi, if: null}
{ exports: [ 'fi', 'if' ], reexports: [] }

module.exports = {fi, if: true}
{ exports: [ 'fi', 'if' ], reexports: [] }

I guess this is because the grammar is specified as (IDENTIFIER (`:` IDENTIFIER )?). Is there any reason to not support arbitrary expressions? Like module.exports = { version: '0.1.0' }.

@evanw this is just a very simple lexer designed to be small and fast and it doesn't even have an expression parser. We could extend the grammar to support direct primitives in the object form, but for now I'm concerned that may be even more confusing too, it's not a clear line by any means yes. See also https://github.com/guybedford/cjs-module-lexer/issues/6.
Supporting numbers and strings would be straightforward, I'm just not sure we want to push the blurry line in that direction here, would value your thoughts.

The latest update has been upstreamed now, and the object form detection remains the same without string support. I'm open to further suggestions here, although the window is closing on the current set of updates. Closing for now but just let me know if we should reopen or make changes here further.

it doesn't even have an expression parser

Ah, this makes sense. I was mainly just surprised that support for this wasn't necessary to handle modules like that in the wild.
I recently shipped support for the 0 && module.exports = {...} pattern in esbuild btw. So far so good.


Compatibility with Laravel 9

Hi,
Currently, I can't upgrade my personal project from Laravel 8 to 9, because of the laravel-maps package, and one of its dependencies : illuminate/support.
By upgrading this dependancy from 8.0 to 9.0, it will allow to upgrade to Laravel 9.

Hey! Thanks for creating the issue. pls see my comment on your pr.


hedgedoc-cli: fix installPhase

When using the hedgedoc-cli script, it requires jq, wget and curl.
We need to make those derivation available in the script by using makeWrapper.
Description of changes

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.11 Release Notes (or backporting 22.05 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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 183398 run on x86_64-linux 1

1 package built:

hedgedoc-cli

Still missing some dependencies I guess
[nix-shell:~/.cache/nixpkgs-review/pr-183398]$ ./results/hedgedoc-cli/bin/codimd
/nix/store/qab58gqpia9lf082zkz4dy1wr32h3vpb-hedgedoc-cli-1.0/bin/.codimd-wrapped: line 374: $1: unbound variable

[nix-shell:~/.cache/nixpkgs-review/pr-183398]$ ./results/hedgedoc-cli/bin/hedgedoc-cli
/nix/store/qab58gqpia9lf082zkz4dy1wr32h3vpb-hedgedoc-cli-1.0/bin/.codimd-wrapped: line 374: $1: unbound variable

This is because of the script, you need to add the URL to the codimd/hedgedoc instance.


Adding cmake option to link against MariaDB (LGPL)

I have taken the liberty to work on issue #1. This solution adds the USE_MARIADB cmake option that, when set to 1, forces linking against MariaDB.
I have noticed that the cmake script for Windows already only links against MariaDB.
I would be glad if the changes could be considered. Please let me know if there is some way I can improve them.

Cool, thanks!


dex2jar: init at 2.0

Motivation for this change
Add dex2jar from my local NUR to nixpkgs. dex2jar is a tool requested in #81418
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build dex2jar

thanks for the review!

Result of nixpkgs-review pr 88425 1

1 package built:
- dex2jar


pythonPath not respected in remote container

Mirroring https://github.com/microsoft/vscode-remote-release/issues/1579
Environment data

VSCode Version: 1.38.1
Extension version (available under the Extensions sidebar): 2019.9.34911
Local OS Version:  Window 10 1903
Python version (& distribution if applicable, e.g. Anaconda): Python 3.5.4
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pipenv
Remote OS Version:  Ubuntu 18.04
Remote Extension/Connection Type: Docker

Expected behaviour
In "Remote Container" mode, I expect VSCode to respect the pythonPath setting and use the interpreter from the virtualenv.
Actual behaviour
The path is not respected, and only the system-installed interpreters of the docker container are shown in the VSCode interpreter selection drop-down.
Steps to reproduce:

Create a project with pipenv
In the folder settings.json point pythonPath to the python interpreter in the virtualenv (i.e. something like C:\Users\username\.virtualenvs\SomeProject-09hncfd\bin\python.exe
Create a custom Dockerfile based of Ubuntu 18.04 and build image (e.g. my-image:latest), in this image create a virtualenv based on the project's Pipfile using pipenv. Assume the virtualenv (in the image) is created at /root/.local/share/virtualenvs/SomeProject-1298hf
Setup remote like so:

"image": "my-image:latest",
"settings": {
  "terminal.integrated.shell.linux": "/bin/bash",
  "python.pythonPath": "/root/.local/share/virtualenvs/SomeProject-1298hf/bin/python",
  "python.linting.enabled": true,
  "python.linting.pylintEnabled": true,
  "python.linting.pylintPath": "/root/.local/share/virtualenvs/SomeProject-1298hf/bin/pylint"
},

Open VSCode remote, run the project.
An error message appears: "You need to select a Python interpreter ..."

yes, same issue here. workplace settings takes precedence over devcontainer's settings.json

Thanks for the bug report! We just wanted to quickly acknowledge we received it and we will be triaging it as soon as we can.

@sandy081 The pythonPath setting's scope is currently resource (presumably this can change per workspace folder). Should this become machine-overridable since it is a machine-specific path that needs to be overridable per workspace folder?

resource scope allow setting to be overridable at all levels (machine, folder).

Is there at least temporary workaround for this issue?

looking for a workaround here. any suggestion please ?

@jahan01  If you use the latest version of the extension (2020.5.78807) you should be able to enter the path directly there. Try this.

@karthiknadig i think we are talking different things here, i don't have issues with list of interpreters vscode detects. When switching from local to remote container development python.pythonPath in devcontanier.json value is overridden by the one in workspace settings which has my local interpreter path. So every time I switch between local and container development I have to reset python.pythonPath in workspace settings.json according to the environment.
However enabling "python.experiments.optInto": ["DeprecatePythonPath - experiment"] this seems to fix the issue, but introduced a new issue where I cannot use ${config:python.pythonPath} in my tasks.json to find the virtualenv interpreter for the project. It defaults to global python installation now, since the settings is removed from workspace.

@jahan01 Try ${config:python.interpreterPath} that is the replacement for ${config:python.pythonPath}.

@karthiknadig I had already tried that, it says
'${config:python.interpreterPath}' can not be resolved because setting 'python.interpreterPath' not found.

@karrtikr Can you help @jahan01 on this?

@jahan01 Filed #11789 to track the issue with interpreterPath

@jahan01 @karthiknadig We can only use ${config:python.interpreterPath} in launch.json. Can you paste that part of tasks.json where you use ${config:python.pythonPath}? What happens if you don't specify the python path at all.

Python Path setting will be deprecated. The python.pythonPath setting was stored in settings.json. This means that when you switched to a container/WSL, it would continue to use the path that was set in the settings.json. The new setting is stored at the machine level, this means you will have to select the interpreter once for your workspace in the host machine and once in the container or WSL and extension will remember this setting for that workspace on that machine/container/server.
This is still an experimental feature so you may run into bugs. You can opt into this by adding "python.experiments.optInto": ["DeprecatePythonPath - experiment"] to your user settings. Right now there is one issue with expanding ${config:python.interpreterPath} when used in task.json/settings.json.

@karthiknadig is the below setting still valid in devcontainer.json. i think you may still need some setting to mention default python location for containers.
// Set *default* container specific settings.json values on container create.
"settings": {
	"terminal.integrated.shell.linux": "/bin/zsh",
	"python.pythonPath": "/usr/local/bin/python",

https://github.com/microsoft/vscode-remote-try-python/blob/29a93447a860ca92d30d9bdfb4960db60668cd36/.devcontainer/devcontainer.json#L11-L14

@karthiknadig I seem to have the container overwriting the python.pythonPath setting in my settings.json file. This is annoying because i have it set properly in devcontainer/settings but it seems to prefer overriting my main settings.json file.
Do you know what I should do about that? You mentioned python.pythonPath is deprecated, what replaces it?

You can do using python.defaultInterpreterPath setting. It's a little different from python.pythonPath at workspace level. Check the wiki https://github.com/microsoft/vscode-python/wiki/AB-Experiments#tldr to understand how it works.
But if you have manually selected an interpreter in the dev container, that setting will not have any effect. You can try using Python: Clear Internal extension cache in that case.

Thanks, I have changed to using python.defaultInterpreterPath and it's the same problem. Once inside the container it is not respecting .devcontainer/settings/python.defaultInterpreterPath and is instead trying to use the one proposed at the top level .vscode/settings/python.defaultInterpreterPath.
It looks like this extension is not properly overriding settings from .devcontainer over .vscode.
I've done a screenshot to illustrate my

problem:

Ah I see, we'll try to have a look as soon as possible, expect by end of next week.

It looks like this extension is not properly overriding settings from .devcontainer over .vscode.

Hi so I tried it out, but the behavior does not seem to be just specific to python.defaultInterpreterPath or python extension related settings. It seems Workspace settings are always prioritized over Remote devcontainer settings for all settings (Personally I tried with "terminal.integrated.defaultProfile.linux" setting)
Can you name any settings for which devcontainer are prioritized over workspace? (.vscode/settings.json)

I see what you're saying, so maybe it's better to put the python.defaultInterpreterPath in my user settings instead of workspace settings and the problem may just solve itself. I assumed the precedence was User Settings -> Workspace Settings - > Dev Container Settings with dev container settings overriding everything before it. But now I see that's not the case. Sorry that wasn't obvious to me, I'm not sure why I thought that was the order.
Do you know why the extension is writing python.defaultPath into my .vscode/settings.json file? I thought this config was deprecated?  It only does it if it can't find the interpreter path and I need to choose one in the drop down.

so maybe it's better to put the python.defaultInterpreterPath in my user settings instead of workspace settings and the problem may just solve itself.

Yes.

Do you know why the extension is writing python.defaultPath into my .vscode/settings.json file? I thought this config was deprecated? It only does it if it can't find the interpreter path and I need to choose one in the drop down.

If no workspace is opened, python.defaultInterpreterPath setting works just like python.pythonPath did, so it'll write the selected interpreter path into it.
I'm closing this issue for the above reasons.


Create a new GitHub repo for the R package version

Create the package according to the steps laid out here: https://r-pkgs.org/whole-game.html

Push as a public repo in the UBC-MDS organization

Following issues related to the R package will be created on that repo.

Done, see: https://github.com/UBC-MDS/mercedestrenzr


Update maintainers

For the next release, remember to update the maintainers list removing my ID kinow :+1: (or before that too, but needs to release a new version/build)

Here's how to add a maintainer: https://conda-forge.org/docs/maintainer/infrastructure.html#conda-forge-admin-please-add-user-username
... but there seems to be no "remove user" directive

The "add user" docs say Please do not modify this PR or adjust the commit message. This PR is designed to skip building the package.
Other than avoiding a rebuild, maybe it merely updates the list of maintainers in the recipe:
# recipe/meta.yaml
 extra:
    recipe-maintainers:
     - hjoliver
     - kinow
     - oliver-sanders

Other than avoiding a rebuild, maybe it merely updates the list of maintainers in the recipe

I believe you will need to take the username out, then ask for a rerender (@conda-forge-admin, please rerender) and then do a rebuild, it will remove said username from everywhere in the repo after the build is completed and merged.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/cylc-flow-feedstock#32.

Ooops sorry, please close that rerender in #32 🤣

(Closed by #33)


windows compile, there has been a wrong" cannot find -lqtdrv"

windows compile, there has been a wrong：
c:> cd visualfc
c:\visualfc> go build
d:/program files/mingw/bin/../lib/gcc/mingw32/4.6.2/../../../../mingw32/bin/ld.exe cannot find -lqtdrv
Please help

I'm getting this error on Archlinux
go build: when using gccgo toolchain, please pass linker flags using -gccgoflags, not -ldflags
# github.com/visualfc/goqt/ui
/usr/bin/ld: cannot find -lqtdrv.ui
collect2: error: ld returned 1 exit status


refactor: change external optimus sensor task name similar with internal sensor name

Resolving https://github.com/odpf/optimus/issues/490

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 77.66%

Files with Coverage Reduction
New Missed Lines
%

ext/notify/pagerduty/pagerdutynotifier.go
2
93.75%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
7700

Relevant Lines:
9915

💛  - Coveralls


cordova resolveLocalFileSystemURI is deprecated. Please call resolveLocalFileSystemURL instead.

Bug Report
Problem
What is expected to happen?
Like previously the plugin was working fine and previous version of app is fine but when I am building app now the error comes
What does actually happen?
Information

Command or Code

Environment, Platform, Device

Version information

Checklist

[ ] I searched for existing GitHub issues
[ ] I updated all Cordova tooling to most recent version
[ ] I included all the necessary information above

Generally speaking when a method is deprecated, it's because it has fundamental flaw. So the only advice I can give with the provided information is to refactor your code to use resolveLocalFileSystemURL.
Please fill the remaining information for further help.

Closing as stale.


Fix quoting in bash script

In this diff:

Use set -eu for a slightly saner bash execution environment
Double quote all the things, to handle paths including whitespace
Use $(...) in place of ``...``

I'm sorry, I wasn't @ mentioned, so I was unaware that this PR existed! I'll merge it now ...

Can't merge this; apparently, source repo is gone. Now watching repo so I'll be much more timely if further PRs come in.


embedded admission controller into helm chart deployment

This patch is to simplify the deployment of the admission controller. Before this patch, we need to manually deploy the admission controller after scheduler is installed. After patch, all can be done by helm chart. And this patch does following:

Pack the init scripts (generating certs, secrets etc) into the scheduler docker image
In helm chart, added a property: embedAdmissionController, if true, the admission controller init hook will be triggered during the deployment of the scheduler, as a postStart hook.
When we delete scheduler resources, it automatically deletes resources of the admission controller, including web-server, secrets, deployment, and service. This is done via preStop hook.
If embedAdmissionController is false, no admission controller will be installed (the old behavior)

hi @wilfred-s  could you help to review the changes when you get a chance? Thanks!

Hi @wilfred-s , thanks, I've done the fixes in 2nd commit.


Windows Authentication

What version of Flyway are you using?
flyway 4.0.3
What database are you using (type & version)?
SQL Server 2008
What operating system are you using?
Windows 7
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
Folks,
I am facing a issue with windows authentication when i tired flyway url as below,
flyway.url=jdbc:jtds:sqlserver://SOMESERVER;instance=SOMEINSTANCE;databaseName=SOMEDATABASE;SendStringParametersAsUnicode=false;useNTLMv2=true;domain=SOMEDOMAIN;
flyway.user=SOMESERVER\SOMEUSER
flyway.password=SOMEPASSWORD
I am referring a issue raised earlier to flyway, https://github.com/flyway/flyway/issues/360 which is in a closed state. But when use the configuration values flyway.url, flyway.user, flyway.password, I am receiving below error.
Login Failed for user 'SOMESERVER\SOMEUSER'
I am able to login for the same user 'SOMESERVER\SOMEUSER' using windows authentication in SQL server and i have logged in the system with the same user 'SOMESERVER\SOMEUSER'.
Whether flyway support windows windows authentication ? If so what is the syntax we have to follow in flyway.config file ? Can we access remote SQL server using windows authentication ? Appreciate your help regarding !!
What did you expect to see?
When i trigger ./flyway info command, it should show the information from schema_version table from DB which i am configured with windows authentication for a particular user.
What did you see instead?
Login Failed for user 'SOMESERVER\SOMEUSER'

This is something that clearly falls under the responsibility of the JDBC driver you use. There is not much we can do on our end. Make sure to try the MS driver as well to see if that helps.


meta(extensions): tracking issue for extensions / plugins

Tracking all tickets related to extensions / plugins or proposed extensions:
Discussion

https://github.com/foundry-rs/foundry/issues/706
https://github.com/foundry-rs/foundry/issues/6509
https://github.com/foundry-rs/foundry/issues/7760

Features

https://github.com/foundry-rs/foundry/issues/1970
https://github.com/foundry-rs/foundry/issues/7760
https://github.com/foundry-rs/foundry/issues/5864
https://github.com/foundry-rs/foundry/issues/5015
https://github.com/foundry-rs/foundry/issues/3714
https://github.com/foundry-rs/foundry/issues/3911
https://github.com/foundry-rs/foundry/issues/4072 (vendor-specific cheatcodes)
https://github.com/foundry-rs/foundry/issues/5576
https://github.com/foundry-rs/foundry/issues/1232
https://github.com/foundry-rs/foundry/issues/5784
https://github.com/foundry-rs/foundry/issues/8354

Repo's

https://github.com/OpenZeppelin/openzeppelin-foundry-upgrades

Previous discussion tickets: https://github.com/foundry-rs/foundry/issues/706 + https://github.com/foundry-rs/foundry/issues/7760 + https://github.com/foundry-rs/foundry/issues/6509


ti.end in the future issue with Tutorial 1.  (related to precision defined in /utils/time_utils.py)

As I just reinstall my Ubuntu I tried to run Tutorial 1 again for a test.
The issue occurs in the (Executing a new interval) section, which I think is caused by the precision of utcnow() defined in hyperstream/utils/time_utils.py.
It is interesting I didn't have this issue in the past...
For instance:
interval.end = datetime.datetime(2017, 8, 18, 9, 41, 14, 262064, tzinfo=)
utcnow() = datetime.datetime(2017, 8, 18, 9, 41, 14, 262000, tzinfo=)
137             if interval.end > utcnow():
138                 print [interval.end, utcnow()]

--> 139                 raise ValueError("Calculated intervals should not be in the future")
140
141         self._calculated_intervals = value
ValueError: Calculated intervals should not be in the future

To avoid all these problems, we should just change the time intervals of the tutorials to at least some minutes or hours in the past.

HyperStream is only millisecond precise, so this comparison should be changed to remove microseconds.

Should now be fixed with https://github.com/IRC-SPHERE/HyperStream/commit/fb58a388c5f5e844032987ac5e180263e9637519
Would be good if someone were to write a unit test to verify this.


import_module with os.listdir() in path of file

I have the following code which imports all modules from the current directory:
__all__ = [
    import_module("." + f.split('.py')[0], __package__)
    for f in os.listdir(os.path.dirname(__file__))
    if "__" not in f
]

I added the files from the directory with a plugin over getImplicitImports. But the exe does not run and gives the following error:
Traceback (most recent call last):
  File "D:\path\to\app\meterlab2.pyw", line 25, in <module>
  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 "D:\path\to\app\xxxx\common\devices\types\__init__.py", line 6, in <module xxxx.common.devices.types>
FileNotFoundError: [WinError 3] Das System kann den angegebenen Pfad nicht finden: 'D:\\path\\to\\app\\xxxx\\common\\devices\\types'

What are my options to solve this?
Regards

This code should be using pkgutil.iter_modules which is tremendously simper to use and more correct in all cases.


Add conditional logic to hide/reveal comment form

Description
Please provide a description of the changes you have made in this pull request. Explain the problem you are trying to solve and how your changes address it. If this pull request closes any open issues, please link to them in the description.
Changes Made

[x] Added new feature
[ ] Fixed bug
[ ] Refactored code
[ ] Other (please describe):

1 - Added CSS styles for a primary button
2 - Check if user is authenticated on post-detail, if they are, show the comment form. If not, show 'Login to comment' button.
After logging in, they are redirected back to post-detail page.
Test Plan
All tests pass
Screenshots (if appropriate)

Checklist

[x] I have read the contributing guidelines
[x] I have added tests to cover my changes and they all pass in addition to the existing tests.
[ ] I have added documentation for my changes (if appropriate)

@freedompraise Does this comment workflow make sense to you?

@freedompraise Does this comment workflow make sense to you?

Yes it does


WeightedPandas index improvements

Description
This provides a few simultaneous upgrades to WeightedPandas, which were sparked by some of the indexing considerations in #194. These are

WeightedDataFrame and WeightedSeries always have a weights column in their MultiIndex. If no weights are passed, the column is filled with 1's (integers).
More complicated MultiIndexes are now possible (previously it was assumed there would be only one other 'iteration' column).
The 'other' column is now no longer called '#', but will retain the name of the previous index/indices (if present at all).

The above changes make the code more robust and extensible, particularly when it comes to extracting/dropping weights (we can assume that df.index.get_level_values('weights') and df.index.droplevel('weights') will always work), but also making sure that code like (a*b).mean() is only possible between weighted objects if they have the same weights.
Checklist:

[x] I have performed a self-review of my own code
[x] My code is PEP8 compliant (flake8 anesthetic tests)
[x] My code contains compliant docstrings (pydocstyle --convention=numpy anesthetic)
[x] New and existing unit tests pass locally with my changes (python -m pytest)
[x] I have added tests that prove my fix is effective or that my feature works

OK, fairly major update.
It turns out that the 'nice to have' idea of default unit weights in the weighted index when weights=None is passed is not going to work. The main issue is that when transposing samples.T.index (i.e. the original samples.columns) would now have unit weights. This is ugly, plays havoc with index merging, and potentially leads to unusual behaviour in any code which calls a transpose (which we cannot easily/should not forbid).
Fundamentally samples.mean(axis=1) should give the same thing as samples.T.mean(). More importantly samples.T.mean(axis=1) should give the same as samples.mean(). In the original code, these were different, with the former giving the unweighted mean by transferring the weights to a column where the hard-coded axis=0  therefore skipped.
The new updates now aim tie weighted sums to any axis with 'weights' in a MultiIndex, and the above sums work (and are tested to work).
I would value feedback from @lukashergt as to the new user interface, in particular the now slightly inconsistent .weights getter and setter and __init__ function, which only extracts weights from an index for a WeightedDataFrame, rather than from a column. It should be noted that reset_index also performs similarly, but this is consistent with pandas.

OK, the new API with get_weights and set_weights, minimising @propertys and conforming to pandas copying scheme is now finished.
One problem this has revealed is that in the 'correct' copying scheme, metadata is not deep copied in pandas. This is even more reason to implement #214 and #215, which I'll get on with now. If we're happy with the tex dictionary doing something potentially funny in copying until then, this is ready to be merged.

Many thanks for your help reviewing @lukashergt

This one may prove fiddly to merge into your active branches -- let me know if you need any help.


Update SDK and Vehicle App utilizing middleware config

Description

Issue ticket number and link

Checklist

[x] Vehicle App can be started with dapr run and is connecting to vehicle data broker

[x] Vehicle App can process MQTT messages and call the seat service

[ ] Vehicle App can be deployed to local K3D and is running

[x] Created/updated tests, if necessary. Code Coverage percentage on new code shall be >= 70%.

[x] Extended the documentation in Velocitas repo

[ ] Extended the documentation in README.md

[x] Devcontainer can be opened successfully

[x] Devcontainer can be opened successfully behind a corporate proxy

[x] Devcontainer can be re-built successfully

[ ] Release workflow is passing

Only update to devenv-runtimes v1.0.3 is missing to make k3d runtime work. Without just local runtime is working.


[RF][SmartContracts] Create CoinviewRule base class

Currently PosCoinviewRule inherits and overrides some members and/or methods of PowCoinviewRule. As these two classes basically share a lot of logic I want to introduce a base abstract class which holds this logic. Then instead of having virtual overrides for methods like GetProofOfWork we implement concrete implementations for each network type, be it Pos or Pow.
The current implementation breaks some SOLID principles like Liskov as if you had to remove PosCoinviewRule's override of GetProofOfWorkReward the node wouldn't certain be concerned otherwise. We need to be more explicit as to each implementation.
As we also require SmartContractCoinviewRule we cannot currently implement this due to the PosCoinviewRule inheriting from PowCoinviewRule approach.
There are NO logic changes in this PR but instead only the shared logic was moved to CoinviewRule.
I confirm a full Testnet sync and staking on Testnet works.

Closing this as will redo for easier reviewing.


AstTypeStubPaths_ExclusiveStubs test fails

https://github.com/Microsoft/python-language-server/pull/86


Using ToString in LINQ filter does not yield a WHERE clause

This was originally filed under issue #3807. I have repro'd this on rc1-final using the Npgsql provider. I found what I believe is the underlying cause but I'm not sure if it's expected behavior or not. It may have something to do with how the expression evaluator is handling the ToString() method. If this is expected behavior, please close. Thanks.
enum Color
{
    Red,
    Blue,
    ...
}

class MyTable
{
    public int id { get; set; }
    public string color { get; set; } // Contains "Red", etc.
}

// Yields no WHERE clause:
var color = Color.Red;
context.MyTableSet.Where(t => t.color == color.ToString());

// Generates a WHERE clause, as expected.
var color = Color.Red;
var colorString = color.ToString();
context.TableSet.Where(t => t.color == colorString);

This is expected. We don't know how to translate ToString method to SQL so the entire filer is performed in memory. Workaround is as you mentioned - perform the client operations outside the query and pass the result in form of closure.


En/Disable button based on availability of appointment in the message

Thank you for this add-on!
Would it be possible to enable the button in the message pane, only if there is an appointment available in the message? If there is no appointment in the message, the button should be disabled (not sensitive).

Thank you so much @Leworoe!!


Set current resolution based of DSP model number

Using both a DPS3005 and DSH5005 with the same ESP32 caused me issues with setting/reading the current because of the different resolution. So I've added automatically setting the resolution based on the DPS model number it returns.
Also added a picture of the pinout for the serial connector to the readme
{This is a great custom component for ESPHome, thanks very much to creating it}

I've refactored your code a bit. The device model based auto-detection is the default now and is called current_resolution: AUTO. It can be overridden if another value (LOW, HGIH`) is used. Could you give it a try?

Gave this a brief test tonight and working fine - thanks.


Feat: Changes config to add support for bytes input/return types

Description
Aims to add support for what was discussed on #77.
@tmm let me know if something like this works or if you had something else in mind.
Additional Information

[x] I read the contributing docs (if this is your first contribution)

Your ENS/address: 0x75EF32C862b8fe2915D49b38a2AECc8744622864

Nice!


CSS is loaded after the slides are rendered. You can see the whole content of the slides before the CSS kicks in and distributes it properly over the other slides

CSS is loaded after the slides are rendered. You can see the whole content of the slides before the CSS kicks in half a second later and distributes it properly over the other slides.

Please help me 👍

here you can see a screenshot...
http://recordit.co/MIaaZekX6u

A quick fix would be, setting .hooper-slide width to 100%, this will make the slide take the whole slider width, until it loads and recalculate its width.
Another way to fix this is to set the carousel opacity: 0, and wait until it completely loads the show it again, this approach will need some enhancements from my side,  emitting event after carousel loads.

Thank you. I appreciate your work and I tried the quick fix but it didn't have an effect. The behaviour didn't change.
What I am also trying to achieve is to animate the transform:translate a little bit with ease-in-out. But it would be even better if I had a currrentSlide Class for the proper styling. Is it possible to add a currentSlide Css-Class so I can only show the currentSlide. Here is a great example, where a new slide is fading in with an animation
https://onlineuebungen.schularena.com/sheets/m698/tasks/13

I have just found the is-active class in your code :-)

setting the opacity of .hooper to 0 also shows the initial loading of the components.

If you’re using webpack in development mode, it will inject the CSS in javascript files, so the style will load after the page loads which will cause flickering for sure. Try to extract CSS files (run webpack in production mode) and please give me the feedback with the results.
For opacity method (which I recommend) it will not work until the following release, when the ‘loaded’ event is available.

Looks like you are right. I don't get the same flickering behaviour if I run it in production mode. I am sorry I havent thought about that. Thank you for your help though. Concerning the loaded event, don't we already have  an @updated emit event which would be very similar to a loaded event, sort of?

Yes, you can use @updated event for now, since it fires each time carousel is updated (including the first time creation).

you can now use @loaded event in version 0.0.9

Hi,
One idea is that Hooper hide yourself (or only your slides) before was loaded (on same moment of the @loaded event or watch data loaded #46) using a class .hooper-preload or anything like that.


How to make mediapipe pose detection faster? (python)

Hi. I'm making a pose estimation script for my game. However, it's working at 20-30 fps and not using the whole CPU even if there is no fps limit. It's not using whole GPU too. Can someone help me?
Here is resource usage while playing a dance video: https://imgur.com/a/6yI2TWg
Here is my code:
import cv2
import mediapipe as mp
import time

inFile = '/dev/video0'

capture = cv2.VideoCapture(inFile)
FramesVideo = int(capture.get(cv2.CAP_PROP_FRAME_COUNT)) # Number of frames inside video
FrameCount = 0 # Currently playing frame
prevTime = 0

# some objects for mediapipe
mpPose = mp.solutions.pose
mpDraw = mp.solutions.drawing_utils
pose = mpPose.Pose()

while True:
    FrameCount += 1
    #read image and convert to rgb
    success, img = capture.read()
    imgRGB = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)

    #process image
    results = pose.process(imgRGB)

    if results.pose_landmarks:
        mpDraw.draw_landmarks(img, results.pose_landmarks, mpPose.POSE_CONNECTIONS)
        #get landmark positions
        landmarks = []
        for id, lm in enumerate(results.pose_landmarks.landmark):
            h, w, c = img.shape
            cx, cy = int(lm.x * w), int(lm.y * h)
            cv2.putText(img, str(id), (cx,cy), cv2.FONT_HERSHEY_PLAIN, 1, (255,0,0), 1)
            landmarks.append((cx,cy))

    # calculate and print fps
    frameTime = time.time()
    fps = 1/(frameTime-prevTime)
    prevTime = frameTime
    cv2.putText(img, str(int(fps)), (30,50), cv2.FONT_HERSHEY_PLAIN, 3, (255,0,0), 3)

    #show image
    cv2.imshow('Video', img)
    cv2.waitKey(1)
    if FrameCount == FramesVideo-1:
        capture.release()
        cv2.destroyAllWindows()
        break

Hi @yigitayaz262, Thank you for sharing your observation. Could you please elaborate your use case with more details w.r.t above observation. Thanks!


fixing position of rotated "no orientation" ports

Hi @joamatab , we found a pretty serious bug today.
When references are rotated, ports with no orientation are not correctly transformed. This PR fixes that bug and adds a simple test for this case.

Thank you Troy!


brew doctor fails with the error: Error: no implicit conversion of nil into String

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[ ] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

What you were trying to do (and why)

After running brew update this morning, brew doctor consistently fails with the error Error: no implicit conversion of nil into String.

What happened (include command output)

-> brew doctor
Error: no implicit conversion of nil into String
Please report this bug:
  https://docs.brew.sh/Troubleshooting
/usr/local/Homebrew/Library/Homebrew/formulary.rb:363:in `extname'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:363:in `loader_for'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:280:in `factory'
/usr/local/Homebrew/Library/Homebrew/dependency.rb:33:in `to_formula'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1564:in `map'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1564:in `missing_dependencies'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:13:in `block in missing_deps'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:12:in `each'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:12:in `missing_deps'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:720:in `check_missing_deps'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:46:in `block in doctor'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:38:in `each'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:38:in `doctor'
/usr/local/Homebrew/Library/Homebrew/brew.rb:95:in `<main>'

What you expected to happen

I expected brew doctor to return either a clean bill of health or a specific set of problems.

Step-by-step reproduction instructions (by running brew commands)

-> brew update
Already up-to-date.
-> brew doctor
Error: no implicit conversion of nil into String
Please report this bug:
  https://docs.brew.sh/Troubleshooting
/usr/local/Homebrew/Library/Homebrew/formulary.rb:363:in `extname'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:363:in `loader_for'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:280:in `factory'
/usr/local/Homebrew/Library/Homebrew/dependency.rb:33:in `to_formula'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1564:in `map'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1564:in `missing_dependencies'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:13:in `block in missing_deps'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:12:in `each'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:12:in `missing_deps'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:720:in `check_missing_deps'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:46:in `block in doctor'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:38:in `each'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:38:in `doctor'
/usr/local/Homebrew/Library/Homebrew/brew.rb:95:in `<main>'

Output of brew config:
-> brew config
HOMEBREW_VERSION: 1.6.6-63-ga674a97
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: a674a972a56850a6f76eb1d9e812438bc435152b
Last commit: 65 minutes ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 44ca0d278857397e87273740bb3229772cd733e5
Core tap last commit: 2 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_AWS_ACCESS_KEY_ID: set
HOMEBREW_AWS_SECRET_ACCESS_KEY: set
HOMEBREW_DEV_CMD_RUN: 1
CPU: octa-core 64-bit haswell
Homebrew Ruby: 2.3.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Clang: 9.1 build 902
Git: 2.17.1 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
Java: 1.8.0_72
macOS: 10.13.4-x86_64
CLT: <IP_ADDRESS>.1.1526532315
Xcode: 9.4
XQuartz: 2.7.11 => /opt/X11

brew doctor --debug --verbose would be even better.


⚠ Bofrost.be has degraded performance

In ccb0f03, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1858 ms

Resolved: Bofrost.be performance has improved in e217a63.


Background transparency not working as expected.

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 10 Professional 64-bit
Hyper.app version: 1.2.1
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/imaginationac/74f42ffa21e41c4a797beaab70b5fa76
Relevant information from devtools (CMD+ALT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A
The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue

After updating to 1.2.1, background transparency/alpha does not behave as expected. Was working on 1.1.0.

Do you mean vibrancy? ✌

@henrikdahl No.
In .hyper.js, when changing the the value of background-color to something like rgba(0, 0, 0, 0.5) I expect to the background to be at 50% opacity, and to be able to see through the windows to any programs or the desktop. That not longer happens for me.

Transparency still works on macOS, I don't have Windows so someone else needs to confirm there.

I have just updated to 1.2.1 from 1.1.0 and I'm experiencing the very same problem.

I have the same problem. I use Windows 10 Professional x64 Version: 1607 Build: 14393.693 and Hyper 1.2.1

Also experiencing this. Win 10 Pro x64: 1607, Hyper 1.2.1.

In Windows 10, also experiencing this issue

Issue still present as of 1.3.1.

@imaginationac you can read this comment in same issue https://github.com/zeit/hyper/issues/1638 by @cvandal it's work

I'm on MacOSX.
I also have this issue. When I remove my old .hyper.js file, and let hyper create a new one, it does not work. After a few quit and reopens, and full reloads, the previous terminal persists in the background . The layers build up and then they eventually make the window look full opaque.

The layers are gone again, if I dump the .hyper.js file and let hyper create a new one for me.

Same problem with Hyper 1.3.3, Windows 10 build 15063.483

Not working here too Windows 10 v1709 Build 16299.248 ... Hyper 1.4.8

Not working with windows 10

It's not supported on windows.

I think this can be solved using browserWindow.setOpacity (electron doc).
It is compatible with Windows and MacOS. It was added in electron in this PR : https://github.com/electron/electron/pull/10667
I made a plugin for hyper using it : hyper-opacity 🙂

I think this can be solved using browserWindow.setOpacity (electron doc).
It is compatible with Windows and MacOS. It was added in electron in this PR : electron/electron#10667
I made a plugin for hyper using it : hyper-opacity 🙂

This plugin doesn not work for me with hyper 3.0.2


CurrentProcessMetrics: MemoryResident on Linux probably should not include shared memory pages (e.g. system file cache)

https://kontur.slack.com/archives/C08DRHZTP/p1613558381350700

I agree that we shouldn't include shared memory pages in this metric, because they don't provide very useful info about application memory usage.
Having RssAnon instead of VmRSS is informative enough.
Of course, if we could calculate RssAnon + something like shared that is used by no one but me metric it would be more useful, but there is no easy way to calculate this.
There is also a metric called PSS.  It looks much more useful, and can be retrieved in /proc/[pid]/smaps.
But the doc placed a warning under /proc/[pid]/statm: "Some of these values are inaccurate because of a kernel-
internal scalability optimization.  If accurate values are
required, use /proc/[pid]/smaps or
/proc/[pid]/smaps_rollup instead, which are much slower
but provide accurate, detailed information."
So I think that the game is not worth the candle, and we should just use RssAnon instead of VmRSS.


Fix issue

Made changes to pass test cases in javascript medium task

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Crashes on any locale other than English and Arabic

This line:
ResourceBundle resource = ResourceBundle.getBundle("com.github.msarhan.ummalqura.calendar.text.UmmalquraFormatData", locale);
crashes the app if run on French / German / Russian etc as the bundle only includes _ar and _en. This should be done in a try / catch and if a locale isn't found, English should be used.

Added check for supported locale.
I think it is not good to fallback to English locale if the provided locale is not supported.
Thanks!

@guidedways , i know this is way too late
but for anyone else coming here, you can always download the library, and use it in ur project, and simply add classes for any other language, then manually translate the months' names to that language, and don't forget to add the condition for your language, otherwise it'd keep throwing the Exception.

@guidedways I was wrong about refusing fallback solution if the current locale is not supported as this library could be used on mobile devices with no control of the used locale.
Thanks to @AmineMessabhia for the suggestion.
I've added the possibility to add new locale using properties files which is much easier than creating Java class for localization. And in case of missing locale, it will fallback to ENGLISH locale.
Please check v1.2.0


create a new .txt file with every batch only works for the first batch, and 90% of the time there is no info in it.

Describe the bug
Write prompts to file: Create a new .txt file for every batch containing the prompt template as well as the generated prompts is checked. When running multiple batches only one text file is generated. In this text file 90% of the time the only information in it is
positive_prompt,negative_prompt
,
and nothing else. When it does "work" it saves the positive prompt twice and does not write the negative prompt.
When generating with dynamic prompts enabled but not using any dynamic prompts variants it creates the file for every image and puts the positive prompt twice.
saving the template to .png also only saves the wildcard information not the actual wildcard entries that were used.
To Reproduce
Steps to reproduce the behaviour, include the prompt you used if applicable:

Go to img2img
enter prompt
click generate batch size of 1 batch count 1
click generate for second batch size 1 batch count 1
go to output folder and open .txt file.
only 1 .txt file and it has no relevant information

Also mention which checkboxes have been set.
ignore whitespace checked
save template to metadata checked
write prompts to file: checked
I've tried with every combination of them checked and unchecked.
Are you using a local install or a colab?
Local
Expected behaviour
Prompt generation information should be saved to the .txt file.
Additional context
Fresh install of auto1111 as of today.
No other extensions installed.
fresh install of dynamic prompts as of today.

Looked it up using gpt4 told me I was missing a __ somewhere in my prompt.


bug: Bower - Can't update to ionic rc2

Type: bug
Platform: desktop browser
I can't update ionic to rc2 from rc1 - wrong angular version

Greetings @piernik!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

I've tried also ionic lib update no luck - rc1

Are you using the Ionic bundle? Use your own version of angular at your own risk.

Yes I started with Ionic bundle

In bower I have
"devDependencies": {
    "ionic": "driftyco/ionic-bower#1.0.0-rc.1"
  },

switching to rc.2  (now rc.3) updates library


Improved gitignore and removed references to Avenir

This is mostly cosmetic, but part of the removing references to Avenir.
Additions

More robust .gitignore rules

Removals

Removed references to Avenir

Review

@Scotchester

Preview
Preview this PR without the whitespace changes
Notes

Part of https://github.com/cfpb/capital-framework/issues/161

Looks good!