Permission Bonus are not applying on 3.6.2

Permission Bonuses granted with the jobs.boost.xxx.xxx permission does not grant the bonuses. In previous version, 3.5.6 it works though.

the same to me,could u fix it?

This seems to be a issue with all boosts!
I had the problem that the restricted areas where loaded, but the multipliers weren't applied.
So I got the source-code and had a look on it with ecllipse.
I couldn't get out what exactly is the problem, but I think it has something todo with the way the plugin is calculating the boosts and rewards.

Should be fixed with latest builds


Handle an empty DVD drive on Windows gracefully

Closes #951
Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[N/A] I have added a changelog entry for my contribution
[N/A] I have added/updated documentation for all user-facing changes
[N/A] I have added/updated test cases

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed

/azp run libertem.libertem-data


Firebase Storage

How can I access Firebase Storage in Unity??

At the moment you can't use the Firebase Storage SDK from a Unity C# interface.

Hello. Any plan to release  Firebase Storage SDK for Unity soon?

Yes very soon.
On Wed, Dec 7, 2016 at 5:10 PM, kobechenyang<EMAIL_ADDRESS>wrote:

Hello. Any plan to release Firebase Storage SDK for Unity soon?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/firebase/quickstart-unity/issues/15#issuecomment-265623786,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFw1RCbcn-jZWG3rfitGWrXYbkYPHdK0ks5rF1jvgaJpZM4K89Et
.

Nice. Thank you!


How to get data?

when running the updated file named "01_SEC_API_10k10q_download.py", it raise FileNotFoundError, saying couldn't find the ' data\03_primary\price_data.parquet'
when running the updated file named "04_data_pipeline.py", it got a price file in pkl format, then raise FileNotFoundError, saying couldn't find the './data/filing_data.parquet'

A new version has just been updated. Please retry it.

A new version has just been updated. Please retry it.

01_SEC_API_10k1-q_download.py can work in the right way, but 01_Alpaca_News_API_download.py still couldn't work. It also need a file named price_data.parquet, which I have no idea to get it.
data = pl.read_parquet(os.path.join("data", "03_primary", "price_data.parquet"))

It is the same as the '1_SEC_API_10k1-q_download.py'  code from line 148 to line 159, which is used to get the target ticker as a list of strings.

It is the same as the '1_SEC_API_10k1-q_download.py' code from line 148 to line 159, which is used to get the target ticker as a list of strings.

No, the '1_SEC_API_10k1-q_download.py' only needs a list of strings, but '01_Alpaca_News_API_download.py' needs a list of tuple, which consist of date and str. The problem is I don't know the structure of data = pl.read_parquet(os.path.join("data", "03_primary", "price_data.parquet")), so I have no idea how to fix it.

How can I obtain the ‘price_data.parquet’ file?

hi!   for "01-Refinitiv_Real_Time_News_download.sql",   Where can I find the database for this SQL file, and is there a reference link available?

Refinitiv Real-Time News is a private dataset of Reuters articles. https://my.refinitiv.com/content/mytr/en/product/thomson-reuters-real-time-news.html

