Move most file handling logic to server

tabularfilemanager.fromURL() could perfectly replace routine of .getDataSource() https://github.com/okfn/datapackagist/blob/master/datapackagist/src/scripts/components/ui/resource-editor.js used for fetching and parsing resources files from remote URL.
It will also help to avoid hanging on large CSVs, as there is an option defining max part of the CSV to be processed and infered into schema.

@gvidon ok, great.
So, we should provide a distinct service on our service for CSV/Excel, that uses tabularfilemanager code (distinct from the service that will take json, for example).
But what about uploads? We'd want to work consistently with URL and upload. I know that tabularfilemanager should do this too, right? If it simplifies things, you can take the current tabularfilemanager code, and strip out the stuff that is there for browser compatibility, so that we have a good server-side solution with no compromises. Once we get it working in the codebase, we can maybe package it up for npm. WDYT?

@pwalsh yes, tabularfilemanager can process both local files and URLs. So, what you mean is backend view that takes either URL from GET (or from url param) or file data from POST. As a result this view should response with infered schema and parsed CSV (and some other properties returned by tabularfilemanager)?
And do we keep local CORS proxy for json?

Yes to all above.
It simplifies the client and puts most processing into the server.
Does this sound good to you? Any problems?

@pwalsh it sounds good in terms of an overall consistency. The only thing that is not absolutely clear to me is how to refine generic upload dialog according to new conditions and still keep it as generic solution — https://github.com/okfn/datapackagist/blob/master/datapackagist/src/scripts/components/ui/upload-dialog.js. But it is a minor problem.

@gvidon great.

tabularfilemanager validates resources over goodtables API. So, https://github.com/okfn/datapackagist/blob/master/datapackagist/src/scripts/components/ui/validationresult.js should be refined with this in mind.

[ ] Replace validation routine with reading of resources attributes which were returned by tabularfilemanager from server side.

@pwalsh should I move this to distinct issue?

@gvidon renaming, and adding a specific list of tasks (and, please comment on stuff that I miss in the list)

Description
We currently do all sorts of file processing purely client side (datapackage.json files and data files). For various reasons (processing speed, CORS), we want to move some logic to the server.
Tasks

[ ] Include a "next iteration" of our tabularfilemanager lib, that runs on the server. If it cleans up the API, then, remove aspects of this code that were originally written for browser + server compat.
[ ] Expose a suite of services service (service > controller + route, service can be unit tested, controller + route can have functional tests) for the following:

[ ] When the client receives a datapackage.json for upload or from URL, get the server to handle this, and it will return the file to the client (i.e.: Avoid CORS via a self-hosted proxy)
[ ] When the client receives a tabular data file (i.e.: CSV or Excel), costume this via an endpoint that wraps tabularfilemanager. The settings for TFM should be exposed as params on the endpoint (e.g.: infer, validate)
[ ] Do we need any special additional service for our datapackage-from-remote routines, or, will it be covered by the above two?

[ ] Make client adjustments as required to work with the new data flow (example, as noted here)
[ ] Write unit tests for the services
[ ] Write functional tests for the API endpoints

@pwalsh what I don't like about server side solution

as we discussed today — when user uploads large CSVs from local drive we will have to cut it locally and only after this send to server, as in other case large CSVs upload will create serious lag. This is like architecture design issue, because tabularfilemanager has a feature for cutting large data files — so, we will have to copy this feature on client side.
when user uploads datapackage.json from local drive why do we need to send it to the server? Probably I just misunderstood you?
having special server side service for datapackage-from-remote would be useful when user gets datapackage.json with resources which schemas are not yet defined and we need to infer schemas for each resource. But I'm not sure if we need this feature.

@gvidon

on the server, we can read the CSV as a stream. I don't think we need to cut in client. Large files take time.... we can't remove that issue. Most files, however, are not large.
don't need to send to server for local file of datapackage.json, for any reason, except to have a consistent API. I prefer we don't have two different code paths.

@gvidon as we agreed now on Skype - the handling of the datapackage.json can be done any way you prefer.

@pwalsh to clarify

will this resolve current CORS issue with processing CSV files referenced in a data package?
During server-side processing will the UI hang or get some indication of progress?

Apologies if these are already answered!

@pwalsh

IE9 doesn't support FormData. I'm able to polyfill it with https://github.com/francois2metz/html5-formdata
IE9 ignores Access-Control-Allow headers, so cross domain requests are almost impossible there. superagent had a solution, but then they removed it https://github.com/visionmedia/superagent/issues/409 We use superagent everywhere for http requests, custom solution will require us to rewrite much code.

Cross domain requests used to fetch registry profiles list. We can move this to backend if IE9 support is critical.
I'm not sure that registry list fetching is the only problem with IE9, possibly there will be others, need to test it.

Firefox 39 is ok

@pwalsh should implement registry profiles fetching on backend?
How about datapackage-from-remote — do we decide to place it on backend too, and do resources validation and infer once data package received from remote source?

Generic upload solution is not suitable for streamed upload or sending file in chunks, as http multipart spec requires file content to be in single request.
Existing solutions like https://github.com/balderdashy/skipper from what I understand still uploads the whole http request data (request body) to the server and then stream it into different sources, not creating temporary file. It doesn't eliminate upload lag.
What should eliminate upload lag is splitting resource file into chunks on the client and use http request Conten-range header thttp://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.16 to send data in chunks.
But probably much simpler would be to just truncate blob from file input on the client, and send already truncated data to the server
Here it is how client side truncation is done.
// .slice() returns Blob which will be sent to the backend
file.slice(0, _.min([file.size, config.maxCSVSize]))

@gvidon

For IE9, CORS, etc: https://github.com/matthew-andrews/isomorphic-fetch
For large files: let's not overthink this. Large files take time to process, it is a fact. The point of moving to server is to prevent lag in client, not to magically solve processing of large files. With server managing this, the client has less to do. I see no need whatsoever to start truncating files on client before sending to the file manager backend.

@pwalsh I'm not worrying about processing large files. I'm worrying about uploading large files to the server from user browser.

@pwalsh we will have to move from superagent to fetch in datapackage-from-remote, datapackage-registry and datapackagist itself. All these packages are used in browser currently.
Is it acceptable?

After lots of investigation, closing in favour of new issues to deal with this issue differently.


[FEA] [SPARK-46743][SQL] Count bug after constant folding

https://github.com/apache/spark/commit/b2f6474848f
Spark introduced a fix for Count bug handling in subqueries. Spark will prevent constant folding in subqueries that contain a COUNT-bug succeptible aggregate.  There is a new configuration flag spark.sql.optimizer.decorrelateSubqueryPreventConstantHoldingForCountBug.enabled  that is defaulted to true. We should ensure that our behavior is consistent with the fix that is now the default behavior.
We could fallback to GPU when the flag is set to false, but that logic is complicated since it's only COUNT in subqueries.

Priority is to investigate that we are not falling back with this new default state (true).


feat: wider page and better columns

