Add TinyURL to URL Shorteners

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been [squashed][squash-link] into a single commit

I've been using TinyURL as an API in one of my personal projects for a while now and I thought it would be useful to this project as well. There were not official documentation on how to use TinyURL as an API so I wrote it myself, They have documentation listed on their homepage on how to make a browser toolbar shortcut that shortens a URL so I figured since making API requests wasn't any different that I can make an official API document found here.

This is "unofficial", so I think we'll skip it.


Vue's unbind of custom directives provides unexpected 'el'

Version
2.5.21
Reproduction link
https://codepen.io/RuphiLau/pen/NebaQB
Steps to reproduce

Click 'HIDE' to make 'show' set to false
See the output of console, and it prints 'b' and 'c'

What is expected?
It should print 'a' and 'c'
What is actually happening?
It prints 'b' and 'c'

If I add a key for each p tag, the problem disappeared, but that's inconvenience for such a non-list situation

It is an expected behavior as I understand.
Actually keys are not bound to v-for. It offers hints each time Vue decides to reuse an existing element or not. This includes v-if.
Anyway, I think we do need to document this further. See https://github.com/vuejs/vuejs.org/issues/1248.


CC: Updated connection pooling guidance

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2869f8d00c1a82eb3f6ff1a47e3af67c5b38360f/
Edited pages:

v19.2/cockroachcloud-production-checklist.md

@lnhsingh Oh no worries! TFTR!

Marc reviewed the text on Slack. Merging now.


Fix Flatlist.js

Summary:
I was facing issue in my project with FlatList, fixed it by editing nodemodules.

Changelog:
[General] [Fixed] - Fixed Flat list constructor

Test Plan:

This issue has been addressed by https://github.com/facebook/react-native/commit/e33767a36daf7901cb46854b6612dd52b0dbdeec already.


Adding update on assistive text.

Per customer request adding a note here referencing company branding. Per current product logic for assistive text; If there is customized text available for the tenant (which is not currently supported), we show the customized assistive text in the password change page. If there isn't customized text available for the tenant, we check if the password type for the user if on-perm or not -If the user's password is "on-perm" or "on-perm & on cloud", we don't show any assistive text. Else we show the assistive text.
At this time we are not supporting configuring assistive text however, it is in the roadmap. Several customers have requested this via support tickets and the feature is being tracked in a work item. Last customer requested it documented here.

@willbrim : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@willbrim Please make the changes above and we need to include which tenants allow assistive text. Thanks!

Haven't heard from @willbrim and updates haven't been made. #please-close

@ajburnle sorry for the delay I had lost this thread. I can go ahead and make the edits you mentioned above. Do I need to submit another entirely? or can I do this here and have it reopened?


Port build.cmd to build.sh

I ported the majority of the functionality from build.cmd to build.sh to make it easier to build/run the various test suites on macOS/Linux.
The usual 'net40' projects build just fine; build.sh didn't accept any arguments before, and the ported script works the same way (and will build the same projects as before if you call it without any arguments).
Please note, this script merely allows different build configurations to be invoked like they can with build.cmd, it does not promise that every possible combination of arguments will work correctly; there are some configurations which still have issues on macOS/Linux and won't build successfully.
Finally, there are still a few edge cases that still need to be cleaned up in this script (but which I'll circle back to), e.g., the ability to specify categories of tests for NUnit to run.

Additional info: I tried running ./build.sh net40 test, and found some of the projects in the test suites have build errors unrelated to build.sh. In particular, once the TEST_NET40_FSHARP_SUITE environment variable is defined, the ILComparer project is included as part of the builds but fails with an unusual error:
/Users/jack/Code/visualfsharp/build-everything.proj (default targets) ->
(Build target) ->
/Users/jack/Code/visualfsharp/tests/fsharpqa/testenv/src/ILComparer/ILComparer.fsproj (Build) ->
/Users/jack/Code/visualfsharp/Proto/net40/bin/Microsoft.FSharp-proto.targets (CoreCompile target) ->

	FSC: error FS0224: Option requires parameter: --version:<string>

@jack-pappas Tricky PR...
Pros:

I really, really like the fact we get build net40 on Linux
I absolutely love that this gets close to giving us a build coreclr on Linux

Cons:

I greatly fear the duplication of pretty horrific logic... :-/   What can we do now to avoid this?

What can we do now to avoid this?

Does anyone know a scripting language that works x-plat?
Sorry could not resist ;-)

@forki Yeah, I know :) Was tempted to say "F#" or "FAKE" too :)

You can use powershell! 👎
Alternatively now the git bash is installed with Visual Studio out of the box you can actually call the .sh file from the .cmd file as well.
But to be honest please use F# if possible ;)

FAKE (or batsh?), really, who claim to be at ease with shell script compared to F#, people contributing here are F# devs, not sysops :)
@jack-pappas was saying to us that he had to search 30 web pages about shell and batch syntax to submit this PR.
I think we should take the PR to enable additional integration, but really, let's write a FAKE script, I had the first build step done as experiment and it worked with macOs/Windows + you could step in it from VS (ionide?).
https://github.com/smoothdeveloper/buildvisualfsharp/blob/master/fakebuild.fsx

@dsyme I too worry about the extra complexity of maintaining a second build script and keeping it in line with build.cmd. I think it's worth doing though as a short/medium-term solution, since having build.sh support all of the same build options makes it much easier to work on shoring up the testing situation on Linux/mac.
Longer term, I think it's worth looking into replacing these build scripts with FAKE, msbuild, or some other tool that'd let us have a single, easily-managed build script that works across all platforms.
@smoothdeveloper It's true I did have to look up a fair amount of shell syntax, but only because I was somewhat rusty with it and I wanted to ensure the script was fully POSIX-compliant (I could have taken some shortcuts and used some bashisms, but I think it's better to have the script be as portable as possible). In the end, porting this script wasn't terribly complicated -- I did it in a few hours while watching TV. I do agree though (as I said above) that a better long-term solution would be to use something else (e.g. FAKE) that'd let us have a simpler, more maintainable, single build script.

There are is even the possibility to just use FAKE as a lib when we need
some helpers. We do need to use it as build framework
Am 28.02.2017 11:18 schrieb "Don Syme"<EMAIL_ADDRESS>
Longer term, I think it's worth looking into replacing these build scripts
with FAKE, msbuild, or some other tool that'd let us have a single,
easily-managed build script that works across all platforms.
Another alternative is to use a plain .fsx script that builds on
src/scripts/scriptlib.fsx. What's being done is not that complex and
there is some sense in not taking a dependency on FAKE in this repo (partly
to reduce dependencies in general, partly to reduce what needs to be
learned)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/2497#issuecomment-283000129,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNHhfdT-w_9Lh-5Ds95BvIPk49lBQks5rg_SKgaJpZM4MMnUQ
.

Another alternative is to use a plain .fsx script that builds on src/scripts/scriptlib.fsx

This is an option, but why implement command line argument parsing, basic target support, why not leverage some of the ready made helpers in FAKE, the pretty output, etc.
I'm using FAKE but I'm not a power user of it, just using it as a better script interpreter (which does compilation + caching, the performance difference with bare FSI is significant) and comes with some  goodies to make our script simpler.
In this use-case, the code is identical to vanilla F# so it won't be much harder than if it was just FSI.

This is an option, but why implement command line argument parsing, basic target support, why not leverage some of the ready made helpers in FAKE, the pretty output, etc.

Because it avoids dependencies (keeps the repo more self-contained). That is important.

FAKE vNext will hopefully split up the huge "FAKE" colloss and that might make it simpler to argue with taking a dependency on one such smaller module of FAKE....


[1/3] Adds Databricks to UI

Describe your changes and why you are making these changes
Adds Databricks Integration to UI
Related issue number (if any)
eng-2087
Loom demo (if any)

Checklist before requesting a review

[ ] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[ ] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[ ] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

Do we nee to add another logo here besides the spark logo?

Looks like you are failing lint in the UI build github action.
Please go to aqueduct/ui/common and run:
npm run lint:fix to take care of these.

@hsubbaraj-spiral is this something that we want to get merged soon, or are we waiting for the rest of the spark implementation to be ready?
Also, do we need to add another icon here? Happy to provide design help and get you the icon you may need.

please get databricks icon from https://brand.databricks.com/

Hi @nfx! Thanks for all the feedback you provided, I'll make the according revisions soon!  Do you have any suggestions on how to go about doing this?


Add gh-pages CNAME support back

"To use your custom domain, whether you're setting up an apex domain or a subdomain, you need to remove the CNAME file from the other repository in which it's used." -> I've done that
" After you have done this, you will also need to push again to trigger a new page build." -> I couldn't do this but you can
TL;DR: Trigger a new build by pushing something onto the gh-pages branch, please

should be ok

It is, you could close now. Thanks !

Because I forgot to add a point in my DNS, I will need to wait for dns cache to reload. Hopefully, I've added a redirection on my webserver for that time. Sorry for the inconvenience.


Pushing a model isn't early alpha anymore

What are you trying to do?
Pushing a model isn't early alpha anymore
How should we solve this?
remove 'early alpha' in the import doc
What is the impact of not solving this?
folks will think its early alpha
Anything else?
No response

It's still an early feature being developed - will update this soon


Persist key by external provider

Since we have containerised the ASP.NET server and have changed the hosted environment to be Linux, we need to manage the key by our own.
2021-03-21T22:11:37.369651644Z warn: Microsoft.AspNetCore.Session.SessionMiddleware[7]
2021-03-21T22:11:37.369710245Z       Error unprotecting the session cookie.
2021-03-21T22:11:37.370422650Z System.Security.Cryptography.CryptographicException: The key {8bc1cea5-0d61-444a-be5e-9f867b14296e} was not found in the key ring.
2021-03-21T22:11:37.370436350Z    at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.UnprotectCore(Byte[] protectedData, Boolean allowOperationsOnRevokedKeys, UnprotectStatus& status)
2021-03-21T22:11:37.371048055Z    at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.DangerousUnprotect(Byte[] protectedData, Boolean ignoreRevocationErrors, Boolean& requiresMigration, Boolean& wasRevoked)
2021-03-21T22:11:37.371061055Z    at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.Unprotect(Byte[] protectedData)
2021-03-21T22:11:37.371080955Z    at Microsoft.AspNetCore.Session.CookieProtection.Unprotect(IDataProtector protector, String protectedText, ILogger logger)

