Rule request: minimal-namespace-depth

TypeScript code being linted
// File one
namespace A.B.C1 {
    export const D = 7;
}

// File two
namespace A.B.C2 {
    // Could also be referenced by C1.D
    const copy = A.B.C1.D;
}

Expected behavior
The A.B.C1.D reference is unnecessary long, which leads to bloated source files and bloated output code. It would be beneficial for folks still in namespace land to have a rule that blocks using unnecessarily prefixed namespace paths.

@andy-hanson does #2008 close this issue?

Yes, I mentioned this actually.


[BUG] Advanced functions causes SyntaxAnalysisException

Describe the bug
I am using AWS OpenDistro 7.10 with SQL queries.
Using advanced functions in select clause gives SyntaxAnalysisException whereas simple functions works.
Example
Working example:

SELECT       DATE_FORMAT(data.timestamp, 'yyyy-MM-dd HH:mm:00.000') data__timestamp1m FROM       data* AS data  WHERE  (data.timestamp between '2021-07-19T08:40:00Z' AND '2021-07-20T13:00:00Z') GROUP BY DATE_FORMAT(data.timestamp, 'yyyy-MM-dd HH:mm:00.000') ORDER BY DATE_FORMAT(data.timestamp, 'yyyy-MM-dd HH:mm:00.000') DESC LIMIT 1000

Failing example:

SELECT      CASE WHEN MINUTE(data.timestamp) % 2 = 1 THEN DATE_FORMAT(ADDDATE(data.timestamp, INTERVAL 1 minute), 'yyyy-MM-dd HH:mm:00.000') ELSE DATE_FORMAT(data.timestamp, 'yyyy-MM-dd HH:mm:00.000') END data__timestamp2m FROM      data* AS data WHERE (telemetry_yuri.timestamp between '2021-07-19T08:40:00Z' AND '2021-07-20T13:00:00Z') GROUP BY data__timestamp2m ORDER BY data__timestamp2m DESC LIMIT 1000

To Reproduce
Steps to reproduce the behavior:

check that a simple query give results without errors
check that an advances query give a SyntaxAnalysisException

Expected behavior
Advanced query to work or having a workaround
Host/Environment (please complete the following information):

OS: aws linux 2
Version 7.10.2

Additional context
Our use case is we want to group data into buckets of some period of time, on 1 min for example it works easily but on other we need more complex logic. It worked when I replace the between in the where clause to a >= and <= but this creates painless scripts which slows down the query.

Moved to sql repo.

Checked again but BETWEEN seems still missing in engine v2.


Can't find a way to remove connector below last item

I'm a noob and I tried to look for options/props to edit this option but can't find any.

Found the solution in the closed issues:
https://github.com/stephane-monnot/react-vertical-timeline/issues/59


Handle special characters

Hello, first of all great port! The only problem I am having is that it is not working with special characters such as *
How can we "rmthrash" all the files in a directory without having to delete them one by one, for example?
Thanks!

Hi, Thanks for reporting your user experiences.
I've just released a version 0.0.3 that now rmtrash can work with wildcard, which you can install from the following command:
go install<EMAIL_ADDRESS>
Please let me know if the latest version works for your reported case or not.

Wow, that was fast, I was going to do a pull request after work... Working perfectly now, thanks a lot!!!

Sounds great, thank you very much for the confirmation :)
Close as it is solved in<EMAIL_ADDRESS>


Auto conversion block doesn't drop

I can create and place the Auto Converter, but when I break it it just disappears. It should drop itself, like virtually every other Minecraft block. And some directions on how to use it (tooltip in JEI, for instance) would be real handy. I thought it was broken when I couldn't connect to it from the side.
Also, XNet will not connect to it, which makes automation using it a bit more difficult.

Could I get what version of AODC you're playing with?

16.1.8, so it says.


Travis reports (apparently) non-existent errors

Note:  this is apparently a bug in Travis, but recording the issue here anyway
This issue is a Bug
Related issue(s) and/or pull request(s)
None
Expected Behaviour
Travis should accept updated files, build the CR and update Firebase
Observed behaviour
Instead, Travis starts the builds but reports that it "errored" but does not report what the error is/was
Steps to reproduce

Clone the readium.github.io repo, master branch
Modify in some safe way (e.g. change a title)  the OPDS file in the epub_content folder
Push the OPDS file to the readium-github.io repo, master branch
Wait....  Check the Travis build here
Travis will report that it "errored" but not where.

Test file(s)
None needed
Product

Readium CloudReader app

Additional information
Tested on Mac OS Sierra, 10.12.6, Chrome 67 on a MacBookPro

It looks like the Surge authentication data has been revoked. I am looking into this.
...
surge --project ./dist/cloud-reader --domain readium.surge.sh
...
   Login (or create surge account) by entering email & password.
          email:
No output has been received in the last 10m0s
...

https://surge.sh/help/integrating-with-travis-ci

Seems fixed: https://travis-ci.org/readium/readium-js-viewer/jobs/408557385#L2761
Both Surge and Firebase deployments have been updated correctly:
https://readium.firebaseapp.com
https://readium.surge.sh/?epubs=https%3A%2F%2Freadium.firebaseapp.com%2Fepub_content%2Fepub_library.opds

Fixed by generating fresh Surge tokens for TravisCI.


Ensure that an attempt is not made to access attributes of non-objects

In my testing of #17, I am finding an issue with twig.attr when it is passed undefined, null, empty object, etc. as the first obj argument, e.g. in welcome.js:
// line 19
sb.append(twig.filter.escape(this.env_, twig.attr(context["__internal_2547295a140ac5330ced4d5f977bde50_1"], "link", ["\/login", "Please login.", "Login"], "method"), "html", null, true));

In this case context == {} because we passed an empty object to renderBlock as in #17 and so typeof context["__internal_2547295a140ac5330ced4d5f977bde50_1"] === "undefined" because that key is non-existent. However, in twig.attr, an attempt is then made to do this:
if (attr in obj) {
    // ...
}

Which is of course illegal because obj is undefined. So the question here is what behaviour do we expect? I've not made a PR or added tests for this case because I am not sure what we want here. The safest and quietest option seems to be to return null at the start of twig.attr if !obj.
Perhaps it would be better if the compiler added a check to the first param of twig.attr such as this:
twig.attr(typeof context["the_key"] !== "undefined" ? context["the_key"] : {}, "the_attr")

And if it so happens that context["the_key"] is a string or an integer or something else, then that is a user-land error.

I know this bug, but there is no easy fix. A first step would be to write a script which ports twig's integration tests.


Turki907

What article on docs.github.com is affected?

What part(s) of the article would you like to see updated?

Additional information

.


Discord DAVE Protocol (E2EE) for voice

DAVE? What? huh?
This mega-PR implements Discord's DAVE protocol.
DAVE is an end-to-end encryption scheme that uses MLS (message layer security). To add support for this we use a fork of libdave (a Discord-made open source library) to which i have changed to our code style and documented.
This also depends on libmlspp, which is a third party dependency from cisco, but is currently only available on GitHub and on vcpkg, as such we have made our own fork. Our own fork of mlspp changes the build scripts, as by default it compiles AND INSTALLS as a shared object libmlspp.so, and also installs libbytes.so, libhpke,so, libmls_vectors.so and libtls_syntax.so. We really do NOT want all these installing alongside dpp and becoming separated requirements so this is linked statically into libdpp.so.
Notes about Discord's libdave and Cisco's mlspp
Note that both libdave and mlspp are significantly smaller libraries in terms of poplarity and userbase than dpp, as the mls protocol is yet to pick up widespread traction beyond innovators at places like cisco, signal, facebook, and google.  In time, as mls becomes a more popular protocol (IF it becomes a more popular protocol) we can expect better documentation and support for it from third parties and things like libmlspp ending up in places like apt repositories.
Note that if the user disables voice, libmlspp isnt built at all, and nor is the entire of src/dpp/dave
Important: mlspp is not applicable for our code review. It is a security library and its implementation should be considered blackboxed and left well alone!
Refactoring
I have done significant refactoring of discordvoiceclient.cpp to split it into several separate files - if voice is enabled, everything under src/dpp/voice/enabled is compiled into dpp, else everything in src/dpp/voice/disabled is compiled into dpp; currently, a simple empty stub of discord_voice_client. This makes dpp with voice disabled an extremely streamlined version of the library.
Todo

[x] MLS negotiation of group
[x] Sending audio
[x] Receiving audio
[x] User join/part of exising MLS group
[x] Protocol downgrade if last MLS user leaves
[x] Documentation
[x] Tidy up and reformat libdave - pay attention to asserts

Code change checklist

[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] My code follows the coding style guide.
[x] I tested that my change works before raising the PR.
[x] I have ensured that I did not break any existing API calls.
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