From: betterJia @.>
Sent: Saturday, June 1, 2024 2:17:16 PM
To: pipiku915/FinMem-LLM-StockTrading @.>
Cc: Yuechen Jiang @.>; Comment @.>
Subject: Re: [pipiku915/FinMem-LLM-StockTrading] How to get data? (Issue #11)
hi! for "01-Refinitiv_Real_Time_News_download.sql", Where can I find the database for this SQL file, and is there a reference link available?
—
Reply to this email directly, view it on GitHubhttps://github.com/pipiku915/FinMem-LLM-StockTrading/issues/11#issuecomment-2143317933, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANUUDRRJRF4Y5HDOEUU6IX3ZFFROZAVCNFSM6AAAAABFAT4TRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGMYTOOJTGM.
You are receiving this because you commented.Message ID: @.***>

You can obtain the price data via Yahoo Finance using the package “yfinance”.

From: betterJia @.>
Sent: Saturday, June 1, 2024 6:02:58 AM
To: pipiku915/FinMem-LLM-StockTrading @.>
Cc: Yuechen Jiang @.>; Comment @.>
Subject: Re: [pipiku915/FinMem-LLM-StockTrading] How to get data? (Issue #11)
How can I obtain the ‘price_data.parquet’ file?
—
Reply to this email directly, view it on GitHubhttps://github.com/pipiku915/FinMem-LLM-StockTrading/issues/11#issuecomment-2143035428, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANUUDRRXT6A74I3RKY4ABD3ZFDXRFAVCNFSM6AAAAABFAT4TRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGAZTKNBSHA.
You are receiving this because you commented.Message ID: @.***>

no activity, closed

The generated data for the Fake-Sample-Data is not clear, Do you mind elaborating on that?
A guide for instance on generating tsla.pkl


Microsoft Defender Exploit Guard inconsistent

There are inconsistencies between 6 device configuration profiles for MDM.
Below are the 6 profiles and their settings.  I've corrected it to disabled on my solution and the 6 conflicts resolve.
NCSC - Application Control - Disabled
NCSC - Attack Surface Reduction Rules - Disabled
NCSC - BitLocker - Disabled
NCSC - Defender AV Security Experience - Disabled
NCSC - Web Protections (EP) - enabled
NCSC - Xbox Services - Disabled

Thank you for this, can you please elaborate which specific configurations were in conflict? This will help us to investigate the issue.

Hi,
It's under this trailing path in "NCSC - Web Protections (EP)"
Microsoft Defender Exploit Guard > Network filtering > Network Protection.  This is Enabled whereas in the rest of the configuration profiles it's 'Disabled.
I hope this helps.

A change by Microsoft of the default values from "userDefined" to "notConfigured" caused all "userDefined" values to be changed to "disabled". Thanks to your issue we identified this and have now resoled it in https://github.com/ukncsc/Device-Security-Guidance-Configuration-Packs/commit/c71aaa32eaa0c9cd802910c3335f965a12923e2b

NCSC - Application Control
NSC - Credential Guard
NCSC - BitLocker
NCSC - Credential Guard
NCSC - Defender AV Security Experience
NCSC - Firewall - It should be Empty.
NCSC - Firewall Rules - It should be Empty.
NCSC - Local Security
NCSC - Web Protections (EP)
NCSC - Xbox Services
All of these Profiles shows me still the "Microsoft Defender Exploit Guard with "userDefined" value. It doesnt change the value to disabled or enabled. But for disappearing it should be updated to "notConfigured".


formula: set java.io.tmpdir

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

We have several formulae setting this (https://github.com/Homebrew/homebrew-core/search?q="-Djava.io.tmpdir"&unscoped_q="-Djava.io.tmpdir") and I think it's clear now that making it the default is better.

Thanks @Bo98!


Add Details Page

[ ] When the user selects a specific item, they can see the details page with its full description (skip the "Rotate image" button).

[ ] In the details page, the user can click the "Reserve" button (in the design you can see the "Configure" button - please replace it with the "Reserve" button).


Follow-ups for Gestalt-DI

Some things that could be improved on after merging #622:

[ ] All remaining uses of ContextWrapper should be removed
[ ] Make @Inject in NUI screens use the game context. This could be quite a bit of work.
[ ] Document the roles of the service registry classes (CoreService, GameConfigurationService, SolGameServiceRegistry etc.)
[ ] Try to find a better alternative to EventReceiverServiceRegistry. It uses the older getSubtypesOf method, which might be better replaced with newer automatic techniques now provided by gestalt-di.
[ ] If ModuleManager initialisation fails, try to find a way of exiting gracefully whilst displaying an error.

Try to find a better alternative to EventReceiverServiceRegistry. It uses the older getSubtypesOf method, which might be better replaced with newer automatic techniques now provided by gestalt-di.

It uses new getSubtypesOf :3
I change behaviour of similar methods from scanning or using reflections cache to use gestalt-di indexing(@Index annotation)
But yeah. I imagine that target usage should be something like injecting some collection with event handlers... But gestalt-di don't generate this code yet. Another throuble - backward injecting(injecting from child beancontexts) is not tested yet.. potentially even work.


there are an android version??

Any solutions??
Thanks

Hey @albertoMrFz, we do not plan to release Android supoort for EZLoadingActivity


Provide option for NetCDF-3 processing

Currently, Axiom is unable to process NetCDF version 3 files, as it attempts to read the data with HF5NetCDF, which only works on NetCDF version 4 files. To work around this issue for now files can be converted from NetCDF-3 to NetCDF-4.

This could maybe be handled by a preprocessor (if format is not v4 do convert-to-v3).

@JakeWeis, the preprocessor is triggered after the file has already been opened. As most of the standards out there require minimum nc4 it is perhaps a little too permissive to back port a lot of the code to nc3.
Files can be converted by the user from nc3 to nc4 using standards tools (i.e. cdo) prior to executing axiom fairly simply.
Alternatively, we can make the backend configurable, it is currently set to h5netcdf to speed, but it is possible to allow the user to select a different engine (i.e. netcdf4, script, cfgrib etc.) if they are prepared to take the performance hit.

Closing this issue for now. Please raise another issue if it looks like we need to make the backend configurable.


service capabilities

spring-cloud-commons portion of https://github.com/spring-cloud/spring-cloud-commons/issues/19

I'm going to close this in favor of #41.


Problems I had with tutorial (July 2023)

Hi!  Thank you so much for making this tutorial available for new users of Quarto. It was very helpful to have a defined project because I have never used GitHub Actions before.  I wanted to let you know what issues I hit in case you would like to incorporate solutions in the documentation (or want me to submit a PR...)

https://openscapes.github.io/quarto-website-tutorial/explore.html#setup-github-action describes make a new file quarto-publish.yml but that file is already in the repo. I didn't notice that, so got a cryptic GitHub error when I tried save a new file to that name.
https://openscapes.github.io/quarto-website-tutorial/explore.html#fork-to-your-account suggests following the instructions at https://github.com/thefaylab/lab-manual/wiki/Quick-steps-to-making-a-copy-of-the-lab-manual-&-publishing-it for setting up the repo. It had instructions for adding a new repository secret called EMAIL but doesn't say why. That was a little confusing because of the next problem I had.
My Quarto workflow would not run. I thought it was due to 2 (above) but in fact, my repository did not give read-write permission to workflows by default. The only had read-read so it could not write the new files.  To fix that, I went to Settings > Actions > General > Workflow Permissions (below the fold at the bottom of the page) and clicked the radio button for Read and write permissions
I reran the Render and Publish workflow manually a few times via the Actions tab. That might be mentioned as an option rather than editting files to trigger the Workflow.
For some reason, Settings > Pages was using the master branch instead of gh-pages. The symptom of that was that my Pages link was showing the README file from the main branch rather than the generated files from Render and Publish. Switching the branch and running the pages build and deployment workflow fixed that.
Finally, I was still getting the README showing on the link, even after I could see in the Action logs that the deployment had been successful using the correct branch and everything.  That's when I noticed at https://docs.github.com/en/pages/getting-started-with-github-pages/creating-a-github-pages-site#creating-your-site that there can be a delay of 10 minutes up to an hour. All I had to do was wait, and eventually the correct content appeared.

It might be worth adding a troubleshooting page in case other people get as mixed up as I did!  But the best part was that I knew there were solutions because you have created and used the tutorial. Thanks again.

Hi, thank you so much for this feedback! I'm so glad it's been useful and that you've found places to improve. We'll be able to work on this more next month (vacation and conference travel at the moment) and would be happy to collaborate with you on it; if you'd like to submit a PR that would be super and we can continue from there :). Thanks again for the issue, I really appreciate it!

Hi @eldobbins. I'm part of the Openscapes team. Thank you for opening this rich and helpful issue! I was testing and responding to #26 just now and realized that some answers are in your issue.
Do you still have the time and energy to submit a PR? Your issue lays things out very clearly and we would really appreciate that. If yes, please tag me as a reviewer. Happy to join you for a virtual screenshare if that helps. If not, I'm happy to make the updates.
Thank you!!

Hi;
I hope you can make the updates and that it isn’t extra trouble for you. I’ve gotten pulled to another project so it will take me a while to get back to this. I’ll get there eventually but it would be faster if you don’t wait for me.
Liz
Sent from my iPhone

On Aug 18, 2023, at 10:26 AM, Stefanie Butland @.***> wrote:

Hi @eldobbins. I'm part of the Openscapes team. Thank you for opening this rich and helpful issue! I was testing and responding to #26 just now and realized that some answers are in your issue.
Do you still have the time and energy to submit a PR? Your issue lays things out very clearly and we would really appreciate that. If yes, please tag me as a reviewer. Happy to join you for a virtual screenshare if that helps. If not, I'm happy to make the updates.
Thank you!!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

I'm happy to make those updates soon Liz. Thank you again for giving such a thorough and helpful description!

Thanks to @amyfrey for reporting a couple more things we need to clarify

In the
Quick steps to making a copy of the lab manual & publishing it tutorial, the first place where I run into a difference is step 3, setting the source branch to gh-pages. I do not have a gh-pages branch. I have tried creating one, as described in the quarto-website-tutorial, and then continuing with the faylab tutorial, but I still get the same issue, just the readme page is publishing.

Solution:

When I was forking I was copying the main branch only, and when I unselected that option and tried again the gh-pages branch was there, and now I can see the whole lab manual.

We've (finally) updated the tutorial and the issues you noted have been updated. Thank you again @eldobbins for being so thorough.


Region Tags in PS files broken

No longer able to collapse/fold on region tags after latest upgrade.

VSCode Version:1.25.1
OS Version:Windows 10

Steps to Reproduce:

open powershell file with vscode

The problem doesn't seem to be VSCode, but it's the PowerShell Extention. Just disabled the extention, reload, and try again, it should work like a charm.
Sending a bug report on their repo.

Yes, to be more specific it has been fixed and awaiting next release https://github.com/PowerShell/vscode-powershell/issues/1428
I found this after I posted this issue.


Bypass not working on intercelestial.com

Domain
intercelestial.com
Link
intercelestial.com
Version
latest
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
nothing else

Duplicate of #816


Update Hardhat

Upgrade to latest
Use Hardhat Toolbox (docs)

Reverting... not loading the contract. Will have to investigate further.


Task async

Closes #15
The client provides an async API

[x] CHANGELOG.md updated
[x] Rebased/mergeable
[x] Tests pass

Codecov Report

Merging #16 into master will decrease coverage by 0.18%.
The diff coverage is 0.43%.

@@            Coverage Diff            @@
##           master     #16      +/-   ##
=========================================
- Coverage    1.41%   1.22%   -0.19%
=========================================
  Files         268     268
  Lines       51706   59501    +7795
=========================================
  Hits          731     731
- Misses      50975   58770    +7795

Impacted Files
Coverage Δ

...t/InfluxDB.Client.Api/Service/DashboardsService.cs
0% <ø> (ø)
:arrow_up:

Client/Writes/Events.cs
0% <ø> (ø)
:arrow_up:

Client/WriteApi.cs
0% <ø> (ø)
:arrow_up:

Client/InfluxDB.Client.Api/Service/UsersService.cs
0% <ø> (ø)
:arrow_up:

Client/TelegrafsApi.cs
0% <ø> (ø)
:arrow_up:

Client/UsersApi.cs
0% <ø> (ø)
:arrow_up:

Client/BucketsApi.cs
0% <0%> (ø)
:arrow_up:

...nt/InfluxDB.Client.Api/Service/TelegrafsService.cs
0% <0%> (ø)
:arrow_up:

Client/TasksApi.cs
0% <0%> (ø)
:arrow_up:

Client/InfluxDB.Client.Api/Service/TasksService.cs
0% <0%> (ø)
:arrow_up:

... and 32 more

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 80ff7e2...646732e. Read the comment docs.


If opcache is enabled, VCR hooks may silently fail to work

I have a CI server running my phpunit tests, which use PHP-VCR.
On that machine, I have:
opcache.enable = 1
opcache.enable_cli = 0

I am finding that PHP-VCR is not successfully installing the Curl hook unless I add
ini_set('opcache.enable', 0);

to my tests.bootstrap.php.
(I am running phpunit from the command-line, so I can't work out why the opcache is running here, but it definitely is. I have added logging to the "CurlCodeTransform" class, and it does not see the Guzzle "CurlHandler.php" file unless I add ini_set('opcache.enable', 0).)
I think that PHP-VCR should warn if you use a "AbstractCodeTransform" based hook and opcache is enabled, as there is a good chance that things will then silently fail.
Perhaps the AbstractCodeTransform::enable() function should call "ini_set('opcache.enable', 0)"?

Can confirm, just got the same issue. Thanks for the workaround :+1:

This will be fixed in version 1.5.


CSRF tokens not sessioned when using scope and memory_sessions

Hi!
I've encountered some strange behavior when using Dream.memory_sessions as middleware within a Dream.scope. Forms that contain injected CSRF tokens via Dream.csrf_tag do not seem to store these tokens in the sessions, per the logger:
dream.csrf  WARN REQ 2 CSRF token not for this session

Confusingly, all works as expected if the Dream.memory_sessions middleware is replaced with Dream.cookie_sessions.
I've created a modified version of d-form to reproduce the bug:
let show_form ?message request =

%   begin match message with
%   | None -> ()
%   | Some message ->

You entered: <%s message %>!

%   end;

    <form method="POST" action="/">
      <%s! Dream.csrf_tag request %>
      <input name="message" autofocus>
    </form>

let () =
  Dream.run
  @@ Dream.logger
  @@ Dream.router [

        Dream.scope "/" [Dream.memory_sessions] [
            Dream.get  "/"
              (fun request ->
                Dream.html (show_form request));

            Dream.post "/"
              (fun request ->
                match%lwt Dream.form request with
                | `Ok ["message", message] ->
                  Dream.html (show_form ~message request)
                | _ ->
                  Dream.empty `Bad_Request);

        ];
  ]

I'm afraid I'm too new to the framework to tell whether this is a bug or mistake on my end. Hope this is enough info! Thanks!

There is a bug at how Dream handles middlewares in a scope. It creates its own instance of middlewares for each route. In our case, each of the routes:
 Dream.get "/"
 Dream.post "/"

will have their own memory_sessions middleware with their own hash table as a back-end storage. Therefore, sessions created on GET requests will never be found in session storage of POST requests. As a result, tokens created on GET and sent back with POST will never match to tokens created in session storage on server, because corresponding session will not be found in POST’s session storage.
I'll prepare PR with fix soon.
For now, you can use middleware for the entire server.
  Dream.run
  @@ Dream.logger
  @@ Dream.memory_sessions
  @@ Dream.router [


Nuke unneeded ClaimsPrincipal extensions

Also fix a bug with custom user id claim type with SecurityStampValidator
cc @divega

Addresses: https://github.com/aspnet/Identity/issues/663

:shipit:

1c26e8f5fa1b36febaca1f9a63b2a2f65fb11727

Premature removal, looks like we need to fix templates before we nuke these, as they still rely on this broken sugar method.

Replaced by https://github.com/aspnet/Identity/pull/701


Uncaught TypeError: Cannot read property 'callbacks' of null

uploader.js:281 Uncaught TypeError: Cannot read property 'callbacks' of null
I'm getting this error after following the steps for creating a custom file uploader template. It seem doesn't affect the call to Uploader.finished but it's still an error. I don't know if the error appears when the provided templates are used.
The code where I'm getting this bug is here: Flare

Hi, sorry for the lack of response. It looks good to me.. are you still having the issue?

yes, it doesn't affect anything as far as I can tell, but it's an error in the console

I wonder if it's because it's a Template method in a Meteor.startup block. If you were move the finished callback to:
Meteor.startup(function() {
    Uploader.finished = function(index, fileInfo, templateContext) {
      fileInfo.state = "new"
      JARSDB.insert(fileInfo);
    }
})

Does the issue still occur? I didn't have time to test.

I think you just need to pass a data context to your custom template
{{> customUpload callbacks=myCallbacks}}
and then define myCallbacks in your helper.

Is fixed in the latest release, no workaround necessary. Thanks for reporting this!

Hi tomitrescak, I have integrated custom template and passed attribute autoStart=true, but doesn't work.
{{> customUpload autoStart=true callbacks=myCallbacks formData=keyname}}
When I use the bootstrap version, it's working fine:
{{> upload_bootstrap autoStart=true callbacks=myCallbacks formData=keyname}}
I have used the customUpload that I found on your github page: https://github.com/tomitrescak/meteor-uploads


Added Maximum Subarray sum in python

Related Issue
Closes: #250 number that will be closed through this PR
Describe the changes you've made
Added Maximum Subarray sum in python
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
Python3
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have documented the code.
[x] New and existing unit tests pass locally with my changes.

Nice work @Kpraful :)

Nice work @Kpraful :)

Thanks, I will try to solve little fast from now on :)


Version of the software
0.1.26
Operating system
Windows 10 Pro
Browser / Native
Native
What happened?
After run job with 25 bots, can't connect to the imap server with BLOCK:ImapAutoConnect, wordlist is only 1 host
>>Trying imap.**.com on port 993...

>>Failed!

>>Failed to query https://live.mozillamessaging.com/autoconfig/v1.1/**.com

If I use BLOCK:ImapConnect

Can connect but too long...

>> Connected to imap.**.com on port 993. SSL/TLS: True

>> Keycheck (CheckCondition) <<

>>Matched key  EqualTo

>> Imap Login (ImapLogin) <<

>>Authenticated successfully

>> Keycheck (CheckCondition) <<

>>Matched key  EqualTo

>>BOT ENDED AFTER 22460 ms WITH STATUS: SUCCESS

Test with OB1 its ok and very fast

>>Succesfully connected to host imap.**.com on port 993. The server says:
>> * OK [CAPABILITY IMAP4 IMAP4rev1 ID AUTH=PLAIN AUTH=LOGIN AUTH=XOAUTH2 NAMESPACE] **Mail XMIMAP4Server ready

>> > A01 LOGIN<EMAIL_ADDRESS>"PASSWORD"\r\n
>> > A01 OK Success login ok

>> Saved Response in variable response.
>> <--- Executing Block KEY CHECK --->
>> ===== DEBUGGER ENDED AFTER 0.793 SECOND(S)

Try download new openbullet2 (both WebUI and native) and restart the computer still 20000ms....
Hope you understand, my english is not good :(
Relevant LoliCode if needed
No response

What do you mean "test with OB1"? OB1 does not have IMAP blocks, it just has a TCP block just like OB2, so maybe you should use the TCP block here as well?

tcpblock openbullet 2 response Operation canceled

Well if you don't tell me how to reproduce the issue how can I fix it? 🤣


React wrapper - no onprocessfile event fired

Hi! First of all, thank you very much for this library. Saved massive amount of time for us.
Here is an example https://codesandbox.io/s/x2zw6jzj24 (server code here: https://runkit.com/lytvynenko/5ab2b90f25eebc0012140268) , where "onprocessfile" event  isn't happening.

Thanks! Glad you like it!
Apparently, I somehow forgot to add the onprocessfile callback to the options object. Just did and published version 1.2.6

Woah! Blazing fast turnaround!  Keep it up!


Fix method naming errors

What is the purpose of the change
Fix method naming errors

pls fix ut

Codecov Report

Merging #7632 (c8843c6) into master (0ddc988) will decrease coverage by 0.47%.
The diff coverage is 60.37%.

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


Turning project Wiki into a community hub?

As proposed in issue #321 it might be an interesting idea to move a copy of the docs to the GitHub Wiki, as it tends to reduce the barrier to contributions for having a simplified interface and workflow.
Given that one of Sanic's biggest shortcomings right now is the lack of documentation regarding several portions of the source code, something that might increase contributions should not be so quickly frowned uppon.
While there is a strong argument for keeping docs checked in and versioned with code, revolving around the benefit of always having docs for the current version you're working on in the source even if you roll back, as of right now the existing documentation poorly reflects the capabilities of the software.
I think it would be more beneficial to the community, especially on such a rapidly changing project, to enable the documentation to evolve as rapidly as the source itself does. This is still pre 1.0 software, and I think there's more benefit in leveraging that we can still get a pass on having less strict versioning rules if it results in a more complete docset. Besides, nothing keeps us from syncing the docs into the repo when tagging new releases.

So this discussion hasn't seen any movement in some time now, but here's a new thought:
What if instead of hosting docs in the Wiki, we turned into a hub for community resources?
Right now we have 2 issues in the tracker concerning nothing but what projects use Sanic and where it's being used, namely #396 and #292. What if we forgot about migrating the docs, turned the wiki into a shrine of community resources instead?
Sounds like it could be useful to have a centralized listing of these things, and moving them away from the issue tracker. Curious to hear thoughts on this.

Sounds like a better idea than moving the docs according to comments. I don't see why wiki docs is such a big deal tbh, its a better idea than having them moved offsite and having "allowed" people to change, because they are the ones the most busy. Tons of legit projects on github use the wiki without issue. Even a simple disclaimer of unofficial docs could be added to slay those naysayers.

tagging https://github.com/channelcat/sanic/issues/699

Just read the Wiki and noticed it sorta became a hub for examples. Closing this...


Configuration options have no effect

Environment
How do you use Sentry?
Sentry SaaS (sentry.io)
Which SDK and version?
Currently installed:

symfony/monolog-bundle: 3.5.0
sentry-symfony: 3.5.4
symfony: 3.4.37

The application is Prestashop <IP_ADDRESS> https://github.com/PrestaShop/PrestaShop
Steps to Reproduce

I configured sentry in config_dev.yml

monolog:
    handlers:
        main:
            type:   stream
            path:   '%env(PS_LOG_OUTPUT)%'
            level:  debug
            channels: ["!event"]
        sentry:
            type: service
            id: Sentry\Monolog\Handler
sentry:
    dsn: "https://....ingest.sentry.io..."
    options:
        environment: dev
        release: !php/const SENTRY_RELEASE
        error_types: "E_ALL & ~E_USER_DEPRECATED"
        capture_silenced_errors: false

The options are applied:
$ php bin/console debug:config sentry
Current configuration for extension with alias "sentry"
sentry:
    dsn: '...ingest.sentry.io...'
    options:
        environment: dev
        release: '20210713154815'
        error_types: 'E_ALL & ~E_USER_DEPRECATED'
        capture_silenced_errors: false
        class_serializers: {  }
        in_app_include: {  }
        in_app_exclude:
            - (...)
        excluded_exceptions: {  }
        integrations: {  }
        prefixes:
            - (...)
        tags: {  }
    register_error_listener: true
    listener_priorities:
        request: 1
        sub_request: 1
        console: 1
        request_error: 128
        console_error: 128
        worker_error: 99
    monolog:
        error_handler:
            enabled: false
            level: DEBUG
            bubble: true
    messenger:
        enabled: false
        capture_soft_fails: true

But they have not the accounted effect. Silenced errors are still logged, as are user deprecations.
Expected Result
Despite capture_silenced_errors: false, Errors like filemtime(): stat failed are logged for if (!@filemtime($fileUri) || @filesize($fileUri) === 0) {. I would expect this error to go unnoticed.
Despite error_types: 'E_ALL & ~E_USER_DEPRECATED' , errors like Creating Doctrine\ORM\Mapping\UnderscoreNamingStrategy without making it number aware is deprecated and will be removed in Doctrine ORM 3.0. are logged for below code, where we have a user deprecation.
@trigger_error(
    'Creating ' . self::class . ' without making it number aware is deprecated and will be removed in Doctrine ORM 3.0.',
    E_USER_DEPRECATED
);

Actual Result
The settings are applied, but do not have the expected effect. Silenced errors are still logged, as are user deprecations.

This is probably due to something on the Monolog bundle side that our.
See https://github.com/symfony/monolog-bundle/pull/393 which is included in 3.7.0.
See also https://github.com/symfony/monolog-bundle/pull/394 which is in the same tag, and fixed another Sentry-related issue.

Unfortunately, I cannot upgrade to symfony/monolog-bundle 3.7, because PrestaShop is stuck with symfony 3.4. I tried many many configurations to no avail.
Is it possible, that symfony does not take into account changes to app/config/config_dev.yaml. Finally I suspect symfony does not read it's configuration on each request, and that all the tests I made were useless because of that. 😩

It surely may depend on that. Config is parsed and dumped in the cache, so if that doesn't get refreshed, you're right, your tests are invalid.

I still didn't find a solution. The next major version of Prestashop is using Symfony 4.4, but it will not be released before many months.
If ever I would advance a step with this problem, I will get this issue updated

I hope this issue will be gone on PS 8.0.
I'm not ready to deploy PS 8.0 for now, so I cannot work on this.

Closing for now, please let us know if you have more information once Prestashop 8.0 was released.


Improve reset CLI: add DecisionStartedTime as resetType and also provide …

…lastest runID as default

What changed?

The primitive reset command will use latest runID by default.
Provide one more predefined reset type "DecisionStartedTime" for reset and reset-batch commands.

Why?

It's annoying to require runID. Most of the time people just want to reset the current run or latest run.
It will be useful to allow reset by time as resetType for https://github.com/uber/cadence/issues/2237

How did you test it?
Local tested

Potential risks
No risk

Coverage decreased (-0.05%) to 59.526% when pulling<PHONE_NUMBER>4b2a45d3f7e4c16ec16c8227d83551 on qlong-reset-timerange into a9337097fb6410818ccf02be99faf3f2b27a3f14 on master.


Should hide the 'actions' part of the 'Duplicated intents recognized' trigger

Describe the bug

Version

Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots

Additional context

@a-b-r-o-w-n we need uischema for this component. Can you take a look?

@cwhitten should we try to get this into the sdk release?

@cwhitten should we try to get this into the sdk release?

we can try, but i doubt that they will take it.


when use etcd-3.4 with https ,confd has ERROR client: response is invalid json.

$ /data/apps/opt/confd/confd --version
confd 0.16.0 (Git SHA: 7217b0ca, Go Version: go1.10.2)
[user1@myserver confd]$ /data/apps/opt/confd/confd -config-file confd.toml
...
 /data/apps/opt/confd/confd[56888]: ERROR client: response is invalid json. The endpoint is probably not valid etcd cluster endpoint.
...

[user1@my-server confd]$ cat confd.toml
backend = "etcd"
#client_cakeys = "/data/apps/config/etcd/ssl/etcd-root-ca-all.pem"
client_cakeys = "/data/apps/config/etcd/ssl/etcd-root-ca.pem"
client_cert = "/data/apps/config/etcd/ssl/etcd-client-ca.pem"
client_key = "/data/apps/config/etcd/ssl/etcd-client-ca-key.pem"
confdir = "/data/apps/config/confd"
log-level = "debug"
interval = 30
nodes = [
  "https://<IP_ADDRESS>:2379",
]
noop = false
scheme = "https"
watch = true

etcd info
# etcdctl endpoint health
https://<IP_ADDRESS>:2379 is healthy: successfully committed proposal: took = 18.186985ms
https://<IP_ADDRESS>:2379 is healthy: successfully committed proposal: took = 19.953428ms
https://<IP_ADDRESS>:2379 is healthy: successfully committed proposal: took = 20.095768ms
# etcdctl member list
90bd3178dd0a7942, started, etcd3, https://<IP_ADDRESS>:2380, https://<IP_ADDRESS>:2379, false
f674e4102d2cb7c2, started, etcd1, https://<IP_ADDRESS>:2380, https://<IP_ADDRESS>:2379, false
fb3d93d2bb979bb1, started, etcd2, https://<IP_ADDRESS>:2380, https://<IP_ADDRESS>:2379, false

After setting backend to etcdv3, it returns to normal
backend = "etcdv3"


refactor(test): remove unnecessary reference

TODO

[x] Use cargo fmt
[ ] Add necessary tests
[ ] Add changelog entry
[ ] Update default config/theme in README (if applicable)
[ ] Update man page at lsd/doc/lsd.md (if applicable)

Codecov Report

Merging #699 (b1e9fa1) into master (b247650) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #699      +/-   ##
==========================================
+ Coverage   88.30%   88.40%   +0.09%
==========================================
  Files          41       41
  Lines        4531     4570      +39
==========================================
+ Hits         4001     4040      +39
  Misses        530      530

Impacted Files
Coverage Δ

src/display.rs
92.83% <100.00%> (+0.90%)
: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 b247650...b1e9fa1. Read the comment docs.


[Question] Typescript Support for React Proxy

Do you have support for typescript that is available for the React Proxy you have provided?
Thanks!

I found this issue looking for the same thing.
The docs suggest importing the wrapper using import PWAInstall from<EMAIL_ADDRESS>and there's no type definitions.
I see the react wrapper is declared in src/fallback/react.ts but it doesn't seem to be exported properly in index.ts. I also see the TypeScript source files are surprisingly included in the bundle but when I tried to import them directly using import PWAInstall from '@khmyznikov/pwa-install/src/fallback/react' the result is of type any.
@spencerdezartsmith I see you marked this as completed, did you find a solution?

@lnhrdt for react wrapper this types was generated https://github.com/khmyznikov/pwa-install/blob/main/dist/types/fallback/react.d.ts

@khmyznikov I suggest you try using this package on codesandbox with react+typescript template and see why it doesn't work. You either need to adapt package.json and define multiple exports like https://www.kravchyk.com/typescript-npm-package-json-exports/ or you need to move files in dist in such a way that react.js and react.d.ts are in the same directory so that something like import PWAInstall from '@khmyznikov/pwa-install/dist/react' works.

@anilanar thanks for hints. I have changed how the react wrapper is built, can you try to use latest commit like this in package.json: "@khmyznikov/pwa-install": "github:khmyznikov/pwa-install" and check how it works for you?
@lnhrdt @spencerdezartsmith also calling you to help verify the fix.

@anilanar should be doable. But, even "new" node16 security support ended 8 months ago. It's actually old 😅

@khmyznikov You are totally right but... Listen to me for a moment 😄
Most FE projects are single-page apps and use just a bundler like webpack or vite with ES modules. They don't use node.js runtime.
Node.js implemented ESMs with their own "flavor" that contradicted all existing code that used ES modules. All imports must contain the file extension like import foo from './foo.js'. Typescript, years later, added support for that with a big BUT: you need to import ts files with a js extension. So if you have foo.ts, it must be imported like import foo from './foo.js'. With TS 5, they tried to fix that by introducing moduleResolution: bundler. Now you can import either without extensions or with a ts extension, again a big BUT, you cannot use async imports! It doesn't end there and there are even more problems, but this should be sufficient for convincing 😃
Therefore, most FE code bases still use moduleResolution: node because there's no reason to use something else, at least for now. In future, if libraries start replacing main, module, types fields with exports, then yes, everybody will have to play along.

@anilanar can you check again for moduleResolution: node ? Works ok on my test project.

I think the react types finally fixed

I'll work on this area again this week, I'll give it a try 👍

@anilanar wrapper code shouldn't be needed.
import React, { useEffect, useRef, useState } from 'react';

import PWAInstall from '@khmyznikov/pwa-install/react-legacy';
import { PWAInstallElement } from '@khmyznikov/pwa-install';

function App() {
  const appName = 'My PWA';

  const [promptEvent, setPromptEvent] = useState(null);
  const pwaInstallRef = useRef<PWAInstallElement>(null);

  useEffect(() => {
      // @ts-ignore
    let lastPromptEvent = window.promptEvent;

    const intervalId = setInterval(() => {
        // @ts-ignore
      if (window.promptEvent !== lastPromptEvent) {
          // @ts-ignore
        lastPromptEvent = window.promptEvent;
          // @ts-ignore
        setPromptEvent(window.promptEvent); // Update state with new value
      }
    }, 100); // Check every 100ms

    // Cleanup interval on component unmount
    return () => {
      clearInterval(intervalId);
    };
  }, []);

  return (
    <div className="App">
      <header className="App-header">

        <button onClick={() => pwaInstallRef.current?.install()}>Install</button>
        <button onClick={() => pwaInstallRef.current?.hideDialog()}>Hide</button>
        <button onClick={() => pwaInstallRef.current?.showDialog(true)}>Show</button>

        <PWAInstall ref={pwaInstallRef} externalPromptEvent={promptEvent} name={appName} onPwaInstallAvailableEvent={(event) => console.log(event)}></PWAInstall>
      </header>
    </div>
  );
}

export default App;

Sample with externalPromptEvent in async mode.

@khmyznikov Sadly it doesn't work.
Here's a repro I initialized with tsc --init and npm init -y and just changed module to esnext and moduleResolution to node.

@anilanar not sure how to fix old module resolution. Here's a fully working sample: https://stackblitz.com/edit/vite-react-ts-2eeiak?file=src%2FApp.tsx

Also got this import issue, seems fine if you use React + TS project
but no luck if you still use JS (which is still many project do), see this minimal repro https://stackblitz.com/edit/vitejs-vite-l7d8d6?file=src%2FApp.jsx


Change the default bucket name for Argo Workflows - currently always mlpipeline

Add configuration value for:
https://github.com/canonical/kfp-operators/blob/60ae2d7cfc3d359f2e2d28c033733c92b36dbfc6/charms/kfp-ui/src/charm.py#L72
Customers can have mlpipeline already in their object storage.

@Barteus Should we put this name into config (maybe)?

Low priority.

This should be configurable because someone can have already mlpipeline folder in the object storage

This needs to be included in kfp-ui sidecar re-write.


Add Hotjar

Just looking to get Hotjar into our Airflow dashboard internally to get a better idea of how people use it.  Eventually I'll make this work via analytics.js but this was the MVP version just to get the data flowing.
(This isn't something we'd ever upstream but I'd just like to run it on our fork internally.)

@perryao Do you see any issues with merging + deploying this today?  I plan to get it in with my prod deploy this evening.

Should be fine. Only affects the web server so any potential issues are minimal.

@tedmiston were you waiting for me to merge or for other things to get worked out?

Not waiting on anything specific at this point besides hoping to get a workaround for the PythonOperator / pickling issue in place in docker-airflow first before changing anything else in staging / prod.


Update readme to explain why a "modified version"

Just a request:

Modified version of react-cl-audio-player

It would be helpful to include a bit of background on why you felt there needed to be a modified version, what you changed and why, etc.

Existing one had some limitations so I fixed them.


Updating readme from branch example

Updates
I updated the readme file. Please review and accept.

Thougts
I probably shouldn't , but okay. 👍


Connection Error (api project)

Hi
I am running a latest neo4j 4.x.x database in docker container.  I can connect from neo4j browser without any issues(create, delete, fetch etc... all works) with bolt driver but not able connect to from the node js grandstack api project.
NOTE: It was working fine with the older version and I have upgraded the database and downloaded the latest API code from this repo.
ERROR #1:  Please ensure that your database is listening on the correct host and port and that you have compatible encryption settings both on Neo4j server and driver. Note that the default encryption setting has changed in Neo4j 4.0. Caused by: connect EADDRNOTAVAIL <IP_ADDRESS>:7687
Another issue: After default installation of the GRAND stack(out of the box) on windows OS, npm start is giving me an error as below. I had to split the commands( in windows bash (build separately then copy..)  and run to make it to work (work around)
ERROR #2:
: Successfully compiled 6 files with Babel.
The system cannot find the path specified.
':' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: babel src --out-dir build && shx cp .env build 2>/dev/null || : && shx cp src/schema.graphql build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Shashikanth.r\AppData\Roaming\npm-cache_logs\2020-06-13T23_38_32_349Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: npm run build && node build/index.js "run"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Shashikanth.r\AppData\Roaming\npm-cache_logs\2020-06-13T23_38_32_427Z-debug.log

It was my mistake....After checking the DB logs in the containers, I found that APOC  plugins were missing. Its all working fine. I am closing this.  Thanks


Bots create forks occasionally

Bots Being Forked: We have identified forked groups issues. Hard to replicate for now. Thread here
Workaround: Removing and re-adding the bot fixes it

cc @rygine get with @fabriguespe to test on the latest node release to see if it's still happening.


Revert "Add version and checksum to Dropbox.app - 11.4.20"

See https://github.com/caskroom/homebrew-cask/pull/24975#issuecomment-250439429 for context.
Not merging yet, in case someone has a good point to not revert this.

👍 , as I think Dropbox is probably the only cask where auto_updates actually is true.


[GLUTEN-1500][VL] Integrate with Velox arbitration API

What changes were proposed in this pull request?
Integrate with Velox arbitration API

@zhztheplayer @zhouyuan This is the initial version of the integrate with Velox arbitration API.

@kerwin-zk there's a minor conflict on Gluten config, could you please do a rebase?
-yuan

@zhouyuan @zhztheplayer  I will add the new API to the result iterator instance and rebase together.

@zhouyuan done

@kerwin-zk @zhouyuan  I'm rebasing this.

@kerwin-zk @zhouyuan I'm rebasing this.

@zhztheplayer After #1732  is merged, some codes still need to be modified, because the API of MemoryPool has changed.

some codes need to be modified

I see. Will it be a big change to this patch? Or just impact a few of lines?

@kerwin-zk Before #1732 is landed, let me take a final look around this then I may directly update the PR based on your current work. Let me know if any issue, thanks :)

@kerwin-zk Before #1732 is landed, let me take a final look around this then I may directly update the PR based on your current work. Let me know if any issue.

@zhztheplayer There will be no more changes until #1732 lands, if you have any ideas, you can update directly.

Sounds good. Thanks for working on this by the way.

Run Gluten Clickhouse CI

@zhejiangxiaomai


add admin api

What this PR does
Adding an admin api for geneva actions
Jira: https://issues.redhat.com/browse/ARO-6238
Link to demo recording:
Special notes for your reviewer

Code looks okay overall but can you please move the internal/adminapi directory to internal/api/admin. (Generally the pattern is internal/api/{version} where {version} is a value that would be passed as an api-version URL parameter.)

For reference this is how it looks in "Classic" after a few years: https://github.com/Azure/ARO-RP/tree/master/pkg/api

Do we need a typespec definition for the admin API
No.


CodeAnalysis change

CodeAnalysis: move from *.ruleset to .editorconfig

Fixed: https://github.com/RegionOrebroLan/.NET-Web-Authentication-Extensions/commit/db12ce5839c27672e7dea29edf23c0890e1af0ef


Dropdown stopped opening when adding Toast component

Hello Sebastiano,
Thank you for putting this project together, it has helped me a lot.
I do have one issue however. When I add the Toast component, all my existing dropdown (Bootstrap 5) elements stop opening. I have been able to isolate it to the Toast component (not Toaster).
I forked your code and added a drop down and it works just fine, so it is something with my code.
Any ideas why this may be happening?
Thanks!
Shai

Hi Shai, could you please elaborate a bit more? Without additional details about your project setup, it's really hard to make a guess.

Yes, of course. Appreciate the help!
Before using your technique I was using Bootstrap (css/js).
agnular.json
...
],
           "styles": [
             "src/assets/styles/styles.scss",
             "node_modules/bootstrap/dist/css/bootstrap.min.css",
             "src/assets/styles/jquery.dataTables.min.css",
             "src/assets/styles/select.dataTables.css",
             "src/assets/styles/font-awesome.min.css"
           ],
           "scripts": [
             "node_modules/jquery/dist/jquery.js",
             "node_modules/bootstrap/dist/js/bootstrap.bundle.min.js",
             "node_modules/datatables.net/js/jquery.dataTables.js",
             "node_modules/jszip/dist/jszip.js",
             "node_modules/datatables.net-buttons/js/dataTables.buttons.js",
             "node_modules/datatables.net-buttons/js/buttons.colVis.js",
             "node_modules/datatables.net-buttons/js/buttons.flash.js",
             "node_modules/datatables.net-buttons/js/buttons.html5.js",
             "node_modules/datatables.net-buttons/js/buttons.print.js",
             "node_modules/datatables.net-select/js/dataTables.select.js"
           ]
         },
...

To get ToastComponent to work, I had to install types (npm @types/bootstrap). This allowed me to import
import {Toast} from 'bootstrap'
When I comment out the import (and corresponding references to the Toast type), my drop-downs start to open again as normal.

Thanks for sharing more information. I think the way Bootstrap is currently imported in your project is not ideal and this why it collides with my approach. Please take a look at my project's styles.scss where I import all the Bootstrap dependencies I need for my project.
The package @types/bootstrap additionally just provides type definitions for Bootstrap to have a convenient way to use them in ts code.

Appreciate the guidance! :)


when spring.net support EF6 and WebApi?

i think EF6 is better orm framework,can spring.net 2.0 support it?

Just fork the code and submit a pull-request; we'd love to have the contributions!

FWIW, there is already support for WebAPI as part of our support for ASP.NET MVC 4 and 5; are you thinking about support of some kind beyond what is already provided for there --?

Where can I find documentation about Spring and earlier versions of EF? I'm intersted too and I'm available to contribute if I have to override some classes from the Spring framework to my project. I simply need to understand how to correctly set up transactions and EF/Spring best practices. I'm struggling from weeks with two Spring-EF projects to work on

I may have inadvertently given the wrong impression.  I was offering that Spring.Net has support for WebAPI as part of its MVC4 and MVC5 support (the two versions of MVC that offer WebAPI).
Spring.Net has no support (at this time) for EF and so there's no documentation to which I can point you per se.  If you (or anyone else) is interested in working on such an effort, we'd be happy to assist with guidance, etc. where we can.
I'd suggest starting by conceptually "mapping" each concept from NH to its corollary in EF (e.g., NH session is like an EF DbContext, and so on).  Then attempt to mimic the classes, etc. in Spring.NHibernate that are necessary to work with those same concepts in EF.
Make sense --?

Oh I see. Actually I don't need full 1:1 mapping with NH features, because EF != NH and EF is a complete ORM per se.
I'm working hard in understanding how internally both Spring and EF work in order to come up with a working code-pattern for setting up transactions and DB interoperability (EF doesn't support AOP transactions out of the box, but works great with transactions).
I won't go in the deep today with my post but I believe that, at least according to my specific needs (transactions + DB interoperability) there should be no need to create a brand new Spring.Data.EF package. EF supports DB interop per se, as soon as you inject the connection string and watch out for pitfalls between MSSQL and MySQL.
Today I found that setting up [Transaction] with a TxScopeTransactionManager does nothing at all when used with EF when the DbContext is instantiated with new(), unless I implement a proper factory that injects the current connection/transaction into the DbContext (see https://msdn.microsoft.com/en-us/data/dn456843.aspx).
More in general, an integration between Spring and EF could offer these features I don't personally need in my project, but can be desirable by others:

Exception translating (EF has its own exceptions)
Exposing DbSet as injectable/autowireable object, because DbSet implements the DAO pattern per se

Just one note for sake of discussion, I hope I'll be implementing it in the next few weeks.
In order to autowire a [Transaction]al DbContext I wanted to do the following:

Classes that need DbContext during a transaction must call a GetDbContext() method that simply calls IApplicationContext's GetObject method
Define DbContext as an object with a factory and a create method
Build the factory class!

Basically the factory class has a reference to the IPlatformTransactionManager and when the factory method is called it checks for an existing transaction. If the transaction exists, grab the connection and return a DbContext initialized with that connection but not owning it (as of constructor). If no transaction is active, it will be my choice to either throw (like Propagation.MANDATORY) or return a fresh new DbContext owning the connection (like Propagation.REQUIRED).
What do you think about this design? It looks trivial to me but shall do the trick of transactions

Hi, another comment on Transactions and Entity Framework.
I initially set up TxScopeTransactionManager on my app. I thought I had to set up a proper DbContext factory to wire an existing DbConnection into it in order to run transactions fine.
I was wrong. Even issuing using(MyDbContext ctx = new MyDbContext()) in a method marked [Transaction] works, because I happened to find the following stack trace in my logs
2015-02-19 13:09:05,525 | 20 | ERROR | DiagnosticoSite.Controllers.BaseController | Failure on Transaction Scope rollback.
Spring.Transaction.TransactionSystemException: Failure on Transaction Scope rollback. ---> System.NullReferenceException: Riferimento a un oggetto non impostato su un'istanza di oggetto.
in MySql.Data.MySqlClient.NativeDriver.GetResult(Int32& affectedRow, Int64& insertedId)
in MySql.Data.MySqlClient.Driver.NextResult(Int32 statementId, Boolean force)
in MySql.Data.MySqlClient.MySqlDataReader.NextResult()
in MySql.Data.MySqlClient.MySqlDataReader.Close()
in MySql.Data.MySqlClient.MySqlCommand.ResetReader()
in MySql.Data.MySqlClient.MySqlCommand.ExecuteReader(CommandBehavior behavior)
in MySql.Data.MySqlClient.MySqlCommand.ExecuteNonQuery()
in MySql.Data.MySqlClient.MySqlTransaction.Rollback()
in MySql.Data.MySqlClient.MySqlTransactionScope.Rollback(SinglePhaseEnlistment singlePhaseEnlistment)
in MySql.Data.MySqlClient.MySqlPromotableTransaction.System.Transactions.IPromotableSinglePhaseNotification.Rollback(SinglePhaseEnlistment singlePhaseEnlistment)
in System.Transactions.DurableEnlistmentAborting.EnterState(InternalEnlistment enlistment)
in System.Transactions.TransactionStateAborted.EnterState(InternalTransaction tx)
in System.Transactions.Transaction.Rollback()
in System.Transactions.TransactionScope.InternalDispose()
in System.Transactions.TransactionScope.Dispose()
in Spring.Data.Core.TxScopeTransactionManager.DoRollback(DefaultTransactionStatus status)
--- Fine della traccia dello stack dell'eccezione interna ---
in Spring.Data.Core.TxScopeTransactionManager.DoRollback(DefaultTransactionStatus status)
in  Spring.Transaction.Support.AbstractPlatformTransactionManager.ProcessRollback(DefaultTransactionStatus status)
in Spring.Transaction.Interceptor.TransactionAspectSupport.CompleteTransactionAfterThrowing(TransactionInfo transactionInfo, Exception exception)
in Spring.Transaction.Interceptor.TransactionInterceptor.Invoke(IMethodInvocation invocation)

The exception is not important by itself, but simply means, and I hope it could be expanded in the future within Spring.Data documentation, that using TxScopeTransactionManager is sufficient to have Entity Framework 6 "autowire" itself to the existing transaction, if any. When an exception propagates, Spring.Data rollbacks the Spring transaction causing the underlying EF transaction to rollback. Could be reproduced in a sample test.
I have not experimented yet with composite/propagated/nested transactions (even because MySQL doesn't support nested transactions), but in response to @kindywu I could say that, at least for the transactions aspect, Spring 1.3, and supposedly 2.0, are perfectly compatible with EF 6.
Personally, I'm not yet interested in any other aspect of Spring/EF. Apart from exception translating, I'd ask @sbohlen if there could be other aspects to take into consideration for a possible Spring.Data.EF6 implementation

I have already created a project as Spring.Data.EF as likes Spring.Data.NHibernateX. It is working properly in sample project. I will commit it as soon as possible.

That would be great-- we're eager to review and incorporate it into the project!

I'm doing a rude experiment with closing this, like what stale bot does, but maybe more human as I will probably actually react to feedback. As this issue has been stale for so long, I'll close it. If it's still an issue you would like to pursue, we can definitely reopen.
With limited resources this is just something that we need to do. Thank you for your understanding.


Added link to homepage

Added link in the help page that links back to the main page

thx for the pr, but unfortunately have to reject because the link you added won't be clickable.  as you can see in screenshot below, the entire title is masked behind the fork on github banner element.

The link is in the help.html which doesn't have the banner element.

ah ok.  can you also do it for function.html as well?

Sure.

thx


Update to use Node 20

Closing, as I somehow failed to notice https://github.com/BedrockStreaming/pr-size-labeler/pull/9


chore: remove duplicate omnibus entries in gitignore

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge


Angular/typescript usage

Are there examples of how to use this with angular?

Not that I know of. Never used it in the browser.
On Mon, Dec 3, 2018 at 00:52 grosch<EMAIL_ADDRESS>wrote:

Are there examples of how to use this with angular?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/paixaop/node-sodium/issues/143, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAT9XkF_2ZqiE-QE1HgpBc14DmuPW0xIks5u1LwygaJpZM4Y969E
.

--
Do It Right or Don't Do It At All!

After doing the npm install sodium I did my ng build but I get a bunch of errors like so:

ERROR in ./node_modules/sodium/lib/keys/dh-key.js
Module not found: Error: Can't resolve '../../build/Release/sodium' in '/Users/scott/VoiceTalentBank/node_modules/sodium/lib/keys'

I'm doing this on macOS Mojave.  During the install I saw a bunch of warnings about deprecated methods being called but it did succeed.

The deprecations are from libsodium and should not cause issues. It seems that NG cannot find the binary module for sodium. Can you check the build directory under node sodium to see if it is there after node sodium install runs successfully?

I get the same errors. Any work around for using it with angular?

Could it be related to issues #144, #146, and #152?


0.64 stable new template fails when using use_frameworks!

Getting an error when I take the default template using "npx react-native init" and then changing Podfile to use "use_frameworks!". (removed flipper)
Cycle inside FBReactNativeSpec; building could produce unreliable results. This usually can be resolved by moving the shell script phase '[CP-User] Generate Specs' so that it runs before the build phase that depends on its outputs.
Cycle details:
→ Target 'FBReactNativeSpec' has link command with output '/Users/robertalmalak/Library/Developer/Xcode/DerivedData/testreact-derxgqraechlsxbpbivbctupykqo/Build/Products/Debug-iphonesimulator/FBReactNativeSpec/FBReactNativeSpec.framework/FBReactNativeSpec'
○ Target 'FBReactNativeSpec' has compile command with input '/Users/robertalmalak/testreact/testreact/ios/Pods/Target Support Files/FBReactNativeSpec/FBReactNativeSpec-dummy.m'
○ That command depends on command in Target 'FBReactNativeSpec': script phase “[CP-User] Generate Specs”
○ Target 'FBReactNativeSpec' has copy command from '/Users/robertalmalak/testreact/testreact/node_modules/react-native/React/FBReactNativeSpec/FBReactNativeSpec/FBReactNativeSpec.h' to '/Users/robertalmalak/Library/Developer/Xcode/DerivedData/testreact-derxgqraechlsxbpbivbctupykqo/Build/Products/Debug-iphonesimulator/FBReactNativeSpec/FBReactNativeSpec.framework/Headers/FBReactNativeSpec.h'
○ That command depends on command in Target 'FBReactNativeSpec': script phase “[CP-User] Generate Specs”

well I found a workaround, but I am not sure why this works or if it has any side effect but my app has started working:

found another workaround instead of skipping it like above you can also just change the build phase order where the Generate Specs is above the headers step:

I also noticed after researching the react code that the podspec file has a environment var "DISABLE_CODEGEN=1" which disables this build phase all together.
so to put it simply:
export DISABLE_CODEGEN="1"
pod install
You will notice that the build phase Generate Specs will be gone. I can't imagine anyone using that build step as of now as I am thinking its only used for the new architecture of react native... this makes me wonder why its not disabled by default.

I had issues also with clean installation and without use_frameworks!, iOS didn't built.
Don't know if related, but my issue was related to thing that I started ios build with yarn. It's related somehow to the fact that this Generate Specs is npm script and won't execute correctly if you run with yarn.  After I tried just npx react-native run-ios from manual -> build success.

Having the same issue here with an existing project and with a fresh template.

found another workaround instead of skipping it like above you can also just change the build phase order where the Generate Specs is above the headers step:

I just moved it above headers build phase and used patch-package for now, i see no reason why moving that script can't be the permanent fix

It's a bit of a hack, but it will fix the issue while waiting for an official fix from RN, add this to your Podfile:
# need to move 'Generate Specs' build_phase to be first for FBReactNativeSpec
post_install do |installer|
    installer.pods_project.targets.each do |target|
      if (target.name&.eql?('FBReactNativeSpec'))
        target.build_phases.each do |build_phase|
          if (build_phase.respond_to?(:name) && build_phase.name.eql?('[CP-User] Generate Specs'))
            target.build_phases.move(build_phase, 0)
          end
        end
      end
    end

@marcelkalveram Thanks you, the fix works!

Just a note for everyone that this is still an unresolved issue for iOS users with static_frameworks as far as I know?
It was linked to the 0.64.2 cherrypicks discussion but is listed as unresolved.
Unsure on status for 0.65 but I imagine it is not resolved there as I see no related PR linked or commit reference
I'm still using the Podfile hack above successfully when instructing others how to build

what is the status of this one?

This is related to react-native 0.64 which is so long ago that in the meantime building for macCatalyst broke complete, then was fixed, and then was fixed for use_frameworks ;-). I don't anticipate any motion on this one. I'm focused on react-native 0.70 cherry-picks, happy to see the CI matrix expanded to check use_frameworks to prevent regression in future (https://github.com/facebook/react-native/pulls?q=is%3Apr+matrix+sort%3Aupdated-desc ) and examining the 0.71 release when it comes out.

Irony (?), I just realized this is still live in my make-demo.sh script and just copy-pasted it in to my Podfile, but I'm not sure if it's strictly required or not. It was actually building correctly without it (react-native 0.69.5 + react-native-firebase@current), but I fear taking it out at the moment as I'm struggling with enough things that this one - which at worst should do no harm - seems okay to keep for a bit until I can test taking it out more thoroughly
Maybe others on this issue have more concrete info / experience

Just chipping in, I used your make-demo.sh script as reference to get a new project working with RNFB and managed to get it building without the above fix, react-native: 0.70.2 + @react-native-firebase/app: ^15.7.0

Thanks, happy to hear a success report on current stable, I'll remove this baggage from my script and see how it goes

Closing as all the use_frameworks! issues have been resolved in the latest stable of RN


Update notebook to use Featuretools Dask Implementation

After https://github.com/FeatureLabs/featuretools/pull/783 is merged, we can update the notebook to the improved api

Added a branch called add-dask-example that contains a notebook that shows how to generate a feature matrix using Dask EntitySets. Once Featuretools Issue 783 is complete, this notebook should be reviewed and updated as needed before being merged into master. Currently this notebook requires users to install Featuretools from the dask-entity branch to run.


ActionController::RoutingError (No route matches [POST] "/"):

Hi Zaidka,
I've got this message when trying to send a inform.
Started POST "/" for <IP_ADDRESS> at 2014-11-18 18:24:15 +0000
ActionController::RoutingError (No route matches [POST] "/"):
actionpack (4.1.4) lib/action_dispatch/middleware/debug_exceptions.rb:21:in call'   actionpack (4.1.4) lib/action_dispatch/middleware/show_exceptions.rb:30:incall'
railties (4.1.4) lib/rails/rack/logger.rb:38:in call_app'   railties (4.1.4) lib/rails/rack/logger.rb:20:inblock in call'
activesupport (4.1.4) lib/active_support/tagged_logging.rb:68:in block in tagged'   activesupport (4.1.4) lib/active_support/tagged_logging.rb:26:intagged'
activesupport (4.1.4) lib/active_support/tagged_logging.rb:68:in tagged'   railties (4.1.4) lib/rails/rack/logger.rb:20:incall'
actionpack (4.1.4) lib/action_dispatch/middleware/request_id.rb:21:in call'   rack (1.5.2) lib/rack/methodoverride.rb:21:incall'
rack (1.5.2) lib/rack/runtime.rb:17:in call'   activesupport (4.1.4) lib/active_support/cache/strategy/local_cache_middleware.rb:26:incall'
rack (1.5.2) lib/rack/lock.rb:17:in call'   actionpack (4.1.4) lib/action_dispatch/middleware/static.rb:64:incall'
rack (1.5.2) lib/rack/sendfile.rb:112:in call'   railties (4.1.4) lib/rails/engine.rb:514:incall'
railties (4.1.4) lib/rails/application.rb:144:in call'   rack (1.5.2) lib/rack/lock.rb:17:incall'
rack (1.5.2) lib/rack/content_length.rb:14:in call'   rack (1.5.2) lib/rack/handler/webrick.rb:60:inservice'
/usr/lib/ruby/1.9.1/webrick/httpserver.rb:138:in service'   /usr/lib/ruby/1.9.1/webrick/httpserver.rb:94:inrun'
/usr/lib/ruby/1.9.1/webrick/server.rb:191:in `block in start_thread'
Rendered /var/lib/gems/1.9.1/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/rescues/_trace.html.erb
(0.9ms)
Rendered /var/lib/gems/1.9.1/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/routes/_route.html.erb
(1.6ms)
Rendered /var/lib/gems/1.9.1/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/routes/_table.html.erb
(0.9ms)
Rendered /var/lib/gems/1.9.1/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/rescues/routing_error.h
tml.erb within rescues/layout (16.6ms)

Sorry, I am new about genieacs and got the similar issue.
Could you please guide me how to "point device to backend's cwmp component (port 7547)"?
error messages as below:
i$ sudo rails s -b <IP_ADDRESS>
=> Booting Puma
=> Rails 5.0.0 application starting in development on http://<IP_ADDRESS>:3000
=> Run rails server -h for more startup options
Puma starting in single mode...

Version 3.5.2 (ruby 2.3.4-p301), codename: Amateur Raccoon Rocketry
Min threads: 5, max threads: 5
Environment: development
Listening on tcp://<IP_ADDRESS>:3000
Use Ctrl-C to stop
Started POST "/" for <IP_ADDRESS> at 2017-08-04 13:42:41 +0800
ActiveRecord::SchemaMigration Load (0.1ms)  SELECT "schema_migrations".*


Please consider renaming your project, the name is misleading

Hi, Icecast maintainer here, just found out about your project.
I appreciate the work you've put into creating a streaming server on node.js.
Could you please consider renaming your project though, as it is not actually Icecast, but an implementation of the ICY streaming protocol as originally introduced in Shoutcast and as legacy protocol present in Icecast.
Icecast supports proper HTTP streaming of many formats, especially free formats using the Ogg container like Vorbis, Opus, Theora and WebM video, in addition ICY based streams are working too.
I think it would get you the recognition for your work and avoid us both the confusion of people looking for Icecast.
Thanks in advance.

Hi Thomas. I appreciate your concern here. Has the name caused any inconvenience/confusion in your circles up to this point?
Also, can you describe what you mean by "Icecast supports proper HTTP streaming of many formats". I believe that my library can handle arbitrary formats just fine, so I want to make sure I understand your concern on this part.

Icecast is a trademark of the Xiph.Org Foundation. You're dealing directly with streaming media, so confusion is inevitable.
Looking at the docs, perhaps something icyhttp would be a more descriptive name? It looks like you only handle the ICY method of interleaving metadata used on mp3 and aac streams.

In case you don't want to read the whole explanation below, which also contains a lot of historical context to explain the problem further:
tl;dr: The name is just wrong and you contribute to confusing people about that.

I appreciate your concern here. Has the name caused any inconvenience/confusion in your circles up to this point?

Well, actually your library has nothing to do with Icecast.
To understand better what I mean, let's go a bit back in history, when Shoutcast was created. They wrote a (not limited to) mp3 streaming server and were face with a problem: How to do mp3 metadata streamable?
One solution are ID3 Tags, they work fine but are not streamable, so they came up with their own solution, called ICY Metadata. As you might know, this works by specifying an interval where metadata are found and the client has to parse and remove them before playing the stream. That is exactly what you are implementing, and after all has nothing to do with Icecast. So if any name, related to a Streaming Server would fit, it would be node-shoutcast.
Shortly after Shoutcast, Icecast was born, as an open source alternative to Shoutcast, and the ICY Metadata thing was already quite popular and some clients already implemented it, thats's why Icecast supports those. And after all there is no other way to properly inject metadata into an mp3 stream, as it has no appropriated container format which supports that.

Also, can you describe what you mean by "Icecast supports proper HTTP streaming of many formats".

Some time has passed now, since we were blessed with mp3 and things have moved on. At least a bit. New great free alternatives to mp3 were developed, for example Vorbis, which has a proper container, Ogg. The great thing is, that those have proper support for Metadata.
Since these support much more metadata with proper timing and a lot of information, those are used by Icecast. Therefore Icecast will not use ICY Metadata for anything else than mp3 and no one should ever use those for anything else than mp3, since they are just very bad designed.
Givent hat your library only supports the old ICY Metadata format, it should be named node-icy or node-shoutcast and not node-icecast, as it is just misleading.
Yes, your library is able to parse the metadata out of mp3 Icecast (and Shoutcast) streams, but that's not a valid reason to name it node-icecast, as people could assume it is a node implementation of Icecast, which is not the case.

Thanks the the detailed explanation, makes a lot of sense to me now. Unfortunately, icy and shoutcast are already taken as package names on npm, so I think I'll rename to icy-parser. Does that work for you guys?

so I think I'll rename to icy-parser. Does that work for you guys?

Good name choice, I think!

Thanks!

Was able to nab the icy package name actually. Thanks for the nudge guys! I feel better knowing that I'm lessening any open source confusion :dancers:


make Daru::DataFrame#covariance faster

Roughly it becomes 2 times faster.
I confirmed that result of #coveriance doesn't change.
Should I write some benchmarks or tests?

No that's OK. Merging.


authz-v2: clean out-dated authorization implementation

This PR removes the out-dated implementation of the old AuthorizationPolicy CRD and the new approved API is here: https://github.com/istio/api/pull/918
The AuthorizationPolicy being removed in this PR is only for internal implementation experiment, it's never released to public and there is no users using or depending on it, so it should be safe to just remove them in the code and there is no need to follow deprecation policy.
istio/istio#12394
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTests-distroless-master
1e1c4948880da76b8a4fa3ec2ec9c44508cddfc9
link
/test e2e-simpleTests-distroless-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTests-distroless-master
1e1c4948880da76b8a4fa3ec2ec9c44508cddfc9
link
/test e2e-simpleTests-distroless-master

istio-lint-master
1e1c4948880da76b8a4fa3ec2ec9c44508cddfc9
link
/test istio-lint-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-lint-master
1e1c4948880da76b8a4fa3ec2ec9c44508cddfc9
link
/test istio-lint-master

e2e-simpleTests-distroless-master
50489d020d0678a8feb3282ffa7f1607269b3ba7
link
/test e2e-simpleTests-distroless-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTests-distroless-master
50489d020d0678a8feb3282ffa7f1607269b3ba7
link
/test e2e-simpleTests-distroless-master

istio-pilot-multicluster-e2e-master
50489d020d0678a8feb3282ffa7f1607269b3ba7
link
/test istio-pilot-multicluster-e2e-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram @costinm @nmittler @liminw Could you review this PR? We don't need the old AuthorizationPolicy CRD and it should be safe to just remove it in the code because it's never used by any users. Thank you.


modified single band obj plot

I have added self.k_points_plot in the file_readwrite.Properties_output().read_cry_bands() to collect the k points form the properties output necessary to plot
I have modified the possibility to plot a single band object in the plot.plot_cry_bands().
I removed the import properties in init.py

Added the possibility to plot multiple band obj on a single plot


public interface for Topics

We need to tease what we have to public (non-authenticated users). This involves making a topics/12345/preview page that looks a lot like topics/12345/summary.
Server Side

[ ] a new /server/views/topics/public.py with methods for public_topics, public_top_stories, public_top_words, public_sentences_over_time, public_topic_info, and top_media
[ ] each of those should return JSON like the existing methods, probably even calling the same _cached_XXX method that the existing authenticated ones do
[ ] bonus points: lets add a public_topic decorator that grabs the topic_id, fetches the info, and verifies the topic has public flag set to true

JS UI

[ ] Mimic the setup in src/topics/summary (which will feel dumb, but needs to happen)
[ ] Each of those containers should do an async fetch, and then render just the results (no links, no buttons, etc)
[ ] a new topics/public route that renders a PublicTopicList container

JS Plumbing

[ ] a new FETCH_PUBLIC_TOPICS action and fetchPublicTopics action generator that calls /api/public/topics
[ ] we will need new fetch actions for all of this, lets duplicate the approach with the topic summary, but call it things like FETCH_PUBLIC_TOPIC_TOP_MEDIA
[ ] lets store all this in state.topics.selected.selected by modifying the createAsyncReducer helper to accept either one action, or an array of actions by changing the switch to an if

wouldn't it be possible -as far as the UI duplication is concerned - to have a thin shell that is the public container, and tells all the UI components within to be disabled? that way we don't have such a proliferation of copied code... eg a view versus an edit

I believe we briefly discussed that we wouldn't use user_mediacloud_client(), but what do we do to indicate the user is anonymous? do we not have an API key in this case?  need more details...

ok, we aren't doing a topics/public route FYI - using current and checking if the user is logged in or not..

oh, I see. We want a topics/public route that does not requireAuth that redirects to a public shell like TopicsSummaryContainer

need a review of this...

notes:
remove: story totals, link map, about this
add: button, stats, header
fix: top media


No checking for errors when loading SBML

I notice there are no checks for errors when loading SBML. eg is one uses load('dfgdg'), it crashes processSBML.

details see commit (https://github.com/SunnyXu/SBMLDiagrams/commit/b305b57a69cec6ef978a5829caa52fda2c6b03f0).

Are yu sure you'r protecting the document = libsbml.readSBMLFromString(sbmlStr) method?
Do you have a test that confirms that is a random string is load the it doesn't crash?


Add Zig Language Server support?

I'm currently torn between making a separate package (LSP-Zig) and rolling that functionality into this package itself.
I'd love to have it be a one stop shop, but maybe some people want to have different packages for different purposes.
I'm open to hearing different opinions on this.

I'm leaning against this because I think the separation might actually be useful. Considering doing a LSP-Zig package instead.

there's also some question about how this compares to the rust-analyzer and RustEnhanced split


Validating Nullable Root Objects

Hi,
i have a problem with my validation rules. When my Validator contains only the rule to check if the instance is null, i'm getting a ValidationException as expected.
RuleFor(item => item).NotNull()
If my validator contains other rules beside that, i'm getting in NullReferenceException instead.
RuleFor(item => item).NotNull();
RuleFor(item => item.Prop).NotEmpty();

So why do we have a different behaviour here?

All rules you specify run against the object you provide. If you want some rules to not run you can add an '''.Unless()''' or a '''.When()`'' depending on your situation.
in your case
RuleFor(item => item.Prop).NotEmpty().Unless(item => item == null);

Just to clarify, FluentValidation is designed to process rules against properties, and it is a requirement that the root object is not null. In fact, newer versions of FluentValidation throw an explicit exception if you have a null instance (rather than a NullReferenceException)

You still can get into this issue with a null property. An easy way to deal with it is to make other rules dependent on the NotNull condition.
RuleFor(x => x.Property).NotNull().DependentRules(c => {     RuleFor(x => x.Property.Value).NotEmpty(); });

@diegofrata Your call to RuleFor inside the DependentRules block must be c.RuleFor not just RuleFor.

@JeremySkinner well spotted! I edited the post.


Allow users to customize search algorithm in DropdownMenu

Fixes #136735
This PR is to add a searchCallback to allow users to customize the search algorithm. This feature is used to fix b/305662376 which needs an exact match algorithm.
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

Added bottom app bar to navigate between lists and profile page


Specific Field Middleware Problem.

Hello,
let's say we have the following book type:
type Book {     title: String     author: String     pagecount: Int     cover: Cover   }
A problem I've noticed in this implementation is that if a middleware is specific to a field (the Book query in our case), then this middleware will be executed 5 times. First time in the Book query resolver, then 4 more times for every field resolver in the Book type.
Is there any way how I can limit the middleware to only be executed once (for the Book query resolver only)? Or did I miss someting?

Hey @7assenTlili 👋,
I suppose you've found the solution to your issue. Let me know if you need any help with it! 🙂


fix(bchjs): Adding configuration options

:tada: This PR is included in version 1.3.10 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Merge may 2022

The merge commits have a short description for each PR. 2-3 PRs were not merged because they were refactors related to Translation methods that would be overhauled shortly.

Looks like we need one more commit to fix CI.

utACK 62c0234e8437edd1a9f5844cd6d24f9f06ffd2e4

Fixed the CI and merged.


the slide/roll is broken

if you roll you can't dive and it doesn't set the sprite speed
and if you slide it just plays the spr_player_backslideland sprite and doesn't set the sprite speed

this is... true!!

It's a bug with scr_characterspr, replace it with the utmt decompiled code for now


File/Image field

add form fields for file/image handling.

@parsch The FileField needs some of your nifty styling.

The solution:
FileField requires an input value in the form { value, label, previewURL }, where

value (optional) is the value to be send when the form is submitted
label (optional) a string (e.g. a file name).
preivewURL (optional) is an URL of a thumbnail if applicable (can also be dataURL)

FileField further requires a prop onSelect which must be a function of the form (file, data) => inputValue, where

file is the File object of obtained from the FileList of the input element
data is the result of the FileReader.readAs<ReadMethod>() call, where <ReadMethod> is given by the prop readAs which is 'DataURL' by default (see below)
inputValue is either the input value described above or a promise resolving to such.

There are several possibilites and API could handle an upload:
a) use Base64
b) use File Upload
c) store file and use a reference (similar to FK)
In the case of a), the onSelect function simply returns the value as required by the backend. E.g.
onSelect: (file, dataURL) => ({ value: { filename: file.name, file: dataURL.split(',')[1] } })