Reference

https://docs.microsoft.com/en-us/aspnet/core/security/data-protection/configuration/overview?tabs=aspnetcore2x&view=aspnetcore-5.0#persisting-keys-when-hosting-in-a-docker-container
https://docs.microsoft.com/en-us/aspnet/core/security/data-protection/implementation/key-storage-providers?view=aspnetcore-5.0&tabs=visual-studio

We may use

Database, or
Redis

Closed via 20faebf


[WIP] Update dependencies

@golem131 There's a conflict since I just released 0.4.3. Can you resolve it and then we'll release 0.4.4? Sorry about that!

@golem131 never mind - github has a tool for that now!

@jeremyrsmith any plans to update to Finagle 7.0.0 series in short term?
Thanks

issued #61


.eh_frame: Writing to the unwind table spuriously errors

Most of the time writing to the table works, but occasionally I can see this error. Looks like it could be a race condition
Adding tables from /proc/189150/root/usr/lib64/libc.so.6 starting at 7f80e9d6d000
Adding tables from /proc/189150/root/usr/lib64/libm.so.6 starting at 7f80e9f55000
Adding tables from /proc/189150/root/usr/lib64/ld-linux-x86-64.so.2 starting at 7f80ea03a000
symbol count 38
level=error name=parca-agent ts=2022-09-28T11:17:58.470287082Z caller=cpu.go:291 msg="failed to check or update unwind tables" pid=189150 err="failed to update unwind table: update unwind tables: failed to update map unwind_tables: operation not permitted"
panic: fatal err

Interestingly, this tends to happen after stopping and starting Parca Agent many times, perhaps this issue is exacerbated by increased memory pressure?

I haven't seen this error since I reworked the implementation to get it ready to land in https://github.com/parca-dev/parca-agent/pull/948.
Closing the issue, let's re-open it if this happens again


User Application not found while trying to send message in Company Communicator

Dear team
I am getting error that user application cannot be found while trying to send message.
The Application is already uploaded via Manage Apps and is visible over there.
Also, while testing company Communicator locally with Author group the application does not get pinned although i have pinned it to myself manually.
Apprecaite your support on priority.
Regards
Nadeem

Hi @Nadeem21210 ,
Thanks for raising the query. Within the sending options page whether able to see the teams names under dropdown. If no, Please refer to question number 8 in troubleshooting guide to understand more on this issue, kindly let us know if it helped in resolving the issue

user

app is installed for everyone.
Attached is the screenshot for reference.

Please give us sometime we will get back to you soon.

Hi @Nadeem21210 ,
Please keep us posted if anything comes up otherwise we can make it to closure.


Prom select stats

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #956 (5811d48) into 2.x (863f751) will decrease coverage by 2.83%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x     #956      +/-   ##
============================================
- Coverage     97.94%   95.10%   -2.84%
+ Complexity     3126     3077      -49
============================================
  Files           299      301       +2
  Lines          7734     8237     +503
  Branches        499      612     +113
============================================
+ Hits           7575     7834     +259
- Misses          158      349     +191
- Partials          1       54      +53

Flag
Coverage Δ

query-workbench
62.76% <ø> (?)

sql-engine
97.91% <100.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...opensearch/sql/planner/logical/LogicalPlanDSL.java
100.00% <ø> (ø)

...java/org/opensearch/sql/storage/StorageEngine.java
100.00% <ø> (ø)

...ain/java/org/opensearch/sql/analysis/Analyzer.java
100.00% <100.00%> (ø)

...org/opensearch/sql/analysis/model/CatalogName.java
100.00% <100.00%> (ø)

...rch/sql/planner/physical/catalog/CatalogTable.java
100.00% <100.00%> (ø)

...sql/planner/physical/catalog/CatalogTableScan.java
100.00% <100.00%> (ø)

...l/planner/physical/catalog/CatalogTableSchema.java
100.00% <100.00%> (ø)

...ava/org/opensearch/sql/utils/SystemIndexUtils.java
100.00% <100.00%> (ø)

...ql/opensearch/storage/OpenSearchStorageEngine.java
100.00% <100.00%> (ø)

...rg/opensearch/sql/ppl/config/PPLServiceConfig.java
100.00% <100.00%> (ø)

... and 22 more

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


Need a way to disable logging fluent.conf content

Currently when fluentd starts up, it will print out the content of the config file it use. However, when using the s3 plugin (https://github.com/fluent/fluent-plugin-s3) or the kinesis plugin (https://github.com/awslabs/aws-fluent-plugin-kinesis), it will print out the aws_access_key and the aws_access_secret_key as a result even though i read them from environment variables.

Please use --suppress-config-dump option.
https://github.com/fluent/fluentd/blob/d3ae305b6e7521fafac6ad30c6b0a8763c363b65/lib/fluent/command/fluentd.rb#L106

In addition, latest fluentd and fluent-plugin-s3 combination doesn't print actual key values > https://github.com/fluent/fluent-plugin-s3/commit/a678d899fab871e97a86e9b621ef2d1a523ef17b

Thank you, I was looking at the documentation and didn't find the option :)

I've created a pull request for the kinesis plugin to do the same thing as your s3 plugin: https://github.com/awslabs/aws-fluent-plugin-kinesis/pull/21

Oops. I will add command line option article on docs.
And I recommend to use <system> in configuration file instead of command line option.
http://docs.fluentd.org/articles/config-file#4-set-system-wide-configuration-the-ldquosystemrdquo-directive


nixos-render-docs: init, use for some manual rendering to docbook

Description of changes
another step toward removing docbook. what was once mergeJSON and is now optionsToDocbook is a good start towards more capable renderers, so let's extend that base into a new tool and use it for both rendering the options docbook fragment and the manual.xml fragment containing all the module chapters.
this can (and will) be extended further: we could sort the manual chapters by their title instead of file system path (which would greatly improve navigability of the module chapters fragment), we could include non-module chapters and release notes (obviating he need for md-to-db.sh), we could add more output formats (like html, man, at some point perhaps even info and epub).
the foundation laid here might even be altered slightly to allow for nixos-render-docs plugins that target entirely different formats, or maybe just collect weird and wonderful information about the documentation.
rendering changes from this PR are minimal and mostly confined to what smartquotes does to the options fragment, though a few docbook quotes in module chapters got turned into non-docbook quotes because pandoc is a bit extra like that.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/any-consensus-on-documentation-generation-tool-for-nixpkgs-manual/15550/30


Housekeeping on CMakeLists.txt and pyproject.toml

We still had the installation of the transformations package hanging about in the CMake integration. I also did some housekeeping on the pyproject.toml in the same instance and this seems to have fixed #304. @awnawab please confirm when you can.

Python packaging is still as badly broken as ever, but it seems I may have found a stable configuration that yields the desired outcome.
In the process I discovered that tests that use f90wrap or f2py fail if they are being run directly using the pytest binary from a venv without activating the virtual environment (e.g., venv/bin/pytest --pyargs loki). To fix this, I piggy-backed a little utility which ensures that the python interpreter's parent directory is in the search path.


Option to have embed button on the left

This is a feature as of 10/12/2016, recommend the issue be closed?

@rauenzi Whoops! Thanks.


Why no concent page display when use authorization code flow to acquire access token

I expect to use APIs in 'APIs my organization uses', there's an API that does not require admin consent, I did not grant the admin consent.

when I acquire the access token for this API, no consent page displayed, is this normal?
Or all the APIs in 'APIs my organization uses' tab and in 'My APIs' tab, the user consent page will not appear when using OAuth2 authorization code flow to apply for an access token.
Please help to confirm, or please tell me when the user consent age will appear? Many thanks!

@moarychan Did you by chance grant consent to the whole tenant? Theres a button right above the API permissions that says "Grant Admin Consent for  Tenant". Have you consented to this permission previously with the same user?

@moarychan Did you by chance grant consent to the whole tenant? Theres a button right above the API permissions that says "Grant Admin Consent for  Tenant". Have you consented to this permission previously with the same user?

Not consented yet, I hope users can see the authorized page, which users can choose to agree, and then return the access token. I used permission 'teams', this is not required by Admin consent.

Not consented yet, I hope users can see the authorized page, which users can choose to agree, and then return the access token. I used permission 'teams', this is not required by Admin consent.

I read the 'teams' permission description, it means not required by Admin consent, then it can be consented by the user. Is that correct?

Yes I believe that is the case.

@moarychan Can you verify if the user you are using to test has consented to that scope before? You can go to the app registration on the portal, "API Permissions" tab, and click on "Enterprise application"

.
Here you can click on "User Consent" and see if the user has consented to this scope before. If so, that's why you're not being asked to consent.

Closing issue as I never heard back. Feel free to reopen if you continue having issues.


Dates range not resets

Steps to reproduce:

Select start date in the first month.
Select end date in the second month.
Rewind for two months next
Select start date in the first month.
Select end date in the second month.
Rewind for two months prev

Expected behavior: first range resets
Actual behavior: first range not resets
http://airbnb.io/react-dates/?selectedKind=DayPickerRangeController&selectedStory=default&full=0&down=1&left=1&panelRight=0&downPanel=kadirahq%2Fstorybook-addon-actions%2Factions-panel

Hiyo! I believe this is fixed now by #575 in v12.2.0
Thanks for the issue (and the PR)!


What is the name of the icon?
Petio
What is the website/page of the icon?
https://petio.tv/
Upload the icon here: [Optional]

adding!

The icon has been added!


setDownloadBehavior issue

Steps to reproduce
Tell us about your environment:

Puppeteer version: ^1.18.1
Platform / OS version: Win 10
Node.js version: v8.12.0

What steps will reproduce the problem?
Please include code that reproduces the issue.

Run script
It fails

Code:
const puppeteer = require('puppeteer');

(async () => {
   const browser = await puppeteer.launch({
      executablePath: 'C:/Program Files (x86)/Google/Chrome/Application/chrome.exe',
      defaultViewport: {
          width: 1920,
          height: 1080
      },
      headless: false,
      userDataDir: "./user_data"
   });

   const page = await browser.newPage();

   await page.goto(
     'https://example.com/page-with-the-file-link',
      { waitUntil: 'domcontentloaded' },
   );

   await page._client.send('Page.setDownloadBehavior', {
         behavior: 'allow',
         downloadPath: 'C:/Users/Me/Downloads/custom/folder/'
   });

   console.log('Start downloading');

   await page.click('a.download-btn');

   await page.waitFor(5000);

   console.log('Complete');
   await browser.close();
})();

What is the expected result?
Catch "download complete" event and to make it work correctly.
Also I want to handle downloading status and monitor downloading proccess.
What happens instead?
Start downloading
Complete
(node:51016) UnhandledPromiseRejectionWarning: Error: WebSocket is not open: readyState 3 (CLOSED)
    at WebSocket.send (C:\Users\Me\Downloads\puppeteer\node_modules\ws\lib\websocket.js:329:19)
    at WebSocketTransport.send (C:\Users\Me\Downloads\puppeteer\node_modules\puppeteer\lib\WebSocketTransport.js:60:14)
    at Connection._rawSend (C:\Users\Me\Downloads\puppeteer\node_modules\puppeteer\lib\Connection.js:86:21)
    at Connection.send (C:\Users\Me\Downloads\puppeteer\node_modules\puppeteer\lib\Connection.js:72:21)
    at gracefullyCloseChrome (C:\Users\Me\Downloads\puppeteer\node_modules\puppeteer\lib\Launcher.js:194:20)
    at Browser.close (C:\Users\Me\Downloads\puppeteer\node_modules\puppeteer\lib\Browser.js:255:31)
    at Browser.<anonymous> (C:\Users\Me\Downloads\puppeteer\node_modules\puppeteer\lib\helper.js:112:23)
    at C:\Users\Me\Downloads\puppeteer\test-download-file.js:97:18
    at <anonymous>
(node:51016) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:51016) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Script does not work properly during the execution. It exits right after clicking download link on the page  w/o waiting for 5 seconds and chrome puts downloaded file into default 'Downloads' folder instead of defined 'downloadPath' in the script.

