Cli csv upload

Best reviewed in conjunction with https://github.com/SeitaBV/timely-beliefs/pull/54.
Note that, if the user doesn't set a horizon explicitly, the server time will be used as belief time. Running the CLI command on the same CSV file twice will then work, and ends up storing the same set of values with a different belief horizon.
If the user sets a horizon explicitly, running the CLI command on the same CSV file twice will throw a database error because the data is already there. I tried but was unsuccessful at supporting an allow_overwrite=True option, which requires an "upsert" implementation like I had done before for the API in play mode (there I used session.merge). If you think it deserves another look, I could share my faulty implementation and maybe you could take a look. Let me know.

Closes #56 if I'm not mistaken.

A few higher-level comments:

I like the allow_overwrite=True feature, and if you show me your attempt, I can see if I find out why it doesn't work.
The belief time issue also deserves a line in the CLI docstring (if you don't tell us what the horizon is, we'll assume that you now is the time you acquired this knowledge).
We have some code in the API to validate if the data has resolution fitting to the event resolution of the sensor, even upsampling if needed. That would be useful here, as #56 also mentions. My plan was to factor that part out somehow (into the data package I guess) and use it on both places.

Should we maybe move the resolution check / upsampling code (see my third point) to timely-beliefs?

Should we maybe move the resolution check / upsampling code (see my third point) to timely-beliefs?

I don't think we should. This code is problematic for sensors measuring instantaneous events (such as temperature and accumulated consumption)

I made separate fm and tb branches for you to test out my attempt at upserting beliefs. See here (for FM) and here (for TB).

That is a very valid concern. In addition to your suggestions, I propose to make the 2 CLI commands for adding sensors and sensor data part of a dev-add CLI group for now, to make that distinction clearer.


server URL for mscodehub

Followed directions for Self-hosted Linux agent...
Our project is on mscodehub, so PAT had to be created on https://mscodehub.visualstudio.com/{Project Name}.  I downloaded the agent tar.gz file, unpacked it, and ran config.sh.  But when it asks for server URL, I couldn't get it to accept my mscodehub.visualstudio.com address, nor the suggested dev.azure.com address.
The log file shows that GET request failed with status of "Not Found".  What URL should I use?

@palladia  Thank you for the feedback, we're sorry you're experiencing issues.
The issue was reported previously here, please see if this addresses the problem.
If not, you'll need to report this to the product team using Report a problem in the Azure DevOps community and link back to this issue.

Closing now, please comment below if there are any further issues.


Invalid value "undefined" for header "Content-Language" with 1.9.0

Since the new version released today, all our apps started failing with this error:
TypeError [ERR_HTTP_INVALID_HEADER_VALUE]: Invalid value "undefined" for header "Content-Language"
    at ServerResponse.setHeader (_http_outgoing.js:488:3)
    at i18nextMiddleware (project/node_modules/i18next-express-middleware/lib/index.js:68:11)

Pinning the i18next-express-middleware version back to 1.8.2 fixes the errors. We are using a fairly vanilla express setup,  very similar to the "wire up i18next to request object" example given in the readme. We do not use the language detection features of the middleware.

@isaachinman any idea?

I can take a look in about an hour, but it looks like http is doing some validation that Express wasn't.

@zsgalusz Can you confirm if your app is casting undefined to a string?

wondering a little how lng can be undefined?
situation a) https://github.com/i18next/i18next-express-middleware/blob/master/src/index.js#L45 --> your app only works on fallbackLng (makes no sense)
situation b) https://github.com/i18next/i18next-express-middleware/blob/master/src/index.js#L29 --> there was a language change call from within the request to undefined (makes also no sense)

ok guess I see, we avoid problems by enforcing the fallback language: https://github.com/i18next/i18next-express-middleware/blob/master/src/index.js#L50 so language can be undefined - but neither using a detector nor setting that on req.lng is rather odd`?!?
@zsgalusz why you're using this without translating? not setting any language for the request?

@zsgalusz could you retry with<EMAIL_ADDRESS>(added a check for lng is set)

@jamuhl If the user's app is casting undefined to a string, eg "undefined", then this will actually pass the truthy check you've introduced with v1.9.1, as would any positive number, etc. Is it possible to check if lng belongs to an array of possible languages?

@isaachinman I guess setting "undefined" is possible -> I believe it is really setting undefined which is not allowed -> I believe the error message output casts it to string

Okay, wasn't sure! That's fine then. This is a weird one either way.

1.9.1 does fix the error, thank you for the fast response!
Our setup in detail:
We are running on a node/express platform that does its own language detection, so we do not need the built-in ability of the middleware. We use the middleware like this:
i18next
  .use(initReactI18next)
  .use(i18nextFSBackend)
  .init({
    keySeparator: '$',
    fallbackLng: 'en',
    interpolation: {
      escapeValue: false,
    },
    backend: {
      loadPath: path.resolve(
        'statics/assets/locale/all/messages_{{lng}}.json'
      ),
    },
  })

app.use(i18nMiddleware.handle(i18next));

// later in request handler:
  let language = _get(req, 'aspects.web-context.language', 'en');
  req.i18n.changeLanguage(language);

So yes I suppose until we receive the languages from the platform in the handler, the language indeed is not set.

@zsgalusz thank you for making your setup more clear -> looks valid - so glad the provided fix works for you


Add rider using Kafka in ASP.net 3.1/5.0 stop without log : The program 'dotnet' has exited with code 0

Is this a bug report?
Yes
Can you also reproduce the problem with the latest version?
Yes, I try to use the preview version or the library, and I still have the same issue
Environment

Dotnet version: 5.0 or Core 3.1
Operating system: aspnet:3.1-buster-slim or aspnet:5.0-buster-slim
IDE (Rider, VS Code, Visual Studio): vs 2019

Steps to Reproduce
Just start the projet with docker compose, after few seconds, these logs will appear :
info: MassTransit[0]
      Bus started: loopback://localhost/
MassTransit: Information: Bus started: loopback://localhost/
The program 'dotnet' has exited with code 0 (0x0).

And the API is stopped
Expected Behavior
The API continue to work and handle incoming message from topic : "test"
Actual Behavior
The API stop after few seconds, even if I try to change the kafka host for : "localhost:9092" or something more stupid like "test:9093" (in this case the error occured after ~30s instead of 3s)
Reproducible Demo
    public class Startup
    {
        public Startup(IConfiguration configuration)
        {
        }

        public void ConfigureServices(IServiceCollection services)
        {
            services.AddMassTransit(x =>
            {
                x.UsingInMemory((context, cfg) =>
                {
                    cfg.ConfigureEndpoints(context);
                });

                x.AddRider(rider =>
                {
                    rider.AddConsumer<UpdateCommandConsumer>();

                    rider.UsingKafka((context, k) =>
                    {
                        k.Host("broker: 9092");

                        k.TopicEndpoint<ITrackingUpdateCommand>("test", "test", e =>
                        {
                            e.ConfigureConsumer<UpdateCommandConsumer>(context);
                        });
                    });
                });
            });

            services.AddMassTransitHostedService();
        }

        public void Configure()
        {
        }
    }

    public class UpdateCommandConsumer : IConsumer<ITrackingUpdateCommand>
    {
        public Task Consume(ConsumeContext<ITrackingUpdateCommand> context)
        {
            return Task.CompletedTask;
        }
    }

Thanks in advance
I can paste my docker compose if needed
Kind regards
Rémi

Kafka works fine for me, so unless you have a complete standalone repository to reproduce it, you're kind of on your own.
This question might be related: https://stackoverflow.com/questions/66062726/accessviolationexception-during-application-exit-while-using-masstransit-with-ka

you should use confluent docker images to make it work without any issues. I think it is the main reason they built their own images

Hello @phatboyg @NooNameR
Thanks,
I juste create a sample repository to explain the issue : https://github.com/odinnou/masstransit-kafka-aspnet
It's clairly associate with Docker integration, because I try with console app mode or IIS express and .. yes everything work fine.
I'll check the stackoverflow and duplicate issue linked above, if it can help me
Thanks in advance
Kind regards
Rémi

Well in my case I only run it on Mac. And I've had apps in both Docker and Mac. Just make sure you're building for the OS on which you're deploying since Confluent uses a lib and PInvoke.

Hello, to be 100% sure of the docker image compliance with the confluence library (Confluent.Kafka 1.5.3)
I chose the same image (mcr.microsoft.com/dotnet/core/aspnet:3.1.9-alpine3.12) used by an existing project which is able to execute the Confluent.Kafka 1.5.3 (without masstransit)
I don't know if you have check my "complete standalone repository to reproduce it" like you ask earlier, but I try to be the more lightweight possible.
I tried to switch from InMemory Bus to RabbitMQ Bus, but same issue
I really want to use your lib because your integration is really great :/
Thanks in advance
Kind regards
Rémi

it looks disappointing, but it is the issue with docker windows? but definitely not with MT, whenever you use pure Confluent client (which is used by MT) - you will get the same problem. I used to have that error on my windows machine (when I was using windows) few years ago

I have this issue with Docker Windows right, but I don't get this error with pure Confluent client (I have 10 microservices running this client in production and we are all on windows when we develop).
I use WSL 2 integration with Docker Windows, so it's technically a Linux host and not a Windows host.
If needed I can build a sample that work without MT (pure Confluent client Consumer with the same configuration)

deleting and recreating topic helped me when I faced the issue

Might try the latest develop package, which contains the latest Confluent Client: https://www.nuget.org/packages/MassTransit/7.1.7-develop.3340

Hello @phatboyg I just update my repository with
   <PackageReference Include="MassTransit.AspNetCore" Version="7.1.7-develop.3340" /> <PackageReference Include="MassTransit.Extensions.DependencyInjection" Version="7.1.7-develop.3340" /> <PackageReference Include="MassTransit.Kafka" Version="7.1.7-develop.3340" />