NOTE: The internals of src/dpp/dave/*.{cpp,h} are not going to be converted to our code style at this point. They use camelCase variable names, private members ending with an _ symbol etc. These can be converted in time, but the priority here is to get the functionality in a usable state to make it available to others.


Kmesh: repair kmesh manager not effect in sockops on oe 23.03

sockops port is not same as other linux vendor. We need to adapt oe 23.03.
What type of PR is this?

/kind bug
What this PR does / why we need it:
repair issue #298
Which issue(s) this PR fixes:
Fixes #
https://github.com/kmesh-net/kmesh/issues/298
Special notes for your reviewer:
NA
Does this PR introduce a user-facing change?:
NA

NA

we donot need manually open a pr on release-0.3, just add a label https://github.com/kmesh-net/kmesh/pull/301#issuecomment-2099812781

/reopen


Exception handling

At this moment we use std::runtime errors and fc_light::exceptions for generate exception events. Need to create uniform exception format for external interface.
Required fields:

exception code
string description
source code location

Optional fields:

trace info

I figured out that fc_light::exception inherited from fc library is useful for this purpose. Trace info field is already consist in fc_light::exception class and more over this is filled automatically. All we need is modify the serialization format to more compact form (need to remove extra fields from final output).


Fix issues in generated code

Subj.

Codecov Report

Merging #34 (2089634) into master (6cced2b) will decrease coverage by 0.06%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master      #34      +/-   ##
============================================
- Coverage     84.72%   84.65%   -0.07%
  Complexity      792      792
============================================
  Files           147      147
  Lines          3515     3519       +4
  Branches        354      356       +2
============================================
+ Hits           2978     2979       +1
- Misses          397      399       +2
- Partials        140      141       +1

Impacted Files
Coverage Δ

...ast/astgen/codegen/java/AbstractNodeGenerator.java
0.00% <0.00%> (ø)

...org/uast/astgen/codegen/java/LiteralGenerator.java
94.11% <60.00%> (-5.89%)
:arrow_down:

...c/main/java/org/uast/astgen/analyzer/Analyzer.java
96.80% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6cced2b...2089634. Read the comment docs.


[Feature request] Add full relative paths for JavaScript files by default

Request
Add automatic detection of folder structures for JavaScript links like in the CSS links.
Justification
For example, my project structure:
|-- example1.html
|-- folder1
|   |-- example2.html
|-- folder2
|   |-- test.js

If I want link to test.js file in example1.html file I need <script src="./folder2/test.js"></script>. But if I want link to test.js file in example2.html file, I need print in my code <script src="../folder2/test.js"></script>. I want, to FuzzyFilePath able to determine automatically how many points need to set the name of the file.
Extended behavior
FuzzyFilePath for CSS:

Actual behavior
FuzzyFilePath for JavaScript:

Thanks.

Example 2
Line: https://github.com/Kristinita/Kristinita.github.io/blob/master/Giologica/Valerywork-Kiravel.html#L233
<img class="rightimg" id="Strelena" src="../anima_images/Strelena.png" alt="Стрелена" border="2px">

Extended autocomplete after src: ../anima_images/Strelena.png
Actual autocomplete after src: /anima_images/Strelena.png
Example 3
Line: https://github.com/Kristinita/Kristinita.github.io/blob/master/Giologica/Демагогия-Нестеренко.html#L91
<aside class="sidebar" style="background-image: url(../aside/Бразилия.jpg)">

Extended autocomplete after url: ../aside/Бразилия.jpg
Actual autocomplete after url: /aside/Бразилия.jpg
Only for css autocompletes I have correct autocomplete path.
Thanks.

Hi Kristinita.
In the correct trigger configuration, try changing the base_directory to false. Currently base_directory  seems to break the type of insertion:
{
    // html - <script src="..">
    "scope": "string\\.quote.*\\.html",
    "prefix": ["src"],
    "tagName": ["script"],
    "auto": true,
    "base_directory": false,
    "extensions": ["js"],
    "relative": true
}

Until i fix this, simply copy the scopes array to your "User Settings" and make the above mentioned changes.
Thanks for your Help.

Fixed with f7c70f61bc3c549ca8383bbbff9d2e475d20bc5d


added section headers and some brief notes under each; also links to headers

I've put some questions in the readme file on things we might want to discuss (e.g., feedback and contributions)
Check the readme rubric here for what we should do: https://utoronto-my.sharepoint.com/:b:/g/personal/j_calver_mail_utoronto_ca/EUS7cexMY2tIhMXxe79Dj0QBQeRtemdrXh0gh0mNNAoEZQ?e=jGxJI4
please add more to each section, there's a lot of info missing right now

This looks good!


Swift Package offer

Hello
I came across this project and it has been an almost perfect fit for us, thank you. We needed to add some functionality and have started moving internally to Swift Packages. I didn't have any trouble converting your project to a package, and would be happy to help you do so if interested. Ours is private and modified, but I could easily create a package and example project for you if interested.
Cheers
Dave

Hi @neodave,
Thank you for the kind words. Sure, feel free to post here any steps I could take to implement Swift Package support and will do so.
If there are any feature requests you think this could do with, do let me know.

Great! What I will do is simply convert your current example app into the app and a separate Swift Package. The wonderful thing is that it eliminates the whole CocoaPod thing for beginners. The example app simply pulls in the framework at launch.
We are heavily adapting your work to include a couple features, specifically the requirement that a contact have a valid email address, and if they have more than one to pick one. I am about to talk to the other dev, but I think we had to break up your logic a bit. You gather that contacts in the nav view, but sort and search them in the other view controller. That was causing us some issues tracking selected contacts through it all as we had them deal with the email issues. Let me see what the final result is and maybe I can share some of that work if you are interested.
But simply as a thank you for the great head start I will get your package done for you. Expect it within the week.

Actually I just went ahead a did it. Could you create a branch other than Master I can put a PR into? Maybe something like SwiftPackage? I've already converted the example app. And I created the package and pushed it to our repo as a temp solution. So when I push the example app back to you it won't include any of the cocoa pod or framework stuff. Just a package entry in settings, and the rest takes care of itself :)
Once you see it working just copy the actual framework to your repo and link to that. I will remove our copy once you have it up and running to avoid confusion

Yup, just tested it. Works great. Here is how your project will look using the package system. Much cleaner!

I have just created the swift-package branch for you.
Amazing work! Thank you!

OK I am ready to push that branch, but I don't have permission. I think you need to add me to that branch or something. Git can be confusing. Just this account neodave, that is what I will be pushing from

Hey,
I think you have to first fork the repo, commit against your version and then submit a pull request.
Let me know if that works.

Ah, the makes sense

On Mar 6, 2020, at 3:16 AM, Dragoș-Robert N<EMAIL_ADDRESS>wrote:
Hey,
I think you have to first fork the repo, commit against your version and then submit a pull request.
Let me know if that works.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dragosrobertn/KNContactsPicker/issues/2?email_source=notifications&email_token=AD6QPONLKP4YRUFIPG7NPXDRGC5OXA5CNFSM4LBD76M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOAUW3I#issuecomment-595676013, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD6QPONTVNDPYR5FVM5BBRLRGC5OXANCNFSM4LBD76MQ.

I have just merged your pull request. Apologies it took so long. Will look into merging it into master in due course. Feel free to continue contributing :)

Great, let me know if you find any issues

We did find a weird permission bug that is not included in this. I would have to ask the other dev, but basically your package will show no connections on first launch after asking for permissions or if unsure. We fixed it our branch, but I am pretty sure it was even in your example app.

Yes, I believe this was on my To Do List. :)


New Tracker Laos

https://track24.net/?code=LAOCC1000000009YQ
Is easy to extrack info from the track24.

https://github.com/hdnpt/geartrack/issues/39


[rcore] Rename GetScreenWidth/GetScreenHeight to GetWindowWidth/GetWindowHeight

GetScreenWidth() and GetScreenHeight() might be a bit misleading. I did not find out about that they return the window width/height until just now. This made me search for ways of getting the window size in a few projects in the past, and I was actually about to start making a pull request for making the functions before I found out they existed with a different name.
Long story short, I think that renaming GetScreenWidth() and GetScreenHeight() to GetWindowWidth() and GetWindowHeight() will be way better, especially because all other window-related functions have "Window" in them. Users might not know if these functions are directed towards the window or monitor (one can also call a monitor a screen).

@JojoPlusPlus Those functions return the screen framebuffer size that could the same as window or not. Also, it's a multiplatform function, Android or Web have no window.

@raysan5

@JojoPlusPlus ... Also, it's a multiplatform function, Android or Web have no window.

I think InitWindow sort of sets the raylib model, independent of how that maps on different platforms.  One can rename the parameters for free of course, using a prototype like
'''C
void InitWindow(winWidth int, winHeight int)

Since it doesn't force any kind of renaming.  that would clean things up a bit.

Renaming setters and getters is more complicated.  It might also be less complicated to introduce

```C
void InitScreen(screenWidth int, screenHeight int)

And either way, there is some explaining to be done, including with respect to platform differences.
I do think it would be devilish to explain
void InitScreenBuffer(screenWidth int, screenHeight int)

From my distance, it seems like a choose-your-poison kind of thing.  I'm not for breaking things though.


Generate pdf file from a Page as Wagtail Document

Hi Jonas, you have a great package, love it!
Do you think is it possible to create easily a function that return pdf file object as a Wagtail Document? Where should I start if I want to implement that? I tried to check your code to do that but I am not that pro. I need to advices.
Many thanks!

Thank you very much, it's nice that the package suits your needs :+1:
From my experience directly hooking into Wagtails Documents will not be too easy, as the documents are directly fetched from the database and extending other package's db-models can be pretty tough.
In my opinion the easiest way therefore would be to leave the Wagtails Document code as it is and create a document which simply points to store a rendered version of the .pdf file on the server instead (Lets assume you are rendering some class PdfModel):

add document reference attribute to you PdfModel , e.g. PdfModel.linkedDocument
if a new Instance of PdfModel is created you trigger the creation of a new wagtail document and store it into linkedDocument
if the PdfModel instance is deleted, you also remove the linked document
You'll need to render and to store the rendered .pdf file after the creation and updating of a PdfModel

This is going to be a little tricky, one possibility is to simply access the pdfs-views URL on your server
The nicer way would be to call the render function (django-weasyprint implementation) and to store the pdf somewhere

Hope this helps to get you started, I think this should actually be quite doable :)


feat: update seed, remove lorem ipsum elements on staging

Jira issue(s)
438 439
Overview
This PR updates seed. On staging we will be use more realistic data. And we have new accounts:

<EMAIL_ADDRESS><EMAIL_ADDRESS>- for testing a newly registered user
<EMAIL_ADDRESS>  \
<EMAIL_ADDRESS>  } for testing with assignments to courses
<EMAIL_ADDRESS>/
<EMAIL_ADDRESS> \
<EMAIL_ADDRESS>/ for testing creators of content

Screenshots

Closed and apply in https://github.com/Selleo/lms-core/pull/293


Does this work automatically or do I have to run the command each time?

Hey,
It's manual yeah. I was thinking about making it run as a daemon, but i figured for now, being able to clear lots of stuff out with 1 command would be enough.
Is it something you would want?
Thanks


xfce4-whiskermenu-plugin is missing

i prefer to use whisker menu than default xfce4 menu and its not available on base image

The base image is supposed to contain a basic set of Xfce packages and attempts to reflect the default Fedora Xfce experience. The whiskermenu plugin is not part of that, so I don't see a reason to include it in the base image.
You can, however, layer the package with rpm-ostree install xfce4-whiskermenu-plugin.

i prefer to use whisker menu than default xfce4 menu and its not available on base image

So I checked out the regular Fedora Xfce spin in a VM and it looks like there are a number of panel plugins installed on the live ISO by default, including the whiskermenu plugin, so I will include some of those in the Vauxite base image.
If there is a plugin or app that I haven't included in the Vauxite base image, and it's installed on the default Fedora Xfce spin, I will include it in the base image upon user request.


Wallet address now showing sent zen on Mac client.

I sent zen from my swing wallet to by Arizen wallet to four different address and the zen never arrived in my my Arizen wallet, even though more than an hour passed since I have sent the zen to the Arizen wallet.  What do I have to do to make sure the Arizen wallet reflects the transactions did occur and makes the balances available to me.  I see the transactions did occur and were updated on the block chain.  Below is the link to the specific address I sent my zen to.  What must I do to update my Arizen wallet to show the zen has been properly transferred so the funds are available to me?
Below is the addresses I sent the zen to in my Arizen wallet.
http://explorer.zenmine.pro/insight/address/znazdKQdnVWj3xKLrFtWT8anEyMjc8a3NBS
http://explorer.zenmine.pro/insight/address/znWdeZz8YGgcRZoxMwWmAuEmreRFTx6wtL4
http://explorer.zenmine.pro/insight/address/znjckuLgGmuWxHFYCGZN3m1RpSwaQVGPwBr
http://explorer.zenmine.pro/insight/address/znjNbExyE28KM1ug3spKmrHVTBJWHyJr7yS
Please help!
Thanks,
David

Hey David,
what version of Arizen do you have? Check it in About menu.
Then go to Settings and check your settings your Explorer and API addresses:

Save your setting, restart Arizen and let me know.
Lukas

I am using version Arizen version: 1.0.1.
I have changed the history setting to 150 items and then clicked saved.
I reopened the wallet and the zen appears to be credited to the addresses.
Thank you for your quick response, I was getting nervous for a while that I had lost my funds.
Best Regards,
David

On Nov 21, 2017, at 4:33 AM, Lukáš Bureš<EMAIL_ADDRESS>wrote:
Hey David,
what version of Arizen do you have? Check it in About menu.
Then go to Settings and check your settings your Explorer and API addresses:
Save your setting, restart Arizen and let me know.
Lukas
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

👍
solved, closing


Only master branch is visible

Gitlist shows only master branch in dropdown control.

@sergey-suloev I've tested the current master of this repository and can see a second branch in a sample branch. Please provide additional information about the used version, environment and repository.


configure initial ci-cd

also copy code from old repository

currently available in http://ec2-3-83-142-5.compute-1.amazonaws.com/


Sygic-App Problem with LocationProvider

Hello,
Sygic App does take a very long time to get a precise GPS location and updates it like stuttering. When I am indoor, it does never get a valid location and shows "inaccurate or no GPS" location error.
I'm on cm 13 nightly 2016-03-05 (A6.0.1) with licensed Sygic and MicroG GMS dated 2016-03-24.
On Catlog, I found some strange errors when recording a log and opening sygic:

03-31 13:03:06.338 E/Finsky  (10544): [1] com.google.android.finsky.utils.da.a(3106): No permission to get location. java.lang.SecurityException: "fused" location provider requires ACCESS_COARSE_LOCATION or ACCESS_FINE_LOCATION permission.

Then, there was an Error line telling me that "com.android.vending" misses permission "android.permission.WRITE_SETTINGS".

=> pm grant com.android.vending android.permission.WRITE_SETTINGS did not solve the problem. pm can't do that.
I checked, that Google Play Store has location rights granted and they already were there. Revoked and reactivated the location rights for Play Store and Sygic. => No solution found.
Tried to set "GPS location" to "Device only" and no solution.
Whats wrong here? When I open Google Maps or Ingress, I get a very precise location immediately (outside and inside). I am using Apple WiFi, Local GSM Backend, Local Wifi Backend with filled/online databases. Ticking them on or off does not change anything regarding sygic.

According to catlog I successfully got a location while running sygic in high accuracy location mode. But the App still complains about Bad vor inaccurate GPS. cm privacy off and location granted.
4-08 17:27:14.117 V/NlpLocationBackendFuser(27125): location=Location[network X,Y acc=907 et=+2d19h59m11s668ms {Bundle[{noGPSLocation=Location[network X,Y acc=907 et=+2d19h59m11s668ms], AVERAGED_OF=1, SERVICE_BACKEND_PROVIDER=gsm, SERVICE_BACKEND_COMPONENT=org.fitchfamily.android.gsmlocation/.GsmService, OTHER_BACKEND_RESULTS=[Location[network X,Y acc=16 et=+2d19h52m3s938ms {Bundle[{noGPSLocation=Location[network
X,Y
acc=16 et=+2d19h52m3s938ms], AVERAGED_OF=11, SERVICE_BACKEND_PROVIDER=wifi, SERVICE_BACKEND_COMPONENT=org.fitchfamily.android.wifi_backend/.backend.BackendService_}]}]]}]}]

It looks like only the local GSM and local WiFi/WLAN backends are giving (or have given) locations. Does this app complain about locations if a different backend is used? (Maybe try the Apple backend or the Mozilla backends by themselves.)
In this case it looks like the off line GSM location provider is showing 907 meters error and the local WiFi backend last reported a position over two days ago. I can imagine that an app looking for a good position might not like nearly a 1km error.
May not be much to help the GSM backend depending on your phone and what mobile/cell towers are in your area. But the local WiFi backend can learn your WLAN/WiFi APs if you run a GPS using app while walking around outside where you have a view to the sky and then might be able to give you a position while inside.

Thanks for your detailed explanation. I got the problem solved by upgrading LocalWifiBackend to the newest release version from F-Droid. Temporarily I upgraded and activated newest release of AppleWifiBackend. Now Sygic gets location very fast (2-3 seconds).


HLZ Touchdown Points doc missing slope type

Help for HLZ Touchdown Points tool in the Helicopter Landing Zone Tools in the capability folder needs clarification. Slope can come in two different types: percent or degrees. The tool doesn't seem to make a distinction between the two, so a user could make an argument that it should work with either. However if you look at the included Aircraft_Specifications the field Ground_Slope_Max_Degrees implies that the unit of slope measure should be degrees. The tool documentation should note this in two places:

in the Input Slope parameter help, and
in the tool Usage where a better description can be inserted.

I'm assigning to Natalie to check the requirements for the tool, whether it should be DEGREES or PERCENT and to add the update to both toolboxes.

@jrweakland any idea if these are supposed to be PERCENT or DEGREES?

PERCENT

Updated tool and doc...

Aircraft_Specifications field name to be: Ground_Slope_Max_Percent.
Input_Slope raster clarified to "Values must be in percent slope."
Added to Usage:

•	Input aircraft maximum slope specifications must be in percent slope.
•	Input slope data must be in percent slope.


[Feature Request] Card expansion to fullscreen

As per the material.io documentation, a card should have the ability to expand to fullscreen. I think this could either be on the card or the expansion panel components.
It would make sense on an expansion panel that if a prop such as fullScreen is set to true, when tapped/clicked, the expansion panel expands from that position to the full screen.

[x] This is a v1.x issue (v0.x is no longer maintained).
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Context

Your Environment

Tech
Version

Material-UI
v1.0.0

React
v16.3.2

I'm closing for #4117

@oliviertassinari 👍 Sounds good. I did a search for expansion panels and cards in specific. Thanks for checking this out.

@Auchindoun But It's unlikely it will come to Material-UI in the near feature. It's very challenging to get it right, and it's better not to implement such transitions if it's not done right.

@oliviertassinari Totally understandable. I just wanted it documented somewhere since I didn't see an issue like it originally 😄


[Experimental] Add table driven scenario support

NOTE: This is now classified as an experimental feature, and is disabled by default.

set allow_scenario_datatable=true to enable this feature.
What works:

step dynamic parameters can be resolved using scenario table
scenarios are run as many times as the table rows
if scenario does not use the table, it is not repeated, and run once.

TODO:

SuiteExecutionResult has to propogate the table driven scenario results appropriately
html-report etc should consume and report appropriately.

there can be spec as well as scenario table. the user should be able to select rows from either

refactoring specExecutor.go:

there are too many branch paths. these can be consolidated, and organized better

ProtoScenarioResult now holds the scenario DataTable. So this is duplicate info. there are as many scenario results as the number of row, and each hold a copy of the datatable. Perhaps there is a need for a meta entity - this needs to handle backward compatibility.

@sriv
Can you provide a sample spec file for this feature? I think that will be more helpful to us.

@osandadeshan - this is an experimental feature, and is implemented only in gauge core. Reports/IDE plugins do not work well at the moment.
Your spec seems ok, can you try setting allow_scenario_datatable=true and run the spec? It should iterate the scenario for every row.
You can see the console output reporting this well, but html etc reports do not recognize this yet.

@sriv I tried this with the following spec.

I executed "Login Scenario"
Below is my "default.properties" file

When I execute this using IntelliJ IDEA, the html-report is showing an parse error.
Errors:

            [Parse Error] C:\Users\onimalarathna\IdeaProjects\Gauge-UI-Automation-Demo\specs\test.spec:20 Scenario should have atleast one step => ''

@osandadeshan yes, I agree.
I admit this feature is too rough to be usable now. Consider this as an experiment we did to see how well this could be accomodated. The results look good, but there's still work to be done.
The biggest reason I don't recommend this feature yet, is the lack of support for reporting.

@sriv Yes. Actually this feature is very helpful and we'll wait until this implementation and testing phase completed.


docs: fix incorrect identifier authorizationCodeRepository

As far as I know, it has been authCodeRepository and AuthCodeGrant since 3.x
https://github.com/jasonraimondi/ts-oauth2-server/blob/036e5e4cedafbb3e8153dbab19c39346085eb0d8/src/authorization_server.ts#L43
https://github.com/jasonraimondi/ts-oauth2-server/blob/036e5e4cedafbb3e8153dbab19c39346085eb0d8/src/grants/auth_code.grant.ts#L42

I've gone through the entire commit history but haven't figured out how the identifier authorizationCodeRepository was introduced in the documentation :(


Running tests not in release mode shouldn't require changes to wasm/checksums.json

We have a wasm/checksums.json file in the repo which tracks checksums of wasms as built by CI (or equivalently by make build-wasm-scripts-docker). When running tests, the wasm/checksums.json file needs to be in sync with whatever wasms you have built which while developing, may change often, so wasm/checksums.json often ends up dirty with regards to git, and you need to remember not to accidentally commit it.
Locally as a workaround, you can run git update-index --assume-unchanged wasm/checksums.json so that it doesn't pollute your diffs. Ideally this wouldn't be necessary when running tests locally. We could use a (.gitignored?) wasm/checksums-dev.json or something for when running tests locally in debug mode, but not sure of the best solution here.

git add -f ... can be used to force add otherwise ignored files on git


Change K-9 funding link

We need to use the k9mail campaign for funding links to Fundraiseup
https://www.thunderbird.net/donate/mobile/?form=k9mail

@kewisch Could you confirm that this is the full link?
https://www.thunderbird.net/donate/mobile/?form=k9mail&utm_source=%s&utm_medium=android_app&utm_campaign=inapp_appeal_v1

Yes, looks good!


select count(*) statement

I have been searching around for the implementation of "select count(*) from ... where ..." but couldn't find it in the documentation / issue list. Is this something planned to be released in the future?

@polymorpher There is a count method, you can call:
MyTable.count.where(..)

That should give you exactly what you want. Please let me know if you get on with it ok.


[RFC] Support for GRUUnit in TVM/NNVM/Relay framework

Thanks for participating in the TVM community! We use https://discuss.tvm.ai for any general usage questions and discussions. The issue tracker is used for actionable items such as feature proposals discussion, roadmaps, and bug tracking.  You are always welcomed to post on the forum first :)
Issues that are inactive for a period of time may get closed. We adopt this policy so that we won't lose track of actionable issues that may fall at the bottom of the pile. Feel free to reopen a new one if you feel there is an additional problem that needs attention when an old one gets closed.
For bug reports, to help the developer act on the issues, please include a description of your environment, preferably a minimum script to reproduce the problem.
For feature proposals, list clear, small actionable items so we can track the progress of the change.

Please have a clear description of features which is actionable if it is a feature request, e.g. support GRUUnit in tensorflow frontend. If it is an RFC, please provide a detailed proposal.

I want support of GRUUnit in ONNX frontend.


Move byte literal validation methods from lexer to parser

Purpose
$subject.
Approach
N/A
Samples
N/A
Remarks
N/A
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #23979 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #23979   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf52629...0ed630d. Read the comment docs.


Add GET:/groups/{id}

We require and endpoint that returns information about a group. Authz shall be handled equivalently to #451 .

Implemented in #453.


How to handle binary download from an http get

Short of the ongoing topic of binary upload and download, I am trying to download some files (and save as binary files) from an http get. By setting the webformat to PlainText, and then saving the response.body to a file via adodb.stream, this works most of the time.  JPEG files have been mostly working well, but for some other types I am seeing the error from Webresponse.CreateFromHTTP:
ERROR - WebResponse.CreateFromHttp: -2147210474 (11030 / 80042b16), An error occurred while creating response from http
-2147023783 (80070459): No mapping for the Unicode character exists in the target multi-byte code page.
This is from the same comment that Tim mentioned here on 110.
https://github.com/VBA-tools/VBA-Web/issues/110#issuecomment-102888615
The raw binary from some files chokes the line:
Me.Content = Http.ResponseText
but CreateFromHTTP doesn't have context to know to skip the content assignment line.
I was trying to think of the easiest way to handle this. Looking at code, I suppose the recommendation might be to register custom converters, but I dont necessarily know the MIME types (all of them, anyway) in advance, and some get complicated (office files, for example).
I know that I'm asking for a binary payload; I was initially thinking I could  specify a WebFormat of RAW or Binary, which would provide some context to not parse the content, but not one of the options today.
Any suggestions?

I don't have an answer for you that uses the WebRequest/WebResponse objects, but I can tell you how I download files with the MSXML2.XMLHTTP60 object and you can try to adapt for this project.
1- send your GET/POST/whatever
2- get the filename from the Content-Disposition header (if needed)
3- use this "hack" to get a byte array from the response
Dim xhr As MSXML2.XMLHTTP60
Dim fileBytes() As Byte
Dim filePath As String
...snip opening and sending request...
fileBytes = xhr.responseBody
Open filePath For Binary As #1
Put #1, , fileBytes
Close #1

At least that is what works for me when my ASP.NET and ASP.NET Core sites send files down.

Thanks for that.  I think my implementation is essentially the same, and if (for this call) I comment out the line:
Me.Content = Http.ResponseText
Then the binary data is saved to file just fine. So its just the content parse I need to avoid (but only for this call).
Since I'm a ton of other calls in the webrequest/webresponse in this section, I was just trying to find a cleaner way without some hacking of  createfromHTTP.  But I guess the first hack is the setting of webformat to PlainText.  Maybe that makes the case for a webformat of binary/raw, and then createfromhttp could have context to not try to parse the content.
But the 'clean' library changes for the above are probably a bit too sophisticated for me.

For the interim, I added a WebFormat enum  value of 'Raw'.  I use that type for the response format, and then added some checks for Raw type to skip trying to parse content.
Then I just save off the body (binary bytes) as a file.  Works.

Seems like a good approach, mind sending a PR if you can?

Afraid that I am somewhat GIT challenged at the moment, but:
In File WebHelpers.Base
'' ' @property WebFormat ' @param PlainText ' @param Json ' @param FormUrlEncoded ' @param Xml ' @param Custom 'JK patch:05/08/19 '@param Raw 'end patch ' @default PlainText '' Public Enum WebFormat     PlainText = 0     json = 1     FormUrlEncoded = 2     Xml = 3    'JK patch:05/08/19     Raw = 4     'end patch     Custom = 9 End Enum
In File WebResponse.cls
In subroutine CreateFromHttp:
'JK patch 05/08/19     'Me.Content = Http.ResponseText     If Request.ResponseFormat <> WebFormat.Raw Then Content = Http.ResponseText    'end patch
in subroutine web_LoadValues
` 'JK patch 05/08/19
' If web_Request.ResponseFormat <> WebFormat.PlainText Then
If web_Request.ResponseFormat <> WebFormat.PlainText Or _
web_Request.ResponseFormat <> WebFormat.Raw Then
'end JK patch`


Unsupported exchange: kraken

I am getting Unsupported exchange: kraken error when trying to run Gekko via CLI. I have made changes in config.js with Kraken API and exchange: 'kraken'

Are you sure you are on the latest version of Gekko? It's working for me:
2017-08-24 16:26:27 (INFO): Setting up Gekko in realtime mode
2017-08-24 16:26:27 (INFO):
2017-08-24 16:26:27 (INFO): Setting up:
2017-08-24 16:26:27 (INFO):    Candle writer
2017-08-24 16:26:27 (INFO):    Store candles in a database
2017-08-24 16:26:27 (INFO):

2017-08-24 16:26:27 (INFO): Setting up:
2017-08-24 16:26:27 (INFO):    Trading Advisor
2017-08-24 16:26:27 (INFO):    Calculate trading advice
2017-08-24 16:26:27 (INFO):    Using the strategy: toggle-advice
2017-08-24 16:26:27 (INFO):

2017-08-24 16:26:27 (INFO): Setting up:
2017-08-24 16:26:27 (INFO):    Trader
2017-08-24 16:26:27 (INFO):    Follows the advice and create real orders.
2017-08-24 16:26:27 (DEBUG):  getting ticker, balance & fee from kraken
2017-08-24 16:26:27 (INFO):

2017-08-24 16:26:29 (INFO): trading at kraken ACTIVE
2017-08-24 16:26:29 (INFO): kraken trading fee will be: 0.2%
2017-08-24 16:26:29 (INFO): kraken portfolio:
2017-08-24 16:26:29 (INFO):    EUR: 97.745500000000
2017-08-24 16:26:29 (INFO):    XBT: 0.000000000000
2017-08-24 16:26:29 (INFO): Setting up:
2017-08-24 16:26:29 (INFO):    Performance Analyzer
2017-08-24 16:26:29 (INFO):    Analyzes performances of trades
2017-08-24 16:26:29 (INFO):

2017-08-24 16:26:29 (INFO): Starting to watch the market: Kraken XBT/EUR
2017-08-24 16:26:29 (DEBUG):  scheduling ticks
2017-08-24 16:26:29 (DEBUG):  Requested XBT/EUR trade data from Kraken ...

I have also had no issues with kraken identifier during past 3 releases.

Ah I think I got the issue:
Gekko just updated the version of the kraken dependency. Can you manually run: npm install to install the now required version of the kraken dep?

Thanks @askmike, issue is resolved now!

I thought this would close the issue, but I am still getting the error even after manually doing npm install

Can you try the following:
npm list | grep kraken-api

(it should output ├─┬<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>
And try again?

Does the same thing after

Can you try the following:

npm list | grep kraken-api
(it should output ├─┬<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>And try again?

@decoye @discosultan @pushkarnagle Can you download this branch and try again?
https://github.com/askmike/gekko/tree/debug/kraken

@askmike Command line with 'kraken' is working fine for me

cool, can @decoye or @pushkarnagle try the above branch?

2017-08-25 19:12:14 (INFO):     Setting up Gekko in realtime mode
2017-08-25 19:12:14 (INFO):
2017-08-25 19:12:14 (INFO):     Setting up:
2017-08-25 19:12:14 (INFO):              Trading Advisor
2017-08-25 19:12:14 (INFO):              Calculate trading advice
2017-08-25 19:12:14 (INFO):              Using the strategy: MACD
/root/debug/node_modules/kraken-api/kraken.js:31
const rawRequest = async (url, headers, data, timeout) => {
^
SyntaxError: Unexpected token (
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/root/debug/exchanges/kraken.js:1:76)
Gekko encountered an error and can't continue
Error:
Unsupported exchange: kraken
Meta debug info:
Gekko version: v0.5.5
Nodejs version: v6.11.2

Ahhh the new version of the kraken dependency requires a newer version of
node. I'll change the dep!
On 25 Aug 2017 18:12, "decoye"<EMAIL_ADDRESS>wrote:

2017-08-25 19:12:14 (INFO): Setting up Gekko in realtime mode
2017-08-25 19:12:14 (INFO):
2017-08-25 19:12:14 (INFO): Setting up:
2017-08-25 19:12:14 (INFO): Trading Advisor
2017-08-25 19:12:14 (INFO): Calculate trading advice
2017-08-25 19:12:14 (INFO): Using the strategy: MACD
/root/debug/node_modules/kraken-api/kraken.js:31
const rawRequest = async (url, headers, data, timeout) => {
^
SyntaxError: Unexpected token (
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/root/debug/exchanges/kraken.js:1:76)
Gekko encountered an error and can't continue
Error:
Unsupported exchange: kraken
Meta debug info:
Gekko version: v0.5.5
Nodejs version: v6.11.2
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1036#issuecomment-324981933, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MDxzylUQMZocDl5nSie6Ku6JzUzTMks5sbwCJgaJpZM4PBOwG
.

Makes sense. I'm running node 8.4.0.

But i had problems with higher node versions to run gekko in general on this server. You remember mike, i had 8.4.0 installed.
null

@decoye and @pushkarnagle can you try latest develop?

works for me as expected.

@askmike How do I pull latest develop? I tried git pull and npm install and checked after that, but still getting Unsupported exchange: kraken error.

Can you try the following:
npm list | grep kraken-api
(it should output ├─┬<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>And try again?

hey hey,
still im getting wrong dema candles.

2017-08-26 11:39:06 (DEBUG):    calculated DEMA properties for candle:
2017-08-26 11:39:06 (DEBUG):             long ema:<PHONE_NUMBER>6
2017-08-26 11:39:06 (DEBUG):             short ema:<PHONE_NUMBER>1
2017-08-26 11:39:06 (DEBUG):             diff: -0.31377
2017-08-26 11:39:06 (DEBUG):             DEMA age: 6 candles
2017-08-26 11:39:06 (DEBUG):    we are currently in a downtrend @<PHONE_NUMBER>0 (-0.31377)

where infact we are:

long ema:<PHONE_NUMBER>2
short ema:<PHONE_NUMBER>5
diff: 0.73787

@pushkarnagle you can run:

git checkout develop
git pull

@decoye not sure where the data from "we have" is from, but this is unrelated from this issue. Please open a new issue and post all relevant details.

@askmike Thanks! All looks good now!

I've actually released a new version of Gekko now (on the deftault stable branch). So if you want you can switch back and still have this fix :)


Add saving and restoring functionality

Need to add in the ability to save/load models using specified locations in the parameter file

Done


⚠ AVS (kty) has degraded performance

In cac47c2, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 8590 ms

Resolved: AVS (kty) performance has improved in 2366fc0.


Add conceal to highlights

One question I have is how much and what to conceal. I personally don't use or like conceal that much for markup, but I'd like to have some good default conceals for those that do.
There are some things that seems clear we'd like to have:

Hide urls in links
Hide delimiters in emphasis/strong/etc
Hide inline attributes

But I'm less convinced to conceal block delimiters such as:

Code block markers
Div start/end markers

I know some people like to hide them too, and it's what the markdown treesitter does, but personally I like having to see where the blocks begin and end.
@NoahTheDuke any input?


Update fluent-plugin-cloudwatch-logs to v0.7.3

Because v0.7.2 implementation contains a typo. :cry:
This typo is reported in https://github.com/fluent-plugins-nursery/fluent-plugin-cloudwatch-logs/pull/141.
Signed-off-by: Hiroshi Hatake<EMAIL_ADDRESS>

Could you update minor version of cloudwatch image?

Where is minor version of cloudwatch image...?

Here: https://github.com/fluent/fluentd-kubernetes-daemonset/blob/master/Makefile#L18
like https://github.com/fluent/fluentd-kubernetes-daemonset/commit/d9e95ad0aaab5ed5e24b64f9b16461fff579c34a

Oh! I've got it.
I updated the PR to reflect your advice. :smile:

Thx!


[FLINK-21532][Table SQL/API]Make CatalogTableImpl#toProperties and Ca…

…talogTableImpl#fromProperties case sensitive
What is the purpose of the change
*make these two functions mentioned above case-senstive, fix the misleading problem introduced in JIRA *
Brief change log

disable key normalizing in these two functions

Verifying this change
This change is a trivial rework.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

The failed test case is not related to this PR.
Will merge it.


feat: add optional json logging format for advanced configurations

Add extra environment variables to further configure the logrus logger in the extension.
This is very useful when the target app has to deal with different logger setups such as zerolog, zap, go-logr and others. This happens when integrating different lambda extensions and other libraries such as OpenTelemetry and the target application has zap or zerolog.
There are not significant code changes and not setting a variable will result in using the defaults previously used in the extension (logrus defaults) ensure compatibility with exiting users.
This feature enables the following setups:

Use json as a formatter
Skip sending timestamps as part of the log entry
Specify the format of the timestamps
Change default field names of logrus allowing log entries to use "ts" instead of "time" for example.

To use it, just setup the corresponding environment variables in the AWS lambda configuration and you can have logs like this ones:
{"level":"debug","msg":"Serving on <IP_ADDRESS>:4040","svc":"pyroscope-lambda-ext-main","ts":"2023-07-18T12:47:58.318877953Z"}
{"comp":"orchestrator","level":"debug","msg":"Starting Server","svc":"pyroscope-lambda-ext-main","ts":"2023-07-18T12:47:58.31884208Z"}
{"comp":"orchestrator","level":"debug","msg":"Starting Server","svc":"pyroscope-lambda-ext-main","ts":"2023-07-18T12:47:58.31884208Z"}
{"comp":"orchestrator","level":"debug","msg":"Starting self profiler","svc":"pyroscope-lambda-ext-main","ts":"2023-07-18T12:47:58.318807397Z"}

Thank you for the contribution @demula! I released a new version with this change: https://github.com/grafana/pyroscope-lambda-extension/releases/tag/v1.5.0.


[stdlib] Add StaticString.UnicodeScalarView

Add a new type StaticString.UnicodeScalarView, accessible via a
property .unicodeScalars on StaticString. Also add initializers for
StaticString that take a UnicodeScalarView or a
Slice<UnicodeScalarView>.
The motivating reason for this is to make it possible to slice a
StaticString down to a substring that's still typed as StaticString,
without losing the guarantee of being well-formed UTF8.
Fixes rdar://problem/23382521

Doesn't this require a swift-evolution proposal since it changes the stdlib API?

It does require a swift-evolution proposal.

Does every single stdlib API require a full proposal that has gone through the entire submission and review process? Seems like that could get very tedious. Or can relatively simple changes get by with just discussing it on the swift-evolution mailing list?

Does every single stdlib API require a full proposal that has gone through the entire submission and review process? Seems like that could get very tedious.

Yes, every API change that affects the user-visible API should go through swift-evolution.  We have been using this process internally, and it worked well.

Or can relatively simple changes get by with just discussing it on the swift-evolution mailing list?

Relatively simple changes don't require long discussions and long proposals :) A simple email to swift-evolution and a small proposal that doesn't have much more than a simple statement of motivation and the API diff would be sufficient.

PR updated to actually run the tests.

The related proposal has now been merged as SE-0010 and is Awaiting Review.

Since the swift-evolution request was rejected with rational, can I close this pull request?

I guess so. It's a shame, since there really should be some way of producing a slice of a StaticString, but it was rejected so I guess that's that.

Thanks!


Fix typos

This pull request fixes typos in comments and docstrings in the codebase.
@abhishekkrthakur

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.

@abhishekkrthakur

@abhishekkrthakur gently pinging here.

@abhishekkrthakur gently pinging here.

hi, if you have more upcoming fixes like this, please add them in this pr

hi, if you have more upcoming fixes like this, please add them in this pr

That's all for now.


CVE-2023-45288 in golang.org/x/net-v0.22.0

https://www.cvedetails.com/cve/CVE-2023-45288/?q=CVE-2023-45288
Identified via internal pipeline policy scanning

Thank you for giving us feedback on this.


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 8f2f853:
:clock10: Pending: waiting for processors (27 builds ahead of you)
:warning: Docs Build is busy, currently there are 27 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 8f2f853:
:clock10: Full build: the average full build time is 5 min(s) 22 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 8f2f853:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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


Create Word Mark for artx-market

New logo in https://github.com/macterra/artx-market/commit/75a0ba0dd8c53b5993e634b407672e038891a9d5

tested


Added a requirements.txt file for easy installation and portability.

This is for installing on one of Steward's web servers.
The server might be taken offline in the next couple years, and this makes installing/migrating trivial.

• added lxml, which I initially missed somehow

Thanks, this is great! However, further development should be on lwhitler/arxiv-mailer


Could not linkify the URL with super-long TLDs.

Currently Linkifier implimentation could not linkify the URL with super-long TLDs.
Linkifier assume the TLD lenght in 2 - 6 chars in the here,
https://github.com/sourcelair/xterm.js/blob/master/src/Linkifier.ts#L14
But, some new gTLDs have over 6 chars. (ex: .enterprises, .foundation, and so on).
Needs to update tldClause to support these super-long TLD names.

Details

xterm.js version: master (ff29fd6)

Steps to reproduce

echo 'https://example.enterprises'
URL does not linked.

PRs welcome, should be an easy fix :+1:

@Tyriar Would happy to contribute, for me 256 ought be enough for everybody? ;)

@jens1o the concern with working more on the web link matcher as is, is that it will eventually be replaced by something much smarter that can understand things like when brackets () are within or outside of a URL, see https://github.com/sourcelair/xterm.js/issues/583
And that one's a bit more work 😛 as it's replacing all the regex logic with a parser that will probably work by finding the start of a link and then move forward until the end is found.

Oh, sorry. I think I won't meet your expectations... Let's hope there will be some nice contributor with some good skills :D

No worries 😄

Merging into https://github.com/xtermjs/xterm.js/issues/583


🎉 1m downloads, and looking for maintainers

The Monokai Pro theme has reached 1 million downloads on JetBrains! 😄
The plugin is doing well, but still needs a lot of work. Unfortunately, I don't have the time and ability to keep up with all the issues or make forward progress on development. Let alone adding support for new apps. (I'm sorry for all the late responses!)
I would love to include another person on this repository as a maintainer to keep things moving forward. There is a lot of potential that I think can be unlocked with a little bit of help.
If you have experience writing plugins for JetBrains or other apps, lets chat. Thanks!

@monokai ❤


Initiate YAML schema registration only when YAML file is open or created

Fixes #160

I have not tested but the code looks great, congrats @hopehadfield

@angelozerr , I think there's a problem here. Without this change, If I open a quarkus project, I can see the properties show up in the application.yaml (renamed from application.properties) :

With this change, it no longer shows the propeties/namespaces (eg. config_ordinal, mp, quarkus, greeting)

@rgrunber I think you need some microprofile yaml config dependency in the project classpath

I think it has to do with whether the yaml language server starts before or after the call to https://github.com/redhat-developer/vscode-microprofile/blob/c15b06b4c3c9e934384be586140c64342d11724e/src/yaml/YamlSchema.ts#L122 . Currently we always start the YAML language server. This change makes it so that while the YAML language server only starts if a YAML file is opened. Even though I can clearly see the registered callbacks being triggered, the completions aren't showing up.
I was able to fix it by simply expanding the activation condition in this PR for yamlOpen() to also include the case when an application.yaml file is anywhere in the workspace.

Ok, I think I found the other problem. It starts here : https://github.com/redhat-developer/vscode-microprofile/blob/c15b06b4c3c9e934384be586140c64342d11724e/src/extension.ts#L68-L69
We start the MicroProfile language server, and once done, we pass the language client reference to the yaml cache, so we can make the necessary queries for properties. However if we don't register YAML support, then the cache is undefined, and this happens when no YAML file is opened.
Possible solutions are :

Just add some code to yamlOpen() (also method should be renamed) that detects if any application.yaml (or the other patterns) are in the workspace so it activates in more cases. However this won't work when a user creates a new file.
Decouple the yaml cache from the activation of the yaml extension. It's just an object. I don't see a reason why we need to define it there. We could always define it on extension activation.


Dont run when in building phase

Hi!
Took some hours of debugging and googling to find out why docker-options failed on dokku v0.3.15.
The same issue as in dyson/dokku-persistent-storage#12, this PR contains the exact same change as fixed it in dyson/dokku-persistent-storage: dyson/dokku-persistent-storage@c61bcfa062aa6729d8c40f2a6ddc51d587fbd6a4

:+1:

This is now being resolved in a new version supporting the build phases added to dokku.
Give it a go if you want to help test it before it is merged into dokku as an official plugin.
https://github.com/dyson/dokku-docker-options/tree/dokku-merge

Oh and sorry for the wasted time you spent trying to work out your problem!

Ahh, thanks for sharing! Congrats on getting an "official plugin" :+1:

Really should have just merged this to keep everyone covered in the interim.
Thanks!


firebase authWithPassword callback is not firing when ionic-service-core is included

The problem occurs on my Android 4.4 device. In the browser it is working fine. The response of the request can be viewed in the chrome dev tools network tab, but the callback is not firing.

 FirebaseAuth.$authWithPassword(user).then(function(authData) {
                console.log(authData);
            }).catch(function(error) {
                console.log(error);
            });

I am using
"ionic": "driftyco/ionic-bower#1.0.1",
"ionic-service-core": "0.1.10",
"firebase": "2.2.9",
"angularfire": "1.1.2"

There is no connection between firebase and ionic-service-core. I'd suggest checking out the firebase help page


Added job for email link click processing

ref https://linear.app/ghost/issue/ENG-1815/
ref https://linear.app/ghost/issue/ENG-1817/

created a job queue entry for processing email click redirect actions

Our existing handling relies on a sequence of DomainEvents, resulting a couple updates by different services. The challenge here is that while we maintain the walls around each particular service, this is creating performance issues for Ghost under heavy load.
This change provides a pathway for us to respond to the redirect lookup as far as possible while encapsulating the actions needed by Ghost's main process within the response lifecycle, such that we do not create additional load on Ghost by accepting a new request before all the processing is done with the previous.
In other words, we have a request queue and gate requests as a means of not overhwleming Ghost. We need to ensure that any processing due to a request is contained in that response lifecycle, even if it means very slight delays in returning the response to the user.
We have a new model here instead of creating and emitting events, we inject the job creation fn into the service package as a cleaner means of creating the job. This allows us to keep the job in the service wrapper (within ghost/core) which is necessary to use knex and other imports like the settings cache.
Got some code for us? Awesome 🎊!
Please include a description of your change & check your PR against this list, thanks!

[ ] There's a clear use-case for this code change, explained below
[ ] Commit message has a short title & references relevant issues
[ ] The build will pass (run yarn test:all and yarn lint)

We appreciate your contribution!

Todo:

[ ] Add support for node events (e.g. member.edited); this is in a separate PR that needs tests
[ ] Add/update tests
[ ] Add config for the job that isn't just based on the queue being enabled (AND logic)

Some thoughts:

Would it make more sense for these jobs to be in a separate package?
Should we split up this job so that we can insert a lastSeenAt job row that is member-uuid-specific, which would be a way to prevent duplicative work? [this would be an optimization, and one that may not be necessary]


Support for multiple statements per instruction

When working with mysql, it is not possible to query back the values of the inserted row like it's possible with postgres' RETURNING (...)instruction.
So the go-to way, to do this at the moment would be to first have an :execresult action and then a seperate :one for querying by result.LastInsertd().
It would be great, to have a feature which allowes multiple SQL queries per action, e.g.
/* name: CreateUser :one */
INSERT INTO users (email, password) VALUES (?,?);
SELECT * FROM users WHERE id = LAST_INSERT_ID();

