fix: Align creation of k8s namespaces and service account in addons

What does this PR do?
Some addons support setting helm_config.create_namespace, but don't support setting helm_config.create_service_account. This makes disabling create_namespace useless when the creation of the service account is enabled since the service account can't be created into a non-existent namespace.
On the other hand, since Blueprints v5 will have different repositories per addon, I would prefer to manage all Kubernetes resources through a custom ArgoCD app. And then, I want blueprints to only create the IAM roles for IRSA.
For both reasons, this PR will allow conditionally creating namespaces (already supported) and service accounts per addon.
Motivation

Related to #1421

More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[ ] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

thank you for the PR - this will be resolved in the new addons module https://github.com/aws-ia/terraform-aws-eks-blueprints-addons


_key was called on null

hi Dane
the navigatorKey that i am getting from Navigation services i am getting this error

update to 1.5.1 it's fixed.

Thanks


Fix annotations issue with lower python version

https://github.com/pydantic/pydantic/pull/8659
Tweaking the existing test for this reveals a bug:
class Mixin:
    pass

class Child(Mixin, Model):
    y: int

Maybe the idea is to make this: https://github.com/pydantic/pydantic/pull/8659/files more like this: https://github.com/pydantic/pydantic/pull/8679/files.

Honestly, I haven't been able to easily track down what we need to do here, and this doesn't seem super high priority. If someone reports some sort of bug related to this I'd be happy to fix it, but there are other things that are more valuable at the moment.


perf: combine block + receipt request

closes #2887
This adds a new cache function to fetch a block and its receipts with one call.
in the caching layer reads are spawned tasks, this will only spawn 1 task now to fetch receipts+block instead of two.
EDIT: need to think about this a bit more

@mattsse Need to rebase on top of main for the required checks unfortunately

thought about this, and this is not worth the extra complexity


Bump dependencies

Fixes dependencies @dependabot isn't able to figure out.
/cc @yevgeny-shnaidman @ybettan

Codecov Report
Patch and project coverage have no change.

Comparison is base (bc48830) 82.86% compared to head (6ec0dd4) 82.86%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #341   +/-   ##
=======================================
  Coverage   82.86%   82.86%
=======================================
  Files          27       27
  Lines        2784     2784
=======================================
  Hits         2307     2307
  Misses        388      388
  Partials       89       89

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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Only display the battery runtime label if the battery is discharging

Only display the estimated remaining runtime if the battery is actively being discharged

I'm thinking of doing this slightly differently. I want to change "Battery Runtime" to "Battery Charging"


Add "Go to home" action to service

Sometime services goes to wrong pages, and seems there are no ways for go to "Home" now, without enabling navigation bar and pressing "Home" button.
So, will be good to add action "Go to home" to "View" menu.
And, maybe, in right-click menu on service icon too, together with "Back" and "Forward"?

Implement in https://github.com/getferdi/ferdi/pull/900


Version v0.2.1

Fix paragonie/random_compat dependency breaking the installation on PHP > 7. See #41
The dependency is not needed on the required version (>7.1.0) of php anymore.

Can this be merged please? Not able to install laravel-database-encryption because of #41 issue.

@mithunsnair you may directly require the 2.0
composer require paragonie/random_compat:^2.0


Flutter url_launcher5.7.6 Unable to make calls on some Android phones

code：
bool flag= await canLaunch("tel:$enginetel");
if(flag == true){
if(enginetel!=null) await launch("tel:$enginetel");
}
The Flutter project was running and packing fine. When the real machine was running, clicking the call button did not respond.Some phones have this problem.

@jkLennon
I verified this using url_launcher's official example and using latest version and was able to make phone call properly without any issues.
Use upgraded plugin version and check if it works.
If issue persists, provide complete minimal reproducible code sample and devices on which you are seeing this issue.