How to get around this with crutches

1) We should catch/wait for file which will appear in default 'Downloads' directory
function checkExistsWithTimeout(filePath, timeout) {
    return new Promise(function (resolve, reject) {

        var timer = setTimeout(function () {
            watcher.close();
            reject(new Error('File did not exists and was not created during the timeout.'));
        }, timeout);

        fs.access(filePath, fs.constants.R_OK, function (err) {
            if (!err) {
                clearTimeout(timer);
                watcher.close();
                resolve();
            }
        });

        var dir = path.dirname(filePath);
        var basename = path.basename(filePath);
        var watcher = fs.watch(dir, function (eventType, filename) {
            if (eventType === 'rename' && filename === basename) {
                clearTimeout(timer);
                watcher.close();
                resolve();
            }
        });
    });
}

Check file just in case (optionally)
function checkFile(path) {
    return new Promise(function (resolve, reject) {
        fs.access(path, fs.F_OK, (err) => {
            if (err) {
                reject(new Error(err));
            }

            //file exists
            console.log('File exists');
            resolve();
        });
    });
}

Move file wherever we need after download complete (optionally)
function moveFile(fromPath, toPath) {
    return new Promise(function (resolve, reject) {
        fs.rename(fromPath, toPath, function (err) {
            if (err) {
                reject(new Error('File did not move.'));
                throw err;
            } else {
                console.log('File moved');
                resolve();
            }
        });
    });
}

Example
const fileName = await page.evaluate(() => {
   return document.querySelector('.download-file-btn').textContent.trim();
});
await page.click('.download-file-btn');
await checkExistsWithTimeout('C:/Users/Me/Downloads/'+fileName, 10000);
await moveFile('C:/Users/Me/Downloads/'+fileName, 'C:/Users/me/Desktop/Videos/'+fileName);

2) Donwload file from url using 'request' package
function download(uri, filename, callback) {
    return new Promise(function (resolve, reject) {
        request.head(uri, function (err, res, body) {
            if (!err && res.statusCode == 200) {
                console.log('content-type:', res.headers['content-type']);
                console.log('content-length:', res.headers['content-length']);

                request(uri)
                .pipe(fs.createWriteStream(filename))
                .on('error', function(response) {
                    console.log(err);
                    reject(new Error(err));
                })
                .on('close', function() {
                    callback();
                    resolve();
                });
            } else {
                reject(new Error(err));
            }
        });
    });
}

Example
const videoSrc = await page.evaluate(() => {
     return document.querySelector('video.vjs-tech').src;
});

await download(videoSrc, "C:/Users/Me/Downloads/Videos/video.mp4", function() {
   console.log('downloaded');
});

It works for me without 'setDownloadBehavior' option, othervise it fails.
Now I can click download button or download file directly from some URL, just by manipulating with filesystem with node little bit.
Hope it will help somebody.

How to get around this with crutches

@lumenier1 Loved the logo.
Overall, this is another workaround for a long-awaited #299. Closing as a dupe.

If there is error on file which you want to download then try this one:
const myDownloadPath = path.resolve(${Directory Where you want to download})
mkdirp(myDownloadPath)
await page._client.send('Page.setDownloadBehavior', {behavior: 'allow', downloadPath: myDownloadPath });
imported:
const mkdirp = require('mkdirp')
const path = require('path')

March 2020 update: the error is still happening, even after using the solution @azazrehman proposed or using anything from #299

Still same, download is not being triggered

When chromium is downloading a file, it creates two files, <filename>, and <filename>.crdownload. It is not enough to just check for <filename>, you also need to check that <filename>.crdownload has been removed.
// make sure tempDownloadDir is an empty directory
async function waitForDownload(tempDownloadDir, downloadDir) {
  let filename;
  while (!filename || filename.endsWith('.crdownload')) {
    filename = fs.readdirSync(tempDownloadDir)[0];
    await new Promise(resolve => setTimeout(resolve, 500));
  }
  if (filename !== undefined) {
    fs.renameSync(path.join(tempDownloadDir, filename), path.join(downloadDir, filename));
  }
  return filename;
};

Page.setDownloadBehavior no longer works for me (07/21/2020). To work around, use createCDPSession to get a CDP Session, then do a Browser.setDownloadBehavior:
const cdpsession = await page.target().createCDPSession();
cdpsession.send ("Browser.setDownloadBehavior", {behavior:"allow", downloadPath: " ... download path ..." });

I believe that it may have to do with Page.setDownloadBehavior being deprecated.

Page.setDownloadBehavior no longer works for me (07/21/2020). To work around, use createCDPSession to get a CDP Session, then do a Browser.setDownloadBehavior:
const cdpsession = await page.target().createCDPSession();
cdpsession.send ("Browser.setDownloadBehavior", {behavior:"allow", downloadPath: " ... download path ..." });

I believe that it may have to do with Page.setDownloadBehavior being deprecated.

I can confirm the problem.
Seems puppeteer 5.2.1 is no more working even with "CDP Session" solution (Every time I try to launch the download, it fails)

I'm also getting download error, is there any way to download files by clicking?


Very wonky boxes

Am unsure how things got in this state. @smoogipoo any ideas?  Draw visualiser looked normal.
https://www.youtube.com/watch?v=XumcpXFb7ic

Can't seem to replicate. Can only think of https://github.com/ppy/osu-framework/pull/2718 as a possible cause.

Could also verify by turning off front-to-back while in the broken state.

Managed to replicate the issue, it seems like SafeAreaContainer doesn't correctly support being completely outside the bounds of the parenting SafeAreaDefiningContainer.
Here's a repro:
public class TestSceneScratch : FrameworkTestScene
{
    public TestSceneScratch()
    {
        Add(new SafeAreaDefiningContainer
        {
            Anchor = Anchor.Centre,
            Origin = Anchor.Centre,
            Size = new Vector2(200),
            Child = new SafeAreaContainer
            {
                Size = new Vector2(100),
                Y = 400,
                Child = new Box { RelativeSizeAxes = Axes.Both }
            }
        });
    }
}

Also I highly suspect that SAC won't work if it's being transformed towards a padded edge of the SADC.
@swoolcock pinging because you know this better than me

Will take a look in a bit.


Rename to wipac-telemetry

Thankfully, GitHub repo redirects exist: https://github.blog/2013-05-16-repository-redirects-are-here/

Renamed and tested (https://app.circleci.com/pipelines/github/WIPACrepo/rest-tools/358/workflows/cac4991c-8245-4f29-842f-70024e12ee31/jobs/350)


⚠ Bofrost.at has degraded performance

In 897e9d5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 971 ms

Resolved: Bofrost.at performance has improved in 1ba3880 after 6 minutes.


Possibly leaking org.apache.kafka.clients.consumer.KafkaConsumer instance

When checking fs2-kafka stream behavior during errors, I noticed  following warning in logs
2019-10-08 23:25:58,472 | WARN | o.a.kafka.common.utils.AppInfoParser - Error registering AppInfo mbean javax.management.InstanceAlreadyExistsException: kafka.consumer:type=app-info,id=client_id 	at java.management/com.sun.jmx.mbeanserver.Repository.addMBean(Repository.java:436)
I managed to reproduce error in following test case (I ran it from KafkaConsumerSpec):
it("should restart properly on error") {
      withKafka { (config, topic) =>
        createCustomTopic(topic, partitions = 3)
        val produced = (0 until 1).map(n => s"key-$n" -> s"value->$n")
        publishToKafka(topic, produced)

        val conf = consumerSettings[IO](config)

        consumerStream[IO]
          .using(conf.withClientId("client_id"))
          .evalTap(_.subscribeTo(topic))
          .flatMap(_.stream)
          .flatMap(Stream.emit(_) >> Stream.raiseError[IO](new RuntimeException))
          .attempts(Stream.emit(1.second))
          .compile
          .toVector
          .unsafeRunSync
      }
    }

Enable logging and check logs. Notice that I have changed client.id property to a specific value. Also I checked out code tagged with "v0.20.1".
Setup:
Java: 11.0.4
Scala: 2.12.8
sbt:1.3.0
fs2-kafka: 0.20.1
cats: 2.0.0
cats-effect: 2.0.0
Let me know if you need more info.

I have created a small sample project https://github.com/retriku/fs2-kafka-consumer. Follow instructions from README.md in order to reproduce issue

@retriku I think this could do the trick:
consumerStream[IO]
          .using(conf.withClientId("client_id"))
          .evalTap(_.subscribeTo(topic))
          .flatMap(_.stream)
          .flatMap(Stream.emit(_) >> Stream.raiseError[IO](new RuntimeException))
          .scope  // <--- this should guarantee that the current consumer resource is actually released
          .attempts(Stream.emit(1.second))
          .compile
          .toVector
          .unsafeRunSync

this should explain why :) https://gitter.im/functional-streams-for-scala/fs2?at=5ce5088decdf942b4c25877e