I think this also opens up the opportunity to handle complex data insertions, which mutate multiple database tables in one transaction with a single function call.

@philippta Thanks for the feature request.
I won't be added support for multiple SQL queries per command. Instead, you can use a transaction with two queries to get the same result.
/* name: CreateUser :exec */
INSERT INTO users (email, password) VALUES (?,?);

/* name: SelectUser :one */
SELECT * FROM users WHERE id = LAST_INSERT_ID();

You'd use the generated code like so:
package main

import (
    "context"
    "database/sql"
)

func (ctx context.Context, tx sql.Tx) error {

    q := New(tx)

    err := q.CreateUser(ctx<EMAIL_ADDRESS>"secret"})
    if err != nil {
        return err
    }

    user, err := q.SelectUser(ctx)
    return err
}

Could you please add this example to https://github.com/kyleconroy/sqlc/blob/master/docs/transactions.md ? It's much clearer to me than the example there. If you'd prefer, I'd also be happy to make a PR with this doc addition.


Adding firebase storage capability (w/ multiple files)

Following the patterns from <firebase-document>, I would like to ask for review for that addition of two components: Polymer.FirebaseStorageBehavior and <firebase-storage>
firebase-storage allows the uploading of a single or multiple files from a File object or an Array of File objects (FileList) fed to the file attribute. You can then call the .upload() method to upload the files manually or just add auto-upload attribute to automatically upload the files.
All files and their statuses are returned via the uploaded-files property, where each element inside will be updated via events inside.
Other methods:
destroy() to delete the file / folder in the path.
resume(i), pause(i), cancel(i): the methods found in the upload tasks. The i parameter is for accessing the file element in the array returned from uploaded-files
Example usage as follows:
<firebase-storage
        id="fs"
        path="/users/{{user.uid}}/files"
        files="[[files]]"
        uploaded-files="{{uploadedFiles}}"
        on-error="catchError"
        auto-upload>