flutter doctor -v
[✓] Flutter (Channel master, 2.1.0-11.0.pre.205, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-GB)
    • Flutter version 2.1.0-11.0.pre.205 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 267f66a7dc (5 hours ago), 2021-03-10 23:15:03 -0500
    • Engine revision 1b68503bc8
    • Dart version 2.13.0 (build 2.13.0-125.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (3 available)
    • SM G975F (mobile) • RZ8M802WY0X • android-arm64  • Android 10 (API 29)
    • macOS (desktop)   • macos       • darwin-x64     • Mac OS X 10.15.4
      19E2269 darwin-x64
    • Chrome (web)      • chrome      • web-javascript • Google Chrome
      89.0.4389.82

! Doctor found issues in 1 category.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Documentation: make it live

Hey, your documentation (under /docs/source/) is great and very helpful!
But I don't see any link to read it comfortably online... (besides nagivating back and forth from dir-index on github.com).
Put it somewhere online (readthedocs, github.io or the like) and put a link in README (or even better in the project head (above dir-index)).
Or, at least, put a link from README to /docs/source/index.rst and update that page to make links work directly on github.com...
Would be great and add value to your package!

Finally done it here: https://django-location-field.readthedocs.io/en/latest/
Thanks for the suggestion!


Component auditing improvements

What
Couple of new bits on the component audits, specifically:

stop using the tag component on the component details list, not appropriate
add JS generated numbers to indicate totals on the component details list

Why
Improve our component auditing.
Visual Changes
New column totals on the component details. These are updated dynamically if the selection is changed.

Thanks for the review @jon-kirwan! Great shout on the forEach loops, didn't realise they were supported as far back as IE9 - I seem to always instinctively write JS for the lowest common denominator...


Minor design issue with openMIDI Submission CC entry

Issue is merely with the openmidi.com "Create OpenMIDI Submission" form.
Please set the default value for the "Maximum CC Value List" to = 127 rather than 0.
It will make submission far less tedious. Thank you for this project

thanks for the feedback., I have updated this.


Control the writing of operational outcomes with https $import

Signed-off-by: Paul Bastide<EMAIL_ADDRESS>

The changes look good to me, but it makes me wonder if we should at least provide a high-level summary of the resources which failed to import...any thoughts/ideas on that?

The changes look good to me, but it makes me wonder if we should at least provide a high-level summary of the resources which failed to import...any thoughts/ideas on that?

Good point... I think we could do that. It shouldn't be so much of a problem

{
    "transactionTime": "2021-02-05T14:00:57.717-05:00",
    "request": "$import",
    "requiresAccessToken": false,
    "output": [
        {
            "type": "OperationOutcome",
            "url": "test-import.ndjson_oo_success.ndjson",
            "count": 4
        }
    ],
    "error": [
        {
            "type": "OperationOutcome",
            "url": "test-import.ndjson_oo_errors.ndjson",
            "count": 0
        }
    ]
* Connection #0 to host localhost left intact
}*

This is what is returned.


Can't install the downloaded .deb file.

After downloading the Monokle-linux-1.5.1-arm64.deb, I tried to install the file.
It says- "The following packages have unmet dependencies".
I downloaded the required dependencies: helm and kubectl, but it didn't work.
Need help.

Thank you @Rohan749 for reporting this issue! We will check what is wrong and inform you back as soon as possible.

judging by https://www.electron.build/configuration/linux#debian-package-options we need to need to remove some dependencies for the deb linux target in package.json

I tested this by installing Monokle using the following command:
sudo dpkg -i Monokle-linux-1.5.1-arm64.deb

and it threw me a similar error, which told me the libxss1 package must be installed
thus, I installed it with the following command:
sudo apt-get install libxss1

and then I ran the first command again, which now worked without any errors and Monokle was installed and works correctly

Same for me

I installed Monokle using the appImage on Ubuntu which worked without any errors - does the appImage include this dependency? or perhaps it's not actually needed but incorrectly set as required in the .deb image?
https://www.electron.build/configuration/linux#debian-package-options doesn't list libxss1 as a default dependency for .deb packages though!?

Yes, it seems that libxss1 is for deb packages.

@Rohan749 have you you tried the new version of Monokle (1.6.3) and installed the dependencies?
sudo apt-get install libxss1


ocamlPackages.lwt_ssl: 1.1.3 → 1.2.0

Description of changes
https://github.com/ocsigen/lwt_ssl/blob/1.2.0/CHANGES
Things done

Built on platform(s)

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

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

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

[x] 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/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

4 packages built:

comby
dune-release
ligo
liquidsoap


use M1 runners for iOS builds

Moved the changes into https://github.com/THEOplayer/flutter-theoplayer-sdk/pull/8


Could not locate zlibwapi.dll. Please make sure it is in your library path!

用GPU的时候一直提示这个：Could not locate zlibwapi.dll. Please make sure it is in your library path! 然后就自己报错退出了

https://docs.nvidia.com/deeplearning/cudnn/install-guide/index.html#install-zlib-windows
You can copy this file somewhere and add it to the PATH or you could copy it to C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.x\bin folder


error ./generate_postmortem.sh 0 --ova --pull-appliance-logs --diagnostic-portal

./generate_postmortem.sh: line 7: syntax error near unexpected token newline' ./generate_postmortem.sh: line 7: '

@kaskiland @subindani Is this issue still happening? I cannot reproduce - can you let me know and I will raise an internal issue if so.
Thanks


ML 3.10 : Outlier Detection using Local Outlier Filter (LOF) (V)

Description
📌  Issues for Week 3
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
Outlier Detection using Local Outlier Filter (LOF).
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Unsupervised_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Video

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Name-Nikita Bhrugumaharshi Emberi
Domain-Machine Learning
Batch-6
Serial No-992
Issue Number-2943
Choice-Video

Hey I'd like to take this issue for contribution.
Name- Pratyaksha Gupta
Type- Video
Github-https://github.com/Pratyaksha-047
Batch- ML batch 3

@NikitaEmberi Complete your previous issue first.

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do video on this issue.

@deepthi1107 Issue assigned to you.

Thanks mam


Improve CSS in Blogs Section

Description
Want to improve the spacing and styling in the blogs section as it looks very scattered right now.
Screenshots
https://user-images.githubusercontent.com/92395053/193438920-116f0b59-7116-4de8-b6f0-c11b754c95c5.mp4

Hi @ankittt20 I want to work on this issue please assign it to me.

Hi @ankittt20   I can fix that issue please assign it to me


Add a tool for managing a cluster.

Currently only has a method to initialise the default entries in etcd.
Laid out as it is with the intention of being able to call into it from ct-server.cc when that's being run in "standalone" mode in order to get its FakeEtcd initialised too.

Minor thing: I'd rename this ct-clustertool, so it'd still make sense when installed in /usr/bin?
Otherwise done with this review.

LGTM


chore: add ruff copyright check

Add a ruff rule to enforce copyright header is presented in python files

Looks like the mirror failed when installing system deps. Should be able to just retry the job?

Looks like the mirror failed when installing system deps. Should be able to just retry the job?

Lemme take a look. seems that protobuf is not installed.

@eddyxu It's just that it failed here:
https://github.com/lancedb/lance/actions/runs/7200989635/job/19616217571?pr=1716#step:5:313
The install command is here: https://github.com/lancedb/lance/blob/c65afc89cd3725be3317bf0c169e699dadcc5e77/.github/workflows/build_linux_wheel/action.yml#L33-L37


Add branch protection to the repo

As we start working on the codebase, nobody including owners should be able to push directly to master. Let us put branch protection in place to make all changes through a pull request.

@omec-project/tst this is already done for this repo. Correct?


Replaced Deprecated Function

Changed deprecated Function with a supported one.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
10 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 83.734%

Files with Coverage Reduction
New Missed Lines
%

bitcoin/src/blockdata/script/borrowed.rs
10
77.29%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
19505

Relevant Lines:
23294

💛  - Coveralls

@apoelstra, safe to land?

No, I'll wait for a second approver before merging, which is our policy here.

Oh, not really. This should've been silenced for the time being and completely deleted with the offending function in the next release.

@Kixunil what's the point of waiting for a release? We already replaced the function and we've had several weeks of testing the deprecated function since then.
Also the new function is obviously a subset of the old one, and the test only covers that subset, and clearly neither function is going to change functionality ever./


Add Button required for lens cluster pages (like namespace addbutton)

What would you like to be added:
Today all the cluster pages has edit and remove functionality available but "Add/Create" functionality not available.so we wanted the add/create functionality for all cluster pages.
Why is this needed:
we want to add a new node to the cluster to achieve this via lens app we have required node creation functionality.same way this functionality will useful for other pages also.
Environment you are Lens application on:
Mac and Linux OS

Kubernetes distribution: [EKS, AKS, GKE, Rancher, ...]

Desktop OS:

This is functionality that we would expect could be added through an extension, since the implementation details would be specific to the k8s distribution, etc.

As @jim-docker said, Lens won't (definitely in the near future) deal with the adding of nodes due to the divers distributions and their associated requirements.


环境异常

环境异常，使用GPTK，因为使用wine渲染有问题，尝试过重启，没有用

macOS: [e.g. 14.5 Sonoma]
Device: [MacBook Pro 2023]
CPU:  [M3Max]
RAM: [ 36G ]

Use Crossover wine.

Use Crossover wine.

I'm using crossover wine

Describe the full problem you are having

Describe the full problem you are having

I had solve this problem by changing an network ，thank you


Update minimum iOS version to 9.0

For the sake of consistency
FUTURE TBD: I wonder if the minimum should be higher, since the iOS ecosystem seems to have moved beyond older iOS versions such as iOS 9 and iOS 10.

[x] updated tests using yarn jest -u
[x] npm t (npm test) succeeds
[ ] TODO: test that generated module with generated example app continues to work on iOS

P.S. This update was made in response to my review and examination of PR #91 (added tvOS support).

/cc @dlowder-salesforce


Hugo: improve distinguishability of inline code

Added background + border as seen in design
Only added this when the inline code is inside a p. This because previous wishes to remove the styling from titles

For https://linear.app/usmedia/issue/CUE-279

Closed via 0e229156965350ab5437ef9061452a23218009c9


Symphony automatically unwraps the first-level properties when creating configmap

If a catalog has a complex type at root like
complex_object:
foo1: bar1
foo2: bar2
The object is unwrapped and only foo1 and foo2 and sent to configmap.
Desired behavior is that the properties are kept as they are.

After discussion, the behavior would be if a property is post-fixed with "/", we'll extract and return the properties under the property. Otherwise, the property is returned as is. Hence, in the above example, "complex_object/" will return "foo1" and "foo2", while "complex_object" returns the object itself.

Dup of #105

Closed this as it dups to #105


error during connect: In the default daemon configuration on Windows, the docker client must be run with elevated privileges to connect.: Post http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.24/build?buildargs=%7B%7D&cachefrom=%5B%5D&cgroupparent=&cpuperiod=0&cpuquota=0&cpusetcpus=&cpusetmems=&cpushares=0&dockerfile=Dockerfile&labels=%7B%7D&memory=0&memswap=0&networkmode=default&rm=1&shmsize=0&t=mymicroservice&target=&ulimits=null&version=1: open //./pipe/docker_engine: The system cannot find the file specified.

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/microservice-tutorial/docker-image
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Looks similar to https://stackoverflow.com/q/40459280/4231460.
@MichaelSimons, I am not very familiar with docker, could you please verify if this is the same issue as the StackOverflow's post that I linked?

Hey @SergeyBorisov, please check issue #6018 that has a couple of things for you to try and some recommendations.


how can i install jobe in Nginx?

how can i install jobe in Nginx?
Nginx is insalled as my httpserver, how can i install jobe in a Nginx server instead of apacher?

Duplicate of #34.


Adds icon to unsubscribe from side bar

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
#1396
I know this is closed, but I think its a great stop gap solution until #1394 is finished. It allows users to quickly prune their subscriptions. It should also allow you to unsubscribe from deleted channels (which I think @eggplantbren mentioned was an issue). It might even be better than #1394, because it keeps things simple for the end user, and I'm not sure how useful editing and tipping without entering the claim would be compared to the complexity cost to the user.
What is the current behavior?
No unsubscribe button next to the list of subscribers.
What is the new behavior?
Unsubscribe button next to the list of subscribers. It looks like this

Other information

hey @abueide , thanks for the PR! We'll get a review on this, but I'm not sure if we need quick unsubscribe actions from the side bar - how often would that actually get used? It also adds lots of repeating elements to the UI. I'll let other team members review.
Can we show you some appreciation for the contribution?

Thanks for the PR @abueide!
I think this is an interesting idea, but I think it will lead to more issues than help. The main one being when someone accidentally clicks on the heart icon. In every other button, clicking on the icon leads is the same as clicking on the button label.
I'm going to close this.


⚠ Bofrost.at has degraded performance

In e67709b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2830 ms

Resolved: Bofrost.at performance has improved in d16bdf2.


Compliance test suite

The issue
This topic has been brushed a few times already (#1098, #892, #733). The idea is to provide a test suite so that library implementers can 1. prove compliance to the spec and 2. uniformly showcase their abilities.
The solution
I went ahead and created some basic infrastructure: jsonapi-grader, which is mainly a bunch of request specs.
The idea with the validation of server libraries is that each maintainer would set up a service (on a free heroku instance or similar) against which users could run the test suite (and possibly the jsonapi.org maintainers would periodically run it as well). The test scenarii would possibly be documented on the jsonapi.org website.
Making it happen
Sounds interesting? Get involved and help build the actual test suite (suggesting scenarii and/or implementing them).

This could be a helpful reference: https://json-ld.org/test-suite/

Since this is the new home for this discussion I'll bring my recommendation here from https://github.com/json-api/json-api/issues/1098
I'm proposing we use 'Author > Books > Reviews' as the basis of an example app.
Lets make it so that all resources in the domain have names that are very easy to grok and can not be in any way misunderstood for code related terms.

@evolve2k Thanks for the input. I agree that we should keep the test scenarii and the tested resources/endpoints simple to reason about. However, I believe it would be easier for implementers that each scenario tests a specific thing (which is not opposed to having resources/endpoints named as you suggest, but I'd like to avoid having 3 endpoints that have to comply to many different scenarii at the same time). One possible solution would be to have each test scenario hit namespaced endpoints (e.g. for the first test http://compliance.example.com/test1/books).

One thing I think should be in there: a model with a two-word name, like CustomerReviews, so that we can test serializing the type to customer-reviews.

@fotinakis I'm not certain we should be concerned with the casing choice of the Server since, although it is a recommendation, is not required. For example, if an implementation only supports camelCased or snake_cased names, it makes result validation a bit more complicated since now we need to check the keys of the returned document against several different casing options. But, I also see the value in being able to state what casing mode the Server uses by default.

I love this discussion! Don't have the bandwidth to add much, but I would like to point to one of @tkellen's old projects—https://github.com/endpoints/fantasy-database—which could offer a schema/test data to work from easily.

IMDB is also a popular example dataset.
on the application side I could offer crnk.io as guinea pig to setup and pass those tests. It has both a full server and implementation.

Thanks for the inputs – a dataset will definitely come in handy.
I think the next step should be establishing together a list of specs (sorted by category) that we want for the first version. I'll open a PR so that people can suggest new tests/categories.

Update: I built part of a test suite. Anybody willing to give a hand?

@beauby thanks for leading this. i'll be taking a deeper dive into it this weekend.

@jaredcnance (and everyone, really): I updated the README with a few explanations, and added a (partial) example compliance implementation for jsonapi-rails.

Partial test suite here as well: https://github.com/endpoints/endpoints/tree/master/test/integration


Add wheel to requirements

The build will fail without wheel. Fairly easy for a python person to fix, but less obvious to the uninitiated. Adding it to the requirements fixes the problem.

@felixs88 do I need to change anything in this commit for it to be accepted?


Travis macOS job failure when triggered by a commit to an non-master branch

I encountered this while trying to set up Travis CI for a tap. Travis job log
I have only seen this failure happen if the Travis CI build is triggered by a commit to an non-master branch, not by PRs. Did not test if the same will happen if commits directly to master.
Looks like it is not isolated. Similar failures occur in denji/homebrew-nginx's log & dunn/homebrew-emacs's log. (Note that these two repos' builds also contains an Linuxbrew job that does not fail. )
The only "Error:" message that is unique to the macOS jobs is:
Error: File name too long @ dir_s_mkdir - /Users/travis/build/tommyang/homebrew-gerbera/brewbot/ (might need to click the raw log button to see, somehow hidden by Travis)

@tommyang Could you try to submit a pull request? We're unlikely to work on fixing this ourselves as we're currently investigating non-Travis CI solutions and Travis is working fine for us.

I would love to submit a PR. However, I was not (and still isn't) sure about the root cause.
I just did a "bisect" using the log of dunn/homebrew-emacs and found that one such commit builds successfully on May 4th while another such commit fails this way on May 7th.
During this timeframe, the only change to test-bot is #155. I can't make sense of why this change would result in the error message above.
Tagging @dunn in case they have additional insight.

I see that error intermittently, but I've never tracked down the cause of it.

Ok, this doesn't seem actionable as-is but we'll consider a PR. Thanks!


Stop renaming everything, you're annoying. Fix the names of the classes and don't rename them.

Bug description
description in title
What version are you seeing the problem on?
v1.x
How to reproduce the bug

Error messages and logs
# Error messages and logs here please
Support for `training_epoch_end` has been removed in v2.0.0. `ResnetTT` implements this method. You can use the `on_train_epoch_end` hook instead. To access outputs, save them in-memory as instance attributes. You can find migration examples in https://github.com/Lightning-AI/lightning/pull/16520.

Environment

Current environment
#- PyTorch Lightning Version (e.g., 2.4.0):
#- PyTorch Version (e.g., 2.4):
#- Python version (e.g., 3.12):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):

More info
I don't understand what you were on when you replaced it training_epoch_end to on_train_epoch_end.


Improve THCCachingHostAllocator performance by making it reclaim less aggressively

@gchanan just fyi this PR breaks on gcc 4.4.7 which only has support for c++0x. I dont care if we want to keep this c++0x-only requirement, maybe it's okay we move to c++11

@soumith it's probably not worth it to remove support for gcc 4.4.7 just for this change.  I had assumed that emplace existed since emplace_back existed for std::vector (and those functions are in the same standards proposals), but it looks like gcc didn't implement them until later: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44436.

https://github.com/torch/cutorch/pull/703


Merge sandboxed prototyping features into main branch

Summary | Résumé
This PR introduces the following features into staging:

Feature Flags
Vault storage retrieval page
Template CRUD API

Linked issues:
Resolves #217
Resolves cds-snc/forms-staging-terraform#26

I've got this hooked up to the sandbox backend locally so I can verify everything still works.
Seems like all the templates in the sandbox vault are there, however when I try to access the form's page I get a 404
(e.g. I can see en/id/155/settings, but en/id/155 gives a 404.


add checkstyle plugin into e2e module.

The example module and the e2e module are single projects under shenyu, the e2e module needs to add checkstyle plugin too. If not, mvn checkstyle:check rule is not as the shenyu project rule.
Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4811 (4ccb619) into master (8d4bfa0) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4811      +/-   ##
============================================
- Coverage     65.32%   65.11%   -0.21%
+ Complexity     8186     8149      -37
============================================
  Files          1138     1138
  Lines         33476    33464      -12
  Branches       2975     2971       -4
============================================
- Hits          21867    21790      -77
- Misses         9885     9941      +56
- Partials       1724     1733       +9

Impacted Files
Coverage Δ

...nyu/admin/discovery/DefaultDiscoveryProcessor.java
10.30% <ø> (ø)

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


Set up config to allow for keybindings to control Amplitude

When a user presses a certain key, let Amplitude control a function.

This has been added to version 3.2 and pushed


Feature request: Show command output

If a script misbehaves, it would be helpful to see the actual output generated by the script. The old BitBar had a "Run in Terminal", which did almost this, it re-ran the command in a Terminal window. This was nice, but even better would be if xbar saved the output of the last time the script was run, and could show that on e.g. Preferences > Advanced > Show plugin output.
(Also, perhaps rename the Preferences menu option to xbar? Since it already contains items that do not really is a "preference" (like Refresh, or Quit xbar) it is not really correct. And putting "xbar" in the main menu would clearly mark the plugin as being a xbar plugin)

I would also love this. I relied a lot on running plugins in terminals with bitbar. I miss the functionality.

@magicus @ehershey How would you feel about the "Run in terminal..." menu item going inside the xbar menu for neatness? or do you use it often enough that the extra level of indirection would be annoying?

It would be annoying but still better than nothing. I used it a lot in BitBar.
I also realized a couple of workarounds/ideas that I would prefer even less than that fwiw:

Open plugins dir and run the script from there (works now).
A "Run" button in the plugin viewer window thingy that opens when I "Open plugin..." from the xbar menu (hypothetical new feature).

Oh, I definitely think that any such functionality belongs in the xbar menu. The way I see it is that plugins should really "own" the layout and design of the drop-down menu. The only concession is that xbar appends an "xbar" item at the very bottom.
The old "Run in terminal" had some flaws, from my PoV:

It used Terminal, not iTerm, and it was not configurable :)
It just re-ran the script. If you have a dynamic script, and were wondering "what on earth kind of output resulted in this display in xbar???" it was not helpful.

So showing the last output as debug info is still a slightly different thing, I believe.
I do like ehersheys suggestion of having a Run button when viewing the script, though!

Hearing the use cases is really helpful, thank you. I'll think about it.

How about seeing the script output on the plugin page so you can see huge errors with ease?

Sounds like an excellent idea.

I'd love to just have a debug mode that captures the input and output for a plugin and either logs it or displays it.

I also think it would be helpful for xbar itself. For instance, perhaps you can set a crash handler (I assume this is possible in go on macOS, in at least some way) that takes the current output from the current plugin being processed, and saves it to a crashdump file, or suggests to open a github bug issue with the output included, etc.


Add CI

Apparently, CI does not run on templates?

The CI runs when I create a PR on my fork...

also, would be great to have stylua, but not strong opinion there

This PR is very old feel free to update it to the current CI of the other repos!


loosen some lifetime restrictions

I think annotating these self references is overly restrictive. Suppose I am parsing a &'static [u8] and call (e.g.) as_tagged within a function. Then 'a will get narrowed to the lifetime of the current function. This makes it impossible to return anything borrowed from the return of as_tagged from the function.
More concretely, the following compiles after merging this PR...
fn foo(i: &'static [u8]) -> der_parser::error::BerResult<&'static [u8]> {
    let (rem, obj) = der_parser::parse_ber(i)?;
    Ok((rem, obj.as_tagged()?.2.as_slice()?))
}

...but fails to compile under master...
error[E0515]: cannot return value referencing local variable `obj`
   --> src/foo.rs:3:5
    |
    |     Ok((rem, obj.as_tagged()?.2.as_slice()?))
    |     ^^^^^^^^^---^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |     |        |
    |     |        `obj` is borrowed here
    |     returns a value referencing data owned by the current function

Thanks for your contribution!


apache-airflow fails to build with python-slugify >=5.0

Describe the bug
As of https://github.com/NixOS/nixpkgs/pull/126255, apache-airflow fails to build due to the constraint here:
https://github.com/apache/airflow/blob/2.1.4/setup.cfg#L143
Notify maintainers
@bhipple @costrouc @ingenieroariel
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: apache-airflow
# a list of nixos modules affected by the problem
module:

Unable to repro this anymore, so closing for now.


Move emacs plugin to a dedicated repository

Could you create an official repository for flow-types.el? Similar to the facebook/vim-flow plugin.
I think it would improve visibility and make it easier to work on it together.

this, please. i would contribute by making a proper mode and adding my company-flowtype auto-complete. having this in the root dir of flow makes it hard to properly submit PRs.

OK cool. If that's holding you back from contributing, let's do it.

nice :)

any progress? is there anybody i can bug with this?

I can take this up
On Wednesday, March 2, 2016, FossiFoo<EMAIL_ADDRESS>wrote:

any progress? is there anybody i can bug with this?
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/flow/issues/999#issuecomment-191125923.

@avikchaudhuri @samwgoldman So is there something in progress related to this issue?

Yeah, expect something to materialize later this week.

@avikchaudhuri ping?

bump

The emacs package has been moved to https://github.com/flowtype/flow-for-emacs/. I close the issue.
See https://github.com/facebook/flow/pull/2731 for details.


httpDefaults implementation

It's really hard to use httpDefaults because of the methods now are different from httpRequest. And there is no way to set only port and protocol for example, so the .url method is not really usefull

Hello, thank you again for asking for such support.
Can you provide example where this usability issue arises?
Regards

for ex - you want to set everywhere protocol to https, after you see that your test has too much dublicates. you found the object httpDefaults, and trying to set there '. protocol', but you cant . you see the only URL setter. okay let's do it like .URL('https'), but doesn't work. why? okey you need to do it like this .url('https://'). clear. after that you see another think, the port method uses so many times too. okay let's specify it once and use everywhere. but you still have only . url() method. what you will do? okay let's try this . url('https://'+'8086')... not working. you got it?)

So, in general, the domain changes in requests, but protocol and port don't (or not so often)?
In such scenario doing something like: httpDefaults().url("https://mydefaultdomain:8080"), and then httpSampler('/my_path').host('myHost')`.
Thank you for describing such scenario, but do you think that an example as the one I provided, and inciting users to use http defaults with a base/initial url and then overwrite the specific field in sampler when host, port and/or protocol changes, might be enough?
I understand that we could provide the methods for consistency and ease jmeter users, who already expect for such fields. I am trying to understand if there is actually a simpler/shortest way of modeling the same problem. For instance, I would in general think is preferable something like:
httpDefaults()
  .url("http://myservice"),
httpSampler("/users"),
httpSampler("/products"),
httpSampler("/products")
  .protocol("https")
  .port(8081)

than following one, which requires more code and would in general be the way jmeter users would by default model it (I believe):
httpDefaults()
  .portocol("http")
  .host("myservice"),
httpSampler("/users"),
httpSampler("/products"),
httpSampler("/products")
  .protocol("https")
  .port(8081)

I am still pretty sure there is a case I haven't thought yet that would make more sense to use a format like the later than using the first approach.
What do you think? Maybe just to ease jmeter users is enough? but I would like to understand more about the scope.

i create the issue only when i faced with problem and discuss it with team. so there is real case with different host (few services and mock). my opinion is simple - similar objects should have similar methods.

but i am not really sure how implement it better :)

Hello,
We have added requested methods in latest release.
Please let us know if is what you expected.
Regards


Sr3 log stdout

change data-consumer and wis2node to write their logs to stdout.   so now the docker logs works as docker people expect it to. First step in containerizing each config.  No more log volume needed.
Future step would be to parametrize docker/data-consumer/Dockerfile to be used for all sr3 configs.
if you look in docker/data-consumer/Dockerfile... there is only one line change for each config:
This line:
COPY ./config/watch/incoming.conf  /root/.config/sr3/watch/incoming.conf
will need to be changed to create a container for each config (the gisc ones as well as the ones in wis2node.)
Then each will run in thei

Replaced by #85


Call API from code not working

Unable to call API via code even the example get's crashes from the "kitchen ensemble app"
Screenshot:

More details:

@Ammar-Ishfaq thanks for reporting the issue. The issue was caused by calling an API that was not defined. The example is updated to call the correct API and it is working as expected. We will enhance the error messaging so it is clear what the issue is.
Do you have this issue in your own app? if so, please let us know.

Issue created for enhancing the error message:
https://github.com/EnsembleUI/ensemble/issues/259

closing it as it is not an issue. We went through it in the meeting.


Implement component name completion inside html! macro for IntelliJ Rust plugin

Description
These changes bring completion of component names inside an html! macro invocation:
https://user-images.githubusercontent.com/3221931/203096240-740557cc-1912-4863-a330-4453d7da974e.mp4
The behavior of html macro has been leaved absolutely unchanged in the case of usual usage during compilation.
How to check it out
You need CLion or IntelliJ IDE with Rust plugin version not lower than 0.4.184 (which includes https://github.com/intellij-rust/intellij-rust/pull/9711)
[dependencies.yew]
git = "https://github.com/vlad20012/yew.git"
branch = "intellij-rust-completion"

use yew::html;

pub struct MyComponent;

fn main() {
    let _ = html! {
        <div>
            </*caret*/>
        </div>
    };
}

Implementation details
When a user performs code completion, IntelliJ Rust duplicates the current file, inserts a dummy identifier at the current caret position and then expands a macro invocation at the current caret position. After that, it maps the caret position to the expansion (based on spans) and then performs code completion on the expansion.
If the macro expansion fails (expands to compile_err!(...)), which is usual when a user is typing, IntelliJ Rust can't provide any completion.
yew::html! expansion fails in any cases of invalid syntax. For example, unmatched tag/component names:
html! {
    <MyC/*caret*/>

    </MyComponent>
}

In such cases html! expands to compile_err!(...), hence the user is left without completion :(
This PR is based on the fact that since https://github.com/intellij-rust/intellij-rust/pull/9711 IntelliJ Rust sets these environment variables when invoking a proc macro during completion:
RUST_IDE_PROC_MACRO_COMPLETION=1
RUST_IDE_PROC_MACRO_COMPLETION_DUMMY_IDENTIFIER=actual dummy identifier
So, I changed the behavior of the parser if RUST_IDE_PROC_MACRO_COMPLETION env is set. Namely:

Ignore unclosed tags
Ignore a close tag without the open tag
Ignore unmatched open/close tag names

Note that this behavior only works when the macro is invoked from an IDE during completion. When an IDE invokes a macro in other cases, it doesn't set these envs and hence usual parser works.
Also, with these env vars, it is possible to make fully custom completion. For example, it's possible complete standard tag names like div (but this PR does not include this).
Inspired by https://github.com/rust-lang/rust-analyzer/issues/7402#issuecomment-770196608 (thanks to @ogoffart & @matklad)

https://matklad.github.io/2022/10/24/actions-permissions.html

Interesting approach. Before reviewing, can you perhaps clarify

[...] inserts a dummy identifier at the current caret position and then expands a macro invocation at the current caret position. [...] yew::html! expansion fails in any cases of invalid syntax

html! {
    <MyC/*caret*/>

    </MyComponent>
}

Would this be expanded as
html! {
    <MyC INTELLIJ_DUMMY_SYMBOL>

    </MyComponent>
}

or

html! {
    <INTELLIJ_DUMMY_SYMBOL>

    </MyComponent>
}

In https://github.com/rust-lang/rust-analyzer/issues/7402#issuecomment-770196608 you mention the possibility to communicate "valid" expansions back to intellisense. Is this sort of thing possible for the IntelliJ plugin?

@WorldSEnder
html! {
    <MyC/*caret*/>

    </MyComponent>
}

Guessed wrong :D
It will be expanded like this:
html! {
    <MyCINTELLIJ_DUMMY_SYMBOL>

    </MyComponent>
}

In https://github.com/rust-lang/rust-analyzer/issues/7402#issuecomment-770196608 you mention the possibility to communicate "valid" expansions back to intellisense. Is this sort of thing possible for the IntelliJ plugin?

Of course! If a macro expands normally and correctly sets spans for output tokens, IntelliJ Rust looks through the macro call at the expansion and provide some IDE features there (like highlighting and, of course, completion). It the case of yew::html!, completion works in Rust code blocks (inside {} braces) right now without this PR.

@hamza1311 done!

@hamza1311, well, at the moment I don't see how such advanced features could be achieved using such an approach. We likely need a special IntelliJ plugin with yew integration.

Exactly. Writing plugins specifically for proc macros would need to be supported by IntelliJ Rust

@hamza1311

well, at the moment I don't see how such advanced features could be achieved using such an approach, unfortunately. We likely need a special IntelliJ plugin with yew integration.

I realize this is a bit off-topic for this issue  in this repository, but the ideally, ide integration shouldn't not need to have special plugin for each possible proc-maco.  What I was discussing with matklat that lead to  https://github.com/rust-lang/rust-analyzer/issues/7402#issuecomment-770196608 was specifically that the yew html! macro, and when run through an ide, would look for the MyCINTELLIJ_DUMMY_SYMBOL (or whatever its name is, probably something more neutral, such as  like MyC__IDE_INTEGRATION_COMPLETION) and then could expand its macro to the list of available token, so the proc macro output would be like  __ide_integration_completion![ "MyComponent", "MyController", "MyC" ]
and the yew's html macro implementation would also be able to recognize the torken in the closing tag ( </__IDE_INTEGRATION_COMPLETION> ) and reply with __ide_integration_completion![ "MyComponent"]
Of course, the whole protocol needs to be well defined so we can do completion items, highlight, gotodefinition, and so on.
But for sure the first thing we can do now as macro authors is to be a bit more resilient to errors in macro implementation, and still try to produce the most possible rust code before throwing a compilation_error.

@ogoffart The features that you describe can be done in the scope of this PR, I guess (even without a special __ide_integration_completion thing that IDE have to understand).
But not the features that @hamza1311 explains in this comment, because that is not about completion at all.

@ogoffart Let me explain. With this PR, when you invoke completion with such a source:
html! {
    <MyC/*caret*/>
}

the macro will see this input:
html! {
    <MyCINTELLIJ_DUMMY_SYMBOL>
}

Since the macro understands it's invoked during a completion inside an IDE, is uses less strict parser and eventually expands to something where MyCINTELLIJ_DUMMY_SYMBOL is used as a type:
{
    ...
    let _ = |_: MyCINTELLIJ_DUMMY_SYMBOL| {};
    ...
}

Of course, the macro sets correct span to the toke in the output.
Then, an IDE see the token in the output in performs completion in that context. This is why we can complete items that out of scope of the macro:

You don't need __ide_integration_completion approach for this, and, furthermore, such behavior is unachievable with __ide_integration_completion approach because a macro can't what names are defined outside of the macro (MyComponent1, MyComponent2) and hence it can't provide completion for them.
Let's take a look at a more complex example with auto-import:
https://user-images.githubusercontent.com/3221931/203565198-b92b6167-7582-4cd9-a50b-cacec4371306.mp4
This is unachievable with __ide_integration_completion not only because the macro don't know anything about MyComponent1 but also because applying completion must lead to the import insertion outside of the macro!
Such tricks works well with closing tag just because they're correctly mapped with expansion as well:

What is not done in this PR is completion for standard html elements:
html! {
    <INTELLIJ_DUMMY_SYMBOL>
}

You likely want div, span and other tag completion here. This is where __ide_integration_completion approach may seem useful, but actually it looks like there is a better approach. If a macro understands that it's invoked during a completion inside an IDE (it is) and that a dummy identifier is placed to the tag name position (it can figure it out by seeking for dummy identifier name), it can expand to something like this:
{
    struct div;
    struct span;
    ...
    let _ = |_: MyCINTELLIJ_DUMMY_SYMBOL| {};
    ...
}

And the IDE will pick up these definitions of div and span and show them in the completion list just because these names are now in the scope. No need for special __ide_integration_completion syntax!

What @hamza1311 suggests is automatic insertion of the corresponding closing tag when a user types >:
https://user-images.githubusercontent.com/3221931/203568049-6bbf3c8a-7c65-4442-8a12-46d64c5f191d.mp4
Or automatically adjust a closing tag when editing the open one:
https://user-images.githubusercontent.com/3221931/203568739-42d8e9e7-6158-47b8-a3eb-23ecf1d8e652.mp4
Completion is just not involved here. So I think the only way to fulfill these feature is implementing a special plugin

I'm sorry for being so late with the response. I was on vacation without a PC :sweat_smile:

I'm not sure how IntelliJ is able to figure out the prop definitions for components with props, considering it's impossible for the macro to know about the props struct. It would be great if it could auto complete too.

I'm going to try making it in a separate PR right after this. I think in this case the macro should expand to a completely different code that puts the dummy identifier in such specific context where the completion will work as expected.
struct FooComponent {
    attribute: i32
}

...

html! {
   <FooComponent attr/*caret*/>
}

In this case, we could expand the macro to
let _ = FooComponent {
    attr/*caret*/: ()
}

Then, when an IDE will invoke completion in such expanded fragment, it will show only fields of the FooComponent struct in the list

Then, when an IDE will invoke completion in such expanded fragment, it will show only fields of the FooComponent struct in the list

FYI: this would need to show fields of FooComponent::Properties. Using FooComponent is wrong as it's not supposed to be constructed directly, instead Component::create function is used


unable to see element if i do not use height property in CardForm

I want to use CardForm Component in my app.
<CardForm
onFormComplete={(cardDetails) => {
console.log('card details', cardDetails);
setCard(cardDetails);
}}
style={{height: 200}}
/>
in Above I am not able to see stripe elements untill and unless provide the height property in style Attrible.
is there any solution that i see the element if i Not provide the height.

Hi! Can I ask why you don't want to provide the height? This is a fairly common practice with UI elements in React Native, I think

I added the button just below the CardForm, and if I select the country for which postal-code required, then this postal-code field doesn't show because of the below button.
is there any solution for that...

Hm, well one option would be to place your components inside of a ScrollView so that you can give CardForm the appropriate height, or you could use CardField which allows you to collect card details on just one line.


Failed to get the image?

Hello!
I'm confused when i use this url.
"http://img0.imgtn.bdimg.com/it/u=1633104621,3019537675&fm=21&gp=0.jpg"
Did i miss something?

You response is a 403 Forbidden, which means the request itself is valid but some restriction is applied and the server refused to response it.  It is usually due to your server is doing some validation on the request and it thinks that the request does not have permission for it.
You may need to investigate what is the restriction. I guess the server might be doing validating on some header field like user agent. If so, you can use the requestModifier of ImageDownloader to modify your request to match the server exception as a workaround.

Thanks! That works for me.


native: update action sheets for consistency

This PR updates all action sheets for consistency with design + adds fixture previews.
Retry send sheet:

Profile sheet (group admin + default versions)

Group preview sheet (various states)

Group join request sheet

Edit section name sheet

Delete confirmation sheet

Create channel sheet

Channel sort sheet

Add gallery post sheet

Attachment sheet

Great, I see we have a form element now as well.


Update sql-database-service-tiers-dtu.md

This line:
Compute sizes are expressed in terms of Database Transaction Units (DTUs) for single databases and elastic Database Transaction Units (eDTUs) for elastic pools. For more on DTUs and eDTUs, see DTU-based purchasing model?
Does not need a question mark at the end, just a period.

@qnal : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@qnal - thanks for the edit!
#sign-off


Patch/collision issues

Made a new ground detection system to prevent various collision issues

Closes #92 #98 #104. turns out they are all the same issue

Closes #97

Closes #105

Closes #107


Ionic 2 Support

Hello,
I realise that this project is not core to Ionic and the wish is for it to become a community one, I was wondering if anyone is going to port it over to Ionic/Angular 2?
Thanks,
Ross

That would be amazing.

+1


Added second constructor

Hello,
I added a second constructor to be able to have a custom pin wiring instead of the default.

LGTM. Thanks again!


Decluttering conflict within feature styles (icon, label, shapes)

When using decluttering in a vector layer, also multiple styles within a single feature have impact on each other.
For example a point that includes a label and an icon, is not displaying the label, when the icon would overlap it. Or a point with a CircleStyle and a label might run into the same thing, hiding the label if the radius of the circle is too big.
In my opinion within a feature, the decluttering should be deactivated, because it is quite possible that a label is supposed to overlap the corner of an icon with transparent background for example.
The decluttering should just affect the conflicts between features, imo.
If you think the current behavior is alright, please close this issue, it's just my 5cents.
Using ol 6.0.1

Use a single style object containing the text and image styles.  If you separate them in a style array they will conflict and one will declutter the other.

Thanks for the hint, unfortunately this is not possible in every case.
For example I have a point feature with an icon and want to display the point together with the icon. Both use image:  so I would have:
new Style({
   image: new CircleStyle({...}),
   image: new Icon({...})
})

which obviously throws an error. Also, for more complex situations (eg modifying) you might want to show different geometries for the same feature (once the center, once the vertices, once the outline etc).

@Chris1234567899 In v5, it would have been possible to achieve what you want by using a MultiPoint geometry. But since #9870, it is not any more. When you look at the comments in #8954, you'll see why that change was made.


Web: Geolocator.isLocationServiceEnabled() not supported: UnimplementedError

🐛 Bug Report
Using Flutter for Web:
Using Geolocator.isLocationServiceEnabled() throws UnimplementedError.
This should either be implemented or documented.
In the docs there is only a mention of other methods and with different expected errors.
Configuration
Flutter for Web, beta channel, latest version.
Version: 7.01
Platform:

[ ] :iphone: iOS
[ ] :robot: Android
[x] Web

This issue is fixed with the release of geolocator_web version 2.0.3.


Fix certification via key hosted in S3

As discussed in https://github.com/Miserlou/Zappa/issues/294
Sorry I haven't been able to spend any time digging into placebo today, but I tested that this works functionally on the real Flask API I am working on.
Hopefully I can play around with placebo more and help with that in a subsequent PR. Also, sorry about the stripped trailing spaces in the diff (my editor automatically does that on save).

Excellent, just waiting for build before merge.

Coverage remained the same at 81.715% when pulling 3593fe37e9f09318cb8a20154f123f0a31929a09 on smoll:fix-certify-from-s3-key into 9207001e818bf9b49367d964c63d229bbfad8f3d on Miserlou:master.


sass/scss extension

For convenience, I suggest adding sass and scss files.

... But why? You can import CSS into SASS and SCSS files.
@use "the-new-css-reset/css/reset.css";


[Home] Fixed problem with external link in Stack Overflow question component

Hey, I just made a Pull Request!
Issue #11674
Added className property to  component in Stack Overflow Plugin and file with css styles.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Hi! Remember to DCO sign all of your commits, check out the bot comment above.

Hello @arsentos I made some other changes in https://github.com/backstage/backstage/pull/12024 (changed the Link component to wrap the entire list item and not just the text). This fixed the issue too! So no need for custom css 🎉  Thank you so much for opening this PR though!


docs: capitalize text

🔗 Linked issue

❓ Type of change

[x] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Hot Module Replacement is written with capitalize in both vite and webpack. I think it would be nice if it is written in capitalized way.
https://vitejs.dev/guide/features.html#hot-module-replacement
https://webpack.js.org/guides/hot-module-replacement/

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

I appreciate the PR. There are two valid title casing conventions and we are moving to only captilize first charrs across docs, this convention is already applied to nitro docs headers.

Ok @pi0. Thank you.


requests.exceptions.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED]

This is not a bug in the library, but a misconfiguration on the server side:
See https://mediafire.mantishub.com/view.php?id=22 for more information
MediaFire API returns download links with http schema (i.e. plaintext, not encrypted).
mediafire.client is programmed to perform http to https schema swap on the URLs returned by the API to make it harder for a download to be intercepted.
Recent MediaFire certificate change left the download servers without a chain, so the SSL certificate cannot be properly validated.

Resolved by MediaFire team


Include breaking changes in changelog

When upgrading PDF.js (using the pdfjs-dist repo) it can be quite difficult to figure out what breaking changes were made in the last release. It would therefore be useful to include a little section in the release that indicates which of the changes were breaking and how we can migrate our code to work with the newer version.
Also just overall a more descriptive changelog instead of a list of PRs that were merged would be really nice.

Each new pdfjs-dist release, as found at https://github.com/mozilla/pdfjs-dist/releases, will link to the relevant release page in the main repository (i.e. directly to a release as found at https://github.com/mozilla/pdf.js/releases).
To find important and/or breaking changes in a release, please look for the strings [api-minor] and [api-major]. Generally speaking, the relevant PRs should/will have a description of the changes that were made, and how to have adapt your code. Additionally, for API changes, there will also be a deprecated message printed in the web-console with (some) migration information, when you're using older functionality.

Also just overall a more descriptive changelog instead of a list of PRs that were merged would be really nice.

Most of the somewhat recent releases[1] haven't had a particular focus, meaning that it's probably not easy/meaningful to summarize them any other way than as a list of the actual changes that were made; see also https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#version
Furthermore, given that most contributions to PDF.js are made by unpaid volunteers, while compiling better change-logs (where it makes sense, given the above) would obviously be nice it could however be that finding the time for someone to actually do so is difficult (since these sort of things tend to take a fair bit of time).

[1] With the exception of the first release on the 2.0 branch, which removed the global PDFJS object.


.Net: Option to control temperature to stop Handlebars planner hallucinating functions.

I'm new to SK in the last few weeks and have only used the later versions (e.g. 1.2.0 and 1.0.0.-beta8).  I'm trying the Handlebars planner and finding that it is hallucinating some basic functions (e.g. "Example-Random") and in this example "length":
{{set "tables" (myCode-GetTablesInDatabase)}}

{{!-- Step 2: Choose the Right Helpers --}}
{{set "chosenTable" (range 0 (length tables) 1)}}
{{set "variables" (myCode-GetVariablesFromTable (lookup tables chosenTable))}}

It would be useful to have a way of turning down the temperature with the HandleBars planner.
This is the exception that I got:
An unhandled exception of type 'Microsoft.SemanticKernel.KernelException' occurred in System.Private.CoreLib.dll: '[HallucinatedHelpers] The plan references hallucinated helpers: Helper 'length''
 Inner exceptions found, see $exception in variables window for more details.
 Innermost exception      HandlebarsDotNet.HandlebarsRuntimeException : Template references a helper that cannot be resolved. Helper 'length'
   at HandlebarsDotNet.Helpers.MissingHelperDescriptor.Invoke(HelperOptions& options, Context& context, Arguments& arguments)
   at HandlebarsDotNet.Helpers.MissingHelperDescriptor.HandlebarsDotNet.Helpers.IHelperDescriptor<HandlebarsDotNet.HelperOptions>.Invoke(HelperOptions& options, Context& context, Arguments& arguments)```

Yep, we have been mentioning this hallucination and handlebar planners imagining the syntax, so they cannot be trusted but as an intermediate step to generate a plan just to review, polish and fix later... but not to use it right away.


repr(sharded_device_array) should print ShardedDeviceArray

Just noticed that we had a trivial regression in how repr(sharded_device_array) works:
In [1]: from jax.api import device_put_sharded, devices

In [2]: device_put_sharded([1], devices())
Out[2]: DeviceArray([1], dtype=int32)

We use repr(x) to show the type of x, e.g. in demos, so we probably want to put this back.

Ah, it's just this change. @jblespiau to answer your question in that TODO, yes we should put it back ;)


[doc] Venice Improvement Proposal -- Design Doc process

[doc] Venice Design Documentation Process

Based on Venice open source contributers sync, we aligned on a design document process inspired by Apache Pulsar. Please read the design doc guide for more details.
How was this PR tested?

Build Complete, Doc change no functional changes.
Does this PR introduce any user-facing changes?

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

Awesome document.
Overall LGTM but I would add a section about 'Rejected altenatives'

Thanks for chiming in @eolivelli! The alternatives, including rejected ones, are currently tucked under the design section. Though maybe we should put them in a section of their own? IDK.
I guess each structure may be ideal for different phases of the design process. In the discussion phase, it may be best not to bias reviewers too much towards a specific alternative, so that the best one emerges based on its tradeoffs, rather than based solely on which section it's in. Whereas for accepted proposals, we may want to highlight more clearly the chosen versus rejected paths. IDK, what do you think?


Added AWS Containers from the Couch Session

I've added my recent live session about ESO from the AWS Container from the Couch YouTube channel
Problem Statement
What is the problem you're trying to solve?
Related Issue
Fixes #...
Proposed Changes
How do you like to solve the issue and why?
Checklist

[ ] I have read the contribution guidelines
[ ] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

Awesom! lgtm!
But you will need to rebase

/approve


Fix: Return helpful error message when get structure fails for mongo db due to lack of read permission.

Description

return helpful error message when get structure fails for mongo db due to lack of read permission. The message returned is: Appsmith has failed to get database structure. Please provide read permission on the database to fix this.
any get structure error is also generically being caught by the getStrucuture method in DatasourceStructureSolution.java
error msg and status code is now returned as part of DatasourceStructure object.

ErrorDTO is used to store error info.
ErrorDTO definition is moved to appsmith-interfaces package so that it can be used all packages.

Other changes:

stop logging PLUGIN_GET_STRUCTURE_ERROR externally, as this error is mostly triggered by credentials error or lack of read permission.
start logging PLUGIN_GET_STRUCTURE_TIMEOUT_ERROR externally, because this is not a user error.

Fixes 1479
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

manually
Test case

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=d87945

/ok-to-test sha=ea988e9

/ok-to-test sha=ea988e9


Update luau to 0.638

Before adding new features and new modules, please go to issues to submit the relevant feature description first.
Write good commit messages and use the same coding conventions as the rest of the project.
Please commit code to dev branch and we will merge into master branch in feature
Ensure your edited codes with four spaces instead of TAB.

增加新特性和新模块之前，请先到issues提交相关特性说明，经过讨论评估确认后，再进行相应的代码提交，避免做无用工作。
编写友好可读的提交信息，并使用与工程代码相同的代码规范，代码请用4个空格字符代替tab缩进。
请提交代码到dev分支，如果通过，我们会在特定时间合并到master分支上。
为了规范化提交日志的格式，commit消息，不要用中文，请用英文描述。

mingw can be fixed by using this patch on isocline which is embedded in the project: https://github.com/xmake-io/xmake-repo/blob/dev/packages/i/isocline/xmake.lua#L13

Thank you.
I don't know why android broke from nowhere now, maybe a xmake change?

Android r27 seems to break some packages.

the fix for android seems to be to require at least cmake 3.6: https://github.com/android/ndk/issues/2032

Note that vcpkg suffers from the same issue: https://github.com/microsoft/vcpkg/pull/40339
I agree to roll back the NDK version update.


Cobb County, GA Parcels

http://www.cobbgis.org:81/arcgislb1/rest/services/tax/cobbparcelsmap_wgs/MapServer/3

Looks like there is a "House Numbers" source there, too:

http://www.cobbgis.org:81/arcgislb1/rest/services/tax/cobbparcelsmap_wgs/MapServer/0

Fills big hole in Atlanta area!

Some PR fixed this.


Firebase 和 PubNub相比较怎么样？

Is the bug present in the demo Chat SDK project?

What modifications have you made to the Chat SDK?

Android Version:

Steps taken to reproduce the problem:

Expected result:

Actual result:

Comments:

https://chatsdk.co/mobile-messaging-service-price-comparison/


Reduced usage of textureSize() in shader

This PR attempts to resolve the issue  https://github.com/godotengine/godot/issues/83894
The field introduced to cache emission_tex_size = textureSize(...) inside struct DisplayParameters may not be the ideal choice, but it requires less amount of changes. Open to further suggestions on how it could be refactored. :smile:

Hey I was looking at the source for this issue as well, but didn't really know how to go about wrapping my head around it - I'd appreciate if you could share some of your thought processes while making this change!
It seemed strange to me that we were writing code to a string, but then I can't seem to find where it gets used after RS::get_singleton()->shader_set_code(shader_data.shader, code);
How would one get started with dealing with this "meta" code? For instance how would we test compile it for syntactic errors?
I'm also not sure where the definition of textureSize() is, it feels like because we're dealing with String, it feels like it would be very hard to investigate/explore the code.
Thanks a bunch!

Hey I was looking at the source for this issue as well, but didn't really know how to go about wrapping my head around it - I'd appreciate if you could share some of your thought processes while making this change!
It seemed strange to me that we were writing code to a string, but then I can't seem to find where it gets used after RS::get_singleton()-shader_set_code(shader_data.shader, code);
How would one get started with dealing with this "meta" code? For instance how would we test compile it for syntactic errors?
I'm also not sure where the definition of textureSize() is, it feels like because we're dealing with String, it feels like it would be very hard to investigate/explore the code.
Thanks a bunch!

The task was  to simply store call  to textureSize() for later use and analyzing branches taken for it. The code written inside string is a custom shader code used by Godot which resembles mostly to GLSL.
The string is compiled by the shader compiler, at later stages, which checks for its syntactic correctness. textureSize() is built-in function of GLSL.
Reading the code, modifying the code and re-building the engine works quite well for testing and debugging. Familiarity with graphics pipeline and rendering APIs like OpenGL/Vulkan helps a lot.


Add feature matrix to README and Symbolizer documentation

This change adds a feature matrix for the symbolization functionality to the README as well as the Symbolizer type. It depicts which features are available in supported formats and whether blazesym supports them.

I don't know if long-term we want to keep this matrix in the README, but for now let's have it there as well.


Bug report: Doesn't export metrics about log files if using timestamps

If log_timezone is set to true in exim configuration, the log format changes from
YYYY-MM-DD HH:MM:SS MAILID-MAILID-ID flag whatever whatever

to
YYYY-MM-DD HH:MM:SS +ZZZZ MAILID-MAILID-ID flag whatever whatever

(where flag is any of =>, <=, **, ==, etc etc)
This breaks checking the flags and generating exim_messages_total since the index (checked on lines 288-296) is offset by one.

Thanks for the report. I've verified the issue is reproducible.

Sorry, this got lost behind some feature work, and I should have just shipped it with a patch release... which I've done now with v0.6.2. Thanks again!


Always run sw-precache's fetch handler before sw-toolbox's

R: @addyosmani @gauntface @wibblymat
This reorders the sw-toolbox inlined code, to ensure that its fetch handler only runs if sw-precache's handler does not call event.respondWith().
This addresses an issue we were seeing in https://github.com/Polymer/docs/pull/1641 and fixes https://github.com/GoogleChrome/sw-precache/issues/128 as well.
There's a chance that folks were relying on the previous behavior (though I don't think that's a great use of sw-precache if they were...), so I'll call this change out explicitly in the 4.0 release notes.

This largely seems ok to me, although difficult to fully appreciate given it looks like template imports to me.

Yes, it's just shuffling around the order of things in the template.


Proposal: Canonical Approach with latest branch

This PR changes the current version docs to latest and adds canonical links to the old docs (where it is easily possible).
Instead of copying the folder and name it for the new version the process would look like:

copying latest for the old version
change the alias in index.md of latest to the new version

All links will stay the same, the only issue which might arise is the CLI update, which we would need to investigate specially.
Also this allows for easier version specific changes/updates.
:information_source: this is just a proposal currently - it would fix some back link issues, it is not perfect, but the canonical URLs, will point to the latest version all the time, therefore SEO should always point to the latest release (at least where we can map it easily, not working for moved pages)

seems like my page exists check is broken, I am gonna fix it :)

lgtm but @mowies should review it.   Also note that, for LTS, we are going to move the Concepts, Installation, etc sections under LTS or 1.x or something so they will all be at the top level, which means that we probably do not need to redo all the xrefs to use "latest" rather than the actual release number.  But having latest in the URL gives us a lot more flexibility for the future and may even mean that we can silently roll the LTS docs into the new doc engine without breaking user bookmarks and such.

https://developers.google.com/search/docs/crawling-indexing/consolidate-duplicate-urls is a good read from google regarding canonical URLs


Works with all playlists except my own?

Hey guys, sorry if this is an issue with my account or something, but the player will load with every playlist I can find on soundcloud except the ones attached to my client's account. His playlists are set to be public.
Any idea why this wont load?
Here is a link to the soundcloud playlist
https://soundcloud.com/dieguisproductions/sets/rock-n-indie-songwriter-1
and a codepen test:
http://codepen.io/handhugs/pen/NrpQdo

Ok i found the problem - the playlist was set to public but the individual tracks were set with the "enable app playback" was disabled. In case this happens to someone else.


⚠ Community has degraded performance

In d06421e, Community (https://community.wrk.com/health) experienced degraded performance:

HTTP code: 200
Response time: 187 ms

Resolved: Community performance has improved in 89eea48.


[camera] Coordinates set by setFocusPoint and setExposurePoint are only valid in a single orientation.

As of right now, the setFocusPoint and setExposurePoint methods in the camera plugin do not correctly rotate the supplied coordinates according to the UI orientation, causing them to only show correct behaviour in one orientation. For Android this is landscape left, on iOS portrait up. This can be reproduced by running the example app in any other orientation, as it can be seen the camera will focus somewhere other than where the preview widget was tapped. (Be sure to lock the autofocus first on Android)

A fix for the Android side is already included in part 9 of the Android refactor that is currently going on (PR flutter/plugins#4059)
I am currently submitting a PR for the fix for iOS. A fix for iOS has now been submitted merged in flutter/plugins#4158

Closing as both fixes for this have been merged.


Performance issues

Hi
I have had impressionist running for a while and the impressions table is currently at about 220.000 rows. This has made my site pretty slow.
Do you have any tips on how to increase the performance? I have all the default indexes as well as the counter cache.
I guess it could be that I'm only recording unique  impressions so it has to look through the database records before creating a new impression...
How do you handle these issues?
I'm running on Heroku using Postgres.
For instance, queries like this isn't very funny:
 (13684.8ms)  SELECT DISTINCT COUNT(DISTINCT "impressions"."request_hash") FROM "impressions" WHERE "impressions"."impressionable_id" = $1 AND "impressions"."impressionable_type" = $2  [["impressionable_id", 60], ["impressionable_type", "Artist"]]

This was on my local machine (which is a bit slow) but anyway....

test

As a workaround for it; I've made a backup model, basically copying the impressions model (adding an extra column for the original id) and moved there all the registries from before 4 months ago using a rake task. That helped a lot and maintained the overall functionallity over the site.


Can there be direct links to heading?

For example, https://pwa.tips/codelabs/serviceworker#how-to-register to

Not sure if that would be possible with client-side rendering but would be nice, since this is documentation.

@sarbbottam Yep you are right. Since we are using react-router we can have a #hotlink like above. But we will if I can find a workaround.

I will give it a try. I implemented something similar at http://sarbbottam.github.io/empower-your-users/tab. It will nice if we can do it.


Update generic_utils.py

invalid operator / for 'list' and 'int'

What's the use case? Could you clarify what this is supposed to achieve?

There was an error while importing the keras - generic_utils.py as mentioned above this error was rectified by using sum() since we are avg


too many bugs. totally unusable.

much much much much too many bugs. totally unusable .... there is anyone on earth who gets to do something with shineisp ???

yes of course :)


Implement callbacks and streamable subset option

Implements seek and tell callbacks so that the stream encoder can insert metadata at the start of the file.
Implements metadata callback, although currently the callback just receives a blob of cdata.
Makes streamable subset optional to allow the use of the full format settings.

This is an example of using the new things:
import numpy as np
import pyflac

class ExampleFLACEncoder(pyflac.StreamEncoder):
    def __init__(self, filename):
        self._filename = filename
        self._output_file = None
        super().__init__(
            write_callback=self.write_callback,
            seek_callback=self.seek_callback,
            tell_callback=self.tell_callback,
            metadata_callback=self.metadata_callback,
            sample_rate=48000,
            blocksize=65535,
            streamable_subset=False,
            verify=True,
        )

    def write_callback(self, buffer: bytes, num_bytes: int, num_samples: int, current_frame: int):
        self._output_file.write(buffer)

    def seek_callback(self, offset):
        print('seek', offset)
        self._output_file.seek(offset)

    def tell_callback(self):
        offset = self._output_file.tell()
        print('tell', offset)
        return offset

    def metadata_callback(self, metadata):
        # Currently this is just a big blob of cdata.
        print('Metadata:', metadata)

    def __enter__(self):
        self._output_file = open(self._filename, 'wb')
        return self

    def __exit__(self, exc_type, exc_val, exc_tb):
        self.finish()
        self._output_file.close()

    def write(self, b):
        self.process(np.frombuffer(b, dtype=np.int8))

def main():
    with ExampleFLACEncoder('test.flac') as enc:
        for n in range(100):
            enc.write(b'\x00\xff'*1024)

if __name__ == '__main__':
    main()

Pull Request Test Coverage Report for Build 789706439

15 of 23 (65.22%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.2%) to 96.44%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyflac/encoder.py
15
23
65.22%

Totals

Change from base Build 777692000:
-2.2%

Covered Lines:
298

Relevant Lines:
309

💛  - Coveralls

I have implemented seek and tell on StreamDecoder. However these are not useful unless user also defines the read callback. Currently that is not possible as it is handled internal to StreamDecoder and as a result the user can't respond to the seek callback by seeking on the file pointer, because there is just an internal memory buffer. Fixing this would require completely changing how StreamDecoder works. You could not have a process method and implementing the read callback would become mandatory in user code.

I see, ok well maybe we should revert commit 243203d. As long as both encoder/decoder data callbacks are now called write_callback this makes sense. I can take care of this if you're ready to pull this out of draft status and merge in?

I have dropped https://github.com/sonos/pyFLAC/commit/243203d0eda3c5f4ea86dc2b2404a52c5a87dd72 and updated the changelog and version so this can be merged. I would like to make the necessary changes in decoder at some point, but it could be a long time before I get around to it.


adds a pr template

Description
A straight forward PR template.
Notes

Open to discussion and changes
Let's get everyone's approval to move forward.

Issue (if applicable)
n/a
Testing

Will test once merged in develop and next PR is open.
should have useful sections for reviewer
should have useful descriptions for author

Screenshots (if applicable)
n/a

LFG 🚀


Add non-interactive mode, allowing commands to be called directly.

Create functions for each quick-stats option. Then, allow direct execution of the function from the top-level (non-interactive).
This uses a simple switch statement to select functions, which means there is some duplication when picking / naming / printing usage. A function table might be better, but to be honest I'm not aware of bash best practices.
Tested by running through each option directly, then running through the interactive menu (which has had its inline code replaced by function calls, where not already replaced).

Why was this closed? It's backwards-compatible to the interactive mode.

Yes, your change a was implemented in master branch


[Enhancement] Please add proxy config

please add proxy config command.
such as
cinst config proxy.type   http
cinst config proxy.ip    <IP_ADDRESS>
cinst config proxy.port 8123

do not force me change ie proxy setting by hand.
chocolatey should had it's own proxy setting

Closing for now, this isn't really something we want in there yet. If this is needed, I'm sure it will come up again in the new choco.


Plot output not full

$ seq 1 200 | plot -c C -x 5:0:0:3:blue -y 10:3:3 -d 50:200 -b 0:250
Seems to give only first 128 values.

working good with changed FILE_INPUT_BUF

Hello. Yes, working good now. Thank you

Awesome, I've just bumped the version


add error message middleware

This could be done as a sample versus an actual piece of middleware we ship but we need a piece of middleware that you install first in your bot and it listens for an error to be thrown and then simply sends a message to the user telling the something wen't wrong. Other middleware on the stack should have already processed/logged the error so this middleware is just about telling the user something happened.
I think the general strategy we want for all middleware error wise is to handle the error and then re-throw it so someone else can handle it.
In JS I think this middleware looks like:
bot.use({
    contextCreated: (context, next) => {
        return new Promise((resolve, reject) => {
            next().then(
                () => { resolve() },
                (err) => {
                    context.responses.push({ type: 'message', text: `Oops... Something went wrong.`});
                    reject(err);
                }
            );
        });
    }
});

@Stevenic, is this something we need to keep a tracking issue around?

I have written the c# version of this middleware.  Once the wider issue of middleware not currently being able to store its own conversation state (see #248), I will finish the last couple of bits.  I am working to reproduce the various bits of middleware @Stevenic has in his toybox in C# over at https://github.com/garypretty/botbuilder-dotnet-toybox.
@cleemullins I do have some thoughts on how to handle middleware that we don't want to include in the main repo, so let's discuss that offline.
Not sure we need an issue open here as I have this on my radar.

Thanks Gary!


Devstack changes for Haproxy lbaasv2 support and PaloAlto Image

@njagadish please review this

Changes are fine for advanced mode. Need to test enterprise mode and revert the user data changes if it wont work.


GEODE-5803: Remove geode-core:distributedTest dependency on *:integrationTest:outout

Built from 6fc60af of PR #2558
Adds JSONFormatHelper to relieve cross-module test source set dependency.  Moved to classes to resolve this as well.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[SEE ABOVE] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[SEE ABOVE] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[n/a] Have you written or updated unit tests to verify your changes?

[n/a] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Shifting resources wasn't as robust as I had hoped.  Closing for now.


Eslint from jscs and jshint & cleanup

So shiny and clean! :) Really like the AirBnB standard now that I see it. Thanks for staying late and getting this done!


[Event Parameter Change] Report 5881 Make Phys. Invt. Recording - Procedure InsertRecordingHeader - Event OnInsertRecordingHeaderOnBeforeInsert

We would like the Sender added to the following event.
Report 5881 Make Phys. Invt. Recording - Procedure InsertRecordingHeader - Event OnInsertRecordingHeaderOnBeforeInsert
Original:
Report 5881 Make Phys. Invt. Recording - procedure InsertRecordingHeader - Event OnInsertRecordingHeaderOnBeforeInsert

[IntegrationEvent(false, false)]
    local procedure OnInsertRecordingHeaderOnBeforeInsert(var PhysInvtRecordHeader: Record "Phys. Invt. Record Header"; PhysInvtOrderHeader: Record "Phys. Invt. Order Header")
    begin
    end;

Change:
Added Sender from the Report
[IntegrationEvent(true, false)]
    local procedure OnInsertRecordingHeaderOnBeforeInsert(var PhysInvtRecordHeader: Record "Phys. Invt. Record Header"; PhysInvtOrderHeader: Record "Phys. Invt. Order Header")
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46663.


fix(module_php): Fixed fetch of URI

Pull Request Test Coverage Report for Build 445

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.603%

Totals

Change from base Build 424:
0.0%

Covered Lines:
448

Relevant Lines:
459

💛  - Coveralls


Usage

Hi :)
Would you mind adding some basic usage examples ?

I started working on my own NeuralNet but if you know what you're doing & actively maintain this then I'll use yours and contribute here

Sure I can add some examples soon. This is still VERY MUCH a work in progress. I'm only part way through the back prop implementation right now. But I like Crystal and it's fun to build new libraries so I'd like to keep up with this project.

Cool , if so I'll try and focus my contributions here so we can have a single project which actually does something instead of 100 projects around half made :)

I don't want to discourage you from working on neural networks or Crystal. I'm happy to take PRs when this project is in a more usable state but I don't want to slow down anyone else's development while I get things off the ground.


Experimental pq: implement EdgesRequest.kind filters [phab:D736]

Created by schroederc at 2016-02-23 22:21:18:

shahms wrote at 2016-02-23 22:24:47:
Lint errors?

schroederc wrote at 2016-02-23 22:30:13:

! In (earlier comment), shahms wrote:
Lint errors?

No idea why it doesn't like the letter k.  I'm sticking to it.

shahms accepted this revision at 2016-02-23 22:31:49

schroederc closed via commit b45d4861096bda3efb81e70dff987041a9f7069e at 2016-02-23 22:32:36


Search by filter doesn't work when the upper price is left empty

Search by filter doesn't work when the upper price is left empty so maybe set a default upper price value to something big?

This issue has been resolved with commit 415ac2b. The search by filters now sets a default upper price value when the upper price is left empty.
Thank you for bringing this to my attention!


Fix incorrect city name

Update Pakistan, Pakistan -> Attock, Pakistan

@jawwad Hi, the url https://www.facebook.com/groups/free.code.camp.attock is either incorrect or the facebook group has closed.  Can you advise?

Or it is not public?

I should clarify my comment, the group might be "secret" both "open" and "closed" privacy groups can still be seen by non-members, for example the bronx group. What I meant was deleted but I had forgotten about the "secret" groups setting. So this fb group is either non longer existing, has an incorrect url entry on the directory, or has been set to privacy "secret". So we should either remove it from the directory, or correct the url.

Lets, remove this.

@jawwad we are going to remove the entry for freeCodeCamp Attock unless we are provided with a correct URL for a group with privacy "open," additionally the co-ordinates appear to be incorrect, can you confirm the correct location if the entry needs retained, thanks

@johnkennedy9147 - Sorry about the delayed response. I'm actually not sure if the group exists, I just saw that there was a mismatch between the url and the city name and so updated the city name. In this case I guess it can be removed.

@jawwad could you please either amend this PR to remove the entry or close and I will do another to remove the entry, thanks
cc @raisedadead


Create video with screenshot

sorry for my English.
I'm trying to make a video, from screenshot, but it's too slow
inside a for, I'm saving the buffer
const buffer = await page.screenshot({fullPage: true})
buffers.push(buffer)

then created a sequence that I sent to ffmpeg
var inStream = streamifier.createReadStream(newBuffer);

ffmpeg(inStream)
.inputOption('-r 30')
.videoCodec('libx264')
.format('mp4')
.on('end', () => console.log('end conversion'))
.on('error', (err, stdout, stderr) => {
  console.error(err);
})
.output('video.mp4')
.run();

this works very very slow
Do you know a better approach to achieve this? Thank you
the right way, I think it's start Screencast but I do not know how to apply it

This kind of screencast implementation is inevitably slow. Implementing fast screencast would require some protocol work, please upvote the #478 if you need one.


[Proposal] Add "evidence" of talk quality

Allow for tracking feedback on a talk. Tweets, maybe others?
adamwathan [10:23 AM]
hey good idea for symposium, storing feedback to use as part of future submissions
if i could log in to my talk and hit like "add evidence!" and paste a twitter link
that would be slick

So this matches na Idea i had and i'm starting to think it belongs in Symposium and not outside of it, but it could go either way.
Anyway, the part that is relevant here, is adding to a talk a link to a joindin talk.
With these id's you could generate a graph of evolution, get a few quotes and even list all conferences where the user spoke at.
i did something similar: http://aggregatein.azurewebsites.net/talk/annotating-with-annotations

@rdohms OH! Clever! I like that graph idea.

Thinking so far:
A "submission" is a link between a talk and a conference. (will need renaming.)
It can progress for "applied" to "rejected" or to "accepted" or to "given" (or "past" or whatever.)
It is the unique entity to represent "When I gave my Empathy talk at Laracon EU 2015" vs "When I gave my Empathy talk at the White House that one time". So each Submission would be the place to add feedback.
Currently thinking:

Submission can have a Joind In Talk URL, from which we pull reviews like @rdohms mentioned above
Submission hasMany Twitter reviews, each of which is a tweet URL
Submission hasMany plain reviews, which are just text (and a name? and an attribution??)
Submission hasMany URL reviews?  or should it just be twitter vs. plain?

I would argue against the "text" review and rather invest in getting people to do that on joind.in, so i would really make it very low priority.
I'm a bit put off by the "applied, rejected" states, for one we can't automate this and why would i want to manually add this before actually presenting the talk? The only states that r "to me" really make sense, would be "accepted or upcoming" and "done" which would be the same state but automated and time sensitive. Thus i could say "this is where i will be at" which is very cool and "where i have been"... not sure i want to say "this is where i was rejected, bastards"
I would suggest the term "performance" if we are focusing only on positives not on rejections.

The "text" review was only for if you have gotten evidence for conference that don't use Joind.in. Not the primary way to do it, for sure.
Applied, rejected, etc.. primarily so that you can mark that you "applied" to a particular conference with a particular talk. I've gotten acceptance emails from a conference and couldn't even remember which talk I submitted. Additionally, it's so you can remember what you submitted to a conference when you submit to the same conference the next year. Does that make sense? Oh, also: the rejected state is only for internal knowing, not for public showing, just read your last sentence.

Performance would be good if it's only positives; trying to think of something that encompasses:

I submitted talk and haven't heard back yet
I submitted the talk and was rejected
I submitted the talk and was accepted; haven't given yet
I have given the talk

 Add evidence of talk quality

Addressed in #392