First of all, this removes the grow effect when hovering around items (fixes #241). The effect wasn't really helping with anything in the interface by causing blurriness. Perhaps in the feature we can find a better way to give focus to the current 'hovered' item.

Secondly, this makes the page wider and stops using flex boxes for the columns and uses CSS columns instead. With this, the items don't need to be vertically aligned, so they don't need to take up the same space. That way, this won't happen anymore:

Of course this has its own drawbacks: the bottom items aren't aligned and perfectly beautiful. But I believe this is a much better looking solution.

The max columns we have is four, and they adapt to 3, 2 and 1 at certain breaking points.
License: MIT
Signed-off-by: Henrique Dias<EMAIL_ADDRESS>

Try it out here: https://ipfs.io/ipfs/QmUsCHJMC9vdub9uuZ4yJSSWhujLhNAhBpM94DZ5FQBiqx


Native types (or GCC-style vectors) for args/retval instead of unions

I'm thinking that it might be a good idea to pass native types (e.g., __m128, int8x4_t, etc.) or GCC-style vectors when possible instead of SIMDe's union.  This would cause a bit of cruft in most SIMDe functions since we'd need to convert to our union, but I think it would make things a bit more reliable.

Compilers are less likely to move stuff in and out of vector registers.  Inlining takes care of most of this issue, but doesn't solve it completely for all code.
Some compilers (cough MSVC) don't like passing around aligned unions, but seem to do okay with the native vectors.
I'm hoping this also gets rid of some warnings from GCC that I don't see another way to silence (the ones about the ABI changing in some GCC version), but I'd need to verify that.

If I decide to go ahead this is going to be a pretty noisy change, but the sooner it happens the better.

Finally done.  11412 insertions, 10327 deletions.  Less of it was search and replace than I'd hoped, but a lot of it was very repetitive (I thought about using Coccinelle, but a lot of the changes involved moving preprocessor conditions around).
This works a lot better.  There are still a few failures on a few compilers when I crank up the optimization level, I'll keep working on getting them all passing..


Using defaultSizes disables the minSize and maxSize

If you set defaultSizes. the min and max sizes on the panes are ignored

#103 is duplicate


Configurable Gateway Fees

Enable configurable gateway fees for routing outgoing payments.
Fees are declared using known RoutingFees structure from lightning crate.
By default, gatewayd configures a base fee of 0 msat and a marginal fee of 1%, which is on par with current behavior.
Next, we need to introduce a gateway API for configuring routing fees
Note: Breaking
This PR recommits db/migrations/lightning-v0 to avoid running a lightning db migration

Thinking about this a little more, I think we need to achieve the following:

Clients are presented a fee that will be charged for the payment before creating outgoing / incoming contract . They can use this fee to choose a gateway, display this fee to the user, and calculate the amount needed for the outgoing / incoming contracts.
For outgoing payments, gateways can set a fee they will charge for routing payments a given payment, net of routing fees they pay. For incoming payments, gateways can set a separate fee to receive a payment for the user compensating them for locking up capital in channel capacity and ecash balance.

So the first conclusion is that I think we need separate fees for incoming and outgoing payments. They fundamentally represent costs of different scarce resources: outgoing lightning liquidity, and incoming lightning liquidity + cost and risk of holding ecash.
Another conclusion is that no matter how we express these fees in the outgoing case, we need to take them into consideration when the gateway pays the actual invoice. We’ll need to pass these fee calculations on to PayInvoiceRequest.max_fee_percent field in our gRPC spec. Without doing this, the gateway only controls the fees they charge the end-user, but could still lose money routing the payment if routing fees exceed fees charged to the user.
I’m also not sure if lightning::routing::gossip::RoutingFees is the best way to encode our fees, because it seems like it’s intended to express the cost for forwarding a payment across a single channel. But the gateway’s job is different: completing the entire payment no matter how many hops it takes. And if we do use RoutingFees to express gateway fees, we'll need another parameter expressing how much of these fees the gateway will reserve for itself when paying the invoice. For example, if RoutingFees translates to a fee of x for a given invoice, the gateway might want to use a value less than x for PayInvoiceRequest.max_fee_percent to guarantee that they make a profit.

LGTM, let's iterate on it

thanks for review.
@justinmoon, take a look if you still need this change asap


CPP: Additional change notes.

Change notes for:

https://github.com/Semmle/ql/pull/194 (external contribution)
https://github.com/Semmle/ql/pull/225 (by Sauyon)

Fixed merge conflict and added a change note for https://github.com/Semmle/ql/pull/264/.


Opening a realm keeps the thread alive forever

Opening a realm keeps the thread alive forever, even if Realm::close is called.
This makes it impossible to exit the application correctly.
Tested on:
Realm version: 0.10.1
Platform: JVM Java 17 on Windows x86_64
Possibly related to #733

Hi @jakobkmar Thank you for the report. I strongly suspect the root cause of this is the same as #733, so I'll be closing this as a duplicate. You can follow #733 for updates.


Fix some memory leak issues

Codecov Report
Base: 100.00% // Head: 97.66% // Decreases project coverage by -2.34% :warning:

Coverage data is based on head (b8924a2) compared to base (2f314c5).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##            master     #315      +/-   ##
===========================================
- Coverage   100.00%   97.66%   -2.34%
===========================================
  Files           21       21
  Lines         2182     2180       -2
===========================================
- Hits          2182     2129      -53
- Misses           0       51      +51

Impacted Files
Coverage Δ

src/viztracer/patch.py
81.89% <0.00%> (-18.11%)
:arrow_down:

src/viztracer/util.py
94.28% <0.00%> (-5.72%)
:arrow_down:

src/viztracer/viztracer.py
95.18% <0.00%> (-4.82%)
:arrow_down:

src/viztracer/main.py
95.97% <0.00%> (-4.03%)
:arrow_down:

src/viztracer/code_monkey.py
98.32% <0.00%> (-1.68%)
:arrow_down:

src/viztracer/attach.py
100.00% <0.00%> (ø)

src/viztracer/viewer.py
100.00% <0.00%> (ø)

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

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


update navigation in navbars in story.html section

Title and Issue number
Title : update the navigation of navbars in the story.html
Issue No. : #345
Code Stack : navigation html css
Close : #345
Video/Screenshots (mandatory)
i have fixed the bug "When navigating from the story.html page back to the index.html page's testimonial section or trip gallery or any other navbars by clicking on the navbar link in the navigation bar, the user is not directed to that section as expected."
Checklist:
[ X] I have mentioned the issue number in my Pull Request.
[X ] I have commented my code, particularly in hard-to-understand areas
[X ] I have gone through the contributing.md file before contributing
Additional context (Mandatory )
Are you contributing under any Open-source programme?
yes i am contributing under GSSOC24 open source program

fixed the issue number #345 bug in navigation bar


Patch 2

Just found a broken link on the web page https://keda.sh/docs/1.4/concepts/scaling-jobs/ that I thought I could try to fix here.

Not very experienced with this. Forgot the signing part. Will try again. Closing this one.


Current project status

Hi, is this project still maintained or are you looking for maintainers?

Obivously no longer maintained...

My company can fork it. I use it all the time. We have had considerable success with other abandoned projects, like ccampbell/chromephp. Let me know.

I've forked it. Using it as part of the Bettergist Collective. I'll open source it as soon as I've released the Bettergist Collective.

Thank you for your interest!
This project is alive and will receive some updates in the future, but I've had to postpone most of these plans due to lack of focus time due to lack of time/funds. Right now, I can only tell you that I am planning to pick this up again as soon as time permits. If you need this for a commercial project and want to sponsor some features, feel free to reach out :+1:

I've picked up development again, reach out if you want to help sponsor some features :shipit:
I believe this has been answered, so I'm closing this for now. Please come back with more details if this problem persists and we can reopen this :+1:

I'd also like to inform you I've just set up a quick roadmap towards a future v2.0.0 release in #88
I'm also still looking for volunteers and/or maintainers to help with this vision, so let's also ping @staabm, @hopeseekr and others who have previously contributed some great additions to this project :+1:


Add incoming endpoint for ratings

@spsina I reviewed the code changes, and @alimahdiyar did a visual review.


Stop wagging tails on crit

About the PR
fixes  #25272

Why / Balance

Tails do stop wagging in death, but it was correctly mentioned that they do not stop wagging while in crit.  This PR fixes that.
Technical details

removed a code block that only toggled wagging if the creature was dead on state change
Media

https://github.com/space-wizards/space-station-14/assets/12686577/601ab857-a1a6-4471-b75c-ff420f7d773c

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: Tails now stop wagging on crit

GitHub question, but why am I seeing the Tritium flammable touch commits in this PR?

GitHub question, but why am I seeing the Tritium flammable touch commits in this PR?

I accidentally commited a change for a different PR to master before I made this branch, so I had to revert the commit.
TL;DR because I'm a dumbass

GitHub question, but why am I seeing the Tritium flammable touch commits in this PR?

I accidentally commited a change for a different PR to master before I made this branch, so I had to revert the commit.
 Thank you for making the mistake so hopefully I can avoid it in the future.


Audit tracing usage to ensure we have enough logs, at the right level.

Audit existing logging
Add debug logs at important steps
Log critical errors
Make sure we have appropriate levels to avoid noise
TRACE, DEBUG, INFO, WARN, ERROR

Is part of this adding an admin endpoint to modify the log level?

No, I think that is different. This is about making sure we are logging useful information, and at a useful level, to make sure the system is debuggable


Method showPopup is not supported in version 6.0

Doesn't working nothing correctly.
I get
telegram-web-app.js:1754 [Telegram.WebApp] Method showPopup is not supported in version 6.0)
telegram-web-app.js:1755 Uncaught Error: WebAppMethodUnsupported

Hi. Usually this error occurs if you open the site in a browser and not as a Mini App inside a Telegram app. Please provide more details on how you open the Mini App.

It makes sense yeah. Thanks for the answer !


Stickers

What size stickers? Just the logo for NC Open Pass or include Civic Camp?

we can talk more at 11, but I check out the spreadsheet I linked in the marketing notes and ticket 1138

I just found the spreadsheet! Sorry about that.
On Thu, Aug 16, 2018 at 10:06 AM, Janel<EMAIL_ADDRESS>wrote:

we can talk more at 11, but I check out the spreadsheet I linked in the
marketing notes and ticket 1138
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1112#issuecomment-413557485,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlJbYE32DBDYOhotcU5UrjBufKMlgPkvks5uRXxzgaJpZM4V29aw
.

Done

Reopening - to track the actual ordering

New StickerMule account is created and credentials shared with @ChrisTheDBA and @jtawes - need to upload designs with the quantities and then the order can be placed

FYI - Your proofs are ready for R223572342 (via NCOpenPass gmail account)

Order on the way


Desired Feature: Page Templates

Pretty self explanatory.
We plan on use BookStack for our SOPs (standard operation procedure) and we use a fixed template to unify all the documents and this feature will help a lot.
thanks

+1

Similar feature here - https://github.com/BookStackApp/BookStack/issues/129

Thank for the suggestion @eplastiq. As @Abijeet has pointed out this has already been required. Vote up issue #129 with thumbs up to make it more visible when new features are decided upon.
I will therefore close this issue.


Sample multi draw indirect throws validation errors at startup