</firebase-storage>

You can play with the demo here: https://practice-play-for-polymer-tjmonsi1.c9users.io/
This is an offshoot of the PR #30 and hopefully a fix for #10.
Some bugs:
cancel, resume, and pause sometimes doesn't respond even when wrapped around a promise. Will try to figure out why.
Would like to get some feedback

@mbleigh Hi, would like to know any updates on this :)

Plan to take a closer look next week, thanks
On Fri, Aug 19, 2016, 9:02 AM Toni-Jan Keith Monserrat <
<EMAIL_ADDRESS>wrote:

@mbleigh https://github.com/mbleigh Hi, would like to know any updates
on this :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-241059294,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAD_u_Nns0SG6CdfbesS10DW6aMFNIoks5qhdOKgaJpZM4JIHKx
.

We're trying to use this and we can upload files fine. We're having trouble understanding how to access files from firebase storage however. One way we imagine this working would be something like:
<firebase-storage
      id="fs"
      app-name="myapp"
      path="/path/to/image.png"
      files="[[files]]"
      on-error="catchError">
      </firebase-storage>

I see that there's a function for getDownloadUrl() but not sure how this should be used.
Or is the intended use to getDownloadUrl() on upload and store that in the database instead?
Could you please clarify how this should be used for downloading ?