In the case of b) or c), the onSelect function must execute the upload and return a promise. For example:
onSelect: (file, data) => {
     return axios.put('/upload/server', data, config).then(response => ({
         value: response.data.fileID,
         previewURL: response.data.thumbnailURL,
     }))
}

FURTHER PROPS:

onRemove is a function of the form ({ value, label, previewURL }) => ({value, label, previewURL}), which will be invoked when user clicks on the remove button. By default, the onRemove function is: () => ({ value: ''})
readAs is one of 'DataURL', 'Text', or 'ArrayBuffer'. The default value is 'DataURL'. See https://developer.mozilla.org/en/docs/Web/API/FileReader for more info about these methods


Proxies in the wild: Vue proxies

https://vuejs.org/guide/essentials/reactivity-fundamentals.html#reactive-proxy-vs-original
@caridy You suggested this.  Can you elaborate on Vue proxies, please?  I'm not that familiar with them.

In vue, any object returned by the data() method, transitively, it if is considered a POJO, or is one of the supported types (e.g.: Array, Map, Set, etc). it will automatically be wrapped by a Proxy whose proxy handler is a membrane implementation. This way, when values from those proxies are used to produce the final UI state, VUE can track their usage, and react if the value is updated at a later time.


add font-display: swap;  to the generated css

