Parent pages scroll position problem

I have a problem with iframe resizing changing parent pages scroll position.
I have:

n+1 iframes on page (admin page).
I close one Iframe and scroll page to the next one by JS (after everything is done in the last iframe)
now something triggers in plugin and browser window jumps back to the position it was before my scroll.

I think the problem is because pagePosition variable holds the coordinates and will not be updated or invalidated by the scroll I trigger myself.
I would like an option to either
a) disable this scrolling alltogether (I can't see currently why it is an function at all).
b) unset pagePosition so it would not use old cached value.
Right now I don't seem to be able to do neither.
And no the scrollCallback is not an option here because this call is not using this callback. It originates from: https://github.com/davidjbradshaw/iframe-resizer/blob/master/src/iframeResizer.js#L140

Can you create a simple test case?

I'll add it to my todo list - but I probably can't allocate time for that in near future.

I have the same problem. Opening new page in the iframe remembers the vertical position of the last page. So if the user scrolls down and opens a new page that page will be shown scrolled to that same position even if the page does not have that much content - in this case it just displays white screen.
It makes perfect sense that every time page is loaded, vertical position is reset to 0.
Any ideas of the quick fix?

You could use onInit To trigger scrolling the page to the top


Fix compile error

Fixes #1

I can confirm that for me this fix did unbreak the build, and seems to resolve #1 :tada: Thanks @marccampbell.
One note for other intrepid open source adventurers: I found that after getting the build working the tool was producing incomplete information run against 2022 firmware on the router/dish. Updating the proto bindings in the companion starlink-grpc-go repo and using the updated dependency in this tool did the trick for me.
Folks looking to do the same can use my WIP PR instead of this fix: https://github.com/starlink-community/starlink-cli/pull/3

{>>" # Hello.h \ Thank you Elon Musk the help package 📦 Ukraine 🇺🇦 STARLINK big help

I just ran into the same bug, see me (recently closed) duplicate fix: https://github.com/starlink-community/starlink-cli/pull/5.
Is this repository/project still active or abandoned?


allow customizing logfmt standard keys

as mentioned on Slack, this can be used to make simpler/cleaner logs in cases where you don't need module, group, id, etc, and are reading the logs directly rather than via an aggregator. I went for "fully customizable" (i.e. order + which keys) rather than special casing some (e.g. you probably always want msg, but might as well make it customizable).

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

Project coverage is 94.18%. Comparing base (14d6f24) to head (6900c63).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   93.67%   94.18%   +0.51%
==========================================
  Files           1        1
  Lines          79       86       +7
==========================================
+ Hits           74       81       +7
  Misses          5        5

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


Hub database displays undefined in table data fetch time

Describe the bug
The database table's data fetch time shows undefined
Steps to reproduce
Steps to reproduce the behavior:

Go to any of your project's hub database in the admin panel
Click on any of the tables so that it will fetch its rows (it will correctly display the time it took to fetch, e.g. 725ms)
Now go to some other tabs in the browser, come back to the admin hub tab after some time
Refresh the table (from just above the table header) so that it will refetch its rows
You'll see undefined along with a number

Expected behavior
It should not show undefined in place of time
Here is a screenshot of the issue (look at 3undefined. Do not worry about 0 rows, same thing happens where there are rows present)

Should be good now!


Remote*Helper classes: have create() call create(projectId, keyStream)

I don't think it's necessary to keep an issue around for this... the code isn't incorrect.


question

hi, nice project !
i was thinking about copying some of your code and use as template for packaging some binaries to guix , more specifically anki, which is hard to build so it's better to get the binary from the official website.

Thank you.
Please do and share your work.
The license is Apache ASL 2.0 .


Fix - using the new useWindowDimensions api to fix unmounting issues

Overview
Unmounting causes issues with the removal of the event listener so I have used the new useWindowDimensions hook to remove the need for a listener all together as the height and width are got dynamically now.
Test Plan
I ran yarn lint and prettify for code structure testing and tested the modal works on differing screen sizes using android studio.

Thanks for working on it! I added a comment

@mmazzarolo must have worked from a cache issue yeah sorry must have missed this. I'll change it to a functional component now :)

@mmazzarolo re-done to lint standards and I tested it locally again but let me know if there's anything else to do.

Thanks for you changes +1 I added a bunch of comments. Let me know if you want me to jump in and make them for you (as soon as I have some free time).

Hey thanks I'll get on them today and tomorrow :)

hey sorry yeah missed lots here, sorry working on multiple things right now but I am sorting it now. :)

Hi, is this PR still active? I would love to help accelerate this

Hi, is this PR still active? I would love to help accelerate this

Hi sorry currently my ability to test the branch has broken. Is there any chance you could pick these changes up when you have time as I wouldn't like to push untested code. Sorry should have kept you updated just been trying to fix the testing issues. Thanks

No worries @joe-brabben i ll pick this up
@mmazzarolo i have a question, this could;ve been fixed using Dimentions.addEventListener (https://reactnative.dev/docs/dimensions#addeventlistener) or do you want to refactor the library to functional components?

@khzouroussama I think any solution is fine as long as we solve the issue without using deprecated APIs 👍


InteractiveViewer 'scale != 0.0': is not true

Steps to Reproduce

Run InterctiveViewer sample on Android emulator
Press CTRL to make pan gesture circles appear
Start panning and collapse circles to the point, so they become the one
See the assertion failed in console

Code sample can be taken here https://api.flutter.dev/flutter/widgets/InteractiveViewer-class.html
Expected results: No error
Actual results: Error

Logs
'package:flutter/src/widgets/interactive_viewer.dart': Failed assertion: line 948 pos 12: 'scale != 0.0': is not true.

Analyzing issues...

   info - Unused import: 'package:flutter_issue/main.dart' - test\widget_test.dart:2:8 - unused_import

1 issue found. (ran in 3.8s)

[√] Flutter (Channel master, 2.3.0-1.0.pre.264, on Microsoft Windows [Version 10.0.19041.928], locale ru-RU)
    • Flutter version 2.3.0-1.0.pre.264 at C:\dev\src\flutter
    • Upstream repository git@github.com:nt4f04und/flutter.git
    • Framework revision b3f7ebe069 (15 hours ago), 2021-05-08 08:54:02 +0800
    • Engine revision f57e986aa8
    • Dart version 2.14.0 (build 2.14.0-74.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\danya\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

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

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.7.7)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.7.30621.155
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • 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-b01)

[√] IntelliJ IDEA Community Edition (version 2020.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.3.3
    • 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

[√] VS Code (version 1.56.0)
    • VS Code at C:\Users\danya\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.22.0

[√] Connected device (4 available)
    • sdk gphone x86 (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)       • windows       • windows-x64    • Microsoft Windows [Version 10.0.19041.928]
    • Chrome (web)            • chrome        • web-javascript • Google Chrome 90.0.4430.93
    • Edge (web)              • edge          • web-javascript • Microsoft Edge 90.0.818.46

• No issues found!

Still an issue in Flutter 3.24.4.


Update ilmerge since current one needs .NET 3.5 installed

Building the sln fails on a machine without .NET 3.5 installed. Updating ilmerge to latest version 2.14.1208 resolves the issue.
I think it would be better to consume it from a nuget package instead of checking it into source - https://www.nuget.org/packages/ilmerge/
I can quickly create a PR if there are no concerns with this.

See #266, ILMerge will be removed soon.
So thanks but the PR isn't needed.


Fix unstable chapter layout for codegen-backend

The section for codegen-backend in the "Unstable features" chapter was placed in the wrong location (it was below the "stabilized features" list.
Additionally, the crate-type feature was stabilized, but not removed from the unstable listing.

@bors r+

:pushpin: Commit 36df68903beed8b1fec2750aea12a4821b3ad7cb has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 36df68903beed8b1fec2750aea12a4821b3ad7cb with merge bb8c0341366bf556b8819c19b6e84ab2786adabf...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing bb8c0341366bf556b8819c19b6e84ab2786adabf to master...


Aligning the view recanters the object being rendered

Align view: Changing the view to a preset, resets the view. For instance, after centering the the object being rendered, when a user switches to a different view (eg. -Z), VAPOR switches both the view and the camera center requiring the user to recenter the object.  Expected behavior: Following other 3D modeling software, chaining the view should not alter the camera center.

Related to all the visualizer refactoring issues such as #3611


C API wrapper proposal

Motivation covered in #26. Similar to LevelDB C API wrapper.

Thanks @isaachier. You might also want to post it to the opentracing-cpp gitter channel.
One question I have: If someone wanted to use tracers that were completely written in C as opposed to using this as a bride to the C++11 API, does this still give them a way to customize the tracer used?

@rnburn that is a much more difficult proposal. It would probably need another project opentracing-c. If anything, the C++ API should implement/extend the C API if that were the goal.

What I'd really like to see for C and C-derived OT APIs is a specification for dlopening tracers, so that we can get e.g. nginx (C) to buy-in just once to OT, and then not have to rebuild it to use different tracer implementations. Perhaps the C++ bindings already permit that.

@rbtcollins There's another issue (#28) to discuss just that.
The current instrumentation for nginx doesn't require you to rebuild nginx for each tracer as it uses nginx's dynamic modules, but there is some configuration overhead that we're trying to remove with dlopen conventions for tracers.

I would say there is at least a hint to this issue in the C API I proposed here. When it comes to initializing a global tracer, there is no way to specify this properly in C. That is why I left it as TODO. The dlopen solution would potentially resolve this.

For those looking for a true C implementation of the OpenTracing model, I would say gRPC's Census library comes pretty close to the specification (if not completely fulfills it). Take a look here: https://github.com/grpc/grpc/tree/master/src/core/ext/census.

The difficulty with wrapping the API stems from a few issues relating to ownership. One difficulty is that I cannot copy any of the classes because there is no clone method for me to use. I'm starting to wonder what a pure C API would look like. It seems the main consumers of pure C would much rather have the implementation be pure C as well.

I think you could fix most of the ownership issues by just changing opentracing_tracer_t to
struct opentracing_tracer_t {
     std::shared_ptr<opentracing::Tracer> impl_;
 };

Hey @heyleke, @jcchavezs -- you guys have asked about a C API before. Would something like this work for you?

Maybe... I'll get back to this soon enough. Thank you for reviewing it.

I am in!

@jcchavezs Could you describe your use case? I'm curious what types of projects would benefit from this API.

@jcchavezs I am also curious to hear. Meanwhile, what motivated me was to ease the process of wrapping/binding other languages to a C/C++ implementation. That way, an implementation need only follow the opentracing-cpp specification, and benefit from a version in Python, Ruby, JavaScript, etc. with no additional code (or minimal code for each).

@rnburn  I'm following this with great interest. Although this approach will probably not solve the biggest issue in our use case: C++11 support in toolchain is still required. But having a start towards a C api to start with is already something, I also read about the census implementation which could be useful to create a pure C implementation if we can't solve the C++11 support.
So, the main use case are:

pure C applications, but linking to C++ is still possible in that case
environments lacking C++ or C++11 support.
(although I have to admit in 2017 you should be capable of doing C++11 in your toolchains ;-) )

Already alot of thx to @isaachier for bringing this on the table. I hope to become more active in this area in the near future.
Btw, if there one day would be a pure c implementation, I would call it opentracing-C99 if it would be pure C99 (e.g.)
And the C 'wrapper' would reside in the opentracing-cpp(11) repo.
Are there any pure C tracers that this would be testable with (with some modifications)? lightstep?

I just submitted a new version in pure C. As of now uses C11, although that
is just for the single atomic pointer used to store the global tracer.
On Oct 27, 2017 5:48 AM, "Jan Heylen"<EMAIL_ADDRESS>wrote:
@rnburn https://github.com/rnburn I'm following this with great interest.
Although this approach will probably not solve the biggest issue in our use
case: C++11 support in toolchain is still required. But having a start
towards a C api to start with is already something, I also read about the
census implementation which could be useful to create a pure C
implementation if we can't solve the C++11 support.
So, the main use case are:

pure C applications, but linking to C++ is still possible in that case
environments lacking C++ or C++11 support.
(although I have to admit in 2017 you should be capable of doing C++11
in your toolchains ;-) )