Hi Saad,
Unfortunately, the current implementation does not behave the same firebase
document where it has a one to one correspondence with the path. It is only
used as of now as an uploader (that's why it uses getdownloadurl) But given
your idea, i think this can be done... Lemme check the code again and then
give a documentation.
On Sat, 17 Sep 2016 23:37 Saad Elbeleidy<EMAIL_ADDRESS>wrote:

We're trying to use this and we can upload files fine. We're having
trouble understanding how to access files from firebase storage however.
One way we imagine this working would be something like:

I see that there's a function for getDownloadUrl() but not sure how this
should be used.
Or is the intended use to getDownloadUrl() on upload and store that in the
database instead?
Could you please clarify how this should be used for downloading ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-247781923,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1sVp2fBvvyXTt37bjT_5MWTpGa4cks5qrAkcgaJpZM4JIHKx
.

@mbleigh would like to follow this up. I like the idea of @sbeleidy though but would like to know what would be the thrust of the polymerfire team on this

@googlebot The added parts are just from the messaging branch from the polymerfire.

@mbleigh hi. would like to know if i should rewrite this or is this still a part of the plan of polymerfire

@tjmonsi, can you fix the commits? I would like to get this to update it with more features.

@mbleigh @cdata @tjmonsi Please resolve this PR.

Will get back on this this tuesday. Just had a surgery. Sorry :(
On Sun, 12 Feb 2017 18:33 Josef Jezek<EMAIL_ADDRESS>wrote:

@mbleigh https://github.com/mbleigh @cdata https://github.com/cdata
@tjmonsi https://github.com/tjmonsi Please resolve this PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-279209436,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1peKeZPzmCjs_1PBqlHmmp2G3LHYks5rbt_hgaJpZM4JIHKx
.

@jorgecasar It is said that the branch has no conflicts with the base branch, so I am just waiting for them to merge. I don't know what @mbleigh 's plans on having a firebase-storage but then again... I am happy to help now that I am back (after my surgery). If there's anything I need to change, please help me and point it out so I can fix it easily. Thanks 😄

@tjmonsi there are still some conflicts in the PR, have a quick look at the changed files, seems there went something wrong resolving merge conflicts
I hope that PR makes it into mainline after all :+1:

Hmmm... what I'm going to do is to check if I can still do a rebase. If not, I'll just open another PR and close this one. Thanks @locomuco

@locomuco how about now? It's still saying that this branch has no conflicts with the base branch. I updated the dependencies though

@tjmonsi there are 2 thins to solve:

Issue with merge in some files firebase-messaging. Take a look file changes (https://github.com/firebase/polymerfire/pull/69/files).
It is not allowed to include commits from others.

I guess I'll just have to create another PR then. I think the one i did before made the conflict

@locomuco and @jorgecasar Thanks guys for the help. All checks have passed I think

:+1:

@tjmonsi it would be great if you revert changes on  demo/firebase-messaging.html. You remove de the new line at the end. It's a minor change, but it's in a file that this PR must not be affected.

@jorgecasar Oh... ok, reverting (adding the new line)

Thanks @tjmonsi! I guess the PR is ready to merge.
I hope @locomuco merge it soon 😄

@mbleigh ready to merge

@mbleigh, could you please merge this pull request?

Recently Firebase team announced multiple storage buckets support: https://firebase.googleblog.com/2017/03/multiplying-power-firebase-storage.html
Currently storage bucket sets in storage-bucket property in <firebase-app> element. Due to recent changes looks like it makes sense to set storage bucket in <firebase-storage> element.

@FluorescentHallucinogen storage-bucket should still be set on firebase-app, but bucket could perhaps also be provided on the individual elements. If not provided, it would use the default from the app so it'd be an override.

Please also take a look at third-party implementations:
https://webcomponents.org/element/olivierlangelaar/firebase-storage-upload (https://github.com/olivierlangelaar/firebase-storage-upload)
https://webcomponents.org/element/convoo/file-fire/file-fire (https://github.com/convoo/file-fire)

I like that idea that you mentioned on having a file element that has a
destroy call.
On Thu, 20 Apr 2017 01:11 Michael Bleigh<EMAIL_ADDRESS>wrote:

@FluorescentHallucinogen https://github.com/FluorescentHallucinogen
storage-bucket should still be set on firebase-app, but bucket could
perhaps also be provided on the individual elements. If not provided, it
would use the default from the app so it'd be an override.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-295349824,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1lgGMovYo5xtGik9ljS7UPWqF1fkks5rxkBJgaJpZM4JIHKx
.

@mbleigh

Since the primary purpose of this element is to upload multiple files, I think it makes more sense named firebase-storage-multiupload

What about download files?

I will work on it on the weekend after the GCP Extended IO here.
On Thu, 20 Apr 2017 01:19 Toni-Jan Keith Monserrat, <
<EMAIL_ADDRESS>wrote:

I like that idea that you mentioned on having a file element that has a
destroy call.
On Thu, 20 Apr 2017 01:11 Michael Bleigh<EMAIL_ADDRESS>wrote:

@FluorescentHallucinogen https://github.com/FluorescentHallucinogen
storage-bucket should still be set on firebase-app, but bucket could
perhaps also be provided on the individual elements. If not provided, it
would use the default from the app so it'd be an override.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-295349824,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1lgGMovYo5xtGik9ljS7UPWqF1fkks5rxkBJgaJpZM4JIHKx
.

@FluorescentHallucinogen that's what my proposed <firebase-storage-file> element would do:
<firebase-storage-ref path="..." download-url="{{url}}" auto></firebase-storage>

In most cases, though, it's better to get the download URL up front after an upload and stash it in e.g. the Realtime Database. @tjmonsi does your element support that workflow fairly easily?

@FluorescentHallucinogen I don't know if it would be possible to get a list of files (or folders) inside a bucket using the firebase api. We might need to get the google cloud api for it (and even that I haven't been successful yet in listing the files inside a bucket... or I think I might be misunderstanding something). That's why "downloading files" are more of just saving the downloadURL after uploading and putting it on firebase RTDB.
What I can think of, and is suggested by @mbleigh is a firebase-storage-file, given the firebase storage path of the file, you can have access to destroy(), getMetadata(), and downloadURL of the file.
I am going to work on it this weekend and push new commits for this.
Later, when it is working... I'll work on the 2.0-preview version of it as well

@mbleigh I think so, because I am just returning a list of tasks, which I think I can transform into storage refs easily.

What about
progress property that notifies progress represents a percentage value for the file upload
overwrite property that represents if a file can be overwritten or not
max-size that represents the maximum size of a file to allow
which implemented in third-party in <file-fire> element?

@FluorescentHallucinogen I feel this is becoming a scope creep now. I think this is for us to make firebase-storage be more like the same as the firebase storage api. I don't think overwrite and max-size is part of the api and can be implemented in a lot of ways. I don't want firebase-storage to be too opinionated or deviate from the firebase storage api. As such, if the direction though leads to have those kinds of properties. Why not right?

Agreed @tjmonsi. overwrite doesn't really make sense in a GCS world (it's not a file system, you'll only have pain if you pretend it is) and you can use rules to enforce e.g. max size constraints, I believe.

@tjmonsi @mbleigh Ok, looks like you are right. It was just a suggestion. :) What about progress property? It looks useful.

Yes it is. It's part of the example in firebase, so it can be useful
On Thu, 20 Apr 2017 14:00 Alexey Rodionov<EMAIL_ADDRESS>wrote:

@tjmonsi https://github.com/tjmonsi @mbleigh
https://github.com/mbleigh Ok, looks like you are right. It was just a
suggestion. :) What about progress property? It looks useful.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-295591728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1jQYL7SAKWBXZLTUFicEPpPYeqsnks5rxvSPgaJpZM4JIHKx
.