add font-display: swap;  to the generated css

It should be block for icons, which is the default value in most browsers anyways, so I don't see that any change is required (with swap browser is gonna quickly show fallback font which for custom icons wouldn't make any sense, ref).
Can't you add your own custom css with that property (right after fontello css) if you need to override it?


Updated Footer and jobs2.css files to make a good ux for dark mode in jobs section

Issues Identification
Closes: #994
Description
Summary
Summary
This pull request improves the user experience for dark mode in the jobs section by updating the Footer and jobs2.css files.
Details
-> Updated styles in jobs2.css to ensure better visibility in dark mode.
-> Enhanced the Footer component to align with the dark mode aesthetics.
-> Made minor adjustments for improved layout and user interface.
Types of Changes
Please check the boxes that apply

[✔ ] Bugfix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Please check the boxes that apply

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

Screenshots

dont create pr without assignment


No schema with key or ref draft-4

What version of Ajv are you using? Does the issue happen if you use the latest version?
I have 6.5.2 version.
Ajv options object

const ajv = new Ajv({
    schemaId: 'id', // draft-04 support requirment.
}),

JSON Schema

JSON Data
{
  "properties": {
    "foo": { "type": "string" },
    "bar": {
      "type": "number",
      "exclusiveMaximum": true,
      "maximum": 20,
      "exclusiveMaximum": true,
      "minimum": 1
    }
  }
}