Already alot of thx to @isaachier https://github.com/isaachier for
bringing this on the table. I hope to become more active in this area in
the near future.
Btw, if there one day would be a pure c implementation, I would call it
opentracing-C99 if it would be pure C99 (e.g.)
And the C 'wrapper' would reside in the opentracing-cpp(11) repo.
Are there any pure C tracers that this would be testable with (with some
modifications)? lightstep?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opentracing/opentracing-cpp/pull/27#issuecomment-339926446,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACdXeWWf1WyxQLENgubMhV_O_CgChI2jks5swabagaJpZM4QE4BJ
.

@heyleke - Could you use a C API wrapper if the C++11 tracer and opentracing libraries were cross compiled and distributed as binaries?

If I understand correctly, the answer would be no. Certain platforms just don't have good support for C++11 (in my experience AIX and Solaris).

The native compilers for those OSs might have somewhat flaky C++11 support, but GCC/Clang support cross compiling to most of the widely used architectures/environments.
I believe he has an embedded target. I'm wondering if packages build with something like ellcc might solve the problem.

Any idea what to do with this issue or should I just close it?

I'd like to have more input from stakeholders and some example project instrumentations before going forward with something for C.
I expect there are some projects that would need a C API, but I don't think there's that many. The major C projects that I've looked at so far (NGINX, Varnish, HAProxy) can all be instrumented with the C++ API by just providing some extern "C" hooks.

Ya that's my feeling. The C problem is interesting but might be a separate project.


Affects Fluid Sprites