gr8 job, I just need this one soon! Ready for test when this one land!

@mbleigh I am nearly done and added two new elements: firebase-storage-ref and firebase-storage-task.
The firebase-storage-ref is what we would need to get metadata and download url given path.
The firebase-storage-task is what we use along side firebase-storage-multiupload to list out tasks and use the snapshot details to either resume, cancel, pause the files. firebase-storage-task can also be used in conjunction with firebase-storage-ref when we use the put/putString method of firebase-storage-ref
I'm just going to write good documentation for everyone

Any progress? ;)

Hi everyone,
I am going to get back on it tomorrow on documentation. But this is already
working
On Tue, May 2, 2017 at 2:32 PM Alexey Rodionov<EMAIL_ADDRESS>wrote:

Any progress? ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-298508465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1gK3k5xkDQItVIWJtUZSrDQc6s7wks5r1s3qgaJpZM4JIHKx
.

hi @mbleigh and everyone, please see if i missed something. Sorry for the delay. I was swamped with work recently

What about rename demo/storage-example.html to demo/firebase-storage.html?

I've tried the demo. If I click "Download" button, it prints the download URL to the console. What about real downloading?
The demo doesn't show previously uploaded files. If I update the page, info about the uploaded files disappears.

@tjmonsi Could you also please add a demo how to upload one single file (not multiple) manually (not automatically)?
(If I try to call the .upload() method I get an error.)