Your code

const Ajv = require('ajv');
const JSONSchemaDraft4Definition = require('ajv/lib/refs/json-schema-draft-04.json');

var data = {
  "properties": {
    "foo": { "type": "string" },
    "bar": {
      "type": "number",
      "exclusiveMaximum": true,
      "maximum": 20,
      "exclusiveMaximum": true,
      "minimum": 1
    }
  }
};

const options = {schemaId: 'id'};
const ajv = new Ajv({options}).addMetaSchema(JSONSchemaDraft4Definition);
var validate = ajv.compile(data);

test(data);

function test(data) {
  var valid = validate(data);
  if (valid) console.log('Valid!');
  else console.log('Invalid: ' + ajv.errorsText(validate.errors));
}

Validation result, data AFTER validation, error messages
/home/user/workspace/myproject/node_modules/ajv/lib/ajv.js:93
    if (!v) throw new Error('no schema with key or ref "' + schemaKeyRef + '"');
            ^

Error: no schema with key or ref "http://json-schema.org/draft-04/schema#"
    at Ajv.validate (/home/snakuzzo/workspace/mockserver/node_modules/ajv/lib/ajv.js:93:19)
    at Ajv.validateSchema (/home/snakuzzo/workspace/mockserver/node_modules/ajv/lib/ajv.js:174:20)
    at Ajv._addSchema (/home/snakuzzo/workspace/mockserver/node_modules/ajv/lib/ajv.js:307:10)
    at Ajv.addSchema (/home/snakuzzo/workspace/mockserver/node_modules/ajv/lib/ajv.js:137:29)
    at Ajv.addMetaSchema (/home/snakuzzo/workspace/mockserver/node_modules/ajv/lib/ajv.js:152:8)
    at Object.<anonymous> (/home/snakuzzo/workspace/mockserver/prova.js:18:32)
    at Module._compile (internal/modules/cjs/loader.js:1151:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1171:10)
    at Module.load (internal/modules/cjs/loader.js:1000:32)
    at Function.Module._load (internal/modules/cjs/loader.js:899:14)