And I still got the same issue :(
@NooNameR  I try to start the API without / with a topic, recreate / delete it but no change :(
I think it's not related with topic configuration
I will try to build an API with a consumer that use the pure confluent client to compare
and maybe, try to clone the MT repository and reproduce the issue without Nuget package (maybe I'll got more logs ?)

As I've suggested to others, you might also raise the issue with Confluent, since it is their client library – they might have other suggestions to try.


Define column header requirements for a CSV schema for universal pipeline

User Story
As a ReportStream teammate, I want to define specific requirements for a new CSV schema for Senders to utilize when submitting to the UP, so Senders can submit their data successfully and ReportStream engineers can successfully support Senders using this service.
Background
Owing to the CSV Support in the UP MVP Product Brief, some 50% of current COVID pipeline Senders send messages in CSV format. As we transition these Senders to the UP, we are considering various strategies to both 'meet users where they are' while helping to support ReportStream values and key OKRs of allowing organizations to send multiple conditions to the UP.
Risks/Impacts/Considerations

A more stringent CSV schema, or one with more requirements, may inhibit CSV Senders from being able to send to ReportStream in the UP
Limiting the scope to only conditions that have LIVD table lookups may not be the best strategy for longevity / planning ahead
However, widening the scope to all single analyte and multiplex conditions may be too complicated to take on currently

Open Questions

What is required to finalize the CSV schema? Which element(s) besides column headers are required to be present?
How much harder would it be to define a CSV schema that supports all single analyte and multiplex conditions, versus a schema that just is limited to the conditions that have LIVD table lookups?
What condition(s) are impossible for any CSV schema to support?

Links

CSV schema proposal in UP Product Brief
CSV File Formats
12613: Business case and logic flow for multiple options of sending CSV messages to the UP

Acceptance Criteria

[x] Finalize CSV schema column headers to accept all single analyte and multiplex conditions (multiplex may be tbd based on research)

Out of Scope

Testing validity of schema (internally and/or with Senders)
Accompanying requirements (type, valueset, HL7 or FHIR mapping, documentation)

@GilmoreA6 and I began the work in the CSV/HL7/FHIR Mapping tab of the CSV in UP spreadsheet. We are working backwards from HL7 to CSV. CC @arnejduranovic @jillian-hammer

Schema column headers appear to be finalized in the document (which was moved to sharepoint). Some comments are less specified in terms of "default explanation", but this is captured in the product brief as work to continue/unfinished work.
Also, multiplex conditions continue to be TBD, except for OTC respiratory multiplex tests, as seen in this ticket. Multiplex being "unfinished work" is also noted in the product brief.
Moving to Ready for Product Review and can move to Closed barring any adjustments or changes from Platform leads. CC @Andrey-Glazkv @jillian-hammer

Closing out of this ticket -- I don't see any further work expected for now! CC @jillian-hammer @Andrey-Glazkv


Issue #1329

For #1329
Allow extended classes to access the LogEntry building, so that
additional information may be encoded in each log entry (eg traceid).
Use addLabel rather than setLabels as it avoids the creation and copy
of a HashMap per log entry.
Signed-off-by: Greg Wilkins<EMAIL_ADDRESS>

Coverage increased (+0.0008%) to 83.84% when pulling df8944d16a568c798a9c8ae6c0a5bf1b5a7a628c on jetty-project:issue-1329 into 81278dc3cb13ec9073e4fb4bb65bb3afe7502952 on GoogleCloudPlatform:master.

@mziccard Work on the jetty-runtime is blocking on this. Is anything else needed to approve this for merging?

@meltsufin I am no longer involved in this project, please refer to @garrettjonesgoogle for help

LGTM if Travis passes

Coverage increased (+0.002%) to 84.228% when pulling 9fd9d0e7955c1a45248856546deb1f5d3207c6ba on jetty-project:issue-1329 into 164878e2ac8c4ad0aaa9704b4da287358d92be4c on GoogleCloudPlatform:master.

Coverage increased (+0.006%) to 84.233% when pulling 9fd9d0e7955c1a45248856546deb1f5d3207c6ba on jetty-project:issue-1329 into 164878e2ac8c4ad0aaa9704b4da287358d92be4c on GoogleCloudPlatform:master.

Coverage increased (+0.002%) to 84.228% when pulling 9fd9d0e7955c1a45248856546deb1f5d3207c6ba on jetty-project:issue-1329 into 164878e2ac8c4ad0aaa9704b4da287358d92be4c on GoogleCloudPlatform:master.


First steps for title entries table MODEUR-6

INFO: api-lint: Using api-lint version: 1.0.6
INFO: api-lint: Assessing API description files: OAS
INFO: api-lint: Processing OAS 3.0 file: src/main/resources/openapi/eusage-reports-1.0.yaml
ERROR: api-lint:
Violation - Syntax error : Expecting 'value' but '}' found
Model: file://src/main/resources/openapi/eusage-reports-1.0.yaml
Profile: OAS 3.0
Conforms? false
Number of results: 1
Level: Violation

Source: http://a.ml/vocabularies/amf/core#syaml-error
Message: Syntax error : Expecting 'value' but '}' found
Level: Violation
Target:
Property:
Position: Some(LexicalInformation([(33,2)-(33,3)]))
Location: file://src/main/resources/openapi/schemas/reportDataEntry.json

INFO: api-lint: Processing OAS 3.0 file: src/main/resources/openapi/tenant-2.0.yaml
INFO: api-lint:     did not detect any errors
ERROR: api-lint: There were processing errors. See list above.

Note: When those ERRORs are fixed, then any INFO and WARNING messages will be gone too. See Jenkins "Artifacts" tab for ApiLintReport.

INFO: api-schema-lint: Using api-schema-lint version: 1.0.4
INFO: api-schema-lint: Assessing schema files (https://dev.folio.org/guides/describe-schema/)
INFO: api-schema-lint: Found 16 JSON schema files under directory 'src/main/resources/openapi'
ERROR: api-schema-lint: Trouble loading src/main/resources/openapi/schemas/reportDataEntry.json: Expecting property name enclosed in double quotes: line 33 column 3 (char 750)
ERROR: api-schema-lint: There were processing errors. See list above.

Note: When those ERRORs are fixed, then any INFO and WARNING messages will be gone too. See Jenkins "Artifacts" tab for ApiSchemaLintReport.


PP-2422 Send ePDQ refund requests and handle immediate responses

Sends RFD (partial refund) requests to ePDQ. Expects either status 81 (refund pending) or 71 (deletion pending) responses (usually be 81 but can be 71 if the capture has not yet been processed), which become REFUND SUBMITTED. Other responses become REFUND ERROR.
Refund references are constructed by concatenating the PAYID (gateway transaction ID) with a solidus followed by the PAYIDSUB (number that increments with each operation) from the response. The same PAYID and PAYIDSUB will be included in the subsequent notification, which will allow us to do the lookup by reference.
Pseudo feature flag (hardcoded boolean set to true for tests and false for real) used to stop refunds from working because we haven’t added support for handling the subsequent notifications yet. This means that pressing the refund button in selfservice or using publicapi will continue to display an error as it currently does.
with @juanjoqmelian
with @alexbishop1

rebuild

Just wondering since we have decided to handle status 71 which according to the comment could only happen if the payment hasn't been captured:
• is it even possible to have such an occurrence considering we only allow refunds post full capture of a charge.
• if it is a valid genuine scenario, what do we do with these refunds?

According to ePDQ, we can actually get a 71 if the payment is in status 9 (“the time scale for the deletion option showing instead of refund can be closer to 24-36 hours, and so it is still possible to delete a transaction in status 9”). From our perspective, we treat it just like any other refund: whether ePDQ treat it as a true refund or a deletion doesn’t matter to us; we just need to make sure we handle all the possible statuses.

Also would you be able to add a little bit more context in the commit message. There are a few changes worth documenting in the commit message.

Yeah, was thinking that. Will rewrite the commit message to make it more like the pull request description.

Addressed comments, rewrote original commit message, added new integration test, changed pseudo feature flag to real feature flag (to facilitate integration test). Can you take another look please, @ckalista?


Split out visualizer

Reason/Context
I have raised this before but lost somewhere, or at least I cant find it 😅
I would like to have the Visualizer split out from Studio so we can start to use it and develop it independent of the studio.
Description
I already had the need for it and had to fork it from the studio and decouple it (can't recall what had to change, but something did). I especially needed it to show external applications from the point of view of the application.

One of the questions that came up while doing this myself is when does Cupid come into the picture, and what should the visualizer do?

The distinction I did, for now, was that Visualizer rendered the view of a single application, and the external applications were a "subtle" link to the external application (see the first image). It can of course use Cupid to get this information about the relations to the application in view.
Cupid then showed the entire picture of the system and how everything was connected (see above image).
I know you said you did not want to split it just yet, but I would argue it should 😆

Last community meeting, I introduced the library I have been working on. Still a work in progress, but as soon as it reach v1 I will provide a PR to the studio, proposing to integrate it 🙂
https://youtu.be/StG6feflHjM?t=1732


Recover from exceptions during placement new

When an exception occurs when running the constructor during placement new, modify the allocated block so it won't try to run the delete operation. When an exception was raised, it means that the block was allocated, but not initialized, so running a destructor will result in invalid operations. We can't "unallocate" the block, so the finalizer is modified to be a no-op.
Closes #3

Hi @smuenzel,
Thank you, I did not anticipate this behavior.
This PR looks good to me and made me realize that this binding misses tests for misuse cases.


problem sidebar

1-journal sidebar (right)problem home page
2-oc sidebar problem (left)
3-home page same prodcuts no see ( interesting..
sametimes no seen :(
<IP_ADDRESS>

I can't really help without detailed, technical descriptions of the issue, specific errors from the log, and so forth.
That said, the Journal theme already has it's own page cache, so perhaps you should use that instead.
Our page cache does work fine with the default theme that comes with opencart.

thank you very fast support and free.
thank you :)
and others prbolem other modules can you help me :(( please :(
I need this file but for oc 2 :)
https://gist.github.com/weismannweb/a16b5d103e89ca1d1209
I try my chance :)

We're a neon sign shop...not generally opencart developers.

Thank you for reply


RigidBody2D Weight field strange behavior

Godot Master

Change the gravity to 0 in project settings leads to this behavior

starting the game with this setting crashes the game.
Parse Error: Unexpected identifier: 'inf'.

This is a duplicate of #16038


Fix pipeline parameters display

There was a bug when a pipeline job had multiple arguments and all arguments are from type Vault, it still redirected you to the arguments input page which was basically empty. This PR fixes that and refactored the start pipeline handler.

Codecov Report

Merging #181 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #181      +/-   ##
==========================================
+ Coverage   66.98%   67.02%   +0.03%
==========================================
  Files          36       36
  Lines        2732     2732
==========================================
+ Hits         1830     1831       +1
+ Misses        665      664       -1
  Partials      237      237

Impacted Files
Coverage Δ

workers/scheduler/scheduler.go
68.87% <0%> (+0.33%)
:arrow_up:

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 ca6e757...4376720. Read the comment docs.


Catboost Documentation

A grading boosting algorithm in which I want to contribute. Please assign this to me

@ashwani-rathee @geekquad assign this issue to me

@geekquad Do we need this?

Go ahead @aman503.

Thank you for your response
On Wed, 19 May, 2021, 1:19 pm Aditya Kumar Gupta, @.***>
wrote:

Assigned #129 https://github.com/Algo-Phantoms/Algo-ScriptML/issues/129
to @aman503 https://github.com/aman503.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Algo-Phantoms/Algo-ScriptML/issues/129#event-4764869879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARR6NW6WYK5FTWO2VZDEBLDTONURZANCNFSM4ZUMUTDQ
.

When do I have to submit?
On Thu, 20 May, 2021, 3:35 pm Aman Dahiya (RA1911027030054), <
@.***> wrote:

Thank you for your response
On Wed, 19 May, 2021, 1:19 pm Aditya Kumar Gupta, <
@.***> wrote:

Assigned #129 https://github.com/Algo-Phantoms/Algo-ScriptML/issues/129
to @aman503 https://github.com/aman503.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Algo-Phantoms/Algo-ScriptML/issues/129#event-4764869879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARR6NW6WYK5FTWO2VZDEBLDTONURZANCNFSM4ZUMUTDQ
.

@aman503 It's a level-1 issue, should have been done in 2 days. Please make a PR asap

@yukti845 can I work on README for this? I'll make PR the same day it gets assigned to me :)

@yukti845 https://github.com/yukti845   Give me some time...Actually I'm
not well and will make a PR by 28th may. Sorry for inconvenience.
On Wed, May 26, 2021 at 11:57 AM Manasi Chhibber @.***>
wrote:

@yukti845 https://github.com/yukti845 can I work on README for this?
I'll make PR the same day it gets assigned to me :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Algo-Phantoms/Algo-ScriptML/issues/129#issuecomment-848499997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARR6NW2YCEXSOMRO4DVUKE3TPSIGBANCNFSM4ZUMUTDQ
.

okay @aman503

I can work as no PR is done yet. I can submit PR today itself

@yukti845 can I work on README for this? I'll make PR the same day it gets assigned to me :)

@geekquad @ashwani-rathee please give me a heads up on this.


Multiplication of HypreParMatrix objects

Hello everyone,
Is multiplication of two HypreParMatrix objects implemented currently? I have two HypreParMatrix objects - one from parallel assembling a bilinear form in MFEM, and another obtained in a different way and I want to multiply them (to put as a block in the resulting linear block system's operator), but I didn't find anything implemented which would make it. Is it really so or have I mislooked?
Thanks,
Kirill

Hi, Kirill,
Have a look around line 480 of "mfem/linalg/hypre.hpp" and you should find a function called "ParMult".  This takes pointers to two HypreParMatrix objects and returns a pointer to a new HypreParMatrix object which contains the product of the two inputs.  As long as the matrices have compatible shapes this should work correctly regardless of how the matrices were constructed.
Best wishes,
Mark

Hi Mark,
Thank you for the hint, this is what I needed. I wonder why there are no mentions of this function in the documentation for HypreParMatrix class.
Best,
Kirill

Feel free to add documentation, @kvoronin :)

Hi, Kirill,
Well, it's a standalone function and not a class member so it is harder to find.  Your point is well taken though.  We should probably add something to the comment block at the start of the HypreParMatrix class to list related functions.
Thanks,
Mark

Thanks for the replies!


Update clusternode struct with correct omitempty directives

@giorgos-nikolopoulos Can you resolve the conflicts? All uint32, int32 and uint64 were mapped by int recently to be consistend with older SDK.


Files lose the color tag after editing

Description
I have Windows 11 and the Files version <IP_ADDRESS>. For all my Excel tables I have a color tag. Every time I edit and save one of the Excel files, the color tag is gone and I have to find and re-tag the file.
Steps To Reproduce

tag (Excel) file
open, modify and save file
tag is gone

Steps to fix
How can the files keep the tag after editing
Files Version
Version <IP_ADDRESS>
Windows Version
Windows 11 Home 22H2
Relevant Assets/Logs
none

I have tried this with a text file, and the bug doesn't happen to me.
I will try it with excel too and let you know.

You will need to upload the logs files directly to GitHub. You can get them in files settings > about > open log folder.

@Josh-65 I know you aren't asking me for the log files, but I am downloading the Lab Test Kit from the Microsoft Evaluation Center right now, to try it with excel. It does not happen with notepad though, as I said earlier. Also, if this bug actually "works" with excel, where can I upload the log files? Here?

You can just drag the two .log files into the text box and wait for it to show something like [debug.log] (generated URL)

Yeah, just like I attach videos and stuff. Ok, so just attach it to the comment. Sure, will do.

Hello @Josh-65 , I did open the log folder but what data should I upload form there? Or is it the whole folder?

There should be two files debug.log and debug_fulltrust.log

Thanks for the help. If you need something else please let me know.
debug.log
debug_fulltrust.log

My guess is Excel removes the alternate data stream that holds the tag. Pinging @hez2010 for thoughts on this issue.


Empfänger and Absender MP-ID has wrong data element type in maus and are missing ahb_expression

Probably also relevant for other pruefis, but here in the 11042.maus the MP-ID for Absender and Empfänger should be a Freefield Data Element and not a Value Pool: https://github.com/Hochfrequenz/edifact-templates/blob/03559bccf5f29d3c457667aec393e57d5338cbe8/maus/FV2110/UTILMD/11042_maus.json#L108
Also they are missing their ahb_expression:
Current state:
{
"data_element_id": "3039",
"discriminator": "$[\"Dokument\"][0][\"Nachricht\"][0][\"MP-ID Absender\"][0][\"MP-ID\"]",
"value_pool": [],
"value_type": "VALUE_POOL"
}

Expected:
{
"ahb_expression": "X",
"data_element_id": "3039",
"discriminator": "$[\"Dokument\"][0][\"Nachricht\"][0][\"MP-ID Absender\"][0][\"MP-ID\"]",
"entered_input": null,
"value_type": "TEXT"
}

[ ] AHB-Expression Absender
[ ] value_type Absender
[ ] AHB-Expression Empfänger
[ ] value_type Empfänger

Can be hardcoded if it is urgent, but this always has the danger to be overwritten again when the maus is newly generated.

should be a Freefield Data Element and not a Value Pool

This is fixed since #201/v0.2.4


ip6: add ip6-related definition

Summary
We will integrate network management tools in future projects. In the process of porting the tools, we encounter some situations where the structure is not defined, or the returned data types do not match the expectations. Refer to the common implementation of other systems and add relevant definitions.
Impact
Testing
sim:local

please add commit message describing the change


x/mod/zip: verify file list without creating zip

golang.org/x/mod/zip provides a Create function that takes a list of files ([]File). On success, it creates a zip file with some of those files. It ignores files in submodules and vendor directories and reports errors for files with invalid names or types (symbolic links). On failure, it reports an error on the first file that caused a problem.
This is not the most useful API for clients creating zip files or verifying that a set of files can be used to create a zip file. Some improvements are possible without making incompatible changes to the API:

We should provide a Verify function that accepts []File and reports whether each file in that list will be included, ignored, or rejected. Verify would not create a zip file, and it would not read files (just stat to get sizes and types), so it should be faster than Create.
When Create encounters an error due to the []File argument, it should return a structured error with information about each file. This should be similar to what Verify returns (perhaps whatever Verify returns can satisfy the error interface).
Unzip should also return a structured error so that issues with multiple zip entries can be reported.
We may also want a VerifyZip function which checks a zip file for errors without extracting it. Currently, Unzip always needs a directory to write files.
We should document more clearly when files are included, ignored, or rejected.

cc @dmitshur

Some improvements are possible without making incompatible changes to the API

Incompatible changes would be fine too, I think: the latest tagged version of that repo is still v0.1.0 and the zip package is not included in that tag, so even with our more conservative approach to breaking changes, most users should be unaffected.

Issue #37397 is related.
It's worth considering that depending on the exact goal of a user, they may want to reproduce that vendor bug in order to ensure consistent behavior with cmd/go, or they want to resolve that bug and avoid stripping some valid packages unneccesarily.
A good high-level way of asking that question is whether the user goal is to stay within the rules of the module proxy protocol, or additionally within the cmd/go implementation restrictions.

A good high-level way of asking that question is whether the user goal is to stay within the rules of the module proxy protocol, or additionally within the cmd/go implementation restrictions.

That's a good way to phrase it. Perhaps zip.Create could be a lower-level API that zips all files unless they would cause an error in zip.Unzip (e.g., too big, invalid file name). Or maybe we would introduce zip.CreateRaw to avoid an incompatible change. zip.CreateFromDir would be the convenience function that ignores vendor directories.
Verify would still be useful for clients to understand whether a file should be included.


node version mismatch

i@raspberrypi:~ $ node

var noble = require('noble');
Error: The module '/home/pi/node_modules/noble/node_modules/bluetooth-hci-socket/build/Release/binding.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 46. This version of Node.js requires
NODE_MODULE_VERSION 57. Please try re-compiling or re-installing
the module (for instance, using npm rebuild or npm install).
at Object.Module._extensions..node (module.js:672:18)
at Module.load (module.js:556:32)
at tryModuleLoad (module.js:499:12)
at Function.Module._load (module.js:491:3)
at Module.require (module.js:587:17)
at require (internal/module.js:11:18)

Duplicate of https://github.com/noble/noble/issues/753

Even the same problem i tried it many times will you please tell me correct versions of node npm etc to avoid this problem actually  i am working on ble beacon noble.startScanning();
noble.on('discover', function(peripheral) {
var macAddress = peripheral.uuid;
var rss = peripheral.rssi;
var localName = advertisement.localName;
console.log('found device: ', macAdress, ' ', localName, ' ', rss);
)};


experiment(ci): Test esy@next

Check esy@next - cc @EduardoRFS

Hoping that this works, because I already released it, it was blocking some other stuff ;/

Looks like there was an error on CentOS, but not related to esy -> seeing the same issue in #3365


blacklist cloth

https://github.com/iLLeniumStudios/fivem-appearance/blob/main/blacklist.json

Coming soon, thank you


Decouple token channel subscription

Decouples registering a token from subscribing to a channel so that they can happen at different times in the lifecycle of an app.
Consider a typical situation where push notifications are registered after a user signs in. Assume permissions for push notifications have been granted already.

App launches and immediately receives the application(_:didRegisterForRemoteNotificationsWithDeviceToken) delegate
User signs in

At step 1 a developer can't register for a channel because it needs the user model to determine the channel name. At step 2 a developer can't register because the device token is only available in the delegate callback. Unless of course the developer has done additional work to make it available.
With this change the situation changes as follows:

App launches and receives callback
Device token is saved in user defaults using courier.deviceToken = deviceToken
User signs in
User is subscribed to a channel using courier.subscribeToChannel(user.uuid)

In essence we've done the extra work for the developer by saving the device token in user defaults whenever the device token is set.

LGTM


InstantCoder: stop boxing Longs unnecessarily

In encode, and similar in decode, the existing path goes
Instant->long (inside of converter)
long->Long (returned from converter)
Long->long (inside of LongCoder).
This is a relatively small improvement, but as we encode timestamps
for every single element, this is likely to make a difference in
lightweight stages of pipelines.
R: @lukecwik @bjchambers

PTAL


branch-2.1 schema change may cause mow duplicate key (#33536)

Proposed changes
Issue Number: close #xxx
cherry-pick #33536
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Do you have the compiled data sets available to share?

Hi,
I am wondering whether you are able to share all the data sets you collected and compiled? It can be useful to when want want to test different models using the same data.
Thanks,
Song

I think I may have the sources of the data here: http://nlpprogress.com/english/dialogue.html

happy to know that you figure it out.


Add published biomass burning info from ESGF

Description
Workaround for issues defined in #133
Fix #128
Checklist
Please confirm that this pull request has done the following:

[x] Documentation added (where applicable)
[x] Changelog item added to changelog/

@znichollscr over to you

@durack1 updated now. You can see the fixes I made in the commits. The human summary is:

there were no new entries in the database, so it looks like the workaround script hadn't been run or the results hadn't been committed
I've updated the docs to hopefully clarify the process, feel free to add any tweaks in a follow up MR (I know you're keen to get this merged and live, so I'm just going to hit the button)

cc @eleanororourke @vnaik60, update should be live now (or in 2 minutes)

@znichollscr probably worth a patch bump, no?

Can't hurt, go for it
On Fri, Oct 18, 2024, 17:54 Paul J. Durack @.***> wrote:

@znichollscr https://github.com/znichollscr probably worth a patch
bump, no?
—
Reply to this email directly, view it on GitHub
https://github.com/PCMDI/input4MIPs_CVs/pull/134#issuecomment-2422779967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3KEXH3ZZGH7UNNRA5NC5CTZ4EVLJAVCNFSM6AAAAABQC74KZWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRSG43TSOJWG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update to bitvec 0.22

No behavior changes, just a few symbol renames.
Sorry about the mess with funty; hewing so closely to names in core is always a fun time.

Just saw you have an MSRV of 1.44! Oops!
I am keeping my 1.51 MSRV; lowering is non-negotiable (array const-generics) but I do not know of any reason I will be raising it before 1.0 at the end of June. You can leave this PR idling until you're ready to raise MSRV to 1.51.
You should be able to patch to 0.19.4 and lose funty. However, this PR contains the symbol changes you'll need to advance past the 0.19 series, including to 1.0 when I complete it. I will not be changing the APIs used here between 0.22 and 1.0

Coverage remained the same at 82.632% when pulling d6b81f5303b0a347726e1f3f428751f376e7b771 on myrrlyn:master into 38bb94e7bc45c2b4ef0c473e0e5c03ab134cdff3 on Geal:master.

Any updates on this?

merged in 03a1025, thank you for your help


oci_tarball fails if called with no tags

this is okay:
oci_tarball(
    name = "image_tar",
    image = ":image",
    repo_tags = [],
)

this fails to build:
oci_tarball(
    name = "image_tar",
    image = ":image",
)

error:
	File "/Users/chancila/Snapchat/Dev/.cache/bazel/arm64/0b63180b51ef7d80cbce3ff6de02bf22/external/rules_oci/oci/private/tarball.bzl", line 60, column 20, in _tarball_impl
		ctx.actions.run(
Error in run: at index 1 of inputs, got element of type NoneType, want File
ERROR: /Users/chancila/Snapchat/Dev/snapci/bin/services/external_proxy/BUILD.bazel:46:12: Analysis of target '//bin/services/external_proxy:image_tar' failed
ERROR: Analysis of target '//bin/services/external_proxy:image_tar' failed; build aborted:
INFO: Elapsed time: 0.273s (0.00m)

The doc https://docs.aspect.build/rules/rules_oci/docs/tarball claim it is optional and a label, while the example shows a list of string passed.
My understand is that it expect a label to a file with one line per tag and is mandatory

oci_tarball rule only accepts a file label but the oci_tarball macro which knows how to generate tags file from starlark list accepts both label and list [].

Ok so change the import:
load("@rules_oci//oci/private:tarball.bzl", "oci_tarball")
# to
load("@rules_oci//oci:defs.bzl", "oci_tarball")

But repo_tags is still not optional, the default value should be moved from None to []

But repo_tags is still not optional, the default value should be moved from None to []

yes, that's a bug. that said you should only imports symbols from @rules_oci//oci:defs.bzl

I agree that importing from a private folder does not seem like a good thing, but following doc seems a safe bet normally :)
Thank you for your answers

#293 fixes the documentation bug


fix demo/docs url pls

hey, looks like this repo got moved.
Can you please adjust the paths to the demo/docs? https://wix-incubator.github.io/kampos/

hey @axe312ger, thanks for noting this! I'll fix right away

Great, thanks!

Did you maybe forget to change in the repository about section? 🙈


⚠ HawkeUno has degraded performance

In 56f2963, HawkeUno ($UNO) experienced degraded performance:

HTTP code: 403
Response time: 1750 ms

Resolved: HawkeUno performance has improved in 7a3d2d4 after 6 minutes.


Change brightness together with unchanged RGB value is not working

Describe the bug
When I send a message with on, brightness and RGB and only the brightness is changing, the brightness is not sent to the hue lamp. When the brightness and RGB values changes, everything is working well.
Flow to Reproduce
[{"id":"290bd37d.cad59c","type":"inject","z":"3da7efa8.dfed4","name":"br100","topic":"","payload":"{\"on\":true,\"rgb\":[153,111,63],\"brightness\":\"100\"}","payloadType":"json","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":820.2143402099609,"y":853.0000610351562,"wires":[["48cf52c6.eacd4c"]]},{"id":"e156b596.005388","type":"inject","z":"3da7efa8.dfed4","name":"br50","topic":"","payload":"{\"on\":true,\"rgb\":[153,111,63],\"brightness\":\"50\"}","payloadType":"json","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":790.2143402099609,"y":912.0000610351562,"wires":[["48cf52c6.eacd4c"]]},{"id":"48cf52c6.eacd4c","type":"hue-light","z":"3da7efa8.dfed4","name":"Küche","bridge":"870de88f.3aee98","lightid":"22","colornamer":true,"x":908.214391708374,"y":617.5715255737305,"wires":[[]]},{"id":"870de88f.3aee98","type":"hue-bridge","z":"","name":"Philips hue","bridge":"<IP_ADDRESS>","key":"xyz","interval":"3000"}]

Expected behavior
Brightness is adjusted at the hue lamp.
Please complete the following information:

NodeRED version: 1.0.2
NodeJS version: 10.15.1
Device running NodeRED: Windows

https://app.leanboard.io/board/f91ab067-05b2-4e59-a4ef-4bfa7c5b6ed5

If I understand you correctly, is a parameter ignored? In your case, does it ignore the brightness setting?

@Foddy

If I understand you correctly, is a parameter ignored?

Yes, it is ignored. The lamps brightness is not changing

In your case, does it ignore the brightness setting?

Yes, because no other values are changing, the lamp does not change anything.
It seems like the brightness is only sending to the lamp, when the RGB value is changing too. Remember this is only when on, rgb and brightness is in the same msg.

@Foddy I'm sorry, the problem was the IKEA TRADFRI blub. This is not a bug of this library.

I know this got closed, but I faced the same issue and possible found a working work around.
I can confirm there seems to be an issue regarding IKEA TRÅRFRI, at least the
TRÅDFRI LED bulb E12 600 lumen is affected. Same issue regardless whether using the Hue Lights node or the Hue Groups node.
I am using the following work around to get the random colors and brightness working.
First we need to send just the desired brightness as the first payload, in example: {"brightness":100}
After that there needs to be a delay, I'm using the random node with 1-5s delay. I couldn't get it to work reliably without at least 1 second delay for some reason.
Then we need to send the second payload, in example: {"color":"random","transitionTime":60}
This has at least so far worked fine and does first set the brightness to desired level followed by smooth color change transition. It does not work for me when skipping the first brightness payload and instead adding brightness to the second payload.


k8s create pod failed in CNI plugin mode

kubectl create -f pocnet-busybox.yaml with kubelet --network-plugin=cni command-line option resulted in repeat of
time="2017-05-16T22:25:15+08:00" level=info msg="==> Start New Log <==
" NETNS=EMPTY
time="2017-05-16T22:25:15+08:00" level=info msg="command: DEL, cni_args: IgnoreUnknown=1;K8S_POD_NAMESPACE=default;K8S_POD_NAME=busybox-poc-net;K8S_POD_INFRA_CONTAINER_ID=311717d4d2b2c95137b61faec8001d73c3dc1c596e266331a722ba5dd8c5dc4f" NETNS=EMPTY
time="2017-05-16T22:25:15+08:00" level=error msg="GET Status '500 Internal Server Error' status code 500
"
time="2017-05-16T22:25:15+08:00" level=error msg="DelEndpoint returned 500 Internal Server Error" NETNS=EMPTY

observation from tailf /var/log/contivk8s.log
kubernetes version is v1.6.2

can you check if  netplugin is running ?
kubectl get pods -n kube-system -o wide

As mentioned in the previous issue of netctl group create overlapping ipv4，we adopted a binary executable deployment rather than container  since we can't make any progress with official guide through making k8s-cluster or demo etc.  because of no access to vagrant https. I'm sure netplugin was running at that time.

the error message shows that contiv plugin is not able to reach netplugin(contiv process in host).
@gogolxdong can you join contiv slack channel, https://contiv.herokuapp.com/
so some can check your setup/debug ?


ci: add locker specific missing tests

Type of Change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[x] CI/CD

Description

This PR adds missing tests that calls locker. This includes:

Addition of delete payment method endpoint (this works only on hosted environment)
Rename files
add recurring mit payment endpoint to ntid connector agnostic flows

Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

when running in local, we're expected to get 5xx since this requires locker to be set up locally which is not intended:

ran in hosted environment:

add missing ntid payment flows for validation:

Checklist

[x] I formatted the code prettier . --write
[ ] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible

Can you handle all the connectors for the connector agnostic change  ?

good catch. by the looks of it (re: stripe), this will require a bit of time and a bit of refactoring in mca create call since we're creating mca independently multiple times in connector agnostic.

marking this pr as draft since there exist a lot of blockers when working with ntid flows

blocked until an issue with ntid is fixed where stripe is getting affected.


Create connectors for FTP and S3 based on spark-hdfs

Hadoop API allows connecting to FTP & S3 so it seems it will be possible to create FTP & S3 connectors based on spark-hdfs almost cost free

[x] investigate FTP connectivity
[ ] investigate S3 connectivity
[ ] add "FTP via Spark" on Slamdata
[ ] add "S3 via Spark" on Slamdata

FTP UI created at https://github.com/slamdata/slamdata/issues/1825


Live query: error: UNION types uuid and integer cannot be matched code: "42804"

Report from Discord:
https://discord.com/channels/933657521581858818/1273001835346657384

Using PGlite in Bun I get error: UNION types uuid and integer cannot be matched code: "42804"

My query:

pg.live.incrementalQuery(`select * from "public"."users";`, [], "id", () => {})

My schema:

import { relations } from "drizzle-orm"
import { pgTable, text, timestamp, uuid } from "drizzle-orm/pg-core"

export const users = pgTable("users", {
    id: uuid("id").notNull().primaryKey(),
    createdAt: timestamp("created_at").notNull(),
    updatedAt: timestamp("updated_at").notNull(),
})

export const posts = pgTable("posts", {
    id: uuid("id").notNull().primaryKey(),
    createdAt: timestamp("created_at").notNull(),
    updatedAt: timestamp("updated_at").notNull(),
    userId: uuid("user_id")
        .notNull()
        .references(() => users.id),
    content: text("content").notNull(),
})

export const usersRelations = relations(users, ({ many }) => ({
    posts: many(posts),
}))

export const postsRelations = relations(posts, ({ one }) => ({
    user: one(users, {
        fields: [posts.userId],
        references: [users.id],
    }),
}))

Using the non-live query works:

pg.query(`select * from "public"."users";`)

i ran into this yesterday and fixed it by changing the primary key to a serial, as apposed to a string/uuid.
the key id in incrementalQuery('select * from table', [], 'id') to run the diff fails if it's a string.

Report of it still being an issue: https://discord.com/channels/933657521581858818/1273001835346657384/1283167493095165982

@samwillis the PR had added a test for the particular case of a TEXT type primary key - do we have a way to reproduce the issue still being present?


Pre-compiled version.

Could you possibly provide a link to perhaps a pre-compiled version?

It seems I am blind


Steam Deck / Linux Support

I recently bought a Steam Deck and found that it uses unified memory, allowing memory to be shared between the CPU and GPU. After digging into it, I found that there is about 9GB available to use as video memory by default, with the ability to increase to a total of 12GB thru the Deck's BIOS.
I was unsuccessful as getting a Vulkan build of llama.cpp to run, but I tried koboldcpp and it did run. This leads me to want to use llama.cpp/sd.cpp/whisper.cpp on Windows and koboldcpp for Mac and Linux. I've found that kobold's start-up time was slow on Windows but now on Linux, which is part of my rationale for having the difference.

Using koboldcpp would give us llama, sd and whisper in one binary, leaving us with only supplying the Piper binary which is simple since it's CPU only.
Supporting Steam Deck well will probably need a separate theme or UI and improving keyboard navigation is a must.
I've been able to use an AppImage on the Deck.

Need to write adapter for using koboldcpp conditionally.
IIRC the app is doing IPC to electron for SD and whisper, and API for llama.cpp
I think it would make sense to make an interface to unify the two sets. This would sit on the client end and be called to generate text, image, etc.
Maybe call it useAI. Also make a useKobold which useAI conditionally uses the appropriate composable depending on the platform. The rest of the client would use the useAI composable instead of others like useSD or useWhisper.

koboldcpp works for me but I'm unsure if it's what I want to do.


Update to scala 2.8.1

I think this is safe to pull now.

Ill merge it tomorrow night. Noticed some markdown parsing issues the the current that I worked around by hand in the content of my slides tonight. Don't have much more energy now. I really need to publish this guy. Thanks so much for doing this though.


Added Custom InfoWindow and Icon on Google Maps

Hi Lars, this is a great package!!
In one of my projects i need InfoWindow and custom icons on Google Map.
I've added this ability to blade template.
Hope is useful and you accept PR
Thanks for your work

Hi @kikojover, thx for the kind words!
Love the idea and the pr.  Do you use the package only for google maps?
I would love to keep the api for both the map types ( google map and leaflet ) the same.
If you add tests for the google maps info window, I can add the version for leaflet + tests.
Sounds like a deal?

Sure it's a deal!! :-)
Only let me use a couple of days to add the tests, sorry, I'm a newbie in the PR World.
About your question, it's the first time I have used your package, my current project requires Google Maps, so the answer is yes, i use your package only for Google Maps (for the moment ;-)), but I hope use leaflet too in he future.
A question about PR,  i must send another PR with the tests or it's possible to include it in current one? (As you told I'm a newbie ...)
Thank you.

Hey, you can add the tests to this pr.

Hi @LarsWiegers , i've added Unit tests to Google Maps :-)

@kikojover I have merged the changes + made a new release


Refactor [Swagger] Service

Also created swagger.tester.js

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @TagnumElite!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS


Revoked tokens persistence

Describe what are you trying to do
Hi, we're considering to use KrakenD as an API Gateway, and it is looking good so far.
The only question is token revocation. In the documentation it states that revoked tokens are stored in memory, which leads to the question - what happens, if a token is revoked and KrakenD instance is shut down before the token is invalidated on an end-user's side (let's say, user did not make any requests to an API after the token was revoked)?
Does KrakenD (somehow) restores the list of revoked tokens after a reload or in the scenario above revoked tokens won't be considered as "revoked" anymore and users would still be able to access an API?
Thank you!

yes, the KrakenD instances keep a set in memory of the revoked tokens. they expect initialization and updates from an external source by using the exposed RPC methods union and add.
here you have two small working examples for the remote client of the RPC service:

https://github.com/devopsfaith/krakend-playground/blob/master/jwt-revoker/main.go
https://github.com/devopsfaith/bloomfilter/blob/master/cmd/client/main.go

the easiest way to implement your client is adding a local bloomfilter into it and using it for sending the initial snapshots to the new  instances

@kpacha Okay, thank you ~


Full screen, as long as the player appears, it will lead to StatusBar

When creating an issue, please ensure that you provide as much information as you can. Issues with a basic description will sadly be ignored as it doesn't help diagnose the issue :(
If you don't help me, I can't help you ¯\_(ツ)_/¯
Here are some common things to include that can help your issue be diagnosed.

[ ] react-native-af-video-player version:

[ ] React Native version:

[ ] OS:

[ ] Has this issue already been raised?

[ ] Have you clearly read and understood the Readme?

[ ] Code and explanation to replicate this issue:

[ ] Are there any console logs?

[ ] Include Screeshots / Video:

Android


fix UI on Firefox on Windows

Chrome

Firefox

Confirm fixed with last version:

Firefox used on windows 10: 84.0.2 (64 bit)

Confirm fixed with last version:

Firefox used on windows 10: 84.0.2 (64 bit)


Is point compression supported?

Is point compression supported? I need a compressed version of my public key which is 22 bytes long for a sect163k1 curve key pair. I can't find a way to get this right now.

Hi @p0fi and thanks for your interest.
I have chosen not to support point compression, because it seems to be patent encumbered:
https://patents.google.com/patent/US6252960B1/en
I am not 100% sure when the patents will expire, but it should be soon.

Oh really? Thats a pitty, I really need a point compressed representation of the keys. Do you have any idea how I can maybe implement it myself easily? The patent link seems overwhelmingly complex since im not really into cryptography.

Not many people know for sure how the patent situation is currently.
Some patents cover only ECC-tech working on prime-field curves, others are for Galois-field (binary) curves. That could explain the difference, since Ken MacKay's library works on prime-field curves and mine works on Galois-fields (so-called binary curves)
ECC is quite the patent mine-field, so I haven't uploaded anything related to point compression, even though it's conceptually somewhat simple.
Basically, you exploit the fact that if you know the x-coordinate, there are only so many y-coordinates the point can have, if it lies on the curve.
For prime-field curves, it requires you calculate a modular square root.
For binary curves, you need to solve a quadratic equation to get the y-coordinate, so it's a bit more involved.
This page has great ECC resources, and pseudo-code for some field-operations:
http://point-at-infinity.org/ecc/
I think this answer on crypto.stackexchange has a pretty good explanation of the math involved:
https://crypto.stackexchange.com/questions/8914/ecdsa-compressed-public-key-point-back-to-uncompressed-public-key-point
There is a thorough description of formats under section 2.3 in this document:
http://www.secg.org/sec1-v2.pdf
And this blog is a really approachable introduction to ECC in general:
https://www.johannes-bauer.com/compsci/ecc/

That patent link states the patent application expires today.

Hi @trstovall and thanks for chiming in.
I read that too :) The problem is, that this is not the only patent on ECC - far from - it's just one of the better known ones. There are no public lists of all patents covering which ECC techniques, and there are not many sources of information on what is unencumbered.
Most ECC patents should expire around 2020-2021, from what I've read, but there are no guarantees. There is even an RFC dedicated to enlightenment on what is covered by patents and what isn't:
https://tools.ietf.org/html/rfc6090
Some patents are only valid in the US etc. etc. -- I don't know what to make of this.


Deployment

Deployment
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN a Prework Study Guide website,
WHEN I visit the website using the URL,
THEN I can access my website from any browser

https://github.com/MrAwesome2127/prework-study-guide
The issue is complete, and the website is created. Closing issue


amd or commonJs export

export throught global variable is old epoch.
could you provide an amd and commonJs export?

Hey! This repository seems to be no longer maintained. Please check out https://github.com/philplckthun/sprint fork which uses UMD :smile:

thanks


Need to state the precedence for include and exclude constraints on triggers

Hi,
For instance, I have the following yaml but the pipeline isn't being triggered when I check-in changes on the refactor/create-single-pipeline branch, obviously because the '*\*' exclude is overriding the include...
trigger:
  branches:
    include:
      - 'main'
      - 'develop'
      - 'refactor/create-single-pipeline'
    exclude:
      - '*/*'

It would be good if the page stated whether excludes override includes, or visa versa, or better still, have an option to state which order of perference I require, or use the relative position/order in the file.  In any case, it needs to be documented.
Thanks,
Jeremy

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 86285f72-9e28-da97-59bb-c29eb60f627d
Version Independent ID: 18d5a591-a7d3-c261-6bff-8808ae433f54
Content: Configure pipeline triggers - Azure Pipelines
Content Source: docs/pipelines/process/pipeline-triggers.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-process
GitHub Login: @steved0x
Microsoft Alias: sdanie

@pridmorej -- Jeremy, thank you for your thoughtful feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@steved0x -- Steve, please look into this issue.


Cannot access custom data

I am trying to pass custom data App\Review and App\Reviewer into the notification message.
However, it seems the email message cannot access those values, resulting in an error.  Both $review and $reviewer are null.
This is how the notification is set up. Where did I get it wrong?
<?php

namespace App\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Notification;

class ReviewDueSoon extends Notification
{
    use Queueable;

    public $review;
    public $reviewer;

    /**
     * Create a new notification instance.
     */
    public function __construct($review, $reviewer)
    {
        $this->review = $review;
        $this->reviewer = $reviewer;
        ...
    }

    public function via($notifiable)
    {
        return ['mail'];
    }

    public function toMail($notifiable)
    {
        $mailMessage = new MailMessage();
        ...

        // $this->review is null
        // $this->reviewer is null

        return $mailMessage;
    }

  }

I think I figure out the issue: notifyAt() only takes exactly parameter.
So, instead of:
$reviewer->notifyAt(new ReviewDueSoon($review, $reviewer), $when, $meta);

This should be used:
$reviewer->notifyAt(new ReviewDueSoon([
        "review" => $review,
        "reviewer" => $reviewer
    ]),
    $when,
    $meta
);

Then, in the notification constructor, we set up the notification like this:
    public function __construct($review, $params)
    {
        $this->review = $params["review"];
        $this->reviewer = $params["reviewer"];
        ...
    }

Thanks @hktang  !

Feel free to PR a fix to this issue, as it would be good to support multi params


Adding helper methods to cxx_gen for higher level code generators

I couldn't see this discussed before (?), sorry if it has been.
Would it be accepted to add some helper methods to cxx_gen (or elsewhere?) for higher level code generators ? (If such methods would be accepted I would be happy to help contribute code to do this.)
In CXX-Qt we have a few places where it'd be useful for reuse some of the code from CXX itself rather than our fragile reimplementation of things, and other code generators using CXX may find this useful (eg maybe autocxx?). Furthermore this prevents breakage if CXX changes what it generates or adds support for more types.
Some of the possible scenarios we have come across so far that would be useful (note that API are just examples and could be different).
Retrieving the C++ name for a Rust type
This is useful when we are generating extra C++ code for Rust CXX bridges and need to match the type names for parameters or return types.
For a given syn::Type or syn::ReturnType it would be useful if there was a cxx_gen method to retrieve the corresponding C++ type name.
Note that this would also have to consider any namespaces and cxx_name attributes from the bridge.
For example with the following extern C++ block
#[namespace = "my_namespace"]
unsafe extern "C++" {
  #[cxx_name = "B"]
  type A;
}

If A is used as a type within a method, we need to have generation code to go from A to ::my_namespace::B. (and any other conversions need to occur like for UniquePtr or function pointers etc).
Would something like the following in cxx_gen be accepted?
pub fn syn_type_to_cxx_type(ty: &syn::Type, cxx_name_map: &BTreeMap<String, String>, namespace_map: &BTreeMap<String, String>) -> String {
    // calculates ::my_namespace::B from
    //  - reads syn::Type A
    //  - cxx_name_map with A -> B
    //  - namespace_map of A -> my_namespace
}

There would also likely need to be a similar method for syn::ReturnType as Result<T> can be used just in returns from my understanding (?).
Determining if the type is unsafe for CXX bridges
If certain types are used in CXX bridge functions the method needs to be marked as unsafe, eg if you use *mut T. Being able to determine by asking CXX which of it's supported types are unsafe would be much better than having to chase any new types.
Would something like the following in cxx_gen be accepted?
pub fn syn_type_is_cxx_bridge_unsafe(ty: &syn::Type) -> bool {
   // determine if syn::Type is unsafe
}

Retrieving the C++ bridge method name for a Rust method
In more advanced cases we need to friend the CXX bridge method (this is so that we can call protected methods from a C++ base class from Rust) it would be good if there was a helper to provide us the method name.
For example with the following CXX block
#[namespace = "my_namespace"]
unsafe extern "C++" {
    #[cxx_name = "B"]
    type A;
    fn method(self: &A);
}

CXX generates internal method names such as my_namespace$cxxbridge1$B$method.
Would something like the following in cxx_gen be accepted?
pub fn method_to_cxx_bridge_function_name(method: &syn::Ident, self_ty: Option<&syn::Type>, cxx_name_map: &BTreeMap<String, String>, namespace_map: &BTreeMap<String, String>) -> String {
    // calculates my_namespace$cxxbridge1$B$method from
    //  - reads the method ident "method"
    //  - reads the syn::Type Some(&A)
    //  - cxx_name_map with A -> B
    //  - namespace_map of method -> my_namespace
}

I would prefer not to build things like this into the cxx-gen crate for now. I'd recommend putting the things you'd like to see added into a separate crate, and if that crate ends up being useful to any other code generator tools, I can look into which of the functionality to pick up in cxx-gen.

Sure :-) We just wanted to check if it was worth pushing anything into cxx_gen so it could be like a library to other code generators. As we add support for parts of these we'll see if it's worth splitting these into our own utils crate or something.


Most recent Lanscape.io checks have failed.

The most recent Lanscape.io checks have failed with "Failure reason: Unknown"
Maybe this is a dependency issue and related to the fails in the ReadTheDocs builds: https://github.com/blakeaw/PyBILT/issues/28

I'm removing Landscape checks which I have replaced with Codacy cheks, so this issue is no longer relevant.


Populate a field in an array (and place data in that field)

Steps to reproduce
This issue is very similar to #142.
I'm attempting to populate a field inside an array, with the data from the population replacing the field.
This is the data I'm working with:
Users
{ _id: '111', name: 'alice', age: '19' }
{ _id: '222', name: 'bob', age: '18' }

Team
{
  _id: '555',
  name: 'Alpha',
  members: [
    { user: '111', role: 'leader' },
    { user: '222', role: 'member' }
  ]
}

And the code I have in an attempt to do this.
{
    childField: "_id",
    nameAs: "members.user",
    parentField: "members",
    select: (_, parent) => parent.members ? ({ _id: { $in: parent.members.map(member => member.user) } }) : {},
    service: "users",
}

Expected behavior
{
  _id: '555',
  name: 'Alpha',
  members: [
    {
      user: { _id: '111', name: 'alice', age = '19' },
      role: 'leader'
    },
    {
      user: { _id: '222', name: 'bob', age: '18' },
      role: 'member'
    }
  ]
}

Actual behavior
Adds this to the response:
"_include": [
        "members.user"
]

Alternative 'half-working' solution.
The following code:
{
    childField: "_id",
    nameAs: "users",
    parentField: "members",
    select: (_, parent) => parent.members ? ({ _id: { $in: parent.members.map(member => member.user) } }) : {},
    service: "users",
}

produces:
{
  _id: '555',
  name: 'Alpha',
  members: [
    { user: '111', role: 'leader' },
    { user: '222', role: 'member' }
  ],
  users: [
    { user: '111', role: 'leader' },
    { user: '222', role: 'member' }
   ]
}

where I do have all the data I need but not formatted in a way that I want (the user models being placed into the user field in the members list).

The way to address your needs is to have a preliminary hook that extracts the foreign ids into a format acceptable to populate.
Iterating through an array of objects is difficult to implement. And where does it end? Those objects may themselves contain arrays of objects.
You can submit a PR that fully addresses this issue if you're interested in to doing so.

As a quick workaround, I changed the nameAsField to user, which created a list of users in the root of the object, and then created a second hook which moved them to their correct place.
When I'm done with prototyping the current project I'm working on, I'll come back and create a more permanent and better solution.

The new fastJoin hook can handle this scenario, as well as others populate can't.
It also makes roughly 10% of the service calls that populate needs, i.e. 2 calls instead of 20.
https://feathers-plus.github.io/v1/feathers-hooks-common/index.html#fastjoin
https://feathers-plus.github.io/v1/feathers-hooks-common/guide.html#fastJoin


poor spacing in fractions

$\alpha _ e = \frac{3K _ e I _ \text{max}}{2 J}$
KaTeX:

MathJax:

Could be a font-size related issue like #41. At 4-5pt I get a similiar rendering to yours, above that, sometime the subscript touch the fraction bar. How small are you rendering? In what context are you using KaTeX?

Yes, this looks very much like #41. Again the vertical spacing appears correct at larger sizes, with “correct” meaning “as TeX does it”:

TeX in green, KaTeX in red. You see that the vertical spacing matches even though the horizontal one does not.
I'm closing this as a duplicate of #41.


backup and restore

Hello,
do you have backup and restore documentation ?
Is it the Kpow stateless o it has to be in synch with his kpow topics?
regards,
Leonardo

Hi @leonardotorresaltez the community edition is completely in-memory. All snapshotted metrics and telemetry disappear after the container has exited.
Our paid product features retention of metrics which are persisted to internal Kafka topics.


Add a method to get the hash only

Need to a new method to only grab the hash without loading the file to IPFS. This can be done by passing a hashonly parameter to the API
add?only-hash=true

There already is a hash only parameter:
https://github.com/ipfs/java-ipfs-api/blob/master/src/main/java/io/ipfs/api/IPFS.java#L81


Add feature or story level design review template to existing design review templates

Added feature or story level design review template based on prior work and incorporating some other ideas based on prior experience.
PR for task #131

Thanks for reviewing and merging it in @arsenvlad and @alkofahi


HNC: restructure object controller to allow explicit enqueuing

The Object controller doesn't allow for manual enqueing; either we can try to update every object of a certain type as a single, serial operation (SyncNamespace); if any one fails, we need to redo all of them. By contrast, the HierarchyConfig reconciler allows for individual enqueues and retries of individual failures.
Once we can enqueue a reconcile for a specific object, this unlocks many other improvements such as enqueuing a reconcile for objects that don't yet exist as a batch operation.

@rjbez17  Since the new object controller is implemented and tested, can we switch it as the default? We may want to keep the old controller code for the coming performance test. After the performance test is done, we can remove the code. WDYT?
I also manually tested the new object controller on Kind. It behaved just as we expected, e.g. manually-modified propagated objects got overwritten by the controller and got propagated to the newly-added children. In addition, everything worked fine when the controller is manually restarted.

Yeah I'm all for it. Since we are still pre-aplpha I don't see any reason to do a typical deprecation policy.

I would be interested in some metrics on the performance differences between the old vs new. I'm also happy to help with that testing, just let me know.

/close


Macports / Homebrew installation scripts

Original issue: https://charm.cs.illinois.edu/redmine/issues/44

Write a Macport script to ease installation for users of Mac OS X.

@ericjbohm How does the desire for ease of use expressed in this issue interact with the nature of our commercial licensing?

It creates a wrinkle of complexity in that a naive Macports/Homebrew approach masks licensing details.  A more conscientious approach would use some sort of click-through license text approach so the user is at least notionally aware of the restrictions on commercial use.

Some research would need to be done into whether Homebrew and MacPorts support that and would allow it. In Homebrew's case, we already are disqualified from brew install charm++ due to our licensing and would have to be brew cask install charm++ instead:
https://docs.brew.sh/Acceptable-Formulae#we-dont-like-binary-formulae

Our policy is that formulae in the core tap (homebrew/core) must be open-source with an OSI-approved license and either built from source or produce cross-platform binaries (e.g. Java, Mono). Binary-only formulae should go to homebrew/cask.

https://github.com/Homebrew/homebrew-cask/blob/master/CONTRIBUTING.md


Luarocks not working

Error
After installing the package luarocks seems to not work. Running the command gives the respons:
'"\lua"' is not recognized as an internal or external command,
operable program or batch file.
More Info on the Problem
Because the shim only calls the luarocks.bat I checked that file out and it uses a variable %LUA_DEV%.  So I solved the problem by adding a envrionment variable to my user called "LUA_DEV" with the path to  ../scoop/apps/lua/current. This solves the path problem but is a bit of a hack and when trying to install anything using the commads it seems to fail the compile phase.
Not sure if this can be fixed from scoop side or needs to be fixed on the luaforwindows repository. Because from what I can see this variable is set on line 515 in LuaForWindows.iss in that repo. I have not tested installing it directly from repository so not really sure what the root cause of the problem is

I'm having the same problem here. :cry:

@mrtnpwn Doesn't it fixed yet? https://github.com/lukesampson/scoop/pull/2849

Please check env vars after install lua-for-windows, in my pc, it works.

Just restart your pc. The environment variable "LUA_DEV" need the take effect after log out.

I am not sure about the status of this bug anymore. I tried using it for a game-jam and then forgot about it. Anyone who have tried the suggested solutions and can verify it works?

@Fronkan It works. you can close it.


Roadmap: scenario diff investigation

Re-enabling some old code that could be used as the basis of a scenario diff tool gives a visualization similar to:

Added a hacky draft PR which shows the changes needed to enable something like the sceenshot above.


Login Security is causing the server to crash in Paper 1.17.1

The plugin is causing the server to crash
It kicks the player out of the server and when they try to join it says player already online
Logs:
https://mclo.gs/Y4AIE0s

Are you sure this is related to LoginSecurity? it doesnt look like it in the logs.
Did you try removing LoginSecurity temporarily (while whitelisted, or locally) to see if the issue still occurs?

All im seeing is LoginSecurity kicking playeds because it thinks they are already online.
This is a symptom of the lag/crash, not the cause.


iOS and Android peripheral manager exposes other platform services

I'm using BLE peripheral manager to expose a single service and advertise peripheral under my own custom name in Android and iOS apps. Also making sure to clear existing services before starting BLE advertise.
However when I connect to that BLE peripheral, I see multiple services, including battery service, device info, current time, some audio stuff, etc.
That is triggering some unwanted side effects, such as my peripheral is recognized as an audio device by the car audio, and other things.
This is also reproducible with the bluetooth_low_energy own example app when peripheral is started.

These services are added by system. I don't think there is a way to remove them.

These services are added by system. I don't think there is a way to remove them.
See the discussion in this doc

I was hoping there is a way to have a separate instance of peripheral manager independent from the one used by the system or other apps.

I was hoping there is a way to have a separate instance of peripheral manager independent from the one used by the system or other apps.
E.g. multi-advertising on Android. Re: BluetoothAdapter#isMultipleAdvertisementSupported(), AdvertisingSet, etc.
https://source.android.com/docs/core/connect/bluetooth/ble_advertising

I don't think there is a way to do that. The advertisement has no effect with GATT services, we can't modify system services through multi-advertising.


Please upload documentation its cool

Deploy Tutorial ?

I am currently updating the docs, See them in wiki section


Create relationships as an admin in edit item

This PR adds the ability to add new relationships from the edit item page as an admin. It builds on #541, #540, #539, #531 and #530
In the edit relationships tab, an add button was added next to each relationship type header that will open a modal similar to the one used in the submission that allows you to search for related entities of that type.
Newly added relationships will only be added after the user submits the edit relationships form, consistent with the way that page already worked.
Note: the PR is still a work in progress.

Could you help me understand why I have 3 types of relations (my model is defined in the comment above - https://github.com/DSpace/dspace-angular/pull/545#discussion_r459530388)?

I forgot to refer that I was editing a Project.


[XMLFeedLinkExtractor] Urls splitted

The characters() method of the FeedHandler is called several times (due to the behaviour of the Sax Parser, see https://stackoverflow.com/questions/13336140/sax-parsing-and-special-characters?noredirect=1&lq=1 and https://stackoverflow.com/questions/4567636/java-sax-parser-split-calls-to-characters?noredirect=1&lq=1), causes splitting of the extracted Url.
We're working on a pull request.

Version 2.8.1 was released with that fix.


Use BoundedBTreeSet for SignedSubmissions instead of Vec

Eventually, once it works, this change should improve overall performance.
However, in the meantime, the trait bounds aren't playing nicely, and
this is turning into too much of a pain to handle right now as part of
/#7910. We can take care of it later.

how was sorting done before?

Insertion-sort, essentially, using a highly idiosyncratic method.
https://github.com/paritytech/substrate/blob/88f856a2d05ac4b04b582d56207a3646ee90c8ea/frame/election-provider-multi-phase/src/signed.rs#L183-L199
An acknowledged consequence of this PR is that we'll end up using a total ordering by score instead. I don't think that this will really hurt us or change the incentives, though it's likely to change the actual winner of at least some elections.

As of 35277ea, only all_in_one_signed_submission_scenario still fails. I strongly suspect that this is related to the difference between Ord and the previously-used insertion sort for SignedSubmission, though I haven't yet proved it.
This is now the time to ask whether we really truly need to use that insertion sort, or whether we can accept that the top T::SignedMaxSubmissions are processed in true score order. It saves us some work if we can process them in the total ordering, but we can work around it if it's essential. In particular, would welcome @kianenigma 's thoughts.

It is fine to sort them in total order.
The importance of  ImprovementThreshold is to prevent us from doing a full feasibility check for a solution that is a miniscule better. In the current scheme, we evaluate 1 signed solution anyhow, so total order is fine.
Note that we still need to keen the threshold_ord for signed submissions competition.

Removing the polkadot PR label because this is merging into a different branch, which already takes care of the polkadot PR for us.

Sorry for the spam, all; accidentally merged in the wrong upstream and codeowners automatically tagged you.

@rphmeier The main impetus for this change comes from https://github.com/paritytech/substrate/pull/7910#issuecomment-853711559. In short: you're right that trie lookups dominate the runtime cost for small items. However, in this case, each SignedSubmission is very large, so decoding cost is a much more significant part of the work. Additionally, by decoding only one at a time instead of the full set of submissions, we can get a much better cap on total memory use.

With my level of understanding it looks good, but some Rust related details are still lost on me, so I am going to try and come back.

@kianenigma I think this is now ready for another look.
As for the Drop impl, I've realized that we don't want that after all. The whole point of the storage overlays is to ensure that we can return early without writing to storage. If writes were triggered on Drop, then we'd push the overlays to storage each time a SignedSubmissions exited scope, regardless of whether the exit was a success or an error. I haven't proved it (or even really tried to), but I suspect that the test failures given the Drop impl may be due to updating storage when we're not supposed to.

but I suspect that the test failures given the Drop impl may be due to updating storage when we're not supposed to.

Yeah, this is the main potential footgun with Drop and I prefer keeping it explicit for now too.

@coriolinus feel free to merge into the base branch when ready. Then you can proceed with urging the audit team to start their work on the main PR, and get the second approve.
I will re-visit the base PR one last time once this is merged into it.


Review: Simplify the spec text

Not a Stage 3 concern and purely editorial, but one thing I noticed was that in https://github.com/tc39/intl-displaynames-v2/blob/main/displaynames.html#L160, you use [[<type>]] while the statement is inside the condition type = "language" so you should just use [[language]] to simplify the spec text.

Actually, this goes beyond the editorial problem I thought it was... the entire set of instructions at https://github.com/tc39/intl-displaynames-v2/blob/main/displaynames.html#L153-L160 go against the description of the internal slot at https://github.com/tc39/intl-displaynames-v2/blob/main/displaynames.html#L223 IIUC.
According to the latter, I get the impression that %DisplayNames%.[[LocaleData]].[[dialects]] is a Record that contains a single field "language" (why) which is another Record with the fields "dialectName"and "standardName" which are both Records with the style fields (all this is a bit poorly specified, let's fix that part up).
However, according to the spec, you are trying to access data from %DisplayNames%.[[LocaleData]].[[dialects]].[[<dialectHandling>]].[[language]].[[language]].[[<style>]] (while instead you should have accessed %DisplayNames%.[[LocaleData]].[[dialects]].[[language]].[[<dialectHandling>]].[[<style>]].
https://github.com/tc39/intl-displaynames-v2/blob/main/displaynames.html#L223
https://github.com/tc39/intl-displaynames-v2/blob/main/displaynames.html#L153-L160

From poorly specified, I meant that the list is out of order, which makes it really hard to follow along and there are a few grammatical mistakes here and there which could also be editorially fixed...

Thanks for the feedback. I will looks into this tonight.


"Not allowed to load local resource" for "file://" URI -- but only on npm start, not after it is packaged

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.0.3
Electron version
v21.2.2
Operating system
Windows 10
Last known working Electron Forge version
No response
Expected behavior
This should successfully display an image:

The renderer process tries to display an image using an  element
The url is formatted using url.pathToFileURL(path).toString() e.g. file:///C:/Users/Christopher/Pictures/screenshot.png
The path is of an image file which exists on the local fie system, e.g. C:\Users\Christopher\Pictures\screenshot.png

The application is based on (i.e. built using) the Webpack Plugin template.
Actual behavior
This works OK, when the application.exe is launched from the local file system -- after it has been packaged using npm run package.
But it fails when it's run using npm start which loads the script from a web server (for hot reloading).
The error message in the console is,

Not allowed to load local resource: file:///C:/Users/Christopher/Pictures/screenshot.png

Steps to reproduce
Try an element like this in the renderer when you load it using npm start (but change the path to that of a file which exists on your machine).

Additional information
I guess that the error message ("Not allowed to load local resource") is coming from Electron not from the web server.
It's a common error message -- but different error causes -- on the Electron site:

https://github.com/electron/electron/issues?q=+Not+allowed+to+load+local+resource

I found I can write a hack as follows which bypasses this problem:

convertPathToUrl encodes any path as a URL using a non-standard local:// scheme instead of the file:// scheme
registerFileProtocol is called once on app start to implement support for this non-standard local:// scheme

import { protocol } from 'electron';
import url from 'node:url';

// use this to determine whether to hack behaviour because it's running from web server instead of from file system
const isRunningFromWebServer = __dirname.includes(".webpack");

// not defined in https://en.wikipedia.org/wiki/List_of_URI_schemes, used as a hack when running from web server
const schemeName = "local";
const scheme = `${schemeName}://`;

export const convertPathToUrl: (path: string) => string = (path: string) => {
  return !isRunningFromWebServer ? url.pathToFileURL(path).toString() : `${scheme}${encodeURIComponent(path)}`;
};

export function registerFileProtocol() {
  if (!isRunningFromWebServer) return;
  protocol.registerFileProtocol(schemeName, (request, callback) => {
    // undo the mangling that was done in convertPathToUrl
    const path = decodeURIComponent(request.url.slice(scheme.length));
    try {
      return callback(path);
    } catch (error) {
      console.error(`ERROR: registerFileProtocol: Could not get file path: error: ${error}, path: ${path}`);
    }
  });
}

When it's run this causes a different error message i.e.:

Refused to load the image '<URL>' because it violates the following Content Security Policy directive: "default-src 'self' 'unsafe-inline' data:". Note that 'img-src' was not explicitly set, so 'default-src' is used as a fallback.

And that error message can be fixed by adding devContentSecurityPolicy to the WebpackPlugin in forge.config.ts to add support for the local: scheme to img-src:
  plugins: [
    new WebpackPlugin({
      devContentSecurityPolicy: `default-src 'self' 'unsafe-inline' data:; script-src 'self' 'unsafe-eval' 'unsafe-inline' data:; img-src local:`,
      mainConfig,
      renderer: {

So that's kind of OK: I can work with it.
But it's inconvenient for developers to discover this.
The reason I'm using Electron is to read files on the local file system, this was unexpectedly difficult.
So it would be nice if you could find a way to avoid this, in the default build-and-run.
It's bizarre that something (i.e. vanilla file:// URLs)  should work in the released/packaged build but not in the debug/start build.

So that's kind of OK: I can work with it.
But it's inconvenient for developers to discover this.
The reason I'm using Electron is to read files on the local file system, this was unexpectedly difficult.

Agreed this is somewhat inconvenient but it's a documentation issue only as this is functionally working as expected.  You shouldn't just be loading files ad-hoc from file:// as it's generically unsafe.  Using a custom protocol and specifically allowed files is much Better ™ anyway.

You shouldn't just be loading files ad-hoc from file:// as it's generically unsafe.

Isn't this, e.g. "loading files", like the main reason for using Electron at all instead of a web browser like Chrome?
I.e. to be a desktop application which CAN access local files?
For example I thought that, famously, VS Code is implemented with Electron: and IT sure can access local files.
And if were unsafe to use file:// then why I can do it in the packaged app but not during development?
IMO the restriction isn't by design, instead it's only a side-effect of using a web browser during development for hot reloading.
You say it's a documentation issue; does that mean you'll document it somewhere, or is it already documented somewhere?

file:// urls are not allowed when the files are loaded via http protocol.
This is the default case when you start the project with npm start.
You need to turn off hot reloading.

You shouldn't just be loading files ad-hoc from file:// as it's generically unsafe.

Isn't this, e.g. "loading files", like the main reason for using Electron at all instead of a web browser like Chrome? I.e. to be a desktop application which CAN access local files? For example I thought that, famously, VS Code is implemented with Electron: and IT sure can access local files.
And if were unsafe to use file:// then why I can do it in the packaged app but not during development? IMO the restriction isn't by design, instead it's only a side-effect of using an embedded web server during development for hot reloading.
You say it's a documentation issue; does that mean you'll document it somewhere, or is it already documented somewhere?

Hi, I'm experiencing this issue as well, I'm able to get the loading to work through disabling webSecurity but I would prefer not to do that. I am still learning the ropes and I dont fully understand the "hacky" solution provided. Would someone be able to provide some guidance on how to navigate this issue? it would be greatly appreciated.

You shouldn't just be loading files ad-hoc from file:// as it's generically unsafe.

Isn't this, e.g. "loading files", like the main reason for using Electron at all instead of a web browser like Chrome? I.e. to be a desktop application which CAN access local files? For example I thought that, famously, VS Code is implemented with Electron: and IT sure can access local files.
And if were unsafe to use file:// then why I can do it in the packaged app but not during development? IMO the restriction isn't by design, instead it's only a side-effect of using an embedded web server during development for hot reloading.
You say it's a documentation issue; does that mean you'll document it somewhere, or is it already documented somewhere?

Hi, I'm experiencing this issue as well, I'm able to get the loading to work through disabling webSecurity but I would prefer not to do that. I am still learning the ropes and I dont fully understand the "hacky" solution provided. Would someone be able to provide some guidance on how to navigate this issue? it would be greatly appreciated.

I have the same question too. Have you found a solution yet? It would be greatly appreciated.


catalyst: Update accuracy check

Recent catalyst versions changed the default formatting of the metrics, based on topk argument: https://github.com/catalyst-team/catalyst/pull/1330
Close #184

Codecov Report

Merging #185 (19f6df0) into master (47eb1e2) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
- Coverage   91.82%   91.62%   -0.21%
==========================================
  Files          17       17
  Lines         416      406      -10
==========================================
- Hits          382      372      -10
  Misses         34       34

Impacted Files
Coverage Δ

dvclive/lightning.py
94.28% <0.00%> (-0.59%)
:arrow_down:

dvclive/data/base.py
90.62% <0.00%> (-0.29%)
:arrow_down:

dvclive/live.py
97.82% <0.00%> (-0.10%)
:arrow_down:

dvclive/data/scalar.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


docs(@angular/cli): add i18n commands and story

As requested by @Brocco this is a first draft for an i18n story. Don't hesitate to ask changes if needed.
I didn't enter too much into the details because a lot of that is covered by the i18n cookbook on angular.io.
Also I removed the "deploy" part that @Brocco suggested because I don't see how the CLI can help with that right now (maybe once universal support is added).
If we need documentation on how to deploy an i18n app it should probably go into the cookbook, not the cli story.

Few suggestions:

Extraction, broken parameters link
Serve, Note - add link with a resource, perhaps https://angular.io/docs/ts/latest/cookbook/i18n.html#!#jit?
Build, last command has --base-href fr twice (--bh fr at the end?)

I don't think that the link is broken, we're not supposed to add .md at the end of links because the docs will be parsed and transformed for the wiki.
Good catch on the --bh though! Thanks


Node does not return to a healthy state after the database is temporarily unreachable

Do you want to request a feature or report a bug?
bug
What is the current behavior?
The graph-node is unable to successfully recover on its own from the being down for several minutes for, as an example, unexpected database maintenance.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
In order to reproduce the issue shutdown the database or otherwise make it unreachable to the graph-node for 10 minutes.
When the database comes back online the graph-node may continues to emit warnings from the ChainHeadUpdateListener (see below), and not receive new blocks. Though it does not seem to happen every time.
The bad state can be resolved by restarting the graph-node.
WARN Error receiving message, error: IO error: communication with the server has desynchronized due to an earlier IO error, channel: chain_head_updates, component: Store > ChainHeadUpdateListener > NotificationListener
What is the expected behavior?
Successfully reestablish a connection with the database when it comes back online.

As for the hosted service, we'll do the following:

Identify whether scaling the db up to multiple zones prevents connection errors like this during/after upgrades.
Implement recovering from connection errors in graph-node.
Add a test that verifies that this works when e.g. the database is stopped and restarted.
Build alerts into graph-node for when there are connection errors.

Just ran into this on Livepeer's graph-node. A side effect, at least in our use case: these messages are extremely verbose, filling up 16gb of Papertrail logs in a day 😃

Looking into this, it would be easy to reconnect to the database after a listener loses the connection but getting disconnected from the database there means that we might have lost notifications, and what we think the world looks like does not match reality. In particular, we might have missed a subgraph assignment event, and therefore might not index something we should be indexing.
I think it would be much more robust and safer to panic if we lose connection in a listener rather than in effect silently dropping notifications; that then would let whatever process supervisor is watching graph-node restart the node, and we already make sure we have the right idea of what to index etc. on restart. It also has the nice effect that the supervisor is in charge of how often to restart, when to give up etc.