Opening JEI and searching for water (or any similarly drawn substance, such as Mek's chemicals) and changing the dark mode changes the shading of the sprite drastically.
Forge: 39.0.64
DarkModeEverywhere: 1.0.1

This does not appear to be fixed as when using the method dump thing it provides mezz.jei.library.render.FluidTankRenderer:drawTextureWithMasking as the method for JEI's fluid rendering, but the one that is blacklisted by default is mezz.jei.common.render.FluidTankRenderer:drawTextureWithMasking


pip not in PATH in python3 tag

Leveraging geerlingguy/docker-centos7-ansible:python3, my Molecule test was failing as pip couldn't be found in the environment. After a little searching, it appears there is a pathing issue in the container:

In the latest tag, which uses Python 2, pip is correctly linked:

One thing to note is the python3 tag was pushed 2 years ago whereas the latest tag was last pushed 4 months ago. https://hub.docker.com/r/geerlingguy/docker-centos7-ansible/tags
https://github.com/geerlingguy/docker-centos7-ansible/blob/master/Dockerfile#L29 was added within the last 10 months, which installs/updates pip, and likely fixes the path resolution as a side effect. I believe the fix would be to pin a python3 tag with the existing Dockerfile (supplanting python3 for python2). Not sure how that was done 2 years ago, I only see latest in CI, so not sure what PR to suggest.
To resolve the error in geerlingguy/docker-centos7-ansible:python3 as tagged today, I ran python3 -m pip install -U pip inside the container and pip was then properly linked.

@artis3n - The reason I haven't maintained that python3 tag is apt support is horribly broken under python3 in CentOS 7, therefore any of the projects where I used this container would break if I tried defaulting to python 3 :(
I understand that use case isn't everyone's, but trying to maintain the separate versions was annoying and caused more grief than it was worth for me.

Yeah that's very understandable. No worries!

You're right it's the absolute worst (not working, but for posterity)
https://github.com/artis3n/docker-centos7-ansible-py3


Allow any note-type with a Gid field, not just the single pre-ordained note-type

My language cards use a bunch of different note types for different purposes, with different fields and generated-cards:

Skimming the code, it doesn't look like this addon needs to know anything about the existing card's note-type — just whether it exists. I took a brief stab at patching this, but I couldn't quite get a Python dev-environment setup within my allocated time. 😅
This should only require patching the following to use the field's name, instead of its index, to find the gid:
https://github.com/JASchilz/AnkiSyncDuolingo/blob/2c712177f0daff0e4332d3513735c1602e9f9b26/duolingo_sync/plugin.py#L42-L44
(Ideally, in the process, it'd be nice to have the field renamed to "Duolingo GID" or something clearer, too.)

I appreciate your thoughts and suggestions here @ELLIOTTCABLE , and this seems like a reasonable idea.
In https://github.com/JASchilz/AnkiSyncDuolingo/issues/78 I call out that I'm not able to give this project the full attention that I'd like to as owner, that I'm unable to provide or consider merge requests for new features, and invite a new maintainer.
If a new owner shows up in #78 , then you could potentially let them know about this idea. :)


Remove dev frontends; use snabb-lwaftr

pcapui has been replaced by snabb-lwaftr check.
This uses the same positional arguments.
nic_ui has been replaced by snabb-lwaftr run.
See snabb-lwaftr run -h for usage details.

Please only merge this after #53

lgtm once #53 is merged


Add fallback for unhandled parameter references

Currently there are some constructs which will make the loop in internal/compiler.resolveCatalogRefs silently skip parameters which were identified in the find_params pass. The only concrete example I know how to trigger is passing a parameter inside a case expression as an argument to a function call, although it looks like there are some other spots in the code which could have similar results. The outcome is that currently sqlc generate completes without errors, but the generated queries will probably fail at runtime due to missing arguments.
This PR adds fallbacks to use a parameter with a default name and type any in those cases. I added a check at the end of the loop to panic if the length of the parameter list doesn't match the length of the input paramRef list.

Currently there are some constructs which will make the loop in internal/compiler.resolveCatalogRefs silently skip parameters which were identified in the find_params pass.

Instead of adding a fallback, can we identify the cases this happens and fix those instead? I'm a bit skeptical of adding the fallback, mainly because defaulting to the "any" type isn't very useful.

Yeah, adding a fallback is definitely just a quick fix. For the one concrete instance I know of (case statements as function call arguments) it should definitely be possible to do something. I can update this pull request to do something nicer, but it may be a while before I can get to it.
One complication is that it would be good to still use the function argument name for the parameter name where it makes sense, if there's nothing better, and currently that would keeping the function call node as the parameter parent and handling the child expression inline. The comment here about handling recursive types seems like it could apply. I'm guessing the same is true for the in and between expressions. I don't know quite what MySQL allows there but there are probably a number of different productions.

@wabain I'm sorry that we never go to a resolution here. It feels bad closing a PR after it's been open for more than six months, but I think it's the best option right now. If you'd still like to see this get fixed, let's start with an issue first.


Add a warning about the JOIN/ORDER BY bug in beta-20180305

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3fe3031d8cc6005349866d9475cdfbcf50cc052a/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/7c6f0d82a21984557f4b715c438dd76e2b8982d7/


bug(composer): flaky test composer::tx_from_two_rollups_are_received_by_sequencer

--- STDOUT:              astria-composer::blackbox composer::tx_from_two_rollups_are_received_by_sequencer ---

running 1 test
test composer::tx_from_two_rollups_are_received_by_sequencer ... FAILED

failures:

failures:
    composer::tx_from_two_rollups_are_received_by_sequencer

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 3 filtered out; finished in 2.11s

--- STDERR:              astria-composer::blackbox composer::tx_from_two_rollups_are_received_by_sequencer ---
service is not attached to tty; using json formatting
thread 'composer::tx_from_two_rollups_are_received_by_sequencer' panicked at 'mocked sequencer should have received a broadcast message from composer: Elapsed(())', crates/astria-composer/tests/blackbox/composer.rs:55:10

Closing since this test was removed in favor of a less flaky unit test.


Addlikes

Hello, I hope you're doing well! I have completed the initial implementation of the Add likes of the ES6-Capstone project and I would greatly appreciate your code review. I've opened a pull request with the changes in a feature branch called test.
In this pull request, you will find the entire codebase for the ES6-Capstone project committed as a single chunk of code. As we discussed, please focus on reviewing the code for issues related to best practices and clean code rules.
I kindly request you to review the code and provide your valuable feedback. Feel free to comment directly on the pull request or raise any concerns you may have. Your input will greatly contribute to enhancing the quality and functionality of the project.
Thank you for taking the time to review my code. I'm looking forward to your feedback!


Convert J2objcPluginExtension comments to groovydoc

@brunobowden we should be able to convert our comments to groovydoc and then publish a description of our DSL automatically.  Or do you want to duplicate all this info in README.md and/or in J2objcPlugin.groovy?  I prefer not duplicating.  When we publish a new version, one of the steps would be publishing the groovydoc to the github.com project site.  How does that sound?

I'm open to this. Particularly the more automated it is. I've had issues in
the project site before that it's difficult to do PRs and make changes. So
I'm concerned that this change might make it painful to work with.
On Thu, Jun 11, 2015 at 1:01 PM Advay Mengle<EMAIL_ADDRESS>wrote:

@brunobowden https://github.com/brunobowden we should be able to
convert our comments to groovydoc and then publish a description of our DSL
automatically. Or do you want to duplicate all this info in README.md
and/or in J2objcPlugin.groovy. I think not duplicating is fine. When we
publish a new version, one of the steps would be publishing the groovydoc
to the github.com project site. How does that sound?
—
Reply to this email directly or view it on GitHub
https://github.com/j2objc-contrib/j2objc-gradle/issues/141.

Either way we should do the groovydoc.  I agree that maybe the site won't be easy to use.


Improve GitHub actions

I've forked this repo to create similar Mill Docker images.
In the process I made minor changes to the Github Actions and thought it might be good to contribute back parts that apply to both repos.
Note that for the official docker/login-action I copy&pasted their instructions, so the variables are slightly different.

Hi @francisdb, I had not seen #217. Sounds reasonable to make the build more reproducible 👍
But Dependabot would only update the base image, right?

The problem is a bit is that the tags are moving targets, so pulling the same image tag the next day will give you a different result. Anyway, need to think that through a bit more before changing anything.


Fix scheduling algorithm

Fixes #162.
The issue was that in some situations, the scheduler would not flush stream B if there was a stream A with higher priority even if stream A had no data to send. Since stream A did not send any data, its priority would not change, so it would remain at the top of the priority queue.
This would not be an issue if streams were constantly sending data at the same rate, but if some streams stopped sending (without closing the body) or streams were sending data at different rates, frames could get be stuck in the scheduler.
The fix to this is to traverse the children until a stream that needs to send data is found, if such a stream exists.
Also fixes a possibly broken test that illustrates the issue. After the changes to the scheduler, a frame that wasn't flushed before is now flushed. The test assumed that only one empty DATA frame should be sent but we closed two streams, so it has to be two frames.

I took a stab at fixing some issues here but couldn't push to your branch, so I rebased your commits + added my fixes in #204. Let's continue the discussion there.

Also fixes a possibly broken test that illustrates the issue. After the changes to the scheduler, a frame that wasn't flushed before is now flushed. The test assumed that only one empty DATA frame should be sent but we closed two streams, so it has to be two frames.

I believe this is only happening because we're now flushing twice for streams that have their (stream-ending) empty DATA frame pending. I don't think that's necessarily bad, though.

superseded by #204


chore(firestore-translate-text): node 10 migration

ToDo

[x] Update functions/package.json to include "engines": {   "node": "10" }
[x] Ensure billingRequired is set to true in extension.yaml
[x] PREINSTALL reflects billing is now required
[x] Check all legacy Node 6/8 environmental variables are no longer used.
[x] Ensure compilerOptions are updated to include es2018
[x] Update firebase-functions dependency to latest
[x] Update console logs and error to use new functions logger

Testing

Logging Testing

Codecov Report

Merging #413 into next will decrease coverage by 0.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             next     #413      +/-   ##
==========================================
- Coverage   39.77%   39.10%   -0.68%
==========================================
  Files          19       19
  Lines         636      629       -7
  Branches       85       85
==========================================
- Hits          253      246       -7
  Misses        381      381
  Partials        2        2

Impacted Files
Coverage Δ

firestore-translate-text/functions/src/index.ts
100.00% <0.00%> (ø)

@googlebot I consent


Typed Dict - typing_extensions

Error:
from typing import TypedDict, Dict, List, Optional
ImportError: cannot import name 'TypedDict'
Fix:
if sys.version_info[0] == 3 and sys.version_info[1] >= 8:     from typing import TypedDict, Literal else:     from typing_extensions import TypedDict, Literal

I do not plan on supporting pre 3.8 versions, there would be other issues as well with my use of the walrus operator in other places.

Yep. I did notice that after I added a fix for every  import used. My bad.
This is totally amazing though, that you created this🥰
ons. 26. mai 2021, 03:55 skrev Tolki @.***>:

I do not plan on supporting pre 3.8 versions, there would be other issues
as well with my use of the walrus operator in other places.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mrtolkien/leaguepedia_parser/issues/11#issuecomment-848397249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALT2X5RLQZM2IPYORMZHNVDTPRIKHANCNFSM45PZDAPQ
.


Converting to MKV in python script being ignored.

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.04.03. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.04.03

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
running things via python scripts on windows. I can confirm that I am on the latest.
(Pdb) print youtube_dl.version.__version__
2017.04.03

youtube-dl: error: You must provide at least one URL.
https://www.youtube.com/channel/UC8KxkYgafaJnzyWz1pq_glg

---

### Description of your *issue*, suggested solution and other information

So I have the following script

--------------------------------------------------------
from __future__ import unicode_literals

import datetime
import getpass
import json
import os
import sys
import youtube_dl

class MyLogger(object):
  def __init__(self, log_file):
    self.log_file = log_file
    self.errors = []

  def debug(self, msg):
    self.log_file.write(msg.encode('utf-8'))
    self.log_file.write('\n')

  def warning(self, msg):
    self.log_file.write(msg)
    self.log_file.write('\n')

  def error(self, msg):
    if 'YouTube said' in msg and msg not in self.errors:
        self.errors.append(msg)
        self.log_file.write('ERROR!')
        self.log_file.write('\n')
        self.log_file.write(msg)
        self.log_file.write('\n')

pwd = getpass.getpass()
timestamp = datetime.datetime.now()
timestamp = timestamp.strftime('%Y-%m-%d-%H-%M-%S')
log_file = open(timestamp+'log.txt', 'w', 0)

download_list = [
    {'name': 'curtis_lasam',
     'outtmpl': 'Z:/Media/Youtube/CurtisLasam/%(upload_date)s-%(title)s [%(id)s].%(ext)s',
     'download_link': ['https://www.youtube.com/channel/UC8KxkYgafaJnzyWz1pq_glg']},
  ]
ydl_opts = {
  'noprogress': True,
  'ignoreerrors': True,
  'restrictfilenames': True,
  'nooverwrites': True,
  'writeinfojson': True,
  'writeannotations': True,
  'writedescription': True,
  'writesubtitles': True,
  'subtitleslangs': ['en'],
  'username': 'my_username',
  'password': pwd,
  'cookiefile': 'cookie.txt',
  #'ratelimit': '2M',
  'outtmpl': '',
  'recodevideo': 'mkv',
  'logger': MyLogger(log_file),
}
while True:
    for download_item in download_list:
      print download_item['name']
      ydl_opts['outtmpl'] = download_item['outtmpl']
      with youtube_dl.YoutubeDL(ydl_opts) as ydl:
        ydl.download(download_item['download_link'])
log_file.close()
------------------------------------

This ends up causing me some problems. I would like to have it download and convert everything to mkv. I believe I have it correct. Looking through the code recodevideo should set the [postprocessors correctly ](https://github.com/rg3/youtube-dl/blob/d212c93d16bbb6aeb8645f1dbb15a78f3d9414a8/youtube_dl/__init__.py#L257). The thing is when I use this script it seems to convert to other formats. See the following lines from my logs

[download] Downloading video 42 of 47
[youtube] jGnrRub3YzQ: Downloading webpage
[youtube] jGnrRub3YzQ: Downloading video info webpage
[youtube] jGnrRub3YzQ: Extracting video information
video doesn't have subtitles
[youtube] jGnrRub3YzQ: Searching for annotations.
[youtube] jGnrRub3YzQ: Downloading MPD manifest
[info] Writing video description to: Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].description
[info] Writing video annotations to: Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].annotations.xml
[info] Writing video description metadata as JSON to: Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].info.json
[download] Destination: Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].f137.mp4
[download] Download completed
[download] Destination: Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].f140.m4a
[download] Download completed
[ffmpeg] Merging formats into "Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].mp4"
Deleting original file Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].f137.mp4 (pass -k to keep)
Deleting original file Z:\Media\Youtube\CurtisLasam\20130202-Cube_Drone_-_Source_Control [jGnrRub3YzQ].f140.m4a (pass -k to keep)
[download] Downloading video 43 of 47

I am wondering why this is happening. I imagine I am the problem. I looked through the code some. I also looked through the FAQs and the other issues and didn't see anything that is strictly related. I think this has to do with the fact it is picking the best quality and ignoring my request to convert it to mkv.

There is no such option recodevideo. You should add postprocessor instead.


FirebaseFirestore unsafe build flags

Since updating to version 9.0.0 I am unable to build the project. I get the following error:
The package product 'FirebaseFirestore' cannot be used as a dependency of this target because it uses unsafe build flags.

Seeing this as well unfortunately.

Thanks @dineshvijaykumar! This was due to another issue I recently fixed, where errors can occur if the -ObjC flag is not set. I removed the flag from Package.swift and added readme instructions to manually set the flag under build settings instead.


When multi questions in msg, the Exchange() call always timeout.

When I add multi dns.Question in a msg and call Exchange , it says

2018/06/11 21:23:36 read udp <IP_ADDRESS>:53874-><IP_ADDRESS>:53: i/o timeout