When running the multi draw indirect sample on Windows with an RTX 4070 and the latest SDK I get multiple validation errors at startup:
[error]<PHONE_NUMBER> - VUID-VkBufferMemoryBarrier-None-09050: Validation Error: [ VUID-VkBufferMemoryBarrier-None-09050 ] Object 0: handle = 0x213ba640090, type = VK_OBJECT_TYPE_COMMAND_BUFFER; | MessageID = 0x441cbd8b | vkCmdPipelineBarrier(): pBufferMemoryBarriers[0].buffer barrier using VkBuffer VkBuffer 0x38ea17000000074c[] created with sharingMode VK_SHARING_MODE_CONCURRENT, has srcQueueFamilyIndex 0 (VALID). Source queue family must be VK_QUEUE_FAMILY_IGNORED or VK_QUEUE_FAMILY_EXTERNAL. The Vulkan spec states: If the synchronization2 feature is not enabled, and buffer was created with a sharing mode of VK_SHARING_MODE_CONCURRENT, srcQueueFamilyIndex must be VK_QUEUE_FAMILY_IGNORED or VK_QUEUE_FAMILY_EXTERNAL (https://vulkan.lunarg.com/doc/view/1.3.290.0/windows/1.3-extensions/vkspec.html#VUID-VkBufferMemoryBarrier-None-09050)

Seems this was fixed with #1218 already, that PR simply didn't reference this issue so it didn't get automatically closed.


specify annotation-tab component

Purpose: Generic container for primary components of each annotation tab (explanation, columns for annotation, missing values, specialized annotation component)
Actions / User flows:
N/A
Required input:

activeCategory: the tab needs to know what category it is handling, so it can pass that via prop to its own children
annnotationComponent: I need to know whether I am handling annotation for a categorical or continuous variable. I should be able to ask the store what annotation component I am supposed to load for the user.

Generated output:

none

Internal state:

none

Purpose: Generic container for primary components of each annotation tab (explanation, columns for annotation, missing values, specialized annotation component)
Actions / User flows:
N/A
Required input:

details - set of info about each annotation tab from the store (state.annotationDetails)
title - title of the annotation tab, sent to the specialized component
column to category map from store
data table from store

Generated output:

emits remove column event to annotation page
emits remove missing value to annotation page
emits update missing column values to annotation page
emits update for data table to annotation page
emits update for 'heuristic' to annotation page (but is not handled)
emits update for missing value to annotation page (addMissingValue - on annotation page)

Internal state:

title of the current tab
category table
getColumnsOfCategory from store
filtered data table
relevant columns
unique values
unique values to subject map

Two things:

If the annot-tab doesn't have any user actions of its own (I agree that it doesn't), then it should probably not have to handle any data flow either (child components can be in charge of that themselves)
I think with the refactor we're moving a ton of responsibility from the pages to the components. This annot-tab thing now feels a lot like it could just be a dynamic layout rather than a component.

Agreed on 1 and 2! This feels like some good progress regarding our Vue/Nuxt development experience.

This thing needs to know what type of category it is handling (categorical or continuous)

Closed by https://github.com/neurobagel/annotation_tool/pull/242


Cannot read property 'ImageId' of undefined [CLI/Linux]

I believe this is because the Parsec image doesn't exist in sa-east-1 yet. Is there anyway I can copy it over from another region?

Probably because sa-east-1a does not have GPU optimized machines at this moment.

Hey Guys, I am starting to have this issue on the macOS Gui version. It was working about 2 months ago but it stopped working now. Region: ap-southeast-2 (Sydney). Any solutions?

The parsec image is only used on the first run. Cloudrig then create a new one on each shutdown.
In which context do you have this error ?
Note: this repo is now deprecated and you should post your issues on cloudrig/cloudrig

Hey, whenever I press start it comes up with the error about 5 sec later. It was definitely working about a month ago. Thanks


Adding candidate_viewer.py

This is a class which contains methods for viewing the extracted candidates in snorkel. In current version, there're two methods:

print_candidates for printing extracted candidates (keyword-based printing can also be done using this method)
get_doc_text for getting information of a doc including its text string and corresponding sentences along with their starting indexes

Thanks for the contribution, @phosseini!


doc: modify markdown format in distribution.md

Modify Markdown List format to generate appropriate preview in GitHub.
Before

After

Thanks!

LGTM! Could you please sign your commit?

@stevvooe Looks like you could use https://github.com/jfrazelle/gh-patch-parser :)

@stevvooe  Just signed && force pushed :octocat:
Please check again.

@dtan4 Thank you for the contribution!


Add a test to confirm that supercharged rewards are properly removed

Lik here to MIP repo: https://github.com/MinaProtocol/MIPs/blob/main/MIPS/mip-remove-supercharged-rewards.md

Looks like standard reward is 720 mina for block-producer.

Looking at the CI, there is a step that executes delegation-integration-tests which might be a good place to see if a test could go here.

The file is src/app/test_executive/delegation_test.ml

A new integration test will have to be written. In it two networks will have to be created - one with a block-producer that has no locked tokens and one that does. Then the two rewards will be compared. I'm not sure any other integration test has two networks, so it remains to see if this is feasible.

Need to do three things with test:

Wait for block to be produced (I can already see the code how to do this)
Check balance on block
Compare increase to amount in config file


Unhandled exception on HTTP::Handler

I'm not sure whether it's a problem within crystal.
require "grip"
class Application < Grip::Application
  def initialize(@port : Int32)
    super(environment: "development", serve_static: true)
  end

  getter port : Int32
  getter pubilc_dir : String = "./public"
  getter static_routes : Hash(String, String) = {
    "/foo/bar" => "./public",
    "/test"    => "./public",
  }
  getter fallthrough : Bool = true
  getter directory_listing : Bool = true
end

app = Application.new(80)
app.run

I made some changes to grip that I think was harmless. When I try to access static resources by routes, it works as expected but with an unhandled exception.
ERROR - http.server: Unhandled exception on HTTP::Handler
Closed stream (IO::Error)
  from A:\Scoop\apps\crystal\current\src\http\server\response.cr:204 in 'check_headers'
  from A:\Scoop\apps\crystal\current\src\http\server\response.cr:40 in 'status='
  from A:\Scoop\apps\crystal\current\src\http\server\response.cr:89 in 'status_code='
  from C:\Users\Sunrise\Documents\CodeRepo\Crystal\test\static-server\lib\grip\src\grip\handlers\exception.cr:16 in 'call'
  from A:\Scoop\apps\crystal\current\src\http\server\request_processor.cr:51 in 'process'
  from A:\Scoop\apps\crystal\current\src\http\server.cr:521 in 'handle_client'
  from A:\Scoop\apps\crystal\current\src\http\server.cr:451 in '->'
  from A:\Scoop\apps\crystal\current\src\fiber.cr:141 in 'run'
  from A:\Scoop\apps\crystal\current\src\fiber.cr:93 in '->'

My fork: https://github.com/yanecc/grip

crystal 1.12.1

If you monkey patch in some pp ex within the exception handler, is the actual exception this Closed stream error?