What results did you expect?
Maybe I did'nt undestand, but I expect to validate a schema with exclusiveMaximum/exclusiveMinimum and boolean values using draft 04.
Are you going to resolve the issue?

You have new Ajv({options}) instead of new Ajv(options) so schemaId option is not set.
Also not clear why you validate schema against itself (data is used as both schema and as data), but it's unrelated to the error message.

@epoberezkin I need to validate a spec file with exclusiveMinimum and exclusiveMaximum with boolean values and not number values...and (I don't konw if I'm wrong), I need to use draft-4.
Is it right ?
Using your suggestion...
Error: schema is invalid: data.properties['bar'].exclusiveMaximum should be number

You need to explicitly put $schema pointing to draft-4 meta schema to your schemas. By default (if $schema is absent) it uses draft-7 meta-schema, that does not allow booleans for these keywords.

If this is not possible (e.g. you don’t control your schemas and can’t change them) you can change default meta-schema - it is also covered in v6 release notes.

are you talking about this ?
https://github.com/ajv-validator/ajv#addmetaschemaarrayobjectobject-schema--string-key---ajv

Yes. Option meta changes the default but does not remove draft-7 meta-schema, that's why you see the warnings. You can either set schemaId to "auto" so it supports both the new and old IDs, or you can set meta to false and then add meta-schema with addMetaSchema and then set default by assigning draft-4 ID to this._opts.defaultMeta - it should be in the release notes.