I have heard some guys said some devices don't support multi questions dns query, so a lot of dns server not support multi questions dns query.
wireshark says the dns query packet created by miekg/dns is correct, so it's not the lib fault, right?

Yeah it's probably not this package.
In practice DNS is limited to a single question because a question may be answered in part by bits in the message header and there's only one message header.

However, with DNSSEC answers, all status conveyed in the header could also be deduced from the RRs (i.e. NXDOMAIN, authenticity etc).  So, when all DNS would be DNSSEC signed, multiple questions in one DNS message would be viable :)

Multiple questions was a fine idea, but as Andrew said, doesn't work in
practice. Consider it unsupported.
On Tue, 12 Jun 2018, 08:21 wtoorop<EMAIL_ADDRESS>wrote:

However, with DNSSEC answers, all status conveyed in the header could also
be deduced from the RRs (i.e. NXDOMAIN, authenticity etc). So, when all DNS
would be DNSSEC signed, multiple questions in one DNS message would be
viable :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/miekg/dns/issues/700#issuecomment-396491323, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVkW__6L05HVXjZ8rHd7fI7lvrf4mEHks5t72vlgaJpZM4Uishe
.

Sure, I just wanted to remark that in theory, when all zones in the whole world would be DNSSEC signed, and we would ignore support in existing software, multiple questions in one message could be viable.

Yes :) and you've also supplied the reasons we won't see this happening in
our lifetimes :)
On Tue, 12 Jun 2018, 08:41 wtoorop<EMAIL_ADDRESS>wrote:

Sure, I just wanted to remark that in theory, when all zones in the whole
world would be DNSSEC signed, and we would ignore support in existing
software, multiple questions in one message could be viable.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/miekg/dns/issues/700#issuecomment-396496507, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVkW8F79k7sEuDb_nL29Rdmb9FVzZIWks5t73CcgaJpZM4Uishe
.


Register optimize commands

Newly introduced in Laravel 11.27, packages can now register the commands that should run during the optimize phase of deployment.
This PR adds the recommended steps for optimizing whenever it's available.

Thanks!

Cool!


Monaco Text Editor

On resolving #571, #1725. Here's an implementation of MonacoTextEditor.  After merging this, we should implement the language client services to a python language server, offered by MonacoEditor, for typehints, autocomplete etc... among others.
Currently, we use the TextEditor (ace based), in the following widgets:

[x]  OperationCodeEditor
[x]  OperationDepEditor
[ ]  LogViewerWidget
[ ]  TabbedTextEditor

I am closing this because this will cause unnecessary changes to all the unrelated panels and reopening from a new branch.


Fixes missing f-string to retry message

When using the --retry flag, the CLI's message isn't interpolated properly:
Current output is:
Retrying export for photo ({photo.uuid}: {photo.original_filename})
This PR adds the missing f-string that interpolates the uuid and filename properly.

Thanks for this!
@all-contributors add @hyfen for code


Standalone Mode SPI transaction 512 bytes

Dear All,
I'm using standalone mode to integrate NRC with a microcontroller and I'm having problems to send AT command via SPI with length lower than 512 bytes. The SPI transaction in a Microcontroller is limited to 255 bytes. I don't undertand why needs to send 512 bytes for a simple AT command? Also I don't understand why CRC calculation only included the header?
I will appreciate your support, thanks in advance. Kinds regards

Dear @aogonzalez75,
The slot size is defined as 512 bytes at sdk/apps/atcmd/hif.h.
/* HSPI */
#define CONFIG_HIF_HSPI_SLOT_NUM		32	/* Don't chagne */
#define CONFIG_HIF_HSPI_SLOT_SIZE		512 /* Don't change */