The exception seems to be caused by the grip framework. It appears to be seting the status code in an error handler at a point where the response connection is already closed (ref. https://github.com/grip-framework/grip/blob/7dbfcde591659e29c21f91e2ed8620ed1748510a/src/grip/handlers/exception.cr#L16C9-L16C112).
I don't think there's anything actionable here. Please report this issue to grip.
If you need more help with debugging (there should be an original error which is overshadowed by a bug in the error handler) please consult https://forum.crystal-lang.org/


Make Python code of CuPy faster

While CuPy enables us to great flexibility and usability of using CUDA, it also adds great overheads due to the slow index manipulations and code generations written in pure Python. We must make it faster to improve the try-and-error cycle on small-scale networks.
There might be three options:

Make the pure Python code faster by finding bottlenecks and improve them.
Port some critical codes to Cython.
Reimplement the ndarray by C and add a Python C extension for it.

I have already tried the approach 1 a bit, and realized it is very hard and seems to be hopeless. Approach 3 is maybe ideal, since the resulting array implementation might be used by other languages as well, though this task is too heavy to complete shortly. Approach 2 is an intermediate solution and seems to be better as the first step of improving CuPy performance.
If we take the approach 2, we have to think about the installation process. Using Cython means we introduce C extensions for the CUDA-related code, which cannot be compiled (linked) in environments without CUDA. In order to support such environments, we have to do either of the following two:
A. Detect whether the CUDA is installed or not in setup.py, and if not, avoid the installation of CuPy.
B. Make CuPy an isolated package (like PyCUDA), and let users decide whether to install CuPy or not.
I recommend to choose the approach B, since it makes it clear which modules are installed from what the user did.

maybe take a look at https://github.com/andersbll/cudarray ?

CUDArray seems to be a fine example of Cython for CUDA array implementation. Thank you! (It cannot be directly used for our purpose, though, since it does not seem to support arbitrary striding yet.)

The type-checking in chainer also creates a big overhead...

@bordingj Yes, I noticed that when I took profiles to tune the CuPy. It seems about 7.35% of running time is consumed by type checking in MNIST example on my environment. We might have to provide "non-debugging mode" to skip such safety codes.

"B. Make CuPy an isolated package (like PyCUDA), and let users decide whether to install CuPy or not."
Is this still planned for the upcoming Cythonization (#499)?
It seems like the right thing to do; CuPy could benefit many scientific computing projects, even those that have nothing to do with machine learning. Conversely, CuPy would benefit from a potentially greatly increased userbase, since users can submit bug reports, bug fixes, speed ups, further numpy compatibility improvements, etc.


Accessing Pretrained Model Weights

Thanks for this phenomenal codebase, OpenSelfSup Team.
One small issue and a minor feature request:
It seems the weights for models like selfsup/byol/r50_bs256_accumulate16_ep300.py, for example, are inaccessible for download (Google Drive restriction, I think). Can these be made available?
Also, I'm wondering if you might be able to provide a minimally working example for the quick building and loading of the weights of a pretrained model with a similar workflow to something like PyTorch Lightning?
Thanks in advance!

Also, a second feature minor feature request:
Would you mind sharing the command for running linear classification on the base supervised resnet50 architecture from the associated weights file (imagenet_r50-21352794.pth)?
Thanks again!

The model now is directly accessible for download.
Thanks for you suggestion. We do not support PyTorch Lightning currently, would appreciate it if you would like to contribute to this.


Assorted UI updates

As discussed on Slack, I have tried to split the commits by content area. Please review the commits for the areas you work on:
Commit 1: Added content for debug pages
Commit 2: Editorial changes
Commit 3: Node list changes
Commit 4: CPU usage changes/known limitation
Commit 5: Statements, statements details page, replication dashboard changes
Things in flight:

[ ] Docs-UI QA

[ ] Screenshots updates

(No major reviewing efforts required for these changes, so don't want to block reviewers on completed doc updates).

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4873e8367b9ac2135e109d8a18fce1b8342256d7/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/356f415579bbbbd8e27d33b455ea8d55bc52ed1f/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/52fe8dfb584868937ef2a446c0ed3272f50ff09a/

v2.1/admin-ui-cluster-overview-page.md, line 38 at r3 (raw file):

The storage capacity used by CockroachDB as a percentage of the total usable capacity on the node.
Fixed.

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

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

@jseldess @couchand @piyush-singh Ready for follow-up reviews!

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

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

LGTM other than that one duplication of "page"

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

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/34b6e1ae04d83c13a7f10bf680796791dd3ef7d1/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0c6f28c9d87104d35248c7ff1fafa8c534aef6d9/


termios: add missing VTIME macros on Darwin

The VMIN macros are already defined, but the VTIME macros are not. The VTIME macro allow you to specify the read timeout before read can return. It complements VMIN.
Edit: Just got informed that the LibC provided by Crystal is for internal purposes and only exposes whatever is needed for the standard library. Closing. 🙂

It's certainly a bit confusing. There are plans to move the libc bindings to a private namespace (#13504) to make this more obvious.


Plan counterfactual interventions for social isolation and loneliness

Would be good to find some government plans on tackling social isolation and loneliness in the ageing population, but not a necessity if nothing perfect exists.
One thing we could do is to find the burden of these risk factors in terms of Life Years and associated outcomes. We can do that by generating an input population where each variable is set to the best possible value (i.e. loneliness avg. set to 1, social isolation index set to 0), and making it impossible for these values to change (either relying on mechanisms inside the FEM to make it impossible to transition to different states, or by switching off the prediction of these variables entirely (i.e. taking them out of the csv file).
We can do this separately and together, and might be really interesting to see how the two risk factors differ in terms of outcomes. This is potentially where we could look at dementia / alzheimers incidence, as I hypothesise that social isolation would have stronger links to this outcome than loneliness.

Possibly also a COVID special? Hypothesis that the number of ageing people who are now widowed/widowers increase due to increased mortality in the 50+ from COVID, so index of social isolation increase by at least 1 point. If we can find some numbers for the amount/proportion of people by demographic group that lost spouse we can create a decent counterfactual to model this.


Error while running "train_pytorch_U2GNN_UnSup.py"

Hi, I am seeing the following error while running "train_pytorch_U2GNN_UnSup.py".
I see, this shows "cannot import name 'LogUniformSampler' from 'log_uniform' (unknown location)". But the file is already present in the mentioned path. I am not sure, why am I getting this error.
Please help me solving this issue.
Traceback (most recent call last):
File "train_pytorch_U2GNN_UnSup.py", line 12, in
from pytorch_U2GNN_UnSup import *
File "/mnt/c/Users/sanagraw/Documents/Graph/Graph-Transformer/U2GNN_pytorch/pytorch_U2GNN_UnSup.py", line 6, in
from sampled_softmax import  *
File "/mnt/c/Users/sanagraw/Documents/Graph/Graph-Transformer/U2GNN_pytorch/sampled_softmax.py", line 7, in
from log_uniform import LogUniformSampler
ImportError: cannot import name 'LogUniformSampler' from 'log_uniform' (unknown location)

You should change to the log_uniform directory to perform make to build SampledSoftmax, and then add the log_uniform directory to your PYTHONPATH.

@daiquocnguyen can you please mention the steps if possible? I am not able to get it fully?

If you are using bash, you use cd to go inside the log_uniform directory, then use make to compile the files. After that, you can use export PYTHONPATH="${PYTHONPATH}:/path_to_log_uniform" to add the log_uniform directory to your PYTHONPATH.

how did you solve this problem? thanks

This really brings about a lot of difficulties for users to apply this model, wish a python version LogUniformSampler can be provided in this repo.


Basic HTTP Backendserver to check merkle airdrop entitlement

Fixes server part of https://github.com/trustlines-network/project/issues/352

And ofc please fix black and tests.

And ofc please fix black and tests.

btw: I would recommend putting essential tasks that should be done in pre-commit hooks.

Codecov Report

Merging #34 into master will decrease coverage by 5.91%.
The diff coverage is 60.97%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage   95.54%   89.62%   -5.92%
==========================================
  Files           5        6       +1
  Lines         202      241      +39
==========================================
+ Hits          193      216      +23
- Misses          9       25      +16

Impacted Files
Coverage Δ

merkle_drop/server.py
50% <50%> (ø)

merkle_drop/cli.py
91.66% <84.61%> (-1.49%)
:arrow_down:

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 5924e42...a03d023. Read the comment docs.


react-native run-android error

Question

This is not the repo for baidu-map, you might wanna open this issue there: https://github.com/lovebing/react-native-baidu-map
Closing


Notification.setLatestEventInfo is deprecated

We have some Notification-related methods for low API numbers that are deprecated ( https://github.com/mixpanel/mixpanel-android/blob/7f310bf93998f5932ebdfc717cbcdcf384677423/src/main/java/com/mixpanel/android/mpmetrics/GCMReceiver.java#L261 ) - we need to decide if we 1) want to drop support for API11 or 2) continue using the deprecated methods or 3) use alternative methods to get the job done.

This popped up for me today while upgrading play services to 8.1.0 from 7.8.0. It's blocking builds with an error. Do you know of any immediate workarounds or how timely a fix would be? I am on mixpanel 4.6.3.

@bricestacey this was fixed on the latest version (v4.7.0)(see https://github.com/mixpanel/mixpanel-android/commit/a32b31e0f5cb6c29113d4f0615c48a7f78e2c6fe). Sorry for getting back to you late!


Is there a version mapping table to describe which version of tikv/pd should use which version of client-go

As the title said~

+1, we have discovered that some client and server are not guaranteed to be compatible, such as the v5.3.x server with the latest client. Ref: https://github.com/tikv/client-go/issues/646
Can we maintain a list of supported servers when releasing versions?

I have been trying to not introducing breaking changes to transactional APIs. So, I expect the latest client to be compatible with any TiKV later than 4.0.
Newer features (e.g. async commit) are enabled through options. If you don't enable them, the client should work with low version TiKV.
I don't think the proto change like in #646 is reasonable. But I'm not so familar with the raw kv, sorry.

I have been trying to not introducing breaking changes to transactional APIs. So, I expect the latest client to be compatible with any TiKV later than 4.0.
Newer features (e.g. async commit) are enabled through options. If you don't enable them, the client should work with low version TiKV.
I don't think the proto change like in #646 is reasonable. But I'm not so familar with the raw kv, sorry.

ok i known, thanks for you reply :)


Creating a custom Dockerfile

I think it's better to use custom Dockerfile. But, the instructions are not clear and getting permission error in the execution of docker-entrypoint.sh. Also, there are no node_modules and strapi files in the container. Not sure if this is the right way to implement this:
I am using external mysql client for database.
Error:

starting container process caused "exec: "/usr/local/bin/docker-entrypoint.sh": permission denied": unknown

Dockerfile
FROM strapi/base:10

RUN yarn global add<EMAIL_ADDRESS>WORKDIR /srv/app
EXPOSE 1337

ENV DATABASE_NAME=strapi
ENV DATABASE_CLIENT=mysql
ENV DATABASE_HOST=$DATABASE_HOST
ENV DATABASE_PORT=$DATABASE_PORT
ENV DATABASE_USERNAME=$DATABASE_USERNAME
ENV DATABASE_PASSWORD=$DATABASE_PASSWORD

COPY docker-entrypoint.sh /usr/local/bin/
ENTRYPOINT ["docker-entrypoint.sh"]
CMD ["strapi", "develop"]

If I run below Dockerfile. I am getting yarn build not found error.
Dockerfile - From custom example file
FROM strapi/base
WORKDIR /srv/app
COPY ./package.json ./
COPY ./yarn.lock ./
RUN yarn install
COPY . .
RUN yarn build
EXPOSE 1337
ENV NODE_ENV production
CMD ["yarn", "start"]

What is the best way to create my own strapi image?

Hey,
Have you tried making the COPY to  /srv/app instead of ./ ?

I had a similar issue and swapped "yarn" for "npm" see below:
`FROM strapi/base
WORKDIR /myApp
COPY ./package.json ./
COPY ./package-lock.json ./
RUN yarn install
RUN npm install
COPY . /myApp
RUN npm build
EXPOSE 1337
ENV NODE_ENV production
CMD ["npm", "start"]

@rrubio I tried the way you mentioned and getting this error:
><EMAIL_ADDRESS>start /srv/app
> strapi start

[2019-12-27T23:05:20.297Z] debug ⛔ Server wasn't able to start properly.
[2019-12-27T23:05:20.298Z] error Error: Missing config folder. Please create one in your app root directory
at module.exports (/srv/app/node_modules/strapi/lib/core/load-config.js:9:11)
at Strapi.load (/srv/app/node_modules/strapi/lib/Strapi.js:317:7)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async Strapi.start (/srv/app/node_modules/strapi/lib/Strapi.js:196:7)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `strapi start`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2019-12-27T23_05_20_309Z-debug.log

Hello @rnkhouse,
You are running the image but do you already have a project in your host folder ?

Hi @alexandrebodin
+1 for OP's issue, I have the same problem.
I have:

This repo in my host folder (I guess that is my "project"?)
Same dockerfile as OP

and got the same issue.
So I did this:

Changed to use npm like @rrubio
Ran npm install to create the package-lock.json

Then I got npm ERR! missing script: start (of course, because I do not have the start script that @rrubio used -- do you mind sharing?)
...By the way in my docker-compose.yml I am using:
volumes:
      - ./app:/srv/app

and WORKDIR /app in my Dockerfile.


install: Add system:nodes Group to cilium ClusterRoleBinding

Create a cilium cluster via cilium.yaml generated by helm template,
agent fails because User system:nodes:NODE_NAME cannot get desired k8s
resource, this fixes the issue.
Signed-off-by: Jaff Cheng<EMAIL_ADDRESS>

This change is

Coverage increased (+0.03%) to 45.705% when pulling e45b881d37c4fb30761f96bbd3d296800b62306e on jaffcheng:add-system-nodes-group-to-crb into eb7837f1d63815906518469648c0ee95e144b4a8 on cilium:master.

Hi @aanm , I have a pre-built v1.13.2 kubernetes all in one cluster, following cilium 1.6.2 installation guide, I downloaded and decompressed https://github.com/cilium/cilium/archive/1.6.2.tar.gz .
After executing
helm template cilium \
  --namespace kube-system \
  --set global.etcd.enabled=true \
  --set global.etcd.endpoints[0]=http://xxx:2379 \
  --set global.etcd.endpoints[1]=http://xxx:2379 \
  > cilium.yaml

kubectl create -f cilium.yaml

cilium-agent started and tried to contanct apiserver, but failed with error:
level=info msg="Establishing connection to apiserver" host="https://<IP_ADDRESS>:6443" subsys=k8s
level=error msg="Unable to contact k8s api-server" error="namespaces \"kube-system\" is forbidden: User \"system:node:dev\" cannot get resource \"namespaces\" in API group \"\" in the namespace \"kube-system\"" ipAddr="https://<IP_ADDRESS>:6443" subsys=k8s
level=fatal msg="Unable to initialize Kubernetes subsystem" error="unable to create k8s client: unable to create k8s client: namespaces \"kube-system\" is forbidden: User \"system:node:dev\" cannot get resource \"namespaces\" in API group \"\" in the namespace \"kube-system\"" subsys=daemon

After applying the changes, the errors are gone.
It seems these lines exist in cilium v1.5 example yaml file, not sure why they are removed in v1.6:
https://github.com/cilium/cilium/blob/v1.5/examples/kubernetes/1.15/cilium-external-etcd.yaml#L621

Thanks @aanm , using cilium service account should be the right approach. Maybe we could add the configmap to examples or mention that in the docs for users running without kube-proxy?

Closing this.


Facebook channel getting flooded with "Failed to validate incoming Facebook message"

Bot Builder v4.0.1
.NET Core 2.0
I've setup the echo bot and a Facebook channel, published to Azure and sending/receiving messages and everything else works fine.
However, I get flooded with these FB channel warnings around every 2 minutes:
"Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly."
I've triple checked the facebook app id, secret, page id, page token.
I'm using a Facebook app in test mode if that matters. Any ideas?
Custom event log details:
Application version: Thu 07/12/2018 21:33:44.42
UTC (factoryvm-az373) [Build 36484] [Repo Intercom] [Branch prod] [Commit c1c58d0f0027fbe311b8d18906c95f78d1cfbf2e]
Bot ID: Bot
Channel ID: facebook
Client IP address: <IP_ADDRESS>
Correlation ID: |/6wjT/qVSf8=.ddc61fad_
Country or region: Ireland
Device type: PC
Event count: 1,00
Event name: Facebook message failed
Event time: 2018-07-13 11:21:58
EventName: Facebook message failed
Message: Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.
Operation ID: |/6wjT/qVSf8=.ddc61fad_
Operation name: POST /api/v1/bots/Bot
Operation parent ID: |/6wjT/qVSf8=.ddc61fad_
Role instance: RD0003FF74BC14
Role name: bc-facebook-weurope
SDK version: dotnet:2.6.4-28384

@Sire I cannot repro this. Can you clarify where you see these events logged to?

Closing due to lack of activity. Please reactivate if you continue to see this or will additional information on where you were seeing these events logged to.

@vishwacsena Sorry was on vacation.
Under my Azure "Web App Bot", page "Channels", under the "Health" column for channel "Facebook Messenger".
After I successfully send and receive a message it says "Warnings (2)" in this column. Clicking to show details displays:
Facebook Messenger
Recent issues
Clear All
Time	Message
7/30/2018, 12:57:33 PM	Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.
7/30/2018, 12:57:24 PM	Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.

Having same problem when using test app.

same problem with ver 3.12

same issue

@stevengum please reopen

@1110770MiguelSilva, and @vovanb, please provide which template you're using if any and the botID and bot handle for your bot.

@jwiley84, can you investigate? Thanks!

@jwiley84 Yes I use those exact page subscriptions (webhooks).
Yes, I use the test version of my app, and it's in development mode. I think I've tried the non-test version as well (that is in live mode), but I can double check.

@Sire,
Just for clarification, you are successfully receiving responses from you bot, and this is only showing on the channels blade of your bot on Azure?
To be the bearer of bad news, I know that messaging_optins was not supported in the v3 SDK, so chances are very high that it isn't currently supported in the v4 SDK. Not that I think that it's the core of the issue, as a simple echobot wouldn't need optins, just that might be a possible impact.

@jwiley84 That is correct.
Ok, should I try to remove messaging_optins and see if the problem goes away?

@Sire
If this is just an echo bot, then you shouldn't be calling 'messaging_optins' in the code at all, so that SHOULDN'T be the issue, but yes, go ahead and turn that one off, and let me know what happens.

@Sire are you still having this issue?

@jwiley84 After removing messaging_optins, I still receive warnings. But fewer, only 2 instead of the normal 5 (I think) per initial message sent.

I'm forced with the same issue. I'm using bot framework 3.16.xx and got the same Problem. I'm 100 percent sure that i took the right secret (According to the documentation (https://docs.microsoft.com/en-us/azure/bot-service/bot-service-channel-connect-facebook?view=azure-bot-service-3.0))
Error-Message:

01/11/2018, 1:01:42 PM
Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.

01/11/2018, 12:56:33 PM
Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.

01/11/2018, 12:55:18 PM
Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.

01/11/2018, 12:54:25 PM
There was an error sending this message to your bot: HTTP status code InternalServerError

01/11/2018, 12:54:03 PM
Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.

Thanks for your support.

@Sire: I know it's been a while, but digging around in Facebook dev show that they've been undergoing changes to their dev app review and publish process you might have caught the tail end of. Just to double check: are you still having this issue, and is it still impacting your bot's ability to function?
@2015PM: please open a separate issue on Botbuilder-v3 and I can work with you on that thread.

Closing for lack of response.

Hello, could you please reopen this flow?
I have the same issue with the message : Failed to validate incoming Facebook message. Make sure your FacebookAppSecret is entered correctly.
My bot works in the test channel but not on Facebook or other channel

still receiving this message on bot framework 4.2 node js.
I can confirmed my app secret is totally correct, and remove and add again FB channel still the same.

Hello,
Any news on it?
I'm getting this error a lot on some bots, but not on others.
Let me know what type of information I can provide for you to investigate it further.
Thank you,

Hi @anagor
This issue is closed, and closed issues are not monitored by the team.  Please open a new issue or ask a question on stack overflow.
Also, something to check is the facebook page subscription fields.  Are they the same on the bots with errors as the bots without errors?


Replace custom code with config

Fixes https://github.com/Azure/autorest.csharp/issues/2802
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Add support for Orange Pi Zero 2W

Orange Pi Zero 2W

Orange Pi's new model Zero 2W using Allwinner H618, almost an alternative to RPi Zero 2W.
Schematic
Can we add support for this?
root@orangepi:~# gpio readall
 +------+-----+----------+--------+---+  ZERO2W  +---+--------+----------+-----+------+
 | GPIO | wPi |   Name   |  Mode  | V | Physical | V |  Mode  | Name     | wPi | GPIO |
 +------+-----+----------+--------+---+----++----+---+--------+----------+-----+------+
 |      |     |     3.3V |        |   |  1 || 2  |   |        | 5V       |     |      |
 |  264 |   0 |    SDA.1 |    OFF | 0 |  3 || 4  |   |        | 5V       |     |      |
 |  263 |   1 |    SCL.1 |    OFF | 0 |  5 || 6  |   |        | GND      |     |      |
 |  269 |   2 |     PWM3 |    OFF | 0 |  7 || 8  | 0 | ALT2   | TXD.0    | 3   | 224  |
 |      |     |      GND |        |   |  9 || 10 | 0 | ALT2   | RXD.0    | 4   | 225  |
 |  226 |   5 |    TXD.5 |    OFF | 0 | 11 || 12 | 0 | OFF    | PI01     | 6   | 257  |
 |  227 |   7 |    RXD.5 |    OFF | 0 | 13 || 14 |   |        | GND      |     |      |
 |  261 |   8 |    TXD.2 |    OFF | 0 | 15 || 16 | 0 | OFF    | PWM4     | 9   | 270  |
 |      |     |     3.3V |        |   | 17 || 18 | 0 | OFF    | PH04     | 10  | 228  |
 |  231 |  11 |   MOSI.1 |   ALT4 | 0 | 19 || 20 |   |        | GND      |     |      |
 |  232 |  12 |   MISO.1 |   ALT4 | 0 | 21 || 22 | 0 | OFF    | RXD.2    | 13  | 262  |
 |  230 |  14 |   SCLK.1 |   ALT4 | 0 | 23 || 24 | 0 | ALT4   | CE.0     | 15  | 229  |
 |      |     |      GND |        |   | 25 || 26 | 0 | ALT4   | CE.1     | 16  | 233  |
 |  266 |  17 |    SDA.2 |    OFF | 0 | 27 || 28 | 0 | OFF    | SCL.2    | 18  | 265  |
 |  256 |  19 |     PI00 |    OFF | 0 | 29 || 30 |   |        | GND      |     |      |
 |  271 |  20 |     PI15 |    OFF | 0 | 31 || 32 | 0 | OFF    | PWM1     | 21  | 267  |
 |  268 |  22 |     PI12 |    OFF | 0 | 33 || 34 |   |        | GND      |     |      |
 |  258 |  23 |     PI02 |    OFF | 0 | 35 || 36 | 0 | OFF    | PC12     | 24  | 76   |
 |  272 |  25 |     PI16 |    OFF | 0 | 37 || 38 | 0 | OFF    | PI04     | 26  | 260  |
 |      |     |      GND |        |   | 39 || 40 | 0 | OFF    | PI03     | 27  | 259  |
 +------+-----+----------+--------+---+----++----+---+--------+----------+-----+------+
 | GPIO | wPi |   Name   |  Mode  | V | Physical | V |  Mode  | Name     | wPi | GPIO |
 +------+-----+----------+--------+---+  ZERO2W  +---+--------+----------+-----+------+

Modules like gpiozero use BCM pins, so I added RPi BCM to OPi GPIO mapping.
I just bought a OPi Zero 2W and the solution worked well with a Waveshare e-Paper made for RPi.


[BEAM-5315] Python 3 port bigquery and pubsub related io modules

This is is part of a series of PRs with goal to make Apache Beam PY3 compatible. The proposal with the outlined approach has been documented here: https://s.apache.org/beam-python-3.
This PR ports the  bigquery and pubsub related io modules.


Error handling within on_record while using stream api

Describe the bug
Version: 4.8.6
I previously opened #275 in regards to errors being thrown within the on_record callback were not effected by the skip_lines_with_error option, although the issue was closed and resolved I think that the issue still persists (specifically under streaming scenarios)
To Reproduce
Playground: https://repl.it/@RobertPitt/CSV-Paese-1
const parse = require('csv-parse') // 4.8.6

const on_record = ([a, b], { lines }) => {
  if(lines === 2)
    throw new TypeError('Something went wrong')
  return [a, b]
}

const parser = parse("a,b\nc,d\ne,f", {
  on_record,
  skip_lines_with_error: true,
})

// Under this configuration, no events actually emit and global exception is throw.
parser.on('skip', () => console.log('Skipped'))
parser.on('error', () => console.log('Error'))
parser.on('data', (d) => console.log('Data', d))
parser.on('end', () => console.log('End'))
parser.end()

Additional context
See issue: #275

This was due to a bug not related to on_record. Fixed now. Notes about your code: you shall not call end since you are passing a string and not using the writable stream api; on('data',..) shall be on('readable',..)


Pre-trained model inference

Would be great if you could provide a sample of using the pre-trained network on a new set of images (MP4 video or a folder of images) for people who would like to test it on new data

Thanks. Sounds like a good idea -- I can do that later.
For now the easiest way is to clone DAVIS file structure and use eval_davis.py on it.

https://github.com/hkchengrex/STCN/blob/main/eval_generic.py is now provided for generic data.
See our extension to MiVOS for a interactive GUI demo.


kernel: Enable new LTS 5.4.x on ppc64le arch

Linux has embraced another LTS kernel version v5.4.x.
Update the kernel config for Power as well.
Fixes: #936
Signed-off-by: Nitesh Konkar<EMAIL_ADDRESS>

/test

/AzurePipelines run

Thanks @nitkon


Stream events, including past ones

Is your feature request related to a problem? Please describe.
Currently there are two ways to query events:

with a filter for a given range:contract.events().from_block(block_number).query().await. This allows to query events that have been emitted in the past.
streaming new events: contract.events().stream().await. This will only yield events that are emitted after the stream has been created.

Describe the solution you'd like
Ideally, there would be a way to get a stream of events from a given point in the past. I.e. a function that combines both above functions.
Additional context
See https://github.com/gakonst/ethers-rs/issues/986

I see, so contract.events().from_block(block_number).query().await should turn into contract.events().stream().await once the head is reached?
So I guess we should make this the default behaviour if no end_block was set

@meetmangukiya wanna go for this?

Still the case for abigen! Contract types?
    let pool_contract = LendingPool::new(lending_pool, client.clone());
    let last_block = client.get_block(BlockNumber::Latest).await?.unwrap().number.unwrap();
    let events = pool_contract.events();
    let filtered = events.from_block(last_block - 10000);
    let mut stream = filtered.stream().await?;
    while let Some(log) = stream.next().await {
        println!("{:?}", &log);
    }

Only get past events if I use
    let last_block = client.get_block(BlockNumber::Latest).await?.unwrap().number.unwrap();
    let events = pool_contract.events();
    let result = events.from_block(last_block - 1000).query().await?;
    println!("{:?}", &result);

Perhaps I'm screwing it up?

I believe we don't do that in abigen. Let's track in a separate issue. Or feel free to add!

Its not quite clear to me what is supported for this and what is not.

Are only web-sockets supported? Or does this work with the filter polling on http as well?
Abigen contracts won't work? So one needs to use the provider functions and lose the type safety?
From the PR code I think you have a race condition. If there is a reorg or another new block between your last query and your subscription, then you'll miss it. Unless this is handled elsewhere?

@Mirko-von-Leipzig did anything ever come of this? Attempting to write an indexer using ethers-rs events.

@Mirko-von-Leipzig did anything ever come of this? Attempting to write an indexer using ethers-rs events.

Not afaik - we ended up doing this using normal HTTP RPC methods with overlapping block ranges to guard against reorgs.


What is correct config for expose-loader in webpack2?

I originally craeted issue #10
after playing around it looks like the way we configure expose-loader in webpack config is not having desired effect.
the error i get is now

Uncaught Error: Cannot find module 'numbro'
at newRequire (handsontable.js:53)
at handsontable.js:60
at Object.23.cellTypes (handsontable.js:4479)
at newRequire (handsontable.js:58)
at outer (handsontable.js:66)
at handsontable.js:71
at webpackJsonp../node_modules/handsontable/dist/handsontable.js.previousRequire (handsontable.js:27)
at Object. (handsontable.js:27)
at Object../node_modules/handsontable/dist/handsontable.js (stupidApp.1793dcd….js:25771)
at webpack_require (bootstrap 1793dcd…:691)

I've looked at the docs and also at issue #3
relevant part of webpack config looks like

 {
    test: require.resolve('handsontable'),
    loader: 'expose-loader?Handsontable'
  },
  {
    test: require.resolve('numbro'),
    loader: 'expose-loader?numbro',
    //options: {'numbro': null}
  },
  {
    test: require.resolve('moment'),
    loader: 'expose-loader?moment'
  },
  {
    test: require.resolve('pikaday'),
    loader: 'expose-loader?Pikaday'
  },
  {
    test: require.resolve('zeroclipboard'),
    loader: 'expose-loader?ZeroClipboard'
  },

inside the relevant react component I import as outlined by the docs:
import moment from 'moment';
import numbro from 'numbro';
import pikaday from 'pikaday';
import Zeroclipboard from 'zeroclipboard';
import Handsontable from 'handsontable';
import HotTable from 'react-handsontable';
import 'handsontable/dist/handsontable.full.css';

but if i do window.numbro I can see that it is undefined.
Why isn't the expose loader doing it's job?

See: https://github.com/webpack-contrib/expose-loader#usage
The following config should do it:

{
  module: {
    rules: [
      {
        test: require.resolve('numbro'),
        use: [{
          loader: 'expose-loader',
          options: 'numbro'
        }]
      },
      {
        test: require.resolve('moment'),
        use: [{
          loader: 'expose-loader',
          options: 'moment'
        }]
      },
      {
        test: require.resolve('pikaday'),
        use: [{
          loader: 'expose-loader',
          options: 'Pikaday'
        }]
      },
      {
        test: require.resolve('zeroclipboard'),
        use: [{
          loader: 'expose-loader',
          options: 'ZeroClipboard'
        }]
      },
    ]
  }
}

thx.
thought i was crazy for not noticing those webpack2 docs before but turns out they were just added.
https://github.com/webpack-contrib/expose-loader/commit/e6125cf526279558f0e7d605545919a27910308c
will try and report back.

unfortunately that updated config still does not work.
module.rules:
{
        test: require.resolve('numbro'),
        use: [{
          loader: 'expose-loader',
          options: 'numbro',
        }],
      },
      {
        test: require.resolve('moment'),
        use: [{
          loader: 'expose-loader',
          options: 'moment',
        }],
      },
      {
        test: require.resolve('pikaday'),
        use: [{
          loader: 'expose-loader',
          options: 'Pikaday',
        }],
      },
      {
        test: require.resolve('zeroclipboard'),
        use: [{
          loader: 'expose-loader',
          options: 'ZeroClipboard',
        }],
      },

error:

Uncaught Error: Cannot find module 'numbro'
at newRequire (handsontable.js:53)
at eval (handsontable.js:60)
at Object.23.cellTypes (handsontable.js:4479)
at newRequire (handsontable.js:58)
at outer (handsontable.js:66)
at eval (handsontable.js:71)
at previousRequire (handsontable.js:27)
at Object.eval (handsontable.js:27)
at eval (handsontable.js:25474)
at Object../node_modules/handsontable/dist/handsontable.js (....

The suggestion by @flatanimals worked for me on webpack 2.6.1 with the vue-handsontable-official v0.2.5 component and handsontable 0.32.0. I know it's not react, but it's webpack 2 so I figure it's the same issue since I had the same error previously. I'll submit an update to the wiki docs for vue-handsontable-official. Thanks!

Hello. I just wanted to chime in and say the solution above worked for me. To any other developers who are trying to install react-handsontable and have been seeing these errors:
cannot find module 'numbro'
cannot find module 'pikaday'
Adding expose-loader to my webpack.config.js file made it so HandsOnTable now appears.

Thank you for sharing the hits. We will be updating the wrapper later this year so thank you all for feedback.


Remove some unused methods in models.

These aren't being used?

not in the API server.
They are pretty view-y methods, though, so we'll probably find new homes for something like them as we go.


[BUG] System.ArgumentNullException when getting SlotConfigNamesResource

Library name and version
Azure.ResourceManager.AppService 1.0.2
Describe the bug
Using Azure SDK for .Net and Azure.ResourceManager.AppService packge, I used webSiteResource.GetSlotConfigNamesResource.GetAsync() to get slot config names of a web app and the folowing exception was thrown:
System.ArgumentNullException: Value cannot be null. (Parameter 'resourceId')
Stack Trace:
Argument.AssertNotNullOrEmpty(String value, String name)
ResourceIdentifier.ctor(String resourceId)
SlotConfigNamesResourceData.DeserializeSlotConfigNamesResourceData(JsonElement element)
WebAppsRestOperations.ListSlotConfigurationNamesAsync(String subscriptionId, String resourceGroupName, String name, CancellationToken cancellationToken)
SlotConfigNamesResource.GetAsync(CancellationToken cancellationToken)
Expected behavior
Get SlotConfigNamesResource along with its data
Actual behavior
Exception is thrown because the resource id is null so the response can't be deserialized.
Reproduction Steps
Get a WebSiteResource with GetWebSiteResource(_appId).GetAsync()
Get the SlotConfigNamesResource from the WebSiteResource with GetSlotConfigNamesResource().GetAsync()
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Are there any updates? FYI, I tried the REST API to list slot configuration names and the id in the response is null indeed.

Hi @wchigit , Thank you for using Azure SDK for .NET.
As ARM design requested, the id of a resource should NOT be a null value.
Unfortunately, this issue is related to the service side and not the SDK side, so we are not able to help you with it. Therefore, we are closing this issue for now. If you need more assistance, please open an Azure support request. We apologize for any inconvenience this may cause you.
We value your feedback and we want to make sure that your problem is solved. If you think that we have misunderstood your issue or closed it incorrectly, please feel free to comment on this thread and reopen the issue. We will be happy to assist you further. Thank you for your understanding and cooperation.


Gen httpdconfig rework

General tidyup; drop unused code.
Create committer access macro references; the file is not yet included by the httpd config

Please can this be applied or rejected?

ping?

Enable

Root

sorry for the delay. this looks good to me.


Use loose mode for modules

Support IE8
Changing to exports.__esModule = true manually in the imported code makes it work. Without it, Object.defineProperty is used.
Object.defineProperty(exports, '__esModule', {
  value: true
});

This is a no-op in core-js for IE8, so the import-transform in babel doesn't work.
And yes, I know IE8 isn't supported by MS anymore, but this is a small change.
Workaround for now is to use const { createSelector } = require('reselect');, but I don't want that :laughing:

Current coverage is 100.00%

Merging #72 into master will not affect coverage as of 6dc0443

@@            master     #72   diff @@
======================================
  Files            1       1
  Stmts           41      41
  Branches         0       0
  Methods          0       0
======================================
  Hit             41      41
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 6dc0443

Powered by Codecov. Updated on successful CI builds.

Cool, I don't see any downsides with supporting ie8 while its feasible. Thanks!

Great, thanks! We're merging a PR tomorrow using reselect. Would it be possible to publish this before that?

I'm on it!

v2.0.2 published to NPM

Awesome!
On Thu, 14 Jan 2016, 22:17 Lee Bannard<EMAIL_ADDRESS>wrote:

v2.0.2 published to NPM
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/reselect/pull/72#issuecomment-171782411.

--
--
Simen Bekkhus


Support for LoadBalancerSourceRanges

What would you like to be added:
It would be nice if the Service API spec supported some version of the loadBalancerSourceRanges field from the existing services spec:
    // If specified and supported by the platform, this will restrict traffic through the cloud-provider
    // load-balancer will be restricted to the specified client IPs. This field will be ignored if the
    // cloud-provider does not support the feature."
    // More info: https://kubernetes.io/docs/tasks/access-application-cluster/configure-cloud-provider-firewall/
    // +optional
    LoadBalancerSourceRanges []string `json:"loadBalancerSourceRanges,omitempty" protobuf:"bytes,9,opt,name=loadBalancerSourceRanges"`

Why is this needed:
Because the existing services resource supports it, and it makes sense to implement some type of ACL system in the Service APIs. Also, we have an interest in seeing this adopted so it is one less thing we need to grab from a services resource instead of reading only the gateways resource.
Thanks!

/remove-lifecycle rotten

/remove-lifecycle stale
/lifecycle frozen

While grooming we saw that this one was open for a long period of time without anyone with a strong use case to champion it. We're going to close this as we don't expect anyone's ready to drive it forward, but if you still want this feature and have a strong use case we will be happy to reconsider this and re-open.


Is Destination class in the CaptureDescription class of the event hub support EventHubArchive.AzureDataLake ?

In the Azure Portal, the Capture settings for an event hub can use Data Lake Storage. I wonder, does the python sdk just support Storage Account now?
https://github.com/Azure/azure-sdk-for-python/blob/2f3e214f5c9344d3e0842b1d7435ccd006ceda0b/azure-mgmt-eventhub/azure/mgmt/eventhub/v2017_04_01/models/destination.py#L42-L43

@v-Ajnava would you know?

@MyronFanQiu , right now it doesn't support  EventHubArchive.AzureDataLake properties.

@v-Ajnava  @lmazuel  I see. Thanks very much. By the way, is there any schedule or plan to support it?

@ShubhaVijayasarathy can you please update on this.

@MyronFanQiu - yes there is a debt and we plan to support this by EOY. By then we might also add the ADLS Gen 2 support as well. Hope this helps.

@ShubhaVijayasarathy @v-Ajnava Thanks! Wish everything goes fine!


Refactor chemicalc.s2n.Sig2NoiseVLT

Refactor VLT ETC query to match the WMKO query with a separate class for each instrument.

Completed in #21. Documentation on the readthedocs page is still required.


Wpc requires md5() and md5_file() functions, but does not test if they are available

Fixed in 926e266


Remove pytorch_lightning.core.memory.get_gpu_memory_map

To resolve issue #12521
What does this PR do?

Fixes #<issue_number>
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
:)
Make sure you had fun coding 🙃

I think for this we need to remove LoggerConnector.gpu_metrics, memory.get_memory_profile too since they are just sequential calls. Probably it can be done in separate PRs starting with gpu_metrics, then get_memory_profile, and finally get_gpu_memory_map.

@rohitgr7 Ok, then let me take get_memory_profile as well from issue #12521.
But where can I find gpu_metrics, because there is no item in #12521

@puhuk added at the end :)


Created volume can't mounted properly when pod running OpenEBS cStor

I use ...
Ubuntu 22.04
Microk8s 1.26.5
OpenEBS 3.7 using cStor Volume
Expected Behavior
PV can be mounted to the Pod
Current Behavior
The container is running, but when i execute df -h inside the container. There is no volume attached
Environment
iSCSI initiator is installed and running on each node

Node

PVC

Pod

YAML Configuration
I use 3 node and each node have 2 block devices
cStor Pool Cluster
apiVersion: cstor.openebs.io/v1
kind: CStorPoolCluster
metadata:
  name: disk-pool-test
  namespace: openebs
spec:
  pools:
    - nodeSelector:
        kubernetes.io/hostname: "kube-1"
      dataRaidGroups:
        - blockDevices:
            - blockDeviceName: "blockdevice-806e429e2cb53e5c9ab3df7fae83b29a"
            - blockDeviceName: "blockdevice-1b3e495f8e92c287270209f271c20488"
      poolConfig:
        dataRaidGroupType: "stripe"

    - nodeSelector:
        kubernetes.io/hostname: "kube-2"
      dataRaidGroups:
        - blockDevices:
            - blockDeviceName: "blockdevice-6b6c67ee3176cd05085ed757814120ac"
            - blockDeviceName: "blockdevice-3b92a39e20c3f177d4448498ea3a1cf2"
      poolConfig:
        dataRaidGroupType: "stripe"

    - nodeSelector:
        kubernetes.io/hostname: "kube-3"
      dataRaidGroups:
        - blockDevices:
            - blockDeviceName: "blockdevice-a16ca6891c4b77e099140bad5d06d918"
            - blockDeviceName: "blockdevice-dde79a1b093b495771354577b6043ee5"
      poolConfig:
        dataRaidGroupType: "stripe"

Storage Class
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: cstor-1
provisioner: cstor.csi.openebs.io
allowVolumeExpansion: true
parameters:
  cas-type: cstor
  cstorPoolCluster: disk-pool-test
  replicaCount: "1"

---

apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: cstor-2
provisioner: cstor.csi.openebs.io
allowVolumeExpansion: true
parameters:
  cas-type: cstor
  cstorPoolCluster: disk-pool-test
  replicaCount: "2"

---

apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: cstor-3
provisioner: cstor.csi.openebs.io
allowVolumeExpansion: true
parameters:
  cas-type: cstor
  cstorPoolCluster: disk-pool-test
  replicaCount: "3"

Persistent Volume Claim
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: test-pvc
spec:
  storageClassName: cstor-1
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 1Gi

and Pod
apiVersion: v1
kind: Pod
metadata:
  name: busybox
spec:
  volumes:
  - name: pvc
    persistentVolumeClaim:
      claimName: test-pvc
  containers:
  - command:
      - sh
      - -c
      - 'date >> /mnt/openebs-csi/date.txt; hostname >> /mnt/openebs-csi/hostname.txt; sync; sleep 5; sync; tail -f /dev/null;'
    image: busybox
    imagePullPolicy: Always
    name: busybox
    volumeMounts:
      - mountPath: /my-data
        name: pvc

All resources is success to deploy/apply, and what confuses me is the volume that I created is unused by the pod.

And there is no error log in command kubectl get events, kubectl describe pod/pvc

nevermind, the problem is solved by myself.
i use this guide to solve this

Thank you


Add summary to ALL functions, classes and interfaces

Description for what the function does, what parameters are required and what it returns.

This issue was defined too broad and could not be worked off properly.
It will be replaced by more granular Issues.


Question: how does Flynn secure my Git repository in my server?

Hi,
I'm coming from Heroku background and I really like Flynn so far. I am aware that everytime I run flynn create command, it will auto-create Git remote for me which I can deploy my app easily later using git push flynn master command just like Heroku.
In Heroku land, I ran heroku login to authenticate myself to the Git repo inside Heroku. But when I use Flynn, there is no equivalent command for that.
So, I am curious here. How does Flynn secure the Git repo inside my server?
Can someone copy my Flynn's Git remote, add it into their dev environment as flynn remote, push and overwrite my app?
Thanks.

@zulhfreelancer when you run the initial flynn cluster add command on your machine, some credentials are written to a .flynnrc file in your home directory which give you access to the Flynn cluster.
Git is also configured to use the Flynn CLI for credentials when pushing to the Flynn cluster via the credential helper (you can see this in your ~/.gitconfig file).
So, if someone copies your git remote, they also need to have Git configured to use the Flynn CLI, and the Flynn CLI also needs to be configured with the credentials of your cluster, so as long as you don't share ~/.flynnrc then they will not be able to access the git remote.

This is really cool and helpful. Thank you for answering my question! 🙂


Getting the Url of the loaded gpx

Hi,
It is certainly an easy thing but I unfortunately do not find.
I am try to get during the loading of a gpx the original url of it in order to let the user click on the gpx and download it if needed
With the code below I am actually trying to get in g.on('loaded', the original url from g ( loopinfo.what[j][i] ) and paste it in ???????????????
Could you help me ?
Thanks

for (var j = 0; j < loopinfo.what.length; j += 1) {
  for (var i = 0; i < loopinfo.what[j].length; i += 1) {

    var g = new L.GPX(loopinfo.what[j][i])

    g.on('loaded', function(e) {
      var gpx = e.target,
      namegpx = gpx.get_name(),
          distM = gpx.get_distance(),
          distKm = distM / 1000,
          distKmRnd = distKm.toFixed(1),
          eleGain = gpx.get_elevation_gain().toFixed(0),
          eleLoss = gpx.get_elevation_loss().toFixed(0),
          cen = gpx.getBounds().getCenter();
    var info = "Name: " + namegpx + "
" +
        "Distance: " + distKmRnd + " km
" +
        "Elevation Gain: " + eleGain + " m
" +
    "Elevation Loss: " + eleLoss + " m
" +
        "Link
" +
        "Share location
";
        // register popup on click
        gpx.getLayers()[0].bindPopup(info);
    });

g.addTo(loopinfo.layer[j]);

The library doesn't have a way to get you back the URL of the GPX that was loaded, because it doesn't just take URLs as input (you can also directly pass GPX XML to the L.GPX(...) constructor). But since you already have that URL in your code, you don't need the library to give it back to you.
Your problem here is a classic Javascript callback scope capture problem. You can read more about this here: https://www.pluralsight.com/guides/javascript-callbacks-variable-scope-problem
You can make it work with this:
for (var j = 0; j < loopinfo.what.length; j += 1) {
  for (var i = 0; i < loopinfo.what[j].length; i += 1) {
    var url = loopinfo.what[j][i];
    new L.GPX(url).on('loaded', (function() {
      var _url = url;
      return function(e) {
        var gpx = e.target;
        var link = 'Link';
        ...
      }
    })() );
  }
}

The library doesn't have a way to get you back the URL of the GPX that was loaded, because it doesn't just take URLs as input (you can also directly pass GPX XML to the L.GPX(...) constructor). But since you already have that URL in your code, you don't need the library to give it back to you.
Your problem here is a classic Javascript callback scope capture problem. You can read more about this here: https://www.pluralsight.com/guides/javascript-callbacks-variable-scope-problem
You can make it work with this:
for (var j = 0; j < loopinfo.what.length; j += 1) {
  for (var i = 0; i < loopinfo.what[j].length; i += 1) {
    var url = loopinfo.what[j][i];
    new L.GPX(url).on('loaded', (function() {
      var _url = url;
      return function(e) {
        var gpx = e.target;
        var link = 'Link';
        ...
      }
    })() );
  }
}

Great thanks I got it work with your help 👍 and learned something important.
Thanks for the references :)

Great thanks I got it work with your help 👍 and learned something important.
Thanks for the references :)


switch to torchvision CelebA

Will need to use SubsetDataset to get the attribute subsets, but that's probably better anyway.

Mostly solved in ba6e558eda324fd177914046b5d671f1527dbbc5 + eb35f270fd1067da6c5f9bf987948f46394c9ec0, but need to fake https://github.com/pytorch/vision/pull/1008 a bit.


use React 16.9

r? @dweedon-stripe
Summary & motivation
This updates the react version we use for demo and test to 16.9
Testing & documentation
Covered by existing tests.

lgtm


Fix 20516 mWeb topbar color

Details
Setting StatusBar for mWeb to keep consistency with Native Apps.
Fixed Issues
$ https://github.com/Expensify/App/issues/20516
PROPOSAL: https://github.com/Expensify/App/issues/20516#issuecomment-1586349155
Tests

Open App in mobile browser
Verify the TopBar of LHN should match with LHN background.
Verify other pages TopBar should match with native app StatusBar.

[x] Verify that no errors appear in the JS console

Offline tests
Same as Tests
QA Steps
Same as Tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/31707153/4e7adffd-e464-453c-86c3-4d23ace83843

Mobile Web - Chrome
https://github.com/Expensify/App/assets/31707153/262edf6c-5a14-4d6a-8755-dc1230d10ab6

Mobile Web - Safari

https://github.com/Expensify/App/assets/31707153/9e839871-0358-4544-b2cc-8a45089a0bbc

Desktop

https://github.com/Expensify/App/assets/31707153/e928b051-80ca-4d2e-bf41-503fedcaf213

iOS

https://github.com/Expensify/App/assets/31707153/3c78d761-6fa9-451b-820f-4e46c7b633db

Android

https://github.com/Expensify/App/assets/31707153/ec355875-6745-456a-addd-9e1f537d5040

@thesahindia Friendly bump so we can merge it within the bonus!

It was a federal holiday yesterday in the US so I was out (it won't count for bonus either), but I am taking a look today.

I noticed this while watching the videos, I think we should make the top look the same color as the IOU:

Ahh interesting, yeah, after digging more on how this works, I think we should stick to the background color since then it might get too complicated for future cases. With that in mind, it looks and works great, I don't think we can do anything about that tiny browser delay.
@thesahindia Can you do a final review and complete the checklist so I can do my final review?

Ah sorry, got confused with the assignments in the PR, my bad for not checking 😄 I will do a final check then before approving.

I keep getting a crash on this branch when running the web version, I can run the app without issues on main, can you check to see if it's only me?

This is the error, I tried even deleting the node modules and reinstalling and still no luck:

Just a small change. I wonder why it requested changes were posted twice lol

🚀 Deployed to staging by https://github.com/pecanoro in version: 1.3.30-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.30-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Multi database test suite

Laravel Version
10.x
PHP Version
8.1.x
Database Driver & Version
No response
Description
I do not understand why in the checks that are done on multiple database system by github, the test suite being tested is only tests/Integration/Database, and for example why is tests/Database not included in those.
This caused (with the help of an error on my part of course) the problems described in https://github.com/laravel/framework/pull/46309 subsequently to its merge.
Steps To Reproduce
Create a PR, only some subset of databases tests are executed on multiple databases

Thanks but this doesn't seems like a bug report. Feel free to send in a PR if you want to change something to the test suite.


Issue no:14 VM can't be Startted

@rajesh-rajagopal
server one seven six

@rajesh-rajagopal
kk change it and how many memory tottaly available
total memory 24 ?

Available memory is only 24 Gb.
Others are used by

Rio/OS software,
Ceph Storage and
System usages

May i convert VM memory size to 24 Gb.

@rajesh-rajagopal
kk change the memory

VM is started.. Kindly check it and close the issue.

@rajesh-rajagopal
thank u and
how much memory default required in

Rio/OS software,
Ceph Storage and
System usages

Memory usage depend upon usage of software. If you don't use Ceph storage or Rio/OS most of them memory are available.
Basically, Rio/OS needs 4 GB of memory,
Ceph storage used on usage like 1 TB of storage used for VM,
System caches minimum of 1 Gb of memory.

Once you clarified, close this issue.


New Contact function has broken

You type in the name of the new contact, but the card to edit the details never appears.
The logs say "Strange no evidence that x has anything to do with contacts".  But the network logs say that the PATCH requests to the pod have been 200 Ok.


[TreeView] disable all selection when disableSelection

[x] I have followed (at least) the PR section of the contributing guide.

https://github.com/mui-org/material-ui/issues/20099

@tonyhallett Thanks for the continue work on TreeView!


Why are error codes and details hidden by default?

When I access a non-existing URL, I got a generic error page.

sorry. something went wrong :-(

After a bit of digging, I understood that the error is known within the page, only displayed in a hidden div.
What is the reason for hiding the error?
Is it a security reason?
If so, perhaps we should only apply this policy for internal sever errors (500), and for other types of errors display that to the user?
Under the current behavior, an innocent user might think the website is broken, when in fact they just followed a stale link.

that should be set upped as a task,
any API Call on the code, should be checked to see that the error response are the correct one.


[stable/kiam] Generatic certs in chart

What this PR does / why we need it:
This chart was not self-sufficient, as it needed some pre-generated certificates. This PR includes a self signed certificate generated by Sprig functions.
Which issue this PR fixes

fixes #10420

Special notes for your reviewer:
Signed-off-by: Endre Czirbesz<EMAIL_ADDRESS>Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] title of the PR starts with chart name e.g. [stable/chart]

@prydonius @davidkarlsen @maorfr
Do I need to do anything else to get this PR reviewed and merged?

/assign
/ok-to-test
hey @endrec,
very nicely done!

/lgtm