Is it possible to upload a Blob?

@tjmonsi Could you please resolve merge conflicts and answer my questions?

I am longing for it too!

Hi guys, sorry for the long delay. Just swamped with work. Will work on it
now :) and thanks for the feedback
On Wed, May 24, 2017 at 5:04 PM Pascal Gula<EMAIL_ADDRESS>wrote:

I am longing for it too!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-303663748,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1rpO9da0NiIxi2a8mk2vj-rh9MxYks5r8_KngaJpZM4JIHKx
.

Th(X/C) Toni-Jan (@tjmonsi)

I've tried the demo. If I click "Download" button, it prints the download URL to the console. What about real downloading?

@FluorescentHallucinogen It's just in the code. Just add window.open(url, '_blank') to do that. It is not part of the firebase-storage-ref (to download the item automatically) to give way to allow other forms of processing the downloadUrl from Firebase

The demo doesn't show previously uploaded files. If I update the page, info about the uploaded files disappears.

That's because you are not really listing the bucket's items but only the upload tasks that were done in this demo. If you want to show previously uploaded files, you may want to save the paths of the files in Firebase. Then, list them out using firebase-query, then use

(If I try to call the .upload() method I get an error.)

Do you have a minimal code for me to use?

Is it possible to upload a Blob?

As per https://firebase.google.com/docs/storage/web/upload-files#upload_from_a_blob_or_file
Yes. You can do that using  and then just calling put method. Like this
<firebase-storage-ref id="storage" path="/some/path"></firebase-storage-ref>

// on script

this.$.storage.put(blobFile, metadata) // this returns a firebase upload task. You can then use this and put it to firebase-storage-upload-task or just do anything with it

@tjmonsi How to get the Google Cloud Storage URI of uploaded file in gs://bucket-id/object-id format?

@FluorescentHallucinogen good catch there. You can get it via firebase-storage-ref by either using the method toString or just by accessing the attribute string
<firebase-storage-ref path="some/path/" string="{{gsString}}"></firebase-storage-ref>

@tjmonsi Thanks!
IMHO, string is not good name for the attribute. What about more meaningful, e.g. storage-uri?
Could you please add the URI output (printing) to the demo? (If you you want I can do it myself and send you a pull request.)

Please do :) I'm going out again today and will be back in the afternoon.
Thanks
On Thu, May 25, 2017 at 4:15 AM Alexey Rodionov<EMAIL_ADDRESS>wrote:

@tjmonsi https://github.com/tjmonsi Thanks!
IMHO, string is not good name for the attribute. What about more
meaningful, e.g. storage-uri?
Could you please add the URI output (printing) to the demo? (If you you
want I can do it myself and send you a pull request.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-303838685,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1nDi7mxi0idwlTyc_EXyO9o_UDI-ks5r9I_QgaJpZM4JIHKx
.

@tjmonsi Please merge my pull request https://github.com/tjmonsi/polymerfire/pull/4 and resolve the merge conflicts. 😉

Thanks.. Will check later
On Thu, 25 May 2017 19:10 Alexey Rodionov<EMAIL_ADDRESS>wrote:

@tjmonsi https://github.com/tjmonsi Please merge my pull request
tjmonsi#4 https://github.com/tjmonsi/polymerfire/pull/4 and resolve the
merge conflicts. 😉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-303985293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1t91ErGgYxGheqhzlJ7PxymzPxGrks5r9WG4gaJpZM4JIHKx
.

I've also found that toString method for some reason is not listed on API Reference page. If I rename it to e.g. toString2, all is ok… :thinking:

@FluorescentHallucinogen I think you need to sign the CLA now for this to work :)

I've also found that toString method for some reason is not listed on API Reference page. If I rename it to e.g. toString2, all is ok…

@FluorescentHallucinogen what do you mean?

@googlebot All is ok.

@tjmonsi

@FluorescentHallucinogen what do you mean?

Run polymer serve and check "Methods" section on http://<IP_ADDRESS>:****/components/polymerfire/#firebase-storage-ref page.

@FluorescentHallucinogen I think CLA is not yet ok. Can you try again?

@googlebot I signed it!

@tjmonsi Looks like a @googlebot bug. I signed Google Individual CLA Mar 12, 2016 06:22 PST. The ID of my CLA is<PHONE_NUMBER>65100465718. What should I do?

Should I try squashing the commit? guys?

@mbleigh PTAL.

@googlebot I confirm

@FluorescentHallucinogen can you try saying you confirm to submit to @googlebot ? It asks that "We need to confirm that they're okay with their commits being contributed to this project. Please have them confirm that here in the pull request."

I think the CLA is fine -- it's in a "terminal state" but it says I can merge on my discretion. I'm going to try to take another look through the code to see if we've got everything we need.

@mbleigh thanks. :)

@mbleigh I am waiting for firebase-storage. Thank you.

@tjmonsi What about the problem with toString method that I reported earlier? Why toString method is not listed in "Methods" section on the API Reference page?

https://firebase.google.com/docs/reference/js/firebase.storage.Reference#toString
On Wed, 31 May 2017 19:38 Alexey Rodionov<EMAIL_ADDRESS>wrote:

@tjmonsi https://github.com/tjmonsi What about the problem with toString
method that I reported earlier? Why toString
https://github.com/firebase/polymerfire/pull/69/files#diff-275a8c1aff9eeaf594afd11a42e9eedeR190
method is not listed in "Methods" section on the API Reference page
https://www.webcomponents.org/preview/tjmonsi/polymerfire/58462a6a9f52b06072c1ee684adfe2bc2b07e793/elements/firebase-storage-ref#methods
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-305164797,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1gmJNKGLN0dAEHQqf96VbAFY7yqhks5r_VEjgaJpZM4JIHKx
.