Thanks a lot @marcodippy! Looks like it really solves the problem.
Closing this ticket

As far as I understand there's recently been a change in the behaviour of scopes (https://github.com/functional-streams-for-scala/fs2/pull/1574); I'm no expert, but it looks like in the latest version we shouldn't need to call .scope explicitly there anymore, but I tried that on fs2 2.0.1 and it still doesn't work as we'd expect. I'll ask the fs2 folks :)


Add support for location-based bounding box queries

A few outstanding points:

Does it make sense to add these methods directly on RLMObject / RLMResults? Since internally they just build NSPredicate they could be exposed as functions that produce NSPredicate. Alternatively, they could live in separate location-related categories on RLMObject / RLMResults?
Is it reasonable to use CoreLocation's coordinate type in our API, or should we define our own?
What should a Swift API look like? Querying in our Swift API is currently via a single filter method. There's no obvious extrapolation like there is with the Objective-C API.

I don't think these need to be available on RLMObject -- filtering based on allObjects should be sufficient.
I think a swift API could look something like filter(boundingBox:latitudeProperty:LongitudeProperty) ?

I'm also not entirely convinced those should be exceptions rather than returning errors?

My main concern about using the CoreLocation coordinate types is that since it's part of the umbrella header it drags in a lot of extra stuff for anyone including Realm, and I think if modules are enabled it'll even result in CL being linked into the user's app. I'm not sure if that's actually a problem in practice, but I'd expect a few complaints about it.

@tgoyne, for Objective-C projects, auto-linking doesn't pull in CoreLocation.framework unless you make use of symbols it defines. Merely using types declared in its headers doesn't result in it being pulled in. Things don't work quite so well for Swift. Realm APIs mentioning CLLocationCoordinate2D doesn't result in CoreLocation.framework being pulled in directly, but it does result in CoreLocation's Swift overlay, libswiftCoreLocation.dylib, being pulled in, which in turn pulls in CoreLocation.framework.
It would be easy to declare our own coordinate type instead. The downside is that those users that choose to use our location-based APIs will have to convert between types when doing so. It's probably worth it to avoid pulling in CoreLocation unnecessarily for all clients using Swift.

Rebased onto master with the updated version of core. This is now ready for review.

I'd rather this be a utility to generate an NSPredicate, which would cut down on the number of API additions we need to make in both bindings.

While this particular query can be represented as an NSPredicate, related queries such as the objects-within-distance-of-point cannot be. I think it'd be weird for these two related types of queries to be exposed in different ways.

I'm going to close this PR for now since it doesn't make sense to add this without broader support for location types. When we attack that, we can revisit this branch.


escape terraform interpolation ${} in rulegroup

Hi grafana team,
i ran into this error when creating the RuleGroup resource
Invalid reference: A reference to a resource type must be followed by at least one attribute access, specifying the resource name.
after some digging, turns out it was caused by having ${}
example, having this in the model expression
"$ErrorsLast24h > ${PercentOfMessagesLast24h}"

should be a similar issue to https://github.com/grafana/crossplane-provider-grafana/issues/59
and
https://github.com/grafana/crossplane-provider-grafana/issues/89
currently bypassing this using $$.  we use the exported rules yaml files (so once get into grafana, it will become $), would be nice to have a fix

This should be fixed in v0.13.0!


CMake: enable building and installing as a library

This is far less ambitious than https://github.com/BinomialLLC/basis_universal/pull/13, but I hope it can be merged more easily as it doesn't reorganize files.
It also generate CMake export files, and a basisu::basisu_lib CMake target, which makes it easy to use basis_universal from other CMake projects
The build can use the ZSTD from the library when available, otherwise fallbacks to the embeded copy.
Tested on Linux (ubuntu 20.04) and Windows (MSVC 2019)

Thank you. It's still a lot of changes, so I'll need to review it. It's queued up.

I would also suggest to add VERSION and SOVERSION properties to the library properties.
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -153,7 +153,7 @@

 set(BASISU_LIB_TARGET "basisu_lib")
 add_library(${BASISU_LIB_TARGET} ${BASISU_SRC_LIST})