Is there a way around this issue for command line validation?


Update download-git-repo

To latest version

+1 for this, which improves support for custom domains on bitbucket/gitlab, among other things

Closing as this has been included in #565, thanks.


Do not track balances for orders that are not open

The field is only of interest for open orders so we shouldn't waste
resources tracking balances of historical orders.
Test Plan
adjusted unit test

I created a relevant issue here: https://github.com/gnosis/gp-v2-services/issues/967
When I was touching the account_balances.rs code, it wasn't obvious to me that the subscription keys determining which balances to track were getting cleaned up correctly.
Just wanted to bring it up in case you noticed anything.


gi.ts rework & merge

This is the second attempt at merging the two codebases, it is still a draft but it is generating types with only a few failures to debug :partying_face:

I'm working on merging the newlib/ folder into the rest of the source.
I used newlib/ for the gi.ts sources so it was easier to merge the codebases side-by-side.

Yay I'll have a look when I find the time. Shall we delete branch 4.x for this and create a new one?

Yay I'll have a look when I find the time. Shall we delete branch 4.x for this and create a new one?

@JumpLink I think it'd be best to create a new branch off of 3.x called 4.x or next and we can target that branch with this MR.

@ewlsh I have now recreated the main branch for this because I have already referred to it in the documentation. I could start helping by removing the code for node-gtk, CommonJS and other exports as NPM package and also adjust the examples accordingly. How do we want to do this? Should I create a PR for gi-ts-rework or should I push directly to it, which do you prefer?