@tjmonsi My question was "Why toString method is not listed in Methods section?". Why, e.g. putString and updateMetadata are listed, but toString is not?

I haven't updated the Readme. Sorry about that
On Wed, 31 May 2017 21:30 Alexey Rodionov<EMAIL_ADDRESS>wrote:

@tjmonsi https://github.com/tjmonsi
[image: screenshot]
https://cloud.githubusercontent.com/assets/7892779/26634169/6e9a9d5c-461e-11e7-9c80-34212fde4fda.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-305186986,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1ijVQUudaQT2RS07EnNF81iA9T_7ks5r_WuGgaJpZM4JIHKx
.

Whoops! I just accidentally pushed this sucker to master based on a bad local git state. Hold on to your branch, please, I'm going to try to untangle this mess.

@tjmonsi can you please open a new pull request from the same branch? I've hard reset master back to where it was before.

Will do
On Thu, Jun 1, 2017 at 8:22 AM Michael Bleigh<EMAIL_ADDRESS>wrote:

@tjmonsi https://github.com/tjmonsi can you please open a new pull
request from the same branch? I've hard reset master back to where it was
before.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/pull/69#issuecomment-305354399,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChe1gP4VbpzVDfvAUx6ESQ-ZRqSNtKJks5r_gQ8gaJpZM4JIHKx
.

 Can you make an EASY example of an image upload process? Including html, scripts in polymer and the API structure?
Can this be used with polymer 2?
Can i use the firebase/polymerfire api or should i use the tjmonsi fork???

@eeerrrttty have a look here at this PR, the PR was moved


Add framework-specific docs

[ ] Remix
[ ] Next.js
[ ] Redwood.js
[ ] Express

Yes pleaseee :)

I got it working, but am not sure if this is the correct way to do it. I first tried in my app/services/emails.server.ts file to include the AccountCreated component. But that didn't work. Of course, I had to make the extension tsx! After that it was straightforward. Just import the <AccountCreated ... /> component like a normal component. Pass it some props like URL from process.env and send the email :)
I imported the AccountCreated component from react-email-starter/emails/AccountCreated.tsx so the preview function in the NextJS app works as well. Very nice workflow. Develop/test/check it in the live preview and import the same file/component in the Remix app. To have some demo values in the preview I created a default function AccountCreatedEmail to add the props. And from the remix side I imported the (non-default) AccountCreated component. I only need to check if the Docker build will work.
export default function AccountCreatedEmail() {
  return <AccountCreated url="test.com" />;
}

export function AccountCreated(props: AccountCreatedProps) {
...

Thanks for testing this out I was a bit  curious as to how I'll get it work. Is it possible to compile the emails programmatically and see if possible to reduce the docker image size. Will try this over the weekend. https://react.email/docs/cli#email-export

Are there any updates on this?
I'm currently unable to get it to work with express.

I find a solution for express, not perfect (because it need an build command) but not to dirty:

Install react-mails in an autonomous project (autonomous install)
Inside install babel depencies npm i @babel/cli @babel/core @babel/plugin-transform-modules-commonjs --dev
Compile (and transform in commonJS)  your components babel ./emails/ --presets=@babel/preset-react --plugins @babel/plugin-transform-modules-commonjs --out-dir dist

If you use esModule in your express server, you can remove --plugins @babel/plugin-transform-modules-commonjs

In your express server, install react and react-mail/render npm i @react-email/render react
Render (without jsx):

const { render } = require('@react-email/render');
const React = require('react');
const {MyMail} = require('../../react-email/dist/MyMail');

const emailHtml = render(
    React.createElement(MyMail, { props1, props2 }, null)
);
console.log(typeof emailHtml) // => string

Work with:
   // react-email project
   "@babel/cli": "^7.23.9",
   "@babel/core": "^7.24.0",
   "@babel/preset-react": "^7.23.3",
   "@babel/plugin-transform-modules-commonjs": "^7.23.3",

   // express project
   "@react-email/render": "^0.0.12",
   "react": "^18.2.0"


Callback function to handle Select category dropdown interaction not functional

Testing phase:

Everything is working fine except for the callback on selection. The interaction response is not recognized and the following error is encountered:

SOURCE FOR SELECT MENU IMPLEMENTATION:  https://gist.github.com/lykn/a2b68cb790d6dad8ecff75b2aa450f23
When I tried using their original options exactly -
import discord
from discord.ext import commands

client=commands.Bot(command_prefix=commands.when_mentioned_or("."))

class Select(discord.ui.Select):
    def __init__(self):
        options=[
            discord.SelectOption(label="Option 1",emoji="👌",description="This is option 1!"),
            discord.SelectOption(label="Option 2",emoji="✨",description="This is option 2!"),
            discord.SelectOption(label="Option 3",emoji="🎭",description="This is option 3!")
            ]
        super().__init__(placeholder="Select an option",max_values=1,min_values=1,options=options)
    async def callback(self, interaction: discord.Interaction):
        await interaction.response.send_message(content=f"Your choice is {self.values[0]}!",ephemeral=True)

class SelectView(discord.ui.View):
    def __init__(self, *, timeout = 180):
        super().__init__(timeout=timeout)
        self.add_item(Select())

@client.command()
async def menu(ctx):
    await ctx.send("Menus!",view=SelectView())

token=""
client.run(token)

still facing the same error -
PS C:\Users\panat\SE\Finbot\src> python .\discordBottest.py
2023-10-04 01:38:49 INFO     discord.client logging in using static token
2023-10-04 01:38:50 INFO     discord.gateway Shard ID None has connected to Gateway (Session ID: 95d00119d1605234e31b1a3b19b5b195).
2023-10-04 01:39:20 ERROR    discord.ui.view Ignoring exception in view <SelectView timeout=180 children=1> for item <Select placeholder='Select an option' min_values=1 max_values=1 disabled=False options=[<SelectOption label='Option 1' value='Option 1' description='This is option 1!' emoji=<PartialEmoji animated=False name='👌' id=None> default=False>, <SelectOption label='Option 2' value='Option 2' description='This is option 2!' emoji=<PartialEmoji animated=False name='✨' id=None> default=False>, <SelectOption label='Option 3' value='Option 3' description='This is option 3!' emoji=<PartialEmoji animated=False name='🎭' id=None> default=False>]>
Traceback (most recent call last):
  File "C:\Users\panat\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\discord\ui\view.py", line 427, in _scheduled_task
    await item.callback(interaction)
  File "C:\Users\panat\SE\Finbot\src\discordBottest.py", line 74, in callback
    await interaction.response.send_message(content=f"Your choice is {self.values[0]}!",ephemeral=True)
  File "C:\Users\panat\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\discord\interactions.py", line 801, in send_message
    await adapter.create_interaction_response(
  File "C:\Users\panat\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\discord\webhook\async_.py", line 221, in request
    raise HTTPException(response, data)
discord.errors.HTTPException: 400 Bad Request (error code: 50035): Invalid Form Body
In name: Username cannot contain "discord"

From my understanding, I think there is something wrong with the syntax in which the callback function is defined but was unable to find details or debug. Tried quite a few alternatives (https://guide.pycord.dev/interactions/ui-components/dropdowns) but no luck!
Measures taken:

posted the problem in https://gist.github.com/lykn/a2b68cb790d6dad8ecff75b2aa450f23 for any comments

Referring other sources to check for alternatives

PR:
https://github.com/vyshnavi-adusumelli/FinBot/pull/33
Notes:
The application name in the developer portal was assigned as "Discord-Fin-Bot" so the username has Discord present in it which led to interaction failures and exceptions. The application name was fixed and the corresponding changes were tested using the discord.ui library and it worked as expected. The test results are attached in the PR above


Change default build job preset to normal

Description
Changes proposed in this pull request:

changed the default build job profile to "normal" due to common limited cpu time budget

Related issue(s)
https://github.tools.sap/kyma/backlog/issues/5562

/retest

/retest

/retest

/retest


Correction for raw affiliation Concordia University, 1455 de Maisonneuve Blvd. W, Montreal QC H3G 1M8 Canada

Correction needed for raw affiliation Concordia University, 1455 de Maisonneuve Blvd. W, Montreal QC H3G 1M8 Canada
raw_affiliation_name: Concordia University, 1455 de Maisonneuve Blvd. W, Montreal QC H3G 1M8 Canada
new_rors: 0420zvk78
previous_rors: 04k83g518
works_examples: W2003661232;W1978057224
contact: 44e891d707bc629394c426ee5621c205:a779519c @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


ptipython does not run ipython startup script

ptipython is awesome! I use it to replace ipython. But I found ptipython do not execute the ipython startup files, e.g. ~/.ipython/profile_default/startup/*
I did a local patch to make this done, i.e. modify site-packages/ptpython/ipython.py, replacing "embed" function with the following code:
def embed(**kwargs):
    config = kwargs.get('config')
    header = kwargs.pop('header', u'')
    compile_flags = kwargs.pop('compile_flags', None)
    if config is None:
        config = load_default_config()
        config.InteractiveShellEmbed = config.TerminalInteractiveShell
        kwargs['config'] = config
    shell = InteractiveShellEmbed.instance(**kwargs)
    initialize_extensions(shell, config['InteractiveShellApp']['extensions'])
    RunStartupScripts(shell)
    shell(header=header, stack_depth=2, compile_flags=compile_flags)

def RunStartupScripts(shell):
    import glob
    import os
    startup_dir = shell.profile_dir.startup_dir
    startup_files = []
    startup_files += glob.glob(os.path.join(startup_dir, '*.py'))
    startup_files += glob.glob(os.path.join(startup_dir, '*.ipy'))
    for file in startup_files:
        shell.run_cell(open(file).read())

But is it OK to support this feature officially.

Yeah this is a good work around. Please do include this is the official release.

Just came here to report this. I, too, was surprised when ptipython didn't pick up my ipython config. Is there any solution besides patching the program code in site-packages?

No news on this front?

Would really love to see this fixed. This is a friction point for using ptipython as a replacement for ipython.
@jonathanslenders do you have any thoughts about directions for fixing this? Might be able to work on a PR but would need some guidance.


Pie chart label not shown when animation is disabled

Version
4.6.0
Reproduction link
https://jsfiddle.net/cLa5db8n/
Steps to reproduce
Define a basic pie chart and set animation to false.
What is expected?
Labels should be displayed on mouse hover.
What is actually happening?
Labels are not displayed. If I enable animation, then labels are displayed normally.

This issue started with 4.3.0 and happens to this day. v4.2.1 was fine.

@maliayas the jsfiddle works with echarts downgraded to 4.2.1, so I wonder if the bug was introduced with this commit in PieView.js line 205.  Still looking into it.  Cc. @100pah