-set_target_properties(${BASISU_LIB_TARGET} PROPERTIES OUTPUT_NAME "basisu")
+set_target_properties(${BASISU_LIB_TARGET} PROPERTIES OUTPUT_NAME "basisu" VERSION "${PROJECT_VERSION}" SOVERSION "${PROJECT_VERSION}")
 target_include_directories(${BASISU_LIB_TARGET} PUBLIC
                            $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/transcoder>
                            $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/encoder>

Any movement on this PR? It would significantly help with integrating the library in other tools.


Broken ReSpeaker Mic Array devices (manufacturing issue?)

Two of the four ReSpeaker Mic Array devices received have been broken on arrival.  I've downloaded the 8 channel raw FW on each of them to figure out what is happening, and recorded the following 8 channel wave files using the following command: 'arecord -c 8 -r 16000 -t wav test2.wav':
Broken Device 1: https://www.dropbox.com/s/v1v0d5b1rjh6bjo/respeaker_mic_array_device2.wav?dl=0
Broken Device 2: https://www.dropbox.com/s/uyydnxm6f1uas86/respeaker_mic_array_device1.wav?dl=0
In each case, multiple microphones were generating noisy audio streams, or completely broken audio streams, for several of the microphones.  This certainly must have been interfering with the DSP's ability to perform proper AEC/beaming/etc.
The other two devices are working as expected.

The audio files can't be downloaded?


Default Data Not Saved

Keep getting a Error  Default Data not saved  when i click save to db

Cab you send a screen shot, if com1 is 124.85 (the default value) it won’t save.  This is to avoid issues with the default settings overwriting real settings when certain things happen in the sim
From: jpoblocki77 @.>
Sent: Monday, March 27, 2023 6:21 AM
To: kenz67/FlightSim-AircraftState @.>
Cc: Subscribed @.***>
Subject: [kenz67/FlightSim-AircraftState] Default Data Not Saved (Issue #9)
Keep getting a Error Default Data not saved when i click save to db
—
Reply to this email directly, view it on GitHub https://github.com/kenz67/FlightSim-AircraftState/issues/9 , or unsubscribe https://github.com/notifications/unsubscribe-auth/APPUDWGXJJ7HS44RMJ25APTW6FSYNANCNFSM6AAAAAAWI7NKUQ .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/APPUDWGOU24PKP7XO3MQFT3W6FSYNA5CNFSM6AAAAAAWI7NKUSWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHGDXC4BE.gif Message ID: @.*** @.***> >


Cleanup: arp-speaker

Make arp-speaker simpler, implement leaderelection and provide the
framework for sending unsolicitated ARPs on acquiring leadership
(actually sending not implemented yet).
I needed access to the k8s client, hence the internal/k8s/kus.go change,
the rest is all local to the arp stuff.
All not tested; but this should give me a base to allow some local
testing.

Moving the master election into internal/k8s sgtm.
On 10 Dec 2017 20:01, "Dave Anderson"<EMAIL_ADDRESS>wrote:

Merged #35 https://github.com/google/metallb/pull/35.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/metallb/pull/35#event-1380485082, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAVkW9JeItmcEKIkXVct7a58hmsFNTO0ks5s_Dh5gaJpZM4Q8cnS
.


feat(#3): release to homebrew

closes #3

PR-Codex overview
This PR updates the Homebrew workflow to include a license and permissions, adds a job to release a formula, and specifies the formula details.
Detailed summary

Added MIT License and permissions
Updated Homebrew workflow with release job
Specified formula details for the release job

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@h1alexbel Hey there, awesome job on your contribution! 🎉 You've snagged +10 points this time: +30 for the base, with a slight dip of -15 for missing code review and -5 for just a few comments (only 3). Don't sweat it though, these things happen! Keep those contributions rolling in - quality and quantity both count. Your current score is sitting pretty at +13. Let's aim higher next time!


Date functions which take into account leap years and seconds

Dear sqlean team,
working with sqlite dates is a real pain because date modifiers don't take into account leap years making the output of them unreliable.
The users of my app would consider this a serious bug.
Example:
select datetime('1993-06-30 23:59', '+20 months') as dt;

should return 1995-02-28 23:59 but it returns 1995-03-02 23:59:00 because sqlite does not take into account leap years.

Do you know of any plugin with proper date and time functions which work bug free?
If not, do you know any hack which can solve this?

I believe a plugin for better date and time handling including convenience functions for formatting, parsing and modifying dates would very much help the community.
Best regards

The problem isn't in leap years, I believe. For example, this query in SQLite:
select datetime('1993-08-31', '+1 month') as dt;

returns 1993-10-01.
While the same query in PostgreSQL (or MySQL):
select '1993-08-31'::date + interval '1 month';

returns 1993-09-30.
I don't know how to make SQLite behave like PostgreSQL in this case, unfortunately.

Sorry, I have no plans to add new features at this time.


web api v3 documentation error.

The documentation in  https://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/index.html#-Request-Body-Parameters
states wrongly that "subject" is not required.
But it is required.

Or here another way to see it.
https://github.com/sendgrid/docs/blob/old-master/source/API_Reference/Web_API_v3/Mail/index.html#L107
{% api_table_param subject string No "" "The subject line of your email." 1 %}

Hi @komkom,
The API reference shown above is out dates and soon-to-be deprecated. I apologize for the inconvenience. Please refer to this set of documentation in the future.
The optional subject you're looking at is nested under Personalizations, and if there's a subject specified within the personalization it is not required in the request body. Please let me know if you have any additional questions.
Thank you,
The Twilio SendGrid docs team


Control is missing on ExternalWorkflowExecutionCancelRequestedEventAttributes

Moreover, control is not propagated to database layer.
This is currently fine due to client issue https://github.com/uber-go/cadence-client/issues/405

since there is an agreement that variable control will only be present in the RequestCancelExternalWorkfllowInitiated event, and we use initiated event ID for mapping from history event to state machine, this issue is no longer relevant


QRCode isn't working.

Hi, i was trying to print a QRCode using the correct function.
The barcode128 works fine, but the QRCode don't, it shows me like this when i try to print:
1A21C1E01P0https://www.google.pt/1Q0
is there any fix for this issue? Thanks

Your link is broken, if you're seeing just a T, and based on the fact that you're using Portuguese Google, I'm guessing it's because of this: https://github.com/mtmsuhail/ESC-POS-USB-NET/issues/31#issue-829422674


Infer on own data

I was trying to apply the model on an own example image. I followed the steps you described in https://github.com/lucastabelini/LaneATT/issues/3. However, I get the following result:
python main.py test --exp_name laneatt_r34_llamas
[2021-12-06 13:38:07,376] [lib.runner] [INFO] Loading model experiments/laneatt_r34_llamas/models/model_0015.pt
Looking for image files with the pattern /disk/vanishing_data/zl254/test_folder/**/*.jpg
[2021-12-06 13:38:09,759] [lib.datasets.lane_dataset] [INFO] Transforming annotations to the model's target format...
[2021-12-06 13:38:09,759] [lib.datasets.lane_dataset] [INFO] Done.
0%|                                                                                | 0/1 [00:00<?, ?it/s]/disk/vanishing_data/zl254/test_folder/Grosse-Autobahn-Baustellen-zu-Ferienbeginn-auf-dem-Berliner-Suedring_reference_4_3.jpg
100%|██████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:00<00:00,  1.15it/s]
Traceback (most recent call last):
File "main.py", line 61, in
main()
File "main.py", line 57, in main
runner.eval(epoch=args.epoch or exp.get_last_checkpoint_epoch(), save_predictions=args.save_predictions)
File "/disk/no_backup/zl254/LaneATT/lib/runner.py", line 106, in eval
self.exp.eval_end_callback(dataloader.dataset.dataset, predictions, epoch)
File "/disk/no_backup/zl254/LaneATT/lib/experiment.py", line 130, in eval_end_callback
metrics = self.save_epoch_results(dataset, predictions, epoch_evaluated)
File "/disk/no_backup/zl254/LaneATT/lib/experiment.py", line 137, in save_epoch_results
predictions_dir = os.path.join(epoch_results_path, '{}_predictions'.format(dataset.split))
AttributeError: 'NoLabelDataset' object has no attribute 'split'
Can you tell me what I need to modify to make it work?

After the model passes through every image in the dataset, it will try to save the results (metrics) to disk. Since you're using the NoLabelDataset class I assume you're not interested in that, only in qualitative results. Have you tried running the code with the flags --view all?

Yes, I got the same error message

That is expected. What are you expecting from running the code? All it can do is show a visualization of the predictions. It should have opened a window displaying them. If you want metrics, then it would be necessary to implement them for your specific dataset.

Sorry, I meant I got the same error instead of a window displaying the visualization. I do not need metrics, I just wanted to see the qualitative results.

That's odd. Are you running this script through SSH? I'm not sure what could cause the window not to show. This is the code line that should open the window: https://github.com/lucastabelini/LaneATT/blob/main/lib/runner.py#L100

@Christin95 Did you solve the problem? I got the same error, and I have already tried to use the flags "--view all". I am trying to do the testing with my own images, and I changed the 'LaneATT/experiments/laneatt_r34_tusimple/config.yaml' as @lucastabelini proposed in issue #3, so I don't know what to do..

@pepesouza81 what is your error message?


Code for nuplan

Hi,
Nice work! Are you going to release the code for nuplan?
Best

Recently, we do not have plans to release the code for nuPlan. I think you only need to modify the data preprocessing scripts.


Glyph text issue

@iBasit are we using the correct glyph (text) to render the Arabic text, I just noticed that there is extra meem's on Surah 2 ayah 19, on Tanzil it's not being displayed even when using the same font Scheherazade.

not sure what has changed but I could not reproduce the issue on http://globalquran.com/en.transliteration+quran-uthmani/2:19

Let  me know if you are still viewing this issue, as I don't see it.

@basit from your screenshot I can still see it but it's overlaping with the text so it's a bit more difficult to see it.


🔧 Update myst dependencies

update myst dependencies to the latest.

When testing this, can you ensure that the epigraph work that @agoose77 completed shows up in the mystmd docs?

Updated to the latest, quote and kbd roles are working. Tested the new frontmatter fixes, they also work.


Investigate root cause for significant time consumed by loader

After investigation it appears that I was incorrect in my initial analysis. It is not the loader itself that is causing the delay. This was a mistake when reading the trace. For the sample app, the time is broken down as follows:
154.6 ms total startup thread time
| - 129.82 ms  - GetActivationFactory (runtime setup time)
|          | - 108.7 ms - Framework CRT init (objc static init) <----- BIG HITTER
|          | - 21.14 ms - Loader, Interop, etc. (mostly fixed cost overhead)
| - 24.29 ms - ActivateInstance (actual business logic time)
The differences seen in different investigations was likely just due to differing environments / surrounding conditions not actual savings. My mistake for misreading the traces (was confused because it broke down the framework init calls into several different sections underneath identical loader calls that made it hard to find them all). This suggests that the work to defer selector registration etc will be more valuable than initially expected and probably worth the risks.


Params not available when in controller

In this case:
@Controller(":id/about")
export class UserController {
  @Get()
  about(req: Request, res: Response) {
    console.log(req.params.id); // undefined
    res.json({ hello: "world" });
  }
}

req.params.id doesn't exist, because it's not defined in @Get() but in @Controller.
Is this intentional, or a bug with Express.js or with Overnight?

Hmm this might be an Overnight bug, let me look into it.

I think you need to do @ClassOptions({mergeParams: true}). I keep forgetting this. It's also required in express. I've had a lot of people post about this issue. Maybe I should update the readme.

That works, thanks! I didn't know Overnight was using express (sub)routers under the hood, for some reason I always imagined some kind of route concatenation, but that's silly of me.
Yes, perhaps it's a good idea to add the mergeParams note in the README section about child controllers. Thanks again!


VS Code Server doesn't use system certificate store on OpenSUSE 15.4

Issue Type: Bug

Install OpenSUSE as a host, or in WSL
Install a custom CA certificate into OpenSUSE
Connect to OpenSUSE as remote using WSL, Remote SSH, Remote Explorer, etc.
Install (or attempt to install) a plugin that makes HTTPS connections.  Observe failure to connect due to a "self-signed certificate"

Note that this issue does not occur on CentOS 8 Stream nor on Ubuntu 20.04.
Also, by way of a work-around, if the remote node executable is replaced by a script that sets NODE_TLS_REJECT_UNAUTHORIZED=0 the environment and then invokes the real node, then the issue is worked around.  Therefore, the issue seems to be that Node on OpenSUSE does not use the system certificate store.
Note that OpenSUSE does store the CA bundle in a different location than some other Linux distributions, though the system libopenssl is built with the proper location (though it might not be in use by libopenssl).
VS Code version: Code 1.69.2 (3b889b090b5ad5793f524b5d1d39fda662b96a2a, 2022-07-18T16:12:52.460Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No
Remote OS version: Linux x64 5.14.21-150400.24.69-default

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10850H CPU @ 2.70GHz (12 x 2712)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.75GB (7.75GB free)

Process Argv
--crash-reporter-id 7d88c287-2f49-4675-bee3-a01b39662908

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: opensuse

OS
Linux x64 5.14.21-150400.24.69-default

CPUs
Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz (6 x 2593)

Memory (System)
11.67GB (3.96GB free)

VM
100%

Extensions (5)

Extension
Author (truncated)
Version

jupyter-keymap
ms-
1.1.0

remote-ssh
ms-
0.84.0

remote-ssh-edit
ms-
0.84.0

codeium-enterprise
Cod
1.2.59

codeium-enterprise-updater
Cod
1.0.9

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.81.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I realized that libopenssl wasn't dynamically linked for node, and figured that there must be a statically linked copy, and that statically linked copy might be expecting the CA bundle to be at a different location.
Therefore, on a whim,  I ran:
sudo ln -s /var/lib/ca-certificates/ca-bundle.pem /etc/ssl/certs/ca-certificates.crt

This functions as a simple work-around, allowing node to find the system CA bundle, and the extensions to make the necessarily HTTPS connections.

This is a limitation in the way Node.js looks up certificate location by default. However, you can use the environment variable NODE_EXTRA_CA_CERTS or specify the location [CA_default] in an openssl config file with --openssl-config

Yes, I was able to work around this at first with an environment variable, but that involved me hacking the node binary installed to instead be a script that calls the actually binary with the adjusted environment. That's really hacky and not sustainable (or obvious).

NODE_EXTRA_CA_CERTS does not need to be hacked into the script, it is only respected by Node.js exectuable so you can set it globally, you will need this variable for any other node programs to respect your system's client certificate installation.

@rjbell4 Is there a /etc/ssl/ca-bundle.pem? https://github.com/openSUSE/ca-certificates suggests that this is the cert bundle.

We check for and load /etc/ssl/certs/ca-certificates.crt and /etc/ssl/certs/ca-bundle.crt. With more information we could add additional paths.


Hook to act after others plugins invocation

What problem does this feature solve?
(Plugins dev API)
I don't know if this already exists or possible, but I need a method to listen other plugins
invocation inside my plugin, to alter files contents.
Simple use case:
I want to replace default import of PWA plugin (import './registerServiceWorker') to :
if (process.client) require('./registerServiceWorker');

ATM, I use api.postProcessFiles() function, but this one only work if I do:
vue add @vue/pwa
vue add my-plugin

But if I do
vue add my-plugin
vue add @vue/pwa

my-plugin is not aware that PWA was added/invoked, so I cant transform the import statement.
What does the proposed API look like?
Something like api.postProcessFiles()
api.hookPluginInvoke('pluginName', (files) => {})

Thanks !

We can also use hook mechanism for events.
For example, apollo plugin has a postprocess where it's checks for typescripts and converts accordingly. Ideally this should be done by other 3rd party plugins too, but isn't it better that only typescript defines this and other plugins doesn't need to care about it? So, the typescript plugin defines something like this,
api.hook('afterGenerate', (files) => {
  convertFiles(files)
});

Here's my proposal. @yyx990803 @LinusBorg @Akryum Would appreciate thoughts.
Callback/Hook API Proposal
There are a lot of different reasons for having a plugin system. But one of the main reasons is to provide flexibility. A good system should allow plugins to interact with each other but still be independent of each other. All related code for certain feature should be contained to a certain plugin rather than relying on other plugins. Given below is the proposal of the API
Type of Callbacks
afterRender (postProcessFiles)
It behaves like how the current postProcessFiles behave. The function provided to this will be executed after template rendering is done and we get an JS object containing the files. It will only be executed for the plugins which were invoked. The new postProcessFiles will just be a wrapper for this.
Example
A plugin which generates modules would want to insert those modules into certain parts of existing code. This hook can be used to do that after the module is created.
afterAnyRender
The function provided to this will be executed after writing the generated files to the disk and installing the injected dependencies. It will be executed for all the plugins installed in the project.
Example
Typescript plugin would want to convert all the files irrespective of which plugin they were generated from before writing them to the disk.
afterWrite (onCreateComplete)
It behaves like how the current onCreateComplete behave. The function provided to this will be executed after writing the generated files to the disk and installing the injected dependencies. It will only be executed for the plugins which were invoked. The new onCreateComplete will just be a wrapper for this.
Example
Similar example to afterRender.
afterAnyWrite
The function provided to this will be executed after template rendering is done and we get an JS object containing the files. It will be executed for all the plugins installed in the project.
Example
Eslint plugin would want to lint --fix all the files.
Drawbacks
Whenever a plugin is invoked, all the other plugins which were installed would be loaded to be checked if they have any callbacks. This might increase the running time of vue invoke.
Why in 3.0.0?
Once we release 3.0.0, there will be a lot of 3rd party plugins created by the users, and it will become an issue if they don't take care of how their plugins will interact with other plugins (especially typescript, lint). In order to make sure that it won't become a big mess that we try to unravel with the next major update, we should try to make sure that we do this before releasing v3.

@yyx990803 Thoughts?

@yyx990803 Does the label you added means I can work on this and submit PR?

Implemented in #2337. Please take a look. Will write docs as a separate PR.


Apply a class to outer MdMenu container

Any way to add a class to the outer menu container so I can nudge it around from an anchor point?

Thats the element I would want to apply the class to.

You can create a global css rule targeting the .tour-step class.  That's a class that ngx-tour-md-menu applies to the <md-menu> component.
You can create the global css rule in a styles.css file or in a component that has encapsulation: ViewEncapsulation.None.
Let me know if that works for you.

Yeah, thats helpful, but for my case I need different nudge properties for each step, so I need to target the rendered MdMenu for each individual step.
I was able to pull it off my using a "shadow" anchor element that I position relatively.. creates an extra dom node, but no big deal -- works fine for my needs.

Hi Isaac, thanks for your package !
I'm try to change the color of the background of the card, no success...
I already try :
this:
body .cdk-overlay-container .cdk-overlay-pane .mat-card{
   background-color: black !important;
}

and this :
.tour-step .mat-card{
  background-color: black !important;
}

And is it possible to change properties on 'popover' class ?
Thanks for your attention.

Those styles should work if you're declaring them in a global css context (not inside a component's styles that is using the default ViewEncapsulation.Emulated).  Where are you declaring those styles?

Thanks for your quick answer,
This declaration is in a standalone file, and this file is imported directly in my app.component.less.
Should I just make a simple CSS file at the root directory and put my style in there ?
Thanks again
Alexis
From: Isaac Mann
Sent: Thursday, 31 August, 18:30
Subject: Re: [isaacplmann/ngx-tour] Apply a class to outer MdMenu container (#42)
To: isaacplmann/ngx-tour
Cc: COLONEAUX Alexis, Comment
Those styles should work if you're declaring them in a global css context (not inside a component's styles that is using the default ViewEncapsulation.Emulated). Where are you declaring those styles?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/isaacplmann/ngx-tour/issues/42#issuecomment-326350942, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARYvNt5ujRxetR-zNGjr8B3lxI0QEsjBks5sdt-igaJpZM4PE9Xg.

Yes, either (1) declare the css in a global css file (imported in the index.html) or (2) change the AppComponent's encapsulation property to ViewEncapsulation.None or (3) put /deep/ before every css rule that you want to apply globally.

Thanks you! Issue solved. I create a css file in src/styles/app.tour.css, and I reference it directly in my app.component.less as @import "styles/app.tour.css";
By the way, /deep/ is recently deprecated by some many browser and it still work without
Thanks again, you save my day
Regards

Is it possible to target specific steps? I thought the stepId would do that, but I don't see it added as an id or class attribute. Perhaps am just doing something wrong.

The reason for the specific target is that the positioning does not appear to be working properly, as I'm lazily loading multiple modules with child components, many of which have a step. As you can imagine, it's proving to be a bit difficult.

Putting the step id in a class is a reasonable feature request, I'll add that to my todo list.
But I would like to figure out why your tour steps aren't showing up in the correct place before resorting to a work around.  Can you provide any more context about what your code looks like?

@kahboom It's look like a CSS properties errors. I've a lot of error on a existing projet a the setup of ngx-tour. Look at this point (these solution worked for me):

Wrong position: add @import<EMAIL_ADDRESS>at the top of your app.component.less.

Cannot display Icon:  add link tab to your index.html :
<link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet">

Have a good day
Alexis


[FLINK-4000] Checkpoint dictionaries null after taskmgr failures

Fix for exception during job restart after task mgr failure, at which point restoreState fails as checkpoint dictionaries can be null.

This kind of check is might be useless and probably wouldn't fix the issue. My debugging shows that the list idsProcessedButNotAcknowledged is null on recovery, not checkpoints itself. This list is initialized in open method, but somehow it doesn't get called in such scenario.

I agree @asavartsov , that was a quick look and was working to reproduce.Does the updated make sense? thank you.

No, it does not make any sense and even makes things worse, sorry.

@asavartsov Ok. Please let me know how you propose to solve this? thanks

Take a look at my pull request at https://github.com/apache/flink/pull/2062

aha, in one of my intermediate runs had done just initializing idsProcessedButNotAcknowledged and  retain pendingCheckpoints , but in last run changed it calling open() :-( . makes sense @asavartsov. closing. thanks


Properly extract password from auth credentials (fixes #218)

Allows urls like postgres://h:password@host.com:1234 to work. I also attempted to rewrite using parse_url & parse_str as suggested in #218, however due to the implementation of socket location parsing, which parse_url doesn't support, this would lead to too much boilerplate code, making it simpler to just add an easy fix.

Thanks, please merge this!

Why would you ever want so specify a user? Redis does not support this?

It is the URL format provided by all of Heroku's Redis providers, so this potentially affects a lot of people. There is also pretty much no downside to adding it.

Well, currently passwords can contain : (there is currently no test that handles this). With this change this won't work anymore. The good thing is that 2.0 is not BC, so it would not be a problem.

@snc Good reason! So we should close this PR - we need to support :.

I would prefer a way to make it configurable then, as it is still a legitimate use case.

Absolutely, both should be supported. Do you still have your parse_url code around?

I force pushed over it I'm afraid, so no.

The way to have : in passwords is to use %3A really. So I would say this is mergeable as is.

Well, strictly speaking, only the first : is considered as splitting user and pass I think, so it should be changed to explode(':', $this->password, 2); to make sure further : stay in the password.

See https://github.com/snc/SncRedisBundle/commit/c5b91aba0947bccb66722e52a9048cf6ca514891 for additional fixes. I hope you're ok with this @snc, I wrote it down in UPDATE.md at least :)


1108 update references to admiraltest data

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[ ] Code is formatted according to the tidyverse style guide
[ ] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[ ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ ] Update to all relevant roxygen headers and examples
[ ] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md
[ ] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[ ] Link the issue so that it closes after successful merging.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done!  Much love to your accomplishment!

Hi all,
This PR changes a lot of files is small ways.  Hoping we can go over with a fine-tooth comb.
I'm still a bit flummoxed with the create_query_data.R  error.  If anyone has suggestions or can pull down and see if it works in there environment, then please! :)

Hi all,
This PR changes a lot of files is small ways. Hoping we can go over with a fine-tooth comb.
I'm still a bit flummoxed with the create_query_data.R error. If anyone has suggestions or can pull down and see if it works in there environment, then please! :)

That issues seems to have been resolved after Stefan's most recent changes on {admiraltest}. Getting this new error, through. Will fix it now.

@thomas-neitmann Thanks Thomas.  Let me know when you have fixed it.  I made some of the changes you requested and will push up once your change is done.

@thomas-neitmann devtools::check() also runs for me with no issues.


Add one json unmarshal test for full coverage

As #22, to make it complete.

LGTM, thanks for 100%!


speed is slow

when I test with 256x256, it takes 4s with cpu, and if I test with 512x512, it takes 25s in cpu.

Try GPU mate and best place to start will be colab @tengshaofeng


Version 0.6.0 not published on crates.io

Seems like version 0.6.0 has not been published on crates.io: https://crates.io/crates/alloy-zksync
We recently integrated alloy-zksync within foundry-zksync , using the published 0.5.0 version, but we are upgrading to a newer version of foundry, which makes use of alloy 0.6.0, so we would need to use the equivalent alloy-zksync version.
Would be lovely to have this version published on crates.io like 0.5.0. Thanks!

Version 0.6.1 is published


📚 Launch HMC Gitbook

GitBook provides a centralized, collaborative, and customizable platform for creating, managing, and publishing comprehensive documentation, enabling HMC to improve knowledge sharing, education, streamline collaboration, and enhance the user experience.

https://coinbox.gitbook.io/harimau-mint/ - HMC Gitbook


add filter option

Discussed in #25 and #20.

Should the filter function accept any additional parameters, or just standard Array#filter?  For example, contract name or artifact path.

For the Dark Forest subgraph use case, we can do all filter based on just the ABI information, but I definitely think there could be value in passing in the fullyQualifiedName. In hardhat-diamond-abi, the filter signature is filter(fullyQualifiedName: string, abi: any) => bool

@phated See latest.  I've structured it as a callback passed to abi.filter(...) rather than a function that accepts the entire ABI and returns a filtered result.  I think it's a bit more intuitive and concise this way.

It's likely that nobody will ever use the full ABI here, but I included it because Array#filter passes element, index, and the full array to the callback.


add --testnet-magic to part 3

I found missing --testnet-magic from sections of cli commands on part 3

Nvm the solutions changed and I see now you export at the top the correct variable, I need to update my solutions repo....


Fixed broken markup

@antonsavin : Thanks for your contribution! The author, @RodSan, has been notified to review your proposed change.

We fixed the code blocks in the files, and the revised article will be live later this afternoon.  Thank you for your contribution, we appreciate it!

@msmbaldwin - Would you like to close the PR if the issue has been addressed? Thanks.

The fix is reflected on the live site. Closing the pull request.


compilerPath in c_cpp_properties.json does not properly interpolate value of ${workspaceFolder} variable

Does this issue occur when all extensions are disabled?: No
Extension bisect identified the offending extension as "ms-vscode.cpptools"

VS Code Version: 1.53.2
OS Version: Linux x64 5.10.26-1rotete1-amd64

Steps to Reproduce:

Open c_cpp_properties.json for current workspace. Add a configuration with the following setting:
"compilerPath" : "${workspaceFolder}/arm-none-eabi-gcc"

VSCode flags the setting with a yellow squiggly line.

Replace the compilerPath setting with the following:
"compilerPath": "/usr/local/google/home/palchak/projects/pointy/boosty/arm-none-eabi-gcc"

Error goes away.

The value of the ${workspaceFolder} variable for this workspace is /usr/local/google/home/palchak/projects/pointy/boosty/.
Additionally, I configured the following task;
    "tasks": [
        {
            "label": "Check compiler",
            "type": "shell",
            "command": "stat ${workspaceFolder}/arm-none-eabi-gcc",
            "problemMatcher": []
        }
    ]

The output is:
> Executing task: stat /usr/local/google/home/palchak/projects/pointy/boosty/arm-none-eabi-gcc <

  File: /usr/local/google/home/palchak/projects/pointy/boosty/arm-none-eabi-gcc
  Size: 1296976         Blocks: 2536       IO Block: 4096   regular file
Device: fe04h/65028d    Inode: 22020197    Links: 1
Access: (0750/-rwxr-x---)  Uid: (268163/ palchak)   Gid: (89939/primarygroup)
Access: 2021-06-04 13:49:45.911436622 -0700
Modify: 2021-06-04 13:49:33.587504276 -0700
Change: 2021-06-04 13:49:33.587504276 -0700
 Birth: 2021-06-04 13:49:33.583504296 -0700

Terminal will be reused by tasks, press any key to close it.

Issue moved to here:
https://github.com/microsoft/vscode-cpptools/issues/7649


Cursor location does not match circle diameter

Environment:
Windows 10. ArcGIS Pro 2.1 build 10224
Using: Military Tools Single add-in installer. Military Tools for ArcGIS 2.0.
Steps to Repro:

With the add-in installed, open the Distance and Direction pane.
Click on the circle tab.
Make sure the From pull down is set to "Radius".
Click on the Map Point Tool.
Click on a point.
Move the mouse outwards, notice the behavior:

The mouse cursor matches the location of the outer circle

Perform the same steps listed above, this time change the pull down to "Diameter".  Notice the cursor does not match the location of the circle.

Confirmed this also occurs at ArcGIS Pro 2.0.

On Pro 2.1.0 with MT build 23 (5Mar) the feedback looks ok with Radius and Diameter


Add Display and Error trait to ClapError

Display trait may have something more to work on...

@sru one minor patch requested before merge. See comment against Cargo.toml.

@sru Nevermind, I'm rebasing your commits in. Removed the Cargo.lock entirely.

:umbrella: The latest upstream changes (presumably 806a525) made this pull request unmergeable. Please resolve the merge conflicts.

Merged manually via rebase.

@sru You'll want to pull -f due to fast forwards.

@james-darkfox Good enough.


It is currently not possible to accept ws and wss connections at the same time via the same websocket server instance.

Is your feature request related to a problem? Please describe.
It is currently not possible to accept ws and wss connections at the same time via the same websocket server instance.
Describe the solution you'd like
Being able to receive ws and wss connections at the same time
Describe alternatives you've considered
I have tried to modify the WebSocketServerFactory but I could not get the solution
Additional context
Please help me fix this. when i use WSS with DefaultSSLWebSocketServerFactory work fine, but when i use WS throw "Connection closed because ofjavax.net.ssl.SSLException: Unrecognized SSL message, plaintext connection?"

Hello @jaimegarzon,
this is currently not supported, and we have no plans to implement a workaround to get this to work.
If you really wanna WS, please use a separate port. I would recommend to only use WSS whenever it is possible.
Best regards,
Marcel


Time limit for buy orders

According to the bot behaviour, once the current price reaches the lowest price, then the bot places a STOP-LOSS-LIMIT order to buy.
However, it often happens that the trigger price is reached after several candles (more than 4 o 5): this scenario indicates that the price is increasing "slowly", so it is more prone to fall. This is a bad scenario, we want to avoid it unless we are sure about the bullish trend (i.e. never).
What about setting a parameter expressing a time limit (expressed either in candles or in minutes) after which the buy order will be deleted if the trigger price is not reached?
In my opinion, on the basis of the current oscillations, this will increase the precision of the buy/sell orders.
The current behaviour could always be replicated by setting a time limit of -1, for example.

Hmm, it's interesting.
What is your buy stop/limit percentage? How does the buy order stay a long time? In fact, how long is long?
For some coins, what you say may apply; however, for some coins, it may not.
For example, this CAKEUSDT was +-7% for many days and it sudden increased over 20%.

But I guess that is why you mentioned optional configuration.
Can you give me some examples for your scenario?
I mean I see what you say, I am curious what settings you made to stay the buy order long time.

I don't know if it`s related, but the bot put a buy order for almost 5 hours, any tip on params to avoid this scenario?

@elvishp2006 You are trying to buy FIAT to FIAT, which is almost not moving at all.
That is why the buy order will never be executed.
Buy coins, not FIAT.

@chrisleekr thanks, I was trying to convert my main FIAT to another by the lowest price

@elvishp2006 oh, in that case, let's wait.

Thenk you @chrisleekr , for both your answer and the project itself.
What is your buy stop/limit percentage? How does the buy order stay a long time? In fact, how long is long?
My bot is running with 15 minute candles. With "long time" I mean that the minimum has not a V shape, that is typical during bearish trends!
In order to limit losses (assuming that we don't know whether the general trend is bullish or bearish), my sell parameters are:
Trigger percentage: 1.008
Limit price percentage: 0.992
In this way, when the price rises of 0.8% after the buy, I have a stop loss at (more or less) the price I bought.
Concerning your example, notice that my goal (and those of many traders) is not to take all the possible opportunities.
On one hand, I wish to maximize the precision of the buy orders, i.e. they are the middle point of a V shape.
On the other hand, I would like to reach the trigger percentage as soon as possible in order to minimize the time in which I don't have a stop loss.
For this goal, I think that this optional (you guessed) parameter may help.
The following image shows a scenario that I would like to avoid (BTC), even if it is followed by a rise.

The following three images (ETH, IOTA, LTC) show good scenarios, in which minimums are followed by a growth of at least trigger % within the next 2 candles, so that
(i) we have a V shape
(ii) the time in which we haven't stop loss is minimized.
ETH

IOTA

LTC

I also thought how it would be possible to implement the behavior of the bot at such moments of the market.
And I have not come up with anything better than to achieve the maximum number of trades with the minimum profit.

make a checkbox - take the minimum profit.
need to - if profit price in usd * (fee*2) > 0 - sell
and I think need to create a limit order right away.(not stop-loss-limit)
But I didn’t think of how it is possible not to lose profit at moments in time - if the coin continues to grow. also keep track of time intervals.

@fsabiu
First of all, let me point out your sell parameters. I know it's your strategy.

In order to limit losses (assuming that we don't know whether the general trend is bullish or bearish), my sell parameters are:
Trigger percentage: 1.008
Limit price percentage: 0.992

The above configuration will chew up your money because of the commission. I assume you would know already.

In this way, when the price rises of 0.8% after the buy, I have a stop loss at (more or less) the price I bought.

Technically, it's not selling at the last buy price because you should take out trade commissions.
With your example, I still don't think it will work. Let me explain why.

Let's assume, you bought the BTC at 53700 USDT.
And the bot will start placing stop-loss limit order once it came back to 53700 USDT. However, within 15mins, it will definitely decrease by 0.8%, which I assume your stop percentage as well. Then it won't follow the rising price, your order would be executed in a short time.
Other examples will have the same issue.
If you want to follow the rise, you would configure stop/limit price percentage much bigger.
Am I making sense?

Thank you @chrisleekr.
I understand your considerations. I was using such a "limit price percentage" also as stop loss for the buy.
In my opinion, your last example highlighted the need of an initial (optional) stop loss that should be different from the limit price percentage (even 2 or 3 times bigger). In this case, we could avoid your scenario, guaranteeing an upper bound for the risk at each transaction. I know from other comments that you are not so favourable to it, it is just a kind proposal.
About parameters, you are right. Which parameters do you advise? And how are they profitable?

@chrisleekr many thanks for sharing your experience, it is useful to everyone!
Don't you agree that with a stop loss of 5% (for example) you would have limited many losses? Is it implementing it very time expansive? Otherwise, could you suggest how to implement it in the project?
Finally, why don't we create a Telegram group or something similar where we all share parameters and related profits? We would save a lot of time and money.
fsabiu

Well I created a discord server for it - https://discord.gg/Cqy35XnppA
And created pr with attached link in readme https://github.com/chrisleekr/binance-trading-bot/pull/136


Fax mail can be took from behind the wall

Description
Fax machines put printed mail slightly inside the tile above them, making it possible for mail to be taken from behind the wall by anyone accidentally.
Reproduction

Put paper into fax
Send fax to fax with wall one tile above it
Try taking the mail from the other side of the wall
Discover all NT secrets

Screenshots
https://github.com/space-wizards/space-station-14/assets/38139742/4941f341-39e3-494c-8445-cd9b36111dc8
Additional context
On Packed it is possible to accidentally take some mail from HoP

https://github.com/space-wizards/space-station-14/blob/0071b095abe477b23546bd3ea86d6c13ba6e4acf/Content.Server/Fax/FaxSystem.cs#L508
Ok, it seems like the mail is just pushed into the wall through collision with the fax machine, as I cant find any position offset for new mail to spawn.


RecyclerView different animation when collapsing above alredy expanded groupview

Here is the outline of the problem: When I click on item to collapse, the item below it first draws the container then the text. However if the view below the clicked one was already expanded, it smoothly scrolls up. I don't understand why they behave differently. It might not be related to animation, but I could not think of better title. The demo in the Readme does not behave this way.

Basically this was caused because in the layout RecyclerView that uses the adapter with parent and child ViewHolders, had its layout_height set to wrap_content, instead of match_parent.

Was facing this too, when collapsing or expanding it was a little bit laggy.
Changing parents and child to match_parent worked flawlessly.
Thanks


Fix error which generates if Delete Line used on the last line in CodeEditor

Fix the following error:

The error is reproduciable on 3.x. I think this PR could be cherry-picked.

Thanks!

Cherry-picked for 3.4.


Why not use ... to agree on multi-line strings?

Why not use `...` to agree on multi-line strings?
'''...''' or """...""" is too cumbersome.

Dart also has concatenation of adjacent strings - when you don't care about indentation I like to use that. FWIW I'd write this example as:
  var query = 'SELECT * FROM Task '
      "WHERE category = 'Personal' "
      'AND priority < 3 '
      'ORDER BY priority DESC';

Not the prettiest, but also probably good enough that we don't need yet another quote option.

@natebosch @lrhn
thank's


deluge: incompatible with libtorrent-rasterbar 1.1

Issue description
Deluge 1.3.12 is incompatible with current libtorrent-rasterbar 1.1 introduced in 0236661. There are a few exceptions thrown in the log and several core features do not work as expected, e.g. seeding torrents get queued, although queues are disabled. The Deluge team does not plan to fix this, but it looks like libtorrent might in version 1.1.1.
References:
http://forum.deluge-torrent.org/viewtopic.php?t=53939
http://forum.deluge-torrent.org/viewtopic.php?f=7&t=54059
https://bugs.archlinux.org/task/49861
Technical details

System: 16.09pre87431.3badad8
Nix version: 1.11.2
Nixpkgs version: 16.09pre87431.3badad8

Should we include an older version of libtorrent-rasterbar in the repos alongside current? We can point deluge to use it instead.

Yes, that would probably best in the meantime. 1.0.9 should work as expected.

Looks like the PR that fixes this has been merged.


Allow renaming of labels in decompiler view

Is your feature request related to a problem? Please describe.
Currently we can rename labels in the disassembly view, but not the decompiler view. This is a bit painful, and it's a standard feature in IDA that I'd love to see done here.
Describe the solution you'd like
Same as IDA: right click / rename (or press the shortcut key) while in decompiler view on top of a label should allow it to be renamed.

You can rename a lot of things in decompiler view.  What exactly are you trying to rename?

Check the label highlighted in the decompiler view in the attached screenshot. It can be renamed in the disassembly view, and then the decompiler view will change to reflect the new name.
However ideally it would mimic IDA behaviour - you could just change the name directly on the decompiler view.

Still relevant

Not sure how this has gone unnoticed for so long.

You guys are da bomb


PRs: DW contributions from docs-mule-runtime

When DW docs were move to docs-dataweave (for 10/21 publication), these DW-related PRs in docs-mule-runtime were still open. Need to assess what content from these PRs is needed in docs-dataweave.
HANDLING VIA A different PR:

https://github.com/mulesoft/docs-mule-runtime/pull/1708 (DOCS-12348 Explain all the mapping operator options #1708) -- SEE COMMENT FROM MDA

DONE:

https://github.com/mulesoft/docs-mule-runtime/pull/602 (DOCS-10744 Added simpler example from the form page (Multipart) #602) -- SEE COMMENT FROM MDA
https://github.com/mulesoft/docs-mule-runtime/pull/603 (DOCS-10745: Added an XML example with more details (Multipart) #603) -- SEE COMMENT FROM MDA
https://github.com/mulesoft/docs-mule-runtime/pull/604 (DOCS-10746: corrected the form function docs #604) --- SEE COMMENT FROM MDA (part of the contribution is not needed)
https://github.com/mulesoft/docs-mule-runtime/pull/1555 (DOCS-8698 Update dw-core-functions-flatten.adoc #1555)
https://github.com/mulesoft/docs-mule-runtime/pull/2021 (DOCS-11606-4.4-dw-cookbook-reorganization-kt #2021) -- FIXED via https://github.com/mulesoft/docs-dataweave/pull/21
https://github.com/mulesoft/docs-mule-runtime/pull/1992 (DOCS-11581: Update dataweave-flat-file-schemas.adoc #1992) -- FIXED in docs-dataweave. See results in https://docs.mulesoft.com/dataweave/2.4/dataweave-flat-file-schemas, https://docs.mulesoft.com/dataweave/2.3/dataweave-flat-file-schemas

There is one outstanding story that remains in our backlog to do as of this closing: W-10589423


Questions: 1. run on save? 2. every file?

I have a couple of questions:

Is it possible to add an option to run on save (like in step 3 of this article)
is it possible to run on every file with one command?

Hi @bigandy
I can investigate the first option but I'm not sure the second would be possible. I don't remember any extension that allows formatting all the files in a workspace. For that, it would be better to use a npm-script. Something like the approach described in this article using ESLint: https://simondosda.github.io/posts/2021-05-10-eslint-imports.html

Thanks for making this extension, it's awesome.
I was able to get it to run on save with https://marketplace.visualstudio.com/items?itemName=fsevenm.run-it-on
That being said, it always modifies the file, so it always results in an unsaved file.
Is there any way to have this extension do nothing if the imports are already correct?

hey!
Thanks for this extension. Upping this topic as it would be great to add it on save. I don't know if it's still in the works.
Thanks!

Hi @Kowaio
Yes, it would be a great feature, but in my opinion, only for the active editor (current edited file). Sadly, I don't have too much time to investigate and develop this at the moment, so any help with this would be appreciated. I'm keeping the issue open and tagging it with Help wanted.
Thank you all for using it and the suggestions.

Hello everyone,
Finally, I got some time to develop this feature. It is available in version 1.2.0 via Code Actions.
To enable it (in your VS Code settings):
"editor.codeActionsOnSave": {
  "source.removeUnusedImports": true
}

I hope this helps :)
Please, leave the extension a rating if you find it useful. It is still unrated.

Hey!
Thanks a lot for your work.
I'm having a weird message in my vscode project and this does not seem to work because of that :
[BABEL] unknown: Preset your preset  requires a filename to be set when babel is called directly,

babel.transform(code, { filename: 'file.ts', presets: [/* your preset */] });

See https://babeljs.io/docs/en/options#filename for more information.

Any ideas? Am i missing something here?

Hey! Thanks a lot for your work. I'm having a weird message in my vscode project and this does not seem to work because of that :
[BABEL] unknown: Preset your preset  requires a filename to be set when babel is called directly,

babel.transform(code, { filename: 'file.ts', presets: [/* your preset */] });

See https://babeljs.io/docs/en/options#filename for more information.

Any ideas? Am i missing something here?

Hi @Kowaio
I've created a new issue for this, as this one is closed and it doesn't seem to be related to this one.
Can you please share more information here about the context in which the error happens? The file that you are editing, etc.
Thanks


Messing w arcs

part of https://github.com/KittyCAD/modeling-app/issues/569

[x] get arc tool
[x] get arc tool to generate code
[x] Move tool should work on lines with arcs in the sketch
[x] get move working with arcs themselves
[x] closing with an arc should generate arc and close lines?
[ ] Write e2e test

Thinks to consider

[ ] Add tangentialLine stdlib function (line that follows an arc and follows the same angle, only takes length)
[ ] Add tangentialConstraint (can be applied to non-tangential arc, or line that follows an arc)
[ ] horizontal constraint when applied to a tangentialLine should snap the arc proceeding it to 90° etc

Finally CI passed ooph


Missing space for searches that use advanced search

Bug Report
Expected behaviour
As you add more to your search, the text in the box on the right that displays your search query spaced appropriately
Actual behaviour
The end of advanced search portions are missing a space.

Steps to reproduce the behaviour

Go to https://dockstore.org/search
Add one input into advanced search
Click some facet option or add another input into advanced search. A space is missing

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7.x
┆Issue Number: DOCK-1035

LGTM,
API - 1.8.0-alpha.2
UI - 2.5.0-alpha.0


Why check all available bots? (checkAllBotsAvailability)[BUG]

Describe the bug / 描述问题
I found that checkAllBotsAvailability is declared in App.vue. This method will check the accessibility of all bots. If a bot is not configured, an error will be reported on the devtools console. Can we only check selected bots to improve performance?
To Reproduce / 复现步骤
刷新页面或者点击设置再关闭。
Expected behavior / 期望行为
只检查已选择的bots
async function checkAllBotsAvailability(specifiedBot = null) {
  try {
    let botsToCheck = bots.value.filter((bot) => selectedBots.value[bot.getClassname()])
    ...

Versions / 版本
1.19.22

I admit, my solution was not good enough. It was a lazy approach I took back then. A better solution would be to only check each bot when it is being activated. Are you interested in completing this improvement?

Have submitted the PR


Separating spin-up and spin-down contributions in sumo-bandplot output .dat file

Using  sumo-bandplot -spin 1 or sumo-bandplot -spin -1, one is able to obtain the bandstructure with either spin-up or spin-down channels, however, the created .dat file will contain both spin-up and spin-down contributions. I would like to ask whether one could have those contributions in two separated .dat files?

You could use the unix cut program to remove the unwanted columns from the file. Just watch out for the header; the extra # means the header has one more entry and the indices are offset.
e.g. to get the spin-up component from a file with s,p,d components:
FILENAME=Ni_dos.dat
head -n 1 $FILENAME | cut -d ' ' -f 2,3,5
tail -n +2 $FILENAME | cut -d ' ' -f 1,2,4

It shouldn't take much extra work to turn that into a shell function or script that takes a .dat file as input and writes a couple of new ones, and would be a bit easier to share/maintain than a Summo fork.


{"message":"Invalid JSON in request body."} for curl -u root:123 -XPUT http://<IP_ADDRESS>:2379/v2/auth/users/fleetuser -d value='{"user" : "fleetuser", "password" : "fleetpw"}'

Bug reporting
version: v3.1.4
I just use below command to create a user but report invaild json in request body, I can not figure out why, everything seems run under proper way:

curl -u root:123 -XPUT http://<IP_ADDRESS>:2379/v2/auth/users/fleetuser -d value='{"user" : "fleetuser", "password" : "fleetpw"}'
report:
{"message":"Invalid JSON in request body."}

@xiang90 , T T