#148 solves the current merge conflicts


Improve polling for HostedFile state

Prevent caching outdated JSON when state changes multiple times in one
second.

Ensure error message is updated in file stage when it changes during reprocessing.q

Coverage decreased (-0.007%) to 99.06% when pulling 3fb35bdedd2b90f3cd04f9c5025e960e1d6c3b15 on tf:hosted-file-polling into 8dad96c487b16dd9783e99e16130711e706b9f25 on codevise:master.

Coverage decreased (-0.007%) to 99.06% when pulling 3fb35bdedd2b90f3cd04f9c5025e960e1d6c3b15 on tf:hosted-file-polling into 8dad96c487b16dd9783e99e16130711e706b9f25 on codevise:master.


Allow {"prefix", "source"} in from

This mirrors what we have in insert_all - it was surprising that you
could do:
insert_all({"prefix", "source"}, [%{x: 1])

But using the same in update_all or delete_all would fail:
update_all({"prefix", "source"}, set: [x: 0])

\cc @josevalim I'm opening this as a PR, as I'm not 100% convinced about this feature.

@michalmuskala update_all and delete_all receive a query and the prefix can be set there. The only exception is indeed insert_all. Therefore I wouldn't add this. :)

On one hand I agree with you, on the other hand setting a prefix on query is significantly more complex than setting the prefix for insert_all and update_all. The current way to do this for update is:
query = %{Ecto.Queryable.to_query("my_table") | prefix: "my_prefix"}
Repo.update_all(query, set: ...)

compared to: Repo.insert_all({"my_prefix", "my_table"}, [...])
I think aligning both uses is the right path following the rule of least surprise. We should unify the API to support the same thing everywhere.

Sometime ago I have proposed a prefix option to be given to all, insert_all, update_all, insert, etc. That would be my preferred approach instead of adding a special case to update_all/delete_all.


Literal types are broaded on object spread

TypeScript Version:  2.2.1
Code
// A *self-contained* demonstration of the problem follows...
interface Type {
    type: "type";
}

var type: Type;

type = { ... { type: "type" } }; // type 'string' is not assignable to type '"type"'

type = { ... { type: "type" as "type" } }; // no error

Expected behavior: I would expect that type broading not happen on object spread.
Actual behavior: Type broading happens on object spread.

The problem is that contextual typing doesn't follow through a spread, so when type: "type" requests a contextual type, it asks for the contextual type of { type: "type" }, which asks for the contextual type of ...{ type: "type" }, which returns undefined.
In a normal assignment like type = { type: "type" }, the contextual type of type = { type: "type" } is var type: Type, which provides the literal type "type".
This is definitely a bug, but it's pretty low priority because the only contextually typable things are literals, function expressions and object literals. The first two are not allowed (or non-sensical) in spreads and object literals can always be flattened into the parent object literal, which is also more efficient at runtime:
type = { ... { type: "type" } };

equivalent to
type = { type: "type" };

object literals can always be flattened into the parent object literal,

Yes, if not doing this (probably the repro was too minimal):
interface SomeFetchParameterBag {
  param1: string;
  param2: string;
  param3: "A" | "B"
}

var injectParam3: Boolean;
myFetch({ param1, param2, ...injectParam3 && { param3 } });

That repro still doesn't look complete. Here's one way (I also shortened the identifiers):
interface SomeFetchParameterBag {
  s: string;
  t: string;
  u: "A" | "B"
}
declare function myFetch(a: SomeFetchParameterBag): void;
function f(s: string, t: string, u: "A" | "B", inject: boolean) {
    myFetch({ s, t, ...inject && { u } });
}

But this doesn't fail because u already has a type that won't widen. If you use a literal instead of a parameter it does fail:
interface SomeFetchParameterBag {
  s: string;
  t: string;
  u: "A" | "B"
}
declare function myFetch(a: SomeFetchParameterBag): void;
function f(s: string, t: string, inject: boolean) {
    myFetch({ s, t, ...inject && { u: "A" } });
}

However, this is barely shorter than a top-level ternary:
    myFetch({ s, t, ...inject && { u: "A" } });
    myFetch({ s, t, u: inject ? "A" : undefined });

And I think the ternary is much less surprising on the first read.  Maybe if there are a lot of defaults like this, then it starts looking better. But in that case the default object should probably be declared on its own line.

If you use a literal instead of a parameter it does fail:

Ah yes, I incorrectly skipped that part.

And I think the ternary is much less surprising on the first read.

In some situation you won't want to define a property:
new URLSearchParams({ u: undefined }).toString()
// "u=undefined"

Fix is up at #15362