If the SPI transaction is limited as 255 bytes on host MCU, CONFIG_HIF_HSPI_SLOT_SIZE should be 128 and then FW should be rebuilt.
FYI, raspi-atcmd-cli application has hspi_ready() at sdk/apps/atcmd/host/raspi-atcmd-cli/nrc-hspi.c which checks slot number and slot size.
static int hspi_ready (hspi_info_t *info)
{
	hspi_regs_t regs;

	if (!HSPI_ACTIVE())
		return -1;

	if (hspi_regs_read_all(&regs) != 0)
		return -1;

	hspi_regs_print_all(&regs);

	if (memcmp(regs.msg, "NRC-HSPI", 8) == 0)
	{
		uint16_t slot_num;
		uint16_t slot_size;
		int que;

		for (que = HSPI_TXQ ; que <= HSPI_RXQ ; que++)
		{
			slot_num = (regs.msg[2 + que] >> 16) & 0xffff;
			slot_size = regs.msg[2 + que] & 0xffff;

			if (slot_num == 0)
				return -1;

			if (slot_size == 0 || slot_size > HSPI_SLOT_SIZE_MAX)
				return -1;

			info->queue.slot[que].num = slot_num;
			info->queue.slot[que].size = slot_size;
		}
...

Best regards,
Newracom

we will try to change it. It's not clear for me, why it uses a fixed length on the SPI transaction? why CRC not include the entire message?

Dear @aogonzalez75,
Have you tried with the change? If you have any further questions, we'd like to close this issue.


Links and image not displayed correctly

In the wiki documentation under here, links does not work and image does not display correctly.

Fixed the link.


dismiss indeterminateProgressDialog or ProgressDialog

I've tried to dismiss both Progress Dialogs but I couldn't. I assign the progress dialog function to a variable and then the function dismiss is not found. It works with normal alerts but not with progress dialogs
val mProgressDialog = indeterminateProgressDialog("Description", "Title").show() mProgressDialog.dismiss() // This won't work

Sorry, it was my fault. I was assigning everything included the .show()
It works like this:
val mProgressDialog = indeterminateProgressDialog("Description", "Title") mProgressDialog.show() mProgressDialog.dismiss()


Added set_scissor_rect to tracked render pass.

Objective And Solution
Add set_scissor_rect from wgpu-rs to the TrackedRenderPass. wgpu documentation can be found here:
https://docs.rs/wgpu/latest/wgpu/struct.RenderPass.html#method.set_scissor_rect
The reason for adding this is to cull fragments that are outside of the given rect. For my purposes this is extremely useful for UI.

bors r+

Since the TrackedRenderPass is reused for all draw functions, does that mean that setting the scissor rect in one draw function will also set it for all subsequent ones? That seems unexpected.

Since the TrackedRenderPass is reused for all draw functions, does that mean that setting the scissor rect in one draw function will also set it for all subsequent ones? That seems unexpected.

I used the same wording exactly as the wgpu documentation. It is a little strange with bevy's draw system and perhaps a new PR could address updating the documentation to be a bit more clear on that. I don't think we should clear out the scissor pass state in-between draw functions because it would likely slow things down. It would also require people to set scissor state for each draw call which is slower from a performance view point.


OPTIONS method causes a crash

If you do a http options request to a URL wrapped with this middleware, it crashes:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/lib/python3/dist-packages/flask/app.py", line 1479, in full_dispatch_request
    response = self.process_response(response)
  File "/usr/lib/python3/dist-packages/flask/app.py", line 1691, in process_response
    response = handler(response)
  File "/usr/local/lib/python3.4/dist-packages/flask_compress.py", line 79, in after_request
    response.content_length < app.config['COMPRESS_MIN_SIZE'] or
TypeError: unorderable types: NoneType() < int()

I guess the fix is to change the condition to:
(response.content_length and response.content_length < app.config['COMPRESS_MIN_SIZE']) or

I'm wondering how it got into that scenario. All of the COMPRESS_* values should be initialized in init_app. A few lines above it it is checking for COMPRESS_MIMETYPES and it doesn't crash. Is there anything unique about your scenario?

My understanding was that it is response.content_length which is None (OPTIONS has no body), but now I realise that I didn't actually check.

I bet you are right, that makes more sense to me. I'll look into it and get back to your shortly.

Can you write a small snippet in test_flask_compress.py that reproduces this for me? So far I keep getting a response.content_length of 0 when doing an options request.
client = self.app.test_client()
response = client.options('/small/', headers=[('Accept-Encoding', 'gzip')])
self.assertEqual(response.status_code, 200)
print(response.content_length)

Thank you

I currently cannot reproduce it locally, only in production. Maybe it is a bug that got fixed already somewhere? I will reopen if/when I can reproduce this in a controlled environment.

Right, I can reproduce it in development now. The problem was that I wasn't passing --compressed to curl, and it was working against the live site since there was a proxy infront of the daemon which supported compression.
I'm suspecting a flask bug right now, since the HEAD method is actually hanging.

I added this test to your code, and it reproduces the problem when running nose.

    def test_content_length_options(self):
        client = self.app.test_client()
        headers = [('Accept-Encoding', 'gzip')]
        response = client.options('/small/', headers=headers)
        self.assertEqual(response.status_code, 200)
        print(response.content_length)


support multiple authnContext

This addresses issue #181 and supersedes PR #183. Please see those other two entries for context.

Looks good. Thanks. Merging.
QA Log

I confirmed that SAML 2 support multiple authentication contexts. ref

As soon as this lands I'll update the TypeScript typings for this. Is there any interest in maintaining the typings for this project here?

@cjbarth Only if you maintain them. My team plans to adopt the TypeScript in the future, but hasn't currently. The project is changing at a slow rate now, so maintenance to the TypeScript bindings should be infrequent.

In that case, I'll leave them over at DefinitelyTyped for now.


Change fake-cluster-id to a real uuid

[x] Change fake-cluster-id to a real uuid
[x] Test the change to ensure it is working correctly

/assign @joelddiaz
/cc @suhanime

Fixes: https://issues.redhat.com/browse/HIVE-1354


Strange issue with inconsistent behavior

I do this:
cal = parsedatetime.Calendar()
datetime_obj, parse_status = cal.parseDT(datetimeString='thursday 6:00 am', tzinfo=timezone('Europe/Bucharest'))
Resulting time I get is this:
2016-06-30 03:00:00+00:00
Here's the problem: I get this result when I run the query at 6:40am on June 30th.
I think what's happening is because it's pre 9am (or some other time), it returns a time that has already passed instead of the following Thursday. eg. I was expecting this: 2016-07-07 03:00:00+00:00
I don't see this in the docs, but will do more testing. Any input appreciated on why this happens. I'm expecting to say "Thursday 6:00am" and if the current time in given timezone is past 6am, return the following Thursday. I'm not seeing this behavior, though.
Thank you -

there should be a flag you can set in PDT that controls whether to use current, prior or next week when evaluating those types of expressions -- I'm on the road and don't have full access to my desktop so I'm having trouble finding it
I'll check back later when I'm at my desk

Thank you, I couldn't seem to find that either – maybe I'm not looking in the right place. Being able to say, "Thursday 6am" (On a Thursday at 6:42am) and force it to return the following Thursday would indeed solve my  issue. Thanks Bear.

k, found it - if you look at https://github.com/bear/parsedatetime/blob/master/parsedatetime/init.py#L2314
you will find quite a few constants that control how the system handles filling in missing information along with comments that describe the behaviour

Thank you Bear. I read through the examples, and your link, and have tried multiple variations (each time thinking, "I got it this time."), but I'm not able to get the results I'm after.
I'm using this:
c = parsedatetime.Constants();
c.StartTimeFromSourceTime=True;
c.DOWParseStyle=+1;
c.CurrentDOWParseStyle=False;
cal = parsedatetime.Calendar(c);
date, status=cal.parseDT(datetimeString='Thursday 2am', tzinfo=timezone('Asia/Manila'));
print(date, status)

Current time in Asia/Manila is (as I ran this):
4:14 AM
Thursday, July 7, 2016 (GMT+8)
Result:
2016-07-07 02:00:00+08:00 3
My expectation:
Since the current time in Asia/Manila is Thursday 4:14am, and I asked for Thursday 2am (and I set starting source time and other constants as I understand them to get the result I want), I'm expecting this result:
2016-07-14 02:00:00+08:00 3 (core difference, next Thursday, 2am, not 2am in the past)
Can you enlighten me? Am I missing something obvious? Thank you. :)

hmm, that is odd for sure -- I won't be able to look at it until this weekend but I sure will

Thanks Mike, I'm baffled. It seems like it should work properly if I'm understanding the docs correctly.

If anyone else has this issue, I'll set up a workaround to add the word "next Thursday" in my code if the time in given timezone is pre-9am. I think that'll be a good workaround in the meantime since I think that's the core of my issue.

Hey Mike, happen to have any idea on this? I appreciate any time you're willing to spend. My "next Thursday" temp. fix still operates differently and sometimes unexpected since the constants don't appear to be working properly. (Unless I'm missing something here)

@zefoo apologies for the delay in responding, I was AFK from the internet for the last week and i'm just now catching up

Sounds like a good time. :) Thanks for the heads up! I'm really curious what you find.

Hey Mike, any updates on this by any chance? I still can't seem to figure this out for the life of me. Thank you so much, appreciate any time you're willing to spend narrowing this down.


Unable to deploy single functions (similar to #189)

I have the same issue as in #189.
Serverless: Docker Image: lambci/lambda:build-python3.6
Serverless: Installing requirements of two/requirements.txt in .serverless/two...
Serverless: Docker Image: lambci/lambda:build-python3.6
Serverless: Invoke package:function
Serverless: Packaging function: one...
Serverless: Excluding development dependencies...
Serverless: Injecting required Python packages to package...

  Type Error ---------------------------------------------

  path must be a string or Buffer

     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

  Stack Trace --------------------------------------------

TypeError: path must be a string or Buffer
    at fs.readFile (fs.js:358:11)
    at go$readFile (/Users/gaston/my-service/node_modules/graceful-fs/graceful-fs.js:73:14)
    at Object.readFile (/Users/gaston/my-service/node_modules/graceful-fs/graceful-fs.js:70:12)
    at Object.readFile (/Users/gaston/my-service/node_modules/universalify/index.js:5:67)
    at moveModuleUp (/Users/gaston/my-service/node_modules/serverless-python-requirements/lib/inject.js:58:6)
    at BbPromise.resolve.filter.map.map.func (/Users/gaston/my-service/node_modules/serverless-python-requirements/lib/inject.js:104:18)
    at runCallback (timers.js:789:20)
    at tryOnImmediate (timers.js:751:5)
    at processImmediate [as _immediateCallback] (timers.js:722:5)

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information -----------------------------
     OS:                     darwin
     Node Version:           8.9.3
     Serverless Version:     1.28.0

serverless.yml:
service:
  name: my-service-name
  awsKmsKeyArn: ${file(./config-${opt:stage}.yml):awsKmsKeyArn}

provider:
  name: aws
  runtime: python3.6
  region: us-west-2
  stage: ${opt:stage}
  profile: ${opt:aws-profile}
  role: my-role
  timeout: 120
  memorySize: 128

plugins:
  - serverless-python-requirements
  - serverless-pseudo-parameters

custom:
  pythonRequirements:
    dockerizePip: non-linux
  accountId: "#{AWS::AccountId}"

package:
  individually: true

resources:
  ...

functions:
  one:
    handler: main.handle
    module: ./one
    package:
      include:
      - ./one/main.py

  two:
    handler: main.handle
    module: ./two
    package:
      include:
      - ./two/main.py

npm list
$ npm list
├──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ ├──<EMAIL_ADDRESS>  │ ├─┬<EMAIL_ADDRESS>  │ │ └──<EMAIL_ADDRESS>deduped
  │ └──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ ├─┬<EMAIL_ADDRESS>  │ │ ├──<EMAIL_ADDRESS>  │ │ ├─┬<EMAIL_ADDRESS>  │ │ │ ├──<EMAIL_ADDRESS>deduped
  │ │ │ └──<EMAIL_ADDRESS>  │ │ ├──<EMAIL_ADDRESS>  │ │ ├─┬<EMAIL_ADDRESS>  │ │ │ └─┬<EMAIL_ADDRESS>  │ │ │   ├──<EMAIL_ADDRESS>  │ │ │   └──<EMAIL_ADDRESS>  │ │ ├─┬<EMAIL_ADDRESS>  │ │ │ └──<EMAIL_ADDRESS>deduped
  │ │ └──<EMAIL_ADDRESS>  │ └─┬<EMAIL_ADDRESS>  │   └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ ├──<EMAIL_ADDRESS>  │ ├──<EMAIL_ADDRESS>  │ ├─┬<EMAIL_ADDRESS>  │ │ └──<EMAIL_ADDRESS>  │ ├──<EMAIL_ADDRESS>  │ └─┬<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>deduped
  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>deduped

on the same lines, with python
Serverless: Injecting required Python packages to package...
Type Error ---------------------------------------------
Cannot read property 'artifact' of undefined
 For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

Stack Trace --------------------------------------------
TypeError: Cannot read property 'artifact' of undefined
at BbPromise.resolve.filter.map.map.map.func (/Users/ranga.reddy/ama-dev/git-repo/att/att-notification-platform/node_modules/serverless-python-requirements/lib/inject.js:113:23)
From previous event:
at PluginManager.invoke (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:390:22)
at PluginManager.spawn (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:408:17)
at Deploy.BbPromise.bind.then.then (/usr/local/lib/node_modules/serverless/lib/plugins/deploy/deploy.js:118:50)
From previous event:
at Object.before:deploy:deploy [as hook] (/usr/local/lib/node_modules/serverless/lib/plugins/deploy/deploy.js:113:10)
at BbPromise.reduce (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:390:55)
From previous event:
at PluginManager.invoke (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:390:22)
at PluginManager.run (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:421:17)
at variables.populateService.then.then (/usr/local/lib/node_modules/serverless/lib/Serverless.js:163:33)
at runCallback (timers.js:672:20)
at tryOnImmediate (timers.js:645:5)
at processImmediate [as _immediateCallback] (timers.js:617:5)
From previous event:
at Serverless.run (/usr/local/lib/node_modules/serverless/lib/Serverless.js:150:8)
at serverless.init.then (/usr/local/lib/node_modules/serverless/bin/serverless:42:50)
Get Support --------------------------------------------
Docs:          docs.serverless.com
Bugs:          github.com/serverless/serverless/issues
Issues:        forum.serverless.com
Your Environment Information -----------------------------
OS:                     darwin
Node Version:           6.14.1
Serverless Version:     1.28.0

Hey there!
Going through old issues, and these are performed against old versions of node, servereless, servereless-python-requirements - so I'm going to close this issue.
If you're still experiencing a failure condition, please open a new issue and provide all relevant debugging data to assist with reproduction!


Print which Networked prefab is missing.

When a Networked prefab is missing it just prints "Networked prefab is missing a NetworkedObject component. It would be nice if it would also print the name because it takes a lot of time to check through all the prefabs.

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


possible "keyBiding" typo

https://github.com/5zig/The-5zig-API/blob/master/src/eu/the5zig/mod/ModAPI.java#L116
I think the name of the method should be registerKeyBinding. At the moment it has been spelt biding instead of binding.
Also, what's the category parameter's purpose? Can I put any string there?

Regarding the category parameter, it is used by Minecraft for the section headers in the controls menu. Just try it out and look at the result in the GUI. :)

How do I update to the latest API in my own project? And does the client itself need updating at all or just the API jar?

The API will be updated when the new update of the 5zig mod has been released.

Where's a list of all of the Keyboard keys possible? I tried using Keyboard.KEY_SEMICOLON but I think it crashed the plugin


Preview

Can you add some info on what you are doing and why? So that other see what works is being done and why and do not duplicate?


[wip] tetsing doc-builder

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Testing

Summary

PR for testing.
Related issues

Fixes #17

Metadata

[ ] Adds new file(s)
[x] Edits existing file(s)
[ ] Removes file(s)

Checklist

[x] My PR generates no warnings or build errors (other than known issues)
[x] My PR follows these contributor guidelines:

Each section of code contains a descriptive comment
Grammar and spelling errors have been fixed

Need to do the following:

run python3 ./title-converter/test.py
take return variable and store it in a custom variable
if custom variable returns False, exit 1


Replace node-keytar with @napi-rs/keyring

Replace the https://github.com/atom/node-keytar with https://github.com/Brooooooklyn/keyring-node since the node-keytar is archived.

We ditched keytar so I'll close this.


Add printing macros

Indoc is currently not usable directly as the format string of a print macro on a stable compiler. (It's fine with the nightly-only "unstable" feature enabled.)
error: format argument must be a string literal
 --> src/main.rs:4:14
  |
4 |     println!(indoc!(""));
  |              ^^^^^^^^^^

It would be nice to provide macros something like printdoc! for this common use case.
printdoc!("
    ...
    ...
    ...
");

This is just for curiosity - a custom trait I wrote as a work-around.
I mostly use it with format!(), so the trick consists of using format!().unindent(). I have added related functions for indenting. I'm not happy enough with the code yet to publish it as a crate, but it works for me.
/// Block indents and un-indents
trait RootIndent {
    /// Remove leading block indentation
    fn unindent(&self) -> String;
    /// Indent by a number of 4-space tabs
    fn indent(&self, count : usize) -> String;
    /// Indent all, except the first line, by a number of 4-space tabs
    fn indent_inner(&self, count : usize) -> String;
}

impl RootIndent for &'_ str {
    fn unindent(&self) -> String {
        if self.trim().is_empty() {
            return String::new();
        }

        let mut ll = self.lines()
            .skip_while(|l| l.trim().is_empty())
            .peekable();

        let next = ll.peek().unwrap();

        let mut space_count = next.as_bytes().iter().take_while(|c| **c == 32).count();

        ll.map(|s| {
                if s.len() > space_count {
                    &s[space_count..]
                } else {
                    ""
                }
            })
            .collect::<Vec<&str>>()
            .join("\n")
    }

    fn indent(&self, count: usize) -> String {
        let indentbare = "    ".repeat(count);
        let indentstr = "\n".to_string() + &indentbare;
        let replaced = self.replace("\n", &indentstr);
        (indentbare + &replaced).trim_end().to_string()
    }

    fn indent_inner(&self, count : usize) -> String {
        let indentbare = "    ".repeat(count);
        let indentstr = "\n".to_string() + &indentbare;
        let replaced = self.replace("\n", &indentstr);
        replaced.trim_end().to_string()
    }
}

impl RootIndent for String {
    fn unindent(&self) -> String {
        self.as_str().unindent()
    }

    fn indent(&self, count : usize) -> String {
        self.as_str().indent(count)
    }

    fn indent_inner(&self, count : usize) -> String {
        self.as_str().indent_inner(count)
    }
}

impl RootIndent for &String {
    fn unindent(&self) -> String {
        self.as_str().unindent()
    }

    fn indent(&self, count : usize) -> String {
        self.as_str().indent(count)
    }

    fn indent_inner(&self, count : usize) -> String {
        self.as_str().indent_inner(count)
    }
}

fn main() {
    // Hello,
    //   world! 123
    //     foo
    //     bar
    //     baz
    // ...

    println!("{}", &format!(r#"
        Hello,
          world! {}
            {inner}
        ..."#, 123, inner="foo\nbar\nbaz".indent_inner(3)
    ).unindent());
}

just a note if you decide to implement these macros, maybe don't use the prefix i, that would conflict with the ifmt crate (like iformat!()). that's a really convenient crate, btw

Just run into this in my own project, so I'm in, MR hopefully will be sent soon.

A small question: how would you test if the stdout of some operation is the expected one? I'd like to add the tests for the new printdoc! macro, as they are here for indoc!, but not sure how to do this (so that they fit here). trybuild does not support checking the output of successful runs, AFAIK?

If printdoc!() is added, then something like formatdoc!() would also be useful. With that, you can build multi-line formatted strings with nice indentation.
Actually, now thinking of it, it'd make the most sense if indoc!() directly worked like this formatdoc!(), taking additional arguments

Well, formatdoc is already added (not pushed yet), and tests for it, adapted from the indoc case, are already passing. Maybe that's enough, given that their implementation is exactly the same, except the macro name.


feat(migrate): WIP

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Jolg42:x: renovate[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

Unnecessary await removed in https://github.com/prisma/prisma/commit/0b090492c4fc00c2a4d4b4b30977eefc3fd82e27


Missing documentation , how to use SVG files ?

What's going on?
I have a lot of assets with svg format , Need to config it , no access to metro.config.js , how to do that ?
I'm using next libraries :  react-native-svg ,  react-native-svg-transformer .

Hi.
Add a metro.config.js.
We are using svg's and a chart lib that used svg's and the libraries you mentioned, with no issues.
Hope it helps.

What's going on?
I have a lot of assets with svg format , Need to config it , no access to metro.config.js , how to do that ?
I'm using next libraries : react-native-svg , react-native-svg-transformer .

Hello here is a more detailed guide on the topic:
https://blog.usejournal.com/how-to-use-svg-in-react-native-e581eca59534


Images become very large on narrow displays

Using rtd 1.8x making a page very narrow will increase some image sizes considerably.
Originally reported here: https://developer.blender.org/T46079
Tested with the default theme and this problem doesn't happen.
See:
http://www.blender.org/manual/render/opengl.html#rendering-an-animation-using-opengl-render

Same here, I have encountered a similar issue.

I'm looking forward to this being fixed because I really need to put small pictures of buttons inline with text.

Sounds like this is the same as #144.

Yes, it's a duplicate of #144, closing.


fix: improvements in token balance fetching methods

FEATURE
Improve SDK getTokenBalancesForChains and getTokenBalances methods LF-3225
TL;DR
Add a boolean parameter to the token balance methods to allow users to hide zero balance tokens. Also, improve error handling for provider errors during token balance fetching.
DESCRIPTION
This feature improves the SDK's getTokenBalancesForChains and getTokenBalances methods by adding a boolean parameter. This parameter allows users to specify whether they want to hide zero balance tokens or not. By default, all tokens are returned, but users can choose to exclude tokens with zero balances by setting the boolean parameter to true.
Additionally, this feature enhances the error handling for provider errors during token balance fetching. It throws the correct validation error when encountering issues with the provider during the retrieval of token balances.
STEPS TO TEST

Call the getTokenBalancesForChains and getTokenBalances methods with the boolean parameter set to true to hide zero balance tokens.
Verify that the returned token balances exclude tokens with zero balances.

EXPECTED RESULTS

When setting the boolean parameter to true, the token balance methods should return balances excluding zero balance tokens.
When encountering provider errors during token balance fetching, the methods should throw the correct validation error.

CHECKLIST

[ ] I have performed a self-review of my code.
[ ] I have commented the code I wrote to ensure clarity.
[ ] I have added tests that cover the new functionality.
[ ] I have made sure that existing tests are still passing.

Can we just hide zero balances without any additional flag? The question here https://lifi.atlassian.net/browse/LF-3225 was mainly for the widget and if it is possible to hide them completely. 🙂

Can we just hide zero balances without any additional flag? The question here https://lifi.atlassian.net/browse/LF-3225 was mainly for the widget and if it is possible to hide them completely. 🙂

Yes, we can filter out the zero balance tokens by default, just to be sure, there is no other use case where we'd have to show it?

Can we just hide zero balances without any additional flag? The question here https://lifi.atlassian.net/browse/LF-3225 was mainly for the widget and if it is possible to hide them completely. 🙂

Yes, we can filter out the zero balance tokens by default, just to be sure, there is no other use case where we'd have to show it?

It's better to double-check where we use this.

This will be integrated into the new version after migration.


Update README.md

The previous link returns a 404; perhaps this link is the one you're looking for? Please review.

That's the right link! Thanks!


[BUG]:YACE Route53 DNSQueries is not matching with Cloudwatch

Hi Team,
We are observing some mismatch with YACE Route53 SNSQueries Metric.
The Sum of metrics Cloudwatch is giving around 150-170 per every 5 mins where as the same in YACE is providing not more than 20-30 for every 5 mins.Can you please check it from your side once and provide a fix?Attached the screenshots form cloudwatch and grafana as well.

closing this and raising with bug label


openAPI spec

Hi,
Are you guys interested in adding an OpenAPI spec yaml file to the repo?
This would be a great plus for interoperability en generating clients
Thanks
M

Hello,
Thanks for showing the interest.
We are looking at a new REST project where there is potential to consider your suggestion.
Closing this issue over here.
Thanks,
Ameya


Localizes content/en/docs/tutorials/kubernetes-basics/update/update-interactive.html to bengali

Signed-off by: PratikforCoding<EMAIL_ADDRESS>Localizes content/en/docs/tutorials/kubernetes-basics/update/update-interactive.html
Tracking issue: #31677

@PratikforCoding Kindly resolve the issues or I will close this PR within the next 1 week.

@PratikforCoding  kindly sign EasyCLA.
You can find about EasyCLA here - https://docs.linuxfoundation.org/lfx/easycla
and the steps to sign EasyCLA are here - https://docs.linuxfoundation.org/v2/easycla/contributors/individual-contributor
You can start your signing EasyCLA from here - https://api.easycla.lfx.linuxfoundation.org/v2/repository-provider/github/sign/18706487/51478266/38056/#/?version=2

/hold

/label tide/merge-method-squash

/lgtm

/remove-hold


Improvements in deployment.toml on enabling email username

Description:

Current implementation of the deployment.toml has some inconsistent keys with the previous XML based configuration.
As you see in the previous configuration for "enable email based username" <EnableEmailUserName> is used in XML. And it is completely different to the new key in the toml file. It's there as enable_email_domain. So there can be a user experience issue, where a person with previous experience of the product (Existing/Old Developers) finds difficult to try out/map or search for the property consciously.
Also this config is under [tenant_mgt], which confuses more.
Also after enabling this, it is not applying to the super admin user credentials. And developer needs to update the super admin username separately, but it is not informed to the developer. I would suggest to do a validation there and inform the developer to update the super admin username in this scenario.
Related Issues:
https://github.com/wso2/docs-is/issues/1254

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[DE] Added areas to homeassistant_HassTurnOff and HassTurnOn

Now sentences like "Schalte den Computer im Wohnzimmer an" or "Mache im Wohnzimmer den Computer aus" will work

Summary by CodeRabbit

New Features

Enhanced HassTurnOff intent to support varied order of <name> and <area> parameters for more flexible commands.
Expanded HassTurnOn intent to include optional area specifications, allowing more versatile device activation commands.

Because there hasn't been any activity on this PR for quite some time now, I've decided to close it for being stale.
Feel free to re-open this PR when you are ready to pick up work on it again 👍
../Frenck


Improve and fix ensemble robustness

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[x] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (minor / major / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?
Sorry... After testing with real data and discussing with real users, I realizes a few problems with the last ensemble robustness PR.

Changed the name knutti_sedlacek to robustness_coefficient. Easier to understand and remember, also keeps the door open to implementing new algorithms of the same nature.
Added the excude_dims={"time"} kwarg to the apply_ufunc calls... I only tested with data without real time coordinates, so didn't see that this was needed when applying a ufunc on multiple arrays with different "time" indexes.
Fixed the signature of robustness_coefficient to accept Dataset too (same duck typing as change_significance) and to match the arg order (fut, ref).

This PR does break things, but those were released this morning, I'm guessing it's ok.

All linux tests (py 3.6, 3.7 and 3.8) were successful on my machine.


fix: upgrade example hz/protobuf using hz v0.6.3

What type of PR is this?
fix
What this PR does / why we need it (English/Chinese):
Upgrade the out-dated example hz/protobuf to use hz v0.6.3.
Which issue(s) this PR fixes:
N/A

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

@li-jin-gou
Please check https://github.com/cloudwego/hertz-examples/actions/runs/4943816148/jobs/8838686550
It took a long time than normal to run.


Revert "auto name change until openSTA signal name parsing is fixed"

Reverts Silimate/yosys#5
This PR is not needed anymore

@akashlevy please merge the revert


Spring Cloud Refresh event does not recognize removed properties?

See http://stackoverflow.com/questions/38932168/spring-cloud-refresh-event-does-not-recognize-removed-properties please
Overview
I have a Spring Boot 1.4 application running with Spring Cloud. My app is using the native profile to pull settings from an external config.properties file. The config server is embedded inside the same application.
In the config.properties file I have added the following setting:
app.setting=helloworld
What works:
When I change the property and send a REFRESH event, relevant beans marked are reloaded and the change is recognized correctly.
What doesn't work:
If I actually remove the property from config.properties, (by commenting it out for instance), the REFRESH event does nothing to actually refresh the application. Spring Cloud seems to not recognize the fact that the property is removed and when the data-binder proceeds to update the state of the world it misses the fact that the property is removed, and the corresponding bean linked to it must also be refreshed and its field set to blank/null, etc.
It looks like the data-binder only looks at what is at the moment available in the configuration and does not keep record of what was vs what is.
The only way to actually disable that setting in the bean configuration state is not by removing it, but by actually setting it to a blank value (which is a new value, given the setting is just a String). Note the field in Java bean mapped to this property has no default value other than null, and the value is not defined anywhere else (as in an embedded application.properties file, etc)
What might I be missing?

You are not missing anything. Binding only affects fields that actually have a value in the configuration. You need to work with that constraint really.

That's fine. Thanks for the confirmation. I am assuming there is no easy way to bypass the constraint or at least have it be some sort of a configuration knob?

Not that I am aware of.

Got it. Thanks again

I've run into a similar issue when refreshing an external config.properties file. The issue manifested itself with my config.properties because it only had a single entry in it.
To demonstrate:
Start by overriding the application properties
application.properties
some.value=abc

config.properties
some.value=xyz

Initially, using some.value will render "xyz"
To show that the value can be updated, change the value in the config.properties
config.properties
some.value=123

Using the /refresh endpoint, refresh the context and then using some.value will render "123"
Now, by removing the property, then we can see the value does not get updated
config.properties
// now empty

Using the /refresh endpoint, refresh the context and then using some.value will  still render "123". It hadn't recognised that the field had been removed, nor used the "abc" value from the application.properties.
The issue stems from the class ConfigFileApplicationListener which on line 428, identifies the properties file as empty, so doesn't load the file into the property sources that are later used to compare the new properties to the old in the ContextRefresher class. The scenario appears to keep the old one in memory.
To workaround this issue, when you only have a single property, you could add property like a.b which would force the file to be loaded with the no value and result in the correct functionality.
config.properties
a.b=true
// item removed, but use some property to make sure it's read later

Hope this helps


IDEM-438

fix take photo/select from device on ios
fix always savable
fix no title on file detail screen
Signed-off-by: Robert Bartlett<EMAIL_ADDRESS>https://github.com/dltxio/idem-mobile/issues/438

Need to update branch


ci: also test nixos-23.05

Fixes #11

FIXME: the nixosModules are still tied to nixos-unstable

Todo addressed in https://github.com/numtide/srvos/pull/193

merged.


Seamless animation!

I think it would be really awesome if users could generate mp4s and gifs by passing arrays with an extra "time" dimension and the keyword arg animate=True to arbitrary plotting functions.
The following approach would allow users to animate stuff in multiple subplots, possibly with multiple plotting commands, simultaneously:

If animate=True, anticipate an extra "time" dimension on the input data, and select the first time index for drawing the initial frame. They can pass FuncAnimation keyword args perhaps by using an animation_kw argument.
When calling e.g. fig.save('animation.gif'), create an animation by calling the same plotting commands with the same keyword args with successive time slices. This should also make sure the "time dimension" lengths are identical for all arrays passed to plotting commands with animate=True.

This is not release critical but seems like such an obvious and useful addition... whenever I have to make animations I have to look up boilerplate code and write code that manually creates and destroys successive artists, which is surely very easy to automate. One difficulty might be allowing users to control how text objects, labels, annotations, legends, etc. are updated with each iteration, but will cross that bridge when we come to it.
Edit: To clarify, the reason I am considering this is ProPlot already standardizes the positional arguments passed to plotting functions with standardize_1d and standardize_2d. So it would be straightforward to have these functions handle a third "animation" dimension.

This would be fun but possibly very complicated. It is no longer high priority.


Add a tool for advertising Eddystone urls from the command line.

What the title says. Super-simple python script for advertising an Eddystone url.

We are creating a new implementations directory for Eddystone-url. You may want to wait a day, then you can place this in the new directory along with several other eddystone implementations being added shortly.

I didn't see this in the implementations directory today. Are you going to send a new Pull Request?


Migrate salmon into the jellyfish monorepo

What would you like to be added:
Migrate all of salmon's packages into the jellyfish monorepo.
Why is this needed:
Faster iteration and less overhead on dependency management

Some questions:

Do we move the lambda packaging e.g. npm run package from the top level package.json to the lambda functions package.json, or do we combine the CI/etc at the root?
Do we keep the e2e workflow, will this slow down jellyfish deployment, how will we structure this and other CI components as part of jellyfish (similar to above)

Yeah we need to keep the packaging workflow, when we move whale and playground, there will be a packaging for them.
Yup I updated the narrative https://github.com/DeFiCh/jellyfish#end-to-end-testing

I think for safety best to just re-run and rebuild everything since that's the only way to ensure consistency.

Over the weekend, I realize the DeFiCh/whale project, which is a critical component has to be migrated before we can effectively migrate salmon to jellyfish.
Initially, I wanted to delay salmon migration until we have the whale project on DeFiCh/jellyfish.
However, the more I think about it, having salmon in a silo is unnecessarily unhealthy. We will do it regardless and have whale-api-client being used fixed to a static version. After we migrate the entire whale project over then we point it to local deps.

I spent another great deal of time thinking about the salmon project yesterday.
It can’t be within the jellyfish project, as jellyfish is a library/server project it doesn't have the necessary ops pipeline to guarantee the quality. It has to be a standalone project, as it is today, and run like a product project with the many ops pipeline.
Everything else is still required but it can't be part of DeFiCh/jellyfish because it needs more ops pipeline than jellyfish and the ops requirement scopes beyond jellyfish setup.

moved to another repo


Unable to find a specification for `PusherSwift (~> 10.1.1)

Compiling flutter project with pusher_channels_flutter: ^2.1.2 the following error:
[!] Unable to find a specification for PusherSwift (~> 10.1.1) depended upon by pusher_channels_flutter
You have either:
* out-of-date source repos which you can update with pod repo update or with pod install --repo-update.
* mistyped the name or version.
* not added the source repo that hosts the Podspec to your Podfile.
Using environment:
flutter 3.7.7
Xcode 14.2
iOS target  13.0
Have tried pod repo update and pod install --repo-update
Kindly provide suggestions how to remedy this as it is blocking iOS development

Are you able to share your podfile.lock for review?

Issue resolved.
The apps' iOS pod file required a source file to a custom package which caused the pod not to be found.
Adding source 'https://github.com/CocoaPods/Specs.git' to the app pod file resolved the issue.
Most probably not the best solution to the issue, but at least it removed the blocker so that I can continue evaluating Pusher for our application.

Thanks for sharing. Do you then receive any errors when you run 'pod repo update'? I know you mentioned you have already run this, but it seems the repo is still out of date according to this error.


Undefined GITHUB_API_URL and  GITHUB_LOGIN_URL

Hii Team
I have used this package to configure github with Cognito. I have deployed the package in my local and have publically exposed it using Ngrok.
In the src/github.js file, the following 2 parameters are not defined in the config.sh file (Because a comment, in the config.sh file, states that it is only required for github enterprises). Can you please help with the place where these parameters are required to be defined because I have currently hard coded these 2 parameters as follow, on line no 12 and 13 of github.js file.
apiBaseUrl = "https://api.github.com"
loginBaseUrl = "https://github.com"

Can you please help me with the same?

Thanks for this! I thought this was fixed already, but apparently not - now fixed by https://github.com/TimothyJones/github-cognito-openid-wrapper/commit/bb3ace5b87aa74200e226e72bfa21f59df90a8d0


feat: add fallback provider to get blockchain data

Add infura fallback provider for reading blockchain data if user doesn't have connected wallet

There is no handling for it. In case if there is a rate limit error, it just won't show results and user needs to connect wallet to see them.


How to: Create an environment variable without a shell command (shared server)

Hello
I'm struggling at sending my first email with Sendgrid in a Php environment (Api v3).
In the "Integrate using our Web API" guide, you ask to "Create an environment variable" by "running in the shell" a command. But as I use a shared server, I can't access to a shell.
So is there another way to set/create an environment variable?
Thx.
PHP Version: 7.0
Server : Mutualized server on Online.net

...same here.. :(
also, setting an environment variable means I'll have to set it again everytime my server goes down and rebots..
Is there any alternative to setting the APIKEY?
I've tried  this but with no luck..
$sendgrid = new \SendGrid($myAPIkey);

I'm also using
$sendgrid = new \SendGrid('COPY-PASTED-API-KEY');
and it's working for me.

An ENV variable is not strictly necessary - although it is a best practice to separate your API keys and such from your code. In situations such as you mentioned (shared hosts) you can skip the getenv() call and either import it from a credentials file (if you are using such), or just paste it right in the code as a string literal (as @DaWe35 posted).
Friendly reminder: Be careful not to commit any code containing API keys or other credentials to publicly-accessible source control (such as GitHub)

Thx for your time and answers @jhalag, @DaWe35, @Virneto.
On my side, it's still not work. I tried to put the api key directly in my php file.
So my following question is about the line "require 'vendor/autoload.php'; // If you're using Composer"
I'm not using vendor, again because of a shared server, so I don't really know how to use or if it's available.
Does it work for you because you're using Vendor ?
In my real example on my website, I don't have any feedback when I trigger to send an email.
With the example.php available in the folder sendgrid, I have at least a message saying "Authorization required"
Thx for any idea/help

Does it work for you because you're using Vendor ?

I'm not using Composer. I recommend you to download the latest release (https://github.com/sendgrid/sendgrid-php/releases/download/v7.3.0/sendgrid-php.zip), and rebuild the code from the tutorial.

Don't know if this helps, but is it to do with this section at the top of the code...
require 'vendor/autoload.php'; // If you're using Composer (recommended)
// Comment out the above line if not using Composer
// require("<PATH TO>/sendgrid-php.php");
// If not using Composer, uncomment the above line and
// download sendgrid-php.zip from the latest release here,
// replacing <PATH TO> with the path to the sendgrid-php.php file,
// which is included in the download:
// https://github.com/sendgrid/sendgrid-php/releases

I'm not using Composer, so mine looks like this
// require 'vendor/autoload.php'; // If you're using Composer (recommended)
// Comment out the above line if not using Composer
require("sendgrid/sendgrid-php.php");
// If not using Composer, uncomment the above line and
// download sendgrid-php.zip from the latest release here,
// replacing <PATH TO> with the path to the sendgrid-php.php file,
// which is included in the download:
// https://github.com/sendgrid/sendgrid-php/releases

And it's require("sendgrid/sendgrid-php.php"); because I've saved the entire sendgrid directory into
.../public_html/sendgrid on my web server
Then I use a $variable in
$sendgrid = new \SendGrid($api_key);
just so that I can set the key elsewhere in my code
Does that help?

Hello @fsimonin, @Virneto,
Please let us know if you are still having trouble.
Thanks to @DaWe35, @jhalag and @Robin-the-Frog for helping out!!!
With best regards,
Elmer


Fix a failing test due to recursion limit

What have you changed?
A test was failing due to recursion limit being set in configuration which I didn't notice in the Prost PR.
What is the type of the changes?

Bugfix (a change which fixes an issue)

How is the PR tested?
Tests pass
PTAL @Hoverbear @yiwu-arbug

/test

/test


Add entry of python-pkg-resources to rosdep/python.yaml

For https://github.com/ros/catkin/pull/798

I don't think this is required for the PR since pkg-resources is afaik always a dependency of setuptools but having this rosdep key is ok.

I don't think this is required for the PR since pkg-resources is afaik always a dependency of setuptools but having this rosdep key is ok.

Yeah. But should I add it as a dependency for safety?

should I add it as a dependency for safety?

If the code imports the package it should indeed be listed as a dependency.


Beginner: data format question

Hi All,
I'm quite new to Chartkick and the code examples work just fine.
However I want to put data from a database in a line chart. I don not understand what the format of this data must be.
If i just put it in an array the dates do not get displayed (but the name of the variable does).
I also tried JSON.stringify on my variables, but it doesn't help (Error Loading Chart).
I also tried to put the dates in a new Date().
I even used the moment() on the dates, just to make sure (-;
Can you please provide me with a code example (or explanation) of how to turn this into a Chart?
var f = 19;
var e = "2016-02-10";
var g = 5;
var h = "2016-02-11";

Thanks

Hey @remcoNL, try to make the format match that of the examples. You shouldn't need to convert anything to dates or use moment. For a line chart, the data above would be:
var data = {"2016-02-10": 19, "2016-02-11": 5};

Alternatively, you can use multiple arrays.
var data = [["2016-02-10", 19], ["2016-02-11", 5]];

I'd use whichever format is easier to get your data into.

Thank you so much for your quick response. When i 'type out' the dates, it works fine. But i'm making some mistake when i do this:
var f = 19;
var e = "2016-02-10";
var g = 5;
var h = "2016-02-11";

var data = [[e,f][h,g]];

new Chartkick.LineChart("chart-1", data);

O no, it works!

Cleaning up issues


Cannot connect when HomeAssistant is using HTTPS

Using simple authentication (http_password) with HomeAssistant and a valid LetsEncrypt certificate on HA's standard port (8123), WallPanel does not connect, does not indicate any error, and only displays a blank screen. Using Chrome on the same device will connect to the page successfully from the same network presence. Main page or subpages have same behavior. Use of automatic browser engine and the loading message disappears quickly without complaint and with blank screen. Use of Crosswalk project engine prompts with Request Denied to Security and net::ERR_INSECURE_RESPONSE
No observable logs to provide.

I can connect to https sites (https://www.google.com) without any issues using the native browser or crosswalk.   The "Request Denied to Security and net::ERR_INSECURE_RESPONSE" you get with Crosswalk it points to a certificate with an incomplete certificate chain specifically with LetsEncrypt certs. More information here: https://github.com/crosswalk-project/cordova-plugin-crosswalk-webview/issues/187.   I am testing with login credentials, it's possible that when I added cache clearing (due to another users feature request).

I am able to login and view HASS using the password with simple authentication.  I don't have LetsEncrypt because I only access my HASS on my own network.  However, I am able to connect to secure sites using valid certificates.   Are you perhaps trying to access the local IP address of your HASS instead of the outside address which you use to access HASS outside your domain?   This seems to be the issue, perhaps the Chrome browser is more forgiving than the WebView component used in Android or CrossWalk.   But so far I am not able to reproduce the issue without of course duplicating your exact setup.

I can investigate that possibility but working with WebView is a major pain and takes time.  If you know of a way to make WebView bubble up security errors or use the certificate store of the Android device, that would help with time and limit my need to investigate.   I will change this to a feature request.

I am not sure what you mean by this part:

Of course, now I cannot load the custom panels that contain javascript in them, but that doesn't relate to a https/blank screen scenario.

I have a few custom panels using the Floorplan for Home Assistant
framework. When I try to view that custom panel within WallPanel, I get the
following error. My assumption is to a javascript dependancy not met within
the browser in use in WallPanel. However, as I stated, with the certificate
alterations I made on my HA environment, I can view the HA web pages with
the exception of the Floorplans. So this is a different issue all together.
https://homeassistant:port/static/custom-elements-es5-adapter.js:13:615
Uncaught TypeError: Class constructor HaPanelDownstairsPlan cannot be
invoked without 'new'
On Tue, Aug 7, 2018, 11:30 AM Michael Ritchie<EMAIL_ADDRESS>wrote:

I am not sure what you mean by this part:
Of course, now I cannot load the custom panels that contain javascript in
them, but that doesn't relate to a https/blank screen scenario.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thanksmister/wallpanel-android/issues/7#issuecomment-411119228,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOIquZ8Tn0BdkVnSXzsjDaHI2YyD0LuDks5uOcCNgaJpZM4VyD1S
.

Can you update your WebView component from Google Play?

My tablet (S3) is autoupdating its apps, running Android 8. It is also using Chrome 68 which has the Android System Webview component built in.

I am talking about this component: https://play.google.com/store/apps/details?id=com.google.android.webview&hl=en_US.  This is installed separately on my Google Pixel running Android 8.   This component is driving the content when using the WebView component in Android applications.

I understand what you were asking. I cannot install or update or enable that component (currently disabled on my device). Quick research indicated Chrome took over that functionality, and looking in the developer options for my device, that is the case. I cannot select the Android System WebView component. Chrome is the only app serving those requests that can be selected. It is up to date.

I assume then you are using the latest WebView component already.   If there are incompatibilities with the JavaScript, I can't address them with WallPanel because this is part of the WebView component.  I think I have found a way to show and ignore security errors and will add this to the next release today.  Maybe that will resolve some of the issues with the SSL certificates.    There is a paid application that you may wish to try if you can't get WallPanel to work, they have done extensive work on the app to make it compatible: https://www.ozerov.de/fully-kiosk-browser/

I have released 0.8.2-beta.8 which contains error handling for SSL certificates, let me know if this helps.

I think this might be an issue for Floorplan, to make it more compatible with different (and older) browsers.   It might be a simple JavaScript issue that can be fixed to increase compatibility.